Use ppm as basename in getResourcePath checks

This fixes the issue with core packages not showing in settings view caused by the change of apm install folder to ppm in pulsar-edit/pulsar#125.

Although while these changes are great, it occurs to me the decafe is done, and I'm in the process of reviewing. Not sure if you'd be open @jonian in opening another PR when I'm done merging those to apply these changes as well, but it is something I can do if needed. Otherwise I'll ping you when that process is over.
As always thanks for the contribution!

@confused-Techie It is no problem if this is merged first. I will redecaffeinate some files anyway.

@confused-Techie It is no problem if this is merged first. I will redecaffeinate some files anyway.

Glad to hear, in that case I'll go ahead and merge this one, thanks!

This is a bit late but I think this pull request is not correct. I think it has caused this error.

@Sertonix is that possible with ppm submodule in pulsar being https://github.com/pulsar-edit/ppm/tree/2f184753bcaf71e32c8a977f6b2e1d548f8298bb?

Also, I have pulsar installed from AUR, that uses this as a patch and I can install lang-gdscript without any issues.

Ok, then this might fix the problem. Do you know how to update the submodule?

@Sertonix @confused-Techie in fact this PR fixes that error. I re-installed the latest binary without applying this fix and the result is:

Glad to hear this PR is fixing the error. To update the submodule thatll have to be done as a PR to pulsar


Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when i have to add multiple icons at the same circle.
Describe the solution you'd like
I want them to adjust their position dynamically equal.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I want them to adjust their position dynamically equal. can you share a little more about this feature request?

Thank you for your response! I’m using the MagicUI library for a circular
animation, where icons rotate around a central element. The issue I’m
facing is that when I add more icons, they stack on top of each other
rather than positioning themselves evenly along the circle’s path. Ideally,
I want the icons to automatically adjust their positions so that they are
spaced equally around the circle, no matter how many icons are added.
Could you please suggest how this could be implemented, or if there's a
feature in the library that I may have missed that handles this case?
Thank you again for your support!
On Wed, Sep 18, 2024 at 11:06 PM Arghya Das @.***>
wrote:

I want them to adjust their position dynamically equal. can you share a
little more about this feature request?
—
Reply to this email directly, view it on GitHub
https://github.com/magicuidesign/magicui/issues/318#issuecomment-2359023622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6QV7KVMGA2ERFUC7ISK32DZXGZCNAVCNFSM6AAAAABOI6AZXWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJZGAZDGNRSGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

working on this #472


Use AST parser to check alt text

What
Closes #30
I updated the alt-text-bot to use an AST (remark) to parse images for bad alt text. Using an AST allows us to ignore any images defined in code blocks.
Why is this cool

In the future, we can add additional linting rules
The workflow is easier to read in javascript
We can test more easily
We can ensure that there aren't false positives
We can provide more feedback. For example, it is easier to indicate what line the alt text is found on.

TODO

[ ] - I was able to add codeblock tests and ensure all of our old test cases were passing. I have not had a chance to check if the action still runs properly.
[ ] - Add test case for blockquotes

Testing
If you find it useful I  used AST Explorer to test remark output.

Excited to see this! First question -- is there a reason you went with remark over markdownlint?  I would personally prefer the latter because it already comes with the no alt text rule & we also have additional rules in markdownlint-github.

Remark:

has a wider user base
is a much small package (107.4kb vs 583.5kb), though I recognize that it's not a huge deal in the case of a github action.
Documentation was a bit easier to follow for this specific scenario.
It is hooked up to the AST explorer making the AST syntax more discoverable
I don't think markdownlint api: lintInput allows you to test one rule I think they also include their default ruleset
Also they have some accessibility focused plugins too!


Macbook Pro Multiple Monitors header bar not hidden when switch to desktop (Mac Native Monitor only)

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

VS Code Version: 1.73.1
OS Version: macOS Ventura 13.0

Steps to Reproduce:

MacBook connect to another monitors: Monitor A -- Monitor B(MacBook) -- Monitor C
In Monitor B ( MacBook ), goto full screen VSCode, hover top menu, keep cursor not moving, then slide to desktop, then move cursor down to monitor middle, then slice back to full screen VSCode
result: VSCode top menu bar not hidden.

not sure is bug of macOS or VSCode.

https://user-images.githubusercontent.com/2208880/205017552-208b3397-7d70-435f-9b3a-568c37256f5a.mp4

This is native macos behavior, it could be electron, but I don't think its specific to VS Code. @vikyd do you have any other electron apps to try this on? e.g. slack?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Unable to build Android version

After following the instruction for flutter integration, I'm trying to build for Android and getting the following error message:
> Manifest merger failed :
Attribute application@fullBackupContent value=(@xml/tw_backup_rules) from AndroidManifest.xml:7:153-201
is also present at [com.appsflyer:af-android-sdk:6.3.2] AndroidManifest.xml:16:18-73 value=(@xml/appsflyer_backup_rules).
Suggestion:
add 'tools:replace="android:fullBackupContent"' to <application> element at AndroidManifest.xml:7:5-99:19 to override.

After some reading, I have tried adding tools:replace="android:fullBackupContent" to the application attribute, but then I get:
> org.xml.sax.SAXParseException; systemId:
file:/C:/dev/12Tech12/git/client/tw_app/android/app/src/main/AndroidManifest.xml; lineNumber: 7; columnNumber: 325;
The prefix "tools" for attribute "tools:replace" associated with an element type "application" is not bound.

Using:
AppsFlyer version 6.3.3-nullsafety.0
Flutter version 2.2.3

Fixed by adding xmlns:tools="http://schemas.android.com/tools" to the manifest attribute and merging the rules from @xml/appsflyer_backup_rules into my own backup rules @xml/tw_backup_rules.


Local engine build failure (missing browser_launcher pkg)

Happens when building without prebuilt SDK (probably that's why it doesn't happen on CI)
With recent master building local engine fails due to missing browser launcher package:
[2469/2670] ACTION //third_party/dart/utils/dartdev:generate_dartdev_kernel_dill(//build/toolchain/mac:clang_x64)
FAILED: gen/third_party/dart/utils/dartdev/generate_dartdev_kernel.dart.dill
vpython3 ../../third_party/dart/build/gn_run_binary.py compiled_action /Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/dart -Dsdk_hash=c17fd9df12 --dfe=/Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot /Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/gen_kernel/bootstrap_gen_kernel.dill --packages=/Users/Matej/Projects/flutter/engine/src/third_party/dart/.packages --platform=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/vm_platform_strong.dill --no-aot --no-embed-sources --no-link-platform --output=gen/third_party/dart/utils/dartdev/generate_dartdev_kernel.dart.dill --depfile=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/dartdev/generate_dartdev_kernel.dart.dill.d -Dsdk_hash=c17fd9df12 /Users/Matej/Projects/flutter/engine/src/third_party/dart/pkg/dartdev/bin/dartdev.dart
Command failed: /Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/dart -Dsdk_hash=c17fd9df12 --dfe=/Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot /Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/gen_kernel/bootstrap_gen_kernel.dill --packages=/Users/Matej/Projects/flutter/engine/src/third_party/dart/.packages --platform=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/vm_platform_strong.dill --no-aot --no-embed-sources --no-link-platform --output=gen/third_party/dart/utils/dartdev/generate_dartdev_kernel.dart.dill --depfile=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/dartdev/generate_dartdev_kernel.dart.dill.d -Dsdk_hash=c17fd9df12 /Users/Matej/Projects/flutter/engine/src/third_party/dart/pkg/dartdev/bin/dartdev.dart
output: ../../third_party/dart/third_party/devtools/devtools_server/lib/src/server.dart:12:8: Error: Error when reading '../../third_party/dart/third_party/pkg/browser_launcher/lib/browser_launcher.dart': No such file or directory
import 'package:browser_launcher/browser_launcher.dart';
       ^
../../third_party/dart/third_party/devtools/devtools_server/lib/src/server.dart:270:13: Error: Getter not found: 'Chrome'.
      await Chrome.start([url]);
            ^^^^^^
../../third_party/dart/third_party/devtools/devtools_server/lib/src/server.dart:963:24: Error: Getter not found: 'Chrome'.
    final proc = await Chrome.start([uriToLaunch.toString()], args: args);
                       ^^^^^^

[2473/2670] ACTION //third_party/dart/utils/kernel-service:kernel-service_snapshot(//build/toolchain/mac:clang_x64)
ninja: build stopped: subcommand failed.

And apparently also webkit inspection:
[70/202] ACTION //third_party/dart/utils/dartdev:generate_dartdev_snapshot_dill(//build/toolchain/mac:clang_x64)
FAILED: gen/third_party/dart/utils/dartdev/generate_dartdev_snapshot.dart.dill
vpython3 ../../third_party/dart/build/gn_run_binary.py compiled_action /Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/dart -Dsdk_hash=c17fd9df12 --dfe=/Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot /Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/gen_kernel/bootstrap_gen_kernel.dill --packages=/Users/Matej/Projects/flutter/engine/src/third_party/dart/.packages --platform=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/vm_platform_strong.dill --no-aot --no-embed-sources --no-link-platform --output=gen/third_party/dart/utils/dartdev/generate_dartdev_snapshot.dart.dill --depfile=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/dartdev/generate_dartdev_snapshot.dart.dill.d -Dsdk_hash=c17fd9df12 /Users/Matej/Projects/flutter/engine/src/third_party/dart/pkg/dartdev/bin/dartdev.dart
Command failed: /Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/dart -Dsdk_hash=c17fd9df12 --dfe=/Users/Matej/Projects/flutter/engine/src/third_party/dart/tools/sdks/dart-sdk/bin/snapshots/kernel-service.dart.snapshot /Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/gen_kernel/bootstrap_gen_kernel.dill --packages=/Users/Matej/Projects/flutter/engine/src/third_party/dart/.packages --platform=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/vm_platform_strong.dill --no-aot --no-embed-sources --no-link-platform --output=gen/third_party/dart/utils/dartdev/generate_dartdev_snapshot.dart.dill --depfile=/Users/Matej/Projects/flutter/engine/src/out/host_debug_unopt/gen/third_party/dart/utils/dartdev/generate_dartdev_snapshot.dart.dill.d -Dsdk_hash=c17fd9df12 /Users/Matej/Projects/flutter/engine/src/third_party/dart/pkg/dartdev/bin/dartdev.dart
output: ../../third_party/dart/third_party/pkg/browser_launcher/lib/src/chrome.dart:10:8: Error: Error when reading '../../third_party/dart/third_party/pkg/webkit_inspection_protocol/lib/webkit_inspection_protocol.dart': No such file or directory
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';
       ^
