Allow enabling replication manually

Fixes https://github.com/projectharmonia/bevy_replicon/issues/310.
I've tested this with the bevy_replicon_renet test app, and made appropriate changes to preludes and imports so that the renet transport code doesn't have to be modified at all to work properly.
A ton of the diffs are just connected_client(s) -> replicated_client(s).
I wrote a test for this as well, but I don't know if the module I put it in is correct.

I separated the rename from this PR into #319. Let's merge it first, will make the review process much easier.

Good catch!


KNLMeansCL: fatal error!

StaxRip x64 v<IP_ADDRESS>
Log-KNLMeansCL.txt

it works here, system problem?
http://forum.doom9.org/showthread.php?t=171379

Not sure, the same system runs v<IP_ADDRESS>_beta without problems.  There was an ongoing v<IP_ADDRESS>_beta encode going on when I was testing v<IP_ADDRESS> with KNLMeansCL, not sure if that made a difference.

Ok, I think I found the problem.  I have a GTX480 and that has OpenCL 1.1.  KNLMeansCL dropped support for that and only supports OpenCL 1.2 starting with their 0.7 releases, which is what you upgraded to in v<IP_ADDRESS> vs v<IP_ADDRESS>_beta.

Ok, I downgraded the KNLMeansCL dll in my v<IP_ADDRESS> copy to v0.6.11 and it works again.


Recalculating computed field on time logging

I am using the following formula for a custom field to calculate a percentage of the actual spent time on an issue to its estimated time:

self.total_estimated_hours ? ( 100 * self.total_spent_hours / self.total_estimated_hours ).round(2) : -1

I noticed that the computed field is recalculated only on the issue updates, but the field is not recalculated  when time logs are added; which defeats the whole purpose of the field itself as it contains invalid values.

Fixed. Please update plugin and try again.

Thank you for the fast response. Works good.
One thing still missing is when time-entries are deleted (from /issues/[id]/time_entries)

I'm glad to help you. Fixed. Try again.

Works great.
Thank you and keep up the good work.

having exactly the same issue.
After deleting the time spent entry the value is not getting updated


endsWith and startsWith fail on null valued columns

The code for these doesn't check if the value of the column is null before running leading to an error:
TypeError: Cannot read properties of null (reading 'indexOf')
at matchItem (/node_modules/prisma-mock/lib/index.js:708:33)

Fixed in v0.10.0


Fixing the absence of mentors for sponsors by making it actually use …

…the data sent in prepareforsegue. Sponsors data no longer stored in model. If we ever need this elsewhere in the app, this will need to change.

Does this include the code for sending the mentors in prepareforsegue? Or was that in the initial pr?

Yeah that was already in. It was just being overidden by null when it tried to do a load from the Model data

Awesome, thats what I figured.

Okay the CI is apparently broken I'll figure that out later. Just gonna push the pr through for now

Broken how? I can't see anything showing that it is

There's just a build that isn't running for some reason, nothing from your code.


WinSDK: extend the WinSock module further

This enables the use of TransmitFile more fully by adding the
Microsoft extensions to the WinSock module.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

@swift-ci please test Windows platform

The JIT regression on Windows is known and is not caused by this PR


Native component for 'BlurView' does not exist

import { BlurView }  from 'react-native-blur';
render() {
  return (
        <BlurView blurType="light" blurAmount={50} style={styles.position}/>
   )
}

Got the same error with react-native version 0.52

Solution in #249; add the libRNBlur.a manually to Build Phases

Thanks @ashleydw !

Thanks @ashleydw !


Adding steps to allow Warp connector to work as a private net offramp

Found during testing the current steps work for S2S connectivity, but if you try to access the private net behind the warp connector from a remote device connected to Warp, it fails due to traffic making it to the private net resources with a source IP in the <IP_ADDRESS>/12 subnet, leading to the return route not being valid since the local subnet isn't aware of how to route to this  <IP_ADDRESS>/12 subnet. Adding the command "sudo iptables -t nat -A POSTROUTING -o eth0 -j MASQUERADE" resolves this by changing the source IP of warp client traffic to the LAN IP of the warp connector, which private resources are able to locate.
Summary

Screenshots (optional)

Documentation checklist

[ ] The documentation style guide has been adhered to.
[ ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
[ ] Files which have changed name or location have been allocated redirects.

@DevinCarr will users need to run this command in the new WARP Connector flow?

Closing because the new WARP Connector flow takes care of iptables configuration.


installing fail on ubuntu server

Hi,
I'm new to use vps to run project. I really like how matterwiki looks. XD
When I run npm run build.
Below is my problems.
`
co@ubuntu:~/matterwiki$ npm run build

<EMAIL_ADDRESS>build /home/connor/matterwiki
cross-env NODE_ENV=production webpack --config ./webpack.config.prod.js --progress --profile --colors

sh: 1: cross-env: not found
npm ERR! Linux 4.4.0-57-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run" "build"
npm ERR! node v6.9.4
npm ERR! npm  v4.1.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>build: cross-env NODE_ENV=production webpack --config ./webpack.config.prod.js --progress --profile --colors
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'cross-env NODE_ENV=production webpack --config ./webpack.config.prod.js --progress --profile --colors'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the Matterwiki package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env NODE_ENV=production webpack --config ./webpack.config.prod.js --progress --profile --colors
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs Matterwiki
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls Matterwiki
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/co/matterwiki/npm-debug.log
`
I do have the node version 6.9.4
and npm version 4.1.2
Could you tell me what happen ?
Thank you

Hey, glad you like the design! :)
Did you run npm install in the Matterwiki repository before npm run build?

@krishnagopinath Hi,
I just followed the install instruction,
I'm sure I did npm install before npm run build.
I will try npm install again and give you the information that I received.
If you can record a video to present how to install matter wiki, I would be very appreciated.

@krishnagopinath
Hi, when I did npm install,
there is a message says:
npm WARN deprecated<EMAIL_ADDRESS>use uuid module insteaddoParallel extract

Funny, seems to work here. Could you give me the npm-debug-log that is generated?

Hi,
The attach file is the npm-debug-log.
Thank you :D
On Thu, Jan 26, 2017 at 10:20 AM, Krishna Gopinath<EMAIL_ADDRESS>
wrote:

Funny, seems to work here. Could you give me the npm-debug-log that is
generated?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Matterwiki/Matterwiki/issues/58#issuecomment-275414794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIpj0Dk-X1CBHKOhKzLv9AWoKlDvCTNEks5rWLnQgaJpZM4LuDR-
.

I found another one inside matterwiki folder. (The previous one I send to
you was outside of matterwiki.)
This one is much smaller.
On Thu, Jan 26, 2017 at 12:13 PM, Misaki Hoshino<EMAIL_ADDRESS>wrote:

Hi,
The attach file is the npm-debug-log.
Thank you :D
On Thu, Jan 26, 2017 at 10:20 AM, Krishna Gopinath <
<EMAIL_ADDRESS>wrote:

Funny, seems to work here. Could you give me the npm-debug-log that is
generated?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Matterwiki/Matterwiki/issues/58#issuecomment-275414794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIpj0Dk-X1CBHKOhKzLv9AWoKlDvCTNEks5rWLnQgaJpZM4LuDR-
.

@krishnagopinath
I did npm install and then did npm run build again.
There is no error this time.
But when I did node index
This time something more interesting happened lol.
co@ubuntu-512mb-nyc1-01:~/matterwiki$ npm run build

<EMAIL_ADDRESS>build /home/co/matterwiki
cross-env NODE_ENV=production webpack --config ./webpack.config.prod.js --progress --profile --colors
721868ms o1ms optimiz37ms emit
Hash: 19d1ec6588b1b27ab67b
Version: webpack 1.13.2
Time: 15088ms
Asset    Size  Chunks             Chunk Names
bundle.js  289 kB       0  [emitted]  main
bundle.js.map  2.5 MB       0  [emitted]  main
[0] multi main 40 bytes {0} [built]
factory:1ms building:3ms = 4ms
+ 258 hidden modules

co@ubuntu-512mb-nyc1-01:~/matterwiki$ node index
Knex: run
$ npm install sqlite3 --save
Error: Cannot find module '/home/co/matterwiki/node_modules/sqlite3/lib/binding/node-v48-linux-x64/node_sqlite3.node'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (/home/co/matterwiki/node_modules/sqlite3/lib/sqlite3.js:4:15)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Client._driver (/home/co/matterwiki/node_modules/knex/lib/dialects/sqlite3/index.js:65:12)
at Client.initializeDriver (/home/co/matterwiki/node_modules/knex/lib/client.js:211:26)
at Client (/home/co/matterwiki/node_modules/knex/lib/client.js:92:10)
at Client_SQLite3 (/home/co/matterwiki/node_modules/knex/lib/dialects/sqlite3/index.js:51:23)
Does this mean something wrong with the database ?

@krishnagopinath I might try other VPS (I'm using Digital Ocean) and try again.
Do you have any recommend VPS server ?
Thank you

@misaki5960x Sometimes when the build fails, the sqlite package is left out and is not installed. Installing the package using npm install sqlite3 should fix the issue :smile:

@nshntarora Thank you! I will try it!

@nshntarora Thank you again! It was the problem with sqlite. I did npm install sqlite3 before I did npm install, and now my server can run matterwiki without any problem!
Thank you guys! @nshntarora @krishnagopinath


♻ Archive '23 details

(Mostly) archives the '23 home page under /23-denmark
New home page says we're planning for '24

CTA to fill out our venue survey
No details on sponsorship yet but contact us if you have sometime specific in mind
Says we hope to bring back the opportunity grants (right??)

I tried not to nuke too much of the old code (eg the ticket form) because presumably we'll want that back at some point and refactoring it felt out of scope of this.

Thanks @hayleigh-dot-dev 👍


Restructuring the directory

Modify the directory structure:

move all function samples to "samples/functions"
move all functions-framework samples to "samples/functions-framework"
create "samples/applications" directory for application samples

Change module name of go.mod in OpenFuncAsync samples

Signed-off-by: laminar<EMAIL_ADDRESS>

Still in the editing process...


Added Microsoft Excel extensions.

Isn't XLSX actually XML-based?

That's right, but XLSX is ZIP-compressed XML, so XLSX can be treated as binary zip file. Same as DOCX, which was added earlier to the list.

Got it, thanks.
:+1:


Remove package: Daum.PotPlayer.Beta version <IP_ADDRESS>

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

It interferes with the stable version.

Microsoft Reviewers: Open in CodeFlow

URL: - https://t1.daumcdn.net/potplayer/beta/PotPlayerSetup64.exe
Invalid URI: The URI scheme is not valid.
(Automated message - build 915)


Improve documentation for more advanced features

This includes, but is not limited to:

[ ] GridFS
[ ] Geospatial queries

As per suggestion in http://stackoverflow.com/questions/36968156/advice-on-mongodb-odm-libraries-supporting-gridfs-and-geospatial-extension?noredirect=1#comment61676206_36968156

@malarzm @alcaeus I have stumbled upon the docs for aggregation framework - which is really nice btw - but they suggest that it is present in version 1.1.3, which is not the case. Should I open a new issue, or can I hijack this one a little? :)

