Adds logic to switch_register cli to accomodate disparate inputs

Currently supporting mock, dell and nexus

@knikolla @henn
simple PR, have a look when you can.

@kylehogan please have a look if you have time.

-1. Instead of using *args, you should use **kwargs.

cli should be taking command line arguments hence args is the appropriate thing.
I do not wish to use kwargs as it will compels the command line user to know what keys are there for each switch driver.
About code duplication, I thought about it but I do not think that I can simply it.
I can make the error as a single function that can be invoked if desired input is not received, but that would make it more complex as it will require defining new variables to suit each case.
Also, this is to keep current HaaS code operational.
Once I am done with the client library, the CLI is going to be completely revamped.

@SahilTikale yeah, I backtracked on my kwargs argument after I saw the cmd decorator.

@knikolla  fixed it.

+1

sorry for the late reply, I missed the initial email.
It looks good to me, your decision on how to handle the errors makes sense.
My only comment is that it might be helpful if the error for 'wrong subtype supplied' was instead either 'invalid subtype' or said that subtype must be 'nexus, mock, or powerconnect55xx'
Listing all the subtypes might not make sense if there end up being many of them, but with only three it seems reasonable. This way it is clear that the subtype supplied is not one that is supported rather than 'wrong' which could imply that the subtype was valid, but a different subtype had been expected.

@SahilTikale, I agree with Kyle. We are already having to handle all the switches in a separate code block. So we know what we support and can provide feedback to the user on the correct types.

This logic should be in the switch driver---that way it's extensible.

After some conversation with @SahilTikale, we agreed we don't need to do this nicely... yet.
However, please add a TODO or FIXME comment saying that this current design needs to be changed so that the CLI doesn't need to know about every backend.

(and once you do that, +1)

+1 and merging


[stable/kubernetes-dashboard] Allow HTTP connection

There are situations where the kubernetes dashboard is hosted behind ingress controllers / proxies that perform TLS termination already. Many ingress controllers / proxies (cloudflare argo ingress, oauth2_proxy) expect proxied services to be HTTP, and do not support HTTPS services.
Recommend adding an input value to setup deployment to allow HTTP traffic (which is supported by k8s-dashboard binary itself via --insecure-port, --insecure-bind-address and --enable-insecure-login option.
Reference: On Securing the Kubernetes Dashboard by Joe Beda @ Heptio

I would be happy to review, although I don't have the mighty /approve power that @kfox1111 have as official maintainer. :)
I would use this feature myself as it seems unnecessary behind ingress.

Preventing this from being auto closed by commenting. Any feedback on my PR #9011 ?

Dear bot, please don't.
cc @kfox1111


Intellisense gives "any" type for the original name in a destructuring assignment

interface A {
	x: number;
}

var a: A;
var { x: x2 } = a;

This gives the type "any" when mousing over x

(Not sure what the corresponding services API is. getTypeDefinitionAtPosition ?)
Mousing over x2 is fine - it gives "number". It would be nice if mousing over x gave the same, since it helps with nested destructuring to see the intermediate steps.

Duplicate of #2675.


Change background-image URL

glyphicons-halflings.png is already available inside the bootstrap folder & also, the link to http://twitter.github.com/bootstrap/assets/img/glyphicons-halflings.png is dead! I guess you forgot to update the URL.

Hello,
Please merge the change i made.

I will be back from my vaca in 1 day. WIll check in on this then. Sorry!


Canonical Transcript IDs selection

Hello,
I've been trying to get the same Canonical Transcript IDs that you uploaded in cfDNA/data/Ensembl_v75/
Though I don't come to the same list when using the above lists, to select the Canonical Transcript IDs. :

http://hgdownload.cse.ucsc.edu/goldenPath/hg19/database/knownCanonical.txt.gz
http://hgdownload.cse.ucsc.edu/goldenPath/hg38/database/knownCanonical.txt.gz

Nor do I come to the same list when selecting the longest transcripts.
Would you mind telling me the way you settled for those particular transcripts ?
Did you use particular attributes obtained with BiomaRt to perform your selection ?
Best regards,
MushuW.

Dear MushuW,
Thank you for your message. We are not using files from UCSC here. As you suspect, we used BioMart to retrieve the list of canonical transcript IDs available in our repository. It seems no longer possible to that over the web, only the Perl and R API interfaces seem to provide access to that. Please have a look here for the response of Thomas Maurel on how to do it with the Perl API: http://gmod.827538.n3.nabble.com/biomart-users-How-to-download-all-human-canonical-transcript-IDs-from-Ensembl-version-75-via-BioMart-td4049474.html
Best,
Martin

Dear Martin,
Thank you for your answer.
Your link to Thomas Maurel answer is very useful; new to Perl, I wasn't sure how to proceed.
Best of luck with your work,
MushuW.


ebs piops metric filtering

@kapilt - Currently, there's no way to easily filter provisioned IOPS (PIOPS) volumes that are underused with Custodian.  We should have a filter that allows us to set a threshold for the 'VolumeConsumedReadWriteOps' metric and return volumes that come in below that threshold.  The problem with using 'VolumeConsumedReadWriteOps', is that the metric is a 'Count' metric.  It looks like the denominator needed to calculate the percentage used varies per volume.  Thoughts?
example policy

name: under-utilized-piops
resource: ebs
filters:

type: under-utilized-piops
threshold: 50
op: lt
actions:
notify

make sense, i'm wondering if we can extend the extant metrics filter syntax in a way that we could add calculations against the resource attribute. the proposed syntax in your example is very specific to a given metric and resource type, where as this seems like a common scenario for cost optimization of verifying reserved utilization.

Modifying the metrics filter sounds like a similar effort to just creating this one filter for piops, so it may very well be worth it.

i guess the question is what would it look like on the metrics filter here's a sketch, suggestions welcome
name: underutilized-provisioned
resource: ebs
filters:
  - VolumeType: io1
  - type: metric
     name: VolumeConsumedReadWriteOps
     op: less-than
     value: 60
     period: 7
     # need to specify conversion and resource attribute to use.
     value_type: percent
     value_attr: Iops

that's basically what I had in mind when you suggested modifying the metrics filter to accomodate.  i'll take a look at the code to get this working as time allows over the next few weeks.  thanks @kapilt !

started looking at the code to do this, and can't think of a single valid use-case that may require anything other than calculating percentages based on metrics and a resource attribute.  this would be a much simpler change to the metrics filter, and we can always add other operations in the future if needed.  unless you think otherwise, I'll submit a PR with this in mind.

sounds good.

implemented, some improvements coming when dealing with metric unit mismatches between resource attribute and metric unit.


Add com.parrable/encrypted_payload/jsonschema/1-0-0

Additional to the encrypted and decrypted ID, we also have an optout value (true/false) that needs to be added


JOSS Review - Lorenz

Everything looking fine

[x] runners in place: linting + testing
[x] tests + examples are working as expected
[x] pointwise comparison to reference implementation in tests
[x] crates items are fully documented
[x] bindings to python interface

Two small things

you can include your readme for crates with https://blog.guillaume-gomez.fr/articles/2021-08-03+Improvements+for+%23[doc]+attributes+in+Rust
https://docs.rs/egobox-ego/latest/src/egobox_ego/lib.rs.html#38

Thanks for your review. I've taken into account your points in the last commits of the joss branch.


Homebrew Support

As a macOS user, I’m used to installing tools through Homebrew. Given that lapce already has a lot of stars, I think it should be possible to get its formula merged into homebrew without any issues

you can brew install --cask lapce https://formulae.brew.sh/cask/lapce.

@ghishadow Any chance to install lapce on Ubuntu using homebrew?

@kraemi perhaps it is better to open another issue.


Adding the pre-processing file

@Sanchit120496  Hey, Sanchit, could you please filter "YearsCodePro != 'Less than 1 year' and YearsCodePro != 'More than 50 years' " in the data?


[WIP] Updates to anydata, open records and error detail

Purpose
This PR

Allows structures with pure types (anydata + error) as anydata
Fixes #13231 - (anydata|error)[] and map<anydata|error> need to be considered as anydata

Updates the default record rest field to be anydata|error (pure type)
Fixes #13181 - Default record rest field type should be anydata|error (pure)

Validates that the detail expression specified in the error constructor is a subtype of map<anydata|error>

Closing this PR since changes need to be introduced to the error type, type signature and info, to support detail types containing the same errors, Will open a new PR once the changes are done.


jest-utils: add a getter for process.domain

Fixes #7247

Codecov Report

Merging #9136 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #9136      +/-   ##
==========================================
+ Coverage   64.77%   64.78%   +<.01%
==========================================
  Files         277      277
  Lines       11722    11724       +2
  Branches     2878     2878
==========================================
+ Hits         7593     7595       +2
  Misses       3512     3512
  Partials      617      617

Impacted Files
Coverage Δ

packages/jest-util/src/createProcessObject.ts
74.46% <100%> (+1.13%)
:arrow_up:

Continue to review full report at Codecov.

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

Node 8 is not happy, could you take a look?

@SimenB thanks for the quick reply!
I'm on it, trying to find a non hacky way for solving it

@SimenB done :)

Thank you!


Handle when webpack config erroneously contains an entry point.

Feature Proposal
When the specified webpack.config contains an entry field, this will overwrite the intended entry field before the internal configuration is merged with the passed webpack.config causing karma-webpack to throw an error and fail.
Since this is something that should never be specified as outlined by the README, we should log a warning if the field is set specifying that it should not be set and that it will be ignored.
We should then delete the entry field before calling webpack-merge to ensure a successful run.
Feature Use Case
Prevent unnecessary failures and give the user feedback about invalid configuration.

