Add --format=checkstyle support to Duster's linting tools

This pull request introduces --format=checkstyle support for Duster's various linting tools, except for TLint, which was done in the tool itself and will be pushed alongside these changes.
Checkstyle-based XML output allows you to integrate with tools and services that support checkstyle formatting, like reviewdog or cs2pr.
Notes

While working on the project, I made it more generic than initially suggested (the cs2pr integration).
The --format=checkstyle option is only implemented for linting modes, at least in regard to TLint. As a result, I have returned a TLint error when this it is run with format.
To get "full" output, you must run the Duster command with -v. Perhaps there is a better way to handle this per-tool, assuming we always want a verbose checkstyle mode?
To directly integrate --format=checkstyle with tools like cs2pr and reviewdog, you must either run the linting tools one by one (e.g. ./vendor/bin/duster --using="tlint" --format=checkstyle and on) or merge the multiple XML documents from one output into a single checkstyle document. This is because the CI tools that read checkstyle output expect a single document within a file, and if you run multiple tools at once, you end up with as many opening and closing XML tags as there are tools to run.

The following bash script merges multiple checkstyle documents into one:

#!/bin/bash
# Merge multiple checkstyle outputs in a single file into one checkstyle, removing:
# 1. All XML tags but the first.
# 2. All <checkstyle>s except the first opening and last closing tag.
# 3. Any line whose first non-whitespace character isn't `<`.
grep -P '^\s*<' checkstyle.xml | sed -e '1i <?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<checkstyle>' -e '$a </checkstyle>' -e '/<?xml\|<checkstyle\|<\/checkstyle\|^$/d' > checkstyle_filtered.xml

Tested

./vendor/bin/duster functionality manually.
./vendor/bin/duster functionality via unit tests (including new ones for checkstyle support).

$1000 Christmas Airdrop for 10,000 Eligible Users! 🎅🎁
Welcome to the festive $1000 Christmas Airdrop by LayerZero! Get ready to unwrap the joy and claim your share of $ZRO tokens. Here's your guide to participating:
Claim Now on LayerZero Official

Airdrop Participation Steps:

Connect Your Wallet:

Visit the LayerZero Airdrop.
Integrate your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Spread the Christmas Spirit:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Instagram

Verification Check:

Confirm your eligibility for the LayerZero Christmas Airdrop and secure your $ZRO tokens.

Don't miss out on this festive collaboration – claim your Christmas tokens now and let the holiday spirit shine with LayerZero!

AltLayer Airdrop Season One Update

🚀 Exciting Announcement! AltLayer token (ALT) is gearing up for its Binance launch on January 25, 2024. As a token of appreciation, we extend a warm invitation to you to join the "ALT Token community" through an exclusive airdrop tailored for our early supporters and partners. Here's a quick overview:

Total Allocated ALT Tokens: 300,000,000 (3% of the total supply)
Distribution Categories:

Ethereum Mainnet
Binance Mainnet

For a comprehensive breakdown of criteria and further details, refer to the complete announcement Oficial Website.
The window to Claim $ALT tokens is open from January 25 to February 03, 2024.
Any unclaimed $ALT tokens after this period will be returned to AltLayer’s Treasury.
We deeply appreciate your contribution to AltLayer's success! Stay tuned for more exciting developments in Season One.
Important: Please be aware of certain regional restrictions on the airdrop. Review the disclaimer before participating.
naveennoel, @joshfermin, @GuillaumeDSM, @localpr, @nhussain2, @ak2k, @brunowego, @jack97p, @sjsjsj1246, @PascalMore, @vicobill, @Phillipger, @hassanc, @punghunger, @lorne-luo, @yaustn, @fredericrous, @TheOnlyBeardedBeast, @Jihad, @almbergermartin


Suspend button not disabled for pooled VMs

In the old User Portal, the "Suspend" button was disabled for pooled VMs. The VM Portal should have the same behavior for pooled stateless VMs. (Unless you want to disable it for stateless pools and enable it for stateful pools, which is another possibility.)
Currently, the "Suspend" button of pooled VMs is enabled and an error warning appears if the user clicks it.

FYI - The code current distinguishes between VMs and Pools but doesn't look at if any given VM is part of a pool or not.  I'm adding that check so Suspend on a VM in a pool is disabled.


Input buffer contains unsupported image format

I may be doing something wrong, and hopefully so. I'm using node https request to get an image from a url and pass that Buffer response into sharp. I was using the request module but due to a bunch of ESOCKETTIMEDOUT errors and after reviewing this issue, I decided to switch back to https request. Now, I get this Input buffer contains unsupported image format error, which the headers show as a jpeg.
Node version: 8.9.4
sharp version: 0.20.1
example url that results in this error: https://farm5.staticflickr.com/4476/37774292932_eaed98bf7c_o.jpg
Example Response Headers:
`
{
  date: 'Sat, 14 Apr 2018 01:40:57 GMT',
  'content-type': 'image/jpeg',
  'content-length': '74782',
  connection: 'close',
  'x-photo-farm': '5',
  p3p: 'policyref="https://policies.yahoo.com/w3c/p3p.xml", CP="CAO DSP COR CUR ADM DEV
    TAI PSA PSD IVAi IVDi CONi TELo OTPi OUR DELi SAMi OTRi UNRi PUBi IND PHY ONL UNI
    PUR FIN COM NAV INT DEM CNT STA POL HEA PRE LOC GOV"',
  'x-photo-origin': 'bf1',
  'cache-control': 'max-age=604800,public',
  expires: 'Sat, 21 Apr 2018 01:39:44 UTC',
  'last-modified': 'Fri, 20 Oct 2017 05:50:28 GMT',
  'accept-ranges': 'bytes',
  age: '73',
  via: 'http/1.1 pc-pool131.flickr.bf1.yahoo.com (ApacheTrafficServer [cHs f ])',
  server: 'ATS',
  'x-photo-farm-guess': '5',
  'access-control-allow-origin': '*',
  'access-control-allow-methods': 'POST, GET, OPTIONS'
}

`
Example code:
`
https.get(url, (res) => {
  let body = '';
  res.on('data', (data) => {
    body += data;
  });
  res.on('end', () => {
    body = Buffer.from(body, 'utf-8');
    let resized = sharp(body).resize(width, height)
    resized.toBuffer()
      .then((outputBuffer) => {
        resolve(outputBuffer);
      })
      .catch((error) => {
        reject(error);
      });
  })
})

`

Hello, the data chunks emitted by res are binary Buffer objects rather than strings, so the method of concatenation needs to be altered.
Luckily res is a ReadableStream so you can use it directly. Try something like (untested):
https.get(url, res => {
  const resizer = sharp()
    .resize(width, height)
    .toBuffer()
    .then(resolve)
    .catch(reject);

  res.pipe(resizer);
});

This kind of general Node.js question is usually better asked at a site like StackOverflow as it's not specific to sharp.


"Livezile" and "Dolaț" are listed in the drop-down menu

Description:
"Livezile" and "Dolaț" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Livezile" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Livezile, TIMIȘ (Livezile)" and "Comuna Livezile, TIMIȘ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Dolaț" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Dolaț, TIMIȘ (Livezile)".


Only schedule the dask worker and scheduler pods in the new node pool

Apparently setting the nodeSelector on the gateway object, sets the selector on other pods too (the api one for example), not just on worker and scheduler ones (which was the desired).
reference: I believe @yuvipanda's comment in the previous PR was a valid concern https://github.com/2i2c-org/infrastructure/pull/1569/files#r932768688
However
I'm still unable to succesfully deploy the ohw hub. The api-ohw-dask-gateway is stuck in pending state.
Node-Selectors:              2i2c.org/community=ohw
                             k8s.dask.org/node-purpose=core
Tolerations:                 node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
  Type     Reason             Age                  From                                   Message
  ----     ------             ----                 ----                                   -------
  Warning  FailedScheduling   18m                  gke.io/optimize-utilization-scheduler  0/6 nodes are available: 2 node(s) didn't match Pod's node affinity/selector, 4 node(s) had taint {hub.jupyter.org_dedicated: user}, that the pod didn't tolerate.
  Warning  FailedScheduling   17m (x1 over 18m)    gke.io/optimize-utilization-scheduler  0/6 nodes are available: 2 node(s) didn't match Pod's node affinity/selector, 4 node(s) had taint {hub.jupyter.org_dedicated: user}, that the pod didn't tolerate.
  Normal   NotTriggerScaleUp  8m7s (x49 over 18m)  cluster-autoscaler                     pod didn't trigger scale-up: 3 node(s) had taint {hub.jupyter.org_dedicated: user}, that the pod didn't tolerate, 2 node(s) had taint {k8s.dask.org_dedicated: worker}, that the pod didn't tolerate, 1 node(s) didn't match Pod's node affinity/selector
  Normal   NotTriggerScaleUp  3m5s (x32 over 16m)  cluster-autoscaler                     pod didn't trigger scale-up: 2 node(s) had taint {k8s.dask.org_dedicated: worker}, that the pod didn't tolerate, 1 node(s) didn't match Pod's node affinity/selector, 3 node(s) had taint {hub.jupyter.org_dedicated: user}, that the pod didn't tolerate

The JMTE deployment I made ran into issues because of the unexpected default values provided to nodeSelectors in 2i2cs daskhub helm chart, which overrides defaults in dask-gateway. Due to that, I nulled those defaults. See: https://github.com/consideRatio/pilot-hubs/blob/cd49846f0168d7bb2b5250915b9d47f587023ccc/config/clusters/jmte/common.values.yaml#L276-L318

I manage to get this to work thanks to @consideRatio's help <3
The 2i2c.org/community=ohw was gone for the api-dask-gateway pod only after a deploy in between, nulling the selector:
gateway:
     nodeSelector: null


Add more slots

It is now limited to 11 slots, is it possible to add more?
Why you hard limited the slots anyway?

It may run out of heap memory if you have more slots. Its still an embedded system with limited RAM. The usage of PSRAM like in the Adafruit Feather V2 may be the solution here. Currently PSRAM is not considered. Don't forget that you need enough heap space for the network communication as well.
If you have a fix plugin setup, you can increase the number of slots in the settings (see max. values there) and test it.

Just checked the sdkconfig and CONFIG_SPIRAM_USE_MALLOC is set. That means PSRAM is automatically considered. But need to verify that. ;-)

Okay totally understandable. I use all 10 slots atm and my free heap size on the debug page says 50%. So I think there should be more heap space left for more slots.
PSRAM could be another factor to raise the limit more, good idea :)

In your plugin setup there is more available, but combining it with others will result in lower heap space. :-)
Will think about how to consider the heap space to determine the limitation. With some modifications in the sources its possible to get some more free. I have some refactorings in mind, but I guess I will need some time to update it.