@Steveb-p good catch, instead of creating new issue would you mind creating a PR with a note like one in Custom Collections chapter instead? :) That must have been overlooked originally as PR was quite huge ;)

Sure, will do in the evening :)


(Build) Fail on dev branch.

This issue is created automatically by CI.
Linux Build State: failure
Windows Build State: success
Commit: 518f49c7150074860faa3ae8874f5157cc222b73
Commit Branch: dev
Commit Author: Monika0000

Auto-closing as build in this branch is successful now. Commit: b404ad48dc815acceaa3a1643ede67d7d770cde3


[DOCFIX] update docs/en/Running-Spark-on-Alluxio.md

I tested the Running spark cluster on Alluxio, but the spark job did not work.
I found the reason is I only place the client jar in Spark master, but do not put them in Spark slaves so that Spark executors cannot find local Alluxio client jar.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18505/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.zookeeper.ZookeeperFailureIntegrationTest.zkFailure
Test FAILed.


api: add test_invoke_multi helper

Small utility function that allows you to chain contract method calls of already wrapped contracts e.g.
    facade = ChainFacade.node_provider_mainnet()
    neo = NeoToken()
    gas = GasToken()
    print(
        await facade.test_invoke_multi(
            [
                neo.symbol(),
                neo.balance_of_friendly("NSgwwTZP122oexuNGDju2u7uQZfKnG8ZLt"),
                gas.symbol(),
                gas.decimals(),
            ]
        )
    )

Because test_invoke cannot change the chain state (=read-only) the order of calling the functions doesn't matter and so it is done concurrently

Coverage decreased (-0.05%) to 81.83% when pulling 5c26acf0c893a84ca15a0db4ae4eb7123a62b8db on multi-invoke into fbc304fd19d435fd20f2c0c00b009f5300f06d7b on master.


Added Declared Licnese

Type: Missing
Summary:
Added Declared Licnese
Details:
Package site says MIT -https://www.npmjs.com/package/regenerator-transform/v/0.10.1
Resolution:
And the license file history in the repo shows it has always been MIT too
Affected definitions:

regenerator-transform 0.10.1

It looks like this should be is BSD-2 and Other due to the patent clause.
https://github.com/facebook/regenerator/tree/regenerator-transform%400.10.1
http://registry.npmjs.com/regenerator-transform/0.10.1
If you agree, I will update the PR.

@capfei - I don't know.  The GitHub repo link points to something from 2017.  Here, we are curating the NPM package and the package site and the site says MIT - https://www.npmjs.com/package/regenerator-transform/v/0.10.1.  Also, if you click on the GitHub link from that site (https://github.com/facebook/regenerator/tree/master/packages/regenerator-transform) it shows MIT.  I would go with this data, which aligns with the NPM publish date vs. the 2017 data.  @pombredanne for thoughts?

@capfei - to close on this issue, we understand the license info and last publish date on the NPM page reflects the most recently published package, so we will go with the registry info for this specific version, which is BSD-2 and OTHER for the patent file (http://registry.npmjs.com/regenerator-transform/0.10.1/https://github.com/facebook/regenerator/blob/regenerator-transform%400.10.1/LICENSE).


Rebaseline JavaFirstUsePerformanceTest

To accept 0.4% and 1.7% regression.

@bot-gradle test and merge

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

OK, I've already triggered a build for you.


Error when clicking button after translation

I tried to translate the name of the enchantment book in the GUI store in the configuration file "config". I clicked on it to report an error.
Server Core: paper-192.jar
Plugins Version: v1.8 Dev Build v6.1
translate:
Name: '&a&l&nCommon' #Name in the GUI
Slot: 31 #The Slot number the item will be in
InGUI: true #Toggle if it's in the GUI or not.
Lore: #The lore of the Item
- '&eCost: &6&l20 &6XP Levels'
TO
Name: '&a&l&n[普通]附魔书' #Name in the GUI
Slot: 31 #The Slot number the item will be in
InGUI: true #Toggle if it's in the GUI or not.
Lore: #The lore of the Item
  - '&e价格: &6&l20 &6级经验'
  - '&e可获得稀有的附魔书'

Bug info:
[18:54:48 INFO]: [Crazy Enchantments]>> The category Common has no enchantments. &7Please add enchantments to the category in the Enchantments.yml. If you do not wish to have the category feel free to delete it from the Config.yml.
[18:54:48 INFO]: [Yum] [Yum Event monitoring] Plugin CrazyEnchantments Handle InventoryClickEvent An exception occurs during an event!
[18:54:48 INFO]: [Yum] Exception name: java.lang.NullPointerException
[18:54:48 INFO]: [Yum] Exception description: null
[18:54:48 INFO]: [Yum] Simple error messages are as follows:
[18:54:48 INFO]: [Yum]     Be located me.badbones69.crazyenchantments.controllers.ShopControler.onInvClick(ShopControler.java:216)
[18:54:48 INFO]: [Yum]    Be located com.destroystokyo.paper.event.executor.asm.generated.GeneratedEventExecutor712.execute(unknow:-1)
[18:54:48 INFO]: [Yum]     Be located org.bukkit.plugin.EventExecutor.lambda$create$1(EventExecutor.java:69)
[18:54:48 INFO]: [Yum]     Be located co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:80)
[18:54:48 INFO]: [Yum]     Be located pw.yumc.Yum.inject.ListenerInjector.execute(ListenerInjector.java:88)

"Name of the category" does not recognize other languages.

Thanks, ill look into trying to make it translatable.


unable to access index type by keyof if index value type is unknown

TypeScript Version:  3.6.2
Code
export const indexByProp = <T extends Record<string, unknown>>(
  propName: keyof T,
  xs: T[],
): Record<string, T> => {
  const seed: Record<string, T> = {}
  return xs.reduce((index, x) => {
    const address = x[propName]
    index[address] = x
    return index
  }, seed)
}

Expected behavior: type checks
Actual behavior:
src/test.ts:8:5 - error TS2536: Type 'T[keyof T]' cannot be used to index type 'Record<string, T>'.

8     index[address] = x
      ~~~~~~~~~~~~~~

Playground Link: link

If I replace unknown with any then it type checks. I would prefer to use unknown as it is safer and clearer.

I feel like this is working as intended. If T[keyof T] points to a non-indexable property, the function would error:
const indexByProp = <T extends Record<string, object>>( // record of string -> object
  propName: keyof T,
  xs: T[],
): Record<string, T> => {
  const seed: Record<string, T> = {}
  return xs.reduce((index, curr) => {
    const address = curr[propName]
    index[address] = curr // error: T[keyof T] cannot be used to index type Record<string, T>
    return index
  }, seed)
}

However, if you guarantee that T values will be indexable types, it will work:
const indexByProp = <T extends Record<string, string | number | symbol>>(
  propName: keyof T,
  xs: T[]
): Record<string, T> => {
  const seed: Record<string | number | symbol, T> = {}; // notice that we change the seed type as well
  return xs.reduce((index, curr) => {
    const address = curr[propName];
    index[address] = curr;
    return index;
  }, seed);
};

Thanks @Barbiero makes sense now. While JS will call upon toString thus enabling any kind of value to be used for index entry name, TS is much stricter (good) here. Another example that works via type narrowing:
link
const indexByProp = <T extends Record<string, unknown>>(
  propName: keyof T,
  xs: T[],
): Record<string, T> => {
  const seed: Record<string, T> = {}
  return xs.reduce((index, x) => {
    const address = x[propName]
    // NOTE could also be as follows if a bit more change:
    // if (typeof address !== 'string' && typeof address !== 'number' && typeof address !== 'symbol') {
    if (typeof address !== 'string') {
      throw new Error(`Can only index over string values. You tried to index over a value of type ${typeof address}`)
    }
    index[address] = x
    return index
  }, seed)
}

I raised a related follow up question on Stack Overflow https://stackoverflow.com/questions/58030413/calculate-union-type-of-key-names-in-object-whose-values-are-indexable. While @Barbiero's solution works I would like the ability to accept objects which may contain non-indexable values and just limit propName to those which are valid indexable types (string, symbol, number).

Seems like more and more senseless errors are coming to TS lately.


Fix case with non-existing date

Sometimes we get a non-existing date:
** (ArgumentError) cannot parse "1942-02-29" as date, reason: :invalid_date
    (elixir) lib/calendar/date.ex:276: Date.from_iso8601!/2
    (ehealth) lib/ehealth/declaration_requests/api/creator.ex:82: EHealth.DeclarationRequests.API.Creator.validate_person/1
    (ehealth) lib/ehealth/declaration_requests/declaration_requests.ex:140: EHealth.DeclarationRequests.create_offline/2
    (ehealth)

Ideally this is fixed on json-schema level. We're using (1) jvalid (2) package.
Relevant request IDs:

c72ad96b-c085-430e-a506-bb77c451ff09#51431
db99429a-50da-43c9-a368-b8e50da42c09#225432
0b40d1b6-6b7b-4b24-95b0-e304a201a4eb#261922