Just documenting, it looks like the issue that is causing these errors is that
registerExtraWebpackFiles and preprocessorFactory are both called independently with the karma configuration options, but preprocessorFactory is mutating the controller object with the webpack configuration while the other does not, this causes an inconsistency in output state.

Just documenting, it looks like the issue that is causing these errors is that
registerExtraWebpackFiles and preprocessorFactory are both called independently with the karma configuration options, but preprocessorFactory is mutating the controller object with the webpack configuration while the other does not, this causes an inconsistency in output state.


Align bytes

How can I align bytes without Condense Filler Bytes? I create arrays now, but it is wrong approach.
That's what I mean:

Create a selection, and apply the filler bytes data type.  The Alignment data type is similar to a string in that it can have a variable size.
You can also apply the Alignment data type with a size from within a script.


Feedback Milestone 4

Hi!
Your Dashboard looks pretty straightforward and clean!
Couple things:

It would be best if "daily distribution of pollutants " graph will get updated without its own dropdown to select the pollutant. you already have that dropdown for your first graph
the sidebar was not explained: what is it for? is it a specific day or range/ than what does the averaging period do?
make sure to make some space for explaining why you  created this dashboard ( you can utilize the stuff from your final report's intro section) and also acknowledge the data source

Feedback:

I could not figure out which feedback was implemented and which was not

Thanks @yuliaUU !
All graphs depend on the same dropdown
The sidebar is now used for the dropdown menus
We've included a description at the top of the dashboard and I've added the data source.


Links: Cortana Skills Kit (2021-02)

Monthly link cleanup.
/cc @Karl-Bridge-Microsoft

Docs Build status updates of commit 3a8e32c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/data-markup/toc.md
:white_check_mark:Succeeded

docs/skills/toc.md
:white_check_mark:Succeeded
View

docs/toc.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Re-implement the traffic replay feature as TCP stream (PCAP) replay

This feature is HTTP specific and poorly implemented. Therefore should be removed.
It should be re-implemented as PCAP replay in the future.

https://github.com/kubeshark/front/commit/dbd561d38df44aeac1a08f69edf5bc61d05c4a5f

https://github.com/kubeshark/front/commit/489def0c2a91bda7f5f7c92ced1559fc4c8bc669

https://github.com/kubeshark/worker/commit/4124d2dea0743d328f8c16f8621bdeb5e440f82a

https://github.com/kubeshark/hub/commit/b53727e0a0013b217bc4c049ab56ead25bc731c3


Game freezes whenever clicking on buy max ants or enabling autobuy

I have quite big numbers also, might be the cause

I've not been able to get numbers like these yet. You should probably try a hard reset in de settings and play from there. It's much more balanced. But I'll definitely look into it!

When buying like 20k ants the game is freezing for good 5-10 seconds


Rename Failure to Rejection

This PR makes the Spine Framework use the term Rejection instead of Failure to avoid the terminology clash with the term from Reactive Manifesto.

@armiol, PTAL.


Possible minor typo in readme.md

readme.md
- Section on Updating over USB.
Shouldn't line 30 and 31's mention of "CP2014" be "CP2104"?
(It seemed like in SiLabs documentation it was 2104 or 210x, rather than 2014).

Fixed, thanks!


[Feature request] unsquashing unsorted_segment_x

Have I written custom code? No
OS Platform and Distribution? Win10
TensorFlow installed from? pip
TensorFlow version? 1.7
Bazel version? N/A
CUDA/cuDNN version? N/A
GPU model and memory? N/A
Exact command to reproduce: unsorted_segment_sum
Describe the problem
Let's say you have 3 sequences of 12 values (shape (3,12)) with 5 segments and you have an input of shape (None,12,3) transposed to (3,12,None)
using any function with unsorted_segement_x the output will be (5,None).
Would it be possible to have a function that does the same but still keep the first dimention.
Something equivalent to:
input.shape==(None,12,3)
segment.shape==(3,12)

outputs=[]
for ix, val in enumerate(tf.unstack(input)):
    ouputs.append(tf.unsorted_segment_sum(val,sequence[ix],5)
output=tf.stack(outputs)
output.shape==(3,5,None)

The use of such a thing would be to iterate over a weighted input and without having to loop over the data or using reshape. I think this should be somewhat straightforward to implement; not concatenate together all the sequences for the superior dimensions and use "keep_dims" similar to reduce_sum to trigger that behaviour.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Have I written custom code? No
OS Platform and Distribution? Win10
TensorFlow installed from? pip
TensorFlow version? 1.7
Bazel version? N/A
CUDA/cuDNN version? N/A
GPU model and memory? N/A
Exact command to reproduce: unsorted_segment_sum
(duplicate info for the bot)

Nagging Assignee @rmlarsen: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 60 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I want to use the same feature too! It is strange that the segment_ops don't have keep_dims parameter.

@zhaoxin19 There are some workaround to this, a good idea might be to start a new thread about this since @rmlarsen  seems to be deceased.  I am doing deep model research and want to add this kind of operation in my new model. Lucily I've found the unsorted_segment_sum implement. But it is a little different as what I want. I think add a keep_dims parameter is a good idea in this operation. I'm not expert in adding or changing a new operation in tensorflow. I will be very grateful if anyone could help.

@roya0045 do you know how can we add this feature in tensorflow? Seems that we just need to change tf.unsorted_segment_sum api as tf.reduce_sum adding a keep_dims parameter.

I'm sure it can be done but I'm not sure how. I'm not familiar with c++ and the code base for tensorflow.
The workaround I have found so far are implemented in  my latest projet
The only issue is that I have not tested if those 4 methods break backpropagation (get the weights first, train on random data, get the weight after and just subtract the initial weight to see if it worked.)
I wanted to test them but I haven't had much time to do so, if you want you could just set up a minimal network with a for loop for each version , train it and compare the weights to see if those workarounds are valid, I'd be interested in the results.

Nagging Assignee @rmlarsen: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 48 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 64 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 80 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 94 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 109 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 124 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 140 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rmlarsen: It has been 155 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/schmupu/ioBroker.lupusec

[ ] [E502] "admin/img/info-big.png" not found, but selectID.js used in index_m.html

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

fixed


fix: reanimated+gesturehandler crash

Why

We get a crash when doing reload via dev menu. The stack trace of crash looks same when we apply codepush updates and do ExpoUpdates.reloadAsync.
Also fixes crash caused by coinbase sdk due to failing assertion.

How
Follows this comment. This would make all the callback passed in gesture handler run on JS thread. Right now this seems to be a good tradeoff, we're mostly setting shared values in callbacks so this should not be a big issue. Best solution would be to migrate to v3 once it's stable enough.

Test Plan

Run the app, do cmd+d and press reload. App should not crash

this is the first thing i'll remove when we migrate to v3 😤

Testing with a prod release and confirmed that reloading does not crash anymore.
https://user-images.githubusercontent.com/504909/235762954-49555342-2cc2-4bb2-8c34-4999c121a7e4.mp4


Several missing features

Hello. Kudos to this plugin. I would be extremely grateful if the link data was displayed in the pop-up window. With a large number of hosts, this is necessary =)
Also, there is not enough scalability so that you can zoom in and out a certain section of the map in the widget.

Could you be more specific? I'm not sure what you mean about the scalability.

It would be convenient in the dashboard to zoom in on a certain zone of the network map and move it away

Not sure if this is what you mean, but you can zoom on the map (outside of edit mode) already? Holding Shift + scrolling or dragging will let you zoom and move around (even as a read-only user).

Or do you mean to be able to select an entire portion of the map to move at once? That would make more sense.

Not sure if this is what you mean, but you can zoom on the map (outside of edit mode) already? Holding Shift + scrolling or dragging will let you zoom and move around (even as a read-only user).

great, didn't know

And do not tell me what about the pop-up window?
And I'm also interested - in one of the updates, it was indicated that you can bend lines between network nodes, I can't find it. Or is it planned?

And I'm also interested - in one of the updates, it was indicated that you can bend lines between network nodes, I can't find it. Or is it planned?

Yes, please see the wiki: https://grafana-weathermap.seth.cx/customization/#advanced


Add /clear command via the ANSI method

Small, but welcome addition in my opinion. Works using the ANSI method from the bottom.
case "/clear":
    fmt.Printf("\x1bc")

Finally checked in the help text w/ PR #1764 which explains how to clear the screen. I'm going to go ahead and close this PR. Thanks again for submitting it though!


ML 1.11: Skewness (D)

Welcome to 'ML' Team, good to see you here
This issue will helps readers in acquiring all the knowledge that one needs to know about Skewness.
To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
Domain : Machine Learning
Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Statistics_for_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Documentation

All the best. Enjoy your open source journey ahead. 😎

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do documentation on this issue.

Great, I am assigning it to you.


transparent login with modern web view

hi,
in the original web we can login with url and token
but in modern web i can't find any way for this action
how can to do that ?

For questions there's the forum.


fix: s3 bucket name constructed differently in legacy amplify apps (p…

…re 2019)

Description of changes

Issue #, if available
#8893

Description of how you validated changes
s3 bucket name was constructed differently in 2018/2019 versions of amplify-cli.
pl find attached legacy-cfn and new-cfn both generated based on the logic.
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
new-cfn.json.zip
legacy-cfn.json.zip

Finally got this figured out huh.


Camera plugin: access to the image stream from Dart example

in https://github.com/flutter/plugins/tree/93d679181427c3b85ac3c989f779e0e263fd34e8/packages/camera/camera
I see "Add access to the image stream from Dart." but I don't see methods to call it.

Can you explain how to get access to image stream? I don't see any methods in example/lib/main.dart
What about overhead? Is it good idea to apply OCR on image streams from dart side?

@Alexufo
Can you check if this open issue is similar to yours ?

@darshankawar  Yes!

Thanks for confirming. I suggest you to follow-up in the original issue and you may post your comment in it, since it is already prioritized and has crowd label.
Closing this as duplicate.


notary-server certificate expired at midnight

the notary-server.crt included with the git clone just expired. hopefully a new one will be added soon?

+1

@lewiada: new testing certs are on the way!

awesome, thank you!  was just getting familiar with notary when they expired :-)

