linalg lsqr libraries not found on OSX for v0.16

I get this image not found error when trying to import lsqr from linalg:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-a4c2463c7356> in <module>()
----> 1 from scipy.sparse.linalg import lsqr as sparse_lsqr

/Users/klay6683/miniconda3/lib/python3.4/site-packages/scipy/sparse/linalg/__init__.py in <module>()
    107 from __future__ import division, print_function, absolute_import
    108
--> 109 from .isolve import *
    110 from .dsolve import *
    111 from .interface import *

/Users/klay6683/miniconda3/lib/python3.4/site-packages/scipy/sparse/linalg/isolve/__init__.py in <module>()
      4
      5 #from info import __doc__
----> 6 from .iterative import *
      7 from .minres import minres
      8 from .lgmres import lgmres

/Users/klay6683/miniconda3/lib/python3.4/site-packages/scipy/sparse/linalg/isolve/iterative.py in <module>()
      5 __all__ = ['bicg','bicgstab','cg','cgs','gmres','qmr']
      6
----> 7 from . import _iterative
      8
      9 from scipy.sparse.linalg.interface import LinearOperator

ImportError: dlopen(/Users/klay6683/miniconda3/lib/python3.4/site-packages/scipy/sparse/linalg/isolve/_iterative.so, 2): Library not loaded: /usr/local/gfortran/lib/libquadmath.0.dylib
  Referenced from: /Users/klay6683/miniconda3/lib/python3.4/site-packages/scipy/sparse/linalg/isolve/_iterative.so
  Reason: image not found

When downgrading to 0.15.1 the error disappears.
System: OSX 10.10.4, iMac 5k
Package Manager: miniconda.
Note: I found this trying to use DBSCAN from scikit-learn.cluster, so that is broken due to this, too.

Can you raise this with the miniconda team?  We don't maintain the conda packages for scipy.

Sure thing, just didn't know if that's a miniconda thing or scipy itself. There are by now already (Mini)conda GH issues about scipy probs with 0.16. You can close this if appropriate.

I think you've found a conda packaging problem here, where you need a gfortran dynamic library and it is not installed.  I would be very surprised if the OSX wheel do ship and maintain has the same problem, but please do test and let us know:
pip install numpy scipy
python -c 'from scipy.sparse.linalg import lsqr as sparse_lsqr'

I believe you can install pip from miniconda, and that you have to make sure that your selected conda environment does not already have numpy or scipy.  I'm afraid I am a Python.org Python / virtualenv person myself.

Here's a conda issue that looks the same: https://github.com/conda/conda/issues/1467

Fixed by https://github.com/ContinuumIO/anaconda-issues/issues/392


Metrics cleanup

Change some of the references to statsd metrics in doc comments to prometheus metrics.
Remove the deprecated prometheus metrics/labels from the previous release.

While I'm here, also add go 1.20 to CI.

Can you update the PR description with a list of all of the changed / removed metrics so it's easy to copy/paste into release notes?

Removed metrics (all the replacements were already added from at least v0.9.12)

httpbp_server_panic_recover_total: use httpbp_server_recovered_panics_total instead
kafkabp_consumer_rebalance_total: use kafkabp_consumer_rebalances_total and kafkabp_consumer_rebalance_failures_total instead
thriftbp_server_panic_recover_total: use thriftbp_server_recovered_panics_total instead
thriftbp_client_pool_exhausted_total: use thriftbp_client_pool_exhaustions_total instead
thriftbp_client_pool_release_error_total: use thriftbp_client_pool_release_errors_total instead
thriftbp_client_pool_active_connections: use thrift_client_pool_active_connections instead
thriftbp_client_pool_allocated_clients: use thrift_client_pool_idle_connections instead

Removed labels:

thrift_client_name from thrift_client_pool_*: use thrift_pool instead.


Why do I only got a 'OK'?

Here is my code:

But i only got a 'OK':

But The service is successfully save my file~

I have few recomendations to your code:

Generally, try to use form.parse() unless there is specific need to handle other events.
Moving "ctx.res.end" into "form.parse()" would definitely do the job (I once had similar code).
Though I'm not sure why it didn't work in "form.on('end')".
Use form.on('fileBegin') to rename the file. You can change the file path there, without using other libs, like this: file.path = path.join(form.uploaddir, fileName);
Don't write "200 OK" in response header, before parsing the request. There may be error (400, 500) while parsing the request.
Last but not least, rename "assests" -> "assets"

Hope it solves your issue(s).
A little question: did you get -> Upload done printed from your form.on('end') ?
(If yes, then it may be a bug in Formidable; otherwise it not be related to the lib).
Waiting for your response to see if issue got solved or not, to decide whether we need to close the issue proceed further.

I fix it by the other way


Bug 2116416: core: add support to rotate log based on logfile size