[ ] medication dispense


refactor: update angular versions to master builds

Updating PR from @daniloff200

It is just shorter to read :)

closes #48


pip install torch==1.13.1+cu117

.

You can install different versions of torch. I use 2.2.0 cuda 11.8


You accidentally used Javascript on your web site, about not using Javascript

I'm sure it was an oversight. You might not need javascript.

I hope opening this issue made you feel better

I hope you took this with some humor. I am a fan of your stuff.
Specifically your open source goals and toolsday. Sorry If I was the
10,000th person to say it. I have mad respect for you.
On Tue, Oct 11, 2016 at 5:27 PM Una Kravets<EMAIL_ADDRESS>wrote:

Closed #16 https://github.com/una/YouMightNotNeedJS/issues/16.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/una/YouMightNotNeedJS/issues/16#event-820154844, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAqlXjsAHp0kPkbeHuWw6V-HnbHZjBrEks5qy_9UgaJpZM4KUH3z
.


NO-ISSUE: Add diffutils package to buildroot image

Certain golangci-lint linters require the diff util, which is not installed by default in the ubi-minimal image, which is used for the buildroot image in the CI. This must be a standalone commit as the CI gets the Containerfile.tools file from the branch HEAD, rather than the PR being tested.

/lgtm

/approve


Tests module: includes moved out from cvtest namespace

Fix for https://github.com/Itseez/opencv/issues/5740

Hm, can you comment please?
Imgcodecs_Tiff.decode_tile16384x16384 seems to fail with timeout, but even in Debug-build this test was passed for 19240 ms. Is this is some buildbot-related problem?

It is sporadic failure. Build is restarted manually.

:+1:


Realm support to proxies

The Relay is not using createClient by design. That function is designed for connecting to servers as a standalone client. It handles boilerplate such as keep-alives and basic connection prerequisites.
That is not desirable for a proxy, which is supposed to only relay authentic packets from one side to another. The client can get kicked, flagged or banned on some servers with the extraneous packets. The purpose of Realy is to be as close to identical in terms of a vanilla client down to the ping behavior, as vanilla clients always ping before they connect.
For this feature to be implemented the relevant code needs to be copied over to the Relay.

Alright I’ll fix it tomorrow

I switched it back to the Client class instead of the createClient function

Have you tested this to work?

Sorry for the late response, but I've restarted it and joined about 5 times and so far I haven't had any problems.

Thanks !


bail handling refactor

What does this PR accomplish?
Refactor #63 and builds on top of #61 which is refactored in the process.

[ ] Bug Fix
[x] Feature
[ ] Documentation

closes #63.
Changes proposed by this PR:
Introduce an intermediate representation between found mistakes and exit code, called Finish.
Notes to reviewer:
Partially refactored your existing code and moved a few things around, I tried to cover all test cases manually - ShouldWork™

Ahhh f*ck, that was accidental..


koreader: 2022.05.1 -> 2022.06

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/koreader/koreader/releases.
meta.description for koreader is: "An ebook reader application supporting PDF, DjVu, EPUB, FB2 and many more formats, running on Cervantes, Kindle, Kobo, PocketBook and Android devices"
meta.homepage for koreader is: "https://github.com/koreader/koreader"
meta.changelog for koreader is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2022.06 with grep in /nix/store/dg6l07wzg77ybxjqw9gx2a5fsy68b5br-koreader-2022.06
found 2022.06 in filename of file in /nix/store/dg6l07wzg77ybxjqw9gx2a5fsy68b5br-koreader-2022.06

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
koreader

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dg6l07wzg77ybxjqw9gx2a5fsy68b5br-koreader-2022.06 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A koreader https://github.com/r-ryantm/nixpkgs/archive/51c82c45bd7d0f2ee138281817044fdcc1fc4fd2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dg6l07wzg77ybxjqw9gx2a5fsy68b5br-koreader-2022.06
ls -la /nix/store/dg6l07wzg77ybxjqw9gx2a5fsy68b5br-koreader-2022.06/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

koreader

Maintainer pings
cc @contrun @neonfuz for testing.

v2022.07 is already out


CS-3124: Updated the terraform for circuit breaker

Closing this for now.
Please re-open if/when this is ready to be merged, at some future point.


Qt: Revert "Force TLS1.0+ for SSL connections"

See: https://github.com/bitcoin/bitcoin/pull/14403
This reverts commit 15e26a6, whose
purpose was to tweak the Qt configuration to force TLS, i.e., to
disable SSLv3, in Qt versions >= 5.5. However, the default behavior
of Qt >= 5.4 is to disable SSLv3 anyway [1], so the configuration
tweak is redundant.
With Qt 5.11.2, the configuration tweak is not only redundant but in
fact provokes a deadlock (#14359) due to Qt 5.11.2 being incompatible
with OpenSSL 1.1.1 [2]. Since the deadlock occurs at the early startup
stage of bitcoin-qt, it renders bitcoin-qt entirely non-functional
when compiled against OpenSSL 1.1.1 and Qt 5.11.2 (and possible future
combinations of OpenSSL and Qt versions).
This commit fixes #14359 by removing the redundant code.
[1] https://code.qt.io/cgit/qt/qtbase.git/commit/?id=3fd2d9eff8c1f948306ee5fbfe364ccded1c4b84
[2] https://bugreports.qt.io/browse/QTBUG-70956

utACK, even though I assume that the fixed issue is not a problem when using the depends subsystem which uses an older Qt version.

True, it works fine when using the depends subsystem to build the Qt wallet but take less time this way:
./autogen.sh && ./configure && make


proposal: net/http: add ability to know which requests are HTTP2 push requests

When I use http.Pusher interface to push a HTTP request/response, there is no way to know which requests are being pushed, once they become handled. Even more, it seems there is no way for me to tie those requests with the original request in my logging code (e.q., using request ID or something similar).
I would propose that http.PushOptions is extended with additional Context: interface{} field which allows to pass additional information to the pushed request, and that http.Request gets PushContext: interface{} field to read it out. This way I could pass through ID or even some other information which could help handle those requests faster (like internal caches or some other data I already have during handling of the original request).

CC @neild @bradfitz


Ensure testing of count as result of write, for all backends.

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
It seems like there is currently no test for count (result of write) for the swift backend, in the event of an error (when 0 < count < len(data)).
(https://github.com/openfun/ralph/blob/master/tests/backends/data/test_swift.py)
Describe the solution you'd like
Add the test mentionned above for swift backend (and others if missing).

@Leobouloc I am not sure to follow: what test would you add? Seems tested here and there


Purpose of cog wheel on an error message is unclear

Purpose of cog wheel on an error message is unclear. The hover help states 'Configure Notification'. Clicking on it states 'Manage Extension'. But one cannot actually perform any function.
What did you do?
I just tried to login in to an empty server to cause an error message to pop up. But this seems to be true for all error messages.
What did you expect to see?
Some action to occur when the cog wheel symbol on a message was selected.
What did you see instead? Under which circumstances?
Nothing.
Possible Solution
Either add a valid action or remove the cog wheel symbol from all messages.
Additional context

It doesn't seem like there is a straightforward way to resolve this with the current vscode api.


"selectRow" checkbox of first selected row doesn't get checked on Firefox (44.0.2) ...

... when clickToSelect is true and you don't click directly on the checkbox.
The backgroundcolor correctly changes, though.

@shyblower, what version did you used? and could you please give me an example? btw, only happened on firefox? Thanks :)

I used react-bootstrap-table1.5.4 on Firefox-44.0.2. To see if it was only a Firefox problem I tried again on the latest Chromium browser where it worked flawlessly :)
Steps to reproduce:

Create a 2 row table with selectRow in "checkbox" mode and clickToSelect set to true in the selectRow properties.
Click on one of the rows (not directly on the checkbox).
The row becomes correctly coloured with the chosen bgColor but the checkbox will NOT display a check mark.
Click somewhere on the second row . Now this row will show the check mark correctly.

It doesn't matter which row you click first, only the first selected row will expose this behaviour.
If you clear the selection and do it again it will still behave as described.

@shyblower, could you try v1.6.0 again for this issues?

Could you please set a v1.6.0 tag as you did with the previous releases, because I my build system downloads a zip file based on the version tag automatically. It would be much easier for me to test it that way.

sorry, I've pushed v1.6.0 as a tag, check it out!

It still does not work. Maybe a thing to mention is that when I click directly on the checkbox (where it kind of works), the check mark appears only after I have moved the cursor outside the checkbox rectangle. But this happens also only with the first row that I select. The second selection shows the check mark immediately.

ok, it's strange, could you please give me your example code? or could your reproduce this issue on http://allenfang.github.io/react-bootstrap-table/example.html#select-checkbox?


Fix --productID parameter documentation

We claimed this was required, but it's not.

@timsneath,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


"display" equation + auto-render = Inline equation

So here's a screenshot of the page before I run renderMathInElement(document.body);:

And now after I run the command:

It may not be apparent from the screenshot but it is being rendered the exact same as an inline equation despite the default for \[ ... \] display: true
Happens even when I pass:
{
 delimiters: [
   {left: "$$", right: "$$", display: true},
   {left: "\\[", right: "\\]", display: false},
   {left: "\\(", right: "\\)", display: false}
  ]
}

for the second argument

@vihanb When you say "rendered the exact same as an inline equation" do you mean "centered"? Here's a jsbin with some inline math and then some display math: http://jsbin.com/sikehuxofu/1/edit?html,output

The display math is centered and on a new line, but it's also in display style, so the \sum is larger. It's also wrapped in a div.katex-display element, which is what turns it into a display element:

So, I'm pretty sure that it's being correctly rendered as inline math, but you must have CSS on your site which is centering it.

If I understand this correctly, the problem is that the katex-display wrapper is absent here.

Happens even when I pass: …

No surprise there: you explicitely set display: false for the \[…\] syntax. With default settings the behavior is more surprising. I can't reproduce that aspect.

