Reports Host OS version and exact type for RHEV-M

The version and product type of Host are reported from the :os
element if present.
http://bugzilla.redhat.com/1360416

@miq-bot assign @blomquisg

@blomquisg I removed the change to the product from this patch and kept only the change to the version part. The vmm_product will describe the nature of the host as before: rhel, rhev-h or ovirt-node (instead of the explicit host os name).

K, this looks good.
One last comment.  Can you update the specs for this to check for the new version info?

@blomquisg I don't think I can - the existing specs are assume to work against ovirt-engine 3.0[1] and 3.1 [2] while the change becomes effective only for ovirt-engine 4.1 and above (as part of fixing this bug I also pushed a patch to the ovirt gem [3] and to ovirt-engine)
[1] spec/models/manageiq/providers/redhat/infra_manager/refresher_3_0_spec.rb
[2] spec/models/manageiq/providers/redhat/infra_manager/refresher_3_1_spec.rb
[3] https://github.com/ManageIQ/ovirt/pull/67

@masayag k, I understand...
Can you guys create an item on the backlog to create tests against an ovirt 4.1 environment?  And, you can just handle these tests in a separate PR.

@blomquisg will do.


remove EFF sites from /shared/data/tracker_lists/entityMap.json

https://panopticlick.eff.org/about
trackersimulator.org
eviltracker.net
do-not-tracker.org

These domains contain files used to simulate various tracking techniques.  By blocking these domains, the browser extension masks testing what tracking techniques might actually be in use elsewhere.

The fingerprint checking still appears to be loading in the test so this appears to be working as intended. Please reopen if this isn't the case.


JSON data getting very escaped

We're getting JSON data like:
{
  'contractors': [{
    'id': 456789,
    'name': 'John Smith'
  }],
  'date': '2017-1-1T00:00:00Z',
  'description': 'GCSE Exam',
  'recipients': [{
    'client_name': 'Brain Johnson',
    'id': 123456,
    'name': 'Cara Johnson'}],
  'reference': [12345, 56789],
}

coming through to Sentry like:
[{
  "'items'": [{
    "'date'": "'\\'\"u\\\\\\'datetime.datetime(2017, 1, 1, 0, 0, tzinfo=<UTC>)\\\\\\'\"\\''",
    "'contractors'": "'\\'\\\\\\'[{u\"\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'name\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'\": u\"\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'John Smith\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'\", u\"\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'id\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'\": u\"\\\\\\\\\\\\\\'456789\\\\\\\\\\\\\\'\"}]\\\\\\'\\''",
    "'recipients'": "'\\'\\\\\\'[{u\"\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'name\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'\": u\"\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'Brain Johnson\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'\", u\"\\\\\\\\\\\\\\'\\\\\\\\\\\\\\'...",
    "'description'": "'\\'\\\\\\'u\"\\\\\\\\\\\\\\'GCSE Exam\\\\\\\\\\\\\\'\"\\\\\\'\\''",
    "'reference'": "'\"\\'[12345, 56789]\\'\"'"
  }]
}]

Let me know if you need any more details.

Same here.. if I call
client.captureMessage("test message", extra={"a": {"b": ["c"]}})

my Sentry report looks like:


ST_GeneratePoints implementation

Summary
Requesting the addition of the ST_GeneratePoints function to the DuckDB spatial extension. This function generates a specified number of random points within a given geometry and is available in PostGis for instance.
Motivation
The absence of ST_GeneratePoints in DuckDB requires me to connect to Postgres for this functionality.
Integrating this function directly into DuckDB will streamline spatial data workflows by eliminating this dependency. It will also improve significantly my jobs runtime by eliminating steps.
Example of workaround (SQL)
-- Connecting Postgres to add a point by point column with st_generatepoints, with mandatory geometry handling
ATTACH 'dbname=postgres user=postgres host=<IP_ADDRESS> port=5439' AS postgres_db (TYPE postgres);
drop table tmp_menages_esri50_discretise;
create table tmp_menages_esri50_discretise as
SELECT ST_GeometryN(st_GeneratePoints(st_geomfromtext(geom),1),1) point_individuel, a.*
FROM tmp_menages_esri50 a;
-- back to DuckDb to handle the end of the job
copy (select
st_geomfromhexwkb(point_individuel) geom_point_individuel, st_x(st_geomfromhexwkb(point_individuel)) x_geom_point_individuel, st_y(st_geomfromhexwkb(point_individuel)) y_geom_point_individuel,  "type_logt" from postgres_db.tmp_menages_esri50_discretise
limit 10000000
)
to "C:\Users\ali.hmaou__LocalEnv\Projets\POINTILLES\export.geojson"
WITH (FORMAT GDAL, DRIVER 'GeoJSON', LAYER_CREATION_OPTIONS 'WRITE_BBOX=YES')

Hello! Thanks for opening this feature request!
I agree that ST_GeneratePoints would be great to have. I've been wanting to implement it to make it easier to produce tests and benchmarks. I'll add it to my TODO list.

https://github.com/duckdb/duckdb_spatial/issues/411
Apparently this function is there, but it's not documented :)

Damn ! And the workaround in the discussion looks fine too. Gonna try it.


fix: tests failing on windows

What this PR does / why do we need it:
This PR modifies the test suite so that it is compatible with Windows. There were 2 issues that made it incompatible:

tempfile.NamedTemporaryFile cannot be read twice while open on Windows -> I switched to a regular file inside of a tempfile.TemporaryDirectory.
On Windows, Python raises a PermissionsError whenever we try to read from or write to a path that is a directory. This is reported as a bug: https://bugs.python.org/issue43095

Which issue(s) does this PR fix:

Fixes #17
Release Notes

Fixed test suite on Windows.

What type of changes to the API does this change introduce?
No user-facing changes. We just modified the test suite.
Checklist

[x] I've read the Contribution Guidelines
[x] Updated the appropriate sections of the documentation.
[x] I've added automated tests covering all success and error scenarios.

Codecov Report

Merging #30 (c403f26) into main (a274cf1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   99.92%   99.92%
=======================================
  Files          53       53
  Lines        1299     1299
=======================================
  Hits         1298     1298
  Misses          1        1

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 a274cf1...c403f26. Read the comment docs.


Fix subcommand initialization

Closes https://github.com/andrey-zherikov/argparse/issues/44

Codecov Report

Merging #50 (701291d) into master (ce4ca59) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   93.84%   93.88%   +0.04%
==========================================
  Files           1        1
  Lines        1056     1063       +7
==========================================
+ Hits          991      998       +7
  Misses         65       65

Impacted Files
Coverage Δ

source/argparse.d
93.88% <100.00%> (+0.04%)
: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 25821a3...701291d. Read the comment docs.


Create an 'Initial' Private Kernel Circuit

Migrated from archived circuits repo: https://github.com/AztecProtocol/aztec3-circuits/issues/200
"""
Currently, there's only 1 private kernel circuit.
That means for the very first iteration, we need to generate dummy 'previous kernel data'. The way we generate this data is inefficient, and the way we generate this data is inefficient (because we're currently generating actual VKs over a mock circuit).
Todo:

Separate c_bind for calling the initial native previous kernel circuit
A new struct for InitialPrivateKernelInputs and InitialPrivateKernelPublicInputs
Remove code which generates a dummy previous kernel (unless needed for testing, but I don't think we'll need that)
Write an initial native previous kernel circuit, which skips 'previous kernel proof verification' and any 'recursive case' logic

Make sure to re-use common codepaths between this circuit and the existing 'recursive' private kernel circuit (refactor to put functions in common files, if needed).
Make sure this circuit can only be called in the 'base case' (by including appropriate assertions inside the circuit)

Update the existing 'recursive' private kernel circuit:

remove:

The signature verification
TxRequest validation (we might even be able to remove the SignedTxRequest input altogether)
The 'base case' logic
Contract deployment logic (for now)

keep:

Verification of the 'previous kernel data'
recursive case logic
etc.

new:

A check on the previous kernel data's verification key -- that it matches the hard-coded one of the initial kernel circuit, when executing private_call_count = 2. (I.e. we're enforcing that private_call_count = 1 used the 'initial' circuit).

Note: for subsequent iterations of the kernel circuit, we can't check the verification key within the kernel circuit, because that's cyclic! We defer such verification key checks to the base rollup circuit. (A separate issue will be opened for this).

Typescript:

Work with someone to make corresponding changes to all the typescript code (mainly in the kernel-prover package), including structs.

"""
Story subtasks:

#502
#503
#504
#505

We'll want to mimic some of what has been done for the public kernel. We can move some logic into a common.hpp. Constructor logic only needs to be in the initial kernel. We may want to consider renaming some public kernel functions and structs if we decide on more intuitive names for their private kernel workalikes.

Note after starting work in #549:

We'll want to add the CombinedHistoricTreeRoots to the PrivateKernelInputsInitial to initialize public_inputs.constants.
We can initialize public_inputs.constants.tx_context from the tx_request.
Since we discovered that there were checks missing comparing tx request against private call, we should add test cases that would catch this
We shouldn't hesitate to refactor the public kernel to match ours as we move forward with the private kernel split

Closing, as status moved to 'Done' (it's a GitHub bug)


Spaces are removed when recognizing Korean language

Microsoft PowerToys version
0.63.0
Running as admin

[ ] Yes

Area(s) with issue?
TextExtractor
Steps to reproduce

Run Text Extractor.
Recognize any Korean text which has more than two words separated by space(s).

✔ Expected Behavior
Spaces between words should be remained when recognizing Korean.
Not like Chinese or Japanese, space should not be removed when extracting Korean.
❌ Actual Behavior
All the spaces are removed.
Other Software
No response

Can you confirm this is still an issue with PowerToys v0.72?
/needinfo

Can you confirm this is still an issue with PowerToys v0.72?
/needinfo

It has been fixed since v0.66.0


[FLINK-19396][Connectors/Kafka]fix properties type cast error #13470

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Properties extends Hashtable<Object,Object> ，The key and value of the Properties class support object objects. If the user's value is a boolean type, this code "(String) k, (String) v" will occur an error.
such as : java.lang.ClassCastException: java.lang.Boolean cannot be cast to java.lang.String
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache
update cast object to String : (String)o to String.valueof(o)

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no )
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no )

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

