[FEATURE] MRU sort for the picker

Is your feature request related to a problem? Please describe.
Session list is not sorted
Describe the solution you'd like
Would be nice if there was a passthrough argument for telescope to provide a custom sorter, also provide an implementation for MRU sort.
Describe alternatives you've considered
There are no alternatives really afaict

for now I have come up with:
    config = function()
      local opts = {
        auto_restore_enabled = false,
        auto_session_use_git_branch = true,
      }

      local rg_available = 1 == vim.fn.executable "rg"
      if rg_available then
        opts.session_lens = {
          find_command = { "rg", "--files", "--color", "never", "--sortr", "modified" }
        }
      end
      require('auto-session').setup(opts)
      require('telescope').load_extension('session-lens')
    end,

although not sure if file's modified timestamp is a good measure of recent


Functions wrapped in __importDefault with allowSyntheticDefaultImports emit incorrect function calls

The following code:
import allSettled from "promise.allsettled";

const resolved = Promise.resolve(42);
const rejected = Promise.reject(-1);

allSettled([resolved, rejected]).then(results => {
  console.log(results);
});

Is transpiled into the following JS with esModuleInterop & allowSyntheticDefaultImports:
"use strict";
var __importDefault =
  (this && this.__importDefault) ||
  function(mod) {
    return mod && mod.__esModule ? mod : { default: mod };
  };
Object.defineProperty(exports, "__esModule", { value: true });
const promise_allsettled_1 = __importDefault(require("promise.allsettled"));
const resolved = Promise.resolve(42);
const rejected = Promise.reject(-1);
promise_allsettled_1.default([resolved, rejected]).then(results => {
  console.log(results);
});

Full tsconfig.json:
{
  "compilerOptions": {
    "target": "es2018",
    "module": "commonjs",
    "strict": true,
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
  },
  "include": [
    "src"
  ]
}

(CodeSandbox.)
Executing this code leads to:
TypeError: #<Object> is not a constructor
    at Object.resolve (<anonymous>)
    at Object.PromiseResolve (/sandbox/node_modules/es-abstract/es2018.js:160:10)
    at /sandbox/node_modules/promise.allsettled/implementation.js:26:24
    at Function.from (<anonymous>)
    at Object.allSettled (/sandbox/node_modules/promise.allsettled/implementation.js:19:22)
    at Object.allSettled [as default] (/sandbox/node_modules/promise.allsettled/index.js:16:9)
    at Object.<anonymous> (/sandbox/src/demo.ts:6:11)
    at Module._compile (internal/modules/cjs/loader.js:776:30)
    at Module.m._compile (/sandbox/node_modules/ts-node/src/index.ts:536:23)
    at Module._extensions..js (internal/modules/cjs/loader.js:787:10)

That is because the promise.allsettled package handles this according to the spec, see https://github.com/es-shims/Promise.allSettled/issues/5. The author of that library is @ljharb who will have deeper understanding of the specifics than I do. (Also, thanks for nudging me to report this issue!)
The problem is most likely the emitted function call, which looks like this:
promise_allsettled_1.default([resolved, rejected]).then(results => {
  console.log(results);
});

In comparison, Babel + TS emits this (which works):
(0, _promise.default)([resolved, rejected]).then(function (results) {
  console.log(results);
});

Babel playground.
TypeScript Version:  3.7.2
Search Terms: esModuleInterop, allowSyntheticDefaultImports, emit, transpile, downlevel compile, ES Modules, default, wrapper, Node.js, module.

A simpler repro case would be importing module.exports = function () { 'use strict'; return !!this; } (with esModuleInterop and synthetic imports enabled, of course) - a correct downleveling would make invoking this function with no implied receiver return false; TS’s output currently seems to make it return true.

AFAICT the same bug exists for named imports.
// foo.ts
export function foo() { return !!this; }

// bar.ts
import { foo } from './foo.js';
foo();

// transpiled bar.js
var foo_js_1 = require('./foo.js');
foo_js_1.foo(); // this line should be `(0, foo_js_1.foo)()`

@ljharb please correct me if I'm wrong.

Yes, that’s the same bug, thanks.

Fix is up at #35877

@rbuckton Sorry to disturb you, but your PR seems to wrap all imported symbols with (0, ...), but I think it's not necessary and may be skipped when the target is a "const arrow function".


[WIP] QFI with the primitives

Summary
This PR adds QFI using Linear Comb. of Unitaries with the primitives.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 98 (1.02%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 84.241%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/algorithms/gradients/utils.py
1
37
2.7%

qiskit/algorithms/gradients/qfi.py
0
61
0.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/squ.py
2
79.78%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
57866

Relevant Lines:
68691

💛  - Coveralls

@dlasecki @woodsp-ibm Thank you so much for reviewing the PR! I fixed the codes based on the comments. I also changed to use options  instead of run_options. I think it's ready now.

Thank you for the comments! I will check them.


Does the order of condition value matter in query?

CrateDB version: 2.2.5
JVM version: 8
OS version / environment description: CentOS 6.5
Hi there,
I found it interesting that the order of condition values matters in a crate query. Pleaes check following sql.
select ymd, h, sum(lrate) from realChanFlow_test
    where ymd in (20180101, 20180102, 20180103, 20180104, 20180105, 20180106, 20180107, 20180108, 20180109, 20180110, 20180111, 20180112, 20180113, 20180114, 20180115, 20180116 )
    and area in ('cn')
    and isp in ('other', 'jyw', 'sarft' )
    and domain in (
        select domain from(select domain from domain_100000 limit 60000) inner_table order by domain)
group by ymd, h order by ymd, h;

The table realChanFlow_test is partitioned by ymd and isp. 10 million rows are populated into one ymd partition, distributed into 8 servers.
The data in domain_100000 table is unsorted and the query returns in 10 seconds.
However, if I remove the order by domain clause. the query never returns.
I am not a cratedb/elasticsearch/lucene expert and I have no idea on this scenario. Please kindly help me to understand the reason. Any inputs would be helpful.
BTW, if the order does matter, is that a good idea to sort it in the SQL optimizer?
Best regards,
Ryan

@matriv Really appreciated. :)

@fanwu72 No worries, thank you for the detailed info you provided!


launch variable extension adds $ again

Verifying #8370
Have the following launch.json
{
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Launch Program",
            "program": "$"
        }
    ]
}

invoke code completion after $ and pick a variable proposal

You end up with something like
"program": "$${relativeFile}"

@aeschli nice catch, I am now adjusting a positoin based on the $ sign since it is always the start of my suggestion. You can check out the code and let me know what you think, I tried it out and seems to work nice

@isidorn Nice work!

@aeschli thanks


[Bug][llama_index-0.9.1]: ValueError: "AzureOpenAI" object has no field "_http_client"

Bug Description
ValueError: "AzureOpenAI" object has no field "_http_client"
Version
0.9.1
Steps to Reproduce
Line 96 in ../site-packages/llama_index/llms/azure_openai.py:96).
# Use the custom httpx client if provided.
# Otherwise the value will be None.
self._http_client = http_client
Relevant Logs/Tracbacks
No response

I am also facing the same issue

Same - yes i can copy class and adapt, but it's not ideal and a bit strange for this very popular method for OpenAI model access. Automatic tests?

I think this is a pydantic v1 vs v2 issue -- all our tests are using v2 now

Fixed in #8955 .
Thanks.

Is the issue fixed for AzureOpenai?

@Arokamal yes you can pass in http_client in the latest version of llama index


Fix for Syck YAML loading error: "method 'load_dj'"

Fixes error when loading job with newer Ruby and Syck is used as YAML engine (tenderlove/syck gem):
"Job failed to load: undefined method `load_dj' for Syck:Module"
Fixes #786

Hey folks, I'm having the same issue and this fix works when YAML uses Syck. Could we merge this change? Thanks!


tf.bitwise.invert throws SystemError when input is uint64 tensor

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
2.13.0.dev20230204
Custom Code
Yes
OS Platform and Distribution
Ubuntu 20.04.4 LTS
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
tf.bitwise.invert throws SystemError when input is uint64 tensor

Standalone code to reproduce the issue
import tensorflow as tf

tf.bitwise.invert(tf.constant(-2147483648, dtype=tf.uint64))

Relevant log output
SystemError: <class 'tensorflow.python.framework.ops.EagerTensor'> returned a result with an error set

@gaikwadrahul8 Thanks for your reply!


feat(example): set outgoing response before writing

Feature or Problem
This PR changes the ordering in our example HTTP components to ensure we are setting the response outparam before writing bytes to the body.
Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

A bit surprised it works like this but 🤷

Me too! But it does kind of make sense. I ran into some issues when writing larger payloads using splice or read/write loops because without setting the ResponseOutparam I can write a certain amount to an internal buffer, but that isn't actually streamed to the requester. This way we will immediately write/flush the bytes back to the requester and then we can "finish" the body at the end.


configure_file() & dependencies

my testcase:
asversion = configure_file(
  input : 'as-version.h.in',
  output : 'as-version.h',
  configuration : conf)
sources = [
  'src.c',
  asversion]

I'm getting error if I will specify manually this name of file: Tried to add non-existing source as-version.h.

You don't need to add the file to a target (and indeed should not). It is generated before build starts so it's always there. The dependency system will pick it up automatically.
The only reason to do this would be to have it show up in a list of files in an IDE. Then it should go into extra_files, though it will probably give you the same error. We'll get that fixed at some point.

@jpakkane I need to specify this file to gobject-introspection. Without it I will have problems with girs.
So I think we need to implement this.

Ah, ok, that makes sense.

Fixed in trunk. Forgot to write the closing comment so closing manually.

@jpakkane actually it's not fixed.


v12: bump tendermint, remove ignite, sdk, ibc

This PR:

updates ibc-go to v3.4.0
updates tendermint to use the informal fork, v0.34.26
updates cosmos-sdk to v0.45.13
moves the openapiconsole so that we no longer pull the ignite cli into dependencies

Not tested on Uni, so going to have to say no to this one. Will be apart of v13 already though

I do not really see a reason for this in v12 since we are moving to v13 so quickly which will have these changes & no way to test easily with Uni since its on v13. Closing


Travis CI: Test on the current Python

OK with that. However, in order to avoid mixing generated code too much with the "source", I maintain this kind of code in the master branch which I then merge into the master_gen_v4_00 along with the generated code. In some way I consider generated code a bit like if it was binary code complied from a source code (on master). So I 'ill merge this on master 1st and then merge it into master_gen_v4_00.

Closing in favor of #33 because Travis seems to be gone.


Custom environment variables

