Issue with environment variables

airflow version: 1.10.0
Running docker-compose_LocalExecutor
Defined environment variables file with a  new MySQL connection set as
AIRFLOW_CONN_MYSQL_CONF=mysql://xxx:xxx@xxx:3306/db_xxxx
Docker-compose file was modified and added env_file section to the webserver container:
env_file:
- ~/local_stage.env
Everything goes up fine, the new env variable is avilable within the webserver container
but for some reason airflow didno't pick it up and set a new connection called mysql_conf
Airflow doc on how to create new connections using env variables:
https://airflow.apache.org/howto/manage-connections.html#creating-a-connection-with-environment-variables
Tnx for your help

I am having this same problem. Any ideas on how to solve?

It seems like the export command in entrypoint.sh doesn't actually set the environment variables in the running Airflow container. Currently I'm working around it by explicitly setting the env variables with docker run -e <env_var>, but it'd be nice if we didn't have to do this manually

Same problem for me. I am trying to set the parallelism param in [core]. Any help would be appreciated.

I also have this problem. In the entrypoint.sh I am able to verify that the environment variables from the docker-compose.yml are set. But the connections I am trying to make aren't being picked up. I was able to make a ugly work around which is to call airflow connections command between the initdb and the launch of the webserver. This does create the connections. It kinda looks like this:
case "$1" in
  webserver)
    airflow initdb
    if [ "$AIRFLOW__CORE__EXECUTOR" = "LocalExecutor" ]; then
      # With the "Local" executor it should all run in one container.
      airflow scheduler &
    fi
    airflow connections -a --conn_id warehouse --conn_uri postgresql://svc_airflow:XXXXXXXXXXXXXXXX@hostname:5432/warehouse
    airflow connections -l
    exec airflow webserver
...

Well, I am back. I was doing some more research and stumbled across this: https://airflow.readthedocs.io/en/stable/concepts.html#concepts-connections
From that section:

If connections with the same conn_id are defined in both Airflow metadata database and environment variables, only the one in environment variables will be referenced by Airflow (for example, given conn_id postgres_master, Airflow will search for AIRFLOW_CONN_POSTGRES_MASTER in environment variables first and directly reference it if found, before it starts to search in metadata database).

Which seems to imply that this is not a bug and that we should not expect connections defined as an environment variable to show up in the UI? They will just be searched when the connection is called for?

Ok I am back again, I made some experiments. And basically my last hunch was correct. If you get the AIRFLOW_CONN_ env var setup correctly -- url encoded, don't wrap it in quotes (why did I do that!?) -- then it will always get used by the dag and it won't show up in your database, ever.
If you are inclined you can name the env var conns after any of the defaults that get setup in the initdb command. Or you can put some airflow connections -a --conn_id {conn_id} --conn_type {conn_type} stubs in the entrypoint that will make the connections show up in your UI but will be missing the connection details.
I think I am just going to have a bunch of ghost connections and rely on env vars for everything. Hope this helps!

The entrypoint.sh adaptation suggested my @mandrews in issue #441 seems to get around this: with that, I can create a connection via an env var and have it show in the UI with proper credentials, and have it appear in the list of connections for ad hoc queries (even if it's name doesn't match one of the pre-existing connections).


Notice: Please add break; after case 1

That segment of code relate to interrupt handling in Computer.java we must put break; at end of the case1 because this case have 2 case.
It is kind of notice for developer not an issue.

thanks for sharing this issue fix now (commit 78)


Visa backend configuration for Windows

Hi @SamSchott,
This is not an issue, but a usage noob question...
How do I go about  setting up the backend visa configuration for windows (Win10, Python 3.8)?
I am seeking to use the NI visa library already installed in my system.
Cheers,
G

Hi @ozumana,
This is a valid question and currently not well documented.
You can pass a keyword argument visa_library to the Keithley2600 constructor to specify the backend. For example:
from  keithley2600 import Keithley2600
k = Keithley2600('TCPIP0::<IP_ADDRESS>::INSTR', visa_library='path/to/library')

It defaults to using the PyVISA-py backend, selected by visa_library='@py'. But if you pass an empty string, it will use an installed IVI library (such as NI-VISA) if it can find one in standard locations and fall back to PyVISA-py otherwise.
You can find more information about selecting the backend in the PyVISA docs.
Hope this helps,
Sam

Brilliant @SamSchott ! Thanks!


Can't unload/load map at runtime

Unloading and loading a map at runtime causes an access violation.
The error can be reproduced by pressing button 1 in the current version to unload the map and then button 2 to reload it.

Apparently the problem occurs in this function:
https://github.com/ngagesdk/nrpg/blob/master/src/tiled.c#L243

Apparently this only affects the emulator. It works on real hardware!

This is no emulator bug.
The problem occurs with all maps that have more than 1 actor entity. That's a clue!


[WIP] Electrostatics joshs development

This is a combination of the electrostatics branch and joshs-development branch.

I need more info....