rebase for https://github.com/apache/flink/pull/13470


NCL_panel_9.py

Closes #284
NCL Plot: https://www.ncl.ucar.edu/Applications/Images/panel_9_lg.png

The original NCL script uses wgt_runave_Wrap to calculate a running average for the data in the XY plot and then plots the resulting line. Is there an equivalent for this in Python yet? I have been unable to find something that is sufficient. @michaelavs @anissa111 Thanks!

I plotted the rolling average using xarray.DataArray.rolling() and it is similar to what is in the NCL plot, but the Python version is still jagged. The NCL version is also a weighted rolling average which hasn't been implemented in xarray yet. I'm going to continue to look for an improved way of doing this, but let me know if any of you have suggestions.

For future reference, the weighted rolling mean of xarray data can be calculated by first making a DataArray object with the weights. Then, call xarray.rolling() on the data in question. Using .construct('window') on the rolling object and then doing the dot product between it and the array of weights returns the weighted average. Note that the weights have to sum to 1 for this to work properly.
The edit to this stack overflow post explains it better.

Current state of the plot using make html

To properly fill the xy plot between y=0 and the data, I had to interpolate points and make that into a new array. Is there a better way to do this? I want to make sure this isn't terribly inefficient. I do this starting at line 118.

I think everything looks good overall. I looked into your question about the xy plot and I couldn't find a better way to do the plotting, but I also don't have much of a problem with how the code is now as long as it doesn't take a super long time to build the html file (which it seems it isn't). The only thing I see needing changed is removing native_1 from the PR/branch because I think it's just sticking from another branch currently. After that, I should be ok with approving this PR!

Thanks for catching that @michaelavs. I accidentally did some work in the wrong branch. This PR should be good to go now.

There is actually a better way of doing this, though it essentially just does what you did on the back end. There's an option on the fill_between(...) call that will do this for you!
I've included the changes as suggestions below along with a typo I found. Other than that, this looks good to go, thank you!

Thank you @anissa111! I had no idea I could use that kwarg. This should be good to go once all of the checks complete.


Gem version 0.9.1

Hi, issue #32 is breaking our Chef infrastructure. Would you mind releasing a new version of the gem for 0.9.1? In the meantime, we'll set an explicit Git SHA in our Gemfile. It'll be a bit messy. Thanks!

I just pushed v0.9.1.
Let me know if this works for you.

It works beautifully. Thank you!


Add powershell version of configure-skeleton script to support windows

Added powershell version of the script to support windows developers.
Update the bash version to delete the powershell script before deleting himself.
I don't know if you will find that PR useful but I use it so
🙋♂

Thank you, but I’m not going to merge this in.
We don’t use Windows and don’t want to maintain this code.


Tap event

Hi, it's nice control! Thanks!
What about tap event?

@NDN84 Please check README. Thank u.

Yes, but if hide by tap is NO, maybe we can using our delegate?


Fix attenuation oscillation not stabilising

I think this is good but should be squashed to one commit. The docs also need updating to say that singleshot stabilises when it hits 0 attenuation or when it increases - currently it just says when it does not cause any adjustment. Also I don't like the formatting...

Is it mainly just the subscribe_endpoints you don't like the formatting of? :sweat_smile:

The whitespace padding on the initialiser list is the most offensive bit for sure. I prefer to have opening brace on the same line rather than the next line, trailing brace on its own line - basically black formatting. I also prefer */& to be on the type, not the name, e.g. here.
I can see it has caught some bad formatting, which is good. The problem is whatever formatting you are using here isn't being enforced / automated and when the next person works on the project it won't be maintained, so I don't think it is worthwhile reformatting everything unless there is the tooling to keep it consistent.

@GDYendell so just make the formatting changes and merge?

I think so. I guess this is what is running on the beamline currently?

The problem is whatever formatting you are using here isn't being enforced / automated

Apparently it was using the default C/C++ formatter built in to VSCode

@GDYendell I'll update the docs on a different PR if that's okay? They want this on the beamlne ASAP...


⚠ Samsung has degraded performance

In 12c30f7, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3234 ms

Resolved: Samsung performance has improved in 9698ae8.


Add support for 128x64 UC1701X LCD

Ebay listing: http://www.ebay.co.uk/itm/292061588410
Datasheet: UC1701x_v1.0.pdf

Implementation notes
Code [WIP] on branch: feature/UC1701X.

LCD Pin
LCD Name
RPi Pin
RPi Function
Remarks

1
ROM_IN
n/c
n/a
Unused

2
ROM_OUT
n/c
n/a
Unused

3
ROM_SCK
n/c
n/a
Unused

4
ROM_CS
n/c
n/a
Unused

5
LED A
P01-12
GPIO 18 (PCM_CLK)
Backlight

6
VSS
P01-06
GND
Ground

7
VDD
P01-01
3V3
3.3V Power

8
SCK
P01-23
GPIO 11 (SCLK)
Serial Clock

9
SDA
P01-19
GPIO 10 (MOSI)
Serial Data

10
RS
P01-16
GPIO 23
?? Suspect this is the Data/Command line ??

11
RST
P01-18
GPIO 24
Reset

12
CS
P01-24
GPIO 8 (CE0)
Chip Select

render time = 40.39 ms, frame rate = 24.58 FPS


feat: recursive folding and decider verifier for Protogalaxy

Implements the recursive folding verifier and recursive decider verifier resembling the native ones as closely as possible. Tests follow the example of the other recursive honk verifiers and native protogalaxy tests.
Additionally, I introduced a function that sequentially computes polynomials in coefficient form in the ProtoGalaxyRecursiveVerifier to avoid having to instantiate the entire Polynomial class on stdlib::bn254.
Closes https://github.com/AztecProtocol/barretenberg/issues/789.
Closes https://github.com/AztecProtocol/barretenberg/issues/834.

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 09dbfcd7 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,412
179,524
716,068

l1_rollup_calldata_gas
221,608
866,824
3,446,884

l1_rollup_execution_gas
314,027
984,496
3,666,473

l2_block_processing_time_in_ms
1,479 (+3%)
5,632 (+2%)
21,211 (+2%)

note_successful_decrypting_time_in_ms
385 (-14%)
957 (+2%)
3,485 (+4%)

note_trial_decrypting_time_in_ms
98.1 (+63%)
85.6 (+26%)
158 (+10%)

l2_block_building_time_in_ms
16,238
64,315 (+1%)
257,271 (+1%)

l2_block_rollup_simulation_time_in_ms
11,794
46,816 (+1%)
187,479 (+1%)

l2_block_public_tx_process_time_in_ms
4,403 (+1%)
17,405 (+1%)
69,496 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
16,096 (-1%)
31,076 (-1%)

note_history_successful_decrypting_time_in_ms
2,198 (-2%)
4,301 (-6%)

note_history_trial_decrypting_time_in_ms
121 (-5%)
184 (-16%)

node_database_size_in_bytes
3,514,514
3,553,341

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
213
44,681
25,561

private-kernel-ordering
166
43,753
14,809

base-rollup
1,194
128,338
881

root-rollup
79.4
4,088
677

private-kernel-inner
293
71,208
25,561

public-kernel-private-input
207
31,863
25,561

public-kernel-non-first-iteration
204
31,905
25,561

merge-rollup
7.74 (+1%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,455
32,219

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
376
1,092

Metric
1 public data writes

tx_sequencer_processing_time_ms
539 (+1%)


test package updates to fix pipeline

@CodyBatt Merging this now. Going to fix Mac in a separate branch.


Show all sessions on all the three pages: rooms, tracks and schedule

Fixes #1327
Changes:

All the sessions from all the days are displayed on the three pages (tracks, rooms, and schedule) now.

Test Link
https://princu7.github.io/websiteTest
@aayusharora @geekyd Please review. Thanks!

@Princu7 I'm not sure if this is the desired behaviour specified in the issue description. I think the desired behavious is:

Each page shows all the sessions when there is no filter applied(no specific date button is clicked)
When a specific date is clicked we show the sessions only for that date.

@mariobehling Please guide. Thanks

@geeekyd Oh. Sorry for the misunderstanding. I will correct the mistake. Thanks!!

@geekyd Updated the link. Please review again. Thanks!!

Thanks.


add pod antiaffinity for fio server pods

Some testing confirms that implementing this anti-affinity will cause even distribution of the fio server pods among the schedulable nodes.


Colored console output

Signed-off-by: Raul Sevilla<EMAIL_ADDRESS>Description
Enable colored log output
Fixes

Codecov Report

Merging #153 (40bedd4) into master (35baaae) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 40bedd4 differs from pull request most recent head 79e3c2c. Consider uploading reports for the commit 79e3c2c to get more accurate results

@@           Coverage Diff           @@
##           master     #153   +/-   ##
=======================================
  Coverage   93.45%   93.45%
=======================================
  Files          24       24
  Lines         993      993
=======================================
  Hits          928      928
  Misses         65       65

Flag
Coverage Δ

gha
93.45% <ø> (ø)

python-3.9
93.45% <ø> (ø)

unit
93.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 35baaae...79e3c2c. Read the comment docs.


Need a 'crash' sound effect

https://soundimage.org/sfx-combat/
I personally like Explosion2 and Explosion8.  (This is Eric Matyas' website, in case you've never visited it before.)

I think they all should have their own sounds, actually.  Or even a pool of sounds that can be randomly drawn from for variety.
One sound/pool for landing, one for light (non-landing) impact, one for heavy impact, and one for zero health.  There's probably other sounds that could be added, but this is a really good starting place.  I haven't done any sound stuff with Lua/Love yet, but it's a topic I've been wanting to learn.  :)