Allow user to specify environment variables to pass through to the Singularity container (using SINGULARITYENV_[variable] as described here)

We already support this (via the env attribute of the input file). Would be good to add some tests though.

This has been in for a while


Ser-42 upgrade to webpack 5

Description
Update webpack via encore so that we can get rid of security issues further down dependency chain.
Had no idea how webpack works so spent ages trying to work out why it wasn't finding correct paths. In true serve pairing style, roped in Ste who was super knowledgeable but still took us ages working out what it was doing with the paths as errors aren't super obvious..
Pretty sure noones using IE11 but added this line to make it compliant: config.target = ['web', 'es5'];
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Issues Resolved
SER-42 [List any existing issues this PR resolves]
Merge check List

[ ] New functionality includes testing
[ ] New functionality has been documented in the README if applicable
[ ] Approved by PMs

Closing in favour of #425


[BUG] Router go-chi not handling URLs with parameters correctly

Describe the bug
When trying to make a request to SMD, URLs that contain a parameter such as /hsm/v2/Inventory/RedfishEndpoints/{xname} or /hsm/v2/Inventory/ComponentEndpoints/{xname} will return a 400 response code.
curl http://<IP_ADDRESS>:27779/hsm/v2/Inventory/ComponentEndpoints/x0c0s1b0
{"type":"about:blank","title":"Bad Request","detail":"HMSDS method arg is nil","status":400}

More relevant information from SMD:
smd              | 2024/02/02 20:33:17.227162 smd-api.go:2640: doComponentEndpointGet(): trying...
smd              | 2024/02/02 20:33:17.227491 hmsds-tx-postgres.go:3272: Error: GetCompEndpointByIDTx(): xname was empty
smd              | 2024/02/02 20:33:17.227863 smd-api.go:2645: doComponentEndpointGet(): Lookup failure: () HMSDS method arg is nil
smd              | <IP_ADDRESS> - - [02/Feb/2024:20:33:17 +0000] "GET /hsm/v2/Inventory/ComponentEndpoints/x0c0s1b0 HTTP/1.1" 400 93 "" "curl/8.4.0"

To Reproduce
Steps to reproduce the behavior:

Start the OChami services
Make a curl request: curl http://<IP_ADDRESS>:27779/hsm/v2/Inventory/ComponentEndpoints/x0c0s1b0
See error

Expected behavior
Making a request to these endpoints should not return a 400. The request should return a response as described in the swagger_v2.yml file.
Desktop (please complete the following information):

OS: MacOS Sonoma 14.2

This looks to me like we're trying to use mux.Vars when we should be using chi.URLParam(r,"xname")
https://github.com/OpenCHAMI/smd/blob/2697169fc26322c4fe81d345df7e451b959f18dc/cmd/smd/smd-api.go#L2641-L2643
Since we dropped mux in favor of chi, anything that relies on mux is likely to fail.

That was the issue and changing the top to lines to xname := chi.URLParam(r, "xname") fixed it.


If nonzero min_value used, ProgressBar.start() fails

ProgressBar.start() always calls ProgressBar.update(0).  When a ProgressBar is create with a nonzero min_value parameter, this call results in a ValueError exception due to 0 being outside the range of permissible values.

That's a fair point, I've just updated the code to start at min_value instead. I believe that should fix it properly :)


Change dropdown options on Publications form

Change "Student Thesis/Dissertation" to "Graduate Student Research"
Change "Other Student Research Paper" to "Undergraduate Student Research"
All other fields should stay the same.

@jamstooks - This might have implications for the Publications IRC import - the names used in the file are the ones we are changing to, so there's a chance this ticket has to be resolved concurrently.


Reimplement a4a06f8016 in terms of symbol ownership

Motivation

Now that we have proper symbol-based package ownership, we don't need to
use this hack which relies on the canonical loc of a Symbol being the
source of truth for that symbol's package.
Test plan

The test included in a4a06f8016 covers this refactor.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_PdtSj8FKRm9Mhf
→ https://go/builds/bui_PdtS1heIhGdRBY
→ https://go/builds/bui_PdtS01wlHKqo4T


More standard date format?

In header, we have "15th and 16th of August, 2018, Harvard University, Cambridge"
Since this is hosted in US, I think it may be more standard to have:
"August 15-16, 2018, Harvard University, Cambridge"

this way it will also be consistent with the rest of the site, such as: "The first annual R/Pharma conference will be held Wednesday, August 15th and Thursday, August 16th at Harvard University, Cambridge, Massachusetts, USA."


⚠ Secret Website 3 has degraded performance

In 6113acc, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 1821 ms

Resolved: Secret Website 3 performance has improved in fcc565a.


Small Sequencer Fixes found during testing

Problems/omissions encountered during debugging of the sequencers in development of VdbBuilder/komodo.

@phantomjinx: in addition to @rhauch's comments, pls open a JIRA for this & attach the PR. Also, if possible, squash the 3 commits into 1. Thanks.

JIRA created and commits squashed.

@rhauch are you ok with this ?

Looks good.


Closing webcam

Can I know how to close the webcam once i click on the back button in java. Because now I am being shown this error "Exception in thread "AWT-EventQueue-0" java.lang.IllegalStateException: Cannot change resolution when webcam is open, please close it first"

Hi, this can be done simply by calling:
webcam.close()

Thanks alot that helped me to solve my problem.


modify standard_error calculation to take absolute value before sqrt,…

… removing ComplexWarning.  Fixes issue #138

minor change fixes issue #138


PR not checked out correctly for current version command

adrianreber/criu#6 added a new make target (show-version) to determine the version. But make show-version prints
No rule to make target 'show-version'

My current assumption is, that there is some issue in checking out the PR, but will need some evidence to prove this.

I'm afraid the problem with Adrian's PR was that current_version_command is broken - can't prove that, but after I dropped that, it started to work for me.
I'm gonna prioritize the card to deprecate current_version_command and the create_archive_command.

Indeed! I was playing with this in csomh/hexchat#1. Using actions."get-current-version" worked, while using current_version_command did not.


Save well position changes during re-analysis

I added code that allows the user to tweak the saved well positions.  I should probably save that in case they want to rerun it using the newer saved well positions.  Should also allow them to select among different saved well positions files upon reanalysis.

This was implemented in branch 'issue16' and was merged into the parent branch 'issue25'.


fix: error dialog should allow scrolling of error text

Description
Bug: error popup was clipping error messages and the modal title was blank
Refactored formatErrorTitle to default to Error instead of blank ''
Task Item
closes #6727
Screenshots

Coverage increased (+0.001%) to 50.89% when pulling 50c317825e2898acf6718db133b2c1fad38882a4 on natalgar/6727-fix-error-popup into 84f3a111cec02ce4dad9c53f64f538b29c32e48a on main.


Dns condition

report a clusterdeployment condition when we are waiting for the dnszone to become ready
retport a metric recording how long we waited for the dnszone to become ready

/lgtm


fix(API): Fix unstable AWSAPICategoryPluginResetTests.testReset()

My best analysis as to what is happening is the following:
Test case Post:
https://github.com/aws-amplify/amplify-ios/blob/main/AmplifyPlugins/API/AWSAPICategoryPluginTests/AWSAPICategoryPlugin+RESTClientBehaviorTests.swift
Test case reset:
https://github.com/aws-amplify/amplify-ios/blob/main/AmplifyPlugins/API/AWSAPICategoryPluginTests/AWSAPICategoryPlugin+ResetTests.swift
Base class
https://github.com/aws-amplify/amplify-ios/blob/main/AmplifyPlugins/API/AWSAPICategoryPluginTests/AWSAPICategoryPluginTestBase.swift
Imagine that test case “post” is running.  The Post test first kicks off an operation.  The operation continues to execute even after the actual test case finishes.  The problem is that we haven’t killed the operation associated with the Post unit test, and this operation executes while the next test (Reset test) has modified the state of the system, which causes a data race.
Since we are immediately assigning apiPlugin in the base class's setup() function, I am proposing to call reset after each of the unit tests.  The only time we do not want reset to be called is if we already called reset.  If we call reset twice, we will hit an implicitly unwrapped optional which will cause a segfault.  To prevent this from happening we set it to nil so that the tearDown() code is rendered inactive.
The following is the data race output by Xcode.  Notice that one of the threads is an AWSAPIPlugin thread, which is completely out of the scope of testReset() :)
Test Case '-[AWSAPICategoryPluginTests.AWSAPICategoryPluginResetTests testReset]' started.
==================
WARNING: ThreadSanitizer: data race (pid=2145)
  Read of size 1 at 0x7b0c000f2060 by thread T7:
    #0 HubCategory.plugin.getter HubCategory.swift:43 (Amplify:x86_64+0xffe7f)
    #1 HubCategory.dispatch(to:payload:) HubCategory+ClientBehavior.swift:15 (Amplify:x86_64+0xfe1f9)
    #2 AmplifyOperation.dispatch(result:) AmplifyOperation.swift:175 (Amplify:x86_64+0x3ce1e)
    #3 AWSGraphQLOperation.complete(with:response:) AWSGraphQLOperation+APIOperation.swift:52 (AWSAPICategoryPlugin:x86_64+0x380cf)
    #4 protocol witness for APIOperation.complete(with:response:) in conformance AWSGraphQLOperation<A> <compiler-generated> (AWSAPICategoryPlugin:x86_64+0x398ce)
    #5 AWSAPIPlugin.urlSessionBehavior(_:dataTaskBehavior:didCompleteWithError:) AWSAPIPlugin+URLSessionBehaviorDelegate.swift:14 (AWSAPICategoryPlugin:x86_64+0x4db43)
    #6 AWSAPIPlugin.urlSession(_:task:didCompleteWithError:) AWSAPIPlugin+URLSessionDelegate.swift:32 (AWSAPICategoryPlugin:x86_64+0x5be5c)
    #7 @objc AWSAPIPlugin.urlSession(_:task:didCompleteWithError:) <compiler-generated> (AWSAPICategoryPlugin:x86_64+0x5c020)
    #8 <null> <null>:2 (CFNetwork:x86_64+0x1bba93)
    #9 _dispatch_client_callout <null>:2 (libdispatch.dylib:x86_64+0x3533)

  Previous write of size 1 at 0x7b0c000f2060 by main thread:
    #0 HubCategory.configure(using:) HubCategory+CategoryConfigurable.swift:23 (Amplify:x86_64+0xfd58f)
    #1 HubCategory.configure(using:) HubCategory+CategoryConfigurable.swift:27 (Amplify:x86_64+0xfdcb5)
    #2 protocol witness for CategoryConfigurable.configure(using:) in conformance HubCategory <compiler-generated> (Amplify:x86_64+0xfdec2)
    #3 static Amplify.configure(_:using:) AmplifyConfiguration.swift:152 (Amplify:x86_64+0x1d87e)
    #4 static Amplify.configure(_:) AmplifyConfiguration.swift:110 (Amplify:x86_64+0x1be5c)
    #5 AWSAPICategoryPluginTestBase.setUp() AWSAPICategoryPluginTestBase.swift:63 (AWSAPICategoryPluginTests:x86_64+0x30a14)
    #6 @objc AWSAPICategoryPluginTestBase.setUp() <compiler-generated> (AWSAPICategoryPluginTests:x86_64+0x314d4)
    #7 __70-[XCTestCase _shouldContinueAfterPerformingSetUpSequenceWithSelector:]_block_invoke_2 <null>:2 (XCTest:x86_64+0x2ac3f)

  Location is heap block of size 33 at 0x7b0c000f2040 allocated by main thread:
    #0 __sanitizer_mz_malloc <null>:2 (libclang_rt.tsan_iossim_dynamic.dylib:x86_64+0x4f2fa)
    #1 _malloc_zone_malloc <null>:2 (libsystem_malloc.dylib:x86_64+0x12a0f)
    #2 static Amplify.reset() Amplify+Reset.swift:67 (Amplify:x86_64+0x9c34)
    #3 AWSAPICategoryPluginTestBase.setUp() AWSAPICategoryPluginTestBase.swift:60 (AWSAPICategoryPluginTests:x86_64+0x3079a)
    #4 @objc AWSAPICategoryPluginTestBase.setUp() <compiler-generated> (AWSAPICategoryPluginTests:x86_64+0x314d4)
    #5 __70-[XCTestCase _shouldContinueAfterPerformingSetUpSequenceWithSelector:]_block_invoke_2 <null>:2 (XCTest:x86_64+0x2ac3f)

  Thread T7 (tid=66130, running) is a GCD worker thread

