Invoking the plugin from a declarative pipeline script

We're using a declarative pipeline script to execute our build, and I'm unsure of how to invoke this plugin to gather and publish the tests.
From the documentation, it appears ant is required for invocation but goes no further, and says nothing about "publishing" the results themselves.
Thank you!

Hi @cspang1 , I haven't really used the plugin in declarative pipeline scripts, only freestyle ones. Will have a look next development cycle, but maybe pinging the users mailing list someone else will know how to use it.

@kinow Thanks for the quick response! I was actually able to figure it out:
It's as simple as using
step([$class: 'TapPublisher', testResults: '**/*.tap'])

in a scripted pipeline, and wrap it in script { } in a declarative pipeline. Some additional options:
step([$class: 'TapPublisher', discardOldReports: false, enableSubtests: false, failIfNoResults: false,
    failedTestsMarkBuildAsFailure: true, flattenTapResult: false, includeCommentDiagnostics: false,
    outputTapToConsole: true, planRequired: true, showOnlyFailures: false, skipIfBuildNotOk: false,
    stripSingleParents: false, testResults: '**/*.tap', todoIsFailure: false, validateNumberOfTests: true,
    verbose: true])


Session storage doesn't save values after navigation when a role is used

What is your Test Scenario?
Retrieve a value from the session storage after a page navigation when using a role.
What is the Current behavior?
The session storage is empty after a page navigation when a role is used.
What is the Expected behavior?
The session storage contains previously stored values after a page navigation when a role is used.
What is your web application and your TestCafe test code?

Your website URL (or attach your complete example):
http://devexpress.github.io/testcafe/example/

Your complete test code (or attach your test files):

import Page from './page-model';

import { Role } from 'testcafe';

// Page model
const page = new Page();

const role = new Role('http://devexpress.github.io/testcafe/example/', () => {});

fixture `A set of examples that illustrate how to use TestCafe API`
    .page `https://devexpress.github.io/testcafe/example/`
    .beforeEach(t => t.useRole(role));

// Tests
test('Handle native confirmation dialog', async t => {
    await t
        .setNativeDialogHandler(() => true)
        .click(page.populateButton);

    const dialogHistory = await t.getNativeDialogHistory();

    await t.expect(dialogHistory[0].text).eql('Reset information before proceeding?');

    await t
        .click(page.submitButton)
        .expect(page.results.innerText).contains('Peter Parker');
});

Steps to Reproduce:
Run the test
Your Environment details:

testcafe version:                   0.23.3-alpha.4
node.js version:                    any
command-line arguments:  N/A
browser name and version: Chrome 71
platform and version:          Windows 10

I maintain a fairly large Testcafe suite on my current project, and we depend heavily on role-switching. I've been struggling with what appears to be this issue since 0.23.2 was released. We're going to pin 0.23.1 in our project for now. Definitely looking forward to the fix, thank you for your work on it!

Hi @morganedmonds, this issue can be reproduced in 0.23.1 as well. Could you please create a new issue and provide the information that I can use to reproduce the problem: an example project, your test files, the page URL or source code?
You could also try<EMAIL_ADDRESS>we've fixed a lot of problem in this version.

Hi @AndreyBelym, yes we figured that out. I was able to work around the issue by clicking on a link to the URL rather than navigating to it directly. I'm afraid I don't have time to put together a sample project in the next couple of weeks with the holidays, and I can't share our app code.  Thank you for following up!

After examing the problem, I consider this issue as a duplicate of issue #2195. A Role should reload the test page, but it doesn't happen in some situations. The workaround is to manually reload the page after using a Role, e.g.:
const reloadPage = ClientFunction(() => location.reload());

await t.useRole(role);

await reloadPage();


1.16.2/indrev-1.5.1-BETA Modular Workbench progression inaccurate

The progression bar in the modular workbench isn't accurate to the percentage of the progression.

I'll look into it and try to fix for the next update, thanks for the report! Also, weird note, those texts are meant to be very colorful... Don't know why it's all white there.

Could there be another mod interfering?

I just probably made a mistake somewhere

Let me know if I can help testing.  I'm currently hosting an experimental 1.16.2 dedicated server for this purpose.

Oh, good to know, this will be helpful to test the next update since it adds a bunch of things (and probably 2x bugs) if the server is still up by then!

It will be!  I'm also open for more players.  We're currently with 5 people, so it's hard to get everything tested.  Though I'm sure the flood of new releases will slow done in a while.  Right now AE2 and its mod interactions are the big thing to test.

This should be fixed as of 1.6.0-BETA which you can download https://www.curseforge.com/minecraft/mc-mods/industrial-revolution/files/3043918. Thanks for the report!


auto alarm atop

hi
is thare a way to auto stop the alarm like after a minut

You can do that by manually call the clearAlarm method after your desired time, e.g. one minute.
Just set a Timeout in the alarm function to clear alarm afterwards. like
setTimeout(function(){
//do your stuff here
$.fn.thooClock.clearAlarm
}, 60000);


Removed Az-copy option from migration techniques for Table API

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

@markjbrown
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

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

#sign-off

@richagaur: I'm sorry - only the author of this article, @rothja, @markjbrown, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


cmake .. returns error of not found libraries

After succesfull tests it fc_blas/fc_lapack  the cmake .. returns error of not found libraries:
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/.py.test -v test/test.py
============================================================== test session starts ===============================================================
platform linux2 -- Python 2.6.6 -- py-1.4.28 -- pytest-2.7.1 -- /usr/bin/python
rootdir: /home/milias/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test, inifile:
collected 14 items

test/test.py::test_cxx_custom PASSED
test/test.py::test_extra_cmake_options PASSED
test/test.py::test_cxx PASSED
test/test.py::test_cxx_static PASSED
test/test.py::test_fc PASSED
test/test.py::test_fc_git_info PASSED
test/test.py::test_fc_int64 PASSED
test/test.py::test_fc_mpi PASSED
test/test.py::test_fc_omp PASSED
test/test.py::test_fc_static PASSED
test/test.py::test_fc_blas PASSED
test/test.py::test_fc_blas_static PASSED
test/test.py::test_fc_lapack PASSED
test/test.py::test_fc_lapack_static PASSED

=========================================================== 14 passed in 70.00 seconds ===========================================================
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/.cd test/fc_blas/
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/.ls
CMakeLists.txt  build-2015-08-10-17-26-38/  build-2015-08-10-17-26-43/  cmake/  setup.py*  src/
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/.cd build-2015-08-10-17-26-43/
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build-2015-08-10-17-26-43/.m
[100%] Built target example
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build-2015-08-10-17-26-43/.cmake ..
CMake Error at src/CMakeLists.txt:16 (message):
  BLAS library not found for the test fc_blas!

-- Configuring incomplete, errors occurred!
See also "/home/milias/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build-2015-08-10-17-26-43/CMakeFiles/CMakeOutput.log".
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build-2015-08-10-17-26-43/.

Please try to create a minimal example to reproduce it. I don't think this problem has to do with test script.
The test script executes the setup script. Try to do this directly. Try to minimize/localize the problem first.

In the fc_blas directory run the setup script, do make and afterward do cmake ..:
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/../setup.py --fc=gfortran --cmake-options="-DMATH_LIB_SEARCH_ORDER=SYSTEM_NATIVE" build_gfortran_sysnative
FC=gfortran cmake -DEXTRA_FCFLAGS="''" -DENABLE_STATIC_LINKING=False -DENABLE_64BIT_INTEGERS=False -DENABLE_BLAS=auto -DENABLE_LAPACK=auto -DMKL_FLAG=off -DMATH_LIB_SEARCH_ORDER="MKL;ESSL;ATLAS;ACML;SYSTEM_NATIVE" -DBLAS_LANG=Fortran -DLAPACK_LANG=Fortran -DCMAKE_BUILD_TYPE=release -G "Unix Makefiles" -DMATH_LIB_SEARCH_ORDER=SYSTEM_NATIVE /home/milias/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas

-- The C compiler identification is GNU 4.4.7
-- The CXX compiler identification is GNU 4.4.7
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- The Fortran compiler identification is GNU
-- Check for working Fortran compiler: /usr/bin/gfortran
-- Check for working Fortran compiler: /usr/bin/gfortran  -- works
-- Detecting Fortran compiler ABI info
-- Detecting Fortran compiler ABI info - done
-- Checking whether /usr/bin/gfortran supports Fortran 90
-- Checking whether /usr/bin/gfortran supports Fortran 90 -- yes
-- BLAS will be searched for based on MKLROOT=/mnt/apps/intel/composer_xe_2013_sp1.1.106/mkl
-- Searching for BLAS using search order SYSTEM_NATIVE
-- Found BLAS: SYSTEM_NATIVE (/usr/lib64/libblas.so)
-- LAPACK will be searched for based on MKLROOT=/mnt/apps/intel/composer_xe_2013_sp1.1.106/mkl
-- Searching for LAPACK using search order SYSTEM_NATIVE
-- Found LAPACK: SYSTEM_NATIVE (/usr/lib64/liblapack.so)
-- Configuring done
-- Generating done
-- Build files have been written to: /home/milias/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build_gfortran_sysnative

   configure step is done
   now you need to compile the sources:
   $ cd build_gfortran_sysnative
   $ make
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/.cd build_gfortran_sysnative
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build_gfortran_sysnative/.m
make: Warning: File `Makefile' has modification time 22 s in the future
make[1]: Warning: File `CMakeFiles/Makefile2' has modification time 22 s in the future
make[2]: Warning: File `src/CMakeFiles/example.dir/flags.make' has modification time 22 s in the future
Scanning dependencies of target example
make[2]: warning:  Clock skew detected.  Your build may be incomplete.
make[2]: Warning: File `src/CMakeFiles/example.dir/flags.make' has modification time 22 s in the future
make[2]: warning:  Clock skew detected.  Your build may be incomplete.
make[2]: Warning: File `src/CMakeFiles/example.dir/flags.make' has modification time 22 s in the future
[100%] Building Fortran object src/CMakeFiles/example.dir/example.f90.o
Linking Fortran executable ../bin/example
make[2]: warning:  Clock skew detected.  Your build may be incomplete.
[100%] Built target example
make[1]: warning:  Clock skew detected.  Your build may be incomplete.
make: warning:  Clock skew detected.  Your build may be incomplete.
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build_gfortran_sysnative/.cmake ..
CMake Error at src/CMakeLists.txt:16 (message):
  BLAS library not found for the test fc_blas!

-- Configuring incomplete, errors occurred!
See also "/home/milias/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build_gfortran_sysnative/CMakeFiles/CMakeOutput.log".
milias@login.grid.umb.sk:~/Work/qch/software/software_projects/autocmake_devel/autocmake_miroi/test/fc_blas/build_gfortran_sysnative/.

This is probably due to caching of variables in the math discovery (or lack thereof).
Here you need to track what happens to variables when the code is configured
and reconfigured.

Well, this is the BLAS_FOUND library set the math_libs.cmake module. Any clues to how to do tracking ? Maybe printing out variables from math_libs.cmake ?

Strange that setting of the crucial BLAS_FOUND to true is only at one place in math_libs.cmake module....

OK, I filled PR for that.

Miro, thanks for the analysis. I will reject the PR because I have a different solution which caches
the result. Note that BLAS_FOUND is set in more than one place (since we sometimes use the SERVICE variable).


Add download_chunk_size property to HTTPRequest.

This allows setting the read_chunk_size of the internal HTTPClient.
This is important to reduce the allocation overhead and number of file
writes when downloading large files, allowing for better download speed.
Fixes #32807 (the user will need to raise the download_chunk_size to a higher value. 64KiB is enough).
extends Control

var done := false
var last_bytes := 0
var time := 0.0 # timer that is restarted ~every second
var http

func _ready():
	http = HTTPRequest.new()
	# Sets a 64KiB chunk size
	http.download_chunk_size = pow(2,16)
	add_child(http)
	http.use_threads = true
	http.download_file = "100MB.bin"
	http.connect("request_completed", self, "_on_file_request_completed")

	# 100MB test file
	http.request("https://speed.hetzner.de/100MB.bin")

func _process(delta):
	if not done and time >= 1:
		var speed_in_bytes = http.get_downloaded_bytes() - last_bytes
		var speed_in_mega_bytes = speed_in_bytes / 1000000.0

		print(speed_in_mega_bytes, " MB/s")

		last_bytes = http.get_downloaded_bytes()
		time = 0
	time += delta

func _on_file_request_completed(result, response_code, headers, body):
	done = true
	print("File downloaded")

In general, and like in other cases in Godot networking, we are doing some extra buffering, which could be cleaned up in the future.

Note I think we should raise the default value to something higher, 32-64KiB, to better cover the download file use case by default. In any case, we should do that for editor-related HTTPRequest where download speed is quite important.

Note I think we should raise the default value to something higher, 32-64KiB, to better cover the download file use case by default.

Sounds good to me. Are there any drawbacks to using a value like 64 KiB by default?

Are there any drawbacks to using a value like 64 KiB by default?

Well, there might be some memory overhead, especially when reading in non-blocking mode from slow servers (or servers that sends little data at a time).
But in many other case (like the referenced issue) it would mean a much lower number of allocations.
In general, HTTPClient and HTTPRequest likely need some refactoring to use pre-allocated buffers instead of allocating memory every time (unless when strictly talking to GDScript). But that's 4.0 material.

Thanks!

I think a default of 64 KiB would make sense. Or 32 KiB if you think 64 KiB is too high as all-rounder value (knowing that most users will use the default). Can you make a follow-up PR?

Now it works like a charm, Thanks Faless.
I think download_chunk_size should be by default 32 KiB because downloaded files are often small sizes.
In this case there's no need to occupy additional memory for such a file. If a user wants to download larger files, he will just increase the value. But the user has to read docs in a first place...


Add script example on how to set system time

Added a commented script for setting the system time, which by default is set to the time needed for flog. Also updated .gitignore with a few things it misses from install.

Switched it to withHandles


Cell not deselected sometimes

Found something. If I selectDates with triggersDelegate false, and then select a date, the previously selected date is not deselected. Are you seeing the same?

Are you setting a cell to be unselectable any where in your app?
 A simple code fix. Looking into it.

I don't think so, can you be more specific?
I am selecting programmatically

It does seem like the last cell's state just isn't being refreshed

issue resolved on branch. Closed.
Cells were not being restored when dequeued. Thus old cells that were marked at disabled were reused for new cells.


Result of SOQL query: fields are shown in alphabetic order but not in the order mentioned in query

Quer: SELECT Id, F__c, B__c, A__c FROM Account
Result table: A__c, B__c, F__c

Hi @tviktar - thanks for raising this
I've made a fix for this and published v0.5.1, hopefully this resolves your issue.

it works. thanks a lot


[QUESTION] About future plan of the project

Dear Catalyst Maintainers,
Sorry for putting a question in issue tracker, I didn't find a better place for this.
May I ask about the future plan of this project? In the document I saw there were quite a few new features proposed for 2018, but seems none of them were accomplished. Will catalyst still be maintained in near future or it's being phased out?  I'm asking because we intend to build our trading bot based on catalyst, but if it will not be maintained / enhanced any longer we may have to reconsider.
Thank you.
Regards
R.

Hi @xiaohai6688 ,
Besides the Github, you can join the Catalyst Forum for questions around Catalyst, algorithmic trading and technical support. We also have a Discord group with the #catalyst_dev and #catalyst_setup dedicated channels.
In regards to your question, catalyst continued to be developed and we continue to refine the project goals during the development process. Many of the milestones set for 2018 were accomplished, others were deprioritized. And as any open source project, we welcome any contribution :)
Could you please share the reference to the proposed features for 2018 (we recently removed a page from the docs to avoid confusion, so we'll be happy to know if there are still misleading doc pages)?
Thanks

Hello
Thank you for the response. It's great to know you guys are still refining the catalyst!
I saws the features for 2018 here: https://enigma.co/catalyst/features.html
In the "Upcoming features" section (like pipeline support, web UI etc ...).  Maybe they've been implemented but I didn't find them. Sorry for the confusion if it is the case.
Regards

Thanks @xiaohai6688 for sharing.
We've removed the link for this page from the docs, however it is still possible to access it, sorry for the confusion.
As mentioned above, some of the milestones set for 2018 were accomplished (API documentation, Support for data ingestion of community-contributed data sets and more), others were deprioritized.
We are closing this, feel free to reopen if you still have questions, or using any of the other catalyst channels.
Thanks


I think it's because tryLock returns false when this.lock object was created, but not in LockState.ACQUIRED state:
https://github.com/apache/hudi/blob/f52cb32f5f0c9e31b5addf29adbb886ca6d167dd/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveMetastoreBasedLockProvider.java#L116
this.lock is never reset between retries, so when Lock manager calls tryLock for the second time the validation fails:
https://github.com/apache/hudi/blob/f52cb32f5f0c9e31b5addf29adbb886ca6d167dd/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveMetastoreBasedLockProvider.java#L136
To Reproduce
A second writer was writing to the table at that time, so it must have acquired a lock.
Expected behavior
Hudi should retry to acquire a lock.
Environment Description

Hudi version : 0.10.1
Spark version : 3.1.1
Hive version : 3.1.2
Hadoop version : 3.2.1
Storage (HDFS/S3/GCS..) : S3
Running on Docker? (yes/no) : no

Additional context
Lock config:
HoodieWriteConfig.WRITE_CONCURRENCY_MODE.key                       -> WriteConcurrencyMode.OPTIMISTIC_CONCURRENCY_CONTROL.name,
HoodieCompactionConfig.FAILED_WRITES_CLEANER_POLICY.key            -> HoodieFailedWritesCleaningPolicy.LAZY.name,
HoodieLockConfig.LOCK_PROVIDER_CLASS_NAME.key                      -> classOf[HiveMetastoreBasedLockProvider].getName,
HoodieLockConfig.HIVE_DATABASE_NAME.key                            -> "test_db"",
HoodieLockConfig.HIVE_TABLE_NAME.key                               -> "test_table",
HoodieLockConfig.LOCK_ACQUIRE_WAIT_TIMEOUT_MS.key                  -> (60 * 1000).toString,
HoodieLockConfig.LOCK_ACQUIRE_RETRY_WAIT_TIME_IN_MILLIS.key        -> (1 * 1000).toString,
HoodieLockConfig.LOCK_ACQUIRE_NUM_RETRIES.key                      -> 15.toString,
HoodieLockConfig.LOCK_ACQUIRE_CLIENT_NUM_RETRIES.key               -> 2.toString,
HoodieLockConfig.LOCK_ACQUIRE_CLIENT_RETRY_WAIT_TIME_IN_MILLIS.key -> (10 * 1000).toString

Stacktrace
22/05/27 02:14:38 WARN HiveConf: HiveConf of name hive.server2.thrift.url does not exist
22/05/27 02:14:38 INFO metastore: Trying to connect to metastore with URI thrift://ip-10-203-95-204.eu-west-2.compute.internal:9083
22/05/27 02:14:38 INFO metastore: Opened a connection to metastore, current connections: 1
22/05/27 02:14:38 INFO metastore: Connected to metastore.
22/05/27 02:14:38 INFO HiveMetastoreBasedLockProvider: ACQUIRING lock at database test_db and table test_table
22/05/27 02:14:38 INFO LockManager: Retrying to acquire lock...
22/05/27 02:14:48 INFO HiveMetastoreBasedLockProvider: ACQUIRING lock at database test_db and table test_table
22/05/27 02:14:48 INFO TransactionManager: Transaction ending without a transaction owner
22/05/27 02:14:48 INFO HiveMetastoreBasedLockProvider: RELEASING lock at database test_db and table test_table
22/05/27 02:14:48 INFO HiveMetastoreBasedLockProvider: RELEASED lock at database test_db and table test_table
22/05/27 02:14:48 INFO TransactionManager: Transaction ended without a transaction owner
java.lang.IllegalArgumentException: ALREADY_ACQUIRED
	at org.apache.hudi.common.util.ValidationUtils.checkArgument(ValidationUtils.java:40)
	at org.apache.hudi.hive.HiveMetastoreBasedLockProvider.acquireLock(HiveMetastoreBasedLockProvider.java:136)
	at org.apache.hudi.hive.HiveMetastoreBasedLockProvider.tryLock(HiveMetastoreBasedLockProvider.java:112)
	at org.apache.hudi.client.transaction.lock.LockManager.lock(LockManager.java:62)
	at org.apache.hudi.client.transaction.TransactionManager.beginTransaction(TransactionManager.java:51)
	at org.apache.hudi.client.SparkRDDWriteClient.getTableAndInitCtx(SparkRDDWriteClient.java:430)
	at org.apache.hudi.client.SparkRDDWriteClient.upsert(SparkRDDWriteClient.java:157)
	at org.apache.hudi.DataSourceUtils.doWriteOperation(DataSourceUtils.java:217)
	at org.apache.hudi.HoodieSparkSqlWriter$.write(HoodieSparkSqlWriter.scala:277)
	at org.apache.hudi.DefaultSource.createRelation(DefaultSource.scala:164)
	at org.apache.spark.sql.execution.datasources.SaveIntoDataSourceCommand.run(SaveIntoDataSourceCommand.scala:46)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult$lzycompute(commands.scala:70)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.sideEffectResult(commands.scala:68)
	at org.apache.spark.sql.execution.command.ExecutedCommandExec.doExecute(commands.scala:90)
	at org.apache.spark.sql.execution.SparkPlan.$anonfun$execute$1(SparkPlan.scala:185)
	at org.apache.spark.sql.execution.SparkPlan.$anonfun$executeQuery$1(SparkPlan.scala:223)
	at org.apache.spark.rdd.RDDOperationScope$.withScope(RDDOperationScope.scala:151)
	at org.apache.spark.sql.execution.SparkPlan.executeQuery(SparkPlan.scala:220)
	at org.apache.spark.sql.execution.SparkPlan.execute(SparkPlan.scala:181)
	at org.apache.spark.sql.execution.QueryExecution.toRdd$lzycompute(QueryExecution.scala:134)
	at org.apache.spark.sql.execution.QueryExecution.toRdd(QueryExecution.scala:133)
	at org.apache.spark.sql.DataFrameWriter.$anonfun$runCommand$1(DataFrameWriter.scala:989)
	at org.apache.spark.sql.catalyst.QueryPlanningTracker$.withTracker(QueryPlanningTracker.scala:107)
	at org.apache.spark.sql.execution.SQLExecution$.withTracker(SQLExecution.scala:232)
	at org.apache.spark.sql.execution.SQLExecution$.executeQuery$1(SQLExecution.scala:110)
	at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$6(SQLExecution.scala:135)
	at org.apache.spark.sql.catalyst.QueryPlanningTracker$.withTracker(QueryPlanningTracker.scala:107)
	at org.apache.spark.sql.execution.SQLExecution$.withTracker(SQLExecution.scala:232)
	at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$5(SQLExecution.scala:135)
	at org.apache.spark.sql.execution.SQLExecution$.withSQLConfPropagated(SQLExecution.scala:253)
	at org.apache.spark.sql.execution.SQLExecution$.$anonfun$withNewExecutionId$1(SQLExecution.scala:134)
	at org.apache.spark.sql.SparkSession.withActive(SparkSession.scala:772)
	at org.apache.spark.sql.execution.SQLExecution$.withNewExecutionId(SQLExecution.scala:68)
	at org.apache.spark.sql.DataFrameWriter.runCommand(DataFrameWriter.scala:989)
	at org.apache.spark.sql.DataFrameWriter.saveToV1Source(DataFrameWriter.scala:438)
	at org.apache.spark.sql.DataFrameWriter.saveInternal(DataFrameWriter.scala:415)
	at org.apache.spark.sql.DataFrameWriter.save(DataFrameWriter.scala:293)

@ganczarek Can you please try latest Hudi (0.11.1)? There were a couple of fixes in the lock provider. See https://github.com/apache/hudi/pull/5255
https://github.com/apache/hudi/pull/5380

This version does not work, either.
Add a reset of this.lock= null in tryLock could solve the problem.

I think it's because tryLock returns false when this.lock object was created, but not in LockState.ACQUIRED state

That's correct!

this.lock is never reset between retries, so when Lock manager calls tryLock for the second time the validation fails

This is not always the case. After tryLock returns false in the first attempt (when this.lock object was created, but not in LockState.ACQUIRED state), then LockManager will sleep for some time and try again
https://github.com/apache/hudi/blob/master/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/transaction/lock/LockManager.java#L72
But the real problem is that we don't reset the lock in this case.
https://github.com/apache/hudi/blob/master/hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/transaction/lock/HiveMetastoreBasedLockProvider.java#L198-L200
We should fix this.  HUDI-4518 to track this.

Closing as the fix landed. This should be released in v0.12.0


Can you tag the commit you based your PyPi package on?

Would help to be able to easily checkout the version of the repository each wheel is built from.

Thanks for the suggestion, here you go:
https://github.com/webfp/tor-browser-selenium/releases/tag/v0.1


Add Phone book example

Added a simple phone book example program written in Moose. Was more of a fun project but if you want you can merge this pull request and add it to existing examples. Overall a really impressive small language and I am quite impressed by the good compiler error messages, good work 👍

So nice! Thank you! :)


Trouble finding feed for 2a10:4741:30::/44

The owner of 2a10:4741:30::/44 has published their feed URL with the geofeed attribute in inet6num records but it isn't discovered in the output CSV with neither of -i ripe or -i arin. What could be the reason? Any guidance is appreciated.
https://apps.db.ripe.net/db-web-ui/query?searchtext=2a10:4741:30::%2F44

Other observations:

Deleting the cache directory to trigger a re-download didn't change the above behavior
Interestingly, geofeed-finder-linux-x64 -t 2a10:4741:30::/44 yields the correct results on stdout

Not reproducible here. How many lines do you have in the output csv file?
sgteq@desk ~/geofeed $ fgrep 2a10:4741: ripe-2023-02-02.csv
2a10:4741:0030:0000:0000:0000:0000:0000/44,,,,
2a10:4741:0032:0000:0000:0000:0000:0000/48,CA,CA-ON,Toronto,
2a10:4741:0036:0000:0000:0000:0000:0000/48,CA,CA-QC,Montreal,
2a10:4741:0037:0000:0000:0000:0000:0000/48,CA,CA-QC,Montreal,
2a10:4741:0033:0000:0000:0000:0000:0000/48,CA,CA-ON,Toronto,
2a10:4741:0035:0000:0000:0000:0000:0000/48,CA,CA-QC,Montreal,
sgteq@desk ~/geofeed $ wc -l ripe-2023-02-02.csv
31128 ripe-2023-02-02.csv

I am using the pre-built Linux AMD64 binary in the releases folder , v1.7.0
~$ ~/bin/geofeed-finder-linux-x64 -v 1.7.0
Command I use:
~/bin/geofeed-finder-linux-x64 --keep-non-iso --keep-invalid-subdivisions --download-timeout 0 -i ripe -o ripe.csv
Line count:
~$ wc -l ripe.csv
32604 ripe.csv
No results:
~$ cat ripe.csv |grep -i 2a10:4741

31741 lines in my ripe.csv but no mention of any 2a10:4741. The CLI options have nothing to do with it. Will try recompiling myself, rather than use the precompiled binary release v1.7.0

Hi @s8mathur,
I just tried with both no parameters (31800 entries) and with the parameters you reported in your previous post (32583 entries), only ripe.
In both cases I can get the prefixes correctly.
2a10:4741:0030:0000:0000:0000:0000:0000/44,,,,
2a10:4741:0032:0000:0000:0000:0000:0000/48,CA,CA-ON,Toronto,
2a10:4741:0036:0000:0000:0000:0000:0000/48,CA,CA-QC,Montreal,
2a10:4741:0037:0000:0000:0000:0000:0000/48,CA,CA-QC,Montreal,
2a10:4741:0033:0000:0000:0000:0000:0000/48,CA,CA-ON,Toronto,
2a10:4741:0035:0000:0000:0000:0000:0000/48,CA,CA-QC,Montreal,

Precompiled or not doesn't matter. However, just to be sure I also tried with the precompiled downloaded from git (version 1.7.0). Same result of above, everything is fine.
I would suggest you to check that you are able to download the geofeed file: check inside logs/ and grep for as208914. Also, your lower count of geofeed suggests me that your cache is not up to date. Try rm -rf .cache in the root directory where geofeed-finder is.

Just a comment on the last statement I did. Removing the cache should not be the way of proceeding in production.
If you find that removing the cache fixed the issue, it means that everything is fine and next time you should wait for the cache to expire. This protects the whois servers and the hosts of the geofeed fils (and probably you from getting blocked). The geofeed file host can set http headers to signal how often they allow the file to be downloaded once discovered.

Thanks for your help, folks. Deleting rm -rf ~/.cache (instead of the rm -rf ~/.cache/ripe and rm -rf ~/.cache/arin that I did previously) seems to have fixed the problem, hence closing this issue.
Somewhat troubling that the cache files go into a silent failure mode, but I will keep an eye out for future inconsistencies of the type. Perhaps ~/.cache/<toolname> should be used as cache root directory to avoid inadvertent collision with many other tools which save files there?

I think you removed the whois dump, but the geofeed file itself was still cached.
I would cd in the location of the binary and run it, in this way the .cache directory is inside the same directory of the binary (however, I think it was just cached and it needed to expire).


from keyword in import tags and of keyword in for loop tags are not highlighted

Same vein as https://github.com/ventojs/vscode-vento/issues/11, with the addition of of. Possibly other keyword too, I'll do a proper check. I don't have a minute to work on this myself yet (likely until this weekend) but thought I'd mention it in case you have some time on your hands!

Hey, thanks for making an issue! The grammar here is pretty naive, since it only assumes a few keywords are part of vento (in theory, it's very flexible due to extensions), so currently it relies a bit on tree-sitter-javascript to fill in the gaps.
I can add the the special cases like import x from "" and such, since those are handled natively in vento, and should be added. Let me know which ones you're missing and I'll add em :).

Gotcha yeah, I was taking a peek around earlier - I've been working with Tree-sitter grammars frequently recently so I'm getting used to it 😅 Would be great if you could add those two keywords, not any others I can think of at the minute.


A mechanism is not available to dismiss the additional content without moving pointer hover or keyboard focus, in Design screen.

Describe the bug
A mechanism is not available to dismiss the additional content without moving pointer hover or keyboard focus, in Design screen.
To Reproduce
Steps to reproduce the behavior:

Go to Design  (http://localhost:9992/cm/app/?view=design&pathType=unknown)
Mouse hover on the "Action" menu. It will show sub menu, i.e. Create Template from Page, Add Template, Export Template and Import Template.
There is no way (Esc key etc.) to dismiss the sub menu, without moving mouse or removing focus using keyboard.
Submenu should d1sappear without touching the mouse i.e. by using the Esc key etc.

Expected behavior
Submenu should d1sappear without moving mouse.
Additional context
Navigate all the options available for similar issue and fix.

This is not a bug as there is no additional content on Hover. In application on Hover, only title is displayed. So closing this.


Finch client Spike - Do Not Merge

This is a quick spike around what I think a finch-client could look like, at least for my typical use cases.  It's heavily biased towards a RESTFul style.  A small syntax showing useage is in the README for the package.  Quick synopsis:

Create a Finch request w/ a host and an endpoint (finagle.Path)
Make the actual Http request w/ an Http Verb => Get(finchRequest)
Verb returns a String Xor Resource from Cats to handle failures better ( super shallow w/ this right now )
A Resource is simply and HttpResponse Status like Ok parameterized on the response content type => Ok(twitter.io.Buf) or a Ok(String) or Ok(argonaut.Json)
Right now I've got two implicit DecodeResource TypeClasses for String and json.Argonaut.

Thoughts on this style?  Love it?  Hate it?  It needs a lot of work, but this is much closer to what I envision that my partial baked gist I put in gitter and then yanked out on Friday.

I'm just starting to look at this, but I'm not sure about introducing a Cats dependency just for a biased disjunction type. Any reason you don't just fail in the future with some type representing a response failure? Then you could use the standard future failure handling mechanisms.

I have no issues removing the right biased disjunction and using a separate type to represent a failure - using Xor came from looking at Dispatch and realizing they return a Future[Either[FailureType, ResultType]] and trying to play with something similar.

In continuation of 1, I should have been explicit in the failure information - honestly I just forgot about it while I was spiking this.

implicit functions make me nervous - the reasoning for them was an attempt at utilizing TypeClasses to handle decoding a com.io.twitter.Buf to some A. In my head, that would allow us parameterize a Resource on the ContentType itself. I'm sure there are better ways to do this though - it's just that most implementations of TypeClasses I've seen use implicits.

The problem with implicit functions is that they can get applied behind your back, just like implicit conversions defined as defs (this is why we've got things like the ToRequest type class in my last PR—it's isomorphic to a A => Request, but you can put one into implicit scope without As being treated as Requests everywhere).


Added Visual Studio projects for the launcher

So Windows freaks like myself can also work on the launcher project ;-)

+437... Ouch. Something something open folder something.
... anyway...

+437... Ouch. Something something open folder something.
... anyway...

Visual Studio project files are too complex and long 🤷♂

Thank you! I guess someone has to fight for poor ol' Visual Studio's native projects :laughing:
And they are at least better than opening the CMake in some ways.

So, my major problem with the VS sln, is that except in the (very) short term it saves you from nothing. You still need to setup CMake to do a device build, which is kind of the whole point... and then you have the same problem we have, you need to maintain duplicate build configs.
So, while you can say it's easier to setup for VS users, unless you for some reason don't want to build for the device it's more work in the end. You might as well setup something that works for both local and device (which could still be VS, cross-builds are possible there). And then there's the fact that CMake is also required to do anything with metadata, and releases...
Maybe I just don't like duplicated work and only being the occasional VS user am probably not the best person to comment on VS .sln vs VS integrated CMake. (Not going to suggest a CMake generated VS .sln, those are... messy)
Anyway, I'll stop complaining now... (probably until it gets in my way again)


event_id null in flat_events and flat_event_params

Hi Ruslan, Per our discussion, there are event_id fields coming in from flat_events as well as flat_event_params.  It looks to be about 1% of the records per day.
Thanks,
Rob

Thank you Robert, I will look into it and will give you an update.

@robertlmyers , I am unable to replicate the issue.
I used the following query:
SELECT DISTINCT event_id FROM `{project}.analytics_{ga4_property}.flat_events_*`
ORDER BY 1 NULLS FIRST
 LIMIT 1000

Are any of the following fields NULL in your source GA4 data?
"stream_id",
"user_pseudo_id",
"event_name",
"event_timestamp"

@robertlmyers,
This issue has been solved in this release:
https://github.com/adswerve/google_analytics_flattener_ga4/releases/tag/v1.3.0


Failed to activate the linter package - cannot find module 'xregexp'

[Enter steps to reproduce below:]

Just open Atom
the message pops-up

Atom Version: 1.0.2
System: "Fedora release
Thrown From: linter package, v1.3.0
Stack Trace
Failed to activate the linter package
At Cannot find module 'xregexp'

Error: Cannot find module 'xregexp'
  at Module._resolveFilename (module.js:328:15)
  at Function.Module._resolveFilename (/usr/share/atom/resources/app.asar/src/module-cache.js:383:52)
  at Function.Module._load (module.js:270:25)
  at Module.require (module.js:357:17)
  at require (module.js:376:17)
  at Object.<anonymous> (/home/users/adriano.vieira/.atom/packages/linter/lib/helpers.coffee:2:11)
  at Object.<anonymous> (/home/users/adriano.vieira/.atom/packages/linter/lib/helpers.coffee:1:1)
  at Module._compile (module.js:452:26)
  at Object.requireCoffeeScript (/usr/share/atom/resources/app.asar/node_modules/coffee-cash/lib/coffee-cash.js:85:19)
  at Module.load (module.js:347:32)
  at Function.Module._load (module.js:302:12)
  at Module.require (module.js:357:17)
  at require (module.js:376:17)
  at Object.<anonymous> (/home/users/adriano.vieira/.atom/packages/linter/lib/linter-plus.coffee:5:11)
  at Object.<anonymous> (/home/users/adriano.vieira/.atom/packages/linter/lib/linter-plus.coffee:1:1)
  at Module._compile (module.js:452:26)
  at Object.requireCoffeeScript (/usr/share/atom/resources/app.asar/node_modules/coffee-cash/lib/coffee-cash.js:85:19)
  at Module.load (module.js:347:32)
  at Function.Module._load (module.js:302:12)
  at Module.require (module.js:357:17)
  at require (module.js:376:17)
  at Object.module.exports.activate (/home/users/adriano.vieira/.atom/packages/linter/lib/main.coffee:48:18)
  at Package.module.exports.Package.activateNow (/usr/share/atom/resources/app.asar/src/package.js:240:19)
  at /usr/share/atom/resources/app.asar/src/package.js:221:30
  at Package.module.exports.Package.measure (/usr/share/atom/resources/app.asar/src/package.js:165:15)
  at Package.module.exports.Package.activate (/usr/share/atom/resources/app.asar/src/package.js:213:14)
  at PackageManager.module.exports.PackageManager.activatePackage (/usr/share/atom/resources/app.asar/src/package-manager.js:452:21)
  at /usr/share/atom/resources/app.asar/src/package-manager.js:436:29
  at Config.module.exports.Config.transact (/usr/share/atom/resources/app.asar/src/config.js:311:16)
  at PackageManager.module.exports.PackageManager.activatePackages (/usr/share/atom/resources/app.asar/src/package-manager.js:431:19)
  at PackageManager.module.exports.PackageManager.activate (/usr/share/atom/resources/app.asar/src/package-manager.js:412:46)
  at Atom.module.exports.Atom.startEditorWindow (/usr/share/atom/resources/app.asar/src/atom.js:645:21)
  at Object.<anonymous> (/usr/share/atom/resources/app.asar/src/window-bootstrap.js:12:8)
  at Object.<anonymous> (/usr/share/atom/resources/app.asar/src/window-bootstrap.js:23:4)
  at Module._compile (module.js:452:26)
  at Object.loadFile [as .js] (/usr/share/atom/resources/app.asar/src/babel.js:162:21)
  at Module.load (module.js:347:32)
  at Function.Module._load (module.js:302:12)
  at Module.require (module.js:357:17)
  at require (module.js:376:17)
  at setupWindow (file:///usr/share/atom/resources/app.asar/static/index.js:96:23)
  at window.onload (file:///usr/share/atom/resources/app.asar/static/index.js:36:7)

Commands

Config
{
  "core": {},
  "linter": {}
}

Installed Packages
# User
linter, v1.3.0
linter-erb, v0.1.1
linter-puppet-lint, v0.2.9

# Dev
No dev packages

Repro steps please? Did you install any of those packages using git? and forgot to do apm install in their dirs?

I just the installation problem.
We're behind a proxy...
I configure the proxy var at .atom/.apmrc with https-proxy = http://<user:pass>@<proxy:port>
but apm appear not authenticate a proxy with digest authentication.

If apm doesn't support that, I can only hope npm does, You can also do npm install to achieve the same thing. Closing for now, but feel free to continue the discussion.

Posting here because the View Issue button in atom says this issue has already been reported links to this page. Maybe someone could look into that too.
[Enter steps to reproduce below:]

First time using atom since upgrading to win10
Clicked on update all
Wait for update to process

Atom Version: 1.0.7
System: Microsoft Windows 10 Home
Thrown From: linter package, v1.4.3
Stack Trace
Failed to activate the linter package
At undefined is not a function

TypeError: undefined is not a function
  at new Linter (C:\Users\George\.atom\packages\linter\lib\linter.coffee:34:54)
  at Object.module.exports.activate (C:\Users\George\.atom\packages\linter\lib\main.coffee:54:21)
  at Package.module.exports.Package.activateNow (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\package.js:243:19)
  at C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\package.js:224:30
  at Package.module.exports.Package.measure (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\package.js:167:15)
  at Package.module.exports.Package.activate (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\package.js:216:14)
  at PackageManager.module.exports.PackageManager.activatePackage (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\package-manager.js:486:21)
  at C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\node_modules\settings-view\lib\package-manager.js:312:60
  at exit (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\node_modules\settings-view\lib\package-manager.js:59:16)
  at triggerExitCallback (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\buffered-process.js:213:47)
  at ChildProcess.<anonymous> (C:\Users\George\AppData\Local\atom\app-1.0.7\resources\app.asar\src\buffered-process.js:235:18)
  at emitTwo (events.js:87:13)
  at ChildProcess.emit (events.js:169:7)
  at Process.ChildProcess._handle.onexit (child_process.js:1057:12)

Commands
 10x -9:15.4.0 core:backspace (atom-text-editor.editor.is-focused)
 18x -8:27 core:move-left (atom-text-editor.editor.is-focused)
     -8:24.9.0 editor:newline (atom-text-editor.editor.is-focused)
 22x -8:24.4.0 core:move-right (atom-text-editor.editor.is-focused)
 14x -8:11.9.0 core:move-left (atom-text-editor.editor.is-focused)
     -8:10.8.0 core:move-up (atom-text-editor.editor.is-focused)
 22x -8:10.3.0 core:move-right (atom-text-editor.editor.is-focused)
 14x -8:08.0 core:move-left (atom-text-editor.editor.is-focused)
 15x -8:06.4.0 core:move-right (atom-text-editor.editor.is-focused)
     -8:04.2.0 core:move-left (atom-text-editor.editor.is-focused)
     -8:00.2.0 core:backspace (atom-text-editor.editor.is-focused)
     -7:58.3.0 core:move-down (atom-text-editor.editor.is-focused)
     -7:55.7.0 core:move-up (atom-text-editor.editor.is-focused)
     -7:54.9.0 core:move-down (atom-text-editor.editor.is-focused)
     -1:23.8.0 core:save (atom-text-editor.editor.is-focused)
     -1:21.8.0 settings-view:check-for-package-updates (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)

Config
{
  "core": {},
  "linter": {}
}

Installed Packages
# User
.bin, vundefined
language-protobuf, v0.5.0
linter, v1.4.3

# Dev
No dev packages

@xiegeo remove and re-install linter

Just restarting atom worked, I think, no more errors. Is there a good way to report non-critical / temporary errors?

@xiegeo well, atom's the one who handles that, so not sure about that.

Also @xiegeo according to that package list you have no linter-* providers installed, so linter won't be doing too much for you :wink:.

I've been getting this error for a while on Yosemite; the stack is identical to @xiegeo's. Win8 is apparently not a critical factor.
@Arcanemagus, you should start by granting the reasonable assumption that @xiegeo removed providers in an attempt to identify a provider as a source of the problem.

@jeffmcmahan It's been ages since this error has been fixed. We've even completely removed xregexp usage and all from this repo. so If you're still getting this error you might wanna have a fresh start with your ~/.atom.

Also @jeffmcmahan you would not believe the number of people filing issues saying it doesn't work who either don't have any linter-* packages installed, or don't have linter installed but have a linter-* package installed (this is now fixable, and providers are updating for this).
Sometimes pointing out what seems obvious helps somebody who is looking at it from a different perspective and can't see that. In the worst case it doesn't hurt anything.


If ECS service-linked role doesn't exist, CF Stack will rollback

Hi,
Trying to provision the CF template without the ECS-generated service-linked role "AWSServiceRoleForECS" will result in a rollback. It would likely be helpful to highlight this requirement and include a pointer to the documentation for creating a service-linked role: https://docs.aws.amazon.com/IAM/latest/UserGuide/using-service-linked-roles.html#create-service-linked-role.
Quick research suggests that CloudFormation may not be able to elegantly create this role, so not sure what's the best way to proceed. Happy to help if given guidance.

@chrisckchang You are absolutely correct. We updated the README and we will get this documented in the implementation guide.


Implement WeakReference, Finalizer and NativeFinalizer in VM

For reference: language proposal.

[x] WeakReference
[x] Finalizable
[ ] Finalizer (in review)
[ ] NativeFinalizer (in progress)

Now that Finalizer and NativeFinalizer has landed, can this issue be closed?


Added Base32 Encoding

The Secret is not treated as Base32 encoded string (which is pretty much the default)
Also I fixed the tests for the Length (6 vs. 0x6)

Thanks @andyHa. I'll review this in the weekend.

Hey @andyHa!
Thanks, but I cannot fully merge this since it will break backwards compatibility. The RFC standard does not mandate the use of base32 encoding so it should be optional.
I'll reuse your commits and work in top of them so I can keep your contribution history.


Inert state for input based components

In the HTML standard there is an inert state for elements which are not interactive. It is very similar to disabled but the element does not change its color.
I have recently seen this used several times in the designs so it would be good to support it as a Diete component property.

I would like to discuss about this. I've read a few articles on the topic and I'm even more confused now why is it needed — it might be for some technical reasons.
In general if an element is not interactive, I think that it should be visually different, otherwise it might feel as a bug, no?
Lately I have addressed this with a slight update on the Disabled state for components like inputs, select, etc. where I wanted the content to remain readable even if the component is disabled.

Maybe we don't need the inert but the readonly. But I can show you examples from recent designs where there is a non-editable, non-disabled component (think of the copyable link for example).
Disabled is still needed to clearly indicate to the user when a field or button is disabled, but there is a distinct, different use-case which is not addressed at the moment consistently.


KAFKA-8715: fix timestamp bug for static member.id generation

The bug is that we accidentally used the loaded timestamp for the group instead of the real current time. Fix is made and unit test to make sure the timestamp is properly encoded within the returned member.id.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@guozhangwang @hachikuji FYI

@abbccdda Thanks for the fix. I think this bug has both a direct and indirect impact. The direct impact is the NPE. The indirect impact is that the memberId is not being uniquely generated. This seems to have an impact for fencing? I think I'm wondering whether we should use UUID.randomUUID() rather than timestamp, similar to the dynamic path.

Assuming the instance-id is unique already, then practically time.millisecond() should be sufficient as you will only break fencing if the clients configured with the same instance id are joining at the exact same milli-second. So I think this theoretical risk is not worth dropping the debugging efficiency?

I still feel timestamp is useful for debugging like Guozhang suggested. For uniqueness I'm fine with both.

Ok, we can stick with timestamp.

LGTM.

Hmm, the reasoning for using a timestamp seems a bit odd. Are we sure about this?

Yea, I think we discussed a couple of times during past reviews, and felt that a timestamp suffix could help debug when a static member actually rejoins the group.

get 1/3 build with jdk 8 unknown failure
retest this please

We can use a time-based UUID or you could add the timestamp in addition to the UUID. Relying on a timestamp for uniqueness is an anti-pattern. In particular, the following statement is weird:
"Assuming the instance-id is unique already, then practically time.millisecond() should be sufficient as you will only break fencing if the clients configured with the same instance id are joining at the exact same milli-second. So I think this theoretical risk"
We know things like this happen, it's not theoretical.

I'm fine with a instanceId + UUID + timestamp format than instanceId + timestamp -- I do feel maintaining the timestamp would be convenient for debugging.

I prefer instanceId + UUID for consistency. I think the debugging benefit of having a timestamp is marginal.

Per offline discussion, we shall use UUID for consistency. Thank you all!


Integrate "Create Issue Branch" Github action

This would save some much time!
https://github.com/robvanderleek/create-issue-branch

Added in #181


reneable daily insiders releases

and update engine to 1.58.0-insider
For #6195

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Tests fails because 1.58 doesn't exist yet. It should by tomorrow.


apply correct bounds for various array inputs

Fixes several out-of-bounds memory accesses that can be triggered via an unauthenticated cellular device.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@nathaniel-bennett, you need to "redo" your commit using the -s flag to pass the DCO check. You see details here: https://wiki.linuxfoundation.org/dco


Is it possible to apply parcels at a dataset of a certain time period for multiple times?

I am working at a 3D case similar to the corefootprint particles example:
https://github.com/OceanParcels/Parcelsv0.9Paper_Scripts/blob/master/experiment_scripts/example_corefootprintparticles.py
Suppose you want to use one year of data in this corefootprint particles example multiple times. Is this possible? So for example to use the data from the year 1980 three times to track a particle for three years.
I am now getting as far as adding snapshots of the same year the second time with fieldset.advancetime() method and get:
RuntimeError: Time of field_new in Field.advancetime() overlaps with times in old Field

Thanks @pdnooteboom. In the short term, the way to fix the RuntimeError is to also change the fieldset.U.time (and fieldset.V.time etc) so that they 'count through'
In the longer term, it might be nice to implement this 'looping' of velocity fields as an option, for example through the allow_time_extrapolation=looping attribute in FieldSet. The could be a useful feature for some users

Thanks @erikvansebille!


(GH-1614) Deep copy config and inventory for the BoltSpec Runner

Inventory can be huge when read from a file and so the Inventory deletes
the version key to avoid copying the whole inventory. This will cause
BoltSpec to copy the inventory before creating an Inventory instance to
avoid mutating the callers inventory hash.

CLA signed by all contributors.

Is there a reason we shouldn't just leave the version key on the inventory in create_version?

Shouldn't this fix also make it into Bolt 1.x branch too?  Seems an egregious error that should be backported.

@glennsarti no, we're not doing any backports.
We chose to invest in making the upgrade process simple rather than long term support for 1.0 Bolt is still relatively young and we want to push users to upgrade to prevent fractures in the ecosystem.
@nicklewis I think it's safer for us to copy at this level and it will allow us to optimize for the more common CLI use case later. I think we'll soon crystalize this as part of the ruby API work.


Build the style guide (Component: Button)

The design team has built the Style guide, which can be found here.
This issue is a chunk of #4, ensure to check it first.

Build the button components similar to the design and document all classes used
Build with great focus on all states and responsiveness.

Assign me, sir @BolajiAyodeji .

Just assigned you chief :fire:

I'm setting up storybook as part of this ticket since that is needed to view the components in isolation.

Also, the Figma link displays 404.

Also, the Figma link displays 404.

@perriefidelis we need the Figma link here too

I'm setting up storybook as part of this ticket since that is needed to view the components in isolation.

Alright chief, sounds great.

@BolajiOlajide the Figma link is updated now, please check now.

@BolajiAyodeji PR ready.


0.5.1 or 0.6.0?

@jbalogh would you mind cutting a 0.5.1 or 0.6.0 release so that the recent fixes/features can be easier to use?

v0.6.0 released - https://pypi.python.org/pypi/django-pylibmc/0.6.0


GUI scaling for XR

Draft PR since it works but I want to take a second look over it, though I'm going on vacation for a week.
If @RaananW has any feedback as well, that would be appreciated. This was the closest I could get to what we had discussed while keeping XR out of GUI and GUI out of XR.
The only thing not included in this is the fact that the user has to call GUI3DManager.useRealisticScaling = true in order to get the XR scaling ratio, which should be included in the official docs for any developer wishing to use 3D GUI elements in XR.

That's a great abstract solution.
I have a few nit-comments. Otherwise, LGTM. Is the 0.1 scaling enough to make it render correctly in XR?

That's the default scaling I've been applying to my local scenes. It sets the GUI buttons to a size of 10x10cm, I'll need to double-check what the MRTK guidelines say about button sizes though before locking the scale in.

@rickfromwork - Any update on this one?


Subdomain can access main domain routes

@thetutlage the issue we talked about on AdonisJs discord.
Package version
"adonis-version": "4.1.0",
"@adonisjs/framework": "^5.0.13",
Node.js and npm version
v9.11.2
6.9.0
Sample Code (to reproduce the issue)
Add this line to your /etc/hosts file <IP_ADDRESS>	test.localhost
Create a simple repo with adonis new yardstick, remove Route.on('/').render('welcome') from the routes.js file and add the following code:
Route.get('/', () => 'You\'re on the subdomain').domain('test.localhost')
Route.get('/', () => 'You\'re on the maindomain')
Route.get('/contact', () => 'You\'re on the maindomain')

Here is the behaviour:
When I go on test.localhost:3333 it shows "You're on the subdomain".
When I go on localhost:3333 it shows "You're on the main domain".
When I go on test.localhost:3333/contact it shows "You're on the main domain".
Apparently it's not the expected behaviour and you shouldn't have to add .domain('localhost') to your main domain routes to prevent your subdomain from accessing them.
BONUS
Repo to reproduce the issue -> https://github.com/mathieuh/domain-bug
An issue about it was opened here -> https://github.com/adonisjs/adonis-framework/issues/971
@RomainLanz kind of answered here -> https://forum.adonisjs.com/t/routing-domains-doesnt-care-for-domain-while-routes-are-same/2178

@annymosse thank you, actually I know how to solve it, its just that I have a lot of routes and I didn't want to put .domain on all my www routes plus I talked to @thetutlage who told me it might be a bug.
I just want to be sure of the behaviour of Adonis on this.

@mathieuh you've a right ,for me i prefer the current methods coz i can handle each route by myself that making me review and organize my routes for each development to improve and scale my projects slow and fast at same time because i'm sure i didn't miss any router and his controller,i think my idea little buggy :man_facepalming: but who know maybe this my secret that making me grow up and faster as a coder .

someone look at #1058

Closing since not actionable.


I would like to make a feature request to include a timeout setting when attempting to connect.

Hello to you Mustafa,
I would like to make a feature request to include a timeout setting when attempting to connect.
Perhaps it could be an optional parameter passed via the dirver.connect command.
In my application for example, the ability to connect to the database is an environmental attribute that the program must be responsive to by facilitating a local storage option while the sensors continue to function. When connection to the remote data becomes viable, the program would then transmit it's local cache to that system.
Currently, if the remote data connection is available the tdslite library connects immediately.
If the remote data is not available the program remains unresponsive during this timeout/retry period.
Please do let me know if there are any questions regarding this request or if I may be of any assistance .
Regards
Originally posted by @bradbeshaw in https://github.com/tdslite/tdslite/discussions/39#discussioncomment-8208817

Scheduled to be released with 0.5.0


Async refresh when experimental setting is on

Description
This makes refresh attempt async slicing by default when its corresponding experimental flag is on. In that case, the many internal calls to Layer.refresh will be executed asynchronously via the viewer rather than directly in the layer. This is especially helpful when a layer is initialized and added to the viewer, which can trigger many blocking calls to refresh some of which can now be cancelled. As Layer._slice_dims is always on the sync path, that forces the refresh to be synchronous.
In most usage of napari this should be enough to avoid #5591 because if the experimental flag is on and the layer supports async, then all slicing should occur on the slicing thread. If the flag is off or the layer does not support async, then all slicing occurs on the main thread.
However, #5591 could still be triggered if a user flips the experimental flag while async slicing is occurring. So this PR does not entirely close #5591, though something like #5610 should.
References
Related to #5591 but does not close it.

@jni : added you as a reviewer because we'd discussed something like this before. See the PR description for my thoughts on this.
I also attempted a different approach to this in https://github.com/andy-sweet/napari/pull/52/files similar to something we had discussed in the past. That's much more complex, but it does a much better job of unifying the sync and async paths. Probably needs a bit more work and some performance optimization before it could be merged though, whereas this is simple enough.

Converted to draft because after some shower thoughts, I think I might be able to make https://github.com/andy-sweet/napari/pull/52 solve both issues.

Hahaha every office should get easy and convenient showers installed. Shower thoughts are the source of so much innovation! 😂 🧠🚿💭

@jni : I decided to make this ready for review because it's simple, likely fixes #5591 in most real cases, and also provides some other performance/laziness benefits from main.
By contrast https://github.com/andy-sweet/napari/pull/52 makes lots of changes for a variety of different reasons. They may be worth pursuing, but probably don't change too many fundamentals.


Tweak self-label dialog refresh

Why
Just a couple of additional tweaks ontop of the work in https://github.com/bluesky-social/social-app/pull/4354

We'll merge this through https://github.com/bluesky-social/social-app/pull/5758


APP-264: Improve "Storm data analytics demo" descriptions

Fix p.4;
Reviewers: @Slash32 and @Rib47

LGTM

LGTM


Nodist can't find npm version 7.14.0

When I run nodist npm 7.14.0, it returns with Unhandled rejection {"message":"Bad credentials","documentation_url":"https://docs.github.com/rest"}.
If I I run npm install -g<EMAIL_ADDRESS>it complains about npm.cmd and suggests that I run the command with --force.. If I run the command with --force it works.

Yeah i too facing the same issue when i try this command nodist npm 6.14.12
getting this below error
npm 6.14.12
{"message":"Bad credentials","documentation_url":"https://docs.github.com/rest"}.
Sorry.

You can do a quick fix by installing npm without using nodist. So by using the following command: npm i -g npm@latest.

Hello,
I will take a look into fixing this tomorrow. Sorry for the trouble.
Thank you
On Sun, May 30, 2021 at 11:03 AM Martijn @.***> wrote:

You can do a quick fix by installing npm without using nodist. So by using
the following command: npm i -g @.***
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nullivex/nodist/issues/233#issuecomment-851030517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARWXXYGQ5NOBAKT7SYCWNLTQJVWFANCNFSM45S4NNZA
.

--
Bryan Tong,
President and CEO
@.***
This communication is for the use of the intended recipient only. It may
contain information that is privileged and confidential. If you are not the
intended recipient of this communication, any disclosure, copying, further
distribution or use thereof is prohibited. If you have received this
communication in error, please advise me by return e-mail and
delete/destroy it.

Having the same issue :(

nodist npm match worked.

nodist npm match worked.

And what npm version will it install if you use it?

nodist npm match worked.

And what npm version will it install if you use it?

npm corresponding to the installed Node will be installed.
see here: https://nodejs.org/ja/download/releases/

Hello,
I spent some time on this issue today and issued a patch here: https://github.com/nullivex/nodist/pull/234
For anyone with an existing version the issue can be easily avoided by replacing the nodist/lib/github.js file. I will attach that file to this comment.
Additionally, I will propagate a new version being that the older key is now invalidated.
#232 is duplicate of this.

Hello,
...
For anyone with an existing version the issue can be easily avoided by replacing the nodist/lib/github.js file. I will attach that file to this comment.

Where is the attached file you mention?  I am running 9.1 - should that already have the patch?

Here is the file he mentions: https://github.com/nullivex/nodist/blob/master/lib/github.js Replace Nodist/lib/github.js with this file

Has been fixed in v.10.3


bug: int overflow.

Codecov Report
Base: 58.94% // Head: 58.94% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (316ebb8) compared to base (d92126a).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #4516   +/-   ##
=========================================
  Coverage     58.94%   58.94%
- Complexity    26910    26911    +1
=========================================
  Files          1268     1268
  Lines        122755   122755
  Branches      23123    23123
=========================================
+ Hits          72355    72361    +6
+ Misses        41381    41377    -4
+ Partials       9019     9017    -2

Impacted Files
Coverage Δ

...ool/ha/selector/RandomDataSourceRecoverThread.java
0.00% <0.00%> (ø)

...ol/ha/selector/RandomDataSourceValidateThread.java
0.00% <0.00%> (ø)

...ol/ha/selector/StickyRandomDataSourceSelector.java
0.00% <0.00%> (ø)

...n/java/com/alibaba/druid/pool/DruidDataSource.java
72.26% <0.00%> (+0.28%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

good job


Upgrade ReactNative version

Avoid having this issue on Android: https://github.com/facebook/react-native/issues/5658

Thanks


Removes deprecated exit code in 'spfx project upgrade'. Closes #1418

Removes deprecated exit code in 'spfx project upgrade'. Closes #1418

Coverage remained the same at 100.0% when pulling 49df580f96e49bf03b8a55608749e56c572dd086 on waldekmastykarz:remove-ERROR_PROJECT_UP_TO_DATE into fe516654e3e2c29dde00a53a00187f392d2097f1 on pnp:v3.

Merged manually


MIGRATED: unable to load local assets or from CDNs along with window.postMessage not working in iOS release build (#20226)

Migrated from https://github.com/facebook/react-native/issues/20226.
@sja1n says:

Environment
React Native Environment Info:
System:
OS: macOS High Sierra 10.13.5
CPU: x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
Memory: 310.16 MB / 8.00 GB
Shell: 5.3 - /bin/zsh
Binaries:
Node: 8.11.1 - /usr/local/bin/node
Yarn: 1.5.1 - /usr/local/bin/yarn
npm: 5.6.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
Android SDK:
Build Tools: 26.0.3, 27.0.3
API Levels: 26, 27
IDEs:
Android Studio: 3.1 AI-173.4819257
Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
npmPackages:
react: 16.4.1 => 16.4.1
react-native: ^0.56.0 => 0.56.0
npmGlobalPackages:
create-react-native-app: 1.0.0
react-native-cli: 2.0.1
Description
I am using webview to display chart for my app using chartjs and jquery library I am able to view charts fine in iOS debug mode but in release config, all I get is a white screen, I tried to check if atleast window.postMessage was working using setinterval code in my .html file, but I dont get any response from webview.
To check for window.postMessage I have also tried to possible solution from issue , still its not working
I have tried to inject the complete html code as well instead of serving the html as local asset file, still not success.
This issue is only present in iOS, works perfectly fine for android.

Hey all :wave: thanks for reporting this issue. Unfortunately, without a reproduction (a repo we can clone and see the unexpected behaviour) we're pretty limited in how we can help sorry.
I'll go ahead and close this issue given that there's been no further activity for a few months.
If you want this re-opened please let me know and help us out with a reproduction.


Have Results know how to summarize themselves

Based on type, scale, etc, results should figure out how to summarize themselves (eg last point for cumulative, mean for prevalence, etc)

Closed by #802


Logistic regression standardized and studentized residuals are slightly different from those given in R

This is apparent in the logistic integration test log_termination_0, which now compares several fit statistics to those given by R. For the standardized/studentized residual equality checks to pass the tolerance has to be increased to ~0.05, so they are approximately equal but not exact up to FPEs.
I'm including these residual functions in 0.0.10 since they are close and should still be useful, but the source of the difference should be investigated further.

This is likely due to the leverage component using the projection matrix from the data alone, not the full adjusted hat matrix from the fit result. These are only identical in the linear case.
This will be addressed in the weighted regression update.


Swap to always loading the configuration module shipped in the PS Version task

Removes the dependency on PSGallery for the VerisonPoweshellTask - fixes #1042
Updates all NPM packages (to address vulnerabilities identified in SYNK) on all Node based tasks in the extension - required else the pipeline fails

Well, this is proving troublesome. I have swapped the code so that the 1.5.0 configuration module shipped in the task is always used. This has a dependency on the metadata module.
If I run the script locally it works in all the PowerShell versions I have on my PC. It also work on a locally hosted build agent, but if I run it via a MS Hosted agent it fails
##[debug]Loading module from path 'D:\a\_tasks\VersionPowerShellModule_036278fe-8a47-426b-b5b3-bef05334db00\2.30.10\Configuration\1.5.0\Configuration.psd1'.
##[debug]Caught exception from task script.
##[debug]Error record:
##[debug]Import-Module : The required module 'Metadata' is not loaded. Load the module or remove the module from 'RequiredModules' in the file 'D:\a\_tasks\VersionPowerShellModule_036278fe-8a47-426b-b5b3-bef05334db00\2.30.10\Configuration\1.5.0\Configuration.psd1'.
##[debug]At D:\a\_tasks\VersionPowerShellModule_036278fe-8a47-426b-b5b3-bef05334db00\2.30.10\Update-PowerShellModuleVersion.ps1:69 char:1

I tried

putting the configuration and metadata modules  in their own folders and loading them both explicitly
I also tried putting them all in one folder

Neither work.
Any ideas @ChrisLGardner.
Just to make it seem a little stranger. On my locally hosted build agent the Pester 10.3.10 task is also failing with the error

VERBOSE: Importing function 'Import-Pester'.
##[error]Get-Command : The term 'Install-Module' is not recognized as the name of a cmdlet, function, script file, or operable
program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
##[debug]Processed: ##vso[task.issue type=error;]Get-Command : The term 'Install-Module' is not recognized as the name of a cmdlet, function, script file, or operable %0D%0Aprogram. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.%0D%0A
##[debug]task result: Failed

Hope there is not an issue there too

An easy fix there would be to store a copy of the Metadata module in the task folder and then update the PSModulePath within the task to include this folder and it should resolve that, you can do that by adding $Env:PSModulePath = "$PSScriptRoot;$Env:PSModulePath" before the Import-Module call.
I think this module used to be a nested module or something within the Configuration module which would explain why we haven't seen this before.
I'll have a look at the Pester issue, I don't think I've seen anyone else reporting problems with it but I'll run some test pipelines.

Sorting the PSModulePath has fixed the issue on the hosted agents. The revised task also work on my private agent.
As to the Pester issue, it seems to be local to my machine, s I would not worry about it too much
Thanks for the help. I will get this fix shipped


Appstudio update test-component-pac-xfor

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-xfor-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
40 seconds
build-container

✅ Succeeded
20 seconds
inspect-image

✅ Succeeded
16 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
13 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
13 seconds
label-check

✅ Succeeded
6 seconds
show-summary

✅ Succeeded
11 seconds
show-sbom


Should autoclick counter be reset after completion?

While working on #12 , I noticed that the autoclick counter isn't being reset after the configured number of clicks has been performed:
// Increment click counter and stop autoclicking if completed
self.click_counter += 1u64;
if click_amount != 0u64 && self.click_counter >= click_amount {
  self.is_autoclicking = false;
}

Therefore, when you re-enter autoclick mode, the app only performs a single click  (unless you set it to 0 = infinite clicking). Is this intentional?

https://github.com/MrTanoshii/rusty-autoclicker/blob/8267a3569e86928951b346436a4bf956edf1762c/src/app.rs#L196-L204
It's reset in start_autoclick().
I tested it and it works as intended on my the main branch on my side, can you provide detailed steps so I can try and reproduce it please?

@MrTanoshii Hm. I can't reproduce it either - neither in my branch nor in the main branch. I guess I had messed it up during development, sorry for the noise.


Add cobertura reporter for easier Jenkins integration

This would useful at reviewing the test failures in Jenkins and tracking tests over time.

Looks fine to me!


Issue #1309

Fixes issue #1309

@ljharb , I am done updating this PR. Thanks 😄

(also it'd be great to have the PR title explain the change being made, and confine the issue reference to the OP description)

@ljharb should be all set. thanks :)


CSV-214: adding cache for the line ending information

Description copied here from the JIRA ticket.
In my use-case , I have to read a CSV file, Mangle some columns and then write out a new csv file with those mangled columns.
I have gone through the parser code and I have found no usable way of getting the line ending information from the CSVParser object. The function readEndOfLine just consumes End of line whether it is CRLF or LF.
Now that problem is that when I am writing my file back using the CSVPrinter, I need to know what line ending my input file was. I could write an external function to do that. A better way would be to store that information in the CSVParser object and just use it.
To do that I am just saving the state in a variable and exposing that using a getter.
I have done some basic testing. and Submitting a pull request regarding that.

Coverage increased (+0.07%) to 94.411% when pulling ea446f003ef96b79f83237b392ac1a973c205957 on nmahendru:master into 4d0f2260061124878a8f884fab0148a5b29f942a on apache:master.

Thanks for the pull request!
Merged by @garydgregory  in https://github.com/apache/commons-csv/commit/aae6f90442ca09e2461e766a987b33316d9fa6be
It would be nice if you could close the pull request.
Thanks!

thanks for accepting it.


comictagger: init at 1.5.5

Adds comictagger, a multi-platform app for writing metadata to digital comics, written in Python and PyQt.
The github project can be found here:
https://github.com/comictagger/comictagger
The following are python dependencies needed:

py7zr
pyinstaller
pyinstaller-hooks-contrib
settngs
text2digits
unrar-cffi
wordninja

In this commit there are also those packages.
This is my first package, I expect "obvious" mistakes.
I've built and successfully run it on linux, for sure Mac dependencies are different.
Note: To work on wayland it needs to be called from the command line with a specific environemnt variable - I do not know how to handle it at package level:
$ QT_QPA_PLATFORM=wayland comictagger
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

I need to work on this before commiting again. All sort of weird issues with my local build.


No tag for 20200726

This version has been commited but no tag is available under https://github.com/pdfminer/pdfminer.six/releases

Done


Update conf files to follow the new file format

What does this PR do?
Updates all configuration file to strictly follow the rules defined here and to fix the errors raised by the new yaml validator https://github.com/DataDog/integrations-core/pull/3679/
Motivation
See https://github.com/DataDog/integrations-core/pull/3679/
Additional Notes
Before merging, we need to write descriptions for:

agent_metrics integration, variable named process_metrics
ceph integration, variables named ceph_cmd and ceph_cluster
docker_daemon integration, variable named tls_verify
pdh_check integration, variable named metrics

Also aerospike integration uses a lot of inline comments, making this integration a special case wouldn't seem good but should we rather allow inline comments all-together, or remove them in the aerospike configuration?
Review checklist (to be filled by reviewers)

[x] PR title must be written as a CHANGELOG entry (see why)
[x] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[x] PR must have changelog/ and integration/ labels attached
[x] Feature or bugfix must have tests
[x] Git history must be clean
[x] If PR adds a configuration option, it must be added to the configuration file.

This is insaaaaane super coool ! I'm going to add the descriptions missing directly in this PR. And fix the aerospike integration.

I just removed inline comments for Aerospike, since they don't bring any real value for a customer. If we get a specific customer feedback about this I will create a dedicated FAQ.


Tests for search-relevance-dashboards  failed due to sample data exists

During the 2.6.0, team noticed that sample data already exists when the page was visited and thus failed to see the button to add sample data.
See the video
https://ci.opensearch.org/ci/dbc/integ-test-opensearch-dashboards/2.6.0/5520/linux/x64/tar/test-results/3169/integ-test/functionalTestDashboards/without-security/test-results/cypress-videos/plugins/search-relevance-dashboards/1_query_compare.spec.js.mp4
Below is the link to all plugins.
https://opensearch-project.github.io/opensearch-dashboards-functional-test/site/?version=2.6.0&build_number=5520&test_number=3169
We need to verify whether other tests are adding sample data but didn't clean up.

Find suspicious tests in observability. Sample data was added but not removed.
https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/22322f04cc80710022e9c665a14e99fa387547d1/cypress/integration/plugins/observability-dashboards/0_before.spec.js#L21
https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/22322f04cc80710022e9c665a14e99fa387547d1/cypress/integration/plugins/observability-dashboards/8_after.spec.js#L9
@joshuali925 Could you please take a look?

@tianleh should cy.deleteAllIndices(); https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/5e601cc1a4fa9be79c75fa39db4086d6ac8def4a/cypress/utils/commands.js#L99-L106 also delete .kibana_*?

@tianleh should cy.deleteAllIndices();
https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/5e601cc1a4fa9be79c75fa39db4086d6ac8def4a/cypress/utils/commands.js#L99-L106

@joshuali925 Deleting .kibana_* may cause Kibana unhealthy.
I just run an experiment. Deleting opensearch_dashboards* shall be sufficient to re-enable the button to add sample data

@junqiu-lei
I notice there two test files are not cleaning up sample data.
https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/22322f04cc80710022e9c665a14e99fa387547d1/cypress/integration/plugins/custom-import-map-dashboards/documentsLayer.spec.js#LL14C20-L14C20
https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/22322f04cc80710022e9c665a14e99fa387547d1/cypress/integration/plugins/custom-import-map-dashboards/opensearchMapLayer.spec.js#L14
There are two ways to fix it.

use UI to remove sample data.
call delete indices to clean up. https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/22322f04cc80710022e9c665a14e99fa387547d1/cypress/integration/plugins/observability-dashboards/8_after.spec.js#L10

Hi @tianleh, checking, thanks the solution, is this a blocking for 2.6 release?

@junqiu-lei I think we fixed this in our repo. Can we add it here?

Yes, I am preparing the PR

observability is already calling cy.deleteAllIndices() https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/22322f04cc80710022e9c665a14e99fa387547d1/cypress/integration/plugins/observability-dashboards/8_after.spec.js#L8-L11, shouldn't cause this issue for others
plugins can call API to add sample data, or use UI but make it work if data is already added to make tests more defensive
https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/5e601cc1a4fa9be79c75fa39db4086d6ac8def4a/cypress/integration/plugins/reports-dashboards/01-create.spec.js#L16-L19


[termcolor]; Add termcolor

Describe the pull request

What does your PR fix? Fixes #

Add new library termcolor

Which triplets are supported/not supported? Have you updated the CI baseline?

support Win and Linux

Does your PR follow the maintainer guide?
Yes

CI failed but not caused by my PR:

          File path: [ D:/downloads/temp/vtk-vtk-m-f2aa6ad5be1a97e3fb41ef4680ee2c76c3434ac0.tar.gz ]
      Expected hash: [ 2f2a273f74d9a583df9e25a4792440d8d89652fa14b3153f2ea5afbd329b50970e7b9bd68e0ccd036baf5c1f3ad7a8302d95c01dbb30d9a46c045987eebf5370 ]
        Actual hash: [ 35e8a2c0ad6cd3c1f02a71a50d781c89f93909ad27030b406fd69f4fea5c1862c48a6e541fd07562947322c3a69bdfdb54206ae51bb86ef7a710f9e9898e9638 ]

  The file may have been corrupted in transit.  This can be caused by
  proxies.  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY
  environment variables to
  "https://user:***@your-proxy-ip-address:port/".

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:191 (test_hash)
  scripts/cmake/vcpkg_from_gitlab.cmake:123 (vcpkg_download_distfile)
  ports/vtk-m/portfile.cmake:40 (vcpkg_from_gitlab)
  scripts/ports.cmake:136 (include)

Error: Building package vtk-m:x64-windows failed with: BUILD_FAILED
Elapsed time for package vtk-m:x64-windows: 1.213 s
Starting package 100/104: vtk:x64-windows
Building package vtk[atlmfc,core,paraview,qt,vtkm]:x64-windows...
Error: Building package vtk:x64-windows failed with: CASCADED_DUE_TO_MISSING_DEPENDENCIES
Elapsed time for package vtk:x64-windows: 32 us

@NancyLi1013  Could you please take a look?

This is not related with your PR. We have submitted a PR #14884 to fix this.

The failures caused by vtk-m are not related with PR. I will rerun this PR once the regressions are solved.

/azp run

LGTM now, thanks for your PR @prehistoric-penguin.

Thanks!


GitHub: New Team for DCMFOSS

Product GitHub team name:
DCMFOSS
GitHub users to invite:
jschu1
ClosedSourcerer
Maintainer of the new GitHub Team:
ClosedSourcerer
validation / approved by: jschu1
New Team required because the old Team name has a company name within it.
Old Team needs to be disbanded at some point.
New Team responsible for the following REPOs:
https://github.com/eclipse-tractusx/demand-capacity-mgmt
https://github.com/catenax-ng/tx-demand-capacity-mgmt

Approved @ClosedSourcerer
Danke!

This ticket needs some internal clarification (System Team + FOSS), as to my knowledge the code will be made Eclipse-Tractus-X compliant (TRGs, legal stuff, etc.) and afterwards the repo is going to be archived as there will be no further development effort. Maybe there has been things changed, since I was a couple of days off.
If a new team will be created, the name will be product-dcm-foss. Maybe it's worth renaming the existing team instead of creating a new one, but that's not a final desicion yet.

@carslen This is exactly what I know. We need clarification. Also about the status on this  branch.

If a new team will be created, the name will be product-dcm-foss. Maybe it's worth renaming the existing team instead of creating a new one, but that's not a final desicion yet.

I am fine with renaming the existing team. However the current team owner told me he could not change the name.

If a new team will be created, the name will be product-dcm-foss. Maybe it's worth renaming the existing team instead of creating a new one, but that's not a final desicion yet.

I am fine with renaming the existing team. However the current team owner told me he could not change the name.

Hi @ClosedSourcerer,
we can take over renaming for you. Then we also do not have to maintain the ownership of repos.
Will you be continuing the DCM efforts in Tractus-X?

we can take over renaming for you. Then we also do not have to maintain the ownership of repos. Will you be continuing the DCM efforts in Tractus-X?

@SebastianBezold
Effort is a high an mighty word considering the circumstances, but yes I will attempt to prevent this from drowning and having wasted a 6 figure investment...
Considering that everyone who is currently part of that team will be off-boarding on friday.
Yeah that is the plan, for now.
However the Ownership of the Depo needs to changed as well, because the current owner will off-board on friday.

Ok understood. So my suggestion would be the following.
I will rename the existing team to get rid of company names. This was anyway never intended.
Second, I will add you to the team and grant you the team maintainer role.
This way you can on- and offboard members to the team.
Repo ownership (or better permissions) are always granted to teams and not individuals, so I don't see an issue there.
Would that work for you @ClosedSourcerer

@SebastianBezold
That should work nicely. Thank you.

Hi @ClosedSourcerer,
finally found the time to actually do that.
You are now maintainer of the renamed team product-dcm
Please check, if you now have access to all the repos you need and close the issue if nothing is missing

Closing for now. Feel free to re-open, if there are additional topics, that we missed


Broken hands.

On my friends HL2RP server i selected my hands tool but the fists are raised on screen but when i click it doesnt punch so when i hold R the fists are still there and i can punch but when i move my mouse the screen slightly shakes everytime i move my mouse so it's hard to aim this also happens with guns how do i fix this problem?

Im currently having the same issue with this aswell. The fists do not do damage, and only push.

Disable Free-Aiming in the configuration of the server.

I haven't been able to reproduce this.


generate pdf only after image has been retrieved from the web

I am including a logo that is saved online into the pdf. By the time the image is downloaded the pdf is generated and the image is not included - any suggestions?

You can try download bitmap first then set that bitmap to image view instead of using Picasso or glide.

Closing this now, above solution should be applied for any such case.


Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.2.1

medallia-word2vec is showing error while i run the function searcher.
[cosineDistance("money", "Honey")]
Error comes like
Exception in thread "main" com.medallia.word2vec.Searcher$UnknownWordException: Unknown search word 'Honey'
at com.medallia.word2vec.SearcherImpl.getVector(SearcherImpl.java:83)
at com.medallia.word2vec.SearcherImpl.cosineDistance(SearcherImpl.java:45)
at com.medallia.word2vec.Word2VecExamples.interact(Word2VecExamples.java:130)
at com.medallia.word2vec.Word2VecExamples.demoWord(Word2VecExamples.java:73)
at com.medallia.word2vec.Word2VecExamples.main(Word2VecExamples.java:33)
BUILD FAILURE
Total time: 15.075s
Finished at: Fri May 22 13:26:27 IST 2015
Final Memory: 5M/15M
Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.2.1:exec (default-cli) on project medallia-word2vec: Command execution failed. Process exited with an error: 1 (Exit value: 1) -> [Help 1]
To see the full stack trace of the errors, re-run Maven with the -e switch.
Re-run Maven using the -X switch to enable full debug logging.
For more information about the errors and possible solutions, please read the following articles:
[Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException.
PLease i need kind help.

Hi
As the error message says, your trained vocabulary does not contain the word "Honey". I think this depends on your text corpus and your training settings.

I think this is a duplicate of #13.

Thank u for your kind reply. If the word is not in vocabulary why "Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.2.1" error comes?
Moreover, how can i handle this type of words rather printing errors? If word is not there, it should print status. Actually i am getting the link where i should do the changes.

@guerda you are right, issue is duplicate of #13. I am trying to overcome since last few days. But could not. In other cases system is working fine.

@dwijenrudrapal It depends, how you call the maven target. It seems that you use mvn execute, which will listen to the exit code of the Java class. As the UnknownWordException is thrown and not catched, it will set the exit code to non-zero. Therefore, the Maven execute will say that the command failed.
I recommend to include the word2vecjava library in an Eclipse or a Netbeans project and write your own code to include the calls. Then you can catch the UnknownWordException and output the status.

based on my English text data i edited the code to handle the errors for non-vocabulary words. I found the implementation works fine for English text only. When i try to train the system for Bengali text, system is not compatible.  Its printing ? for each Bengali characters in the train file. Can you kindly help me on this issue?

based on my English text data i edited the code to handle the errors for
non-vocabulary words. I found the implementation works fine for English
text only. When i try to train the system for Bengali text, system is not
compatible. Its printing ? for each Bengali characters in the train file.
Can you kindly help me on this issue?
On Fri, May 22, 2015 at 4:44 PM, Philip Gillißen<EMAIL_ADDRESS>wrote:

@dwijenrudrapal https://github.com/dwijenrudrapal It depends, how you
call the maven target. It seems that you use mvn execute, which will listen
to the exit code of the Java class. As the UnknownWordException is thrown
and not catched, it will set the exit code to non-zero. Therefore, the
Maven execute will say that the command failed.
I recommend to include the word2vecjava library in an Eclipse or a
Netbeans project and write your own code to include the calls. Then you can
catch the UnknownWordException and output the status.
—
Reply to this email directly or view it on GitHub
https://github.com/medallia/Word2VecJava/issues/21#issuecomment-104623018
.

Are you using a working UTF-8 console to print the words? Arabic text works fine with Word2VecJava, see https://github.com/guerda/Word2VecJava/blob/bug-arabic-text/src/test/java/com/medallia/word2vec/Word2VecTest.java#L179
I suspect you are printing the words with an incorrect encoding.

Finally I managed to work your implementation for Bengali Language. Both
Write and read function i little modified. I used "UTF-8" as an argument in
these functions. So files are opening in and writing to in desired script.
One important observation is that, for two Bengali words i am getting
cosine distance more than 1 (like 1.65874), which is not expected. Cosine
distance must be 0-1. So, while i am calculating sentence level similarity,
score is coming more than 1. Working on this issue.
I hope, You know Bengali is one major language in India. I am doing
research on Bengali language. I hope, I will get your kind help from you in
future also.
Regards
Dwijen
On Sun, May 24, 2015 at 5:23 PM, Philip Gillißen<EMAIL_ADDRESS>wrote:

Are you using a working UTF-8 console to print the words? Arabic text
works fine with Word2VecJava, see
https://github.com/guerda/Word2VecJava/blob/bug-arabic-text/src/test/java/com/medallia/word2vec/Word2VecTest.java#L179
I suspect you are printing the words with an incorrect encoding.
—
Reply to this email directly or view it on GitHub
https://github.com/medallia/Word2VecJava/issues/21#issuecomment-105006133
.

Good that this could be resolved. About the cosine distance larger than 1: can you provide a test case, where this happens? So we can reproduce this error and hopefully fix it. A new issue would be suitable for this.
I would suggest to close this issue, as your initial problem seems to be fixed, OK?


rework chart

follow up of https://github.com/owncloud/ocis-charts/pull/1
Description
Adds extra flexibility and configurability to the chart and adds the option to add labels, configure volumes...
I took the grafana charts as a role model.
TODO

this PR does only functional changes
documentation and CI still needs to be added / adapted: tracking in https://github.com/owncloud/ocis-charts/issues/3

Uuuuh nice using micro kubernetes registry!

@rhafer I would like to merge it as it is and tackle documentation and CI in a separate PR. The state above is running at our partner's rancher based cluster and I would love to have a look together with you.
I'm using a helmfile for this setup like below:
releases:
  - name: ocis
    namespace: ocis-test
    chart: ./ocis
    values:
      - image:
          pullPolicy: Always

      - externalDomain: ocis.ocis-kube.owncloud.works

      - ingress:
          enabled: true
          ingressClassName: nginx
          annotations:
            cert-manager.io/issuer: "ocis-certificate-issuer"
          tls:
            - hosts:
              - ocis.ocis-kube.owncloud.works
              secretName: ocis-ocis-kube-tls

      - replicas: 5

      - autoscaling:
          enabled: false
          minReplicas: 1
          maxReplicas: 10
          metrics:
          - type: Resource
            resource:
              name: cpu
              targetAverageUtilization: 60
          - type: Resource
            resource:
              name: memory
              targetAverageUtilization: 60

      - resources:
          limits:
            cpu: 100m
            memory: 256Mi
          requests:
            cpu: 100m
            memory: 256Mi

      - thumbnails:
          persistence:
            enabled: false
      - settings:
          persistence:
            enabled: false
      - storageMetadata:
          persistence:
            enabled: false
      - storageSharing:
          persistence:
            enabled: false
      - storageUsers:
          persistence:
            enabled: false
      - store:
          persistence:
            enabled: false

      - extraResources:
          - |
            apiVersion: cert-manager.io/v1alpha2
            kind: Issuer
            metadata:
              name: ocis-certificate-issuer
              namespace: ocis-test
            spec:
              acme:
                server: https://acme-v02.api.letsencrypt.org/directory
                email<EMAIL_ADDRESS>                privateKeySecretRef:
                  name: ocis-certificate-issuer
                solvers:
                - http01:
                    ingress:
                      class: nginx