Good ideas!
There are already two sounds incorporated. Did you want to follow that example and suggest some code in your own time?
You'll need to note that the asset loader makes this a little bit different than normal.

Yeah, I'll look into this when I have time - likely next week.  And thanks for the heads-up about the asset loader.  I'll make sure I understand it before monkeying.  :D


Refactored Robot Code to slim down methods

Also incorporated name registry checking to ensure multiple names don't exist.

Hey @paulghadad would you mind offering some more advice or a short example of what you'd recommend here? New with Ruby still

@Manbearpixel I suggest you focus on the following for your next revision. This is in a nice place, but I'd like to see one more revision.

Extract a Registry class to adhere to the Single Responsibility Principle.
Make methods only called inside your class private.
See if there are ways of simplifying your code in the spots I pointed out.

If you'd like to pair on this next week, I'd be glad to! Just let me know.

Are you referring to the private methods comment, or one of the others? I
think it would be great to pair next week and go over this together. Let me
know if you would like to do so.
On Fri, Aug 21, 2015 at 7:24 PM, Vaughn<EMAIL_ADDRESS>wrote:

Hey @paulghadad would you mind offering some more advice or a short
example of what you'd recommend here? New with Ruby still
—
Reply to this email directly or view it on GitHub
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-133595965
.

Yeah that would be great! Hands on run through would help out.  My week is free so if you want to set a date / time that works best for you invite me to an event. I'm Vaughn Bullock

How about Thursday afternoon around 2 or 3? Would you mind sending a calendar invite?

Hey Paul,
Sorry was OOF yesterday. I have sent the calendar invite to you for this
Thursday at 2pm.
Thanks!
On Mon, Aug 24, 2015 at 10:13 AM, Paul Haddad<EMAIL_ADDRESS>wrote:

How about Thursday afternoon around 2 or 3? Would you mind sending a
calendar invite?
—
Reply to this email directly or view it on GitHub
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134242373
.

That works! I don't believe I received the invite, though.
On Tue, Aug 25, 2015 at 10:08 AM, Vaughn<EMAIL_ADDRESS>wrote:

Hey Paul,
Sorry was OOF yesterday. I have sent the calendar invite to you for this
Thursday at 2pm.
Thanks!
On Mon, Aug 24, 2015 at 10:13 AM, Paul Haddad<EMAIL_ADDRESS>wrote:

How about Thursday afternoon around 2 or 3? Would you mind sending a
calendar invite?
—
Reply to this email directly or view it on GitHub
<
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134242373
.

—
Reply to this email directly or view it on GitHub
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134617118
.

Hey Paul, did you get the invite yet? I tried re-creating the event.
On Tue, Aug 25, 2015 at 10:58 AM, Paul Haddad<EMAIL_ADDRESS>wrote:

That works! I don't believe I received the invite, though.
On Tue, Aug 25, 2015 at 10:08 AM, Vaughn<EMAIL_ADDRESS>wrote:

Hey Paul,
Sorry was OOF yesterday. I have sent the calendar invite to you for this
Thursday at 2pm.
Thanks!
On Mon, Aug 24, 2015 at 10:13 AM, Paul Haddad<EMAIL_ADDRESS>wrote:

How about Thursday afternoon around 2 or 3? Would you mind sending a
calendar invite?
—
Reply to this email directly or view it on GitHub
<

https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134242373

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134617118
.

—
Reply to this email directly or view it on GitHub
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134647379
.

I have not...what's your Enova email? I can send one.

<EMAIL_ADDRESS>On Aug 25, 2015 3:15 PM, "Paul Haddad"<EMAIL_ADDRESS>wrote:

I have not...what's your Enova email? I can send one.
—
Reply to this email directly or view it on GitHub
https://github.com/jmmastey/level_up_exercises/pull/930#issuecomment-134724832
.


Feature Request: Configure Standby adapter on a team

I've tried looking through all the options and I don't see one that allows me to configure a 'standby adapter'.  If this isn't available could it possibly be added?
It can be done in PowerShell with...
Set-NetLbfoTeamMember -Name "NIC4" -AdministrativeMode Standby

Label this as an enhancement, and help wanted so anyone in the community can run with it.


Prevent migrations from executing on multiple nodes in a web farm

In web farm setups, data migrations should execute on just one node. This could be implemented using a simple lease implementation leveraging MessageBus and Redis, where the lease would be a key in CacheManager and invalidated using a distributed signal (MessageBus + Redis) and maybe a timeout as a fallback (IClock.When).

Don't we already have a similar mechanism (TaskLease) that could be utilized here? Or rewritten/updated to use Redis instead of the database?

My thought as well initially, but @sebastienros correctly pointed out, data migrations services live in Orchard.Framework while ITaskLeaseService lives in its own module. Also the implementation for this could be very simple where the data migration manager simply leverages ICacheManager to create a "lock" and releases it when its done. All we'd have to do next is enable distributed signals. This way it's also reusable for things other than migrations as you suggest.

I don't see how ICacheManager fits into the picture. You can't use that to create a lock unless it's distributed, the features for which also live in separate modules. And I don't see the point - TaskLease simply uses a database record, couldn't the migration manager do the same?

I suppose the Migration Manager could use database records too, but it can't use ITaskLeaseService for that since that interface does not live in Orchard.Framework while the data migration manager does. ICacheManager does live in the Framework so it could potentially be used (it's OK that the distributed features live in separate modules). But @DanielStolt makes a good point -how does creating a key in local memory allow you to use it as a lock that other nodes can monitor, since the key is local? Seems to me the key must be stored in a shared cache. If we would go the Redis cache route, I imagine the nodes would see if the cache contains a certain key. If not, it creates it and therefore has the lease. Other nodes checking for that key would know that a lease was given to someone else so they would not execute the migrations. To @DanielStolt 's point: why use Redis over a simple database record? Or perhaps the framework itself should not provide the implementation, just the contract. Which seems to be the point of ITaskLeaseService, so perhaps we should bring that thing into the framework.

Yeah, exactly - I didn't mean to use ITaskLeaseService I just meant to do what it does, i.e. use a database record for cross-farm critical section. IMO it's by far the simplest and most robust solution if all you want to do is lock. The database provides very robust ACID properties, a distributed cache solution does not.

Me neither - it was just an example. I'd rather go with some simple "lock provider service" that would sit in Orchard.Framework, with some simple implementation based on database. I totally agree that it's the simplest and the most robust solution. And maybe TaskLeaseService could be then refactored to use the new mechanism.
The only thing I'm concerned about is using database as a lock mechanism when Migration Manager runs DDL stuff.

DDL?

So maybe we could copy just the ITaskLeaseService interface to Orchard.Framework, which itself would provide an empty implementation. The actual implementation would still come from Orchard.TaskLease. This has the benefit that installations that don't require task lease won't have the TaskLeaseRecord table created and better matches Orchard's overall architectural design. I'll create a branch with some suggested changes as well as a data migration manager leveraging task lease which we can review.

TaskLease and its implementation is not useful in scenarios where a lease/lock shouldn't span multiple transactions (and even then its limited in use to scenarios where tasks won't try to start within the duration of the transaction acquiring the lease), see: https://github.com/OrchardCMS/Orchard/issues/4313 and https://github.com/OrchardCMS/Orchard/issues/3468.
Orchard.Caching (because it can supply a distributed implemenation) can be used for such server farm-wide locking (see a decorator for TaskLease): http://helpfullibraries.codeplex.com/SourceControl/latest#Libraries/Tasks/TaskLeaseServiceDecorator.cs As can Media files used for the same purpose: http://helpfullibraries.codeplex.com/SourceControl/latest#Libraries/Tasks/Locking/FileLock.cs
Since neither of these are (rightfully) in the Core I'd suggest adding a distributed/shared locking feature into the Framework with an implementation that's good enough for a web garden scenario (e.g. by using lock files in App_Data) and e.g. Orchard.Redis (or Orchard.Caching on a higher level) and Orchard.MessageBus could supply implementations usable in a web farm setup. The default implementation could even use IStorageProvider in the same way the linked services in Helpful Libraries does since that's a Framework interface and Media is the first thing you should think about when running on multiple servers (though, depending on the solution, it's not always usable for locking, but in this case you could use one of the other implementations).

The best solution here is to use a table to lock the migrations.

And use a separate transaction for handling that lock? Or ReadUncommitted isolation level?

To minimize the impact of row/table locking and issues that may come with this I'd go for a separate transaction in ReadUncommitted isolation level, yes.

I think you'll have to use ReadCommitted or higher for this to have the desired effect. ReadUncommitted won't suffice, because you're looking to mitigate the potential race condition that two or more nodes try to lock at exactly the same time. If you use a dedicated transaction for only the synchronization and a dedicated table for this and keep clear of any foreign key relationships etc, I don't think it will cause interfering locking issues with other stuff.

Here's an interesting Redis-based implementation we might steal:
https://github.com/schyntax/cs-schtick.redis


Feat/add sender name

What does this PR do?
Adds the ability to customize the sender name displayed in the inbox, instead of just showing the email address.
This feature is important because it allows users to personalize the sender name that appears in the inbox of the recipient. By default, most email clients display the sender's email address as the sender name, which can be less recognizable to the recipient.
Which issue(s) does this PR fix?
If this PR affects any API reference documentation, please share the updated endpoint references

@miqe released as part of 1.1.48


Functionality for users to drag/resize app modules

using a react-draggable package.
https://www.npmjs.com/package/react-draggable

Guess this was added back in December 2017? I should've closed out these issues awhile ago. Eep.


SLAM selection