If this is still an issue, please reopen it. Thank you!


node-red: Updates to not update base stack packages and some perf changes

Signed-off-by: Steven Groeger<EMAIL_ADDRESS>Checklist:

[x] Read the Code of Conduct and Contributing Guidelines.

[x] Followed the commit message guidelines.

[x] Stack adheres to Appsody stack structure.

Modifying an existing stack:

[x] Updated the stack version in stack.yaml

Contributing a new stack:

Describe how application dependencies are managed:

Explain how Appsody file watcher is utilized:

Describe other Appsody environment variables defined by the stack image:

Describe any limitations and known issues:

Related Issues:

Related issues:

#709
#710

@knolleary Would like to get a review from you (if possible)


Segmentation fault (core dumped) during FFT initialization

I am facing the same issue
./glfft_cli help outputs [1]    6691 segmentation fault  ./glfft_cli help
If anyone has any fix, please do let me know


cmd/oci-image-tool/man: Document multiple --ref for 'validate'

Catch up with 6ca74be5 (cmd/oci-image-tool: support multiple --ref, 2016-09-06, #265).
While I'm touching it, reformat this entry to one line per sentance per the README's Markdown style section.

On Tue, Sep 06, 2016 at 10:07:49PM -0700, W. Trevor King wrote:

While I'm touching it, reformat this entry to one line per sentance
per the README's Markdown style section.

Although it appears go-md2man does not actually mark up these
definition lists:
$ go-md2man -in cmd/oci-image-tool/man/oci-image-tool-validate.1.md -out /dev/stdout
…
.PP
\fB--ref\fP NAME
The reference to validate (should point to a manifest).
Can be specified multiple times to validate multiple references.
\fB\fCNAME\fR must be present in the \fB\fCrefs\fR subdirectory of the image.
Defaults to \fB\fCv1.0\fR&.
Only applicable if type is image or imageLayout.
…
I'm not really sure what the plan is here.  I'd be in favor of
switching to AsciiDoc or other man generator with a bit more polish
1.

LGTM (IANAM)

lgtm

LGTM


prettify console output

share code for this between cook and goglizer

create color.py and add this in

[X] dolmades
[X] goglizer
[ ] cook

dolmades doesnt need it yet...


IoT Latin America 2017 - Environment and demo script

https://www.youtube.com/channel/UCK1iQ-d-K-O2mOLahPOoe6w


$route is not computed correctly until after app is mounted with async components

Version
3.5.1
Reproduction link
https://codesandbox.io/s/happy-tdd-qh8vk?file=/src/App.vue:273-317
Steps to reproduce
Create a router with async components, log the content of the route on mount
Load the page, notice the $route property is not correct on the mounted hook, it will be after the route component has loaded though, it is also correct if the route component is not async
What is expected?
The $route property to be correctly reflecting the current route
What is actually happening?
The $route property doesn't contain anything related to the current route

Since the route is still being matched properly before the router is ready it should contain the correct info even though the component it contains has still not loaded, currently it waits for the component to be fetched to trigger the "ready" event and set the correct $route, the route should be set before the "ready" event fires to avoid this problem
PS: I'm running into major troubles with server side rendering and v-if because of this bug

This is expected: the route object is set once the navigation is validated. Please search existing issues before opening one

I did search but this creates a problem with server side rendering
With SSR if you use a computed property with this.$route.meta it will be correct, but on the front it will not creating an invalid node state that throws an error
Have a look there https://staropramenjatek.hu/belepes/
I cannot wait for the router to be ready in the computed property or then the component is not prerendered

Okay so I did solve the problem by doing
const v = new Vue({
  router,
  //...
});

router.onReady(() => {
  v.$mount('#app');
});

Effectively delaying the app mount until the component is loaded, but it does delay the mounting time by a while, surely there must a more efficient way of doing this without getting an error. But for now this is enough for me

Okay so I did solve the problem by doing
const v = new Vue({
  router,
  //...
});

router.onReady(() => {
  v.$mount('#app');
});

Effectively delaying the app mount until the first component is loaded, but it does delay the mounting time by a while and it's slightly noticeable (the page looks like it's jittering on load), surely there must a more efficient way of doing this without getting an error. But for now this is enough for me

Hi @Tofandel I had a very similar issue as you and after some fiddling I found a different solution and wanted to share.
const router = useRouter();
const route = useRoute();

router.isReady().then(() => {
   ...
});

https://router.vuejs.org/api/#isready
(method) Router.isReady(): Promise
Returns a Promise that resolves when the router has completed the initial navigation, which means it has resolved all async enter hooks and async components that are associated with the initial route. If the initial navigation already happened, the promise resolves immediately.
This is useful in server-side rendering to ensure consistent output on both the server and the client. Note that on server side, you need to manually push the initial location while on client side, the router automatically picks it up from the URL.


Nested type declaration in separate file causes specs to fail to load

Hello,
If the yaml spec is splitted in 2 files as follows:

api.yaml: paths that refers to models.yaml#components/schemas/XXX
models.yaml: components that re sub component #components/schemas/YYY

At startup the GetSwagger() function returns an error that complains that #components/schemas/YYY is not found.
You'll find an example to reproduce this behavior here: https://github.com/ydarma/oapi-codegen-subtype
Thancks

Do you have this problem all the time or does it regenerating the sources a few times makes it work?

I remember that the problem always happened. It is an old thread, I remember that we decided to work this around and to not use a separate model specification.
Recently on another project we went with a separate model file with nested components and now it works. Maybe the problem has been fixed since.


Extra step needed to prevent error

[Enter feedback here]
Installing the cmdlet using the steps on this page sometimes gives "CommandNotFoundException" related errors when user runs any PowerApps command such as "Add-PowerAppsAccount".
To prevent/fix that, as pointed out in this post, the following commands should be executed by the user:
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Force
dir . | Unblock-File

Please address the problem and/or add the instructions above so that users will not encounter the same problem in the future.

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

ID: 02f1be79-2b3a-31da-7a7d-ace1eac5d1f1
Version Independent ID: fa7c7450-15a6-edc8-5671-f45763267ca6
Content: PowerShell support - Power Platform
Content Source: power-platform/admin/powerapps-powershell.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

I was not able to reproduce this error when I installed this Administration module on a new VM.  If you continue to face this error please let us know via support ticket the repro steps.  Thanks!


Coverage icon is broken

@davorrunje it is working, but it grabs information from commit status.
To fill this information you need to add smokeshow CI task #630

I temporarely commented it out since it is broken on the PyPi main page and README on github, we'll add it back once we complete #630. This is a reminder task for that.


Apple Mach-O Linker Error

Hi I'm newbie to develop a app using xcode.
I tried to launch ios app with google map on simulator and got this error.
no such file or directory: '/Users/ruci.k/Library/Developer/Xcode/DerivedData/snappinPhotographer-ciezjvwndpeipgglzatkopsrsmnz/Build/Products/Debug-iphonesimulator/libReact.a'
There was a React dir in Debug-iphonesimulator dir, and libReact.a was in it.
'libReact.a' file is seem to generated by some kind of xcode project build setting automatically,
But I can't find which setting decides the path of 'React' library file.
Is there any information about it?
I did just follow instruction guide
npm install react-native-maps --save
 rnpm link
vim podfile
pod install
pod update

This might help: https://github.com/decosoftware/deco-ide/issues/84

@symmetriccurve Thanks, but that option already checked :'-(

Did you try linking it by hand instead of Podfiles or rnpm link?
I did it by hand myself and it worked for me.

Thanks for your reply.
I tried to link it by hand, and it worked 😄

https://github.com/ivpusic/react-native-image-crop-picker/issues/21#issuecomment-239710351

Hi ,
I am facing same issue why archiving the build.
The app is running fine in iphone 7 simulator. But when i am archiving the app i am getting linker error.
Please help me.

I deleted a folder of a test project in Project Navigator and Targets. It solved my problem.

@asimolmez, I deleted test project folder in Project Navigator but i did not find the Targets area, and the error still occurring.
I not even understand what you do mean with Targets.
Could you be more didactic, please?
Thanks!

@rajeshkos hi, am facing  same issue have you found a way to workout the issue >??

I ran into a similar problem and clearing my DerivedData and re-building fixed this issue for me.
you can do this by running Command + option + shift + k in xcode and rebuilding

@zyskowsk thank you so much

@zyskowsk God bless you!

@zyskowsk Am I supposed to get a dialog when running your solution? I get no notice whether the cache cleared or not. 🤷♂

@mjstelly

@zyskowsk Am I supposed to get a dialog when running your solution? I get no notice whether the cache cleared or not. 🤷♂

This changed in XCode10, there is no longer a dialog. It seems that now the only notice is a flask of cleaning and clean finished in the build bar

have you tried adding javascriptcore.framework under linked libraries? That worked for me

decosoftware/deco-ide#84

Hi thanks it really helped me apart from that please make sure try to check the workspace setting if  it has following setting :
Build System:
-Legacy Build System
1.Build Architecture Yes
2.Workspace Setting:
Build System:
Legacy Build System
Per-User Workspace
DerivedData:
Workspace relative location
Build System
Use User Setting

https://stackoverflow.com/questions/5329001/apple-mach-o-linker-error-when-compiling-for-device


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:17: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:36: missing_pip_check: For pypi packages, pip check should be present in the test commands
INFO: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:39: invalid_url: http://www.cairosvg.org/ : URL domain redirect www.cairosvg.org ->  cairosvg.org
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:40: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, closest match: LPPL-1.0
INFO: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:44: invalid_url: http://www.cairosvg.org/documentation : URL domain redirect www.cairosvg.org ->  cairosvg.org
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:45: http_url: http://www.cairosvg.org/documentation is not https
WARNING: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:45: missing_description: The recipe is missing a description
ERROR: /tmp/abs_53acuc0svf/clone/recipe/meta.yaml:45: http_url: http://www.cairosvg.org/ is not https
Errors were found


When should we add a dataset to the published list?