../../third_party/dart/third_party/pkg/browser_launcher/lib/src/chrome.dart:56:9: Error: Type 'ChromeConnection' not found.
  final ChromeConnection chromeConnection;
        ^^^^^^^^^^^^^^^^
../../third_party/dart/third_party/pkg/browser_launcher/lib/src/chrome.dart:56:9: Error: 'ChromeConnection' isn't a type.
  final ChromeConnection chromeConnection;
        ^^^^^^^^^^^^^^^^
../../third_party/dart/third_party/pkg/browser_launcher/lib/src/chrome.dart:63:31: Error: Method not found: 'ChromeConnection'.
      _connect(Chrome._(port, ChromeConnection('localhost', port)));
                              ^^^^^^^^^^^^^^^^
../../third_party/dart/third_party/pkg/browser_launcher/lib/src/chrome.dart:123:7: Error: Method not found: 'ChromeConnection'.
      ChromeConnection('localhost', port),
      ^^^^^^^^^^^^^^^^

I confirm this, I saw the same issue yesterday so I had to delete all local engine entirely

I confirm this, I saw the same issue yesterday so I had to delete all engine entirely and download fresh setup

That would only help if you built it with prebuilt SDK after.


Delete untranslated contents in the l10n(ko) directory

After #10996, untranslated contents are not needed in the contents/ko directory.

/assign
/lgtm
/approve


Transaction callback error: Cannot read property 'then' of undefined

I tried to catch transaction callback but it gives undefined error.
(I'm passing the initial codes, define db, openDatabase etc..)
  componentDidMount() {
    const { db } = this.state;

    db.transaction(tx => {
      tx.executeSql('SELECT * FROM tblUser;', [], (tx, results) => {
        const rows = results.rows;
        let users = [];

        for (let i = 0; i < rows.length; i++) {
          users.push({
            ...rows.item(i),
          });
        }

        this.setState({ users });
      },
      )
    }).then(() => console.log("TRANSACTION DONE"))  // <== GIVES ERROR
  }

Error output:
C:\Users\ezran\Desktop\TestingProjects\testProject\node_modules\react-native\Libraries\Core\ExceptionsManager.js:44 TypeError: Cannot read property 'then' of undefined

This error is located at:
    in App (at renderApplication.js:40)
    in RCTView (at AppContainer.js:101)
    in RCTView (at AppContainer.js:119)
    in AppContainer (at renderApplication.js:39)
reportException @ C:\Users\ezran\Desktop\TestingProjects\testProject\node_modules\react-native\Libraries\Core\ExceptionsManager.js:44
handleException @ C:\Users\ezran\Desktop\TestingProjects\testProject\node_modules\react-native\Libraries\Core\ExceptionsManager.js:113
showErrorDialog @ C:\Users....
.....
.....
....native\Libraries\BatchedBridge\MessageQueue.js:109
(anonymous) @ debuggerWorker.js:80
Show 7 more frames
C:\Users\ezran\Desktop\TestingProjects\testProject\node_modules\react-native\Libraries\Core\ExceptionsManager.js:44

TypeError: Cannot read property 'then' of undefined

This error is located at:
    in App (at renderApplication.js:40)
    in RCTView (at AppContainer.js:101)
    in RCTView (at AppContainer.js:119)
    in AppContainer (at renderApplication.js:39)

Versions:
    "react": "16.9.0",
    "react-native": "0.61.1",
    "react-native-sqlite-storage": "^4.1.0"

I guess you didn't enable promises with
SQLite.enablePromise(true);

So you can't use db.transaction() as a Promise

@Al10s SQLite.enablePromise(true); did the trick.


[AutoRelease] t2-servicelinker-2024-03-07-57247(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5035
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=3074

/check-enforcer override


Goggle Nexus 7 (2012) added to devices

Tested my google Nexus 7 running LineageOS 14.1 (Andriod 7.1.2). Works with a USB hub. Can add a video of it working :)

thanks for your feedback :)


Map takes tap event when tap is performed on marker

Hello,
I have added a feature to add pins on tap of map. Pin also displays info window when tap is performed on pin. When I tap on pin to open info window, map also adds new pin because map takes tap event. I could not find anything to fix it and looks like a defect.

I have a similar problem: when I tap on a marker onTap callback is triggered. The second time I tap on the same marker the onTap callback is NOT triggered.

I have same issue @dev4jam. Have you found any solution?

I faced the same issue and found a very hacky workaround for it. I just append a random uuid to the marker id and call setState() when tapping the marker. E.g.
PlatformMap(
                initialCameraPosition: initialCamera,
                markers: locations.map(
                        (location) {
                          return Marker(
                            markerId:
                                MarkerId(location.locationId + uuid.v4()),
                            position: LatLng(
                              location.geoCoordinate.latitude,
                              location.geoCoordinate.longitude,
                            ),
                            consumeTapEvents: true,
                            onTap: () {
                              // handle tap logic
                              setState(() {});
                            },
                          );
                        },
                      ).toSet(),
                gestureRecognizers: const {
                  Factory<OneSequenceGestureRecognizer>(
                    EagerGestureRecognizer.new,
                  ),
                },
)

We are still experiencing this issue. Any ideas?


qingstor support private cloud and replace the sdk repository

Codecov Report

Merging #1303 (5df8746) into main (8f95046) will increase coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1303      +/-   ##
==========================================
+ Coverage   59.32%   59.39%   +0.07%
==========================================
  Files         126      126
  Lines       21466    21470       +4
==========================================
+ Hits        12734    12752      +18
+ Misses       7201     7186      -15
- Partials     1531     1532       +1

Impacted Files
Coverage Δ

pkg/object/qingstor.go
1.14% <0.00%> (-0.03%)
:arrow_down:

pkg/meta/sql_lock.go
72.84% <0.00%> (-2.65%)
:arrow_down:

pkg/meta/sql.go
67.91% <0.00%> (+0.25%)
:arrow_up:

pkg/chunk/disk_cache.go
81.17% <0.00%> (+0.76%)
:arrow_up:

pkg/chunk/cached_store.go
72.90% <0.00%> (+2.44%)
:arrow_up:

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 8f95046...5df8746. Read the comment docs.


Update releases and nuget

The releases haven't been updated recently and now with XChaCha20-Poly1305 supported by the library, I'd like to use it in my project. Unfortunately I'm unable to do it due to the fact that the nuget is still on version 20.2.0 and hasn't been updated since 2020

There are 3 preview versions on NuGet now.

Done. 😃


Fixed various UBSan warnings about working with NULL pointers

Any pointer arithmetic with NULL pointers is technically UB, even if you don't end up dereferencing the pointer.

Many thanks!


How to delete log files based on maximum number of files

Hi,
I want to limit log files to a maximum number so that if the number of log files exceed, and a new log file comes, the oldest gets deleted. I could do it using log file archiving, but I am trying to avoid archiving.
This is my present code:-
NLog.Targets.FileTarget target = new NLog.Targets.FileTarget();
fileTarget.FileName = "${basedir}/logfile.${shortdate}.log";
fileTarget.AutoFlush = true;
fileTarget.ArchiveFileName =  "${basedir}/ArchivedLogs/logfile.${shortdate}.log";
fileTarget.MaxArchiveFiles = 3;
fileTarget.ArchiveEvery = NLog.Targets.FileArchivePeriod.Day;
fileTarget.ArchiveNumbering = NLog.Targets.ArchiveNumberingMode.Date;
fileTarget.ConcurrentWrites = true;
fileTarget.KeepFileOpen = true;
This will work, but I would like to avoid archiving (so that my application would only have log files and not log archives which is not required for my application).
Please let me know if you need additional info. Thanks.
Type :

Question

NLog version:

<IP_ADDRESS>

Platform:

.Net 4.0

This is currently only possible with archiving.

PR #1993 will allow you to specify only FileTarget.FileName and FileTarget.MaxArchiveFiles, and it will cleanup old files within the same folder.


MAP_TILEMAP_URL or MAP_TILEMAP_OPTIONS_MAXZOOM are not taken into account

Describe the bug
here is my docker compose
  kibana:
    ports:
      - '<IP_ADDRESS>:5691:5601'
    image: amazon/opendistro-for-elasticsearch-kibana:1.11.0
    environment:
      ELASTICSEARCH_URL: https://elasticsearch:9200
      ELASTICSEARCH_HOSTS: https://elasticsearch:9200
      MAP_TILEMAP_URL: 'https://xxxxx/{z}/{x}/{y}.png'
      MAP_TILEMAP_OPTIONS_MAXZOOM: 19

Those two last value are not taken into account. but when I manually set in kibana.yml it's working
I don't know if this file bellow is used, but it's old notation for map plugin :
https://github.com/opendistro-for-elasticsearch/opendistro-build/blob/master/kibana/docker/build/kibana/bin/kibana-docker
To Reproduce
set env file in the yaml for map
Expected behavior
new map layer
Configuration (please complete the following information):
odfe 1.11

Hi @DariusMR is this a new bug that only shows in 1.11, or in any of the previous versions?
Thanks.

1.9 also didn't try the other versions

Hi @DariusMR seems like this is a xpack feature not ODFE.
https://www.elastic.co/guide/en/kibana/current/tile-layer.html

Hi @DariusMR seems like this is a xpack feature not ODFE.
https://www.elastic.co/guide/en/kibana/current/tile-layer.html

Hi @DariusMR,
I am closing this issue as there has been no response and also the feature doesn't belong to ODFE. Please feel free to re-open the issue if any concern.
Thanks for reaching out to us.

Hi @DariusMR,
I am closing this issue as there has been no response and also the feature doesn't belong to ODFE. Please feel free to re-open the issue if any concern.
Thanks for reaching out to us.


Make h5py a travis dependency

This implements a change to the tests that I discussed earlier today with @aphearin: it adds h5py as a dependency for the travis tests.  Testing h5py functionality is generally desired because basically the only reason h5py is not a required dependency is because it won't build on RTD.
This also adds an additional Travis build that's the same as the standard py2.7 build except that it does not have h5py this is here to make sure that an "accidental" dependency isn't injected (e.g., a module-level import of h5py), as in that case the RTD builds will start failing.

Great, thanks @eteq for hopping right on this and making informative in-line comments.

Not sure if you found a better fix, but see #154 for officially sanctioned solution


nixos/nginx: Enable TLS 1.3 support

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

LGTM, test deployed to a server and works as expected! Thanks! :)

I've taken the liberty of rephrasing the release notes documentation myself so we can get this in today before the branch-off. A quick review would be appreciated!

Do not see any changes https://github.com/NixOS/nixpkgs/compare/3cedb8cbbf46b61bdfb9b0d138661c61eb057307..b3fbb0498a5a36c3e8082b4e73fc279acbde849a