Hello,
Thank you for sharing your appreciation for the work. I've followed many of your videos to compare SLAM algorithms and need your help in selecting a SLAM algorithm for my use case.  I have Livox Avia and Livox Mid 360, and we have experimented with the following SLAM algorithms. We need a SLAM algorithm for mapping small and large rooms. However, we are facing issues when generating point clouds for complex spaces that combine large areas, narrow paths, and small rooms.

FastLio: Works well in open areas but drifts in narrow paths and small rooms.
R3Live: FastLio performs better.
Immesh: FastLio is superior.
PvLio: Performs better than FastLio, but we are facing issues with Z drift. The whole point cloud bends over the length. It also requires adjusting the voxel size for different sizes of rooms.

I am considering trying two different approaches: one with a dual LiDAR setup (Avia + Mid and Mid + Mid) and the other with a Mid360 + external IMU. However, I'm not sure which IMU to try.  Could you please share your feedback on which IMU I should consider and what other factors I could explore?
Thank you very much.

The z drift is like this. The bend is more when we reduce the voxel size in PV LIO

@aditdoshi333 Hi. I guess you want to map the environments accurately without fast motions, right?
In that case, I think the built-in IMUs in LiDARs are enough. Anyway if you want to get expensive and good one, I recommend XSens IMUs. They have MTi-100, 300, 610, 630, etc. My lab members tried 100, 300, 630 for Visual Inertial Odometry methods and they were all good.
ImMesh uses VoxelMap for its odometry estimate and that is same for PV-LIO.
They estimate probabilistic planes and use them to estimate the odometry.
So more constraints, should be better performance, more computational resources are used.
But, more constraints do not always provide better performance, as you experienced already.
The matter is how good points are used to estimate odometry depending on the estimation methods they use.
You can refer these papers RMS and Quatro.
They both point that too many redundant points (or constraints) rather degrade the odometry estimate, as they include more outliers, noises, and worse localizable ones.
As the environments you are targeting include different sizes of spaces (large and small rooms), you can consider to try my lab's Ada-LIO. If you want, send me your datasets. I can test them and if you like the result.
Otherwise, you can develop your own algorithm based on Fast-LIO (I just recommend this algorithm for its compact structure and good performance with fast computations) that uses the good points only depending on the environments where the robots are located.

Thanks a lot for sharing your thoughts.
Please share your email address I would like to share our dataset with you.

@aditdoshi333 Hi. My email address is<EMAIL_ADDRESS>

Shared over mail


Add option: Adjust lightness of indistinguishable text

Description of the new feature/enhancement

ConEmu has an option  to:

‘Adjust lightness of indistinguishable text’ on the ‘Colors’ settings page. The idea is to make text readable if certain foreground and background colors are indistinguishable for current palette. ConEmu will try to change the lightness of the foreground text.

This is useful when two light colors are displayed with each other. Here is an example using Solarized Git color scheme with the feature off:

and with it on:

Proposed technical implementation details (optional)

It would be great to have a similar flag to set as an option in settings.json
ConEmu's method could be an inspiration. Here is the commit that added it.

Thanks for the request! This is tracked at /dup #2368 (which is also inspired by the ConEmu feature!)

I'm sorry, I meant #2638.

Awesome, I didn't quite find it. I'll check out that issue!


fix: Add backgroundColor type on CoreScaleOptions

Very similar to issue https://github.com/chartjs/Chart.js/issues/11082 with same fix, but in CartesianScaleOptions added to CoreScaleOptions so this isn't duplicated in various extensions of the CoreScaleOptions

Can you let me know if there is anything to further change here? Would love to have the corrected type merged :) Thanks!

MB for the late response, since its now in the core scale opts it can be removed from the RadialLinearScaleOptions at line 3436. After that it should be fine

I don't have merge permissions, so if you would like to merge this please do :)

@Mer-cat we'll merge once we release a 4.3.3 to fix a bug


Improve handling of duplicate products

As illustrated in #233, when duplicate products are flagged (i.e. produced with same reference frame) in some instances the longer of the two may be discarded, which introduces a gap in the path. The proposed improvements compare the duplicate files and pass the longer of the two.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- tools/ARIAtools/ARIAProduct.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- tools/ARIAtools/ARIAProduct.py  1

See the complete overview on Codacy

@dbekaert codacy was vague in regards to how the code I introduced was complex, it just reporting some percentage without suggestions as we see for other cases regarding duplication and pythonic conventions.
Perhaps it would be appropriate to address general codacy issues in a separate PR and push this bug-fix through?

The codacy issue is ok as it passed the PR standard for codacy. If its a severe issue it will report it.
To your question if we should in merge PR (in future) that fail Codacy and are bug fixes; would say no as the premise is for the PR to be up to coding standards we set before we merge it.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- tools/ARIAtools/ARIAProduct.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- tools/ARIAtools/ARIAProduct.py  1

See the complete overview on Codacy

To your question if we should in merge PR (in future) that fail Codacy and are bug fixes; would say no as the premise is for the PR to be up to coding standards we set before we merge it.

I agree going forward, but in this case it isn't transparent how complexity was added. I was trying to say for minor cases where codacy doesn't fail, we could address in a separate PR. Code could be cleaned up as we did with RAiDER

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- tools/ARIAtools/ARIAProduct.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- tools/ARIAtools/ARIAProduct.py  1

See the complete overview on Codacy

yes, agreed on the above; in case of sever codacy issues it would flag it. your PR still passed codacy, but had other issue on
GDAL.


Force NODE_PATH environment variable

Is it possible to do force NODE_PATH from atom's exports folder to a different folder?
I've tried using:
NODE_PATH="D:\WebDevTools\nodejs,D:\npm-modules\node_modules"
but it seems it gets rejected and NODE_PATH in the opened terminal still shows:
NODE_PATH=C:\Users\Gary\AppData\Local\atom\app-1.0.19\resources\app.asar\exports

I just researched a little more and found a related issue reported to Atom editor repo: https://github.com/atom/atom/issues/3311

@jagged3dge Where did you try changing the environment variable?

If you change the environmentVariables setting of this package to NODE_PATH=D:\WebDevTools\nodejs,ATOMERNIAL_FILE=$file,ATOMERNIAL_DIRECTORY=$directory,NODE_ENV= at least atoms value should be overriden. Unfortunatly you can not set environment variables to anything containing a comma right now.

This issue hasn't had any activity for a long time, closing.

Oops, did not realize this issue was still open. I've since moved on to VSCode as my editor, ditching Atom.
Regardless, thank you for your package and continued support :+1:


Error with access logging and unix socket

I run the server on unix socket with asyncio 0.22.5, python 3.5.2 and get the following error:
aiohttp.access ERROR 2016-08-16 13:46:34,653: Error in logging
Traceback (most recent call last):
  File "/var/www/polygon_venv/lib/python3.5/site-packages/aiohttp/helpers.py", line 405, in log
    [message, environ, response, transport, time]))
  File "/var/www/polygon_venv/lib/python3.5/site-packages/aiohttp/helpers.py", line 392, in _format_line
    return tuple(m(args) for m in self._methods)
  File "/var/www/polygon_venv/lib/python3.5/site-packages/aiohttp/helpers.py", line 392, in <genexpr>
    return tuple(m(args) for m in self._methods)
  File "/var/www/polygon_venv/lib/python3.5/site-packages/aiohttp/helpers.py", line 348, in _format_a
    return args[3].get_extra_info('peername')[0] if args[3] is not None \
IndexError: index out of range

With disabled access log or without unix socket problems not observed.

Duplicate of #816 (https://github.com/KeepSafe/aiohttp/issues/816#issuecomment-236359667)
fixed in master


Add Readme section for creating a new beamline: standards, best practices

Better documentation on creating a new dodal beamline: dummy devices, use of device_instantiation.
Should also update p38 and p45 and commit to keeping those beamlines updated to serve as examples.
Acceptance Criteria

A new beamline should be able to be created by following the documentation without any further assistance for an example beamline by its DAQ Support engineer without further support

Closed by #323, but always open to more improvements.


Merged Main for Auto Scroll PR

Description ✏
Just updated the branch with a merge from main.  This is the main to scroll branch only.  There are no other edits.
Checklist ✅

[ ] I have added screenshots (if UI changes are present).
[ ] I have done a self-review of my code.
[ ] I have manually tested my code (if applicable).

Yo there seems to be a bunch of unrelated changes on this PR 👀

I see we using #161

This was a PR to merge main into the scroll branch. Since I didn't author the PR for "Scroll," I couldn't keep it up to date with main. That was what this PR was for.


reconciliation wait, infoblox logout()

To prevent Infoblox connection will be repeatedly open I updated reconciliation queue.
When error in CreateZoneDelegationForExternalDNS occurs, the error is logged, but controller-runtime is
informed to start next loop after RECONCILE_REQUEUE_SECONDS. If I don't do it, and Infoblox connection fail, the controller runtime will schedule next loop ASAP.
Second change concerns implementation of standalone logout() which is now called after ObjectManager
makes all interaction with Infoblox. For that reason I also encapsulated infoblox client into standalone class.
Now logout is called explicitly in CreateZoneDelegationForExternalDNS and Finalize, see:
objMgr, err := p.infobloxClient.login()
if err != nil {
  return err
}
defer p.infobloxClient.logout()
...  // interaction with Infoblox

instead of
p. infobloxConnection() // defered logout() occured within infobloxConnection()
... // interaction with Infoblox

Signed-off-by: kuritka<EMAIL_ADDRESS>

#462


add kaggle starter

Added a notebook starter in kaggle environment.
https://www.kaggle.com/code/ipythonx/google-mediapipe-starter

Add a huggingface gradio web app : Hugginface Space

@sgowroji @sureshdagooglecom
Could you please take a look into this PR?


Circular flow type definitions trigger no-use-before-define