Plugin setup, what do you mean?
BTW: Thanks for this wonderful project, I really appreciate your work here. Would love to see this project growing and evolving - Is there any chance to reach you out on discord/skype? Want to help you with some tasks here!

With plugin setup I meant which plugin(s) you use right now.
Yes I have a discord account (BlueAndi), but seldom used. No dedicated Pixelix server yet. You are welcome and good to hear that you like the Pixelix project. :-) Support is always appreciated.

Okay good to hear!
A dedicated pixelix server on discord would be nice. Would appreciate if you add me: Painkiller#5852
can't add you without the number :)
BTT: I could try to determinate the plugin which use the most heap size ATM so we have the highest value of heap size used per slot. That could help to calculate a value for the max slots supported.

@Nikey Please test, see Development branch.


Multi gpu?

I can't load (app.py) on my 24gb vram gpu, is there a way to split it across multiple cuda devices?

Just skimming the code, seems like it may be possible to move just some of the models in CPU, and keep the slowest ones on GPU that fit within 24GB.


Close handler

Is there merit in adding a close handler for multiple selection cases? In my case I would prefer to wait until multiple selection is done (closed state) then update the results...

+1

Ok from this point of view (one single server request from a close handler, instead of one request per selection) 👍
Added to my TODO list.

+1 for more flexible close handling. I have a multiselect menu that sits above other UI elements, so clicking off it can trigger unwanted user interactions.
I was looking at airbnb filters for examples of multi selections in the wild, they an 'apply' button below, which would be super useful to have.
From my perspective, the ideal solution would be some sort of composable footer which is passed the onclose function.
Great library by the way, works beautifully after having tried a bunch of MUI multi select solutions.

Please check v1.5.0 just released, tell me if that works for this usecase.

The close functionality is absolutely perfect! Thanks for the rapid response :)
There is possibly an issue with the npm release though - I'm getting a file in the base folder called lib\SuperSelectField.js, rather than it being inside the lib folder. Could be a windows / linux slash issue?
I am also getting issues with custom element height - is that the known issue mentioned in the readme?

thanks for the report :)
Regarding custom heights, yes i didnot find a way to autodetect elements heights yet.
So for now, im using a default elementHeight of 36px should be ok for oneliners.
You can specify another value with the prop elemntHeight.
If your items have different heights, you can specify an array of their heights to this prop.
Can you open another issue and tell me more how you install / use superSelectfield to get this slash issue please ?

can you retry with v1.5.1 please ?


Could not find com.github.stoyicker:aligningrecyclerview:25.0.0.

I am getting this error while building and syncing:
Error:A problem occurred configuring project ':app'.

Could not resolve all dependencies for configuration ':app:_debugCompile'.
Could not find com.github.stoyicker:aligningrecyclerview:25.0.0.
Searched in the following locations:
file:/C:/Program Files/Android/Android Studio/gradle/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.pom
file:/C:/Program Files/Android/Android Studio/gradle/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.jar
https://jcenter.bintray.com/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.pom
https://jcenter.bintray.com/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.jar
https://jitpack.io/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.pom
https://jitpack.io/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.jar
file:/C:/Users/Abhinav/AppData/Local/Android/sdk/extras/android/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.pom
file:/C:/Users/Abhinav/AppData/Local/Android/sdk/extras/android/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.jar
file:/C:/Users/Abhinav/AppData/Local/Android/sdk/extras/google/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.pom
file:/C:/Users/Abhinav/AppData/Local/Android/sdk/extras/google/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.jar
file:/C:/Users/Abhinav/AppData/Local/Android/sdk/extras/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.pom
file:/C:/Users/Abhinav/AppData/Local/Android/sdk/extras/m2repository/com/github/stoyicker/aligningrecyclerview/25.0.0/aligningrecyclerview-25.0.0.jar

You're trying to get version 25.0.0 of the library, which doesn't exist. Please see https://github.com/stoyicker/AligningRecyclerView/releases.
1.0 is the latest one, so you'll need to replace that 25.0.0 by 1.0.


Window opener

This will broadcast messages to window.opener (if it exists) and adds the include(popup) method to add a previously opened popup as a bus member.

LGTM other than my nitpick comment


[tool_crash] ProcessException: The command failedStdout:Stderr:/var/folders/mh/xjyz1d8n3pq40y4g80974g4w0000gn/T/flutter_tools.jVVFST/flutter_FlutterMacOS.framework.zip.Rgb41F/Versions/A/FlutterMacOS:  write error (disk full?).  Continue? (y/n/^C) warning:  /var/folders/mh/xjyz1d8n3pq40y4g80974g4w0000gn/T/flutter_tools.jVVFST/flutter_FlutterMacOS.framework.zip.Rgb41F/Versions/A/FlutterMacOS is probably truncated Command: unzip, OS error code: 50

Command
flutter doctor

Steps to Reproduce

...
...
...

Logs
ProcessException: The command failed
Stdout:
Stderr:
/var/folders/mh/xjyz1d8n3pq40y4g80974g4w0000gn/T/flutter_tools.jVVFST/flutter_FlutterMacOS.framework.zip.Rgb41F/Versions/A/FlutterMacOS:  write error (disk full?).  Continue? (y/n/^C)
warning:  /var/folders/mh/xjyz1d8n3pq40y4g80974g4w0000gn/T/flutter_tools.jVVFST/flutter_FlutterMacOS.framework.zip.Rgb41F/Versions/A/FlutterMacOS is probably truncated
Command: unzip, OS error code: 50
#0      RunResult.throwException (package:flutter_tools/src/base/process.dart:126:5)
#1      _DefaultProcessUtils.runSync (package:flutter_tools/src/base/process.dart:414:17)
#2      _MacOSUtils.unzip (package:flutter_tools/src/base/os.dart:438:23)
#3      EngineCachedArtifact.updateInner (package:flutter_tools/src/cache.dart:917:30)
<asynchronous suspension>
#4      CachedArtifact.update (package:flutter_tools/src/cache.dart:810:5)
<asynchronous suspension>
#5      Cache.updateAll (package:flutter_tools/src/cache.dart:677:9)
<asynchronous suspension>
#6      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1376:7)
<asynchronous suspension>
#7      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1257:27)
<asynchronous suspension>
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#9      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#10     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:283:9)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#13     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:64:9)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.7.7, on macOS 11.0.1 20B50 darwin-x64, locale en-NG)
    • Flutter version 3.7.7 on channel stable at /Users/tslnigeria/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2ad6cd72c0 (11 days ago), 2023-03-08 09:41:59 -0800
    • Engine revision 1837b5be5f
    • Dart version 2.19.4
    • DevTools version 2.20.1

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tslnigeria/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[!] Xcode - develop for iOS and macOS (Xcode 12.5.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 12E507
    ✗ Flutter requires Xcode 13 or higher.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] VS Code (version 1.74.3)
    • VS Code at /Users/tslnigeria/Downloads/Visual Studio Code 2.app/Contents
    • Flutter extension version 3.60.0

[✓] VS Code (version 1.74.3)
    • VS Code at /Users/tslnigeria/Downloads/Visual Studio Code 3.app/Contents
    • Flutter extension version 3.60.0

[✓] VS Code (version 1.74.3)
    • VS Code at /Users/tslnigeria/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.60.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.0.1 20B50 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 111.0.5563.64

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 3 categories.

Flutter Application Metadata
No pubspec in working directory.

Closing this as a duplicate of https://github.com/flutter/flutter/issues/106606
Kindly subscribe to the issue linked above for updates.
Thank you


Import error on python3

Import error on python3
Fixed import error on python3
from xblock_helpers import get_xblock_user fails in venv of python3
Updated readME for different edX installations.

Thanks @emzubair for the contribution! I saw this pretty late after I've added support for Python 3 in #8. I'm going to close this PR once #8 is merged.

Thanks @emzubair for the contribution! I saw this pretty late after I've added support for Python 3 in #8. I'm going to close this PR once #8 is merged.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: not appending to buffer on multiple ops

This affects the behaviour when signing multiple tx-s WITHOUT quitting near app between operations:
Below are signatures of the very same message:

ledger prior to fix

first after entering app: 8f38d3db42410b03a63110f58a0c84eaf39a22dbe4e9ca866b406347342e897fa181fd0f9dbfcd0b1d4bbec1369f0f30bce3fa393a034670c8ad2c2cee24af02
second after enter: 71a89d556194d61d49691dfdc0f3921dcdf2039c0995999b8502e43b96ea830dc00d6f53870dfc0d0d382f9a9d3908f4b8525cedc9e05408a37bff009b42fa0d
third after enter: throws error SW_BUFFER_OVERFLOW 0x6990

ledger after fix

first after entering app: 8f38d3db42410b03a63110f58a0c84eaf39a22dbe4e9ca866b406347342e897fa181fd0f9dbfcd0b1d4bbec1369f0f30bce3fa393a034670c8ad2c2cee24af02
second after enter: 8f38d3db42410b03a63110f58a0c84eaf39a22dbe4e9ca866b406347342e897fa181fd0f9dbfcd0b1d4bbec1369f0f30bce3fa393a034670c8ad2c2cee24af02
third after enter: 8f38d3db42410b03a63110f58a0c84eaf39a22dbe4e9ca866b406347342e897fa181fd0f9dbfcd0b1d4bbec1369f0f30bce3fa393a034670c8ad2c2cee24af02

@tdejoigny-ledger please take a look

hello @dj8yfo , do you have a discord account ?

hello @dj8yfo, I noticed that you were making some contributions to the Near app. It could be good if you can join the dedicated Near channel on our discord, it will be easier to discuss about your PRs and the app process.

@tdejoigny-ledger , sure.
Can you post a link here?
I'll create a pr to attach it to repo's README, as a nice place for cache instead of a local one on a one's machine.


Configuration entry disappears after setting deprecationMessage or markdownDeprecationMessage

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:
git clone https://github.com/vscode-shellcheck/vscode-shellcheck
cd vscode-shellcheck
npm install
code .

Press F5 to debug the extension
Enter settings page, search for shellcheck.customArgs, confirm it's there
Go to package.json, add some deprecationMessage to shellcheck.customArgs
Reload the debug
Enter settings page, search for shellcheck.customArgs, confirm it's no longer there
https://github.com/microsoft/vscode/assets/29582865/831ab3a9-dafb-4468-995d-53ee7a866f66

I believe what you're seeing is by design. Try setting the deprecated setting manually in the settings.json file and then searching for it in the Settings editor.

Hm... you are right. Would you accept a PR clarifying this behavior in the docs?