Argh, my mistake. New attempt here.

@lheckemann can you (back)port this to release-19.03?

Done! Thanks for the pointer.

Uhm… did commit 051e852 (actually build nginx with openssl_1_1) just fall under the table in the release-19.03 channel? »You're not going to space^W^W^Wenabling TLS 1.3 today« ;)

:fearful: oops. Good catch, fixed!


Add eslint and github action for eslint

I added config file for eslint and GitHub Actions to fix errors easily. More, LINTER.md file is added to explain how to use eslint and GitHub Actions.

I'm closing this pull request because there is no an action for a long time.


Cannot read property 'shadowRoot' of null

Ha 0.115.3
Custom header 1.8.0
Noticed a CH error in the log after opening HA using the official android app.
There were some similar  issues in previous versions.
Logger: frontend.js.latest.202009182
Source: components/system_log/init.py:193
First occurred: 20:53:06 (1 occurrences)
Last logged: 20:53:06
https://<IP_ADDRESS>:8123/hacsfiles/custom-header/custom-header.js:146:49227 Uncaught TypeError: Cannot read property 'shadowRoot' of null

This applies to both issues you just submitted. Ensure that the latest version is installed using the the info from at the beginning of the bots reply above on a computer or another browser that can show you the dev tools console. If that reports version 1.8.0, make sure that the companion app is using the latest by clearing it's cache. Hold down on the app icon and select "App Info", then click "Storage and cache", then click "clear cache".
If both of your issues are still a problem, please follow the bots comment above and provide the as much of the necessary information as you can so I can try to replicate the issue.
My phone is an Android phone and I use the companion app, I have neither of these issues. In order for me to replicate the issues, I need info and to make sure it's not a cache/install issue.

This also seems like an issue that may have been fixed by 1.8.3, please let me know if updating helps. If not I will reopen this issue and investigate further.

Will do. Thanks!
Ryan Meek<EMAIL_ADDRESS>schrieb am Mi., 30. Sep. 2020, 22:13:

This also seems like an issue that may have been fixed by 1.8.3, please
let me know if updating helps. If not I will reopen this issue and
investigate further.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/maykar/custom-header/issues/267#issuecomment-701619669,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACO7FKEALHSGCGS362ST5ILSIOGQPANCNFSM4R43YF7A
.


Update external dependencies

Update Cimg to 1.7.3
Update giflib to 5.1.4
Update libjpeg to 9b
Update libpng to 1.6.23
This addresses some potential security flaws in lwip (libpng in the version used currently contains some potentially high-level security bugs).
Updated all other libs in the same go. tests pass.
Relevant CVEs for libpng specifically:
https://www.cvedetails.com/cve/CVE-2015-8126/
https://www.cvedetails.com/cve/CVE-2015-8472/

Coverage remained the same at 98.311% when pulling f0b5d81bac641df54a3b9a40853e3b05ae9ef479 on WatchBeam:master into a65f6086a26c22736eb4d7cfa43c096afad5b5e8 on EyalAr:master.

In order to make lwip work with node 7.6.0 I had to update libpng and zlib to latest versions. If you are brave enough you can use https://www.npmjs.com/package/pajk-lwip v0.2.0.
https://github.com/Pajk/lwip/tree/development

Thanks @Pajk . I was able to install lwip on node 7.7.1 with your development branch.


Inlcude changed keys when broadcasting to model class? (updates_for :only)

My use case is that I would like to put this in a view:
<%= updates_for StaffMember, only: 'current_task' do %>
 <%# show what people are working on #%>
<% end %>

That is: trigger a refresh when the current_task of any StaffMember changes. However, it seems like broadcasts to the subscription ID of a model class are always an empty hash, so the :only option is effectively ignored.
I'm pretty sure that's happening here: https://github.com/stimulusreflex/cable_ready/blob/40bc658cc500b0715dfb9fb49cf539f23f24f98c/app/models/concerns/cable_ready/updatable/model_updatable_callbacks.rb#L22-L25
Could we include the changed attributes when broadcasting to the model.class as well? I feel like it would at least be harmless, but maybe I've missed the reasoning for doing it this way.

My understanding of line 23 is that broadcasts to the Class are specifically to respond to records being created or destroyed. The intended use case is that you have an index view, and you want to track items being added or removed from the list.
Your intention sounds perfectly reasonable to me, though.
What I suggest is that you clone the CR repo and try out what you're looking to do, and if it works, please make a PR. I'll even give you the code:
def broadcast_update(model)
  changeset = model.respond_to?(:previous_changes) ? {changed: model.previous_changes.keys} : {}
  ActionCable.server.broadcast(model.class, changeset)
  ActionCable.server.broadcast(model.to_global_id, changeset)
end

My understanding of line 23 is that broadcasts to the Class are specifically to respond to records being created or destroyed.

Those are handled by broadcast_create (which is also aliased as broadcast_destroy).

What I suggest is that you clone the CR repo and try out what you're looking to do, and if it works, please make a PR.

I actually monkey-patched this locally in my app already to test it and it works. However, I am not using CableReady very extensively so I thought I'd ask here first. I've created PR #193 with those changes.

Sorry, wrt line 23 I wasn't speculating on how it works, I was speculating on the intended functionality of the feature. Apologies, that wasn't as clear as it could have been.
So really what this PR is about (since I think I already gave you the code, such as it is) is the bigger "why" question, a close cousin of "wait, does this make sense?"
I'm hoping that @julianrubisch can help me get my thoughts straight on this. Does it make sense to broadcast changed columns to the class form components? I think so, but need another pair of eyes.


URL links for rst not formatted correctly

See description here: https://github.com/ModECI/modelspec/blob/main/examples/document.rst#section
Propagating to docs being generated here: https://mdf.readthedocs.io/en/latest/api/Specification.html#id9

Probably an issue in here:
https://github.com/ModECI/modelspec/blob/d04c5f66209884c9845f94b4876c1b034c1dd2cb/src/modelspec/base_types.py#L583

@pgleeson while referring any sub section inside an RST document (what I got from internet) instead  of
Document

we must use
Document_ .._Document

I tried hard-coding this and it works correctly i.e a hyperlink of sub section inside the RST doc is created
and I will try to add this inside the
def insert_links

function.
this is what we want right?

Fixed in #18


Bandpass filter option not working

Hi there,
Trying to use the bp filter option (--fmin and --fmax) with analyze.py in command line. I get the error that --fmin and --fmax are unrecognized arguments. Are these no longer settable?
Thanks! Happy to provide additional detail if needed.
Morgan

No, it's still there. Make sure you are using the current version.

Thanks! I must have had an old version downloaded.
--Morgan
Morgan Ziegenhorn, PhD.
she/they
"*All that is in our minds is also in theirs." *- William Henry Hudson
On Tue, Mar 5, 2024 at 12:30 AM Josef Haupt @.***>
wrote:

No, it's still there
https://github.com/kahst/BirdNET-Analyzer/blob/7c6fe30231525a233217e28f3d7609bde7be7e5f/analyze.py#L501.
Make sure you are using the current version.
—
Reply to this email directly, view it on GitHub
https://github.com/kahst/BirdNET-Analyzer/issues/279#issuecomment-1978209736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJUZGNVUT5DIC4ALG3KRVZDYWV7DFAVCNFSM6AAAAABEF4IMJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYGIYDSNZTGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Semantic versioning

I've moved the AgonConsole8 fork of agon-vdp to use semantic versioning.
There's several reasons why.  On other projects I've used CI tooling that will do things like automatically increment version numbers and create new releases when new commits get pushed to main, and at some point I'd like to implement that.
Another reason is that I think it would be good for us to make more frequent releases.  By having "major.minor.patch" it feels like the stakes are lower for cutting a new release that contains only patch changes.  It's also not such a big deal to add in new minor features.
This PR sets the version at "2.0.0 Alpha 1".  This is a straw-man number.  Given that there has been quite a lot of changes made since the last full release, and this moves to a new version numbering system it feels like a justifiable number.
From the commit message:
move version info out to version.h file.
version info now a semantic version, with major, minor, patch, and “candidate” info, as well as an indicator for candidate type (displayed if candidate number is non-zero)
support for showing an automatically-generated build number added, which can be provided by a build script and is intended to be derived from the git hash
with this approach it should also be possible in the future to add other tooling to automatically increment version numbers
version info also includes the codebase variant, allowing us to easily differentiate between Quark and Console8

I'll switch to this after 1.04 has been released properly, save confusing everyone (including myself)


Latest version not published to Terraform registry?

Latest version seems to be 2.7.1 https://registry.terraform.io/providers/Telmate/proxmox/latest could it be updated to 2.7.2 please?

Cheers 2.7.3 worked.


Fix：ComQuery write row not correct

What this PR does:
Fix：ComQuery write row not correct
Which issue(s) this PR fixes:

Fixes #2

action-golangci-lint 应该加上


GRPC status should check against HTTP status code if not otherwise provided

Fixes #82 .

Yup, that's what would happen. As it turns out grpc-status is usually only
set on error... So this happened to work. Ish.
On Mon, Sep 10, 2018, 7:59 PM Sean McArthur<EMAIL_ADDRESS>wrote:

@seanmonstar approved this pull request.
Seems good to me!
In src/client/streaming.rs
https://github.com/tower-rs/tower-grpc/pull/83#discussion_r216532715:

     // Destructure into the head / body
     let (head, body) = response.into_parts();

   if let Some(status) = super::check_grpc_status(&head.headers) {

I'm just trying to be sure I understand, but it seems like the behavior
before was that the existence of any grpc-status trailer would return
an error, always? That seems like a woops...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tower-rs/tower-grpc/pull/83#pullrequestreview-154034878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4FwXPKA4UGpeWRBgCA6ZxbFzf_4L6Wks5uZycugaJpZM4Wid1b
.

Wait, so then that'd mean that this would be turning cases that were OK into UNKNOWN, right?

I remember now. As stated in the spec https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md#responses, it's possible for failures to include a Trailers-Only response. This is when the response headers contain the grpc-status immediately, so that 2 headers frames aren't needed if there would otherwise be no body.
So, in that case, it should only check the headers, and map the status code in trailers.

How do we check the trailers-only case here? What confuses me is that this change (as it stands) actually works in the case where there is a real gRPC body (and I would expect it to fail).

If the initial headers include a grpc-status, then it is mostly meant as an error, as indicated in the spec. If they aren't there, then the status depends on the final trailers (or, if no trailers come, then map the original HTTP status code). Does that make sense?

@seanmonstar I implemented roughly the logic we're discussing here in the newest commit on the branch. With local testing it seems to work correctly, although that was also true of the prior diff...


cdb_cookie management

fixes https://github.com/CartoDB/cartodb/issues/13894 and as a bonus, creates the cookie when you log in from cartodb

not sure about this approach. WHat do you think @javitonino ?

:red_circle: Cookies are not deleted in logout. Looking at the headers and comparing with the session cookie, it seems to be missing the domain in the cookie deletion.


Add permissive license

We need to add a license to this repo: I was thinking of MIT or similar.

MIT seems fine to me, please go ahead.


choice of DEFAULT_FLUSH_INTERVAL for StatsDAggregator

The aggregation documentation says:

...so long as the flush interval configured configured on the client side is smaller than said flush interval on the server side there should no loss in resolution

But from what I can tell, the default values for the client here in java-dogstatsd-client of 3 seconds is quite a bit bigger than the default flush interval in the datadog agent of 100 milliseconds.  From what I can see the default in the datadog agent hasn't changed in some time, if ever.
Does it make sense to choose a smaller number here to make the defaults "safe"?

@dbyron0 thank you for showing your concern with regards to the aggregation feature and its implementation and for digging deeper into it.
The value we're really interested in for the dogstatsd server is the aggregation granularity of buckets which is defined here: https://github.com/DataDog/datadog-agent/blob/master/pkg/aggregator/aggregator.go#L32 and has a hard-coded value of 10s; that's the size of the aggregation buckets. The packet buffer flush period of 100ms has no impact in the actual aggregation of contexts. And as you very well pointed it was important to not be overly aggressive and define conservative sane values which is why we went for a 3s flush interval in the client.
I hope this clarifies things. I am going to go ahead and close this issue, but please feel free to reach out if anything remains unclear. Once again, thank you for reaching out! 🙇

@dbyron0 thank you for showing your concern with regards to the aggregation feature and its implementation and for digging deeper into it.
The value we're really interested in for the dogstatsd server is the aggregation granularity of buckets which is defined here: https://github.com/DataDog/datadog-agent/blob/master/pkg/aggregator/aggregator.go#L32 and has a hard-coded value of 10s; that's the size of the aggregation buckets. The packet buffer flush period of 100ms has no impact in the actual aggregation of contexts. And as you very well pointed it was important to not be overly aggressive and define conservative sane values which is why we went for a 3s flush interval in the client.
I hope this clarifies things. I am going to go ahead and close this issue, but please feel free to reach out if anything remains unclear. Once again, thank you for reaching out! 🙇

Thanks for clarifying.  This would be helpful info to have directly in the readme.  I'll try to carve out some time to make a PR, but wouldn't be surprised if someone else beats me to it.
FWIW, the code you linked to wasn't super clear to me at first glance either...The line above has a comment about a 15 second flush interval, and it takes some digging to see that the units of bucketSize are seconds.  Probably the key thing is that any kind of flush interval in the agent sounds like it's not relevant...and it's the aggregation buckets that matter.  I bet that's something you (or someone else) can explain way better than me.

Thanks for clarifying.  This would be helpful info to have directly in the readme.  I'll try to carve out some time to make a PR, but wouldn't be surprised if someone else beats me to it.
FWIW, the code you linked to wasn't super clear to me at first glance either...The line above has a comment about a 15 second flush interval, and it takes some digging to see that the units of bucketSize are seconds.  Probably the key thing is that any kind of flush interval in the agent sounds like it's not relevant...and it's the aggregation buckets that matter.  I bet that's something you (or someone else) can explain way better than me.


chore(master): release 1.5.9

:robot: I have created a release beep boop
1.5.9 (2023-07-22)
Miscellaneous

deps: update andreasaugustin/actions-template-sync action to v1.1.1 (2b0067e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-project-services/releases/tag/v1.5.9 :sunflower:


added support for custom React ID Attribute

This commit adds support for cases when React is changed to use a custom React ID attribute. In browserify, the react ID can be changed as follows:
require('react/lib/DOMProperty').ID_ATTRIBUTE_NAME = 'data-reactid-...';
So this commit makes sure that ReactShadow gets the right ID Attribute in case its running as a commonjs module

Thanks @ziahamza! :+1:


Bug: VRP env bug

It should be:
self.order_dict[order_idx][‘Status’] == 1
['Status'] missed

Thank you for pointing this out. It's been fixed!


allow embedding heatmap_slicer

allows passing a figure instance to heatmap_slicer so that it can be embedded into a gui. I made an example but ideally this will make it into the docs one day as well. Perhaps as a standalone embedding section?
attn: @drfeinberg I think this fixes your issue.

based on helpful discussions here: https://discourse.matplotlib.org/t/mpl-iteractions-with-gui-outside-of-jupyter-notebooks/21523/5

failing formatting is fixed by https://github.com/ianhi/mpl-interactions/pull/107

Looks awesome.  I'll check it out asap.  Thanks for working on this!

not quite yet though! I won't have time to get it on pypi until sometime this afternoon


Allow for alternative compiler

Ran into this having clang-11 rather than clang
Description
This PR fixes compiling on certain linux distributions/configurations

I will recommend using the pre-compiled binaries when possible

Yeah very true, but I'm working on aarch64 which doesn't have any yet, so I thought I might as well upstream some of the needed changes

I will open an issue to add official support for aarch64


Doxygen documentation issue

Summary
Having the error "Unable to resolve reference to clio/src/rpc/README.md' for \ref command" with doxygen version 1.11.0 when trying to commit code.

duplicate of #1431


Mahout 0.10.x ora 0515 MAHOUT-1660 MAHOUT-1713 MAHOUT-1714 MAHOUT-1715 MAHOUT-1716 MAHOUT-1717 MAHOUT-1718 MAHOUT-1719 MAHOUT-1720 MAHOUT-1721 MAHOUT-1722 MAHOUT-1723 MAHOUT-1724 MAHOUT-1725 MAHOUT-1726 MAHOUT-1727 MAHOUT-1728 MAHOUT-1729 MAHOUT-1730  MAHOUT-1731 MAHOUT-1732

Cumulative patch for the issues above.

@avati @Anand Avati: support for MAHOUT-1717 needed for H20 that comes with this PR #135 (allreduceBlock)

tracked as a part of this #135 : (for asfgit to leave a reference in these issues):
MAHOUT-1660 MAHOUT-1713 MAHOUT-1714 MAHOUT-1715 MAHOUT-1716 MAHOUT-1717 MAHOUT-1718 MAHOUT-1719 MAHOUT-1720 MAHOUT-1721 MAHOUT-1722 MAHOUT-1723 MAHOUT-1724 MAHOUT-1725 MAHOUT-1726 MAHOUT-1727 MAHOUT-1728 MAHOUT-1729 MAHOUT-1730 MAHOUT-1731 MAHOUT-1732

MAHOUT-1713 is part of #135

MAHOUT-1714 is part of #135

MAHOUT-1715 is part of #135

MAHOUT-1716 is part of #135

@dlyubimov Yes, I will have a look into it. How do you prefer:

You merge this by leaving ??? in H2O, and I raise a new PR after that replacing ??? with actual code

I work against this development branch, you merge my code and re-submit this PR that includes H2O changes

MAHOUT-1717 is part of #135

MAHOUT-1718 is part of #135

MAHOUT-1719 is part of #135

MAHOUT-1720 is part of #135

MAHOUT-1721 is part of #135

MAHOUT-1722 is part of #135

MAHOUT-1723 is part of #135

MAHOUT-1724 is part of #135

MAHOUT-1725 is part of #135

MAHOUT-1726 is part of #135

MAHOUT-1727 is part of #135

MAHOUT-1728 is part of #135

MAHOUT-1729 is part of #135

MAHOUT-1730 is part of #135

MAHOUT-1731 is part of #135

MAHOUT-1732 is part of #135

@avati -- you can start on this PR branch if you prefer so however, #135 will go in on Wednesday. at that point the history of this issue will be squashed; however, i don't imagine it should create issues for you to merge in the 0.10.x branch later, since squashed commit will be equivalent to the history prefix you'd have.

ok so this will break h20 tests.
for one, it replaces physical operator of AewScalar with AewUnaryFunc (elementwise application of a function).

Ah yes, that will break existing tests. I think we need a workflow strategy for such changes in the future, especially as more engines are added.

yeah.. although this might be tough, it doesn't look terribly likely that we want to introduce a lot. We had a conversation about it with @pferrel who added a bunch of ??? stuff to H20Engine which do not seem terribly relevant to algebraic stuff (to me). We agreed that if indexed format is needed, perhaps this is something to solve ouside of individual backend scope. Either way, something pending as ??? forever is not the effect we want -- and i suggest to apply a higher level of scrutiny on what is going on there.
in my case, motivation of adding (replacing) dedicated physical operator computing elementwise scalar operation with elementwise function operator is pretty clear. it allows to detect and optimize and fuse more general elementwise cases on upper level of optimizations, such as for example 1 + drmP * dlog(drmP), into one physical map. Also it detects some very important and frequent enough  cases like X * X (self-hadamard) and sends it into this unary elementwise function instead of AewB.
But in case of indexed dictionary structure i am not sure i have heard  convincing arguments this needs to be there... at least in the algebraic engine itself... i am perfectly fine with quasi-algebraic support with what @pferrel  is doing. I can quite imagine this will not be an outlier case...

@avati ok i created two issues, MAHOUT-1736 to track allreaduceBlock() and MAHOUT-1737 to track AewUnaryFunc physical operator in H20.
there are a couple of samplers there as well that would be nice to support at least with approximate methods similar to what was applied in spark but if they are not supported oh well.
But the former two are much more essential for portability support.

closing manuall after commit. @asfgit seems to fail to auto-close requests that are not on master. bummer.

Looks great, amazing drop, and very readable even for a Scala novice like me.


[ Schema ] MITRE ATT&CK Representation: Plural “Attacks”

Is the Security Controls and its attacks Object final?
Can this be be renamed to mitre where the array of the techniques lives under?
It was confusing to want to see an attacks object I thought was how I would group telemetry of attacks (telemetry),   and upon closer look, I realized that’s just the array of `mitre semantics.
Ref: https://schema.ocsf.io/1.0.0-rc.3/objects/attack

attacks is an array of attack objects.  Security Controls is a profile that includes attacks.  They are both part of 1.0 and that sense, 'final.'
We are thinking that the Attack object itself is missing sub-techniques, so that is one area that could be improved.
We don't want an attribute named 'mitre' as that is the name of the organization, (and also, for example MITRE has D3FEND as well as ATT&CK).

Thanks Paul, that makes sense for a schema, one final question.  Why is the Tactics and array and coupled with a single technique object?

The way we did it, for simplicity in not have arrays of arrays, is that the attacks attribute holds one Technique/Tactic pair per element of the attacks array.  If a Tactic has more than one Technique reported, then the Tactic would be repeated with different Techniques.  We might have to do something similar if we implement sub-techniques.  Mainly we are judicious with deeply nested arrays if possible.

Thank you, @pagbabian-splunk .


extra data found in  OneToMany collection

still using hbase jpa client, this is my entity definition:
@Entity
@Table(name="Oswaldl")
public class Man {
	@Id
	@GeneratedValue(strategy = GenerationType.TABLE)
	private String id;

	@Column(name = "name")
	private String name;

	@OneToMany(cascade=CascadeType.REMOVE)
	@OrderBy("id ASC")
	private Set<Man> boys = new HashSet<Man>();
}

test code:
        Man m1 = new Man();
        m1.setId("id1");
        m1.setName("attr1");

        // add one boy here
        Man subm2 = new Man();
        subm2.setId("id2");
        subm2.setName("attr2");
        subm2.getBoys().add(subm2);

after save 'm1'  and refetch it, i got two boys in m1 instead of one(the extra one is m1 itself)!

Are you using kundera-hbase or kundera-hbase-v2 ?

com.impetus.kundera.client:kundera-hbase-v2:2.17

We are marking it as a bug. We will add a fix soon.

Thanks for raising the issue. Fix has been added in the trunk. Please build it and let us know.

Find this testcase for more details.

ok, thanks

closed


Enabling gofmt checking on our CI

added script for running gofmt
fix gofmt in entire project

Example fail: https://travis-ci.org/mzylowski/kubernetes-entrypoint/jobs/192772946

Split this to 2 PRs. First is changing CI another one is foxing tests.

@PiotrProkop Why it can't remain as a single PR?

My bad.

LGTM


Rinohtype test failing on Python versions <3.8

Describe the bug
The test for pdf generation using rinohtype is failing due to an update to an unpinned dependency.
To Reproduce
Haven't been able to reproduce locally. See failing CI.

No longer testing Python <3.9


Set the ID for the byoip prefix during import.

Support for splitting the account_id and prefix_id out of the user-supplied ID had been added in f9b3d1e, but when the object is being read, the prefix id supplied to the API includes both the account id and prefix id.
This sets the ID of the schema object to the prefix id in a similar manner as resourceCloudflareBYOIPPrefixCreate
Closes #1890

solid work @wolfmd 👏

Yeah, real monumental change here 😅
Thanks for the review and merge!


Package  on PyPI is out-dated.

Hello!
Please upload new version of package on PyPI.
Because version installed with pip don't supported Python 3, and out-dated (2012-07-06).

This package is not updated anymore.


Area code 226 should be in Canada, not the US

The area code 226 is a Canadian area code, but intl-tel-input thinks it's a US area code. Wikipedia can confirm this is a Canadian area code: http://en.wikipedia.org/wiki/Area_codes_519_and_226
Steps to reproduce:

Navigate to http://jackocnr.com/intl-tel-input.html
Select Canada from the country dropdown.
Type "226" into the text field.

The country dropdown will change to the United States.
In contrast, libphonenumber demos seem to yield the correct results. E.g.:

Navigate to http://libphonenumber.googlecode.com/svn/trunk/javascript/i18n/phonenumbers/demo-compiled.html
Enter the phone number "2265551234".
Enter the default country "US".
Click Submit.

The validation results show "Phone Number region: CA"

Thanks for reporting this. Fixed in v3.8.7.


feat(edit appointment): implement Edit Appointment

Fixes #1807
Changes proposed in this pull request:

Implement Edit Appointment
Rename translation keys to be consistent across Patient and Appointment
Move createAppointment from appointments-slice to appointment-slice
Rename AppointmentsRepository to AppointmentRepository to be consistent with PatientRepository
Add createAppointmentSuccess action to set isLoading to false and be consistent with createPatient
Refactor AppointmentDetailsForm to use single Typeahead for patient instead of conditional typeahead / text field

The tool bar button on the /appointments route is showing the internationalized key rather than the value.

This should also be fixed now.

@MatthewDorner I've accepted 2 PRs and there's been an "avalanche" of conflict. Sorry, dude!

@MatthewDorner I've accepted 2 PRs and there's been an "avalanche" of conflict. Sorry, dude!

Should be fixed now. Not sure what's going on with the tests now, but they're passing for me locally (Ubuntu).


poke: 1.3 → 1.4

Motivation for this change
Update to the latest version: https://savannah.gnu.org/forum/forum.php?forum_id=10076
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@AndersonTorres

Result of nixpkgs-review pr 148385 run on x86_64-linux 1

1 package built:

poke


comb-to-cggi: add secret memref allocation conversion

Fixes https://github.com/google/heir/issues/361
This commit: https://github.com/google/heir/pull/363/commits/987335eb8d1f59434a94fb216ab0200f6ca0757c
Fixes https://github.com/google/heir/issues/365

This is finally ready for review! It took my a very long time to understand how to deal with the memref accesses and resulting layouts. I'm happy to walk through an example.

Test fixed, thanks!

Looks like it still needs a fix.


nat - migrate test from Junit4 to Junit5

Module nat
eg
public class NatServiceTest

Add dependency for junit.jupiter (junit5)
Migrate per https://blogs.oracle.com/javamagazine/post/migrating-from-junit-4-to-junit-5-important-differences-and-benefits
Remove dependency on junit 4

Here's an example https://github.com/hyperledger/besu/pull/5525/files

I'm working in this issue.

@macfarla This issue was resolved by PR #6122

thanks @fspirola


Crash using str_format in MySQL functions

Server seems to crash using the return string of str_format in MySQL functions, as mysql_tquery**_s** and mysql_pquery**_s**.
Used plugins (in order of load)

Crashdetect 4.20
PawnPlus 1.3.1
MySQL R41-4

Crash log
[00:14:01] [debug] Server crashed while executing bare.amx
[00:14:01] [debug] AMX backtrace:
[00:14:01] [debug] #0 native mysql_tquery () in mysql.DLL
[00:14:01] [debug] #1 0000025c in public OnGameModeInit () in bare.amx
[00:14:01] [debug] Native backtrace:
[00:14:01] [debug] #0 6bddefae in ?? () in log-core.dll
[00:14:01] [debug] #1 6bddf10c in ?? () in log-core.dll
[00:14:01] [debug] #2 6bddd8ee in ?? () in log-core.dll
[00:14:01] [debug] #3 68075e76 in ?? () in plugins\mysql.DLL
[00:14:01] [debug] #4 6808d7bd in ?? () in plugins\mysql.DLL
[00:14:01] [debug] #5 004010b6 in ?? () in samp-server.exe
[00:14:01] [debug] #6 5b79f96e in ?? () in plugins\crashdetect.DLL
[00:14:01] [debug] #7 5b7a5a81 in ?? () in plugins\crashdetect.DLL
[00:14:01] [debug] #8 5b7991a5 in ?? () in plugins\crashdetect.DLL
[00:14:01] [debug] #9 5b79f9dc in ?? () in plugins\crashdetect.DLL
[00:14:01] [debug] #10 10014c6e in ?? () in plugins\PawnPlus.DLL
[00:14:01] [debug] #11 0040174f in ?? () in samp-server.exe
[00:14:01] [debug] #12 1001866e in ?? () in plugins\PawnPlus.DLL
[00:14:01] [debug] #13 0046f654 in ?? () in samp-server.exe
[00:14:01] [debug] #14 65646f6d in ?? () in samp-server.exe
[00:14:01] [debug] #15 61622f73 in ?? () in samp-server.exe
[00:14:01] [debug] #16 612e6572 in ?? () in samp-server.exe
[00:14:01] [debug] Registers:
[00:14:01] [debug] EAX: 00000000 EBX: 00000001 ECX: 00000001 EDX: 00000000
[00:14:01] [debug] ESI: 00000000 EDI: 00000004 EBP: 0019f574 ESP: 0019f568
[00:14:01] [debug] EIP: 6bddefae EFLAGS: 00010246
[00:14:01] [debug] Stack:
[00:14:01] [debug] ESP+00000000: 00000004 0019f82c fdb96d90 0019f598
[00:14:01] [debug] ESP+00000020: 00000001 0019f5ac 00000000 00000000
[00:14:01] [debug] ESP+00000040: fdb96d90 6bddd795 6bde2674 00000022
[00:14:01] [debug] ESP+00000060: 007a2010 00775898 00000024 007a2034
[00:14:01] [debug] ESP+00000080: 02bd8990 007a1f38 007a2010 1005ff6f
[00:14:01] [debug] ESP+000000a0: 00770acc 0295ab00 02bdc9e4 fdb96d90
[00:14:01] [debug] ESP+000000c0: 6bde24f4 0019f638 00000010 000001f4
[00:14:01] [debug] ESP+000000e0: 007758bc 02bd8990 02bd8990 0019f838
[00:14:01] [debug] ESP+00000100: 0019f698 9fe17f7e 00000150 77440ec1
[00:14:01] [debug] ESP+00000120: 0019f7f4 02bdc9f0 0076f644 02bd8990
[00:14:01] [debug] ESP+00000140: 1007bc16 007a205c 02bd89e4 00000020
[00:14:01] [debug] ESP+00000160: 773bb135 00000026 00000020 00783af8
[00:14:01] [debug] ESP+00000180: 007a2034 007a1f38 02bd8994 00000020
[00:14:01] [debug] ESP+000001a0: 773bb135 00783f00 00000000 00000000
[00:14:01] [debug] ESP+000001c0: 00000000 00000002 00020000 000201c8
[00:14:01] [debug] ESP+000001e0: 00020bd0 0074b5f8 0019f770 773bae8e
[00:14:01] [debug] ESP+00000200: 00000014 1021cd28 0019f78c 751ea9f6
[00:14:01] [debug] ESP+00000220: 1021cd28 0072eaa8 1021cd28 00783878
[00:14:01] [debug] ESP+00000240: 101b2380 00000002 0019f808 10039012
[00:14:01] [debug] ESP+00000260: 1021cd28 0019f814 101b06cf 00783878
[00:14:01] [debug] ESP+00000280: 0295ab00 02bdc9d8 00000000 00000000
[00:14:01] [debug] ESP+000002a0: 6bdd37dd 0019f848 0019f84c 0074dae8
[00:14:01] [debug] ESP+000002c0: 1b383933 00770a00 00000000 0074b854
[00:14:01] [debug] ESP+000002e0: 0019f8b8 68075e76 00770acc 0295ab00
[00:14:01] [debug] ESP+00000300: 00000008 0295ab00 b61045ad 00000007
[00:14:01] [debug] ESP+00000320: 0073d000 1b3839d3 00000000 0295ab00
[00:14:01] [debug] ESP+00000340: 00770aa0 0019f8dc 6809452f ffffffff
[00:14:01] [debug] ESP+00000360: 02bdc9dc 6809a898 1b38398f 0295ab00
[00:14:01] [debug] ESP+00000380: 0019fa24 004010b6 0295ab00 02bdc9dc
[00:14:01] [debug] ESP+000003a0: 0019fa94 02bdc9dc 7597b1b3 000041b8
[00:14:01] [debug] ESP+000003c0: 00000000 000041d8 0000025c 00000008
[00:14:01] [debug] ESP+000003e0: 00000020 00000000 00700000 00000020

Used code
#include <a_samp>
#include <a_mysql>
#include <PawnPlus>

native mysql_tquery_s(MySQL:handle, AmxString:query, const callback[] = "", const format[] = "", {MySQL, Float,_}:...) = mysql_tquery;

new MySQL:Handle;

main()
{
	print("Test");
}

public OnGameModeInit()
{
	new MySQLOpt:options = mysql_init_options();
	mysql_set_option(options, AUTO_RECONNECT, true);
	mysql_set_option(options, MULTI_STATEMENTS, true);

	Handle = mysql_connect("localhost", "root", "", "testing", options);
	mysql_log(ALL);

	if(mysql_errno(Handle) == 0)
	{
	    new String:name = str_new("Heix"), String:ip = str_new("<IP_ADDRESS>");
	    mysql_tquery_s(Handle, str_format("SELECT * FROM `TEST` WHERE `NAME` = '%E' OR `IP` = '%E' LIMTI 1;", name, ip), "OnQueryFinish");
	}

	return 1;
}

forward OnQueryFinish();
public OnQueryFinish()
{
	new rows;
	if(cache_get_row_count(rows))
	{
	    if(rows) print("1");
	    else print("0");
	}

	return 1;
}

str_format does not have anything to do with it; any PP string crashes when used in this code. The crash happens inside log-core, which is in turn invoked by the construction of CScopedDebugInfo in mysql_tquery. PawnPlus isn't invoked anywhere in the process.
This is an issue in samp-log-core here. It for some reason uses its own copy of the AMX functions, bypassing PP's hooking mechanism, so dynamic strings could never work with it, unless it properly sticks to the AMX API. However, that could not cause the crash on its own, because amx_GetCppString is actually wrong and assumes that amx_GetAddr never fails, which it normally does when used on a PP string.
This makes samp-log-core violate the first two points of the GCA for AMX API consumers and thus PawnPlus is not expected to work with it. I suggest you post an issue there so that it can be fixed.
Fortunately for you, the MySQL plugin itself uses the AMX API properly, so this can still be fixed. Simply don't use mysql_log(ALL);, since the debug logging was what led to the crash. Don't use it all or use mysql_log(ALL & ~DEBUG);. If you want to keep this logging level, you will have to use pawn_call_native since that has native support for PP strings and can translate them to normal strings without intermediate AmxString.
By the way, the query is incorrect. There is LIMTI instead of LIMIT.


Add style URL to the hash

Closes #48
If you add a style via url, that style will be added to the hash. Then on refresh, the style will be fetched again. If you add a new style via drag and drop, the url will be removed from the hash.

@ebrelsford good catch! Just updated.

Ok, now I think this should be good to go.

I'm not sure if that was introduced here, if it wasn't let's open a follow-up issue for it.

Yea, that one's a bit deeper of an issue, but an intentional choice. We decided to always flip back to the fill page to replenish the background layer: https://github.com/stamen/chartographer/blob/fe8a56a7e2b994ed0da7e39416dc13433fa9de0d/src/App.svelte#L125-L126
This carries over to styles in the URL since they're treated as a new style and need a background layer.


On tracking initialized

From these docs: https://docs.viromedia.com/docs/viroarscene

Function called when the AR system has properly initialized. Until this function is called, the camera position is

The sentence is incomplete, is this [0,0,0]?

Hi Rafael,
Thanks for reaching out pointing this incomplete documentation! I have updated the documentation as well, but to answer your question, the camera position is unset / not in sync with the coordinated system until onTrackingInitialized is called. It is set at [0,0,0] at the moment onTrackingInitialized is called.
Thanks!

Awesome!
One more question, I'm able to add objects to the scene before its initialized and I'm placing them in front of the user, how is this possible if it hasn't been initialized yet?

Hi Rafael,
Great question. This is explained on this page here https://docs.viromedia.com/docs/augmented-reality-ar. Specifically this section.

When the user first enters the AR experience, the camera will remain at [0,0,0] while underlying AR system gets its bearing. After a few moments, the camera tracking will complete, the developer should receive a callback on onTrackingInitialized in ViroARScene, and the camera should now track the user's movements around their world.

So to answer your question -  nothing changes with the camera per se, it’s the nodes / objects in the scene that would move as the camera moves (before init) vs (after init) objects moving/tracking with points in the world.
Hope that explains the setup process.
Thanks!

@manbod
Yup, answers my question perfectly, thank you so much!


Genome version label is incorrect

It is hg18 and hg18, not hg38 and hg 19.  Which version naming nomenclature are you wanting to use?  hg or GRCh?  The latter uses numbers like 37 and 38, so it is confusing to have as an option on this tool hg38 - that is meaningless.  Thanks for checking it out!

UCSC uses "hg19" and "hg38" (for example, here: https://genome.ucsc.edu/cgi-bin/hgGateway).
The main reason I use "hg38" instead of "GRCh38" is because it's shorter : )

Remains confusing and not consistent with clinical practice guidelines – UCSC is not the correct authority on this.
  Bob

Bob Wildin, MD, ABMG Clinical Geneticist
From: bw2<EMAIL_ADDRESS>Sent: Tuesday, November 10, 2020 6:54 PM
To: broadinstitute/cma-search<EMAIL_ADDRESS>Cc: bwildin-clingen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [broadinstitute/cma-search] Genome version label is incorrect (#1)
UCSC uses "hg19" and "hg38" (for example, here: https://genome.ucsc.edu/cgi-bin/hgGateway).
The main reason I use "hg38" instead of "GRCh38" is because it's shorter : )
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/broadinstitute/cma-search/issues/1#issuecomment-725038308 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARAYI7XHE73BUMHU7Z6M5QTSPHHCXANCNFSM4RPF6JIQ .

ok, I've updated the labels.

Thank you! 😊
  Bob

Bob Wildin, MD, ABMG Clinical Geneticist
From: bw2<EMAIL_ADDRESS>Sent: Thursday, November 12, 2020 12:30 AM
To: broadinstitute/cma-search<EMAIL_ADDRESS>Cc: bwildin-clingen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [broadinstitute/cma-search] Genome version label is incorrect (#1)
ok, I've updated the labels.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/broadinstitute/cma-search/issues/1#issuecomment-725847475 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ARAYI7TPNWZTVELZJJAIOOLSPNXGDANCNFSM4RPF6JIQ .


[Feature request]: Add Kolors Support

It is SDXL architecture with a different noise scheduler(Kolors Scheduler?) and text encoder (ChatGLM3-base).
Would be great to be able to fine tune it as it is better than any other openly available base model now.
https://github.com/Kwai-Kolors/Kolors

I agree, it's a very good base model

+1

+1


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2256 ms

Resolved: Bofrost.at performance has improved in 720bd07.


After router collateralization, still not enough funds in channel, sender is left hanging

The router reconciled a deposit which did not increase the balance. The router continued to try to forward the transfer, which did not relay back to the sender that the transfer was failing.

Opened in main repo.


Where is index.html after publish?

When I run the publish command the output does not include the index.html file required to deploy the project in client-only mode to a static-file web host such as github pages.
Command Used:
dotnet publish -c Release
Expected Output Directory:
bin\release\net*******\publish\wwwroot\

Dual mode uses the https://github.com/Suchiman/BlazorDualMode/blob/master/BlazorDualMode.Server/Pages/_Host.cshtml file because index.html does not work for server mode. You could have both an index.html and an _Host.cshtml which you'd need to keep in sync.

Thank you for the explanation and suggestion. I may attempt something like this and will update if it goes well.

Adding the index.html file did allow me to publish and serve the client project from a static http server, however doing this broke the FetchData tab because /api/SampleData/WeatherForecasts/ is no longer being served by the Server Host.

Comparing with my standalone WebAssembly application it seems the difference is that the standalone loads the data from a json file rather than from a .net core hosted controller, and so this endpoint not working should be expected.

I wanted to follow up in case anyone else is interested in doing the same.
I've upgraded this project to support deploying just the client in this commit:
https://github.com/vanderstack/BlazorDualMode/commit/8f974033722f9995764793b164e5877d760e477c
I've gone further and integrated BlazorWebView to add a WPF host to run natively on windows (mac and others are also available using other projects in the BlazorWebView library), and then included the latest Test Suite prototypes from Steve Sandersons blog.
A stable commit with these features is available here:
https://github.com/vanderstack/BlazorDualMode/commit/d58dbe5a499aaa9cd3a519f4af07e9a8e18036d8


expose window handle para

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity


Revert "Update Makefile"

Reverts suleymanakbas91/kyma#133

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-integration

pre-master-kyma-gke-upgrade
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-upgrade

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-integration

pre-master-kyma-gke-upgrade
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-upgrade

pre-master-kyma-gke-central-connector
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-central-connector

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@suleymanakbas91: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pre-master-kyma-gke-integration
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-integration

pre-master-kyma-gke-upgrade
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-upgrade

pre-master-kyma-gke-central-connector
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-gke-central-connector

pre-master-kyma-integration
81fadda5ea510552b5fce7f7ecf390f52a5541b2
link
/test pre-master-kyma-integration

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


PDB file repostirory over http should not use ftp.wwpdb.org DNS name

Instead of https://ftp.wwpdb.org it should be https://files.wwpdb.org. See announcement
This file contains some references to the bad DNS name:
https://github.com/rcsb/py-rcsb_utils_repository/blob/b1b16b5d206b3e74e7d8bdefb0138d0146ea440e/rcsb/utils/repository/CurrentHoldingsProvider.py#L2
But there might be others in the repo. I didn't check.

Thanks for pointing out, @josemduarte!


feat: Added validation for --client-listen-urls and --peer-listen-urls

This PR resolved this issue: https://github.com/xline-kv/Xline/issues/691
Please briefly answer these questions:

what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
During the startup process of Xline, certain essential command-line parameters such as --member and --storage-engine are checked. and there is a lack of validation for --client-listen-urls and --peer-listen-urls
Here's how it looks -

what changes does this pull request make?

are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

@Phoenix500526 @iGxnon  Please review this PR. I closed the previous one and remove unnecessary commits and also just wanted to push from proper PR branch name from my local. Please tell me if there is anytime more to be done for this PR.

@Phoenix500526 is their any order in which a particular arguments needed to be shown, like right now --client-listen-urls and --peer-listen-urls are first and then rest of the flags, i will change if required.
While playing with Xline, i found a small ip-address issue with the quick_start.sh script for prometheus and quick-start DOC. Should i raise a issue for this or are we going to change the quick_start soon in this issue https://github.com/xline-kv/Xline/issues/692.

@Phoenix500526 is their any order in which a particular arguments needed to be shown, like right now --client-listen-urls and --peer-listen-urls are first and then rest of the flags, i will change if required. While playing with Xline, i found a small ip-address issue with the quick_start.sh script for prometheus and quick-start DOC. Should i raise a issue for this or are we going to change the quick_start soon in this issue #692.

Is the issue you mentioned above described in issue #692? If not, I think it's better to initiate a new issue to describe it.

Thanks @Phoenix500526 .Got it! I will raise a issue for this although its just a minor fix.


I believe that the construct name is missing in each package under GFM.

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions
2.0.0
Link to runnable example
No response
Steps to reproduce
source code
Expected behavior
When I want to turn off certain constructs in the parser of remark, such as strong, I can pass attention to disable.null. But when I want to turn off constructs in GFM, such as strikethrough, I have no way unless I stop using remark-gfm altogether and instead selectively incorporate the functions I need.
Actual behavior
Upon examining the code, I discovered that it is this particular line of code that activates the passed disable.null.
    if (construct.name && context.parser.constructs.disable.null.includes(construct.name)) {
            return nok(code)
 }

But in https://github.com/micromark/micromark-extension-gfm-strikethrough/blob/ffa5ca5d0014667ed6c90625037b942cd7761c3c/dev/lib/syntax.js#L38, there isn't a name attribute.
So, would it be more appropriate to include the name attribute?
Relevant
https://github.com/zestedesavoir/zmarkdown/issues/416#issuecomment-982812961
https://github.com/remarkjs/remark-gfm/issues/34#issue-1096426681
https://github.com/micromark/micromark-extension-gfm/issues/6#issue-1114755539
Runtime
Node v16
Package manager
pnpm
OS
macOS
Build and bundle tools
Vite

Hey!
It isn’t exactly an issue: name is optional; it isn’t here because these are extensions, so you should be able to turn them off by not adding them.
But I do welcome PRs, I think it’s nice/good to have: I think I have it on my todolist somewhere too
Closing as this has to be solved in each of the extensions that implement constructs, not in this collection package!

Hey!
It isn’t exactly an issue: name is optional; it isn’t here because these are extensions, so you should be able to turn them off by not adding them.
But I do welcome PRs, I think it’s nice/good to have: I think I have it on my todolist somewhere too
Closing as this has to be solved in each of the extensions that implement constructs, not in this collection package!

Thank you for your response. This feature is quite important to me, and I will submit PRs.


Improve output.

This commit replaces almost all uses of print with the logging module, and more cleanly separates artman output from the output spit out by the processes that artman runs.
Additionally, it adds a -v and -q flag to alter how much output you get. Adding -v shows all shell commands.

@ethanbao This should make it a bit easier to work on artman; it is the first step toward my GitHub changes. :-) I wanted it to be easier to see what artman is doing.

This change looks great. One question, artman can also listen to a task queue, fetch task(s) and run tasks in parallel. In addition to the default logger, would it be possible to attach additional stackdriver logger handler for each execution thread, so that each thread will upload the log to cloud with unique log id?

One question, artman can also listen to a task queue, fetch task(s) and run tasks in parallel.

I have some ideas to how to do this. Will discuss with you later.

@ethanbao has approved in-person. Merging.

I derped and did not see the CI failure. My own fault. Will fix first thing tomorrow morning.

Oh, I wasn't aware that github will allow merging with CI failure.

It is configurable. We should turn it off. I intend to do that soon on every repo we own.

#154 is the fix. Dumb on me. Feel free to just merge as soon as CI passes.


V2 - Return the values from iTree to Firebase

Use Case:
IF values are generated successfully from iTree - save them to Firebase when the user selects "SAVE".

Values we NEED to save to firebase are (in the iTree API XML..you can use the same naming convention in firebase - though I would make the FIRST letter lowercase):

(* we convert to gallons, iTree API saves it as cubic meter - would be better to save it as gallons)

2.

Return the values from iTree to Firebase #94 done

Return the values from iTree to Firebase #94 done

@sorratanak close - but we actually need to save more from the API then what the 'Tree Benefits Calculator' displays. There also seems to be an error happening with 'RunoffAvoided'... We need ALL The values from the xml tags listed above individually saved in Firebase.
For example, we don't need 'AirPollutionRemoved'  'AirPollutionRemovedValue' - but we do need all the data that makes that field (in the front end):

Can you help with this?

@sorratanak close - but we actually need to save more from the API then what the 'Tree Benefits Calculator' displays. There also seems to be an error happening with 'RunoffAvoided'... We need ALL The values from the xml tags listed above individually saved in Firebase.
For example, we don't need 'AirPollutionRemoved'  'AirPollutionRemovedValue' - but we do need all the data that makes that field (in the front end):

Can you help with this?

Yes, No problem. I did. Please check the video and GitHub codes.
https://reccloud.com/u/ig8jzxw

Yes, No problem. I did. Please check the video and GitHub codes.
https://reccloud.com/u/ig8jzxw

Hello, @TreeMama .
I am Lenh.
Can you share your contact information please?
my email address is<EMAIL_ADDRESS>Hope your reply soon. thank you.

Hello, @TreeMama .
I am Lenh.
Can you share your contact information please?
my email address is<EMAIL_ADDRESS>Hope your reply soon. thank you.


added readmes to IoTHubBuddy and IoTHubBuddyClient

Hi @DivyaMahadevan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;


Port ZapToken from ZapJS to Python

Summary
Port ZapToken from ZapJS to Python
Reference
https://github.com/zapproject/zap-monorepo/blob/master/packages/ZapToken/src/index.ts
Definition of Done
All the functions are able to called and return the correct values.

I'll be picking up on this issue.


Should we keep mutiny-reactive-streams-operators in this repo?

See $title 😉
We have 3 options:

keep it in this repository, but pin the mutiny-reactive-streams-operators project Mutiny version to 1.x stable
move to another smallrye repo
remove from the main branch and consider that the code will be maintained in the 1.x branch

Or... 4 migrate to Mutiny 2.0 (but shift the baseline)

We will have to migrate the implementation to mutiny 2.0 and so use lots of adaptation.
This is mainly because this module is required in reactive messaging, and so we need a version using the latest mutiny version. Without it, we will end up with two versions of mutiny on the classpath (which leads to package name changes and so on).


Issues with CUDA , how to run on CPU only?

Hi guys, I have an issue using CUDA on my machine, how can I tell the model to use CPU only?
thank you very much

Hi @vitotitto ,
for inference you can use the lib that I wrote here. It automaticlly checks if you have a GPU and uses the cpu otherwhise. If you don't want to use your GPU you have two options:

run  ``export CUDA_VISIBLE_DEVICES="0"``` to disable the gpu (I did not test this)

or

install pytorchs cpu only version

Please feel free to reopen if you have any issues


Added full state to product card

Added full state to product card

Changes 👷♀

- when debt ceiling is reached, product card will show it
How to test 🧪

- tweak isFull props in product card (locally)
Definition of done ✔

[ ] Acceptance criteria for each issue met
[ ] Unit tests written where needed and passing
[ ] End-to-end tests for happy path
[ ] Documentation updated
[ ] Project builds without errors
[ ] Non-functional requirements met
[ ] Code reviewed and functionality tested by the reviewer (locally, Heroku, etc.)
[ ] Feature verified and accepted by Product Owner
[ ] Project deployed to production environment

Something is wrong here. They are not full.


Color information lost after boolean operations

Great work on the library. If I understand it correctly in V1 API after applying boolean operations colour data was transferred further down the chain and existed as a property on each polygon. In V2 I see that applying booleans on coloured meshes returns objects that 'forget' about original colour. Is there some way to force that in V2? Thank you.

Correct. Colors are not transferred, and apply only to the geometry as a whole.
When using V2, be sure to set color as the last step, after applying other operations.

Thanks, I will keep that in mind.

@bitbybit-dev thanks again.
just one more tip... a design can return one or more geometries. many of the export formats (like DXF) will convert each geometry into appropriate objects. so, your design can choose to combine everything into ONE BIG GEOMETRY, or return a bunch of little geometries. it just depends on what you want to do with it.

I sort of imagined that coloured 3D printers would use this info from V1 when printing single mesh, but saving multiple objects with different colours before going to 3D print probably is also ok. My experience in that is a bit limited. In any case, users of my platform will need to make this choice themselves, I am exposing csg as it is and so far it works really well.

@bitbybit-dev nice feedback. keep in touch and let us know if there are any issues. i would expect a few in V2. ;)
fyi, i discussed color prints with Mr. Prusa at Makers Faire Tokyo. (We had a great discussion). For multi-color prints, each part must be seperate, and have a unique color. The Prusa slicer uses that information to activate the nozzle (color), and create the multi-color print. P.S. His file format of choice is 3MF, which is coming in JSCAD.

That sounds great! I will follow the development of 3MF integration then. I get that V2 can be a bit unstable, but I like the API choices you've made and unit tests make a big difference :) I'll be testing csg quite extensively through https://bitbybit.dev UI, if something comes up I'll let you know. Prusa slicer looks great, I was only familiar with Cura...

@bitbybit-dev oh... by the way, this library is now part of OpenJSCAD.org repository, and known as the 'modeling' package. you should be working with that repository now, as this one (CSG.js) is no longer maintained.

Ah :) That is some very valuable information, had an idea that csg.js was part of OpenJSCAD.org as it was not marked deprecated.  I will clone it through OpenJSCAD.org then and use modeling package. Thanks!

