modspeech doesn't work on iPad Safari 13.x and 14.1 (works on 14.5)

On #109 we fixed audio for older Safari by using the prefixed APIs and older style of node/buffer creation.
It didn't fix speech because modspeech uses a 19000 sample rate but old Safari has a lower bound of 22050 (contrary to the spec but it was early days).
Coincidentally (I assume!), modspeech has a mode=3 option that doubles the sample rate. That mode works in Safari 13. There’s also some code for linear interpolation but not sure it happens with mode=3 or whether it's needed for an OK result.
So far as I can tell we can only trigger mode=3 by passing an undocumented mode keyword argument from user code.
@dpgeorge what is the mode used for? Do you think we could use this or something similar to get a >= 22050 sample rate in the simulator? We could consider doing this just for older Safari if we felt it would impact the output. We could detect the need to do this in the speech hal init function or any point after a user interaction with the simulator.
Alternatively we could look at resampling in JavaScript between modspeech and Web Audio. The older Web Audio API has no support for this so we'd be doing it ourselves.

CC @microbit-robert

Since #111 (unreleased, available on staging) we now handle the error and just skip playing rather than leaving the sim in a bad state. Still interested in persuing a sample rate change.

Coincidentally (I assume!), modspeech has a mode=3 option that doubles the sample rate. That mode works in Safari 13. There’s also some code for linear interpolation but not sure it happens with mode=3 or whether it's needed for an OK result.

I've just tested mode=3 and fixed a small bug related to it (now pushed to master micropython-microbit-v2).  I think it would be fine to use this mode, it sounds similar enough to the default mode=1.
There's also a mode=4 which is a linearly interpolated version of mode=3.  You can also try this mode 4, it may be better (or worse!) on a PC's speaker.

So far as I can tell we can only trigger mode=3 by passing an undocumented mode keyword argument from user code.

Yes, that's correct.
I guess you would like to have the default mode compile-time configurable?

I guess you would like to have the default mode compile-time configurable?

@dpgeorge yes, I think it would make sense for the simulator build to define the mode.

OK, I've now pushed a commit to the micropython-microbit-v2 repository which allows setting the default mode.  You can add this line to the simulator's mpconfigport.h file:
#define MICROPY_PY_SPEECH_DEFAULT_MODE (3)

OK, I've now pushed a commit to the micropython-microbit-v2 repository which allows setting the default mode. You can add this line to the simulator's mpconfigport.h file:
#define MICROPY_PY_SPEECH_DEFAULT_MODE (3)

Thanks, confirmed this works in Safari 13. I'll close this issue when the simulator changes are merged.


Store reasonable frame IDs in expData for randomizer-generated frames

Right now frames are named using position in the entire sequence + highest-level frame name, e.g. 32-test-trials; would like to allow use of sub-names within randomizers.

Moved to lookit/ember-lookit-frameplayer#49


chore: update controller-gen 0.11.1 to 0.11.3

Please regenerate the manifests.

/ok-to-merge

Shouldn't this be handled by renovate? 🤔

@phisco it's to fix another backport that update the version I think renovate wants to update to another version


chore: update import template response parsing

Description
The response meta was added as mentioned in https://github.com/appsmithorg/appsmith/issues/11661. So the response structure changed which needs an update on the frontend.
Previous response structure
{
  applicationId: ...
  ...
}

After the response meta addition from the backend
{
   responseMeta: {
      ...
   },
   data: {
     applicationId: ...
     ....
   }
}

The frontend change is to conform to this spec.
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/reposnse-parsing

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.63 (0)
36.88 (0.01)
34.95 (0)
55.94 (0.01)

:red_circle:
app/client/src/sagas/TemplatesSagas.ts
26.83 (-2.12)
8.33 (-1.67)
16.67 (0)
32.35 (-2.03)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
85.25 (1.64)
64.71 (2.95)
73.33 (0)
90.59 (2.35)

:green_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.94 (0.23)
41.67 (0.84)
36.21 (0)
56.99 (0.25)

/ok-to-test sha=262633c

/ok-to-test sha=c40a1cf


is gas conduit supposed to handle multiple gasses ?

I am curious if the gas conduit is supposed to work like the fluid conduits and handle multiple gasses on one conduit or just one gas per conduit?

It behaves like a normal (not the Ender) fluid conduit - or the Mekanism gas conduits - only a single gas type at a time.

thanks :) mekanism is single.. i was hoping it would do more now that the versions are compatable .. :/ maybe in the future having a option to do multiple ?

I can't tell you how much I would love it if the gas conduits could handle multiple Mekanism gases. One of my favorite features of the  conduits is to assign color channels to inputs and outputs. It makes IO networks so tidy and organized!


te-Malta

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Is your app/project/product currently shipping and/or have the source code available?
[x] If an app, does your homepage include links to download the Android and/or iOS apps?
[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~10 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No cached entry, and https://github.com/GOVMT-MITA/mt.gov.dp3t.android has issues with stats fetching: Cannot read property 'length' of undefined
No cached entry, and https://github.com/GOVMT-MITA/mt.gov.dp3t.android has issues with start date fetching:, 404 - {"message":"Branch not found","documentation_url":"https://docs.github.com/rest/reference/repos#get-a-branch"}

Build failed because of:
No cached entry, and https://github.com/GOVMT-MITA/mt.gov.dp3t.android has issues with stats fetching: Cannot read property 'length' of undefined
No cached entry, and https://github.com/GOVMT-MITA/mt.gov.dp3t.android has issues with start date fetching:, 404 - {"message":"Branch not found","documentation_url":"https://docs.github.com/rest/reference/repos#get-a-branch"}

LGTM


textDocument/didChange full send should be array

When telling the LSP about content changes, the path for when sync_kind is .Full and the server doesn't support incremental changes sends a manually constructed textDocument/didChange event.  See lsp/init.lua L961.
The quite simple nelua-lsp server can't cope with the way contentChanges is written.
It's written as a field that is one object, containing .text with all the text -- but according to the LSP spec DidChangeTextDocumentParams definition, .contentChanges must be an array of change events.
I've changed lsp/init.lua L970, L972 locally to add array [ ] braces around the constructed text change event.

Thanks a lot for investigating this!
I remember when implementing the lsp client initially I didn't supported sending incremental changes so client didn't reported support for incremental changes and the data was sent as you pointed out, it worked with the servers I tested or that is what I thought, crazy hehe, maybe it happened after implementing the sending raw functionality, those where some long nights trying to understand spec :).
Looking at what you say then it should look like this:
.. '"contentChanges": ['
.. '{"text": "'..text..'"}'
.. "]"

Will commit this change right now!


FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

When I run modulizer --out . I get the following error:
C:\me\myProject>modulizer --out .
[1/2] ��  Converting Package...
Out directory: C:\me\myProject

<--- Last few GCs --->

[20284:000002000B7E6970]    98725 ms: Mark-sweep 1331.6 (1461.2) -> 1331.5 (1461.7) MB, 1729.7 / 0.0 ms  allocation failure GC in old space requested
[20284:000002000B7E6970]   100165 ms: Mark-sweep 1331.5 (1461.7) -> 1331.5 (1426.2) MB, 1438.9 / 0.0 ms  last resort GC in old space requested
[20284:000002000B7E6970]   101887 ms: Mark-sweep 1331.5 (1426.2) -> 1331.5 (1426.2) MB, 1721.4 / 0.0 ms  last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 00000323F3BA57C1 <JSObject>
    1: stringSlice(aka stringSlice) [buffer.js:560] [bytecode=000001E4864E4D19 offset=94](this=00000145C76822D1 <undefined>,buf=000003B2A364ABF1 <Uint8Array map = 0000039A83F43231>,encoding=00000323F3BB4EB1 <String[4]: utf8>,start=0,end=511355)
    2: oncomplete(aka readFileAfterClose) [fs.js:~489] [pc=000003A4F9AF91C2](this=000002D4084D79B1 <FSReqWrap map = 0000003B0AD34B89>,err=00000145C7682...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node_module_register
 2: v8::internal::FatalProcessOutOfMemory
 3: v8::internal::FatalProcessOutOfMemory
 4: v8::internal::Factory::NewRawOneByteString
 5: v8::internal::Factory::NewStringFromOneByte
 6: v8::internal::Factory::NewStringFromUtf8
 7: v8::String::NewFromUtf8
 8: v8_inspector::V8InspectorClient::currentTimeMS
 9: node::Buffer::New
10: v8::internal::wasm::SignatureMap::Find
11: v8::internal::Builtins::CallableFor
12: v8::internal::Builtins::CallableFor
13: v8::internal::Builtins::CallableFor
14: 000003A4F99843C1

setting node options doesn't help on linux.
others things I tried
npx --max_old_space_size=8000 modulizer --out .
node --max_old_space_size=8000 modulizer --out .

@derhuebiii I gave up on this and wrote my own https://github.com/EvolutionJobs/HtmlImport2ESModule
It's much dumber than this is trying to be, it's basically a bunch of regexes, but it worked for me for this and #2811 (which was a deal breaker for me as we're continuing to maintain and develop after the conversion and the hidden <div> tags that this outputs are a mess).

Thanks all. I finally managed to convert the files (see EDIT in previous post), but I had to put all in one directory (no subdirectories). Also all references to Polymer components where removed. I did similar to you a search and a replace macro using my old <link rel="import" being replaced to import ...
The tool was not very smooth for me but all in all it looks I iwll get my project to work. Only missing polymerfire support, but that's offtopic :)

@jkanche try node --max-old-space-size=8000 modulizer --out .

@bicknellr will you have some time to look into this after the holidays?
Our users are experiencing issues with running relatively big projects (~ 200 HTML files) through polymer-modulizer. It's quite annoying to converting files one by one because of that.


fix: don't use css transitions while the page is still hydrating

This is really a major issue only on sites with an extensive load time and even then only on mobile/tablet.

moving to theme: https://github.com/carbon-design-system/gatsby-theme-carbon/pull/638


Automated cherry pick of #205

Cherry pick of #205 on master.
#205: fix for #187 failure when using RFC 6902 json-patch

Codecov Report

Merging #211 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #211   +/-   ##
=======================================
  Coverage   94.56%   94.56%
=======================================
  Files           9        9
  Lines         681      681
=======================================
  Hits          644      644
  Misses         37       37

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 b5eac1f...058b81b. Read the comment docs.


Add validator method returning default value.

Not very fluent in Kotlin so this may be syntax error, but I think you get the idea.

@vrozkovec what is the use case for this? There is already default overloads like ctx.queryParam(key, class, default) (although I think like your version better).

@vrozkovec what is the use case for this? There is already default overloads like ctx.queryParam(key, class, default) (although I think like your version better).

Huh, missed those. However those require you to pass String as default value and once you pass String, there should be no need for .getOrNull(), using .get(defaultValue: T) feels more straightforward.
Or perhaps changing the signature of ctx.queryParam(key, class: Class<T>, default: String) to  ctx.queryParam(key, class: Class<T>, default: T), but that would have to go to next major version, right?

using .get(defaultValue: T) feels more straightforward.

I agree, but I want to avoid having two ways of doing it 🤔
If we merge this PR, it needs to include deprecations (and tests).

but that would have to go to next major version, right?

Yes, unfortunately.

If we merge this PR, it needs to include deprecations (and tests).

Ok, I'll see what I can do.


Backfilled a few Player tests and moved opengl and shader stuff into Window

Backfilled a few tests for Player
Updated readme with testing instructions
Moved opengl and shader stuff into Window in order to hopefully narrow the scope of World to just managing objects.

I noticed that blocks are now a bit fuzzy... not really sure what happened, but would love if someone else could take a look for me before merging :)