@lewiada just opened #842, so should be fixed soon!  You can check out that branch for now if you'd like to continue playing with the notary docker-compose setup :)

pulled the branch and back in business!  Thanks for the quick turn around :-)


Problem training Penn treebank example with large hidden size on GPU

There seems to be a problem training the
 penn treebank tutorial example
using large networks on a single GPU. When using a large hidden state and large minibatch, (in my case minibatch = 100 and hidden state size 4000 on a 12 GB tesla k40 GPU, although others have replicated my issue on other GPUs in python 2.7 ,python 3.4 and python 3.5), the program freezes at line 139 at the call to loss.backward(). This issue can be fixed in a very hacky way by inserting a statement print(loss.data) right before loss.backward() is called. With this print statement, training works correctly.

No longer able to reproduce the problem, so I will close the issue. It works for me using the specified parameters on 16GB P100.


Bridge to Defichain: Burned, but not received

Hi,
tried to bridge to Defichain.
BSC Burn Tx ID 0xa5681287c4131855f482569e59aed91df1a10e8c78b4ab5b55b1ac3e1ba82257
Defichain Receiving address df1qs7uza0nxz9pq9tuwrh7dmslfkqs63aey7yeghv.
Tokens not received on Defichain. Recovery doesn't work, Error message "could not sign the payout request".
Please help! Thx in advance.

Above Tx could be recovered now - which didn't work previously.
New BSC Burn Tx that is stuck now is 0x56496a467c4044068215212a6b6785c9df652a7dd2bf521cbce97458db846843.
1800 DFI, recovery doesn't work, same error as I had above earlier.

I had lost a lot of coins in some scam case just like this thanks to the best hacker in the world who helped me recover all my funds,message him on telegram his telegram username is https://t.me/Blackhathacker110

Is the problem still present? If so, please let us know your DFI address to which the funds should be sent.


Expose user schedules as an iCalendar server

Depends on #47; resolves #8.
Building on top of our ability to export schedules as iCalendar files (#22), this feature would expose users' schedules publicly over HTTP so they can be accessed by any calendar client and, crucially, update as the user edits their schedule. Users would have to opt into exposing their schedules public and the path to the schedule would be a hash/UUID as to avoid enumeration attacks and other kinds of information leaking.
We would also need to link to documentation explaining to users how to add their schedules to popular calendar clients; for MIT students, the most important ones to link to would be Google Calendar and Outlook Web.

This could also be decoupled from account functionality; we currently have a functionality for URLs pointing directly to schedules, and we could similarly encode the schedule data into the URL itself to generate ics data and avoid keeping any knowledge of what user generated it (thereby presumably evading most privacy concerns).

encode the schedule data into the URL itself to generate ics data

this loses the "update as the user edits their schedule" aspect of the feature, although it would be a nice feature too

Oh, true. So, I guess this requires us to grow our own authentication or else somehow co-opt FireRoad authentication, either of which would require a proper backend.

Maybe it could edit the iCalendar URL using Google's OAuth, as the user updates their schedule, all completely from the client-side? Although this would go into why did Google Calendar export stop working in the first place?


Use tslint

There's a tslint: https://github.com/palantir/tslint
We should use it.  (Maybe in a precommit script).

We're using it in uproxy-lib!
https://github.com/uProxy/uproxy-lib/pull/355

:+1:

Now in uproxy too!


Mark/microchip

Thank you for sending a pull request :heart:
Checklist
Please review the guidelines for contributing code to this repository. And check [x] all the boxes that apply:

[ ] This request pulls a feature/bugfix branch (right side) from my fork. Not my master.
[ ] This request pulls against ockam-network/ockam’s develop branch (left side).
[ ] Build succeeds ./build with no linter warnings or test failures.
[ ] All code matched our code formatting conventions.
[ ] Commit messages match our conventions and all commits
are signed.
[ ] This request includes tests and documentation for all new code.

Proposed Changes
Please describe the changes in your pull request.
If this pull request resolves an already recorded bug or a feature request, be sure to link to that issue.

Could you squash the branch so it has only signed commits
Also follow the commit message convention here
https://github.com/ockam-network/ockam/blob/master/CONTRIBUTING.md#commit-messages

In the meantime, I'll read the code.


[WIP] Fix NeuralMagic tests

@mgoin do you have any ideas as to why the following new test failures might be occurring with the SparseML integration?
FAILED tests/models/test_neuralmagic.py::test_sparseml_eval[facebook/opt-125m-lambada_openai] - ImportError: cannot import name 'SparsityCompressionConfig' from 'compressed_tensors' (/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/compressed_tensors/__init__.py)
FAILED tests/models/test_neuralmagic.py::test_sparseml_eval[hf-internal-testing/tiny-random-gpt2-wikitext] - ImportError: cannot import name 'SparsityCompressionConfig' from 'compressed_tensors' (/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/compressed_tensors/__init__.py)
FAILED tests/models/test_neuralmagic.py::test_sparseml_eval[mgoin/tiny-random-llama-2-quant-gsm8k] - ImportError: cannot import name 'SparsityCompressionConfig' from 'compressed_tensors' (/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/site-packages/compressed_tensors/__init__.py)

I can see that in the sparseML repo this SparsityCompressionConfig class does exist, but there seem to be problems importing it in the transformers SparseML classes for some reason.

@bfineran it seems that sparseml nightly has breaking changes related to compressed-tensors, do you have a recommendation for a version to pin as the version ceiling or if this might be resolved with a new nightly pushed? Reference for current version requirement: https://github.com/EleutherAI/lm-evaluation-harness/blob/86319a9b14ddae2030bc6e0fdddd47fc7d0bb525/pyproject.toml#L74

Hi @mgoin we pushed a new nightly build today and have processes in place to get them released regularly. Let me know if there's further issues

That's great, @haileyschoelkopf could you please try re-running the checks so the latest nightly should be accessed?

@mgoin sorry for the delay--looks like the tests still fail with the same import error?

@lintangsutawika this small PR moves around some of the external LM tests into a separate workflow so that we don't run them for every Python version--ready for review!
Unit test failures are unrelated, seems like some caching issue on our CI


Incorrect parsing (get/set parents)

<?php
include 'vendor/autoload.php';
use gymadarasz\xparser\XNode;
$x = new XNode(
'

		<title>Test page</title>

');

$before = $x->find('#hello1 div')->outer();

// show document
echo $before;

outputs

which is not valid html (there is a missing
)

I fixed it but I'm not sure to understand the problem. "there is a missing ", It means you would find the "#hello1" element but your selection was "#hello1 div" so there the opening tag <div id="hello1"... was unnecessary OR did you try to get a parent element with the outer() function instead of outerHTML text?
Just because the parent element doesn't supperted yet so I'll leave open this issue for parents getter/setter functions otherwise bug fixed.

What I'm saying is just that the output is not html
There are two div opening tags. And only one closing.
The output should be :

I pulled the last version I now I only get

Which is the inner of #hello1 whereas my code is asking the outer. outer (according to the spec) should return the element queried entirely. Before the closing tag was missing. Now the opening and the closing are missing (which is even worse)

It's perplexing me because I added some new test for every single variant and all test was success. In your test case your expected result was the find('#hello1 div')->parent() or $x->find('.hello2')->parent() but the parent() function is not included in this version yet, it is why this issue already open and I working on this..
Momently, you sould try to get: $x->find('#hello1')->outer(); instead of $x->find('#hello1 div')->outer(); in your test.
(current version commit is 8739cf9)

I'm sorry I was wrong the issue was indeed fixed in 2d3b179

No prob, in meanwhile I added a getParent function on a develop branch, it will be merge into the master in the next version, so thanks for this issue (y)


Add jealous by eyedress

Song Name
jealous
Artist Name
eyedress
Source
Youtube
Youtube Link
Y_WqmAmT1rpCnOGp
Direct File Link
No response
Song ID
1818291
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 91948655

reject
invalid YouTube link


Allow of setting user generated Request Id

As a feature request I am suggesting a way for a user to set their own request Id. I already track unique Id's for the things I download and having to track the request Id's on top of that seems unnecessary. This makes it easy for me to use the Id's that are already stored in my own database. Any thoughts?

Appears to be working in the application I am developing. If I encounter any issues I'll reopen.


Better parameter typing

Is your feature request related to a problem? Please describe.
Decompiled functions are frequently littered with type casts,
Describe the solution you'd like
Doing an additional pass over the decompiled code, in which all casts that were used in the previous pass are noted and tried as function parameters. The permutation with the fewest number of casts required is the one chosen.
Describe alternatives you've considered
Brute force all reasonable types (i.e: not casting more than one level of indirection) and select the one which produces the fewest number of casts.
Additional context
Before:

After:

This would be great! Going over the parameters manually and figuring out the types is very time-consuming.


Working with LocalizationModelContext

Hi again @damienbod
I'm curious, how would I work with the underlying DbContext in my app?
Suppose I want to seed localization data at Startup (populating the LocalizationRecords table).
Or maybe I want to create a web console in my app, where I can query the records, do CRUD operations etc.
Thanks in advance :sun_with_face:

Hi @inexuscore
Seed should be possible as you have the DbContext in your project. If changes are required to the DbContext itself, then I would use the Localization.SqlLocalizer directly instead of the NuGet package and change it. We could then add this to the package if it helps others.
https://github.com/damienbod/AspNet5Localization/tree/master/AspNet5Localization/src/Localization.SqlLocalizer
Greetings Damien

Hi @damienbod
Great, I think I'll pull in the source to my project and have it as another layer.
The DbContext doesn't require any changes, at least not now, not that I can think of any. But there's only one change I would make: configuring a default value for the UpdatedTimestamp column. That way we'll be able to manually add records in SSMS, Visual Studio or other clients.
Thanks for the repo!

I'll add an interface for the LocalizationModelContext : ILocalizationModelContext
Then you can write your own if required, for example configuring a default value for the UpdatedTimestamp column

interface won't work, need to think of a better solution.


python3Packages.flux-led: 0.27.20 -> 0.27.28

Motivation for this change
https://github.com/Danielhiversen/flux_led/releases/tag/0.27.28
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.flux-led
python39Packages.flux-led

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

2 packages built:

python38Packages.flux-led
python39Packages.flux-led


updated documentation for quickstarters and slaves

updated documentation on quickstarters and slaves

@rattermeyer .. Status? Either Update and Merge or close

Will close for now, because we needed to make more changes in the way we handle Java discovery to get it working on CentOS. Will open a seperate request for all changes.


Using stryker & vitest runner & JSDOM in UI test cannot PASS dry run

Summary
When using vitest to test UI components, run with npx vitest run will PASS, but npx stryker run will FAIL in dry run.
The root cause is the global jsdom environment was shared in stryker testing, while isolated in vitest unit test.
I found that it was because of this setting:
https://github.com/stryker-mutator/stryker-js/blob/master/packages/vitest-runner/src/vitest-test-runner.ts#L46C24-L46C24
this.ctx = await vitestWrapper.createVitest('test', {
  config: this.options.vitest?.configFile,
  threads: true,
  coverage: { enabled: false },
  singleThread: true, // <----- This single thread causing issue
  watch: false,
  dir: this.options.vitest.dir,
  bail: this.options.disableBail ? 0 : 1,
  onConsoleLog: () => false,
});

This issue was also introduced in vitest document
I use patch-package and set singleThread to false, seems everything works fine for me. Is it OK to use false here, and what's the side effect?
Stryker config

{
  "$schema"<EMAIL_ADDRESS>  "_comment": "This config was generated using 'stryker init'. Please see the guide for more information: https://stryker-mutator.io/docs/stryker-js/guides/react",
  "tempDirName": "stryker-tmp",
  "mutate": ["src/**/*.{j,t}s?(x)", "!src/**/*.test.tsx", "!src/setupTests.ts"],
  "ignorePatterns": [
    "node_modules",
    "build",
    "coverage",
    ".git",
    "/reports",
    "*.tsbuildinfo",
    "/stryker.log",
    ".stryker-tmp"
  ],
  "testRunner": "vitest",
  "vitest": {
    "configFile": "./vite.config.ts"
  },
  "reporters": ["progress", "clear-text", "html"],
  "symlinkNodeModules": true,
  "coverageAnalysis": "off",
  "tsconfigFile": "tsconfig.json",
  "dryRunTimeoutMinutes": 20,
  "timeoutMS": 60000,
  "timeoutFactor": 4
}

Stryker environment

+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>

+--<EMAIL_ADDRESS>
Test runner environment

# Test command
vitest run
# Mutation test command
stryker run

import { defineConfig } from "vite";
import react from "@vitejs/plugin-react";
import "vitest/config";

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [react()],
  test: {
    globals: true,
    setupFiles: ["./src/setupTests.ts"],
    environment: "jsdom",
    singleThread: true,
  },
});