Excellent. Here's a starting point...
https://openjscad.org/dokuwiki/doku.php?id=early_v2
FYI, you can also find the @jscad/modeling library on NPM. It was published today! :)

Haha, that's some top-notch support. NPM package will be very helpful as I only need that part at this moment, thanks! Its probably not the best place to throw these questions, but if it is possible to publish io package to npm that would be awesome as well ;)

Should be there... here's the list of published packages.
Successfully published:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>lerna success published 22 packages

OK, amazing @z3dev, thanks a lot for all the help here! I have everything I need to continue ;) And it all started with colors... Now I just need to code it out.


navCtrl.getActive().component.name returns "t" in RC0

Getting the component name or using various methods give the letter t in return.
This has been seen in the Tabs template.
Note that this only happens on the device. It works fine in the browser.
    console.log(this.navCtrl.getActive().component.name);
    console.log(nav.getActive().component.name);
    console.log(this.tabRef.viewCtrl);

What behavior are you expecting?
Name of the component or view is expected
Steps to reproduce:

Create a tabs project.
Add a button to log the current component name
Build for android and run on device or emulator

Which Ionic Version? 1.x or 2.x
Ionic 2 RC0

Hello, thanks for using Ionic! This is actually expected due to our AoT compiling that happens when doing a prod build. Ionic does a prod build when doing ionic run android/ios, ionic emulate android/ios or ionic build android/ios. All of the methods will still work the same as the name, which is t in this case, still refers to the same component.