Some conflicts with the docs stuff I just merged in.

Yup, I think I'll just abandon this one, since it wasn't really many changes, and I'm not sure it's worth it


slider seems to be broken

when bringing up a dataset the slider show a NaN

slider for GAIA data only

fixed for Gaia but data not displaying

fixed. error in how data was read properly


Add PDF page count to response

Requested by @Atinux in #10

+1!

Page counts can be used with css3. It is 4am right now but I had to use this for a ton of stuff so I am going to ask author if I can contribute docs for some issues everyone seems to have, this being one of them:
Mozilla Link for docs on CSS 3 page counter
I pass that metadata obj to html-pdf, it is my options object. it is exported in node as a js file, but you can see from the code how it works, it has sloppy 1999 level's of   but they don't render. You may have to put a span tag in and give it a large margin.
working example:
options.js
 metadata: {
        doctype: doctype,
        company: normalizedCompany,
        directory: outbox + '/' + normalizedCompany,
        html: outbox + '/' + normalizedCompany + '/' + normalizedCompany + '.html',
        logo: outbox + '/' + normalizedCompany + '/' + normalizedCompany + '-logo.png',
        pdf: outbox + '/' + normalizedCompany + '/' + normalizedCompany + '-' + doctype + '.pdf',
        options: {
          height: '11in',
          width: '8.5in',
          border: {
            top: "0.1in",
            right: "0.25in",
            bottom: "0.1in",
            left: "0.25in"
          },
          header: {
            height: "1.5in",
            contents: '

' + doctype + '  ' + req.body.jobInfo.referenceNumber + '

' + req.body.jobInfo.host + ' ' + req.body.jobInfo.name + '

'
          },
          footer: {
            height: "0.5in",
            contents: '
 ' + req.body.companyInfo.name + ' – ' + req.body.companyInfo.city + ' – ' + req.body.companyInfo.email + ' – ' + req.body.companyInfo.website.replace(/.*?:\/\//g, "") + '  ' + '    Page
'
          }
        }
      },

main.css
body {
  counter-reset: pagecounter;
}
.page {
  counter-increment: pagecounter;
}
.page:after {
  content: counter(pagecounter);
}

