Cannot run importer

Hello,
i'm trying to run the importer on different glb files (including the example DamagedHelmet) but i get this error everytime :
ProcessException: No such file or directory

  Command: cmake -Bbuild -DCMAKE_BUILD_TYPE=Debug
#0      _ProcessImpl._runAndWait (dart:io-patch/process_patch.dart:488:7)
#1      _runNonInteractiveProcessSync (dart:io-patch/process_patch.dart:633:18)
#2      Process.runSync (dart:io-patch/process_patch.dart:68:12)
#3      main.<anonymous closure> (file:///PATH/appdata/local/pub/cache/hosted/pub.dev/flutter_scene_importer-0.1.2-3/hook/build.dart:26:36)
#4      build (package:native_assets_cli/src/api/build.dart:92:16)
#5      main (file:///PATH/appdata/local/pub/cache/hosted/pub.dev/flutter_scene_importer-0.1.2-3/hook/build.dart:7:9)
#6      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#7      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)
Building native assets for package:flutter_scene_importer failed.
file:///PATH/AppData/Local/Pub/Cache/hosted/pub.dev/flutter_scene_importer-0.1.2-3/hook/build.dart returned with exit code: 255.
To reproduce run:
C:\PATH\Documents\flutter\bin\cache\dart-sdk\bin\dart.exe '--packages=C:\PATH\.dart_tool\package_config.json' 'C:\PATH\.dart_tool\native_assets_builder\a63996ef2018865896090302e099b2bf\hook.dill' '--config=C:\PATH\.dart_tool\native_assets_builder\a63996ef2018865896090302e099b2bf\config.json'

I tried on Windows, WSL and macOS and encounter the same issue.
My flutter doctor and dart version
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel master, 3.27.0-1.0.pre.105, on Microsoft Windows [version 10.0.22631.4317], locale fr-FR)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Professional 2022 17.5.2)
[!] Android Studio (version 2022.1)
    X Unable to determine bundled Java version.
[√] Android Studio (version 2024.1)
[√] VS Code (version 1.94.2)
[√] Connected device (3 available)
[√] Network resources

! Doctor found issues in 1 category.

Dart SDK version: 3.7.0-34.0.dev (dev) (Wed Oct 16 21:01:59 2024 -0700) on "windows_x64"

Thanks for your help

Hey @LeoCrenon , have you tried following the guide here?
If so, are you trying to use the importer manually via the command line or are you trying to use it automatically via a build hook?

Yes i followed this guide, and i was trying to use it via the command line

Okay, and if you ran this command:
dart --enable-experiment=native-assets \      run flutter_scene_importer:import \      --input "path/to/my/source_model.glb" \      --output "path/to/my/imported_model.model"
Then it would seem the issue is with your input argument. Have you tried specifying an absolute path instead of a relative path?
If you can, please provide the exact command that you're running. Feel free to redact any personal information in the directory path, such as your name.

I tried using absolute path on Windows and macOs with nothing new
PS C:\test_3d> pwd

Path
----
C:\test_3d

PS C:\test_3d> dart --enable-experiment=native-assets run flutter_scene_importer:import --input "C:/test_3d/assets/DamagedHelmet.glb" --output "C:/test_3d/assets/DamagedHelmet.model"

Unhandled exception:
ProcessException: No such file or directory

Command: cmake -Bbuild -DCMAKE_BUILD_TYPE=Debug
#0      _ProcessImpl._runAndWait (dart:io-patch/process_patch.dart:488:7)
#1      _runNonInteractiveProcessSync (dart:io-patch/process_patch.dart:633:18)
#2      Process.runSync (dart:io-patch/process_patch.dart:68:12)
#3      main.<anonymous closure> (file:///c:/users/leocr/appdata/local/pub/cache/hosted/pub.dev/flutter_scene_importer-0.2.0-0/hook/build.dart:26:36)
#4      build (package:native_assets_cli/src/api/build.dart:93:16)
#5      main (file:///c:/users/leocr/appdata/local/pub/cache/hosted/pub.dev/flutter_scene_importer-0.2.0-0/hook/build.dart:7:9)
#6      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#7      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)
Building native assets for package:flutter_scene_importer failed.
file:///C:/Users/leocr/AppData/Local/Pub/Cache/hosted/pub.dev/flutter_scene_importer-0.2.0-0/hook/build.dart returned with exit code: 255.
To reproduce run:
C:\Users\leocr\Documents\flutter\bin\cache\dart-sdk\bin\dart.exe --packages=C:\test_3d\.dart_tool\package_config.json C:\test_3d\.dart_tool\native_assets_builder\a63996ef2018865896090302e099b2bf\hook.dill --config=C:\test_3d\.dart_tool\native_assets_builder\a63996ef2018865896090302e099b2bf\config.json

And if i reproduce it using the "to reproduce run ... command":
PS C:\test_3d> C:\Users\leocr\Documents\flutter\bin\cache\dart-sdk\bin\dart.exe --packages=C:\test_3d\.dart_tool\package_config.json C:\test_3d\.dart_tool\native_assets_builder\a63996ef2018865896090302e099b2bf\hook.dill --config=C:\test_3d\.dart_tool\native_assets_builder\a63996ef2018865896090302e099b2bf\config.json
Running cmake gen step...
Unhandled exception:
ProcessException: Le fichier spécifié est introuvable.

  Command: cmake -Bbuild -DCMAKE_BUILD_TYPE=Debug
#0      _ProcessImpl._runAndWait (dart:io-patch/process_patch.dart:488:7)
#1      _runNonInteractiveProcessSync (dart:io-patch/process_patch.dart:633:18)
#2      Process.runSync (dart:io-patch/process_patch.dart:68:12)
#3      main.<anonymous closure> (file:///c:/users/leocr/appdata/local/pub/cache/hosted/pub.dev/flutter_scene_importer-0.2.0-0/hook/build.dart:26:36)
#4      build (package:native_assets_cli/src/api/build.dart:93:16)
#5      main (file:///c:/users/leocr/appdata/local/pub/cache/hosted/pub.dev/flutter_scene_importer-0.2.0-0/hook/build.dart:7:9)
#6      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#7      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

Not sure if it will make a difference, but it looks like you're using forward slashes "/" in your path for separators whereas Windows uses backslashes ""

i tried it too :
dart --enable-experiment=native-assets run flutter_scene_importer:import --input "C:\test_3d\assets\DamagedHelmet.glb" --output "C:\test_3d\assets\DamagedHelmet.model"

Okay. Have you installed CMAKE? I remember I had to do that when I followed the guide myself the first time.

Awesome! Feel free to close the issue if you're all set.

Thanks @kwill39. :) People keep getting snagged on this, so I added a note about this error to the medium post as well as an FAQ to the readme.


Add API testing with mocks

Currently, there is no API testing setup with Mocks. In fetching.test.js directly calling the fetchContributions function and getting the data from API is not a good practice while testing. Instead of making real API calls, use Mocks.

Mocks are added with Commit: 177

Mocks are added with Commit: 177


custom Webpack loaders support in v3/eject command not working?

The migration doc says this:

The standalone CLI has been rewritten with Gatsby, and the following CLI flags are no longer supported:
--webpack - use the Gatsby theme directly to customize webpack features

...but it's unclear to me what "use the Gatsby theme directly" means, as none of the Gatsby files are exposed in an mdx-deck repo. (I have used Gatsby directly before.)
I did a mdx-deck --help and saw there was an eject command, so I tried that, but it doesn't seem to work/didn't do anything. And I see that's an issue that's already been filed: #511
In case it's additionally useful, here's the output from my attempt:
[mdx-deck] ejecting Gatsby site
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 1530k    0 1530k    0     0  1189k      0 --:--:--  0:00:01 --:--:-- 1189k
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of tsne-js@* but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta but none is installed. You must install peer dependencies yourself.

audited 44642 packages in 8.824s
found 2 low severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details
git version 2.20.1 (Apple Git-117)
Reinitialized existing Git repository in /Users/janabeck/GitHub/flatland/.git/
On branch take-three-ejected
nothing to commit, working tree clean
[err] { Error: Command failed: git commit -am "Init"
    at checkExecSyncError (child_process.js:629:11)
    at execSync (child_process.js:666:13)
    at gitInit (/Users/janabeck/GitHub/flatland/node_modules/initit/index.js:26:3)
    at create (/Users/janabeck/GitHub/flatland/node_modules/initit/index.js:81:23)
    at process._tickCallback (internal/process/next_tick.js:68:7)
  status: 1,
  signal: null,
  output: [ null, null, null ],
  pid: 64272,
  stdout: null,
  stderr: null }

Am I correct in assuming that eject is the first step towards adding a custom Webpack loader? If so, it might be worth amending the migration instructions and/or README to make that a little clearer; happy to do this if you agree!

use the Gatsby theme directly to customize webpack features

See https://github.com/jxnblk/mdx-deck/blob/master/docs/gatsby.md for docs on how to use the theme with Gatsby. From there, you can use Gatsby's APIs to adjust the webpack config

Closing this as it's no longer relevant to me. My epic tale of slide deck build problems isn't worth your time, so I'll just save us both and close it :)


Feature Request: Package For Debian

Hey, it would be really cool if this module could be added to debian archives.

I forked the project: https://github.com/kmille/silk-guardian
I don't have a deb package, but a dkms deployment


FIX/TST: Accept one_1d_step or one_nd_step-like signatures in per_step.

This fixes a regression in the backward compatibility of scan. If
a user passes in a per_step parameter with has a 1D signature
(a signature like one_1d_step) this PR uses a local "adapter"
function to translate this into one_nd_step.
It also adds validation and user-friendly error messages in the event
that the per_step parameter provided by the user does not have one
of the expected signatures.
There is still a potential for breaking user code: we now insist that
the parameter names in per_step functions match the ones we expect
-- either one_1d_step and one_nd_step. Before, users could technically
get away with a per_step function that had different parameter names
because we called the per_step function with positional arguments.
However, I think it is fair to assert that we always had the right to pass in
the arguments by keyword, i.e. to require that when we specify an "expected
signature" that this includes the names. So I would not consider this an
API break.

Codecov Report

Merging #988 into master will increase coverage by 0.02%.
The diff coverage is 96.77%.

@@            Coverage Diff             @@
##           master     #988      +/-   ##
==========================================
+ Coverage   90.26%   90.28%   +0.02%
==========================================
  Files          49       49
  Lines        7879     7909      +30
==========================================
+ Hits         7112     7141      +29
- Misses        767      768       +1

Impacted Files
Coverage Δ

bluesky/plans.py
81.57% <100%> (+0.68%)
:arrow_up:

bluesky/tests/test_new_examples.py
99.38% <93.75%> (-0.3%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good. Thanks for taking the time to make succinct error messages. In my opinion, one of the hardest parts of learning to usebluesky is that tracebacks can be very obscure.

Agreed. We've heard that feedback from a couple directions, and I think it's probably the most important general area of development in bluesky and ophyd at this point.


Invalid org token in Vite build

Environment
node 20.15.0
vite 5.3.2
@sentry/vite-plugin 2.20.1
Steps to Reproduce
Configure vite.config.mts with
import { sentryVitePlugin } from '@sentry/vite-plugin'

export default defineConfig({
  plugins: [
    vue(),
    sentryVitePlugin({
      org: 'myorg',
      project: 'my-project',
    }),
  ]
})

Note that the org is my "organization slug".
The main error being Invalid org token (http status: 401)
Expected Result
I should be able to run npm run build and my org setting (my organization slug) should be valid.
The build should pass.
Actual Result
When I run npm run build and get the following error.
error: API request failed
  caused by: sentry reported an error: Invalid org token (http status: 401)

Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
Please attach the full debug log to all bug reports.
[sentry-vite-plugin] Info: Successfully uploaded source maps to Sentry
x Build failed in 5.47s
error during build:
[sentry-debug-id-upload-plugin] Command failed: /home/soviut/projects/my-project/node_modules/@sentry/cli-linux-x64/bin/sentry-cli releases new 18fb9ab7f75046e4085423e2aa660629d8e354aa
error: API request failed
  caused by: sentry reported an error: Invalid org token (http status: 401)

Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
Please attach the full debug log to all bug reports.

    at genericNodeError (node:internal/errors:984:15)
    at wrappedFn (node:internal/errors:538:14)
    at ChildProcess.exithandler (node:child_process:422:12)
    at ChildProcess.emit (node:events:519:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5)

Hello, thanks for writing in!
Just to confirm, do you have a valid auth token in your .env.sentry-build-plugin file? If you run our wizard it should generate a valid token for you, see https://docs.sentry.io/platforms/javascript/sourcemaps/uploading/vite/#automatic-setup.

@andreiborza That was it. I completely forgot about the .env.sentry-build-plugin file.
What's frustrating is I even explicitly set a SENTRY_AUTH_TOKEN in my .env.local but it was missing quotes so the = sign later on in the key must have confused it.


Account for file deletion and renaming in Export Presets

Previously, when a file was deleted after an export preset was created, Godot
would still attempt to export the deleted file, and issue a spurious error in the
process. There was also no way to change this from the Project Export window.
This PR ensures that presets stay updated, by removing any deleted or renamed
files from the presets. It does this either when the editor starts up, or when the Project
Export window is opened, ensuring that the user always sees the latest versions.
Closes #39269
Can be cherry picked for 3.2

Thanks!

Cherry-picked for 3.2.2.

Why was the cherrypick label removed?

Because I cherry-picked it already.


Change classnames module imports and usage to classNames

Pretty self-explanatory
I also removed those deeply-nested object props in outcome-trade-summary.jsx

Looks good!


Slider validation casues a lot of rerender to show the error message

https://github.com/user-attachments/assets/2507fccd-95eb-4a90-ad6f-1138dfd26e0e
based on my inspection to this, we should be considering validation on mouse leaves, not by monitoring the value

I would be willing to say this is a CSS issue, typically you would want to reserve the space for the error message beneath/next to the field to avoid layout shifts.
This will also happen with number inputs incrementing/decrementing, so I think we should maybe do that for the inputs in the docs and add a styling tip in the styling guide.

Setting a fixed width for the container will help with the size shift when the error message is displayed.
About having the error/description positioned absolutely, I would recommend having the error/description always rendered with a minimum height to avoid the hassle of positioning absolute elements

Sounds reasonable, do you want to take this on? The docs live here.


Refactor/preventing duplicate client order ids 2

NOTE: The original PR can be found here. This PR was created as a means of solving the issues that arose from the fact that the original PR branched off of another feature branch. This caused many issues with resolving conflicts with development.
Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Duplicated client order IDs can cause orders to be rejected by the exchange and can cause issues with historical analysis of strategy performance. Currently, the client uses a nonce derived from the current timestamp to generate "unique" client order IDs. The issue is that those IDs can be duplicated across multiple instances of hummingbot running in parallel. To prevent this from happening, this PR introduces a new method of generating the IDs that will be unique across instances as well.
The two main changes are:

Propagates the Binance approach of ensuring unique client order Ids into the rest of the CoinAlpha-maintained exchanges with slight modifications.
Adds a default function to use for that purpose with options for adding an exchange-specific prefix and limiting the length of the ID string.

The new order ID schema

Optional exchange-specific hbot_order_id_prefix
Order side ["B", "S"]
The base_str, which is the first and last letter of the base token
The quote_str, which is the first and last letter of the quote token
Tracking nonce (low-res version, with milliseconds)
The client_instance_id, which are generated so that they differ between instances on the same machine, but also between instances on different machines (eng: see concerns below)

Tests performed by the developer:

Tests for the new function
Tests for all of the exchanges affected:

AscendEx
Binance spot
Binance perp
Bybit perp
Gate.io
Huobi
KuCoin
OKEX
ProBit

Successfully placed orders on the following exchanges (expired creds for ByBit perp; invalid ones for Huobi and OkEx):

AscendEx
Binance spot
Binance perp (testnet)
Gate.io
KuCoin
ProBit

Tested and confirmed working QA.
Tests performed:

Created a strategy using the affected connectors and went through initial prompts successfully
Started the strategy without issues, confirmed orders are created and cancelled based on config
Confirmed that the orders created now uses the new order ID schema
Confirmed that no duplicate client order IDs were created when running between instances on the same and different machine
Trade information are correct (price, size and order ID) based on history command output, sqlite and csv file
Confirmed all issues reported are now fixed

PR update:

Binance perpetual, Huobi and Kucoin: ok
Okex and Probit: In progress


idea: browserslist for npm packages

The current implementation of browserslist good fit for applications, but is it a good fit for libraries?
I'm thinking out loud. https://github.com/ai/browserslist/issues/75 wants to add node.js support. Same time there is engines in package.json. Should engines directive allow to list browsers?
Like:
{
  "engines": {
    "Firefox": "> 20",
    "not ie": "<= 8"
  }
}

Can this information be used by browserslist to verify that all required packages met application browser requirements?

I like this idea. Maybe not in Browserslist directly, but as some addition tool on top of Browserslist queries.

Unofficial, but the widely used package.browser (version of the package transpiled/written with changes for browser usage) property would also be nice to 'support' as mixture of package.engines && browserslist somehow :). Though this is not really browserslist issue 🙃
{
   name: "",
   version: "",
   module: 'path/to/esm.js', // (Source Code)
   // e.g babel-preset-env (targets via browerslist)
   main: 'path/to/cjs.js', // engines.node
   browser: 'path/to/browser.js', // engines.browsers
   engines: {
     node: ">= 4",
     browsers: {
        ...list
     }
  }
}

@michael-ciniawsky there is an issue with browser field - how to support module and browser simultaneously. Options:

browser-module https://github.com/webpack/webpack/issues/4674
esm: true https://github.com/nodejs/node-eps/pull/60

I'm collecting package.json fields, if you are interested https://github.com/stereobooster/package.json

Maybe not in Browserslist directly, but as some addition tool on top of Browserslist queries.
@ai yes definitely, was rather interested in your opinion

Also worth to note that browserslist queries tend to have moving target, like "2 last versions of browse", but engines queries should be more static, like "Chrome >= 48 or Firefox >= 50".

I'm collecting package.json fields, if you are interested https://github.com/stereobooster/package.json

👍
@stereobooster The pkg.browser field should definitely be unofficially 'specced' by some entity and at best officially by npm. E.g please not another field for the same purpose, it's very bad practice to do that imho :) (no offense to anyone here). Why not pkg.browser.module = {String}  && pkg.browser.script = {String} with or simply an {Object}
// ESM
pkg.browser = { src: 'path/to/module.js', type: "module" }
// Script
pkg.browser = { src: 'path/to/script.js' }

to start with the basics?
pkg.browser = {
  type: 'module', // If omitted => script
  src: 'path/to/script.js', // I'm mandatory
  custom: 'option' // I'm not mandatory (webpack, rollup, ... Options)
  // ...add what you like to the {Object} (import.meta Style)
}

But we derailing the issue here... 😛