Hi @jgw96 ,
What I should do to get the name property, instead t, or e, or r property?

What I should do to get the name property, instead of t, or e, or r property?

+1

We can get component-name with "this.nav.getActive().id"  in AoT.
not use "name".

its working "this.nav.getActive().id" in AoT.
thanks

This doesn't seem to work as of currently (ionic CLI 3.18.0, ionic-angular 3.7.1). "this.nav.getActive().id" returns a shortened string like "tn-4", not the original component name. I'm using this method in production to send telemetry about view navigation, so the original component name is what I'm looking for. Is there any other way to obtain it?

+1

+1

+1
Could be very useful to have component name also in "ionic build --prod"!

+1 plz....

If you need name just to compare with specific page / component, you can copare active component with that component directly:
let activeView = this.navCtrl.getActive(); if (activeView.component == ConversationPage) { }
as ruben0909 mentioned here. It helped me a lot.


Exception caught during observer callback: TypeError: Cannot read property 'replace' of null

I'm not 100% sure this is an issue with marked so I'll start here
I get the following exception when using Marked in a web component. The result displays so I don't think it's a Marked error. May this be related to why the page is not updating when I make css changes after the first load?
This is how I'm using Juicy Markdown
<juicy-markdown value="{{doc}}"   ghcss="false"></juicy-markdown>