SUMMARY: ThreadSanitizer: data race HubCategory.swift:43 in HubCategory.plugin.getter

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

Great find, really appreciate the deep dive on this!


No documentation for "Run job" stage

This stage is part of the Kubernetes v1 provider. See spinnaker/spinnaker#2587 for a possible v2 version.

Fixed in #1015.


feature: provide mock module

Ⅰ. Describe what this PR did
Provide mock module for increase unit test coverage rate.
Ⅱ. Does this pull request fix one issue?

#1436
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1478 into develop will decrease coverage by 0.42%.
The diff coverage is 23.7%.

@@              Coverage Diff              @@
##             develop    #1478      +/-   ##
=============================================
- Coverage      45.76%   45.33%   -0.43%
- Complexity      1666     1692      +26
=============================================
  Files            347      352       +5
  Lines          12711    12943     +232
  Branches        1589     1592       +3
=============================================
+ Hits            5817     5868      +51
- Misses          6252     6430     +178
- Partials         642      645       +3

Impacted Files
Coverage Δ
Complexity Δ

...va/io/seata/rm/datasource/mock/MockConnection.java
100% <100%> (ø)
3 <3> (?)

...ata/rm/datasource/mock/MockExecuteHandlerImpl.java
100% <100%> (ø)
3 <3> (?)

...ava/io/seata/rm/datasource/mock/MockResultSet.java
100% <100%> (ø)
3 <3> (?)

...seata/rm/datasource/mock/MockDatabaseMetaData.java
4.97% <4.97%> (ø)
6 <6> (?)

...n/java/io/seata/rm/datasource/mock/MockDriver.java
84.37% <84.37%> (ø)
11 <11> (?)

...server/store/file/FileTransactionStoreManager.java
45.99% <0%> (-1.4%)
19% <0%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


bump sync plugin for credential/secret change

for https://trello.com/c/SGvNKyMM/1154-5-make-openshift-secrets-available-in-jenkins-as-credentials-jenkinsintegration

[merge]

Plugin list now:
ace-editor:1.0.1
authentication-tokens:1.1
blueocean-autofavorite:1.0.0
blueocean-commons:1.1.6
blueocean-config:1.1.6
blueocean-dashboard:1.1.6
blueocean-display-url:2.0
blueocean-events:1.1.6
blueocean-github-pipeline:1.1.6
blueocean-git-pipeline:1.1.6
blueocean-i18n:1.1.6
blueocean:1.1.6
blueocean-jwt:1.1.6
blueocean-personalization:1.1.6
blueocean-pipeline-api-impl:1.1.6
blueocean-pipeline-editor:0.2.0
blueocean-pipeline-scm-api:1.1.6
blueocean-rest-impl:1.1.6
blueocean-rest:1.1.6
blueocean-web:1.1.6
branch-api:2.0.9
cloudbees-folder:6.0.4
conditional-buildstep:1.3.1
config-file-provider:2.16.2
credentials-binding:1.12
credentials:2.1.13
display-url-api:2.0
docker-commons:1.8
docker-workflow:1.9
durable-task:1.13
favorite:2.3.0
git-client:2.4.4
github-api:1.85.1
github-branch-source:2.0.8
github:1.26.0
github-organization-folder:1.6
git:3.3.2
git-server:1.5
handlebars:1.1
icon-shim:2.0.3
jackson2-api:2.7.3
job-dsl:1.60
jquery-detached:1.2.1
junit:1.20
kubernetes:1.0
mailer:1.20
mapdb-api:<IP_ADDRESS>
matrix-auth:1.5
matrix-project:1.7.1
mercurial:1.54
metrics:<IP_ADDRESS>
momentjs:1.1
multiple-scms:0.6
openshift-client:1.0.3
openshift-login:1.0.1
openshift-pipeline:1.0.53
openshift-sync:0.9.2
parameterized-trigger:2.35.1
pipeline-build-step:2.5.1
pipeline-github-lib:1.0
pipeline-graph-analysis:1.3
pipeline-input-step:2.8
pipeline-milestone-step:1.3
pipeline-model-api:1.1.4
pipeline-model-declarative-agent:1.1.1
pipeline-model-definition:1.1.4
pipeline-model-extensions:1.1.4
pipeline-rest-api:2.9
pipeline-stage-step:2.2
pipeline-stage-tags-metadata:1.1.4
pipeline-stage-view:2.4
pipeline-utility-steps:1.5.1
plain-credentials:1.3
pubsub-light:1.8
run-condition:0.10
scm-api:2.1.1
script-security:1.31
sse-gateway:1.15
ssh-credentials:1.13
structs:1.7
subversion:2.9
token-macro:2.0
variant:1.1
workflow-aggregator:2.5
workflow-api:2.18
workflow-basic-steps:2.3
workflow-cps-global-lib:2.7
workflow-cps:2.39
workflow-durable-task-step:2.11
workflow-job:2.10
workflow-multibranch:2.14
workflow-remote-loader:1.2
workflow-scm-step:2.4
workflow-step-api:2.10
workflow-support:2.14

There is currently a problem with building the openshift jenkins rhel plugin from the above list ... have emailed the CI/CD team.

quick response from @jupierce and https://buildvm.openshift.eng.bos.redhat.com:8443/job/devex/job/devex%252Fjenkins-plugins/13/console is cooking

RPM job done
the next build in this repo should now pick up that RPM and push it to ci.openshift  ... per discussion with @bparees in scrum, we can have QE work off of that for card validation, and then once validated, go through the CI/CD process to populate brew, etc.


refactor(discordx): add nsfw option to slash command objs

Please describe the changes this PR makes and why it should be merged:
Added nsfw option to slash decorator to match discord api types
Package

discordx

@SteeledSlagle13 please make PR for main branch. I didn't see it before merging it.


Cherry pick fixes and update change logs

Codecov Report

Merging #9498 into release-2019.12 will decrease coverage by 0.68%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           release-2019.12    #9498      +/-   ##
===================================================
- Coverage            57.29%   56.61%   -0.69%
===================================================
  Files                  551      551
  Lines                29438    29068     -370
  Branches              4479     4479
===================================================
- Hits                 16867    16457     -410
- Misses               11623    11661      +38
- Partials               948      950       +2

Impacted Files
Coverage Δ

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-60%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
50% <0%> (-50%)
:arrow_down:

src/datascience-ui/native-editor/redux/store.ts
50% <0%> (-50%)
:arrow_down:

src/datascience-ui/history-react/redux/store.ts
50% <0%> (-50%)
:arrow_down:

src/datascience-ui/react-common/logger.ts
0% <0%> (-40%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
33.33% <0%> (-33.34%)
:arrow_down:

src/datascience-ui/react-common/locReactSide.ts
16.66% <0%> (-33.34%)
:arrow_down:

...jupyter/kernels/jupyterKernelPromiseFailedError.ts
0% <0%> (-33.34%)
:arrow_down:

...ience/jupyter/jupyterDebuggerRemoteNotSupported.ts
50% <0%> (-30%)
:arrow_down:

src/datascience-ui/react-common/arePathsSame.ts
33.33% <0%> (-29.17%)
:arrow_down:

... and 43 more

Continue to review full report at Codecov.

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


Upcoming release date

Hey,
a while back we merged this PR together https://github.com/jscutlery/semver/pull/708 and I was wondering when you were planning on doing the next release? We are currently still using a fork that I made of this project and would like to switch back to the published version.
Thanks in advance :)

@lucavb Sorry we had trouble releasing the lib due to a problem in Nx itself, I reported the issue and waited to get a fix to release. I found a workaround without the patch and just published the new version of semver. Thank you.


Supported Plugins - PostgreSQL 15 is missing in the list

Issue on page:        https://github.com/opentelekomcloud-docs/relational-database-service/blob/main/umn/source/working_with_rds_for_postgresql/plugin_management/supported_plugins.rst
Here the current supported PostgreSQL version 15 from RDS is missing in the table.

Hi @SebastianGode,
I can actually see the PG15 supported plugins in the table on the HelpCenter.
https://docs.otc.t-systems.com/relational-database-service/umn/working_with_rds_for_postgresql/plugin_management/supported_plugins.html#rds-09-0045
Can you please recheck it again?
We uploaded a new document version on RDS in the recent 2 weeks, so hopefully you should see it know.

Thanks @onszekel
Now the document is updated :)
Issue closed.


Make org.gradle.api.tasks.bundling.Zip use Gradle workers