dataset - a unit of useful data/information, can be assembled from numerous collections.
We need to decide at what point a dataset should appear on the list of datasets on the digital land site.
Should they appear as soon as we have specified the schemas it is made up of?
In this case, we might have defined schemas but not yet collected any data for it. An example of which is developer contributions. Should we list this 'empty' dataset?
Should we only publish once we have collected some data? How much? Would a single record be enough? A single record would at least act as a real-life example what the data should look like, which would be useful for data publishers but not necessarily data consumers.
Another thing we could do is indicate on the list page how 'active' the dataset is. We could include metrics such as number of records, data last collected date, number of publishing organisations. Anymore?

The creation of new datasets should be cheap. A new one could just be assembled from existing collections.
This will be useful when we create a dataset that doesn't quite satisfy the needs of a data consumer. We will be able to quickly create a new one that is more closely aligned to their needs.
At the moment we think there is still merit in listing any dataset we have created. If we create a new one it doesn't have to replace an existing one. They can keep both available.


[tests] Tweak the regex for jest unit tests to explicitly check for integration.

The regex bit (?!.integration.) checks that we don't match a test like integration.test.ts.
Our copy of this repo lives under the directory integrations/figma-for-jira. Currently, this regex inadvertently matches all tests within integrations/figma-for-jira because the folder name has the string integration
To fix the regex, I added a check to exclude integration. vs. just the string integration.
I checked that npm run test:unit runs both in this copy and in our copy.

That's a funky problem.. nice catch!


autoplaySpeed is changed by clicking nextBtn when fade equals true

https://jsfiddle.net/penguin3069/L96rm9j6/
set autoplaySpeed, then click nextBtn very quickly,and speed will be changed

I encountered the same issue.
@penguin3069 I downgraded to the v0.10.0 and it seems to work.

@dalpo Thanks


How to set ALB Security Policy to ELBSecurityPolicy-TLS-1-2-2017-01

By default, ALB Security Policy is set to ELBSecurityPolicy-2016-08. I want to set it to ELBSecurityPolicy-TLS-1-2-2017-01. I know I can manually set it after ALB is created. But is there a way to automatically do it?

If you want to try https://github.com/lareeth/aws-alb-ingress-controller/tree/feature-ssl-policy this should work, will create a pull request so it can be reviewed properly

This is now supported with #415


Improve GitHub Actions for pull requests

See https://github.com/Senzing/community-roadmap/issues/71

Pull requests:

https://github.com/Senzing/docker-base-image-debian/pull/38


HDDS-11017. Migrated to ECharts, Vite and AntD v4 with eslint, prettier

What changes were proposed in this pull request?

While conducting a fossa scan we found that node-notifier and jsdom is not compatible with Apache license
This PR migrates from the react-scripts used in the current project to Vite and Vitest which resolved the licensing issue
Apart from that this will bring additional improvements like:

Faster local development time
Faster builds
Migration to AntD v5 which will allow us to directly upgrade to newer versions
Shift from xo to eslint and prettier - this will make the development experience more smooth and consistent for contributors

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11017
How was this patch tested?
The patch was tested manually on local build

Currently library selection no longer has any licensing issues.

@devabhishekpal
Can we update node and pnpm version also?

Hi @smitajoshi12 , so for Node v18 - it no longer supports CentOS 7 and older linux distros.
We would need to then specify it that the project cannot be built on older linux dist.
Hence we are stuck with Node v16 for the time being.
pnpm v9 required Node 18. So I am upgrading to the latest supported pnpm v8.15.7.

@devabhishekpal
In Disk Usage CSS aligmnet has been changed for Show Metata Data button.

@devabhishekpal
Can we move pi chart alignment to left side.

@devabhishekpal
Css Issue in heatmap when selecting dropdown

Hi @smitajoshi12, for https://github.com/apache/ozone/pull/6841#issuecomment-2188075777, if we move the pie-chart to the left side it would look odd as without the metadata panel open, it would have lots of empty space on the right hand side.
So we are keeping it in the center as when the metadata is viewed the user will either way not be interacting with the chart.
Please do let me know if you want to shift to the left or if you have any other alternate inputs.
Thanks

Hi @smitajoshi12, for #6841 (comment), if we move the pie-chart to the left side it would look odd as without the metadata panel open, it would have lots of empty space on the right hand side. So we are keeping it in the center as when the metadata is viewed the user will either way not be interacting with the chart.
Please do let me know if you want to shift to the left or if you have any other alternate inputs. Thanks

@ivandika3  @dombizita
Can suggest but if  entity name is lengthy then meta data summary pop is overlapping or we can mimimize metadata summary pop up so we will get clear visibility of entity names.

@smitajoshi12 could you let me know the screen resolution you are using for https://github.com/apache/ozone/pull/6841#issuecomment-2188080629?
It seems in my resolution this issue is not present.
https://github.com/apache/ozone/assets/43001336/493257b8-712a-4bc9-b1bb-0fa8fe4e40fc
It is good to have other resolutions to test as well. Thanks a lot for bringing this to my attention.

Hi @smitajoshi12, for #6841 (comment), if we move the pie-chart to the left side it would look odd as without the metadata panel open, it would have lots of empty space on the right hand side. So we are keeping it in the center as when the metadata is viewed the user will either way not be interacting with the chart.
Please do let me know if you want to shift to the left or if you have any other alternate inputs. Thanks

@ivandika3 @dombizita Can suggest but if entity name is lengthy then meta data summary pop is overlapping or we can mimimize metadata summary pop up so we will get clear visibility of entity names.

So I changed the current legend to go on the left hand side for better visibility.
After my current changes it looks like below:

Hi @smitajoshi12, for #6841 (comment), if we move the pie-chart to the left side it would look odd as without the metadata panel open, it would have lots of empty space on the right hand side. So we are keeping it in the center as when the metadata is viewed the user will either way not be interacting with the chart.
Please do let me know if you want to shift to the left or if you have any other alternate inputs. Thanks

@ivandika3 @dombizita Can suggest but if entity name is lengthy then meta data summary pop is overlapping or we can mimimize metadata summary pop up so we will get clear visibility of entity names.

So I changed the current legend to go on the left hand side for better visibility. After my current changes it looks like below:

@devabhishekpal
These disk usage changes looks good to me.

After adjusting for media queries in the heatmap we are having the following UI:
Resolution was reduced to adjust for Github media size
https://github.com/apache/ozone/assets/43001336/a3469127-9df4-4465-b0c8-1038548363ea

@devabhishekpal
while doing cluster testing  volume1 was having data  and 100% and Total Data Size: 63.1 KB
It should be inside pie chart Size and Percentage is accurate just position is changed.

{
"status": "OK",
"path": "/",
"size": 64614,
"sizeWithReplica": -1,
"subPathCount": 39,
"subPaths": [
{
"key": false,
"path": "/volume1",
"size": 64614,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/s3v",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-0-79279",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-1-38323",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-10-54251",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-11-70290",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-12-44556",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-13-18787",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-14-46679",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-15-55529",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-16-51548",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-17-87656",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-18-09750",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-19-95360",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-2-75113",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-20-41102",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-21-75670",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-22-58601",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-23-97664",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-24-56683",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-25-42101",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-26-76437",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-27-82747",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-28-42441",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-29-31116",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-3-08351",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-30-12401",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-31-88150",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-32-90258",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
},
{
"key": false,
"path": "/vol-33-64420",
"size": 0,
"sizeWithReplica": -1,
"isKey": false
}
],
"sizeDirectKey": -1
}

@devabhishekpal
while doing cluster testing volume1 was having data and 100% and Total Data Size: 63.1 KB It should be inside pie chart Size and Percentage is accurate just position is changed. How we will come to know which are zero size entities and which entities are having data.

Hi @smitajoshi12, so in general when we are hovering over an area the area becomes highlighted and elevated in ECharts.
Apart from that I added new changes to also display the current path element name in the tooltip in the legend color.
After changes it looks like below:

With Same Cluster data results are different in pi chart
filecount
[
{
"volume": "volume1",
"bucket": "bucket1",
"fileSize": 32768,
"count": 5
},
{
"volume": "volume1",
"bucket": "bucket2",
"fileSize": 32768,
"count": 3
}
]
container count
[{"containerSize":536870912,"count":1}]
Old Library

New Library

5:41
"fileSizeCounts": [
{
"volume": "volume1",
"bucket": "bucket1",
"fileSize": 32768,
"count": 5
},
{
"volume": "volume1",
"bucket": "bucket2",
"fileSize": 32768,
"count": 3
}
]

Hi @smitajoshi12, thanks for pointing out this issue.
Fixed it in the latest patch.
With your data now it is looking like below:

One change from the original is that we are referring to the range instead of the actual value as in the original it seemed like multiple entities were having the same fixed container size.
Showing the range let's the user know that it is not a fixed size but a range in which the entities are spread

@devabhishekpal
Looks like Container ID column alignment has been changed.

@devabhishekpal
Can we increase width of Pi Chart align more to left side so can get visibility for all entities.

@devabhishekpal In Overview Page In Route.json entries are also missing for local run getting 404 in local.

Hi @smitajoshi12 , I checked the older routes.json as well. I mean from older branch (Ozone 1.4.0) it seems the path mapping to /triggerdbsync/om was never present, hence the 404 response.
But if we test it on a cluster instead of in dev mode, the API is being called. So it is okay in built files.

@devabhishekpal In Overview Page In Route.json entries are also missing for local run getting 404 in local.

Hi @smitajoshi12 , I checked the older routes.json as well. I mean from older branch (Ozone 1.4.0) it seems the path mapping to /triggerdbsync/om was never present, hence the 404 response. But if we test it on a cluster instead of in dev mode, the API is being called. So it is okay in built files.

No Issues closing this comment

@devabhishekpal
Looks like Container ID column alignment has been changed.

It seems this is the default behaviour with expandable rows.
This is also present in the existing code, I cross checked with the current code.
The following is built on master

@devabhishekpal
Can we increase width and Height of Pi Chart align more to left side so can get visibility for all entities.