You can attach external css sheets with  tag1: 'file://' + basepath + '/app/themes/reset.css',
that syntax. That code is passed to handlebars and renders as something like (note that it is an absolute root path so there are actually 3 slashes when handlebars calls it`, you can put this in the head of your document :
file:///Users/you/Desktop/style/style.css
In testing I loaded 4 external sheets, however if possible, I would suggest compiling/minifying it and including it.
Base64 images render like shit.
also, if you need an image in the header include it with similar syntax to above in the body with a display: none, the header will recognize the external file and render it.


refactor: upgrade React to 18 and mui/* to latest

This PR upgrades React to 18 and mui/* packages to the latest stable release.

Looks good to me.

This is wonderful! I hope someone can merge this in and publish a new version ASAP!

@patorjk Can this be merged in? I know a lot of people are waiting for it.

@Lori-Becker I'll take a look at this tonight.

Thank you for putting this together. Just published it in the latest release.

This is awesome. Thank you for this. @Usama-Tahir  🙌


Some publish script changes and updated contributing documentation

This still needs to be fully QA'd but I think this is an improvement? I was more hopeful that we'd have better ways to test a WIP version of shared-components in community, but this invalid hooks error was kind of a pain. Feel free to share feedback, I'm sure this isn't 💯 and I'm probably missing easier ways to do things/explain things/etc.
Also I'd like to confirm my changes to the publish script. In my mind this makes sense but I also don't want to test it by publishing unnecessary versions if I can help it, so please check my logic at least :)

:rocket: PR was released in v0.19.0 :rocket:


Prepare start tender data

Not ready to merge

Reviewed 4 of 4 files at r1, 2 of 2 files at r2, 3 of 3 files at r3.
Review status: all files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

Comments from the review on Reviewable.io

Мені пасує

Reviewed 1 of 4 files at r1, 1 of 2 files at r2, 3 of 3 files at r3, 1 of 2 files at r4, 2 of 2 files at r5.
Review status: all files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

Comments from the review on Reviewable.io

Reviewed 2 of 2 files at r4, 2 of 2 files at r5.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from the review on Reviewable.io

Reviewed 8 of 8 files at r6.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r7.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


feat: Install operators in batch.

user can use the flag "-batch-chunk-size" to control how many subscriptions will be deployed simultaneously
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #624
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

/test all

/retest

/retest

@Daniel-Fan @ZhuoxiLi
This PR is ready for review.
This PR is changed much from the last review. It still uses locks to update status. I tried pushing status update into a channel, but the effect is not very good.
Let's use the lock as version 1. We can make further refactor in future PRs.

Looks, it didn't pass the e2e test. I will make more testings on it

/retest

/retest

@Daniel-Fan @ZhuoxiLi
It finally passes all the tests. Please review this PR.
We can take it the stage 1 of the concurrence support. We can do more reactors on it in the future.


[BUG] Command config market [value] not working

Describe the bug
As indicated in the docs, we can reconfigure a parameter on a single line by doing config [key] [value]. Tested on all pure market making params and worked except for config market and config price_source_market.
config market [value]
config price_source_market [value]
Steps To Reproduce
Enter config market [value] or config price_source_market [value] on your PMM strategy.
Screenshots

Release version
version-0.29.0
┆Issue is synchronized with this Clickup task by Unito

Same behavior for market pairs on both Cross exchange and Arbitrage strategy;
Cross Exchange:

Arbitrage:

Same behavior for config market [value] where the hyphen is emitted

but the bot shows different behavior for config price_source_market [value] where the hyphen is emitted.
Right pane doesn't generate message, while left pane (running logs) generates message.

Another similar one-line command  issue but not on config market or config_price_source_market,
It's on config paper_trade_account_balance

Failed to initialize one line command for changing market (Example: config market XRP-USDT)
Steps:
Enter config market [value] or config price_source_market [value] on your strategy (upon checking all strategies affected)
Actual:
For the active pair, Bot showing that pair is not active market on the exchange and showing promt for manually search another pair.
Expected:
User could change the market with one line of code


I am not able to see the onlyoffice options in main menu

Hi @LinneyS,
I restart the dockers and rerun the $ sudo bash set_configuration.sh
But I am not able to see again the docs on the menu.
I shared here some images in order to help.
1ª) Trying to create a new document writer

2ª) Some errors showed on backend

3ª) Apps Management

4ª) Market

Thanks in advance

Hello @ivanoterouvigo, please specify some more information, as we cannot reproduce the described issue on our side:

OS of the server;
Did you use docker-compose to deploy Document Server and Owncloud?
Version of Owncloud;
Which command did you use to restart containers?
Specify if you were able to create documents before you restarted docker containers.

Hello @alexanderonlyoffice,
1. OS of the server;
Webservers: nginx-server 1.14.2
Distributor ID:	Ubuntu
Release:	16.04
--------------------------------------------------------------------------------------------------------------
2. Did you use docker-compose to deploy Document Server and Owncloud?
Yes I did,  $ sudo docker-compose up -d

--------------------------------------------------------------------------------------------------------------
3. Version of Owncloud;
OCM Provicer: Owncloud10.0.10
--------------------------------------------------------------------------------------------------------------
4. Which command did you use to restart containers?
$ sudo service docker restart
--------------------------------------------------------------------------------------------------------------*
5. Specify if you were able to create documents before you restarted docker containers.
Yes I was able to create documents before.

@ivanoterouvigo, unfortunately, we cannot reproduce the issue on a default installation, please specify some additional information:

Did you perform default installation or changed some parameters (ports or something else)?
Screenshot of ONLYOFFICE integration app address settings from OwnCloud;
Specify what was the reason for rebooting the system?
Please send us .yml file to<EMAIL_ADDRESS>with a note 'For technical support';
Please send us all logs of Document Server within the same message;
Please send us log of OwnCloud within the same message.

@ivanoterouvigo, we received the .yml file, there are no errors in it. Please send us the rest requested information.
Please specify also if you changed address settings. We could partially reproduce the issue only when changed the settings underlined red on this screenshot.

Hi @alexanderonlyoffice,
1. Did you perform default installation or changed some parameters (ports or something else)?
I made the default installation.
But when I made I had the IP: <IP_ADDRESS>
Now I have other IP: <IP_ADDRESS> but behind a firewall.
So I access via. http://firewall.teleco.uvigo.es:57228
2. Screenshot of ONLYOFFICE integration app address settings from OwnCloud;

3. Specify what was the reason for rebooting the system?
Trying to solve this issue.
https://github.com/ONLYOFFICE/onlyoffice-owncloud/issues/243
4. Please send us .yml file to<EMAIL_ADDRESS>with a note 'For technical support';
File sent.
5. Please send us all logs of Document Server within the same message;
6. Please send us log of OwnCloud within the same message.

@alexanderonlyoffice, sorry but I was not able to see the  screenshot.
I got this error.

@ivanoterouvigo, we received the log of ownCloud.
Please also do the following:

Click on the 'Save' button on the page with ONLYOFFICE integration app address settings and check if there are any errors. If yes, please send us a screenshot of the error.
Send us logs of Document Server and ownCloud once again after you click on the 'Save' button. Logs of Document Server are in folder /app/onlyoffice/DocumentServer/logs/documentserver/ or in /var/log/onlyoffice/documentserver/logs inside the container with Document Server.

@alexanderonlyoffice,
1. Click on the 'Save' button on the page with ONLYOFFICE integration app address settings
Saved and this is the error.
Error when trying to connect (Bad Request or timeout error)

2. Send us logs of Document Server and ownCloud once again.
Logs
 **Owncloud log sent.**

2.1. /app/onlyoffice/DocumentServer/logs/documentserver/ it doesn't exit.

2.2. /var/log/onlyoffice/documentserver/logs - it doesn't exist.

@ivanoterouvigo, it seems that address firewall.teleco.uvigo.es:57228 is inaccessible for the Document Server. Please try to enter http://nginx-server/ in 'Server address for internal requests from the Document Editing Service' and click on the 'Save'.

@alexanderonlyoffice ,sorry, How could I do that?

@ivanoterouvigo, type in the address http://nginx-server/ in the field 'Server address for internal requests from the Document Editing Service' and click on the 'Save' button.


Custom fetch hook

Thought these hooks might be useful additions to Gateway.
The idea would be to use what's there in useFetchById as boiler and adjust as needed.

Closing pull request because proposed addition does not fit vison of repo


Setup ArchUnit rules banning Guava Collection factory methods

The Javadoc on these methods notes that they should be treated as deprecated.
This PR replaces these methods with simple constructor calls.  It adds an ArchUnit rule that prevents them being added.  It only affects the no-arg versions, as the argument-taking overloads are still valid.
Affected methods:

Lists.newArrayList
Lists.newCopyOnWriteArrayList
Lists.newLinkedList
Maps.newConcurrentMap
Maps.newHashMap
Maps.newLinkedHashMap
Maps.newIdentityHashMap
Maps.newTreeMap
Sets.newCopyOnWriteArraySet
Sets.newHashSet
Sets.newLinkedHashSet
Sets.newTreeSet

The large number of files changed here should just be replacing these calls with the appropriate constructor calls and cleaning imports as needed.  The new rules are in GuavaCollectionFactoryUsageTest.java.

@bot-gradle test this

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


Update command line arguments when running the examples locally

Dear maintainers,
apparently, webpack > 5 replaced "content-base" with "static". Changing this line allows to run the examples successfully on more recent webpack versions.
All the best

I have realized more fixes are needed in order for certain examples to work as expected.
I will close this PR and open a new one that will contain all of the fixes.


Consistent tabs-spaces usage in CMakeLists.txt, and remove template-file

This PR brings two tiny (and slightly unrelated) changes that weren't worth one PR each:

the first one fixes converter.py so that the generated CMakeLists.txt uses 4 spaces consistently, instead of a mix of tabs and spaces;

the second removes CMakeLists_template.txt before the program exits.

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


New release with latest commit

We have an issue with the current mutating webhook and require it update its version to v1. There is a commit that was already pulled into the master branch for this, but it was not put into a release. We would like to please have this in a release. We have lost K8s monitoring in our clusters because of this issue. Until bottlerocket OS is supported by Dynatrace, we cannot move to the newest operator.

Hi, as this operator is deprecated there will be no new release to it. However the new dynatrace-operator is capable of doing everything the old oneagent-operator did aswell, so Bottlerocket OS should also work with classicFullStack in the dynatrace-operator. Only cloudNativeFullStack currently has the limitation with the CSI Driver not yet running on that OS (we currently work on removing that limitation).

@luhi-DT  Are we able to store the image of the oneagent codemodule in a private ECR with the new dynatrace-operator?


This will allow for the rename-function to also move files, in Linux

In the rename-template, use an absolute path to move files
for example
/Comics/%publisher%/%series%/%series% %issue%

This feature has been implemented in #129


Update Swift 4.2

Is this library abandoned? I've tested this update and everything is working fine.

last I've try this library in Xcode 10, using swift 4.2 not work @dielsonsales

@balitax I mean after your PR. It is a straightforward update but wasn't merged yet.


Mac Desktop app often locks up

I have been having issues with the Mac Desktop app continually locks up.  Usually when adding a new timer or modifying the time on a previous timer.  The entire UI locks up for a couple of seconds before it allows to me do anything else with the app.  This is ongoing and does not seem to be related to any particular project I am working on.
This seems to have been an issue in the past but was resolved in a later version.  Is it possible that it has the same issue again?
Reference ticket: https://github.com/toggl/toggldesktop/issues/1483
Mac Version 10.10.5
Toggl Desktop 7.3.319

Please send feedback from the app when this happens. Mention that it's for Indrek. That way we get logs and can investigate. Alternatively clearing cache could help fix the issue. https://support.toggl.com/clearing-the-local-data-from-toggl-desktop/#mac

I experience same behaviour. Will send report from app as well.

Clearing the Cache seemed to help and I have not had any timeouts since September 2nd.  It must have been an issue with my Cache that was causing the issue.

Did that with little result. Also cache corruption was the case for #1921. Which brings an idea, that asking users to clear cache is only a workaround, not the solution.


"next" footer links don't pull title from source

Instead of pulling the title attribute from next target in nav metadata, the link generator seems to just strip the file extension and -. This looks weird if the title is different than the title generated from the link.
https://github.com/reactjs/reactjs.org/blob/184a7ef9476c524df0ff82362257c02bfbe8698d/src/templates/components/NavigationFooter/NavigationFooter.js#L89
EDIT: found an existing TODO for this 😄
https://github.com/reactjs/reactjs.org/blob/184a7ef9476c524df0ff82362257c02bfbe8698d/src/components/MarkdownPage/MarkdownPage.js#L125-L132

@alexkrolick Is there someone working on it already? If not, I'd love to take a crack at it.

Sure, it's all yours @raunofreiberg

Gave it a shot at #688. Would appreciate it if you took a look :-) @alexkrolick


Compatible with ppc64le linux

Is this exporter is compatible with ppc64le linux. What are prerequisite for running on ppc64le linux.

Prometheus/Common/log  is missing

I'm not work with db2,not powerpc cpu.send log to me发自我的平板-------- 原始邮件 --------发件人： bhattanil @.>日期： 2023年1月9日周一 14:17收件人： glinuz/db2_exporter @.>抄送： Subscribed @.***>主    题： Re: [glinuz/db2_exporter] Compatible with ppc64le linux (Issue #8)
Prometheus/Common/log  is missing
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Vendor/GitHub.com/ibmdb/go_ibm_db/api_unix.go:13:22: fatal error: sqlcli1.h: no such file or directory.
I am using go version go1.16.15 linux/ppc64le.
//#include <sqlcli1.h>
Compilation terminated.     Logs

Check installed db2 client,ODBC driver.And set ENV path,mybe db2 clidriver files path in ppc63le is differrent to x86.-------- 原始邮件 --------发件人： bhattanil @.>日期： 2023年1月10日周二 01:31收件人： glinuz/db2_exporter @.>抄送： glin @.>, Comment @.>主    题： Re: [glinuz/db2_exporter] Compatible with ppc64le linux (Issue #8)
Vendor/GitHub.com/ibmdb/go_ibm_db/api_unix.go:13:22: fatal error: sqlcli1.h: no such file or directory.
I am using go version go1.16.15 linux/ppc64le.
//#include <sqlcli1.h>
Compilation terminated.     Logs
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Thanks now working after changing go version for ppc64le and path


Multi-module prep

A couple changes to get the tree in shape for multi-module:

Make decisions about whether a particular node should ever be shared local to that node (removing the CompilationModuleGroup's role in sharing decisions).

Remove the static CompilationUnitPrefix property that has caused problems with not being set at the correct time. Instead, ISymbolNodes are offered the chance to prepend it when AppendMangledName is called.

@dotnet-bot test this

uh, CI seems to have given up on this PR. I'll close it out and make another.

Or maybe CI is just borked..


Why Fiddler can't catch the requested data。I use 1.4.9 version

In  my  old  project   I  can  catch  the  request date  but  when i  use  1.4.9  this version  i  can't  .   I don't know  my  old project use  which  version .  I  use  this  version in 2013year .  anyone  can  help me  ..

I've had the same problem. why?

+1

+1

https? I Fiddler version is [Fiddler Web Debugger (v<IP_ADDRESS>)].not you version....
I seem to be the latest.
English is not good, Google Translation


Multi-columns sortValues()

Is your feature request related to a problem? Please describe.
It's very common to sort the dataframe on many columns, and it's currently not possible.
This can be problematic for many use cases.
Describe the solution you'd like
DataFrame.sortValues() can be extended to support array of columns name to sort on, like with Pandas.
Additionnaly, it will be convenient to allow providing different sorting order for each column.

You can achieve what I think you want by just calling sortValues multiple times.
The order of elements that have the same value for the column you are sorting on are stable, i.e. they don't move at all if the are equal. So to get what you might call "sort by column 1 then column 2", you just sort by those column, in reverse order, i.e. sort by column 2 first, then by column 1.
import { DataFrame } from 'danfojs-node'

const data = [
	{ col1: 'b', col2: 'd', str: '1 -> 1 -> 2' },
	{ col1: 'b', col2: 'c', str: '2 -> 2 -> 1' },
	{ col1: 'c', col2: 'a', str: '3 -> 3 -> 3' }
]
const df = new DataFrame(data)

// to get "sort by col1 then col2"
df.sortValues('col2', { inplace: true })
df.sortValues('col1', { inplace: true })
df.print()

note: rows that have the same value for both col1 and col2 will be in whatever order they started in - I guess you can look at this like sorting by the least most important columns first to the most important.

Actually, here is a quick utility function to do it for an arbitrary number of columns:
const sortByColumns = (orig: DataFrame, columns: string[], options?: { ascending?: boolean, inplace?: boolean }) => {
	const df = options?.inplace ? orig : orig.copy()
	for (const column of columns.reverse()) {
		df.sortValues(column, { ...options, inplace: true })
	}
	return df
}


pgbouncer-rr-patch install fails with pgbouncer release v1.7,1

[ec2-user@ip-172-31-2-63 pgbouncer-rr-patch]$ ./install-pgbouncer-rr-patch.sh ../pgbouncer
Merging pgbouncer-rr changes to: ../pgbouncer/Makefile
patching file Makefile
Hunk #3 succeeded at 84 with fuzz 1 (offset -4 lines).
Merging pgbouncer-rr changes to: ../pgbouncer/src/client.c
patching file src/client.c
Merging pgbouncer-rr changes to: ../pgbouncer/src/main.c
patching file src/main.c
Hunk #1 succeeded at 139 (offset 1 line).
Hunk #2 FAILED at 269.
1 out of 2 hunks FAILED -- saving rejects to file src/main.c.rej
Merging pgbouncer-rr changes to: ../pgbouncer/include/bouncer.h
patching file include/bouncer.h
Hunk #1 succeeded at 84 (offset -4 lines).
Hunk #2 succeeded at 458 (offset -4 lines).
copying pgbouncer-rr files: README.md LICENSE.txt NOTICE.txt pgbouncer-example.ini users.txt rewrite_query.py routing_rules.py images/diagram1.jpg images/diagram2-routing.jpg images/diagram3-rewrite.jpg src/pycall.c src/rewrite_query.c src/route_connection.c include/pycall.h include/rewrite_query.h include/route_connection.h
Failures encountered during merge of pgbouncer-rr with pgbouncer.
See error messages above.
Possible causes:
pgbouncer-rr-patch already installed in target directory?
new version of pgbouncer with changed source files that can't be patched?
- tested with pgbouncer v1.7
Status: pgbouncer-rr-patch merge FAILED

Merge context had changed in src/main.c.
Diffs file src/main.c.diffs updated - validated on pgbouncer v1.7.1


Give FTLs insulated gloves

About the PR

Title.
Why / Balance

CM-13 parity, they get insulated gloves. Yeah, same as CombatTech.
Technical details

Media

Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: FTLs get insulated gloves as part of their kit

thats not the FTL gear thats the corpsman gear

dangit

Silly goose


mandatory-script-verify-flag-failed (Script evaluated without error but finished with a false/empty top stack element) (code 16)

I have a contract that's having some inconsistencies I can't explain .. I've tried searching for the error, but none of the hits offered a solution..
meep debug --tx=02000000017296be2da833d7b7e31a7b8e15fdf589e475026328287043cd6a84dfe70cc8dc00000000fd400308cbd0090039070000473045022100c2d72abce0fd542df2930698eddce94271f4d879e7dfa6e9264c29e1dadcb91002207ece5c08bfe499954c34a2f0e9ab1f76a016c7d716e242819cf2515cc9e94b314114731693c66e94a4b68d087b8bf6434e37a36384659e314a8f8fb3247fb41a13a1945bf1d58d7e4000e63aebe27ff3c97ee807d0d63eb31b89877c709d83a4e741210344b28c3daf7c88741c483f068fa4525572587263af3344c631bccf5df6492f124d910102000000a5fcba62f38eae614cf7723686ece695377208659da365de1c321ae1ea763de418606b350cd8bf565266bc352f0caddcf01e8fa789dd8a15386327cf8cabe1987296be2da833d7b7e31a7b8e15fdf589e475026328287043cd6a84dfe70cc8dc00000000f40333d009210344b28c3daf7c88741c483f068fa4525572587263af3344c631bccf5df6492f12144dec2661eb4de14a3b8cda28fb7ca19f0e98cad01455670d530c2feb8a7a2d0bb2d888d6db02e1a1cb14212afcde653565e8121a180aebee97f5d235817d5579009c63567901697f77820134947f587f547f7701207f755b7a5b796e7c828c7f755d7aa87bbbad5a7a5b79587abb5979547f7581597aa9577a876376b1756776b17568567aa269567a547f7781023905537a8152795880041976a9147e567a7e0288ac7e7c537a947b9458800317a9147e537aa97e01877e7eaa87777767557a519d5579a97b8871ad6d6d5168720c000000000000feffffff6594114b69be15493cda32acca1a0e5725b66fc5a843ccf446a77dc0e92b3450fbd0090041000000004cf40333d009210344b28c3daf7c88741c483f068fa4525572587263af3344c631bccf5df6492f12144dec2661eb4de14a3b8cda28fb7ca19f0e98cad01455670d530c2feb8a7a2d0bb2d888d6db02e1a1cb14212afcde653565e8121a180aebee97f5d235817d5579009c63567901697f77820134947f587f547f7701207f755b7a5b796e7c828c7f755d7aa87bbbad5a7a5b79587abb5979547f7581597aa9577a876376b1756776b17568567aa269567a547f7781023905537a8152795880041976a9147e567a7e0288ac7e7c537a947b9458800317a9147e537aa97e01877e7eaa87777767557a519d5579a97b8871ad6d6d5168feffffff0139070000000000001976a914212afcde653565e8121a180aebee97f5d235817d88acfbd00900 --idx=0 --amt=3186 --pkscript=a914133c04f70740a2f5d567679a4b37c095830200fe87

for some reason, I'm unable to install meep to run this and "maybe" shed some light on the issue .. could someone that has a "working" version of meep, run this and post the results??
~I have a few workarounds to this issue that so far HAVE been consistent, but they would degrade the effectiveness of this contract.~ fyi, I have SEVERAL versions of this contract, but I can post one or more if that would help in resolving the issue..
thanks!

edit: here's the error when I try to make meep .. also go get github.com/gcash/meep is hanging, probably for the same reason
env GO111MODULE=on go build .
../../go/src/github.com/gcash/bchd/txscript/pkscript.go:10:2: cannot find package "github.com/gcash/bchutil" in any of:
	/usr/lib/go-1.10/src/github.com/gcash/bchutil (from $GOROOT)
	/home/dev/go/src/github.com/gcash/bchutil (from $GOPATH)
main.go:5:2: cannot find package "github.com/jessevdk/go-flags" in any of:
	/usr/lib/go-1.10/src/github.com/jessevdk/go-flags (from $GOROOT)
	/home/dev/go/src/github.com/jessevdk/go-flags (from $GOPATH)
debug.go:16:2: cannot find package "github.com/nsf/termbox-go" in any of:
	/usr/lib/go-1.10/src/github.com/nsf/termbox-go (from $GOROOT)
	/home/dev/go/src/github.com/nsf/termbox-go (from $GOPATH)
../../go/src/github.com/gcash/bchd/txscript/opcode.go:15:2: cannot find package "golang.org/x/crypto/ripemd160" in any of:
	/usr/lib/go-1.10/src/golang.org/x/crypto/ripemd160 (from $GOROOT)
	/home/dev/go/src/golang.org/x/crypto/ripemd160 (from $GOPATH)
Makefile:9: recipe for target 'all' failed
make: *** [all] Error 1

So I ran the meep command through meep and it looks like your tx.hashOutputs do not exactly match the outputs of the transaction. Note that the CashScript SDK automatically adds a change output for the leftover money in the contract, which might be a source of error here. You can disable this change output by calling withMinChange(Number.MAX_VALUE) when building the transaction.
If that is not the cause of the error, try to double check if the outputs you added using the to() function exactly match the expected output of the smart contract.

Note that the CashScript SDK automatically adds a change output for the leftover money in the contract, which might be a source of error here

hmmm, yeah that sounds about right, considering the inconsistency came with varying UTXO amounts for the same Script .. ugh! likely NOT gonna go back and re-test this right now .. really determined to get the Mecenas contract fully tested (w/ all edge cases) so I can launch Causes Cash asap..

withMinChange(Number.MAX_VALUE)

if I hit this issue again, I'll definitely give this a try..
appreciate you making the time to sort me out 🙏


CAMEL-13230: Error starting SQS consumer due to config option missing that's required for producer only

Thanks, this has been merged on master and backported on 2.x, 2.23.x and 2.22.x


updates for schedule

Outside of the sleep statements I added a comment for, Kevin and I reviewed this and I think it looks good. Kevin will consider this on his next iteration of these scripts since they are currently not under pressure to be delivered. Going to approve based on this.

/lgtm


Syntax error line 5

Hi.
There is no log. The stdout shows the following (Debian Jessie)
eas@debianMaster:~$ ./import-mailbox-to-gmail.py --json /home/eas/TJemail-842823h79543.json --dir /home/eas/mbox/
./import-mailbox-to-gmail.py: línea 5: error sintáctico cerca del elemento inesperado `newline'
./import-mailbox-to-gmail.py: línea 5: `<!DOCTYPE html>'

In english:
./import-mailbox-to-gmail.py: line 5: syntax error unexpected element 'newline'
./import-mailbox-to-gmail.py: line 5: `<!DOCTYPE html>'

Any ideas?
thanks!

I think that instead of performing a GIT checkout or downloading the RAW data, you downloaded the HTML page, which has the doctype exactly at line 5:

Can you please try to download the RAW version instead:
https://raw.githubusercontent.com/google/import-mailbox-to-gmail/master/import-mailbox-to-gmail.py

You're wright. My bad.
I ran the script again with the RAW version and MAGIC.
I'll cry.
Thanks for your effort PMox. It works like a charm!


add pkg.svelte

This field allows Svelte apps using this component to import the source directly, rather than importing the compiled bundle, meaning some code is deduplicated. (Currently only works if the app is built with rollup-plugin-svelte, but the intention is to use it elsewhere.)
For non-Svelte apps, nothing changes — they still import the built file.

Awesome, thank you for this 👍


add ntpc.edu.tw

This is a domain to student, which is provided by education department of New Taipei City government (https://www.ntpc.edu.tw/index.jsp).
The single-sign-on page is https://sso.ntpc.edu.tw for all students to login their own email account.
Thank you.

@innocentliny I am sorry, our internal investigation shows that there is a high risk for this domain to be misused. Please use another option to apply. Please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).
I have to close the request without merging.


Some clarifications

Thanks for the library.
I don't understand the math behind HMC, I tried this implementation because its one of the easiest to use.
I have an energy function which can be any Real. And its gradient. For me, samples that have low energy should have higher probability of being sampled. So, I use your implementation with a function that returns e^(-energy) and gradient of e^(-energy).
Since I don't understand the math here, my first question is, does HMC require that the distribution be normalized (that is, sum of probabilities sum to 1)?
Additionally, I have been facing the following issue. Lets assume a fixed step size. I run the code exactly as in the Readme, but with my energy function and its gradient function.
If I sample N samples, and lets number it sample 1 to sample N, then I find that sample 1 is closer to what I expect, but sample N is much further from it. As I go from 1 to N, I find that I get more violating examples (in terms of my energy).
Given an initial sample, I expect that all N samples should be independent. But I actually find that sample "m" has higher energy (lower probability thus), than sample "m-1".
Is there a reason for this?

Hi,
It seems that from you closing this issue that you managed to resolve your problem?
The potential energy function should correspond to the negative logarithm of the (potentially unnormalised) density of the target distribution of interest (in the log domain any normalising constant will be an additive constant which will not affect the simulated Hamiltonian dynamics or density ratio in the acceptance probabilities).
Regards,
Matt


[TC-SOACONC-2.1] SDK has to be implemented for Sulfate Concentration Measurement Cluster

All-clusters-app has to be enabled with below mentioned Attributes :

[ ]  MeasuredValue
[ ]  MinMeasuredValue
[ ]  MaxMeasuredValue
[ ]  PeakMeasuredValue
[ ]  PeakMeasuredValueWindow
[ ]  AverageMeasuredValue
[ ]  AverageMeasuredValueWindow
[ ]  MeasurementUnit
[ ]  MeasurementMedium
[ ]  LevelValue

Bug prevalence
Whenever I try to read all these attributes
GitHub hash of the SDK that was being used
7ae5919a046a18c7e60537aff814977ee424b9cf - Master
Platform
raspi
Platform Version(s)
RPI - 4, 8GB RAM

Will be fixed by https://github.com/project-chip/connectedhomeip/issues/27662

Test case removed in https://github.com/CHIP-Specifications/chip-test-plans/pull/2995


Error on Location(s)

I followed this guide to configure the sample deployment,
https://alien4cloud.github.io/#/documentation/1.4.0/getting_started/new_getting_started.html
At step Setup and deploy your application, I got warnings triangle and the "Matching" tab is disabled.

I also noticed that Puccini simple orchestrator is disabled and cannot be re-enabled.

Hello,
glad to see you are looking inot Alien4cloud :)
The orchestrator enablement seems to have gone wrong. Can you check the logs file alien4cloud-getstarted/alien4cloud/logs/alien4cloud.log  and see if you can spot any error log?
I Just re-tried the procedure on my end, and I do not have any problem... Could you retry and let me know if you face the issue again?
Thx