Make org.gradle.api.tasks.bundling.Zip use Gradle workers
Expected Behavior
If you have a project that has two Zip tasks today, they will run in parallel to each other.
Current Behavior
If you have a project that has two Zip tasks today, they will run serially one after the other.
Context
In our builds we found that Zip tasks can get on the critical path blocking each other. Example: https://ge.androidx.dev/s/nft6iyacctnqg/timeline?details=ciapjxixayigu

Instead of just changing the ZIP task the entire AbstractArchiveTask that serves as the base for it and many others should be changed.


Getlist fields / Get API

GET_LIST
The SDK for getlist currently does not support optional selection of fields.  This pull requires updates the getlist function to optionally support specifying which fields to include in the getlist results.
GET
The SDK did not include a function for get.  The pull request provides an implementation of get, that also supports optionally parameters for specifying which fields to include.
Unit Tests
Unit tests for both getlist and get are included.

@doflynn, GET functionality is already handled via the Read.ts file in the "Common" directory. Please revert those changes related to GET api.


[SPO] Rules for adding mandatory and optional consent

Additional info: TP does not have to sign "Optional" consent to continue participation in the study. TP either can sign it (and then withdraw), decline, or just live in the list of consents (no action). Related issue (TP): #305

Adding more information as clarifying with the business:

There should be only one mandatory consent per trial

first consent should be mandatory and the rest optional. implement this by adding a business rule .e.g getAllowedNumberOfMandatoryConsents that for the moment will return a constant.
if there is no more than one choice the select should be disabled: so first consent will have Mandatory disabled, the next consents will have Optional and disabled.

Commited

When I'm trying to add a consent version

Please also change the modal name from New Consent to New Consent Version

Commited @Mastaleru

all good


Split the Config class in multiple files

Right now the Config class is all in config.py and it's huge and unwieldy. The class already contains sections, and we should split them in their own files, which will contain a parent class for the new Config class.

fixed by #89


net: wifi_mgmt: Fix size calculation for net_mgmt info

Bug: net_iface_status is larger than scan_result, causing net_iface events to be dropped due to info size exceeding NET_EVENT_INFO_MAX_SIZE.

