[BUG] zipFile can't download completely in windows

Describe the bug
A clear and concise description of what the bug is.
zipFile can't download completely in windows
Exception or Stack Trace
Add the exception log and stack trace if available
no error file, can't see what happend, file download more less than upload zip file.
To Reproduce
Steps to reproduce the behavior:
Code Snippet
Add the code snippet that causes the issue.
blobClient.downloadToFile(fileInfoDTO.getFileName())
Expected behavior
A clear and concise description of what you expected to happen.
I want download complete zipFile.
Screenshots
If applicable, add screenshots to help explain your problem.

in azure , upload file is work ,but download file is not  work。
Setup (please complete the following information):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)]
Java version: [e.g. 8]
App Server/Environment: [e.g. Tomcat, WildFly, Azure Function, Apache Spark, Databricks, IDE plugin or anything special]
Frameworks: [e.g. Spring Boot, Micronaut, Quarkus, etc]

in docker, springboot project, tomcat,java8
If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please check out Troubleshoot dependency version conflict article first. If it doesn't provide solution for the problem, please provide:

verbose dependency tree (mvn dependency:tree -Dverbose)
exception message, full stack trace, and any available logs

Additional context
Add any other context about the problem here.
BlobClient blobClient = BlobHelper.getBlobClient(storageConfig.getContainerName(), storageConfig);
GetFileInfoDTO fileInfoDTO = tcFileUploadService.selectUrlByUid2(uid);
// 下载文件
blobClient.downloadToFile(fileInfoDTO.getFileName());
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[Y ] Bug Description Added
[ Y] Repro Steps Added
[ Y] Setup information Added

I hope help as soon sa possible

Thanks for filing this @YLIU356Volvocars.
Would you mind providing a translation for the error message included in the second picture you included and provide a full reproduction of this issue, as what you've shared may be where the problem is happening in your application but doesn't give us any context into what is happening and would be extremely difficult for us to reproduce as it doesn't include application settings, how the blob is stored in the service.

Thanks for filing this @YLIU356Volvocars.
Would you mind providing a translation for the error message included in the second picture you included and provide a full reproduction of this issue, as what you've shared may be where the problem is happening in your application but doesn't give us any context into what is happening and would be extremely difficult for us to reproduce as it doesn't include application settings, how the blob is stored in the service.

I will give me demo, I hope you give me hand.
fileoperation.zip

@alzimmermsft  can you help me?

Thanks for sharing the demo @YLIU356Volvocars.
What is missing that would really help is an example payload used, and if using the SDK to upload the payload that logic as well. Without those it is impossible to root cause this possible problem as it won't be possible to recreate the scenario.
Additionally, in the demo you gave I see that Spring 2.2.x is being used. This version of Spring is older than what is compatible with azure-storage-blob, which may also be causing this issue. I believe only Spring 2.5.x and later are compatible with the dependencies used by the SDK.

Hi @YLIU356Volvocars
Is this still an issue? If so, can you please provide additional information on the kind of data you are using to upload, size, etc.? That would help us in investigating this better. If your issue is resolved, we can go ahead and close this.

Hi @YLIU356Volvocars Is this still an issue? If so, can you please provide additional information on the kind of data you are using to upload, size, etc.? That would help us in investigating this better. If your issue is resolved, we can go ahead and close this.

I think you can provide relevant version instructions for adapting to the SpringBoot version。If version not adapting， more time time is needed to resolve version issues .


Feature Request: Running on Containers

Why is this needed:
Recently, many applications run on containers such as Docker, Kubernetes, OpenShift Container Platform and so on.  But the EXIST is now able to run only on single virtual / physical machine.  If the EXIST could run on containers, many people would be able to use the EXIST in many situations.
And I could have almost installed the EXIST on the OpenShift Container Platform which is one of the Kubernetes distributions.
What would you like to be added:
First, I would like to separate an EXIST server to Web, MariaDB, Redis servers. Next, I would like to modify some softwares to separate servers. Finally,  I would like to add new "Get Started" for running on containers.
I would like to contribute the EXIST software. If you have any ideas, please give me how to contribute this software for running on containers. (For example, I will have to make another EXIST repository to run on containers, and so on..)

@mamoru1112 Thank you for your proposal. I just would like to create a container image because I often hear that users want to run EXIST on containers. I would like to merge your contribution in this repository. I'm looking forward to get your pull request!

@m-jingu Thank you for your reply! I would like to make a new directory "openshift" in this repository, and I will add OpenShift YAMLs to the directory, and I will make a new README.md for OpenShift users.


Remove key derivation function from ed25519-recovery

Since the key derivation that is currently implemented isn't properly specified and will soon be replaced we should get rid of it.

The WASM implementation is in https://github.com/spacemeshos/ed25519-WASM as long as we don't update that library to drop the function SMAPP should be fine.


Tutorial's mistakes

Where did you get the code?

[ ] Downloaded from GitHub
[ ] Downloaded from the Microsoft Graph quick start tool
[x] Followed the tutorial from Microsoft Graph tutorials

Describe the bug
In Russian tutorial step 1 two mistakes in file name:

Создайте новый файл сauth.js и добавьте следующий код.

Создайте новый файл сui.js и добавьте следующий код.

extra C at the beginning of the filename
Next:
On registration app step need next step with add permissions for  Calendars.ReadWrite and MailboxSettings.Read
because by default after registration we have only User.Read permission in new app
in config.js
const msalRequest = {
  scopes: [
    'user.read', // set by default
    'mailboxsettings.read', // not set by default and app crashed with error
    'calendars.readwrite' // not set by default and app crashed with error
  ]
}

sorry for this, not complete, typing errors

I'll report the Russian error to our localization team, thanks for reporting!
On the app registration - you do not need to configure the permissions ahead of time. The app uses dynamic consent to request those permissions, see https://docs.microsoft.com/azure/active-directory/develop/v2-permissions-and-consent#incremental-and-dynamic-user-consent. The app should not be crashing due to these not being configured, there must be something else happening.


Suggestion: Simple Filter Mode

Here is a suggestion: How about a filter mode without buttons and pop-ups? Only input fields underneath row headers, defaulted to “contains”, refreshing table as you type.

Ahem http://mvc6-grid.azurewebsites.net/Grid/FilterModes

Thanks for your reply, interesting. I implemented a global search following the instructions on your website. It filters all columns as you type. I am quite happy with the performance. I would have assumed that filtering a single column would be even faster. No?
Also, I thought it would be nice to have a filter option without pop-ups offering advanced options. Some users don’t need them and feel confused.

On small tables it's ok, but if you have a site with more users and a bit bigger data set it you start feeling it in cpu usage and average rps. It would also not work on inline column filters, because the grid is repainted on every request so user is not able to keep input focus while typing.
As for no filters, I would just hide them through css, because filters can work without them (using enter key).

Many thanks, I appreciate the time you take to explain what’s going on behind the scenes. Hiding the buttons via CSS sounds like a nice solution. Otherwise, I could also upgrade my external search controls to allow for simple filtering.


Is this sentence missing something?

Another editorial comment on your JOSS submission.
On lines 84-87 you write

In the case of mixtures, it is necessary to map a set of continuous mixture components into a lower dimensional representation of exposure using a pre-determined algorithm then estimate a target parameter on this more interpretable exposure.

This is hard to read. I suggest adding ", and" between "algorithm" and "then", so it becomes

In the case of mixtures, it is necessary to map a set of continuous mixture components into a lower dimensional representation of exposure using a pre-determined algorithm, and then estimate a target parameter on this more interpretable exposure.

Updated.


cupy.around gives wrong output for complex argument

cupy.around gives the correct output for real arguments, but incorrect output for complex arguments:
import cupy as cp
import numpy as np

print(cp.around(0.1234567,decimals=4))
print(np.around(0.1234567,decimals=4))

print(cp.around(0.1234567+0.1234567j,decimals=4))
print(np.around(0.1234567+0.1234567j,decimals=4))

Output:
0.1235
0.1235
(0.12350000000000001+0.12350000000000001j)
(0.1235+0.1235j)

Note that I am not giving an cupy array as input.

I'm using version cupy-cuda101==6.5.0

This is considered by-design. We don't guarantee exact output match with NumPy in terms of floating point accuracy. See also: https://docs.nvidia.com/cuda/floating-point/index.html
This is one of FAQ and needs to be documented in https://docs.cupy.dev/en/stable/reference/difference.html


Manage token

Manage token expiration
#367
What does this PR do?

Extended time to expired token basic
When token expired logout user

How should this be tested?

Run make run
Go to localhost:3000
Change time to expire token in 30 in the file jw.js in line 20 for fast a prove expiration token

Take a look at this

This happen when you logout, being in the profile


Restructure integration test scripts

Also removed some old configuration and simplified some duplicate configuration of integration tests.

Description
Changes proposed in this pull request:

Restructure integration test scripts used in Prow jobs
Simplify integration test configuration and remove old configuration

Related issue(s)

/retest


the related videos show more button not work still!

yup

Works for me. Are you using the latest version of the extension? And which browser version?

It only sometimes works for me, but rarely.

It also does not work for me with the extension version 0.4.0 and Microsoft Edge Version 93.0.933.1 (Official Build) dev (64-bit).

newest version of extention and cent browser also newest version

I can reproduce if I navigate by middle click, or by directly entering a video page from the address bar. I have no idea why.


Feature/card members

https://github.com/aQaTL/go-trello/pull/1

Closing in favor of upstream merge


[Concept]: Arrow Functions

This introduces the concept for arrow functions. Looking forward to feedback on my first full concept.
closes #1340

@junedev @SleeplessByte I have made the changes. Please take a look when you get a chance.

@junedev - thank you for the awesome feedback. I will keep the points in mind moving forward.


Could you please take a look at my stackoverflow question?

I followed the exact documentation, but it's still not working after trying numerous things.
https://stackoverflow.com/questions/63470364/nestjs-and-nestjsxautomapper-resulting-in-error-cannot-read-property-plugin-of
Thanks in advance!

Answered. For the Mongoose question, please visit https://discord.gg/sszmzqn for further support. Thanks!


Standard Lambda Package To Handle Reusable Processes

Value Statement
As an engineer
I want to not repeat past work or copy/paste from past work
So that I can effectively develop new features
We would like a tool that can help ensure consistency, expedience, and confidence regarding new lambda creation. Many lambdas share common problems:

Need to get data from SSM
Need to connect to the database
Need to make requests
Need to push to SQS
etc.

Acceptance Criteria
GIVEN Lambda work is required
WHEN doing common operations (ssm params, db queries, post requests, etc.)
THEN we instead use the common package
Checklist

[ ] Package setup as  a lambda layer
[ ] SSM lookup (maintains/refreshes the connection)
[ ] Able to handle db connections
[ ] Able to push to SQS

Constraints

Each functionality is self-contained
Each functionality is only called if used in the lambda (necessary due to infra permission requirements)
Exists as a module we import e.g from notify_common import db

Additional Info/Resources

Out of Scope

Open Questions

Actively working on this. Doesn't need to be a standalone ticket.


No log output on Debian-based distros

Describe the bug
In #1697 we set StandardOutput=null in scripts/rpm/falco.service to avoid logging to /var/log/messages in Red Hat-based distros. In #2242 we moved away from distro-specific unit files and added scripts/systemd/falco-kmod.service. This file has StandardOutput=null.
When running Falco using scripts/systemd/falco-kmod.service on Ubuntu, I'm getting no output in journalctl -u falco-kmod. Removing StandardOutput=null from the unit file fixes the problem.
Based on https://github.com/falcosecurity/falco/pull/1697#issuecomment-1281044891 I suspect the logging to /var/log/messages by default doesn't affect Debian-based distros. I'm not sure how to tackle this but IMO not seeing logs in journalctl is counterintuitive to Debian users. We may want to do one of the following:

Provide a default which works for most common distros.
Provide commented-out alternatives in the systemd unit files and document distor-specific configuration.
Find a different way to handle the Red Hat-specific problem. Do all systemd units without StandardOutput=null log to /var/log/messages on Red Hat-based distros? 😕

How to reproduce it
Run Falco using the provided scripts/systemd/falco-kmod.service on Ubuntu (I used 18.04.6 but I don't think the precise version matters).

Expected behaviour
Logs show up in journalctl.

Screenshots

Environment

Falco version:

Thu Jun 15 14:03:01 2023: Falco version: 0.35.0 (x86_64)
Thu Jun 15 14:03:01 2023: Falco initialized with configuration file: /etc/falco/falco.yaml
{"default_driver_version":"5.0.1+driver","driver_api_version":"4.0.0","driver_schema_version":"2.0.0","engine_version":"17","falco_version":"0.35.0","libs_version":"0.11.2","plugin_api_version":"3.0.0"}

System info:

Cloud provider or hardware configuration:
OS:

NAME="Ubuntu"
VERSION="18.04.6 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.6 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel:
Linux <redacted> 5.4.0-1109-azure #115~18.04.1-Ubuntu SMP Mon May 22 20:06:37 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Installation method:
DEB

Additional context

Hi! thank you for reporting this, we will take a look ASAP!

/cc @leogr  and, if i remember correctly, @therealbobo was the last one looking into this.

@FedeDP, what a memory! I practically opened the same PR some time ago: https://github.com/falcosecurity/falco/pull/2499 If the common opinion is to keep StandardOutput=null, I strongly suggest to document somewhere the reason for it. 😄

Sorry for the delay, I tried just now on an ubuntu machine:
6.2.0-1010-aws #10~22.04.1-Ubuntu

and with StandardOutput=null I can see logs in journalctl. So I would say that each distro has its own behavior.

Provide commented-out alternatives in the systemd unit files and document distor-specific configuration.

Probably this is the most reasonable solution at the moment :thinking:

I was opening a PR to comment on our systemd units but probably the best place is our website, probably we should add something in our systemd section, this is what we know at the moment:

Ubuntu 18.04: remove this line to see logs with journalctl
Ubuntu 22.04: keep this line as it is to see logs with journalctl
RHEL distros: keep this line as it is to avoid polluting /var/log/messages

When running Falco using scripts/systemd/falco-kmod.service on Ubuntu, I'm getting no output in journalctl -u falco-kmod. Removing StandardOutput=null from the unit file fixes the problem.

IIRC, regardless of StandardOutput=null, alerts should directly forwarded by Falco's syslog_output which is enabled by default :point_down:
https://github.com/falcosecurity/falco/blob/b2374b3c196cafb1bc2c2dce3b92b2d6db7af371/falco.yaml#L339-L343
So, I'd expect to see logs show up in journalctl event with StandardOutput=null in any distros. Does it make sense?
cc @Andreagit97 @johananl @FedeDP @therealbobo
If this is not the case, the root cause may be somewhere else, and not a documentation problem :thinking:

I can't reproduce this now on Ubuntu 18.04.6, namely I can see logs in journalctl.
I suspect I may have used the wrong systemd unit name in the first instance 🙈 Since I can't tell for sure, let's assume a mistake on my end. I'll report here again in case I am certain there is an issue.
I'll leave it to you folks to decide whether to close the issue or update the docs just in case.
Thanks for your help anyway!

Hey @johananl, I haven't been able to reproduce it either. I've observed the same behavior (despite a difference in the color of the journald entry) in both 18.04 and 22.04.
To achieve what you reported, we should disable the syslog_output as well, so it could be that when trying different options, one of them was misconfiguring Falco.

Thank you all guys for the help, i will close this for the moment, feel free to reopen if you face new issues :)


update community joining link in README.md

the old link was directing to the google group which is not active anymore and it took me a while to join the community via new link. I hope this new link will minimise the confusion and will help other new contributors too.

Can one of the admins verify this patch?


Autocomplete Issue when deleting text

Hi,
your ComboBox is great and works fine. However i discovered a little issue and i would be very greatful if you could solve that:
I use the combobox like this:
<dotNetKitControls:AutoCompleteComboBox ItemsSource="{Binding myList}"  DisplayMemberPath="myStringItemOfmyList" TextSearch.TextPath="myStringItemOfmyList" SelectedItem="{Binding anotherValue}" SelectionChanged="Method_SelectionChanged" />
The first item of the list is auto selected when starting the program. Also this item is on focus (the text is marked) when starting.
When I now press any key to clear the marked text and search for another string, this new entered character is displayed for a very short moment. After that, the ComboBox jumps back to the initially displayed value.
Also when I delete characters of the current displayed string one after another, the following behavour occurs: Once i deleted enough characters, that more than one string would match on my current input, the ComboBox also jumps back to the initial value.
Hope my description helps. If you need more info, please ask me.
Thanks a lot in advance vor your support.

Hello. Thanks for reporting.
I reproduced the behavior in question, fixed and deployed.
Repro steps
(1)

Select Alanna Cloninger
Select all text
Input al.

(2)

Select Alanna Cloninger
Delete characters with pressing the backspace key
When Alanna, the text changing to Alanna Cloninger


Margin Auto?

Just curious why doesn't margin Auto work? Any recommendations on what to use instead?

Margin auto is in the pre-release  version (0.43.0-rc.4) of react-native. So unless you are using that version it will not work.


Add -no-prompt flag to drive delete

There should be an option to be able to shot onself in the foot. \o/
(Want to purge some Google Drive Photos using drive delete -no-prompt */*/G*.JPG in Google Photos Directory with fish the Friendly Shell)

Hello @fabianhjr! Thanks for reporting this issue and welcome to drive.
So I have always been reluctant to add a --no-prompt flag to a permanent deletion operation and moreoever one with matches because users have posted issues where they used operations without prior knowledge of what it does and then drive trash [paths...], and the remedy to undo it would be usually to drive untrash but with drive delete there is no going back. Let me know what you think after seeing my rationale.
For starters if you want to do this before this issue is decided, you can do so by
$ function yYes { while [ 1 ]; do echo "yes";done; }
$ yYes | drive delete */*/G*.JPG

or lookup Unix utility "yes"

Hello @odeke-em C:
Thanks for the suggestion I will definetly give that a try though I noticed
rm'ing and drive push does delete the server version. Only thing is I am
being rate limited now :(
I understand the rational behind not having this issue; and now that
yes was mentioned I am quite sure that if anyone else wishes this
destructive no-prompt they should be able to archive it by themselves.
Maybe have the no-prompt in drive delete point to this issue and if
someone really wants to bulk delete have them use yes or a bash/zsh/fish
script?
On Wed, Oct 28, 2015 at 12:17 AM, Emmanuel Odeke<EMAIL_ADDRESS>wrote:

Hello @fabianhjr https://github.com/fabianhjr! Thanks for reporting
this issue and welcome to drive.
So I have always been reluctant to add a --no-prompt flag to a permanent
deletion operation and moreoever one with matches because users have posted
issues where they used operations without prior knowledge of what it does
and then drive trash [paths...], and the remedy to undo it would be
usually to drive untrash but with drive delete there is no going back.
Let me know what you think after seeing my rationale.
For starters if you want to do this before this issue is decided, you can
do so by
$ function yYes { while [ 1 ]; do echo "yes";done; }
$ yYes | drive delete //G*.JPG
or lookup Unix utility "yes"
—
Reply to this email directly or view it on GitHub
https://github.com/odeke-em/drive/issues/448#issuecomment-151735852.

Yeah the server side version could easily be put in a .trash folder in the future. However, am still hesitant about enabling --no-prompt for the same reasons that the Google Drive UI doesn't expose the actual delete because that's the line of defence that a user has.

Well, yeah, that's why I opened the ticket with "There should be an option to be able to shot onself in the foot. \o/" Though having the yes information for the Icarus that likes to live dangerously would be enough.

Alright alright. So what would the purpose of --no-prompt in code linking to this issue be?

Explaining why it isn't a feature and what can the user do if he/she
desires to bulk delete a bunch of stuff.
On Wed, Oct 28, 2015 at 2:05 AM, Emmanuel Odeke<EMAIL_ADDRESS>wrote:

Alright alright. So what would the purpose of --no-prompt in code linking
to this issue be?
—
Reply to this email directly or view it on GitHub
https://github.com/odeke-em/drive/issues/448#issuecomment-151757658.

Good reasoning, thanks! PR coming up in a second.

PR here https://github.com/odeke-em/drive/pull/450

Let me know and I'll merge it into master whenever you are ready.

Yeah, sure. If someone is crazy enough to do drive delete --no-prompt they should be able to know why it wasn't implemented and what to do if they really want to live dangerously.

Addressed by https://github.com/odeke-em/drive/pull/450. Thank you for opening and advocating this!

No, thank to you for maintaining this awesome tool. C: Came real handy to declutter stuff.

Thanks for the kind words!


UI Error: Unable to create credentials

Hi 👋 I'm trying the on-prem/server deployment for trains through docker-compose and am getting a Unable to create credentials error when trying to customize my configuration through trains-init. Where would be a good place for me to start to debug this?

Hi @rcox771,
Start by opening your browser's developer tools (F12) and select the Network tab. Now, try to create credentials again and look at the request sent to the server (should appear as the last network request sent) - it should contain a more detailed error.

@jkhenning I think I got it from here. Pretty sure I need to forward 8008 through my vpn. I'll close this for now thanks!


Add [KPHP] prefix to tooltips

No plans to add this

Why? Some hints or warnings are confusing and it's quite hard to recognize who is misleading, the plugin and PHPStorm itself.


TypeError: can't compare datetime.datetime to NoneType

I ran your example, and I got the following error -->

schedule.run_pending()
Traceback (most recent call last):
File "", line 1, in
File "//anaconda/lib/python2.7/site-packages/schedule/init.py", line 462, in run_pending
default_scheduler.run_pending()
File "//anaconda/lib/python2.7/site-packages/schedule/init.py", line 74, in run_pending
for job in sorted(runnable_jobs):
File "//anaconda/lib/python2.7/site-packages/schedule/init.py", line 73, in
runnable_jobs = (job for job in self.jobs if job.should_run)
File "//anaconda/lib/python2.7/site-packages/schedule/init.py", line 368, in should_run
return datetime.datetime.now() >= self.next_run
TypeError: can't compare datetime.datetime to NoneType

Same here, running Python 3.5.2 :: Anaconda 4.2.0 (64-bit)

@adtrombley Can you give some more info on what lines of code came before the run_pending() call? Thanks!

I'm using python 2.7, this must be the problem.
On Fri, Jan 13, 2017 at 5:40 AM, Daniel Bader<EMAIL_ADDRESS>wrote:

@adtrombley https://github.com/adtrombley Can you give some more info
on what lines of code came before the run_pending() call? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dbader/schedule/issues/125#issuecomment-272444944,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACU-QM2DLBwUrAUQQ1OdRbEvfiTSFGSOks5rR37KgaJpZM4LiPhP
.

--
regards,
Austin D. Trombley, MBA
CTO - Random Forest Capital, LLC
cell:<PHONE_NUMBER>

I've also encountered this error.
Looks like, an interrupted job definition chain breaks the module in current session, e.g.:
import schedule
def job():
    print("I'm working...")

schedule.every(10).seconds   #AttributeError: 'NoneType' object has no attribute 'args'

schedule.every(10).seconds.do(job)
schedule.run_pending()  #TypeError: can't compare datetime.datetime to NoneType
# and at this point it's broken until the shell is restarted

The issue would mainly hit those who just wants to play with the module in a python shell.

I'm also getting this error using an ipython notebook

Thanks @KennyRIM, will look into this. I'm not sure when I'll be able to get to it, so if anyone want's to get started with a PR fix I'd appreciate it.

The issue would mainly hit those who just wants to play with the module in a python shell.

IPython runs repr() on job object created in Scheduler.every(), which accesses self.job_func.args (where self.job_func is None at this time) BEFORE it is set in Job.do().
repr() on Job can be called only AFTER Job.do() initializes the Job instance.

I think I'm getting something similar:
Traceback (most recent call last):
  File "/home/jbliss/miniconda3/envs/xonsh/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/home/jbliss/.xonshrc", line 32, in run
    time.sleep(min(schedule.idle_seconds(), nextsched))
  File "/home/jbliss/miniconda3/envs/xonsh/lib/python3.6/site-packages/schedule/__init__.py", line 487, in idle_seconds
    return default_scheduler.idle_seconds
  File "/home/jbliss/miniconda3/envs/xonsh/lib/python3.6/site-packages/schedule/__init__.py", line 150, in idle_seconds
    return (self.next_run - datetime.datetime.now()).total_seconds()
TypeError: unsupported operand type(s) for -: 'NoneType' and 'datetime.datetime'

I assume it's from trying to run the scheduler without having any jobs configured.

Will be fixed in the next release now that #143 is merged.

Hi dbader,
Could you tag your update?

Has been released as part of 0.5.0 on nov 16, 2017

Hi @dbader ,
I had the below error for Schedule 1.1.0 which seems to be the same bug but not sure.

Fatal Python error: none_dealloc: deallocating None: bug likely caused by a refcount error in a C extension
Python runtime state: initialized
Current thread 0x00007f019244c740 (most recent call first):
File "/usr/local/lib/python3.11/site-packages/schedule/init.py", line 661 in run
File "/usr/local/lib/python3.11/site-packages/schedule/init.py", line 172 in _run_job
File "/usr/local/lib/python3.11/site-packages/schedule/init.py", line 100 in run_pending
File "/usr/local/lib/python3.11/site-packages/schedule/init.py", line 780 in run_pending
File "/usr/app/src/./app.py", line 363 in
Extension modules: charset_normalizer.md, _mysql_connector, yaml._yaml (total: 3)

I can't share the full code but its essentially scheduling two different function which each run some API's calls (REST/SOAP) and store the results to MySQL. One runs every minute and one runs every day.
Looking a the logs, the scheduler for the every minute tasks kept running about 30 times before it stopped suddenly. The fatal error is the only clue I was able to find.
schedule.every(1).minutes.do(function1)
schedule.every().day.at("03:30").do(function2)

schedule.run_all()

while True:
    schedule.run_pending()
    time.sleep(1)

I am updating to schedule 1.2 and hopefully it resolved the issue.


Fix to stop interpolation of escaped asterisk

I noticed that I cannot pass an asterisk to --group(using docker run or marathon-lb/run) and get the marathon-lb.py script to find all marathon apps.
This is a possible fix. Other options are set -f in marathon-lb/run or another --all-groups option in marathon-lb.py.

Makes sense. Thanks!


how to customize background and textColor?

Can i customize the background and textcolors?
If i can, how to do it?

See the sample code, you should provide your own implementation of BootstrapBrand.


SQL error while querying the highest available LOD: column pc.lod4multi_solid_id does not exist

Hello,
We are using the importer exporter to load citygml files into 3dcitydb web map.
We have multiple feature types. We have started with buildings and that went well – we were able to import the citygml into 3dcitydb, then export the kml, and load this kml into the 3dcitydb web map. So far so good.
Then we have tried the same with a different feature type, this one of the veg:PlantCover type. Unfortunately it went wrong on the second step: exporting the kml.
We are getting this error:

This seemed odd to us, so we checked our data against the citygml spec – and it checks out, nothing weird to see here. Then we decided to check the code of the importer exporter and came across this snippet:

The geo-domain and the software is pretty new to us, but this code snippet seemed odd to us. If nothing else, it is inconsistent: _multi_surface_id vs multi_solid_id. Notice the missing underscore in the second example, and the error message: pc.lod4multi_solid_id does not exist Hint: Perhaps you meant to reference the column “pc.lod4_multi_solid_id”.
It seems to us that our issue could be caused by this mismatch. Could this be correct? And if so, are there any workarounds / solutions for this problem?
Thank you in advance,
Dominik Kepinski

Thanks for reporting this bug. Yes, as you assume an underscore is missing in the Java code.

Fixed in 843fcc6d5009fe6e4b9d2a3cd128ef28c404752c. You can checkout the master branch and build the Importer/Exporter from source (see here). Please report here if the fix does not solve the problem.

Thank you for quick response!
After building from master i am getting this error:

This probably has something to do with a discrepancy between master branch and the release, so i will have to find out why this is happening first.
I'll keep you posted when i know more.

It works!
You have made us very happy!
Thank you very much :)


Fix RateLimitPerUser on Channel Create/Edit

I noticed you couldn't properly turn off "slow-mode" (RateLimitPerUser) as 0 is considered empty in the struct, I also noticed someone else tried (#811)  to fix this issue and was denied as if (RateLimitPerUser) was not specified it would remove slow-mode entirely. My proposal of a fix includes passing through a pointer instead of the integer itself. It works functionally the same, just with a pointer, It is not the "cleanest" fix but at least it works.
My proposal:
  seconds := 0
  s.ChannelEditComplex("ChanID", &discordgo.ChannelEdit{
  	RateLimitPerUser: &sec, // this is what has been changed
  })

Currently (Non-Functioning):
	s.ChannelEditComplex("ChanID", &discordgo.ChannelEdit{
		RateLimitPerUser: 0, // this will do nothing as it was omitted as it was considered "empty".
	})

Looks like this is abandoned.


multiple levels of keys

Hey!
How can I access multiple levels of keys?
For exemple, here's a shortened json
"{""level1"":{""item1"":""A"",""item2"":""B"",""item3"":[{""item3.1.1"":""AA"",""item3.1.2"":""AB"",""item3.1.3"":""AC"",""item3.1.4"":""AD"",""item3.1.5"":""AE",""item3.1.6"":{""item<IP_ADDRESS>"":2,""item<IP_ADDRESS>"":3,""item<IP_ADDRESS>"":2,""item<IP_ADDRESS>"":1,""item<IP_ADDRESS>"":1},""item3.1.7"":{""item<IP_ADDRESS>"":49,""item<IP_ADDRESS>"":151,""item<IP_ADDRESS>"":49,""item<IP_ADDRESS>"":1162},""item3.1.8"":{""item<IP_ADDRESS>"":2,""item<IP_ADDRESS>"":-1,""item<IP_ADDRESS>"":0,""item<IP_ADDRESS>"":0}},{""item3.2.1"":""BA"",""item3.2.2"":""BB"",""item3.2.3"":""BC"",""item3.2.4"":""BD"",""item3.2.5"":""BE"",""item3.2.6"":{""item<IP_ADDRESS>"":8,""item<IP_ADDRESS>"":6,""item<IP_ADDRESS>"":8,""item<IP_ADDRESS>"":2,""item<IP_ADDRESS>"":5},""item3.2.7"":{""item<IP_ADDRESS>"":36,""item<IP_ADDRESS>"":135,""item<IP_ADDRESS>"":36,""item<IP_ADDRESS>"":1124},""item3.2.8"":{""item<IP_ADDRESS>"":1,""item<IP_ADDRESS>"":-6,""item<IP_ADDRESS>"":0,""item3.2.8."":0}}}],""item4"":1,""item5"":true}
}"
How can i get item <IP_ADDRESS> value?
I've tried
Set json = JsonConverter.ParseJson(json)
Debug.Print json("level1")(1) 'does not work
I also tried, as the example
For Each Value In json("level1")
Values(i, 0) = Value("item1") 'it works
Values(i, 1) = Value("item2") 'it works
Values(i, 2) = Value("item3") 'does not work
Values(i, 2) = Value("item3")(1) 'does not work
Values(i, 2) = Value("item3")("item3.1.1") 'does not work
i = i + 1
Next Value
Thank you!

I changed "Values" and "Value" for sth else and it worked. I dont know why.
valor(i, 2) = valores("item3")("item3.1.1") 'this worked
I'm sorry.

No worries, I'm glad it worked!


stalebot: add no stalebot label for wip issues

I imagine this label will primarily be used for identified owners that may callify the
issue as second priority. These issues may fall into a future
release and should be ignored by the stalebot.
Signed-off-by: Christopher M. Luciano<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Ex. #1430

@cmluciano: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b05a0f2f293c13872f78952f2ae2dda272cf26f1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

/retest

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b05a0f2f293c13872f78952f2ae2dda272cf26f1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-presubmit.sh
b05a0f2f293c13872f78952f2ae2dda272cf26f1
link
/test istio-presubmit

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b05a0f2f293c13872f78952f2ae2dda272cf26f1
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-presubmit.sh
b05a0f2f293c13872f78952f2ae2dda272cf26f1
link
/test istio-presubmit

prow/istio-unit-tests.sh
101d5cbc6fdd7a5460f8a2196c93217bc17b4239
link
/test istio-unit-tests

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
101d5cbc6fdd7a5460f8a2196c93217bc17b4239
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
101d5cbc6fdd7a5460f8a2196c93217bc17b4239
link
/test e2e-bookInfo-envoyv2-v1alpha3

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

/assign @louiscryan @sebastienvas

@geeknoid are you OK with the label ?

/assign @geeknoid

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


map location selection

will probably be similar to toon selection in behind the scenes functionality. Need to make sure location is saved across pages

aaaaand done


Add watch-option

Fixes #1
Some points:

multiple watch-options allowed
when provided, the src-path is watched as well

Happy to hear your feedback

I did the little style tweaks myself, all merged and bumped to 1.0.0. Thanks a lot! :smile:

@Olical sorry for not getting back to this. Great you wrapped it up! Thanks

Not a problem, thanks for the feature!


the genereated ._manifest\spdx_2.2\manifest.spdx.json scaned with trivy got no vulnerbiltiy

when use trivy to scan the image, got some vulnerbiltiy,
but using this sbom tool to getnerated ._manifest\spdx_2.2\manifest.spdx.json, then run  trivy sbom ._manifest\spdx_2.2\manifest.spdx.json
returns
2024-11-04T...       INFO    [vuln] Vulnerability scanning is enabled
2024-11-04T...       INFO    Detected SBOM format    format="spdx-json"
2024-11-04T...       INFO    Number of language-specific files       num=0
._manifest\spdx_2.2\manifest.spdx.json contains about same number od packages as the sbom file genereated by trivy.

Hi @baoj-dfo, thanks for reporting! Could you please provide more details on what are you trying to do, and the expected outcomes?

@baoj-dfo you can send the files through Github Gist and give the links or upload them here!

@baoj-dfo, we triaged this today. We will only be able to take action if you can provide a demonstrable case where the SPDX data generated by sbom-tool is non-compliant with the SPDX spec. We know that we're specifying 2.2 as our SPDX version (instead of 2.2.2), but other than that we believe that our output is SPDX-compliant.
As mentioned previously, please put the non-compressed manifest.spdx.json file into a Gist and we can investigate it from there. Thanks!

@baoj-dfo I've noticed in the files that they are in different versions of SPDX format, our tool generates the SPDX 2.2 version format while the Trivy one is using the 2.3 version format. Maybe that's a issue you should bring to Trivy's team so they can investigate why the result is different for different formats. Also, we noticed that the files field is empty in one of the JSONs. But I'm not sure on how Trivy looks for vulnerabilities or generates the spdx, so you should try opening a issue with them for getting a better support on this scenario.

@baoj-dfo yes we don't have a public tool for that unfortunately, but as I told you, what caught my attention is that your files section on the SPDX is empty for the one generated by our sbom-tool, I don't know how you ran the tool so the json was generated that way, but since I don't know how trivy works, that could be one of the reasons they are not finding any vulnerabilities, because besides that, the only other noticeable difference between the files seems to be the SPDX version.

@baoj-dfo I believe the files section of the SPDX is only populated if you specify the -b or -bl options. It won't detect files within the ncdsapi image.
I could be completely misunderstanding what you are doing, but maybe something like this would get the results you are expecting?
# export the docker image filesystem to a local directory that sbom-tool can scan it
docker container create --name my-build-container apsabacacr.azurecr.io/ncds/ncdsapi:latest
docker export my-build-container -o my-build-container-fs

# generate a sbom using the exported filesystem (files) and docker image (packages)
sbom-tool generate -b my-build-container-fs -di apsabacacr.azurecr.io/ncds/ncdsapi:latest -m . -ps 'aa' -pn 'bb' -pv "cc"

@baoj-dfo ignore me, looks like I misunderstood what you were trying achieve; Sounds like your issue is that the number of spdx "packages" reported by sbom-tool is not consistent with what is reported by other tools, which has nothing to do with the spdx "files".

Our tool relies on Component Detection's library to detect packages. If there is an issue with the packages we surface versus other tools, you would be better served filing an issue with CD here: https://github.com/microsoft/component-detection. If there is something I'm missing that we can address, please feel free to re-open this issue.


feat(consensus): proposer rotates across validators

#2405  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

This change is

Merge activity

Dec 3, 4:31 AM EST: A user started a stack merge that includes this pull request via Graphite.


infra: Automatically deploy makemake

As requested by @fricklerhandwerk.
Looks like this: https://github.com/ngi-nix/ngipkgs/deployments/makemake
I wanted to implement a fancier deployment via webhooks, but that turned out to be a rabbit hole. Can implement that when we have time. This works for now.

Security note: The secret which is the private key for deployment is scoped to the environment "makemake". This means that "regular" CI jobs don't have access, but only those that are using that environment via jobs.<job_id>.environment. So, let's watch out for that when merging changes to .github/workflows.

@lorenzleutgeb could you please document these details right in the code or contributor/maintainer instructions? Consider these threads ephemeral, no one will ever read them again.


"Unable to reach app. Please try again." When implementing sample 54 task module as a bot skill

🚨 The issue tracker is not for questions 🚨
If you have a question, please ask it on https://stackoverflow.com/questions/tagged/botframework
[question]
I'm implementing sample #54 Teams Task Module as a skill as shown in sample 80.skills-simple-bot-to-bot. I'm using the simple-root-bot in sample 80 to consume the Teams Task Module skill. When I install the root bot app in teams I get this error

I have deployed the skill directly to teams (same code, same App ID) and it works, the problem seems to be when the bot is consumed by another bot as a skill. I have tried both the adaptive card task module and the HTML iframe task module to test whether it could be a problem with "ValidDomains" but it doesn't seem to be because both task modules show the same error.

If there's an error on Teams, there'll be a matching error somewhere in your code. Can you post the error your code is kicking out? In the meantime, I'll try to repro this, see what's going on.

Thank you so much @jwiley84 I tried to look at the error in the code but I can't find the logs of the App Service. I'm probably looking in the wrong place. I enabled all logs in the App Insights. When I got to the Apps Insights page in Azur I can see the activity of the bot but it doesn't report any Failures. Any clues on how or where to see activity logs to find the error code?

Hi @jwiley84  this is the error code I get:
2020-08-05T19:26:28.301Z Err	Unhandled exception. Error: Unable to reach app. Please try again., cause: , stack: new t@https://statics.teams.cdn.office.net/hashedjs/3.2-app.min-7699799.js:3:201585 > {anonymous}()@https://statics.teams.cdn.office.net/hashedjs/3.2-app.min-7699799.js:3:207762 > f@https://statics.teams.cdn.office.net/hashedjs/0-angular-jquery.min-eee9041.js:127:115 > {anonymous}()@https://statics.teams.cdn.office.net/hashedjs/0-angular-jquery.min-eee9041.js:127:345 > {anonymous}() (m.)$eval@https://statics.teams.cdn.office... 2020-08-05T19:26:28.300Z Err	Unhandled exception. : , cause: , stack: {anonymous}(#object,null) > printStackTrace(#object)

Perfect. Let me dig into that and see what's coming up. Usually 502s on Teams based bots mean something is funny with the manifest.json. It might be that the skill needs it's own manifest.json.

Hi @MariaMendietaSpur Just wanted to let you know I haven't forgotten this issue. I'm having some difficulty getting the two (root and skill) to connect. I'm still working on this.

@MariaMendietaSpur Can you post your task module bot file? (ie TeamsTaskModule.cs or TeamsTaskModule.js)

@jwiley84  we made it work. Not sure exactly what was the problem, we started over and tried different things and it worked.
`// Copyright (c) Microsoft Corporation. All rights reserved.
// Licensed under the MIT License.
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Threading;
using System.Threading.Tasks;
using AdaptiveCards;
using Microsoft.Bot.Builder;
using Microsoft.Bot.Builder.Teams;
using Microsoft.Bot.Schema;
using Microsoft.Bot.Schema.Teams;
using Microsoft.Extensions.Configuration;
using Newtonsoft.Json;
using Newtonsoft.Json.Linq;
using Microsoft.BotBuilderSamples.Models;
namespace Microsoft.BotBuilderSamples.Bots
{
public class TeamsTaskModuleBot : TeamsActivityHandler
{
private readonly string _baseUrl;
    public TeamsTaskModuleBot(IConfiguration config)
    {
        _baseUrl = config["BaseUrl"];
    }

    protected override async Task OnMessageActivityAsync(ITurnContext<IMessageActivity> turnContext, CancellationToken cancellationToken)
    {
        var reply = MessageFactory.Attachment(new[] { GetTaskModuleAdaptiveCardOptions() });
        await turnContext.SendActivityAsync(reply, cancellationToken);
    }

    protected override Task<TaskModuleResponse> OnTeamsTaskModuleFetchAsync(ITurnContext<IInvokeActivity> turnContext, TaskModuleRequest taskModuleRequest, CancellationToken cancellationToken)
    {
        var asJobject = JObject.FromObject(taskModuleRequest.Data);
        var value = asJobject.ToObject<CardTaskFetchValue<string>>()?.Data;

        var taskInfo = new TaskModuleTaskInfo();

        //taskInfo.Url = taskInfo.FallbackUrl = _baseUrl + "/" + TaskModuleIds.UploadDoc;
        // SetTaskInfo(taskInfo, TaskModuleUIConstants.UploadDoc);

        //ADDING CARD

        switch (value)
        {
            case TaskModuleIds.YouTube:
                taskInfo.Url = taskInfo.FallbackUrl = _baseUrl + "/" + TaskModuleIds.YouTube;
                SetTaskInfo(taskInfo, TaskModuleUIConstants.YouTube);
                break;
            case TaskModuleIds.UploadDoc:
                taskInfo.Url = taskInfo.FallbackUrl = _baseUrl + "/" + TaskModuleIds.UploadDoc;
                SetTaskInfo(taskInfo, TaskModuleUIConstants.UploadDoc);
                break;
            case TaskModuleIds.AdaptiveCard:
                taskInfo.Card = CreateAdaptiveCardAttachment();
                SetTaskInfo(taskInfo, TaskModuleUIConstants.AdaptiveCard);
                break;
            default:
                break;
        }

        return Task.FromResult(taskInfo.ToTaskModuleResponse());
    }

    protected override Task OnEndOfConversationActivityAsync(ITurnContext<IEndOfConversationActivity> turnContext, CancellationToken cancellationToken)
    {
        // This will be called if the root bot is ending the conversation.  Sending additional messages should be
        // avoided as the conversation may have been deleted.
        // Perform cleanup of resources if needed.
        return Task.CompletedTask;
    }

    protected override async Task<TaskModuleResponse> OnTeamsTaskModuleSubmitAsync(ITurnContext<IInvokeActivity> turnContext, TaskModuleRequest taskModuleRequest, CancellationToken cancellationToken)
    {
        var reply = MessageFactory.Text("OnTeamsTaskModuleSubmitAsync Value: " + JsonConvert.SerializeObject(taskModuleRequest));
        await turnContext.SendActivityAsync(reply, cancellationToken);

        return TaskModuleResponseFactory.CreateResponse("Thanks!");
    }

    private static void SetTaskInfo(TaskModuleTaskInfo taskInfo, UISettings uIConstants)
    {
        taskInfo.Height = uIConstants.Height;
        taskInfo.Width = uIConstants.Width;
        taskInfo.Title = uIConstants.Title.ToString();
    }

    private static Attachment GetTaskModuleAdaptiveCardOptions()
    {
        // Create an Adaptive Card with an AdaptiveSubmitAction for each Task Module
        var card = new AdaptiveCard(new AdaptiveSchemaVersion(1, 2))
        {
            Body = new List<AdaptiveElement>()
                {
                    new AdaptiveTextBlock(){ Text="Upload Document from Adaptive Cardio", Weight=AdaptiveTextWeight.Bolder, Size=AdaptiveTextSize.Large}
                },
            //Actions = new[] { TaskModuleUIConstants.UploadDoc}
            //ADDING CARD
            Actions = new[] { TaskModuleUIConstants.AdaptiveCard, TaskModuleUIConstants.UploadDoc, TaskModuleUIConstants.YouTube }
                        .Select(cardType => new AdaptiveSubmitAction() { Title = cardType.ButtonTitle, Data = new AdaptiveCardTaskFetchValue<string>() { Data = cardType.Id } })
                        .ToList<AdaptiveAction>(),
        };

        return new Attachment() { ContentType = AdaptiveCard.ContentType, Content = card };
    }

    private static Attachment CreateAdaptiveCardAttachment()
    {
        // combine path for cross platform support
        string[] paths = { ".", "Resources", "adaptiveCard.json" };
        var adaptiveCardJson = File.ReadAllText(Path.Combine(paths));

        var adaptiveCardAttachment = new Attachment()
        {
            ContentType = "application/vnd.microsoft.card.adaptive",
            Content = JsonConvert.DeserializeObject(adaptiveCardJson),
        };
        return adaptiveCardAttachment;
    }
}

}
`

I'm very glad you got it to work! Thank you for the update. I'm going to keep this open until I can repro the issue and then solve it, so that future devs can learn from the knowledge.


Add support for collecting and plotting using detection fractions to sinter

Detection fractions are directly measurable on hardware, and less sensitive to the details of the noise model, so they make a much better X axis than "physical noise parameter".

The thing that makes this tricky is there's not an obvious place to put it in the CSV data. If it's stored as a detection fraction column then it doesn't aggregate trivially. If it's stored as a pair of num_detector and num_detection_event columns, then the fraction is massively oversampled compared to what is required, doing unnecessary work. It's also likely for there to be a lot of variations on this concept, e.g. focusing on the detection fraction of specific classes of detectors or including/excluding discarded runs or failed runs in the computation.
A very flexible non-commital thing to do would be to add a separate "collect_det_fraction" command that produced a CSV file that could be joined with the main CSV file by the plot command using the strong id as the key.


Add gcloud image!

Fixes:
Related:
Pre-review Checklist

[X] IMPORTANT: 'image-request' tag has been applied if this PR is adding any images, including new versions or variants

For new image PRs only
If you have an apko.yaml file in this PR you need to follow this checklist, otherwise feel free to remove.

[X] Image is marked experimental or stable as appropriate
[ ] The last two minor versions are available
[X] The latest tag points to the newest stable version
[ ] There is a dev tag available that includes a shell and apk tools (by depending on 'wolfi-base')
[X] The image runs as nonroot and GID/UID are set to 65532

[X] Alternatively the username and GID/UID may be a commonly used one from the ecosystem e.g: postgres
[ ] See above for exceptions to nonroot rule

[X] ENTRYPOINT

[X] For applications/servers/utilities call main program with no arguments e.g. [redis-server]
[X] For base images leave empty
[X] For dev variants set to entrypoint script that falls back to system

[X] CMD:

[X] For server applications give arguments to start in daemon mode (may be empty)
[X] For utilities/tooling bring up help e.g. –help
[X] For base images with a shell, call it e.g. [/bin/sh]

[X] Consider where and how the image deviates from popular alternatives. Is there a good reason and is it documented?
 Let's make this excellent. Include usage example.
[X] Error logs write to stderr and normal logs to stdout. DO NOT write to file.
[X] Include tests, at the very least a basic smoke test.

Thanks!


Sharing pages from Firefox only shares their title

Reproducible on 0.7.11 (TestFlight).
Using the "Share Page With..." feature in Firefox only shares the title of the page. The URL is not included in the message.
Downgrading to riot-ios 0.7.8 shows that it's working fine there (both page title and URL are included in the message).
So this seems to be some regression that happened after 0.7.8.
If sharing is done from Safari, it works identically in both 0.7.8 and 0.7.11 -- only the URL is part of the message (not the page title). Maybe that's a limitation caused by the payload that Safari delivers to riot-ios. In any case, it's the URL that is important, so Safari is fine.

Unfortunately the issue is still present in 0.7.12, basically no sharing of any URLs from apps are working. They only produce an empty line at the receiver.
Was working fine prior to 0.7.11 ... we use it a lot and would be happy to see it fixed again.


PYP 23/24 Sem 1 Q9

Hi all!
I was looking at the PYP video and for the thenApply(Task task) method, is there a reason why this return is valid?
return UOpOrCF.applyB.... should return a CompletableFuture but the return type of thenApply(Task task) is Task. Is there something that I'm missing?

Thanks in advance!

prof edited it in the last minute of the video


N-01 Redundant Code

Issue by Open Zeppelin team:
Multiple instances of redundant code were identified in the OUSD contract:
The initialize function cannot be executed since the contract was already initialized in previous versions. To avoid unnecessarily increasing code size, consider removing the function or commenting it out for future reference.
In the delegateYield function, checking both the yieldFrom/yieldTo mappings and the rebaseState mapping is redundant, as the first check will only pass if the second passes, and vice versa. Consider removing the first check, as it involves more storage reads than the second one.
In the undelegateYield function, there is no need to set the credit balance of the delegation source, as it was already set to their balance within delegateYield.
Consider removing these redundancies to enhance the clarity and efficiency of the codebase.
Origin comment:

We require the initialize function for any future fresh token contract deploys that would extend OUSD.sol
We recognize the redundancy of code in the delegateYield function and have intentionally written in that way in the spirit of code readability and defensive programming. The exaggeration of verbosity is in place to prevent any current or future changes that could introduce an error in this critical part.
Redundant storage slot write in undelegateYield has been removed

Perhaps we just make no changes on this one? Not sure it's worth it.


Problem when changing the type of table-per-hierarchy entry

I am building an example of using the table-per-hierarchy (TPH) feature in EF Core. I have a base class called Book and at derived class called BookWithPromotion. I want to update an entry to change from one to the other, i.e. I want to change an entry that is currently a Book type to a BookWithPromotion type. But when I do that I hit the "...cannot be tracked because another instance of this type with the same key is already being tracked" issue - see issue #7340.
I through I had found one way round this by reading the original entry with AsNoTracking(), building the new entry and use Update. (not a nice solution, but possible). However, this causes another problem - see issue #7510.
Exception and stack trace
The instance of entity type 'BookPromotion' cannot be tracked because another instance of this type with the same key is already being tracked. When adding new entities, for most key types a unique temporary key value will be created if no key is set (i.e. if the key property is assigned the default value for its type). If you are explicitly setting key values for new entities, ensure they do not collide with existing entities or temporary values generated for other new entities. When attaching existing entities, ensure that only one entity instance with a given key value is attached to the context.
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.IdentityMap`1.Add(TKey key, InternalEntityEntry entry)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.StartTracking(InternalEntityEntry entry)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.InternalEntityEntry.SetEntityState(EntityState oldState, EntityState newState, Boolean acceptChanges)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.EntityGraphAttacher.PaintAction(EntityEntryGraphNode node)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.EntityEntryGraphIterator.TraverseGraph(EntityEntryGraphNode node, Func`2 handleNode)
   at Microsoft.EntityFrameworkCore.DbContext.SetEntityState[TEntity](TEntity entity, EntityState entityState)
   at TryEfCoreTHP.UnitTests.TestThp.WriteBookPromotion() in C:\Users\Jon\Documents\Visual Studio 2015\Projects\TryEfCoreTHP\src\TryEfCoreTHP\UnitTests\TestThp.cs:line 50

Steps to reproduce
The following Unit Test fails on the second call to SaveChanges.
[Fact]
public void UpdateBookToPromotion()
{
    //SETUP
    var inMemDb = new SqliteInMemory();

    using (var context = inMemDb.GetContextWithSetup())
    {
        var book = new Book {Title = "Book", Price = 2};
        context.Books.Add(book);
        context.SaveChanges();

        //ATTEMPT
        var bookProm = new BookPromotion
        {
            BookId = book.BookId,
            Title = book.Title,
            OldPrice = book.Price,
            Price = 1,
            PromotionText = "This promotion"
        };
        context.Books.Update(bookProm);
        context.SaveChanges();

        //VERIFY
        context.Books.Single().GetType().ShouldEqual(typeof(BookPromotion));
    }
}

The classes are:
    public class Book
    {
        public int BookId { get; set; }
        public bool HasPromotion { get; set; }
        public string Title { get; set; }
        public decimal Price { get; set; }
    }

    public class BookPromotion : Book
    {
        public decimal OldPrice { get; set; }
        public string PromotionText { get; set; }
    }

The DbContext is:
public class MyDbContext : DbContext
{
    public DbSet<Book> Books { get; set; }

    public MyDbContext(
        DbContextOptions<MyDbContext> options)
        : base(options) { }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<Book>()
            .HasDiscriminator<bool>("HasPromotion")
            .HasValue<Book>(false)
            .HasValue<BookPromotion>(true);
    }
}

Further technical details
EF Core version: 1.1.0
Database Provider: Microsoft.EntityFrameworkCore.Sqlite
Operating system: Windows 10
IDE: Visual Studio 2015, Update 3.

@JonPSmith Closing this as a dupe of #7340.
Some more details: EF doesn't support type mutation for a number of reasons, one of them being that the CLR doesn't support type mutation--.NET does not let an object that was a Book become a BookPromotion, or vice versa. That being said, the code you posted is obviously not doing this. It is instead attempting to delete an entity with a given identity in the same unit-of-work where a new entity with the same identity is being created. This is what #7340 is about. As with all cases like this, it can, of course be handled by calling SaveChanges to do the delete first, followed by adding the new entity and calling SaveChanges again.

Hi @ajcvickers ,
Thanks for that. Yes, I agree it is a duplicate of #7430. Having seen your comments on #7510 I think the code above is the only way I could achieve mutation. I think my use case is valid use of TPH, but I accept that you may define mutation of TPH as invalid in EF Core.
PS. Two SaveChanges is not an option I would ever use in an example, for obvious reasons.

@JonPSmith What are the reasons for now using two SaveChanges?

Oh, sorry. In my examples I am always aiming for atomic updates - they either all work or all fail. I think that is good thing to get in people's mind.
Also, in my own/client projects, which are all web applications, I never use multiple calls to SaveChanges (other than inside a transaction) - my rule is, if the request contains a database update then one, and only one call to SaveChanges should be done.

@JonPSmith This would be a great place to use a transaction.

Yep - that would be a way round it. Bit heavy though. Is there any view on whether #7340 is going to be changed?

@JonPSmith When you say it's a bit heavy, do you mean because there is more code? In both cases there will be a transaction, the only difference is whether EF creates the transaction for a single SaveChanges call or whether you tell EF to use the transaction for more than one call. So something like this:
using (var context = new MyDbContext())
{
    var book = context.Books
        .Include(p => p.Promotion)
        .First(p => p.Promotion != null);

    book.Promotion = new PriceOffer
    {
        NewPrice = book.Price / 2,
        PromotionalText = "Half price today!"
    };

    context.SaveChanges();
}

Becomes:
using (var context = new MyDbContext())
{
    var book = context.Books
        .Include(p => p.Promotion)
        .First(p => p.Promotion != null);

    using (var transaction = context.Database.BeginTransaction())
    {
        book.Promotion = null;
        context.SaveChanges();

        book.Promotion = new PriceOffer
        {
            NewPrice = book.Price / 2,
            PromotionalText = "Half price today!"
        };

        context.SaveChanges();

        transaction.Commit();
    }
}

Doesn't look too bad to me; only a few lines more code, and very similar in terms of what is sent to the database.
With regard to #7340, we have been thinking about it, but there are ambiguities around the semantics that add complexity because of having two entities tracked at the same time that both have the same identity. For example, if we find another FK that refers to that identity, should it be fixed up to point to the one marked for deletion or the newly added on? It is possible to come up with scenarios where both behaviors would be correct. My best guess at this point is that nothing will change in this area for 2.0, and it may still be that we choose to never do this.

Hi @ajcvickers,
Thanks for a really excellent and helpful comment. Yes, my comment that is was "a bit heavy" was about extra code needed for transactions. I think I might personally use that, but for the examples I am producing for others then it's a bit heavy.
Thanks for the feedback on #7340. It's helpful as I am writing examples and I can avoid/warn people about this.
Thanks again.


Add bulk enable and disable account service

Replaces #534  (Fixes merge conflict.)
from @kalpatel01:
Type: generic-large-feature
Why is this change necessary?
We've run into instances where we have wanted to reconfigure watcher run
intervals to improve performance or even remove watchers that were
causing problems. Currently this requires logging into the server and
physically changing the code
This change addresses the need by:
Provides a Watcher Config page in Settings and an api where a user can
configure the run interval and the active flag
Potential Side Effects:
No known side effects

Coverage increased (+0.3%) to 52.732% when pulling 50fe5dc2cf5b53848e04abba01d2ab2305b1ec13 on Bridgewater-7961_BulkToggleAccounts into 714775b2dde29f840584e2945257ceceb4af71dd on develop.

👍 - we're trying to prep a handful of additional PRs, mostly minor changes, so thanks for helping to resolve these conflicts!

I was wondering why #532 didn't allow for sorting on the Active column.  Looks like it was because this PR turned the Active column header into a submit button.

@monkeysecurity - yes, that's correct.  The active column updates where made along with this feature so we thought it was more appropriate to include in this PR


Fix handling of nullable structs

Fix handling of nullable structs.
Add missing verification tests for the Basic website.

Resolves #3013.

Before fix:
TestStruct: {
  type: object,
  properties: {
    hasValue: {
      type: boolean,
       readOnly: true
    },
    value: {
      $ref: #/components/schemas/TestStruct
    }
  },
  additionalProperties: false
}

After fix:
TestStruct: {
  type: object,
  properties: {
    a: {
      type: integer,
      format: int32
    },
    b: {
      type: integer,
      format: int32
    }
  },
  additionalProperties: false
}

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.61%. Comparing base (30fb944) to head (581ac15).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3015   +/-   ##
=======================================
  Coverage   90.61%   90.61%
=======================================
  Files          74       74
  Lines        2962     2962
  Branches      469      469
=======================================
  Hits         2684     2684
  Misses        278      278

Flag
Coverage Δ

Linux
90.61% <100.00%> (ø)

Windows
?

macOS
90.61% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


the editor freezes while running "npm install" in terminal panel

VSCode Version: Code 1.9.0 (27240e71ef390bf2d66307e677c2a333cebf75af, 2017-02-02T08:31:00.827Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

html-snippets
abusaidm
0.0.18

Bookmarks
alefragnani
0.10.2

path-intellisense
christian-kohler
1.2.0

gitignore
codezombiech
0.5.0

githistory
donjayamanne
0.1.5

jquerysnippets
donjayamanne
0.0.1

vscode-generate-getter-setter
DSKWRK
0.4.2

gitlens
eamodio
2.0.2

vscode-html-css
ecmel
0.0.33

EditorConfig
EditorConfig
0.3.4

json-tools
eriklynd
1.0.2

auto-close-tag
formulahendry
0.3.9

auto-rename-tag
formulahendry
0.0.11

code-sync
golf1052
2.1.2

output-colorizer
IBM
0.0.10

vscode-tfs
ivangabriele
0.5.2

docthis
joelday
0.3.10

Angular2
johnpapa
1.0.2

typelens
kisstkondoros
1.4.1

Angular-BeastCode
Mikael
0.3.2

csharp
ms-vscode
1.6.2

typescript-javascript-grammar
ms-vscode
0.0.18

autotrim
NathanRidley
1.0.6

typescript-hero
rbbit
0.12.0

Spell
seanmcbreen
0.9.1

code-settings-sync
Shan
2.4.3

autoimport
steoates
1.2.2

code-spell-checker
streetsidesoftware
0.16.0

gitflow
vector-of-bool
1.0.0

gitblame
waderyan
1.3.0

bootstrap-3-snippets
wcwhitehead
0.0.9

vsc-angular-cli
web-dave
0.1.1

JavaScriptSnippets
xabikos
1.4.0

html-css-class-completion
Zignd
1.0.3

Steps to Reproduce:

Open terminal panel
The terminal should point to powershell.exe
run npm install
the editor goes into no responding state
after reopening again it loses the state of the previos opened windows

@ivanstefanov does this happen if you run it within cmd?
@daviwil have you experienced anything like this?

Nah, I just ran npm install from PowerShell inside of VS Code's terminal UI and it completed successfully:
[daviwil/dsc-breakpoints +0 ~2 -0 !] C:\dev\vscode-powershell
> npm install

><EMAIL_ADDRESS>postinstall C:\dev\vscode-powershell
> node ./node_modules/vscode/bin/install

Detected VS Code engine version: ^1.7.0
Found minimal version that qualifies engine range: 1.7.1
Fetching vscode.d.ts from: https://raw.githubusercontent.com/Microsoft/vscode/1.7.1/src/vs/vscode.d.ts
vscode.d.ts successfully installed!

npm WARN enoent ENOENT: no such file or directory, open 'C:\dev\vscode-powershell\node_modules\tar\package.json'
npm WARN enoent ENOENT: no such file or directory, open 'C:\dev\vscode-powershell\node_modules\sshpk\package.json'

[daviwil/dsc-breakpoints +0 ~2 -0 !] C:\dev\vscode-powershell
>

There was a recent report of long running processes freezing VS Code after 30 seconds on Windows as well, might be related?

I couldn't reproduce the issue on my other PC, which was with SSD. and way more faster. It may be because of the issues @Tyriar  talked about before. Thank you for the fast response


Implement Lamport Scaler Versions

Import version numbers as (Lamport Scaler, Replica ID).

This is complete except that the Replicas need to modify the
LamportVersion.counter[replica.id].value = latest_seen_value
When a version comes in that has a version number higher than their current counter (e.g. to implement the Lamport Scalar clock.
Unfortunately we haven't not implemented this in order to record global staleness numbers.


How to invoke Up function inside a legacy Lambda?

Prerequisites

[x] I am running the latest version. (up upgrade)
[x] I searched to see if the issue already exists.
[x] I inspected the verbose debug output with the -v, --verbose flag.
[ ] Are you an Up Pro subscriber?

Description
I'm trying to invoke my Up function inside a legacy Lambda function:
var invokeInp lambda.InvokeInput
invokeInp.SetFunctionName("up-function")
invokeInp.SetPayload(...)
// ... what else?

lambdaS.InvokeWithContext(ctx, &invokeInp)

Specifically I would like to invoke POST https://<up-function-url>/staging/users, is it possible?
I'd assume for Up case we need to pass some context to InvokeInput but i can't figure it out.

Thanks for clarify :+1:


swift support broken

Starting Visual Studio 8.10 on MacOS the swift libraries are no longer included in the created IPA.

I'm having the same issue.

@visuall @caldrian how can you reproduce that?
Is the dylibs inside the .app/Frameworks folder or the SwiftSupport inside the IPA that is missing?
Are you using the Archive for Publishing or the "generate IPA" build option?

@Flash3001 , I am using Archive for Publishing. The dylibs are not in .app/Frameworks.

Looks like Xamarin now removes Headers folder in each framework during the build process. The CheckSwiftVersionsTask does not detect a swift header and thinks the framework isn't depending on swift libraries. The workaround I came up with is to apply "true" value to the _ShouldIncludeSwiftDylibs variable inside the target and not re-apply it in Output tags. Take a look at modified file: https://gist.github.com/ololomax/1e72c65c5d69ae092c22c52aa2b45f73 (you should replace your packages/Xamarin.Swift.1.0.7/build/xamarinios/Xamarin.Swift.targets with this one).

My support for this project has been awful lately, sorry about that.
I just published @ololomax changes as 1.0.8. That means that even if you target a version of iOS that already includes the Swift runtime the files will still be included in your bundle.
I was having a hard time trying to reproduce the issue. Turns out the X-Swift.h headers only happens on Release builds.
My goal for now was to only provide the workaround so you don't need to keep changing the Nuget download folder.
I still need to stop and think about what to do. Microsoft published a package on NuGet with the same functionality (Xamarin.iOS.SwiftRuntimeSupport) and I have yet to test it and see how it behaves alongside this one.
We may have 3 options:
A) Stop this project and delegate all functions to the official Microsoft package using the Nuget dependency feature.
B) Find a workaround to get the required Swift version and continue with the same functionality.
C) Keep the current workaround, which will always include the Swift runtime.


Seek till end then video is restart

Expected behaviour
Actual behaviour
Steps to reproduce
Environment

Browser:
Version:
Operating System:
Version:

Console errors (if any)
Link to where the bug is happening

Any description?


embed html/css into binary

It doesn't embed?

I don't think so. Look into the embed package if you want to do this.


Not asking for email address

When I try to configure the software it is only asking for the API key and the domain(s) but it's not asking for the email address. Also, I do not see the API key in the config file, only the domains. Where is it stored?
Can you give me a sample for the config file so I can configure it manually and make it work?

Thanks

@zaanposni Perhaps this is a regression related to #35?

Hi @LINKIWI,
this issue is caused by a poorly spelled prompt.
In Line81 we are asking the user to specify whether he wants to use an apikey or token.
If something else than "[K]ey" or "[T]oken" is provided, the program does not handle this correctly.
However, @zozidalom provided his credentials there. I cannot really blame him, as the printed text is not very clear to be honest.
I can add a loop which will stick until the user says "[K]ey" or "[T]oken" and ignores anything else?

@zozidalom Here is a screenshot of the wanted dialog, I added an output of the file so you can create it locally until I provide a fix for this issue.

Thank you I was able to configure by putting the keys and values manually into the file.

Thanks!


fix: Copy button overlaps with content

Proposed fix for #782

Added right padding
Handled overflowing text when scrolling to the right

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

#790 is also covered by this fix.

@third this is not working on firefox for me; works on chrome though.

@third this is not working on firefox for me; works on chrome though.

@crowlKats I updated the code to fix Firefox issue. Let me know how it works on your end.

seems the import statement is collapsed now:

@crowlKats Good eye. Here's my latest fix and screenshots. Tested in Chrome, Firefox and Safari.

Hovered version

When you triple click on any of the words in the import statement, only that word is now selected. Previously the entire statement was selected.


Chebyshev polynomials don't work.

Chebyshev polynomials should be drop-in replaceable with Legendre ones. Right now it doesn't work, the curves go crazy pretty quick.

Seems to work with -D instead of D_hat (WHY??) and no fluxes for a while. There is a disparity at boundaries because there should be fluxes.

Maybe has something to do with the fact that a function should be multiplied by 1/sqrt(1 - x^2) when integrating?

Maybe the derivative matrix is not correct?

Maybe the derivative matrix is not correct?


Rm NVTX calls

NVTX calls are currently why we can't use ClimaTimeSteppers#main.

Superseded by #210


feat: add loading indicator to cat-input component

https://github.com/user-attachments/assets/9bcc0f78-fe82-4797-b991-6f7a41bc35cf

moved to #597 due to some github f**kup


Directioners or Members ONLY

LOVE FOR 1D

Do you know what GitHub is for?

@TomSparkLabs why revive 2.5-year-old useless PRs .-.

@abalabahaha In the hopes it gets closed, which it did


Update README.md how to use prerelease

Suggest adding this to the README.md so it is easily dicovered.

This change is

@RamblingCookieMonster/PSDepend small update for the README.md ready for review.


Github set-output deprecation notice

What did you have in mind?
set-output is deprecated:
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
We use it in .github/workflows/release.yml.
Are you trying to fix a problem?
Prepare for deprecation by GitHub.
Any lead on how this feature can be implemented?
Using environment files.

For reference, an example of using GITHUB_OUTPUT environment file.


Intake Subsystem

Hardware:

2x Kraken X60 motors (non-pro)
1x IR Digital Sensor

Details:

Each motor controls a different roller on the intake, The rollers will spin in opposite directions on the top and bottom to bring notes into the robot.
The sensor is an IR receiver. An IR emitter will be sending out a beam of infrared light which will be detected by the receiver. This beam of light will cross through the intake, such that when a note comes into the robot, it breaks the beam and the receiver detects that the beam of light is no longer there. This is used to detect when we have possession of a note.

Desired Behavior:

Button to spin the intake inward to the robot. Press and Hold.
Button to spin the intake outward away from the robot. Press and Hold.
When the sensor detects the robot has a note, stop the intake and prevent it from spinning inwards.

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


Calendar bug

Fixes #2349
Changes: Added a check to make sure a calendar app is present before navigating to the calendar.

@iamareebjamal can you review this PR?

@iamareebjamal done.

@iamareebjamal done


Installing & Running RML Mapper

Hi @mcm104 :
I'm wrapping two (2) things into one issue here:

Letting you know that I've suggested changes to Installing & Running RML Mapper
Proposing that we move this into the repo as a markdown file to see what you think

These instructions are part of the README in the gettingstarted repo: https://github.com/uwlib-cams/rml/tree/master/getting_started I've updated it to include the changes from the Google doc!

@mcm104 your excellent documentation brings a tear of joy to my eye 💯


with_function fails to pass args properly when number of args > 3

see this example: https://github.com/dpzmick/llvm-rs/blob/master/examples/add3.rs
For me, this spits out nonsense.
built against llvm-3.8, will try against 3.9 and update ticket.
I'm still looking for the source of the issue. It doesn't appear to be an issue with calling conventions but I wouldn't mind being proven wrong about that.

it looks like the implementation of get_function() used by with_function() is correct.
llvm-sys doesn't implement llvm::ExecutionEngine::getPointerToFunction() but, looking at the code in 3.9, its basically exactly the same thing as getGlobalValueAddress()

llvm-3.9 does not resolve the issue.
I've also isolated the issue to this crate. The equivalent example using only llvm-sys works correctly.
see: https://bitbucket.org/dpzmick/llvm-sys.rs/src/622927a96d48e6548b8ed71d75115c5ecc19953c/examples/jit-function.rs?at=default&fileviewer=file-view-default

I've found the source of the issue.
Memory layout for tuples is not well defined in rust, so using a tuple in with_function is causing all sorts of havoc.
Doing something like this with the function passed back by with_function fixes the problem:
    ee.with_function(func, |add:extern "C" fn((f64, f64, f64)) -> f64| {
        let f: extern "C" fn(f64, f64, f64) -> f64 = unsafe { mem::transmute(add) };
        println!("{} + {} + {} = {}", 1., 2., 3., f(1., 2., 3.));
    });

I'll start working on a fix


Remove docker insert command

"docker insert command" has been removed.

Looks fine to me.
CC @ aeonazaan @ bobmaerten @bergvandenp
(When you want to contribute to a plugin, please CC the maintainer(s) when applicable.)

:+1:

OK thanks @bobmaerten.
@waynelkh I want to add this to #3809 but your fork seems unavailable. Care to submit the same PR to my branch?

@ncanceill I had submit a PR to you repo
Please check it, thanks!


Line isn't graphed properly for specific values

Given the array, [0.6137386018237082, 0.2815539332538736, 0.17518342474101156, 0.24931948665991222, -1.1515280464216635, 1.1460104011887073, 0.2818422889043964, -1.3491616766467065, 1.5056442831215968, 0.6401420838971583, -56.699696969696966], of values, I get the following graph

Some content is clipped off. I looked at the source code and it has something to do with step height.

Hi, check out 1.4.5 brand new jsut for you fixed the clipping please test it!


[FLOC-1476] Sketch of design: ZFS "volume" deletion.

https://clusterhq.atlassian.net/browse/FLOC-1476

Thanks @itamarst
This looks good.

I liked it when you included proposed tests in your previous designs, but I guess you're intending to do the implementation here, so not necessary.
Will there need to be handling of the case where we attempt to delete a mounted / in-use dataset?
Or are we going to assume that the convergence agent ensures that applications are stopped and datasets unmounted before making any attempt to delete?

Anyway, please proceed or add some further test ideas if you'd like someone else to write the code.

This isn't even on the level of convergence, this is just a low-level operation. The convergence agent (next issue queued up after this one) will ensure stuff is detached, yeah.

Thanks @itamarst
This looks great.

Tests pass on buildbot.
Coverage is complete

Please merge.
PS I was a bit puzzled by the coverage change report:

http://build.clusterhq.com/results/change-coverage/zfs-deletion-FLOC-1476/flocker/volume/service.py%2Ccover

...which seems to have got mixed up with the changes to flocker.volume.service from another branch.
      # XXX or maybe caller should just do that directly?
      def destroy(self, volume):
          """
          Destroy an existing volume.

          :param Volume volume: The ``Volume`` instance to delete from the
              storage pool.

          :return: A ``Deferred`` that fires when deletion is done.
          """
          #return self.pool.destroy(volume)
!         pass

@tomprince Any ideas?


Create own error message in clap's style

 Issue by LoganDark
Saturday Jul 25, 2020 at 11:40 GMT
Originally opened as https://github.com/clap-rs/clap/issues/2035

Make sure you completed the following tasks

[x] Searched the discussions
[x] Searched the closed issues

Describe your use case
clap emits its own error messages. They look quite nice, they have color, all that good stuff. Maybe I want to take advantage of this feature - perhaps instead of using panic!, or manually emitting a message to stderr and then exiting, I want to emit an error message that matches the ones clap gives.
The main reason for this is consistency. When your error message looks different from clap's, it kind of feels weird to the end user. I've never encountered a utility that has different error messages when parsing arguments than when running the program itself, and it would be weird to make the developer try to mimic clap instead of offering them an API for it.
Describe the solution you'd like
My use case specifically would benefit from a panic!-like solution that emits an error message of my own choosing and then exits. However, that's a very minimal solution and there are better ones, see below.
Alternatives, if applicable
(package names might not be 100% accurate as I'm using file paths)
Really any way to format error messages in the same style as clap would be nice. For example, you could just make clap::output::fmt::Colorizer public. Although, if you do that, you should make clap::parse::errors::start_error public too.
Making these things public would benefit consumers of your API as they'd be able to format their error messages and color their messages the same way as clap does, allowing the application to look more consistent. They could even use the Colorizer for general output.
However, since these things are implementation details at the moment it's possible that yet another solution could be implemented instead, more well-suited to consumers of clap's API. I could come up with many such solutions, but I think it should be up to the maintainers of clap.

 Comment by LoganDark
Saturday Jul 25, 2020 at 20:45 GMT

I think I would wait until after we're done with the refactorings and stuff.

That is a reasonable goal, but you are in the process of refactoring, why not make this change while you can? Major version 3 still isn't fully released, so you have more freedom. Once the version 3 API is stabilized, you can't make any more breaking changes (like renaming that method for example).

 Comment by CreepySkeleton
Saturday Jul 25, 2020 at 20:55 GMT

Because it's not yet decided what the colorizer will look like. Once we're settled on that, we can make it public and document.
If you have any alternative solutions, I'm all ears.

 Comment by epage
Monday Nov 29, 2021 at 14:37 GMT

In clap3, we now have a App::error() (added in #2890) which provides the full clap-style error message, including the usage, recommendation for help, and WaitOnError support (#2967), unlike with_description.  It does not provide a smart usage (adapting to your specific command or to your place in a subcommand hierarchy) and it doesn't expose the coloring API to end users (the clap boiler plate does get colored though).

 Comment by medwards
Friday Dec 03, 2021 at 15:22 GMT

Thanks @epage that works quite nicely for my case at least (not OP).
Small paper-cut for future users: youhave to pre-construct the error or clone their app because once get_matches is called it consumes the entire App.
    let mut app = clap::App::new("app")
        .subcommand(
            clap::App::new("run")
                .about("a silly subcommand but makes my app look nice")
                .arg(clap::Arg::new("COMMAND").required(true)),
        );
    let subcommand_error = app.error(
        clap::ErrorKind::MissingSubcommand,
        "Missing subcommand which wasn't expected. Did you mean 'run'?",
    );
    let matches = app.get_matches();
    if let Some(subcommand) = matches.subcommand_matches("run") {
        subcommand.value_of("COMMAND").expect("command was not provided");
    } else {
        subcommand_error.exit();
    }


Springer styles - new list

Good news: I received last week an updated list from springer about their citation and reference styles. These are information for almost 2.200 journals. I would like to work on that list a little and add the shortTitle as well as the primary language from the other large Springer list (this is connected to https://github.com/citation-style-language/styles/issues/1122). This would enable us to update the Springer journals and give language support (overwrite local). @rmzelle There are also some Dutch journals in the list, e.g. "Jeugdbeleid" (no idea what this word means;). This may take some time until I can do that.
(BTW if you will ever ask Springer about this list, they will say that they do not have such a list, which is partially true, because the list has to be generated from the database [I have the exact query].)

Great, thanks! ("jeugdbeleid" is "youth policies")

Any news on this? Let me know if you need help.

I worked a little in Excel on the list and combined the data with other tables to obtain one main table with all information (I can send you that if you want). I certainly would need much longer for everything in Ruby and I don't have your experience with the creation of dependent styles. Would you like to take it over from here?

Once you have Ruby set up, using the script itself should be rather straightforward. It has an option to replace all dependent styles generated from the "springer" data subdirectory with newly generated styles (ruby generate_styles_from_data.rb --dir springer --replace).
But I can do that as well. Can you share the spreadsheet via email or Google Drive?


Cfbot not working after pushing the application into space

I pushed the application into cloud foundry and while starting the application, i'm getting the following error,

2017-06-03T18:05:17.630+05:30 [APP/PROC/WEB/0] [OUT] info: Ahoy! Attempting to start CF Bot...
2017-06-03T18:05:17.631+05:30 [APP/PROC/WEB/0] [OUT] info: Connecting to Slack using the following token: xoxb-xxxxxxxxxxxxxxxx
2017-06-03T18:05:17.632+05:30 [APP/PROC/WEB/0] [OUT] [Sat Jun 03 2017 12:35:17 GMT+0000 (UTC)] INFO Connecting...
2017-06-03T18:05:17.661+05:30 [APP/PROC/WEB/0] [ERR] error: Connection Error From Slack Websocket
2017-06-03T18:05:17.661+05:30 [APP/PROC/WEB/0] [OUT] [Sat Jun 03 2017 12:35:17 GMT+0000 (UTC)] INFO Reconnecting in 1000ms
2017-06-03T18:05:17.661+05:30 [APP/PROC/WEB/0] [ERR] error: ENOTFOUND

App is getting crashed continuously.

I'm not actively using or maintaining this project anymore. If you are able to resolve this issue, feel free to send a PR?
If you run the application locally, do you get the same issue?


Fix if statement in setup-pre-container.sh

Thanks for your change. Can you fill a CLA otherwise LGTM


mojibake ("é" to "Ã©" etc.) when typing utf-8

Apologies if this is not the true source of the issue i am seeing, but oddly enough it does look like it is.
The issue i am seeing:
When i type "é" in neovim, i get "Ã©". If i save and cat the file i will see "Ã©", so my input was definitely replaced somehow.
❯ nvim testMojibake.text

❯ cat testMojibake.text

Ã©

❯ xxd testMojibake.text

00000000: c383 c2a9 0a

With a file created in nano:
❯ nano testGood.text

❯ cat testGood.text

é

❯ xxd testGood.text

00000000: c3a9 0a

If i paste "é" instead of typing it, it gets written correctly.
So: to reproduce you need to type the character, not copy and paste it, which will be tricky if you have an english keyboard!
Why i think this is from which-key:
These characters are working fine in my terminal, every other app, and i don't see the issue if i start neovim with nvim -u NONE. I proceeded to deactivate plugins one by one. The problem disappears when i deactivate which-key, and reappears if i put it back, with the standard setup from the readme.md.
For what it's worth, my config can be found here: https://github.com/gtnbssn/dotfiles/tree/main/nvim/.config/nvim

I have no problem typing é in nvim but I am using the latest from master branch you did not state wich version you where using.
you can check that you have the correct file encoding
by using this command
set fileencoding
If you whish to set it to utf-8 that can be done same manner:
set fileencoding=utf-8

Thanks for trying to reproduce!
It might be an issue with my configuration, but somehow i only see it when which-key is set up, including when i set it up with the standard config.
I am using nvim v0.6.1 on Manjaro.
fileencoding and encoding are both utf-8. (default value for encoding, and set in my config for fileencoding.)
In the console (or any app, even nvim without which-key) i can type (and paste) é, in nvim with which-key it works if i paste, but if i type i get mojibake.
This is really weird and i am so so puzzled at where this might be coming from. Here's my locale, i believe everything is correctly set (i do have a French keyboard):
❯ locale
LANG=en_SG.UTF-8
LC_CTYPE="en_SG.UTF-8"
LC_NUMERIC=en_SG.UTF-8
LC_TIME=en_SG.UTF-8
LC_COLLATE="en_SG.UTF-8"
LC_MONETARY=en_SG.UTF-8
LC_MESSAGES="en_SG.UTF-8"
LC_PAPER=en_SG.UTF-8
LC_NAME=en_SG.UTF-8
LC_ADDRESS=en_SG.UTF-8
LC_TELEPHONE=en_SG.UTF-8
LC_MEASUREMENT=en_SG.UTF-8
LC_IDENTIFICATION=en_SG.UTF-8
LC_ALL=
❯ localectl status
   System Locale: LANG=en_SG.UTF-8
       VC Keymap: fr
      X11 Layout: fr
       X11 Model: pc105
     X11 Variant: oss_nodeadkeys

I'm on a nordic keyboard so I had to type it in this manner:

´
e

Maybe you could try something like that even if it's a longshot
But like you said if it works when which key is disabled it feels like that is the problem or a combo with  nvim 0.6.1.
This is my version of nvim
nvim -v
NVIM v0.7.0-dev+1086-g1bd6c0a05
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by dev_user@buildkitsandbox

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"

I think i found the cause in my config. Thank you so much! You did put me on the right way!
I found it smart to add these keymaps in my configuration:
local keymap = vim.api.nvim_set_keymap

-- easy way out
keymap("i", "ù", "<Esc>", opts)
keymap("v", "ù", "<Esc>", opts)
keymap("c", "ù", "<C-C>", opts)

I very rarely type ù, and it's a lot easier to reach than esc.
When i comment these out the problem is gone. I need both these keymaps AND whichkey for the issue to show up.
So i guess the issue could be on neovim's side, but it's definitely only showing when i have which-key set up, so there might be something wrong here, i don't know.
I'll leave this open with an updated title for the devs to decide on which side this is.
Thanks!

Ok i think i'm getting close to the bottom of the rabbit hole. The issue would be here:
https://github.com/folke/which-key.nvim/blob/main/lua/which-key/util.lua#L28
  for i = 1, #keystr, 1 do
    local c = keystr:sub(i, i)

And here's something that would explain it all: https://dev.to/samyeyo/lua-strings-are-not-strings-6f4
The code is getting the length of the string and doing a substitution. But it is in fact not really operating on a string and has no idea of encoding. Characters like é will cause issues.

I have a hunch this fix in neovim 0.6.1 is what caused the bug to appear: https://github.com/neovim/neovim/commit/93f1ec0a59ec43e8f66deba0e3e660e0829e74d3

Nicely done!
👍

Unfortunately my patch causes other issues.
Using "v" for visual selection and then arrows triggers this:
E5108: Error executing lua .../pack/packer/start/which-key.nvim/lua/which-key/util.lua:33: invalid UTF-8 code
stack traceback:
        [C]: in function 'error'
        .../pack/packer/start/which-key.nvim/lua/which-key/utf8.lua:128: in function '(for generator)'
        .../pack/packer/start/which-key.nvim/lua/which-key/util.lua:33: in function 'parse_keys'
        .../pack/packer/start/which-key.nvim/lua/which-key/keys.lua:90: in function 'get_mappings'
        .../pack/packer/start/which-key.nvim/lua/which-key/view.lua:250: in function 'on_keys'
        .../pack/packer/start/which-key.nvim/lua/which-key/view.lua:240: in function 'open'
        .../pack/packer/start/which-key.nvim/lua/which-key/init.lua:51: in function 'show'
        [string ":lua"]:1: in main chunk

I guess the arrows are not sending a character code at all.
Is there a more proper way of dealing with UTF-8 in neovim plugins?

Can you check if #275 fixes this?


Data cleanup

https://github.com/rancher/rancher/issues/7049

https://github.com/rancher/rancher/issues/7050

https://github.com/rancher/rancher/issues/7037


fix: multiple lambda s3 trigger event creation

AWS operations need to be applied to the bucket once for all events that use that bucket.

Did a session with @willkelleher and we fixed this PR to work as expected. LGTM - Do we want to require tests for this section of code that was improved? There are none to date.

@sbasgall It would makes sense to do tests cases. (It doesn't need to be 100% coverage.)
It would be better to do them now as the code is fresh in everyone's mind. Going back and doing it later will be much harder, I think.

@sijis @sbasgall added a couple tests