The trajectory and topology classes cannot have pure virtual methods. This is the error I get if I try. I think it is because you cannot call new on an abstract data type because it is not well defined, I quote
"You cannot instantiate an abstract class."
Error:
In file included from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/../fileformatfactory.h:25:0,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/../topologyreader.h:24,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/groreader.h:22,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/topologyreader.cc:23:
/usr/local/include/votca/tools/objectfactory.h: In instantiation of ‘parent* votca::tools::create_policy_new() [with parent = votca::csg::TopologyReader; T = votca::csg::XYZReadervotca::csg::CSG_Topology]’:
/usr/local/include/votca/tools/objectfactory.h:101:11:   required from ‘void votca::tools::ObjectFactory<key_t, T>::Register(const key_t&) [with obj_t = votca::csg::XYZReadervotca::csg::CSG_Topology; key_t = std::__cxx11::basic_string; T = votca::csg::TopologyReader]’
/home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/topologyreader.cc:40:61:   required from here
/usr/local/include/votca/tools/objectfactory.h:89:10: error: invalid new-expression of abstract class type ‘votca::csg::XYZReadervotca::csg::CSG_Topology’
return new T();
^~~~~~~
In file included from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/topologyreader.cc:27:0:
/home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/xyzreader.h:45:7: note:   because the following virtual functions are pure within ‘votca::csg::XYZReadervotca::csg::CSG_Topology’:
class XYZReader : public TrajectoryReader, public TopologyReader {
^~~~~~~~~
In file included from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/groreader.h:22:0,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/topologyreader.cc:23:
/home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/../topologyreader.h:36:16: note: 	virtual bool votca::csg::TopologyReader::ReadTopology(std::__cxx11::string, boost::any)
virtual bool ReadTopology(std::string file, boost::any top) = 0;
^~~~~~~~~~~~
In file included from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/../fileformatfactory.h:25:0,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/../trajectoryreader.h:22,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/dlpolytrajectoryreader.h:34,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/trajectoryreader.cc:23:
/usr/local/include/votca/tools/objectfactory.h: In instantiation of ‘parent* votca::tools::create_policy_new() [with parent = votca::csg::TrajectoryReader; T = votca::csg::XYZReadervotca::csg::CSG_Topology]’:
/usr/local/include/votca/tools/objectfactory.h:101:11:   required from ‘void votca::tools::ObjectFactory<key_t, T>::Register(const key_t&) [with obj_t = votca::csg::XYZReadervotca::csg::CSG_Topology; key_t = std::__cxx11::basic_string; T = votca::csg::TrajectoryReader]’
/home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/trajectoryreader.cc:47:61:   required from here
/usr/local/include/votca/tools/objectfactory.h:89:10: error: invalid new-expression of abstract class type ‘votca::csg::XYZReadervotca::csg::CSG_Topology’
return new T();
^~~~~~~
In file included from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/trajectoryreader.cc:29:0:
/home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/xyzreader.h:45:7: note:   because the following virtual functions are pure within ‘votca::csg::XYZReadervotca::csg::CSG_Topology’:
class XYZReader : public TrajectoryReader, public TopologyReader {
^~~~~~~~~
In file included from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/groreader.h:22:0,
from /home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/trajectoryreader.cc:25:
/home/jbrown/Documents/Software/VOTCA/csg/src/libcsg/../../include/votca/csg/io/../topologyreader.h:36:16: note: 	virtual bool votca::csg::TopologyReader::ReadTopology(std::__cxx11::string, boost::any)
virtual bool ReadTopology(std::string file, boost::any top) = 0;
^~~~~~~~~~~~

Still need tests for topolgy type container, dlpoly file readers and writers.

ping @JoshuaSBrown

This pull request was kept open for all the review comments which are addressed in https://github.com/votca/csg/pull/381 I believe they have been addressed now closing.


Fix errors by linkchecker.py in tasks folder

This PR fixes "Should use relative paths errors" which are shown by scripts/linkchecker.py on en/docs/tasks/*.

/assign @zparnold
Could you review this?

/lgtm
/approve

@tengqm
Thank you for quick approving. :smile:


Add ORDER BY time when fetching metrics

This is more of a discussion on how to downsample/rollup metrics that are older than retention period.  I'm a noob when it comes to timescaledb so this may be an extremely naive way to do downsampling so if any experts out there has a better way to do it, I'm all ears :)
That said, the way we do downsampling is as such:

Run aggregate function (e.g., AVG) over a time period for which raw metrics need to be aggregated into buckets, and save the result to a temp table.  e.g.,

SELECT time_bucket(interval '30 minutes', time) AS tb, labels_id, AVG(value) AS value
INTO tmp_table
FROM metrics_values
GROUP BY tb, labels_id
HAVING tb > %(start_time)s AND tb < %(end_time)s

Delete raw metrics in the metrics_values table within the time range:

DELETE FROM metrics_values WHERE time > %(start_time)s AND time <%(end_time)s

INSERT downsampled metrics back to the metrics_values table:

INSERT INTO metrics_values (time, labels_id, value) SELECT * FROM tmp_table

Drop the temp table

What comes out of this process is that the downsampled metrics will be appended at the end of the metrics_values table.  The table itself does not declare a canonical ordering on the time column so when Prometheus does a range_query, the SQL query result will return metrics_values rows out of order if there are downsampled metrics in the time range, causing Prometheus to return the wrong metrics.
This PR adds the ORDER BY time clause so the metrics_values rows are always sorted by time.
I did a quick EXPLAIN on the two queries on a database:
Without ORDER BY:
postgres=# EXPLAIN SELECT * FROM metrics;
                                      QUERY PLAN
--------------------------------------------------------------------------------------
 Hash Join  (cost=29.12..161.26 rows=1519 width=112)
   Hash Cond: (m.labels_id = l.id)
   ->  Append  (cost=0.00..109.18 rows=1519 width=20)
         ->  Seq Scan on metrics_values m  (cost=0.00..0.00 rows=1 width=20)
         ->  Seq Scan on _hyper_1_9_chunk m_1  (cost=0.00..47.04 rows=4 width=20)
         ->  Seq Scan on _hyper_1_10_chunk m_2  (cost=0.00..62.14 rows=1514 width=20)
   ->  Hash  (cost=18.50..18.50 rows=850 width=68)
         ->  Seq Scan on metrics_labels l  (cost=0.00..18.50 rows=850 width=68)
(8 rows)

With ORDER BY:
postgres=# EXPLAIN SELECT * FROM metrics ORDER BY time;
                                         QUERY PLAN
--------------------------------------------------------------------------------------------
 Sort  (cost=241.53..245.33 rows=1519 width=112)
   Sort Key: m."time"
   ->  Hash Join  (cost=29.12..161.26 rows=1519 width=112)
         Hash Cond: (m.labels_id = l.id)
         ->  Append  (cost=0.00..109.18 rows=1519 width=20)
               ->  Seq Scan on metrics_values m  (cost=0.00..0.00 rows=1 width=20)
               ->  Seq Scan on _hyper_1_9_chunk m_1  (cost=0.00..47.04 rows=4 width=20)
               ->  Seq Scan on _hyper_1_10_chunk m_2  (cost=0.00..62.14 rows=1514 width=20)
         ->  Hash  (cost=18.50..18.50 rows=850 width=68)
               ->  Seq Scan on metrics_labels l  (cost=0.00..18.50 rows=850 width=68)
(10 rows)

It appears that they both inspect roughly the same amount of rows, so the performance impact should be minimal, right?
Again, I'm new to timescaledb and I'm not sure if that's the best way to do downsampling.  Happy to hear any suggestions!

 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.

Hi @kevinjqiu,
thanks for the PR. It addresses the problem correctly. I left some nits to look into.

@kevinjqiu Looks good! Thanks!


[#5214] Improvement(catalog-lakehouse-paimon): add jdbc-driver to Paimon JDBC catalog backend configuration

What changes were proposed in this pull request?
add jdbc-driver to Paimon JDBC catalog backend configuration
Why are the changes needed?
Fix: https://github.com/apache/gravitino/issues/5214
Does this PR introduce any user-facing change?
updated the related doc
How was this patch tested?
New IT and modified UT.

LGTM, @caican00 could you fix the ci failure?

LGTM, @caican00 could you fix the ci failure?

@FANNG1 fixed.

@caican00 , I forgot to tag branch-0.7, could you propose a new PR based on branch-0.7 manually?

@caican00 , I forgot to tag branch-0.7, could you propose a new PR based on branch-0.7 manually?

sure

@caican00 , I forgot to tag branch-0.7, could you propose a new PR based on branch-0.7 manually?

@FANNG1 done


Rust client library and plugable databroker cli

This PR addresses the need of a client library for rust for the new API (kuksa.val.v1) and makes the databroker-cli plugable. TODO: Documentation update.
To try this PR you need to know that per default it will be built with the old API client lib. If you want to use the new one use ```
cargo run --bin databroker-cli --features feature_kuksa --no-default-features

How does Rust compiler handle features? Does it ignore features not included (like old fashioned #ifdef) or does it anyway compile them even if it does not include them in result.
If it ignore it totally - would it then be an idea to extend our CI so that it also builds the "feature_kuksa" variant, to make sure that we do not introduce any regressions that can be detected at compile time, i.e. that all code is compiled by CI.

How does Rust compiler handle features? Does it ignore features not included (like old fashioned #ifdef) or does it anyway compile them even if it does not include them in result.
If it ignore it totally - would it then be an idea to extend our CI so that it also builds the "feature_kuksa" variant, to make sure that we do not introduce any regressions that can be detected at compile time, i.e. that all code is compiled by CI.

Yes that needs to be added. Good point!

cargo clippy --all-targets -- -W warnings -D warnings

would this be enough or would you actually fully build it


send a serializable dict in py3

Not sure if runallserver should work already, but at least one needs to decode some bytes in PY3 before JSON serialization.

It's not finished yet, and that bit is going to be changed around, so I'll close this and fix it separately.


Update socials.mdx

Fixed Parsing error: ',' expected when copying the code.

Hey @AtharvPorate1, thanks for making this PR. I took the liberty to also fix other missing trailing commas within this doc page. I'll approve and merge this shortly!


Add vectorized plotting functionality to improve performance

Overview: What does this pull request change?

added use_vectorized: bool parameter to coords_to_points in CoordinateSystem and Axes

added vectorized logic to coords_to_points in Axes

added use_vectorized: bool parameter to plot in CoordinateSystem

added use_vectorized: bool parameter to ParametricFunction

added vectorized logic to generate_points in ParametricFunction depending on self.use_vectorized

added use_vectorized: bool parameter to plot_antiderivative_graph

added vectorized logic to plot_antiderivative_graph

added number_to_point_array to NumberLine

vectorized inverse_function in scale.py (might be able to add a second version just for this case?)

vectorized angle_of_vector in space_ops.py

changed naming of test data to support fixture parameters because ":" is not allowed in a file name

added use_vectorized pytest fixture

added use_vectorized fixture to

test_axes::test_plot_functions
test_axes::test_plot_derivative_graph
test_axes::test_plot
test_axes::test_get_riemann_rectangles

added test test_coordinate_systems::test_plot_log_x_axis_vectorized

added test test_space_ops::test_angle_of_vector_vectorized

Motivation and Explanation: Why and how do your changes improve the library?

Manim does a ton of function calls internally and reducing them step by step might be a good idea. At some point we could make a breaking change and vectorize functions by default to remove branching code versions. The performance increase is not that much but a step in the right direction. I think this vectorized version could also improve VectorField or other Classes that use coords_to_points 🤔

Before
After

 coords_to_point (yellow) on the right taking 1/3 of generate_points
 coords_to_point on the left taking a lot less

Data collected via vprof and performance difference is from 42 to 36 seconds.
To view the profile_data.zip use
vprof -i vectorized.json
vprof -i not_vectorized.json

Scene used for testing

from manim import *
class Bench(Scene):
    def construct(self):
        ax1 = NumberPlane((-3, 3), (-4, 4))
        track = ValueTracker(0)
        def func(t):
            out =  np.array([np.sin(2 * (t+track.get_value())), np.sin(3 * (t+track.get_value())), 0])
            return out

        vectorized = False
        graph = ax1.plot_parametric_curve(func, color=RED, t_range=np.array([0, 1]), use_vectorized=vectorized)
        graph2 = ax1.plot(lambda x: x**2, use_vectorized=vectorized)
        graph3 = ax1.plot_derivative_graph(graph2, use_vectorized=vectorized)

        graph.add_updater(lambda m: m.become(ax1.plot_parametric_curve(func, color=RED, t_range=np.array([0, 1]), use_vectorized=vectorized)))
        graph2.add_updater(lambda m: m.become(ax1.plot(lambda x: (x**3)*track.get_value()-x/2, use_vectorized=vectorized)))
        graph3.add_updater(lambda m: m.become(ax1.plot_derivative_graph(graph2, use_vectorized=vectorized)))
        rects = ax1.get_riemann_rectangles(graph)
        rects.add_updater(lambda m: m.become(ax1.get_riemann_rectangles(graph)))

        self.add(ax1, graph, graph2, graph3, rects)
        self.play(track.animate.set_value(TAU),run_time=10)

Links to added or changed documentation pages

Further Information and Comments

This is a extended implementation of

#2513

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Maybe it would be a good idea to add a use_vectorized option in the config dictionary

Maybe it would be a good idea to add a use_vectorized option in the config dictionary. This would remove a lot of the unnecessary parameter passing and would make extending this to other functions easier!

I'm not sure about this. The respective classes or objects should hold this sort of behavioral information directly, the global config should only do this for classes that the user should normally not access (like the renderer, the camera, the file writer, ...). And what if you'd want to use vectorized functions in some parts, but not in others?
I'm open to further discussion, but there has to be a very clear and thought through proposal if we want to head in this direction. :-)

And what if you'd want to use vectorized functions in some parts, but not in others?
This is not really possible anyway, beacuse then the function all need to contain logic to decide what their input is and if they need to translate an input array to to elements and then go through a for loop and apply itself. The idea behind that is to slowly optimize the code behind the scenes and give the option to use it but not result in breaking changes so that some function are not working correctly anymore.

The only thing that would still be useful is on the high level end of the plot functions to decide manually if someone would like to have f(x) or f([x_0,x_1,...])
One option would also be to completely vectorize the low level functions in any way or add a vectorized version for all of them. But this results in a lot of code doubling. And when the goal is to optimize in the long term it seems rather counter productive.


Hook for setting initial session state

It would be super cool to have a hook in handleSession for setting the initial session state:
const generateInitialState = () => {
  return {
    something: 'default'
  }
}

export const handle = handleSession({
  init: generateInitialState
});

This would enable us to make sure the shape of the object we're storing in sessiondata is always what we expect (and not "undefined")

Hey you can now set the initial session state with version 3.3.0
export const handle = handleSession({
	init: (event) => ({
		views: 0
	})
});

❤


doc/go1.17: document archive/zip changes for Go 1.17

For #44513. Fixes #46000

@googlebot I signed it!


iOS supports requestLEScan without service filter, but the plugin prevents it

iOS supports the scanning of BLE devices without specifying a service list. This requires
the service list to be nil:
https://developer.apple.com/documentation/corebluetooth/cbcentralmanager/1518986-scanforperipherals#discussion
However, on this line:
https://github.com/capacitor-community/bluetooth-le/blob/main/ios/Plugin/Plugin.swift#L509
an empty array is used when no services are provided in the call, preventing any possibility to get nil.
Since an empty array would not do anything, I propose to return nil in the aforementioned getServiceUUIDs method. ~If developers still want the scanning of an empty array for some reason, they can just pass it along.~
EDIT: After reading up on it, there does not seem to be a valid reason to 'scan' for an empty list, as this would not start a scan

EDIT: After reading up on it, there does not seem to be a valid reason to 'scan' for an empty list, as this would not start a scan

Where did you read this? In my tests, whether services is nil or an empty array doesn't matter and I obtain the same scan results.


Parallel support

It seems like there are consistent InvalidArchiveError's when attempting to run tox-conda in parallel on Jenkins similar to the way tox advises it to be set up. As shown here: https://tox.readthedocs.io/en/latest/example/jenkins.html#running-tox-environments-in-parallel
It looks like conda install is not thread-safe. https://stackoverflow.com/a/58228286
Are there any workarounds for this? Is this a conda problem? Or maybe just user error?

You might be able to get around this by installing the dependencies before running the tests in parallel. e.g.,
tox --notest
tox -p auto

Looks like this workaround works. Thanks!


Unable to make public long com.sun.management.internal.OperatingSystemImpl.getTotalPhysicalMemorySize() accessible: module jdk.management does not "opens com.sun.management.internal" to unnamed module

OrientDB Version:
Community 3.0.2
Java Version:
openjdk version "10.0.1" 2018-04-17
OpenJDK Runtime Environment (build 10.0.1+10)
OpenJDK 64-Bit Server VM (build 10.0.1+10, mixed mode)
OS:
Ubuntu 16.04 x64
Expected behavior
The server would be started and a prompt for a password would appear
Actual behavior
2018-06-24 16:59:56:333 INFO  Loading configuration from: /opt/orientdb/config/orientdb-server-config.xml... [OServerConfigurationLoaderXml]WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.sun.xml.bind.v2.runtime.reflect.opt.Injector$1 (file:/opt/orientdb/lib/jaxb-impl-2.2.3-1.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int)
WARNING: Please consider reporting this to the maintainers of com.sun.xml.bind.v2.runtime.reflect.opt.Injector$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

2018-06-24 16:59:57:747 INFO  OrientDB Server v2.2.22 (build fb2b7d321ea8a5a5b18a82237049804aace9e3de) is starting up... [OServer]
2018-06-24 16:59:57:763 INFO  Databases directory: /opt/orientdb/databases [OServer]Error during initialization of engine 'plocal', engine will be removed
java.lang.reflect.InaccessibleObjectException: Unable to make public long com.sun.management.internal.OperatingSystemImpl.getTotalPhysicalMemorySize() accessible: module jdk.management does not "opens com.sun.management.internal" to unnamed module @7fe3240a
	at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:337)
	at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:281)
	at java.base/java.lang.reflect.Method.checkCanSetAccessible(Method.java:198)
	at java.base/java.lang.reflect.Method.setAccessible(Method.java:192)
	at com.orientechnologies.common.util.OMemory.getPhysicalMemorySize(OMemory.java:66)
	at com.orientechnologies.orient.core.engine.OMemoryAndLocalPaginatedEnginesInitializer.configureDefaultDiskCacheSize(OMemoryAndLocalPaginatedEnginesInitializer.java:80)
	at com.orientechnologies.orient.core.engine.OMemoryAndLocalPaginatedEnginesInitializer.configureDefaults(OMemoryAndLocalPaginatedEnginesInitializer.java:63)
	at com.orientechnologies.orient.core.engine.OMemoryAndLocalPaginatedEnginesInitializer.initialize(OMemoryAndLocalPaginatedEnginesInitializer.java:52)
	at com.orientechnologies.orient.core.engine.local.OEngineLocalPaginated.startup(OEngineLocalPaginated.java:56)
	at com.orientechnologies.orient.core.Orient.startEngine(Orient.java:881)
	at com.orientechnologies.orient.core.Orient.loadStorage(Orient.java:474)
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:172)
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:153)
	at com.orientechnologies.orient.server.OSystemDatabase.init(OSystemDatabase.java:151)
	at com.orientechnologies.orient.server.OSystemDatabase.<init>(OSystemDatabase.java:44)
	at com.orientechnologies.orient.server.OServer.initSystemDatabase(OServer.java:1283)
	at com.orientechnologies.orient.server.OServer.activate(OServer.java:350)
	at com.orientechnologies.orient.server.OServerMain$1.run(OServerMain.java:46)
Error during engine shutdown
java.lang.NullPointerException
	at com.orientechnologies.orient.core.engine.local.OEngineLocalPaginated.shutdown(OEngineLocalPaginated.java:115)
	at com.orientechnologies.orient.core.Orient.startEngine(Orient.java:887)
	at com.orientechnologies.orient.core.Orient.loadStorage(Orient.java:474)
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:172)
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:153)
	at com.orientechnologies.orient.server.OSystemDatabase.init(OSystemDatabase.java:151)
	at com.orientechnologies.orient.server.OSystemDatabase.<init>(OSystemDatabase.java:44)
	at com.orientechnologies.orient.server.OServer.initSystemDatabase(OServer.java:1283)
	at com.orientechnologies.orient.server.OServer.activate(OServer.java:350)
	at com.orientechnologies.orient.server.OServerMain$1.run(OServerMain.java:46)
com.orientechnologies.orient.core.exception.ODatabaseException: Error on opening database 'plocal:/opt/orientdb/databases/OSystem'
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:192)
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:153)
	at com.orientechnologies.orient.server.OSystemDatabase.init(OSystemDatabase.java:151)
	at com.orientechnologies.orient.server.OSystemDatabase.<init>(OSystemDatabase.java:44)
	at com.orientechnologies.orient.server.OServer.initSystemDatabase(OServer.java:1283)
	at com.orientechnologies.orient.server.OServer.activate(OServer.java:350)
	at com.orientechnologies.orient.server.OServerMain$1.run(OServerMain.java:46)
Caused by: com.orientechnologies.orient.core.exception.OConfigurationException: Error on opening database: the engine 'plocal' was unable to start. URL was: plocal:/opt/orientdb/databases/OSystem. Registered engines was: [memory, remote, plocal]
	DB name="plocal:/opt/orientdb/databases/OSystem"
	at com.orientechnologies.orient.core.Orient.loadStorage(Orient.java:475)
	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.<init>(ODatabaseDocumentTx.java:172)
	... 6 more

Steps to reproduce
Download the community edition .gz
Extract it
mv the extracted directory to /opt/orientdb
cd there
try to sudo bin/server.sh

Hi @StrangeTcy
From the logs I see it's OrientDB v 2.2.22, please upgrade to 2.2.36, it should be already fixed
Thanks
Luigi

Sorry, I'm confused.

I download, extract and copy an archive named orientdb 3.0.2 (that's the actual link I use)
I try to sudo bin/server.sh there
I get errors that, among other things, indicate that the actual version of OrientDB is 2.2.22 and not 3.0.2 as I would (reasonably, I think) expect.

Anyway, try to do the same with OrientDB 2.2.36 didn't resolve the issue, unless I've done something wrong.

Update: a version that was provided in an archive that's pointed to by the "for any OS" link contains the 3.0.2 version, and it does work.
Still, I think that's confusing.

Hi @StrangeTcy
Oh, got it, thank you very much for pointing it out, I'll fix the website
Thanks
Luigi

Hi @StrangeTcy
I checked the website and everything seems to be OK.
So just to recollect the information, if you have a chance to use v 3.0.x I'd strongly suggest you to do so. For v 2.2.x we will do a check, probably there is an incompatibility with Java 10
Thanks
Luigi

Hi @StrangeTcy
I just checked the source code and saw that OMemory.getPhysicalMemorySize() no longer exists in v 2.2.36, so it's unlikely that the error is the same.
Could you please post the error you obtain with latest version?
Thanks
Luigi


Support displaying X-axis of graph in local timezone

As a prometheus user who live in Japan(UTC+0900), displaying X-axis timestamp of a graph in local timezone is very desired feature.
As I read the FAQ entry(https://prometheus.io/docs/introduction/faq/#can-i-change-the-timezone?-why-is-everything-in-utc?) about timezone on prometheus, I understand that the prometheus want to respect UTC as much as possible, so I added an option to switch the display mode of X-axis as a server option.
I also think about introducing "localtime" button on the graph UI but realized something like following which makes me feel it don't mean much.

Depends on organizations standard but for most cases people who see the same prometheus server wants to rely on the same timezone on the view. At least all people around me says they want to see the graph with local timezone.
It's really cumbersome to click timezone button every time opening a graph UI or adding a graph.

Any feedbacks are welcome :)
Just for reference, here's the picture of how each display option works:
As-is(built from 82a961):

Option disabled(resulting same view as before):

Option enabled:

This change is

Thanks for your attempt to tackle this nasty issue.
In general, I'd like it more to have a switch in the WebUI itself rather than a command line flag.
Do you know how the display will look like during a "daylight saving time" switch-over?

I also think it would be preferential on the web ui side. I work with
people in other timezones within the US, but also people around the world.
We also have datacenters around the world. Setting this on the server would
not be ideal for what I believe is the common pattern. However, a change in
the web ui would be great!
On May 6, 2016 3:39 AM, "Björn Rabenstein"<EMAIL_ADDRESS>wrote:

Thanks for your attempt to tackle this nasty issue.
In general, I'd like it more to have a switch in the WebUI itself rather
than a command line flag.
Do you know how the display will look like during a "daylight saving time"
switch-over?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/prometheus/prometheus/pull/1616#issuecomment-217384101

See #500
This should be a checkbox in the UI IMHO

I share the same concerns. The problem of persistence of that UI choice can be tackled with cookies and/or local storage.

my fork,  https://github.com/andelf/prometheus , make local time works by modifying ui.

This review seems stale, if you make the requested changes please let us know.


MGMT-9332: Add endpoint for rootfs and kernel

Description

This PR adds a new endpoint to the image service
which will serve the rootfs or the kernel directly
from an ISO.
The new endpoint is of the form:
/boot-artifacts/{artifact-name}?{params}

For example:
/boot-artifacts/rootfs?version=4.9&arch=x86_64
/boot-artifacts/kernel?version=4.10&arch=arm64

How was this code tested?

Ran the image service locally and downloaded the rootfs and kernel using the new endpoints.
Ensured that they matched exactly the same files extracted from the iso using the iso-read command line tool.
Assignees

/cc @mhrivnak
/cc @danielerez
Links

Enhancement doc: https://github.com/openshift/assisted-service/pull/3389
JIRA: https://issues.redhat.com/browse/MGMT-9332
Checklist

[x] Title and description added to both, commit and PR
[x] Relevant issues have been associated
[x] Reviewers have been listed
[ ] This change does not require a documentation update (docstring, docs, README, etc)
[x] Does this change include unit tests (note that code changes require unit tests)

Can someone else review this since @CrystalChun and I effectively paired on it and @mhrivnak is out?
Maybe @danielerez, @avishayt or @filanov (or maybe one of you could suggest someone that could look it over?)

@romfreiman 's question about metrics also reminded me of the issues we had with CORS.
We don't expect anyone to be downloading these files directly from the UI, right?

I assume we gonna present the script in he ui at some point, won't we? And allow to download it.
About rootfs/kernel, why not? Similar to what we do with the iso

I assume we gonna present the script in he ui at some point, won't we? And allow to download it.

The script is going to be served by assisted-service, so CORS is already handled there for all endpoints I think.

About rootfs/kernel, why not?

:man_shrugging: I'll add CORS headers for this also then.

Marked a WIP to work on metrics and CORS.

/assign @avishayt

/lgtm


Update okhttp to 4.12

Re: androidx.media3:media3-datasource-okhttp
Currently, okhttp 4.11 is being used. However, okhttp 4.11 includes a version of okio that contains a vulnerability (CVE-2023-3635). This was addressed in okhttp 4.12 (which includes a newer okio that has fixed the vulnerability).
Could okhttp be updated to 4.12 for the next release?

Thank you for reporting, we will update. In the meantime there's no need to wait for a new media3 release, you can update the version used when building your app using Gradle constraints: https://docs.gradle.org/current/userguide/dependency_constraints.html


JS: Unreachable overloads

Depends on #1953
A query for detecting unreachable overloads in TypeScript.
E.g:
interface Foobar {
    maybeConvertToNumber(a: string): number;
    maybeConvertToNumber(a: string): string; // <- can never be used.
}

var foo: Foobar = null as any;

var a : string = foo.maybeConvertToNumber("foo"); // <- compile error

This version is based on the new API from @asger-semmle (#1953)
Performance is good. There were a few bad outliers, but those got fixed by introducing MethodCallSig.

In creating a pull-request based on this query I found some false positives.
https://github.com/Semmle/ql/pull/1988/commits/0709f7d50b9189f8cb1828c69dacf86a0c2f1151 fixes it, altough I don't like how the fix looks, but that was how I could get super-types to work for now (@asger-semmle ?)

I'll review this shortly

@erik-krogh - thanks for addressing my comments, and for adding a reference.
I have a further comment but it is minor, so I leave it up to you. whether you think it's worth addressing. I'd replace In the below example (which reads a bit too formally to me) with In the example below, (which moves the word below, and adds a comma).


Add a 6th Ed Call of Cthulhu character sheet (French)

Hi,
I made a character sheet for the 6th French edition of Call of Cthulhu. Unfortunately I do not know how to make a preview of this sheet to include in the PR.
Also, I used in the sheet a logo from the French Editor website (to make the title) and I'm not 100% sure that it's ok for the copyright. Let me know what I should do about that.
Thanks,

To create a preview, just use a cropped screenshot of the sheet. As for using the official logo, that's definitely something we can't do so I'm going to close this and you can open a new Pull Request with a preview and without the official images. Also, as far as images go, you can upload them to github as well and then you can reference their raw link for image in the sheet. This makes sure that a 3rd party site doesn't go and remove the image eventually.

Thanks for your answer. I remove the official images.
About the preview I really don't see how to do it. I found something in the
wiki (https://wiki.roll20.net/Building_Character_Sheets) that suggests a
Preview Pane exists, but this option seem to be only accessible for
Pro-suscribers - which I'm not. With a little trick I managed to see what
the sheet should look like but I can't view it as it will eventually be (it
lacks the buttons "add/modify" that will be added by roll20). For this time
I will include the best first preview that I can, and update it as soon as
the sheet is integrated in roll20, unless you suggest otherwise.
2016-02-29 23:01 GMT+01:00 Phil<EMAIL_ADDRESS>
To create a preview, just use a cropped screenshot of the sheet. As for
using the official logo, that's definitely something we can't do so I'm
going to close this and you can open a new Pull Request with a preview and
without the official images. Also, as far as images go, you can upload them
to github as well and then you can reference their raw link for image in
the sheet. This makes sure that a 3rd party site doesn't go and remove the
image eventually.
—
Reply to this email directly or view it on GitHub
https://github.com/Roll20/roll20-character-sheets/pull/1424#issuecomment-190416913
.

Since you do not have a Pro account, you can just ignore the preview image part for now. I will create one for you.

Ok thanks, I create another pull request for these files as this one is closed now.


aws-sam-cli: 1.14.0 -> 1.23.0

Motivation for this change
Version bump.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120896 at b1abe322 run on aarch64-linux 1

1 package built successfully:

aws-sam-cli

2 suggestions:

warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'python-dateutil~=2.6, <2.8.1' doesn't match anything in file 'requirements/base.txt'
Near pkgs/development/tools/aws-sam-cli/default.nix:53:0:
   |
53 |     description = "CLI tool for local development and testing of Serverless applications";
   | ^

warning: stale-substitute
Stale substituteInPlace detected.
substituteStream(): WARNING: pattern 'boto3~=1.14.23' doesn't match anything in file 'requirements/base.txt'
Near pkgs/development/tools/aws-sam-cli/default.nix:53:0:
   |
53 |     description = "CLI tool for local development and testing of Serverless applications";
   | ^


Fix code scanning alert - Insecure randomness

Tracking issue for:

[x] https://github.com/RicardoGEsteves/s3mer/security/code-scanning/1

Fixed!!

done


Add feature to call the log writer

I did not really test all switches, but I only tested the main ones. I think this should work.


Yeah, looks good.

Yeah I think so too.

Done https://github.com/EmmittJ/PoESkillTree/releases/tag/2.1.2


⚠ Bofrost.fr has degraded performance

In d55fdc0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1537 ms

Resolved: Bofrost.fr performance has improved in 7407fa5 after 30 minutes.


AttributeError: 'Orbit' object has no attribute 'frame'

>>> from poliastro import *
>>> r = [-6045, -3490, 2500] * u.km
>>> v = [-3.457, 6.618, 2.533] * u.km / u.s
>>> ss = Orbit.from_vectors(Earth, r, v)
>>> ss
7283 x 10293 km x 153.2 deg (GCRS) orbit around Earth (♁) at epoch J2000.000 (TT)
>>> ss.frame
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'Orbit' object has no attribute 'frame'

Hi @iamabhishek0, I tried this with master and everything worked:
>>> from astropy import units as u
>>> r = [-6045, -3490, 2500] * u.km
>>> v = [-3.457, 6.618, 2.533] * u.km / u.s
>>> from poliastro.twobody import Orbit
>>> from poliastro.bodies import Earth
>>> ss = Orbit.from_vectors(Earth, r, v)
>>> ss.frame
<GCRS Frame (obstime=J2000.000, obsgeoloc=(0., 0., 0.) m, obsgeovel=(0., 0., 0.) m / s)>

could you please tell us what poliastro version are you using?

I am using the current version which is on the master.

I am not at all sure. Are you using Linux? Could you please paste the output of
$ which python
$ python -m pip list | grep poliastro
$ python -c "from poliastro.examples import iss; print(iss.frame)"

?

Yes, I am using Linux.
(polienv) [uchiha@pop-os src{master}]$ which python
/home/uchiha/Desktop/poliastro/polienv/bin/python
(polienv) [uchiha@pop-os src{master}]$ python -m pip list | grep poliastro
(polienv) [uchiha@pop-os src{master}]$ python -c "from poliastro.examples import iss; print(iss.frame)"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
AttributeError: 'Orbit' object has no attribute 'frame'

Well, python -m pip list doesn't find it, so I wonder where is it finding it.
Can you do this?
$ python -c "import poliastro as p; print(p.__file__)"
$ python -c "import poliastro as p; print(p.__version__)"

(polienv) [uchiha@pop-os src{master}]$ python -c "import poliastro as p; print(p.__file__)"
/home/uchiha/Desktop/poliastro/src/poliastro/__init__.py
(polienv) [uchiha@pop-os src{master}]$ python -c "import poliastro as p; print(p.__version__)"
0.14.post0.dev0

Okay, so you're running the code inside src/poliastro. You won't need to do that if you flit install --symlink where pyproject.toml is, but let's not get there yet.
Last thing: do git status, since unfortunately the __version__ does not record the commit...

(polienv) [uchiha@pop-os src{master}]$ git status
On branch master
Your branch is up to date with 'origin/master'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)

        ../polienv/

nothing added to commit but untracked files present (use "git add" to track)

Sorry, I forgot the most important one :man_facepalming: git show

(polienv) [uchiha@pop-os src{master}]$ git show
commit 9994fd6a8e939ac74e0385232b689c46aa59a906 (HEAD -> master, origin/master, origin/HEAD)
Merge: 77b2f9f 5224680
Author: Juan Luis Cano Rodríguez<EMAIL_ADDRESS>Date:   Tue Mar 3 17:01:49 2020 +0100

    Merge pull request #853 from iamabhishek0/fixes_#852

    Fixed Broken benchmarks Link

Actually when I was setting up my virtual environment I did pip install .
But then I had to work on python 3.7.4 which was giving segmentation fault because of numba's bug.
So I created virtual environment using python3 -m virtualenv polienv and installed all the dependencies manually after which everything was working fine.
Sorry for installing the dependencies this way because I wasn't aware of flit install --symlink, I will give it a try today.

All right, sorry for the loooooooooong delay. I reproduced the "issue", now let me remember if it's actually a bug or a feature.

This change was introduced in https://github.com/poliastro/poliastro/pull/827 and now the equivalent thing is to do ss.get_frame().
If you saw ss.frame in any place of the documentation then this is a bug, otherwise let's close it.

@Juanlu001 Thank for your support, We could close this one after fixing the documentation.


Tx history logic

Codecov Report

Merging #1229 (e6848e4) into master (240b556) will increase coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head e6848e4 differs from pull request most recent head 877b100.  Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Social Icons

Hi,
with Casper v.5.0.0 the option to add social icons to header has dissapeared?
It's very useful....
Can it be implemented again?
Thanks.

Hey @viriatusX, thanks for the feedback! Social icons are now back in 5.1.0.


PlatformNotSupportedException - RedisCache .NET Core / macOS

While testing out RedisCache on macOS I am getting a PlatformNotSupportedException (full details below) whenever I try to get or set a value. Is this a known issue and/or am I doing something wrong? Note that I also tried this in an ASP.NET Core application using the IDistributedCache service from the services IoC. Same result.
OS: macOS Sierra Version 10.12.3
Redis 3.2.8
Microsoft.Extensions.Caching.Redis: 1.1.0 which uses StackExchange.Redis.StrongName: 1.1.605
Example
using System;
using System.Text;
using Microsoft.Extensions.Caching.Distributed;
using Microsoft.Extensions.Caching.Redis;

namespace ConsoleApplication
{
    public class Program
    {
        public static void Main(string[] args)
        {
            var cache = new RedisCache(new RedisCacheOptions
            {
                Configuration = "localhost",
                InstanceName = "SampleInstance"
            });

            var key = "foo";
            cache.Set(key, Encoding.UTF8.GetBytes("bar"), new DistributedCacheEntryOptions());

            var result = cache.GetString(key);
            Console.WriteLine("$Cached value: {result}");
        }
    }
}

Stack trace
Unhandled Exception: System.AggregateException: One or more errors occurred. (This platform does not support connecting sockets to DNS endpoints via the instance Connect and ConnectAsync methods, due to the potential for a host name to map to multiple IP addresses and sockets becoming invalid for use after a failed connect attempt. Use the static ConnectAsync method, or provide to the instance methods the specific IPAddress desired.)
---> System.PlatformNotSupportedException: This platform does not support connecting sockets to DNS endpoints via the instance Connect and ConnectAsync methods, due to the potential for a host name to map to multiple IP addresses and sockets becoming invalid for use after a failed connect attempt. Use the static ConnectAsync method, or provide to the instance methods the specific IPAddress desired.
   at System.Net.Sockets.Socket.ThrowIfNotSupportsMultipleConnectAttempts()
   at System.Net.Sockets.Socket.BeginConnect(String host, Int32 port, AsyncCallback requestCallback, Object state)
   at System.Net.Sockets.SocketTaskExtensions.<>c.<ConnectAsync>b__5_0(String targetHost, Int32 targetPort, AsyncCallback callback, Object state)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncImpl[TArg1,TArg2](Func`5 beginMethod, Func`2 endFunction, Action`1 endAction, TArg1 arg1, TArg2 arg2, Object state, TaskCreationOptions creationOptions)
   at System.Threading.Tasks.TaskFactory.FromAsync[TArg1,TArg2](Func`5 beginMethod, Action`1 endMethod, TArg1 arg1, TArg2 arg2, Object state)
   at System.Net.Sockets.SocketTaskExtensions.ConnectAsync(Socket socket, String host, Int32 port)
   at StackExchange.Redis.SocketManager.BeginConnect(EndPoint endpoint, ISocketCallback callback, ConnectionMultiplexer multiplexer, TextWriter log)
   at StackExchange.Redis.PhysicalConnection.BeginConnect(TextWriter log)
   at StackExchange.Redis.PhysicalBridge.GetConnection(TextWriter log)
   at StackExchange.Redis.ServerEndPoint..ctor(ConnectionMultiplexer multiplexer, EndPoint endpoint, TextWriter log)
   at StackExchange.Redis.ConnectionMultiplexer.<ReconfigureAsync>d__119.MoveNext()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at StackExchange.Redis.ConnectionMultiplexer.ConnectImpl(Func`1 multiplexerFactory, TextWriter log)
   at Microsoft.Extensions.Caching.Redis.RedisCache.Connect()
   at Microsoft.Extensions.Caching.Redis.RedisCache.Set(String key, Byte[] value, DistributedCacheEntryOptions options)

there are already many issues open for this on the StackExchange.Redis and dotnet coreclr repos.
You have to use the IP Address for now. (See error message for explanation why).


SOLR-11646: document v2 api (WIP)

Description
Continuing to add V2 api documentation.
Solution
Running bin/solr -c -e techproducts and then testing the commands.  Using https://docs.google.com/spreadsheets/d/1d3scMpjxSt5HAURkDHmMhQ7FrFuV0umyJyBT0lV_HX0/edit#gid=654750626 as a reference.
Tests
This is all docs.
Checklist
Please review the following and check all that apply:

[X ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ X] I have created a Jira issue and added the issue ID to my pull request title.
[ X] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ X] I have developed this patch against the main branch.
[ X] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ X] I have added documentation for the Reference Guide

@epugh Can I do anything to help this along?  I think it was initially dependent on #40 but with that merged I wasn't sure if this was ready to go or not...

@epugh Can I do anything to help this along? I think it was initially dependent on #40 but with that merged I wasn't sure if this was ready to go or not...

I think you did step one, a gentle nudge ;-)   It's always motivating to know someone else cares about a PR!   Will make progress today....   See if I can't get to another reasonable commit point, and then start another one.

okay, I think I am there on this...   Have I missed any other api's that are v2 equivalent?

Have I missed any other api's that are v2 equivalent?

I think you've covered all the collection-admin APIs at least.

Hey @epugh I pushed up a few fixes for my previous review comments.
In the process though I noticed something odd.  On certain API calls, the docs still have two sets of examples.  Take DELETEREPLICA in replica-management.adoc for example.  The section gives a short overview, then has some tabbed v1/v2 examples, then it has detailed description for each API parameter.  All great so far!  But then before the next API command, it gives more DELETEREPLICA examples.
In some cases (e.g. ADDREPLICA) the second set of examples are different enough from the first set that they might provide value.  But in other cases it looks like the two sets of examples are near duplicates and we just forgot to remove them.
Was this something you did thoughtfully?  Or was it just an oversight?  Happy to go through and fix myself - just wanted to make sure I wasn't undoing something you did intentionally.

Hey @epugh I pushed up a few fixes for my previous review comments.
In the process though I noticed something odd. On certain API calls, the docs still have two sets of examples. Take DELETEREPLICA in replica-management.adoc for example. The section gives a short overview, then has some tabbed v1/v2 examples, then it has detailed description for each API parameter. All great so far! But then before the next API command, it gives more DELETEREPLICA examples.
In some cases (e.g. ADDREPLICA) the second set of examples are different enough from the first set that they might provide value. But in other cases it looks like the two sets of examples are near duplicates and we just forgot to remove them.
Was this something you did thoughtfully? Or was it just an oversight? Happy to go through and fix myself - just wanted to make sure I wasn't undoing something you did intentionally.

If you are in there, I think go ahead and delete them!   I think I ended up changing the structure to make the v1 and v2 examples always at the top, and then the more detailed down below, that might have caused some duplicates.

your edits look great!
WDYT about merging after those last edits?  Feel free to click the magic button!

I rm'd the stuff that looked redundant to me, so I'm fine w/ merging whenever at this point.  I'll leave it open for the weekend to let you do the honors, or in case you want to make sure I wasn't overzealous in ripping out redundancy, but otherwise I'll click the button on Monday.


please fix that names of folders and files doesn't show up

What's the Problem?
file names and dir names are overflow hidden that's so annoying when using small group icons or ctrl + shift + 3 mode
also the side bar doesn't render dirs of device i must close it and open it to work
Solution/Idea
please fix the text overflow in group modes
and also make sure sidebar rerender when closed and opened again
Alternatives
no idea
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.376
Windows Version
window 11
Comments
No response

Merging with #1033


feat: expose elasticsearch action results as metric

Pull Request Description
This PR includes changes related to Issue #60. The following modifications have been made:
Changes

New Metric Addition: A new metric has been added to monitor Elasticsearch actions

Tests

It has been confirmed that the newly added metric works correctly and returns the expected values.

Conclusion
These changes are expected to improve the project's provide a better user experience. Thank you for your review and feedback!

what do you think about adding index_name as a label to the Elasticsearch index success/error count metric? including the index name could make error identification more precise, particularly for mapping errors or other index-specific issues. this might also improve monitoring by allowing us to track issues per index. (when have multiple collection mapping)

what do you think about adding index_name as a label to the Elasticsearch index success/error count metric? including the index name could make error identification more precise, particularly for mapping errors or other index-specific issues. this might also improve monitoring by allowing us to track issues per index. (when have multiple collection mapping)

Done 👌
Thanks for advice 🙏


release: 0.9.0-alpha.3

Codecov Report

Merging #626 (59d92c9) into master (ba4fa6c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #626   +/-   ##
=======================================
  Coverage   19.34%   19.34%
=======================================
  Files          26       26
  Lines        2078     2078
=======================================
  Hits          402      402
  Misses       1676     1676

Impacted Files
Coverage Δ

reana/version.py
100.00% <100.00%> (ø)


multi-core support

I have a question regarding the use of TensorFlow Microcontroller in the case of some multicore device (ARM CortexM cores). How to use TensorFlow Microcontroller within a multi-core device and switch between cores (heterogenous)?

Currently  TF Micro does not have a multi-core support.


Aditya Samavedam CPT Final Individual

Link to my post about collegeboard requirements and how I satisfied them.
Link to my video

Component A: Your work in your feature of the project is very evident, and I can see that the code fulfills the collegeboard requirements. The explanations are very clear and detailed, making it easier to read and understand. All your games are unique and fits the critera. Excellent job on this!
Component B: The video is perfect in timing and is concise, clear to view. It shows the output and the change to the data, which is perfect for the criteria. Great video
overall, you did a very good job! 0.97/1


Add Dokuwiki integration

Creates toggl button anywhere on the site.
Button is placed right bellow the header node and can be used for any activity on dokuwiki (reading, editing, comparing...)

This is a screenshot for bootstrap version

Hi @syko would it be possible to have this #849 and #851 integrations reviewed? Thank you


125 Provide UI for Users to Rename The Input

Description

Implemented RenameDialog for Users to rename user input
Enable users to edit the title of UserInput through the UI.
Display the new title in the UI as the user types.
Implement input validation for input
Error handling with SnackBarNotification is in place
React Query setup in https://github.com/Linkta-org/core/issues/250 is integrated in the component
Enhanced zod

Fixes #125
Type of change

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

How Can this be tested? Testing Plan to review this PR

[X] Where in the app does this change take place?

[X] What are the steps to test this change?

At root dir, run npm run dev

Regular input:
Click 'tuxedo cats' -> Rename -> input 'new tuxedo cats' -> save

Error input:
Click 'tuxedo cats' -> Rename -> input  -> save

[X] What is the expected output?
Regular:

Error:

Checklist:

[X] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[ ] Any dependent changes have been merged and published in downstream modules.
[X] I have reviewed the Security Best Practices Document.
[X] I have followed the Naming Conventions Guide for my changes.
[X] I have ensured that my pull request title is descriptive.

Additional Information
Validation will be improved to have a shorter length restriction for input titles
Note: this branch was branched off #121 so there are duplicate changes. Please test this PR based on the instruction provided.

Closing this PR after merging it to #306


[question] Is it possible to configure the plugin so that it will work with both JDK 8 and 11/17/...?

We're in the process of migrating projects from Java 8 to 11/17. For now, the target compilation level will remain 8 but I am already testing the build with JDK 17.
I learned that I have to change
<libs>
  <lib>${java.home}/lib/rt.jar</lib>
  <lib>${java.home}/lib/jsse.jar</lib>
  <lib>${java.home}/lib/jce.jar</lib>
</libs>

to
<libs>
  <lib>${java.home}/jmods</lib>
</libs>

Is there a Java version agnostic configuration syntax or some other way to tell the plugin to just use whatever is in java.home?
(I am also missing configuration examples for modern Java versions in the documentation)

Similar problem is discussed in https://github.com/wvengen/proguard-maven-plugin/issues/350
You might want to open an issue with Proguard itself https://github.com/Guardsquare/proguard-core / https://github.com/Guardsquare/proguard
to make this whole lib passing automatic.

My current solution (might be of interest for people in the same situation).
Projects that use specific execution ids need to define their on jdk-11+ profile.
    <profile>
      <id>jdk-11-on</id>
      <activation>
        <jdk>[11,)</jdk>
      </activation>
      <build>
        <plugins>
          <plugin>
            <groupId>com.github.wvengen</groupId>
            <artifactId>proguard-maven-plugin</artifactId>
            <configuration>
              <libs>
                <lib>${java.home}/jmods</lib>
              </libs>
            </configuration>
          </plugin>
        </plugins>
      </build>
    </profile>


Skip missed samples

A simple (and ugly) change to skip missed samples rather than religiously playing back all samples (this is important for cases where we cannot provide samples to fill the output buffer, such as in the case of a network blip).

Whoops! Didn't mean to make this PR. I'm trying out naudiodon for an online music performance app I'm building (feel free to check it out here!) and needed to make a change for my own purposes. This change should not make it into naudiodon proper. Thanks for maintaining this package!


mac-23, mac-24, mac-8 & mac-9 lost external connection from phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-23, mac-24, mac-8 & mac-9 lost external connection from phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting all machines above to restore connection.


[Spike] Prototype and Validate SSO in tIDIR

Steps to be added

pre-step: Nick - Enterprise Application registration -> Needs permission to create groups.
⚠ Reach out to ADMS to get permissions -  ⚠
Disconnect Forge from Live IDIR
Connect Forge to tIDIR
Create groups - decide in naming structure (Is there a convention?)
Adding users to groups
Enabling AWS Sync
Unknown to lift - 40 minutes waiting period between Azure ID -> AWS permissions.

For #5 here checking in with ADMS on naming convention that we should align with to fit with OCIO standards for ease of operations, reading/understanding and how we can incorporate cloud-specific identifier in each to note the platform its used for (example: AZ, AW, GC)

As per #5, we can follow the OCIO std we followed before creating these groups which would normally be PIM enabled, but in this case not (yet).  So, we would simply not have the beginning part ('PIM_') in the naming convention - something like the following could be done:  DO_PuC_123ldf12_AG_Mgmt_O,  DO_PuC_123ldf12_AG_Dev_C,  DO_PuC_123ldf12_AG_Billing_R, etc.
Later, if we can get PIM enabled on these then the naming convention would simply be updated with std PIM indicator (example:  PIM_DO_PuC_123ldf12_AG_Dev_C)

Also, we may want to add a place in the Registry where the PO can designate 2-3 key people to be able to submit tickets and others who could Read all tickets for their project set subscriptions - ie. where once added people would be added to that project set's Support Request groups:  DO_PuC_123ldf12_AG_SuppReq_C,  DO_PuC_123ldf12_AG_SuppReq_R


Google V2 endpoint dont support anymore approval_prompt parameter

Hi,
according Google documentation https://developers.google.com/identity/protocols/oauth2/openid-connect#authenticationuriparameters and info I found on
https://community.boomi.com/s/article/oauth2refreshtokenonlyworksforanhouringoogleintegration
approval_prompt is not support and you need to delete usage of it in
https://github.com/thephpleague/oauth2-client/blob/0facae9209aef5643bdd780e99996a21b6769c36/src/Provider/AbstractProvider.php#L347
We using SimpleSamlPHP foun that yesterday, but it is look that otehr implementation is efected to check Drupal issue https://www.drupal.org/project/social_auth_google/issues/3129494

Please use the league/oauth2-google package, available here: https://github.com/thephpleague/oauth2-google
The approval_prompt is properly unset in the oauth2-google package:
https://github.com/thephpleague/oauth2-google/blob/f70336461a1cd6399d3db014b30a5c8a165e8691/src/Provider/Google.php#L80-L82


Not working in 7.17.0 with Docker

I try to install the plugin with this docker container but it behaves as if no plugin is installed.
I have already tried Shift+F5, Ctrl+F5, Ctrl+Shift+Del and another browser. However, the view still looks like this:

ARG ELK_VERSION

# https://www.docker.elastic.co/
FROM docker.elastic.co/kibana/kibana:${ELK_VERSION}

# Add your kibana plugins setup here
# Example: RUN kibana-plugin install <name|url>

RUN kibana-plugin install https://github.com/fbaligand/kibana-enhanced-table/releases/download/v1.13.0/enhanced-table-1.13.0_7.17.0.zip

kibana:
    env_file:
      - .env
    build:
      context: kibana/
      args:
        ELK_VERSION: $ELK_VERSION
    volumes:
      - ./kibana/config/kibana.yml:/usr/share/kibana/config/kibana.yml:ro,Z
    ports:
      - "${KIBANA_PORT}:${KIBANA_PORT}"
    networks:
      - elk
    depends_on:
      - elasticsearch

Hi,
I’m very surprised of this issue because I tested v1.13.0 with Kibana 7.17.0 (without Docker) and it works fine. A community user also confirmed me that it works fine.
So it seems to be tied to your Docker setup.
Questions:

have you followed instructions here?
https://github.com/deviantony/docker-elk#how-to-add-plugins
have you error or warning logs at build or run time?
Could you connect to your Docker instance (with /bin/sh) and check that $KIBANA_HOME/plugins contains enhancedTable folder, and inside, there is kibana.json, package.json files and target directory?

Given that there is no feedback since several months, I close the issue.


Filter spacing is off (iPad Air Safari vertical)

Seen in iPad Air safari vertical

dupe of #3057


Path Sum III (Leetcode)

Aim

Details
I would like to solve Path Sum III in C++ under LGMSOC'21.
Programming language

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

@ravikr126 Kindly assign me this issue under LGMSOC'21


feat(#417): Add sourcesDir/outputDir to AssembleMojo

Add the following directories:

sourcesDir defaults to ${project.build.directory}/generated-sources/jeo-xmir
outputDir defaults to ${project.build.outputDirectory}

Closes: #417

History:

feat(#418): add input/output directories for assemble mojo
feat(#418): verify targetDir for assemble mojo

PR-Codex overview
This PR focuses on updating the default directory name from "xmir" to "jeo-xmir".
Detailed summary

Updated the default directory name from "xmir" to "jeo-xmir" in multiple files.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here


STORM-2226: Fix kafka spout offset lag ui for kerberized kafka

+1

Thanks @priyank5485, merged into master and 1.x-branch.


ML backend returned empty prediction

Dear developers,
I set up all things according to your documentation, except that I use Python 3.10.4, and I can only install torch==1.11.0+cu113 torchvision==0.12.0+cu113 torchaudio==0.11.0
Everything looks fine except that the auto segmentation does not work when I pick a point.
Below is the output. It says "ML backend returned empty prediction".

Also I notice that the model version is "unknown" instead of "initial"

Please help. Thank you very much!
best regards

I solved my problem. I installed a label-studio of lower version 1.5. After changing label-studio to the latest 1.7.3, it works.


Map Control visibility

Hi there,
I've been trying to incorporate some custom controls on a map as per this example:
http://tombatossals.github.io/angular-openlayers-directive/#/more/control
The controls appear to be loaded in the DOM but are not actually appearing onscreen. Ultimately I'd like to include a custom control called ol-layerswitcher but I can't even get the 'basic' controls to appear.
A plunkr would be too difficult to include but here is a link to a demo:
http://dev.rgsit.com/ux/xped2AngBuild/index.html
Thanks in advance for any help.

First you should add this to your defaults to avoid creating the default controls:
angular.extend($scope, {
defaults : {
  controls: {
                    zoom: false,
                    rotate: false,
                    attribution: false
                }
},
...

Then you missed the really important ol-control-properties attribute on your control directive:
<ol-control name="{{ control.name }}" ol-control-properties="control" ng-repeat="control in controls|filter: {active: true}"></ol-control>

This should work better although i cannot test on my side.

I'm doing some housekeeping and closing the issue since it's quite old. Please feel free to re-open it if it still persists. Also maybe try to upgrade to the latest version as it might fix it for you.


Add Winget Releaser workflow & readme installation instructions

This action automatically generates manifests for Winget Community Repository (microsoft/winget-pkgs) and submits them.
Before merging this:

Add a classic PAT with public_repo scope as a repository secret named WINGET_TOKEN.

Fork https://github.com/microsoft/winget-pkgs under @nefarius. The action will use that fork for making a branch and creating a PR with the upstream winget-pkgs repository on every release.
Install Pull on the winget-pkgs fork to ensure that it is constantly updated.

You can test out the action inputs in this playground.
If you want to see an example of a PR created using this action, see microsoft/winget-pkgs/pulls (Pull request has been created with WinGet Releaser).

Works:

https://github.com/microsoft/winget-pkgs/pull/100351


feat: add paths repro/example

Reproduction of issue described in #31

If you run bazel build //examples/paths:transpile and have a look at the import paths of the resulting output in bazel-bin/examples/paths/src/index.js, it should be similar to the import path described in the original issue


Add -gen-MDER flag in asn1c

MDER stuff is only  generated if -gen-MDER flag was provided.

ping

 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.


do not modify dns.Rcode when packing to wire format

When the message has an EDNS0 option in the additional section and
dns.Msg.Rcode is set to an extended rcode, dns.Msg.PackBuffer() modifies
dns.Msg.Rcode.
If you were to Pack the message and log it after, the Rcode would show
NOERROR.
Running the test before the change would error with:
=== RUN   TestPackNoSideEffect
--- FAIL: TestPackNoSideEffect (0.00s)
	msg_test.go:51: after pack: Rcode is expected to be BADVERS

after fixing dns.Msg.PackBuffer(), all tests are still passing.
Fixes #674

Codecov Report

Merging #675 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #675      +/-   ##
==========================================
+ Coverage   58.14%   58.18%   +0.04%
==========================================
  Files          37       37
  Lines       10017    10018       +1
==========================================
+ Hits         5824     5829       +5
+ Misses       3145     3140       -5
- Partials     1048     1049       +1

Impacted Files
Coverage Δ

msg.go
78.2% <100%> (+0.03%)
:arrow_up:

defaults.go
60.81% <0%> (+2.7%)
:arrow_up:

Continue to review full report at Codecov.

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

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] do not modify dns.R..." ]

chantra commented on this pull request.

}

// Convert convenient Msg into wire-like Header.
dh.Id = dns.Id

dh.Bits = uint16(dns.Opcode)<<11 | uint16(dns.Rcode)

dh.Bits = uint16(dns.Opcode)<<11 | uint16(rcode)

@miekg what would be a tad cleaner? using dns.Rcode & 0xF inline? or the variable?
The fact that you are asking for a comment makes me believe you mean the former, but just want to make sure I get you right

yep inline
/Miek
--
Miek Gieben

addressed comments


Error Message during download

Hi, I tried the program, it was downloading well last night. But this morning when I tried again It shows error. Not sure where went wrong.

Because the error information you provided is incomplete (you should at least show the bottom of traceback), I'm not pretty sure whether it's a net error or bug in bilix.

Hi HFrost0,
I'd installed the ffmpeg. I think its okay. Cos I realised that it will check for the files already downloaded, and continue to process the rest. Although its a bit fussy to press a few times to run the code, it is still so much better than compared to manual doanlod via different sites and has to bear thru the advertisments.
Thanks a lot! Your script is a saviour!
Cheers!

One Question, your simple chat, can use it on Powershell command line?

@williamsoo Of course

Awesome!

@williamsoo It's just a simple program with only about 30 lines, feel free to use or modify it. By the way, If you have any question about Simple-Chat, let's discuss in Simple-Chat Issues instead of bilix issues.

I apologise. I am very new to github and not sure where is the discussion board for simple-chat.


Shapes data does not conform to LayerDataProtocol

🐛 Bug
Since Shapes is a Layer, its data should fulfill LayerDataProtocol. However, (some) internal ways of populating this don't preserve the required properties
To Reproduce
Steps to reproduce the behavior:
Run the test:
def test_data_protocol():
    shp = Shapes()
    # Add 2-D ellipse
    data = np.ones((2, 2))
    data[:, 0] = [20, 20] # Center
    data[:, 1] = [10, 10] # Axes
    shp.add_ellipses(data)
    # Check for properties
    assert hasattr(shp.data, 'dtype')
    assert hasattr(shp.data, 'shape')

Expected behavior
I would expect that shp.data would have the shape and dtype properties.
Environment

Please copy and paste the information at napari info option in help menubar here:

napari: 0.4.13
Platform: Linux-5.13.0-28-generic-x86_64-with-glibc2.31
System: Ubuntu 20.04.3 LTS
Python: 3.9.10 | packaged by conda-forge | (main, Feb 1 2022, 21:24:11) [GCC 9.4.0]
Qt: 5.12.9
PyQt5: 5.12.3
NumPy: 1.20.3
SciPy: 1.7.3
Dask: 2022.01.1
VisPy: 0.9.4

OpenGL:
- GL version: 4.6 (Compatibility Profile) Mesa 21.0.3
- MAX_TEXTURE_SIZE: 16384

Screens:
- screen 1: resolution 2560x1440, scale 1.0
- screen 2: resolution 1920x1080, scale 1.0

Plugins:
- Labeling: 0.1.dev9+g4bf5a30
- console: 0.0.4
- scikit-image: 0.4.13
- svg: 0.1.5

yep, shapes and surfaces were noted omissions from the original LayerDataProtocol PR.  Shapes are rather heterogeneous, and require some re-working to provide the protocol from https://github.com/napari/napari/pull/2683 ... It would need something like the MultiScale wrapper there

yep, shapes and surfaces were noted omissions from the original LayerDataProtocol PR

Ah, so they are! Shapes still seems to be from the LayerDataProtocol docstring.
I agree that it is a large undertaking to make it conform - maybe it would just be best to better specify that it does not conform? Maybe it should either just not use data, favoring some different terminology?

That seems like a reasonable intermediate step.  The whole layer data protocol is rather new: it's an acknowledgement that we should at least try to define the methods expected of a given layers's data object. But as a late addition, it still needs some attention

What do you think of changing the current Shapes.data object then to Shapes.shapeData, and adding throwing an clear NotImplementedError in the Shapes.data getter and setter? That seems preferrable to getting an error saying that the data just doesn't have these attributes.

cc @sofroniewn

What do you think of changing the current Shapes.data object then to Shapes.shapeData, and adding throwing an clear NotImplementedError in the Shapes.data getter and setter? That seems preferrable to getting an error saying that the data just doesn't have these attributes.

While I can understand this - it would be a big breaking change to take away layer.data from shapes, everyone using the layer would have to change. We might just need to bite the bullet and add a .shape and .type

It would need something like the MultiScale wrapper there

Indeed I think it would. I could imagine how this might be done. Maybe we should give it a go? What do @andy-sweet @jni @nclack think?

What do you think of changing the current Shapes.data object then to Shapes.shapeData, and adding throwing an clear NotImplementedError in the Shapes.data getter and setter? That seems preferrable to getting an error saying that the data just doesn't have these attributes.

While I can understand this - it would be a big breaking change to take away layer.data from shapes, everyone using the layer would have to change.

That's a great point, yeah. For that reason, we probably shouldn't do this.


close() with option to close TagUI and SikuliX but not Chrome - ideas to explore

Any parameters for the close() core function to close TagUI & SikuliX but keep the opened chrome browser ?

I'm afraid not because the code to close all the TagUI processes including Chrome is embedded quite deeply in upstream TagUI engine. Would doing a long r.wait() or an Python input() to wait for user to confirm to continue before closing help?
# wait 1 hour
r.wait(3600)

input('Press Enter key to continue.. ')
r.close()

Adding on, let me know more about your usage scenario why this is needed, I'll see if there's any other workarounds.

Oh I see.. Your scenario does not seem to require SikuliX. In that case, you can try doing your automation but just don't include r.close() at the end. The memory and CPU overhead for TagUI should be considered relatively very low.
And the next time you run automation again with r.init(), it will automatically do the clean-up of dead processes before start.

And if you are really interested for a foolproof solution and willing to hack, then you can do the following -

print(r.tagui_location())
look for a tagui or .tagui folder (depends on your OS)
look for src\end_processes (mac/linux) or end_processes.cmd (win)
delete the section involving chrome_process_id
in your script don't use r.close()
but instead do os.system('full path to the end_processes')

Above hack should kill everything except the Chrome browser.


Rename RuboCop::AST::Token#regexp_dot? to RuboCop::AST::Token#regexp_dots

Follow up https://github.com/rubocop/rubocop-ast/pull/235#issuecomment-1179649807.
This PR renames RuboCop::AST::Token#regexp_dot? to RuboCop::AST::Token#regexp_dots.
It's been determined that it doesn't need to be an alias because it's the API that has just been released and isn't widespread.

Cool, thanks. Released as 1.19.1. I simplified the Changelog too.

Great! Thank you Marc-André!


Prisma Twistlock Flagging 6 Containerd Vulnerabilities in Flatcar 3510.3.3 LTS

Name: containerd
CVEs:
CVE-2022-1996
CVE-2023-27561
CVE-2024-21626
CVE-2023-47108
CVE-2023-44487
CVSSs:
CVE-2022-1996  = 9.1
CVE-2023-27561 = 7.0
CVE-2024-21626 = 8.6
CVE-2023-47108 = 7.5
CVE-2023-44487 = 5.3
Action Needed:
These have been flagged by Prisma Twistlock so will likely need some further analysis. If they are false positives that would be good news for us!
Summary:
We have been using Flatcar LTS Kubernetes nodes in our Lab environments. On these environments we are running the security DaemonSet Palo Alto Prisma Twistlock. From Prisma we can see the vulnerabilities flagged above. Here they are in a table format:

CVE ID
Severity
Package Version
Package Path
CVSS
Package URL

CVE-2022-1996
critical
v2.9.5
/run/torcx/unpack/docker/bin/containerd
9.1
<EMAIL_ADDRESS>

CVE-2023-27561
high
v1.1.2
/run/torcx/unpack/docker/bin/containerd
7.0
<EMAIL_ADDRESS>

CVE-2024-21626
high
v1.1.2
/run/torcx/unpack/docker/bin/containerd
8.6
<EMAIL_ADDRESS>

CVE-2023-47108
high
v0.28.0
/run/torcx/unpack/docker/bin/containerd
7.5
<EMAIL_ADDRESS>

CVE-2023-44487
high
v0.0.0-20220722155237-a158d28d115b
/run/torcx/unpack/docker/bin/containerd
5.3
<EMAIL_ADDRESS>

CVE-2023-44487
high
v1.47.0
/run/torcx/unpack/docker/bin/containerd
5.3
<EMAIL_ADDRESS>

As you can see it delves into Golang dependencies so if these vulnerabilities have been incorrectly flagged then I can just take that information back to our internal security team.
refmap.gentoo: TBD

Some of this was discussed on Matrix but it was flagged that for CVE-2022-1996, as described in https://github.com/containerd/containerd/pull/7117, the vulnerability only affects CORS which containerd does not use. The runc vulnerabilities CVE-2023-27561 and CVE-2024-21626 seem to affect runc itself and I haven't been able to find if it can affect specifically containerd. CVE-2023-47108 is for DoS with the opentelemetry-go library, and CVE-2023-44487 is a DoS when using HTTP/2.
I think the Denial of Service vulnerabilities are fine and the runc vulnerabilities should also be fine if I can find some concrete information on if it can affect programs using runc as a library or just the runc binary itself.


add google login

I was attempting to create a ruby port of the API (but failed) but manages to port over gpsoauth in the proccess (https://github.com/xssc/gpsoauth). I built and tested this on my local machine and it works great.

I downloaded your branch and tested it on my account, I confirm that it's working fine without any problems :+1:

Thanks @xssc, does this work for non-browser based machines as well? (I'm not able to test it until evening).

@nabeelamjab yup

@xssc - confirmed working, I'll merge it in.
I might do some refactor work at some point (to keep it consistent with Ruby style guide). I'll release a new tag tomorrow as well for the addition of Google login.
Thanks again!

@nabeelamjad okay, and oops, I accidentally yanked v0.1.0(first gem, thought I could push the same version so it didnt need to be updated), so poke-api.gempsec needs to updated to require v0.1.1, I updated the gem so it doesn't require libcurl (can be a hassle to install on windows).  sorry, I'm still learning. Nothing else needs to be changed though.

Alright will do.

@xssc - updated and pushed as 0.0.2, thanks once again!


Cron renew forgets certificates if one fails

tested on Arch Linux and dehydrated version 0.7.1 (git revision unknown)

If domains.txt defines the following certificates
example.com www.example.com
example.net
www.example.net

and for example the challenge for www.example.com fails (due to DNS changes or anything else),
then the certificates for example.net and www.example.net will not be renewed even if the challenges there would work.
This should be fixed so dehydrated tries to renew all certificates and complains about all where the challenge failed instead of just complaining about the first failed.

Not a bug. There is a documented KEEP_GOING flag in the config that allows dehydrated to continue after an issue occurred.


Update smithy version to 1.23.0

Update Smithy version to 1.23.0 to pick up the IDL-2.0 Changes.

Description of changes
This changes updates the version of Smithy to 1.23.0. Given the changes that this release brings there are some shapes in the prelude that are now @deprected, to avoid creating any changes to the code generation we will skip adding the available attributer if the target of the members is a shape in the prelude. This should keep the generated code the same.

Scope

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Creating a new one


Linking anonymous orders to newly created customers by OIDC Plugin

What I'm trying to achieve
Customers should be able to first anonymously buy product(s) and later authenticate with OIDC Plugin.
Describe a proposed solution
Anonymously created orders should be linked to (later created) customers by OICD Plugin
Other solutions I've tried and won't work
I could not find a mutation in the API wo link those orders
Screenshots or mockups
see also:

https://github.com/saleor/saleor/discussions/9568
https://github.com/saleor/saleor/discussions/9577

Hi @rolego,
The user orders are linked when a user account is created by the OIDC.
Here is the line in the code responsible for that.
Please let me know if you find out that it's not working.


Updating typescript documentation for create vs put overwrite default

Close #359

Pull Request Test Coverage Report for Build 575

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 83.881%

Files with Coverage Reduction
New Missed Lines
%

lib/Table.js
3
78.42%

Totals

Change from base Build 572:
-0.2%

Covered Lines:
1843

Relevant Lines:
2124

💛  - Coveralls