Brought here from https://github.com/eslint/eslint/issues/8703, by @soda0289 request:
When using core rule no-use-before-define, default for example in standard, it is applied to type definitions.
This could be a good thing, but the current implementation gives an unavoidable error on circular definitions:
/* @flow */
type File = {chunks: Array<Chunk>}
type Chunk = {file: File}

produces
.../src/commands.js
  2:21  error  'File' was used before it was defined  no-use-before-define

✖ 1 problem (1 error, 0 warnings)

OR
/* @flow */
type Chunk = {file: File}
type File = {chunks: Array<Chunk>}

produces
.../src/commands.js
  2:28  error  'Chunk' was used before it was defined  no-use-before-define

✖ 1 problem (1 error, 0 warnings)

respectively.
Currently, disabling no-use-before-define in the files that have flow enabled (all for me) is a fix when actual flow is checked, since that catches undefined.
However, it is still probably a bug, since:
// This is an actual error
const a = {b}
const b = {a}
// While this works and is often necessary
type A = {b:B}
type B = {a:A}

Had to revert this because this was causing many issues in other rules due to the AST change in https://github.com/babel/babel-eslint/pull/697, https://github.com/babel/babel-eslint/releases/tag/v10.0.1

Tip for folks putting types in a types/ (etc) specific directory, you might add an eslint config file to disable the rule for that directory:
// .eslintrc.js
module.exports = {
  'rules': {
    'no-use-before-define': 'off',
  },
};

Maybe add another option in config to allow type be ignored by this rule? Both flow and typescript.

Thank you for the issue. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


Values not set in StatsServerHandler

Describe the bug
It seems that func StatsServerHandler in main.go always return 0 for databasesConfigured, databasesMonitored and unreachableDBs. is it even called from anywhere? I don't see it being added to any server mux?
To Reproduce
Observation from code. Or am I missing something?
Expected behavior
StatsServerHandler added to server mux so it can serve HTTP calls and return correct values.

It's broken. Will be fixed later

Thanks. Looking for using pgwatch3 in our organization and this looked to be not implemented yet.
I also found these small problems:

In file src/db/sql/metric/README.md in row 2 file roll_out_*.psql is referenced, but I can't find it anywhere? Same with 00_schema_base.sql and 01_old_metrics_cleanup_procedures.sql.
In file docs/ENV_VARIABLES.md you state that environment variable PW3_PG_SCHEMA_TYPE can contain values metric-time or metric-dbname-time, but in file docker/bootstrap/init_dbs.sh you test this variable only for value timescale?

Should I create separate bugs for these, or those are known problems? 😄

Oh! Fair enough! Those are doc issues and should be fixed. I'd you can create separate issues that would be great! Thanks for your help.
PS. Pgwatch3 is in a dev cycle at the moment, so it better to try pgwatch2 in production meanwhile.

Thanks. Issues https://github.com/cybertec-postgresql/pgwatch3/issues/423 and https://github.com/cybertec-postgresql/pgwatch3/issues/424 created.
I know it is in dev cycle now - we are too, so I'm fine with that for now 😃. I want to use pgwatch3 (apart from Postgres monitoring) as a event store for Grafana. Is there a way to somehow provide timestamp of last event written to a metric for particular database - like give me all events newer than this timestamp? I was looking at how index change is implemented here, but that is not exactly what I need. I'm fine with developing this feature if you point me in right direction.


what's the meaning of test set, doesn't only dev set is okay?

what's the meaning of test set, doesn't only dev set is okay?
can someone explantation it? I will be very appreciate.

In general, we only want to use the dev set to tweak our model, and we
select the model that performs best on the dev set as our best model.
We use the test set to double check our best model, and report the
performance. In this way, we can avoid selecting and reporting a model that
just overfits on the test set.
For example, you can set test_prefix to None when you doing hyper
parameters search.
On Sun, Sep 24, 2017 at 2:58 AM zhangpeng<EMAIL_ADDRESS>wrote:

what's the meaning of test set, doesn't only dev set is okay?
can someone explantation it? I will be very appreciate.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/nmt/issues/124, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEZCMKtede2K3zNQectiBCHtzbnqieF9ks5slf1zgaJpZM4PhzE3
.

@oahziur thank you !


Add config to disable live-reload of pages

Users should be able to turn off the live-reload of pages.

fixed by #367


Bump kubevirtci