Thanks you for reply. You are doing a great job. Instead investigating on the logs, I re-executed the script to see if there is something wrong.
You are right, the orchestrator is not enabled by the getting_started.sh script
Actually, I noticed that the following code doesn't set the proper orchestrator id. it outputs Created orchestrator with id None on terminal.
echo "Create puccini orchestrator"
ORCHESTRATORID=`curl "$ALIEN_URL/rest/latest/orchestrators" \
-b curlcookie.txt \
-XPOST \
-H 'Content-Type: application/json; charset=UTF-8' \
-H 'Accept: application/json, text/plain, */*' \
--data-binary '{"name":"Puccini simple orchestrator","pluginId":"alien4cloud-plugin-puccini","pluginBean":"puccini-orchestrator"}' | \
    python -c "import sys, json; print json.load(sys.stdin)['data']"`

echo "Created orchestrator with id $ORCHESTRATORID"

I have installed Python, I cannot figure out why.
This leads to the anormal exit of the script, though alien4cloud is successfully started on port 8088. Following are the final part of terminal output
starting alien4cloud
Waiting for alien4cloud to start
............Initializing getting started data
Create puccini orchestrator
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   193    0    79  100   114    607    876 --:--:-- --:--:-- --:--:--   883
Created orchestrator with id None
Update orchestrator configuration
Enable orchestrator (takes a few secs as it checks and configure puccini)
Create local docker location
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   135    0    82  100    53   1675   1083 --:--:-- --:--:-- --:--:--  1708
Created location None
Creating compute location resource
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   304    0    78  100   226   2019   5850 --:--:-- --:--:-- --:--:--  5947
Traceback (most recent call last):
  File "<string>", line 1, in <module>