So even if we increase the pie-chart width the percentage of entity area will be the same.
I added the changes to spread out the pie-chart and added padding to the Tab panels as you suggested.
However with a larger pie-chart the labels will overlap on the chart as seen below.

So sticking to current size. I believe your PR to add a normalized width to the small entities would solve this issue of small areas.

It looks good to me now.

@devabhishekpal
Response and Pi Chart are different

Response:-
{
"status": "OK",
"path": "/vol1/bucket1/dir1/key1",
"size": 3800,
"sizeWithReplica": 11400,
"subPathCount": 0,
"subPaths": [],
"sizeDirectKey": 0
}

@devabhishekpal
Response and Pi Chart are different

Response:- { "status": "OK", "path": "/vol1/bucket1/dir1/key1", "size": 3800, "sizeWithReplica": 11400, "subPathCount": 0, "subPaths": [], "sizeDirectKey": 0 }

It  looks Good to me after your recent changes

Hi @ArafatKhan2198, yes - the changes have been tested with the the datasets that were used by @smitajoshi12 in the original normalization PR and it is working correctly.
The commit https://github.com/apache/ozone/pull/6841/commits/d8cdd9214036467ca91341cf369eadf1c3c83e50 adds the changes for the normalization.

Thanks for the work @devabhishekpal & thanks @devmadhuu @smitajoshi12 @ivandika3 @adoroszlai for the review.


Datepicker add possibility to change today highlight

Feature Description
Today is currently highlighted considering the browser timezone, it is not possible to configure a custom timezone or configure a different date for "today".
Use Case
This feature would be useful in applications that use a custom timezone configuration in application settings instead of browser's timezone.

This should be possible with the startAt input on the mat-datepicker.


About diffloss

Hi,
In this part https://github.com/LTH14/mar/blob/fe470ac24afbee924668d8c5c83e9fec60af3a73/models/mar.py#L232-L238, why you did not filter unmasked tokens (by mask) and just pass the masked tokens to the self.diffloss? You then use the mask to compute the loss only for masked tokens.
Thanks

Both implementations are ok

Thanks for the prompt response.


Add more resources in Web Development

The directory structure to add resources is as follows: ../pages/web-development-resources.md

Shall I?

Let's get you both assigned, but please make sure that you don't create merge conflicts, @Rishav1707 @ashutosh0022


Proto options

This addresses #816
Here's a summary of the commits:
Tweak existing tests
Improve the tests while making them compatible with what's coming. (4 commits)
Tweak existing code
Small changes to the code base but no change to any functionality or function interface (4 commits)
Use prototype for global options
Gradually modify the interface to ParsleyUtils to OptionFactory so global options are handled using ParsleyConfig as prototype. (5 commits)
Use protoype for field & form options
Creates _resetOptions that is in charge of building the options with the right prototype chain. Note that only a single test changes, because we now give priority to explicit options over DOM options.  (1 commit)
Cleanup
The only thing not handled as nicely as before is if the parent changes (only happens in one spot). Then we must rebuild the options (since the sequence of prototypes have changed). The last commit shows something that wasn't possible before. (last 3 commits)

For some reason, github choses to display things in the wrong order.
Please use this list of commits that shows the actual order of the commits.

That seems a pretty neat work.
I'll try to take some time tonight to make a in-deep review. Thanks !

Any chance to look at it? I realize it's not a trivial PR...
Let me know if I can do anything.

This is a truly great PR. Thanks again for your awesome work.


Objgraph images not saved

Hello, while using memory_analyzer I found out that the images from objgraph were not being saved even though the path of them is being shown in the summary of the analyse. I’ve searched everywhere and couldn’t find any of the images. Any idea on why this happens?
Thanks.

Can you provide a little more info about your environment and how you're running it, including some logs of running memory-analyzer?  If there's a chroot, container, etc involved, are you looking in target's filesystem?

I’m running the latest version directly pip installed from GitHub. I run the tool directly in a virtual-environment with python 3.8, I’m also using Ubuntu 18.04. The target process is not located in any container, it is just running using another virtual environment.
memory_analyzer run -s 2 --no-upload --debug --quiet 1847903
Analyzing pid 1847903
Setting up GDB for pid 1847903
/home/tests/tests-env/bin/python3.8-dbg: No such file or directory.
[New LWP 1847905]
[New LWP 1847910]
[New LWP 1847911]
[New LWP 1847913]
[New LWP 1847914]
[New LWP 1929080]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
0x00007fbff1ebe5ce in epoll_wait () from /lib/x86_64-linux-gnu/libc.so.6
+set directories /home/tests/tests-env/lib/python3.8/site-packages/memory_analyzer/
+py sys.path.append("/home/tests/tests-env/lib/python3.8/site-packages/memory_analyzer/")
++file_command /tmp/tmppia4vfp4/rendered_template-1847903.py.out
GIL $1
+++call (void) PyRun_SimpleString("with open('/tmp/tmppia4vfp4/rendered_template-1847903.py.out') as f: exec(f.read())")
+++call (void) PyGILState_Release($1)
[Inferior 1 (process 1847903) detached]
Gathering data...
Writing output to file memory_analyzer_out/memory_analyzer_snapshot-20210322174120

The only thing which looks suspicious is the fourth line, but if I run the following the message disappear but the images are still not saved.
memory_analyzer run -s 2 --no-upload --debug --quiet 1847903 -e /usr/bin/python3.8
Analyzing pid 1847903
Setting up GDB for pid 1847903
[New LWP 1847905]
[New LWP 1847910]
[New LWP 1847911]
[New LWP 1847913]
[New LWP 1847914]
[New LWP 1929080]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
0x00007fbff1ebe5ce in epoll_wait () from /lib/x86_64-linux-gnu/libc.so.6
+set directories /home/tests/tests-env/lib/python3.8/site-packages/memory_analyzer/
+py sys.path.append("/home/tests/tests-env/lib/python3.8/site-packages/memory_analyzer/")
++file_command /tmp/tmp8sav8i6l/rendered_template-1847903.py.out
GIL $1
+++call (void) PyRun_SimpleString("with open('/tmp/tmp8sav8i6l/rendered_template-1847903.py.out') as f: exec(f.read())")
+++call (void) PyGILState_Release($1)
[Inferior 1 (process 1847903) detached]
Gathering data...
Writing output to file memory_analyzer_out/memory_analyzer_snapshot-20210322174441

I have made some research and looks like that when graphviz is not installed, the objgraph images are saved as .dot file in /tmp/ folder. I have installed graphviz and now it's working fine. Thank you for helping and sorry for not noticing this before.


Can you provide the data preprocessing code？

This repo is an incredibly intriguing work and I am eager to replicate your experimental results starting from the raw check-in data. I would appreciate more details on how you preprocess those raw check-in data. If you can share the data preprocessing code, that would be immensely helpful.

Thank you for your interest in our work. The data provided in the repository has already been preprocessed, so no additional processing is required. If you want to use your own dataset, please refer to GetNext, as we follow their preprocessing method. However, as far as I know, they have not open-sourced the code for preprocessing the data.


Update title on save

Sets placeholder as title on first visualisation save avoiding empty name warning

Updated naming flow, please check the changes.


MaterialApp.title doesn't affect iOS title

From the Flutter docs on MaterialApp.title, this property is used to identify the app:

A one-line description used by the device to identify the app for the user.
On Android the titles appear above the task manager's app snapshots which are displayed when the user presses the "recent apps" button. Similarly, on iOS the titles appear in the App Switcher when the user double presses the home button.

On Android, this works as intended. But on iOS, the name used in the App Switcher is the name of the project, even when this property is set.
Repo

From a command line, run flutter create materialtitle
In a code editor, edit lib/main.dart so that MaterialApp.title is initialized to foobar
Run flutter run -d iPhone to launch the app in a simulator
Press Command+Shift+H twice in quick succession to launch the App Switcher
Expected: App Switcher shows foobar. Actual: App Switcher shows materialtitle