https://github.com/kubevirt/kubevirtci/pull/626 should fix k8s-1.21-sig-network (see PR for more details)
Changes:
f0f1952 Switch to booting via kernel and initrd from outside the image (https://github.com/kubevirt/kubevirtci/pull/593)
68abe5b cnao, provision, Bump k8s-1.19/1.20/1.21 cnao to v0.56.0 (https://github.com/kubevirt/kubevirtci/pull/626)
5760734 conformance tests: point to alternative registry (https://github.com/kubevirt/kubevirtci/pull/624)
Signed-off-by: Or Shoval<EMAIL_ADDRESS>None

/uncc @enp0s3 @rmohr

/test pull-kubevirt-e2e-k8s-1.21-sig-network

Will add the missing files, once https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/5923/pull-kubevirt-e2e-k8s-1.21-sig-network/1407951176879575040 done

/test pull-kubevirt-e2e-k8s-1.21-sig-network

@oshoval thanks, but we run a periodic two times a day which will create this kind of PR: https://github.com/kubevirt/kubevirt/pull/5926
I don't t hink that you have to create them manually.

https://prow.ci.kubevirt.io/?job=*kubevirtci-bump*

@oshoval thanks, but we run a periodic every day which will create this kind of PR: #5926
I don't t hink that you have to create them manually.

I understand, but i had to check that k8s-1.21-sig-network is fixed by that, so on the way did it
i can close it if needed once i finish to change, just saw on slack that sometimes if we do need something without waiting the next run (24 hours) its ok to do that

https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/5923/pull-kubevirt-e2e-k8s-1.21-sig-network/1407959931767754752
Passed (at least the ones that are not XFAIL)
Closing upon Roman request

Closing upon Roman request

I have no issues if you create your own PR. I just wanted to point out that it is there to make this work obsolete.


Wrong file naming example

In the section file naming you got some examples with different object types.
But in the section for page extension, you only show an example of an action instead of the page extension name.
Is this intended?
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/compliance/apptest-bestpracticesforalcode#page-extension

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3c7b7057-f9c5-6587-5d8b-267935685262
Version Independent ID: 11a70e81-f0a2-ec66-d63e-d1b0b5edeb5c
Content: Best Practices for AL code - Business Central
Content Source: dev-itpro/compliance/apptest-bestpracticesforalcode.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for catching this. This is confusing, and the header meant to say Action - there are plenty of examples of the object naming, which becomes quite repetitive, so we've fixed the header and moved the sections a bit to hopefully make it clearer. Live with the next doc refresh. Thanks.


Loading csv found from a github url (or uploaded from laptop) and showing as an asset

This issue is partially blocked by: https://github.com/DARPA-ASKEM/data-service/issues/198


🚀 [New Contribution]: The Machine Learning Techniques

Describe Your Contribution Here
Understand the concepts of Supervised, Unsupervised, and Reinforcement Learning Techniques in simple words.

Hey @PikuSpeaks123 this issue is a part of hacktoberfest 2022. The concepts are already explained just need to be added to the repository.
Please go through readme.md file to know more.

Btw @PikuSpeaks123 in my blog i did not explain indepth concepts of Machine Learning Techniques. I tried to keep it as simple as possible. If you are interested then you can explain the concepts such as Regression, Classification etc in depth instead of repeating the one already existing.
How does that sound?

Btw @PikuSpeaks123 in my blog i did not explain indepth concepts of Machine Learning Techniques. I tried to keep it as simple as possible. If you are interested then you can explain the concepts such as Regression, Classification etc in depth instead of repeating the one already existing. How does that sound?

You no longer need to create ISSUE/PR for the low-code and no-code contributions. Please check this communication here and follow the same(especially the ground rules). We will pick the contributions from there. Thanks!
https://www.showwcase.com/opensource


[meta] 2FA for gem pushers

The RubyGems account of one of the maintainers of rest-client was breached, enabling the attacker to push code with a backdoor. (details here: https://github.com/rest-client/rest-client/issues/713)
That's not the first time this kind of breach has happened.
Since there's no way of knowing if the authors of a given gem actually use 2FA, I'm opening this issue so we can keep an open discussion regarding this.
As a user of this gem, I would feel a lot safer if everyone with push rights had 2FA enabled.
Thanks!

Hi @lmansur,
Fair concern.
I do have 2FA enabled for RubyGems.org UI, RubyGems.org API and GitHub.

cc/ @frewsxcv @ktheory

@lmansur: Good call. I enabled 2FA for my rubygems account.

same here!

There are 2 more owners in RubyGems, tiegz and pje.


Upgrade Docker Container to OpenJDK 11

Some months ago, @juan-medina contributed in the project adding the Docker Support for testing.
Next September 2018, OpenJDK will release OpenJDK 11, the next LTS version of Java.
It is necessary to upgrade Dockerfile to OpenJDK 11:
# Pull base image.
FROM openjdk:11-jdk-slim

ADD app /app

WORKDIR /app

CMD ["bash"]

Testing in local, Gradle process failed.
* What went wrong:
Could not create service of type ScriptPluginFactory using BuildScopeServices.createScriptPluginFactory().
> Could not create service of type PluginResolutionStrategyInternal using BuildScopeServices.createPluginResolutionStrategy().

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 59s

> Task :dockerTest FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':dockerTest'.
> Process 'command 'docker'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 5.0.
See https://docs.gradle.org/4.7/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 3m 32s
8 actionable tasks: 8 executed

It is necessary to review in detail the results.

Fixed

Reorganize the project


(javascript) comment does not terminate at end of line

The version of highlight.js is 9.12.0, and the style is monokai-sublime
Here is my code written by markdown

And something seems wrong

By the way, ES2015(JavaScript) was not supported ? I feel the color is different from Sublime Text 3

Can you confirm this is still and issue and if so provide a jsfiddle case that reproduces it?
You can use this template:
https://jsfiddle.net/ajoshguy/nagkqytv/11/

Weekend ping.

Closing due to inactivity (we're cleaning out old stale issues that have likely been fixed).  If this is still a CURRENT issue please respond and we will happily re-open.


Checklist items go missing

I've been using version 1.1.6 since May or June with whatever the latest version of DAVDroid is, both from the F-droid repo, on Android 5.0.2. The checklist feature is convenient, but the items are not reliably synced. I discover checklist items missing or edits reverted so often that I've developed a nervous 'save' tic. Even then, I end up missing items when I go shopping because they mysteriously vanish at some point.
*constantly = today, twice last week, half-dozen times this month

I have still the same error. Almost always if I change, delete or add an item to an already existing checklist, after sync the changes are gone.

Yes, I have the same problems. I also loose all items when I change a list from local to a davdroid calendar.
CM13, opentasks 1.18.2 , davdroid 1.2

Maybe a problem with your backend? Which software do you use on the server? For me, ownCloud works fine.


Props components not working with default exports

Issues
https://github.com/pedronauck/docz/issues/717
https://github.com/pedronauck/docz/issues/758
Notes
Make sure you cleared cache before checking this. I had issues with babel cache (helped when I uninstall all node_modules and caches folders, and wrote yarn, yarn bs, yarn packages:build)

@lesha1201 I updated the docgen typescript algorithm to have some kinda a cache, because it was so slow and this was a big problem for large bundles. Please, resolve the conflict, then I can merge your pull request :pray:

@pedronauck Ok, I will do it tomorrow or day after tomorrow

@pedronauck But there is a problem, that was fixed in docgen.ts - several exports works incorrectly

@pedronauck This Doesn't resolve the path correctly when the folder structure is as follows.

src
| index.jsx     --- this file imports and exports all components

Component
| index.jsx      --- component file

I'm currently importing the Component via the parent index.jsx.
In this case, the metadata added should include the file name as **src/Component/index.jsx** But it actually has the value **src/index.jsx**.
I could have directly imported the component but have styles loaded from parent index.jsx file.


Display rendering problem suddenly appeared when launching gvim on Windows with right-click or macro

Using 32-bit gvim 8.1.1935 from github on an up-to-date Windows 10 box (v1903). I've also tried various other versions including 64-bit. I've been using gvim on this system for years without problem and just yesterday I started getting an unusable display when launching gvim from the right-click menu or via autohotkey macro, but when I launch gvim from the Start Menu or Run Command it always works fine. The screenshot shows typical behavior, which is 100% reproducible:

I'm also unable to use arrow keys or edit a file from this mode, because there is no way to see the text.  for redraw has no effect, nor does maximizing or resizing the window.
I've also tried removing my vimrc file just to make sure it isn't related to a local setting; this makes no difference.

Finally figured it out. In some invocations, term was getting set to win32. Letting term have its default value (builtin_gui in the case of vim) solved the display corruption. Not sure why the term setting was taking effect differently depending on the vim invocation, but getting rid of all term settings in all rc files solved the problem.


fix(ft/fennel): fix double semicolon commentstring

This PR fixes the double semicolons used to comment Fennel files.

Just curious, Is this a syntax error? Because even in the home page they are using double semicolon for comments.

It is not a syntax error, but despite the official Fennel home page, the common practice is to use only 1 semicolon. Also, the commentstring for Fennel uses 1 semicolon by default.
verbose set commentstring
"  commentstring=;%s
"        Last set from /usr/share/nvim/runtime/ftplugin/fennel.vim line 11

I was mistakenly referring to semicolon as "comma", sorry.

Fair enough! I don't have a strong opinion as I don't write fennel myself, so this is good to me :)


Is it possible to remove the default class of .pagination?

I'm removing the page numbers, and adding a class of .pager like this:
<%= will_paginate @posts,
  renderer: BootstrapPagination::Rails,
  page_links: false, class: :pager %>

However, because it's automatically adding the .pagination class as well, I'm getting some funkiness.
I don't have a live example, but here's how it looks with first .pager .pagination and second with just .pager

Thanks in advance :)

Is this still open?  I'm still getting two pagination classes.

@Aufree does your @comments model belongs_to another model like "Posts" or something? If so, this might be the cause of the issue. Would need to investigate after confirming. I bet that is what causes two paginations.


Return image asset on call

Previously, the application returned a page with the svg in it. This change will
cause it to return the asset instead.

LGTM.


How to enable prometheus metrics with the helm chart?

With stable/traefik ,  we can --set metrics.prometheus.enabled=true to enable prometheus metrics, but here I can't find the relevant option.

hello @katepangLiu
You can enable Prometheus metrics by passing the following parameter to Helm:
helm upgrade  --set="additionalArguments={--metrics.prometheus=true}" traefik traefik/traefik
You can also create a values.yaml file and add all argumentes e.g.
additionalArguments:
  - "--metrics.prometheus=true"

then just run:
helm upgrade traefik traefik/traefik -f values.yaml

I found this confusing as well.
Not finding anything about metrics in the Helm chart I looked up the environment variables as that seemed like the next level deeper for configuration ( https://doc.traefik.io/traefik/reference/static-configuration/env/ ) and set:
env:
 - name: TRAEFIK_METRICS_PROMETHEUS
   value: "true"

That had no effect.
Then looking at https://doc.traefik.io/traefik/observability/metrics/overview/ it seems I have to enable metrics itself with --metrics=true and then --metrics.prometheus=true, so I added
additionalArguments:
  - "--metrics=true"

That caused the container to crash loop since only enabling metrics without a defined exporter is apparently invalid. And the environment variable does nothing.
I then removed the env var and went for arguments instead:
additionalArguments:
  - "--metrics=true"
  - "--metrics.prometheus=true"

And it's working.
However after reading this thread I thought about trying with just
 additionalArguments:
  - "--metrics.prometheus=true"

And that also works!
I'm left with a feeling of confusion. Where and how is the correct way to configure Traefik deployed via Helm to K8s? How are arguments either depending, excluding or implying others? And what about the env vars?

@jakubhajek @StianOvrevage Thanks a lot,  It work for me.


worker_state is lost between map calls if the input is too large

This is very related to #15.
Since your awesome release v2.3.0 (which fix #15), I've been using mpire a lot, I love it :)

But I'm having a problem, very similar to #15.
In the following script, each worker get to deal with several numbers i, which they keep in state. Then I retrieve these values in another call.
from mpire import WorkerPool

N = 12
W = 4

def set_state(w_state, i):
    w_state[i] = 2 * i + 1
    return None

def get_state(w_state, i):
    return w_state[i]

if __name__ == "__main__":
    pool = WorkerPool(n_jobs=W, use_worker_state=True, keep_alive=True)
    s = N // W

    pool.map(set_state, list(range(N)), iterable_len=N, n_splits=s)
    r = pool.map(get_state, list(range(N)), iterable_len=N, n_splits=s)

    print(r)

If I run this script, everything works perfectly, I get my expected output :

[1, 3, 5, 7, 9, 11, 13, 15, 17, 19, 21, 23]

Now, if I change N (the number of tasks) to a higher number (like 128) and I run the script again, I get the following error :

KeyError: '\n\nException occurred in Worker-2 with the following arguments:\nArg 0: 32\nTraceback (most recent call last):\n  File "/root/miniconda3/envs/housing_sb3/lib/python3.8/site-packages/mpire/worker.py", line 322, in _run_safely\n    results = func()\n  File "/root/miniconda3/envs/housing_sb3/lib/python3.8/site-packages/mpire/worker.py", line 276, in _func\n    results = func(args)\n  File "/root/miniconda3/envs/housing_sb3/lib/python3.8/site-packages/mpire/worker.py", line 415, in _helper_func_with_idx\n    return args[0], self._call_func(func, args[1])\n  File "/root/miniconda3/envs/housing_sb3/lib/python3.8/site-packages/mpire/worker.py", line 442, in _call_func\n    return func(args)\n  File "housing_drl/sb3/swag.py", line 14, in get_state\n    return w_state[i]\nKeyError: 32\n'

It's the exact same error as in #15, so it seems the worker state is somehow erased ?

@sybrenjansen Do you have any idea what's the problem ? Did I do something wrong in my script ?

I just noticed that if I set the number of worker (W) to a higher number (say, 40), it produces the right output.
I don't know what's going on behind the scenes, but this seems more like a dirty-workaround.

I'd like to be able to process my 128 tasks with a specific number of workers (like 4) to not engorge my CPUs.

Updated example, working perfectly with multiprocessing.Barrier :
from mpire import WorkerPool
from multiprocessing import Barrier

N = 1024
W = 4
B = Barrier(W)

def set_state(w_state, i):
    w_state[i] = 2 * i + 1
    B.wait()
    return None

def get_state(w_state, i):
    B.wait()
    return w_state[i]

if __name__ == "__main__":
    pool = WorkerPool(n_jobs=W, use_worker_state=True, keep_alive=True)

    pool.map(set_state, list(range(N)), iterable_len=N, n_splits=W)
    r = pool.map(get_state, list(range(N)), iterable_len=N, n_splits=W)

    print(r)

Thanks again @sybrenjansen for the super clear explanation 🙏

Actually, let me re-open this issue...
Barrier is working fine, but only if the number of tasks is "round".
If the number of tasks is not "round", then each worker will have to deal with a different number of tasks. In that case the code hangs (processes are waiting for W workers to be at the barrier, but only a subset are there).

Ok, so I created a working example, by using 2 different barriers : One for the general case, and one for the last indexes (which might not be round).
I wrapped everything in a new class so it's easier to use :
from mpire import WorkerPool
from mpire.utils import chunk_tasks
from multiprocessing import Barrier

N = 1025
W = 4

class AdaptativeBarrier:
    def __init__(self, n_workers, n_tasks):
        self.general_barrier = Barrier(n_workers)
        self.last_barrier = Barrier(n_tasks % n_workers)

        # Compute the indexes of the last pass
        chunks = chunk_tasks(range(n_tasks), n_splits=n_workers)
        n = n_tasks // n_workers
        self.last_idx = [c[-1] for c in chunks if len(c) > n]

    def wait(self, i):
        if i in self.last_idx:
            return self.last_barrier.wait()
        else:
            return self.general_barrier.wait()

B = AdaptativeBarrier(W, N)

def set_state(w_state, i):
    w_state[i] = 2 * i + 1
    B.wait(i)
    return None

def get_state(w_state, i):
    B.wait(i)
    return w_state[i]

if __name__ == "__main__":
    pool = WorkerPool(n_jobs=W, use_worker_state=True, keep_alive=True)

    pool.map(set_state, list(range(N)), iterable_len=N, n_splits=W)
    r = pool.map(get_state, list(range(N)), iterable_len=N, n_splits=W)

    print(r)

I'll keep this issue open, as I think having an additional flag in mpire could be easier from a user perspective.
But feel free to close @sybrenjansen !

Let's keep it open.
Like I said, it's not that much work and I do think it can be a useful feature for others as well. I think I can find some free time next week or so.

Had a bit less free time last few weeks than I thought. This is just a reminder that I didn't forgot about it, it's still on my to-do list

Now available in v2.5.0


could you release the code to extract optical flow?

do I need to normalize RGB images to [0,1] from [0,255] before calculate optical flow?
I will follow the code here. https://github.com/opencv/opencv/blob/master/samples/gpu/optical_flow.cpp
And 'd_flow' is only output of TVL1.
how to split dx_flow and dy_flow?

Hi,
from the person who wrapped TVL-1 at google:
(...) "we truncate large motions at +/- 20 pixels along each dimension and then quantize values to 256 buckets using a linear map from [-20, 20] to [0, 255]."

convert_to_flow.ipynb.zip
Hi, this is the .ipynb file to extract the flows from the images. This file extract images from videos and then convert frames to optical flow files.


Error load javascript

Type: Bug
Not link function to file  create function
VS Code version: Code 1.86.0 (05047486b6df5eb8d44b2ecd70ea3bdf775fd937, 2024-01-31T10:28:19.990Z)
OS version: Windows_NT x64 10.0.19041
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4600U CPU @ 2.10GHz (4 x 2694)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.91GB (6.11GB free)

Process Argv
--crash-reporter-id b84c5101-f453-4e9c-9748-a1b0907d3401

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

prettier-standard-vscode
num
1.0.2

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30951796
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
accentitlementst:30887150
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
dsvsc019b:30953937
3ef8e399:30949928

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

/needsMoreInfo
Please provide full steps to reproduce problem. Is it a problem with VS Code editor itself or when you try running code outside of editor?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Added option to skip ssl cert check plus skipping on IndexError when there is no disk details

Minor changes needed to make the cml cli working with vCenter using self-signed ssl certificate. There is also a quick fix of a minor issue with VM details display when no disk data is retrieved from vCenter.

Thanks for taking the time to update with ssl cert and disk try.


Query for Midburn2017 all camps, and names with their members count , and tickets

The query should be:
camp_name_he
camp_name_en
camp_manager_id ,
profile emai of camp_manager_id,
profile name of camp_manager_id
total approved camp members
total approved camp members with MIDBURN2017 ticket
total approved camp members that was linked in the gate from camp

@asihud , I'm on it

"total approved camp members that was linked in the gate from camp"
see entrance_group_id in tickets table

Data to Adir Ron,
I can't link it here since it's not anonymized.
Queries:
select
    id,
    camp_name_he,
    camp_name_en,
    main_contact,
    email,
    first_name,
    last_name
from
    spark.camps as c
left join spark.users as u
    on c.main_contact = u.user_id

select
    id,
    camp_name_he,
    camp_name_en,
    count(id) as num_of_approved_members
from
    spark.camps as c
left join spark.camp_members as m
    on c.id = m.camp_id
where
    (m.status = "approved" or m.status = "approved_mgr")
group by id;

select
    id,
    camp_name_he,
    camp_name_en,
    count(id) as num_of_approved_members_with_midburn_ticket
from
    spark.camps as c
left join spark.camp_members as m
    on c.id = m.camp_id
left join spark.tickets as t
    on m.user_id = t.holder_id
where
    ((t.event_id = "MIDBURN2017") and (m.status = "approved" or m.status = "approved_mgr"))
group by camp_id;


Please wrap the text file with a normal code editor

I often want to add some minor changes to a website that's already been deployed using this awesome service.
Anyway, it would be nice if we can add a code editor like jQ Ace or the zillion others out there to the  that uses to edit the text.
I'm sure there are open-source twitter-bootstrap compatible code-editors we can find.
I'd add a pull myself if I'd know what plugin you'd prefer using.

Current editor is definitely super primitive, and it could make sense to add Ace support.
Do note that you can also enable VS Online "Monaco" on your site, which gives you a super fancy in-browser editor.

I'm sure there are better bootsrap-compliant editors out there.
Never heard of Monaco, will check it out.

See for info: http://channel9.msdn.com/Series/Visual-Studio-Online-Monaco

We actually have this now in e3ce7a165d793348f4f4e37c0059f47ddccaacae thanks to @snobu


Please consider using SemVer

I know that SemVer allows to change the API if the major version number is 0, but this is like not using SemVer at all.
I prefer to use rails_admin 44.0.0 instead of having a Bootstrap 2 to Bootstrap 3 switch between 0.6.5 and 0.6.6
Please consider to bump the major version number if there is a potential breaking change, so we can immediately check if a gem needs attention
Thanks!

Thanks for your advice, we're moving toward 1.0.0 now.

Reopening, because 1.0.0 release will take some more time...

Closing because we're on 1.0.0 now!


Support for HTTP Proxy

Currently the connection to the Github API doesn't seem to use the HTTP Proxy that is configured in Nexus in System -> HTTP . Maybe this can be fixed as part of: https://github.com/larscheid-schmitzhermes/nexus3-github-oauth-plugin/issues/27 .

Even I tried to setup the proxy config inside nexus.vmoptions  (Which nexus suggest) still no luck.


x/pkgsite: shortcut redirect fails if a trailing slash is added

pkg.go.dev supports shortcuts for packages from the standard library (for example, https://pkg.go.dev/http redirects to https://pkg.go.dev/net/http). However, these don't work if a trailing / is appended to the URL.
Opening https://pkg.go.dev/http/ results in a redirect to https://pkg.go.dev/http/net/http (and not https://pkg.go.dev/net/http).
Below are the request and the server's response (shortened):
> GET /http/ HTTP/2
> Host: pkg.go.dev
> user-agent: curl/7.71.0
> accept: */*
>
< HTTP/2 302
[...]
< location: /http/net/http

In the case of https://github.com/golang/pkgsite/blob/master/internal/frontend/details.go#L432, the url parameter to http.Redirect() is a relative path. Therefore, the (full) redirect path is constructed by appending the relative URL to the current location (https://golang.org/src/net/http/server.go#L2082).
I have a CL ready but it is based on a fix for golang/go#39663.
Related: golang/go#39663.

@julieqiu

Thanks @akolar! #39663 has been fixed. Feel free to rebase, and I'll review https://golang.org/cl/242138.


require should not enforce prefix to load from file

Currently, the require method enforces (line 115 of Module.class) that all modules loaded from the filesystem must be prefixed with '/', '../' or './'.
This makes it difficult to import existing modules that in turn require other modules that do not include a prefix.
Can you just remove this check and leave it up to the module loader (i.e. Folder implementation) to determine whether to load from filesystem or not?

As far as I know the algorithm is supposed to mimic Node's behavior, as described here: https://nodejs.org/api/modules.html
You can see that it explicitly looks for a prefix before loading anything from the filesystem. Non-prefixed modules are always loaded from the node_modules folder.
This being said, maybe there is something incorrect in the code. Can you provide a sample that works in Node and breaks with this library?

If you've just followed the spec then i guess thats right approach.
I had my modules stored under a different path (not node_modules), but as mentioned, this meant modules failed to load dependencies as they are typically referenced without a path prefix.
I've now restructured things to use a node_modules directory instead which resolves the issue.
Thanks for your help :)


Normalisation of \discretionary

At present, the extra \discretionary lines added to the log by LuaTeX are normalised out in a rather hackish way by l3build: the line containing \discretionary and the following one are killed, but that may or may not be an 'extra'. A cleaner way, which will target purely LuaTeX, is to set
\exhyphenchar=-1 %

at the TeX end in a .cfg file. This would mean things are not 100% automatic, but may be better for cross-engine testing (as it can be turned off for LuaTeX-specific tests in this area).

My mistake: that would only help with explicit hyphen chars :-(


Rework OpEthApi type

Describe the feature
OpEthApi should wrap EthApiInner<...>, just like EthApi struct does, instead of the generic Eth. Calling Eth::some_trait_method in implementation of EthApiServer helper traits is bug prone, since any default trait method overrides for OpEthApi, won't be considered if called inside scope of Eth::some_trait_method.
Additional context
No response

:shipit:

@emhane, OpEthApi would need to add generics to all that implement it.

yeah that's right @mvares

for OP I can use FullNodeComponents though to unify the generics Provider, Pool, Evm, we just didn't make that change for EthApi because it would require a lot of work fixing existing tests

right… thanks for explanation


Elasticsearch: Playground app testing

Description
Set up a playground app using elasticsearch to use for manual testing of the new instrumentation. This should use a variety of types of requests through elasticsearch in order to have a robust understanding of any differences seen between different requests.
Different playgrounds:

[ ] Elasticsearch 8 with security
[X] Elasticsearch 8 without security (Available using docker-compose and running multiverse)
[X] Elasticsearch 7 without security (Songs API, docker-compose + multiverse)

Estimates
Please provide initial t-shirt size. S = 1-3 days,  M = 3-5 days (1 week), L = 1-2 weeks (1 sprint)
S

The songs_api playground app is working well on Elasticsearch 7.x with security disabled

Elasticsearch 8 with security app is in progress here: https://source.datanerd.us/ruby-agent/ruby_agent_playground/pull/new/elasticsearch-8-secure (Internal)


Fixed deadlock in CRYPTO_THREAD_run_once for Windows platform.

Fixed deadlock in CRYPTO_THREAD_run_once() if call to init() is causing a recursive call to CRYPTO_THREAD_run_once() again that is causing a hot deadloop inside do { } while (result == ONCE_ININIT); section.
Example of such call stack:
test.exe!CRYPTO_THREAD_run_once(long * once, void()() init) Line 79	C
test.exe!OPENSSL_init_crypto(unsigned _int64 opts, const ossl_init_settings_st * settings) Line 506	C
test.exe!ERR_get_state() Line 662	C
test.exe!ERR_put_error(int lib, int func, int reason, const char * file, int line) Line 358	C
test.exe!DSO_pathbyaddr(void * addr, char * path, int sz) Line 315	C
test.exe!DSO_dsobyaddr(void * addr, int flags) Line 325	C
test.exe!ossl_init_base() Line 105	C
test.exe!ossl_init_base_ossl() Line 66	C
test.exe!CRYPTO_THREAD_run_once(long * once, void()() init) Line 82	C
test.exe!OPENSSL_init_crypto(unsigned __int64 opts, const ossl_init_settings_st * settings) Line 506	C
test.exe!ENGINE_load_builtin_engines() Line 18	C
Here DSO_pathbyaddr() caused an error log message via ERR_put_error() which called OPENSSL_init_crypto() recursively via ERR_get_state().
Setting *lock = ONCE_DONE before init() effectively fixes this issue by causing true for if (*lock == ONCE_DONE) condition on recursive re-entry during init() call.

+1 from me.  is this trivial, or will you sign our CLA?

richsalz
is this trivial, or will you sign our CLA?

The bug-fix is trivial, I believe CLA would be an overkill for this case.

349d1cf on master, edc1874 on 1.1.0; thanks!

This fix is wrong and should be reverted. This marks the "run_once" code as having finished before it has. The intended semantics of run_once is that no threads should proceed until the code has run exactly once. With this change the "second" thread will think the run_once code has already been run and will continue, even though it is still in progress. This could result in a crash or other incorrect behaviour.

+1 to revert

Thanks @t-j-h. Reverted.
This issue is a dupe of #1899, so we should track progress for a proper fix there. Re-closing this.


Benefit from this package without using AzureInsights? (Offline Use within docker-compose?)

Hi,
I wonder can I already benefit from this package without using AzureInsights? I.e. offline when testing outside of azure. E.g. in docker-compose? Are there tools to easily evaluate and possibly visualize the results offline?

You could add a console exporter for each signal so that the telemetry is outputted directly to you. For instance, to export traces to console, you can call:
configure_azure_monitor()
trace_exporter = ConsoleSpanExporter()
span_processor = BatchSpanProcessor(
trace_exporter,
)
get_tracer_provider().add_span_processor(span_processor)

@andife
Is your issue resolved?

Thanks for asking. The theme is taking a bit of a break right now, which means I honestly haven't gotten around to evaluating it yet, but I hope to very soon.

Will be closing for now. Feel free to open back up if you want to revisit.


23w03a/1.19.4: Translation fallback field

In the newest Snapshot, Mojang introduced a change to translate components, a fallback field which contains a string to be used in the event that a translation key can't be found.
Full information can be found here:

Added an optional fallback field to translate text components.

The new field is a string that will be used in place of translation if it is missing
If fallback is missing, the old behavior (i.e. using the key itself as the translation) is preserved

Thanks!

Fixed in #868


Get Data object by UDP

In the streaming script, using the UDP protocol, how do I parse the message to obtain the same Data object as with TCP?
In this code in Python example: https://github.com/SICKAG/sick_visionary_samples/tree/main/image_streaming_and_storing#receive-frames-and-convert-to-point-cloud.
How do I create a point cloud or obtain an array of distances, intensities, and camera parameters using UDP?
Thanks for the help

We have seen your question and will get back to you soon.

Hello @xaimepardal ,
we are currently working on an implementation that provides the requested functionality also for UDP.
This update of our samples will be made for both Pyhon and C++.

@zimmefaSICKAG Any news regarding this issue?

Release is in preparation. Expected end of this week latest next week.


Can not get it to run

I get this error
`var io = require('socket.io.').listen(server, { log: false });
^
TypeError: require(...).listen is not a function
`

¿Did you install the correct dependencies versions?

i think so...
I rolled back socket.io to the version you have in the code. but still getting errors:
`/WEB/mmo/node_modules/socket.io/lib/store.js:35
Store.prototype.proto = EventEmitter.prototype;
^
TypeError: Cannot read property 'prototype' of undefined
at Object. (/WEB/mmo/node_modules/socket.io/lib/store.js:35:42)
at Module._compile (internal/modules/cjs/loader.js:1072:14)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1101:10)
at Module.load (internal/modules/cjs/loader.js:937:32)
at Function.Module._load (internal/modules/cjs/loader.js:778:12)
at Module.require (internal/modules/cjs/loader.js:961:19)
at require (internal/modules/cjs/helpers.js:92:18)
at Object. (/WEB/mmo/node_modules/socket.io/lib/manager.js:16:13)
at Module._compile (internal/modules/cjs/loader.js:1072:14)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1101:10)
`
This game you have written sounds really intruiging and I want to check it out. Making games without canvas is fascinating to me haha!
I'll keep poking at it when I get the time and see what I can figure out.

Okay figured it out.

installed the dependencies listed on the package.json

fixed the errors by switching every instance of:
var EventEmitter = process. EventEmitter

To:
var EventEmitter = require('events')
After that it worked! Im going to poke around your code and see how you made this game because the map without a canvas is interesting to me.

Thanks @funjspi i will correct it later


[Azure-PG-Flex] - Typo

#Sign-off

Learn Build status updates of commit 6f724a3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/postgresql/flexible-server/concepts-pgbouncer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Looks good; thanks, @varun-dhawan


Can´t send Direkt Messages

Hello
I can´t send Direct Messages from within Mikutter, except to me.
If I open the Profile of a Twitter user go to the Direct Message Tab, enter a Message and push send it won´t get send but a new message bar is created and the first keeps displayed the only way to go back to normal is to close Mikutter and start it anew.
I am using Mikutter 3.5.9-7-g6a94e69 in Kubuntu 16.04 LTS
Thanks,

Will there a version out in the future with which I can send Direct Messages?
The Version comming out tonight 3.5.12 still can not do it.
Thanks,

Could it be possible that Direct messages could only be send to followers?
If that is so There should be an error "The Recipient is not following you." or so instead of Quietly not sending and hanging the Editor. Or to send it Quietly as normal message such as for example Twidere seems to do.
Thanks,

I think two reason.

Invalid permission for Twitter access token
Twitter has 3 access level:

read
read&write
read&write&DM

If your first mikutter is old token, you can't send/receive DM.
We have workarounds. try:

open Preference

Choose your account.
Click "Delete" in right button. (It will remove register in mikutter.)
Click "Add".
Register your account again.

It will modify Twitter access token. retry send DMs.

Third-party plugin (Environment)
mikutter is extensible. But It causes strange side effect in rarely.
If you can't fix this issue in above, the problem is rarely and difficulty.

I´ve tried both but it won´t help.
Hope this won´t get forgotten.
Have you gotten DirectMessages from me in the past?
I´ve send you one by Twidere but this shows only as normal message.
Could you follow me to try if that is the Problem?
Thanks,

I don't follow you and don't accept direct message out of followee by twitter settings. It will deny send direct message.
Can you send another account that mutually following or yourself?

I´ve send you one by Twidere but this shows only as normal message.

I don't know Twidere. but it looks wrong behavior. You should use official client or web for inspect this issue.

Hello,
that explains much.
Than the behaviour is quite normal.
nonetheless it would be better if there would be a message that tje message could not be send, if it is possible with an explanation such as you cannnot send a Direct Message to this Person or so.
With the following me was only a Question for testing purposes nothing more, sorry if I annoyed you.
Twidere is an openSource Twitter Client for Android.
With the behaviour of that Programm in that sense you are perfectly right I think.
The official Client is such an ugly and hard to use part of Software that I don´t want to use it.
Thanks,


Add support to require local modules as an addition to installed pack…

Description
When running CLI commands it might be necessary to require local modules, that are, then, relative to the working directory.
Normally using -r argument with node cli would correctly load installed/external modules (from node_modules folder) as well as local modules when passing the path to a local js file.
However, currently Mesh CLI does not support this, and so, passing the path to a local module, like -r ./scripts/myCustomModule.js doesn’t work.
The issue is caused by the fact that the import is requested within the mesh cli package, which is in myProject/node_modules/@graphql-mesh/cli/.
In this case, the following require doesn't work:
-r ./scripts/myCustomModule.js
but this does work:
-r ../../../scripts/myCustomModule.js
This PR addresses this by checking if the passed argument value (module name or local path) can be found in a local file relative to the working directory; if so it will import the local path, otherwise continue with the existing behavior which is to import the passed argument value directory (most likely the name of an installed package/module)
Type of change

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

How Has This Been Tested?
Added 2 require arguments on a CLI command, one using the path to a local module and one using the name of an installed module; both are working as expected.
Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] New and existing unit tests and linter rules pass locally with my changes

Thank you @santino <3