this commits add new field MaxLogSize inside LogCollectorSpec of
cephCluster cr which will take max size of log after which we want to
rotate the log.
Signed-off-by: subhamkrai<EMAIL_ADDRESS>(cherry picked from commit 62f73dcd989dde83eca48f58c47a7d27d98b6bc1)
(cherry picked from commit cb7728141fae86384d81e11297e0be2158b871fe)
core: fix logrotate bash check and periodicity logic
we need use != for string comparision in bash instead of
-ne. Also, need to correct periodicity if condition to
make it work as expected.
Signed-off-by: subhamkrai<EMAIL_ADDRESS>(cherry picked from commit https://github.com/red-hat-storage/rook/commit/9d4d5bc7025a026a7f11c52f48f1db0c00c4d5c9)

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
[ ] Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

@travisn ^^^

@subhamkrai we are just waiting on BZ approval for 4.11

/hold

waiting for https://github.com/rook/rook/pull/10751

/nohold

/hold cancel

/assign @travisn

/lgtm
/approve


Capacitor V4 support

Hello,
Capacitor V4 has been released.
Will this plugin support Capacitor V4 anytime soon ?
Thanks!

Yes, I will publish the update over the next week.

You can give it a try:
npm i<EMAIL_ADDRESS>

It works great ! Thanks !


NullGraphicsDevice implementation allowing to run the application without rendering

a NullGraphicsDevice implementation that allows to run PlayCanvas engine without graphics output (server environment for example).
engine examples framework is updated to add NULL option as a graphics device - examples all work but output nothing.

This is great. Will be more efficient when running it server-side using node.js.
We will be updating https://github.com/meta-space-org/playnetwork to use this feature.


[Incremental] Bug: Index failed: 'fatal error: virtual filesystem overlay file not found'

Description
Index failed when run examples/rules_ios in incremental mode. Howerver it works in legacy mode.
Detail error shown as below:
fatal error: virtual filesystem overlay file '/*/rules_xcodeproj/examples/rules_ios/bazel-output-base/rules_xcodeproj.noindex/indexbuild_output_base/execroot/_main/bazel-out/ios-sim_arm64-min15.0-applebin_ios-ios_sim_arm64-dbg-ST-a4a9813474a3/bin/iOSApp/Test/MixedUnitTests/iOSAppMixedUnitTests_vfs.yaml' not found
Reproduction steps

checkout to main branch (currently at commit 46f7a1f47c02e8bec3398913c5d018dd3cc71da5) of 'rules_xcodeproj'
cd examples/rules_ios
run bazel run //:xcodeproj-incremental
open rules_ios.xcodeproj
wait for Index

Expected behavior
Index works in incremental mode.
rules_xcodeproj version
46f7a1f47c02e8bec3398913c5d018dd3cc71da5
Xcode version
15.1
Bazel version
7.0.0
rules_apple version
No response
rules_swift version
No response
Additional information
No response

That vfsoverlay file is produced by Bazel target defined by rule framework_vfs_overlay, which is considered as unsupported target in both incremental mode and legacy mode.

Thanks for the detailed report. I'm looking into it.

Which file failed to index? do you have a fuller error? I'm not able to reproduce this. I do get this error:
/<unknown>: error: cannot open file '/*/rules_xcodeproj/examples/rules_ios/bazel-output-base/rules_xcodeproj.noindex/indexbuild_output_base/execroot/_main/bazel-out/ios-sim_arm64-min15.0-applebin_ios-ios_sim_arm64-dbg-ST-a4a9813474a3/bin/iOSApp/Test/MixedUnitTests/iOSAppMixedUnitTests_swift_vfs.yaml' (No such file or directory)

But it goes away once the correct Index Build that generates that file finishes.

It's file `rules_xcodeproj/examples/rules_ios/iOSApp/Test/MixedUnitTests/ObjCUnitTests.m`. And as shown in the above picture, there are also some other files  failed due to `virtual filesystem overlay file not found`

The errors occur in process of Indexing, at which time the Index Build has already finished.

Thanks! #2859 fixed that in my testing.


Facebook connector error

Hi
I have a bot, conected to telegram and facebook. With telegram working properly but with facebook not.
The error is
Facebook graph API call failed with: BadRequest {"error":{"message":"(#100) Length of param message[quick_replies][1][title] must be less than or equal to 20","type":"OAuthException","code":100,"fbtrace_id":"DRAKp97EzXc"}}
this appeare in the dev.botframework.com as issue
Thanks

Do you feel like the error message is inaccurate?  It's suggesting that quick reply titles must be less than 20 characters.

Hi
There is no title in the code:
connector = new builder.ChatConnector({
appId: "",
appPassword: ""
});
server.post('/api/messages', connector.listen());
bot = new builder.UniversalBot(connector);
bot.dialog('/',[function (session){
session.send("helloooo")
}])
The message error says that is in the Facebook Oauth Api Framenwork. I guess it's between the connection of Microsoft with facebook
This bot works with the cable Telegram, but not with facebook

I'm getting smilar error. It was working good but suddenly I have started to get this error on facebook :
Facebook graph API call failed with: BadRequest {"error":{"message":"(#100) Length of param message[quick_replies][0][title] must be less than or equal to 20","type":"OAuthException","code":100,"fbtrace_id":"BuoXth+PPOf"}}

I'm getting the error below at the same time on the console, may be this helps :
Prompts.choice - WARN: Callstack is invalid, resetting session.

Is this issue still going on?


Debug status bar item has visual glitch

It only shows up in some windows, but survives reloading. Inspected it with devtools and didn't see an obvious problem.

Can not reproduce this. And actually does not look that bad
Do you have some steps on how to end up in this state?
When inspecting did you check the width of the item? Is there some border which should not be there?
Could you see the glitch for other status bar items?

I was seeing this consistently for at least a couple days but I haven't seen it again recently 🤷♂

Might be some unrelated status bar work. Anyways if you see it again just try to inspect more and let me know how it looks


Add options to specify node-exporter user uid and gid

Hello,
it would be very useful if we had an option to explicitly specify the uid/gid of the node-exp user.
Our use case for this option is node-exporter access to nfs mounts (node_filesystem_* checks et al.).
Since default node-exp uids/gids can vary from nfs client to client (depending on already created local users/groups) we would need to be able to "anchor" them to a specific ones in order to allow access on the nfs server.
This PR doesn't change the behavior of the role, only allows the new variables to be uncommented and set if a user needs this functionality.
If there is anything else i should do re. this PR feel free to let me know.
br,
Petar

This role has been deprecated in favor of a the prometheus-community/ansible collection.


Add exit dialog to all the forms

I want to merge this change because...

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://qa.staging.saleor.cloud/graphql/

Task linked: SALEOR-2990 Add global util for exiting dirty form


ThinkPad 13 Gen 1

Model
ThinkPad 13

Patchset
Default for generation (SkyLake)

TPM
No

Notes
Advanced UEFI menu unlocked, wwan whitelist removal not tested. No wwan slot availlable on this model, I will test in the wan slot if I get my hands on a card. Relevant hex was found and replaced during patching.

I will wait until you test WLAN  whitelist removal until marking it as supported, Thanks


Add new field types

textarea (via options on text)
check (checkboxes)
selection / comboboxes (with chip support?)
range

int - may discuss config with rating

Fix for options and response type parsing issue: parse body with seerde_json::Value and try to match the exact type later


Qutebrowser-qt6 does not follow system cursor theme

Describe the bug
Similar to the issue mentioned in #181176, when qutebrowser with qt6 is installed it does not follow the system cursor theme.
It appears as if all qt6 applications share this problem right now.
Steps To Reproduce
Steps to reproduce the behavior:

Install qutebrowser-qt6 package.
Set a default cursor in your nix config / homemanager config
Open up qutebrowser and move the mouse over the window => Should show the default X Mouse theme instead of the custom one.
The screenshot below should help to understand the problem

Expected behavior
qt6 apps should also adhere to the cursor theme.
Screenshots

Additional context
With the regular qutebrowser package the cursor theme works fine.
Relevant package source: https://github.com/NixOS/nixpkgs/blob/61a8a98e6d557e6dd7ed0cdb54c3a3e3bbc5e25c/pkgs/applications/networking/browsers/qutebrowser/default.nix#L153
I am using bspwm + lightdm.
Relevant homemanager config snippet:
home = {
    # ...
    pointerCursor = {
      x11.enable = true;
      name = "Bibata-Original-Amber";
      package = pkgs.bibata-cursors;
      size = 16;
    };
};

Relevant nix config snippet:
services = {
    # ...
    xserver = {
          # ...
          gtk.cursorTheme = {
            name = "Bibata-Original-Amber";
            package = pkgs.bibata-cursors;
            size = 16;
          };
   };
}

Notify maintainers
@jagajaga @rnhmjoj @ebzzry @dotlambda @nrdxp
Thanks for your work! This is not urgent and I just wanted to leave it here for others with similar problems.
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.0.9, NixOS, 23.05 (Stoat), 23.05.20221125.5dc7114`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This issue has been lingering around for long. But qtbase does have replaced the use of libXcursor with libxcb per https://github.com/qt/qtbase/commit/012132c60d625b2de0039bdda3c22a0a8fe2dfe5, I'm not seeing why adding libXcursor to the LD_LIBRARY_PATH of the affected applications do fix the issue as in https://github.com/NixOS/nixpkgs/pull/196269. Hopefully we can fix it one and for all this time.

Or could it be related to https://github.com/NixOS/nixpkgs/pull/182580, as gtk themes does not automatically get applied to qt applications.


co-founder tag

add's a little co-founder tag for the cofounders -

See - https://github.com/FixPlanet/FixPlanet.github.io/compare/cofounder-tag?expand=1 - for the implementation on the website.


Fix a small bug on the document about testing

@cli.command already registered the sync command with the cli group.


phantomjs Syntax error: word unexpected (expecting ")")

Hello.
I try run selenium + phantomjs(Ghost.py, python phantomjs) on linux hosting(not vps \ vds), and it does not have ssh.
When i run script on cron, i get error:
raise WebDriverException("Unable to start phantomjs with ghostdriver.", e)
selenium.common.exceptions.WebDriverException: Message: Unable to start phantomjs with ghostdriver.
Screenshot: available via screen
I try manually run phantomjs(from cron) and get this error:
phantomjs: 1: Syntax error: "(" unexpected
I write full path to phantomjs in script.
Can You help me? What the problem?

I resolved this issue by uninstalling and reinstalling phantomjs-prebuilt:
npm uninstall phantomjs-prebuilt
then:
npm install phantomjs-prebuilt


Ensure we copy boltdb values

These slices are backed by a mmap’ed memory area and can cause segfaults if not copied out.
/cc @benbjohnson

LGTM

:+1:


Search for nvdisasm in the conda environment / CUDA_HOME / default CUDA install location in addition to the path

Fixes #9.
I'm not sure how good the unit test coverage is for this, or the other path search functionality. Ideally we'd need contained tests that cover:

Conda
A default CUDA install (e.g. /usr/local/cuda):

With the full toolkit.
With only libraries, no nvdisasm etc.
With only libraries, but nvdisasm in PATH.

CUDA_HOME:

With the full toolkit.
With only libraries, no nvdisasm etc.
With only libraries, but nvdisasm in PATH.

Just merged develop and pushed so that CI can run on this PR.

/ok to test

Looks like there's some issue with nvdisasm tests with this PR - I haven't looked into it yet, but let me know if any guidance would be helpful.

Rebased on main - I'm trying to get everything away from develop now as it no longer seems necessary.


Language is not loading

Describe the bug
Please check the issue. i can see the language in it
To Reproduce
just do a call to countries database
https://jmp.sh/bJLukcG
Expected behavior
Need to see country name
Screenshots
If applicable, add screenshots to help explain your problem.

@wasiqaftab Hi, thank you for reporting this. Can you please tell me how are you using the package? did you fork the repo or did you install the package in composer? did you publish php artisan vendor:publish --tag=world
from the video you attached it shows that it is not being able to resolve the world tag.

HI
Thanks for your quick response. Yes i publish the tag it woked with when i
choose local as per your documentation but not without that. so i assume i
need to configure something globally. Sorry which i dont understand as its
a default laravel installation
On Tue, Dec 14, 2021 at 9:42 AM Najm Njeim @.***> wrote:

@wasiqaftab https://github.com/wasiqaftab Hi, thank you for reporting
this. Can you please tell me how are you using the package? did you fork
the repo or did you install the package in composer? did you publish php
artisan vendor:publish --tag=world
from the video you attached it shows that it is not being able to resolve
the world tag.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nnjeim/world/issues/10#issuecomment-993355390, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3PTZM7AIFGTDZSJOHA4HTUQ4GQ7ANCNFSM5J7LN57Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Np thank u.


Introduce app ID management mechanism

Fixes part of #244 (server apps only)
Spec: https://g0v.hackmd.io/51wwLHgvSUqtBDaP-yAVnA

May be replaced by Cloudflare Zero Trust
https://g0v.hackmd.io/DxOR8llUQZiA3O1hTBgTVg#infra-Cloudflare-設定


[Table] allow objects to be passed as data values without throwing a propType error

What package is this for?

[x] React
[ ] Angular

Summary
We currently limit the data types for a table to be String, Boolean, or Integer to allow searching and filter, this was well intentioned, but overkill I think as we should allow consumers to pass objects are values if they choose and use a renderer to render the value.   Probably we should throw a warning if they pass and object as the value and don't override the customFilter and customSort logic for a row, and if they don't pass a customRenderer for the row
Here is the list of propTypes warnings we get when passing an object to a Table as value
Warning: Failed prop type: Invalid prop `values.expression` supplied to `TableBodyRow`.
    in TableBodyRow (created by TableBody)
    in TableBody (created by Table$$1)
    in table (created by Table)
    in div (created by Table)
    in Table (created by Table$$1)
    in div (created by Table$$1)
    in div (created by TableContainer)
    in TableContainer (created by Table$$1)
    in Table$$1 (created by MappingsViewModal)
    in div (created by Modal)
    in div (created by Modal)
    in div (created by Modal)
    in Modal (created by MappingsViewModal)
    in MappingsViewModal (created by Context.Consumer)
    in injectIntl(MappingsViewModal) (created by Wrapper)
    in Wrapper
    in form (created by Form)
    in Form (created by Formik)
    in Formik (created by Container)
    in div (created by Story)
    in Story (created by Container)
    in div (created by Container)
    in IntlProvider (created by Container)
    in HelmetProvider (created by Container)
    in Container (created by storyFn)
    in storyFn
    in ErrorBoundary
printWarning @ vendors~main.c65aea5156fc268b72d5.bundle.js:1101298
vendors~main.c65aea5156fc268b72d5.bundle.js:1101298 Warning: Failed prop type: Invalid prop `row.expression` supplied to `TableCellRenderer`.
    in TableCellRenderer (created by TableBodyRow)
    in TableBodyRow (created by TableBody)
    in tbody (created by TableBody)
    in TableBody (created by TableBody)
    in TableBody (created by Table$$1)
    in table (created by Table)
    in div (created by Table)
    in Table (created by Table$$1)
    in div (created by Table$$1)
    in div (created by TableContainer)
    in TableContainer (created by Table$$1)
    in Table$$1 (created by MappingsViewModal)
    in div (created by Modal)
    in div (created by Modal)
    in div (created by Modal)
    in Modal (created by MappingsViewModal)
    in MappingsViewModal (created by Context.Consumer)
    in injectIntl(MappingsViewModal) (created by Wrapper)
    in Wrapper
    in form (created by Form)
    in Form (created by Formik)
    in Formik (created by Container)
    in div (created by Story)
    in Story (created by Container)
    in div (created by Container)
    in IntlProvider (created by Container)
    in HelmetProvider (created by Container)
    in Container (created by storyFn)
    in storyFn
    in ErrorBoundary
printWarning @ vendors~main.c65aea5156fc268b72d5.bundle.js:1101298
vendors~main.c65aea5156fc268b72d5.bundle.js:1101298 Warning: Failed prop type: Invalid prop `children` supplied to `TableCellRenderer`.
    in TableCellRenderer (created by TableBodyRow)
    in TableBodyRow (created by TableBody)
    in tbody (created by TableBody)
    in TableBody (created by TableBody)
    in TableBody (created by Table$$1)
    in table (created by Table)
    in div (created by Table)
    in Table (created by Table$$1)
    in div (created by Table$$1)
    in div (created by TableContainer)
    in TableContainer (created by Table$$1)
    in Table$$1 (created by MappingsViewModal)
    in div (created by Modal)
    in div (created by Modal)
    in div (created by Modal)
    in Modal (created by MappingsViewModal)
    in MappingsViewModal (created by Context.Consumer)
    in injectIntl(MappingsViewModal) (created by Wrapper)
    in Wrapper
    in form (created by Form)
    in Form (created by Formik)
    in Formik (created by Container)
    in div (created by Story)
    in Story (created by Container)
    in div (created by Container)
    in IntlProvider (created by Container)
    in HelmetProvider (created by Container)
    in Container (created by storyFn)
    in storyFn
    in ErrorBoundary

Yes!  Thank you :dancing_men:


implement end of utterance detection for AssemblyAI

End of utterance detection allows users to specify the duration of silence to trigger a FinalTranscript from AssemblyAI.
Remove string encoding for terminate_msg

@CodiumAI-Agent /review

PR Analysis

🎯 Main theme: Implementing end of utterance detection for AssemblyAI
📝 PR summary: This PR introduces end of utterance detection for AssemblyAI by allowing users to specify the duration of silence to trigger a FinalTranscript. It also removes string encoding for terminate_msg.
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
⏱ Estimated effort to review [1-5]: 2, because the changes are straightforward and limited to a few lines of code.
🔒 Security concerns: No security concerns found

PR Feedback
💡 General suggestions: The PR is generally well done. However, it would be beneficial to add tests to ensure the new functionality works as expected. Additionally, consider adding error handling for the new end_utterance_silence_threshold parameter.
 🤖 Code feedback:relevant filevocode/streaming/transcriber/assembly_ai_transcriber.pysuggestion
Consider adding error handling for the end_utterance_silence_threshold parameter to ensure it is a valid integer and within a reasonable range. [important]
relevant lineend_utterance_silence_threshold: Optional[int] = Nonerelevant filevocode/streaming/transcriber/assembly_ai_transcriber.pysuggestion
It would be beneficial to add a comment explaining the purpose of threshold_msg and how it works. [medium]
relevant lineself.threshold_msg = (

 ✨ Usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:
[pr_reviewer]
some_config1=...
some_config2=...

  Utilizing extra instructions
The review tool can be configured with extra instructions, which can be used to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Specify the relevant sub-tool, and the relevant aspects of the PR that you want to emphasize.
Examples for extra instructions:
[pr_reviewer] # /review #
extra_instructions="""
In the code feedback section, emphasize the following:
- Does the code logic cover relevant edge cases?
- Is the code logic clear and easy to understand?
- Is the code logic efficient?
...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  How to enable\disable automation

When you first install PR-Agent app, the default mode for the review tool is:

pr_commands = ["/review", ...]

meaning the review tool will run automatically on every PR, with the default configuration.
Edit this field to enable/disable the tool, or to change the used configurations

  About the 'Code feedback' section
The review tool provides several type of feedbacks, one of them is code suggestions.
If you are interested only in the code suggestions, it is recommended to use the improve feature instead, since it dedicated only to code suggestions, and usually gives better results.
Use the review tool if you want to get a more comprehensive feedback, which includes code suggestions as well.

  Auto-labels
The review tool can auto-generate two specific types of labels for a PR:

a possible security issue label, that detects possible security issues (enable_review_labels_security flag)
a Review effort [1-5]: x label, where x is the estimated effort to review the PR (enable_review_labels_effort flag)

  Extra sub-tools
The review tool provides a collection of possible feedbacks about a PR.
It is recommended to review the possible options, and choose the ones relevant for your use case.
Some of the feature that are disabled by default are quite useful, and should be considered for enabling. For example:
require_score_review, require_soc2_review, enable_review_labels_effort, and more.

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the review usage page for a comprehensive guide on using this tool.

PR Review

 ⏱ Estimated effort to review [1-5]
2, because the changes are straightforward and localized to specific functionalities within the codebase. The addition of end of utterance detection and the modification of the terminate message are clear and do not seem to introduce complex logic.

 🧪 Relevant tests
No

 🔍 Possible issues
Possible Bug: The end_utterance_silence_threshold is an optional parameter but is not handled defensively in the assembly_ai_transcriber.py. If it's None, the code still attempts to send a threshold_msg which is None. This could potentially cause issues when interacting with the websocket.

 🔒 Security concerns
No

 Code feedback:
relevant filevocode/streaming/transcriber/assembly_ai_transcriber.py
suggestion

Consider adding a check to ensure self.threshold_msg is not None before sending it through the websocket. This prevents any potential errors or unexpected behavior when end_utterance_silence_threshold is not provided. [important]

relevant lineawait ws.send(self.threshold_msg)
relevant filevocode/streaming/models/transcriber.py
suggestion

It might be beneficial to add a brief comment explaining the unit of measurement and acceptable range for end_utterance_silence_threshold. This improves code readability and usability for future developers or users configuring the transcriber. [medium]

relevant lineend_utterance_silence_threshold: Optional[int] = None
relevant filevocode/streaming/transcriber/assembly_ai_transcriber.py
suggestion

For consistency and clarity, consider renaming threshold_msg to end_utterance_threshold_msg. This makes it immediately clear what the message is related to, especially since there could be other thresholds or messages in the future. [medium]

relevant lineself.threshold_msg = (
relevant filevocode/streaming/transcriber/assembly_ai_transcriber.py
suggestion

Since self.terminate_msg is no longer encoded, ensure that all usages of self.terminate_msg are compatible with a string format instead of bytes. This might involve checking how self.terminate_msg is sent over the network and ensuring compatibility. [important]

relevant lineself.terminate_msg = json.dumps({"terminate_session": True})

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:
/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:
[pr_reviewer]
some_config1=...
some_config2=...

  Utilizing extra instructions
The review tool can be configured with extra instructions, which can be used to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Specify the relevant sub-tool, and the relevant aspects of the PR that you want to emphasize.
Examples for extra instructions:
[pr_reviewer] # /review #
extra_instructions="""
In the 'possible issues' section, emphasize the following:
- Does the code logic cover relevant edge cases?
- Is the code logic clear and easy to understand?
- Is the code logic efficient?
...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  How to enable\disable automation

When you first install PR-Agent app, the default mode for the review tool is:

pr_commands = ["/review", ...]

meaning the review tool will run automatically on every PR, with the default configuration.
Edit this field to enable/disable the tool, or to change the used configurations

  Auto-labels
The review tool can auto-generate two specific types of labels for a PR:

a possible security issue label, that detects possible security issues (enable_review_labels_security flag)
a Review effort [1-5]: x label, where x is the estimated effort to review the PR (enable_review_labels_effort flag)

  Extra sub-tools
The review tool provides a collection of possible feedbacks about a PR.
It is recommended to review the possible options, and choose the ones relevant for your use case.
Some of the feature that are disabled by default are quite useful, and should be considered for enabling. For example:
require_score_review, require_soc2_ticket, and more.

  Auto-approve PRs
By invoking:
/review auto_approve

The tool will automatically approve the PR, and add a comment with the approval.
To ensure safety, the auto-approval feature is disabled by default. To enable auto-approval, you need to actively set in a pre-defined configuration file the following:
[pr_reviewer]
enable_auto_approval = true

(this specific flag cannot be set with a command line argument, only in the configuration file, committed to the repository)
You can also enable auto-approval only if the PR meets certain requirements, such as that the estimated_review_effort is equal or below a certain threshold, by adjusting the flag:
[pr_reviewer]
maximal_review_effort = 5

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the review usage page for a comprehensive guide on using this tool.

Hey @ajar98 and @arpagon! Thank you for your help with this PR. I've gone ahead and implemented end_utterance_silence_threshold together with EndpointingConfig.
I have checked and we do not send a message of None when end_utterance_silence_threshold_msg is None so I think the Codium bot was hallucinating.
Please let me know if you have any other questions!


Discussion: Rename app to data-upload or something more descriptive?

Low priority, but maybe renaming the app/ folder to something less vague would be nice. Since we're doing data-upload, that makes sense. But not sure...

I agree

This is already done, now as sprout.


"panic: Offset for higher number argument asked for than reserved"

I get panic: Offset for higher number argument asked for than reserved for code generated using clang -O3 -masm=intel -mstackrealign -fno-asynchronous-unwind-tables -fno-exceptions -fno-rtti -S mdb.c
with clang 7.0.0 on windows 7 x64
When this check panics, the values are iarg=5; len(registers)=6.
After defining the go signature for func _mdb_txn_renew(txn unsafe.Pointer) (errCode int) the code panics the first time. I think the following asm belongs to it.
	.def	 mdb_txn_renew;
	.scl	2;
	.type	32;
	.endef
	.globl	mdb_txn_renew           # -- Begin function mdb_txn_renew
	.p2align	4, 0x90
mdb_txn_renew:                          # @mdb_txn_renew
# %bb.0:
	push	rbp
	sub	rsp, 32
	lea	rbp, [rsp + 32]
	and	rsp, -16
	test	rcx, rcx
	je	.LBB9_3
# %bb.1:
	mov	eax, 131073
	and	eax, dword ptr [rcx + 124]
	cmp	eax, 131073
	jne	.LBB9_3
# %bb.2:
	call	mdb_txn_renew0
	mov	rsp, rbp
	pop	rbp
	ret
.LBB9_3:
	mov	eax, 22
	mov	rsp, rbp
	pop	rbp
	ret

-inline-threshold=1000 was too low.


Open 2023-10-25-mirrorbits-arm64-migration

Ref: https://github.com/jenkins-infra/helpdesk/issues/3619 & https://github.com/jenkins-infra/kubernetes-management/pull/4577#issuecomment-1778619413

LGTM, self-merging.


Restart clears minimal document resize history

Enable split view and resize a document to its minimal horizontal size
clicking the left and right document expands and shrinks the documents (very useful feature)
if the window size expands past the point when the document was resized. (either manually or simply shrinking a side pannel)
both documents will expand, rather than just the larger one (another bug/feature)
sadly the size at which it starts expanding is reset when vscode restarts
so everytime I shrink a side pannel after restarting, I then have to resize the small document again back down to its minimal size

This needs to be a setting at the splitview layer which makes sure that minimally sized views remain minimally sized, even as the layout expands.

Actually... duplicate of https://github.com/microsoft/vscode/issues/70083


Wrong  value in docs

Docs generated via crystal docs have <meta name="generator" content> value from the crystal version used to build the compiler instead of the compiled one, being one-of - ie. docs generated with crystal 0.35.1 have crystal 0.34.0 given and docs build for master have crystal version 0.35.1 given.

The value used by the docs generator is Crystal::VERSION which is the actual version of the compiler, also reported by crystal --version. So that should be correct.
And CI should use the freshly built compiler to build the docs, so it should really be identical to project_version.
In fact, the build_docs job for 0.35.1 docs reports running with Crystal 0.35.1 [5999ae29b] (2020-06-19) (
https://app.circleci.com/pipelines/github/crystal-lang/crystal/3875/workflows/528b3e80-5102-4267-a3f5-1855d3f4b667/jobs/48703). No idea what's going wrong here.

The value used by the docs generator is Crystal::VERSION which is the actual version of the compiler, also reported by crystal --version. So that should be correct.
And CI should use the freshly built compiler to build the docs, so it should really be identical to project_version.
In fact, the build_docs job for 0.35.1 docs reports running with Crystal 0.35.1 [5999ae29b] (2020-06-19) (
https://app.circleci.com/pipelines/github/crystal-lang/crystal/3875/workflows/528b3e80-5102-4267-a3f5-1855d3f4b667/jobs/48703). No idea what's going wrong here.

FWIW crystal --version seems to be using Crystal::Config.version which reads the VERSION file directly while Crystal::VERSION is a built in constant. So might be some issue with using a built in constant within the compiler itself? Could maybe just replace this tag with Crystal::Config.version and call it a day?

I don’t understand why this is an issue. I think the doc generator works as expected. It makes sense to me that the generator version matches with the version of the compiler performing the doc generation.

It makes sense to me that the generator version matches with the version of the compiler performing the doc generation.

@potomak it doesn't, that's the issue.

@Sija wouldn’t you expect to see docs with a generator version value equals to the value of ‘crystal -v’? I think that’s reasonable.
Maybe the issue is that the doc isn’t generated using the latest compiler build rather than what value is used in the template.

@potomak I'd expect that the generator version put in the <meta/> would contain the value of the compiler version used to generate the docs.

What is the job to generate the docs?


UTF-8 accents on triggers

Hello,
in my 1.14 python version,
and with ai = RiveScript(utf8=True) on my python file,
I can't enter any accents "éèàç..." on question.
here is the error :
Traceback (most recent call last):
File "C:\Users\neo\Downloads\rivescript-python\RS_AI.py", line 17, in
reply = ai.reply("localuser", msg)
File "C:\Users\neo\Downloads\rivescript-python\rivescript\rivescript.py", line 999, in reply
return self._brain.reply(user, msg, errors_as_replies)
File "C:\Users\neo\Downloads\rivescript-python\rivescript\brain.py", line 46, in reply
self.say("Get reply to [" + user + "] " + msg)
File "C:\Users\neo\Downloads\rivescript-python\rivescript\brain.py", line 41, in say
self.master._say(*args, **kwargs)
File "C:\Users\neo\Downloads\rivescript-python\rivescript\rivescript.py", line 149, in _say
self._log.write("[RS] " + message + "\n")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe9 in position 25: ordinal not in range(128)
What did I miss ???
Thanks Kirsle
Vincent

Well, utf8 bool doesn't seems to run ?!

It's a Python 2 Unicode problem, and I can't reproduce it under Python 3. It also seems to be a regression as it used to work in Python 2 in the past.
I looked into it for a while yesterday but it'll require digging deeper into it. Usually it was the case that as long as you make sure your input message is a proper Unicode object (if printing repr(message) gave a string beginning with u', for example u'pok\xe9mon') then everything else would work as the code itself uses unicode_literals, so interpolating your Unicode string into a quoted string would work fine, but that didn't seem to help this time.

There seemed to be two different problems:

A couple of the source files did not do from __future__ import unicode_literals, so I fixed them so they all import this (that fixes the interpolation errors when putting unicode strings into quoted messages for debug logging etc.)
The interactive mode needed to additionally decode the user's input message into Unicode, but only for Python 2 (in Python 3 the result from input() is already Unicode to begin with).

I'll release a fix pretty soon.

Try version 1.14.1.

hi Kirsle,
last version tested right now,
with python 3 question with accents (écoute moi ça) seems to work. No errors
however, can't enter this trigger : + écoute moi ça
returns :
raise Exception(syntax_error)
Exception: Syntax error in ./eg/neo_brain\neo_Questions.rive line 106: Triggers may only contain lowercase letters, numbers, and these symbols: ( | ) [ ] * _ # @ { } < > = (near: + écoute moi)
See you and thanks again for help.
U'll wait for fix. Have a nice day

Exception: Syntax error in ./eg/neo_brain\neo_Questions.rive line 106: Triggers may only contain lowercase letters, numbers, and these symbols: ( | ) [ ] * _ # @ { } < > = (near: + écoute moi)

For this you have to use the utf8=True constructor option. By default it won't allow any non-ASCII letters in triggers (accented letters would also be removed from the user's message anyway and would make matching impossible).

Works nice on python 3,
Working on python2 (i try some possibilities)
Thanks


Enable modification of config. for endpoints with reserved labels

Master currently disallows users from modifying the configuration of endpoints with reserved labels, except for the reserved:init label. This restriction prevents e.g., the use of the debug mode for the host endpoint.
This commit removes that restriction for some of the configuration options:

Config
Default
Can modify

Conntrack
Enabled

ConntrackAccounting
Enabled

ConntrackLocal
Disabled

Debug
Disabled
x

DebugLB
Disabled
x

DropNotification
Enabled
x

MonitorAggregationLevel
None
x

NAT46
Disabled

PolicyAuditMode
Disabled
x

PolicyVerdictNotification
Enabled
x

TraceNotification
Enabled
x

To summarize, for endpoints with reserved labels, only audit mode and configuration options that decide what logs are sent to cilium monitor can be changed for endpoints with reserved labels.
Fixes: #12037

test-me-please

Coverage decreased (-0.03%) to 36.961% when pulling e6b1341452b094267f34ff101003a974a3a21914 on pr/pchaigno/modify-config-reserved-endpoints into c4b2c1e6c7245bdaf2bb4a904ed80a4087623f4d on master.

retest-net-next

PolicyAuditMode	Disabled	x

... only configuration options that decide what logs are sent to cilium monitor can be changed for endpoints ...

Not sure this is consistent with the code :-)
$ git grep -A 7 AUDIT bpf/lib/
bpf/lib/policy.h:#ifdef POLICY_AUDIT_MODE
bpf/lib/policy.h-       if (IS_ERR(ret)) {
bpf/lib/policy.h-               ret = CTX_ACT_OK;
bpf/lib/policy.h-               *audited = 1;
bpf/lib/policy.h-       }
bpf/lib/policy.h-#endif
bpf/lib/policy.h-
bpf/lib/policy.h-       return ret;
--
bpf/lib/policy.h:#ifdef POLICY_AUDIT_MODE
bpf/lib/policy.h-       if (IS_ERR(ret)) {
bpf/lib/policy.h-               ret = CTX_ACT_OK;
bpf/lib/policy.h-               *audited = 1;
bpf/lib/policy.h-       }
bpf/lib/policy.h-#endif
bpf/lib/policy.h-       return ret;
bpf/lib/policy.h-}

That said I suspect that you're specifically interested in allowing POLICY_AUDIT_MODE to influence the host endpoint.
Taking a step back and looking at the reasoning behind the config restrictions endpoints with reserved:___ labels, I had originally only thought as far as the health endpoint. For that endpoint, I figured it's entirely managed by Cilium internally and users would not (and should not) tinker with it; At the time we were considering extending it to perform regular services / policy checks to validate underlying connectivity and I didn't want to somehow end up in a situation where we rely on specific datapath configuration in the logic that manages the health checks, but then a user inadvertently modifies one of the endpoints and causes those checks to fail.
I don't think there's much of a security barrier question here, because even if we were to continue to disallow something like the CONNTRACK or this POLICY_AUDIT_MODE option from being configured via the API, someone with local access could still import directly a policy that allows some set of traffic. Cilium daemon would argue with it and likely clear such a policy out, re-syncing with k8s from time to time but I guess my point is that local socket access to the API is not considered a barrier between security domains; someone with access to the Cilium unix socket is likely to be able to do much more dangerous things than tweaking these specific datapath options.
At this point I don't think many of the original arguments really apply. Even for the health endpoint, I don't think we got to a point where it really made a difference and most likely the only folks who know about this API or regularly use it would be Cilium developers anyway.  So from that perspective I'm OK with loosening these restrictions.
Regarding enabling datapath debug, I think it's reasonable to locally get access to a Cilium socket and enable debug via CLI / API like this. I think if we really wanted to make this usable we'd probably support some kind of pod or node annotation that Cilium respected, but I'd guess that's more effort than it's worth for what you're trying to achieve here.

PolicyAuditMode Disabled x

...

... only configuration options that decide what logs are sent to cilium monitor can be changed for endpoints ...

Not sure this is consistent with the code :-)

Ouch, yes. I've updated the commit message to clarify that audit mode can also be changed.

That said I suspect that you're specifically interested in allowing POLICY_AUDIT_MODE to influence the host endpoint.

Yes, I'd like to be able to switch the host firewall in audit mode without switching all of Cilium. There's still some work required to allow that AFAICS.
I'm also interested in being able to easily switch the host endpoint's debug mode and general verbosity (ex. log aggregation), but that's mostly for debugging and troubleshooting.
In general, my approach was to only allow setting configurations that shouldn't have a huge impact on forwarding (except in case of complexity issues of course...) to prevent users from shooting themselves in the foot too easily. The audit mode is a bit of a corner case here, but as far as I can see, switching it on shouldn't result in any connectivity breakage.

test-me-please

test-me-please


The tag of input is lost immediately after entering characters on the outer styled-component wrapping component

The tag of input is lost immediately after entering characters on the outer styled-component wrapping component. I want to know why,
the following is the address of the link：https://codesandbox.io/s/610mrrmnlw

Lose focus

👋🏻 please use stack overflow for questions relating to use of the library. If you end up finding a reproducible bug, feel free to open up a more specific issue. Thanks!


remove description field from viewed statement template example

from @gmoger-jisc

Also for viewed statements there is no ‘description’ field as part of the tsv spec or json template but it is listed example under activity types:
https://github.com/jiscdev/xapi/blob/1.0.1/vocabulary.md#activity-types – should it be removed?  I understand it is recommended by xAPI but we don’t have data to use for it and would most likely just replicate the name.

Weekly call: remove from the example


Can session tokens be used for access control instead of keys

I am attempting to limit the use of master/secondary keys in the application, or possibly even control tighter controls when using them.  One way to do this is to use session tokens with the query requests.  Is there a way to do this with Spring Data for Cosmos DB?

@mnolan98-tmk - Not yet, but soon we are going to provide AAD feature, which should allow you to have tighter controls.


Fixed issue #1595 | Easy Issue (CSS)

Pull Request Type
What kind of change does this PR introduce?

Solved an issue/task

What is the current behavior?

Issue Number: https://github.com/fork-commit-merge/fork-commit-merge/issues/1595
What is the new behavior?

The Color of The Header From Black to Red

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Fix typo

a -> an

@changeworld : Thanks for your contribution! The author, @georgewallace, has been notified to review your proposed change.


Fix local image path when using a custom device

Do I need to review this @jonastreub ?

Yes please!

Thanks for the feedback @koenbok! Code is much clearer now.


accuracy = 0.000000, best accuracy 0.000000

I'm getting zeros for accuracy while the loss is decreasing.
# python train.py --data_dir ./data --logdir ./logs_train_0514_run2
Start training
train.py:99: UserWarning: invalid index of a 0-dim tensor. This will be an error in PyTorch 0.5. Use tensor.item() to convert a 0-dim tensor to a Python number
  datetime.now(), step, loss.data[0], learning_rate, examples_per_sec)
=> 2018-05-14 10:08:17.556565: step 100, loss = 7.605348, learning_rate = 0.010000 (507.5 examples/sec)
=> 2018-05-14 10:08:23.371309: step 200, loss = 6.634058, learning_rate = 0.010000 (556.2 examples/sec)
=> 2018-05-14 10:08:29.204335: step 300, loss = 6.444423, learning_rate = 0.010000 (554.7 examples/sec)
=> 2018-05-14 10:08:35.037947: step 400, loss = 6.654078, learning_rate = 0.010000 (554.6 examples/sec)
=> 2018-05-14 10:08:40.876440: step 500, loss = 6.415401, learning_rate = 0.010000 (554.1 examples/sec)
=> 2018-05-14 10:08:46.724192: step 600, loss = 6.980000, learning_rate = 0.010000 (553.6 examples/sec)
=> 2018-05-14 10:08:52.578867: step 700, loss = 7.336755, learning_rate = 0.010000 (552.7 examples/sec)
=> 2018-05-14 10:08:58.457534: step 800, loss = 6.166699, learning_rate = 0.010000 (550.8 examples/sec)
=> 2018-05-14 10:09:04.360389: step 900, loss = 6.186161, learning_rate = 0.010000 (547.6 examples/sec)
=> 2018-05-14 10:09:10.669834: step 1000, loss = 6.420802, learning_rate = 0.010000 (512.7 examples/sec)
=> Evaluating on validation dataset...
/notebooks/evaluator.py:16: UserWarning: volatile was removed and now has no effect. Use `with torch.no_grad():` instead.
  images, length_labels, digits_labels = (Variable(images.cuda(), volatile=True),
==> accuracy = 0.000000, best accuracy 0.000000
=> patience = 99
=> 2018-05-14 10:09:31.108286: step 1100, loss = 5.796063, learning_rate = 0.010000 (524.8 examples/sec)
=> 2018-05-14 10:09:37.420145: step 1200, loss = 5.399920, learning_rate = 0.010000 (512.9 examples/sec)
=> 2018-05-14 10:09:43.742597: step 1300, loss = 5.895159, learning_rate = 0.010000 (512.1 examples/sec)
# python train.py --data_dir ./data --logdir ./logs_train_0514_run2
Start training
train.py:99: UserWarning: invalid index of a 0-dim tensor. This will be an error in PyTorch 0.5. Use tensor.item() to convert a 0-dim tensor to a Python number
  datetime.now(), step, loss.data[0], learning_rate, examples_per_sec)
=> 2018-05-14 10:08:17.556565: step 100, loss = 7.605348, learning_rate = 0.010000 (507.5 examples/sec)
=> 2018-05-14 10:08:23.371309: step 200, loss = 6.634058, learning_rate = 0.010000 (556.2 examples/sec)
=> 2018-05-14 10:08:29.204335: step 300, loss = 6.444423, learning_rate = 0.010000 (554.7 examples/sec)
=> 2018-05-14 10:08:35.037947: step 400, loss = 6.654078, learning_rate = 0.010000 (554.6 examples/sec)
=> 2018-05-14 10:08:40.876440: step 500, loss = 6.415401, learning_rate = 0.010000 (554.1 examples/sec)
=> 2018-05-14 10:08:46.724192: step 600, loss = 6.980000, learning_rate = 0.010000 (553.6 examples/sec)
=> 2018-05-14 10:08:52.578867: step 700, loss = 7.336755, learning_rate = 0.010000 (552.7 examples/sec)
=> 2018-05-14 10:08:58.457534: step 800, loss = 6.166699, learning_rate = 0.010000 (550.8 examples/sec)
=> 2018-05-14 10:09:04.360389: step 900, loss = 6.186161, learning_rate = 0.010000 (547.6 examples/sec)
=> 2018-05-14 10:09:10.669834: step 1000, loss = 6.420802, learning_rate = 0.010000 (512.7 examples/sec

Any ideas?

Fixed in my fork. I'm running pytorch 0.4.0.


if trying custom yolov4 training, only get result of yolov4.index and yolov4.data-00000-of-00001

I have no idea how to do next to test on linux or deploy for android

@KYUNGGUK-CHOI   i have same issue, how can you do?

@mathpopo
which one? linux or android?
after running custom train.py then run convert_tflite.py
you would get tflite file then adapt it on anroid

I got an error CUDA_ERROR_UNKNOWN: unknown error then Fatal Python error: Aborted when converting to tflite. Did you got this error too?


Async credentials support CAE

They would need to claim client capability "CP1" in token requests and accept a claims keyword argument to get_token(). Sync credentials do this and rely on MSAL for the details. Doing the same for async credentials is blocked on AzureAD/microsoft-authentication-library-for-python#88

CP1 capabilities (i.e. CAE) were handled in the AAD Client via https://github.com/Azure/azure-sdk-for-python/pull/30777


nvcc fatal   : Unsupported gpu architecture 'compute_75'

I have encountered the following problem when i tried to execute luarocks install cudnn

CMake Error at /usr/local/Cellar/cmake/3.13.2/share/cmake/Modules/FindCUDA.cmake:696 (message):
Specify CUDA_TOOLKIT_ROOT_DIR
Call Stack (most recent call first):
CMakeLists.txt:7 (FIND_PACKAGE)

The problem was solved by installing cuda from https://developer.nvidia.com
But when i run this command for second time, a new problem has arisen.

nvcc fatal   : Unsupported gpu architecture 'compute_75'
nvcc fatal   : Unsupported gpu architecture 'compute_75'
CMake Error at THC_generated_THCReduceApplyUtils.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCReduceApplyUtils.cu.o
CMake Error at THC_generated_THCBlas.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCBlas.cu.o
nvcc fatal   : Unsupported gpu architecture 'compute_75'
CMake Error at THC_generated_THCSleep.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCSleep.cu.o
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCReduceApplyUtils.cu.o] Error 1
make[2]: *** Waiting for unfinished jobs....
nvcc fatal   : Unsupported gpu architecture 'compute_75'
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCBlas.cu.o] Error 1
CMake Error at THC_generated_THCTensorCopy.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCTensorCopy.cu.o
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCSleep.cu.o] Error 1
nvcc fatal   : Unsupported gpu architecture 'compute_75'
CMake Error at THC_generated_THCStorageCopy.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCStorageCopy.cu.o
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCTensorCopy.cu.o] Error 1
nvcc fatal   : Unsupported gpu architecture 'compute_75'
nvcc fatal   : Unsupported gpu architecture 'compute_75'
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCStorageCopy.cu.o] Error 1
CMake Error at THC_generated_THCStorage.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCStorage.cu.o
CMake Error at THC_generated_THCHalf.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCHalf.cu.o
nvcc fatal   : Unsupported gpu architecture 'compute_75'
CMake Error at THC_generated_THCTensor.cu.o.Release.cmake:219 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-2575/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCTensor.cu.o
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCHalf.cu.o] Error 1
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCStorage.cu.o] Error 1
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCTensor.cu.o] Error 1
make[1]: *** [lib/THC/CMakeFiles/THC.dir/all] Error 2
make: *** [all] Error 2
Error: Failed installing dependency: https://raw.githubusercontent.com/torch/rocks/master/cutorch-scm-1.rockspec - Build error: Failed building.

How can i solve this problem?

I followed this solution and solved the issue by setting the following environment variable export TORCH_CUDA_ARCH_LIST="7.0" before installing cudnn


Update API-Reference.md

fix spawn entities lists URLs for Bukkit and CanarayMod (they were reversed)

by the way, I was just reading the documentation but it may be not useful to fix the doc, as the spawn plugin itself doesn’t seem to work with CanaryMod

Hi Thierry,
Please see the note at the top of the file you changed (reproduced here)...
IMPORTANT NOTE FOR CONTRIBUTORS
Contributors: This file is generated from comments in javascript source files src/main/js/*
If you would like to make changes, change the comments in the src/main/js/* files instead.

shame on me !
Thanks to remind me to read all the document before modifying ;)


Issue#440 image metadata form in uploader

Resolves https://github.com/GlobalDigitalLibraryio/issues/issues/440

Great work, @Mkohm!!


Ensure compatibility with xarray>0.15

This uses part of #31 and solves #30

Codecov Report

Merging #32 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master     #32   +/-   ##
======================================
  Coverage    79.6%   79.6%
======================================
  Files           3       3
  Lines         152     152
  Branches       24      24
======================================
  Hits          121     121
  Misses         30      30
  Partials        1       1

Impacted Files
Coverage Δ

xorca/lib.py
98.3% <100%> (ø)
: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 597bbe8...5751eda. Read the comment docs.


Organize template according to your own idea

Hi,
Now that the demo display is ready, I would like to compile 1 template with my own layout, how can you change this without getting completely confused. eg the train number is now under the time, can it also be placed in the position of the running text?
The layout I want =
top left : the time - top center : train number, right top : train company logo.
Bottom stays the same
is this possible and if so which numbers do i need to adjust to get it that way?

Hi, I managed to make my own Dutch template, with some German accents, so I left the platform number, which is officially not on the Dutch screen and on the Dutch screen it always says via .... information eg Amsterdam via Arnhem etc.
What I do wonder is, you enter the departure time, but do you have to create a new button for each departure time that departs from platform 1 for example? eg 09.00 The Hague and 10.00 Amsterdam, both platform 1 or do you have to set them later so that they automatically switch to the next departure text?
still wondering how this can be solved?
Greetings

Looks good.
Rocrail sends the message in the text field to the display.
There are several mechanisms in rocrail to write data to a textfield. Actions, Scripts, ...
Some people in the Rocrail forum started to use data from the train schedule.
I'm not a rocrail expert and cannot help with that.

Hi,
Do you have a direct link to that forum section, I can see if there is something possible. But if you do it manually do you have to create a new button for each time?
Greetings


Mvc-start

initial api ok

initial api for endpoints - working
next branch will begin:
PKFK relationships
models/controllers for other tables
cascading update/delete


Investigate print statement race conditions

Might be a situation where you call print(), it fills the global TX buffer, calls send_uart(). Then an interrupt occurs, it calls print(), overwriting part/all of the global buffer, etc.

It's true that the print buffer might be corrupted, but there would be problems if we made all calls to print() atomic, e.g. when receiving UART it operates at the same rate as sending UART. If any code needs to guarantee uninterrupted UART it can call print() within an atomic block.


'latin-1' codec can't encode character

When downloading songs with certain characters in them, the downloader throws the following error and skips that song
user@raspberrypizero2w:~ $ spotify_sync run auto --profile profile
[2023-11-18 22:58:35,983] INFO      SpotifySync      : Loading config from profile: profile
[2023-11-18 22:58:35,985] INFO      SpotifySync      : Script started with auto command
[2023-11-18 22:58:37,193] INFO      SpotifyService   : Fetching liked songs
[2023-11-18 22:58:37,663] INFO      SpotifyService   : Added 0 new liked song(s) from Spotify (cached:0)
[2023-11-18 22:58:37,664] INFO      SpotifyService   : Fetching playlist songs
[2023-11-18 22:58:37,977] INFO      SpotifyService   : Adding 14/14 from playlists
[2023-11-18 22:58:38,176] INFO      MatchService     : Attempting to match 14 Spotify songs to Deezer
[2023-11-18 22:58:40,212] INFO      MatchService     : Matched 14/14 new liked song(s)
[2023-11-18 22:58:40,222] INFO      DownloadService  : Downloading 14 song(s) from Deezer
[2023-11-18 22:58:42,828] INFO      DeemixDownloader : Gathering song information in preparation for download..
[2023-11-18 22:58:43,696] INFO      DeemixDownloader :  1/14 - STARTING [Korn - My Gift to You]
[2023-11-18 22:58:43,698] INFO      DeemixDownloader :  2/14 - STARTING [Korn - Pretty]
[2023-11-18 22:58:43,700] INFO      DeemixDownloader :  3/14 - STARTING [Korn - It's On!]
[2023-11-18 22:58:43,703] INFO      DeemixDownloader :  4/14 - STARTING [Korn - Justin]
Ko\u042fn - My Gift to You 'latin-1' codec can't encode character '\u042f' in position 33: ordinal not in range(256)
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.11/site-packages/deemix/downloader.py", line 651, in downloadWrapper
    result = self.download(extraData, track)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.local/lib/python3.11/site-packages/deemix/downloader.py", line 421, in download
    makedirs(filepath, exist_ok=True)
  File "<frozen os>", line 215, in makedirs
  File "<frozen os>", line 225, in makedirs
UnicodeEncodeError: 'latin-1' codec can't encode character '\u042f' in position 33: ordinal not in range(256)
[2023-11-18 22:58:44,571] WARNING   DeemixDownloader :  1/14 - FINSIHED [Korn - My Gift to You] - Failed, download for DeezerId: 134523802 error: 'latin-1' codec can't encode character '\u042f' in position 33: ordinal not in range(256)

Korn is stylised as KoЯn on Deezer but not on Spotify, figured I would bring this issue up as other artists may be stylised with similar characters and cause this error to appear.

@meltham44 can't replicate this on MacOS, which operating system are you using? Can you replicate this on any other OS's?


Serialization issues

Why does laravel-settings unserialize the stored value with the ['allowed_classes' => false] option? This causes some issues when storing an object or class. For example:
$project = Project::find(1);

Settings::set('project', $project);

$p = Settings::get('project');

dd($p);

This returns a __PHP_Incomplete_Class(App\Models\Project) object instead of the actual Project eloquent model. Now storing an object in the settings might not be very useful, but this also happens when storing a Carbon object (i.e. Settings::set('last_import', Carbon::now()))
I would expect the Settings helper to work the same as the default Laravel Cache and Session helpers. These also (un)serialize data using the default PHP serialize() and unserialize() functions but without the allowed_classes option.

The allowed_classes option is used because unserializing objects can be a security risk, especially if the input is coming from users. If that's a risk you're willing to take and/or you trust the data being unserialized, the easiest way to allow for this is to override the default ValueSerializer class.
namespace App\Support;

use Rawilk\Settings\Support\ValueSerializers\ValueSerializer;

class CustomValueSerializer extends ValueSerializer
{
    public function unserialize(string $serialized): mixed
    {
        return unserialize($serialized);
    }
}

Then, in the config/settings.php file, update the value_serializer key to the custom one:
// config/settings.php
'value_serializer' => \App\Support\CustomValueSerializer::class,

I may also consider adding in a whitelist option to the config for objects that are allowed to be unserialized, which would be fine for a few classes, however the solution I showed above is probably more ideal if you're going to storing many different kinds of objects in the settings.
Also, imo it's better to store just the id of a model as a setting, or the actual timestamp from a date object, like now()->unix(). This, however is more of a personal preference though.

__PHP_Incomplete_Class_Name
new version 3.3 seen not compatible with version 2.2.

@irvine48 Not sure what you mean by this. v3+ is a major version change, and has breaking changes from v2.

The safelist seems like a good idea! Perhaps allowing a wildcard like  ['unserialize_safelist' => '*'] would be usefull if you explicitly allow unserializing of all classes.

I'm going to close this issue since the ability to safelist has been added. Feel free to re-open in the future if necessary.


BZ1007859 - Add 'select all' checkbox to Process instances

Use ResizableHeader from uberfire-ext.
Change the check column, adding the header for select all instances. Depending on https://github.com/uberfire/uberfire-extensions/pull/277

Hi @nmirasch looking really good! Only two details I would like to add:

Give a fixed width to the check column enough to display the check box, so we can have more spce available to the other columns.
When I bulk abort all instances I had active, the column header was left with a check state that I couldn't uncheck. Should we review to clean the check status when the table becomes empty? See screenshot.

@cristianonicolai done. Can you take a look? Thanks!

@nmirasch looks good now, just missing the fixed width.

@cristianonicolai  now this pr depends on https://github.com/uberfire/uberfire/pull/366

+1 Looks great, thanks @nmirasch!
Waiting for https://github.com/uberfire/uberfire/pull/366 to get merged so I can merge this one as well.


Added Mikrotik platform support for device_tracker

Description:
Mikrotik Platform for Device Tracker
Example entry for configuration.yaml (if applicable):
device_tracker:
  - platform: mikrotik
    host: <IP_ADDRESS>
    username: admin
    password: admins

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Hi @Xstasy,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Any updates on this PR?

@Xstasy Daniel, do you plan to finish this PR?

I can say /ip/dhcp-server/lease/getall is totally wrong for presence detection. DHCP leases can stay for minutes (~10), while you're already disconnected from network. The most reliable combination is netwatch + dhcp leases (to get MAC from IP).


[chart/grafana-agent][REQUEST] Allow enabling debug mode

Is your feature request related to a problem? Please describe.
Sometimes we need to have debug mode enabled via this line in agent.yml file:
server:
  log_level: debug

But in current chart there is hardcoded only the log_level: info value.
Describe the solution you'd like
Make value of log_level configurable using values.yaml.

@MurzNN Oops. Thank you, I'll jump on that quickly.


fix link error

Signed-off-by: Poor12<EMAIL_ADDRESS>What type of PR is this?
/kind cleanup
What this PR does / why we need it:
fix link error in docs and link articles to web pages whenever possible
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
None

cc @RainbowMango


RLM_ARRAY_TYPE has been deprecated. Use RLM_COLLECTION_TYPE instead.

How frequently does the bug occur?
All the time
Description
Hello,
i would like to ask related to deprecated RLM_ARRAY_TYPE, when i upgrade realm version from 3.20 to 10.28.2 i see the warning that said: RLM_ARRAY_TYPE has been deprecated. Use RLM_COLLECTION_TYPE instead. in my RLMObject
@property RLMArray<DisplayID> *displayIds;

Since the documentation https://www.mongodb.com/docs/realm/sdk/swift/model-data/define-model/relationships/#define-a-to-many-relationship-property did not mention RLMArray is the deprecated, what is the alternative to replace RLMArray?
Stacktrace & log output
No response
Can you reproduce the bug?
Yes, always
Reproduction Steps
Build source code that use RLMArray
Version
10.28.2
What SDK flavour are you using?
Local Database only
Are you using encryption?
No, not using encryption
Platform OS and version(s)
13.0
Build environment
Xcode version: xcode 13.4.1
Dependency manager and version: Cocoapod v1.11.2

Hi @dwirandytlvk thanks for reporting this. CC @dacharyc can we update the docs to use RLM_COLLECTION_TYPE?

Hey @dacharyc that is correct.

how about backward compatibility?
is it safe to use RLM_COLLECTION_TYPE macro when i use realm v.3.20? since we are still on progress to migrate realm to 10.28.0


Show conflict marker in PR list

We already show the CI and review status in the PR overview. It would be nice to also have a marker that shows if a PR is currently blocked due to unresolved merge conflicts.

You want to show a marker on this list, correct?
https://github.com/sindresorhus/refined-github/pulls

We already show the CI and review status in the PR overview

FYI GitHub does that natively, not us.
This can be done via joint GraphQL API call: https://developer.github.com/v4/object/pullrequest/#mergeable

You want to show a marker on this list, correct?
sindresorhus/refined-github/pulls

yes, correct

FYI GitHub does that natively, not us.

I see, it's sometimes hard to figure out what GitHub improved and what comes from the extension 😉

Sounds useful. PR welcome :)

I'd love this. Dependency upgrade PRs often conflict after merging one of them, and it's so annoying to go through the list just to see that many are conflicting and are awaiting an automated rebase.

It looks like this is possible https://developer.github.com/v4/object/pullrequest/#mergeable

I'd like to implement this feature. Already worked a bit on this and I think I should be able to create a PR sometime this week.

Thanks for the PR @Esemesek
Suggestions welcome for the marker style. It should match GitHub’s style. Perhaps it should just be an icon from Octicons


[FLINK-13062] Set ScheduleMode based on boundedness of streaming Pipeline

What is the purpose of the change
The new Blink-based Table Runner needs "streaming pipelines" to be executed with ScheduleMode.LAZY_FROM_SOURCES if all sources are bounded. The current Blink code base uses a global flag for this and configures the StreamGraphGenerator accordingly.
We propose to add an isBounded() property to Transformation (formerly known as StreamTransformation). The property would only be explicitly settable on sources, other transformations inherit the property from their inputs. The StreamGraphGenerator must use ScheduleMode.LAZY_FROM_SOURCES if all sources are bounded, otherwise, it should use ScheduleMode.EAGER, as is the currently existing behaviour.
Brief change log

ccf6f96 refactors DataStreamSource as preparation
03aa30b adds the isBounded() property to Transformation, along with tests for the inheritance logic
277c947 adds a new public method on StreamExecutionEnvironment for adding bounded sources
c37ffb9 changes StreamGraphGenerator to set the ScheduleMode based on the newly added boundedness property, along with tests

Verifying this change
This change added tests and can be verified as follows:

new tests in 03aa30b for the isBounded() property on Transformation
new tests in c37ffb9, i.e in StreamGraphGeneratorTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes, there is a new public API method for adding bounded sources
If yes, how is the feature documented? documented in Javadocs, as for now this is more of an under-the-radar featre

This is an alternative proposal for #8931
cc @ifndef-SleePy

Wow, It's amazing that you have already finished this proposal!
I have taken a glance at it. I got your point. And I believe it could make planner work for 1.9. Furthermore I need to discuss with the guys who are responsible for planner implementation to make sure it could satisfy their requirements for both 1.9 and the further. Since it's midnight in my timezone, I guess I can't give you feedback until tomorrow :(

Sure, please take time to sleep! 😃

Thanks @aljoscha . I think it is ok for BlinkPlanner to choose a proper schedule mode  in 1.9.
I have a little question about how planner set the configuration of runtime.  Sometime a planner might need directly configure the engine to schedule the Job in a specific way. For example there are bounded sources but the planner want use the third scheduling strategy.
My point is we might need a common way to let the planner configure how the engine is execute in the future.  What do you think ?

Thanks @aljoscha .  I think boundedness of streaming Pipeline is not related to the streaming/batch behavior. A bounded source(eg. Collection source) can run not only batch job, but also streaming job.

Another alternative would be to set the ScheduleMode based on the ExecutionMode, which is already configured in the ExecutionConfig.


gcloud playground

Crazy idea, but I think it'd be great if we had a "gcloud playground" where users could basically hit our docs in their browser, select a method, give it arguments, and see the results. Like a gcloud version of the API Explorer sections of the gcloud docs: https://developers.google.com/apis-explorer/#p/storage/v1/storage.objects.list
This would be a bit tricky to implement, mostly around having a server somewhere that would process these requests, as well as the user/project authentication process. Though, if we can ship a browser version of gcloud, this should become a lot easier.
Just wanted to log the idea if we ever choose to explore it further.

It's a cool idea. We could use 3LO to make this work... See the "enable OAuth2 switch" thing in Google-hosted docs.
It might be worth kicking around, but I think way lower in priority than getting Cloud Search and others out the door.

:+1: :mag:

I think this is a bit too optimistic of a goal. We've been linking from our docs to the official docs, which has its own "playground" for a user to experiment with the raw API response.


Calculate timestamp for washer delayed start

Address issue #73
by Introducing a calculated sensor that calculates timestamps for estimated Start/End times when the delay function of the washer is used.

Looks good! Could you kindly resend the collaboration invitation?
I have fixed all pre-commit issues locally.

Looks good! Could you kindly resend the collaboration invitation? I have fixed all pre-commit issues locally.

Absolutely - Done.

@thecrisb I adjust some things 😄
Could you test the updated code? If you confirm everything works, I'll merge this PR.
Thanks for your contributions! 👍

Its working here, found a couple of typos and removed a note i left for myself whilst  working on it.


Update Demo App

[ ] Use https://flutter.github.io/samples/web/material_3_demo/#/ for the demo app
[ ] make it work in both iOs ad Android
[ ] Test the app on a real ios device
[ ] Test the app on a real Android device

https://github.com/flutter/samples
https://github.com/flutter/samples/tree/main/material_3_demo
Need to identify the license associated


ddms0.1.7-alpha

Minor changes:
ddms\classes\command\AssignToResponse: Minor refactor, added newline to beginning of each new assignment.

ddms0.1.7-alpha
Minor changes:
 ddms\classes\command\AssignToResponse: Minor refactor, added newline to beginning of each new assignment.


Add language assistance

Description
This PR adds language assistance helpers (like Hemingway) to the MarkdownEditor component.
Visual changes

Functional changes

[x] Code style has been fixed by running the composer cs command.
[x] Changes have been tested to not break existing functionality.
[x] Documentation is up-to-date.

Just to note, I removed inline styles (replaced with tailwind classes) shortly before recording the demo but didn't build the stylesheet before recording. The dots are circular with a re-build of the stylesheet. In case you noticed and it bugs you as much as it does me.

To be honest, I am a little overwhelmed with having to maintain something like this

I originally thought that your package would be doing a bit more of the heavy lifting and it was just a case of requiring it

Would it be better if that (roughly) 120 lines of component JS was moved to another lib? I suspect the template stuff is equally to blame for the apprehension; but I don't know how I could package that separately in a way that would keep it just as tightly coupled as it is now.

Yeah, is there a way you can bundle a CM plugin? Maybe you can inject the HTML through that plugin also? I'm not familiar there

I think there's still be significant stuff that needs to be elsewhere (templates, alpine state etc.)

Ok, I'll make another plan. Disappointing amount of work making it this far down a dead end road though. :/

Yeah, I really am sorry about that. Hopefully you can see it from our perspective too though. I'm happy to include it in core as long as we're not the ones responsible for maintaining that code.

Yep, no worries. Thanks for your time.


issue[122] Add username customization

What this PR does / why we need it:
1、Solve the issue where [122] the user name cannot be customized
2、The readme file has been updated and this modification is backward compatible.

@opencmit2 Please sync the main branch to your repo and retry the action.


DSCResKit url moved to attributes

Since the CLA hasn't been signed in over 6 month for this PR I'm going to close it out at this point.  If the author would like to sign the CLA feel free to open up the PR again.

@tas50 which CLA needs to be signed?


site_diff: Relative URLs with  tag not supported

Some websites use a <base href="http://www.example.com/" /> tag to specify the base for relative URLs. The site_diff script currently generates wrong URLs in these cases (the relative path is appended to the current URL of the page).

Good catch! That's a tricky one. What's your expectation of that behavior?
I figure the way it would have to work is to set the prefix here:
https://github.com/bslatkin/dpxdt/blob/master/dpxdt/tools/site_diff.py#L141


[FEATURE REQUEST]: Implement Sliding Navbar for Mobile View

Is your feature request related to a problem? Please describe.
Currently, the mobile view of the website lacks a sliding navbar feature. Users cannot easily navigate the site due to the absence of a responsive navigation solution, which can lead to a cumbersome experience when accessing different sections.
Describe the solution you'd like.
Implement a sliding navbar that appears from the left side of the screen when the user clicks on the hamburger icon in mobile view. This functionality should provide an intuitive and seamless navigation experience for users on mobile devices.
Add any other context or screenshots about the feature request here.
Implementing this feature will significantly improve the usability of the site on mobile devices, allowing users to navigate more easily and efficiently.

@Harshdev098  Please assign this issue to me under gssoc-ext level3 and hactoberfest-accepted, as I am interested in working on it.

@PrityanshuSingh What is the tool you are using to check the responsiveness?

@PrityanshuSingh What is the tool you are using to check the responsiveness?

It's a chrome extension, viewport resizer. It's quite handy.

@Harshdev098 are there any conflicts or necessary changes you want me to include in the PR in order to get it merged.

Will let you know more till night

Will let you know more till night

@Harshdev098 Should I generate a new PR regarding this issue before moving to other issues?

@Harshdev098 if there are any conflicts with the existing codebase, kindly inform, i will generate a new PR accordingly.


terraform_0_15: init at 0.15.0

Motivation for this change
https://github.com/hashicorp/terraform/releases/tag/v0.15.0
Things done

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

[x] NixOS
[x] 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 nixpkgs-review --run "nixpkgs-review wip"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119495 at e18275de run on x86_64-linux 1

1 package built successfully:

terraform_0_15

5 suggestions:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/networking/cluster/terraform/default.nix:143:9:
    |
143 |         ./provider-path.patch
    |         ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/networking/cluster/terraform/default.nix:163:17:
    |
163 |     patches = [ ./provider-path.patch ];
    |                 ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/networking/cluster/terraform/default.nix:155:17:
    |
155 |     patches = [ ./provider-path.patch ];
    |                 ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/networking/cluster/terraform/default.nix:171:17:
    |
171 |     patches = [ ./provider-path-0_15.patch ];
    |                 ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/networking/cluster/terraform/default.nix:144:9:
    |
144 |         (fetchpatch {
    |         ^


Priority for Active Messages, and perhaps timers

Would be cool to be able to send messages with a description and priority and show the active message with the highest priority. Having a timer that goes back to the previous description would be helpful too.

No more messages, so this is obsolete.


Switch to ember-d3 now that ember-cli-d3-shape is deprecated

It's coming time to retire ember-cli-d3-shape and use ember-d3 once we cut a new release. I've opened this PR to test compatibility with the new module resolving behaviour before calling it Production Ready™.
Update 10/17/16
Tests run, everything seems to be good to go. Can someone confirm this in another app?

[ ] Confirmed in a parent app

@ivanvanderbyl thanks so much! yes, I'll confirm in a parent app.

alright, works great! Merging.


How does the simulation work with time?

I was exploring the simulation.tick function and found that it used no concept of delta-time while applying physics. I was curious about how d3-force-3d tackles this. I also explored the d3-timer which is the internal timer used by the library. Can anyone explain to me how d3-timer helps get around this? If I understand it correctly, d3-timer runs a timer that matches the frame rate. If my browser runs on 60fps, the tick function inside d3-timer gets called 60 times every second. If the frame rate changes, calls to the tick function will change, affecting the speed of the simulation accordingly. I don't seem to understand how this is prevented. Can any please help me with this?

@Laytoder thanks for reaching out.
The concept of time in this simulation is purely derived from the frame rate. This module makes no attempt at trying to ensure that simulations progress at the same speed across different frame-rates. If there's lower frame-rate than 60fps, the iterations will just take longer to complete, leading into a slow-down of the simulation progress.
So essentially, the simulation just runs as fast as it can given the existing browser conditions. This is intrinsic to the design of this module. You are also correct that d3-timer uses frame-rates, not time-deltas as iteration units.

Thank you so much for letting me know :)


change to the relocation notice

please make this change

there should be no space between words and **. Please consider click preview next time if change the .md files
Fixed by https://github.com/dmlc/mxnet-notebooks/commit/5872e3d8d590e4472375f36380d9cb140a926e91


CleanUp all the dbs2 and dls related code.

remove dbs3UploadOnly flag
remove the sql set by the flog
remove DBSUpload, DBSBuffer package.
need to be careful since some of code used outside those packages (i.e. PhEDExInjector)

Ticket #5225 is related

remove dbs3UploadOnly flag
remove the sql set by the flog
remove DBSUpload, DBSBuffer package.
need to be careful since some of code used outside those packages (i.e. PhEDExInjector)

@ticoann unless I'm mistaken it looks like at least the first two of these have been done already in:
 clean up dbs3 configuration code #6003
Which is merged. Did I miss something?

also whilst attempting to remove the DBSUpload I see reference to it in:
dbsFix.py line 13
from WMComponent.DBSUpload import DBSInterface
<snip>
newFiles.append(DBSInterface.createDBSFileFromBufferFile(localFile, processed))

Given that the DBSUpload package is never used, Is this file ever used or can it too be removed?

Alex, I think it's safe to remove dbsFix.py, it was made in 2010 and apparently never used (possible used at some point for Ops or DBS transition only..).

ok here's an interesting one. There are two references to DBSUpload package in the DBS3Buffer Package:
WMComponent.DBS3Buffer.Oracle.LoadBlocksByDAS
WMComponent/DBS3Buffer.Oracle.SetDatasetAlgo
Are these files used at all? I would have assumed that these files would have wrapped their DBS3Buffer.MySQL equivalents as with the other Oracle ones?

... and that's why we want to get this DBS stuff cleaned up, it's a mess! :-(
LoadBlocksByDAS seems to be using the wrong package in Oracle. Checking those two classes, they are a bit different w.r.t. to the amount of arguments basically.
I suggest you investigate a bit further and maybe run a test or two to make sure we won't break stuff.

Yes, any DAO use from the wrong package should be fixed. I am also not really sure if all the DAO we have in the right package are even used at all.

This is another right?
WMComponent.DBS3Buffer.DBSBufferFile
        uploadFactory = DAOFactory(package = "WMComponent.DBSUpload.Database",
                                   logger = self.logger,
                                   dbinterface = self.dbi)
        setDatasetAlgoAction = uploadFactory(classname = "SetDatasetAlgo")

presumably this should be
        uploadFactory = DAOFactory(package = "WMComponent.DBS3Buffer",
                                   logger = self.logger,
                                   dbinterface = self.dbi)
        setDatasetAlgoAction = uploadFactory(classname = "SetDatasetAlgo")

right?

another:
WMComponent/DBS3Buffer/Oracle/AlgoDatasetAssoc.py


Update README.md

changed "data" to "person" in "Good" example under "Use searchable names (part 1)"

I was going to create a PR with the same change but then I noticed this here. Why has it not been merged (and all other approved PR-s)?


Fix doc formatting for Layout/CaseIndentation

The current formatting is a bit off, see here:

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Add transaction expiry slot for pending transactions

Issue Number
Relates to ADP-93 / #1838.
Overview

Adds expiry slot to pending transactions in order to implement transaction TTL.
Expiry slot and time is reported in the transaction history.
At the expiry slot, pending transactions are marked Expired.
Default TTL of 7200 slots is still hard coded.

bors try

@KtorZ I made the API types more consistent (last 2 commits).
ApiBlockReference is just ApiSlotReference with block information added.
ApiNetworkTip is replaced with ApiSlotReference.
The end result is that all references to a slot/time look the same in the API.
The Haskell API types have SlotId and block info nested rather than inline. I hope in future that we can change the swagger spec to match. But for now there are some non-generic Aeson instances to maintain backwards compatibility.

The Haskell API types have SlotId and block info nested rather than inline. I hope in future that we can change the swagger spec to match. But for now there are some non-generic Aeson instances to maintain backwards compatibility.

I pointed that out in my review from yesterday which I apparently didn't submit :| ... I agree that having one flat and the other nested is sort of inconsistent and I'd very much prefer getting rid of this inconsistency but its also neat-picking at this stage. So happy to move forward with that one.

bors merge


Provide a how to for installing Shiny apps

Provide a simple "how to" document for people to install & test their own Shiny apps.
Possibly related to this recent thread on reddit https://www.reddit.com/r/rstats/comments/79aa3s/best_way_to_dockerize_a_shiny_app/

Done here https://github.com/kinow/nz-shiny-server/wiki/Adding-new-R-applications


added "a" on "personl"

Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@EdwardSanglayGarcia - Thanks for opening a PR! Great catch!
I'll get this triaged for review 👀