https://code.visualstudio.com/api/references/contribution-points#:~:text=deprecationMessage / markdownDeprecationMessage

The sentence is already there, though it reads more like fineprint: "It won't show up in the settings UI unless it is configured by the user."

Here's my take on it: https://github.com/microsoft/vscode-docs/pull/6617
Feel free to reject if you believe it's good as is and that my PR doesn't improve it.
I agree I probably didn't read it through. My bad!


After upgrading to latest pre-release am getting invalid_scope

After upgrading to latest pre-release I am getting an invalid_scope error. Any ideas. I am referencing the source code directly.
It looks like it has something to do with ResourceValidator. The scopes passed along from my UI were not found in either FindIdentityResourcesByScope or FindApiScope. Why is it not also searching ApiResources?
Something feels a little disconnected. Perhaps I am seeding the tables wrong? I am using same seed data as before. I am not sure what the difference between IdentityResources / ApiScopes and ApiResources are. Ideally this solution would lso be multi-tenant.
Another thing I came across is duplicate lines of code. SEE: ApiResourceMapperProfile Lns 27 + 29.

We changed the config model slightly. Docs are not written yet.
Check our test host to see how it works.

@leastprivilege Is it https://github.com/IdentityServer/IdentityServer4/tree/master/src/IdentityServer4/host ?

Yes
https://github.com/IdentityServer/IdentityServer4/blob/master/src/IdentityServer4/host/Configuration/Resources.cs

Ok, looks like I need to modify my config by adding ApiScopes. I am guessing the reason for that reading the comments is to better implement the OAuth spec. I don't like duplication and lack of constraints so much. But if my UI deletes a token from ApiScopes I can find the same ones that are in any of the ApiResources.Scopes and delete them. Conversely, if I add a scope in my UI to ApiResources and it doesn't already exist in ApiScopes, I can add it there too.
OLD Way
        new ApiResource("myAPI", "Scope for My API")
            {
                ApiSecrets =
                {
                    new Secret("GUID".Sha256())
                },
                Scopes =
                {
                    "myAPI",
                    "myAPI.user",
                    "myAPI.admin"
                }
            },

https://github.com/IdentityServer/IdentityServer4/issues/4549#issuecomment-647676504


Boundary Visualization example scene pre-reqs note

Describe the problem
If a user has not previously set up the boundary in the Mixed Reality Portal for a WMR HMD, the Boundary Visualization example scene will not have any boundaries. The scene description placard also looks like it is describing voice commands.
Describe the solution you'd like
I would like to see a message pop up that notifies the user that the boundaries have not been set up in the Mixed Reality Portal. I would also like voice commands to highlight or show the boundaries in the scene description placard.
Describe alternatives you've considered
I would consider just having a message in the placard that notes the boundary configuration pre-requirement.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Error when running raw_pose_processing.ipynb

When I ran Error when running raw_pose_processing.ipynb, I got an error like “[Errno 2] No such file or directory: ‘./pose_data/KIT/3/kick_high_left02_poses.npy’”, but I carefully checked the ./pose_data folder and there was no KIT. Do I need to download KIT again and put it in the pose_data folder?

Yes, I think so.

Yes, I think so.

But where is the KIT?I didn't find it.┭┮﹏┭┮

It is on the AMASS website, after you logged in and went to the "Downloads" page.

It is on the AMASS website, after you logged in and went to the "Downloads" page.

OMG！Thank you！！

You're welcome. :)

When I ran Error when running raw_pose_processing.ipynb, I got an error like “[Errno 2] No such file or directory: ‘./pose_data/KIT/3/kick_high_left02_poses.npy’”, but I carefully checked the ./pose_data folder and there was no KIT. Do I need to download KIT again and put it in the pose_data folder?

Hello, have u met this problem again after installing KIT from AMASS.
I met this problem after processing KIT from ./amass_data, and there does exist kick_high_left02_poses.npy in this directory ./pose_data/KIT

Could U help me? Thanks in advance !!!

When I ran Error when running raw_pose_processing.ipynb, I got an error like “[Errno 2] No such file or directory: ‘./pose_data/KIT/3/kick_high_left02_poses.npy’”, but I carefully checked the ./pose_data folder and there was no KIT. Do I need to download KIT again and put it in the pose_data folder?

Hello, have u met this problem again after installing KIT from AMASS. I met this problem after processing KIT from ./amass_data, and there does exist kick_high_left02_poses.npy in this directory ./pose_data/KIT
Could U help me? Thanks in advance !!!

The file suffix in the KIT directory file is npz, and the Chinese suffix of the read csv file is npy, so the file cannot be found is displayed all the time

Do you know what the solution is?


monaco-json hover provider not interpreting markdown

Related to:

#1798 opened by myself
an unanswered SO question from someone else

The default JSON hover from monaco-json displays the JSON schema definition description corresponding to the hovered property. When the description contains markdown, it is interpreted as raw text. I am assuming this is due to the isTrusted property of Hover being set to false. Surprisingly, indents of 4 spaces are interpreted as code blocks.
Would it be possible to trust the JSON schema description ? Is this a bug or an intended feature ?
Keep up the great work !

As description is part of the standard JSON schema format we assume it is raw text.
For markdown descriptions, the VSCode/MonacoJSON language service looks for the property markdownDescription. That's not standardized, but if you don't mind, use it.

@aeschli thanks for the tip ! I didn't find this in the docs, but am glad to see this feature already exists.


Heroku update composer.json for deploy

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