TypeError: 'NoneType' object has no attribute '__getitem__'
Configure template

Ok that is weird! Just to be sure, do you have all that is mentioned in the Prerequisites section? Specially the Docker part?
Can you please attach the log file alien4cloud-getstarted/alien4cloud/logs/alien4cloud.log? I'll try to investigate on it.
Thx.

@igorng Thank you for your advice, the docker part. My problem is solved.
I removed completed the alien4cloud-getstarted repository, launch docker and re-execute the script. Ça roule ! ;)


Admin metrics

As an admin, we should have metrics about badges, users & other entities.

Please don't update HTML/CSS part of admin in this card before #107 is done.


Update Supported-Windows-guest-operating-systems-for-Hyper-V-on-Windo…

The header states that this is also for Windows Server 2022 - the body of the text does not reflect this.

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

This PR appears to be obsolesced by https://github.com/MicrosoftDocs/windowsserverdocs/commit/9493a670206dac9a0ea59eade31b2743db8e8828.

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


feat: nomad docker image remove

Avoid nomad to delete docker images after job stops

This PR returns error for previous changes in particular the force on the creation on the configuration files.
In particular this commit: https://github.com/ai4os/ai4-ansible/commit/99e23093057d1915042d930fffea55e96a2246cf

I understand that they are failing because once the tests have been executed, it is expected that successive executions will not cause changes, right?
Actually the principle of idempotence, as I understand it, is being fulfilled given that the content of the modified files is not changing between these executions, but since the ansible recap says that if there have been changes the tests assume that this principle has not been fulfilled. I'm wrong?

The problem is that some steps change the same file, particularly the client.hcl in both consul and nomad.
First, it creates the client.hcl using the template, then it changes it, replacing the consul token, so when you execute the playbook again, as the token has been replaced, the template creates again the client.hcl and the replace perform again the replacement, so it will always raise errors in idempotency.
But as the role need a first version of the client.hcl file to generate the tokens it is not easy to solve this issue.
Any ideas?

It was what I thought. The thing is that we are putting the template with a default token and then recovering the token and replacing it in the template. Why don't we recover the token before loading the configuration file? So ansible if it should be able to detect that the configuration file has not received new changes.

Could we create the acl token before creating the client.hcl file?

The thing is that we are putting the template with a default token and then recovering the token and replacing it in the template. Why don't we recover the token before loading the configuration file? So ansible if it should be able to detect that the configuration file has not received new changes.
Maybe I'm wrong, but as far as the tokens are received in the zip file that the ifca admin gives to the site admin, right? Why is it necessary to have a first version of the template to be able to recover the tokens? It is a process that until now I have not had to do manually, so I may be saying things that make no sense.

Maybe I'm wrong, but as far as the tokens are received in the zip file that the ifca admin gives to the site admin, right? Why is it necessary to have a first version of the template to be able to recover the tokens? It is a process that until now I have not had to do manually, so I may be saying things that make no sense.

The tokens are received in a zip in case of joining a cluster, but if you are creating a new one, you need to generate them.

The tasks that I see that are involving the tokens are those in the files copy_tokens.yml and set_tokens.yml. If I'm not overlooking anything it's simply a substitution. Can't it be done before?

Maybe I'm wrong, but as far as the tokens are received in the zip file that the ifca admin gives to the site admin, right? Why is it necessary to have a first version of the template to be able to recover the tokens? It is a process that until now I have not had to do manually, so I may be saying things that make no sense.

The tokens are received in a zip in case of joining a cluster, but if you are creating a new one, you need to generate them.

You are rigth

I have a possible solution, I will test it and create a new branch with the fix.

PR: created #26

Now tests seems to be failing due to something related with docker daemon.