Environment
$ flutter doctor -v
[✓] Flutter (Channel master, v0.2.5-pre.57, on Mac OS X 10.13.3 17D102, locale en-US)
    • Flutter version 0.2.5-pre.57 at /Users/timsneath/flutter
    • Framework revision c5288c7346 (2 hours ago), 2018-03-28 08:32:27 -0700
    • Engine revision dd6f46c485
    • Dart version 2.0.0-dev.41.0.flutter-2f68e82526

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/timsneath/Library/Android/sdk
    • Android NDK at /Users/timsneath/Library/Android/sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.4.0

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] VS Code (version 1.21.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.11.0

[✓] Connected devices (2 available)
    • Android SDK built for x86 • emulator-5554                        • android-x86 • Android 8.1.0 (API 27) (emulator)
    • iPhone 7                  • 81F64B40-5DDD-45CF-A56E-E816369DA5CB • ios         • iOS 11.2 (simulator)

• No issues found!

cc @xster

cc @mklim @GaryQian in case one of you wants to tackle this before @xster gets back.

This eventually traces down to a deliberate non-op with a comment saying that the functionality is Android-only. Looking into this to see if and how it could be added in for iOS, or if it's the docs that need to be updated to specify that this does nothing in iOS.

I don't think it's possible to alter the title of an app in the Switcher in iOS dynamically at runtime, like the original code comment indicated. It looks like this is completely controlled by CF(Bundle)DisplayName, from the app's Info.plist. I'll update the documentation to clarify the limitation.


Japanese Translation: Set the policyId field to default so that Azure Storage will define the policy ID.

[Enter feedback here]
In English,
Set the policyId field to default so that Azure Storage will define the policy ID.
In Japanese,
Azure Storage でポリシー ID が定義されるように、policyId フィールドを既定値に設定します。
It sounds like Set the policy field to the default value.
This should be
Azure Storage でポリシー ID が定義されるように、policyId フィールドに default を設定します。

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

ID: 73edc3d1-d0db-0ee3-7928-19bd1e509d55
Version Independent ID: 4b0e8f7c-591c-c5c4-37f4-e5a47914b873
Content: Configure object replication - Azure Storage
Content Source: articles/storage/blobs/object-replication-configure.md
Service: storage
Sub-service: blobs
GitHub Login: @tamram
Microsoft Alias: tamram

@himachida Thanks for the feedback! We are currently investigating and will update you shortly.

@SumanthMarigowda - could you please assign this issue to the localization team? thanks!

Thank you for your dedication to our documentation.
We sincerely apologize for the delayed response. After a careful review, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Gradle executor is not detected if only build.gradle.kts file is present

https://github.com/pulumi/pulumi-java/blob/af0e43990042b9649a8f04a4aed7902bce8e8212/pkg/internal/executors/executor_gradle.go#L88-L92
The code above is used to detect an executor, and if I have only build.gradle.kts file in my project, then it falls through to sbt executor
Also, this function might be worth looking at as well:
https://github.com/pulumi/pulumi-java/blob/af0e43990042b9649a8f04a4aed7902bce8e8212/pkg/internal/executors/executor_gradle.go#L58-L62

Thank you for noting this. Would you be interested in contributing a PR? Perhaps something as easy as adding build.gradle.kts  to the markers list would work?

I have run into this problem in a few projects now, and in most cases adding an empty settings.gradle.kts was doing the trick. However, the problem now is that for a multi-module project these settings.gradle.kts are causing Gradle to think that this a root module (while in fact it's sub-module of a multi-module project).

Facing the same problem within a monorepo, where all the Gradle stuff is under /root and the infrastructure is in /infra
An empty settings.gradle.kts did not help and produces

Error(error: failed to discover plugin requirements: language host could not determine Pulumi packages: could not find ./gradlew, gradle on the $PATH: exec: "gradle": executable file not found in %PATH%

indeed, an empty settings.gradle.kts helps as it is part of the gradleRootMarkers posted
without it the error is as follows:

error: failed to discover plugin requirements: language host could not determine Pulumi packages: could not find ./sbt, sbt on the $PATH: exec: "sbt": executable file not found in $PATH


Error: Unable to resolve action cloudflare/pages-action@1, unable to find version 1

Hi! Testing this out for the first time, hitting some bumps:
Current runner version: '2.291.1'
Operating System
Virtual Environment
Virtual Environment Provisioner
GITHUB_TOKEN Permissions
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Error: Unable to resolve action `cloudflare/pages-action@1`, unable to find version `1`

I attempted removing the version number, but got:

the `uses' attribute must be a path, a Docker image, or owner/repo@ref

Then setting version 1.0.0, but:

Unable to resolve action<EMAIL_ADDRESS>unable to find version 1.0.0

Ahh! The correct syntax for owner/repo@ref seems to be:
# ...
  - name: Publish
     uses<EMAIL_ADDRESS>     # ...

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsuses

The docs over at https://developers.cloudflare.com/pages/how-to/use-direct-upload-with-continuous-integration/#use-github-actions are also incorrect on this point 👍

To maintain the intention of locking only the major version, the ref should be @v1 which is resolved to the latest version under the specifier major.
Check https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-using-versioned-actions

I did try this initially, but it errors with:

Unable to resolve action cloudflare/pages-action@v1, unable to find version v1

You are correct in that it would still require the team to tag the release commit with v1 as well as v1.x.x - which is the convention for most actions.

Aha! Thanks for clarifying!
Long story short, as it stands, only one config currently works, and it's different from both sets of docs:

✅ @v1.0.0
❌ @v1 (requires additional release tagging) per cloudflare docs
❌ @v1.x.x
❌ @1 per README.md
❌ @1.0.0

@GregBrimble, reckon you could tag each release for its major version, too?

Error: Unable to resolve action<EMAIL_ADDRESS>unable to find version v1.x.x

Hey folks, sorry about that. We've just switched over to 1.0.0 and 1 rather than v1.0.0 to make things a bit easier for everyone. And I've just put up a PR to update the docs. If you're still having difficulty referencing the Action with uses: cloudflare/pages-action@1, please give let us know.

Works for me, thanks 🙏


multiple selects don't work in the elements plugin

In the elements plugin:

the multiple select only posts one value
when clicking back the selected array is not updated and it is added to every time.

this can be tested by using the default emy form post function (only posts one value)
if you change the form method and add target="_self, you can see the multiple values.
if you click back and change the values these are not representative of the selected values

after a bit more investigation looks like there are two issues.

the emy form parser doesn't seem to cope well with multiple selects, in fact it only selects the first value. To replicate go here -> link . Select all the values and submit the form. Only one value is displayed.

the multiple select just doesn't work - select all six items in the multi select list here - link . once selected, unselect them all, go back (the list displays nothing) and submit the form. you will see values have been submitted.

fixed Issue 1.
added in a check for a form multiple:
if (input.getAttribute("multiple")) {
	args[input.name] = getSelectValues(input);
}
else if (value === undefined) { // If parm is 'empty' just set it
	args[input.name] = input.value;
} else if (value instanceof Array) { // If parm is array, add to it
	value.push(input.value);
} else { // If parm is scalar, change to array and add to it
	args[input.name] = [value, input.value];
}

and added function top parse the multiple values
	function getSelectValues(select) {
	  var result = [];
	  var options = select && select.options;
	  var opt;

	  for (var i=0, iLen=options.length; i<iLen; i++) {
	    opt = options[i];

	    if (opt.selected) {
	      result.push(opt.value || opt.text);
	    }
	  }
	  return result;
	}

retested issue 2:

works on ios/android
doesn't work on any desktop browser

works for targetted device (so good), but makes it hard to test on desktop....
Any ideas why values once selected remain selected (even when unselected) on desktop gladly received.....


Python with UR3e robot and Moveit2

Hi everyone!
I'm using Ros2Humble, in ubuntu 22.04, and I was wondering if any of you know how to code a launcher and a executable file to move the robot in python. I have this doubt because I only know how to code this in C++ and I didn't find any info about python yet.
Thanks in advance

Sorry, but this seems to be in no relation to any issue regarding this driver.
To ask general ROS implementation questions, please use https://robotics.stackexchange.com/

Why it is not related @fmauch ? I'm 100% that ROS2 can be programmed with python. And I also think, ( not 100% sure) that this driver implemented in ROS1 also can be done with python. I'm just asking if in this repo, with your controllers and all that stuff can be done with python. I see a very clear relation.

@marioesteban11 I do not know if you are referring to launch files or nodes. Probably .launch.py files. If so, see the bringup package for detailed examples and everything related to what you need.
launch files in ROS1 are not .py (only ROS2), so i do not think you are correct there either.

It's not related, since your question is about using this driver (even through MoveIt!). This driver provides common interfaces to be integrated into applications. Writing those applications is discoupled from the driver itself. This issue tracker is about issues with the driver. If anything from this driver is malfunctioning, please open an issue here. For questions regarding your ROS application, this is simply not the right place. I hope you understand, that we try to focus our work here on the driver itself and cannot give general ROS support.


Does it support IOS and Android?

I want to use it in my project immediately, but I don't know if ios and android are supported?

Hey! I never tested before. Please, if you can test it, let me know if it
works.
Thank you for your suggestion.
Be in mind that ImGui has some issues in your core related to mobile
On Fri, 14 Jul 2023 at 07:46 Personuo @.***> wrote:

Assigned #50 https://github.com/psydack/uimgui/issues/50 to @psydack
https://github.com/psydack.
—
Reply to this email directly, view it on GitHub
https://github.com/psydack/uimgui/issues/50#event-9820622594, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHK3M5NACVHRSAOL3P7VJLXQEPRZANCNFSM6AAAAAA2KFFIGY
.
You are receiving this because you were assigned.Message ID:
@.***>

I've just tested on android. It doesn't render anything. Later I'll check the logs

So the error is in loading cimgui.dll on android:
2023/08/11 16:37:47.683 18317 18338 Error Unity DllNotFoundException: Unable to load DLL 'cimgui'. Tried the load the following dynamic libraries: Unable to load dynamic library 'cimgui' because of 'Failed to open the requested dynamic library (0x06000000) dlerror() = dlopen failed: library "cimgui" not found
2023/08/11 16:37:47.683 18317 18338 Error Unity   at ImGuiNET.ImGuiNative.igCreateContext (ImGuiNET.ImFontAtlas* shared_font_atlas) [0x00000] in <00000000000000000000000000000000>:0
2023/08/11 16:37:47.683 18317 18338 Error Unity   at UImGui.UImGuiUtility.CreateContext () [0x00000] in <00000000000000000000000000000000>:0
2023/08/11 16:37:47.683 18317 18338 Error Unity   at UImGui.UImGui.Awake () [0x00000] in <00000000000000000000000000000000>:0

Same here works flawlessly inside the editor but won't render any uimgui once build ( app runs fine on its own ) tried including the .so with x86 & armv7 file from ImGui.NET-nativebuild/releases ( although I suspect the versions are way off ) , tried both mono and cpp build but still no luck , not very versed with native plugins , maybe someone could figure it out here are a few links I came across : DllNotFoundException on Android , Create a .jar from a .dll

Also tried building my own .so file without any luck so far

Disclaimer: this is a guess.
I read the description of what you did by the link in your last comment, and I think it won't work like that in Unity.
The problem is likely that the code that requires the dll/so is inside the already built ImGui.NET.dll.
I suspect that in order for this to work, all the [DLLImport] statements should be resolved and compiled directly by Unity itself.
If this guess is correct then you might probably have success with:

Copying the source form of the ImGui.NET, including the generated code into the Unity project
Copying the source code of uimgui (this project) into your Unity project
Reorganizing the positioning of cimgui according to the default Unity pattern of multiplatform native libraries (see the article cited below)
Maybe doing something similar to all the other dlls used by uimgui

In that case the whole project, including the linking information for the native libraries is going to be produced and managed by Unity, which should allow it to correctly include the libraries in the build, and be able to resolve them in runtime.
A brief article with generic steps, just for reference: https://matiaslavik.wordpress.com/2021/01/21/cross-platform-native-plugins-in-unity/

Disclaimer: this is a guess.
I read the description of what you did by the link in your last comment, and I think it won't work like that in Unity. The problem is likely that the code that requires the dll/so is inside the already built ImGui.NET.dll. I suspect that in order for this to work, all the [DllImport] statements should be resolved and compiled directly by Unity itself.
If this guess is correct then you might probably have success with:
* Copying the source form of the ImGui.NET, including the generated code into the Unity project

* Copying the source code of uimgui (this project) into your Unity project

* Reorganizing the positioning of cimgui according to the default Unity pattern of multiplatform native libraries (see the article referenced below)

* Maybe doing something similar to all the other **native** dlls used by ImGui.NET and uimgui

In that case the whole project, including the linking information for the native libraries is going to be produced and managed by Unity, which should allow it to correctly include the libraries in the build, and be able to resolve them in runtime.
A brief article with generic steps, just for reference: https://matiaslavik.wordpress.com/2021/01/21/cross-platform-native-plugins-in-unity/

Can you dm me on discord ? same handle as my github username

Hi everyone! I tried to compile cimgui for android arm32/64 as a separate .so file and it worked out of the box.
First of all I've compiled the cimgui for a correct version of imgui used in this repo 1.90.1:

Cloned ImGuiNet-nativebuild for release 1.90.1
Opened cimgui submodule in my terminal (I am using Win11)
Checked that I've android sdk and ndk installed
Compiled cimgui .so for arm32 and arm64 with that command (run that in cimgui/build):

cmake .. -DANDROID_ABI=<armeabi-v7a or arm64-v8a> -DANDROID_PLATFORM=22 (you can use whatever SDK version you want) -DANDROID_NDK="<Android SDK path>\ndk\27.1.12297006" -DCMAKE_TOOLCHAIN_FILE="<Android SDK path>\ndk\27.1.12297006\build\cmake\android.toolchain.cmake" -G Ninja

After that I run another cmake command to compile the .so
cmake --build build --config Release --parallel <cpu count>
After that I renamed cimgui.so -> libcimgui.so and put it to my Unity project
In Unity I've set that it must be used only on Android platform and set the correct architecture

After that I've launched my game and android and got the ImGui window, but it renders with incorrect colors, beside that everything works as good as on desktop

Just a small note I've configured font size and imgui scale for window to be usable size on the device

            var io = ImGui.GetIO();
            var style = ImGui.GetStyle();
            float defaultScale = 1.0f;

            if (Application.platform == RuntimePlatform.Android || Application.platform == RuntimePlatform.IPhonePlayer)
            {
                defaultScale = 2.5f;
            }

            Debug.Log($"[CheatUIManager] Default imgui scale: {defaultScale}");

            io.DisplaySize = new Vector2(defaultScale, defaultScale);
            style.FrameRounding = 0.0f;
            style.FrameBorderSize = 0.5f;
            style.ScaleAllSizes(defaultScale);

            var font = io.Fonts.Fonts[0];
            font.Scale = defaultScale;

@psydack Do you have any info on how to fix that incorrect colors on android (I am using only OpenGL unity backend) and it seems theirs a problem with the transparency

can you share the .so file ?

libs.zip
Here you go, arm32 and arm64 versions attached

I've tested this with my project and it's pretty much plug and play! The colors are indeed a bit off but it's working.
Thank you @rbetik12 for providing the .so files!

Now we need someone to test that on iOS

apk on the Quest-2 crashes , no unity errors / logcat doesn't show any errors - no clue how to check what's going on

I guess you can ry compiling .so files by yourself, using specific SDK Quest-2 uses

Have you tried disabling in Project Settings > Player > Other Settings > Script define symbols > Apply > Restart Unity Editor.
UIMGUI_REMOVE_IMPLOT
UIMGUI_REMOVE_IMNODES
UIMGUI_REMOVE_IMGUIZMO
Also do you have any callstacks related to crash?


New Release Requestment

Hello:
Firstly, Thanks to you contributors for the efforts.
Secondly, The last release is built on March,2021. A newer release is advised to be announced.
Thanks.

Thanks for reminder. Done.


fixing syntax part 2

Looks like the automated build check is not passing.

Yep, working on fixing it now.

If you wish, I can get up a pre-commit hook so that you can't commit if there is a syntax error.


Brain mask nii derivative file filled with NaN values

What version of fMRIPrep are you using?
20.2.1
What kind of installation are you using? Containers (Singularity, Docker), or "bare-metal"?
Docker
What is the exact command-line you used?
fmriprep-docker
--fs-license-file "/usr/local/freesurfer/license.txt"
"${input_dir_fmriprep}"
"${preprocess_dir}"
participant
--aroma-melodic-dimensionality=-200
--bold2t1w-dof=6
--bold2t1w-init=register
--dvars-spike-threshold=1.5
--fd-spike-threshold=0.5
--fs-no-reconall
--ignore fieldmaps slicetiming
--mem_mb="${mem}"
--n_cpus="${cpus}"
--output-spaces MNI152NLin2009cAsym anat
--skull-strip-t1w=force
--skull-strip-template=OASIS30ANTs
--use-syn-sdc
--verbose
--notrack
--participant-label "${subject}"
--bids-filter-file "${working_dir}/filter.json"
-w "${output_dir_fmriprep}/fmriprep_working"
Have you checked that your inputs are BIDS valid?
Yes
Did fMRIPrep generate the visual report for this particular subject? If yes, could you share it?
Yes it did.
Attached as pdf
Can you find some traces of the error reported in the visual report (at the bottom) or in crashfiles?
No
<please copy&paste all the information you can gather about errors>

### Are you reusing previously computed results (e.g., FreeSurfer, Anatomical derivatives, work directory of previous run)?
No

### fMRIPrep log
If you have access to the output logged by fMRIPrep, please make sure to attach it as a text file to this issue.
Attached
[fmriPrepLog_B001_fromYann.txt](https://github.com/nipreps/fmriprep/files/6972205/fmriPrepLog_B001_fromYann.txt)

<!--
List the steps you performed that revealed the bug to you.

I found that downstream steps that require the derivative file ending with brain_mask.nii.gz are failing, and then opened it in an nii viewer and found that it contained only NaN values. I found this across multiple runs with this machine and this Docker, but not on another machine that also uses fmriprep-docker.
I am not sure if this qualifies as a bug but cannot find any reason to why it is happening
The functional data output , sub-B001_ses-baseline_task-resting_space-T1w_desc-preproc_bold.nii.gz looks good.

Include any code samples. Enclose them in triple back-ticks (```)
Like this:

```
-->
[fmriPrepLog_B001_fromYann.txt](https://github.com/nipreps/fmriprep/files/6972212/fmriPrepLog_B001_fromYann.txt)
[B001_visualReport.pdf](https://github.com/nipreps/fmriprep/files/6972214/B001_visualReport.pdf)

Hi Adimaron,
I have the same problem. Brain masks derivatives are empty. I cannot even open them with MRICron. My final report does not indicate any errors. Have you solved the problem

In my processing stream I have noticed a WARNINGS where the brain_mask.nii.gz files are transformed from float32 to uint8. This is the only warning that I get for those .nii images (only brain masks) that I cannot open. I do not know if this is a bug or what

Hi,
We noticed this also on our processed brain_mask.nii.gz images with fmriprep v20.2.2. We are able to display them with fsleyes but this presence of NaN instead of zeros makes SPM crash.
However when we preprocessed the same images with fmriprep v20.2.1, we did not noticed the same behavior, we had no issue to use SPM on them.
For now, we add this line in our script as a small patch on this issue:
fslmaths in_image -nan out_image

 -nan   : replace NaNs (improper numbers) with 0
 -nanm  : make NaN (improper number) mask with 1 for NaN voxels, 0 otherwise

Hi,
There is a bug, which will be fixed in the next release. Please, look at the link below:
https://github.com/nipreps/fmriprep/issues/2507#issuecomment-908662822
Loris

Fixed in 20.2.4.


fix autoconfig_exclude_features / autoconfig_include_features to match the code

What does this PR do?
Fix autoconfig_exclude_features / autoconfig_include_features parameters to fit the code.
Motivation
There is a difference between the values that can be specified in the code and in the configuration file.
https://github.com/DataDog/datadog-agent/blob/191c12924e3d207ce99cd74932d202ee64c56920/pkg/config/env/environment_container_ features.go
Reviewer's Checklist.

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

I couldn't add changelog/no-changelog label, so would someone add this label?

/merge


Apply publish-plugin to automatically close sonatype repository on re…

…lease.
Off chance that it will fix the mysterious missing snapshot builds (they work fine for the similarly configured aws-otel-java-instrumentation). But add --info to CI build to debug in case the problem persists.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@willarmiros Can you check this PR out? Thanks!

this will now also auto-publish the artifacts from the staging repo in Nexus without the need to manually log in and release them?

Yup


Sync/Implement kyle3wynn's Audio Updates

@kyle3wynn has updated a bunch of audio files and added some new ones. These need to be brought across from the assets repo and/or implemented.

Whoops, wrong issue!

Let's call this closed by #97


Bug: Status, Category on contact needs more help text to understand

Create a Resource
Status - i understood that it is  resource status that it should be under development as i was creating the resource
Contact - Category in contact is not self explanotry on possible values

So i expect that there is some text that explains on what it is

fixed in altinn-studio repo


How to split a heterogeneous graph for link_pred targeting specific edge_type?

Hi
I want to split a heterogeneous graph into train, dev, and test set for link_pred, but I need to have the positive and negative instances of the edges/links from a specific edge_type. Predicting that kind of edge is the problem description, other edges are just informative. Is there a way to do that with split?
Thanks,
Soha

I might be wrong but I think deepsnap’s splitting functionality will give you positive and negative supervision edges for all edge types in a hetero graph. If you want to specify one edge type for your supervision edges maybe have a look at pytorch-geometric’s RandomLinkSplit, see for example how it’s used here. In the same folder there is also a hetero_link_pred example which I contributed, so I hope it doesn’t seem like showing off that I share it😅,  it just seems fairly relevant to what you’re trying to do).

Thanks @anniekmyatt both RandomLinkSplit functionality and the code example are very useful for me.