This is the exception I get in Chrome 39.
 Exception caught during observer callback: TypeError: Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked.
    at Lexer.lex (http://localhost:8888/athena-document/bower_components/marked/lib/marked.js:137:5)
    at Function.Lexer.lex (http://localhost:8888/athena-document/bower_components/marked/lib/marked.js:128:16)
    at marked (http://localhost:8888/athena-document/bower_components/marked/lib/marked.js:1201:31)
    at juicy-markdown.Polymer.parseText (http://localhost:8888/athena-document/bower_components/juicy-markdown/juicy-markdown.html:72:53)
    at juicy-markdown.Polymer.valueChanged (http://localhost:8888/athena-document/bower_components/juicy-markdown/juicy-markdown.html:68:10)
    at juicy-markdown.properties.invokeMethod (http://localhost:8888/athena-document/bower_components/polymer/polymer.js:9355:12)
    at juicy-markdown.properties.notifyPropertyChanges (http://localhost:8888/athena-document/bower_components/polymer/polymer.js:9344:20)
    at Object.Observer.report_ (http://localhost:8888/athena-document/bower_components/polymer/polymer.js:4890:24)
    at Object.createObject.check_ (http://localhost:8888/athena-document/bower_components/polymer/polymer.js:5296:12)
    at callback (http://localhost:8888/athena-document/bower_components/polymer/polymer.js:4788:20)polymer.js:4893 Observer.report_polymer.js:5296 createObject.check_polymer.js:4788 callback

Got same error from marked. But I'm not using juicy.

Got it, because of async loading. marked was getting null passed in.


Invalid handling of uppercased elements in query selector

Thanks for the great library! I have ran into the following issue.
(I'm using Deno v1.11.5, but I think that shouldn't matter.)
This snippet produces the following result using your library:
import {
  DOMParser,
} from "https://cdn.skypack.dev/linkedom@v0.11.0/worker.js?dts";

const documentString = `
<Something>
  <Element>Text</Element>
  <Element>Text</Element>
</Something>`

const d = new DOMParser().parseFromString(documentString, "text/xml");

const nodeListUpper = d.querySelectorAll("Element");
const nodeListLower = d.querySelectorAll("element");

console.log(nodeListUpper.length, nodeListLower.length);

Output: 0 2. (I.e. it only finds the elements with the lowercased query selector)
Running the same snippet (without the import statement) on Chrome results in Output: 2 0, it only finds the uppercased element.
I believe your library should produce the same result, as the document string actually uses uppercase tags.
Thanks again!

If you care about case sensitive names you should parse as text/xml

oh, I've missed that! definitively a bug then 👍


[docs] Demo AnchorPlayground crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

Visit https://mui.com/components/popover/#anchor-playground
??
demo AnchorPlayground crashes

Your Environment

Tech
Version

MUI
v5.2.5

netlify deploy
https://61c098e520eae9000736c0df--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.110 Safari/537.36

Duplicate of https://github.com/mui-org/material-ui/issues/30304


The wave doesn't fit the floating btn

You see, wave doesn't fit all the button which make a really non-proper render for those as exigeants as me

Are you zoomed in on the browser?

In this case I wasn't but it appears that zoom can fix the problem sometimes

I close as it's a bit annoying for maniac but I think Materialize team has higher priority as they have more than one hundred issue and 270 PR (and it's growing)


C# ConfigurationManager not working

Godot version: 3.0.6 mono, 3.0.5 mono

OS/device including version: windows 10

Issue description:
I tried to use System.Configuationmanager to get a value from an app.config on a new project. I should recieve a label that should print a value in the appconfig however it errors out with
Unhandled Exception:
System.Configuration.ConfigurationErrorsException: Error Initializing the configuration system. ---> System.MissingMethodException: string[] string.Split(char,System.StringSplitOptions)
at System.Configuration.SectionGroupInfo.ReadRootData (System.Xml.XmlReader reader, System.Configuration.Configuration config, System.Boolean overrideAllowed) [0x00007] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.Configuration.ReadConfigFile (System.Xml.XmlReader reader, System.String fileName) [0x000ce] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.Configuration.Load () [0x00043] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.Configuration.Init (System.Configuration.Internal.IConfigSystem system, System.String configPath, System.Configuration.Configuration parent) [0x0005d] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.Configuration..ctor (System.Configuration.InternalConfigurationSystem system, System.String locationSubPath) [0x00056] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.Configuration..ctor (System.Configuration.InternalConfigurationSystem system, System.String locationSubPath) [0x00038] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.InternalConfigurationFactory.Create (System.Type typeConfigHost, System.Object[] hostInitConfigurationParams) [0x0000d] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.ConfigurationManager.OpenExeConfigurationInternal (System.Configuration.ConfigurationUserLevel userLevel, System.Reflection.Assembly calling_assembly, System.String exePath) [0x000ef] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.ClientConfigurationSystem.get_Configuration () [0x0000e] in <2177b67bbb3541218d1aeb3d05b52181>:0
--- End of inner exception stack trace ---
at System.Configuration.ClientConfigurationSystem.get_Configuration () [0x0002a] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.ClientConfigurationSystem.System.Configuration.Internal.IInternalConfigSystem.GetSection (System.String configKey) [0x00000] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.ConfigurationManager.GetSection (System.String sectionName) [0x00005] in <2177b67bbb3541218d1aeb3d05b52181>:0
at System.Configuration.ConfigurationManager.get_AppSettings () [0x00000] in <2177b67bbb3541218d1aeb3d05b52181>:0
at Newdir.Runner.Dosomething () [0x00001] in C:\Users\squid\source\repos\NEWDIR\Runner.cs:14
at Label._Ready () [0x00007] in C:\Users\squid\source\repos\NEWDIR\Label.cs:16

Steps to reproduce:
make a call in a mono project that reads conf
Run the attached project through godot or:

make a project
add a label node
add a C# script to get a value through app config via System.Configurationmanager

This affects other projects that require Configurationmanager such as Nakama(which claims to be compatible on the website but fails on instantiating a client object)
**Minimal reproduction project:**attached
NEWDIR.zip

What version of mono is installed?

The one I have installed is 5.14, I can test it on 5.12 (not sure if that will do it)

strange, godot doesn't seem to run when I use mono 5.12(though it said it was compatible). It runs when I use 5.14.

That was it, 5.12 was needed, configurationmanager breaks on 5.14.


fix(build): call process.exit(1) when esbuild fails

This ensures CI will not succeed when a module has a syntax error.
Closes #708
Reproduce

Clone https://github.com/aleclarson/repro/tree/vite-708
Run yarn and yarn build
See how the build succeeds, despite a failed transform

We also encouter transform error with css. We should deal with it as same as esbuild transform.
Code line is here https://github.com/vitejs/vite/blob/master/src/node/build/buildPluginCss.ts#L83

@underfin Added process.exit(1) for failed CSS transform, failed CSS minify, and failed PostCSS config load. 👍
LGTM (rebased)


How would I integrate socket.io

It seems to me that there is no elegant way to integrate this with sockets? Or am I wrong? I've been trawling different socket.io implementations with vue and typescript and they all seem cluncky, and I've found none using vuex-class with socket.io.

This is not scope of this lib... vuex-class is just an adapter between Vuex store and class style Vue components.


shadps4: 0.3.0-unstable-2024-10-13 -> 0.3.0-unstable-2024-10-14

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for shadps4 is: Early in development PS4 emulator
meta.homepage for shadps4 is: https://github.com/shadps4-emu/shadPS4
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.0-unstable-2024-10-14 in filename of file in /nix/store/vbl41ywyz52nvchqcv2xaqz51lc5j2cl-shadps4-0.3.0-unstable-2024-10-14

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

shadps4

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vbl41ywyz52nvchqcv2xaqz51lc5j2cl-shadps4-0.3.0-unstable-2024-10-14 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/68ef45a759525d61684d7fb61e3912dbcb535469#shadps4

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vbl41ywyz52nvchqcv2xaqz51lc5j2cl-shadps4-0.3.0-unstable-2024-10-14
ls -la /nix/store/vbl41ywyz52nvchqcv2xaqz51lc5j2cl-shadps4-0.3.0-unstable-2024-10-14/bin

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

x86_64-linux

:white_check_mark: 1 package built:

shadps4

Maintainer pings
cc @ryand56 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge