talksapp: accept markdown-based present files for talks.godoc.org

CL https://golang.org/cl/222846/ (and following) added support for markdown-based present files (as per https://github.com/golang/go/issues/33955)
it would be great if the present instance running at talks.godoc.org was updated to accept them as well.

@dmitshur was this the right place to file that issue?

Sorry I missed this.
This is indeed the right issue tracker for tracking the work for talksapp.

ping?
AFAIK, it's still using the old, non-markdown compliant, code...

That is true, the work of updating talksapp hasn't been done. I'm not aware of someone planning to do this work soon.
Another factor to consider here is that it's not easy to deploy a new version of talksapp now because its deployment script hasn't been kept up to date, so resolving this issue will require also getting talksapp deployment fixed.

thanks circling back on this.
what would involve updating that deployment script? (ie: could this be done by a non-googler?)

It should be possible, although it's harder because non-Googlers won't have access to the GCP project it's currently deployed to test everything works. But it can be deployed to another project for testing purposes.
The deployment app.yaml file is in talksapp directory. See commit https://github.com/golang/gddo/commit/9bd4a32950217197bb319565e843237cca7f85d2 for when it was done last time. This time, there are more changes to make because of changes between Go 1.9 and 1.11 runtimes, as documented at https://cloud.google.com/appengine/docs/standard/go111/go-differences.


Local dependencies options

This adds some options when uploading local dependencies to the Image registry
Thanks !

Release Note
NONE

@johnpoth E2E failed, which appears to be caused by this pull req.
--- FAIL: TestImageRegistryIsAMavenRepository (14.52s)
    --- FAIL: TestImageRegistryIsAMavenRepository/image_registry_is_a_maven_repository (0.41s)

@johnpoth E2E failed, which appears to be caused by this pull req.
--- FAIL: TestImageRegistryIsAMavenRepository (14.52s)
    --- FAIL: TestImageRegistryIsAMavenRepository/image_registry_is_a_maven_repository (0.41s)

Thanks ! Yeah I saw that ! The sign of good tests and a bad commit :D will fix this ASAP


Add gap, columnGap, rowGap to style

See https://reactnative.dev/blog/2023/01/12/version-071#simplifying-layouts-with-flexbox-gap

Yes, indeed, sorry for the duplicate.


Reduce amount of CI tasks

Is your feature request related to a problem? Please describe.
Fraud detection mechanism of Cirrus CI is triggering on this repo. It thinks you mine some crypto coins. 😅
Describe the solution you'd like
Two possible things come to mind:

Combine some of the tasks if possible to reduce overall CPU footprint of a single change. Two matrixes which results in 38 tasks with 4 cores for each change.
Maybe run only a few configurations for PRs and all for master branch? Do you really need both Release and Debug?

/cc @cjdb

Fraud detection mechanism of Cirrus CI is triggering on this repo. It thinks you mine some crypto coins. 😅

Oh dear. That's not good. Thanks for reporting this! 😄

Combine some of the tasks if possible to reduce overall CPU footprint of a single change. Two matrixes which results in 38 tasks with 4 cores for each change.

I think this is the preferred option, but I might need some help.
This is a project template, which I'm planning to use as a starter for other projects. I can dial down the number of cores on this project to 2 (if it's still a resource sink, I can go to 1, but I'd rather have two to show that parallel building is working). This project is rather cheap to build: it only has a few CMakeLists.txt and a few C++ regression tests. The CI is designed to prove that all of the configurations are supported, so that derivative projects can choose what is appropriate for them. Derivative projects will be significantly more expensive to build, and might even benefit from using all eight cores across everything that I plan to support (I still need to add support for Windows, Windows using MinGW, and Mac OS). That is beyond my control, but I'm open to looking into ways to lead by example here, which is what my .cirrus.yml is intended to serve as.
Both debug and release builds are necessary, so I'll combine those, to halve the number of jobs created. I'll also leave this open until the remaining platforms are added: can you please keep me updated on this?

I'll whitelist your account in the meantime and will take a closer look a little bit later today. 🙌

@cjdb could you please explain to me what affects what conan is installing? I noticed that's what takes the most time in CI. I wonder if it's possible to use Docker files to cache an environment with everything pre-installed. Cirrus CI has a pretty cool feature for that.

Conan installs all of the necessary package dependencies for a project, and is controlled by the top-level file conanfile.py.
The packages need to be installed once per toolchain, per build type (so GCC 8 Debug and GCC 8 Release are treated independently). I don't know if it's possible for this to be effectively cached, but I'll look into it.

@fkorotkov I think I have worked out why conan install is taking so long. After some light investigation, it seems that only the clang-8* profiles are taking extreme amounts of time to install, and this is likely due to the fact that one or many of the packages haven't yet been built remotely for Clang 8: the install component is instructed to build any missing binaries, and given the amount of time taken to install, I'm wagering that the Boost libraries are what's chewing up time. For the record, LLVM 8 is still bleeding-edge: it hasn't even been officially released yet, so this is not surprising after having thought about it.
I see a few options to minimise the amount of time spent building libraries until this is fixed upstream:

Do nothing, bear the cost for now. This doesn't impact me that much, but it might have ramifications for other users.
Use eight CPUs to build. This will temporarily strain the system (does 19 jobs * 8 cores cause the same problem as 38 jobs * 4 cores?), but will also theoretically cut the total amount of time spent using Cirrus hardware from eight minutes to four minutes.

Is it possible to have only certain jobs be spun up with eight cores? If just those three can go to eight cores, then it might be okay.

@cjdb I think #6 did the job of reducing the overall amount of CPU minutes. The fraud detection is checking for overall amount of CPU minutes: 8 cores for 1 minute is equal to 2 cores for 4 from that perspective. So if you don't have 4x time reduction then it's kind of doesn't make too much sense.
Thank you so much for addressing the concerns! 🙌


combine soft body sim with other sim envs

It is said in the paper that we can combine any rigid simulator with the soft body simulator.I would like to combine the soft body simulator with mujoco. Could someone tell me how to achieve this? Thanks

This would likely be highly non-trivial amount of engineering work to do that. Perhaps @fbxiang could summarize some helpful points

The principle to couple multiple simulators can be straightforward. You just need to repeat the following steps.

step simulator A
step simulator B
compute forces resulted from any interactions between simulator A and B
apply the forces to A and B

In practice it would require deep understanding of both simulators and quite a bit of work as described in the paper. If you really want to make your own coupled simulator, you could take the soft body environments in ManiSkill 2 as an example.


One or more plugins require a higher Android SDK version.

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
flutter run --no-sound-null-safety
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 3.3.0-0.5.pre, on macOS 13.2 22D49 darwin-arm, locale zh-Hans-CN)
[!] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[!] Xcode - develop for iOS and macOS (Xcode 14.2)
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without CocoaPods, plugins will not work on iOS or macOS.
For more info, see https://flutter.dev/platform-plugins
To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.
[✓] Chrome - develop for the web
[!] Android Studio (not installed)
[✓] VS Code (version 1.75.1)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
Actual results
Warning: The plugin path_provider_android requires Android SDK version 33.
For more information about build configuration, see https://docs.flutter.dev/deployment/android#reviewing-the-build-configuration.
Warning: The plugin shared_preferences_android requires Android SDK version 33.
For more information about build configuration, see https://docs.flutter.dev/deployment/android#reviewing-the-build-configuration.
One or more plugins require a higher Android SDK version.
Fix this issue by adding the following to /Users/cupid/androidProjects/zhaoShang/invest_moudle-master/sg_module/.android/Flutter/build.gradle:
android {
compileSdkVersion 33
...
}
Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

<!-- Paste your output here -->

@helloworkcupid
As the message gives you solution on how to fix the log that you are seeing, you need to follow it and fix to see it working.

It was just an error caused by the upgrade of a plug-in. Later, I returned the upgrade, but the error was still there, which I could not understand. So, I don't think it's such a superficial reason.

@helloworkcupid As the message gives you solution on how to fix the log that you are seeing, you need to follow it and fix to see it working.

It was just an error caused by the upgrade of a plug-in. Later, I returned the upgrade, but the error was still there, which I could not understand. So, I don't think it's such a superficial reason.

@helloworkcupid
I see that you are on old version of beta. Maybe try to upgrade to latest master or stable version and make sure all of the plugins you are using are uptodate.


Is it possible that we can display things other than images inside the alert?

Hi @ivanvorobei
I'm trying to use this SPAlert to display a loading indicator, but for now, it only accepts a couple of default presets and a custom UIImage. Is there any changes that we could pass something like
CAShapeLayer in it? For example https://github.com/relatedcode/ProgressHUD/blob/master/ProgressHUD/Sources/ProgressHUD.swift#L743-L785
I know I can probably address this with a custom loading image. But with so many different screen resolutions, I'm not sure if it's a good idea to go about :(
Thanks for creating and publishing OSS like this :)

Hello!
If need, I can add indicator preset

Sorry for big delay. New feature added in 3.5.0 version, called preset .spinner. Check readme for me details please.


Refactors and bugfixes

timedelta_to_float helper
patch raise statements
update var names to align with conventions
type annotations

Wanting to wrap some more refactorings into here

I didn't fix the #1738 into this. I wanted to do a bit of a deeper refactoring of .reltime and the TimeConverter class in general, but was running into some roadblocks


fix: Fix logPath handling

Currently, it's not possible to set an empty value for the LOG_PATH environment variable because the related Viper config setting is missing.
Additionally, the stlib.log file isn't actually being written to logPath.
This PR fixes both, making it easier to run the container image with restricted permissions.

Hi, sorry for the delay in the response.
Could I ask you to pull request from the "latest" branch, I would like to release this as an hotfix and currently master already has features from a next release.

Sure thing 👍

I've rebased off upstream/latest and changed the PR base to latest as well. If there's anything else, let me know


sequencer: multisig accounts

a potential alternative to #833 for bridge withdrawals is to implement multisig accounts. these will be a new account type that can be initialized with n pubkeys, where m of these keys are required to sign any transaction for it to go through.
questions: do all signatures need to be aggregated into one tx, or can they be submitted separately? for the former, the ux will not be much different from off-chain threshold signatures. for the latter, we will need to figure out timeout/invalidation strategies.

This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue
be closed in 7 days.

This issue was closed because it was stale


generate .pb file for yolov4 weights ?

Can this be used to generate .pb file for yolov4 weights ?

Not sure about what you meant. The protocol works for any model.

yolov4 weights convert to .pb file ?

This is very ambiguous and I could answer.


Flash Attention 1 request

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
I couldn't find how to enable flash attention 1. Is it possible? Can you add it?
✔ Solution
I'd like to train faster on my T4 (ex models I'm running TinyLlama, Mistral, Phi-1.5.)
❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

FA 1 has been deprecated in favor of FA2. Unfortunately, the only place for FA 1 would be in an older commit. However, that would not work with Mistral out of the box.

Thanks 👍

But it's not a huge change.
You could easily patch your local axolotl


serve-docs Doesn't Stop on Invoke stop

Environment

Nautobot version (Docker tag too if applicable): Latest
Python version: N/A
Database platform, version: n/a
Middleware(s): n/a

Steps to Reproduce

poetry install
invoke --list to get the available commands
invoke serve-docs
invoke stop

Expected Behavior
Expected the docs container to stop running

Observed Behavior
Container keeps on serving those docs.

I don't see that behavior in develop:
❯ invoke serve-docs
Starting Nautobot in detached mode...

Running docker compose command "down"
PYTHON_VER=3.12 \
docker compose \
    --project-name "nautobot" \
    --project-directory "/Users/glennmatthews/Documents/git-stuff/nautobot/development/" \
    -f "/Users/glennmatthews/Documents/git-stuff/nautobot/development/docker-compose.yml" \
    -f "/Users/glennmatthews/Documents/git-stuff/nautobot/development/docker-compose.postgres.yml" \
    -f "/Users/glennmatthews/Documents/git-stuff/nautobot/development/docker-compose.dev.yml" down
Container nautobot-celery_beat-1  Stopping
Container nautobot-mkdocs-1  Stopping
Container nautobot-celery_worker-1  Stopping
Container nautobot-celery_beat-1  Stopping
Container nautobot-mkdocs-1  Stopping
Container nautobot-celery_worker-1  Stopping
Container nautobot-celery_beat-1  Stopped
Container nautobot-celery_beat-1  Removing
Container nautobot-celery_beat-1  Removed
Container nautobot-celery_worker-1  Stopped
Container nautobot-celery_worker-1  Removing
Container nautobot-celery_worker-1  Removed
Container nautobot-nautobot-1  Stopping
Container nautobot-nautobot-1  Stopping
Container nautobot-nautobot-1  Stopped
Container nautobot-nautobot-1  Removing
Container nautobot-nautobot-1  Removed
Container nautobot-redis-1  Stopping
Container nautobot-redis-1  Stopping
Container nautobot-db-1  Stopping
Container nautobot-db-1  Stopping
Container nautobot-selenium-1  Stopping
Container nautobot-selenium-1  Stopping
Container nautobot-db-1  Stopped
Container nautobot-db-1  Removing
Container nautobot-db-1  Removed
Container nautobot-redis-1  Stopped
Container nautobot-redis-1  Removing
Container nautobot-redis-1  Removed
Container nautobot-selenium-1  Stopped
Container nautobot-selenium-1  Removing
Container nautobot-selenium-1  Removed
Container nautobot-mkdocs-1  Stopped
Container nautobot-mkdocs-1  Removing
Container nautobot-mkdocs-1  Removed
Network nautobot_default  Removing
Network nautobot_default  Removed

❯ docker ps
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES

❯

Version is 2.26.1
❯ docker compose version
Docker Compose version v2.26.1

I'm getting the following:
❯ inv stop
Stopping Nautobot...
Running docker compose command "down"
PYTHON_VER=3.8 \
docker compose \
    --project-name "nautobot" \
    --project-directory "/home/joshv/projects/nautobot/development/" \
    -f "/home/joshv/projects/nautobot/development/docker-compose.yml" \
    -f "/home/joshv/projects/nautobot/development/docker-compose.postgres.yml" \
    -f "/home/joshv/projects/nautobot/development/docker-compose.dev.yml" down
time="2024-04-26T11:45:45-05:00" level=warning msg="/home/joshv/projects/nautobot/development/docker-compose.yml: `version` is obsolete"
time="2024-04-26T11:45:45-05:00" level=warning msg="/home/joshv/projects/nautobot/development/docker-compose.postgres.yml: `version` is obsolete"
time="2024-04-26T11:45:45-05:00" level=warning msg="/home/joshv/projects/nautobot/development/docker-compose.dev.yml: `version` is obsolete"
 Network nautobot_default  Removing
 Network nautobot_default  Resource is still in use

Checking the interwebs about it.

I was able to replicate it:
root@ntc ~/nautobot develop nautobot-bzAc2NSp-py3.8 ❯ inv serve-docs
Starting Nautobot in detached mode...
   4 seconds ago   Up 4 seconds   <IP_ADDRESS>:8001->8001/tcp, :::8001->8001/tcp, 8080/tcp   nautobot-mkdocs-1
root@ntc ~/nautobot develop nautobot-bzAc2NSp-py3.8 ❯ inv stop
Stopping Nautobot...
Running docker compose command "down"
PYTHON_VER=3.8 \
docker compose \
    --project-name "nautobot" \
    --project-directory "/root/nautobot/development/" \
    -f "/root/nautobot/development/docker-compose.yml" \
    -f "/root/nautobot/development/docker-compose.postgres.yml" \
    -f "/root/nautobot/development/docker-compose.dev.yml" down
time="2024-04-26T11:55:41-05:00" level=warning msg="/root/nautobot/development/docker-compose.yml: `version` is obsolete"
time="2024-04-26T11:55:41-05:00" level=warning msg="/root/nautobot/development/docker-compose.postgres.yml: `version` is obsolete"
time="2024-04-26T11:55:41-05:00" level=warning msg="/root/nautobot/development/docker-compose.dev.yml: `version` is obsolete"
 Network nautobot_default  Removing
 Network nautobot_default  Resource is still in use
root@ntc ~/nautobot develop nautobot-bzAc2NSp-py3.8 ❯

Thanks for the data. I can see that I need to update my local docker, but interesting that this has changed.

@smk4664 would love to also see what you have to think on this? Do you have the same behavior?

Might just be as simple as changing the stop task definition to perform docker compose down --profile '*'? Haven't tried.

Needs to be docker compose --profile '*' down but that works for me both with docker 4.15 and after upgrading to 4.29.


Mb 4361 upgrade storybook

Description
Upgrades storybook from v5.3.19 to v6.1.10.
Changes include the following:

Moves configurations into main.js (removes addons.js)
Replaces @storybook/addons-info with @storybook/addons-docs
Removes use of decorator withKnob which is now automatically added as a preset
Adds @storybook/essentials
Adds @storybook/preset-create-react-app
Upgrades hierarchy separators to use / instead of |
Replaces use of storiesOf with component story format

Reviewer Notes
I'm not seeing any errors when running storybook.  Anything I missed?  Many of the "best practice" items in https://storybook.js.org/blog/storybook-6-migration-guide/ were covered in the green check changes.
Setup
yarn storybook

Code Review Verification Steps

[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
this article explains more about the approach used.

Fails

:no_entry_sign:

Danger failed to run dangerfile.ts.

Error SyntaxError
Cannot use import statement outside a module
dangerfile.ts:1
import * as child from 'child_process';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at wrapSafe (internal/modules/cjs/loader.js:931:16)
    at Module._compile (internal/modules/cjs/loader.js:979:27)
    at Object.requireFromString [as default] (/home/circleci/transcom/mymove/node_modules/require-from-string/index.js:28:4)
    at Object.<anonymous> (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:144:63)
    at step (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:32:23)
    at Object.next (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:13:53)
    at /home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:3:12)
    at exports.runDangerfileEnvironment (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:105:136)

Dangerfile
-------------------^

  Generated by :no_entry_sign: dangerJS against 3dbcc9b7c66c43e6984739b25a331dbd84f826d8

I just noticed, while I do see storybook is successfully updated, that it seems that Public Sans is not getting loaded into storybook, and it is substituting Times! See here:

I just noticed, while I do see storybook is successfully updated, that it seems that Public Sans is not getting loaded into storybook, and it is substituting Times! See here:

FIXED!  I missed some imports-  This is fixed through the imports in preview.js


⚠ Bofrost.lu has degraded performance

In 8a3d01f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2385 ms

Resolved: Bofrost.lu performance has improved in 0e7383f after 13 minutes.


[BrokenLinksH2] Fixed the link path

Fixed the link path

Docs Build status updates of commit 9bd2d27:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Articles/DevChangelist.md
:warning:Warning

Details

Articles/DevChangelist.md

Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 11, Column 443: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/Kaizala/connectors/tokens' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 11, Column 443: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/Kaizala/connectors/tokens' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix broken links
@dstrome
The Content & Learning team is fixing broken links on docs.microsoft.com for the rest of H2. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes and does not change other content. Please review within five business days and merge, or comment in the PR with any changes you'd like to see. Thanks!

@dstrome - The C+E Skilling team is fixing broken links on docs.microsoft.com. This will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes/removals and does not change other content. This PR is ready for merge. Thank you!


Upgrade service, start behavior default

In 0.43.1, when doing a service upgrade through the UI, the page has a checkbox for "Start behavior". The default is unchecked which means "stop before start". Checked starts the new container and then stops the old one.
Seems to me that the default should be checked so that a new container is started before stopping the old one so as to maintain the service up.

No; start-before stop doesn't necessarily play well with containers with data volumes (e.g. databases), and public ports can only be bound to one container at a time.


Revert "buildDualBolusRecords now consistent with other bolus build records

Going to merge this, as I'll be releasing some other work to prod first while this is in testing.


.NET Alloc profiler activation on .NET (Core)

The .NET Alloc option requires the activation of a specific CLR profiler. The activation is implemented in PerfView by configuring HKLM\Software\Microsoft\.NETFramework\COR* registry keys. It does not seem to have any effect on my net7.0 application:

Action
Target Framework
Has ETWClrProfiler events

Run/Collection
net461
Yes

Run/Collection
net7.0
No

Run/Collection
net7.0 + manual profiler activation using environment variables
Yes

Is this behavior a configuration issue on my side?
I could not find any documented way to activate profilers from registry keys for the .NET (Core) runtime, so maybe it is a known and expected behavior.
However, if the profiler cannot be activated using registry keys, I suppose that PerfView should be improved:

By explicitly documenting the behavior ("Collect" cannot enable the profiler and it must be manually enabled).
By automatically configuring the expected environment variable when collecting with the "Run" action.

It can eventually create a PR for the last point.

I did a few experiments with GCAllObjectAllocation and it seems to generate unsampled Microsoft-Windows-DotNETRuntime/GC/SampledObjectAllocation events. So, indeed, it is a good option to capture allocations for .NET (Core) programs.
But my main issue is that many collection UI options are linked to the CLR profiler. This fact is not clearly documented and these options have not effect on .NET (Core) programs unless the profiler is manually enabled.
I think that both the collection UI and the documentation (PerfView Help) should be improved to make more explicit the behaviors. Also, it would be nice to have an option to enable GCAllObjectAllocation.
I clearly lake experience on the subject to describe a complete solution. I can only make a few suggestions:

Either move all CLR-profiler-related options to a specific section, or add a "Profiler" prefix to the options labels.
Add a CLR profiler documentation entry to explain its role, usage and activation.
Add an option to automatically enable the profiler in the "Run" collection.
Rename "ETW .NET Alloc" to "CLR SampAlloc".
Add an option "CLR Alloc" (related to  GCAllObjectAllocation).

I can probably help on these tasks if needed.


Fix example of using blockwise for an outer product

Closes #7118.

[ ] Tests added / passed
[ ] Passes black dask / flake8 dask

Thanks! It'd be nice if the doctests ran on this code. Would you be able to add the setup so that that could work?

Thanks! It'd be nice if the doctests ran on this code. Would you be able to add the setup so that that could work?

It'd be nice if the doctests ran on this code. Would you be able to add the setup so that that could work?

I'll try to take a look next week - I have a few things on fire at the moment.

It'd be nice if the doctests ran on this code. Would you be able to add the setup so that that could work?

I'll try to take a look next week - I have a few things on fire at the moment.

Thanks! It'd be nice if the doctests ran on this code. Would you be able to add the setup so that that could work?

Done

Thanks! It'd be nice if the doctests ran on this code. Would you be able to add the setup so that that could work?

Done

Thanks!


Update TCK code with yarpl library

This PR is to get the tck code building with yarpl library.
The TCK code has some eventBase logic to ensure we run operations in the right threads (was needed for previous rsocket-api).   That logic is no longer needed, and I will clean them up in another PR.

This is very exciting? I took a quick look at the Travis run and it doesn't look like the TCK is running there? Is there more work needed for that?

This PR is just to get the TCK building with yarpl.  I will cleanup the TCK code and enable it in travis in subsequent PRs

I am going to merge the changes to unblock Mani and let him address the CR comments.


Append Area

leader-line SVG's appended to the body element, is any way to change it? I wanna render it in my element parent.
and then I can use over-flow hidden and ... (because I have scroll on my list)

Hi @Deler-T-Abdolqader, thank you for the comment.
You can move the SVG element to another element from <body> if you really want it.
For example:
document.getElementById('container').appendChild(document.querySelector('.leader-line'));

Note that it is not supported, and it is deprecated.

No reply came, then this abandoned issue is closed.


Update scalafmt-core to 3.0.3

Updates org.scalameta:scalafmt-core from 3.0.2 to 3.0.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-patch

Codecov Report

Merging #182 (b87d17a) into develop (5504d50) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #182   +/-   ##
========================================
  Coverage    50.58%   50.58%
========================================
  Files            5        5
  Lines           85       85
========================================
  Hits            43       43
  Misses          42       42

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 5504d50...b87d17a. Read the comment docs.


Build Error on VS 22

Severity	Code	Description	Project	File	Line	Suppression State	Details
Error	MSB4036	The "MASM" task was not found. Check the following: 1.) The name of the task in the project file is the same as the name of the task class. 2.) The task class is "public" and implements the Microsoft.Build.Framework.ITask interface. 3.) The task is correctly declared with  in the project file, or in the *.tasks files located in the "C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64" directory.	ChaiLdr	C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\BuildCustomizations\masm.targets	70

Right-click on the project in VS, go to "Build Customizations -> tick the masm box" . Then right click on the .asm file and go to Settings-> Drop down box -> Microsoft Macro Assembler" . This should get the asm to be included into the project.

Already did , tried solutions from github etc
it is not working somehow

Change the option to Release and try again, also I have pushed some other fixes , pls clone the repo again and try. Do not compile in Debug, compile as Release x64


Project selection fails if folder names are lower case

Describe the bug
Project selection stage of the tool installer will fail, if the target project has lower case or mixed case folders, it will think its not a unity project and complain that the folders are missing
To reproduce
Make a dummy project with any of the three "Assets", "Packages", "ProjectSettings", with the case of any character in any folder changed from the example.
Selecting the dummy project will fail, with the "needs folders..." type error message.
Expected behavior
The folders are there, its a valid project, this warning should not occur.
Screenshots
Your setup (please complete the following information)

Unity Version N/A
MRTK Version MixedRealityFeatureTool-1.0.2104.4-Beta

Target platform (please complete the following information)

N/A

Additional context
You need to sanity check the comparison when running on windows, since windows does not allow folders to be created using same characters of different casing.
Run a lower-case-pass on all folder strings before comparing, or use a compare function that does so.
For targeting linux; find a suitable folder for the given name and set that as the name to use, or use this method in general.

Hello @Vectrobe thank you for submitting this issue on the Mixed Reality Feature Tool! I have assigned @davidkline-ms to investigate this, and @keveleigh added it to the appropriate Project for the Mixed Reality Feature Tool

I've just opened a PR on the MR Feature Tool repo to fix this. Thanks for the report!


Error after starting the program

I'm not much of a python user, so I did try my best.
Not sure if I did right:)
I followed these commands when trying to install:
Installing python3
cd /usr/local/lib
sudo wget http://www.python.org/ftp/python/3.2.3/Python-3.2.3.tgz && sudo tar -xvf Python-3.2.3.tgz
cd Python-3.2.3/
./configure
make
sudo make altinstall

Installing dependencies
sudo apt-get install xsel
sudo git clone https://github.com/raymontag/kppy /usr/local/lib/kppy
sudo git clone https://github.com/dlitz/pycrypto /usr/local/lib/pycrypto
cd /usr/local/lib/kppy/ && sudo python3.2 setup.py install
cd /usr/local/lib/pycrypto/ && sudo python3.2 setup.py build && sudo python3.2 setup.py install

Installing KeePassC
sudo git clone https://github.com/raymontag/keepassc /usr/local/lib/keepassc
cd /usr/local/lib/keepassc/ && sudo python3.2 setup.py install

I then got this error when trying to start my program:
Traceback (most recent call last):
  File "/usr/local/bin/keepassc", line 34, in <module>
    from kppy import *
  File "/usr/local/lib/python3.2/site-packages/kppy.py", line 23, in <module>
    from Crypto import Random
ImportError: No module named Crypto

Any ideas how to fix would be mucho appreciated.

Hi, I had the same /var/empty missing problem on my Ubuntu
Can you please add the mkdir(/var/empty) in the setup.py ?
BTW: ssh does work without this directory:
$ ssh b206
xxxxx@b206's password:
Welcome to Ubuntu X.Y

 * Documentation:  https://help.ubuntu.com/

Last login: Wed Apr 13 19:17:41 2016 from <IP_ADDRESS>
‘/datas/ImagesVersets/bibliorama.fr/1680_desert.jpg’ -> ‘/boot/grub/today.jpg’
‘/boot/grub/today.jpg’ -> ‘/iso/boot/grub/today.jpg’

     "Be of good courage, and he shall strengthen your
      heart, all ye that hope in the LORD."
                             Psalm 31:24
xxxxx @ xxxxx-B206 : ~ ls -ld /var/empty
ls: cannot access /var/empty: No such file or directory


Influx package

Provides an auto-scaling point writer, which writes points in batches and simple writer, which just writes points one-by-one
see TheThingsIndustries/noc#223 for usage example

❯ golint influx
influx/writer.go:15:7: exported const DefaultScalingInterval should have comment or be unexported
influx/writer.go:41:1: exported function NewSinglePointWriter should have comment or be unexported
influx/writer.go:110:1: exported function NewBatchingWriter should have comment or be unexported


kontrol summary: add top comment indicating the files are auto-generated

The files generated by kontrol summary don't currently indicate that have been generated automatically.
We should add a comment along the lines of
// This file was autogenerated by running `kontrol summary`. Do not edit this file manually.

Or better, add an option to pass in the text to be displayed in that comment.

Implemented in https://github.com/runtimeverification/kontrol/pull/263.


Allow tests to be marked "doctest: +XFAIL"

Needs a decision I think:
pro: I like this feature
con: doctest: +XFAIL doesn't seem to be what Python uses.  I haven't sorted out how one does this in Python yet...

Good for me
On Wed, May 20, 2015 at 03:42 Colin Macdonald<EMAIL_ADDRESS>wrote:

Needs a decision I think:
pro: I like this feature
con: doctest: +XFAIL doesn't seem to be what Python uses. I haven't
sorted out how one does this in Python yet...
—
Reply to this email directly or view it on GitHub
https://github.com/catch22/octave-doctest/pull/70#issuecomment-103810952
.

I also could not find a Python analogue. +XFAIL looks good to me. :+1:


add Product and Serializable to Field

following the good practice of having ADT extending these two traits
https://stackoverflow.com/questions/36525804/scala-case-class-extending-product-with-serializable

Thanks @kailuowang!


Support multiple @Watch decorators per function

Hey there,
vue-property-decorator supports multiple @Watch decorators per class method.
It would be amazing if this library would have it as well.

Thanks, it is now working!
Side note: I have looked at your according commit and the actual test case should be something like:
@Watch('defaultWatchKey')
@Watch('fullWatchKey')
multiWatcher() {
  // ...
}

@maxwrlr That's a case in test, I will add it.


Store Data-History in PostgreSQL

So we have a "complete" History of the Data.
Maybe this should be solved in the status-Project, so we keep this generic.

I'm not sure whether data logging should be part of the SpaceAPI server. A more scalable approach would probably be to write a microservice that fetches the API regularly and logs the data to a database (like status logger does it).
@dns2utf8 @rnestler opinions?

@dns2utf8 @rnestler ping? :)

the status logger works very good.
Closing for now.


admin actions to re-schedule task's run_at and change priority

[ ] run_at +1 hour
[ ] run_at +24 hours
[ ] set prio to default
[x] increase prio by 1
[x] decrease prio by 1

https://github.com/arteria/django-background-tasks/commit/af3c8572bcfe89d5bfa80dddea8e144c83835755


Tracing on Vulkan does not support empty beamlines

If Beamline is empty, quadric buffer (Elements' meta-data) will also be empty.
This case is a testcase, in which we only test MatrixSource (RML with only source)
Consequence: Vulkan can't create/allocate 0-sized buffers.
Temporary solution: A dummy Plane is added to the RML with [0,0] size.
This feature might become handy later on. So keep it in mind that for now vector<opticalElement>.size() has to be != 0

Waiting for SYCL decision

Is this still an issue? Why do we need to trace empty beamlines?

This was literally 2 lines of code. Good thing we had this ticket for 9 months :D


⚠ Bofrost.at has degraded performance

In accde70, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 956 ms

Resolved: Bofrost.at performance has improved in 72e3615 after 6 minutes.


KAFKA-8501: Removing key and value from exception message

Messages containing key and value were moved to the TRACE logging level, however the exception is still adding the key and value.
This commits remove the key and value from StreamsException.
*Summary of testing strategy (including rationale)
The content of the exception message is not tested anywhere, and I think it would be really difficult to provide a reliable test to check for content inside a string.
Committer Checklist (excluded from commit message)

[ X ] Verify design and implementation
[ X ] Verify test coverage and CI build status
[ X ] Verify documentation (including upgrade notes)

retest this please

Hi,
How/Who can merge this PR?
Regards

Merged #6904 to trunk

Thanks for the contribution @carlosduclos.
For merging, we usually require two +1's, but in this case, since the change was simple I went ahead and merged.


Implement Debug for generic::Variant

As the title says. This would be a useful thing to have.

Added to #383

Close as now no more Variant


InsertMention should support optional data

Describe the solution you'd like
The insertMention helper provided by useBeautifulMentions should support passing in data for mentions that contain things like id/email.
Describe alternatives you've considered
I'll need to handle inserting myself for now but I'd much rather let this library do that!

Thanks for the quick enhancement and release, works great!


MudDataGrid: Support Column with Func Field Pointer

Feature request type
Enhance component
Component name
MudDataGrid
Is your feature request related to a problem?
Currently the MudDataGrid.Column Component uses a string to point to a specific Field.
Describe the solution you'd like
MudDataGrid.Column should support a Func<TItem, object> for the Field
Currently Columns look like this:
<Column T="TItem" Field="Name" />
I would like to be able to use:
<Column T="TItem" FieldFunc="(x => x.Name)" />
This would also allow us to display objects nested in the graph:
<Column T="TItem" FieldFunc="(x => x.SubObject.Name)" />
Have you seen this feature anywhere else?
My BlazorTable component has this functionality, https://github.com/IvanJosipovic/BlazorTable
Describe alternatives you've considered
No response
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

hey @tjscience,
I know you are working on the v3 of the MudDataGrid.
I was wondering if you considered the feature above, if not, I can look into implementing it.

Hi @IvanJosipovic, I'm sure tj or someone else of more authority will respond as well, but I have seen in other feature requests for Datagrid, that because it's still in development, new requests for features are not being considered. Once its production ready then they can start looking at new stuff.
 :D

@IvanJosipovic, other than being able to resolve nested properties (for which there are work-arounds) what other benefits does this highly breaking change provide? All of the other major component libraries that I have seen just use a string for the field name.

Hi @IvanJosipovic, I'm sure tj or someone else of more authority will respond as well, but I have seen in other feature requests for Datagrid, that because it's still in development, new requests for features are not being considered. Once its production ready then they can start looking at new stuff.
 :D

Fake Data Generators are very useful, try https://mockaroo.com

@IvanJosipovic, other than being able to resolve nested properties (for which there are work-arounds) what other benefits does this highly breaking change provide? All of the other major component libraries that I have seen just use a string for the field name.

Can you provide me details on the workarounds? My attempts all ended up with partial solutions, ie sorting/filtering wouldn't work.

<Column T="TItem" FieldFunc="(x => x.Name)" />

Big advantage is use of real instance object, no magic strings.

<Column T="TItem" FieldFunc="(x => x.Name)" />

Big advantage is use of real instance object, no magic strings.

While, I prefer the Func approach, the current model can also be Type bound by using:
<Column T="ICluster" Field="@nameof(ICluster.Name)" />

@IvanJosipovic Hmmm... in the end it's still a string.
Only advantage now it will be picked up when refactored.

<Column T="TItem" FieldFunc="(x => x.Name)" />

Big advantage is use of real instance object, no magic strings.

You can achieve the same thing by using nameof().

Hi @IvanJosipovic, I'm sure tj or someone else of more authority will respond as well, but I have seen in other feature requests for Datagrid, that because it's still in development, new requests for features are not being considered. Once its production ready then they can start looking at new stuff.
 :D

Fake Data Generators are very useful, try https://mockaroo.com

@IvanJosipovic, other than being able to resolve nested properties (for which there are work-arounds) what other benefits does this highly breaking change provide? All of the other major component libraries that I have seen just use a string for the field name.

Can you provide me details on the workarounds? My attempts all ended up with partial solutions, ie sorting/filtering wouldn't work.

There are two main ones.

Splat the properties for each model into a flat class so that the properties you are binding to are not in nested classes.
Use the SortBy, FilterBy, GroupBy parameters to tell the data grid where the property live and route to the nested property using the CellTemplate.

There are two main ones.

Splat the properties for each model into a flat class so that the properties you are binding to are not in nested classes.
Use the SortBy, FilterBy, GroupBy parameters to tell the data grid where the property live and route to the nested property using the CellTemplate.

1: This workaround is far from ideal.
2: I played with these options, "FilterBy" does not exist in Column.cs which means filtering is broken.
Example, https://try.mudblazor.com/snippet/cEwwapEsUrMIDjsA

Sorry for the misunderstanding. FilterBy is part of phase 3 of the data grid.

Hey @tjscience, I noticed that FilterBy was missing in the latest update. Are you still planning on adding it?

<Column T="TItem" FieldFunc="(x => x.Name)" />

Big advantage is use of real instance object, no magic strings.

You can achieve the same thing by using nameof().

I am actually having a bit of an issue with that, I want to go one layer deeper into the object and this does not allow for that. Something like User.PersonalData.Name instead of User.Name.


When using a MERGE in place of CREATE, get planner error.

Executing this query:
CREATE.node(applicant).label("Applicant")
.property("firstName").value("John")
.property("lastName").value("Smith")
.property("UniqueId").value("JohnSmith");
Works fine.
If I instead try to use a MERGE:
MERGE.node(applicant).label("Applicant")
.property("firstName").value("John")
.property("lastName").value("Smith")
.property("UniqueId").value("JohnSmith");
I get this:
---------------DB Errors:
codeOrType: Neo.ClientError.Statement.InvalidSyntax
message: Parameter maps cannot be used in MERGE patterns (use a literal map instead, eg. "{id: {param}.id}") (line 1, column 48 (offset: 47))
"CYPHER planner=rule MERGE (applicant1:Applicant{props_0}) CREATE (streetAddress1:Address{props_1}), (applicant1)-[:HAS_ADDRESS]->(streetAddress1), (ipAddress1:IpAddress{ip:{param_0}}), (applicant1)-[:HAS_IP]->(ipAddress1)"
^
---------------end Errors:
QUERY
CYPHER --------------------
CYPHER planner=rule
MERGE
(applicant1:Applicant{firstName:'John', lastName:'Smith', UniqueId:'JohnSmith'})
CREATE
(streetAddress1:Address{street:'25 WORKER WAY', city:'DANBURY', state:'CT', zip:'06810'}),
(applicant1)-[:HAS_ADDRESS]->(streetAddress1),
(ipAddress1:IpAddress{ip:'<IP_ADDRESS>'}),
(applicant1)-[:HAS_IP]->(ipAddress1)
JSON   --------------------
{
"query":"CYPHER planner=rule MERGE (applicant1:Applicant{props_0}) CREATE (streetAddress1:Address{props_1}), (applicant1)-[:HAS_ADDRESS]->(streetAddress1), (ipAddress1:IpAddress{ip:{param_0}}), (applicant1)-[:HAS_IP]->(ipAddress1)",
"params":{
"props_0":{
"firstName":"John",
"lastName":"Smith",
"UniqueId":"JohnSmith"
},
"props_1":{
"street":"25 WORKER WAY",
"city":"DANBURY",
"state":"CT",
"zip":"06810"
},
"param_0":"<IP_ADDRESS>"
}
}

If I run the same Cypher query in the Web Interface is works fine:
MERGE
(applicant1:Applicant{firstName:'Frank', lastName:'Farrel', UniqueId:'FrankFarrel'})
CREATE
(streetAddress1:Address{street:'25 PAULDING TER', city:'DANBURY', state:'CT', zip:'06810'}),
(applicant1)-[:HAS_ADDRESS]->(streetAddress1),
(ipAddress1:IpAddress{ip:'<IP_ADDRESS>'}),
(applicant1)-[:HAS_IP]->(ipAddress1)
Added 3 labels, created 3 nodes, set 8 properties, created 2 relationships, statement executed in 168 ms.

I've spent some time debugging, and I think this change MIGHT fix the problem.
QueryParamSet.java:
public static boolean canUseSet(WriterContext context) {
if (context.currentParamOrSet instanceof QueryParamSet && context.currentClause != ClauseType.MERGE)
return ((QueryParamSet)context.currentParamOrSet).canUseSet();
return false;
}
It feels like we need to prevent using property sets, when using MERGE clauses.  I'm not anywhere near 100% sure that this is the absolute cause.

More details...
If I set:
query.setExtractParams(false);
The merge starts working. Obviously this makes all the queries slower.
Would be be possible to set the ExtractParams per "Clause"?
I still think the best alternative might be to change HOW params are extracted for merges. but I'm a Neo4J noob, so I'm not 100% sure.

Hi Brian,
I will have a look into the problem, you have brought it perfectly to the point with your analysis.
I will provide a fixes release of JCypher (3.3.1) by next week.
Best regards,
Wolfgang

Hi Brian,
release 3.3.1. containing the fix for the MERGE problem will be out on monday, April 4th.
Best regards,
Wolfgang

Thank you sir!

Hi Brian,
JCypher 3.3.1 has now been released.
Best regards,
Wolfgang


Langsmith fetch errors in production

When running locally, Langsmith populates as I would expect with my LLM runs.
However, in prod, I'm getting this error:
Error in handler tracer_langchain_LangChainTracer, handleLLMStart: TypeError: fetch failed
No further context is provided in the error.
Thoughts?

@dosu-beta I didn't mention which version I'm using, but I'm using the latest: 0.0.160.

This was an environment variable issue on my end. It's fixed.


Buttons

I have a suggestion: why not allow buttons in the context menu? This saves space and allows for more actions intuitively.
Something like this:

I have a suggestion: why not allow buttons in the context menu? This saves space and allows for more actions intuitively.
Something like this:

What program is used in the screenshot?

#337 (comment)

That's awesome! ETA?

What program is used in the screenshot?

GIMP! This is just a mockup!


Desktop Support

Teams need to have multi-window support with components. This would include features like opening a tooltip in a new window and opening each Cascading Menu list in new windows.
### Tasks
- [ ]  Discuss the current abstractions

Requirements:

Injection of window
Window positioning
Styling injection

How do we see this work's priorities dovetailing with List creation ?


fix(npm): Ran npm audit fix, 0 vulns

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


UnicodeDecodeError install with python3.5 on ubuntu docker

Python 3.5.2
Running
uname -a
I get
Linux 0dd086988daa 4.9.49-moby #1 SMP Wed Sep 27 23:17:17 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
Running
pip3 install pyahocorasick
I get
Collecting pyahocorasick==1.0
  Downloading pyahocorasick-1.0.0.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-11rm5xis/pyahocorasick/setup.py", line 44, in <module>
        long_description = get_readme(),
      File "/tmp/pip-build-11rm5xis/pyahocorasick/setup.py", line 6, in get_readme
        return f.read()
      File "/usr/lib/python3.5/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xc5 in position 1268: ordinal not in range(128)

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-11rm5xis/pyahocorasick/

Update:
Got it to work by installing from source.
RUN cd /tmp; git clone https://github.com/WojciechMula/pyahocorasick/; cd pyahocorasick; pip3 install .
But I guess thats 1.1.5.dev1? Can this be pushed up to pypi?
btw awsome lib - https://stackoverflow.com/questions/192957/efficiently-querying-one-string-against-multiple-regexes/47319512#47319512 worked awsome, gonna use it a lot more.

Hi, sorry for inconvenience. I'm wondering why you got the version 1.0.0, while the latest stable is 1.1.4. It's strange.
Thank you the stackoverflow entry, it's a really nice example of use. And a great advertisement of my library. :)

@gjthompson1 1.0.0 is a reall old dog... so it makes sense to use the latest from Git

Hey guys,
Sorry for taking so long to get back to you, downloading from master branch worked in my project like I said so thats good.
RUN cd /tmp; git clone https://github.com/WojciechMula/pyahocorasick/; cd pyahocorasick; pip3 install .
So I tested with Docker, ubuntu:latest and python3-dev, and added pyahocorasick==1.1.4 to requirements file.
Below is the Dockerfile:
FROM ubuntu:latest

RUN apt-get update -y
RUN apt-get install -y python3-pip python3-dev build-essential libpq-dev libenchant-dev

COPY requirements.txt /tmp/
RUN pip3 install --requirement /tmp/requirements.txt

requirements.txt file
pyahocorasick==1.1.4

Still error's my brother got the same thing as me.
Might be something to do with ubuntu and python3.5 vs 3.6.
Step 5/5 : RUN pip3 install --requirement /tmp/requirements.txt
 ---> Running in b5c047cca55a
Collecting pyahocorasick==1.1.4 (from -r /tmp/requirements.txt (line 1))
  Downloading pyahocorasick-1.1.4.tar.bz2
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-2jlrtkpx/pyahocorasick/setup.py", line 86, in <module>
        long_description=get_long_description(),
      File "/tmp/pip-build-2jlrtkpx/pyahocorasick/setup.py", line 24, in get_long_description
        readme = [line for line in f if not line.startswith('.. contents::')]
      File "/tmp/pip-build-2jlrtkpx/pyahocorasick/setup.py", line 24, in <listcomp>
        readme = [line for line in f if not line.startswith('.. contents::')]
      File "/usr/lib/python3.5/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xc5 in position 1006: ordinal not in range(128)

    ----------------------------------------

That's a mistake of mine :] please use 1.1.6 (the current latest version) instead

FWIW, setuptools does not like having the future unicode literals import in it. Somehow is makes weirdly Python 3 choke.  I should know better

Great, that worked.
Glendons-MacBook-Pro:test glendonthompson$ docker build . -t test-aho
Sending build context to Docker daemon  3.072kB
Step 1/6 : FROM ubuntu:latest
 ---> 2d696327ab2e
Step 2/6 : RUN apt-get update -y
 ---> Using cache
 ---> a58b419744ba
Step 3/6 : RUN apt-get install -y python3-pip python3-dev build-essential libpq-dev libenchant-dev
 ---> Using cache
 ---> 34e81ef31aa3
Step 4/6 : RUN pip3 install --upgrade pip
 ---> Using cache
 ---> 64f2e5ba4e45
Step 5/6 : COPY requirements.txt /tmp/
 ---> 321aeb52be7e
Step 6/6 : RUN pip3 install --requirement /tmp/requirements.txt
 ---> Running in 84e82c2e381b
Collecting pyahocorasick==1.1.6 (from -r /tmp/requirements.txt (line 1))
  Downloading pyahocorasick-1.1.6.tar.gz (69kB)
Building wheels for collected packages: pyahocorasick
  Running setup.py bdist_wheel for pyahocorasick: started
  Running setup.py bdist_wheel for pyahocorasick: finished with status 'done'
  Stored in directory: /root/.cache/pip/wheels/fd/05/a0/a6ae157593031bacde36bf9cee8f92f984dbaddee25b678ce6
Successfully built pyahocorasick
Installing collected packages: pyahocorasick
Successfully installed pyahocorasick-1.1.6
 ---> 04f59869755e
Removing intermediate container 84e82c2e381b
Successfully built 04f59869755e
Successfully tagged test-aho:latest

@gjthompson1 Thank you for testing this and using pyahocorasick! You rock.

@estnltk you had a fix in your fork at https://github.com/estnltk/pyahocorasick/commit/5773b4634a3a263c50f66191a59f0303adc7cbd3 that is likely no longer needed with the latest FWIW. You might want to check it out


add vendor github.com/aanand/compose-file

vendor for docker compose v3 parser

Codecov Report

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

@@           Coverage Diff           @@
##           master     #547   +/-   ##
=======================================
  Coverage   19.18%   19.18%
=======================================
  Files          28       28
  Lines        1496     1496
=======================================
  Hits          287      287
  Misses       1170     1170
  Partials       39       39

docker/libcompose  doesn't support compose v3 yet, use aanand/compse-file


Usage deprecated in pandas 0.23

There was initially a failure due to pandas 0.22 -> 0.23 in #3017,
fixed by a tiny tweak.
However there is an outstanding deprecation warning emerging from the tests ..
$ python lib/iris/tests/test_pandas.py -v TestAsSeries.test_copy_float_false
test_copy_float_false (__main__.TestAsSeries) ... /home/h05/itpp/git/iris/iris_main/lib/iris/pandas.py:133: FutureWarning: Series.base is deprecated and will be removed in a future version
  if hasattr(pandas_obj, 'base'):
/home/h05/itpp/git/iris/iris_main/lib/iris/pandas.py:134: FutureWarning: Series.base is deprecated and will be removed in a future version
  base = pandas_obj.base
ok

----------------------------------------------------------------------
Ran 1 test in 0.004s

OK

That arises from the code here
This is not so trivial a problem :
as existing code indicates, the usage of the 'base' property already changed.
If I understand right, potentially we can now just replace 'base' with 'values'.
- except that would no longer work with older versions of pandas.
However, right now, even testing for the presence of a '.base' attribute is triggering the deprecation warning.

Apologies for butting in.  Isn't the answer here to use (and check for) df.values.base instead of df.base, which would, I think, work with both older versions and 0.23?
As for deprecating this module, while I have used it for using iris with UGRID data, it's a bit of a niche usage so I'm not sure I could make a compelling case against deprecation.

@hdyson - I'm not close enough to the pandas API to answer definitively, but that suggestion sounds sensible. Would be happy to merge if you want to submit a PR?

@pelson A couple of quick questions here.  Has there been a resolution to the conversation about deprecating the cube to/from dataframe/series functionality?  And if it is to be deprecated, will this functionality leave iris before the switch over to a pandas version which will have an error rather than a deprecation warning?
For the fix itself, I'd be happy to provide it; I'm just trying to confirm we're not in "re-arranging deckchairs on the Titanic" territory.

Lol. Love the metaphor! No decision has been made really. It seems like one
of those corners of Iris that doesn't get any attention, and is therefore
ripe for consideration to be deprecated. No matter what though, code won't
be deleted until v3.
On Mon, 11 Jun 2018, 12:20 hdyson<EMAIL_ADDRESS>wrote:

@pelson https://github.com/pelson A couple of quick questions here. Has
there been a resolution to the conversation about deprecating the cube
to/from dataframe/series functionality? And if it is to be deprecated, will
this functionality leave iris before the switch over to a pandas version
which will have an error rather than a deprecation warning?
For the fix itself, I'd be happy to provide it; I'm just trying to confirm
we're not in "re-arranging deckchairs on the Titanic" territory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SciTools/iris/issues/3018#issuecomment-396209687, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAxep-6pYZ6QlhVlOjk8pM5apSnXcsZsks5t7lKSgaJpZM4UBhTb
.

Looks like this was fixed by #3079.


Compiler emits FS0988 warning with a reference to non existing location in source file

Description: In some cases compiler emits FS0988 (Main module of program is empty: nothing will happen when it is run) warning and gives a reference to non existing code location. In "wrong" cases location is always (1, 81), even if there is no such location in source file.
Repro Steps:
Compile the following file
namespace MyNamespace1

module MyModule1 =
    let irrelevant = 10

Expected:
Compiler emits warning FS0988 pointing at the end of the last module in file MyModule1
Actual:
Compiler emits warning FS0988 pointing at location (1, 81)
Severity: Cosmetic - for this particular warning it is just a matter of taste to have a correct source code location, but might be useful when navigation to warning/error location is used in some editors.
Version: 12514308c79453bac38dea11e0e3c549619a79dc
Likely Cause: build.fs#L3054
Workaround: no workaround

nice bug report @mstyura !

I got this one in a F# dotnet core project. Is there a known workaround since last year?

fixed in https://github.com/Microsoft/visualfsharp/pull/1852


Possible Cell size problem.

You might want to limit the cell size to eight bits as many BF programs assume this.
You can do it by adding @memory[@data_ptr] &= 255 at the appropriate locations.

Hey, thanks a lot! Sure will do:)


Semantic API's Symbol at Cursor API not working for annotations

Description:
Consider the following source snippet and the symbol at cursor is not working for the cursor positions mentioned.
@v<cursor>1 {
    val: 2
}
[int, string] [intVar, stringVar] = [1, "myString"];

public function func(@v<cursor>2 { val: "v63 value required" } int id,
                     @v<cursor>2 { val: "v63 value defaultable" } string s = "hello",
                     @v<cursor>2 { val: "v63 value rest" } float... others) returns @v<cursor>3 float {
    return 1.0;
}

const annotation map<int> v1 on source var;
annotation Annot v2 on parameter;
public annotation v3 on return;
const annotation map<string> v4 on source annotation;

@v<cursor>4 {
    str: "v4 value"
}
const annotation map<string> v5 on source annotation;

type Annot record {
    string val;
};

Affected Versions:
sl-alpha2 at least

What's the scenario here? Completions at the above mentioned cursor positions (in which case, an issue with visibleSymbols())?

Ah, just noticed that this was the PR sent for the annotation fixes.


Default front side

Hello,
I am having problems with List of Flip Cards, when i press FlipCard to flip it and then render next Flipcard from list, then its starts from the back side, is there any solution for that ?
E.g. to create FlipCard with default front side.
Thanks !

@PjoterKacprzak Thanks for opening an issue.
Your problem is that you are not using Keys. Provide a UniqueKey to each of your flip cards and your problem should be solved.


