Implement support for find_package in CONFIG mode

Please add support for finding this package with a call to find_package in CONFIG mode.
It looks like CMakePackageConfigHelpers is included, but not used to create the package configuration file or version file.
Reference
The utility of this is you can install the library in your system headers or somewhere on your CMAKE_PREFIX_PATH, then just call find_package in the consuming application.
Note: There is a guide here, but that is the manual way of doing it, CMakePackageConfigHelpers alleviates a lot of the boilerplate.
Why do this? See here.

Hmm, if I add this to CMakeLists.txt, it generates and installs a package version file, but for some reason CMake doesn't like it...
write_basic_package_version_file( ${CMAKE_CURRENT_SOURCE_DIR}/cmake/isobusConfig.cmake VERSION ${PROJECT_VERSION}
COMPATIBILITY SameMajorVersion)

install(FILES
${CMAKE_CURRENT_SOURCE_DIR}/cmake/isobusConfig.cmake
DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME})

From another project that does find_package(isobus 0.1 REQUIRED)
CMake Error at CMakeLists.txt:13 (find_package):
  Could not find a configuration file for package "isobus" that is compatible
  with requested version "0.1".

  The following configuration files were considered but not accepted:

    /usr/local/lib/cmake/isobus/isobusConfig.cmake, version: unknown

-- Configuring incomplete, errors occurred!

Any thoughts on what I might be missing?

Push it on a branch and I'll try it?

Push it on a branch and I'll try it?

Branch is meta/cmake-package-version-file

You misnamed the version file.
It needs to be isobus-config-version.cmake or isobusConfigVersion.cmake. Also, setting version to PROJECT_VERSION is redundant, that's already the default.
Once I changed that, the package can be found, but the targets aren't part of it (yet)

I would follow this guide; https://cliutils.gitlab.io/modern-cmake/chapters/install/installing.html
Working on a fix, can push it to another branch for your review.

I would follow this guide; https://cliutils.gitlab.io/modern-cmake/chapters/install/installing.html
Working on a fix, can push it to another branch for your review.

I appreciate the help & resources, this is a bit outside my CMake knowledge so it's a learning process for me :)

Check out commit bece837a7165423776f5faaeaa13790ba6659b3a
That does get the export working, but it uncovered the include directories are not portable.
Yea, I have to look up the tutorials every time, the cmake docs themselves have you using finicky functions and never recommend not using something where there are many ways to do it.
CMake Error in CMakeLists.txt:
  Imported target "isobus::Isobus" includes non-existent path

    "ISO11783-CAN-Stack/install/ISOBUS/include"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

Ah, it's because this line in isobus/CMakelists.txt:
target_include_directories(Isobus PUBLIC
  $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
  $<INSTALL_INTERFACE:${ISOBUS_PUBLIC_NAME}/include>
 )

This conflicts with
install( TARGETS Isobus SystemTiming HardwareIntegration
         EXPORT isobusTargets
         LIBRARY DESTINATION lib
         ARCHIVE DESTINATION lib
         RUNTIME DESTINATION bin
         INCLUDES DESTINATION include
)

Per INCLUDES DESTINATION docs:
This option specifies a list of directories which will be added to the [INTERFACE_INCLUDE_DIRECTORIES](https://cmake.org/cmake/help/latest/prop_tgt/INTERFACE_INCLUDE_DIRECTORIES.html#prop_tgt:INTERFACE_INCLUDE_DIRECTORIES) target property of the <targets> when exported by the [install(EXPORT)](https://cmake.org/cmake/help/latest/command/install.html?highlight=install#install-export) command. If a relative path is specified, it is treated as relative to the $<INSTALL_PREFIX>.

Basically, this doesn't actually install the header files anywhere to the install directory. They need to be added.
The modern-cmake cpaters installing resource also has a not on this:
The various destinations are only needed if you have a library, static library, or program to install. The includes destination is special; since a target does not install includes. It only sets the includes destination on the exported target (which is often already set by target_include_directories, so check the MyLibTargets file and make sure you don't have the include directory included twice if you want clean cmake files).

The library is definitely doing it twice right now. I have to do some research to figure out what's better.
Do you want the public name to be ISOBUS or isobus?

Also, do you want the includes flat, or in folders? IE, do you want consumers typing
#include <can_hardware_interface.hpp>
OR
#include <hardware_integration/can_hardware_interface.hpp>
OR
#include <ISOBUS/hardware_integration/can_hardware_interface.hpp>
I'd recommend against the first one. For example, from boost: #include <boost/lambda/lambda.hpp>

I think my preference is isobus as the public name, as well as being able to do #include <isobus/hardware_integration/can_hardware_interface.hpp> rather than the flat structure, as I agree it leaves out a lot of implicit context for consumers

Good. This leaves two approaches.

Use install(DIRECTORY) with a  file matching .hpp pattern. I think this would mean moving install to top level only.
Upgrade cmake version to build the library to 3.23 to support file sets, which actually solves this issue the right way. This does mean consumers who want to submodule or compile from source will need to upgrade. For consumers who don't want to upgrade cmake, they can consume the debian or prebuilt binaries at some release cadence.

I'm hesitant to jump CMake versions past 3.16 (the default for Ubuntu 20.04, raspian, and maybe RHEL) also, so I am thinking option number 1 might be the way to go!

For approach 1, I got it working here: 6a0cea3f20508967f1061c7f853821f873dbbc21
It's a bit of hack because the installed header files use the flat include style, so you have to have targets support both. Overall, it seems to work fine, so let me know if you agree with the behavior there.
Important bits are as follows:
Consumer example include:
#include "isobus/isobus/can_general_parameter_group_numbers.hpp"
#include "isobus/hardware_integration/can_hardware_interface.hpp"
#include "isobus/isobus/can_network_manager.hpp"
#include "isobus/isobus/isobus_diagnostic_protocol.hpp"
#include "isobus/hardware_integration/socket_can_interface.hpp"

The install tree looks like this:
$ tree install/
install/
├── include
│   └── isobus
│       ├── hardware_integration
│       │   ├── can_hardware_interface.hpp
│       │   ├── can_hardware_plugin.hpp
│       │   └── socket_can_interface.hpp
│       ├── isobus
│       │   ├── can_address_claim_state_machine.hpp
│       │   ├── can_badge.hpp
│       │   ├── can_callbacks.hpp
│       │   ├── can_constants.hpp
│       │   ├── can_control_function.hpp
│       │   ├── can_extended_transport_protocol.hpp
│       │   ├── can_frame.hpp
│       │   ├── can_general_parameter_group_numbers.hpp
│       │   ├── can_hardware_abstraction.hpp
│       │   ├── can_identifier.hpp
│       │   ├── can_internal_control_function.hpp
│       │   ├── can_managed_message.hpp
│       │   ├── can_message.hpp
│       │   ├── can_NAME_filter.hpp
│       │   ├── can_NAME.hpp
│       │   ├── can_network_configuration.hpp
│       │   ├── can_network_manager.hpp
│       │   ├── can_parameter_group_number_request_protocol.hpp
│       │   ├── can_partnered_control_function.hpp
│       │   ├── can_protocol.hpp
│       │   ├── can_transport_protocol.hpp
│       │   ├── can_warning_logger.hpp
│       │   ├── isobus_diagnostic_protocol.hpp
│       │   └── isobus_virtual_terminal_client.hpp
│       └── utility
│           ├── iop_file_interface.hpp
│           ├── processing_flags.hpp
│           ├── system_timing.hpp
│           └── to_string.hpp
└── lib
    ├── cmake
    │   └── isobus
    │       ├── isobusConfig.cmake
    │       ├── isobusConfigVersion.cmake
    │       ├── isobusTargets.cmake
    │       └── isobusTargets-debug.cmake
    ├── libHardwareIntegration.so
    ├── libIsobus.so
    └── libSystemTiming.so

8 directories, 38 files

You can see the properly nested includes by folder. You also get the cmake libs (.so) in lib/, and ALL of the cmake scripts get dumped into lib/cmake/isobus.
I think that approach is fine, just be aware if this library gets really big, you start supporting COMPONENTS level install and finding to modularize it, perhaps from optional GUI things, this install method will install all the header files regardless if the library was actually built. It's not too much work to change in the future. File sets support this, by only installing headers for something that was actually built and requested to install.

Looks OK to me - if you put together that PR I would be happy to review and approve it.

Ok. Looks like I have to update all the internal include paths to be the same, otherwise the examples can't build from top level. Shouldn't take too long.

@RFRIEDM-Trimble  Now that #58 is merged is this issue closable?

Yes, it can be closed. Thanks for the patience as I worked though all the mods.
This should greatly increase the usability of this package to be integrated into higher level applications.


Docker support for Hygieia

A simple setup to run Hygieia in a Docker container

@tyrion70 for UI setup, instead of using 'gulp serve' can you change it to use apache server ?

Gave a try by running createdocker script. The build went well but it failed when it tried docker rm hygieia.
Step 39 : CMD /usr/bin/supervisord -n
 ---> Running in 742586701a53
 ---> e970f0d0a90b
Removing intermediate container 742586701a53
Successfully built e970f0d0a90b
Error response from daemon: no such id: hygieia
FATA[0000] Error: failed to remove one or more containers

Hmm that kinda makes sense. The first time u run it there is nog container yet. I'll fix that tonight thanks

Updated the createdocker script so it won't give an error the first time (it did start the container eventually right @srid99).
Also added Apache support, please anyone take a good look at what I did, I'm not sure if I have the document root completely correct. Its currently set to the /UI/dist folder now. I tried the src folder, but then it needed a symlink to include bower_components and it was missing css.
A Reverse proxy for /api was added to the apache configuration as well.

@tyrion70 Correct. It had error but the build was success and the container was running.
For the Apache support, what you did is correct. dist folder and the reverse proxy should do the job.

@tyrion70: can you submit the CLA? Thanks

already did yesterday :) u didn't get it?

So in looking at this vs. pull #115 - to be honest i'd like to see something in the middle.. This pull has jenkins,sonar, etc all in the same container and lacks configurability.  The other pull has them separated out but lacks the collectors.. So I'm thinking about merging the best of both of them into a branch and then once it's solidified a bit merge it back to master.  Sound fair?

Hey!
It's actually its different containers. One container is for the complete Hygieia solution including mongodb and apache. It also allows you to start other containers for sonar and jenkins (and mysql) using compose.
Nevertheless there are some nice things in the other PR as well so feel free to solidify and merge it together :)
What do you mean by lack of configurability by the way? All config files needed are in the docker folder and can be modified before building the container.
Let me know if you need any help merging or have any questions about what I did..

I've started down the path of dockerizing some of the collectors, will add those as separate pulls

closed by pull #129


Runnable examples: Connect directly to dpaste.dzfl.pl, skiping the pr…

…oxy written in PHP. + Use JSON instead of XML + add some debug info when response fails for some reason
@MartinNowak: as we discussed before, please merge ASAP.