Your Environment

software
version(s)

node
18.8.0

npm
9.7.1

Operating System
MacOS 13.4 (M1 Pro)

Demo Project
It's really easy to reproduce this issue in the demo project
https://github.com/InfiniteXyy/stryker-vitest-issue

Thanks for opening this issue. I believe singleThread might not be strictly needed. We started with singleThread as requirement because we tried to communicate with the test environment via the global object, but we have since moved away from using the global object for this.
We might be facing issues with concurrency. Vitest must run with concurrency 1, since Stryker is already running mutation testing in parallel (1 worker per core). As long as we can force that (maxConcurrency to 1?), it should be fine.
@odinvanderlinden, what are your thoughts?

Ah yes that makes sense maxConcurrency = 1 it is

Thank you for the updating!
But when I tried to use singleThread = false and maxConcurrency  = 1, seems the mutation score is very slow (most files 0%) Don't know what happened

When i set singleThead = false and maxConcurrency = 1 on your demo project it seems to work fine. Can you confirm this? If so, what is the difference with your real project?

When i set singleThead = false and maxConcurrency = 1 on your demo project it seems to work fine. Can you confirm this? If so, what is the difference with your real project?

This issue only happens when there are multiple mutated files.
Sometimes, the App.tsx has 100 score, sometimes, the App2.tsx has 100 score
I pushed a new commit to the demo project. Maybe you could have a try, not sure

Thanks for the quick reply, i will further investigate this issue this afternoon!

I think I found the problem. The so called projectName of the vitest suites are undefined. Before iterating all the test suites the duplicates are filtered out by the value of the project name. So one of the two test suites will always be filtered out. The solution will be simple by checking for the filename for duplicates, but I don't really know the reason behind the decision to use the project name.
https://github.com/stryker-mutator/stryker-js/blob/cf5d4d04ee24a1e2b65c1f7ae35dbc840edbd48a/packages/vitest-runner/src/vitest-test-runner.ts#L168
@odinvanderlinden can you fix this?


SVG overflowing margins of HTML output from author tools.

Sample SVG ... https://github.com/woobagooba/draft-ietf-is-csmp/blob/main/RegistrationConfigurationMetrics.svg
Margins are fine in the PDF output from author tools.
HTML output from author tools extends SVG far beyond the right margin.

@becarpenter suggestion of deleting height and width works for me.
Maybe this is a tip that should be in https://authors.ietf.org/en/diagrams ?

The problem is that most drawing tools include these elements. I don't know whether they are useful for the PDF generation. My private tool for fixing up SVG files includes this (a bit crude but it usually works...):
def nowh(l):
    """Remove width and height to allow scaling"""
    if 'viewBox' in l:
        print("Seen viewbox")
        if ' width="' in l:
            l1,l2 = l.split(' width="',1)
            _,l3 = l2.split('"',1)
            lout= l1 + l3
            print("Removed width")
        else:
            lout = l
        if ' height="' in lout:
            l1,l2 = lout.split(' height="',1)
            _,l3 = l2.split('"',1)
            lout= l1 + l3
            print("Removed height")
        return(lout)
    else:
        return(l)

A few additional comments.
If it matters, the source draft is authored in kramdown.
What happened to the online version of svgcheck?  https://xml2rfc.tools.ietf.org/experimental is now landing on https://author-tools.ietf.org/.
Should/is svgcheck issuing a warning when height and width are encountered?  Can it / should it automatically remove height and width?
Should 7996 be corrected to address the height and width issue?
I make these points because my tool vendor did their best to adhere to the RFC and author tools.
We should not depend on an ever lengthening list of manual edits required to make SVG source work for IETF drafts.  The goal should be direct export from the SVG drawing tool and unaltered import into the draft.
Cheers

Paul:

What happened to the online version of svgcheck? https://xml2rfc.tools.ietf.org/experimental is now landing on https://author-tools.ietf.org/.

All of tools.ietf.org was taken down, per the plan announced over a year ago.
We haven't added svgcheck to author-tools as it was clear the community expected the 7996 restrictions to change. We'll look at adding it for people like you who are still attempting to conform to what turned out to be really bad dance steps.

Should 7996 be corrected to address the height and width issue?

No,  7996 should be replaced with a document that specifies a more reasonable profile of SVG allowed to appear in published RFCs. This has been discussed quite a lot on rfc-i, the xml2rfc list, the lead-up to the creation of the new editorial stream, and is not being at least pointed to strongly in RSWG.
This is very frustrating for the tools team - the community is not ok with the tooling continuing to make non-consensus changes to what the vocabulary allows, so until 7996 changes or we get strong and clear direction that we should work ahead of publication of its replacement, we are stuck at the current behavior.
From your early comments, it looked like you chose to push into this pain-point aware of what you were getting into. If not, the situation isn't the best, and it would serve everyone well for you to (re)skim the discussion on 7996 so far (or read one of Brian's summaries). If you are fully aware of the corners, then help push where it will matter - help the RSWG to approve a replacement for 7996.

My tool vendor had been using experimental svgcheck until it disappeared only a few days ago (?), thus the question.  I guess now they will have to pull the source and build locally.  NBD
I agree that 7996 (or its replacement) should be the authoritative spec for IETF SVG and non consensus changes should be prohibited.  I do strongly advise any overhaul of 7996 reflect the reality of SVG diagramming tools available today.  I was fortunate in that my tool vendor added an IETF special SVG export.  That will not always be the case.
Unlikely that I will be resourced to assist untangling the IETF SVG knot.  I might be able to rally other assistance.  I will check.
You are probably already aware of this, but I will underline it anyway.    I am able to take SVG from any number of drawing tools and have yet to find a viewer that is unable to display the SVG ... except the IETF RFC tools.

All I can speak to is the issues I discovered personally. I'm not sure I did write a summary but what I've learned is captured in https://github.com/becarpenter/misc/blob/main/fixupSVG.py. Jim Schaad did some fixes a while before his final illness.
The RSWG is probably going to do RFC7991bis first, I think. I suspect that we won't get peoples' attention for SVG until that is done.