Moved issues to another tracker (https://github.com/stereobooster/package.json/issues/4, https://github.com/stereobooster/package.json/issues/2), because, indeed, this is not Browserslist issue. Thank you for your comments

The pkg.browser field should definitely be unofficially 'specced' by some entity and at best officially by npm

@michael-ciniawsky FWIW there's already this: https://github.com/defunctzombie/package-browser-field-spec


chore: Change timedelta(seconds=1) with >= to no timedelta with >, for easier reasoning

closes #313

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.159%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/sources/colombia.py
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
1967

Relevant Lines:
2283

💛  - Coveralls


raise Error when model  has attribute unicode which print some foreignKey value.

If A model has attribute __unicode__, foreignKey and print the foreignKey in __unicode__, when I import some new data, it will raise an Exception says RelatedObjectDoesNotExist: XX has no yy.
For example
## models.py
title = models.CharField(verbose_name=u'需求内容')
proposer = models.ForeignKey(User, verbose_name=u'需求提出人', related_name='proposer')
    def __unicode__(self):
        return u'%s %s' % (self.proposer.username, self.title )

After reading the code in django-import-export, I found something wrong as follow:
## resource.py
instance, new = self.get_or_init_instance(instance_loader, row)  # if row is new, will return an None instance
row_result.object_repr = force_text(instance) # try to get the foreignKey value, and raise Exception

instance, new = self.get_or_init_instance(instance_loader, row)  # if row is new, will return an None instance

The instance loader should always return an object for instance. If the row is new, then the only difference is new will be True. That's what the name of the function means: if the instance exists, get it, otherwise init a new instance.

Does it mean I need to override the init_instance , otherwise it will raise Exception when using __unicode__  for  new row

I'm not sure if you need to override init_instance or if it's necessary.
What exactly is raising an Exception? Do you have a callstack?

You can except the error in your model's  __unicode__ method as in #439. However it's probably better to only except the specific error instead of using the broad except with
from django.core.exceptions import DoesNotExist

Yet I think this is an issue with django-import-export and should be fixed at some point.

Thank  @JarnoRFB  and  @thauk-copperleaf  for replying.

@JarnoRFB I'm not sure how this could be fixed. In #439 was your proposal to not call Resource.save_instance() (which calls Model.save()) in Resource.import_row()?

I think my proposal in #439 is pretty much bullshit. I thought if the models are first instantiated in the dry_run and saved only in the real import they would have the related_model, but isn't that what is already happening?
Now I think just wrapping
row_result.object_repr = force_text(instance)

in a try  - exept  block would maybe do it. Or maybe the issue is with
instance, new = self.get_or_init_instance(instance_loader, row)

returning something strange for new rows as @GuihaiHU pointed out.

I think you're on to something there. I see that line twice inside Resource.import_row(), actually. That's a little wonky. I guess it's recording what the object is immediately since it might get deleted or skipped, and then again since saving it could change its representation (like in this case, with a field that is a foreign key). I suppose wrapping the first call in a try/except would be sufficient...


escape $_POST values (fixes #10)

fixes #10

Weird...I can't seem to replicate this bug. Do you think you could submit a short PHP file that contains the issue?
Thanks for contributing!

Here is the code that I am using to perform a post request on the php file
angular.module('myApp')
  .factory('formsApi', function($http) {
    function create(form) {
      return $http({
        method:   'POST',
        url:      '/api/contact_form.php',
        transformRequest: function(obj) {
          var str = [];
          for(var p in obj)
            str.push(encodeURIComponent(p) + "=" + encodeURIComponent(obj[p]));
          return str.join("&");
        },
        data:     form,
        headers:  {'Content-Type': 'application/x-www-form-urlencoded', 'Accept': 'application/json'}
      }).then(function(formReponse) {
        return formReponse.data;
      });
    }

    return { create: create };
  });

Lets say that I type the following code into a textarea
Hello, it's a beautiful day!
a part of the output in the console will look like this:
parse_str('_subject=Contact%20Request&message=Hello%2C%20it's%20a%20beautiful%20day!....', $_POST);

So it says something like this: ``parse_str('..content..'..still content....')`
// After escaping the ' it works fine and I can work with the values in my php code.

So even there are other possible ways to get around this bug (I set 'Content-Type': 'application/x-www-form-urlencoded' and did not try it with setting it to 'Content-Type': 'application/json'), it is still good to fix it.
Also I did not have time to try it with get requests. Maybe that bug can occur there as well.

Great, thanks for the contribution!


[nrf fromtree] logging: Changed log template to consider default log …

…level
Added using LOG_DEFAULT_LEVEL kconfig option in log module template for assigning the default log level to newly created module instance, instead of assuming always INFO level.
Signed-off-by: Kamil Kasperczyk<EMAIL_ADDRESS>(cherry picked from commit 6fe83ea810ca35ca21bc3bc460dbc0fde332a559)

Do you want me to cherry-pick it in #1096?

Hmm, if there is no problem with that, it's always nice to have it on main branch sooner than later. However I assumed that it will not go in, until upmerge ends, as it's not a bug fix.

@jfischer-no btw I don't know if sdk-nrf CI will pass on this PR, so maybe let's wait to not break upmerge PR unnecessarily.


Tests are not run as part of the chef_ca workflow.

:speaking_head: Foreword
Thank for taking the time to fill this feature request fully. Without it we may not be able to , and the issue may be closed without resolution.
:person_frowning: Problem Statement
We don't run test kitchen as part of the cookbook GitHub workflow.
:grey_question: Possible Solution
Run the cookbook using test kitchen and run the inspec tests.
:arrow_heading_up: Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
:heavy_plus_sign: Additional context
Add any other context about the problem here. e.g. related issues or existing pull requests.

Fixed by #3


Collapse database migrations for ATC 6

I'm submitting a ...

improvement request (usability, performance, tech debt, etc.)

Traffic Control components affected ...

Traffic Ops (database)

Current behavior:

Our database migrations have not been collapsed since #3524 (first included in 4.0.0).
New behavior:

We should squash database migrations in each major release, including the upcoming 6.0.0 release.

I'll take a look


First Steps for Native App Support

SetUp: new properties to select an app in SauceLabs
Supported first Steps:

Click  = <by_value>
Type
Type  = <by_value>
Assert  = <by_value> exists
Assert  = <by_value> contains
Switch to context <context_regexp>
Print contexts

Native App Support should be first implemented as a PoC for SauceLabs only.
Afterwards the driver_factory should be refactored anyways. Then we can combine the local Appium setup with the SauceLabs setup.

Closed with #63


fix: redirect to /login if can't fetch user profile

This fixes the blank page you see if you haven't logged in

Preview workflow is working now, but I don't think we're pushing the built app correctly.

thanks, will test it on #2


Problem when running on Ubuntu 16.04 and mono 4.2.1

Here's what I'm getting after starting the server. All dependencies were installed. Tried reverting mono to earlier version, but it's very problematic on 16.04. Downgrading to Ubuntu 14.04 and reinstalling with mono 3.2.8 solved this problem.
[registered] /v (Rocket.Unturned.Commands.CommandV/v)
[registered] /vanish (Rocket.Unturned.Commands.CommandVanish/vanish)
This server will automatically save every 1800 seconds
System.SystemException: Error running gmcs: Cannot find the specified file
  at Mono.CSharp.CSharpCodeCompiler.CompileFromFileBatch (System.CodeDom.Compiler.CompilerParameters options, System.String[] fileNames) [0x00000] in <filename unknown>:0
  at Mono.CSharp.CSharpCodeCompiler.CompileAssemblyFromFileBatch (System.CodeDom.Compiler.CompilerParameters options, System.String[] fileNames) [0x00000] in <filename unknown>:0
  at Mono.CSharp.CSharpCodeCompiler.CompileAssemblyFromFile (System.CodeDom.Compiler.CompilerParameters options, System.String fileName) [0x00000] in <filename unknown>:0
  at System.Xml.Serialization.XmlSerializer.GenerateSerializers (System.Xml.Serialization.GenerationBatch batch, System.CodeDom.Compiler.CompilerParameters cp) [0x00000] in <filename unknown>:0
  at System.Xml.Serialization.XmlSerializer.RunSerializerGeneration (System.Object obj) [0x00000] in <filename unknown>:0

Are there any problems ingame too?
Because I get this error to and use debian 8. I think you can ignore this error message.

Its not a issue... Just a ugly message ..

Actually I might have had directory structure wrong. Install script is a bit confusing as to the directories. Are you accepting pull requests ? Could probably improve a bit on this.

Sure, go ahead

My Unturned server is showing up on the server list, but always times out when I try to connect. This appears to be the only error message in the log. Can anyone tell if it's related or not, and hopefully how to fix it. It's on a Debian 8 DigitalOcean droplet and I've ran an Unturned server on it before without this issue, so that makes me think it's not a port forwarding issue. (but I may be mistaken.) thanks!


[Question] The usage of datasets during pre-training

Question
Thanks for your impressive work!
I just have a question about the datasets. The paper says "For pretraining, we use 50M image-text pairs randomly sampled from COYO-700M and CC12M . We use additional 500K image-text pairs from COCO training split to standardize the caption outputs". However, the code uses laion for pre-training. What is the difference in the choice of datasets?

Hi @CyrilSterling ,
The code currently has support for datasets in webdataset format, such as LAION, COYO, and CC. The usage of 'laion' in the code only refers to all datasets in webdataset format, not a specific LAION dataset. In practice, we reorganized the sampled data and stored them in a new folder named 'laion'.

Thanks very much for your reply!
And could you open source the reorganized 'laion' folder?

Oh, the data cannot be made open-sourced due to PCL's data management regulations. However, you can use any open-sourced image-text data like LAION and COYO.


hallo aschaffenburg | new app store pictures for Google Play and Apple App Store

Is your feature request related to a problem? Please describe.
The hallo aschaffenburg team has sent us new app store pictures to replace the current ones to reflect the current state of the content.
Describe the solution you'd like
Replace the old store pictures as I believe they are part of the automatic release pipeline.
Additional context
I have attached the files with the new pictures.
If there are any questions (or the pictures do not fit the app store requirements) please let me know.
If possible please try to have a release with the new pictures in January 2024 :) Thank you :)
Screenshots Android Phone.zip
Screenshots Android Tablet.zip
Screenshots iOS Phone 1242x2208.zip
Screenshots iOS Phone 1242x2688.zip
Screenshots iOS Tablet.zip

Thanks for your issue. Actually, the screenshots are not part of the pipeline for aschaffenburg so they can just be changed in the play console/app store connect. Sadly, apparently all of the screenshots have wrong sizes/aspect ratios. Do you think we should adjust them ourselves or you want to request fixed screenshots from aschaffenburg?

@steffenkleinle
Thank you for your response. Let me take over. I will ask Aschaffenburg to change those screenshots which do no fit and then chenge them in both stores :)


Metron can't reach doppler, fills up disk with logs

Hello,
I came in this morning and saw that one of my CF installations had failed over the weekend. I triggered it again, and saw another fast failure. When I investigated, the failed VM had no ephemeral disk left (out of 4GB), and the culprit was a set of metron backup log files, with repetitions of the following lines:
2017/07/16 20:40:02 Failed to dial first-az.doppler.service.cf.internal:8082: context canceled; please retry.
2017/07/16 20:40:02 grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp: lookup doppler.service.cf.internal on <IP_ADDRESS>:53: no such host"; Reconnecting to {doppler.service.cf.internal:8082 <nil>}
2017/07/16 20:40:02 error dialing doppler [first-az]doppler.service.cf.internal:8082: error establishing ingestor stream to [first-az]doppler.service.cf.internal:8082: rpc error: code = 14 desc = grpc: the connection is unavailable

I looked at consul's logs, and it may have been temporarily failing but was now unable to start due to the disk being full. I deleted all the backup logs and everything started successfully.
It would be helpful if this kind of failure resulted in fewer logs, so that the fix doesn't require manual intervention.
Thanks!
Raina
PCF Release Engineering

This has been resolved by #274.


Upsert content types and fields

I'd like to be able to update spaces that may be partially migrated, such that some content types and fields may already exist in the target space in one form or another. However the create and edit methods exposed by the API are exclusive and validation will fail if the space is in the wrong state. It would be nice to have "upsertContentType" and "upsertField" methods that continue the migration regardless of the whether the type/field already exists.
I tried rolling my own upsert functions with try/catch but that obviously doesn't work since the validator doesn't throw. Is there any way to accomplish this with the current API?
module.exports = {
	upsertContentType(migration, contentTypeId) {
		try {
			return migration.createContentType(contentTypeId);
		} catch (error) {
			return migration.editContentType(contentTypeId);
		}
	},

	upsertField(contentType, fieldId) {
		try {
			return contentType.createField(fieldId);
		} catch (error) {
			return contentType.editField(fieldId);
		}
	},
}

Hey @jody-zeitler
you're right that the validator doesn't throw, as this would result in a very annoying "one error at a time" feedback loop. We're trying to provide you with as much information about what is going on with your migration at once. That also means we don't execute every single step "as you write them", but chunk things up (e.g. we can't create a Content Type while you write createContentType as required properties like name are missing)
What is possible (but not documented before the release of the migration standalone lib) is to return a promise in your migration script. (This also enables you to use async/await in case you're on a Node version that supports it)
With this information you could build it yourself by fetching the Content Type in your migration and handle the existence of specific fields. Or catch the error if no Content Type exists).
@TimBeyer suggested something like this:
return sdk.getContentType('foo').then((ct) => {
  migration.editContentType('foo').createField('bar')
}).catch(() => {
  migration.createContentType('foo')
});

Is this sufficient for your use-case?
Best,
Stephan

Is sdk the contentful-management module? I see getContentType exists on Space instances. Is there any way to access the client that the migration is using so I don't have to sort out the spaceId and accessToken myself?

Ah true, this was an oversimplified solution. sdk should reference a space here, correct. So something like this first:
contentfulManagement.createClient(yourConfig).getSpace(yourSpace)

This is rather a workaround than longterm. I could potentially see passing in makeRequest to the migration function as a second parameter for such cases, or a config object with e.g. spaceId - this isn't decided yet!
Especially because it uses rawRequest directly rather than the convenience functions this would yield a difference experience than you're used to.
https://github.com/contentful/migration-cli/blob/7f50a16ea66f25cdecd86f241575a3fc302d8154/bin/contentful-migration.js#L80-L84
// with management SDK
space.getContentType('dog')
// all the convenience functions like contentType.publish()

// with makeRequest
request({
  method: 'GET',
  url: '/content_types/dog',
});
// only JSON object response

I've got something that should work, but I'm seeing some odd behavior when resolving Promises with migration ContentTypes.
util.js:
const {createClient} = require('contentful-management');
const yargs = require('yargs');

const argv = yargs
	.option('space-id', {alias: 's'})
	.option('access-token', {alias: 'a'})
	.demandOption(['space-id', 'access-token'], 'Provide a space ID and access token.')
	.argv;
const config = {
	accessToken: argv.accessToken,
	spaceId: argv.spaceId
};
const client = createClient(config);

async function getContentType(contentTypeId) {
	const space = await client.getSpace(config.spaceId);
	return space.getContentType(contentTypeId);
}

module.exports = {
	async upsertContentType(migration, contentTypeId) {
		try {
			const ct = await getContentType(contentTypeId);
			return migration.editContentType(contentTypeId);
		} catch (error) {
			return migration.createContentType(contentTypeId);
		}
	},

	async upsertField(contentType, fieldId) {
		try {
			const ct = await getContentType(contentType.id);
			if (ct.fields.some((fld) => fld.id === fieldId)) {
				return contentType.editField(fieldId);
			}
			throw new Error(`Field ${fieldId} does not exist.`);
		} catch (error) {
			return contentType.createField(fieldId);
		}
	},
};

model.js:
const {upsertContentType, upsertField} = require('./util');

module.exports = async function (migration) {
	console.log('get application');
	const application = await upsertContentType(migration, 'application');
	console.log('got application');
	application
		.name('Application')
		.displayField('displayName');
};

The event loop is exhausted and "got application" never prints. However if I short-circuit upsertContentType by returning a primitive like 'create' or {}, the message does print and errors out appropriately. I'm using Node 8.6.0.

@jody-zeitler I just published v0.3.0 which should fix your issue 🙂

Yes! The syntax is more awkward than I would like it to be, but it suits my needs.


Spaced repetition

Learning from flashcards is excellent, thanks for making this project. Other flashcards apps, like Anki, use a spaced repetition algorithm.
What do you think about implementing something like this?

Thanks for taking interest in my project!
I have been thinking about adding this feature in later updates but I stopped working on my KaiOS apps. In future I will rewrite the code and surely implement this.

This means is won't be implemented or that is already implemented?
I found dolphinsr to be interesting as it uses (almost) the same algo than Anki.

Yes, this feature was implemented in today's update. You can download new versuon directly from this repo or by using BHackers store.
Algorythm in this app fkr calculating days till next repetition is very simple: Math.ceil(levellevellevel*0.2)
"level" value starts from 0 and gets incremented every time you get the answer right and resets back to 0 when you answer wrong.


Add BigWarpLauncher

Hi @NicoKiaru
If you are interested you could have a look and comment how this compares to your approach.

Thanks for the suggestions!
Do you think it makes sense that I implement your suggestions into my code, which I am happy to do, or is it more efficient for us to use your code here?

Just copy whatever you need. No issue, I'd be pleased to try it myself but I'm still struggling to understand the logic behind behaviour and input/action map. I'm working on it, but for now I cannot contribute to your repo. Conversely, if you do not understand something in https://github.com/BIOP/bigdataviewer_scijava/blob/1c5c1e9fc06de0e93ccc8cb30de54a995ad2cc78/src/main/java/ch/epfl/biop/bdv/scijava/command/edit/register/BigWarpInitWithBdvSources.java , just let  me know.
But on a general manner, I think you/we need to decide what would be the way of passing an input which would be a list of  bdv Source.

One way to do it is to pass a BdvHandle and a String defining the index of the sources. -> good thing about this is that we know which BdvHandle they originate from. That was my choice. But it's not optimal. If the user removes a Source from a BdvHandle, then the indexation should change ? or not ? How to listen ?
Another way is to pass a List<Source> or a List<SourceAndConverter>, but then : are they attached to none/one/many BdvHandle

Generally speaking, how would you like to deal with multiple  bdv sources and multiple BdvHandle ?
Because a completely different manner is just to hold a list of bdv sources somewhere, independently of any BvdHandle, and then keep for each Source (or SourceAndConverter) a list of BdvHandle where they are displayed, in order to update when necessary.
I'm sorry I couldn't come in Dresden, so maybe you have discussed this before and made some choice already. Do you ? What have they been ?

One way to do it is to pass a BdvHandle and a String defining the index of the sources. -> good thing about this is that we know which BdvHandle they originate from. That was my choice. But it's not optimal. If the user removes a Source from a BdvHandle, then the indexation should change ? or not ? How to listen ?

For exactly this reason, Tobias is working on getting rid of the indexing logic and rather to refer to Source objects directly instead. However, when this code becomes useable for us it unclear.

Another way is to pass a List or a List, but then : are they attached to none/one/many BdvHandle

My preferred choice would be to pass bdvHandle and List<SourceAndConverter>. What would you think?

How do you get the SourceAndConverter from a bdvHandle? Like this?

final List< SourceState< ? > > sources = bdv.getBdvHandle().getViewerPanel().getState().getSources();

Another thing:

I'd be pleased to try it myself but I'm still struggling to understand the logic behind behaviour and input/action map.

The point here is that one does not have to understand this logic to contribute. The idea is to contribute pure functionality, such as this class: https://github.com/haesleinhuepf/bigdataviewer-playground/blob/master/src/main/java/sc/fiji/bdv/sources/read/SourceLoaderAndAdder.java#L21
How this is later put into the UI is another layer.
So, for example, if you provide a class:
class BigWarpSomething
{
BigWarpSomething( BdvHandle bdvHandle, List< SourceAndConverter > fixedList, List< SourceAndConverter > movingList )
}

This would be sufficient.

Like this?

final List< SourceState< ? > > sources = bdv.getBdvHandle().getViewerPanel().getState().getSources();

Exactly. And BigWarp can be initialized with SourceAndConverters. Looks like you can also transfer the ConverterSetup.

The point here is that one does not have to understand this logic to contribute.
Yes but I'd like to understand in order to add (on a separate repo) the scijava layer. But ok, it's not the most important right now.

My preferred choice would be to pass bdvHandle and List. What would you think?

This is probably unsufficient if your SourceAndConverter is displayed in several BdvHandle. An 'Observer` pattern could be an option : it's a way to synchronize multiple display for instance, if any of them is updated. (https://en.wikipedia.org/wiki/Observer_pattern)
I know it looks painful, but whether we want to be able to handle multiple BdvHandle is critical. We can make the choice of supporting only one (could be limiting : launching BigWarp from an existing BdvHandle already gives 3 windows)

it's a way to synchronize multiple display for instance.

I think we do should support this! Could you look how it is done in BigWarp?
I would go via a listener-observer pattern. ViewerPanel has some of those methods:
			/**
	 * These listeners will be notified about changes to the
	 * {@link #viewerTransform}. This is done before calling
	 * {@link #requestRepaint()} so listeners have the chance to interfere.
	 */
	protected final CopyOnWriteArrayList< TransformListener< AffineTransform3D > > transformListeners;

	/**
	 * These listeners will be notified about changes to the
	 * {@link #viewerTransform} that was used to render the current image. This
	 * is intended for example for {@link OverlayRenderer}s that need to exactly
	 * match the transform of their overlaid content to the transform of the
	 * image.
	 */
	protected final CopyOnWriteArrayList< TransformListener< AffineTransform3D > > lastRenderTransformListeners;

	/**
	 * These listeners will be notified about changes to the current timepoint
	 * {@link ViewerState#getCurrentTimepoint()}. This is done before
	 * calling {@link #requestRepaint()} so listeners have the chance to
	 * interfere.
	 */
	protected final CopyOnWriteArrayList< TimePointListener > timePointListeners;

	protected final CopyOnWriteArrayList< InterpolationModeListener > interpolationModeListeners;

...maybe at some point we could also implement a DisplaySettingsListener such that B&C changes could be communicated (I think this does not exist yet, does it?).

If you have the same SourceAndConverter in two windows, this is automatically done. Two issues nonetheless : 1 - both windows need to repaint. This is not automatically done, but you can 'drag' a little bit both windows to updat them. 2 - If you do it quick and dirty, the ConverterSetup is still attached to its original window, and not transfered to the second window. So you can set the B&C only from the first window... and if you close it, you loose it.

I would like to try another way of launching BigWarp on a different branch:
My idea is the following : not specifying BdvHandle as an input, but Sources only.
Then in the output, you will get:

The BdvHandle of the moving sources
The BdvHandle of the fixed sources
A way to get the current BigWarp transform

What's important in the associated BigWarp Command, is to put the newly created BdvHandles into the Object Service, so that you can work on it like any other Bdv Window
This should make everything accessible. Good for you ?

I would like to try another way of launching BigWarp on a different branch

Please do it, because my solution does not work at the moment as it would require a possibility to add a TransformListener to BigWarp, which is only possible in a version that we do not yet have as a maven dependency.

I'd suggest closing this branch because it has conflicts and got some limitations mentioned before. @tischer if you agree I let you do that.
See https://github.com/haesleinhuepf/bigdataviewer-playground/pull/39 for an alternative option on BigWarp integration.


Multitargeting Support

Added .NET 8.0 and .NET 6.0 LTS releases
Removed .NET 7.0 as it is no longer supported by Microsoft

Hey @robertmclaws, thanks for the interest. 😊
The wrapper is not done, so I tend to wait until I have covered all the planned API parts before changing the version and only releasing incremented alpha versions until this is done. This enables me to make breaking changes while I'm still implementing all the base parts of the wrapper in case I find out that other parts of the API make it necessary to change the existing parts.
Blazor.WebIDL, a dependency of Blazor.DOM, uses features only available in .NET 7 and forward, so the library cannot multi-target for .NET 6. So, to keep this from breaking people who might already have used this in .NET 7 projects, this will have to keep targeting .NET 7.
Next, on whether to target .NET 8. Targeting newer versions of .NET can improve performance, but for this scenario this is not the case. Any improvements to performance would be implemented in the dotnet/blazor runtime, which is not limited by which version of .NET this library targets. So, I don't think it would add enough value to warrant targeting .NET 8 as well.
Even though Microsoft no longer supports .NET 7, it is still okay to target this version as the lowest common denominator. There are still people using Blazor in .NET 5, and if I could have made support for this as well, I would have.
Lastly, on using version ranges for the Microsoft.AspNetCore.Components.Web. I would rather use a specific package version for references to ensure the package works for that specific version. This does not stop others from targeting newer versions. Additionally, targeting a range can cause dependency changes when releasing a new version of the package without specifically having specified that a new version should be used. Normally, this is not a problem, but it could cause unexpected behavioral changes unrelated to other changes made in a new library version, which I would like to avoid where possible.
A lot of this is obviously my personal preferences, which I should have made clear through a CONTRIBUTOR.md file. I will try to incorporate the above points in my existing template for a CONTRIBUTOR.md file and add it to the repository very soon.

So, thanks for the detailed info. The problem with pinning a specific version of Components.Web is that pulling in this NuGet package also tries to install the .NET 7 version in my .NET 8 app, when I then have to override manually.
Even if you use the technique to keep .NET 7 & .NET 8 support, you still need to be targeting the right version of your dependencies to the right versions of the Framework, otherwise you will have unexpected side-effects.

As it is a part of all Blazor templates to install the appropriate version of Microsoft.AspNetCore.Components.Web for your version of .NET, I expect that all have already referenced a specific version of the package before adding my library to their project. My project should not enforce that people use the correct version of Microsoft.AspNetCore.Components.Web.

It does when you're using it in a library and not part of an app, and up to that point you have not needed to reference that NuGet package before.
I see what you are saying, however NuGet packages should (and most do) enforce that you use the right package for the version of the framework you are using. That's why the technique exists.
I just want to point out that I submitted the PR because this library messed up my library when I tried to install it.


Update package.json

Updating ts-node version to ^10.7.0 and typescript to ^4.6.4
Issue #, if available:
https://github.com/aws-samples/amazon-cloudwatch-alarms-repeated-notification-cdk/issues/4
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Sorry for the delay. Thanks @weijing24 for the PR.


Infinity and MatrixVerb

Hello,
a user from Linuxmusicians mentioned these two reverb plugins from Airwindows: Infinity and MatrixVerb. Both can be used for sustained reverberation, which is a bit different from what the reverbs already available in LV2 format offer, AFAIK. I suggest inclusion. Thank you in any case, as usual! :smile_cat:

MatrixVerb is now ported with 492c3de46720bc2e0772f1afaf4cbcb09382a5f1 and it will be available in the next release. I wasn't able to get it to pass my null test. But I guess that's due to the way the plugin works. I'm quite confident that it sounds the same as the original MatrixVerb.

With 1d64d145be04d4aeceae8143e74d2486c2432813, Infinity2 should also be ported :)


Creation process hangs pub-sub

Required information
Operating system:
Win 11
Microsoft Windows [Version 10.0.22631.3235]
Rust version:
rustc 1.78.0-nightly
Cargo version:
Output of: cargo --version
iceoryx2 version:
main
Observed result or behaviour:
Im getting this error, seems like the creation process hangs:
...
   ix: ".dynamic", creation_timeout: 500ms,
              connection_suffix: ".connection" } }, def
              aults: Defaults { publish_subscribe: Publ
              ishSubscribe { max_subscribers: 8, max_pu
              blishers: 2, subscriber_max_buffer_size:
              2, subscriber_max_borrowed_samples: 2, pu
              blisher_max_loaned_samples: 2, publisher_
              history_size: 1, enable_safe_overflow: tr
              ue, unable_to_deliver_strategy: Block },
              event: Event { max_listeners: 1, max_noti
              fiers: 16 } } }, _phantom_data: PhantomDa
              ta<iceoryx2::service::zero_copy::Service>
               }
              | Unable to check if the service is avail
              | able since it is currently being create
              | d.
      231 [T] FileBuilder { file_path: FilePath { value
              : FixedSizeByteString<255> { len: 87, dat
              a: "C:\Windows\Temp\iceoryx2\services\iox
              2_8ac634184541a67f312ed9232f19c58ab4e7804
              1.service" } }, access_mode: Read, permis
              sion: OWNER_READ | OWNER_WRITE | OWNER_EX
              EC | OWNER_ALL, has_ownership: false, own
              er: None, group: None, truncate_size: Non
              e, creation_mode: None }
              | opened
      232 [D] "static_storage::file::Storage::does_exis
              t_cfg()"
              | Unable to check if storage "8ac63418454
              | 1a67f312ed9232f19c58ab4e78041" exists s
              | ince the underlying resources are curre
              | ntly being created or the creation proc
              | ess hangs.
      233 [D] BuilderWithServiceType { service_config:
              StaticConfig { uuid: "8ac634184541a67f312
              ed9232f19c58ab4e78041", service_name: Ser
              viceName { value: FixedSizeByteString<255
              > { len: 20, data: "My/Funk/ServiceName2"
               } }, messaging_pattern: PublishSubscribe
              (StaticConfig { max_subscribers: 8, max_p
              ublishers: 2, history_size: 12, subscribe
              r_max_buffer_size: 5, subscriber_max_borr
              owed_samples: 2, enable_safe_overflow: tr
              ue, type_name: "usize", type_size: 32, ty
              pe_alignment: 16 }) }, global_config: Con
              fig { global: Global { root_path_unix: "/
              tmp/iceoryx2/", root_path_windows: "C:\\W
              indows\\Temp\\iceoryx2\\", prefix: "iox2_
              ", service: Service { directory: "service
              s", publisher_data_segment_suffix: ".publ
              isher_data", static_config_storage_suffix
              : ".service", dynamic_config_storage_suff
              ix: ".dynamic", creation_timeout: 500ms,
              connection_suffix: ".connection" } }, def
              aults: Defaults { publish_subscribe: Publ
              ishSubscribe { max_subscribers: 8, max_pu
              blishers: 2, subscriber_max_buffer_size:
              2, subscriber_max_borrowed_samples: 2, pu
              blisher_max_loaned_samples: 2, publisher_
              history_size: 1, enable_safe_overflow: tr
              ue, unable_to_deliver_strategy: Block },
              event: Event { max_listeners: 1, max_noti
              fiers: 16 } } }, _phantom_data: PhantomDa
              ta<iceoryx2::service::zero_copy::Service>
               }
              | Unable to check if the service is avail
              | able since it is currently being create
              | d.
      234 [D] Builder { base: BuilderWithServiceType {
              service_config: StaticConfig { uuid: "8ac
              634184541a67f312ed9232f19c58ab4e78041", s
              ervice_name: ServiceName { value: FixedSi
              zeByteString<255> { len: 20, data: "My/Fu
              nk/ServiceName2" } }, messaging_pattern:
              PublishSubscribe(StaticConfig { max_subsc
              ribers: 8, max_publishers: 2, history_siz
              e: 12, subscriber_max_buffer_size: 5, sub
              scriber_max_borrowed_samples: 2, enable_s
              afe_overflow: true, type_name: "usize", t
              ype_size: 32, type_alignment: 16 }) }, gl
              obal_config: Config { global: Global { ro
              ot_path_unix: "/tmp/iceoryx2/", root_path
              _windows: "C:\\Windows\\Temp\\iceoryx2\\"
              , prefix: "iox2_", service: Service { dir
              ectory: "services", publisher_data_segmen
              t_suffix: ".publisher_data", static_confi
              g_storage_suffix: ".service", dynamic_con
              fig_storage_suffix: ".dynamic", creation_
              timeout: 500ms, connection_suffix: ".conn
              ection" } }, defaults: Defaults { publish
              _subscribe: PublishSubscribe { max_subscr
              ibers: 8, max_publishers: 2, subscriber_m
              ax_buffer_size: 2, subscriber_max_borrowe
              d_samples: 2, publisher_max_loaned_sample
              s: 2, publisher_history_size: 1, enable_s
              afe_overflow: true, unable_to_deliver_str
              ategy: Block }, event: Event { max_listen
              ers: 1, max_notifiers: 16 } } }, _phantom
              _data: PhantomData<iceoryx2::service::zer
              o_copy::Service> }, verify_number_of_subs
              cribers: false, verify_number_of_publishe
              rs: false, verify_subscriber_max_buffer_s
              ize: true, verify_subscriber_max_borrowed
              _samples: false, verify_publisher_history
              _size: true, verify_enable_safe_overflow:
               true }
              | Unable to open publish subscribe servic
              | e since the service hangs while being c
              | reated, max timeout for service creatio
              | n of 500ms exceeded. Waited for 504.852
              | 1ms but the state did not change.

Expected result or behaviour:
That it would run without any modifications
Conditions where it occurred / Performed steps:
ran the two binary pub sub example in README in both debug and release. Also tried changing the config to change the root path, did not help.

@EmilLindfors Thanks for the report.
This could happen when for whatever reason a process is non-gracefully killed. Could you please go into the directory c:\temp\ and remove the directory iceoryx2 and every file starting with the prefix iox2_ and retry again?
Currently, we are working on a mechanism to detect stale resources and clean them up automatically so that you do not have to do this manually. This should come in 0.3.0 beginning of April.
If this does not work, could you please attach the full log file so that I can dig into this.

Here's the error this time:
     Running `target\release\ox_r.exe`
        0 [D] FileBuilder { file_path: FilePath { value:
              FixedSizeByteString<255> { len: 20, data: "
              config/iceoryx2.toml" } }, access_mode: Rea
              d, permission: OWNER_READ | OWNER_WRITE | O
              WNER_EXEC | OWNER_ALL, has_ownership: false
              , owner: None, group: None, truncate_size:
              None, creation_mode: None }
              | Unable to open file since it does not exi
              | st.
        1 [D] Config { global: Global { root_path_unix: "
              /tmp/iceoryx2/", root_path_windows: "C:\\Wi
              ndows\\Temp\\iceoryx2\\", prefix: "iox2_",
              service: Service { directory: "services", p
              ublisher_data_segment_suffix: ".publisher_d
              ata", static_config_storage_suffix: ".servi
              ce", dynamic_config_storage_suffix: ".dynam
              ic", creation_timeout: 500ms, connection_su
              ffix: ".connection" } }, defaults: Defaults
               { publish_subscribe: PublishSubscribe { ma
              x_subscribers: 8, max_publishers: 2, subscr
              iber_max_buffer_size: 2, subscriber_max_bor
              rowed_samples: 2, publisher_max_loaned_samp
              les: 2, publisher_history_size: 1, enable_s
              afe_overflow: true, unable_to_deliver_strat
              egy: Block }, event: Event { max_listeners:
               1, max_notifiers: 16 } } }
              | Failed to create config since the config
              | file could not be opened.
        2 [W] "Config::get_global_config()"
              | Unable to load default config file, popul
              | ate config with default values.
< Win32 API error > C:\Users\emill\.cargo\git\checkouts\iceoryx2-1e1d4490bb91753f\9d8c061\iceoryx2-pal\posix\src\windows\unistd.rs:204 GetFileAttributesA(pathname as *const u8)
 [ 3 ] The system cannot find the path specified.

< Win32 API error > C:\Users\emill\.cargo\git\checkouts\iceoryx2-1e1d4490bb91753f\9d8c061\iceoryx2-pal\posix\src\windows\unistd.rs:204 GetFileAttributesA(pathname as *const u8)
 [ 3 ] The system cannot find the path specified.

< Win32 API error > C:\Users\emill\.cargo\git\checkouts\iceoryx2-1e1d4490bb91753f\9d8c061\iceoryx2-pal\posix\src\windows\stat.rs:34 GetFileAttributesA(path as *const u8)
 [ 3 ] The system cannot find the path specified.

        3 [T] Directory { path: Path { value: FixedSizeBy
              teString<255> { len: 33, data: "C:\Windows\
              Temp\iceoryx2\services" } }, directory_stre
              am: 0x1, file_descriptor: FileDescriptor {
              value: 0, is_owned: false } }
              | created
        4 [T] Builder { storage_name: FileName { value: F
              ixedSizeByteString<255> { len: 40, data: "8
              ac634184541a67f312ed9232f19c58ab4e78041" }
              }, has_ownership: false, config: Configurat
              ion { path: Path { value: FixedSizeByteStri
              ng<255> { len: 33, data: "C:\Windows\Temp\i
              ceoryx2\services" } }, suffix: FileName { v
              alue: FixedSizeByteString<255> { len: 8, da
              ta: ".service" } }, prefix: FileName { valu
              e: FixedSizeByteString<255> { len: 5, data:
               "iox2_" } } } }
              | Created service root directory "C:\Window
              | s\Temp\iceoryx2\services" since it did no
              | t exist before.
        5 [T] FileBuilder { file_path: FilePath { value:
              FixedSizeByteString<255> { len: 87, data: "
              C:\Windows\Temp\iceoryx2\services\iox2_8ac6
              34184541a67f312ed9232f19c58ab4e78041.servic
              e" } }, access_mode: ReadWrite, permission:
               OWNER_READ | OWNER_WRITE | OWNER_EXEC | OW
              NER_ALL, has_ownership: false, owner: None,
               group: None, truncate_size: None, creation
              _mode: Some(CreateExclusive) }
              | created
< Win32 API error > C:\Users\emill\.cargo\git\checkouts\iceoryx2-1e1d4490bb91753f\9d8c061\iceoryx2-pal\posix\src\windows\mman.rs:225 CreateFileA(shm_file_path(name, SHM_STATE_SUFFIX).as_ptr(),
    GENERIC_WRITE | GENERIC_READ,
    FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE,
    core::ptr::null::<SECURITY_ATTRIBUTES>(), CREATE_NEW,
    FILE_ATTRIBUTE_NORMAL, 0)
 [ 3 ] The system cannot find the path specified.

        6 [D] SharedMemoryBuilder { name: FileName { valu
              e: FixedSizeByteString<255> { len: 53, data
              : "iox2_8ac634184541a67f312ed9232f19c58ab4e
              78041.dynamic" } }, size: 570, is_memory_lo
              cked: false, has_ownership: false, permissi
              on: OWNER_READ | OWNER_WRITE | OWNER_EXEC |
               OWNER_ALL, creation_mode: Some(CreateExclu
              sive), zero_memory: false, access_mode: Rea
              dWrite, enforce_base_address: None }
              | Unable to create shared memory since it a
              | lready exists.
        7 [D] Builder { storage_name: FileName { value: F
              ixedSizeByteString<255> { len: 40, data: "8
              ac634184541a67f312ed9232f19c58ab4e78041" }
              }, supplementary_size: 370, has_ownership:
              false, config: Configuration { suffix: File
              Name { value: FixedSizeByteString<255> { le
              n: 8, data: ".dynamic" } }, prefix: FileNam
              e { value: FixedSizeByteString<255> { len:
              5, data: "iox2_" } }, path: Path { value: F
              ixedSizeByteString<255> { len: 25, data: "C
              :\Windows\Temp\iceoryx2\" } } }, _phantom_d
              ata: PhantomData<iceoryx2::service::dynamic
              _config::DynamicConfig> }
              | Failed to create dynamic_storage::PosixSh
              | aredMemory since a shared memory with the
              |  name already exists.
        8 [D] BuilderWithServiceType { service_config: St
              aticConfig { uuid: "8ac634184541a67f312ed92
              32f19c58ab4e78041", service_name: ServiceNa
              me { value: FixedSizeByteString<255> { len:
               20, data: "My/Funk/ServiceName2" } }, mess
              aging_pattern: PublishSubscribe(StaticConfi
              g { max_subscribers: 8, max_publishers: 2,
              history_size: 12, subscriber_max_buffer_siz
              e: 5, subscriber_max_borrowed_samples: 2, e
              nable_safe_overflow: true, type_name: "usiz
              e", type_size: 32, type_alignment: 16 }) },
               global_config: Config { global: Global { r
              oot_path_unix: "/tmp/iceoryx2/", root_path_
              windows: "C:\\Windows\\Temp\\iceoryx2\\", p
              refix: "iox2_", service: Service { director
              y: "services", publisher_data_segment_suffi
              x: ".publisher_data", static_config_storage
              _suffix: ".service", dynamic_config_storage
              _suffix: ".dynamic", creation_timeout: 500m
              s, connection_suffix: ".connection" } }, de
              faults: Defaults { publish_subscribe: Publi
              shSubscribe { max_subscribers: 8, max_publi
              shers: 2, subscriber_max_buffer_size: 2, su
              bscriber_max_borrowed_samples: 2, publisher
              _max_loaned_samples: 2, publisher_history_s
              ize: 1, enable_safe_overflow: true, unable_
              to_deliver_strategy: Block }, event: Event
              { max_listeners: 1, max_notifiers: 16 } } }
              , _phantom_data: PhantomData<iceoryx2::serv
              ice::zero_copy::Service> }
              | Failed to create dynamic storage for serv
              | ice.
        9 [D] Builder { base: BuilderWithServiceType { se
              rvice_config: StaticConfig { uuid: "8ac6341
              84541a67f312ed9232f19c58ab4e78041", service
              _name: ServiceName { value: FixedSizeByteSt
              ring<255> { len: 20, data: "My/Funk/Service
              Name2" } }, messaging_pattern: PublishSubsc
              ribe(StaticConfig { max_subscribers: 8, max
              _publishers: 2, history_size: 12, subscribe
              r_max_buffer_size: 5, subscriber_max_borrow
              ed_samples: 2, enable_safe_overflow: true,
              type_name: "usize", type_size: 32, type_ali
              gnment: 16 }) }, global_config: Config { gl
              obal: Global { root_path_unix: "/tmp/iceory
              x2/", root_path_windows: "C:\\Windows\\Temp
              \\iceoryx2\\", prefix: "iox2_", service: Se
              rvice { directory: "services", publisher_da
              ta_segment_suffix: ".publisher_data", stati
              c_config_storage_suffix: ".service", dynami
              c_config_storage_suffix: ".dynamic", creati
              on_timeout: 500ms, connection_suffix: ".con
              nection" } }, defaults: Defaults { publish_
              subscribe: PublishSubscribe { max_subscribe
              rs: 8, max_publishers: 2, subscriber_max_bu
              ffer_size: 2, subscriber_max_borrowed_sampl
              es: 2, publisher_max_loaned_samples: 2, pub
              lisher_history_size: 1, enable_safe_overflo
              w: true, unable_to_deliver_strategy: Block
              }, event: Event { max_listeners: 1, max_not
              ifiers: 16 } } }, _phantom_data: PhantomDat
              a<iceoryx2::service::zero_copy::Service> },
               verify_number_of_subscribers: false, verif
              y_number_of_publishers: false, verify_subsc
              riber_max_buffer_size: true, verify_subscri
              ber_max_borrowed_samples: false, verify_pub
              lisher_history_size: true, verify_enable_sa
              fe_overflow: true }
              | Unable to create publish subscribe servic
              | e since the dynamic service segment could
              |  not be created.

@EmilLindfors Ah I see what the problem is. The default temp dir entry in the toml file is pointing to C:\Temp\iceoryx2 but when the toml file is not found the default value is C:\Windows\Temp\iceoryx2
I created a PR to fix this: #152
Until the PR is merged, could you please remove all files in C:\Windows\Temp\iceoryx2 and all files starting with iox2_ in C:\Windows\Temp\. Then it should work.
When the PR is merged, the real default temp directory is used and everything is stored in C:\Temp\iceoryx2


Allow unlocking of keyring from greetd

Both LightDM and GDM include the login module where a keyring can be unlocked, for example by setting  security.pam.services.login.enableGnomeKeyring = true;.
I suggest adding the following:

text = ''
auth      substack      login
account   include       login
password  substack      login
session   include       login
'';

to
https://github.com/NixOS/nixpkgs/blob/d3a6adc9b6f11e260da168714ad8dcc938621610/nixos/modules/services/display-managers/greetd.nix#L59
and maybe hide them behind a boolean?
Maybe someone with more knowledge of the PAM system could give some feedback on this, as I am not sure if there could be negative or unforeseen consequences lurking.

@luc65r, you are mentioned as the maintainer so I tought I ping you on this :)

You can use security.pam.services.greetd.enableGnomeKeyring = true. But yeah a template for all those login managers makes sense.

I have had success with security.pam.services.greetd.enableGnomeKeyring = true

security.pam.services.greetd.enableGnomeKeyring = true works just fine for me, although IMO forcing people to do this manually is quite dumb — I would propose to enable this if services.gnome.gnome-keyring.enable is set to true.

Still cannot get this to work for the live of me. I'm using Hyprland with greetd but my default keyring remains locked on boot. My config hyprland.nix


Update README.md

Minor updates to README.md
What is this PR for?
Replace this line by a few sentences describing the overall goals of the pull request's commits.
What type of PR is it?
Replace this line by any of { Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring }
Todos
Remove this line and add todos if necessary. Otherwise put N/A here.

[ ] - Task

What is the related issue?
Replace this line with a link to the corresponding issue (open an issue at github). Otherwise put N/A here.
How should this be tested?
Replace this line with an outline the steps to test the PR here. Are UnitTests sufficient?
Screenshots
Questions:
Does the licenses files need update?
Replace this line by either YES or NO. Add details if required.
Are there breaking changes for older versions?
Replace this line by either YES or NO. Add details if required.
Does the change require additional documentation?
Replace this line by either YES or NO. Add details if required.

This has already been addressed.


Incorrect resources duration when containerd sets incorrect startedAt

Pre-requisites

[X] I have double-checked my configuration
[X] I have tested with the :latest image tag (i.e. quay.io/argoproj/workflow-controller:latest) and can confirm the issue still exists on :latest. If not, I have explained why, in detail, in my description below.
[X] I have searched existing issues and could not find a match for this bug
[X] I'd like to contribute the fix myself (see contributing guide)

What happened? What did you expect to happen?
When running Argo Workflows we occasionally see that our workflows report surprisingly high CPU and Memory resource durations.

We have traced this back to an issue with Containerd which will sometimes fails a Pod and for some reason sets the startedAt date to "the epoch" (ie: 1970-01-01T00:00:00Z).
...
"initContainerStatuses": [
{
  "containerID": "containerd://b55ff9eb8687235ae0fecd4216522b75b6f7a0fab5eca18de8c06365bd2636fc",
  "image": "quay.io/argoproject/argoexec:v3.5.11",
  "imageID": "quay.io/argoproject/argoexec@sha256:bb3938480cbe7a7c0f053eb77a5d3edb22c868bf57407bf066fd105961f26c72",
  "lastState": {},
  "name": "init",
  "ready": false,
  "restartCount": 0,
  "started": false,
  "state": {
	"terminated": {
	  "containerID": "containerd://b55ff9eb8687235ae0fecd4216522b75b6f7a0fab5eca18de8c06365bd2636fc",
	  "exitCode": 128,
	  "finishedAt": "2024-09-13T13:28:03Z",
	  "message": "failed to create containerd task: failed to create shim task: context deadline exceeded: unknown",
	  "reason": "StartError",
	  "startedAt": "1970-01-01T00:00:00Z" <---- here
	}
  }
}
],
...

This results in the incorrect calculation of a duration due to the function below which assumes that a startedAt data of 1970 is expected.
https://github.com/argoproj/argo-workflows/blob/25bbb71cced32b671f9ad35f0ffd1f0ddb8226ee/util/resource/summary.go#L16-L22
I appreciate this feels like more of a containerd bug that an argo-workflows bug, however I think it is worth adding some logic to handle this case more gracefully.
For additional context, we are running on top of managed GKE when we see this bug. I am not attaching a reproducible workflow because... well... this bug is challenging to reproduce!
I will propose a fix very soon and will let the maintainers decide whether it is worth patching. From our perspective it's worth it because we use the Argo Workflows reported CPU/Memory usage to bill downstream customers... 😨
Version(s)
v3.5.11
Paste a minimal workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
N/A

Logs from the workflow controller
N/A

Logs from in your workflow's wait container
N/A

Why isn't it possible for you to detect this case when you are collecting the data?
I'm uncomfortable with working around this bug in workflows. Have you managed to find or raise an issue in containerd?

We have traced this back to an issue with Containerd which will sometimes fails a Pod and for some reason sets the startedAt date to "the epoch" (ie: 1970-01-01T00:00:00Z).

I appreciate this feels like more of a containerd bug that an argo-workflows bug

Well that is a containerd bug. I think it makes sense for Argo to assume the data is accurate. If you assume unreliable dependencies, then most of the assumptions go out the door too; it's not really possible to handle all such cases.

however I think it is worth adding some logic to handle this case more gracefully.
I will propose a fix very soon and will let the maintainers decide whether it is worth patching.

Possibly, though that is a workaround rather than a fix. I don't have a strong opinion on it personally although I'd lean toward relying on upstream containerd to fix this.

From our perspective it's worth it because we use the Argo Workflows reported CPU/Memory usage to bill downstream customers... 😨

Huh, I don't think I've seen that pattern before. At my last job, we used KubeCost for that purpose, which also can have bugs, but is at least purpose-built for cost tracking, unlike Argo's calculations.

https://github.com/containerd/containerd/issues/9080

Yeah this does flag a Containerd issue. I appreciate the desire to keep fixes upstream where possible. It looks like Containerd is unlikely to address this anytime soon but will close this issue.


Difference between setup.sh/bash/zsh and local_setup.sh/bash/zsh

Could you explain the different use cases for sourcing setup.sh/bash/zsh vs local_setup.sh/bash/zsh?
It looks like the docs are using local_setup.bash now?

The local_setup files set the environment for the current workspace. It doesn't consider other parent workspaces. If there are any you need to source them manually using their files.
The setup files set the environment for all the parent workspaces first (by sourcing their local_setup files in the order they appear in COLCON_PREFIX_PATH when this workspace was built) and then sources the local_setup file of this workspace.


juliaup info crashes on MacOS Ventura 13.0

Hi,
I followed the instructions to use juliaup 1.8.10 , then change to the releasepreview channel juliaup self channel releasepreview, and that worked fine, but then when executing the juliaup info got the following problem:
name = 'Juliaup'
operating_system = 'unix:OSX'
crate_version = '1.8.10'
explanation = '''
Panic occurred in file '/Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/ureq-2.5.0/src/rtls.rs' at line 74
'''
cause = 'Failed to add native certificate too root store: BadDer'
method = 'Panic'
backtrace = '''

   0: 0x1082a6a85 - core::result::unwrap_failed::ha988429942445917
   1: 0x108226069 - core::ops::function::FnOnce::call_once::ha0d6f48b9a614fb5
   2: 0x108121612 - once_cell::imp::initialize_or_wait::hca53ff2183dd744c
   3: 0x1082b17df - once_cell::imp::OnceCell<T>::initialize::h3f1d155e6e2aef41
   4: 0x1082231f2 - ureq::agent::AgentBuilder::new::hb1a69231fb9fd917
   5: 0x1080fae98 - juliaup::operations::get_ureq_agent::h8ca3e86b253070b5
   6: 0x108106657 - juliaup::operations::download_juliaup_version::h1a7e6db0c6685e7c
   7: 0x1080109b0 - juliaup::main::hf9faef8b0b0035cc
   8: 0x107ff3916 - std::sys_common::backtrace::__rust_begin_short_backtrace::h2ccf0636b6322bbe
   9: 0x107ff3a32 - std::rt::lang_start::{{closure}}::hbaf7903ca9db3d7f
  10: 0x108032e02 - _main
  11: 0x7ff8178c4310 - <unresolved>'''

I'm working on MacOS Venture 13.0. Let me know if more info is needed.

I found https://github.com/pact-foundation/pact-reference/issues/156 and https://github.com/rustls/rustls-native-certs/issues/15 that sound somehow related... Not really clear to me what is going on or how we can fix this, but we probably should...

@davidanthoff If I understand the following discussion correctly, the reqwests crate had a similar issue:
https://github.com/seanmonstar/reqwest/pull/1316
I suspect a similar root-cause in ureq and that the solution would follow the lines of whatever reqwests did. The first step may be opening an issue in the ureq repository and pointing to this issue and the reqwests PR.
What do you think?

Yes, from my (very quick and sloppy) read I came to a similar conclusion, i.e. this probably needs to be fixed in ureq. If you have the bandwidth (and want) to engage with the ureq folks that would be fantastic :)

I'll set up an issue and give some context. Maybe the author is already aware of this or knows of a quick way to improve this / can adopt the reqwests solution.

Good news:
https://github.com/algesten/ureq/pull/564
The ureq maintainers reacted very quickly and already have a PR open. Once the PR is merged, we could update the version bounds and see if that fixes the issue.

I just tried juliaup info on an M1 Mac mini with juliaup version 1.8.12 and macOS 13.0.1 and there was no crash. Maybe it's fixed?

Yes, it seems to be fixed on juliaup v1.8.12 Thank you!


Add link to configuration.yaml in README

configuration.yaml provides, in particular, description of logging options that is useful to bring it closer to a user, so s/he would be able to quickly find it. Requested by QA.

bors r+


DXF/DWG .ini notification pops up each time

Create a flag that only allows for one notification per procedure.

Flag created (Version <IP_ADDRESS>)


All Browsers - Config Files in Git

Please name the task using the following information: Browser - Name. Give this issue a "FED" or "BED" label, alongside the "Bug" label.
Description
Config files and passwords are currently checked into Git. This was originally a result of the team not having admin access to the server (stated by Gregg). This bug includes externalizing all config files as necessary.
https://github.com/GCTC-NTGC/TalentCloud/tree/master/public_html/tc/config
Steps to Reproduce
Steps to reproduce the behavior:

View files on Github.

Completed during refactor


fix(app, robot-server): notify on maintenance run status change

Overview
closes RQA-2941 and RQA-2940.
listen to run status change for maintenance runs and show error in app when failed calibration.
Test Plan and Hands on Testing
explanation in ticket.
Tested on the Flex, works as expected.
Changelog

added a callback to handle status change for a maintenance runs.
pass in status to gripper wizard flow.

Review requests
changes make sense?
Risk assessment
low.

they

funny you say that! @SyntaxColoring and I spoke about wanting to refactor and merge runs and maintenance runs in the future. @mjhuff
will test tomorrow all calibration flows and will fix accordingly. not merging for now.


Videos are not playing.

I use Jtube on Nokia E6 Symbian Belle Refresh. Jtube homescreen loads as normal but videos are not playing. I tried reinstalling but it did not help. It says General: System Error.

Turn off second option in network settings


[events] Add support for actedUpon as filter

Description
Currently, events filter doesn't support filtering based on actedUpon ids (entities on which the event was performed).
This would be helpful in adding support of filtering events for Workspaces.
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

@senali-d @yash37158

// @leecalcote


resetUploader for customImageHandler isn't working

Hi.
I'm doing the example to upload images through customImageHandler, but I can only upload the same image one time if I try it consecutively.
The example with resetUploader option isn't working, despite I've installed the last version.
Any ideas?
Thank you.

Hi.
I found what's the problem.
The file upload isn't cleaned because I had multiple editors in the document, then there are multiple file inputs with the same name 'file-upload'.
Any ideas how to solution this?
Thanks.

Hi there. I'm having trouble recreating this issue.
Is this the example you were following?
-> https://www.vue2editor.com/examples/#custom-image-handler
Also, can you provide anything else such as a code example in CodeSandbox, link to a repo, screenshot, etc?


Color of text for today's date

Seems like this documentation should include what controls color of text (day of month) for the highlighted day.
Also, the function of the Theme on the calendar should probably be in here too(?)

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

ID: 95dceb94-5142-17e4-ef8a-53fab1966ece
Version Independent ID: a2c84e18-8581-6816-bc01-3f5761c92021
Content: Reference for the calendar-screen template for canvas apps - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/screen-templates/calendar-screen-reference.md
Service: powerapps
GitHub Login: @emcoope-msft
Microsoft Alias: emcoope

Thank you for your feedback. Content is updated, please allow some time to reflect the changes.


Markdown Tables Failing to Render

Describe the bug
Tables in markdown are not rendering as expected.
To Reproduce
Steps to reproduce the behavior:

npm link cicero-ui (after this PR) to template-studio-v2
Open Volume Discount

Screenshots

@irmerk CommonMark does not have tables. Those | things are just text.

@irmerk CommonMark does not have tables. Those | things are just text.

(I don't think any of our UI components have table support either? am I being mistaken?)

Ah that could be the case. I was thinking it would render like this https://www.markdownguide.org/extended-syntax/#tables

Ah that could be the case. I was thinking it would render like this https://www.markdownguide.org/extended-syntax/#tables

That would be nice!
For reference, there is an issue in Cicero about supporting tables in CiceroMark. https://github.com/accordproject/cicero/issues/472
Loooong thread about tables in Markdown and CommonMark is here: https://talk.commonmark.org/t/tables-in-pure-markdown/81/160

This issue does seem to be about this project. I'm moving it to TSv2.


FR: Support custom JWT token creation with ApplicationDefault credentials

Right now it's not possible to create custom JWT token with firebase-admin SDK on Java and this leads to the problems of credentials confidentiality.
The most efficient way to obtain credentials while you're on AppEngine or GCE - obtain them via GoogleCredential.getApplicationDefault(), or in case of Firebase - FirebaseCredentials.applicationDefault(), but FirebaseAuth.createToken lacks support for this type of credentials.
Implementing application default credentials support for custom JWT tokens will help people not to save their credentials in the VCS or gamble with cumbersome environment variables.

This is actually a popular feature request. The problem is application default credentials often don't expose a private key, which is required for token signing. We have communicated this requirement to the core Google cloud team on multiple occasions, and there is some work underway to provide a working solution. We will follow up and try to provide an update soon.

@hiranya911 thanks for update. We are really waiting for this feature. I hope it could be implemented soon.

Hello guys,
Are there any updates the the feature?

Finally making some progress on this one. Feature under development in the hkj-jwt-sign branch.

This has been released.


Create WindowsToolchain.md

Add some documentation explaining the installed components and how the toolchain installation is structured.  Add some details about the packaging components.

@swift-ci please smoke test

@swift-ci please smoke test


Transform export

Include the command or code you used.
const Json2csvTransform = require('json2csv').Transform;

Include a sample dataset that we can test against.
any

Include your output/error.
Transform not found

I looked at the code installed by npm and the code in master and Transform is not exported. I will use json2csv-stream instead and I will be happy to to issue a PR with an updated readme, but I want to make sure I understand the expectations. Is this supposed to be part of the feature set? Is the new-api branch's readme the correct one?
Terry

Are you testing against the master branch version of the library (we haven't released a new version to npm yet)?

I just published an alpha version 4.0.0-alpha.0

cc @juanjoDiaz

Yup, sounds like using the old version of the library. Version 4.X should solve it.
Any reason to publish alpha instead of publishing 4.0.0 directly? What are the steps to move away from alpha?

Just wanted to do a quick release that we can try out before doing a 4.0. Mainly to catch anything we missed without affecting too many people.

I'm running this now. It has not caused any issues yet and the readme matches the API I am seeing. Thanks for the quick response. If there is anything further I can do to help, let me know.
I assume this should stay open until 4.0 is released?

v4.0.0 Released


Some Proxy Hosts Have No logs

Hi,
I have 7 proxies set up in NPM with only 2 disabled. That leaves me 5 proxies in use but in the log folder it only has files for 4 of the proxies. Among those files only 2 proxies have actual log entries.
So the 5 services are 2 PhotoPrism, Guacamole, Bookstack and Ghost. Only 1 Photoprism instance and Bookstack have actual content in the access log.
NPM is running in docker (jlesage/nginx-proxy-manager) version v1.26.1

Insert stay now... Bluetooth wifi...
On Sun, Jun 5, 2022, 2:27 AM HT @.***> wrote:

Hi,
I have 7 proxies set up in NPM with only 2 disabled. That leaves me 5
proxies in use but in the log folder it only has files for 4 of the
proxies. Among those files only 2 proxies have actual log entries.
So the 5 services are 2 PhotoPrism, Guacamole, Bookstack and Ghost. Only 1
Photoprism instance and Bookstack have actual content in the access log.
NPM is running in docker (jlesage/nginx-proxy-manager) version v1.26.1
https://hub.docker.com/layers/nginx-proxy-manager/jlesage/nginx-proxy-manager/v1.26.1/images/sha256-884cb243f4372cbfcd1d1ee4979b362e9f6126e98feb7d36e121a1b5248a973f?context=explore
—
Reply to this email directly, view it on GitHub
https://github.com/NginxProxyManager/nginx-proxy-manager/issues/2094,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVMDK3X7IOGKEROLI2OPQT3VNRXGPANCNFSM5X4Y4V7A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Since you aren't using jc21's docker image, perhaps you should ask in jselage's github also.
But there are many reasons that you think nginx isn't logging but is somewhere you aren't finding it.

Is buffering turned on? it will mean there may be a very large lag between website hit and writing logs. Check the proxy_host/*.conf files for access_log section
If the app uses websockets, the majority of the hits will not appear in access logs and I don't think NPM sets up websocket logging.
Access log names have changed over time, make sure by checking the filesystem. Old and new sites can write as different naming styles.
Check the /data/nginx/proxy_host/*.conf files and check where they are being written to, perhaps the location has changed.
Is this an issue with docker/host filesystem syncing? Check both inside the docker (using docker exec) and on the docker host.


site crashing due to bug in error dispatch actions

Some states in the users and posts reducer are failing when there is a server error(500) as we are dispatching error to state, but we are not setting the arrays to empty arrays. And in the components we are trying to map over the arrays which are not defined
To Reproduce

this happens only if there is an internal server error

Solution

either we can set empty arrays when we are dispatching errors in reducers
or we can check for errors before mapping over the arrays in the components

There is no Database in Backend server

Ohh Okie... but still I think it needs to be fixed.


choices property is not recognized

Reading "the docs" of the argparse it says there is a property on every argument called choices, where you can set wich values are accepted for that argument, but that property doesn't exist

My bad, i saw the documentation covered on the docs if tor the luarocks version, getting that version now!

The luarocks package doesn't seem to have the choices() function either, and both the github code and luarocks code are version 0.6.0

Version 0.7.0 is at https://luarocks.org/modules/argparse/argparse


Add check, which branch was cloned

For improved debugging `git statusˋ prints, which branch was cloned.

Hi @s-martin
could you rebase, there is a conflict now. Thanks for your contributions!

Just did (-:


Improving UI of Contact section

UI of contact section must be improved , placeholders and icons should be aligned correctly .

@skmirajulislam  please assign me this issue under gssoc-extended , hacktober fest accepted and with appropriate level.

@skmirajulislam  please assign me this issue under gssoc-extended


Removing note about Humble performance (fixed in H.2.2)

close #475

Notes about startup time on Humble release pages remain.


DSD-1052: Initial Footer component

Fixes JIRA ticket DSD-1052
This PR does the following:

Adds the initial version of the DS Footer component.
This replicates the current production footer that exists on NYPL.
Concerns that will be new tickets:

The DS social media icons don't match with production icons. There is already a ticket to get the new icons into the DS.
The media query breakpoints are different. Production uses these four: "320px", "768px", "1024px", "1500px". The DS uses these four: "320px", "600px", "960px", "1280px", "1536px". While not vastly different, the visual changes are noticeable. We should discuss this for a separate PR but do we want to replicate the legacy breakpoints and import them into the DS, or ignore them and use the current ones in the DS?

This is the React component. A solution still needs to be created for the static HTML/CSS version. This is TBD in a separate ticket.

How has this been tested?
Locally.
Accessibility concerns or updates

N/A

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

One thing that I noticed as I was comparing with the footer on NYPL.org is that we use font-smoothing, whereas they do not. Before we roll out the Header and Footer components, we may need to make a push to have that attribute added to other projects. It would be weird to have a page with mixed font treatments.

I don't have a keen eye to notice this right away... can you make a ticket for this?

Why are the breakpoints so different between Reservoir and DXP? Are the breakpoints you are referring to the same as what Bill is using in Scout? If so, how does he deal with the discrepancies?

The breakpoints from the Header/Footer are from the older design system based on what was at that point, the most common breakpoints for all devices. These have change since now devices have vastly different sizes. It looks like Scout is using the DS breakpoints: https://github.com/NYPL/dxp-react-search/blob/development/src/styles/base/_breakpoints.scss
Instead of importing the Header/Footer breakpoints, we are building the two new components with the DS breakpoints. That's the only big discrepancy between the old and new (besides the font treatment).

Updated. Got the image by alt text now in the unit test.