Will deploy it when I get home.

@MartinNowak sure, thanks!

done

@MartinNowak nice, it works :+1:


Fix dependency name.

Changed dependency from 'kf5-cmake-extra-modules' to 'kf5-extra-cmake-modules'

Thank you! I must have glanced over it 😅


Revise the configuration doc to mark PFLT_INDEXIMAGE as mandatory

This PR fixes the configuration doc to mark PFLT_INDEXIMAGE as a mandatory field.

Good catch, thank you!


Create and delete yum repositories

Bonus fix to make Chef::Mixin::Which work in class methods, so we can use it for provider resolution.
cc @chef/client-maintainers

yep 👍

Could maybe use some debug logging in action_create in the provider, but 👍 as is

It's about time 👍


Unique columns not being set

I found another issue with unique columns that are not being set as unique in the rules. For instance I have the guid column that is set as  unique in the table definitions, and when generating the rules it sets as follows: 'guid' => ['required', 'string', 'min:1', 'max:191'] where it should be 'guid' => ['required', 'string', 'min:1', 'max:191', 'unique:table_name,guid']
Please note that these are constructive contributions to make the package even better.

That's also a very good idea. Thanks for your feedback, I'll try to implement it the next few days...


[bcrypt] Doesn't work on alpine

I'm still looking into it.
I tried installing gcompat or libc6-compat:
Error relocating<EMAIL_ADDRESS>__register_atfork: symbol not found