Apologies that I misremember, but I thought you had a shortlist of the things that whatever format we should move to should/shouldn't do to avoid the issues that our custom profile landed us in. Nevil made some changes too I think (at least he suggested he would at https://mailarchive.ietf.org/arch/msg/tools-discuss/yR78l9xZ4doOWDHUgkpJO56HyuI/). And if its still worth us making that checker reflect 7996 where it doesn't, we can work to do so. But I wonder if that's the better use of time.

It's possible that I am the one misremembering. This is a discussion we do need to start over at RSWG, I think.

On 2022-08-15, at 03:18, pbduffy @.***> wrote:

Should/is svgcheck issuing a warning when height and width are encountered? Can it / should it automatically remove height and width?
Should 7996 be corrected to address the height and width issue?

Oh, and by the way: this discussion is NOT theoretical.
Archived-At: https://mailarchive.ietf.org/arch/msg/auth48archive/nZXEC8ORpZycdmcFjuvUsSQg_6s
Grüße, Carsten

@cabo, I agree, it seems that each case needs to be checked for weird behaviour.

My experience with this is that width and height are extremely useful attributes for ensuring that things don't get scaled up.  @cabo's example is something we had to fix with width and height attributes in RFC 9113.  Without them, the diagram is - just like Carsten's example - disproportionately large.
If your diagram is too wide, then removing width and height will allow it to be scaled down to fit, but that probably isn't the right answer.  If the image is getting cropped because it is too wide for the HTML (or PDF) view, then perhaps the problem is that the diagram is indeed too wide.  A reasonable width for HTML is about 600px; the default xml2rfc stylesheet has a width of 724px; @woobagooba's example is 992px wide.
The @woobagooba includes here has text that essentially too small to read at 600px width[^1].  On a smaller screen it is totally unreadable; I can only imagine what it is like for someone with worse eyesight than mine.
[^1]: Even at its native font size of 13.3333px it's still arguably too small.  Sure, the default xml2rfc stylesheet has body text at 14px, but that is also too small already.

I've massaged the width on https://github.com/woobagooba/draft-ietf-is-csmp/blob/e9c763102bee617e4f6f6a526d39de081660a25f/RegistrationConfigurationMetrics.svg and also split one of the longer diagrams into two.  They all now render nicely in the PDF, still spill over the right HTML margin.

Folks, I've noted that there is a desire (requirement?) that drafts/RFCs adequately display on small screens.  By that I assume the intention is a cell phone.  IMO any serious user of these documents is not developing on a cell phone.  I would not let the small screen tail wag this dog.  Cheers

I don't develop on a cell phone but I definitely look at RFCs on my phone now and then.  They don't have to look fabulous but they do need to be readable.
I also don't think it is a great idea to make designs starting with "nobody I know does X."

The requirements need to be crystal clear for priority use cases.  We should not be optimizing for "now and then" readability on "small screens" (whatever that means).  Is an Apple watch a small screen?

IMHO, we shouldn't mix an editorial judgment issue ("Is this diagram too busy/complex/big for an RFC") with a technical issue ("is scaling for any screen size a required or optional property?"). The former can only be answered case by case but the latter needs to be clearly resolved in 7996bis and in the toolset.

@becarpenter is right, there are two issues here.
The number of characters might be an editorial issue that can be decided on a per-document basis, though probably in a style guide.
Ensuring that this renders - if not well - but reasonably on a small screen is a technical problem.  Though the technical solution might be informed by the bounds set by editorial policy.  For instance, if editorial policy only needs 75 characters per line and SVG diagrams are created that fit that bound, they will look pretty good on a small screen, with only a modest reduction in text size or scrolling needed.  Of course, if the goal is to include very large and complex images, we might need to look into something more advanced.  I don't think we need anything advanced, but that is based on my opinion of what an appropriate editorial policy might be.

As for what editorial policy is reasonable, a two column diagram with >50 characters on the left and 75 characters on the right - more than 125 on a line - is probably going to blow right past any reasonable line length limit.  The best advice I have says line length should be less than 75 characters.  Unsurprisingly, the text format is almost ideal at 69 characters per line.
On a phone, you get slightly less than 75 characters, even with a small-ish font you only get around 50 (and there is good advice that suggests that the default RFC stylesheet uses too small a font size).  But small screens are often high DPI and we don't need a perfect rendering for diagrams, just a good-enough one.  People are able to scroll or zoom to get to details.
Here is what Paul's draft looks like on a phone (scaled down 4x, which is a fair representation of its readability):

That picture is several screens wide and more than a screen high.  It's pretty comprehensible, but hard to work with.  To see all of it, it is a bit squinty.  It could probably be simplified a fair bit so that the important and structural parts are conveyed by the image, leaving the details to text.

Let me add that the PDF generated by author tools is fully legible on an IPhone.  As I assume will be the HTML when the SVG scaling issues are resolved.  I hope the equivalent PDF generation will be made available by Datatracker.

First of all, I don't understand why datatracker offers PDFized plaintext where it should offer proper PDF.
Second, I'm trying to mine this conversation for the controls an authoring tool (such as kramdown-rfc) should provide.
Generally, authors don't want small figures to expand to full-width (see my example), and these are quite likely to occur, so scaling up needs to be default-off (as it is in kramdown-rfc now).
But we never want parts of the figure to become inaccessible, so scaling down should be automatic.
This needs to interact properly with any default CSS that is applied to SVG for both HTML and PDF-through-HTML (I don't know yet why the latter has a different effect).
This CSS might change any week, and I don't think we have good test cases so people changing the CSS get to know when they have broken the scaling.

On 2022-08-17, at 16:44, John L @.***> wrote:

I don't develop on a cell phone but I definitely look at RFCs on my phone now and then. They don't have to look fabulous but they do need to be readable.

+1.
I’m forced by random circumstances to do this way more often than I’d like to.
(It actually works very well with HTMLized…)

I also don't think it is a great idea to make designs starting with "nobody I know does X."

Related to the concept of “Proof by lack of imagination” :-)
Grüße, Carsten


[FLINK-29310] Added license check

Added maven plugin to use in ci workflow for license check.
Note:

License check current runs on mvn install directory, which will need to be updated when switched to deploy mode.
We are using flink-ci-tools dependency which is only available as SNAPSHOT at the moment. https://repository.apache.org/content/repositories/snapshots/org/apache/flink/flink-ci-tools/

Previous closed PR: https://github.com/apache/flink-connector-dynamodb/pull/8. Created new PR to update branch to pull from.

I realised this has the wrong Jira label, after merging https://issues.apache.org/jira/browse/FLINK-29442


Fix update.sh

@satnam6502 @filbranden

One minor nit, otherwise looks good.

Comment addressed.  I'm merging since e2e is broken.


nest: init at 3.3

Description of changes

NEST is a simulator for spiking neural network models that focuses on the dynamics, size and structure of neural systems rather than on the exact morphology of individual neurons.
Homepage: https://www.nest-simulator.org/
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

6 packages built:

nest
nest-mpi
python310Packages.nest
python310Packages.nest-mpi
python39Packages.nest
python39Packages.nest-mpi

Result of nixpkgs-review pr 198872 run on aarch64-darwin 1

6 packages built:

nest
nest-mpi
python310Packages.nest
python310Packages.nest-mpi
python39Packages.nest
python39Packages.nest-mpi

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

6 packages built:

nest
nest-mpi
python310Packages.nest
python310Packages.nest-mpi
python39Packages.nest
python39Packages.nest-mpi

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1386

@ofborg eval


implement Consensus Hit Support

https://github.com/reubwn/hgt