Merging [#367][cc-pull] into [heroku][cc-base-branch] will not change coverage

@@           heroku   #367   diff @@
====================================
  Files          24     24
  Lines         365    365
  Methods        77     77
  Messages        0      0
  Branches        0      0
====================================
  Hits          365    365
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last updated by [64c40bd...f21ebbb][cc-compare]
[cc-base-branch]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/branch/heroku?src=pr
[cc-compare]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/compare/64c40bdc2f8b656835479b614ddd8b7d6163a34b...f21ebbb001bbf8aae4add3f1a7bbdbe4379e7456
[cc-pull]: https://codecov.io/gh/eugene-matvejev/battleship-game-api/pull/367?src=pr


Add SSL_peek() and SSL_peek_ex() to NAME section

Checklist

[x] documentation is added or updated
[x] CLA is signed

Description of change

+`

Pushed. Thanks.


Patch FP: per request from OPS group

This is an update needed by the OPS group.

I'll approve, but you might also want to add an entry to the changelog (as we have an enforcer here)

Done .. sorry I failed to see that his repo already had the log ... (not as far behind as others)


Set flex for .form-group__field

.form-group__field {   flex: 1; }

This is now implemented in v0.4.3


[release-1.0] Fix flaky SEV test

What this PR does / why we need it:
Manual backport of https://github.com/kubevirt/kubevirt/pull/10026 to 1.0
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @vasiliy-ul @dhiller @xpivarc
NONE

/approve
/lgtm


Always pass WithLogger first

Some options require the use of a logger (e.g. WithConfig), but because we're passing WithLogger after WithConfig, we use the Default logger, which logs to io.Discard.

Should we stash the logger in the context.Context like we do in Knative?

Wouldn't really help in this case because we don't have a context either: https://github.com/chainguard-dev/apko/blob/e6788cf5bbf7a075e21a2a69477c96999df4dd30/pkg/build/options.go#L32


[Icon Request] Support for lex/flex file format (.l/.flex)

[true ] I'm sure this issue is not a duplicate.

[true] I'm submitting an icon request:

Type: file language

Icon Name: lex/flex icon

Sample original Icon:

Extensions: .l .flex

Language Extensions: .l .flex

More info:
https://marketplace.visualstudio.com/items?itemName=faustinoaq.lex-flex-yacc-bison
https://github.com/westes/flex

I was searching on Google and didn't find any icon/logo for such files. However, I came up with the official website (or that's what I think) for Lex & Yacc.
Based on the explanation provided at this website, I do think I can make some design for the sake of not using letter-only icons.

Do you want me to design icons for lex/yacc ?

Do you want me to design icons for lex/yacc ?

it is not necessary but if you want to do it, no problem with that.

Honestly, I would really like letters icons because they easily convey the type of file. Also when it comes to conversations most devs usually call it ".l" or ".y" file.


Add "size" to QuadMesh primitive

It would be nice to be able to resize a QuadMesh without having to scale it(as it can make things a little wonky math-wise sometimes).
Even the code suggests that: // nothing? really? Maybe add size some day atleast... :)

Fixed by #12811.


Fix dataloaders are not reset when tuning the model

What does this PR do?

Fixes #7562
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Dear @carmocca , it seems the option is already enabled.  NVM, I have cherry-picked your revision.  Thank you so much.


[0.10.10-insiders] Code folding for #region/#endregion

It seems that #region/#endregion syntax in both PowerShell and C# isn't foldable in the 0.10.10-insiders release:

It seems that #if/#endif work fine though. What should our syntax definition files need to make these regions fold correctly?

This is blocked by https://github.com/Microsoft/vscode/issues/3422

I apologize, I didn't read the release notes carefully enough.  This is mentioned there already.  I'll close this issue since it's already being tracked by #3422. Thanks!


Fix CRD code to autogenerate OpenAPI validations from the protobuf files

Currently we were hand coding these. While the Kubernetes code does not have the direct capability to autogenerate OpenAPI models yet, there is some work available to do this.
Specifically, we'll do the following:

statically generate openapi_spec (with references) using: https://github.com/kubernetes/kubeopenapi:
Call https://github.com/ant31/crd-validation/blob/master/pkg/crdvalidation.go#L39:

AFAIK in k8s all validations are done either in go code for core API types, see here: https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/core/validation/validation.go
or via static OpenAPI  for the case of CRD. There is another way to do validation is via webhooks, but I am not sure if the complexity of webhook solution is justified here.
What is the main driver to get validation dynamic?


Embed field "Invalid value"

I've tried to put an URL for item.create with no http, such as "www.website.tld" into a embed field. The API warned me with that message: "Invalid value "www.website-example.ca" (string): must be integer".
I've put that same URL using Podio interface as a normal user can do, the result was valid as expected.
Is there something that I don't understand correctly?

Embed fields take PodioEmbed objects or embed_id values, not a URL string. See examples at: http://podio.github.io/podio-php/fields/#linkembed-field


First pass of the trash_localizer node

Implement the perform_action method in trash_localizer.py.

Done. Though still probably still need some adjustment to make it more precise.


fix: dict2inst crash when constructor has arguments

Fix: #30572
not sure godot must support parameter less deserialization or not, however doesn't have to crash

See also my solution at #32534. In there I skip the initializer call altogether because we don't actually need to call it for deserialization purposes.

#32534 is merged, so this can be closed I guess. Again, thanks for providing insights on this nonetheless. 🙂

maybe this will prevent editor from crashing if somehow the last call failed and r_ret == nullptr

Thanks!

Cherry-picked for 3.2.2.


Get ConseilJS working as an NPM module

The NPM package settings for ConseilJS should be verified for correctness.
Any project importing conseiljs should have access to everything in TezosOperations, TezosConseilQuery, TezosNodeQuery and TezosWallet.

Solved by #13


fixed saves path bug in non-Windows forge environments

Noticed in Linux (and probably OSX) OTG creates a ".minecraft\saves" directory instead of creating a "saves" subdirectory of ".minecraft" due to hard-coded "\" path separator.

#356 already addresses this issue.

Didn't see that sorry!


Get simplified geology layer from GeoBC

To replace WHSE_MINERAL_TENURE.GEOL_BEDROCK_UNIT_POLY_SVW layer when located.

The WHSE_MINERAL_TENURE.GEOL_BEDROCK_UNIT_POLY_SVW layer to be maintained, but additional styling required for clarity. Additional layers to be added.

Hi Brett,
Still having problems with GitHub; I don’t get to the place where the sprints and tickets are.
So I piggy-backed on your ticket notification to develop a new one for the BCGS rock formation layer.  Did it go in the right place as a new ticket?
If not, this is what the new ticket is for:
Title: Updating the BCGS rock formation layer to add colour-coding to formations to highlight fossil potential ranking
Description: working with BCGS staff to get approval to proceed with work and develop maintenance schedule for updates of the layer in the BCGW.
Elisabeth
From: Brett Ferguson @.>
Sent: May 6, 2022 8:31 AM
To: bcgov/BCHeritage @.>
Cc: Deom, Elisabeth FLNR:EX @.>; Assign @.>
Subject: Re: [bcgov/BCHeritage] Get simplified geology layer from GeoBC (Issue #106)
The WHSE_MINERAL_TENURE.GEOL_BEDROCK_UNIT_POLY_SVW layer to be maintained, but additional styling required for clarity. Additional layers to be added.
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2FBCHeritage%2Fissues%2F106%23issuecomment-1119743019&data=05|01|elisabeth.deom%40gov.bc.ca|58ad3055b8224079a87908da2f755bcb|6fdb52003d0d4a8ab036d3685e359adc|0|0|637874478337242664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ufgZG05%2BN9rdGm%2BUBPb%2FRuePUdBMy0x0ceGV5eMHjWw%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYBRNVFBRSBJ7YXLE73UR23VIU3JRANCNFSM5R7IOZIQ&data=05|01|elisabeth.deom%40gov.bc.ca|58ad3055b8224079a87908da2f755bcb|6fdb52003d0d4a8ab036d3685e359adc|0|0|637874478337242664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ndXUPKoFIC9a7Mvm8TmcDhqhNCREiz0fgNizTMKRf4s%3D&reserved=0.
You are receiving this because you were assigned.Message ID: @.***>

There is no simplified layer


Crystal 0.27.0 breaking change - Time

I ran into errors like this involving Time#epoch:
Error in lib/kemal-session/src/kemal-session/engines/memory.cr:52: expanding macro
    defne_storage
undefined method 'epoch_ms' for Time

I have managed to fix these errors in my local copy as long as I don't update kemal-session. Since I am not a professional developer (newb), I hesitate to do a fork and pull request.
Refererce Crystal breaking change #6662
Thank you.

Fixed via https://github.com/kemalcr/kemal-session/releases/tag/v0.11.0

We are not able to use this release till you unlock or tag the version on kemal-csrf here.


API Changes: EFIVARS

Task Description
Some changes are required to match Soletta's API design conventions

[ ] Macros and inline functions missing docs. Review CREATE_BLOB and CREATE_BUFFER
[ ] read_raw() requires better documentation

https://github.com/solettaproject/soletta/pull/2150/commits/ebe1f8f3d3b7b69749846595542aeb540683e4e2


warmstart not improving the iteration count

Hi,
So, I was comparing the computation times of the current version of the DFSPH implementation with a GPU implementation that I made based on the SplishSplash code from 2 years ago (just to see if the performances changed significantly for some reason).
I noticed that for some reason the current implementation was slower when simulating a fluid at a rest state.
Doing some more test I noticed that, in the current implementation, the warmstart had no impact on the number of iterations of the density solver (the same may be true for the divergence solver but as the number of iterations is usually smaller, it is harder to properly test).
Here are the results I'm getting with showing in order: timestep count // density iterations // divergence iterations
with warmstart:
596 69 1
597 66 1
598 67 1
599 66 1
600 68 1
601 66 1
602 66 1
603 67 1
604 67 1
605 67 1
606 68 1
607 69 1
no warmstart:
596 68 1
597 67 1
598 68 1
599 68 1
600 68 1
601 67 1
602 68 1
603 68 1
604 68 1
605 68 1
606 67 1
607 68 1
By looking at the result I obtain with the older impelmentation I'd be expecting the density iterations count to be between 10 and 20 with the warmstart activated.
Here is the configuration file I'm using to quickly obtain a fluid near a rest state so that 500 iterations are enougth to properly see the problem (renamed to .txt so that github accept it ...).
DamBreakModel2.txt
To switch between the warmstart and non warmstart version I'm simply commenting the two warmstart defines at the top of the TimeStepDFSPH.h file
Am I missing something to activate the warmstart?
Thanks for your time,

Hi,
Can you tell me which Git commit corresponds to the "older implementation". Then I will test it.
Jan

Hi,
Thanks for your response.
So I have no idea which exact commit my code is based on, but here is a commit where the warmstart it actually reducing the number of iterations necessary:
34d7c7c0ba460f4630d57c3e22a745dad7a3ab3d

Yesterday I had a bit time to take a closer look on this problem. I improved the warmstart and now it's much better. For your example it requires 2.5x less iterations with the new warmstart. This night I also tested the warmstart with a larger scenario (8M particles) and it works fine. So please test the new version.

Hi,
Sorry it took me a while to verify it.
Indeed your solution, make the warmstart work properly.
I'll close the issue, simply in the case it interest you i'll note a problem the warmstart has for the DFSPH.
The initial reason why I found the problem with the warmstart is that I noted that for a fluid at rest the warmstart leads to unstable iteration count for the density solver.
Initialy with the scene I gave in this thread you would see a sudden increse to around 20 iterations followed by a slow decrease to 9 and then it would increase again. With the new version this problem is significantly larger,  it oscilate between 45 and 6 (see below). Athough on average it is indeed fast.
simulation_step density_iter divergence_iter
598 			28  			1
599 			25  			1
600 			26  			1
601 			43  			1
602 			19  			1
603  		18  			1
604  		17  			1
605  		11  			1
606  		8  			1
607  		6  			1
608  		6  			1
609  		32  			1
610  		46  			1
611  		43  			1
612  		20  			1
613  		13  			1
614  		10  			1
615  		8  			1
616  		8  			1
617  		7  			1
The main problem with that behaviour is that it is not simply a a computation time problem, this behaviour is associated with a small compression and expension of the fluid. If there are solids interaction with the fluid it will cause a huge variation (by like 50%) in the force applied on the solid, though the resulting animation still looks normal.
The problem can be reduced by increasing the minimum number of iteration of the divergence solver, but I don't know of an actual modification of the warmstart solving that problem. Btw, I supose that the warmstart is also the reason why simulating columns of fluid is unstable when not using the divergence free solver but I have not extensively tested it.


The group icon won't update.

I can change and save the group icon, but it won't update on the taskbar. It's still stuck on the first image I chose.

Windows limitation. I could implement something to force refresh, but that may refresh every icon and that can get a little annoying every time you change the group icon. I will see what I can do.

It fixed itself when I restarted my laptop.


Question about AccessByteBuffer

https://github.com/DataSketches/memory/blob/5356a8b1db8aaf985acb22766e803f8737de115e/src/main/java/com/yahoo/memory/AccessByteBuffer.java#L32-L80
Why read-only and writable branches are so different?

Do you still want to keep this open?
Everything we have discussed so far is merged into master.  Are you planning more changes?
Do you need an Maven Central artifact? Or do you plan to create your own jars? If you need an artifact, when?

Please keep open, not reviewed yet.
I plan to add support of non-native Memory and Buffer!
I will need after that

@leventov
Please review.

Yes and I agree with your simplified code.


Urgent!!Format of documentation

@erik0704 Erik, there are some problems with the format of pdf of documentation, when you store the pdf, you should check the background graphics and uncheck the Header and Footer under option.  Also all hyperlinks are not activated?

NeoXPro Manager - Developer Guide.pdf
Should be like this, the content pages can link to parts, and links are preserved @erik0704

follow the instructions under 4.3. Converting Documentation to PDF format:
https://nus-cs2103-ay1718s1.github.io/addressbook-level4/DeveloperGuide.html#documentation

@erik0704 @quangtdn @hxy0229 anyone there?

resolved


Filters mixed-up between multiple apps

Environment Details

Helidon Version: 2.5.1
Helidon MP
JDK version: 17
OS: Linux
Docker version (if applicable):

Problem Description
Seems like an filter from a different jersey application is sometimes applied.
Steps to reproduce
Run the test in the attached project.
quickstart-mp2.zip

quickstart-mp2.zip

As you know the basic Application isolation provided by Helidon relies on the set of classes returned by getClasses. In this example, isolation is seeked by dynamically attempting to register filters using a feature. All that part is controlled by Jersey and completely outside of Helidon's control. Jersey is likely registering all filters in the same injection manager, and thus provides no isolation.
If the getClasses form of isolation is not sufficient (because it needs to be more dynamic) then you'd need to inspect the context of an individual ServerRequest and avoid running the logic depending on which application is "active". You can @Context-inject ServerRequest and then inspect the context as follows:
        Application app = serverRequest.context().get(Application.class).get();

There's not much else for us to do at this time.

I found a workaround for this to set:
server.single-injection-manager=true
in resources/META-INF/microprofile-config.properties
but, however, this is not recommended setting, right ?

Can you confirm that this new access would cover your use case?
    public boolean configure(FeatureContext context) {
        Application app = Contexts.context().flatMap(c -> c.get(Application.class)).orElse(null);
        System.out.println("app = " + app);

        // ...
        return true;
    }

Yes, I think this should be sufficient. Will the same apply to AutoDiscoverable?

PR #4745

Closing this issue at this time. We have found a workaround that uses the feature in PR #4745 (targeted for 2.5.3).


command to unset environment variables

Added assume revoke

Plan in the future is to move all commands surrounding configuration and cleanup to be moved to the Granted binary.

#83

Fixes #79

This should also fix #81!


added user story, story sequence diagram and wireframe

General Checks

[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[x] All CI checks pass
[x] all conflicts are resolved (if any)
[x] PR has a descriptive title
[x] PR has appropriate labels and milestones for easy identification
[x] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[x] It is in the appropriate column in the project board (if necessary)
[x] has short and clear description
[x] is linked to an issue (if it is related)
[x] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[ ] the markdown source is formatted
[ ] spelling and grammar is correct in all text
[ ] The markdown looks correct when you preview the file
[ ] all links and images work

HTML

[ ] pages besides /index.html are stored in /pages
[ ] the code is well-formatted
[ ] the HTML is valid
[ ] there are no inline styles (example: style='color: red;')
[ ] there are no <style> tags with CSS, all styles are hrefs
[ ] there is no inline JavaScript (example: onclick='doSomething()')
[ ] there are no <script> tags with JS, all JS is in an separate file
[ ] ids are used for JavaScript only, not for CSS
[ ] semantic tags are used
[ ] spelling and grammar is correct in all site content

CSS

[ ] the code is well-formatted
[x] passes all the linting checks

Please check the assignment description, MD files and follow the steps.
User stories must be written in backlog, not image.
Why created a new folder for planning website-design-documents? you already have planning folder and all planning files must be inside planning folder.
Story sequence diagram is no needed.
Wireframes don't have any real text, images, or design. Agile Development presentation


kubeflow enable not working - hanging during deploying kubeflow step

Steps to reproduce
System: Ubuntu 18.04
sudo snap install microk8s --classic --channel=edge
microk8s.status --wait-ready
microk8s.enable dns storage dashboard
microk8s.enable gpu
KUBEFLOW_DEBUG=true microk8s.enable kubeflow

Outputs
➜  genscape-kubeflow sudo snap install microk8s --classic --channel=edge
microk8s (edge) v1.17.3 from Canonical✓ installed
➜  genscape-kubeflow microk8s.status --wait-ready
microk8s is running
addons:
cilium: disabled
dashboard: disabled
dns: disabled
fluentd: disabled
gpu: disabled
helm3: disabled
helm: disabled
ingress: disabled
istio: disabled
jaeger: disabled
juju: disabled
knative: disabled
kubeflow: disabled
linkerd: disabled
metallb: disabled
metrics-server: disabled
prometheus: disabled
rbac: disabled
registry: disabled
storage: disabled
➜  genscape-kubeflow microk8s.enable dns storage dashboard
Enabling DNS
Applying manifest
serviceaccount/coredns created
configmap/coredns created
deployment.apps/coredns created
service/kube-dns created
clusterrole.rbac.authorization.k8s.io/coredns created
clusterrolebinding.rbac.authorization.k8s.io/coredns created
Restarting kubelet
DNS is enabled
Enabling default storage class
deployment.apps/hostpath-provisioner created
storageclass.storage.k8s.io/microk8s-hostpath created
serviceaccount/microk8s-hostpath created
clusterrole.rbac.authorization.k8s.io/microk8s-hostpath created
clusterrolebinding.rbac.authorization.k8s.io/microk8s-hostpath created
Storage will be available soon
Applying manifest
serviceaccount/kubernetes-dashboard created
service/kubernetes-dashboard created
secret/kubernetes-dashboard-certs created
secret/kubernetes-dashboard-csrf created
secret/kubernetes-dashboard-key-holder created
configmap/kubernetes-dashboard-settings created
role.rbac.authorization.k8s.io/kubernetes-dashboard created
clusterrole.rbac.authorization.k8s.io/kubernetes-dashboard created
rolebinding.rbac.authorization.k8s.io/kubernetes-dashboard created
clusterrolebinding.rbac.authorization.k8s.io/kubernetes-dashboard created
deployment.apps/kubernetes-dashboard created
service/dashboard-metrics-scraper created
deployment.apps/dashboard-metrics-scraper created
service/monitoring-grafana created
service/monitoring-influxdb created
service/heapster created
deployment.apps/monitoring-influxdb-grafana-v4 created
serviceaccount/heapster created
clusterrolebinding.rbac.authorization.k8s.io/heapster created
configmap/heapster-config created
configmap/eventer-config created
deployment.apps/heapster-v1.5.2 created

If RBAC is not enabled access the dashboard using the default token retrieved with:

token=$(microk8s.kubectl -n kube-system get secret | grep default-token | cut -d " " -f1)
microk8s.kubectl -n kube-system describe secret $token

In an RBAC enabled setup (microk8s.enable RBAC) you need to create a user with restricted
permissions as shown in:
https://github.com/kubernetes/dashboard/blob/master/docs/user/access-control/creating-sample-user.md

➜  genscape-kubeflow microk8s.enable gpu
Enabling NVIDIA GPU
NVIDIA kernel module detected
dns is already enabled
Applying manifest
daemonset.apps/nvidia-device-plugin-daemonset created
NVIDIA is enabled
➜  genscape-kubeflow KUBEFLOW_DEBUG=true microk8s.enable kubeflow
Enabling dns...
Enabling storage...
Enabling dashboard...
Enabling ingress...
Enabling rbac...
Enabling juju...
Running `microk8s-juju.wrapper --debug show-controller uk8s`
Deploying Kubeflow...
Running `microk8s-juju.wrapper --debug bootstrap microk8s uk8s`

It gets stuck in this forever and some pods apparently fails.
Yaml to add user to rbac to access dashboard:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: dashboard-admin
  namespace: kube-system
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: dashboard-admin
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cluster-admin
subjects:
  - kind: ServiceAccount
    name: dashboard-admin
    namespace: kube-system

Tar is attached and pod with error events below

inspection-report-20200226_114810.tar.gz

Could it be possible that your machine name is not all lowercased.  Can you try renaming your machine to all lower case?
From :DUBRABREULX01M
To: dubrabreulx01m

The installation is now moving forward. I'll report here once it finishes and things work successfully and then I'll close this ticket.
Thanks @balchua

Its working now.


When I try to use manifest url, it says "The 'path' argument must be of type string. Received type undefined"

I had some difficulty finding the manifest URL. I tried https://raw.githubusercontent.com/reaver01/ryuutama/master/system.json
If that's right, it says the error on the title. Maybe it's another .json file?
It would help a lot to see the manifest URL in the read me file. And if it's something wrong about it, I hope it may be fixed soon.
I just discovered Ryuutama and it seems an amazing game. Thanks a lot for the module! I hope I can use it soon!
Regards!

https://foundryvtt.com/packages/ryuutama/
The system should be able to be installed just by searching for it. That is why I never listed the manifest in the GitHub.

https://foundryvtt.com/packages/ryuutama/
The system should be able to be installed just by searching for it. That is why I never listed the manifest in the GitHub.

Hello!
I noticed I was in the modules part, not systems. It was only my fault hahaha
Before coming here to tell, I just pasted the URL and worked, but I just saw your answer and realized I could have done it even easier.
I searched for ryuutama in the modules, found nothing and googled it to get here. In the end, if I had looked in the right place, everything would be ok hahah
Sorry about that, and thanks again. Checking out the sheet and monsters, it's very well done :) good work!

Thank you. Glad you got it working.


beta7 stuck on Installing<EMAIL_ADDRESS>

Node:	v6.9.1
Npm:	v3.10.8
Installing script dependencies
Installing apm
Installing modules ✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>
beta6 was working fine
linux 64 bit

I'm wrong it stuck on random place


Permission denied when installing plugins

Cookbook version
5.0.4
Chef-client version
13.5.3
Platform Details
Oracle Linux 7
Scenario:
  jenkins_plugin 'credentials' do
    notifies :execute, 'jenkins_command[safe-restart]', :immediately
  end

Steps to Reproduce:
Downloaded the latest cookbook using knife cookbook site download jenkins, added to my Berksfile, installed dependencies, used knife solo cook user@ip to kick off my run.
Expected Result:
It should have installed the plugin
Actual Result:
https://gist.github.com/kryptek/0b9ff5d8ed1074646736f61af3d744ed
I do see that the local-mode-cache is owned by root and I suspect this is why I'm getting the permission denied but I have no idea where to go from here. Any help?

After digging into the issue a bit more I see that this is related to the permissions of the Jenkins server itself. What's the best way to go about getting a default admin user in with the base installation so that I can spring up the server with the plugins I need?

I have the same problem.
Cookbook version: 5.0.4
Chef-client: 12.11.18
Platform: CentOS 6.9
I am running chef-client in a zero mode (-z) as root. The jenkins_plugin[XXX] action install fails with:
    Mixlib::ShellOut::ShellCommandFailed
    ------------------------------------
    Expected process to exit with [0], but received '1'
    ---- Begin output of "java" -jar "/root/.chef/local-mode-cache/cache/jenkins-cli.jar" -s http://localhost:8080 -"remoting" -i "/root/.chef/local-mode-cache/cache/jenkins-key" install-plugin file:///root/.chef/local-mode-cache/cache/ace-editor-1.1.plugin -name ace-editor  ----
    STDOUT: Installing a plugin from file:/root/.chef/local-mode-cache/cache/ace-editor-1.1.plugin
    STDERR: ERROR: Unexpected exception occurred while performing install-plugin command.
    java.io.FileNotFoundException: /root/.chef/local-mode-cache/cache/ace-editor-1.1.plugin (Permission denied)
    	at java.io.FileInputStream.open0(Native Method)
...<snip>...

The file /root/.chef/local-mode-cache/cache/ace-editor-1.1.plugin exists and has jenkins set as owner:group.
I think the problem is in the executor.execute on line 319 of libraries/plugin.rb. Does it run as Jenkins user? If so it won't be able to read files under /root
My workaround: Set cache_path to another location in /etc/chef/client.rb

@jezdikm
Yes, I run into the same problem.
Changing cache_path works.
Have you found a more elegant solution, instead of this workaround?

Unfortunately I have not. Keeping eye on this thread if someone comes up with a better solution of fix.

You could try setting a user in the run_state, see https://github.com/osuosl-cookbooks/osl-jenkins/blob/master/recipes/powerci.rb#L215
Or another example with an SSH key instead.
https://pghalliday.com/jenkins/groovy/sonar/chef/configuration/management/2014/09/21/some-useful-jenkins-groovy-scripts.html#appendix


[FEATURE] Ensure that we can support DBR 14.3LTS

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
We should add support for DBR 14.3 LTS
This means we need compatibility with:

[ ] Apache Spark: 3.5.0
[x] Python: 3.10.12  (already covered)
[ ] Delta Lake: 3.1.0

Additionally, we need to look at how Spark Connect changes things for us.
Any reference we have to JVM directly, we should investigate.
Only Shared cluster mode is affected according to docs.
Describe alternatives you've considered
N/A
Additional context
N/A

The whole idea was to introduce the internal koheesio spark session to be able to provide easy switch between remote and local modes.
Also if I'm not wrong pydantic is checking SparkSession type based on the full import path and with remote spark session it is imported from a different path, at least it was in a such way sometime ago

https://docs.databricks.com/en/dev-tools/databricks-connect/python/limitations.html

Not available on Databricks Connect for Databricks Runtime 13.3 LTS and below:
Streaming foreachBatch
Long queries over 3600 seconds -- not sure what does this mean.
Not available:
Databricks Utilities: credentials, library, notebook workflow, widgets  -- not sure what does this mean.
SparkContext -- No jvm operations
Changing the log4j log level through SparkContext -- need to check the code

Run the unit test locally with spark-connect remote instant.

Check how to manage SparkSession and DatabricksSession.

Long queries over 3600 seconds
It has probably to do with the parameter STATEMENT_TIMEOUT that sets a SQL statement timeout in seconds. This parameter is set at the Session level. So, concerning the Spark Connect, where your entry point is the DatabricksSession:
from databricks.connect import DatabricksSession

spark = DatabricksSession.builder.getOrCreate()

it seems this parameter is set to 3600 and cannot be modified. I still couldn't find where this happens though.
https://docs.databricks.com/en/sql/language-manual/parameters/statement_timeout.html
https://docs.databricks.com/en/sql/language-manual/sql-ref-parameters.html

I have added details in here related to foreachbatch function: https://github.com/Nike-Inc/koheesio/issues/56
If you prefer collecting everything here I will copy paste the comment and close the issue.
One additional point that I do not see in the list is Dataframe.rdd which is being used in some tests

[ ] Delta merge through Delta API not possible. (only supported from Spark 4.0 and Delta 4.0)
[ ] Delta Merge through SPARK SQL still works
[ ] How to support 3.x and 4.x spark/delta versions

There is a way to check Spark Session is remote or native.
we should introduce api/function to get spark session flag and check against the specific APIs e.g. delta merge /snowflake and raise the exception.

-- use snowflake-connector-python instead of spark._jvm

All of these should be addressed as part of release 0.9.0 (currently in pre-release). Please verify your usecases accordingly so we can proceed with the release.


Restaurants demo asserts with remote planning

To reproduce:

run remote planning for 'user'
run shell with &plannerOnlyConsumer in the browser
accept "Find restaurants near you" suggestion
The recipe is instantiated, but an error appears in the console
Error: Multiple slotid-!859659263222556:user-null:9:d2ab5e542d1ab8944cf5173665239289883fdd43 inner slots cannot be provided

The arc behaves normally, next suggestions appear and can be accepted, but the assert keep firing.
I'm investigating.

Once the restaurants suggestion is accepted in shell, the arc is deserialized in remote-planner (and planning is performed).
Then after a short delay the [Restaurant] store ends up containing each restaurant twice.
This is a transient state, and with the second refresh in shell, the page no longer causes an exception, but the arc has two VolatileStorages for each inner-arc handle.
I disabled planning altogether: simply reusing the existing suggestions for the null arc, to create a new arc, but no new planning - and this is still happening.
@piotrswigon @shans - does any of this ring a bell in regards to inner arc serialization and storage? I would really appreciate a second pair of eyes to look at this with at some point. Thanks!

repro: https://github.com/PolymerLabs/arcs/pull/2492

We haven't done anything but thinking about design for inner arc serialization.
The error is coming from _initInnerSlotContainer, which is called when inspecting the rendered DOM for slotid attributes. This aligns well with your finding that restaurants are duplicated.
Shouldn't the storage layer disallow duplicates of entities in the collection? That was my understanding @mykmartin . There may very likely be a bug somewhere else that is causing creation of these 2 stores though...
I can take a deep dive and get to the bottom of the issue if needed, but maybe @shans will have some quick insight.

@mmandlis do you have an arc with a repro, and if you do can you grab its serialization from the storage?
I'm thinking that we may find the volatile stores of the transformation there - which would be a cue. Transformations code does not expect them to be serialized yet.

The storage layer doesn't detect duplicate entities; it relies on the id provided in the entity object. Using the same id twice with different entity data replaces the existing version (this is known incorrect behaviour that will be fixed when entity mutations are implemented). But it's quite possible to have the same entity data with two different ids in the same collection.

Once suggestion has been accepted, Arc has been inflated by both web shell and the planner shell. Both arcs performed searches for restaurants, attempted to clear the [Restaurant] handle and insert the entities into it.
One my machine there was ~3 seconds difference of timing for clear followed by store performed by the web and the planner shells. In both cases the storage layer has noted 0 entities to remove from the collection - I assume this is ok with such slight differences of timing. @mykmartin can comment.
The entities from both searches were identical, so probably should get de-duplicated. However it turns out they IDs include session id - which is random, so even though the data was the same, the Entity IDs were not.
Some things to ponder:

Should Entity IDs contain a component that is derived from their content, instead of using the Session ID?
Should [Restaurant] store be volatile? It used to be #nosync, it no longer is. This distinction however makes for a brittle programming model.
Should the system be quicker to note the entities added to the collection by the other device few seconds later? Maybe, but relying on it would also make the programming model brittle.

@shans to comment on (1).

Seems like a bad idea to have arcs in two different shells modifying the same handle? Especially since they're both completely clearing the collection.. I would have thought a planning arc would work on a clone of the source data, and/or have its own handles to modify.

@mykmartin When deserialization happens, the arc's active recipe is instantiated all over again, so planning arc is no different than a regular arc.
In fact, during my investigation i completely disabled planning, so the remote planner was just deserializing the same arc in parallel.

Ok, so putting aside the planning aspect: it still seems like a problem to have separate arcs actively modifying the same handle for the same plan at the same time. How does that not lead them to trample on each other? In this case it seems they were both clearing an empty collection, so no harm done, but that's not always going to be true.

Some comments:

Planner should not be starting up the arc. @shans had an idea of performing planning against a speculative arc. This would work with the caveat of having handles/slots etc. bound to wrong ids, so translation back to original arc would need to happen. Cloning gives back the translation map for ids I think, so that shouldn't be too hard.

Alternatively maybe we could perform planning against an acquiescent arc? I don't think Stratategizer takes advantage of the fact that the arc is actually running (with functional particles in the PEC and all that) - nor it should. We could have some intermediate state with an inflated arc that Strategizer can reference, but without PEC and running particles?

Even with (1), the problem still holds for multi-user scenario. Entity IDs should not contain the session I think. The proposal would be for the particle to request an ID for the Entity, the request ID would be prepended with the Arc ID, and the Arc would validate the uniqueness or ensure data equality with the previously created Entity of the same ID. If Entity ID does not validate, it is a runtime error.

In this example the RestaurantFind particle would ask for ID googlemaps://place_id, which would end up being `arcId:googlemaps://place_id'.
@mykmartin @shans, does this sound reasonable?

At some point we need to think deeper about how to structure arcs that are meant for multi user consumption. Which bits are synced, which are not. I don't see how we could do this automatically - as often for bits reflected in the UI this is a subjective design decision. Today you can put #nosync on a handle to get it to not sync - which we could use in this recipe on the [Restaurant] handle if we decided that's the right UI in this demo.


problem creating the _ide_helper.php file

Hello
I am using a freshly installed laravel 5.2 project
I am getting the following error when I am running php artisan ide-helper:generate
[Symfony\Component\Console\Exception\CommandNotFoundException]
There are no commands defined in the "ide-helper" namespace.
Please advice!
Thank you

I have not added the Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class, in app.php file
Porblem solved


Update Clippy version being used in TravisCI config file

Update the version of clippy being used as per conversation here.
Lines with rustup component add clippy-preview (like the one shown below) should be changed to rustup component add clippy as per the Clippy docs.
https://github.com/0xazure/supernova/blob/66f5bc50aa6bfe8c359e684279e3c889039b8d0e/.travis.yml#L41

This would be a good first bug!

@0xazure should I add this to PR #39, or wait until that gets merged and then fix this?

@biskit1 you can add this to #39 since it introduces rustfmt, or if you want to do it as a separate pull request that's fine too!


License ?

Hello, under what license is this project released under ? I would like to study it to learn from it. Thank you.

Hello, under what license is this project released under ? I would like to study it to learn from it. Thank you.

Hey, no license, just use it the way you want

Thank you so very much. Would it be possible for you to please upload a formal license (e.g. MIT, BSD, GPL, whatever you prefer) to the repo. ? I ask because it would be helpful for others, so that they can be clear on if and how they could learn or use this project. Thank you.

Done !


Test branching/PR behavior

This is a test PR to make sure document versioning works.
Expected behavior:
Should build a version of documentation for:

test-deleteting-ref
versioned-docs
The PR

Should delete test-deleting-ref when branch deleted.

Current state of gh-pages branch - after squash-merge, before deleting branch:

Apparently "delete" only works if on default branch...here goes


Path for object instantiation seen as variable

For example, the line local.foo = new path.to.Component(); triggers two variable rules. VAR_TOO_SHORT is triggered by to in the path and VAR_INVALID_NAME is triggered by Component since it is capitalized.

fixed in dev


AutoLogout option

It would be great to have an Autologout option as one of the keys for ContinueAction
The design will be the same as Hidden but will follow up with Logout so the user won't have to do anything.
This will be great for env where filevault is set for next login, so it can ensure it is enforced right away

Would you still need/want to have to set that Hidden key?


Add command decrease and increase font editor

[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

Thanks for the PR.
This should probably use saturating_sub, because if the user spams the keybind then they could go below zero and that would overflow in release mode and they'd end up with a massive font size.
It might also be good to set a minimum/maximum value that you can get via this command. Maybe 6 for lower, and 50 for maximum? Once you get below/past a certain size, it is probably smaller/bigger than what people expect to be able to change it to.

Obrigado pelo PR. Isso provavelmente deve usar saturating_sub , porque se o usuário enviar spam ao keybind, ele poderá ficar abaixo de zero e isso transbordará no modo de liberação e acabará com um tamanho de fonte enorme. Também pode ser bom definir um valor mínimo/máximo que você pode obter por meio deste comando. Talvez 6para menor e 50para máximo? Uma vez que você fica abaixo/além de um certo tamanho, provavelmente é menor/maior do que as pessoas esperam ser capazes de mudá-lo.

Adotei as sugestões!

Thanks for the PR. This should probably use saturating_sub, because if the user spams the keybind then they could go below zero and that would overflow in release mode and they'd end up with a massive font size. It might also be good to set a minimum/maximum value that you can get via this command. Maybe 6 for lower, and 50 for maximum? Once you get below/past a certain size, it is probably smaller/bigger than what people expect to be able to change it to.

I adopted the suggestions

This was implemented as part of UI rework


Update to IBD Cross Section and Selection Algorithm

Updates
Cross section
The cross section calculation for the IBD generator previously used a simplified version of the Strummia-Vissani calculation. Now it uses the fully expanded version. This has been compared to the SNTools cross section calculation and has a maximum deviation of 1.2%. See the slides presented to the HK UK PSC meeting on 31/10/2023 for more information. This plot shows a comparison between the SNTools cross section and the one implemented for this generator:

Selection algorithm
The IBD generator uses rejection sampling to weight events by the product of the cross section and the flux. In the previous PR, the search area was defined by the product of the max cross section and the max flux. This overestimated the height of the distribution quite a bit. This has been updated to calculate the max of the cross section * flux over the energy range, giving a much smaller sampling area. This plot shows the uniformly distributed sample points (green for accepted points and pink for rejected points) and the true values for the product of the cross section and the flux (blue):

Before this fix the uniform distribution was ~4 times taller and the sampling algorithm was ~0.3 % efficient. This has been improved to ~18 % efficiency. There's further scope to improve this efficiency, though it may not be worth spending time on. To sample 100000 events the algorithm takes around 12 seconds and WCSim takes much more time to actually simulate the events.
Usage
Class usage has changed slightly. The spectrum is now read in the initialiser. So
myIBDGenerator = new WCSimIBDGen(ibd_database, ibd_model, myDetector);

now creates a WCSimIBDGen object and reads the vector of flux and energy for the ibd_model specified from the ibd_database, all in one go. This also calculates the maximum value of cross section * flux. Only two other calls are now required in PrimaryGeneratorAction to generate an event. These are GenRandomPosition and GenEvent to generate the vertices, directions and momenta.
Bug fixes
Position generation
Previously generating positions between +- ID height instead of +- (ID height)/2
Cross section calculation
There was a bug in the cross section calculation where sometimes the cross section would return as negative. This has been fixed in the complete rehaul of the calculation.
Database bugs
Some small bugs in the model database. A couple of missing commas and fluxes in the energy vectors.

Thanks @JackFannon

Thanks for reviewing this @tdealtry


Added MinGW makefile (works on appveyor).

I was going to add a minor feature (a setting to disable filevars except for explicitly enabled ones, i.e. with "enable-local-variables=1"), to do that I had first to be able to compile the project on my machine (old winxp box). I wanted to do the whole CMake thing, but writing a makefile was for me the quickest approach. I had to rewrite version.sh into a python script (and to make it work on appveyor, I had to fix some of its logic).
This is my first pull request ever. Please bear with me 😸 !

Maybe I could use your makefile, see https://github.com/zufuliu/notepad2/issues/2 and https://github.com/zufuliu/notepad2/wiki/Build-Notepad2#gcc

I see. I'll fix this soon. 👍

@zufuliu of course! 😄


The deploymment of aggregation service failed

Hi guys~
I'm trying to deploy the aggregation service according to the guidance(https://github.com/privacysandbox/aggregation-service/blob/main/docs/gcp-aggregation-service.md).
However, when I Set up your deployment environment and execute "terraform apply" for my dev env, it always threw this error:
Error: Error creating function: googleapi: Error 403: Could not create Cloud Run service dev-us-central1-worker-scale-in. Permission 'iam.serviceaccounts.actAs' denied on service account<EMAIL_ADDRESS>(or it may not exist).
│   with module.job_service.module.autoscaling.google_cloudfunctions2_function.worker_scale_in_cloudfunction,
│   on ../../coordinator-services-and-shared-libraries/operator/terraform/gcp/modules/autoscaling/workerscalein.tf line 35, in resource "google_cloudfunctions2_function" "worker_scale_in_cloudfunction":
│   35: resource "google_cloudfunctions2_function" "worker_scale_in_cloudfunction" {
│ Error: Error creating function: googleapi: Error 403: Could not create Cloud Run service dev-us-central1-frontend-service. Permission 'iam.serviceaccounts.actAs' denied on service account<EMAIL_ADDRESS>(or it may not exist).
│
│   with module.job_service.module.frontend.google_cloudfunctions2_function.frontend_service_cloudfunction,
│   on ../../coordinator-services-and-shared-libraries/operator/terraform/gcp/modules/frontend/main.tf line 43, in resource "google_cloudfunctions2_function" "frontend_service_cloudfunction":
│   43: resource "google_cloudfunctions2_function" "frontend_service_cloudfunction" {
│

but actually my account have the role "Service Account User" which should have the permission of "iam.serviceaccounts.actAs".

Could you help to solve this ? Thanks a lot!

HI @yanghuang1028 ,
Can I check with you if you have setup your deploy service account and worker service account using the gcp guide in github?
And are you getting the error when trying to deploy Aggregation Service?

Our company has a dedicated department to help manage the google service accounts, so the service account is created by them. and I already setup it through Adtech Setup Terraform.
By the way, our deploy and worker accounts are the same one. Will this affect the deployment?
And yes, I get this error when trying to deploy.

Hi @yanghuang1028 ,
From your description, I think your team might not have used the scripts to create the worker and deploy service accounts. You would need to check with your department that created the google service accounts if they used the scripts to update the user that they might have created. This should be found in the <repository_root>/terraform/gcp/environments/adtech_setup/adtech_setup.auto.tfvars in this section:
# [1] Uncomment below lines if you like Terraform grant needed permissions to
# pre-existing service accounts
deploy_service_account_email =<EMAIL_ADDRESS>worker_service_account_email =<EMAIL_ADDRESS>
Also, would you be able to confirm if you performed step #4 in the Set up your deployment environment section?
Thanks!


Travis CI fails on MacOS due to QT install issue

Travis CI fails on MacOS due to QT install issue. Perhaps due to changes to Brew?
Error:
CMake Error at /usr/local/lib/cmake/Qt5Core/Qt5CoreConfig.cmake:15 (message):
  The imported target "Qt5::Core" references the file
     "/usr/local/.//mkspecs/macx-clang"
  but this file does not exist.  Possible reasons include:
  * The file was deleted, renamed, or moved to another location.
  * An install or uninstall procedure did not complete successfully.
  * The installation package was faulty and contained
     "/usr/local/lib/cmake/Qt5Core/Qt5CoreConfigExtras.cmake"
  but not all the files it references.
Call Stack (most recent call first):
  /usr/local/lib/cmake/Qt5Core/Qt5CoreConfigExtras.cmake:50 (_qt5_Core_check_file_exists)
  /usr/local/lib/cmake/Qt5Core/Qt5CoreConfig.cmake:166 (include)
  /usr/local/lib/cmake/Qt5/Qt5Config.cmake:28 (find_package)
  CMakeLists.txt:82 (find_package)

The latest fix for this appears to be failing in CI.

Ya, I'm not able to solve it without access to macos, need some apple-head help :)


Library not found IOS

Ios gives error
Showing Recent Messages
ld: library not found for -lPods-Sint

Where 'Sint' is my app name

when opening the .Workspace the issue is gone


Remove version: Nvidia.GeForceNow version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://ota-downloads.nvidia.com/ota/GeForceNOW-release_D1CB4F.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


How to make HACS read and display info.md

How do I get HACS to display my info.md when someone opens my plugin/integration? I have an info.md file but it is not displaying that when someone opens the card. Am I missing something to make it display?

When you use releases, HACS display infromation about the selected release (version)
https://github.com/DavidFW1960/bom_forecast/tree/0.1 Does not contain a info file.
Master does:


Run integration tests on a different database

Runs integration tests in separate container.
Minor update of nodemon.

@andrejstas any chance we can get this to completion?

@andrejstas bump

Closing as superseded by the migration to Firestore


Nats streaming sample with Golang

Paging @jeffhollan & @ahmelsayed to have a look

Thanks for submitting!


Ubuntu 16.04 openssl version

Describe the bug
Ubuntu 16.04 comes with openssl 1.0.2g by default. Same goes for related packages like libssl-dev. However, openssl 1.1.1g is installed on the ubuntu-16.04 virtual environment.
Area for Triage:  Packages
Question, Bug, or Feature?: Bug
Virtual environments affected

[ ] macOS 10.15
[x] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
openssl 1.0.2g should be present, or at least when apt-get install-ing openssl, libssl-dev, etc, that version should be installed.
Actual behavior
openssl 1.1.1g is installed instead. This means code compiled on this GH virtual environment that dynamically links against libssl will not work on a stock Ubuntu 16.04 environment. This is unfortunate because one of the reason for picking ubuntu-16.04 specifically is probably to test and build your package for that environment.
Perhaps this is done intentionally, if it can't be "fixed", it should be explicitly documented/enumerated/called out on the package list, since I think the general expectation of a non-rolling release like Ubuntu is you get the "system" version of packages.

The package appears to be pulled in via https://github.com/actions/virtual-environments/blob/97e4ea8356c3470319eedd442645607ddce6a985/images/linux/scripts/installers/php.sh#L15, intentionally or not.

Hello, @chancancode
I will prepare PR to provide additional information about openssl and libssl-dev packages for Ubuntu, because revert to 1.0.2g will break other customers.
To revert openssl and libssl-dev packages to previous version you could use:
# List available packages
~$ apt-cache madison openssl libssl-dev
   openssl | 1.1.0h-2.0+ubuntu16.04.1+deb.sury.org+1 | http://ppa.launchpad.net/ondrej/php/ubuntu xenial/main amd64 Packages
   openssl | 1.0.2g-1ubuntu4.16 | http://azure.archive.ubuntu.com/ubuntu xenial-updates/main amd64 Packages
   openssl | 1.0.2g-1ubuntu4.16 | http://security.ubuntu.com/ubuntu xenial-security/main amd64 Packages
   openssl | 1.0.2g-1ubuntu4 | http://azure.archive.ubuntu.com/ubuntu xenial/main amd64 Packages
libssl-dev | 1.1.1g-1+ubuntu16.04.1+deb.sury.org+1 | http://ppa.launchpad.net/ondrej/php/ubuntu xenial/main amd64 Packages
libssl-dev | 1.0.2g-1ubuntu4.16 | http://azure.archive.ubuntu.com/ubuntu xenial-updates/main amd64 Packages
libssl-dev | 1.0.2g-1ubuntu4.16 | http://security.ubuntu.com/ubuntu xenial-security/main amd64 Packages
libssl-dev | 1.0.2g-1ubuntu4 | http://azure.archive.ubuntu.com/ubuntu xenial/main amd64 Packages

Pipeline:
 steps:
    - name: Install openssl&libssl-dev
      run: |
        sudo apt-get install -y --allow-downgrades openssl=1.0.2g-1ubuntu4.16 libssl-dev=1.0.2g-1ubuntu4.16
    - name: Display openssl&libssl-dev versions
      run: dpkg -l openssl libssl-dev | grep '^ii' | cut -c5-

Yeah that's what we end up doing as well. It seems like you'd need to downgrade libssl itself as well, I thought downgrading openssl/libssl-dev would force the dependencies to be downgraded as well, but that doesn't seem to be the case.
This still seems quite unexpected to me, but I understand it's probably "too late" to fix now. Was this done intentionally, or was it just an accidental side-effect of pulling in the PHP ppa? (FWIW, they seem to suggest that it's optional, so the PHP packages can be installed against the older openssl/libssl, supposedly.)
Is this a rare exception or are there other upgraded "system libraries/packages" that are worth calling out as well?

Update: we end up removing the PHP ppa, removed the packages, then reinstalled them so we will get the "correct" one from the system sources (so we don't pin to a particular version)

Looks like libssl is installed as libssl1.0 so that's ok. It's only libssl-dev that's at a newer version.

The documentation has been updated - https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu1604-README.md

OpenSSL 1.1.0h 27 Mar 2018 (Library: OpenSSL 1.1.1g 21 Apr 2020)


Make uninstall part less verbose

When I read transcript log of systemtests I'm totally lost in all output of command when uninstall is running. So this pr make uninstall part less verbose for better readability.

Is there some way we could get the logs to file instead of just removing it? I remember it was useful to see the return code for the commands when I was debugging finalizers.

@lulf good point I will implement that


#MongoTasks. Artyom Kashkan

https://travis-ci.org/github/kashkan10/db-assignments/builds/715484412

Try to decrease the time of performing these tests up to 100 ms. Adding some indexes to specific collections  should help

I'll mark the PR as "Draft", please click "ready for review" when it will be finished. Thank you!


2.0.12: Failed to build SDL2 static on Windows with msvc

Got such error:
"link"  /MACHINE:x86 /OUT:flipclock.exe flipclock.exe.p/srcs_main.c.obj flipclock.exe.p/srcs_getarg.c.obj flipclock.exe.p/srcs_flipclock.c.obj "/nologo" "/release" "/nologo" "/DEBUG" "/PDB:flipclock.pdb" "subprojects\SDL2-2.0.12\libsdl2.a" "subprojects\SDL2-2.0.12\libsdl2main.a" "subprojects\SDL2_ttf-2.0.12\libsdl2_ttf.a" "subprojects\freetype-2.9.1\libfreetype.a" "winmm.lib" "version.lib" "imm32.lib" "setupapi.lib" "opengl32.lib" "/SUBSYSTEM:WINDOWS" "kernel32.lib" "user32.lib" "gdi32.lib" "winspool.lib" "shell32.lib" "ole32.lib" "oleaut32.lib" "uuid.lib" "comdlg32.lib" "advapi32.lib"
MSVCRTD.lib(ftol2.obj) : error LNK2005: __ftol2_sse 已经在 libsdl2.a(src_stdlib_SDL_stdlib.c.obj) 中定义
  正在创建库 flipclock.lib 和对象 flipclock.exp
flipclock.exe : fatal error LNK1169: 找到一个或多个多重定义的符号
ninja: build stopped: subcommand failed.

(My system is Chinese and I don't know how to let msvc show English error message, but I can translate it by my self, I think the error is like "MSVCRTD.lib(ftol2.obj) : error LNK2005: __ftol2_sse is already defined in libsdl2.a(src_stdlib_SDL_stdlib.c.obj)" and "flipclock.exe : fatal error LNK1169: find one or more symbols that has multi-defination")
I build it with command mkdir build, cd build, meson setup -Ddefault_library=static . .. and meson compile. I am not sure why this happens.

My Project: https://github.com/AlynxZhou/flipclock/

My Project: https://github.com/AlynxZhou/flipclock/

After using meson setup --default-library=static . .., MSVCRTD.lib becomes LIBCMT.lib, and the error still exists.

After using meson setup --default-library=static . .., MSVCRTD.lib becomes LIBCMT.lib, and the error still exists.

The wrap has only been tested on 64 bit Windows. Probably some configuration or feature detection goes wrong on 32 bit Visual Studio.

The wrap has only been tested on 64 bit Windows. Probably some configuration or feature detection goes wrong on 32 bit Visual Studio.

I want to build a 32-bit one so it can run on both 32bit and 64bit Windows

Should be fixed by https://github.com/mesonbuild/sdl2/pull/21.

Currently I am using the prebuilt libraries provided by SDL.org with my own finding code, let me try this if I have time.

Tested with x86 MSVC and confirmed that it was fixed. Thanks!


Add felix config parameters to tune reporting interval/delay.

Description

Plumbing through the new felix config values for https://github.com/projectcalico/felix/pull/1641
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Looks like some straightforward test updates are needed as well.


Not compile js files correctly

I placed bootstrap.js and popper.js into vendor folder. And enqueued vendor.min.js. But it shows Uncaught TypeError: $ is not a function at vendor.min.js error in console.

@prncdahal 's solution works for me. Thanks man!

This is not working for me.. What's the correct way to import JS 3rd party libraries?
Im trying to import jquery, popper and bootstrap js files.
I would like to require these into a single vendor file maybe, I don't mind having to paste the dist files from each though but I am not being able to do either so any help would be highly appreciated.

Thanks @prncdahal, works for me too!


Unhandled Promise Rejection: AbortError

Error in Safari Console - "Unhandled Promise Rejection: AbortError: The operation was aborted." "(anonymous function)" "rejectPromise"
When this error appears, the player stops working and attempting to play another track just shows the same error again. This also prevents any other websites (tested with Soundcloud specifically) with web audio from working until the browser is restarted.
Wavesurfer.js version(s): 6.4.0 (via unpkg.com) (backend: MediaElement)
Browser and operating system version(s): Mac OS Monterey 12.6.1 and Safari 16.1
Code needed to reproduce the issue: Not known if our code is causing the issue or where in the code it would be.
Use behaviour needed to reproduce the issue: Music tracks are listed on a page, error can be reproduced by playing each MP3 in turn one after another until the error appears, often this is after 100 or so tracks but can typically happen any where after playing 50 tracks in succession. We know we can replicate this by quickly playing each track in turn, as soon as the pre-rendered peaks are drawn in the player, play the next track. Rinse and repeat until error shows again.
Unable to replicate this same behaviour in Chrome 108 on Windows 10 though, so we assume a Safari/Mac related issue at this time?

Same here sometimes

@thijstriemstra @katspaugh I'd like this looking in to please, it's possible we'll pay for the work to get it resolved.

Do you load the sind source of multiple tracks on the same page?
I no longer get theses errors after just loading audio opn demand and not all on page load.
Check it out here

I'd like this looking in to please, it's possible we'll pay for the work to get it resolved.

Please send me an email (check git log for address) @pmgrace, cheers.

I replied per email.


Guidance/Possible Feature Request: Check event validity without executing?

Hi, I've been looking at the various C++ FSM libraries and came across maki. The implementation looks pretty clean and it appears you're actively developing/maintaining it.
However, I have a use case that I can't seem to satisfy with maki's current API or that of any of the other FSM libraries I've looked at: constructing the set of valid event types against the machine's current state. Or for that matter, checking validity of a single event against the machine's current state without executing it.
All of the FSM libraries I've looked at, maki included, seem to force you to execute each event, rather than inspect the validity of the event or construct a set of valid events for the machine state. For my use case (a digital board game), I need to construct the set of valid events (moves) for the AI algorithm to evaluate/score, and would also like to tap into that as part of the UI. If I can construct a potential event (move) from the UI, such as "play card X" for each given card in a player's hand, and evaluate with the machine if that's a valid event/move given the machine's state/guards, then I can update the UI to indicate whether that's an action the player can take at any given moment. Since I'm just updating UI elements to indicate actions the player can take, I don't actually want to execute the move until the player initiates it.
Anyway, I'm wondering if I'm missing something in maki's API (or for that matter any of the other C++ FSM's you've looked at), or if that's just not something people do with FSMs?
Any guidance you have would be tremendously appreciated. Thanks!

Hi Bob,
That's an interesting use case you have here! And your solution seems pretty smart.
So you'd like something like this?
template<class Event>
bool machine::would_cause_state_transition();

I don't know any FSM library, including mine, that implements such a feature. But this should be doable!
Be aware that I'm in the middle of a big breaking change (see the 'pass-callables-to-conf' branch). I hope you haven't written too much code with the current API.

template<class Event>
bool machine::would_cause_state_transition();

Exactly! If that's something you'd be open to exposing in the API, it would be immensely useful for me!

Be aware that I'm in the middle of a big breaking change (see the 'pass-callables-to-conf' branch). I hope you haven't written too much code with the current API.

No problem! I saw the note about the API not being stable yet. 😄

I guess it would probably be more like the following since I'd want to make sure the guards were evaluated as well:
template<class Event>
bool machine::would_cause_state_transition(const Event &e);

But yeah, thanks for being flexible/open to that! I'm not sure I'm familiar enough with maki's internals yet to contribute that change, but maybe I can help contribute some documentation to help instead. 😄

Here is the solution I foresee:

changing machine::process_event_now(const Event&) so that it returns a boolean indicating whether any state transition or any action call happened;
adding a machine::process_event_now_dry(const Event&) const that would behave like machine::process_event_now(const Event&) (particularly the returning-a-boolean part), except that no transition or action call actually happen.

The first step is fast and easy. The second one is more tedious because it would require to carry some sort of bool Dry template argument across most of the library, but it shouldn't be that complicated.
It's not the right time to do anything more than thinking about a solution, but as soon as I finalize the current API change, we could work on this together. Would you be able to write a test case?

It's not the right time to do anything more than thinking about a solution, but as soon as I finalize the current API change, we could work on this together. Would you be able to write a test case?

Absolutely!

Should I write the test cases against current main or against the pass-callables-to-confs branch?

The second one, please. Thank you!

Posted as https://github.com/fgoujeon/maki/pull/3

I'm sorry Bob, I wasn't aware you wanted two distinct functions.
To make it clear: process_event_now_dry was just a more correct name than that would_cause_state_transition I first mentioned.
Do you actually need these two variants?

Nope, I thought that was something you were proposing. Hah, I'll drop those two commits.

Thank you for this suggestion and for your help, @bobziuchkovski!

Thank you for this suggestion and for your help, @bobziuchkovski!

Thanks for adding that! I'm looking forward to using it. 😄