Hi @LuskeyNoah, this should be fixed upstream, please send a PR in Zephyr (https://github.com/zephyrproject-rtos/zephyr/), we'll integrate the change on the next sync of our fork.

@rlubos - I've created a PR on the upstream. Closing this PR. Thanks!


Linking error with imgui[opengl3-glad-binding]

Describe the bug
Trying to build an executable with imgui[opengl3-glad-binding] leads to a linking error. As you can see in the log snippet below, for some reason the linker is looking for glew symbols instead of glad symbols. glew and imgui[opengl3-glew-binding] packages are not even installed.
Environment

OS: Ubuntu 21.04
Compiler: GCC 11.1.0

To Reproduce
The minimal example is as follows.
main.cpp:
#include <glad/glad.h>
#include <imgui_impl_opengl3.h>

int main()
{
  ImGui_ImplOpenGL3_Init();
}

CMakeLists.txt:
cmake_minimum_required(VERSION 3.18)
project(hello)

find_package(glad CONFIG REQUIRED)
find_package(imgui CONFIG REQUIRED)

add_executable(hello main.cpp)
target_link_libraries(hello PRIVATE glad::glad imgui::imgui)

Expected behavior
I expect the executable to compile and link correctly. (The same example worked as expected when I tried it on Windows.)
Failure logs
[build] : && /bin/x86_64-linux-gnu-g++-11 -g  CMakeFiles/hello.dir/main.cpp.o -o hello  /home/dylan/repositories/github/Microsoft/vcpkg/installed/x64-linux/debug/lib/libimguid.a  /home/dylan/repositories/github/Microsoft/vcpkg/installed/x64-linux/debug/lib/libglad.a  /home/dylan/repositories/github/Microsoft/vcpkg/installed/x64-linux/debug/lib/libglfw3.a  /usr/lib/x86_64-linux-gnu/librt.so  -lm  -ldl  /usr/lib/x86_64-linux-gnu/libX11.so  -lpthread && :
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/installed/x64-linux/debug/lib/libimguid.a(imgui_impl_opengl3.cpp.o): in function `ImGui_ImplOpenGL3_Init(char const*)':
[build] /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:163: undefined reference to `glGetIntegerv'
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:164: undefined reference to `glGetIntegerv'
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:168: undefined reference to `glGetString'
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:232: undefined reference to `glGetIntegerv'
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/installed/x64-linux/debug/lib/libimguid.a(imgui_impl_opengl3.cpp.o): in function `ImGui_ImplOpenGL3_SetupRenderState(ImDrawData*, int, int, unsigned int)':
[build] /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:251: undefined reference to `glEnable'
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:252: undefined reference to `__glewBlendEquation'
[build] /usr/bin/ld: /home/dylan/repositories/github/Microsoft/vcpkg/buildtrees/imgui/src/v1.83-af20d68e6e.clean/backends/imgui_impl_opengl3.cpp:253: undefined reference to `__glewBlendFuncSeparate'
...

Have you defined IMGUI_IMPL_OPENGL_LOADER_GLAD  ? Make sure that IMGUI_IMPL_OPENGL_LOADER_GLEW isn't defined somewhere. The CMake wrapper don't add any compile definitions. So that should be on your side.
So if you haven't defined anything, imgui tries to detect the loader by itself. https://github.com/ocornut/imgui/blob/ad5d1a8429ea219d3d34e6a36a48918650402697/backends/imgui_impl_opengl3.h#L68
Not a vcpkg bug

And just to expand it a bit further:
You might get different results depending on the platform you are using. If some application has installed libglew-dev the headers are present in the global include dir of linux. So the line #if __has_include(<GL/glew.h>) resolves to true and therefore defines IMGUI_IMPL_OPENGL_LOADER_GLEW . But the libraries aren't present in the link defines and resulting in the present link errors.

Have you defined IMGUI_IMPL_OPENGL_LOADER_GLAD ?

Yes, I have thought of this. Adding this define to my source doesn't change anything. But that's not a surprise since the linker errors are emited from libimguid.a not from my object file. There are glew headers in my /usr/include/ directory and maybe they were picked up when libimguid.a was built, but that would be a vcpkg bug.

Oh yeah. I see. Sorry for the false info I gave. Actually that will be tricky now. Since the loader have to be known at compile time. I think there was somewhere a discussion to be able to add a custom imgui config file. But that's currently only possible with overlay ports. (would be know the fastest solution to your problem. Copy all sources from the imgui port, add a compile definition to the glad option in the cmake file and add the port as a overlay port)
The long term solution needs discussion I think.

Just to help you getting started with overlay ports:
General overlay ports:
https://github.com/microsoft/vcpkg/blob/master/docs/specifications/ports-overlay.md
If you are using manifests:
https://github.com/microsoft/vcpkg/blob/master/docs/users/manifests.md
add the path (could be relative to your projet root) to VCPKG_OVERLAY_PORTS and set the CMAKE_TOOLCHAIN_FILE

@Dylan-Davies
You may try to solve the problem refer to this https://github.com/microsoft/vcpkg/issues/14753#issuecomment-782899582.
@mathisloge
Thanks for your help to look into this issue.

Thanks for the suggestions. I have managed to make it work with an overlay port where I just added
target_compile_definitions(${PROJECT_NAME} PUBLIC IMGUI_IMPL_OPENGL_LOADER_GLAD)

as proposed in the other issue. Any reason why the problem shouldn't be solved just by adding the compile definitions for all backends like that?
BTW, in the related issues some users complain about unable to use multiple bindings side by side. Maybe it would be a good idea to provide separate targets for the backends? That way we would have imgui::imgui with just the core library and then say imgui::opengl3_glad, imgui::opengl3_glew etc. that could just live side by side.

@Dylan-Davies Please see https://github.com/microsoft/vcpkg/issues/14753#issuecomment-889693559 and https://github.com/microsoft/vcpkg/issues/14753#issuecomment-889731841.

@Dylan-Davies
As you can see as follows:
If you have installed glew on your machine, namely checks GL/glew.h, it will not check other headers such as glad/glad.h.
We need to confirm with upstream if this is by design.

#endif
#else
    #define IMGUI_IMPL_OPENGL_LOADER_GL3W   // Default to GL3W embedded in our repository
#endif

@NancyLi1013  please see https://github.com/microsoft/vcpkg/issues/14753#issuecomment-889739471 and https://github.com/microsoft/vcpkg/issues/14753#issuecomment-889741206

@NancyLi1013

If you have installed glew on your machine, namely checks GL/glew.h, it will not check other headers such as glad/glad.h.

Yes, my understanding is that if you don't want the bindings to be auto-detected, you are supposed to define one of the IMGUI_IMPL_OPENGL_LOADER_* macros before including imgui_impl_opengl3.hpp. This is what target_compile_definitions command that I cited accomplishes.


[BUG] Twitch SVG, not aligned proper

The twitch svg isn't aligned properly, it's a basic css fix I believe.

Hi Can I work on the above issue? If so, please assign it to me.

Hi Can I work on the above issue? If so, please assign it to me.

Hi, sure. Assigning you.

Yes, that's correct for the twitch and insta icon.
On Tue, 12 Jul 2022 at 16:18, Goutham-erram @.***>
wrote:

[image: image]
https://user-images.githubusercontent.com/88138359/178472639-bdeef8b3-3884-4ca9-a7bf-763a9137c999.png
I guess the image size is little bit small compared to others, that makes
it look not aligned properly. Is this is where you are having an issue or
some where else. Kindly let me know if we have issue some other place.
—
Reply to this email directly, view it on GitHub
https://github.com/KendallDoesCoding/ludwigextension/issues/1#issuecomment-1181608499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTPD4BNYMKTOVQXVFX3VTVEOXANCNFSM5JUQ4E2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>


OpenSSL allows the client to present RSA certificates with the key usage keyAgreement only.

Affected Versions
1.1.1g, 1.0.2u
Setup
OpenSSL has been compiled from source in an alpine docker container. The following excerpt of the used docker file shows the build related commands:
RUN ./config --prefix=/build/ --openssldir=/build/ no-async
RUN make -s && make install_sw -s

After building it the respective OpenSSL version and required libraries have been copied into an minimal docker container (from scratch).
The server is later started using the following command:
openssl s_server -accept 4433 -Verify 100 -cert /cert/inputCerts/rootv3.pem -key /cert/keys/rootv3.pem -CAfile /cert/inputCerts/root.pem -verify_return_error
All files mentioned in this report (certificates, keys, etc.) have been included in the attached zip archive.
Issue
OpenSSL allows the client to present RSA certificates with the key usage set to keyAgreement only.
As there is no key agreement algorithm that uses RSA keys this certificate may not be used at all.
Reproduction
Uses OpenSSL 1.1.1 client on Ubuntu 18.04.
Connect to the started server using the following command:
openssl s_client -connect localhost:4433 -cert ROOTv3_CAv3_LEAF_RSAv3_KeyUsageKeyAgreement__leaf_certificate1.pem -key rsakey_2.pem -CAfile ROOTv3_CAv3_LEAF_RSAv3_KeyUsageKeyAgreement__ca_certificate1.pem
Expected Result
OpenSSL should reject the certificate since it may not be used to create/verify signatures.
Actual Result
OpenSSL happily accepts the certificate and proceeds with the handshake.
Attachment
opensslRSAKeyAgreement.zip

Bug confirmed. OpenSSL is generally pretty sloppy on checking key usages, at least for SSL/TLS use.
See the new bug report #12721 covering this report and its duplicate #12140.

Marking as inactive, to be closed at the end of 3.4 dev, barring further input

This is a nice example for the generalized issue #12721.

Ok @DDvO thats a fair statement, but it seems to me to illustrate the fact that we have #12721 to cover this issue, and (regardless of its closure status), that issue links back here.  So is there any reason that this shouldn't be closed as a duplicate of that?

Well, it's not really a duplicate, but a small subset.
As I wrote, it's an example including a test case, so provides nice detail that should be helpful tackling https://github.com/openssl/openssl/issues/12721.
So in order to minimise the number of open issues. I'd suggest copying the OP there, including the attachment,
and then closing this one.

@DDvO copy that, good suggestion, thank you!


gh #233 dsAudio Enhance python tests

Updated the L3 tests to avoid the repeated code
Added ds helper class
Updated dsAudio class to deploy the binaries which are listed in configuration file to target

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


Use vim.notify not to block outputs

print() causes hit-enter prompts when we are using set cmdheight=0 in the latest Neovim.
Telescope-Frecency: Imported 2 entries from oldfiles.
Press ENTER or type command to continue

Neovim has vim.notify for such cases. We can overwrite it with other plugins, such as rcarriga/nvim-notify, so that Neovim does not show the prompt and not block outputs.

thanks :)


Can add Loader ,   give schema and []byte , parse to map or struct ?

I previously used github.com/xeipuuv/gojsonschema to validate JSON schema and extract data from JSON requests, but it has not been maintained for a long time. I'm considering switching to another library, but I noticed that it doesn't have the Loader functionality. Is there any plan or possibility to add it in the future?
Need
// w http.ResponseWriter, r *http.Request
result,err := jsonschema.SchemaDataLoader(r.Body)
if err == nil {
    data := result.(map[string]interface{}) // or some struct
}

Thank you~

see following examples from godoc (https://pkg.go.dev/github.com/santhosh-tekuri/jsonschema/v5)

FromString
FromStrings
UserDefinedLoader


Recording simulations

Is there an easy way to record the simulations with the agent navigating the environment? For all outputs - depth, rgb_filled as well the pybullet gui window? or will I have to store each frame myself and then compile them together?

I ended up using OBS Studio to screen capture


Switch Resident Key to Client Side Discoverable

See https://www.w3.org/TR/webauthn-2/#enum-residentKeyRequirement
This would even remove the need to specify the username. Would obsolete 14.6.3 in #12. Would also remove the need to have the username in login_config in views.py

Among other not supported in Firefox under Linux. For me this is a blocker to implementing it.
See Bug 1530370 and Bug 1294514.


Feature: Keep the selection centered when editing the URL

Now when clicking on the urlbar, I hover over the part that i would like to change/copy, but after clicking on it is moved to the left again. So my cursor is in the wrong position. It would be nice I we can get the edit URL also centered.

I'll probably add it as an option in the next version when i got the time


"make test_cc" failing with undefined reference errors in libortools.so

I'm attempting to build the latest release for C++ per the instructions on this page:
https://developers.google.com/optimization/install/cpp/source_linux
When I try to run "make test_cc", I get the output in the attached file. I'm using gcc 7.2.0 on CentOS 7 Linux. It seems to compile fine, no obvious errors there. Attached is the terminal output I get when trying to run "make test_cc".
ortools.txt

Fresh checkout ? or upgrade from an older version ?
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 5 juil. 2019 à 17:50, ForrestBurt<EMAIL_ADDRESS>a
écrit :

I'm attempting to build the latest release for C++ per the instructions on
this page:
https://developers.google.com/optimization/install/cpp/source_linux
When I try to run "make test_cc", I get the output in the attached file.
I'm using gcc 7.2.0 on CentOS 7 Linux. It seems to compile fine, no obvious
errors there. Attached is the terminal output I get when trying to run
"make test_cc".
ortools.txt https://github.com/google/or-tools/files/3363358/ortools.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1411?email_source=notifications&email_token=ACUPL3PBPCB674LSOHIDWL3P55UV5A5CNFSM4H6MNEJKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G5SW2LQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3M5SKWFQ7XIPF4Z4NTP55UV5ANCNFSM4H6MNEJA
.

Fresh checkout.

I just build 7.2 on centos without any issue
Is gcc 7.2 the default compiler ?
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 5 juil. 2019 à 17:58, ForrestBurt<EMAIL_ADDRESS>a
écrit :

Fresh checkout.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1411?email_source=notifications&email_token=ACUPL3KPNAY4ERBTQ2MGTJDP55VR7A5CNFSM4H6MNEJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZJ3LQQ#issuecomment-508802498,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3O5SMGBTHVXVWDSLJ3P55VR7ANCNFSM4H6MNEJA
.

which gcc gives me 7.2.0. This is being built in a cluster environment, so once the module for gcc is loaded, it'll use it. Also, apparently we've got different versions, I was under the impression that the included file on that page was the latest; I'm trying to build 7.1, though I'll give 7.2 a try.

Update, I've tried 7.2, but found that it throws about the same errors. I'm trying it locally with some updated packages, and I'll post again once I get the results.

No news, closing.


Feature/mkh 27

:tada: This PR is included in version 1.2.0-develop.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


🐞 Bug :"Unnecessary # symbol displayed on the main page"

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
On the main page of CircuitVerse, an unnecessary "#" symbol is being displayed, which makes the UI look cluttered and unpolished.
Steps To Reproduce
1.Open the application and navigate to the main page (http://localhost:3000 or equivalent URL).
2.Observe the # symbol on [specific location of the page, e.g., header, footer, sidebar].
Expected Behavior
The # symbol should not appear unless it serves a functional purpose. If it’s unintentional, it should be removed to improve the user experience.
Screenshots

Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
Yes

It seems that the # symbol appears in the local setup but not on the production site. This could be due to differences in the environment setup or the way assets are loaded.
To resolve this, I would recommend checking the local and production environment configurations for any discrepancies, especially around asset paths and routing.
Is there anything else I can help with or any further information you need from my side to investigate this issue?

Hi,
I'll debug this issue and look into the local & production environment setup. Kindly assign me this issue.
Thanks

i want to work on this issue...please assign it to me

I think i can solve this issue by inspecting page and just finding out from where that symbol is coming from. Its my humble request sir please assign this issue to me

Hey @tanmayshi ,
I think I'll be able to work on this.Could you
please assign me to the task?
thanks.


Subscription missing fields

Why are so many fields missing from the subscriptions response? For example, the three below and others are not available
"customer_id": =
"customer_name":=
"email": =

Thanks for reaching out. @digitalgym - Their API are always evolving, I am not actively maintaining or keeping it up-to-date. So normally I look for "Hey, please add following new properties in" Github request / issues and I update the NuGET package for it in next coming weekend. Let me know if you have a specific properties I can add in


Separate backend and frontend

Backend API will be placed at gifsync-api. The backend will have Flask code, talk to a PostgreSQL database, talk to S3, and post jobs to an RQ worker.

Completed!


Materializecss Transitions

Hello!
How can I use Transitions using Angular 2 and Materializecss?
I want to run Transitions when the page is loaded.
For example: When the page loads, I want items in the menu to come up with the Transitions effect.
http://i.hizliresim.com/WgL5DY.png
Thank you very much in advance. :)

I haven't used it myself, but here's an example usage:
Click Me

List Item

This is a description

List Item

This is a description

List Item

This is a description

List Item

This is a description

List Item

This is a description

animate() {
    Materialize.showStaggeredList('#staggered-test');
  }

Thank you very much. I will try this. :)

@mekkem02 have you had a chance to try it? I'd like to know if we can close this issue. Thanks.

Hi @rubyboy
I tried your solution but it won't work. Angular is not able to recognize  "Materialize.showStaggeredList".
Would you share from which module or component we can get the reference to "Materialize"?
Thanks


DROOLS-1040 - Kie server on Tomcat with JACCValve fails authorisation

merged into 6.4.x thanks @sutaakar


ruby warnings

every time you run the program, ruby starts recording several warnings
https://pastebin.com/raw/Gx0aQwNi

Hi! Thank you for issue.
It's an issue I'm aware of and I'll try to correct it. Thank you.
Same Issue : https://github.com/hahwul/XSpear/issues/45

@Th3F0x-code
Updated XSpear 1.3.2!
$ gem update XSpear


NLP support

What changes are proposed in this pull request?
Creating a Spark MLlib Estimator API which can integrated with tensorflow code, with a reference implementation.
Creating a Spark MLlib Transformer convert text column to 2-D vector which can be feeded to CNN/LSTM directly.
It provides a taste of how to process text  in a DataFrame and use them to train a NLP  model developed by tensorflow.
Also fix issue: https://github.com/databricks/spark-deep-learning/issues/53
The changes consist of these components.
TFTextFileEstimator/TFTextTransformer
New shard params
How is this patch tested?

Unit tests
Manual tests

Thank you for contributing to the project @allwefantasy!

The CI fails because there is no kafka lib in the env. Is there something i can do to fix this ?

@allwefantasy thank you very much for the contribution. I will have more comments for the estimator, so would you mind splitting your PR into the transformer part and into the estimator?
Also, I see that the transformer is embedding Word2Vec. Have you considered chaining them in a pipeline instead?
https://spark.apache.org/docs/2.1.1/ml-pipeline.html
Regarding kafka, you should be able to add it in this file:
https://github.com/databricks/spark-deep-learning/blob/master/python/requirements.txt

@thunterdb  TFTextTransformer is a tool like StringIndexer in MLlib, which we can use to transform the dataframe and feed the new dataframe to TFTextFileEstimator. It seems no need to split into two PRs.
Using word2vec is in order to compute a Map which contains the mapping of word to vector. We do not need the word2vec model's transform function.

Codecov Report

Merging #56 into master will decrease coverage by 3.24%.
The diff coverage is 64.17%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   82.82%   79.57%   -3.25%
==========================================
  Files          23       25       +2
  Lines        1217     1469     +252
  Branches        5        5
==========================================
+ Hits         1008     1169     +161
- Misses        209      300      +91

Impacted Files
Coverage Δ

python/sparkdl/transformers/named_image.py
93.51% <100%> (ø)
:arrow_up:

python/sparkdl/transformers/keras_applications.py
93.93% <100%> (+2.1%)
:arrow_up:

python/sparkdl/transformers/utils.py
100% <100%> (ø)
:arrow_up:

...ython/sparkdl/estimators/tf_text_file_estimator.py
50% <50%> (ø)

python/sparkdl/transformers/tf_text.py
78.26% <78.26%> (ø)

python/sparkdl/param/shared_params.py
80.88% <82.5%> (+0.67%)
:arrow_up:

Continue to review full report at Codecov.

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


Polling Events?

Thank you for your work.  I would love to use Amcrest2MQTT to automate turning on an outside light when someone presses the AD401 doorbell button after dark.  However it looks like you are polling data with a default of 1 hour interval. Is this true even for events such as the doorbell button being pressed?  If so, is it practical for Amcrest2MQTT to send a MQTT event when event occurs?  Is shorting this interval to a few seconds practical?  Thank you again.

Opps, it looks like the polling is only for getting the amount of storage used.  Events are triggered as they happen, right? I have installed things like mqtt on my Synology NAS using Portainer but I had to follow step by step instruction videos.  Does anyone have a video or step by step instructions to install amcrest2mqtt?

Yeah, the polling only applies to fetching storage data. All events should happen immediately and I have automations based on my AD110 doorbell working well 🙂
I'm not too familiar with Portainer but it should be a case of adding a Docker container in the same way you have for your other containers and then setting the correct environment variables as per the README.


Set player locale when creating Audience

The pull request sets the client locale for Player audiences in the adventure-platform-bukkit module.
Until now, the client locale was only correct if the player changed his or her locale via settings while playing on the server.

So it is running fine on 1.12 up to 1.20.4, can't tell about 1.20.5 yet.
1.8 has been tested but does not work, it instead returns system language via Locale.getDefault()
Anything I can do to get this merged?

Why do i feel like on 1.8 getLocale() was on .spigot() or something? is that just me making things up?

oh yes, https://hub.spigotmc.org/stash/projects/SPIGOT/repos/spigot/browse/Bukkit-Patches/0018-Implement-Locale-Getter-for-Players.patch?at=e4d4710834f7b292bc496f41f1cf9ac363225c0e#21
Any chance you'd be able to revise this PR to include that variant? I'd love to spin off a -platform release soon including this PR.

.spigot().getLocale() doesn't seem to work properly either. In forums I read that for 1.8 you'd have to listen to the settings packet. The PR removed the event listening and replaced it with a direct getLocale call on the player object, but the event didn't even exist on 1.8 and has been introduced in 1.12 I think? So either way it'd be an improvement already but doesn't fix the 1.8 issues yet
At least, the following code did not work, it still returns Locale.getDefault():
    private static final MethodHandle SPIGOT_SUPPORTED = findMethod(Player.class, "getLocale", findClass("org.bukkit.entity.Player.Spigot"));
    private static final MethodHandle SPIGOT_LOCALE_SUPPORTED = findMethod(findClass("org.bukkit.entity.Player.Spigot"), "getLocale", String.class);

//...

      builder.withDynamic(Identity.LOCALE, () -> {
        if (LOCALE_SUPPORTED != null) {
          try {
            return Translator.parseLocale((String) LOCALE_SUPPORTED.invoke(viewer));
          } catch (final Throwable error) {
            logError(error, "Failed to call getLocale() for %s", viewer);
          }
        }
        if (SPIGOT_SUPPORTED != null && SPIGOT_LOCALE_SUPPORTED != null) {
          try {
            final String localeStr = (String) SPIGOT_LOCALE_SUPPORTED.invoke(SPIGOT_SUPPORTED.invoke(viewer));
            return Translator.parseLocale(localeStr);
          } catch (final Throwable error) {
            logError(error, "Failed to call spigot().getLocale() for %s", viewer);
          }
        }
        return Locale.getDefault();
      });

You know, if 1.8 is fixable then let's just let the 1.8 people PR that fix and get this release out already, otherwise the PR looks fine


NOISSUE - Fix opcua-adapter events warnings

Signed-off-by: Manuel Imperiale<EMAIL_ADDRESS>

Codecov Report

Merging #965 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #965      +/-   ##
==========================================
- Coverage   83.75%   83.71%   -0.04%
==========================================
  Files          75       75
  Lines        5307     5307
==========================================
- Hits         4445     4443       -2
- Misses        587      589       +2
  Partials      275      275

Impacted Files
Coverage Δ

things/service.go
88.97% <0%> (-1.48%)
:arrow_down:

Continue to review full report at Codecov.

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


Update README.md

Add code-coverage option in the README.md.

I signed it!

Thanks for integrating those changes, this looks ready to merge!


merge package from arlac77/npm-package-template

package.json

chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(engines<EMAIL_ADDRESS>chore(package): add xo from template
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

.travis.yml

chore(travis): remove node versions 10.14.1
chore(travis): add node versions 10.14.2

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Digital COVID certificates validity check: List of countries is always in the language which was active during the onboarding

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[X] Bug is not already reported in another issue

Technical details

Device name: iPhone 6
iOS version: 15 Beta 6
App version: 2.8.1

Describe the bug
The names of the countries which can be selected in the "Validity of Certificate" screen does not adapt its translation correctly. They always stay in the language which was active during the onboarding.
Steps to reproduce the issue

Open the iOS Settings
Scroll down to "Corona-Warn"
Select "Sprache"
Select "English"
Open the Corona-Warn-App
Click on "Certificates"
Click on "Add Certificate" and scan the QR code of a certificate if you don't have any certificates registered in the app
Click on a certificate
Click on "Check Validity"
Click on "Country"
See that the names of the countries appear in German instead of in English

Expected behaviour
The names of the countries should be localized correctly.
Additional context

There is no way for me (currently) to get the app back to show the English names...
Please let me know if you need an error log.

Credits
This issue was discovered by @dsarkar!

Internal Tracking ID: EXPOSUREAPP-10180

With version 2.8 the issue is exactly the other way around. English always shows the German names...
Will adapt the OP later.

Analysis: the list of countries is stored persistently including the localised names of the countries. As long as the list does not change (e.g. additional countries participate), it uses this stored list and does not recalculate the localisation.

@ndegendogo Thanks for analysing, I forward your info to the internal ticket.

I found three variables in the Store related to Countries:

validationOnboardedCountriesCache - this is used to populate the picker for the certificate validity check
lastSelectedValidationCountry - this is the initial value of the picker for the certificate validity check
submissionCountries - this is not used during certificate validation; it seems to be related to the ExposureSubmissionService, but I didn't analyse this in more detail

@dsarkar @heinezen I have submitted PR #3802 with a fix for this - please apply the labels. Thanks!

Thanks for the PR @ndegendogo!

Confirm fixed in version 2.15.
Thank you, again, for the PR fixing this @ndegendogo.
Closing as fixed.

@ein Thanks for following up. I will remark in the internal ticket, that the fix was confirmed by the community.


Marking mail moves it to cur unconditionally

I don’t know if this is right in the first place; it seems this should have been written to leave the mail wherever it is.
What I do know is Mutt makes a distinction between non-S-flagged mail in new (which it considers really unread) vs non-S-flagged mail in cur (which it considers seen-but-not-read). It can mark unread mail as seen-but-not-read (i.e. move it from new to cur) without adding an S flag, but the user can configure it not to.
I’m not sure what the right thing is, or how to proceed.

One option I thought of, given the circumstances, is to add an optional second argument to ->mark, which may be cur or new to specify the target explicitly, or just a 1 to say “leave it wherever it was”. If the argument is false or missing, the mail is moved to cur as usual.

I thought about this one, and I can't quite think of any problems allowing one to specify {cur,tmp,new} explicitly, since it doesn't break any default behaviors described by DJB's Maildir specification.

Good enough for me!


[BUG] ConformalIntervals wrapped ARIMA/AutoARIMA breaks with default parameters

Discussed in https://github.com/alan-turing-institute/sktime/discussions/2869

Originally posted by linaamic June 26, 2022
Hi everyone, I am currently trying out the ConformalIntervals feature from sktime.forecasting. However, I can't get it to work for both ARIMA and AutoARIMA. Is there an inherent difference between ARIMA forecasters and others? Do I have to call or wrap them differently or is ConformalPrediction not possible with ARIMA?
With other forecasters I have no issues.
This is the error I get:
File "...\site-packages\statsmodels\tsa\statespace\sarimax.py", line 843, in _conditional_sum_squares
Y = endog[r:]
IndexError: too many indices for array: array is 0-dimensional, but 1 were indexed
Code:
from sktime.datasets import load_airline
from sktime.forecasting.arima import AutoARIMA, ARIMA
from sktime.forecasting.conformal import ConformalIntervals

y = load_airline()
trial_forecaster = AutoARIMA()
conformal_forecaster = ConformalIntervals(trial_forecaster, method="conformal")
conformal_forecaster.fit(y, fh=[1,2,3])

Any help is much appreciated!

FYI @bethrice44

Have looked into this and it appears to just be too short initial_window since auto-arima does some slicing of the data and it  requires more than one data point. The exact number of data points it requires is defined by r from sarimax, i.e. it will be r+1:
    def _conditional_sum_squares(endog, k_ar, polynomial_ar, k_ma,
                                 polynomial_ma, k_trend=0, trend_data=None,
                                 warning_description=None):
        k = 2 * k_ma
        r = max(k + k_ma, k_ar)

        ....

        Y = endog[r:]

I'm not sure if there is an easy way to pre-calculate this for the model. Thoughts @fkiraly?

Possible solutions I can think of:

making the default initial window of ConformalInterval longer. That does not cover all possible cases (since r can be arbitrarily high), but perhaps most cases appearing in actual use
adding a note or caveat at somewhere in the algorithm description, and/or next to the initial_window parameter
?

Yes. One option is we don't pass initial_window at all and we take (say) 5% of the data points as the initial window, as long as it is >= some value. Another option is just adding to the initial_window docstring, but I would assume an  initial_window of just 1 will always be a poor model and maybe over-estimate the overall interval? I think I need to test this and also check the literature.

Hmm, I see - your suggestion is to allow float for initial_window, interpreting this as a fraction, and then leave a fraction as the default?
Makes sense!
Another thing I noticed when I was thinking about the impacts of runtime, sample_frac might also be confusing for a user to control, since it is a fraction of a total that the user will not have a grasp of, rather than a parameter that links to absolute runtime. Would a parameter n_samples or similar be better to control the runtime here?

Yes thats right on initial_window although may run in to problems if that dataframe passed to the function is small (i.e. we pass 0.05 as initial_window fraction and the data only has 10 items say...), so could do something like max(initial_window*len(y), 1), but that wouldn't solve the priblem above if len(y) is too small.
On the second point, happy to update it to n_samples and I guess default to n_samples=len(y_index)

On the second point, happy to update it to n_samples and I guess default to n_samples=len(y_index)

Perhaps - I would not remove arguments, but we can make it like in train_test_split that accepts multiple types (integers, floats, etc) which are interpreted differently.
We also need to be cautious about deprecations, i.e., all calls to the estimator need to behave the same until version 0.15.0, possibly raising deprecation messages if we want to change the behaviour. But, optimally, we can figure something out so we do not even need to deprecate? Would have to sit down and think about this carefully, but probably you already have a good idea what/how to change?

This might be another instance of "too small training set": https://github.com/alan-turing-institute/sktime/issues/3194


SGPR, method compute_qu incorrect for multiple latent gps

Bug

I'm not an expert in GPs but I believe compute_mu should return one covariance matrix per output / latent gp, so when you  construct such a model, the tensor corresponding to the covariance term should have rank=3 instead of 2. At the moment regardless of the number of outputs same q covariance matrix is returned (q mean tensor seems to be computed correctly, taking into account the number of outputs).
In addition, it would be great if compute_qu would return q objects that are consistent with SVGP where q objects also depend on q_diag argument - perhaps that could be an argument to compute_qu method?
To reproduce
Minimal, reproducible example

import tensorflow as tf
import numpy as np
import gpflow

x = tf.constant(np.arange(5).reshape(-1, 1), dtype=gpflow.default_float())
y1 = 3.0 * x + 10
y2 = y1 * 0.5
model = gpflow.models.SGPR((x, tf.concat([y1, y2], axis=-1)), gpflow.kernels.Matern32(), x)
q_mu, q_var = model.compute_qu()

Expected behavior

If we are guided by SVGP (see here), q_mu should be of shape (5,2), while q_var should be of shape (2,5,5) where 2 is num_latent_gps. However, q_var is of shape (5,5) regardless of number of latent gps.
System information

GPflow version: 2.4.0
GPflow installed from: with "pip install gpflow"
TensorFlow version: 2.5.3
Python version: 3.7.5
Operating system: Ubuntu 18.04

No time to do the testing right now, but I believe it is caused by using a single-output Kernel.
You need to use multi-output kernels since you are modeling two outputs.


calculated column fails

the following column expression
=if(quantity=0, 0, min(1, filledQuantity / quantity))
fails on server with error
line 1:43 no viable alternative at input 'if(quantity=0,0,min(1.0,filledQuantity/'
The parser seems to terminate the expression after the '/'
Thought maybe it was an issue with the '/' character itself and I would need to escape it, but this works more or less ok
=filledQuantity / quantity
(more or less because divide by zero cases obviously yield wrong results)
this also fails
=(filledQuantity / quantity)

Will take a look over weekend

Looking at the tests I think it's possible that min expects a literal column name rather than an expression but will confirm in test

Hmm just added this test case, it works.....
Are you doing this on the orders table...


Edge web driver creation fails with "End of Central Directory record could not be found."

Starting today, we get the following exception when calling DriverSetup.AutoSetUp("Edge"):
OpenQA.Selenium.WebDriverException : Creating the web driver failed with the message "End of Central Directory record could not be found.". This can mean that there is a leftover web driver process that you have to kill manually. Full exception: System.IO.InvalidDataException: End of Central Directory record could not be found.
	   at System.IO.Compression.ZipArchive.ReadEndOfCentralDirectory()
	   at System.IO.Compression.ZipArchive..ctor(Stream stream, ZipArchiveMode mode, Boolean leaveOpen, Encoding entryNameEncoding)
	   at System.IO.Compression.ZipFile.Open(String archiveFileName, ZipArchiveMode mode, Encoding entryNameEncoding)
	   at Atata.WebDriverSetup.DriverSetupExecutor.ExtractFromZip(String archiveFilePath, String destinationFilePath)
	   at Atata.WebDriverSetup.DriverSetupExecutor.ExtractDownloadedFile(String sourceFilePath, String destinationFilePath)
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   at Atata.WebDriverSetup.DriverSetupExecutor.SetUp(String version)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.ExecuteSetUp()
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUp()
	   at Atata.WebDriverSetup.DriverSetup.AutoSetUp(String browserName)

This only seems to be happening under GitHub-hosted Ubuntu runners of GitHub Actions. It doesn't happen on Ubuntu runners hosted by WarpBuild, or under GitHub-hosted Windows runners (nor can I reproduce it on my Windows machine).
I suppose something recently changed in how the Edge web driver is made available. This run on the 22nd succeeded, and this other one failed today, without any changes made to the affected code. Running the previously succeeding older run again also fails, confirming that this is something with the driver's download URL.

Thanks @Piedone for reporting this. I'm also able to reproduce on Ubuntu on Azure Pipelines. I'm looking at this...

Thank you for your quick reply, Yevgeniy, and for looking into this.

While I was trying to solve this issue, seems that it is gone now. EdgeDriver ZIP appeared in Azure drivers storage and the library is able to handle it. You can try on your side. But anyway, let me describe the problem.
So the library was trying to setup the driver for Edge. If you don't specify the version explicitly, the library tries to detects the version of Edge browser on a machine and use that version as desired EdgeDriver version. When it cannot detect the Edge version, then it gets the latest EdgeDriver version by URL https://msedgedriver.azureedge.net/LATEST_STABLE. In my case today both approaches returned version 126.0.2592.113, which is the current stable version according to https://developer.microsoft.com/en-us/microsoft-edge/tools/webdriver/. Then the library composes a URL to download ZIP file with the driver. In this case it was https://msedgedriver.azureedge.net/126.0.2592.113/edgedriver_linux64.zip. But at that moment there were no ZIP files for Linux and macOS, there were only ZIPs for Windows. The version specific drivers can be checked by such link: https://msedgewebdriverstorage.z22.web.core.windows.net/?prefix=126.0.2592.113/. So we were getting "404 (The specified blob does not exist.)" by requesting that URL. Your error "End of Central Directory record could not be found." is just a case of bad error handling in the library, it was trying to unzip "404 (The specified blob does not exist.)" text. I'm fixing the error handling for such cases.
So at the moment we are good with the current version of Edge. But in case of the similar problem in future I'm trying to find a fallback solution that will try to get the driver of the lower version, the closest one.

I can confirm the previously failing runs pass now (see here and here).
Better error handling would be great here, thank you.

This has resurfaced just now, e.g.: https://github.com/Lombiq/Open-Source-Orchard-Core-Extensions/actions/runs/10133111098/attempts/1#rfc97f24c5s2:~:text=Creating the web driver failed with the message

I've just released v2.12.0 of Atata.WebDriverSetup package with Edge driver download improvement. Please upgrade and let me know how it works for you.

Thank you! I've tested the updated package and it appears to be working. However, let me get back to you because due to the global Azure networking issues currently the download can randomly fail.

Yeah, looks good both under Ubuntu and Windows when run in GHA. Locally, I could also get the tests to finally run, catching a break in the networking issues.

Thanks for the information. I don't know what I can do for a case when Azure network has issues. Anyway, I think we handled the current issue and can close it.

Yep, thanks.
Nothing to do, really, I just couldn't properly test first due to this outage.

I got this now:
OpenQA.Selenium.WebDriverException : Creating the web driver failed with the message "Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.". This can mean that there is a leftover web driver process that you have to kill manually. Full exception: Atata.WebDriverSetup.DriverSetupException: Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.
	 ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 404 (The specified blob does not exist.).
	   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
	   at Atata.WebDriverSetup.HttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.<>c__DisplayClass4_0.<DownloadFile>b__0()
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.ExecuteWithRetries[TResult](Func`1 operation)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   --- End of inner exception stack trace ---
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   at Atata.WebDriverSetup.DriverSetupExecutor.SetUp(String version)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUpDriver(DriverVersionResolver driverVersionResolver, DriverSetupExecutor setupExecutor, String driverVersion)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.ExecuteSetUp()
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUp()
	   at Atata.WebDriverSetup.DriverSetup.AutoSetUp(String browserName)
	   at Lombiq.Tests.UI.Services.WebDriverFactory.AutoSetup(String browserName) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 206
	   at Lombiq.Tests.UI.Services.WebDriverFactory.CreateDriverAsync[TDriver](String browserName, Func`1 driverFactory) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 190
	---- Atata.WebDriverSetup.DriverSetupException : Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.
	-------- System.Net.Http.HttpRequestException : Response status code does not indicate success: 404 (The specified blob does not exist.).

I this expected and https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip for some reason now randomly giving a 404, or should it use a different URL?

A Linux driver for 127.0.2651.74 is missing according to https://msedgewebdriverstorage.z22.web.core.windows.net/?prefix=127.0.2651.74/. Strange that Atata.WebDriverSetup decided to download that version.

OK. I'm able to reproduce that. Let me try to fix that.

Thank you!

@Piedone, I've just released v2.13.0 of Atata.WebDriverSetup package with the fix.

Thank you! I can confirm it works (see run here).

I got this now here:
OpenQA.Selenium.WebDriverException : Creating the web driver failed with the message "Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.". This can mean that there is a leftover web driver process that you have to kill manually. Full exception: Atata.WebDriverSetup.DriverSetupException: Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.
	 ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 404 (The specified blob does not exist.).
	   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
	   at Atata.WebDriverSetup.HttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.<>c__DisplayClass4_0.<DownloadFile>b__0()
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.ExecuteWithRetries[TResult](Func`1 operation)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   --- End of inner exception stack trace ---
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   at Atata.WebDriverSetup.DriverSetupExecutor.SetUp(String version)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUpDriver(DriverVersionResolver driverVersionResolver, DriverSetupExecutor setupExecutor, String driverVersion)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.ExecuteSetUp()
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUp()
	   at Atata.WebDriverSetup.DriverSetup.AutoSetUp(String browserName)
	   at Lombiq.Tests.UI.Services.WebDriverFactory.AutoSetup(String browserName) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 206
	   at Lombiq.Tests.UI.Services.WebDriverFactory.CreateDriverAsync[TDriver](String browserName, Func`1 driverFactory) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 190
	---- Atata.WebDriverSetup.DriverSetupException : Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.
	-------- System.Net.Http.HttpRequestException : Response status code does not indicate success: 404 (The specified blob does not exist.).

I this expected and https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip for some reason now randomly giving a 404, or should it use a different URL?

Same error is happening again ☹ With the exact same error message.
OpenQA.Selenium.WebDriverException : Creating the web driver failed with the message "Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.". This can mean that there is a leftover web driver process that you have to kill manually. Full exception: Atata.WebDriverSetup.DriverSetupException: Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.
	 ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 404 (The specified blob does not exist.).
	   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
	   at Atata.WebDriverSetup.HttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.<>c__DisplayClass4_0.<DownloadFile>b__0()
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.ExecuteWithRetries[TResult](Func`1 operation)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   --- End of inner exception stack trace ---
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   at Atata.WebDriverSetup.DriverSetupExecutor.SetUp(String version)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUpDriver(DriverVersionResolver driverVersionResolver, DriverSetupExecutor setupExecutor, String driverVersion)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.ExecuteSetUp()
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUp()
	   at Atata.WebDriverSetup.DriverSetup.AutoSetUp(String browserName)
	   at Lombiq.Tests.UI.Services.WebDriverFactory.AutoSetup(String browserName) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 206
	   at Lombiq.Tests.UI.Services.WebDriverFactory.CreateDriverAsync[TDriver](String browserName, Func`1 driverFactory) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 190
	---- Atata.WebDriverSetup.DriverSetupException : Failed to download Edge driver v127.0.2651.74 by URL 'https://msedgedriver.azureedge.net/127.0.2651.74/edgedriver_linux64.zip'.
	-------- System.Net.Http.HttpRequestException : Response status code does not indicate success: 404 (The specified blob does not exist.).

Currently using 2.13.0 of Atata.WebDriverSetup.

@wAsnk, I've released v2.14.0 of Atata.WebDriverSetup package with a fix. Please verify again using the latest.

It works now, thank you.

I hate to bring this up but we again see the same error, see here, with v2.14.0:
	OpenQA.Selenium.WebDriverException : Creating the web driver failed with the message "Failed to download Edge driver v129.0.2792.89 by URL 'https://msedgedriver.azureedge.net/129.0.2792.89/edgedriver_linux64.zip'.". This can mean that there is a leftover web driver process that you have to kill manually. Full exception: Atata.WebDriverSetup.DriverSetupException: Failed to download Edge driver v129.0.2792.89 by URL 'https://msedgedriver.azureedge.net/129.0.2792.89/edgedriver_linux64.zip'.
	 ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 404 (The specified blob does not exist.).
	   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
	   at Atata.WebDriverSetup.HttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.<>c__DisplayClass4_0.<DownloadFile>b__0()
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.ExecuteWithRetries[TResult](Func`1 operation)
	   at Atata.WebDriverSetup.ReliableHttpRequestExecutor.DownloadFile(String url, String filePath)
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   --- End of inner exception stack trace ---
	   at Atata.WebDriverSetup.DriverSetupExecutor.DownloadDriverIfMissing(String version, String driverDestinationDirectoryPath, String driverDestinationFileName)
	   at Atata.WebDriverSetup.DriverSetupExecutor.SetUp(String version)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUpDriver(DriverVersionResolver driverVersionResolver, DriverSetupExecutor setupExecutor, String driverVersion)
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.ExecuteSetUp()
	   at Atata.WebDriverSetup.DriverSetupConfigurationBuilder.SetUp()
	   at Atata.WebDriverSetup.DriverSetup.AutoSetUp(String browserName)
	   at Lombiq.Tests.UI.Services.WebDriverFactory.AutoSetup(String browserName) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 224
	   at Lombiq.Tests.UI.Services.WebDriverFactory.CreateDriverAsync[TDriver](String browserName, Func`1 driverFactory) in /home/runner/work/Open-Source-Orchard-Core-Extensions/Open-Source-Orchard-Core-Extensions/test/Lombiq.UITestingToolbox/Lombiq.Tests.UI/Services/WebDriverFactory.cs:line 201
	---- Atata.WebDriverSetup.DriverSetupException : Failed to download Edge driver v129.0.2792.89 by URL 'https://msedgedriver.azureedge.net/129.0.2792.89/edgedriver_linux64.zip'.
	-------- System.Net.Http.HttpRequestException : Response status code does not indicate success: 404 (The specified blob does not exist.).

@Piedone, looking into this. Unfortunately, algorithm of resolution of driver version corresponding to Edge browser version is not perfect. Will try to find some solution for improvement.

Thank you! I tested the new version and it appears to be working properly.

Great. You are welcome. As always, please keep letting me know when you face any issues.


Error When Donating To collective with different host

Pia Got this when tried to make a donation to opencollective.com/operationcode-collective from Open Collective Inc. (opencollective.com/opencollective-company):

This stripe error usually happens when the customerId is sometimes too old and was deleted(most cases) or by any other reason the customerId was deleted. Readding credit card may make it work.


newStronghold should clear all status effects

Rationale
If someone has been poisoned by a witch or a cave spider or has other status effects from a gapple/god apple, that shouldn't carry over into the next stronghold.

A potential exception to this is night vision - but I mostly agree.


EEG-inception MI results

As the decription in EEG-inception[1], it has the accuracy 88.39% even without the augumentation, but it has no description of Preprocessing steps
In my attempt on BCIC-IV 2a dataset with the default preprocessing tutorial of 2, the 5-fold model validation acc is 57.64%±8.70% ，seems has the overfit problem. How should I prepocrssing the origin data, and reproduce the results in the paper?
Looking forward your reply, thanks!
[1] Zhang, C., Kim, Y. K., & Eskandarian, A. (2021). EEG-inception: an accurate and robust end-to-end neural network for EEG-based motor imagery classification. Journal of Neural Engineering, 18(4), 046014.

Hi @NormanGod,
Thank you for using braindecode!
This could be happening for several reasons, and unfortunately, it is a huge part of the challenge of open-source science. That is why the authors must publish the codes in some reproducible framework, and reproductions usually report between 15~5% less than the authors write.
I can help you, but I need more information.
In reality, the EEG-Inception has two homonym papers, one for ERP and another for Motor Imagery, from different authors. Here, we have the implementation for both, but only in the more recent version and not released.
What version of braindecode do you have installed? To install the version directly from the repository, you will need to install it with the command:
pip install -U https://api.github.com/repos/braindecode/braindecode/zipball/master

Try to import EEGInceptionMI to ensure that you are using the correct model. Could you also provide our code? It could be a minor bug that is hard to catch, even copying some braindecode tutorial. I recommend trying to remove the pre-processing step if your goal is reproduction.
Also, a disclaimer, in models that the braindecode team has reimplemented, there is no guarantee that they are entirely correct. We always try our best, but several components can impact the final result. Unfortunately, the core team have limited time for extensive studies to reproduce every choice not reported in the article.

Hi @bruAristimunha thanks for taking a look!
Here are my code for reproduction
https://colab.research.google.com/drive/1Oav9DE0vXdBh6UTLfBoegZYYTTyyfftV#scrollTo=GWU1fdumwaOG
Since it has no GPU, I training on my own PC with same code. the result of 5-fold validation on first person is in the attachment zip.
reproduce_EEG_inceptionMI.zip
Validation setup
We want to use the validation scheme like [How to train test and tune] (https://braindecode.org/stable/auto_examples/plot_how_train_test_and_tune.html) of option 3. But utilize the validation model with the highest validation accuracy and test session_E only once. How should the best model be selected, and what changes should be made to the code?
Shuffle problem
We utilized to divide the train dataset into numerous epochs (3s in BCIC IV -2a), shuffle epochs (between epochs, not within each epoch ), and then use kfold validation to distinguish training data from validation data. Is this correct? or may result in time series data leaky? If we used a longer epoch, would this problem be resolved?
Reproduction problem
As the description in code, the author mention their hyper parameters and preprocessing settings in details, only say they use Adam optimizers with the learning rate of 5e-3 and batch size of 32.
In result of k-fold cross-session on subject 1, it seems can not reach the same performance mention in paper, which part might be wrong?
looking forward to your reply, thanks.

Hi @NormanGod,
Unfortunately I can't access the colab, can you change the view option to public?
I also applied for access three days ago

Hi @bruAristimunha,
Sorry for the inconvenience :( , please try this new webpage: https://colab.research.google.com/drive/1YKnwBKpTFyTlH0Id6nk_VhfREy-eSgv9?usp=share_link


Refactor StepChainParentage thread to resolve by workflow

Fixes #11693
Status
ready
Description
Refactor how data is organized in the StepChainParentageFix cherrypy thread, such that it goes through each workflow at a time; instead of aggregating all of the datasets that require parentage fix, dealing with all of them, and only then making the status transition.
With these changes, we have a smaller memory footprint but a larger number of HTTP calls to the DBS server, as multiple workflows could have the same input and/or output dataset names.
Note-1: spike in memory can still occur, whenever a large workflow needs to have its parentage fixed (e.g.: cmsunified_task_SMP-RunIISummer20UL18wmLHEGEN-00542__v1_T_230314_093054_8766).
Note-2: the parentage resolution logic has not been changed at all. Only the order of datasets and reqmgr2 updates.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
None
External dependencies / deployment changes
None

I tested this with a real workflow and it works fine, so I decided to patch the production reqmgr2-tasks service and let it go through a cycle. After this cycle, we will know what is stuck because the service was misbehaving, and what is stuck because we need to have the relevant changes in in DBS server.

Quick update: WM central services have been redeployed with a stable docker image, which means the changes provided in this PR are no longer applied to the StepChainParentageFix cherrypy thread. I don't see any restarts of the thread today, but if I start seeing those, I will re-apply this patch such that we can skip the very large workflow that brings the application down.

I have just applied this patch again to the production reqmgr2-tasks pod and restarted the service. Timestamp is around Thu Sep 14 11:05:39 UTC 2023.

@vkuznet I should have addressed all of your comments now, besides the python shebang which I am not sure it's meaningful.

Thank you for the review, Valentin and Todor. The only modification I have done after your review was to remove an already planned commit, which was hacking the code to skip a given very large workflow. Initial description has been updated accordingly.