I chose to do a modified aggregate hit score instead (#28) https://doi.org/10.1371/journal.pcbi.1010686


Add support for set retention on simplivity backups

Description
Add support for set retention on simplivity backups
Issues Resolved
None
Check List

[ ] New functionality includes testing.

[ ] All tests pass($ tox).

[ ] New functionality has been documented in the README if applicable.

[ ] New functionality has been thoroughly documented in the examples (please include helpful comments).
[ ] New endpoints supported are updated in the endpoints-support.md file.

[ ] Changes are documented in the CHANGELOG.

Closing this pull request as we are seeing the changes in other files as well


[sdl2-image] update to 2.6.2

Library name: SDL_Image
New version number: 2.6.2
Other information that may be useful (release notes, etc...)
Link to 2.6.2 release notes is here: https://github.com/libsdl-org/SDL_image/releases/tag/release-2.6.2

Never mind, the website is outdated...


deprecated #ifNotNilDo: -> #ifNotNil:

change deprecated #ifNotNilDo: -> #ifNotNil:

a lot of mess from STON reformating of the metadata

Thanks!
The repo should be metadata-less so I am not sure why it is complaining about that...

It is very common. Someone changed the formating for STON in filetree some
time ago and since then you can see such mess even for the metadata-less
repositories (once).
Cheers,
-- Pavel
2017-03-22 16:40 GMT+01:00 Peter Uhnak<EMAIL_ADDRESS>
Thanks!
The repo should be metadata-less so I am not sure why it is complaining
about that...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/peteruhnak/geometry/pull/2#issuecomment-288439953,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAi8TnsffUJUeq0uYoaYIoVqyoO8zS5Nks5roUDXgaJpZM4MlWga
.


feat: optimize starknet storage access

Feature Request
Update the get_storage_at RPC endpoint to remove the need of the runtime API query.
Describe the Feature Request
The current implementation of get_storage_at endpoint makes use of the runtime API in order to access the starknet storage. We should offload work from the runtime to the override.
Describe Preferred Solution
Implement storage access in the override and replace the storage query in get_storage_at.

Related Code

Current storage query
override location

I will take this.


Add ru_ru.json (Russian)

Hello! Here's the russian translation of your mod
Feel free to correct me or ask me anything about it, I will answer you and commit changes as soon as I can.
I've been working as a mod translator for quite a while now, so if you like my work, you can consider donating! Thank you! (link in bio)

Thank you for the translation. I will merge before Update 4.7.0 is complete.

Finished Codding the Update, could you rebase the PR please, thx (I probs should have merged before quashing, by bad)

Looks like there are some conflicts between theese two branches, I will resolve them and open a new pull request


Pulling tasks from archived projects

I have just realised that the plugin is somehow pickiong up tasks that are in archived projects. I.e. a filter like overdue|today doesn't appear the same between todoist and what the plugin produces

Hey, thanks again for filing!
Hmm, I'm having a hard time reproducing. I tried creating a new project with one task in it that's due today. Then, I archived the project. Then, I ran the plugin with the query "today|overdue". The task did not show up in obsidian.
Mind showing me how to reproduce?

So it appears that there's something odd occuring with Todoist that is likely causing this. It turns out it was only archived projects with the GCal 2-way sync on (apparently the 2-way sync continues to populate the archived project, but not visibly in Todoist).
I'd say this is a bug with Todoist rather than the plugin


[Feature] Update a KPS table

User story
Some APIs need updates in KPS tables. Swagger promote should support a generic way to specify a KPS table and properties so it can do the insertions/removals. (The key of the KPS table is always the api.id field.)
Additional context

With the next version I plan to add initial support to manage KPS in the same way as other entities. This means to provide the following features:

get existing entries (Console, JSON, CSV, etc.) from a given table/alias

Filter support

import/replicate KPS entries based on JSON desired state

to replicate the KPS-ID is used to decide if update or create an entry
the desired state may contain multiple KPS Tables/Aliases and entries for each

delete entries based on the given filter

In the version after KPS-Dependencies can be added to an API. This dependency will be the KPS-Alias plus the Entry-ID. This kind of fragment will then be added to an API on the root level:
kps: [
  {“kpsAlias”:”tableABC”, “kpsKey”:”123456”},
  {“kpsAlias”:”tableXYZ”, “kpsKey”:”987654”}
]

With that, during replication of an API the CLI validates that the required KPS entries do exists. The required KPS entries must be imported before using the KPS import feature added with the next version.
The KPS dependency information is planned to be stored in a custom property and based on that also exported.

Duplicate of #478


Protocol Buffers 3 parser in Go?

Asking here as it seems the most relevant and specific audience.
Is there in existence a Go-based package for parsing protocol buffers 3 .proto files?

There's github.com/dsymonds/gotoc, but that's not an official project. Other than that, no, there's no Go-based .proto parsers that I know of.

@dsymonds thanks very much, exactly what I was after.

Having just seen https://github.com/golang/protobuf/issues/155, I thought I should note here that I am maintaining a copy of github.com/dsymonds/gotoc/internal/* here:
https://github.com/myitcv/protobuf
The code is quick and dirty for now but has added the following improvements:

Fix parser file search algo
Allow parsing of multi-line strings
Parsing of custom field options
Parsing of custom method options

Contributions welcomed and of course thanks to @dsymonds for the original code

https://github.com/emicklei/proto is another package that also accepts version 2. Thanks to @myitcv for mentioning the @dsymonds package and this issue.

@emicklei, thank you for the great work
Still, sometimes exact file positioning is needed to locate errors and warnings. For instance, we are thinking about specific documentation generation where each enum, fiield, service (grpc) and message(grpc) must have a comment and we should point exact position of an element what lacks it. Am I missed something, or does your version not having attached lexems indeed?

Related to https://github.com/emicklei/proto/issues/7, someone is working on adding LineNumbers to the nodes. Column position would also be needed.

Great!

Someone recently told me that the bnf has become available:
https://developers.google.com/protocol-buffers/docs/reference/proto2-spec
https://developers.google.com/protocol-buffers/docs/reference/proto3-spec
something that I have been waiting for years.
Maybe you will find it useful.

https://developers.google.com/protocol-buffers/docs/reference/proto3-spec

https://github.com/yoheimuta/go-protoparser is yet another package. It is designed to strictly follow the bnf. I hope this will help.

FWIW, you can convert a .proto source file into a manipulable form using protoc:
protoc --include_imports --include_source_info -oFILE foo.proto

This will write a FileDescriptorSet message (defined in descriptor.proto) containing the input files to FILE. The --include_imports and --include_source_info flags will include dependencies and source positioning information.

@neild this doesn't work that well. For instance, you don't have positions (file, line, column)  for options.

I see source_code_info annotations for options when I try it, FWIW.

syntax = "proto3";

import "google/api/annotations.proto";

message Message {
		string id = 1;
};

service Service {
		rpc Method(Message) returns(Message) {
			option (google.api.http) = {
				   post: "/v1/message"
				   body: "*"
			};
		};
};

Here's is the protoc dump: json-dump.zip
Please point, what part of source_code_info.location points to body key in google.api.http option.

            {
                "path": [
                    6,
                    0,
                    2,
                    0,
                    4,
                    999,
                    0,
                    8
                ],
                "span": [
                    10,
                    51,
                    13,
                    25
                ]
            }

Doesn't drill into the option content, so if you want to identify the position of elements within the HttpRule message you still have work to do. Still, it gets you past the .proto file parsing.


Accept future enrollment

There could be a scenario where a node does not have an
frozen UTXO externalized for an enrollment and receives
the enrollment. We improve to deal with the scenario by
making a node accept the enrollment with a transaction
that has the UTXO in the pool.
Fixes #1975

@hewison-chris thanks for the correction.

Ready for review.
Please tell me if there's something more to do.


Support for jq functions (eg keys)

Thanks for releasing this - I'm a big fan :)
Is there interest in covering off more of the jq builtins?
For instance:
echo '{"abc": 1, "abcd": 2, "Foo": 3}' | jq 'keys'
["Foo", "abc", "abcd"]

jid prevents me typing keys at all at the moment.

This could maybe include 'jq' as a submodule or something, to not reinvent the well

Shameless self-promotion: jiq, my fork of jid uses jq and supports all the fancy queries, operators and functions it offers.

So, jid doesn't actually use jq, so this can't quite be done... :(  However, @fiatjaf's jiq fork of jid does, which is pretty cool.  It might be nice to have golang bindings for libjq...  Oh wait!  There is at least one: https://github.com/threatgrid/jq-go !

That's awesome. I should use that in jiq but maybe it would be better if it got used on jid instead.

@fiatjaf yeah.
I've got a branch in my jq clone that adds FFIs, I/O, co-routines, eval.  After it gets integrated we'll be able to build a REPL out of eval.  jid shows me that a REPL shell should really use curses so as to leave the program in the same part of the terminal screen, and show/page results in another.

Besides the jq-go mentioned in https://github.com/simeji/jid/issues/33#issuecomment-486938365 above, there is also https://github.com/itchyny/gojq as a Go library for jq query.


Create HelloWorldInC.c

congratulations on your hactoberfest contribution. if you like this repo then give it a star. Happy hacking..


docs: proposal for providing PV key rotation

The design and implementation details for rotating the encryption keys for volumes.

@black-dragon74 please send PR's to https://github.com/ceph/ceph-csi not to this downstream repo


Fixed bug where Ray2/Aabb2 intersection returned false positives for rays parallel to an axis

Also adding intersection tests for reverse order aabb vs ray/ray vs aabb.

This commit fixes #25.


ASR example with pipeline for multi-modal efforts (SimulST)

Current state should work, have to double check preprocessing pipeline is functional.


Group heading on select and on hover event

Hi, I am working on a tree like drop down with single hierarchy of groups. I want to select all the elements of the group if the user clicks on the group heading. Is there a way to do it with Multiselect in ui-select?

+1

@billccn I have incorporated that in my fork. https://github.com/prakashn27/ui-select check and let me know the feedback. I have made the change in dist/select.js repo. Use that file and let me know the feedback.

+1 nice feature. I implemented your fork in my project for now, it works well.

+1

+1

May i know why my code is still not being merged?

@wesleycho

Can this be merged?


feat: add fennel-ls support

Untested on windows.

does this server not support windows?

does this server not support windows?

does this server not support windows?

I guess it's possible, but I haven't tried it yet.

so we can simply add the server cmd in windows. if it not works fine we can fix it according the error message

local cmd = 'fennel-ls'
if vim.fn.has('win32') then
  cmd = { 'cmd.exe', '/C', 'fennel-ls'}
end

I have tried in wsl2, but it's not work.
The error in lsp.log is  .../vim/lsp/rpc.lua:732  "rpc" "cmd.exe" "stderr"  "'fennel-ls' is not recognized as an internal or external command,\r\noperable program or batch file.\r\n"  , but I do not known how to compile is to a windows EXE file.

hmmm okay then just need fix the commit message.

hmmm okay then just need fix the commit message.

ok


Documentation Landing Page

@qzhou1607 we have a new css file for documentation. Can we have it pulled in only for the documentation pages, landing page included?
@alrra it looks like my commits from my other PR for search results were pulled into this PR. How can I fix this?

it looks like my commits from my other PR for search results were pulled into this PR. How can I fix this?

@ststimac Should be fixed.

@ststimac Can you fix the errors in the css file running lint?

@qzhou1607 sorry, yes, should be fixed now.

@ststimac @alrra Done.

@qzhou1607 can you please squash all these commits and add a new one following the convention of the other repo?

@molant Done. I didn't realize that this page includes a search bar, so it will be best to merge this PR after I'm done with https://github.com/sonarwhal/sonarwhal.com/pull/164. Also @ststimac I tried to fix the mixed files from the other PR, can you double check the current commit to see if it looking okay?

@qzhou1607 #164 is merged now. Can you fix the conflicts and update this PR please?

@molant Done.


provide table schema

There should be a parameter that one can provide the table schema. Default is dbo. One should have an option to provide another value, so that the table belongs to this schema.

Any word on when this will be available in the NuGet package?

It should be out there in the -dev package now 👍

Thanks for the quick response @nblumhardt. When can we anticipate the next release? How stable is the -dev package? I want to use this in production fairly soon, and can't in good conscience rely on a dev package. Thanks!

I've created the release PR for 5.0 to track it: https://github.com/serilog/serilog-sinks-mssqlserver/pull/77
The release timing depends on the level of confidence we have in the package. Any and all testing is helpful. Would you be able to try it out and check that the table schema customisation works as intended?
Thanks!

I'd more than happy to!

Tested it out and it works like a charm.

Hello.. I don't find info how to config the Scheme. Can you help me?
Currently I have this web.config and I need to add the scheme Temp:
    <add key="serilog:minimum-level" value="Debug" />
    <add key="serilog:using:MSSqlServer" value="Serilog.Sinks.MSSqlServer" />
    <add key="serilog:write-to:MSSqlServer.connectionString" value="My-Conexion" />
    <add key="serilog:write-to:MSSqlServer.tableName" value="Logs" />
    <add key="serilog:write-to:MSSqlServer.autoCreateSqlTable" value="true" />

Thanks

Testing with several options it worked with:
    <add key="serilog:write-to:MSSqlServer.schemaName" value="temp" />


Account Module - Old Project Billing Issue

@vinpahilwani , Diddi , @sureshsirvi
need Default Filter to Trim UnBilled Projects
Default Filter -
Any Project above 3 Years Create Date is ignored
Required -
Date Setting to stop showing Old project whose Create Date is Less than this Date

"Unbilled Project Minimum Date" setting Created
Not sure about where it needed to place, so testing required, in case some module may ignore this settings


Exception during interaction with TrackScheme and BranchSpot table open

How to reproduce

Open https://github.com/mastodon-sc/mastodon-example-data/tree/master/tgmm-mini
Open Feature Computer and Compute all Features
Open a TrackScheme Window
Find a cell division
Delete all spots of on daughter branch, except the first spot after the division
Click on the remaining spot of the branch where the spots have been deleted

The following exception appears:
Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: bitIndex < 0: -1
	at java.util.BitSet.get(BitSet.java:623)
	at org.mastodon.model.DefaultSelectionModel.isSelected(DefaultSelectionModel.java:137)
	at org.mastodon.model.branch.BranchGraphSelectionAdapter.isSelected(BranchGraphSelectionAdapter.java:101)
	at org.mastodon.model.branch.BranchGraphSelectionAdapter.getSelectedVertices(BranchGraphSelectionAdapter.java:289)
	at org.mastodon.views.table.TableViewFrameBuilder$GraphTableBundle.selectionChanged(TableViewFrameBuilder.java:589)
	at org.mastodon.model.DefaultSelectionModel.resumeListeners(DefaultSelectionModel.java:382)
	at org.mastodon.adapter.SelectionModelAdapter.resumeListeners(SelectionModelAdapter.java:157)
	at org.mastodon.views.trackscheme.display.TrackSchemeNavigationBehaviours.select(TrackSchemeNavigationBehaviours.java:271)
	at org.mastodon.views.trackscheme.display.TrackSchemeNavigationBehaviours.access$1100(TrackSchemeNavigationBehaviours.java:65)
	at org.mastodon.views.trackscheme.display.TrackSchemeNavigationBehaviours$ClickSelectionBehaviour.click(TrackSchemeNavigationBehaviours.java:395)
	at org.scijava.ui.behaviour.MouseAndKeyHandler.mouseClicked(MouseAndKeyHandler.java:265)

Version
Reproducable in Mastodon-Beta-31 Release

Normally this is fixed in https://github.com/mastodon-sc/mastodon/commit/c19c5abc7a475fb4ea62da8e8f434bc8dd7eac35

Normally this is fixed in c19c5ab

I just tested this with the current Dev branch. The bug is still reproducible.

Normally this is fixed in c19c5ab

I just tested this in the current dev branch. It seems to be fixed there.


⚠ sandbox-api.piste.gouv.fr healthcheck has degraded performance

In 86a881b, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 4705 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in b40a76a.


Add validation to create evidence form

Related to #1088
This pull request introduces validation logic to the CreateEvidenceModal component to ensure that both the description and evidence content fields are not empty before submission. This change addresses the issue of being able to submit blank evidence, enhancing the form's data integrity.

Adds a validateForm function that checks for non-empty description and evidence content. It generates an array of error messages if validation fails.
Modifies the handleSubmit function within useForm hook to include a call to validateForm. If validation errors exist, the submission is halted, and errors are displayed to the user.
Maintains the existing functionality of evidence type selection and file upload, ensuring that the validation logic seamlessly integrates without disrupting the user experience.

For more details, open the Copilot Workspace session.

Not sure about the react specific stuff here but we probably want to implement this on the backend rather than the frontend so that it's not possible to create on the API routes either. For quick feedback it might make sense to keep some frontend validation as well.

Implements frontend validation for creating evidence in frontend/src/pages/operation_show/evidence_modals/index.tsx and backend validation for evidence content in backend/services/evidence.go and backend/services/evidence_metadata.go.

Adds a validateForm function to check for empty description and content fields before evidence creation on the frontend.
Integrates form validation checks into the evidence creation process, rejecting the operation with an error if validation fails.
Implements backend validation in CreateEvidence function to ensure the description is not empty and content is provided for evidence types other than 'event'.
Adds validation in CreateEvidenceMetadata and UpsertEvidenceMetadata functions in evidence_metadata.go to ensure source and body fields are not empty, enhancing data integrity for evidence metadata operations.

For more details, open the Copilot Workspace session.

There's probably a better solution than what copilot provided this first shot. Going to try some other attempts at this issue.

One other thing that could be done would be to introduce validation at the dissector level. I didn't finish this, but it could look something like this:
// inside Coercable, we'd introduce some validator:
type ValidateFunc func(fieldName str, value interface{}) error

// Then, we'd create custom validators -- like this one to ensure a (string) field isn't empty...
func strNotEmpty(fieldName string, v interface{}) error {
  s, _ := v.(string)
  if strings.TrimSpace(s) == "" {
    return fmt.Errorf("%v cannot be empty", fieldName)
  }
  return nil
}

// we'd then use it when validating routes
	route(r, "POST", "/operations/{operation_slug}/evidence", jsonHandler(func(r *http.Request) (interface{}, error) {
		dr := dissectFormRequest(r)
		i := services.CreateEvidenceInput{
			Description:   dr.FromBody("description").Required().Validate(strNotEmpty).AsString(),
			//                                                   ^^^^^^^^^^^^^^^^^^^^^ Tell the dissector to flag this as an error if it fails validation
       // rest of the function

There are a couple of downsides:

This works only at the field level, not at the request level -- so your check of contentType != event && content != nil won't work here.
While some validation occurs here (specifically vetting the incoming shape of the data), most of the validation occurs within the handler, as it understand what is required to complete the function. Adding validation here could be seen as a breaking of this pattern
The validation is somewhat inefficient, because it has to operate on data as an empty interface, rather than the real type, so in nearly every situation, we'll have to coerce the value back into the proper type (again).

Separately, we could look into using this (popular) library: https://github.com/go-playground/validator


improved refresh_rate fix, now actually works on OS X...

So the last refresh rate fix didn't really fix anything, in fact it broke resolution without refresh rate all together when using multiple screens... Turns out the refresh rate was never removed at all when you have multiple screens. Now it works perfectly, by looping over the individual refresh rates and removing them there (all still in pure bash)

This seems a little too complicated.
What's the resolution output normally? 1920x1080 @ 60hz, 1280x720 @ 30hz ?

I'm all for a simpler solution. The output I get normally in my setup is 1920x1080 @ 60Hz, 1440x900 @ 60Hz, 1440x900 @ 60Hz

This way works with 2 lines:
resolution=${resolution// @ [0-9][0-9]Hz}
resolution=${resolution// @ [0-9][0-9][0-9]Hz}

I'm trying to reduce this to one.


Implement str_len method for StringList

Does this method behave like this:
>>> import ulist as ul
>>> strs = ul.from_seq(["hello", "world!"], dtype='string')
>>> strs.str_len()
UltraFastList([5, 6])

Does this method behave like this:
>>> import ulist as ul
>>> strs = ul.from_seq(["hello", "world!"], dtype='string')
>>> strs.str_len()
UltraFastList([5, 6])

Exactly, the dtype of the result should be int64 by the way. Please feel free to submit a PR if you are interested.


osds_per_device is used literally, breaks all batch ceph-volume tests

Bug Report
What happened: Undefined variables are being used, still coming from PR #3113 from commit 27027a17d
In this case, the following error is raised:
ceph-volume lvm batch: error: argument --osds-per-device: invalid int value: 'osds_per_device'

Because the command being sent to ceph-volume is not using a integer:
      command: "ceph-volume lvm batch --report --format=json --osds-per-device osds_per_device {{ devices | join(' ') }}"

In that case osds_per_device is being used literally.
What you expected to happen: the correct number of osds per device is used
How to reproduce it (minimal and precise): Attempt to run any batch test case for ceph-volume using the current master version of ceph-ansible

TASK [ceph-config : get number of osds for lvm-batch scenario] *****************
task path: /tmp/tox.hTs4ghLxdR/centos7-bluestore-mixed_type/tmp/ceph-ansible/roles/ceph-config/tasks/main.yml:27
skipping: [mon0] => {
    "changed": false,
    "skip_reason": "Conditional result was False"
}
fatal: [osd0]: FAILED! => {
    "changed": true,
    "cmd": [
        "ceph-volume",
        "lvm",
        "batch",
        "--report",
        "--format=json",
        "--osds-per-device",
        "osds_per_device",
        "/dev/sdb",
        "/dev/sdc",
        "/dev/nvme0n1"
    ],
    "delta": "0:00:00.296023",
    "end": "2018-09-18 18:26:49.871305",
    "rc": 2,
    "start": "2018-09-18 18:26:49.575282"
}

STDERR:

usage: ceph-volume lvm batch [-h] [--bluestore] [--filestore] [--report]
                             [--yes] [--format {json,pretty}] [--dmcrypt]
                             [--crush-device-class CRUSH_DEVICE_CLASS]
                             [--no-systemd]
                             [--osds-per-device OSDS_PER_DEVICE]
                             [DEVICES [DEVICES ...]]
ceph-volume lvm batch: error: argument --osds-per-device: invalid int value: 'osds_per_device'

Testing out with these changes:
-      command: "ceph-volume lvm batch --report --format=json --osds-per-device osds_per_device {{ devices | join(' ') }}"
+      command: "ceph-volume lvm batch --report --format=json --osds-per-device {{ osds_per_device | default(1) }} {{ devices | join(' ') }}"


Cached types.Packages cannot be unloaded

Similar to https://github.com/KurokuLabs/margo/issues/97
While experimenting with a TypeCheck linter that's always running (while typing), I observed some failures due to types not matching even though their names are identical.
This will need further investigation, but it appears to be caused by the unloading of cached packages.
Theory:

A depends  on B so when we import it, B is also imported and cached.
B gets edited so it's dropped from the cache.
C depends on A and B so when we import it, A is taken from the cache and B is re-imported.
When we type-check C,  we get a typecheck failure because the B imported by C is not the same B that was imported by A.

Kim-Porter doesn't have this problem...


FE-homework-12

Metric
Value

alert_status
b:white_check_mark: OK/b

bugs
2

code_smells
13

reliability_rating
C (at least 1 Major Bug)br/Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)br/Note: A being best, E being worst

vulnerabilities
0


"GraphQL error" shows up on all Expenses pages

Describe the bug
"GraphQL error" shows up on all Expenses pages.
To Reproduce
Steps to reproduce the behavior:

Go to a collective like https://opencollective.com/streetlives/expenses
See error.

Expected behavior
Seeing expenses rather than a blank error page.
Screenshots
Desktop

OS: [e.g. iOS] macOS Mojave
Browser [e.g. chrome, safari] Chrome Version 78.0.3904.108 (Official Build) (64-bit)
Version [e.g. 22] 10.14.6

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
I only checked pages under https://opencollective.com/foundation.

Hi @audreyr, thanks for reporting this bug.
Is this still happening on your side? I tried to reproduce but the page works on my machine:

Closing. Feel free to reopen if this happens again in the future.


Standard Mode not working

Hi,
is there also a test-program for the standard mode available. I do not get it running when I change the mode to standard in the example program:
`#include "rpLidar.h"
#include "rpLidarTypes.h"
rpLidar lidar(&Serial2,1000000);
void setup() {
Serial.begin(115200);
lidar.resetDevice(); //reset the device to be sure that the status is good
lidar.setAngleOfInterest(5,175); //Set the field of view that is saved to Data
lidar.start(standard); //start the express scan of the lidar
}
void loop()
{
lidar.readMeasurePoints();// reads a full scan and save it to Data
for(int i=0;i<sizeof(lidar.Data)/sizeof(point_t);i++)
{
Serial.print(lidar.Data[i].angle);
Serial.print("deg\t|\t");
Serial.print(lidar.Data[i].distance);
Serial.println("mm");
}
}`
My problem is that I need the data continuously and save it in a background task to a buffer where I can get the newest measurements. They should be overwritten at least all 200ms that they are real time.
How can I solve this?

@jakobofner
You can read the Data out of stScanDataPoint_t DataBuffer[1500].  In your example lidar.data is alway empty becouse it´s the Buffer for the express scan.
If you need the Data fast, you should use express scan. It is faster and also more accurate.

copy the Buffer after a scan and then use the Data.
programm a cicular buffer

Thats not part of this repo and it´s not planed to include this.


DOC: clarify use of shapely.transform vs shapely.ops.transform

Documentation of shapely.ops.transform does not reference shapely.transform, and vice-versa.  Despite the clarification that shapely.transform is a newer addition that was ported in from pygeos.apply, I'm still confused as to which transform I should use when, where and why.
The documentation can be improved by:

Referencing the alternate transform function from within the documentation of at least one option
Explaining which one is preferable for some/all use cases

Discussed in https://github.com/shapely/shapely/discussions/1895

Originally posted by ijessen-mitll September 26, 2023
The two transform functions seem to significantly overlap each other:

shapely.ops.transform(func, geom)
Applies func to all coordinates of geom and returns a new geometry of the same type from the transformed coordinates.

shapely.transform(geometry, transformation, include_z=False)
Returns a copy of a geometry array with a function applied to its coordinates.

There are very slight differences in their parameters, and some logic within ops.transform seems to try to suss out the interface of the provided transform function.  Besides that, can anyone speak to why both exist?  What are the independent use cases for each?  Can the documentation be improved to direct the use of one over the other?

Hi @ijessen-mitll ,
You raise a valid point. Activity on shapely has been kind of low as of late, however, we are working on it. Ref. https://github.com/shapely/shapely/pull/1902
The reason that the current situation is there is that the ops.transform was there in 1.x and we left it there to not needlessly break existing code. The new transform is new in 2.x and is better in most situations.
For a detailed review of differences, please look at https://github.com/shapely/shapely/pull/1902 (mind that this is a work in progress).
I hope this answers you questions?


Initial MongoDB setup for MSUnmerged.

Fixes #10798
Status
Ready
Description
This PR is providing the basic functionalities we need to be able to write the proper structure to MongoDB for the MSUnmerged micro service.
Is it backward compatible (if not, which system it affects?)
NO, once we have that merged we  need to have a MongoDB instance configured and running. It will also break several stuff related to the microservice work itself:

We may no longer rely on refreshing the timers from memory, since we are about to be preserving those in the database
We will keep accumulating garbage records in the fields  of the RSE object if we just update and never clean the RSE record at the database upon refresh of the RSE read on the RucioConMon side

Related PRs
Here are the configuration commits related to the initial MongoDb setup [1]. We are using the MongoDB as a service instance so far and distinguishing between service instances by database names (fetched from service_config).
[1]
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/121/diffs
https://gitlab.cern.ch/cmsweb-k8s/services_config/-/merge_requests/122/diffs
External dependencies / deployment changes
We need to propagate the correct passwords in the relevant password file (this is happening in a private mode with the cmsweb Team)

Hi @amaltaro, since we just introduced MongoDB into the picture, those unit tests for MSUnmerged started to fail, because the machine to run the tests at Jenkins has no access to any MongoDB instance, neither as a service nor local.  As of code-wise the changes are now completed and well tested manually with the already existing MongoDB as a Service instance from my virtual environment and the results were all fine.
So I'd suggest you may start you review of the code until I figure out what to do with the broken unit tests.
BTW any ideas on how to simulate the database or to connect to the already existing one without exposing any usernames or passwords here are very welcome.

And just for refreshing, we had the very same problem with unit tests for MSOutput and it is still not resolved. The GH issue related to the problem is this one: https://github.com/dmwm/WMCore/issues/9789
There are few options here.:

The simplest one - we can install MongoDB at the machine running the tests - it will always work since all services by default point to local host.
Use libraries as mongomock for in-memory emulation of MongoDB. Or a test library relying on the above one and available through pipy pytest-mongodb. Still need to figure out if this would be just a database emulation or a full server emulation. Because we will actually need the full server emulation to simulate  the communication protocol itself. Reason is, the MongoDB connection for our objects happen at init time [1] so the server client communication is unavoidable.
The third option would be working just for database emulation with some simple code like:

@mock.patch("pymongo.collection.Collection.find")
def test_name(self, mock_find):
    mock_find.return_value = MongoDummybject(return_data)

NOTE: example taken from stackoverflow
[1]
https://github.com/dmwm/WMCore/blob/dd3d3252b2f545ef84d65573d3e69216da06ad4a/src/python/WMCore/MicroService/MSUnmerged/MSUnmerged.py#L121

Hi @amaltaro as we spoke earlier today I did add the functionality to write the whole RSE object to the database. And I did keep the PR split in two separate commits, but the first one has some broken database write logic, which I've  fixed in the second one. So I'd suggest we go with them both and not looking back.
The databse read && write operations are now well tested for every possible case both manually and combined with the calls from MSUnmerged code itself. What I didn't do so far, is changing the flag name for triggering the behavior (dumpRSE). This will require some changes in the config repositories as well. I'd better proceed with the whole change in a separate commit, which may also include the needed configuration changes to add a simple connection to MongoDB as a service.

@amaltaro Alan please take another look. I think I have addressed most of your comments. Some had to be left for the other issue. And I am currently working  on the configuration PR separately. I'll push that  one soon too. Thanks!

Hi @amaltaro I just did all requested :

Squashed all the commits
Made the configuration changes
Properly linked in the comments
Variable renamed

Could you please take another look
Thanks in advance!

Even though we want to stop using our VMs for central services tests, we still do not have a full k8s dev/test setup. So here is a fix that should have gone with this PR: https://github.com/dmwm/deployment/pull/1115


⚠ Bofrost.fr has degraded performance

In 49928f9, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1574 ms

Resolved: Bofrost.fr performance has improved in b17b800 after 9 minutes.