Docker error (https://github.com/docker/docker-py/issues/3256) fixed.


FindByXPath gives error

According to the documentation:

FindByXPath finds exactly one element by XPath selector.

but when I do:
//*[contains(@class, "mdl-menu")]
test := AgoutiPage.FindByXPath(findText)

It tells me that: failed to select element from selection 'XPath: //*[contains(@class, "mdl-menu")] [single]': ambiguous find
I know it occurs more times in the html I'm trying to investigate

My first guess would be that there are other items on the page with the same class, since it is claiming an ambiguous find.

That is completely true, the page has more of the same class.
But as the documentation says: FindByXpath find one element, I think it is not working.
(classes can occur more than one in a HTML page, even id's can if the programmer was sloppy)
or.. perhaps we need a FindElementsByXpath, which could return slices.

All of the Find methods intentionally fail when more than one element that matches the selection is present. You can use AllByXPath or FirstByXPath if you don't need this behavior.

Hello,
Would it be possible refactor the function FindByClass in order to return a list of *Selection, in this way I would be able to select different element that got the same class name.
With the current approach I do not understand how can I interact with an element that has common class name and it is not the first in order of appearance (behaviour that I found very common in all the web app that I tested)
Thank you

Actually never mind I found I can achieve with the function AllByClass


Correct filter models

I moved every filter to a separate file, created filter condition classes. All filters have correct format now.
All filter properties with default values were wrapped in Nullable<T>.
I also wrote some test cases for filters. For this I changed  RestClient.defaultSerializerSettings access to protected, so I could use in tests serializer settings identical to settings in RestClient

@KoditkarVedant thank you for reviewing!
Yeah, moving conditions to be nested in filter class is a good idea, should've done that from the start. I will update my PR with this changes later today.

@asevos I've added editorconfig to the solution. you might wanna run the dotnet format.

@KoditkarVedant I moved conditions, formatted my files, added constructors to filters (as discussed in my constructor proposal ) and added to readme a section on how to query a db


Add support for system prompt on vertex

Status
I've just made the code and some tests, in next days I'll start to work on bellow checklist. If you want this feature you can use  my branch
PR Checklist

[ ] Read the Contributing documentation.
[ ] Read the Code of conduct documentation.
[ ] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[ ] Describes the source of new concepts.
[ ] References existing implementations as appropriate.
[ ] Contains test coverage for new functions.
[ ] Passes all golangci-lint checks.

@ImPedro29 is this ready for review? please mark as non-draft if so.

Closing to follow repository rules. Fork bad used. I'll send a new PR soon


Enhancement make UT-SelectRollupTest more stable

Proposed changes
Issue Number: #12964
Root cause:
        there is still a short gap between "job finish" and "table become normal",
        so if user send next alter job right after the "job finish",
        it may encounter "table's state not NORMAL" error.

Describe your changes.
Add table state check
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Merge it


Make max_iterations_per_try dynamic

First I tried installing with Python3.11 installed and got errors from numpy. Rolled back to 3.10 and installation worked, but attempting a poetry update failed (again due to numpy). Another roll back to Python3.9 and both installation and poetry update went smoothly. Didn't investigate the versioning beyond that, but our packages were clearly incompatible with Python3.11 and 3.10


Change 'logical' backup type to 'text' backup type

@DimitriosLisenko @lvangool @shahriarb @babakgh This should be changed to Text as default to avoid change in the API. Please fix.


paste does not work in firefox

CTRL + V is not working in firefox (42.0)
any work around for this?
works fine in chrome.

Since version 10.1.9 add ngf-enable-firefox-paste="true" to enable paste in firefox.
The trick is to make the input as contenteditable but disable editing the content and just capturing the paste events.

Still does not work.

Upload

Am I doing something wrong?

If you copy your code into demo page it works on Firefox for me.

It works when you copy an image from the web but if you are copying from the snipping tool for example, does not work.

Pasting images in Firefox is broken for me as well.  I'm installing ngf-drop on the parent element of a textarea, which also kills pasting text (due to the preventDefault on paste events that ng-file-upload implements in Firefox).  When I add ngf-enable-firefox-paste="true", pasting images still doesn't work in Firefox because the focus remains in the textarea, but on top of that all keyboard events are eaten so I can't even type into the field!
I've worked around this by patching preventDefault on paste events, disabling paste handling in ngf-file-upload, and using https://github.com/layerssss/paste.js instead, which works pretty well (though it requires some extra contortions to make text pasting work in Firefox, and undo/redo of pastes gets wonky).  Perhaps you can borrow some ideas from that project and integrate them into ng-file-upload?

@AdamRollinson
Pasting from file system does not work in any browser. The paste feature is for in browser paste, so you can copy the image from any tab in the browser and paste it in the area, but not from file system.

@pkaminski Could you create a jsfiddle and explain what is not working?
You could have the image paste on a separate div so it doesn't interfere with the textarea.

@danialfarid
it works on chrome from filesystem.

@AdamRollinson what operating system?

@danialfarid
Windows 10

@danialfarid Check out http://plnkr.co/edit/i41Yx5MtRFRZFVVmMaSz?p=preview.  It works fine in Chrome, but on Firefox pasting of both text and images fails.  If you add ngf-enable-firefox-paste="true" then typing stops working as well.
Perhaps I misunderstand something, but doesn't the ngf-drop need to be on an ancestor of the textarea?  Otherwise, how would it receive the paste event?  Maybe you could include an example of how to use ngf-drop on a textarea in the project's README.

@pkaminski the example doesn't ready work in Chrome:
http://plnkr.co/edit/wUt6wVkqfHSbl9xOOAfY?p=preview
It doesn't assign the pasted file to the model.

@danialfarid My original example does work in Chrome.  I don't care about a model -- I just use the callback directly, passing in $file.  Also, your version is actually broken in Firefox in the same way: you can't paste either text or images in the textarea.

@pkaminski For now I would suggest use separate divs for text area and image paste. Text area is for text and doesn't make much sense to paste image in it. So you could have the image paste div separate from textarea and it should work in both.

Text area is for text and doesn't make much sense to paste image in it.

@danialfarid On the contrary, I think that's a very common context for pasting images.  You have to look no further than the GitHub comment area you're using right now for an example...  Anyway, if ng-file-upload isn't meant to support file uploading through a text area, you might at least want to note it prominently in the README.  But as I mentioned, it's really not far off, if you could just adopt some code from https://github.com/layerssss/paste.js.

Correct, but pasting images on github is only working in Chrome not Firefox.
I suggested that approach as temporary solution. I will have a look to see if it is fixable later on but it is kind of a low priority considering it is just a hack for one browser.
If you are willing to contribute PRs are welcome.

Correct, but pasting images on github is only working in Chrome not Firefox.

True, but 1) it can be made to work in Firefox too (see my product, https://reviewable.io), though there are some trade-offs, and 2) even if you decide to only support it in Chrome it should at least not break Firefox textareas altogether.
I don't really have the time to dig into ng-file-upload code to make a clean patch (sorry!) but the key insight from paste.js in non-Chrome browsers is simply to intercept {ctrl,cmd}-v and switch the focus to an off-screen (and out-of-hierarchy) contenteditable to get a paste event, then switch focus back to the text box afterwards.  Then you don't need any hacks to make the contenteditable not actually editable and everything is happy.

Fix firefox paste text in textarea inside ngf-drop.

firefox paste text in textarea inside ngf-drop is fixed at version 10.1.11

ngf-enable-firefox-paste would be experimental for now until I find a way to integrate it better probably by using paste.js.

I just want to let you know that I'm testing the latest version of your plugin and it is not working in FF

@xavier-figueroa what is not working?

in https://angular-file-upload.appspot.com/ the paste functionality only works in chrome. I integrated it in my application with the same results. I installed it using bower

yes that's what this issue is about. You can enable it by https://github.com/danialfarid/ng-file-upload/issues/1183#issuecomment-177043394 but experimental


add time-slice filter

Similar to event / column / row filter, support a filter that excludes all events outside a given time-range. Having this built-in would allow it to be used with tdb_cons_append for efficient materialized views.

Any plans on this one? I have some use cases where I'd like to calculate metrics on many (possibly millions) trails, but over a small time frame. Having a time filter builtin would come in handy.

yes, we should add this soon

I am not sure how you guys think this feature might best be implemented, but here's a thought:
If the filter functionality was extended to include additional operators, <, >, <=, and so on, the time slice functionality could leverage the existing filter framework and you could filter a time slice by using the time field from a trail.
I think this approach would leverage the new index functionality, but I am not entirely sure on that point.
If would tell tdb_event_filter_add_term the type of operator used, it currently accepts '==' and '!=' then I think this might let us just filter times (and other field values) using simple LT GT operators.
Just a thought.

Working on a an example of this using a bit mask for the filter operator.

Interesting proposal! There are pros and cons with this approach:
Pro:

Reuse existing APIs
Can support multiple time slices (kind of)
Can be combined with other filters

Con:

LT and GT operators wouldn't work with other fields, which may be confusing
Expressing time slices in Conjunctive Normal Form isn't very natural. For instance, consider the following sequence of timestamps with two slices:
1 3 [5 6 7] 8 10 20 [25 26 30] 40 41. You can't really use LT and GT in this case, since this doesn't work (time > 5 OR time > 20) AND (time < 8 OR time < 31) but you need to expand slices to large unions which is not very efficient.

To address the last issue, maybe there could be tdb_event_filter_add_time_slice which you could use to define a slice as one "term" as a part of a normal CNF query. The above example would look something like:
tdb_event_filter_add_time_slice(filter, 5, 8)
tdb_event_filter_add_time_slice(filter, 20, 31)

Probably there needs to be another parameter to control whether the slice is inclusive:exclusive etc.
Just thinking aloud. I like the idea of being able to combine this with the existing event filter functionality.

@tuulos Agreed, there would be several pros and cons with treating time slices the same as other Trail fields and reusing the filter API for both.
To your points, Con #1 occurred to me, where a field contains a normal string value, say derp=foo, you are correct that LT / GT is not much help here (or any other place where we are comparing a TDB_ITEM that is the representation of a string value. Even in cases where the item is a string representation of an integer value, say speed=200 the 200 part might not be easily compared with LT / GT operators without some type casting. (ASIDE Is there anything preventing me from storing another integer type in an event field? Say speed? /ASIDE)
I think that it would be very convenient and natural to use time intervals along side other field comparisons, even if the other field comparisons are limited to equal / not equal (PRO # 3). However I don't want the time slice functionality to suffer needlessly from performance hits because it was easy to roll this into the filter API.
Anyway, really excited about this feature :)

All fields are currently simple byte-strings with no typing / semantics. You can store any data in the fields (a 64-bit int takes an 8 byte string etc) but TrailDB doesn't know that the bytes correspond to an int, so it doesn't treat them specially.
One idea that has been floating around is to eventually add support for continuous values (integers) that can be compressed efficiently. With fields like this, GT / LT operators would make more sense too.
Maybe tdb_event_filter_add_time_slice() is the way to go.

Implemented by PR #114


fixes 895

Fixes #895
Type of change
Please delete options that are not relevant.

[ ] 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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Kubernetes Version:
Helm Version:
Hardware:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked that my issue complies with the Contributing guidelines

@JeremyTheocharis guess what's stuck at docker hub again ?
Ill prob check alternatives for building there on friday.


Fix: TypeError - 'Link' object is not iterable

In the ebooklib code below, book.toc may not iterable and may be a Link class.
https://github.com/aerkalov/ebooklib/blob/1cb3d2c251f82c4702c2aff0ed7aea375babf251/ebooklib/epub.py#L1567-L1588
In this case, it throws "TypeError: 'Link' object is not iterable" exception,
so fixed to convert to a list if it is not iterable.
About 1% of epub files got this error.

@mwbyeon We realized that ebooklib used a license that is incompatible with our Apache 2.0 license and had to remove it from our dependencies. Does that make this PR moot?

@qued  If you're not using ebooklib, this PR is meaningless, so let's close it.


Processing steps

Collect data from preprocessing derivatives and raw dataset.

Raw phase data from raw dataset.
Preprocessed magnitude data from preprocessing derivatives.
Transforms from preprocessing derivatives.

Rescale from arbitrary units to radians.
Remove non-steady-state volumes?
Unwrap phase data with ROMEO/MEDIC.

Combine unwrapped phase data across echoes with MEDIC?

Calculate phase jump and phase jolt time series with LayNii's LN2_PHASE_JOLT.
Extract confounds (if any can be extracted).
Apply transforms from fMRIPrep to get data into the appropriate spaces.

Probably need to ignore STC.
In idevicente/Phase-based_denoising_pipeline, they combine mag + phase into complex, then split into real + imag, then apply the transforms, then combine into complex, then split into mag + phase. Not sure if that's necessary.

Related to https://github.com/nipreps/fmriprep/issues/3336.

Useful links:

https://neurostars.org/t/processing-for-complex-valued-fmri-data/5093
https://github.com/tsalo/complex-flow (a past attempt at something similar)


⚠ Admissions Portal has degraded performance

In 7983f3f, Admissions Portal (https://admissions.cranleigh.org) experienced degraded performance:

HTTP code: 200
Response time: 11621 ms

Resolved: Admissions Portal performance has improved in a2fd275.


Doubly LinkedList Implementation

In ds implementations there can also be a dooublyLinkedList implementation which i want to provide in below mentioned pull request, Please accept it if you like it.
Peace :)

#122  this is the pull request with Full implementation of doublyLinkedList in CPP.
Please accept if You Like It :)
Regards


Document the new classPathFromResources with bundled jars

As reported on Slack there was a bit of confusion on how (and when) best to use the new classpathFromResources to setup a recipe test classpath. So far the best documentation to that is in private Slack threads one and two.

Another useful thread to add to this list: thread.

Want to double-check my understanding of classPathFromResources. @sambsnyd @timtebeek - would one of you mind double-checking that the below information is correct? I can then clean this up and make a doc for it.
The problem
If a recipe needs more than two versions of a library, it is can be confusing on how to set that up so tests can run correctly. This is common with migration recipes that span multiple versions.
Let's consider a recipe that migrates something from Java 11 to 15, but along the way breaks it up into Java 11 -> 12 and Java 12 -> 13 and so on. The recipe itself will produce code that is Java 15 code. The tests, however, might expect code to be Java 11 or 12 or 13 or 14, though. If you specify multiple dependencies on different versions in your build files, only the top one will be used.
This can cause people to struggle to get the appropriate libraries configured for specific tests. Please note, though, that this is only an issue when the tests need many different versions in them. If the tests only need one version, then using classPathFromResources doesn't have much use (?)
The solution
The rewrite recipe library now offers some new functionality to assist with this. It will look in the META-INF folder for dependencies. Many different versions of the same dependency can be included in there. Your code can then specify exact versions to run for a particular test.
For instance, if you wanted to specify that one test should use assertj-core-3.24.2, you could add this to your spec:
spec
    .parser(JavaParser.fromJavaVersion()
        .classpathFromResources("assertj-core-3.24.2")

This would look in the META-INF folder for an assertj-core-3.24.2 directory and use it accordingly.
To further help make developer's lives easier, new functionality has been added to build files. In build.gradle files, you can add a recipeDependencies section and specify specific versions in there. This will then be automatically pulled in via the rewrite recipe library to the META-INF folder for you.
In the build.gradle file this would look like:
recipeDependencies {
    parserClasspath("org.assertj:assertj-core:3.24.2")
}

I would maybe not use something from Java 11 to 15 as an example, since we're not really putting different versions of Java on the classpath here through that mechanism. I think an illustrative use case here is where you have versions 2, 3 & 4 of a library, such as Mockito (maybe not exactly Mockito, but at least a library), and then a deprecation in version 2, removal in version 3, and new additions in 4. Modules that want recipes both for the 2 -> 3 removal and 3 -> 4 additions would otherwise not be able to have the corresponding classes on the classpath without this mechanism. I hope that's clear~ish.

Makes sense! Will keep that in mind when I write the full doc.

Open questions that I'll need to figure out / get answers for before I can finish the doc:

What is the rewrite recipe library? Is that a new thing? Where would someone go to find out about it? Does it serve any point other than adding recipeDependencies and parseClasspath functionality to build files?
How do you configure the recipe library for use in your project? Is it as simple as applying the plugin in your build.gradle file and then using JavaParser.Builder to load the resources? Or is there anything else that needs to be done?
Can Maven projects also use the recipe library? If so, do we have any examples of what that looks like?
Does this special configuration of adding multiple versions of the same library serve a point for projects that do not need 3 or more versions of the same library?

The rewrite recipe library is built from this repository and published to the gradle plugin portal. It is not at all required for recipe development, but it collects our conventions for building a recipe module. And provides the recipeDependencies and parseClasspath functionality.
After applying the plugin, users can manually run downloadRecipeDependencies which will download the parserClasspath jars into the src/main/resources/META-INF.rewrite. Jars can also be placed here manually, using the task or our build plugin are not necessary, it just facilitates populating this folder
No. If anyone is using Maven they can put the jars they want into that folder directly.
Not required for projects that don't need to interact with multiple versions of a library.


Automate extension packaging/publishing

At least for Chrome: https://developer.chrome.com/webstore/using_webstore_api

I'm guessing an easier plan would be to use mozilla/web-ext to create the .zip file and upload it to the Chrome store with DrewML/chrome-webstore-upload-cli. Uploading to AMO will probably remain manual for now, pending mozilla/web-ext#804.


let broken

{
  let one;
  let two;
  two;
}

Transpiles to:
{
  var one = undefined;
  var _two = undefined;
  two;
}

Fixed in 3.0.4

@sebmck I'm running 3.0.5.

@lukescott Are you sure? I can't replicate this locally or on the REPL. Does running $ 6to5 -V output 3.0.5?

@sebmck The simple test case works, but I'm still having an issue. Here's actual code:
	function load(object = {}) {
		for (let key in object) {
			if (object.hasOwnProperty(key)) {
				let value = object[key];
				let child = this.find(key)[0];
				if (!child) {
					continue;
				}
				if (hasTrait(child, Input)) {
					child.load(value);
				}
			}
		}
	}

The child var is named _child.

@lukescott Ah. I'll look into it now.

@lukescott Tracked it down. I'll push out a new patch shortly.

@lukescott Fixed as of 3.0.6. Sorry about that!

@sebmck That did it, thanks! All my tests are passing now :+1:


Modal Overrides

After some testing I noticed that if we removed a large amount of the modal styling, it makes no difference to the components styling.
For example;

add modal--large to a modal component
remove the following styles from lib/Modal/styles.scss;

left: 0;
right: 0;
width: auto;
margin: auto;
background-color: transparent;
box-shadow: none;
border: none;

testing in gather-content-ui removing these don't change the outlook of the component
this is the same for a lot of the styling for the Modal component

Further investigation points that we're overriding the styles of bootstrap with the exact same values which is confusing. If we want to remove Bootstrap from our deps then lets do so but at the moment its very difficult to see what Bootstrap is meant to handle and what our styles are. Especially for the Modal component.
I recommend we clean out everything that isn't required.

This may also be the case for most of overrides/_modal.scss which was abstracted from the modal component stylesheet in #41 .

Fixed in #106


Updating tenant switch faq

This PR intends to update the tenant switch FAQ to make it easier for the client to access relevant FAQ if they run into issues resolving users after a tenant switch

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

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

#sign-off


Tag a release?

Could you tag a new release for the laravel 5.5 support?
Thanks

Latest release (5.5.0) has support for laravel 5.5. Check this travis build https://travis-ci.org/melihovv/laravel-log-viewer/builds/270061293

Oh, weird. https://github.com/melihovv/laravel-log-viewer/releases shows 1.0.0 from January as the latest release, but I see it on the tags tab. Thanks


Unable to install \pyaedt.[tests] with python 3.8

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
Unable to install \pyaedt.[tests] with python 3.8 with the error shown below

ERROR: Ignored the following versions that require a different python version: 1.11.0 Requires-Python <3.13,>=3.9; 1.11.0rc1 Requires-Python <3.13,>=3.9; 1.11.0rc2 Requires-Python <3.13,>=3.9; 1.11.1 Requires-Python <3.13,>=3.9; 1.11.2 Requires-Python <3.13,>=3.9; 1.11.3 Requires-Python <3.13,>=3.9; 1.11.4 Requires-Python >=3.9; 1.12.0 Requires-Python >=3.9; 1.12.0rc1 Requires-Python >=3.9; 1.12.0rc2 Requires-Python >=3.9; 1.2.3 Requires-Python >=3.9; 1.25.0 Requires-Python >=3.9; 1.25.0rc1 Requires-Python >=3.9; 1.25.1 Requires-Python >=3.9; 1.25.2 Requires-Python >=3.9; 1.26.0 Requires-Python <3.13,>=3.9; 1.26.0b1 Requires-Python <3.13,>=3.9; 1.26.0rc1 Requires-Python <3.13,>=3.9; 1.26.1 Requires-Python <3.13,>=3.9; 1.26.2 Requires-Python >=3.9; 1.26.3 Requires-Python >=3.9; 1.3.0 Requires-Python >=3.9; 1.3.1 Requires-Python >=3.9; 1.4.0 Requires-Python >=3.9; 1.4.0rc1 Requires-Python >=3.9; 2.1.0 Requires-Python >=3.9; 2.1.0rc0 Requires-Python >=3.9; 2.1.1 Requires-Python >=3.9; 2.1.2 Requires-Python >=3.9; 2.1.3 Requires-Python >=3.9; 2.1.4 Requires-Python >=3.9; 2.2.0 Requires-Python >=3.9; 2.2.0rc0 Requires-Python >=3.9; 3.8.0 Requires-Python >=3.9; 3.8.0rc1 Requires-Python >=3.9; 3.8.1 Requires-Python >=3.9; 3.8.2 Requires-Python >=3.9; 8.13.1 Requires-Python >=3.9; 8.13.2 Requires-Python >=3.9; 8.14.0 Requires-Python >=3.9; 8.15.0 Requires-Python >=3.9; 8.16.0 Requires-Python >=3.9; 8.16.1 Requires-Python >=3.9; 8.17.0 Requires-Python >=3.9; 8.17.1 Requires-Python >=3.9; 8.17.2 Requires-Python >=3.9; 8.18.0 Requires-Python >=3.9; 8.18.1 Requires-Python >=3.9; 8.19.0 Requires-Python >=3.10; 8.20.0 Requires-Python >=3.10
ERROR: Could not find a version that satisfies the requirement scipy==1.11.4; extra == "tests" (from pyaedt[tests]) (from versions: 0.8.0, 0.9.0, 0.10.0, 0.10.1, 0.11.0, 0.12.0, 0.12.1, 0.13.0, 0.13.1, 0.13.2, 0.13.3, 0.14.0, 0.14.1, 0.15.0, 0.15.1, 0.16.0, 0.16.1, 0.17.0, 0.17.1, 0.18.0, 0.18.1, 0.19.0, 0.19.1, 1.0.0, 1.0.1, 1.1.0, 1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.3.0, 1.3.1, 1.3.2, 1.3.3, 1.4.0, 1.4.1, 1.5.0, 1.5.1, 1.5.2, 1.5.3, 1.5.4, 1.6.0, 1.6.1, 1.6.2, 1.6.3, 1.7.0, 1.7.1, 1.7.2, 1.7.3, 1.8.0rc1, 1.8.0rc2, 1.8.0rc3, 1.8.0rc4, 1.8.0, 1.8.1, 1.9.0rc1, 1.9.0rc2, 1.9.0rc3, 1.9.0, 1.9.1, 1.9.2, 1.9.3, 1.10.0rc1, 1.10.0rc2, 1.10.0, 1.10.1)
ERROR: No matching distribution found for scipy==1.11.4; extra == "tests"
Steps To Reproduce
With python 3.8 installed, clone pyaedt (says at C:\pyaedt ) and in command line windows type:

py -3.8 -m venv C:\venv_3.8
call C:\venv_3.8\Scripts\activate
python -m pip install pip -U
python -m pip install wheel setuptools -U
python -m pip install C:\pyaedt.[tests]

Which Operating System are you using?
Windows
Which Python version are you using?
3.8
Installed packages
attrs==23.2.0
colorama==0.4.6
exceptiongroup==1.2.0
importlib-resources==6.1.1
iniconfig==2.0.0
packaging==23.2
pluggy==1.4.0
pytest==7.4.4
pytest-azurepipelines==1.0.5
pytest-nunit==1.0.4
tomli==2.0.1
zipp==3.17.0

Hi @anspraksaph, could you please type:
pip install .[tests]
from pyaedt repo directory?

Hi @anspraksaph, thanks for notifying us of such an issue.
Seems like a side effect of #4118, I'll try to fix that today.

I did a little digging and it seems that it's not the first time that such an issue happened (cf #3532).
I'll try to update the CI to perform smoke testing on multiple python versions.

@SMoraisAnsys If not already the case, please use matrix for Python version at least for the smoke tests.


Make ConfigurableFileCollection.from() honor conventions

Make from() honor conventions similarly to HasMultipleValues.append() and MapProperty.insert().

Superceded by #28189


Right Click Token Icons unreadable

Right click Token options unreadable

Hi, could you please tell me which other modules you are running? It seems like they are completely invisible rather than their normal white. Thank you!


Feature/platform support

[Current coverage][cc-pull] is 91.55%

Merging [#63][cc-pull] into [master][cc-base-branch] will increase coverage by 1.15%

@@             master        #63   diff @@
==========================================
  Files            17         15     -2
  Lines          1845       1848     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1668       1692    +24
+ Misses          177        156    -21
  Partials          0          0

Powered by Codecov. Last updated by [659735b...3d935d5][cc-compare]
[cc-base-branch]: https://codecov.io/gh/devlucky/Kakapo/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/devlucky/Kakapo/compare/659735bae4417bf2bfa62d0c23526f97d5eeafd3...3d935d5c443fe28c0eccd92f8cb006e35ca5aa42
[cc-pull]: https://codecov.io/gh/devlucky/Kakapo/pull/63?src=pr

250 files changed :no_mouth: What am I supposed to review here?

Can't do anything about it since it's all about changing the project structure, code isn't changed. I will merge


Fix code scanning alert - subprocess call - check for execution of untrusted input.

Problem at main python script.

Not gonna fix it.


Observatory Mozilla on pending when results aren't fetched in a while

Are you experiencing an issue with...
shields.io
🐞 Description
When visiting my readme page i see that the observatory is pending:

The weird thing is, that when i was writing this issue, it didn't work (tried it with a different one aswell). But when inserting it in this issue, it seems to fetch the data again? You need to wait about a day or 3-4 until it goes back to pending.
🔗 Link to the badge

💡 Possible Solution
Only when i click on the button/visit the observatory URL i see the score after refreshing the readme page.

One thing to be aware of is that this badge gets served with a cache-control: max-age=120, s-maxage=120 header so our downstream cache and your browser will cache it for 2 mins so if you've viewed the badge while the job is pending and then it completes it will take a couple of mins for the badge to update. Does that explain what you're seeing?

One thing to be aware of is that this badge gets served with a cache-control: max-age=120, s-maxage=120 header so our downstream cache and your browser will cache it for 2 mins so if you've viewed the badge while the job is pending and then it completes it will take a couple of mins for the badge to update. Does that explain what you're seeing?

Hmm, I’m pretty sure I’ve checked the readme page/badge with an interval longer the 2 minutes
but, isn’t it strange that the badge returns to pending after a few days? No matter the cache?

Right. I now see what you mean by "results aren't fetched in a while". This is basically a characteristic of the upstream service: https://github.com/mozilla/http-observatory/blob/master/httpobs/docs/api.md#invoke-assessment It'll cache your result for 24 hours. If it doesn't have a cached result then it will initiate a new scan. I wonder if it always returns a pending result immediately, or only if the scan is taking too long to return an immediate response..

Right. I now see what you mean by "results aren't fetched in a while". This is basically a characteristic of the upstream service: https://github.com/mozilla/http-observatory/blob/master/httpobs/docs/api.md#invoke-assessment It'll cache your result for 24 hours. If it doesn't have a cached result then it will initiate a new scan. I wonder if it always returns a pending result immediately, or only if the scan is taking too long to return an immediate response..

Hm, maybe. I'm sure i've included the force part correctly.
But that could perhaps be the reason, but it somehow doesn't line up all, perhaps it's best to check in a couple of days here, hoping nobody has clicked on the badge meanwhile, so that either one of us can see it being on pending, and checking it a few times with 2 mins interval.

I retract my statement!
I checked this page of mine: https://github.com/TheVoxSiren/voxsiren.net
And while i checked it, was on pending, and ivé checked about  ~15mins later, and it sure enough was showing the results!
I assume we can rather ignore the fact that'll likely need a minute or 5 after 24h to show the results :-)
Closing it now, but feel free to comment if People think otherwise!


Catalog-of-Things PR

In this project, We accomplished the following tasks:

Insert and query data in SQL.
Use primary key & foreign key mechanisms for joining tables.
Understand the different types of relationships between tables.
Query multiple tables.
Prepare complex queries that answer analytical questions.
Run a program using the command line.
Use Ruby syntax for basic programming operations.
Apply Ruby best practices and language style guides in code.
Describe the SOLID principles of OOP.
Implement classes and objects in Ruby.
Implement encapsulation and inheritance with Ruby.
Set up associations between classes and objects.
Store data in JSON files.
Build interactive console apps.

@Javed-Shahbaz , @Sparktan24 and @abiola-arobieke Kindly, would you add the video presentation


Travis on OSX

It seems this is becoming more of a reality, so let's see if we can actually test this repo on OSX.

I guess not yet :(

send a quick email with the repo info to<EMAIL_ADDRESS>and ask for osx.
we really should reduce the matrix in the yaml file before, since it will double the build matrix.
anyway they'll hook it up, they even gave it to me before on random personal open source repos.

Glad to hear it.
But indeed, let's reduce the size of it first. Didn't we have a PR dealing with that somewhere? :)

ya it was a while back, basically my idea was to preserve the amount of work but instead of spreading it out across a matrix just build 3 versions of libzmq and keep reinstalling node/iojs versions on top of that to run the test. the gist of it was something like:
# ./configure && make && whatever for libzmq
$ nvm install 0.08
$ make
$ make test
$ make clean # rm -rf build node_modules
$ nvm install 0.10
$ make # etc. you get the point...

that way we get our same test coverage but it drops the matrix size by a factor of node versions we test


UTR fixes

Update for 3' UTR sequence calculation
Check for condition where there is no CDS linked to an mRNA

[Current coverage][cc-pull] is 80.31%

Merging [#201][cc-pull] into [develop][cc-base-branch] will decrease coverage by -0.00%

@@            develop       #201   diff @@
==========================================
  Files            24         24
  Lines          3484       3489     +5
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2799       2802     +3
- Misses          685        687     +2
  Partials          0          0

Powered by Codecov. Last updated by 09b2729
[cc-base-branch]: https://codecov.io/gh/kbase/data_api/branch/develop?src=pr
[cc-pull]: https://codecov.io/gh/kbase/data_api/pull/201?src=pr

Addressed Dan's valid point about needing a comment for filtering out missing CDS ids, in the case where an mRNA does not have an associated CDS.


write config for sunset addon

write a config (json) for sunset

Done. Cant remember in which merge / commit


How to trigger external workflow, or make an activities for external workflow (defined in different SDK)

Hello,
I am trying to writing a Ruby Temporal activity that works with an (external) workflow defined in a different micro-service (writing Typescript SDK). When a Temporal server tries to pick up the activity from same queue, I have a issue "Workflow is not registered with this worker"
It seems that an activity forced to be defined inside Ruby's Workflow, which we not want. What is the appropriate way to do it?
Also if I want to trigger any external workflow, which the current micro-service does not hold ownership for that workflow, what is the appropriate way to trigger it.
Compared to temporalio/ruby-sdk, it claims support "Starting a workflow (defined in any other SDK)", e.g.
# Start a workflow
handle = client.start_workflow('MyWorkflow', 'some input', id: 'my-id', task_queue: 'my-task-queue')

How we can achieve the same with this SDK?
Thanks in advance.

Hi, sorry for the delay here.
For your second question, I don't think we have an example in the repo but you can specify the name of a workflow to start by setting the name property on options. Link

thanks for your clear response.


When will this be usable on the Blue?

Currently also not availabe from the Ledger Manager for the Nano S with 1.4.1 Firmware.

It's been... nearly a year now. We should likely assume at this point it's never coming :(

@TheSpiritof69 Holy crap, you're right. I saw the agent and I assumed it was there. Well now I'm just double fucked.

It's available.

No, it is not.

My bad.. On the Nano it is.

@TheSpiritof69 I'm very happy it's back though, I'd just really like to use my blue too.
Especially since it gets less attention for firmware updates, therefore less chance for mia gpg I can only hope, eh?

For now, I ve no ETA for Blue.

@cslashm

Does that mean "never" or "someday"?
What roadblocks stand in the way? Is it that hard to port it? I thought these archs were rather similar, but you would know best of course!

Seconding the last question - what still needs to happen before this can be available for Blue? I bought one expecting it to be a flagship product, but it turns out it's a 5th-class citizen.  I'm more than a little disappointed, but I'm also an embedded software developer by trade, so if I can help I'd like to know how.

Agreed Emily. A pointer as to what is missing would be more than enough.
Now it's been well over a year since then, so things may have improved by
now, but the last time I tried the dev environment it wasn't in a usable
share for the blue..
On Fri, Sep 14, 2018, 8:00 PM emilyofearth<EMAIL_ADDRESS>wrote:

Seconding the last question - what still needs to happen before this can
be available for Blue? I bought one expecting it to be a flagship product,
but it turns out it's a 5th-class citizen. I'm more than a little
disappointed, but I'm also an embedded software developer by trade, so if I
can help I'd like to know how.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LedgerHQ/ledger-app-openpgp-card/issues/17#issuecomment-421525945,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQGIibFHyRORxjIHwAEIjXxqkBlbpc0ks5ubG1RgaJpZM4RL8ZM
.

--
Ty,
Trevor

@cslashm hi since this and #23 were both closed, does this mean it's available on the blue?


fix: fix the bug in restore_test

What problem does this PR solve?
The restore_test will fail when we run function test now. This pr is used for fix this bug.
What is changed and how it works?
In our implement, backup interval must be greatger than cold_backup_checkpoint_reserve_minutes, which is 10 minutes in default.
And in restore_test, the backup interval is 1 seconds, while cold_backup_checkpoint_reserve_minutes is 10 minutes. So it will produces some error in running restore_test.
Check List
Related changes

Need to cherry-pick to the release branch
yes
Need to update the documentation
Need to be included in the release note

If cold_backup_checkpoint_reserve_minutes will only affect restore function test, I suggest update it dynamically like config cold backup interval.

If cold_backup_checkpoint_reserve_minutes will only affect restore function test, I suggest update it dynamically like config cold backup interval for restore function test.

Done.