Without them:
Error: Error loading shared library ld-linux-x86-64.so.2: No such file or directory (needed by<EMAIL_ADDRESS>
My dockerfile:
FROM node:14-alpine

# Create app directory
RUN mkdir -p /usr
WORKDIR /usr

COPY package*.json ./
RUN apk add --no-cache gcompat \
    && npm install --production

Might be related : https://stackoverflow.com/questions/58766740/how-can-i-target-my-rust-application-and-its-dependencies-to-musl

@Mokto<EMAIL_ADDRESS>should support linux musl, and it tested on node:12-alpine and node:14-alpine


Pokémon type component with better background color.

A component that would take a single type as an input and return a <Text>{type}</Text> with a different background color.

@0Verlord-41  would it be better to have a Type color class in the core library for consistency?


[mobility-theme] Referencing to data categories from EC Delegated Regulations

We plan to reference all “categories” & “sub categories” (from our Controlled Vocabulary Mobility Theme (mobilitydcat-ap.github.io) to the corresponding data categories from EC Regulations.
Reason: see Wikipage, section 3.2.
There will be an additional entry under each “category” & “sub category” in our Controlled Vocabulary.
We will use the property “dcatap:applicableLegislation” for that addition.
This will be done in a future version of the Controlled Vocabulary.
By the way: there was already a referencing to the EC Regulations for the Coordinated Metadata Catalogue:
https://www.its-platform.eu/wp-content/uploads/ITS-Platform/AchievementsDocuments/NAP/EU EIP_Coord. Metadata Catalogue_Annex I_v2.0_191115.xlsx
We need volunteers to re-do such referencing in the Controlled Vocabulary "Mobility Theme".

I propose to add the Excel table for "mobilityTheme" as follows:

New column "dcatap:applicableLegislation" - reusing definition and range from the same property  in mobilityDCAT-AP 1.0.1. It links to the relevant Delegated Regulation via the ELI identifier syste,
New column "mobilitydcatap:delegatedRegulationCategory" - this is a new property which we would define in a future release of mobilityDCAT-AP. It denotes exactly the relevant data categories under the relevant EC Delegated Regulations.
@marioscrock : please let me know if I can proceed this way

I propose to expand the Excel table for "mobilityTheme" as follows: [...]

Yes, this is what I had in mind. I will then work out how to ensure that the association in the column is also automatically serialised in RDF.

New column "dcatap:applicableLegislation" - reusing definition and range from the same property  in mobilityDCAT-AP 1.0.1. It links to the relevant Delegated Regulation via the ELI identifier.

Yes, I agree on the property chosen and the usage of ELI identifiers. You can use the comma to list multiple ELI identifiers for the same category if needed.

New column "mobilitydcatap:delegatedRegulationCategory" - this is a new property which we would define in a future release of mobilityDCAT-AP. It denotes exactly the relevant data categories under the relevant EC Delegated Regulations.

I would like to discuss this one a little bit further. Do we expect a 1:1 match between our vocab and the delegated regulation categories? In this case, this can be seen as an "alternative label" of the same concept in the controlled vocabulary.
I am not sure if it makes sense to add to mobilitydcatap: two properties to specify "two types" of categories (mobilityTheme and delegatedRegulation Category). I would prefer to keep this association in the controlled vocabulary to reduce the "burden" on the user filling out the metadata fields.
Finally, I am not convinced of having a property (mobilitydcatap:delegatedRegulationCategory) that requires "complex string" values. From the example provided, it seems that also the values in the last column could be represented hierarchically, e.g., 1 -> a -> ii. If this property is needed, we should either define these as alternative labels of the concepts already defined in our controlled vocabulary (thus already having a IRI) or define new identifiers (IRI) for these categories. An alternative option may be to keep this info only in the Excel as "provenance information" but without serialising it in RDF.

@peterlubrich if it is 0..n the idea of using alternative labels is valid, we can add as many alternative labels as we want. If we have also the case that a single Del.Reg. Category corresponds to multiple Mobility Theme it is still "technically feasible" but I would say less "logically consistent".

I propose to expand the Excel table for "mobilityTheme" as follows:

New column "dcatap:applicableLegislation" - reusing definition and range from the same property  in mobilityDCAT-AP 1.0.1. It links to the relevant Delegated Regulation via the ELI identifier.
New column "mobilitydcatap:delegatedRegulationCategory" - this is a new property which we would define in a future release of mobilityDCAT-AP. It denotes exactly the relevant data categories under the relevant EC Delegated Regulations.

@marioscrock : please let me know if I can proceed this way

I completed the expanded Excel file, as proposed above:
mobility-theme_added_applicableLegislation_240628.xlsx
There are now additional columns:

F,G:  dcatap:applicableLegislation (value 1,2)
H,I,J,K: mobilitydcatap:delegatedRegulationCategory  (value 1,2,3,4)

The reason for the multiple values is the 0...n cardinality, as discussed above.
To do:
We need to sort out how to do the RDF representations correctly. @marioscrock had some suggestions above.
The expanded Excel file serves as a starting point for a future revision of the Controlled Vocabulary.

Status 27.8.24:
To do:

Check if mapping proposal by Peter is correct : domain experts needed @EdNDW @CRVH : send out Teams meeting to SWG4.4 to discuss mapping
Apply in update of Controlled Vocab of "Mobility Theme" @Mario

Email sent by Peter 29.08.24:
Dear Ed, Christian, Jonas, Mario,
One of the open actions from our last meeting is:
Mapping of our controlled vocab “mobilityTheme” vs. the various EC Delegated Regulations.
Next, I am compiling some support information.
General goal:
Add new information to existing controlled vocab “mobilityTheme”, linking each value from this controlled vocab to :
(a)	one (or more) Delegated Regulations
(b)	one (or more) data categories from these Delegated Regulations
Baseline documents:
Current version of “mobilityTheme”: https://mobilitydcat-ap.github.io/controlled-vocabularies/mobility-theme/latest/index.html#/
Related GitHub issue: https://github.com/mobilityDCAT-AP/controlled-vocabularies/issues/8
Previous mapping exercises (including Coord. Metadata Catalogue from 2019; SE mapping from 2023; DK mapping from 2024): see NAPCORE Sharepoint
New mapping proposal by Peter: see NAPCORE Sharepoint
What is to do:

Check if mapping proposal by Peter is correct, compare with previous mappings
Apply mapping information as update of Controlled Vocab of "Mobility Theme" (Mario will add additional properties as RDF codes)

Working proposal:
Ed, Christian please:
•	Propose an online meeting, send out to entire SWG 4.4 group
•	Moderate meeting, work for consensus
@Christian: what is your GitHub user name?
@Jonas: I am adding you to this action, as you had sent me some comment about using the “mobility theme” during 2023.
Thanks everyone!
Peter

Update:
Ed Ooms and Christian von Huth (DK) have provided a mapping on the level:
1 Value from Vocab "Mobility Theme " -> Reference to 1..n Delegated Regulations
I have verified this mapping with our own mapping attempts (see above "New mapping proposal by Peter").
Result:

Some refinements done in our mapping table
Produced a new "source excel table" for Vocab "Mobility Theme", see on Sharepoint

To do now:
@marioscrock , please:

produce a new RDF vocabulary based on the new "source excel table"
1.1. insert a dcatap:applicableLegislation property for each entry in columns F,G,H,
1.2 if "n/a" is noted, do nothing here!
Deprecate the current versionof "Mobility Theme"
Publish a new version 1.1 of  of "Mobility Theme"

To do later:
The above actions ignore the more granular referencing to "mobilitydcatap:delegatedRegulationCategory", as proposed by me above. This needs more analyses / discussions, and will be considered for another update of "Mobility Theme".

Thank you @peterlubrich! I edited the file on Sharepoint to keep all the values as a comma-separated list in a single column.
TODO

[ ] The property is currently not visible in the "human-readable" documentation, find a way to describe the mapping in the vocabulary docs
[ ] finalise other issues related to mobility-theme before releasing 1.1.0 of the vocab

Meeting 05.12.2024:
Decision procedure for the proposed change:
Mobility Theme: Added references to EC Delegated Regulations via additional dcatap:applicableLegislation properties for all values used in this vocabulary. This allows for an automated tagging of EC Regulations to individual datasets.  DECISION POINT: YES / NO
Decision result: no opposition, change to be announced in v1.1.0

@peterlubrich EC Delegated Regulations added to v1.1 of the controlled-vocabulary, now available online at https://w3id.org/mobilitydcat-ap/mobility-theme


Add macos target

I've been looking for the release for MacOS, for the cross build for clang + Mac. I've found some releases here https://github.com/messense/cargo-xwin, but it seems to be fork, so less trust.
Never set up GitHub action, this is my first time) Just moved build action to separate file and added mac os target.
Example of release:
https://github.com/ZzEeKkAa/xwin/releases/tag/untagged-1cce06ead6861d7fecf6

I've merged #57 which supersedes this change.


[QE] Continual tests for KafkaSource for upgrade testing

Fixes #67

Proposed Changes

Adds an implementation of KafkaSource continual testing by sending events to Kafka topic directly

Additional notes
This PR requires merge of the following PRs:

knative/eventing#4815
knative/eventing#5321

This lint error: https://github.com/knative-sandbox/eventing-kafka/pull/563/checks?check_run_id=2658845322 is due to https://github.com/knative-sandbox/eventing-kafka/pull/648

/lgtm


can't install in new project

Getting this:
npm install ember-cli-sass --save-dev
npm ERR! fetch failed https://registry.npmjs.org/center-align/-/center-align-0.1.1.tgz
npm WARN retry will retry, error on last attempt: Error: getaddrinfo ENOTFOUND registry.npmjs.org
npm ERR! fetch failed https://registry.npmjs.org/async/-/async-1.2.1.tgz
npm WARN retry will retry, error on last attempt: Error: connect ETIMEDOUT
...
npm keeps spinning forever.. is it just me?

Yup, just you. It doesn't use that package.
On Tue, 23 Jun 2015 06:05 Yoni Dabush<EMAIL_ADDRESS>wrote:

Getting this:
npm install ember-cli-sass --save-dev
npm ERR! fetch failed
https://registry.npmjs.org/center-align/-/center-align-0.1.1.tgz
npm WARN retry will retry, error on last attempt: Error: getaddrinfo
ENOTFOUND registry.npmjs.org
npm keeps spinning forever.. is it just me?
—
Reply to this email directly or view it on GitHub
https://github.com/aexmachina/ember-cli-sass/issues/58.


Bugfix/get trace graph

Builds on the earlier fix by jccurtis. This pull request fixes issue #66 . It makes hiddenlayer work with Pytorch 1.4, where get_trace_graph became private and ONNX formatting appears to have changed.

Works for me with PyTorch 1.4.0

Can this be merged in

Thank you for the fix, and sorry for the delay. This is now merged.

hi, but in pytorch 1.5, the problem of no attribute get trace graph happens again


Bad load_path for COFDetector

julia> MLJModelInterface.load_path(COFDetector)
"COFDetector"

This should be "OutlierDetectorNeighbors.COFDetector".

Okay, it looks like they are all wrong:
julia> eval.(OutlierDetectionNeighbors.MODELS)
5-element Vector{DataType}:
 ABODDetector
 COFDetector
 DNNDetector
 KNNDetector
 LOFDetector

julia>

julia> load_path.(ans)
5-element Vector{String}:
 "ABODDetector"
 "COFDetector"
 "DNNDetector"
 "KNNDetector"
 "LOFDetector"

Okay, I see why this was not caught before. The bug is that the string returned by load_path currently depends on the namespace. If instead of using OutlierDectectionNeigbors (what I did above) you do import OutlierDetectionNeighbors, the you get:
julia> load_path(OutlierDetectionNeighbors.COFDetector)
"OutlierDetectionNeighbors.COFDetector"

Closed in favor of https://github.com/OutlierDetectionJL/OutlierDetectionInterface.jl/issues/7


bump sync plugin to 0.1.24

v0.1.24 of the sync plugin brings in the setting of user-agent of the underlying http client that came out of the recent diagnosis for the us-east prod cluster struggles
@bparees - I'm assuming we want this in 3.6 - that your opinion?
@bparees @jupierce @smunilla - if so, we need to build a 3.6 rpm and a new 3.6 jenkins image, and presumably consider whether the resulting image is what goes into the pipeline for the upcoming upgrade of the jenkins image running in us-prod east .... what is everyone thoughts?

that's my preference since it'll help us get to the bottom of the excessive watches we're seeing in prod, but if it's too late in 3.6 to respin this plugin+image, we can probably survive picking it up for 3.6.1.

[test]

per discussion with @bparees [merge] ing

also stopped by @jupierce 's desk re: the new rpm ... he's got it on his to-do list; should be part of the next jenkins image headed to QE for regression

@gabemontero The openshift-synch plugin version has been updated in dist-git to 0.1.24, per your request. Subsequent builds of 3.6 should pick it up.

On Fri, Jul 21, 2017 at 12:05 PM, Justin Pierce<EMAIL_ADDRESS>wrote:

@gabemontero https://github.com/gabemontero The openshift-synch plugin
version has been updated in dist-git to 0.1.24, per your request.
Subsequent builds of 3.6 should pick it up.

great thanks @jupierce

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/jenkins/pull/336#issuecomment-317042057,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADbadG4mfWhabkoFA6CfPQvdMqoT54U-ks5sQMxMgaJpZM4Oe0yR
.


az ad app permission admin-consent does not work from Cloud Shell

This is autogenerated. Please review and update as needed.
Trying to use az ad app permission admin-consent. Works fine when installed locally. Does NOT work from Cloud Shell. Grr.
Command Name
az ad app permission admin-consent
Errors:
400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 631, in execute
    raise ex
cli/core/commands/__init__.py, ln 695, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 686, in _run_job
    cmd_copy.exception_handler(ex)
...
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 486, in get_msi_token
    result.raise_for_status()
python3.6/site-packages/requests/models.py, ln 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ad app permission admin-consent --id {}

Expected Behavior
Environment Summary
Linux-4.15.0-1082-azure-x86_64-with-debian-stretch-sid
Python 3.6.5
Installer: DEB

azure-cli 2.5.1 *

Additional Context

@arrownj  please take a look

this is known Cloudshell issue caused by #11749. To workaround it, try az login.

#11749

This should be fixed now.  I cannot repro, please close.


Scheduled AV build not working

https://github.com/chocolatey-community/chocolatey-coreteampackages/issues/1273#issuecomment-494035818

@gep13 hmm, it is odd that the schedule doesn't run then.
I've been triggering it manually for the past few days because it didn't run.

Looks like it stopped 2 days ago.

@majkinetor it stopped long before then, I've been manually running them now and then for some time now.

Even linux platforms could work as it looks like AU works on pwsh without problems.
Gitlab too is an option to although it requires setup of git mirror.
@pascalberger Isn't AV on Azure ? What are limitations on Azure ?
AV can be easily fixed by using external scheduler...
Any potential replacement should be thought out from the perspective of the entire AU community and not only this repo, as major point of current AU system is that everybody can use it in the same manner.

Isn't AV on Azure ? What are limitations on Azure ?

My issues was that builds in the free-tier can only run up to 60 min, which was not enough time for the build to download a 400 MB file and calculate the checksum. Beside that Azure DevOps doesn't have any limitation on run time, it seems to have also much faster network connectivity in my case. Maybe it was just a temporary issues on AppVeyor.
I don't see how this affects AU (beside the report generation where it adds an AppVeyor badge). But otherwise it's the same scripts running and choice of build server shouldn't have any impact on migraton of package between different repos using AU, even with different build servers.

My issues was that builds in the free-tier can only run up to 60 min, which was not enough time for the build to download a 400 MB file and calculate the checksum.

System that can not download 400 MB in an hour in 2019 is not usable in general sense. Are you sure this is not a problem of different nature.

I don't see how this affects AU

It doesn't affect AU - it affects AU community as it adds complexity.
Are you willing to create a wiki page pandan to the one we have now for AV?
We also need build templates pandan to .appveyor.yaml.
The point is to make on-boarding as quick as possible, otherwise, some of the maintainers may lose patience and revert to local run which will stop working sooner or later.

beside the report generation where it adds an AppVeyor badge

Report plugin has a type parameter which you can use for example to have textual reports without badge. Each type also has its own parameters so its easy to create badges based on other providers and you could even disable badge and let users specify whatever in custom text that is among updateall options.

I was informed that it had been enabled here:
https://twitter.com/appveyor/status/1130581959225163776?s=21

@gep13 can you confirm that the schedule is still set on on the correct build vm? (https://ci.appveyor.com/project/chocolateycommunity/chocolatey-coreteampackages).
I don't have access to the settings so I can not see it myself (I'm thinking it perhaps got removed when they enabled the schedule, I know that happened on my own repository).

From what I can see, yes.

I am going to modify it slightly and re-save the setting, to see if that has any effect.

@AdmiringWorm I have just re-saved the settings.  There wasn't any error when I did this, which tells me that scheduled builds are enabled, as I got an error the first time I tried to create the schedule.  Let's see if this helps any...

@gep13 seems like that change fixed the issue with the schedule.
I see a scheduled build that ran ~3 hours ago: https://ci.appveyor.com/project/chocolateycommunity/chocolatey-coreteampackages/builds/24835994#L163

@AdmiringWorm nice one, thank you for confirming!


[Removal]:

Hotspot Name
Sticky Zinc Bat
Hotspot b58 Address
1124GEEgck8FJMtrLLts5UWK9jSGguKYREuspCXxkkjaiPMGEoWK
Discord Handle
No response
Hotspot Manufacturer
PantherX2
Removal Reason
我在线下买了9台黑豹x2设备  还有10个hnt代币   结果到手发现他们都在拒绝名单里，我不知道该怎么办了，联系不上卖家，请求官方帮忙，谢谢
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
No response

because your used miners have been already on the denylist for cheating, which the sellers didn't told you. there is no way helium can help based on my understanding.


Is the Institution byline MANDATORY?

Good morning,
I just want to know if the institution byline is mandatory.
One of our QA person pointed out the redundancy of the byline since we have the institution name in the breadcrumbs.
Thanks!

Nope. Check out https://design.canada.ca/common-design-patterns/institutional-byline.html. If it doesn't say "Mandatory" then it's not mandatory.
See also the complete list of mandatory elements at https://www.canada.ca/en/treasury-board-secretariat/services/government-communications/canada-content-information-architecture-specification/mandatory-elements.html and note that the byline is not included.

Nope. Check out https://design.canada.ca/common-design-patterns/institutional-byline.html. If it doesn't say "Mandatory" then it's not mandatory.
See also the complete list of mandatory elements at https://www.canada.ca/en/treasury-board-secretariat/services/government-communications/canada-content-information-architecture-specification/mandatory-elements.html and note that the byline is not included.

Thank you very much for your quick answer!!


Generalize FilterMonadic operations of Value implementations

Traversable<T> implements FilterMonadic<IterableKind<?>, T>, which results in
<U> Traversable<U> flatMap(Function<? super T, ? extends Kind<? extends IterableKind<?>, ? extends U> mapper)

This is (more or less) equivalent to
<U> Traversable<U> flatMap(Function<? super T, ? extends Iterable<? extends U>> mapper)

This is a great generalization over
<U> Traversable<U> flatMap(Function<? super T, ? extends Traversable<? extends U>> mapper)

(which does not scale regarding inheritance).
In the same way we want to treat implementations of Value. These are:

Either.LeftProjection, Either.RightProjection
Lazy
Match.MatchMonad
Option
Try

All of these classes Xxx will implement FilterMonadic<Value<?>, T>, which leads to
<U> Xxx<U> flatMap(Function<? super T, ? extends Kind<? extends Value<?>, ? extends U>> mapper)

which is (effectively) the same as
<U> Xxx<U> flatMap(Function<? super T, ? extends Value<? extends U>> mapper)

Note: Value<T> has to implement Kind<Value<?>, T>

This will not work without having plenty of RuntimeExceptions when values are undefined regarding get(). Therefore I will close this issue.


[network_info_plus] Warning: Deprecated methods

I'm getting a warning while building my flutter app with network_info_plus version 2.1.2:
C:\flutter\.pub-cache\hosted\pub.dartlang.org\network_info_plus-2.1.2\android\src\main\java\dev\fluttercommunity\plus\network_info\NetworkInfo.java:46: warning: [deprecation] getConnectionInfo() in WifiManager has been deprecated     if (wifiManager != null) wifiInfo = wifiManager.getConnectionInfo();                                                    ^ C:\flutter\.pub-cache\hosted\pub.dartlang.org\network_info_plus-2.1.2\android\src\main\java\dev\fluttercommunity\plus\network_info\NetworkInfo.java:49: warning: [deprecation] getIpAddress() in WifiInfo has been deprecated     if (wifiInfo != null) i_ip = wifiInfo.getIpAddress();                                          ^ C:\flutter\.pub-cache\hosted\pub.dartlang.org\network_info_plus-2.1.2\android\src\main\java\dev\fluttercommunity\plus\network_info\NetworkInfo.java:108: warning: [deprecation] getDhcpInfo() in WifiManager has been deprecated     DhcpInfo dhcpInfo = this.wifiManager.getDhcpInfo();                                         ^ C:\flutter\.pub-cache\hosted\pub.dartlang.org\network_info_plus-2.1.2\android\src\main\java\dev\fluttercommunity\plus\network_info\NetworkInfo.java:122: warning: [deprecation] getConnectionInfo() in WifiManager has been deprecated     return wifiManager == null ? null : wifiManager.getConnectionInfo();                                                    ^ 4 warnings
flutter docotr -v:
`[√] Flutter (Channel stable, 2.10.0, on Microsoft Windows [Version 10.0.19042.1466], locale de-DE)
• Flutter version 2.10.0 at C:\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 5f105a6ca7 (8 days ago), 2022-02-01 14:15:42 -0800
• Engine revision 776efd2034
• Dart version 2.16.0
• DevTools version 2.9.2
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at C:\Android_SDK
• Platform android-31, build-tools 32.1.0-rc1
• ANDROID_SDK_ROOT = C:\Android_SDK
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.0.5)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.0.32112.339
• Windows 10 SDK version 10.0.19041.0
[√] Android Studio (version 2020.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
[√] VS Code, 64-bit edition (version 1.64.1)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension version 3.34.0
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.1466]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 97.0.4692.99
• Edge (web)        • edge    • web-javascript • Microsoft Edge 96.0.1054.62
[√] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
`
Do you have any suggestions, to fix it?

Hi. It is not a bug, but just warnings.
Network info plus supports lots of Android versions, including very old ones and to get network information on old device the plugin also uses APIs which are already deprecated in newer Android versions.
These warnings that you get during the builds aren't something that will fail the build or cause any issues, so you shouldn't worry about it.


Exact matches in gem/project search should be preferred

When searching for a gem/project, exact matches should be preferred and at the top of the list.
As an example: Searching for "ramaze" currently yields the results "ns-ramaze-ext" and "ramaze", in that order. "ramaze" should be the first result though, since it exactly matches the search term.

bump

It's definitely on the list. In the meantime, patches are always appreciated :).


Update token.py

This enables Google's oauth2 authentication. For example when using something like:
AUTH_URL = 'https://accounts.google.com/o/oauth2/v2/auth'
TOKEN_URL = 'https://oauth2.googleapis.com/token'
CLIENT_ID = 'XXXXXXXXXXXXXXXXXXXXX'

auth = DashAuthExternal(
    AUTH_URL, TOKEN_URL, CLIENT_ID,
    client_secret='YYYYYYYYYYYYYYYYYYY',
    scope='email',
)

Thanks for the input. I would prefer if we didn't add specfic code for certain OAuth providers, could we adapt this to allow user to access other arbitrary params that might have been returned in the token response. Possibly adding an 'extras' optional dict to the  class? and then exposing a new method to access the entire token object, called get_token_data?

That would work! Only our knowledge about programming and this module is limited to something simpler as proposed.

not to worry! I'll draft a release at some point in the next few days.

@subsurfaceiodev added get_token_data method in v1.2.0


Unable to edit blogs

Microsoft.JSInterop.JSException: Cannot read properties of null (reading '__quill')
TypeError: Cannot read properties of null (reading '__quill')
at Object.getQuillHTML (https://localhost/js/quill-interop.js:30:30)
at https://localhost/_framework/blazor.web.js:1:3244
at new Promise ()
at y.beginInvokeJSFromDotNet (https://localhost/_framework/blazor.web.js:1:3201)
at gn._invokeClientMethod (https://localhost/_framework/blazor.web.js:1:62841)
at gn._processIncomingData (https://localhost/_framework/blazor.web.js:1:60316)
at connection.onreceive (https://localhost/_framework/blazor.web.js:1:53957)
at i.onmessage (https://localhost/_framework/blazor.web.js:1:82102)
at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
at Oqtane.Modules.Controls.RichTextEditor.OnAfterRenderAsync(Boolean firstRender) in C:\Source\Projects\oqtane.framework\Oqtane.Client\Modules\Controls\RichTextEditor.razor:line 192
at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle, ComponentState owningComponentState)
Oqtane Version 5.1.0
Blog Module Version 5.0.3

I am also getting the same problem

@vnetonline are you getting the error from the code in this repo… or from installing the package in the Marketplace?

From installing it from market place

I just foredthe repo and I get the same error when trying to edit an existing blog

the blogs module was downloaded from the marketplace, oqtane from github compiled @sbwalker

Is it possible it could have been resolved by this recent PR - https://github.com/oqtane/oqtane.framework/pull/4111 (Oqtane Framework dev branch)

I am not able to reproduce this problem using the latest code in the dev branch of the Oqtane Framework

I am still getting the issue i have run the latest changes from the Dev branch and have just updated the SDK to 8.0.4 on my local copy of the blog still have the same issue
Adding content is fine
Unable to edit the content once entered,
I have noticed that the OnInitializedAsync is hit twice and on the second run, the error occurs

@tranpl @vnetonline I am able to reproduce the problem now... I will investigate further today. Its very strange, as I am using the Blog module on https://www.oqtane.org (and other sites) without any issues.

@tranpl @vnetonline this PR https://github.com/oqtane/oqtane.framework/pull/4126 resolves the issue in the RichTextEditor. The problem was basically a race condition due to JS Interop. The JS Interop call to CreateEditor on firstRender can sometimes take time (depending on the environment) which means that OnAfterRenderAsync could be called a second time with firstRender = false BEFORE the first OnAfterRenderAsync call completed. This could result in the component trying to access a method on the RichTextEditor before it actually exists. The resolution is to explicitly control the flow within the component by tracking whether the firstRender event completed or not.

I can confirm that PR https://github.com/oqtane/oqtane.framework/pull/4126 resolves the issue in the RichTextEditor and now I am able to edit the blogs

thank you @sbwalker


@Value / @EqualsAndHashCode callSuper false warning: lombok.config ignored

Hi, the @Value annotation is marked as warning although lombok.config is present with 'lombok.equalsAndHashCode.callSuper = skip', same with @EqualsAndHashCode.
I attached a minimal demo project.

Thanks,
Igor
Lomtest.zip

It should be fixed in the next release 0.12

Thanks a lot for taking care!


[Chore] Review browserlist options

This PR closes #43

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Refactoring and fine grained vol adjustment

The refactorings and their reasons are explained in the commit message of dbd3707.
I made them in a separate commit intentionally, so you have the option to merge only this commit in case you dislike the feature in 5e72534.
If you don't want the option of fine-grained volume adjustments or if you want different keybindings, then that's fine by me, I'll just have it on my fork.

Let me know if you've tested this with the new volume display and then I'll merge it :)
But yeah I think this is a good solution for granularity without introducing even more keybinds.

Let me know if you've tested this with the new volume display and then I'll merge it :)

yes, I tested it of course. Works as it should.

Alright, LGTM!


Multiple parameter binding?

Greetings,
First off, the package works great, thank you for the time you've spent on it.
I'm building a project in which the node server emits a 3 argument message. This is tested and working properly. In my .NET project, I want to be able to handle those 3 arguments, but I seem to be missing if/how this is possible with this client? Only the first argument is bound properly, while the other two are missing completely from the payload. My C# code is as follows:
            socket.On("remote:request", (data) =>
                {
                   //Do something with the args
                });

I have looked through your unit tests and didn't notice this covered anywhere. I apologize if I overlooked it.
Any and all help is appreciated. I would be happy to provide a more detailed code snippet if necessary. Thanks!

You can always write your own IListener implementation. See my comment from commented on 20 Nov 2014 in https://github.com/Quobject/SocketIoClientDotNet/issues/12

Ah, thanks! Totally /facepalmed, I even looked at that thread and didn't notice the follow-up comments. Thank you again!

Here is an example for an IListener implementation taking three params:
public class ListenerImpl : IListener
{
    private static int id_counter = 0;
    private int Id;
    private readonly Action<object,object,object> fn;

    public ListenerImpl(Action<object, object, object> fn)
    {

        this.fn = fn;
        this.Id = id_counter++;
    }

    public void Call(params object[] args)
    {
        if (fn != null)
        {
            var arg0 = args.Length > 0 ? args[0] : null;
            var arg1 = args.Length > 1 ? args[1] : null;
            var arg2 = args.Length > 2 ? args[2] : null;
            fn(arg0,arg1,arg2);
        }
    }

    public int CompareTo(IListener other)
    {
        return this.GetId().CompareTo(other.GetId());
    }

    public int GetId()
    {
        return Id;
    }
}


Server Issue or ban message?

When starting up the bot, I receive this error:
[2017-02-22 14:48:25] [PokemonGoBot] [INFO] Setting start location.
[2017-02-22 14:48:25] [PokemonGoBot] [INFO] [x] Coordinates found in passed in location, not geocoding.
[2017-02-22 14:48:25] [PokemonGoBot] [INFO] Location found: 34.0083861,-118.4971739 (34.0083861, -118.4971739, 8)
[2017-02-22 14:48:25] [PokemonGoBot] [INFO] Now at (34.0083861, -118.4971739, 8)
[2017-02-22 14:48:25] [PokemonGoBot] [INFO] Login procedure started.
[2017-02-22 14:48:26] [pgoapi.auth] [ERROR] PTC User Login Error - invalid JSON response: No JSON object could be decoded
[2017-02-22 14:48:26] [PokemonGoBot] [INFO] Login process failed: Invalid JSON response: No JSON object could be decoded
Wed Feb 22 14:48:26 EST 2017 Pokebot  Stopped.
Press any button or wait 20 seconds to continue.
Is this a ban message, or are the servers down? Because I recall that the bot prints something like "Game over, make a new account" if you received a ban.

Same here...

Got this on an old account. Assumed it was a ban message and made a new one. Same message on new account. So maybe not a ban...

It is Niantic server's outage. Closing :)


HTTPS is less likely to be blocked by a firewall

Some customers has reported that they are  unable to build meta-pelion-edge
as their corporate firewall only allows outbound HTTP/HTTPS but not SSH. This
should help with that use case.
Also updated SHAs of devicejs-ng and global-node-modules as they were internally referring to ssh links.

@costanic Ya I don't know the right approach here. I looked at the meta-openembedded and followed what they have done.

./meta-openembedded/meta-oe/recipes-kernel/kpatch/kpatch.inc:6:SRC_URI = "git://github.com/dynup/kpatch.git;protocol=https
./meta-openembedded/meta-oe/recipes-extended/libimobiledevice/libplist_2.0.0.bb:12:SRC_URI = "git://github.com/libimobiledevice/libplist;protocol=https
./meta-openembedded/meta-oe/recipes-extended/libimobiledevice/libusbmuxd_git.bb:12:SRC_URI = "git://github.com/libimobiledevice/libusbmuxd;protocol=https"
./meta-openembedded/meta-oe/recipes-extended/rsyslog/libfastjson_0.99.8.bb:9:SRC_URI = "git://github.com/rsyslog/libfastjson.git;protocol=https
./meta-openembedded/meta-oe/recipes-extended/rsyslog/librelp_1.2.16.bb:9:SRC_URI = "git://github.com/rsyslog/librelp.git;protocol=https
./meta-openembedded/meta-oe/recipes-extended/boinc/boinc-client_7.12.bb:34:SRC_URI = "git://github.com/BOINC/boinc;protocol=https;branch=${BRANCH}
./meta-openembedded/meta-oe/recipes-extended/libreport/libreport_2.9.5.bb:14:SRC_URI = "git://github.com/abrt/libreport.git;protocol=https"
./meta-openembedded/meta-oe/recipes-extended/socketcan/can-isotp_git.bb:6:SRC_URI = "git://github.com/hartkopp/can-isotp.git;protocol=https"
./meta-openembedded/meta-oe/recipes-test/gtest/gtest_1.8.0.bb:13:    git://github.com/google/googletest.git;protocol=https;
./meta-openembedded/meta-oe/recipes-graphics/fvwm/fvwm_2.6.7.bb:37:    git://github.com/fvwmorg/fvwm.git;protocol=https
./meta-openembedded/meta-oe/recipes-connectivity/libuv/libuv_1.20.3.bb:10:SRC_URI = "git://github.com/libuv/libuv.git;protocol=https;branch=${BRANCH};"
./meta-openembedded/meta-oe/recipes-connectivity/libimobiledevice/libimobiledevice_git.bb:15:    git://github.com/libimobiledevice/libimobiledevice;protocol=https
./meta-openembedded/meta-oe/recipes-connectivity/usbmuxd/usbmuxd_git.bb:13:SRC_URI = "git://github.com/libimobiledevice/usbmuxd;protocol=https"
./meta-openembedded/meta-oe/recipes-connectivity/libwebsockets/libwebsockets_3.0.0.bb:10:SRC_URI = "git://github.com/warmcat/libwebsockets.git;protocol=https;"

Should we change it to https and remove "protocol=" specifier?


PLT-4267: ADR: Plutus Tx's Evaluation Strategy, Compiler and Standard Library

To be clear, at a high level I am pro the overall direction of:

Go for strict-by-default
Provide user-controllable mechanisms for getting non-strict let bindings and if possible some kinds of non-strict functions

I just have issues with the proposed mechanisms.

I like this proposal a lot. We just have to make sure that all these great information and examples are properly catalogued in readthedocs.
 Are we going to direct the plutustx users to use an older plutustx compiler version to retain the old behaviour or are we going to expose some extra plutus-tx-plugin flags to "control" this semantic change?

@bezirg has a good point. We should probably discuss the implications for existing scripts in the ADR?

With regard to the stdlib, I think we can provide a non-short-circuiting foldr, and then implement monomorphic any etc. for lists and other concrete types, without using foldr - like some other languages do. Does that make sense?

Yeah, I think that's the only option, really.


Int conversion bug when converting intensities to xarray

WaveBreaking version: main
Python version: 3.10
Operating System: Unix

Description
I wanted to create an xarray file capturing the intensity of anticyclonic northern hemispheric wavebreaking at every time/lat/lon. The intensities should always be >0, but after converting to xarray, the intensities in the array were coming out as positive and negative, with a min of -128 and a max of 127.
Example
I followed the example code for how to create the relevant xarray.
rwb = gpd.read_parquet("./era5/era5_pv_streamers_340K_1959-2022.parquet")

tropospheric = rwb[abs(rwb.mean_var) < 2]
anticyclonic = tropospheric[((tropospheric.level > 0) & (tropospheric.intensity > 0) | # Northern hemisphere
                    (tropospheric.level < 0) & (tropospheric.intensity < 0))] # Southern hemisphere

nh = anticyclonic[np.asarray(list(anticyclonic.com))[:,1] > 0] # restrict to northern hemisphere

# create dataset with correct dimensions
times = pd.date_range(start='1959-01-01T00', end="2022-12-31T18", freq='6H')
lats = np.arange(-89,90,1)
lons = np.arange(-180,180,1)
da = xr.DataArray(dims=["time", "lat", "lon"], coords={"time":times, "lat":lats, "lon":lons}).to_dataset(name="init")

da["tropos"] = wb.to_xarray(data = da.init, events = nh, flag='intensity')
da = da.drop("init")

Running print(nh.intensity.values.min() returns 0.0 as expected, but running print(da['tropos'].values.flatten().min()) returns np.int8(-128).
The problem
The offending line in the to_xarray function is the following: data_flagged = data_flagged.astype("int8"). This is fine if flag = 'ones', but generates a signed-unsigned float/int conversion bug when running flag = 'intensity'. I solved this in a local install by just adding an if flag == 'intensity' statement, and saving as float64 in that case. Maybe there are better ways.

Hi Kristian
Please excuse my late response. I'm actually not working on this project anymore and have had to find some time to look at it.
Thanks a lot for pointing that out! I initially added the line data_flagged = data_flagged.astype("int8") to reduce the size of the xarray.Dataset when using the flag='ones'. Of course, this does not work when using other flag names. I added an if flag=='ones'  statement for the conversion to an int8.
I committed the change in the latest release (v0.3.8). Please let me know if you still experience issues converting the event dataframes to a xarray.


get current dragging item in setOnItemDragEventListener

i'm using onLongPress for start dragging,what i want do is to check if item position is chganged or not and if its not changed do something.
using  setOnItemDragEventListener i have 3 mothod i needed but i cant figure out how get current dragged item in these methods.

Hi. Have you checked the declaration of the OnItemDragEventListener?
    /**
     * Used for listening item drag events
     */
    public interface OnItemDragEventListener {
        /**
         * Callback method to be invoked when dragging is started.
         *
         * @param position The position of the item.
         */
        void onItemDragStarted(int position);

        /**
         * Callback method to be invoked when item position is changed during drag.
         *
         * @param fromPosition The old position of the item.
         * @param toPosition   The new position of the item
         */
        void onItemDragPositionChanged(int fromPosition, int toPosition);

        /**
         * Callback method to be invoked when dragging is finished.
         *
         * @param fromPosition Previous position of the item.
         * @param toPosition   New position of the item.
         * @param result       Indicates whether the dragging operation was succeeded.
         */
        void onItemDragFinished(int fromPosition, int toPosition, boolean result);
    }

OK, I'll give some examples how the listener works.

example scenario - 1

Long press the first item
After entering to dragging state, move the item (slowly) to position 3
Release the finger on position 3

how OnItemDragEventListener works

Long press the first item

onItemDragStarted(position = 0)

After entering to dragging state, move the item (slowly) to position 3

onItemDragPositionChanged(fromPosition = 0, toPosition = 1)
onItemDragPositionChanged(fromPosition = 1, toPosition = 2)
onItemDragPositionChanged(fromPosition = 2, toPosition = 3)

Release the finger on position 3

onItemDragFinished(fromPosition = 0, toPosition = 3, result = true)

:point_right: fromPosition != toPosition ... position changed

example scenario - 2

Long press the first item
After entering to dragging state, move the item (slowly) to position 3
Then move the dragging item (slowly) to position 0
Release the finger on position 0

how OnItemDragEventListener works

Long press the first item

onItemDragStarted(position = 0)

After entering to dragging state, move the item (slowly) to position 3

onItemDragPositionChanged(fromPosition = 0, toPosition = 1)
onItemDragPositionChanged(fromPosition = 1, toPosition = 2)
onItemDragPositionChanged(fromPosition = 2, toPosition = 3)

Then move the dragging item (slowly) to position 0

onItemDragPositionChanged(fromPosition = 3, toPosition = 2)
onItemDragPositionChanged(fromPosition = 2, toPosition = 1)
onItemDragPositionChanged(fromPosition = 1, toPosition = 0)

Release the finger on position 0

onItemDragFinished(fromPosition = 0, toPosition = 0, result = true)

:point_right: fromPosition == toPosition ... position not changed

ty @h6ah4i i think i explained my question really bad.
i know how to find out position is changed or not (its done as you described), what need is to get the item and viewHolder that currently is dragging, i want to change something on that view and get information after change.
for example user do longPress but wont start dragging after releasing item,user can change name of that item (a EditText will be replaced by TextView),and after his done changes will be commit to data and DB.

@BlackAngel1991
Those features are not provided by this library. But...
Obtaining a view holder from position is a common technique of RecyclerView. You can use the RecyclerView.findViewHolderForAdapterPosition() method.
Obtaining an item from position is not provided by this library and RecyclerView. Just implement it to your adapter like this
Item getItem(int position) {
     return mItems.get(position);
}

unfortunately RecyclerView.findViewHolderForAdapterPosition() return null value.
and im afraid this method :
     return mItems.get(position);
}```
wont work for bcoz im using Expandable Draggable Swipeable adapter.

@h6ah4i
as a workaround for my problem i modified OnItemDragEventListener and passed mDraggingItemViewHolder as a parameter to it.
i changed :
public void onItemDragFinished(int fromPosition, int toPosition, boolean result)
to
public void onItemDragFinished(int fromPosition, int toPosition, boolean result,,RecyclerView.ViewHolder holder)

@BlackAngel1991 I have just pushed an modified master branch that added how to handle drag & drop event listener with expandable list. Check the following code and please let me know if you have more questions about it ;)
https://github.com/h6ah4i/android-advancedrecyclerview/blob/demo_issue_187/example/src/main/java/com/h6ah4i/android/example/advrecyclerview/demo_eds/RecyclerListViewFragment.java#L90

wow @h6ah4i that's exactly what i needed . thank you so much for answering, you are awesome.


prefix and suffix props seem to be applied globally

Adding a prefix or suffix prop to one <number> component applies it to all of them.
<script setup>
import {ref} from 'vue'
import { number } from '@coders-tm/vue-number-format';

const count = ref(5)
const count2 = ref(3)
</script>

<template>
  <number prefix="$" v-model="count" />
  <number v-model="count2" />
</template>

Here I'd expect that the prefix would only appear on the first component only.

@martinnobis Please use latest version. I have fix it. Thanks for finding the issue. :)

Thanks, that fixed it!


Update readme examples to metalsmith-layouts

The examples in the readme use the now deprecated metalsmith-templates. Also, the examples folder should probably be updated as well.

I've created a simple but useful boilerplate for static site generation with metalsmith that might be useful for newcomers: https://github.com/superwolff/boilerplate-in-place. Feel free to include it somewhere.

Fixed by #182


Add option to skip fields when creating the parser

Either as a flag to the metadata, or as an argument to for example skip all fields that end with a certain postfix.

My vote would go to explicitly indicating it in the metadata


change manifest.json path for vite 5.0

manifest.json default path got moved into a .vite folder cf vitejs/vite#14230

@tsnobip sorry, missed this! We need to upgrade Vite versions of the router here as well, right?

@tsnobip sorry, missed this! We need to upgrade Vite versions of the router here as well, right?

yes, sorry for the half-baked PR, you need to upgrade vite to v5 too, otherwise it won't work.
I just did it as a reminder because I got bitten by this and I knew you would too eventually once you'd upgrade vite.

I've upgraded Vite to 5.x now, and moved the server stuff into its own Vite plugin. We can continue fixing these things whenever we go back to solving server stuff.


Plugins: Upgrading doesn't always seem to persist in UI

The buttons return after a refresh. Seems like the upgrades worked, but nothing.

Ah, so the issue only happens when using --no-cluster..

right, it uses cluster to do the self restart


Users/semurdoc/enable fpp

swap asset upload ordering to enable FPP when it is ready.

@Seann-Murdock please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"


Export GPX Route via QR Code

The initial work for this feature was done by @EssBee59. I've integrated the changes and made some adjustments (using a modal dialog instead of opening a new browser window; cleanup code; add support for go-to areas).

a new button “Generate QR Code” was added to the export dialog
when that button is clicked a new modal dialog opens
the new dialog shows a QR Code which contains the “Export as GPX” URL
if that URL is opened the GPX file can be downloaded and opened directly on a mobile phone (works on Android and iOS; I've tested it successfully with Firefox and Garmin Connect on iOS 👍)
it's possible that the route definition is too long for encoding it into a QR Code, in this case an error message is shown in the modal dialog (happens when a large go-to area is active)
in the QR Code dialog three buttons are shown – it's possible to switch the QR Code size (helpful if the camera fail to recognize a smaller QR Code)
the initial size of the QR Code is chosen according the to URL length (< 500 characters => small, < 1800 characters => medium, else large)
i18n strings are extracted to en.json
the MIT-licensed QR Code library was included (js/library/qrcode.js) because it's unavailable as npm package

New Button in Export Dialog:

QR Code Dialog:

Error when URL is too long:

Want to check out this feature? I've deployed it to my BRouter-Web instance at https://brouter.m11n.de/

Hello,
the current implementation is vlaid only for brouter-web version smaller 0.17.0  !!!!!
(as the way to dowload the GPX file changed in version 0.17.0 to support "Draw a direct line between two route points")
So please freeze this pull request, I will have a look at the new download and try to adapt this pull request.
@ mjaschen:
Be aware of this problem / current limitation: On your server, the "QRCODE" export do not deliver the right track when a direct line was drawn
Regards

A straightforward way might be to use the client URL (window.location.href) for the QR code and add a search/query parameter like /?export# to open the export dialog after the route is loaded, e.g. by adding something like this to index.js:
        var searchParams = new URLSearchParams(window.location.search);
        if (searchParams.has('export')) {
            routing.once('routing:setWaypointsEnd', function (evt) {
                $('#export').modal('show');
            });
        }

I have been thinking about this feature for the last few days. A user friendly workflow could look like this:

the QR code contains the current link of the planned route with an additional URL parameter download=gpx (kml, xml, ...)
if BRouter Web is opened via this link, a modal dialog "Download route or view the planned route" appears with a button for each of the two actions
a click on "Download" initiates the download, closes the dialog and removes the URL parameter download=*.
a click on "View" closes the dialog and removes the URL parameter download=*.

Norbert,
Thank a lot for your help!!
With your tipp I think it could also be possible to download the GPX file directly on the device that reads the QRCODE (smartphone)?
The other way read the QRCODE and download the GPX file (that I prefer as more confortable) is to use a special app on the android smartphone: also here I think the download can be automated...
@mjaschen:
Do you plan to implement/ adapt the change on the brouter-web side?
If so, I will only change/adapt the app when your part is ready...
Regards
Ess Bee

With your tipp I think it could also be possible to download the GPX file (=write the track-data) directly on the device that reads the QRCODE (smartphone)?

Don't know what you mean by "directly", but my proposal is that when reading the QR code on the phone, it opens brouter-web in the browser and automatically opens the export dialog. It could also automatically start a specific (e.g. GPX) download after starting brouter-web. The download will be the same as when you open brouter-web on the phone and do a manual export there.

The other way read the QRCODE and download the GPX file (that I prefer as more confortable) is to use a special app on the android smartphone: also here I think the download can be automated...

As the track formatting now is done in brouter-web JavaScript this could be bit difficult now. You would probably need to make a hybrid app that loads brouter-web in a web view. Or make a minimal headless bundle if Android can run JavaScript.

the QR code contains the current link of the planned route with an additional URL parameter download=gpx (kml, xml, ...)

For the sharing use case, e.g. embedding the QR code in a blog or website, it would be better to let the reader choose the format.
It might then make sense to have the "generate QR code" button in a separate "share" dialog (see #299) and only open the "export" dialog when reading.

Yes, that are two valid points 👍 It also keeps redundancies low as the markup for export/download is already available.
Maybe we can add in that case an “Info” section or the like to the export modal which shows one or two sentences about BRouter Web. So people get an idea of where they ended up and what they can do there besides just downloading a GPX file.


silverlight only?

I believe XNA also runs the same piece of code that silverlight does.
I had to change to this in order to get XNA working :
if WINDOWS_PHONE
define USE_CSHARP_SQLITE
define SILVERLIGHT
endif
sqlite-net is also ideal for embedded database for XNA games.

Thanks to the nuget package sqlite-net-pcl this is all taken care of for you!


Cannot install on Python 2.7 due to bsddb

macOS Sierra
Python 2.7.12
Gutenberg 0.4.2

$ pip install gutenberg
Collecting gutenberg
  Downloading Gutenberg-0.4.2.tar.gz
Collecting bsddb3>=6.1.0 (from gutenberg)
  Using cached bsddb3-6.2.1.tar.gz
    Complete output from command python setup.py egg_info:
    Can't find a local Berkeley DB installation.
    (suggestion: try the --berkeley-db=/path/to/bsddb option)

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/zm/drwrq_ld04s7vk50p00c9lnw0000gn/T/pip-build-TGvyoe/bsddb3/

The installation instructions in the README say bsddb is only needed for Python 3:

Installation
is project is on PyPI, so I'd recommend that you just install everything from there using your favourite Python package manager.
pip install gutenberg

...

Python 3
This package depends on BSD-DB. The bsddb module was removed from the Python standard library since version 2.7. This means that if you wish to use gutenberg on Python 3, you will need to manually install BSD-DB.

Is that not the case? Is bsddb3 needed for Python 2?

No, you shouldn't need to install bsddb3 and I guess there might be a bug on when setup.py tries to figure out what requirements to install for you.
However, you do need to install Berkeley DB separately from Gutenberg as python just comes with the module to connect to it and not the actual DB (which should be clarified in the instructions).
On Mac though, use homebrew to install it via brew install berkeley-db4, and then try again to do pip install gutenberg

It'd be great to put this stuff in a Dependencies section for the README before Installation, with the instructions for each OS.

Thanks, @MasterOdin, for the macOS instructions. I created #69 to include the instructions in our readme.


Use python 3.8 for tests

Motivation
Our system tests use the Google Cloud Platform SDK, which is incompatible with Python 3.9, released on Oct. 5th, 2020.
Since our dockerfile specifies python:latest instead of a fixed version, the system tests stopped working.
Changes
Specify Python 3.8 to be used for our system tests.
Test Plan
Run bors try, triggering a full system test suite run.

bors try

bors merge

I canceled the last bors try to free up Travis to run bors merge, since we've passed the critical stage of the run successfully.


Use static_assertions to check for trait impls

Objective

Tests are manually checking whether derived types implement certain traits. (Specifically in `bevy_reflect.)
#11182 introduces static_assertions to automatically check this.
Simplifies Reflect test in #11195.
Closes #11196.

Solution

Add static_assertions and replace current tests.

I wasn't sure whether to remove the existing test or not. What do you think?

Awesome! I'll remove the TODO comments.
I'm going to check if any other crates use the same should_impl test style, and I'll fix CI too.

@cBournhonesque, have time for a quick review?

I couldn't find any other tests that would work for this. As long as CI passes, I think this is ready to merge. :)


Add trino controller to control all the trino ecosystems.

Trino controller needs to control all the trino ecosystems.

create trino operator
create helm operator.
create nginx ingress controller.
create cert manager.
create trino gateway.
create trino cluster.
create prometheus.
create grafana.
scale out trino cluster.
register trino cluster to trino gateway.
deregister trino cluster from trino gateway.
detect exhausted trino cluster.

redis caching added to trino gateway to cache query id and nextUri which will be used to route the queries to the selected backend  trino coordinator in nextUri.

https://github.com/cloudcheflabs/dataroaster/pull/26


[IMP] enable database manager

On OCA repos, sometimes there are incompatible modules. Or, at least, modules that introduce pollute others' behaviors.
When doing functional tests of a PR and the DB already contains all addons for the branch, it's easy to find out a bug that really won't be a bug in the current PR, but instead a side effect of another addon in the same repo. Many times, that addon isn't even used in production and the flaky use case is irrelevant.
It's helpful to have the possibility of installing an empty database. This way, only the wanted addons can be installed, and there's less chance for unwanted side-effects.
On a recent real use case that we have recorded, a user lost 30 minutes in the functional review, only to uninstall modules that polluted the one he wanted to test. At the end, there were still failures that we're not sure if belong to the important module.
For these reasons, I'm configuring Odoo runboat instances to have the database manager enabled by default, with a standard password: admin.
I took the chance to add a couple of good default parameters for any database.
@moduon @rafaelbn

But then what is going to drop those user-created databases?

So yeah, I'm -1 on this, as this changes the promises of this image.
If we want a runboat db with only base installed that could be added as a runboat feature.

I created https://github.com/sbidoul/runboat/issues/94 to track this.


Add more Knative-sandbox repos for CLOTributor

Add more Knative-sandbox repos for CLOTributor
Related: https://github.com/cncf/clomonitor/pull/1074

/hold
Wrong URLs!

/unhold

It looks like GitHub is having some issues now.


Localised Strings in addition to Locale.preferredLanguages

Adds optional preferredLanguages parameter to generated strings functions. Used as a replacement for Locale.preferredLanguages.
Usage:
// Assuming the app is localised in English and French
// Returns French localisation, even if Phone is set to English
let hello = R.string.welcome.hello(preferredLanguages: ["nl", "fr"])

Fixes https://github.com/mac-cain13/R.swift/issues/219

Note: As of iOS 13, end users can select the preferred language for an app in Settings.

@tomlokhorst Could we set the preferred language in general instead of insert it in each call?

No, that's not possible with this API.

@tomlokhorst Thanks for your response! In my app, I let the user choose the language he wants in the start of the app. Therefore, I try to find a way to make this the default localization language. I see you set the applicationLocale with the first preferred language. Couldn't we set it dynamic?

Would be great if preferredLanguages: would be a method of string resource. I too face a situation where I need to change locale throughout the app, an the current API does not allow me to do that without every piece of the app knowing what the current locale is.

@IharAndreyeu @abdelmagied94  were you able to have work around regarding this?

@tomlokhorst wanted to check with regards to the above comment, setting preferred language globally for the entire app. Is there anything in the roadmap to address this use case ?

Yesterday I change all my strings(~1k) to R.string in my app. In my app I can change language of app.
But after change strings to R.string - change language in app not working.
I need global change of locale. How I can do it? Variant with preferredLanguages does not suit me.

With help of Article: Strong Typed Resources Management with R.swift - I found my workaround.
I create module Languages (where enum Languages and extension R)
import Foundation
import Localize_Swift
import Rswift

enum Languages: String {
    case ru
    case en
    case lt
    case lv
    case et
}

extension Languages {
    var langcode: String {
        switch self {
        case .en:
            return "en"
        case .ru:
            return "ru-RU"
        case .lt:
            return "lt-LT"
        case .lv:
            return "lv-LV"
        case .et:
            return "et-EE"
        }
    }

    static private var preffered: Languages = Languages.en
    static private var langBundle: Bundle?

    static func getLang() -> String {
        return preffered.langcode
    }

    static func setLang(_ langCode: String) {
        preffered = Languages(rawValue: langCode) ?? Languages.en
        let url = Bundle(for: R.ClassR.self).url(forResource: preffered.langcode, withExtension: "lproj")
        langBundle = Bundle(url: url!)
    }

    static func getBundle() -> Bundle? {
        if (langBundle == nil) {
            let url = Bundle(for: R.ClassR.self).url(forResource: preffered.langcode, withExtension: "lproj")
            langBundle = Bundle(url: url!)
        }
        return langBundle
    }

    static func getName(_ code: String) -> String {
        let locale : NSLocale = NSLocale(localeIdentifier: code)
        return locale.displayName(forKey: NSLocale.Key.identifier, value: code) ?? code
    }

    static func getLocalName(_ code: String) -> String {
        return Localize.displayNameForLanguage(code)
    }
}

extension R {
    static var text = R.string.localizable.self

    fileprivate class ClassR {}

    static func local(_ string: StringResource) -> String {
        return NSLocalizedString("\(string.key)", bundle: Languages.getBundle()!, comment: "")
    }
}

And when I change language of app I run this:
Languages.setLang(langCode)

And strings...
Before R.swift:
"test".localized()

With standard R.swift:
R.string.localizable.test()

With my workaround:
R.local(R.text.test)


Question: why an age requirement?

Why do you put an age requirement on a simple notes taking app?

Hey Thewisem,
thanks you for your feedback.
I was planning to implement a cloud storage to store user data. But I ditched the plan in favor of a local database. So the app actually does't need to collect the date of birth. We will try to remove the age verification in the upcoming releases!
Regards, Michael

Thank you for making this app local only. As privacy minded person, I would like to not have my diary have internet access at all. This is a great app you made

This is just a suggestion, but could you encrypt the backup in a future update? Would be kinda great if your app is the only one that could read the backup and it is encrypted.
With Regards,
Wisem

Thank you for your suggestion, Wisem! Encrypting the backup would indeed make the app more private. We will try to implement that!
Regards, Michael


⚠ ClamAV has degraded performance

In 2eeba78, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in a6195e4.


Can anyone explain this early problem?

Hi,
Love the idea. Needs a bit more documentation perhaps?
I got as far as
$ ./gradlew build
and then this failed after a bit:

Task :installProject FAILED
Install PyLookup2019-09 ............................................. [STARTING]
Obtaining file:///home/mike/software%20projects/eclipse-workspace/PyLookup2019-09
No files/directories in /home/mike/software projects/eclipse-workspace/PyLookup2019-09 (from PKG-INFO)

"No files/directories"... what can that mean? There was a very basic starter file (__init__.py). Also can you explain what you mean by a "standard python project"? Does setup.cfg have to contain something specific perhaps?
Also, what might it mean by "(from PKG-INFO)"?
Thanks

try moving __init__.py to src/__init__.py


Remove carbon registry dependency

Carbon registry dependencies need to be removed from the Identity Server to improve performance.
First unnecessary registry dependencies will be removed
Related PRs

https://github.com/wso2/carbon-identity-framework/pull/4758
https://github.com/wso2-extensions/identity-user-account-association/pull/48
https://github.com/wso2/identity-apps/pull/3962
https://github.com/wso2-extensions/identity-data-publisher-authentication/pull/115
https://github.com/wso2-extensions/identity-conditional-auth-functions/pull/135
https://github.com/wso2-extensions/identity-user-ws/pull/58
https://github.com/wso2-extensions/identity-outbound-provisioning-salesforce/pull/30
https://github.com/wso2-extensions/identity-outbound-auth-samlsso/pull/163
https://github.com/wso2-extensions/identity-carbon-auth-saml2/pull/96
https://github.com/wso2-extensions/identity-governance/pull/721
https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/2111
https://github.com/wso2-extensions/identity-inbound-auth-saml/pull/400

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Are we there yet ?

Hey devs, its been ages. Just doing a quick sanity check 😄 to see if you're all doing well.
Missing your activity!
@H-K-R @adarshswaminath @foss2cyber @takanome-dev @arcVaishali

The current state of our landing page. Simplicity won the battle at the end of the day.
Thank you all for your support throughout this project 🙏

Hey y'all, I'm doing pretty good 😃
haven't been contributing to open source for like 4 months (😱)just been busy with work and freelance projects but will start being more active soon hopefully.
Thanks for checking in 🤗

That's great! I'm confident you're pushing through! Looking for ideas on how to monetize this project. Working on changes for v2
Will update you soon 🚀
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: El Hadji Malick Seck @.>
Sent: Monday, January 22, 2024 11:13:44 pm
To: prjctimg/huetiful @.>
Cc: Dean Tarisai @.>; Author @.>
Subject: Re: [prjctimg/huetiful] Are we there yet ? (Issue #127)
Hey y'all, I'm doing pretty good 😃
haven't been contributing to open source for like 4 months (😱)just been busy with work and freelance projects but will start being more active soon hopefully.
Thanks for checking in 🤗
—
Reply to this email directly, view it on GitHubhttps://github.com/prjctimg/huetiful/issues/127#issuecomment-1904827695, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZAJNB23GTA2Z4C7W6EYZ2LYP3JARAVCNFSM6AAAAABCD6HXAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBUHAZDONRZGU.
You are receiving this because you authored the thread.Message ID: @.***>


backupccl: add exportrequest.delay.total metric

PR: https://github.com/cockroachdb/cockroach/pull/67310
From release notes:

Introduced a new metric exportrequest.delay.total to track how long ExportRequests (issued by BACKUP) are delayed by throttling mechansisms. [#67310][#67310]

Closing this issue for public docs, @dt not sure if anything needs to be documented internally for TSEs?


ECRECOVER ethereum addresses

Closes #1068

Adds std::vm::evm namespace to the std-lib, and implements ecr
Recovers a 20-byte ethereum address. Value is returned as a left-padded Address type.
Currently recovers address from the literal signed data: EIP-191 domain separators (\x19Ethereum ... etc. ) are not yet taken into account. In ethereum web3 libraries these are often prepended automatically so we will probably need to add support to handle them later, e.g. for #1065

Sorry -  I was a bit quick to request review, and now you all get to witness my first struggle with the sway CI

@simonr0204 run the formatter on all the sway files... QUICK... before John sees it!

@simonr0204 run the formatter on all the sway files... QUICK... before John sees it!

But... CI is passing :o

@simonr0204 run the formatter on all the sway files... QUICK... before John sees it!

But... CI is passing :o

We don't yet have formatting CI checks for Sway.
The current CI pipleline check for Sway formatting is just John first waving from his desk over PR reviews.


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 3518 ms

Resolved: Bofrost.lu performance has improved in 6139071.


Crash

Issue description:
Crash with gregtech
What happens:
when trying to manipulate(via autocrafting or manually) some gregtech liquids stored in external storage game crashes
What you expected to happen:
game continues to run
Steps to reproduce:

setup a barebones network
add an external storage to a tank with gt chlorine
add some chlorine to the tank
when you try to take chlorine from fluid terminal the game crashes
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>60
Refined Storage: 2.3.2
Gregtech Ceu: 2.8.6-beta

Does this issue occur on a server? [yes/no]
both on server multiplayer and local singleplayer
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

Default rs doesn't crash

same issue here with an industrial forgoing black hole tank
doing nothing except removing the tank solves the crashing
https://pastebin.com/TMM0v4EX
Minecraft: 1.12.2
Forge: <IP_ADDRESS>60
Refined Storage: 2.3.2
Industrial Forgoing: 1.12.13-237


Add LoggingConfiguration resource

Issue #, if available:

https://github.com/aws-controllers-k8s/community/issues/1530

Description of changes:
Adding the logging configuration resource for AMP controller. API Reference
Note: This is a draft PR since the following change to enable acktest to bootstrap CW Log group needs to be merged first to run the end to end tests. Added bootstraping for CW log groups #339
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/unhold

/retest

Seems like tests/test_alert_manager_definition.py::TestAlertManagerDefinition::test_creating_two_alert_manager_for_one_workspace  failed due a created resource that was in a DELETING state will retry to run the tests
/retest

/retest

/test all

Making sure that the tests are not flaky. Running them for the 3rd time
/test all

/lgtm


Potential to expand support for HIP Apps?

Hi,
Just curious how challenging it would be to add support for HIP applications.  Any thoughts?
Thanks,
Jeff

Sorry for the delay in getting to this.
From a design perspective, I don't think this would be too hard to support. It would mostly involve another wrapper library, like the OpenCL wrapper contained in ./src/cl_wrapper/ . However, I expect that actually implementing the checkers will lead to a number of roadblocks that would add up to a reasonable amount of work.
For instance, most of our OpenCL overflow checkers are configured to be be able to run on the GPU immediately after the user kernel runs. We take advantage of the fact that the user kernel will require a work queue and working OpenCL device, so we don't allocate our own. As such, we would need to port all of this to work in a HIP/HCC environment as well.
In any case, I believe this is a good feature request but I can't give an estimate as to when/if it would be done.


How can we reproduce the mask AP of COCO or LVIS as Table 5?

Hope to release the eval codes as soon as possible, thanks.

same question

same question


Reset timeslider on close

It would be useful to have the timeslider reset the temporal extents when the timeslider widget it is closed so that all data date ranges are shown.

This is a good idea. I can add it if I get some time someday. Otherwise pull requests welcome 😀
Something like this would work:
if(this.parentWidget){
    this.own(this.parentWidget.on('show', lang.hitch(this, 'resetTimeslider')));
}

Just found this example: https://github.com/tmcgee/cmv-widgets/commit/80809d48915c52fc47bbd7a583e8265d64490bc6


error: expected type, found keyword impl

I'm running cargo build on Piston/conrad and came across an error involving rayon.
/home/cortland/.cargo/registry/src/github.com-1ecc6299db9ec823/rayon-0.6.0/src/scope/test.rs:95:34: 95:38 error: expected type, found keyword `impl`
/home/cortland/.cargo/registry/src/github.com-1ecc6299db9ec823/rayon-0.6.0/src/scope/test.rs:95     pub fn iter<'s>(&'s self) -> impl Iterator<Item = &'s T> + 's {
                                                                                ^~~~
error: aborting due to previous error

Any idea why this is happening?

What's your Rust version? Rayon requires 1.12 to understand the impl Trait syntax, even though that's only used in tests.

What's your Rust version? Rayon requires 1.12 to understand the impl Trait syntax, even though that's only used in tests.

1.12 >= or only 1.12?

1.12 >= or only 1.12?

1.12 or greater.

1.12 or greater.

Much obliged. Appears that my version wasn't correct. I thought otherwise.

Much obliged. Appears that my version wasn't correct. I thought otherwise.


Bump cassandra3 to 3.11.12

Cassandra installation failing with Version '3.11.11' for 'cassandra' was not found. Looking at the apt repo shows only listed version now in 3.11.12: https://downloads.apache.org/cassandra/debian/dists/311x/main/

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

thx incorporated the change in #188


Enable cache-control in fontsource API request header

Describe the feature you'd like to request
I have noticed Fontsource currently doesn't enable caching font files downloaded from API requests. Is it possible to enable Cache-Control in the request header with JSDelivr CDN?
Describe the solution you'd like
Users don't need to download font files from CDN every time if they choose to use the Fontsource API.

We're currently working with jsDelivr for a custom CDN endpoint. The current API was an extremely quick rewrite from the old one (which completely stopped working when v5 was released) so we're still missing out on a lot of caching rules. I've just had not much time to work on this project these days so progress to implement all the handlers for the rewritten API properly has been slow.
I've got a branch which implements a much better Download API that follows optimal practices, which we'll then layer the jsDelivr CDN on top for edge caching.

This has been resolved in multiple PRs such as #812 or #829.


For npm usage, yarn/npm test fails due to a bug inside update.npm.full.js:  "TypeError: Cannot read property 'indexOf' of undefined"

Hi,
After I did "npm install browser-update --save" and did a yarn test on my front-end project which uses browser-update, I'm getting this error:
● Test suite failed to run
TypeError: Cannot read property 'indexOf' of undefined
at $buo (packages/core/dist/webpack:/packages/core/node_modules/browser-update/update.npm.full.js:317:1)
Line 317 of update.npm.full.js is this:
op.already_shown=document.cookie.indexOf("browserupdateorg=pause")>-1 && !op.ignorecookie;
Therefore, it seems that "document.cookie" is undefined and needs to be initialized to something before being used.
Can anyone look into resolving this bug?
Thanks.

Hi,
What is your use case? What tests are you running. I could detect the test scenario and det the cookie.


Upload changes from main

remove envVars from index.ts

Hm, what's going on here? The code was already merged to main.


Update 07-bootstrapping-etcd.md

there is no such section in prerequisites

@b10s I think a much better fix would be to bring back that section as it's very useful:
https://github.com/kelseyhightower/kubernetes-the-hard-way/blob/master/docs/01-prerequisites.md#running-commands-in-parallel-with-tmux


SSH tunneling in Windows Server 2019 does not seem to work

Description
When trying to use the built-in ssh command in windows-latest (Windows Server 2019) for local port forwarding, the specified ports are not being listening on or forwarded.
Area for Triage:
SSH
Question, Bug, or Feature?:
Question/Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[x] Windows Server 2019

Expected behavior
Upon issuing the ssh command and having ssh running, the ports specified in the ssh command should be listening and forwarding the packets to the destination via ssh.
Actual behavior
The ports specified in the ssh command are not opened or listened locally.
Repro steps
The GitHub Actions script is as follows (with the sensitive info replaced by placeholders):
name: CD

on:
  push:
    branches: [ master ]
  workflow_dispatch:

jobs:
  build:

    runs-on: windows-latest

    steps:
    - name: Checkout Code
      uses: actions/checkout@v2

    - name: Cache Dependencies
      uses: actions/cache@v2
      id: cache-dependencies
      with:
        path: ~/.nuget/packages
        key: ${{ runner.os }}-nuget-${{ hashFiles('**/packages.lock.json') }}

    - name: Open SSH tunnel
      run: |
        echo "${{ secrets.SSH_KEY }}" > key.pem
        $acl = Get-Acl key.pem
        $acl.SetAccessRuleProtection($True,$False)
        $rule = New-Object System.Security.AccessControl.FileSystemAccessRule("$env:UserDomain\$env:UserName", "FullControl", "Allow")
        $acl.AddAccessRule($rule)
        Set-Acl key.pem $acl
        Start-Process ssh -ArgumentList "-tt -L 1433:<server int ip>:1433 -L 8172:<server int ip>:8172 <username>@<ssh host ip> -i key.pem"
        Start-Sleep -s 10
        Get-Process -Name ssh | echo
        netstat -an

    - name: Deploy to Dev
      run: |
        Get-Process -Name ssh | echo
        netstat -an
        dotnet publish -c Release `
        -p:PublishProfile=<publish profile> `
        -p:Password=${{ secrets.DEPLOY_PASSWORD }} `
        -p:AllowUntrustedCertificate=True

    - name: Close SSH tunnel
      run: kill -Name ssh

`
Notes:

The commands in the script to open the SSH tunnel and do the deployment run fine locally in a Windows 10 Pro 1909 machine and I am able to build and deploy to the server from my local machine.

The commands  Start-Sleep -s 10, Get-Process -Name ssh | echo and  netstat -an are for debugging purposes only. I included the  Start-Sleep -s 10 command to account for the time taken to make the SSH connection, which when connecting to the server using a local machine takes less than 5 secs (turns out adding this command made no difference). While the Get-Process -Name ssh | echo command shows that the ssh process is indeed running in both the Open SSH tunnel and the Deploy to Dev steps, the  netstat -an command shows that the specified ports 1433 and 8172 are not being listened on locally in either step, so the deployment fails with the error
MSDEPLOY : error Code: ERROR_DESTINATION_NOT_REACHABLE
MSDEPLOY : error : Unable to connect to the remote server
MSDEPLOY : error : No connection could be made because the target machine actively refused it <IP_ADDRESS>:8172

Is there something wrong in my code, or something else that I should/shouldn't have done to make SSH tunneling work? Any help would be greatly appreciated.

@amoszheng i suppose you use ssh tunnel to communicate between the processes on the same host. Can you please try to use localhost instead of network interfaces addresses. I am not sure right now but there's high probability the opening listen sockets are disable on agent hosts.
Also, can you intercept the output of ssh server, please?

@amoszheng i suppose you use ssh tunnel to communicate between the processes on the same host. Can you please try to use localhost instead of network interfaces addresses. I am not sure right now but there's high probability the opening listen sockets are disable on agent hosts.

Correct me if I am wrong, but localhost would resolve to <IP_ADDRESS> as well, right?

Also, can you intercept the output of ssh server, please?

Could you give me instructions on how to do that without interrupting the main process?

Hello @amoszheng,
You are right, localhost resolves to <IP_ADDRESS>. I verified it using the folowing commands on agents:
ping localhost
Pinging WIN-61K18E5PQA9 [::1] with 32 bytes of data:
Reply from ::1: time<1ms
Reply from ::1: time<1ms
Reply from ::1: time<1ms
Reply from ::1: time<1ms
ping localhost -4
Pinging WIN-61K18E5PQA9 [<IP_ADDRESS>] with 32 bytes of data:
Reply from <IP_ADDRESS>: bytes=32 time=1ms TTL=128
Reply from <IP_ADDRESS>: bytes=32 time<1ms TTL=128
Reply from <IP_ADDRESS>: bytes=32 time<1ms TTL=128
Reply from <IP_ADDRESS>: bytes=32 time<1ms TTL=128

I've tried to reproduce the issue on the agent's side, and it seems it's not possible to open the local socket on agent machines. I tried to add additional flags to ssh command -vvv and to Start-Process command "-Verb open", but there is no additional information about what is causing the issue.
We are investigating the issue.

@amoszheng since Start-Process starts ssh in background mode it's really hard to get logs from there. So, I suggest to start ssh without "Start-Process" just for debugging purposes with -vvv flag, in order to understand why local sockets do not work.
We are looking forward to your reply.

@Darleev I found what the problem was. Since every workflow run on GitHub runners happens in a fresh VM, there was no known_hosts file, so every time ssh is called it is stuck in the ssh prompt
The authenticity of host <host ip> can't be established.
RSA key fingerprint is <fingerprint>.
Are you sure you want to continue connecting (yes/no)?

So a solution that worked for me was creating a new SSH_KNOWN_HOSTS secret and adding the following before the ssh command:
mkdir ~/.ssh
echo "${{ secrets.SSH_KNOWN_HOSTS }}" > ~/.ssh/known_hosts

Since the server fingerprint is probably not a secret, it could also be defined in an environment variable or hard-coded in the Action code instead of being stored in a GitHub secret.
An alternative would be to add the argument -o "StrictHostKeyChecking no" to the ssh command, but that would make the script susceptible to ssh MITM attacks.
Thanks for all the help troubleshooting the issue!

@Darleev I found what the problem was. Since every workflow run on GitHub runners happens in a fresh VM, there was no known_hosts file, so every time ssh is called it is stuck in the ssh prompt
The authenticity of host <host ip> can't be established.
RSA key fingerprint is <fingerprint>.
Are you sure you want to continue connecting (yes/no)?

So a solution that worked for me was creating a new SSH_KNOWN_HOSTS secret and adding the following before the ssh command:
mkdir ~/.ssh
echo "${{ secrets.SSH_KNOWN_HOSTS }}" > ~/.ssh/known_hosts

Since the server fingerprint is probably not a secret, it could also be defined in an environment variable or hard-coded in the Action code instead of being stored in a GitHub secret.
An alternative would be to add the argument -o "StrictHostKeyChecking no" to the ssh command, but that would make the script susceptible to ssh MITM attacks.
Thanks for all the help troubleshooting the issue!