Update limits-specifications-teams.md

Added clarification about 500 endpoint limit for meetings with breakout rooms.

@MicrosoftHeidi PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@MicrosoftHeidi PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@MicrosoftHeidi PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


feat(quey): add desc for builder

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

GitHub release
v0.9.0

Your semantic-release bot :package::rocket:


testing primary uids on created keys

How could this issue be tested? Since it was fixed without adding any test, it may get broken in the future.

I agree, I didn't find a way in PGPainless to check for primary user ID value. It gives all the users list for which test was already added.

Normally I assume the first returned uid is the primary one. Is that guaranteed in pgpainless? Or is actually the way to choose primary uid a different mechanism?
Originally posted by @tomholub in https://github.com/pgpainless/pgpainless/issues/60#issuecomment-769621392
Just re-posting here so it doesn't get lost

PGPainless currently returns the first userID in the userID list, so it does not check for the userId subpacket.
We should change that.

PGPainless currently returns the first userID in the userID list, so it does not check for the userId subpacket.
We should change that.

Now there is keyRingInfo.getPrimaryUserId() which will only return the userID that is marked as primary by a signature.

Now there is keyRingInfo.getPrimaryUserId() which will only return the userID that is marked as primary by a signature.


Typeahead shows the same result

I've spent mega hours, and can't get it to work.
I have a remote method that returns a JSON array of string (about 300), I confirmed that it's valid JSON! Somehow Typeahead only shows the first 10 strings, repeatedly. What am I missing!
$(document).ready(function() {
  var brands = new Bloodhound({
      datumTokenizer: Bloodhound.tokenizers.obj.whitespace('name'),
        queryTokenizer: Bloodhound.tokenizers.whitespace,
        limit: 10,
        remote: {
           url: '/brands_typeahead.json',
           filter: function(list) {
           return $.map(list, function(brand) { return { name: brand }; });
           }
        }
    });

    brands.initialize();

    $('.typeahead').typeahead(null, {
       name: 'brands',
       displayKey: 'name',
       source: brands.ttAdapter()
    });
});

remote data is not searched through, it's rendered as is.


Removed deprecation warning „constant OpenSSL::Cipher::Cipher is deprecated“

Tests pass without deprecation warning in Ruby 2.4.0.
The gem "test-unit" had to be added additionally to make tests run.

@chicks is this PR slated to be accepted and a new version released?

Anything we can help with to get this merged?

Hi folks.  The original author isn't interested in maintaining this any longer.  I'll be taking over.  Sorry this sat in limbo for so long.  This change looks good to me, thanks for contributing!  I'll merge it and follow up soonish with a new release.

Hi @vanaltj , thanks for taking over maintaining this gem 🙏
When do you think you can release a new version?


updated vo version 0.8.5

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


my fills is incorrect when making trades at same price

scenario:
user gets into a position at 100 0.6 on outcome A.
then closes position at 100 0.6.
The transactions are grouped together as bought they should be separate, a bought and a sell

fixed-closing


Vnet peering in route tables?

I'm really confused... is this document accurate, or 2017 is way too out of date?
I've got a route table, and I've got a Vnet with peering enabled. I can't see this option "Virtual network peering" anywhere in the option of next hub in my route table!

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

ID: 38bd69ac-a1b5-a401-184d-37b55188a7c4
Version Independent ID: ef976d45-36eb-8a32-4b29-ae03412165d2
Content: Azure virtual network traffic routing
Content Source: articles/virtual-network/virtual-networks-udr-overview.md
Service: virtual-network
GitHub Login: @malopMSFT
Microsoft Alias: malop

@sarah026 , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@sarah026 , Appreciate your patience. The document was accurate and as said earlier, Azure adds additional default system routes for different Azure capabilities, but only if you enable the capabilities like peering. Depending on the capability, Azure adds optional default routes to either specific subnets within the virtual network, or to all subnets within a virtual network.
So, You will not be finding those additional routes in route table associated with that subnet instead you will be finding those routes in the list of effective routes of a NIC attached to a NSG that was linked to your subnet. Also, the NIC has tobe associated to a VM for viewing the routes. Here is the screenshot of those effective routes for my NIC,

We are closing this issue for now.If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Dropdown with custom button, better eslint for more consistency.

Maybe we should introduce husky for when we get pull requests, because it automatically runs eslint. What do you think?
    "husky": {
        "hooks": {
            "pre-commit": "./node_modules/.bin/eslint --fix . --ext .js,.vue"
        }
    }

511596358


Problem: using expect may cause a panic and in most cases this isn't the desired behaviour

We need to explicit handle our error conditions and decide which are recoverable and which should actually panic.

@jhult Let me know if you have the desire to audit the code base (even partially) for improper use of 'expect' (causing failure to log errors correctly, or premature failure of the process).

Running ripgrep inside src directory.
❯ rg -iFl "expect("
block/precomputed.rs
proof_systems/mod.rs
proof_systems/signer/seckey.rs
store.rs
state/mod.rs
receiver/filesystem.rs
bin/mina-indexer.rs
ledger/genesis.rs
state/branch.rs
server.rs
command/signed.rs
ledger/mod.rs
command/mod.rs
protocol/serialization_types/signatures.rs

rg -iF "expect("
bin/mina-indexer.rs
156:    fs::create_dir_all(log_dir.clone()).expect("log_dir should be created");
166:        fs::create_dir_all(parent).expect("log_file parent should be created");
205:    fs::create_dir_all(watch_dir.clone()).expect("watch_dir should be created");

block/precomputed.rs
94:            let file_name = path.file_name().expect("filename already checked");
96:            let state_hash = get_state_hash(file_name).expect("state hash already checked");

store.rs
94:            .expect("blocks column family exists")
100:            .expect("lengths column family exists")
106:            .expect("slots column family exists")
112:            .expect("canonicity column family exists")
118:            .expect("commands column family exists")
124:            .expect("ledgers column family exists")
130:            .expect("events column family exists")
136:            .expect("snarks column family exists")
850:                            .expect("signed commands with state hash")
1005:                            .expect("snark work with state hash")

receiver/filesystem.rs
56:                .expect("should not crash");

proof_systems/mod.rs
99:            .expect("Failed to serialize field");

command/signed.rs
161:                .expect("valid transaction hash"),

ledger/genesis.rs
75:                    .expect("Parsed Genesis Balance has wrong format"),

state/mod.rs
245:                    .expect("ledger add succeeds");
248:                    .expect("set best block to root block");
594:                            .expect("has root")
1072:                .expect("block is in diffs_map");

proof_systems/signer/seckey.rs
170:                .expect("failed to decode sec key"),
180:                .expect("failed to decode sec key"),
204:                .expect("failed to decode sec key")
232:                .expect("failed to decode sec key")

command/mod.rs
345:                            Value::String(serde_json::to_string(&r).expect("serialize reason"))

server.rs
103:    let mut term = signal(SignalKind::terminate()).expect("failed to register signal handler");
104:    let mut int = signal(SignalKind::interrupt()).expect("failed to register signal handler");
145:    fs::create_dir_all(startup_dir.clone()).expect("startup_dir created");

ledger/mod.rs
281:            .expect("ledger diff application");
283:        let account_after = ledger.accounts.get(&public_key).expect("account get");
308:            .expect("ledger diff application");
310:        let account_after = ledger.accounts.get(&public_key).expect("account get");

state/branch.rs
64:        let root = self.branches.root_node_id().expect("root node id");
67:            .expect("traverse level order ids")
75:            .expect("branch always has root node")
82:                .expect("branch always has root node")
88:                .expect("node_id from iterator")
123:                .expect("node_id comes from branches iterator, cannot be invalid");
132:                    .expect("node_id comes from branches iterator, cannot be invalid");
225:            .expect("branch always has root node");
229:            .expect("incoming_root_id valid, branch always has root node")
235:            .expect("junction node exists in self")
245:            .expect("merge_on called with valid junction_id");
252:            .expect("incoming_root_id guaranteed by root_id() call")
256:                .expect("guaranteed by call structure");
260:                .expect("old_node_id valid");
267:                    .expect("child_id valid")
276:                    .expect("under_node_id guaranteed by call structure");
294:            .expect("insert as root always succeeds");
301:            .expect("new_root_id received from .insert() call, is valid")
308:                .expect("node_id from iterator, cannot be invalid");
345:            .traverse_level_order(self.branches.root_node_id().expect("root node id"))
346:            .expect("traverse level order")
385:            for node in self.branches.ancestors(&node_id).expect("node_id is valid") {

protocol/serialization_types/signatures.rs
130:                    .expect("Wrong number of bytes encountered when converting to BaseField"),
151:                    .expect("Wrong number of bytes encountered when converting to FieldElement"),
243:                .expect("Wrong number of bytes encountered when converting to BaseField"),
245:                .expect("Wrong number of bytes encountered when converting to ScalarField"),
268:                    t.rx.to_bytes().as_slice().try_into().expect(
271:                    t.s.to_bytes().as_slice().try_into().expect(

This is about the journey rather than the destination. Getting to the end of the long tail is likely untenable but we've improved our codebase. Closing.


default REST CA Management  should be "ignore certificate Checking"

when setting the connection, the default REST CA Management should be  "ignore certificate Checking"

I disagree. That's the less secure option.

@shiramax I see your point, there is a small chance that the CA which is used to sign the engine certificate is already trusted by the device you are on. On the other hand falling back to the less secure option by default is not a good solution. What we do is this:

by default we hope the CA is already trusted by the device
if not and the connection fails due to certificate issue, we advice the user to download the engine's certificate and pre-fill the dialog with default address where the certificate is living
and if the user explicitly decides to, he can decide to ignore the certificate checking

I think the current situation is secure while still not that user-unfriendly, so I would close this issue.

@jelkosz well, could you download the certificate for the user and ask if he could trust it?

@mykaul we kinda do this - the user is redirected to the screen where everything is pre-filled and the user can decide if he wants to download this cert and trust it or not.


Pom1 broken

Pom1 example not working any more, because uses amici.sol.xdot field (https://github.com/ICB-DCM/PESTO/blob/master/examples/Pom1p_gradient_formation/logLikelihoodPom1.m#L99).
Easiest solution: xdot has moved to amici.sol.diagnosis.xdot, but this should be also possible (nicer) via pre-equilibration now?

Sounds good. Would be great if you implement the solution.


Add Nation of Makers to US partnerships on FAQ page

We’ve launched findthemasks.com/makers, and we’d like to acknowledge our maker data partners at Nation of Makers on the US partnerships page. https://www.nationofmakers.us/
Dorothy Jones-Davis and Claire Cocciole are our main contacts there if you need more info, high resolution logos, etc.

Hi. Can you use a png file? If so, here is the Nation of Makers logo.

Yes we can!

I added Nation of Makers to our Dev Ready spreadsheet: https://docs.google.com/spreadsheets/d/1bzf_ZxWtDV_fagpOB9f8dy1yKJN1Elqy11ak2S4IuLs/edit?usp=sharing

Devs please add row 13's logo and URL to the "Collaborators - US" subheader on /partners - thank you!!


Material Icon barcode_scanner,barcode are missing

Use case
I cant find the icons barcode_scanner,barcode in flutter icon library
Proposal
Add missing material icons

Those icons are material symbols, not material icons.
Material symbols are not yet supported in flutter. See https://github.com/flutter/flutter/issues/102560
Closing this as a duplicate of the issue linked above.
Thank you


[type:fix] pin the spring-cloud version to 3.1.2

// Describe your PR here; eg. Fixes #issueNo
fix startup failed caused by wrong spring-cloud version in binary package.

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3494 (2a4a578) into master (55037b8) will decrease coverage by 0.38%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3494      +/-   ##
============================================
- Coverage     62.83%   62.45%   -0.39%
+ Complexity     5553     5517      -36
============================================
  Files           840      840
  Lines         23258    23256       -2
  Branches       2109     2107       -2
============================================
- Hits          14615    14524      -91
- Misses         7295     7378      +83
- Partials       1348     1354       +6

Impacted Files
Coverage Δ

...plugin/logging/rocketmq/LoggingRocketMQPlugin.java
22.72% <0.00%> (-72.73%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...ogging/rocketmq/body/LoggingServerHttpRequest.java
0.00% <0.00%> (-20.00%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
69.10% <0.00%> (-12.20%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...rg/apache/shenyu/plugin/rewrite/RewritePlugin.java
66.66% <0.00%> (-5.56%)
:arrow_down:

... and 30 more

Continue to review full report at Codecov.

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

I have did this thing.

@zhc00 @moremind  this pr maybe close?

@zhc00 @moremind this pr maybe close?

yes, I have did it, I will fix it in next pr.


This was abandoned?

I had an issue that was giving me an error that it coulnd't find the php-cs-fixer (on windows), and what I've changed is that is now calls the executable path directly, instead of php @executablePath.
You intend to keep developing this plug-in? Or can I fork and add my fixes?
Sorry for any problem I'm giving, I just wanted to keep this project on going as the CS Fixer is still on going as well.

No, it is not abandoned, but I have not much time atm. If you can make a pull request with your changes, I would like to merge it...

I sure did the fix for myself, but I'm not actually sure if this fix would have much impact on linux users, as I develop on Windows.
Actually, the plugin itself know doesnt needs the php call, just the command itself, can you assure me that it's now working the same way as the linux? Just to know if I will need to add any test to check wheter we are on unix or windows env.

the problem is, that I added this feature because of windows...

have you tried to use the full path to both, the php executable and the fixer executable?

Yes I did, actually what I've tested is that calling like php path/to/php-cs-fixer no longer works on windows. To get it to work, I needed to call it alone like just path/to/php-cs-fixer instead, without the PHP in front. What I'm asking is that this behaviour is only working for windows or in both it and unix OS based are working this way.

have you tried it with path/to/php path/to/php-cs-fixer?

Yes, it no longer works this way, I think it's because the new .bat file has the php paremeter in it, so when you call it php path/to/php-cs-fixer actually what it does is php php php-cs-fixer

I can not recheck this... can you confirm this behavior @mappedinn or @Dgame?

I'm no longer using Atom, sorry.

I was just waiting for any answer... So, can I develop the fix?

You can, but I would not recommend removing the php exe completely... perhaps a "setting" to use php or composer directly...

that was what i was looking for, and option to be able to call it directly, ok, I will pull request soon as I finished

nice, thanks :)

@hotwer are you still working on that?

@pfefferle Sorry, recently I've been working with VS Code, so, sorry, I only did the fix for myself, but didn't have time to go any further. Sorry about that.

no problem, but then I will close this issue for now.


Teams packages unavailable

The Teams Linux packages (rpm and deb) links from the download page point to non-existing files (404 http error) and indeed the repository for teams packages is empty since at least yesterday (2023-02-22).

Microsoft discontinued the Teams client for Linux.


Added the documentation and the script for lightweight solution

Hi @zoneout215, thanks for the update, but I think you missed many of the comments I made. I noticed that you marked some of the comments as resolved, but there was no update on them.
I agree with dropping support for Windows.

Hi @zoneout215, thanks for the update, but I think you missed many of the comments I made. I noticed that you marked some of the comments as resolved, but there was no update on them.
I agree with dropping support for Windows.

Oh, my appologies, I thought that when you resolve them, the changes are accepted, will do in a moment


WATERTABLED is on tile only

bug fix.

Thanks, @weiyuan-jiang, for identifying and fixing this bug.  It slipped through the cracks in 0-diff testing vs. develop because no test was writing out this new export.  And we never ran PEATCLSM with this output in ensemble mode, I think.
In any case, did you test the fix in the sense that we can now write out the ens_avg WATERTABLED?  Or do we just assume that it'll work?  It should from just looking at the src code.

@gmao-rreichle , I found this bug because of Qing's work.  @gmao-qliu please let us know if the fix works for you.

@gmao-rreichle , I found this bug because of Qing's work. @gmao-qliu please let us know if the fix works for you.

Yes, it works now. @weiyuan-jiang  @gmao-rreichle


nixos/networking: make setting up dhcpcd with networkmanager more straightforward

This PR fixes #341092, specifically using approach 2. As described by @Ten0:

Have the dhcpcd module still create the dhcpcd config files even if useDHCP is false, provided that networking.dhcpcd.enable == true, and have the networkmanager module set networking.dhcpcd.enable = true if networking.networkmanager.enable == true and networking.networkmanager.dhcp = ="dhcpcd".
So that we don't create those files for nothing if useDHCP is false but nothing otherwise enables dhcpcd, we could have networking.dhcpcd.enable's default value be config.networking.useDHCP, instead of just true, so that the config files are still not generated if useDHCP is false (unless of course systemd.services.dhcpcd is explicitly enabled).

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test dhcpcd

@ofborg test networking.networkmanager

I looks like you applied the autoformatter. Could you please make a separate commit for the autoformatting.


DSP-329 Fix sort button

resolves DSP-329
resolves DSP-314

In this PR I fixed the bug we had with sort projects and sort users.
I updated the default sort value:

"Project name" in projects
"Username" in users

This PR has to wait for new release of dsp-ui: DSP-693


docs: Change Microsoft to AKS & ACA and introduce name for managed support options

Change Microsoft to AKS & ACA and introduce name for managed support options
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Will fix DCO

Thanks for the help @thisisobate!


Introducing workspaces into pipelines

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] All static analysis checks passed.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Closing this PR for now.


texlive: package "mfirstuc"

Motivation for this change
Closes #15828
Things done
Add the mfirstuc package in the texlive distribution.
This is not well-tested, it fixes my issue but the doc and src hashes might be wrong as I don't know how to build the appropriate outputs to test them

Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

@vcunat

Hashes are correct, could you please also include it as a dependency of collection-latexextra?
Just add   deps."mfirstuc" = tl."mfirstuc"; around line 5442 in pkgs/tools/typesetting/tex/texlive-new/pkgs.nix

Many thanks, it works now for me by using just collection-latexextra.
@vcunat could you merge it to the mainline? Thanks in advance.

As of the version of texlive currently in master, mfirstuc is included in glossaries. Therefore I see no reason to add it separately; moreover such individual updates seem a risk of introducing inconsistencies into our snapshot.

See https://github.com/NixOS/nixpkgs/pull/16391

For the record: glossaries is not in latest unstable channel and therefor the commit introduced in this PR fixes the issue I've reported in #15828

Ah, right, I forgot some of the packages were updated unnaturally in 7abe192b so the set is rather inconsistent ATM and the glossaries version is probably without mfirstuc now.

@vcunat

Ah, right, I forgot some of the packages were updated unnaturally in 7abe192 so the set is rather inconsistent ATM and the glossaries version is probably without mfirstuc now.

This is exactly the case and prevents one from using glossaries on the unstable channel. How fast can we get the current version in master to propagate to unstable? Do we need an intermediate solution?

@steveeJ: nixpkgs-unstable and nixos-unstable have exactly the same texlive.* as master. There were no changes that recetly.

@vcunat

nixpkgs-unstable and nixos-unstable have exactly the same texlive.* as master. There were no changes that recetly.

Ah, I was mislead by your comment from June. So this makes the situation is even worse for my configuration. I have a mix of unstable and master nixpkgs, from which I can't see how to get the glossaries latex package. :-/ Am I missing something?

Applying this to current master doesn't result in a working glossaries package when texlive.combined.scheme-full is used as a scheme. The package gets built but latexmk can't find it.

is this still required?

@Mic92 My thesis is ready. I used this patch in nixpkgs to build my latex env all the time, so I guess it is somehow required. I don't care anymore about it, though. So feel free to close this PR unmerged!

The package was included within the full update #16391, so this PR isn't required anymore.


Add sell all functions to router and AMMs

Similar to https://github.com/open-web3-stack/open-runtime-module-library/blob/23c8363d53fe8580f636a7f820c54b994d1b96f7/tokens/src/lib.rs#L522 this would allow us to create simpler UX and pass through functionality with XCM programs allowing better composability in similar sense that buy_for works for fee payment.

We start with sell_all for router which solves the Max button issue.


Migrate events data (1.37.0)

I would kindly like to ask for support. I use events a lot, I have thousands of records. As is mentioned with the 1.37.0 release of Umami:

When you start the app for the first time, the database changes will be automatically applied. Your data however will not be automatically migrated because we can't determine what event_name you will want to use. Your old event table has been renamed to _event_old, which you can use to manually move data to the new table. If you've never used events, feel free to delete the _event_old table.

If I knew how, I would migrate the data from one to another table but data table migrations may be quite tricky. Has anyone done the migration yet please?

You may got some issues when upgrade to 1.37.0 from an old instance for now. See https://github.com/umami-software/umami/issues/1406

Assuming you stored only string values in your event_value table, you can run the following script to run migrations of your _event_old data into your new event table.  The most important thing is for your time-series data to stay intact (created_at).
INSERT INTO "event" (website_id, session_id, created_at, url, event_name)
SELECT website_id, session_id, created_at, url, event_value
FROM "_event_old"

We didn't include this in the migration because some users were not using the former type/value structure as intended (ex: storing json in event_value, etc).

Thanks
This script should be more clearly highlighted in the migration instructions, many default users run into an issue that has easy resolution. Maybe the db migrator could prompt users to migrate with this default behavior or skip).
Cheers

@briancao
I suggest that you add ; to the SQL commands after FROM _event_old – ie FROM _event_old;. Or highlight that you need to close out the SQL commands with ;.
I did not know (not that experienced in mysql), but then I saw @ajfriesen comment and it worked. I got all my events back.

Maybe I am missing something but as I understand the 2 new tables, I should also populate event_data table, right?
INSERT INTO event_data (event_id, event_data) SELECT (event_id, event_type) FROM _event_old ;
Don't use this query on production data, I have not tested it.

After this migration, is it still possible to track events using css tags?

Documentation seems to point that yes: https://umami.is/docs/track-events
In practice we lost css events tracking.

@kunago "event_type" has been dropped all-together and the event_data table does not need to be populated.  "event_data" can be seen as more of an additional feature going forward.  There is currently no way to even access that data in the UI at this time.  Unless you were one of the custom users that held JSON in their old event_type/event_value that is.

Okay, cool,, thank you for the explanation. So this is the solution:

Assuming you stored only string values in your event_value table, you can run the following script to run migrations of your _event_old data into your new event table. The most important thing is for your time-series data to stay intact (created_at).
INSERT INTO event (website_id, session_id, created_at, url, event_name)
SELECT website_id, session_id, created_at, url, event_value
FROM _event_old;

We didn't include this in the migration because some users were not using the former type/value structure as intended (ex: storing json in event_value, etc).
(Edit: added semi-colon to end of script per suggestion)


[IOTDB-6073] Add ClientManager metrics

Description
In IoTDB's cluster architecture, ClientManager is used to manage the client pool connected to specific services of other nodes, and the client is obtained through ClientManager for cross-node RPC communication.
Therefore, adding ClientManager metrics can show the communication between the current node and other nodes, which is helpful for locating exceptions and managing client resources.


Create a configuration file for setting download directory

I would be nice if it had a way to configure a default download directory and other possible options of the program.

Hello!
Thanks for the issue, @tarsobcaldas. I think this is a great idea that I hadn't really considered before. I'm envisioning potentially using TOML as well as allowing users to provide their own mirrors in case the hardcoded ones no longer work.
I'll whip something together and submit a PR.

@tarsobcaldas are you able to test this out from https://github.com/ciehanski/libgen-cli/pull/93?

It works!
Do you think it'd also be possible to set a default naming scheme? For example, my filenames for books are always something like:
klabnik2023-rust-programming-language.epub

That is, first author last name, year, and the title in dashes. Everything in lowercase. Maybe there some way of parsing the information and do the substitutions with some regex. Either let the user write the regex himself, or have some supported options.


Is it possible to "log"/write to the "Messages" field of the cron log entries?

The field is populated with errors/traceback.
I would like to be able to log some messages, so that when it's successful I can see some of the data/results as well.

I guess you could do it by just instantiating it's logger:
logger = logging.getLogger('django_cron')
May this could be included in the Docs?

There is an easier way to do what you want:
just return string from do method in your Cron class.

The problem is that after a certain number of lines it gets cut off.

as mentioned above: just return a string from do method in your Cron class.


time out error while connecting to kubernetes api

Hello,
I  am trying to set up a new Azure Kubernetes Cluster but straingly I got a time out error on half of "helm ls" or deployment commands. It is very slow responding as well.
22:44 $ kubectl version
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-11T18:14:22Z", GoVersion:"go1.13.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.7", GitCommit:"e2d9f8479783020904aba3de7499a49be6c75ebd", GitTreeState:"clean", BuildDate:"2020-04-09T02:31:15Z", GoVersion:"go1.13.6", Compiler:"gc", Platform:"linux/amd64"}
✔ ~/noo/portfolio/network-infrastructure [master L|●2✚ 18…11]
22:46 $ helm version
Client: &version.Version{SemVer:"v2.16.6", GitCommit:"dd2e5695da88625b190e6b22e9542550ab503a47", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.16.6", GitCommit:"dd2e5695da88625b190e6b22e9542550ab503a47", GitTreeState:"clean"}
helm ls
https://ggyzxcvadrqwerqweaxvnecentraluasdfamk8s.io:443/api/v1/namespaces/kube-system/configmaps?labelSelector=OWNER%!D(MISSING)TILLER: dial tcp: i/o timeout
Here are the tiller pod's logs
storage] 2020/06/04 19:33:25 listing all releases with filter
[storage] 2020/06/04 19:35:24 listing all releases with filter
[storage] 2020/06/04 19:36:16 listing all releases with filter
[storage] 2020/06/04 19:37:46 listing all releases with filter
[storage/driver] 2020/06/04 19:38:16 list: failed to list: Get https://gyzxcvadrqwerqweaxvnecentraluasdfamk8s.io:443/api/v1/namespaces/kube-system/configmaps?labelSelector=OWNER%3DTILLER: dial tcp: i/o timeout
[storage] 2020/06/04 19:41:26 listing all releases with filter
[storage/driver] 2020/06/04 19:41:47 list: failed to list: Get https://gyzxcvadrqwerqweaxvnecentraluasdfa.io:443/api/v1/namespaces/kube-system/configmaps?labelSelector=OWNER%3DTILLER: dial tcp: lookup gavnets.azmk8s.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:54995-><IP_ADDRESS>:53: i/o timeout
[storage] 2020/06/04 19:44:07 listing all releases with filter
[storage/driver] 2020/06/04 19:44:37 list: failed to list: Get https://gyzxcvadrqwerqweaxvnecentraluasdfas.azmk8s.io:443/api/v1/namespaces/kube-system/configmaps?labelSelector=OWNER%3DTILLER: dial tcp: i/o timeout
you can see that some call succeeds but some are not.
Thanks in advance.

@mehmetekici I have a couple questions to help diagnose the problem:

Did it work and then stop working?
Are you using RBAC in the cluster? If so, is Tiller setup to work with RBAC?

Hi, @mehmetekici - You are using Helm 2. As explained in the AKS docs, AKS allows you to start using Helm 3: https://docs.microsoft.com/en-us/azure/aks/kubernetes-helm#verify-your-version-of-helm - and if you check out https://helm.sh/blog/helm-v2-deprecation-timeline/ you will see instructions to migrate to Helm 3.
As we don't have more information to help diagnose, I will resolve this issue now. If you can give us additional information about your situation, we can help with Helm 2 questions until mid-November. Thanks.