Set the Ready condition to False when a CertificateRequest has been denied for all CertificateRequests that reference a cert-manager.io signer

This PR changes all internal signers so that they set the Ready condition to False with the Reason "RequestDenied" when a CertificateRequest is Denied. This is to keep the same behavior where a terminal state of a CertificateRequest should have a Ready condition.
This should be backported to release-1.3 and released in v1.3.1.
/assign @munnerz
Set the Ready condition to False when a CertificateRequest has been denied for all CertificateRequests that reference a cert-manager.io signer

/cherry-pick release-1.3

Set the Ready condition to False with the Reason "RequestDenied" when a CertificateRequest is Denied. This is to keep the same behavior where a terminal state of a CertificateRequest should have a Ready condition.

I struggle to understand why this change is needed. From my perspective, this feels like a new feature, what is the rationale for backporting it?

This is to keep the same behavior where a terminal state of a CertificateRequest should have a Ready condition.

I struggle to understand why this change is needed and who (or what) will benefit from it. Might be worth adding the user stories in the PR description.

This should be backported to release-1.3 and released in v1.3.1.

From my perspective, this feels like a new feature, what is the rationale for backporting it?

There could be CR controllers which are just watching the ready condition. Since we never set it, they would consider a denied CR as unprocessed and pending forever. This PR keeps the same behaviour of all terminal states having a ready condition.
We don’t use a Failed reason, since that is semantically different to Denied.

To be clear, this is a regression of API. Previously, clients of the API could rely on the Ready condition being populated. This reinstates this behaviour.

There could be CR controllers which are just watching the ready condition. Since we never set it, they would consider a denied CR as unprocessed and pending forever. This PR keeps the same behaviour of all terminal states having a ready condition.
We don’t use a Failed reason, since that is semantically different to Denied.

The docs say: https://cert-manager.io/docs/concepts/certificaterequest/#ready

Failed: The certificate has failed to be issued - either the returned certificate failed to be decoded or an instance of the referenced issuer used for signing failed. No further action will be taken on the CertificateRequest by it’s controller.

So Failed already covers two cases: permanent Issuer failure, or failure to decode the certificate set by the issuer.
So I'd argue that Failed could also be used for the Denied case.
That text also implies that the Failed reason can be set by a different controller than the issuer CR controller. i.e. issuer sets Ready: Issued  + corrupt signed certificate bytes.
Then some other cert-manager controller changes the condition to Ready: Failed after it detects the corrupt certificate.
If that is the case, then I'd argue (again, see above) that cert-manager should set the Ready: Failed condition at the same time it sets the Denied condition.
And we should probably update the sample-external-issuer to ignore CRs that are Ready: Failed, like here: https://github.com/cert-manager/sample-external-issuer/blob/4e8bff2ce903d939538730b5e5b2883a522cdc8d/internal/controllers/certificaterequest_controller.go#L84

/lgtm
/hold cancel

/cherry-pick release-1.3


Push Replication fails randomly

Hello,
I'm noticing something really strange: We've a small number of clients Running right now ( around 100 ) creating docs locally and pushing them to one remote sync gateway server and pull a few docs once in a while ( channel filtered replication ). Now i'm seeing from time to time clients that won't submit their newly created documents until the whole app restarts. It seems to happen especially if the client updated his Apk just before. ( App is Running - App Update - and now continue working with the app seems to fail often )
To make it short the Push Replication itself works but won't send any Data.
While monitoring everything i've seen that replication requests got executed correctly but the client didn't tried to send anything until the bespoke app restart.
It's important to say that the client is parallel tapped in a pull replication feed which didn't have any updates for him.
Anyone has an Idea where to look for this issue? I'm seeing this nearly daily on a "random device". So it's really hard to tell where to look for.
If i have to guess i would say it has something to do with loading the last replication sequence (checkpoint) from database.
Best Regards,
Leon

Hi @K-Leon,
Some questions:

Can you check Sync Gateway Log? Does Sync Gateway receive requests from clients?
Can you check LogCat on Android? Can you see any error message? and Can you see the indication of sending data to the sync gateway?

As you said, restarting the app can fix the problem. I wonder if the application loose internet connection or if replication is stopped after APK updated.
Please share further detailed information.
Thanks!

Hi @hideki
I had luck and could start a Logcat tool on an instance that was already "broken". My findings:
a) Internet Connection present - WIFI
b) Sync Gateway received requests - even though it had been POST request
c) I stopped Sync Gateway - looked at the logs and restarted it. I have seen connection failures coming from continious pull and of the One Shot Push Replication i ran additionally on a regular base to rule out issues with continious replication after seeing this issue the first time. Even though no PUT Request have been issued - only the local, Change Tracker requests etc. have been visible on a regular base.
d) No Error message - nothing that indicates something is going wrong. At least nothing "continious" - i could only tap in when it did happen already.
So this leaves me with two ideas:
a) Change Listener attached to Database failed
b) Mismatch between local and remote replication id.
As authentication i use basic auth.
Best Regards,
Leon

Hi @hideki,
I dove a little bit in the Code and i'm currently suspecting this part: https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/replicator/ReplicationInternal.java#L1020
Am I right: if there's an actual mismatch between both state the current revision isn't reset? I would expect that the replicator starts from the remote rev or starts over from the beginning ( rev 0 ).
Best Regards,
Leon

start over from 0.

@hideki
I just stumbled upon an instance with upgrade issues - this one was totally dead because upgrade caused an SQLiteDatabaseCorruptException.
Is it possible that the SQLite Database won't get closed correctly before updating?
Small snippet of the Log - it goes like this all the time: https://gist.github.com/K-Leon/28bda2a04970a09e29aaa76356cf8d8c

Hi @K-Leon,
Do you close the database in Application.onTerminate() method?

Apparently not: https://github.com/couchbaselabs/react-native-couchbase-lite/blob/master/android/src/main/java/me/fraserxu/rncouchbaselite/ReactCBLite.java
I'll look tomorrow into adding this - thank you very much! This could explain all of the weird behavior.

Sorry :) Now i started working on it and could read the docs - the whole Database won't get closed. Just added an onDestory Hook and i'll let you know if i'm still experiencing this behavior.

@hideki I dig a little bit in - but i haven't found a solution to close Databases on App Upgrade.
03-24 11:12:46.768 790-812/? I/ActivityManager: Force stopping de.thisismy.app appid=10102 user=-1: uninstall pkg
03-24 11:12:46.769 790-812/? I/ActivityManager: Killing 10740:de.thisismy.app/u0a102 (adj 9): stop de.thisismy.app
This won't stop Couchbase Lite Manager - it keeps running and will easily fail while device moves Data around. Mostly nothing bad happens but from time to time we see strange things happen.
Is there a simple way to kill all already running instances before creating a new one or something like this?

@K-Leon,

Non of REST API methods explicitly close the database. We need to add the way to close the database through REST API.

https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/router/Router.java

Can you confirm Application.onTerminate() is called when Application is upgraded?
If yes, I think this is right time to make sure if database is closed.

Application.onTerminate ist never Called on Production Devices - so it's out. I tried all day to find a hook that allows execution of closing before running the upgrade without success.
The disturbing thing i noticed Lite Listener and Subsequently Couchbase Lite kept alive - even though App updated and didn't restarted Lite Listener / Couchbase Lite.
The best thing would be binding the CB / Lite Listener Lifetime to the Calling Process.

Hi @K-Leon,
I am not sure how application update behaves on Android. Also, I am not familiar with react-native. It seems Listener (web app for REST) threads become a deamon thread after the upgrade?
Do you use CBL 1.3.1 from react-native? Can you try 1.4.0 instead? We compiled SQLite serialized mode which allows multiple connections. It might prevent to cause Database Corruption.
https://github.com/couchbaselabs/react-native-couchbase-lite/blob/master/android/build.gradle

Hi @hideki
I'm already on 1.4

Yes Listener somehow Deamonizes - RN isn't really different to all other apps using rest API.

Hi @jamiltz,
do you have any idea about this issue?


Missing Action setting for Button (for Drill through)

Hi, that was an awesome experience using the new format pane. But, here is the thing - I liked the new option to navigate / bookmark pages for buttons but I am using Drillthotugh to buttons which I usually select through the action setting, but now in the new format pane, I couldn't see that. Can anyone explain how can I do that using the new Action pane?

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

ID: 99728831-9577-faee-e82a-8def045dadea
Version Independent ID: 99728831-9577-faee-e82a-8def045dadea
Content: The new Format pane in Power BI Desktop (preview) - Power BI
Content Source: powerbi-docs/fundamentals/desktop-format-pane.md
Service: powerbi
Sub-service: pbi-fundamentals
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi @NikhilPodishetti -- good catch, thanks for asking! A new release of Power BI Desktop should be coming out this week, and it'll feature updates to the Format pane. Check it out and see if that answers your questions!

Thank you. That's all I need :)


Update documentation

Add information on using toil-lib and Docker to develop toil pipelines
Remove Spark documentation that is being moved to toil-lib

@benedictpaten these aren't done yet, but I'd appreciate comments!

@almussel , please make updates asap, will review this afternoon.

@benedictpaten @almussel is this still being worked on?

@almussel @fnothaft What's the block on this? I am about done with known bugs and service deadlocks. I'm pretty motivated to do some spark service testing!

@benedictpaten @cket This is in my court. I've been distracted with a different project but I'll get back on it this week.

I know you have other things going on @almussel , but this is a bit insane. Getting impatient.

@benedictpaten I've addressed your comments

@benedictpaten do you want to take another look before I merge?

@benedictpaten I've added additional docker_call documentation. Will you review?

Couple of small changes. Thanks @almussel!

Docker documentation has been moved to #1403

@almussel Close this once #1403 is merged.

@benedictpaten What about the other parts of this pr?

Hmm, I don't like toil-lib. I would like to have toil-spark (as we discussed offline) and to then have a section (few sentences) pointing from toil to it.

Closing this for now.


[v4] Prisma client raw (SQL) query Action

Supports executeRaw and queryRaw

[x] queryRaw
[x] executeRaw

https://prisma.pub/queries/raw-database-access.html


transfer screens update

What does this PR?
This PR updates the design of transfer screens according to design of delegate screen. For now I had just moved the same parts from transfer screen to separate components and updated the UI style as used in delegate screen. Functionality is still dependent on code in transfer screen.
Note: Promote and Boost screen is not updated as they had different design than the other transfer screens
Issue number
fixes #2370
Screenshots/Video
https://user-images.githubusercontent.com/48380998/179895403-b98e6526-ec10-4d23-a668-498a56d29ae2.mp4

@feruzm updated keybaord aware view and tested on both android and ios


Sound plays through speakers with connected bluetooth device on Mac

OS: MacOs 10.14.6
GPMDP Version: 4.6.1
Issue Descriptions: Sound not playing via bluetooth. System sound goes via bluetooth as expected but gpmdp plays through speakers at the same time anyway
Steps to Reproduce: connect bluetooth device, try to play anything

Having the same issue after updating to MacOS 10.15

There is a device picker in settings that may be causing this. It's possible for the app to select its own audio device output. Make sure yours is set to whatever the device is you want it to play through.

I have this same issue. But, even if I select the bluetooth audio device from the list in Audio Devices, it does not play through bluetooth. I have to exit the app and reopen it for it to work. Likewise, if I have it playing through bluetooth, then disconnect bluetooth, I am unable to play through the speakers (even if I pick the device from the list) unless I quit and reopen the app

My issue is similar except that the music doesn't come out the bluetooth headset or the builtin speakers.


CommandRecommender should be called after the error is shown

Description
When a command fails, it take longer to show the error message than before (without Aladdin) because CLI is querying Aladdin which involves a network request:
> az vm disk list
CommandNotFoundError: 'list' is misspelled or not recognized by the system.
Try this: 'az disk list --resource-group <myresourcegroup>'
Still stuck? Run 'az vm disk --help' to view all commands or go to 'https://aka.ms/cli_ref' to learn more

Possible solution
It will be more user-friendly if CLI can show the error message
CommandNotFoundError: 'list' is misspelled or not recognized by the system.

first and then do the Aladdin query.

Error handling performance


Implement null insert on glass type delete

This will allow deleting glass types even if they have cocktail assigned to them
Implement programmatically, handle cocktails index updating

Implemented via code


import restructuring

based on #36
I also added a basic instantiation test, so at least we'll be able to tell if something fundamental has broken across python versions.

ping @justinkterry

@fabianvf remind me if i don't get to this this weekend.

@fabianvf
What exactly is the all for? It works with it, but I don't follow after googling it.

__all__ defines the objects that will be exported when you import *


Enhanced Conversation Organization with Optimized Folder Structure

Building upon the initial implementation in PR #296, this pull request introduces further optimizations to the conversation organization feature within our chat interface. The new enhancements aim to streamline user experience and improve performance.
Key Updates:

Refined folder-based structure for better categorization and management of conversations.
Enhanced UI interactions for creating, renaming, and deleting folders. Currently only a flat list.
Seamless integration with the new UI changes introduced by Big-AGI.

This update addresses feedback received on the initial version and aligns with the sleek design of the new UI. I look forward to your suggestions and further feedback.
@enricoros, I've seen your message and the exciting news about Big-AGI's new UI. I have ported the code to the main branch as part of this update. Thanks for the New Year wishes, and I hope you've had a great start to the year too!
Please review the changes at your convenience, and let's make our chat interface even better!
Screenshot:

Thanks for porting the changes. I like that your work and the changes in main complement each other, and thanks for doing the port, much appreciated. I've made some changes on main for animations and your patches work well with that too.
Working on your changes now. I'll verify it all, check if there are optimizations, and the merge.

Note sure I'll finish the merge today, as I found a couple of bugs in my code as I was code-reviewing yours.
The work-in-progress is on https://github.com/enricoros/big-AGI/tree/joriskalz-folders (branch), and will be merged into main (with all my fixes as well) ASAP.

Great news! MERGED :)
Here is positive feedback from the review:

Code for UI and graphics is really good great CSS work!, as well as the 'state.'
I've done some minor UI changes to have a consistent style with the rest of the app

not sure about removing the ok/cancel from the folder creation; it seemed good

the feature looks good with the new UI

Here are the issues that need to be resolved:

Please minimize the changes in a PR in the future. In the updated code, AppChat was re-formatted, which is okay for a new file, but not for a core file that needs review. I had to review and verify each line, and I went through many hundreds of lines. It took 3 hours on that file. The "diff" on that file is overflowing here on GitHub. It does not do good to the "git blame" (history), and it makes it really hard for me to see the changes character by character.
I like the react-beautiful-dnd library; despite adding 30kb to the app, I'm leaning towards keeping it for now and eventually working to remove it in the future
[Scroll issues] with many folders, the chats disappear, as there are no scrollbars - what can be a good scrolling mechanics for folders + chats?
[Mobile?] I have not tested it on mobile yet

Please test and let me know if the feature is working or what are the edge cases, or if I disabled some functionality while merging.

Hey @enricoros,
Thrilled to see the PR merged – and your UI refinements are spot on! The animations add a nice touch, and the tweaks to the new folder feature really polish it up. Also like the many improvements like putting the colors into folders-store.ts.
On the chat folders, I believe the current flat list is a good starting point for chat organization. It's straightforward and already adds value. While a tree structure could be an option down the line, focusing on AI to auto-sort chats might be the more efficient route for now.
Addressing the issues:

I apologize for the oversight with the formatting changes in AppChat. I understand the hassle it caused and will be more vigilant about keeping PR changes focused and minimal in the future.

Regarding the react-beautiful-dnd library, I'll look into alternatives that are more size-efficient. However, if we find more places to use drag and drop, it could validate the library's size.

On the mobile front, I've noticed a z-index issue where the folder menu is obscured by the side menu.

Thanks for your patience and for the valuable feedback. It's a great learning experience for me, and I'm keen to keep improving.

Hi, thanks for the comments: can you attach a screenshot of the Mobile issue? I cannot reproduce and would love to release this today as Big AGI 1.10.0

I tested it on IOS and also in Chrome on the desktop:

@enricoros, also I made a short video to demo the feature. Maybe you can make use of it. Let me know if you need more specific ones.
Video Folders:
https://github.com/enricoros/big-AGI/assets/1590910/6c2f37a5-cb77-4ebc-bf99-3ac3773041bc

@joriskalz Thanks for showing the Mobile issue! Now I know. Let me fix that!
And the video looks AMAZING - I'm gonna share it right away on Discord, let's see if we get comments.

@joriskalz I've finally added the Folder selector to the Chat Items in the drawer as well.
https://github.com/enricoros/big-AGI/assets/32999/9edc84b9-0f1c-4e34-a137-4744454546ab


How to enable zooming by pinch gesture on mobile devices in presentation mode?

There are a lot of issues related to these topics. I'm a little bit confused with them, and after reading them all I still don't know if the solution of my problem exists.
Pdf.js viewer in presentation mode on mobile devices doesn't react to any gestures: press, swipe, pinch/zoom. But if PDF has big format (A4, for example), it's impossible to read it without zooming, because the document fits to screen and font is too small in presentation mode.
I've hacked viewer.js file and switched on the zooming in presentation mode for desktop browser (for example, if you click Ctrl + "+" in Chrome, the doc is zoomed in, and if you click Ctrl + "-", it is zoomed out). But PDF viewer doesn't react to zoom/pinch gesture on mobile phone touchscreen.
I've tried to attach Hammer.JS to html element with id="viewer". I haven't got an error, but my handler of "pinch" event wasn't fired too.
Is there any possibility to use zoom/pinch gestures on mobile devices in presentation mode?

Zooming was purposely disabled in Presentation Mode, please refer to the discussion in issue #5336; this is thus out-of-scope for the default PDF.js viewer.
Closing as answered.

Hello bevalorous,
Can you please tell me how you have  hacked viewer.js file and switched on the zooming in presentation mode for desktop browser (for example, if you click Ctrl + "+" in Chrome, the doc is zoomed in, and if you click Ctrl + "-", it is zoomed out)?
Thanks

Hello @ibaritakis,
Just open web/viewer.js file and find out these pieces of code:
if (!isViewerInPresentationMode) {
    PDFViewerApplication.zoomIn();
}

and
if (!isViewerInPresentationMode) {
    PDFViewerApplication.zoomOut();
}

You should change conditions in these if-statements to something that always evaluated to true, for example, if (true || !isViewerInPresentationMode) {.
That was my dirty trick ;-)

Thanks @bevalorous!
I also did
`
enableHandToolOnLoad: true,
enterPresentationMode: function handToolEnterPresentationMode() {
if (this.handTool.active) {
this.wasActive = true;
//this.handTool.deactivate();
}
},
exitPresentationMode: function handToolExitPresentationMode() {
if (this.wasActive) {
//this.wasActive = null;
//this.handTool.activate();
}
}`
for enabling hand tool so you can move around when zoomed in

Thanks @ibaritakis !

Hi @ibaritakis
where did you add this code? can you please mention the file names to be more clear.
Thanks


Scroll to Top Button

Is your feature request related to a problem? Please describe.

Make a scroll to top button using html css and Javascript.

Describe the solution you'd like

Button should be visible at every part of the page(not only at the bottom).

hey! @urvashi-code1255 I want to work on this issue  . should I make different structure or should I add it to main webpage  ?

@Nagsenwaghmare Add it to the main webpage itself

hey urvashi I am done with the scroll to top button , Have a look :blush:
https://user-images.githubusercontent.com/64947831/107626255-87375c80-6c83-11eb-9bc7-160ae6318ae0.mp4

hey urvashi I am done with the scroll to top button , Have a look 😊

Hey @Nagsenwaghmare Please change this top button to some up arrow and also change its color to some grey or black.
Also pull the recent changes from upstream before making a PR.

I am unable to pull changes from upstream  urvashi  can you please help ?
when I added  remote it is showing" remote upstream already exists". but when I am fetching it ti is showing  "Repository not found".

hey @Nagsenwaghmare Try to do "remote -v"
if it gives 4 urls( 2 of origin and 2 of upstream) then you have added upstream.
But if it gives only 2 urls of origin then you have to do "git remote add upstream https://github.com/Vishal-raj-1/Awesome-JavaScript-Projects.git"

@urvashi-code1255 does this looking good should I make PR?

@urvashi-code1255 does this looking good should I make PR?

@Nagsenwaghmare Increase the size of arrow not circle !! You can increase fontsize !!

@Nagsenwaghmare Increase the size of arrow not circle !! You can increase fontsize !!

done!


[Feature Request] Hotkeys to Enable/Disable the feature

Feature Request
It would be great to have hotkeys to use them in macros/etc. to disable / enable the plugin features.

Hi!
I took over this plugin in a fork that you can find here. Installed Obsidian plugins should auto-update to it automatically. The first release is basically the same, with the #5 proposal added that addresses #2.
If this issue is still relevant to you, please submit your issue there too.

My bad, issues weren't open on my repo for some reason. They are now.


Tinkerbell generate worker nodes capi objects

Issue #, if available:
Description of changes:
Implemented code to generate worker nodes CAPI objects for Tinkerbell provider
Testing (if applicable):
unit tests
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/retest

/approve


node-sass 4.13 incompatible with latest node 12 version (Utf8Value)

Overview
Using latest version of node-sass with latest version of node 12 fail with this error:
/Users/jeandat/.node-gyp/12.13.0/include/node/v8.h:3046:5: note: candidate constructor not viable: no known conversion from 'v8::Local<v8::Value>' to 'const v8::String::Utf8Value' for 1st argument
    Utf8Value(const Utf8Value&) = delete;
    ^
/Users/jeandat/.node-gyp/12.13.0/include/node/v8.h:3039:5: note: candidate constructor not viable: requires 2 arguments, but 1 was provided
    Utf8Value(Isolate* isolate, Local<v8::Value> obj);
    ^
1 error generated.

Full error log
Trace
Context
❯ npm -v
6.12.0

But I'm using yarn:
❯ yarn -v
1.19.2

❯ node -v
v12.13.0

❯ node -p process.versions
{
  node: '12.13.0',
  v8: '7.7.299.13-node.12',
  uv: '1.32.0',
  zlib: '1.2.11',
  brotli: '1.0.7',
  ares: '1.15.0',
  modules: '72',
  nghttp2: '1.39.2',
  napi: '5',
  llhttp: '1.1.4',
  http_parser: '2.8.0',
  openssl: '1.1.1d',
  cldr: '35.1',
  icu: '64.2',
  tz: '2019a',
  unicode: '12.1'
}

❯ node -p process.platform
darwin

❯ node -p process.arch
x64

❯ node -p "require('node-sass').info"
node-sass       4.13.0  (Wrapper)       [JavaScript]
libsass         3.5.4   (Sass Compiler) [C/C++]

❯ npm ls node-sass
<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └── UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
❯ yarn list --pattern node-sass
yarn list v1.19.2
├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>

We actually build and test with this combination in CI. It's likely that you're running into uses somewhere with the combination of Yarn and NVM handling native modules at all, rather than anything particularly with node-sass. the errro line is coming from core Node headers

We actually build and test with this combination in CI

Are you testing with yarn too?

It seem to me is that the compile error occurs in /Users/jeandat/Developer/jeandat/liveaction/project-eyelo-ng2/node_modules/@angular-devkit/build-angular/node_modules/node-sass/bui ld/config.gypi which contains node-sass 4.11 and not 4.13.
Y

I'm going to migrate to angular 9 when it is released which will upgrade @angular-devkit/build-angular dependency. Thus if problem persist with angular 9 I will post an update. Thanks.

What version of node-gyp is being used?

What version of node-gyp is being used? The error log above says:

gyp info using<EMAIL_ADDRESS>
Does upgrading to<EMAIL_ADDRESS>change things? https://github.com/nodejs/node-gyp/releases

Didn't help for me. But downgrading node to 10 did.


Add main help for java sdk

Codecov Report

Merging #67 into dev will decrease coverage by 0.20%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev      #67      +/-   ##
============================================
- Coverage     36.32%   36.11%   -0.21%
  Complexity      127      127
============================================
  Files            47       48       +1
  Lines          1250     1257       +7
  Branches         79       79
============================================
  Hits            454      454
- Misses          779      786       +7
  Partials         17       17

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/webank/wecrosssdk/Main.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...crosssdk/performance/BCOS/BCOSPerformanceTest.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...performance/Fabric/FabricSendTransactionSuite.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

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 3fdfdbe...3d71ddf. Read the comment docs.


Documentation and Demo request

Thanks for the good work with the react-hook-tracer project, i really applaud you for that.
I believe your projects specifically this one needs more traction, purposefully because of its ability to help programmers and developers understand how different components work in the hood.
That's why i am requesting for a demo probably a worked out video for many people out there who would love to use this package.

Thanks, glad you like the package! The general interest in it is a bit underwhelming though, which make it tricky for me to justify spending much more time on it. A demo video would be great, but since the repo only gets 1 or 2 page views a day, I frankly don't think it's worth the effort as no one will see it.
I might be able to add a few bits of documentation over time though. Is there something specific you feel that's missing or unclear in the current example? There are a few more small examples in /apps/react-hook-tracer-demo/src/demos/, which are also live on the CodeSandbox playground, but maybe that's not made clear enough on the landing page.

I'll close the issue for now. Feel free to reopen it if you have any specific documentation requests.


Merge branch '5.4.x'

@confluentinc It looks like @rigelbm just signed our Contributor License Agreement. :+1:
Always at your service,
clabot


Indent fallback not working?

Neovim version (nvim -v | head -n1)
NVIM v0.8.0
Operating system/version
macOS 12.5
How to reproduce the issue

Use this ufo setup from the readme

vim.opt.foldenable = true
vim.opt.foldlevel = 99
vim.opt.foldlevelstart = 99

require("ufo").setup({
	provider_selector = function(bufnr, filetype, buftype)
		return { "treesitter", "indent" }
	end,
})

Navigate to an indented section of code that doesn't have a treesitter fold
Try and fold with zc

Expected behavior
Since no treesitter fold is found, ufo should fallback to indent folding and fold the code by indent
Actual behavior
The message E490: No fold found is displayed

Can't reproduce it, check out your shiftwidth, please.

Maybe that should be added to the readme?

I don't think it's necessary for Vimer, without a correct shiftwidth value, insert a newline is always buggy.
Does the correct shiftwidth work for you?

@kevinhwang91 I don't think my shiftwidth value is the issue. I had it set to 2 and it didn't work, I just tried shiftwidth=4 and the bug is present with both values.
Do you have any other ideas what the issue might be?

If it's relevant. I am experiencing this bug in a Scala file

Make sure that UfoInspect output Selected provider: indent.
Trim the issue file and post it here.

@kevinhwang91 oh wait maybe I misunderstood how the fallback works. I am in a file that has some treesitter folds but it isn't able to fold some sections of the file, I thought it is meant to fallback if there is a section of code that treesitter isn't able to fold. Is that not how it works? Does the fallback only work at the file level?

Does the fallback only work at the file level?

All providers work at the file level. If you want them to work together, you must write an external provider for provider_selector, doc/example may help you understand it.

Ah gotcha! Thanks for the help, sorry for my misunderstanding!


refactor: Remove querystring usage

Today we import and use Node.js's querystring module throughout the library.
However, the module is considered 'legacy': https://nodejs.org/api/querystring.html#query-string
Using other solutions should cover us; preferably the standard URL whenever possible.

Looks like it has been un-deprecated as of v18 https://nodejs.org/docs/latest-v18.x/api/querystring.html


Remote Controll between two mobile deivces

I'm working on new project just like you did. thanks for your project , let me have good start.
My project has two basic features(for ios deivce):

DeviceA connect DeviceB, when connect is ok, A will get B screen.

2.also,A can controll B ,such as when draw a circle on A,B will show the circle.
so If I want to achive these feature base on your current project,is it possible?
Can you give me some advice?I really need yours help.thanks a lot.:)

You made it? If i try this project on iPad i hard lag.


PAGE checks do not work when the "external_modules" development version of the module framework is present

@tertek, specifically the PAGE === "ExternalModules/index.php" check does not work as expected.  I'm scheming the best recommendation to fix this...  I wonder if updating the $module->isPage() method as follows is the most intuitive:

isPage([$path]) | 9.7.6 | Prior to framework version 14, $path is required and true is returned if the current page matches a path under the REDCap directory (e.g. Design/online_designer.php).  For framework version 14 and later, $path is optional.  If specified, true is also returned if $path matches the name/path for a module page (e.g. some-module-page).  If $path is not specified, true is returned if any page for the current module is being displayed.

What do you think?  Do have have any alternative ideas?

@mmcev106  I am not sure if  handling this with $module->isPage() is the most intuitive.
In my opinion, a better approach could be the introduction of a new method: isModulePage:

isModulePage([$path]) | From framework version 14 and later , $path is optional. If specified, true is returned if  $path matches the name for a module path/page (e.g. some-module-page). If $pathis not specified, true is returned if any page for the current module is being displayed.

This would also be practical, since currently $module->isPage used APP_PATH_WEBROOT that breaks as well when "external_modules" development version of the module framework is present.
I could not find it, but is there any flag/constant set, in case "external_modules" development version is active/present?
In that way it would be much easier to handle the case.

Some quick write up:
        /**
	 * @return bool
	 */
	function isModulePage($path=""){

        $url_prefix = "?prefix=" . $this->getPrefix();

        $url_extmod_web = APP_PATH_WEBROOT . "ExternalModules/" . $url_prefix;
        $url_extmod_dev = APP_URL_EXTMOD_RELATIVE . $url_prefix;

        //  check if URI contains extmod url and module prefix
        $check_web = strpos($_SERVER['REQUEST_URI'], $url_extmod_web) !== false;
        $check_dev = strpos($_SERVER['REQUEST_URI'], $url_extmod_dev) !== false;

        //  return false if not a module page at all
        if(  !$check_web && !$check_dev ) {
            return false;
        }

        //  return true if no specific page has been defined
        if($path === "") {
            return true;
        }

        //  return true if url parameter for p
        if($_GET["page"] == $path) {
            return true;
        }

        return false;

    }

@tertek, I'm now thinking of adding an isModulePage() method as you suggested, but also effectively renaming isPage() to isREDCapPage() (while continuing to support existing isPage() calls).  I'm also considering adding javascript versions of each method.  Do you have any more thoughts on this before I do a community post about it?

Great news!Will return to you after Tuesday. Thank you for keeping me in the loop. -------- Original Message --------From: Mark McEver @.>Date: Fri, May 26, 2023 6:36 PM +0200To: Research-IT-Swiss-TPH/redcap-pdf-injector @.>CC: Ekin Tertemiz @.>, Mention @.>Subject: Re: [Research-IT-Swiss-TPH/redcap-pdf-injector] PAGE checks do not work when the "external_modules" development version of the module framework is present (Issue #35)
@tertek, I'm now thinking of adding an isModulePage() method as you suggested, but also effectively renaming isPage() to isREDCapPage() (while continuing to support existing isPage() calls).  I'm also considering adding javascript versions of each method.  Do you have any more thoughts on this before I do a community post about it?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.>
[
{
@.": "http://schema.org",
@.": "EmailMessage",
"potentialAction": {
@.": "ViewAction",
"target": "https://github.com/Research-IT-Swiss-TPH/redcap-pdf-injector/issues/35#issuecomment-1564647386",
"url": "https://github.com/Research-IT-Swiss-TPH/redcap-pdf-injector/issues/35#issuecomment-1564647386",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
@.***": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

I think the separation into isREDCapPage() and isModulePage() is very good.
In the future, they might support checks such as $isAuthenticated or $hasRole.
Also the Javascript versions are very neat!

@tertek, I just created a PR adding isREDCapPage() and isModulePage() methods.  I opted to pass on the JS versions, as they would add complication, and it is easy to JSON encode any PHP calls needed for use in JS.  Don't hesitate to speak up if you have any thoughts on this PR, or would like more than a few days to consider it.

@mmcev106 thanks for letting me know.
I am currently in the process of working down my huge backlog from 1 month off.
I may give it a look in beginning of December.

It's only a few lines, and my biggest concern is just the general idea being on point.  I made a reminder to merge it on 12/15 if I haven't heard from you.  Good luck w/ the backlog!


reserve returns 404 although a node is reserved

Method reserve returns 404 as a reaction for HTML output even a node is reserved.
I'm not an expert on Ruby, but I guess, this line of code needs a fix, it should be format.html { render :json => hosts } as in other methods.

Hi,
Unfortunately, this plugin in not maintained anymore.
If you use it, I can give you admin rights to merge any fix you see fit.

It looks like I don't have rights to merge as well, @david-caro who is the original maintainer can do it and also assign you admin rights.
But maybe its best if you fork the project to your own user and continue development there?

I had done that before PR created.

I'll give you rights to the repo (both, @eedri and @pjanouse), though as @eedri says, the repo is unmantqined, so if you want to maintain it, might be nicer to add a note in this one pointing to your fork.

I just gave you access to the repo, let me know if you need anything else 👍


🐛 [BUG] Profilers does not exist

Summary
After installing the library, he tried to remove a table named Profilers from old migrations while migrating the database, but because this table does not exist, I get the error does not exist for Profilers.

Necessary checks were made and it was determined that there were no errors.
If you have used the EasyProfiler library before and then removed it, clean the EasyProfiler records in _EFMigrationHistory in your database for new uses.


Add github support

Only pre-configured packages from dep-repository are available right now. Adding gh support might be nice.
gh:<user>/<repository>[@commit-ish]

Note to self: instead of plain github support, add support for local & remote tarballs.
This will implement github support but also add support for uncommon hosts.


Add Skylake to the Linux hardware list

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Skylake was missing from the Linux hardware list.
Original commit courtesy of @maxim-belkin.

Thanks again @rwhogg, @maxim-belkin!


Deployment fails if the initial response from instagram fails

This causes the app not to be deployed. Better to handle and log the failure, and allow the owner to retry.

Fixed in 0370d0b05856c72bf50a97434a4c35832e29142e


Update README.md

previous PR broke the link above  mine

Fixed in https://github.com/bnb/awesome-developer-streams/pull/46 :)


Query for multiple nodes, ways, or relations at once by ID

It would be nice if I could query for multiple ids at once per osm element and then have the result piped to stdout in a common data format such as CSV or GeoJSON. Something like
osmx query ./database.osmx way 1001 1002 1234
// or
osmx query ./database.osmx way 1001,1002,1234

I'm not that familiar with C++ or these APIs but it looks like if I wanted to do that myself using the osmexpress headers in code I'd have to make one getReader(id) call for each node, way or relation element that I wanted to pull from the database. Is that correct or is there another API call I can use to return a List of results?

@CloudNiner that's correct. calling getReader(id) costs almost nothing.
I think using the programmatic API is better than extending the command line API to handle multiple objects, it would be complicated to specify if you only wanted a subset of tags, or if you had more than several thousand IDs to look up. are you using Python? that may be more convenient to query the DB with, but will be slower than if you compile a C++ program against the headers

👍 that makes sense -- we'll press forward with the programmatic API. Thanks again for the quick response!


UPSTREAM: : Chore: Update OWNERS_ALIASES

/lgtm

/approve

Seems we can't merge to this repo anymore and it will soon be archived anyway.
https://issues.redhat.com/browse/DPP-14039
/close


ACM-7477: Destroy modal incorrect info

https://issues.redhat.com/browse/ACM-7447

/hold

/unhold


Avoid overlapping of tooltips

When changing hover from one button (for example, from "Move up" to "Move down"), one tooltip overlaps the other, because of the default tippy.js duration config.
This PR adds duration=0 to instantly remove the tooltips on mouseout.

Good spot, thanks!


Adding jsoncat, json-sharp and jsonschema (python)

This pull request has the following contributions:

jsonschema Python implementation;
Pretty-print json command line tool (jsoncat);
Json transformer directly in objects (json-sharp);

thanks!


Expose Host level errors via Admin API

Currently we serve function level errors via admin/functions/{name}. We should also allow a way to access the host errors if the host is not running successfully.

Resolved via https://github.com/Azure/azure-webjobs-sdk-script/pull/144


XVim not reading ~/.xvimrc in XCode 6.2

Here is my .xvimrc file. It's name xvimrc.xvimrc and is located in the home directory.
imap jk Esc
(with carrots around the Esc)
It is not read by XVim, though running ':imap jk Esc' works as expected within XCode. This suggests its a different issue than #645

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/10731553-xvim-not-reading-xvimrc-in-xcode-6-2?utm_campaign=plugin&utm_content=tracker%2F252770&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F252770&utm_medium=issues&utm_source=github).

the file should be named just .xvimrc. I have the same command in my xvimrc and it is working for me.

I get this error when trying to rename it to .xvimrc
"You can’t use a name that begins with a dot “.”, because these names are reserved for the system. Please choose another name."

Ah figured it out, you need to do it with terminal. In my case:
cd ~
mv xvimrc.xvimrc .xvimrc
Successfully renamed it and is now working! Thanks Lando! Closing issue.

Great to hear.
On Wed, Apr 8, 2015, 4:36 PM andrewsyntertainment<EMAIL_ADDRESS>wrote:

I get this error when trying to rename it to .xvimrc
"You can’t use a name that begins with a dot “.”, because these names are
reserved for the system. Please choose another name."
—
Reply to this email directly or view it on GitHub
https://github.com/XVimProject/XVim/issues/725#issuecomment-91043200.

Anyone knows why 'set cursorcolumn' not  work in .xvimrc?


The Profiler namespace should be prefixed by :: to avoid name collision

Hello
If the brofiler macros is used inside a namespace that itself has a profiler namespace, there will be a name clash (the internal namespace will be preferred).
So I suggest to change the macros this way :
#define BROFILER_THREAD(FRAME_NAME) ::Profiler::ThreadDescription currentFrameDescription(FRAME_NAME);\
(Profiler is now prefixed with ::)

Hi, thanks for the report.
It's a good idea. I've submitted the suggested fix.
By the way, main namespace has recently changed from "Profiler" to "Brofiler".
"Profiler" namespace was too popular :)
This version hasn't been released yet, but the code is already available.


Replace email functions with apprise notification functions.

Just playing around but through I'd share this minor change.
It should allow most methods of notifications (email,slack,pushbullet,etc).
Need a bit more work still.

PLaying with apprise and this script too !

I've been using my fork for a while, works fine using it with pushbullet but the apprise yml should work with most provides, but I've not tested every one, so can't be 100% sure on the change.


list members' emails in README

so, after #77 has landed, it might make sense to list the commcomm members' emails in the readme, just like the TSC, so that folks that have questions can individually contact us

I'm curious. Has there been an issue with people being unable to individually contact commcomm members? Or, is there a need for this?

@RichardLitt i was thinking it's more for prevention of those issues. like, the other top-level committees do a similar thing, so i'm assuming they've had a similar kind of issue before?

+1 for consistency with the TSC - maintaining this consistency is important from my point of view to keep uniformity and maintain quality across the committees.

+1 for consistency too

Alright! Works for me.

+1

@nodejs/community-committee if everyone is ok with #89 being a thing, does anyone mind if i create a PR for this using the same emails that are used for the email alias?

Go for it @pup, and thank you!

+1

FWIW, these emails can now be found in the aliases file in the emails repo - would be great to get this PR'ed!

I took some time this morning to whip up this PR. I'm missing email addresses from @msmichellegar and @rachelnicole.

I think this can be close since it was PR'd in. Closing.  Please re-open if that was not the right thing to do.


Undocumented error when distributing a package via WSUS Package Publisher.

[Введите отзыв здесь]
Error downloading updates 0x8024b303

Сведения о документе
⚠ Не вносите правки в этот раздел. Это необходимо для связывания страницы сайта docs.microsoft.com с вопросом на GitHub.

ID: f76d96ac-5343-20e2-9d99-9c1cea7ccc2e
Version Independent ID: a95c9b39-4a66-d726-4404-de5af60ac982
Content: Список кодов ошибок Центра обновления Windows с разбивкой по компонентам - Windows Deployment
Content Source: windows/deployment/update/windows-update-error-reference.md
Product: w10
Technology: windows
GitHub Login: @jaimeo
Microsoft Alias: greglin

@dougeby

@dougeby

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Coordinate Location icon should be a configurable symbol

From @MarkTorrey on June 24, 2016 15:28
The icon used to display the current coordinate on the map is hard to small and hard to visualize against different basemaps (i.e. imagery). This needs to be updated to use standard map symbology symbol that can be adjusted by the user in the widget settings. @mcalamito  @topowright
Copied from original issue: Esri/military-tools-webappbuilder#81

@elinz @Dbarnes1
I have updated the config for this widget can somebody verify

@adgiles verified. closing.


fix random behaviour in executeFilters

Issue:
#160
Description of changes:
Sometimes (~3 / (100 * ~600 (instance types))), when all exec executions were finished, and:

some abort <- true was fired
done <- true was fired after wg.Wait()

there was situation, that in select:
select {
	case <-abort: ...
	case <-done: ...

there were both cases to proceed, and then, according to specification:

If one or more of the communications can proceed, a single one that can proceed is chosen via a uniform pseudo-random selection.

I've merged abort and done channels into one verdict channel, and now, if some exec will return false, it will be always ordered within shared channel before always last verdict <- true (after WaitGroup.Wait()).
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Were you able to rerun your test posted in #160 ? If so, did this resolve the anomalous returns?

Were you able to rerun your test posted in #160 ? If so, did this resolve the anomalous returns?

Yes, and so far, anomalies did not reproduced.


Public assets return 500 server error

What version of Hono are you using?
3.5.8
What runtime/platform is your app running on?
Cloudflare worker
What steps can reproduce the bug?
server.ts
const app = new Hono();

app.get("/public/*", async (ctx: Context) => {
  return await ctx.env.ASSETS.fetch(ctx.req.raw);
});

Public directory

Error

What is the expected behavior?
Public assets should work
What do you see instead?
500 error:  message: Can't modify immutable headers.
Additional information
No response

Sorry, fixed by following docs from scratch - https://hono.dev/getting-started/cloudflare-workers#serve-static-files


Error after dev server: Typescript package and config not included with Quick Start

Overview
Just flagging this as it happened today in the current install. Looks like some typescript packages are missing from the install?

I was following the Quick Start guide in the main repo, after running dev server at this point:

the following error is thrown:
Error output
=> Booting Puma
=> Rails 6.0.0 application starting in development
=> Run `rails server --help` for more startup options
Puma starting in single mode...
* Version 3.12.1 (ruby 2.6.3-p62), codename: Llamas in Pajamas
* Min threads: 5, max threads: 5
* Environment: development
* Listening on tcp://<IP_ADDRESS>:34092
Use Ctrl-C to stop
[error] sewing-kit cannot build a typescript project without typescript as a project dependency. Run yarn add typescript in your project directory and try again.
Error: sewing-kit cannot build a typescript project without typescript as a project dependency. Run yarn add typescript in your project directory and try again.
    at /Users/adriancorcoran/Repos/github.com/AdrianCorcoran/test-quilt-adrian/node_modules/@shopify/sewing-kit/dist/src/tools/typescript/verify-typescript-task.js:31:33
    at Generator.next (<anonymous>)
    at fulfilled (/Users/adriancorcoran/Repos/github.com/AdrianCorcoran/test-quilt-adrian/node_modules/@shopify/sewing-kit/dist/src/tools/typescript/verify-typescript-task.js:4:58)

Looks like typescript is not installed? So I then followed the manual commands given underneath in the readme:

After then running dev server I still got errors:
Error output
=> Booting Puma
=> Rails 6.0.0 application starting in development
=> Run `rails server --help` for more startup options
Puma starting in single mode...
* Version 3.12.1 (ruby 2.6.3-p62), codename: Llamas in Pajamas
* Min threads: 5, max threads: 5
* Environment: development
* Listening on tcp://<IP_ADDRESS>:34092
Use Ctrl-C to stop
[error] sewing-kit cannot build a typescript project without typescript as a project dependency. Run yarn add typescript in your project directory and try again.
Error: sewing-kit cannot build a typescript project without typescript as a project dependency. Run yarn add typescript in your project directory and try again.
    at /Users/adriancorcoran/Repos/github.com/AdrianCorcoran/test-quilt-adrian/node_modules/@shopify/sewing-kit/dist/src/tools/typescript/verify-typescript-task.js:31:33
    at Generator.next (<anonymous>)
    at fulfilled (/Users/adriancorcoran/Repos/github.com/AdrianCorcoran/test-quilt-adrian/node_modules/@shopify/sewing-kit/dist/src/tools/typescript/verify-typescript-task.js:4:58)

Same error? I then looked at the commands from the Rails with Node walkthrough and noticed that the following packages were installed there but not in the quilt readme:

typescript
@types/react
@types/react-dom

So I ran, yarn add typescript @types/react @types/react-dom and copied the tsconfig.json file example given in the Rails Walkthrough and everything then worked fine :)
Perhaps the quilt installer needs to be updated? As well as the quilt readme commands in the Manual Install section?

Consuming repo
My own simple test repo: https://github.com/adriancorcoran/test-quilt-adrian
Scope

Is this issue related to a specific package?

I'm not 100% sure on this one :)

Checklist

[x] Please delete the labels section before submitting your issue
[x] I have described this issue in a way that is actionable (if possible)

cc @cartogram - looks like template rework has turned TypeScript into a hard dependency for all projects.  Was this intended?

✅ Just followed the same instructions and ran into the same issue. Attempted to follow the Quick start, but ended up filling in the gaps from the manual guide.

From the audit trail above, looks like this was done but not closed - closing now :)


ACL fixes - idempotentcy and port bug.

Implemented prefetch so that multiple puppet runs will not attempt to recreate the ACLs. Also fixed ACLs not working due to a typo introduced by the addition of the port parameter.
Note: I've only been playing with puppet for a couple of weeks so this isn't necessarily the best implementation. I'm also a bit uncomfortable about the reliance on 'port' and 'acl_api_token' needing to be correct across all tokens or bad things might happen.

@sigerber does this actually make puppet resource consul_acl spit stuff out?
@michaeltchapman or @mdelagrange can you code review this?

Sorry, no. You need to implement 'self.instances' for that - but self.instances doesn't take any parameters, so there isn't a way AFAIK to feed in the port and ACL token. This just fixes the port and id bug and stops puppet trying to create the resource every time there is a puppet run.

seems reasonable to me

@sigerber can you merge master then I'll merge it in?

@solarkennedy Done - and hopefully did it right.

Thanks! I (personally) am not a user of this particular feature of this module, and we don't have a lot of test coverage on these, so we're going to depend on you and the other users on this thread to test this irl.


expose libusb_claim_interface and release_interface

The current libhoth implementation does not support multiple
applications talking to the same device at the same time, this can be
fixed by claiming/releasing the USB device periodically.
Signed-off-by: Robi Buranyi<EMAIL_ADDRESS>

Obsoleted by https://github.com/google/libhoth/pull/36


Could not resolve all dependencies for configuration

Hello everyone!
When I install the knowrob_addons,I meet a difficult problem.
I input the commands in the terminal:
cd ~/catkin_ws
catkin_make
Then I find the errors in the terminal:
.........................................................................................................................................................
FAILURE: Build failed with an exception.
What went wrong:
Could not resolve all dependencies for configuration ':knowrob_meshes:compile'.

Could not find any version that matches org.knowrob.knowrob_cram:knowrob_cram:[0.1,).
Required by:
org.knowrob.knowrob_meshes:knowrob_meshes:1.0.0

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
make[2]: *** [stacks/knowrob_addons/knowrob_meshes/CMakeFiles/gradle-knowrob_meshes] error 1
make[1]: *** [stacks/knowrob_addons/knowrob_meshes/CMakeFiles/gradle-knowrob_meshes.dir/all] error 2
make[1]: ....

FAILURE: Build failed with an exception.
What went wrong:
Could not resolve all dependencies for configuration ':knowrob_memory_graph:compile'.

Could not find any version that matches org.knowrob.knowrob_cram:knowrob_cram:[0.1,).
Required by:
org.knowrob.knowrob_memory_graph:knowrob_memory_graph:0.1.0

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

FAILURE: Build failed with an exception.
What went wrong:
Could not resolve all dependencies for configuration ':knowrob_plan_summary:compile'.

Could not find any version that matches org.knowrob.knowrob_cram:knowrob_cram:[0.1,).
Required by:
org.knowrob.knowrob_plan_summary:knowrob_plan_summary:0.1.0

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
make[2]: *** [stacks/knowrob_addons/knowrob_memory_graph/CMakeFiles/gradle-knowrob_memory_graph] error 1
make[1]: *** [stacks/knowrob_addons/knowrob_memory_graph/CMakeFiles/gradle-knowrob_memory_graph.dir/all] error 2
make[2]: *** [stacks/knowrob_addons/knowrob_plan_summary/CMakeFiles/gradle-knowrob_plan_summary] error 1
make[1]: *** [stacks/knowrob_addons/knowrob_plan_summary/CMakeFiles/gradle-knowrob_plan_summary.dir/all] error 2

FAILURE: Build failed with an exception.
What went wrong:
Could not resolve all dependencies for configuration ':knowrob_chemlab:compile'.

Could not find any version that matches org.knowrob.knowrob_cram:knowrob_cram:[0.1,).
Required by:
org.knowrob.knowrob_chemlab:knowrob_chemlab:1.0.0
Could not find any version that matches org.knowrob.knowrob_sim_games:knowrob_sim_games:[0.1,).
Required by:
org.knowrob.knowrob_chemlab:knowrob_chemlab:1.0.0

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
make[2]: *** [stacks/knowrob_addons/knowrob_chemlab/CMakeFiles/gradle-knowrob_chemlab] error 1
make[1]: *** [stacks/knowrob_addons/knowrob_chemlab/CMakeFiles/gradle-knowrob_chemlab.dir/all] error 2

FAILURE: Build failed with an exception.
What went wrong:
Could not resolve all dependencies for configuration ':knowrob_openease:compile'.

Could not find any version that matches org.knowrob.knowrob_cram:knowrob_cram:[0.1,).
Required by:
org.knowrob.knowrob_openease:knowrob_openease:1.0.0

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
make[2]: *** [stacks/knowrob_addons/knowrob_openease/CMakeFiles/gradle-knowrob_openease] error 1
make[1]: *** [stacks/knowrob_addons/knowrob_openease/CMakeFiles/gradle-knowrob_openease.dir/all] error 2
make: *** [all] error 2
Invoking "make -j8 -l8" failed
....................................................................................................................................................

I find the package knowrob_cram is located in the folder knowrob_addons,and the package knowrob_meshes, knowrob_memory_graph, knowrob_plan_summary, knowrob_chemlab, knowrob_openease are also located in the folder knowrob_addons.In addition,these packages are all depend on the package knowrob_cram.
As a result, when I catkin_make the workspace,the system could not resolve all dependencies for configuration?
Which one can help me solve the problem?
Thanks a lot!

Hi,
Please don't replicate the issues you have already opened.
Regarding to your problem. Which ROS distro do you use? Did you already install rosjava?
Greetings,
Asil

The version of ROS which I use is indigo,and I have already installed rosjava.
I find the information of errors in the terminal:
..................................................................................................................................
FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':runtime'.

Could not find org.ros.rosjava_messages:designator_integration_msgs:0.0.6.
Required by:
org.ros.rosjava_messages:iai_robosherlock_msgs:0.0.6

FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':knowrob_cad_parser:compile'.

Could not find any version that matches org.knowrob.knowrob_common:knowrob_common:[0.1,).
Required by:
org.knowrob.knowrob_cad_parser:knowrob_cad_parser:1.0.0

FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':knowrob_srdl:compile'.

Could not find any version that matches org.knowrob.knowrob_common:knowrob_common:[0.1,).
Required by:
org.knowrob.knowrob_srdl:knowrob_srdl:0.1.0

FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':json_prolog:compile'.

Could not find any version that matches org.knowrob.knowrob_common:knowrob_common:[0.1,).
Required by:
org.knowrob.json_prolog:json_prolog:0.1.0

FAILURE: Build failed with an exception.

What went wrong:
Could not resolve all dependencies for configuration ':semantic_map_to_owl:compile'.

Could not find any version that matches org.knowrob.knowrob_common:knowrob_common:[0.1,).
Required by:
org.knowrob.semantic_map_to_owl:semantic_map_to_owl:1.0.0
.......................................................................................................................................................
Could you help me?Thanks a lot!

@asilx ,Could you help me?Thanks a lot!

again a duplicate: Duplicate of: #137  :/ closing this one


Handle Buycraft being offline better

As of writing, Buycraft is currently suffering some downtime.
During this downtime, restarting the server results in BuycraftX waiting a fairly long time to try to verify the server key, then waits a fairly long time once again to grab server packages after trying to verify the secret key. This for me took around a minute or so for BuycraftX to abort and let the server continue.
Once started, whenever BuycraftX attempts to retrieve information from the Buycraft service, a large stacktrace indicating a timeout is printed to console. This should probably be turned into a "Service unavailable" message and look much friendlier then a stacktrace.
http://pastebin.com/raw/uQRSDxvL

The newest versions already respect a much shorter timeout instead of waiting a minute. We will look at scrubbing exceptions from console output at a later date.
BuycraftX actually handles downtime better than the old plugin, which would choke and frequently require a server restart to make it work. That being said, its logic could be improved and this is something we'll investigate.


Clarify default number of counterparties

The JM usage guide mentions a default of 5-7 counterparties which is what we implemented. Adam mentioned on Telegram that it should be 9.
Tracking this here so we don't forget that. I'd say let's err on the side of caution and go with the higher number here.

The current default is 8 to 10, https://github.com/JoinMarket-Org/joinmarket-clientserver/pull/1199

Thanks! Looks like the docs are outdated there. Will update!


Replace rocket with warp

Closes #313

Replaces the examples' rocket server with a similar warp test server. The test server supports graceful shutdown and a serve_once function that will automatically shut down the server after one request (as is often the case with the redirection functions)
Updates project dependencies for tokio/reqwest/etc to remove some of the duplicates
Also includes some clippy test/example fixes.

So I also made changes for removing rocket, not knowing you were working on that too, because right now the tests/builds are failing due to Rocket now requiring a newer rustc version. I also accidentally referenced your PR in my commit :facepalm: instead of #313 so that is why you see GitHub showing I referenced your PR.
To make this easier, I am going to go ahead an incorporate most of your changes into my branch and go ahead and get it put on master, and then ill take a look at your other two PR's. Let me know whether or not this is ok with you, meaning the commits won't show that you did the work.

Also, you will need to pull in the latest master.

Yeah, that's totally fine with me! I think so long as the result is the same. I'm more interested in just using the library on stable, so updating the examples is just a means to an end there.
If it's cleaner, I can always close this PR if you prefer that!

Yeah, that's totally fine with me! I think so long as the result is the same. I'm more interested in just using the library on stable, so updating the examples is just a means to an end there.
If it's cleaner, I can always close this PR if you prefer that!

I would definitely rather you get credit for your work, and by merging PR's, but thank you for contributing as well. On this one occasion, it was just easier that I add your changes to my branch. But in the future, im gonna do normal merges.


openbsd: ignore ci error about unexistent KERN_NSELCOLL constant

This sysctl(2) constant has been removed.
https://github.com/openbsd/src/commit/9bf42e341afcf2dd868da323254a70b8eb9463cf

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

it unbreaks the CI on OpenBSD when using a recent -current snapshot (upcoming 7.2 version)

Thanks! @bors r+

:pushpin: Commit 5872135f9d3d237dec44c7b5758895ca58fa1c2d has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 5872135f9d3d237dec44c7b5758895ca58fa1c2d with merge 5b0cfe5d7dd9502df5a768c5f5176aa36c33f378...

:broken_heart: Test failed - checks-actions

@bors retry

:hourglass: Testing commit 5872135f9d3d237dec44c7b5758895ca58fa1c2d with merge 2c7f384d915071692f9353825cb896640b3f30ef...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 2c7f384d915071692f9353825cb896640b3f30ef to master...


Build: Don't run CI on unrelated changes

@manuzhang looks like theres some linter issues, can you run make lint?

@kevinjqliu I think it would be better to squash commits and merge for such PR next time.


Data Sharing and Privacy

Summary
@friedger made an excellent argument regarding Issue #113: We should consider restructuring the backend model to figure out how data will be shared so that it does not interfere with privacy laws.
Example:
Sharing private data such as location should be done in accordance with privacy laws.
Motivation
We need to be aware of the important legal and regulatory requirements of healthcare clients so we can avoid being liable in the future. Some standards to consider are HIPAA (US/CAN), GDPR (EU), etc..
Describe alternatives you've considered
For anonymity, we could follow some de-identification steps found [here] (https://www.hhs.gov/hipaa/for-professionals/privacy/special-topics/de-identification/index.html#standard).

I agree. Here's some points I made with @SomeMoosery:

Ownership: We can have permissions for everything (every data field essentially). At first, all input will be on the client side. After a set daily time, all data with the granted permissions e.g. an indicator function will be allowed to transfer when pulled from the client to our backend. This way the user owns the data they want to share.

Privacy: We are not a hipaa compliant entity: "HIPAA applies to healthcare providers, health plans, and healthcare clearinghouses if those organizations transmit health data electronically in connection with transactions for which the Department of Health and Human Services has adopted standards." source: https://www.hipaajournal.com/who-does-hipaa-apply-to/. If we work with an entity that is a hupaa compliant entity, then we have to be careful. Howeever, we're not financially liable: "Since the Enforcement Final Rule of 2006, OCR has had the power to issue financial penalties (and/or corrective action plans) to covered entities that fail to comply with HIPAA Rules." source: https://www.hipaajournal.com/what-are-the-penalties-for-hipaa-violations-7096/.

For v1: we can have anything we want set in the code because no one is using this yet.
Prior to release, we can always take things out. As well as if we partner with a hipaa compliant entity we can not share some data with them based on field names. We can also add an attribute in the data model e.g. child to flag potential hipaa conflicting information.
I'm happy to hear GDPR regulations.

Different privacy laws have different stipulations. Software like the ones developed by OneTrust exist to assist with compliance. Also, there are many privacy regulations that are on the verge of being passed. However, in general, privacy compliance ensures that users know what data is being collected from them, how it is being used, and have assurances regarding the risk of data collection. My suggestions are:

Have a clear privacy policy that outlines the major information and data flows
Prepare an open source and accessible privacy impact assessment (PIA)
Follow the seven principles of privacy by design. Implement privacy-preserving techniques from the onset. Instead of relying on just anonymization, it is likely worth investing in differential privacy. If machine learning is implemented, consider federated learning and secure multi-party computation as well
Always ask for permission as @ngiangre stated

We are and will further consider all of these points!

Closing this issue as we've taken note of these security measures and will look to implement them ASAP


System.js or minified library

Trying to integrate with angular2 using system.js. Installed with npm and then pointed system.js to ng2-smart-table.js in /build. However, it has a problem with
__export(require('./src/ng2-smart-table/lib'));
in ng2-smart-table.js as it tries to find lib.js and fails.
Could you provide steps to get system.js working or provide a minified library thats easier to include?

@dbogdantech101 I solved the problem for lib.js by adding a new entry for it in systemjs.config.js, but I still have 404 on the .html.js and .scss.js files (wrongly suffixed with .js extension).
(function (global) {
  System.config({
    paths: {
      // paths serve as alias
      'npm:': 'node_modules/'
    },
    // map tells the System loader where to look for things
    map: {
      // ...
      'ng2-smart-table': 'npm:ng2-smart-table',
      'ng2-smart-table/lib': 'npm:ng2-smart-table/build/src/ng2-smart-table/lib'
    },
    packages: {
      'smart-table': {
        main: 'build/ng2-smart-table.js'
      },
      'ng2-smart-table/lib': {
        main: 'index.js'
      }
    }
  });
})(this);

Actually, it could be working like this :
(function (global) {
  System.config({
    paths: {
      // paths serve as alias
      'npm:': 'node_modules/'
    },
    // map tells the System loader where to look for things
    map: {
      // ...
      'ng2-smart-table': 'npm:ng2-smart-table',
      'ng2-smart-table/lib': 'npm:ng2-smart-table/build/src/ng2-smart-table/lib',
    },
    // packages tells the System loader how to load when no filename and/or no extension
    packages: {
      // ...
      'ng2-smart-table': {
        main: 'build/ng2-smart-table.js',
        meta: {
          '*.html': {},
          '*.scss': {}
        }
      },
      'ng2-smart-table/lib': {
        main: 'index.js'
      }
    }
  });
})(this);

doesn't work for me


[Feature]: Make Decimal Rounding Configurable

What feature would you like?
Currently, the number of decimal places in checkAndEvaluateMath.ts is fixed to 3. We want to make this number configurable by introducing a platform option called mathRoundDecimals (or a similar name). This will allow users to define the number of decimal places according to their requirements.
Acceptance Criteria:

Introduce a new platform option called mathRoundDecimals (or similar).
Use this option to determine the number of decimal places.
Ensure a default value is used if mathRoundDecimals is not provided.

Here is the relevant code snippet:
// the outer Number() gets rid of insignificant trailing zeros of decimal numbers
const reducedTo3Fixed = Number(Number.parseFloat(`${result}`).toFixed(3));
result = resultUnit ? `${reducedTo3Fixed}${resultUnit}` : reducedTo3Fixed;
return result;

Related TS plugin ticket: https://github.com/tokens-studio/figma-plugin/issues/2874
@jorenbroekema
Would you be available to contribute this feature?

[ ] Yes, I would like to contribute this

Suggested API seems good to me, goal is to configure like so:
{
  platforms: {
    css: {
      mathRoundDecimals: 4,
      // rest of the options
    }
  }
}

and the check evaluate math transform can pick up on that platform option, I'm also okay additionally to set the default value to 4 instead of 3 because em/rem and the rounding to 4 could be significant for pixels, where 5th usually isn't.


Implementation in pytorch

Is there a Torch version code accessible?

Sorry, but there isn't. But you can substitute model part in the code base with the pytorch.


Upgrade yargs-parser

User Story
https://github.com/chef/automate/security/dependabot/components/chef-ui-library/package-lock.json/yargs-parser/closed
Definition of Done
Demo Script / Repro Steps

Issue is no more found in dependabot


Editing a blog post in flasky/app/main/views.py Tag 11h

Am wondering Miguel at line 95 - 96 is it supposed to if current_user != post.author or not current_user.can(Permission.ADMINISTER): instead of if current_user != post.author and  not current_user.can(Permission.ADMINISTER):

This is actually correct in the code. The conditional ensures that the user is either the author or an adminsitrator, but the if statements tests the reverse case. So "author or admin" becomes "not author and not admin".

I finally figured that it was correct, I should have closed this issue. Thanks for getting back to me though and keep up the good work Miguel.


Trouble installing posh-docker

File: docker-for-windows/index.md
The commands for installing post-docker do not work. You need to run import-module posh-docker after the install-module posh-docker command.note
The command in $PROFILE should be corrected to  "Import-Module posh-docker"

I'll take this one


bottom pixel overflow issue resolved

Description
I added a SingleChildScrollView propery over the Scaffold which enabled me to solve the bottom pixel overflow error
Fixes #11
Mention issue number in the above issue bracket.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Screenshots (if appropriate):
Before

After

Additional Notes:
Include any additional information that you think is important for the reviewer to know.

@himanshiiiiii  whenever creating a new issue, please include a description of the issue. If it's related to the UI, also attach screenshots. Additionally, your home_screen.dart file conflicts with the main. Please resolve these conflicts first. If you haven't pulled the latest commit from this repo, do so first

@himanshiiiiii  whenever creating a new issue, please include a description of the issue. If it's related to the UI, also attach screenshots. Additionally, your home_screen.dart file conflicts with the main. Please resolve these conflicts first. If you haven't pulled the latest commit from this repo, do so first

Ok I'll do it

Good work @himanshiiiiii , please do similar changes in menu_screen also, it also has some pixel overflow issue, then I will merge this PR in medium section

Ok I'll do it

Good work @himanshiiiiii , please do similar changes in menu_screen also, it also has some pixel overflow issue, then I will merge this PR in medium section

I've done it plzz check it once


Automatically generate missing drag and drop functions

Should be able to be automatically generated but I don't know how to do it since it's based around gdk::DragContext. I pretty much just need gtk_drag_finish() to be implemented (which is necessary for drag and drop functionality to work). I think all 7 functions listed here where their first argument is a gdk::DragContext should be done.

I think this was resolved by #472.


error / any user forum?

Hi
I installed pydcop successfully on windows 10, but when I run  dcopy_cli.py --version or the following command from tutorial . I get error
> dcop_cli.py solve --algo dpop graph_coloring.yaml
  File "C:\Users\908682\pydd\pyDcop\pydcop\dcop_cli.py", line 164
    def _configure_logs(level: int, log_conf: str):
                             ^
SyntaxError: invalid syntax

How may I resolve this? Do you have any user forum to ask questions, may be this is not the right place?

Hi,
This is the right place for such question, I prefer to have all questions as issues on github, so that other people can find them.
I suspect you're using an old version of python (probably 2.x)  : the syntax error points to the annotation used for optional typing since python 3.5.
To use pydcop, you need at least python 3.6, and I would recommend using the last version available (3.8 currently).

Thank you. I installed 3.8 and it works. Should it not work with the pydcop shortcut also? It works only on the python call.
(pydcop_env) C:\Users\908682\Documents\Projects\pydd\pyDcop\pydcop>python dcop_cli.py --version
pydcop 0.1.2a1

(pydcop_env) C:\Users\908682\Documents\Projects\pydd\pyDcop\pydcop>pydcop --version
'pydcop' is not recognized as an internal or external command,
operable program or batch file.

I asked about a user forum because I am not in AI nor into computer science. I just want to use dcop as an application. I would like to develop my constraint graph and give my constraints and variables. So my questions could be very basic :( .  Could you advice some easy introduction to understand not too much math.  I understand a little about defining variables and agents, but I am not able to understand how to define or assess the global objective.  Thank you.

The pydcop shortcut is actually a shell script, it works on unix-like platform (e.g. linux or mac) or on windows when using tools like cygwin or WLS (Windows Subsystem for Linux).
Out of curiosity, what are you trying to do with pydcop ? if you don't care about distribution and multi-agents, it might not be the best tool for you, If you simply want a (centralized) optimization tool it would probably be simpler to a solver like glpk or cplex.

Thanks for your reply. I am indeed looking for decentralized optimization in a agent based model. I have a network of immobile agents who are in a particular state (one of 5 possible high to low states).  I want to improve their state to high (if not high already), an agent should try to be in a similar state as its few neighboring agents. To improve their state they can use one of the available 1-5 possible improvements. These improvements depend on their current state, so not always 5 improvements to every agent. Each improvement has a cost. And there is a global budget. So the agent talks to a limited set its neighbors and select improvements that do not go beyond a global budget. I am intentionally opting to do a decentralized optimization to differ from current work and its suits my agent model.

Hi, sorry for the delay.
If you're looking for decentralization and agent model, then pyDcop may indeed be what you're looking for :)
In your scenario, the global budget might be an issue : it means there is a global constraints that involves all variables : in your constraint graph every variable would be connected to every other variable though this constraint.
Except this point, it would be quite easy to model your system as a DCOP, if you elaborate a bit I might be able to help you.

Thank you for your reply. May I have your email? It could be long to write here.  if you do not wish to make your email public, Please drop a mail to me here, just to get your email : b dot srirama at gmail dot com
I will reply to that email. If you do not agree to that, I will write problem description here.


[Backport] [1.0] [FLINK-30998] Apply adding failureHandler on top of current apache:main branch

Backporing FLINK-30998 to 1.0 branch

@MartijnVisser I think we could backport to 1.0, low risk change, thank you

@MartijnVisser I think we could backport to 1.0, low risk change, thank you

Fine with me, will you fix the CI?

Yes!


Release 2.1.1

Added revocations for remaining Golden League challenge charts we expect to be released

Smoke testing LGTM


Avoid reassignments to Frame fields after initializing with default

Use chunks_exact in Frame::from_[rgb|rgba]_speed
Use [copy|extend]_from_slice instead of manual operations
Correct overlapping range patterns in match statement
Remove unnecessary semicolon in benchmark
Remove unnecessary & in if let statement
Remove unnecessary statement in reader::fill_reader
This seemed to be a 2-3% performance improvement on some of the benchmarks.

Thanks!


Please create a tag 'cdb' for this extension

release 0.1 is not compatible with Postgres 9.4 and it is the only released tag for this project..

I don't think this extension was created to be compatible with postgres 9.4, in any case, what's the problem ?

It won't compile with 9.4.4 headers. /cc


Update source versions

The script is not using the latest versions of all sources, this should addressed in the next update of the script.

Resolved, see: https://github.com/nsuchy/nginx-naxsi-installer/commit/a4f3ac3b667bb9415ab6f3eda12ec20eec215038


Add new header pattern to loading of Neuralynx files

Hello all,
I am using spikeinterface and Neo 0.12.0. I get an error concerning the time informations in the header due to, I guess, new patterns not supported yet by Neo. One is from new recordings and one from very old recording.

First case:
I get an error on the following line :

 info = NlxHeader(filename)

Error trace:
NlxHeader.__init__(self, filename)
    246 else:
    247     dt1 = sr.groupdict()
--> 248     self['recording_opened'] = datetime.datetime.strptime(
    249         dt1['date'] + ' ' + dt1['time'], hpd['datetimeformat'])
    251 # close time, if available
    252 if 'datetime2_regex' in hpd:

File c:\Users\agnes.kandracs\AppData\Local\anaconda3\envs\si_env\lib\_strptime.py:568, in _strptime_datetime(cls, data_string, format)
    565 def _strptime_datetime(cls, data_string, format="%a %b %d %H:%M:%S %Y"):
    566     """Return a class cls instance based on the input string and the
    567     format string."""
--> 568     tt, fraction, gmtoff_fraction = _strptime(data_string, format)
    569     tzname, gmtoff = tt[-2:]
    570     args = tt[:6] + (fraction,)

File c:\Users\agnes.kandracs\AppData\Local\anaconda3\envs\si_env\lib\_strptime.py:352, in _strptime(data_string, format)
    349     raise ValueError("time data %r does not match format %r" %
    350                      (data_string, format))
    351 if len(data_string) != found.end():
--> 352     raise ValueError("unconverted data remains: %s" %
    353                       data_string[found.end():])
    355 iso_year = year = None
    356 month = day = 1
 ValueError: unconverted data remains: .000

And here is the associated header:
######## Neuralynx Data File Header
-FileType NCS
-FileVersion 3.4
-FileUUID 189b12c7-ac30-4a8e-9e03-ecd6bfead8b7
-SessionUUID 2c4921e7-e248-409b-ae37-b98584ea243b
-ProbeName
-OriginalFileName "/network/lustre/iss02/epimicro/patients/raw/pat_03249_1711/eeg/03249_2023-01-04_13-50/03249_2023-01-04_13-50_AmT2_1.ncs"
-TimeCreated 2023/01/04 13:50:16.000
-TimeClosed 2023/01/04 15:50:15.999

-RecordSize 1044
-ApplicationName Pegasus "2.2.2 "
-AcquisitionSystem Micromed ATLAS
-ReferenceChannel "AD Channel 0110"
-SamplingFrequency 4096
-ADMaxValue 32767
-ADBitVolts 0.000000122070312500000003
-AcqEntName AmT2_1
-NumADChannels 1
-ADChannel 16
-InputRange 4000
-InputInverted True

-DSPLowCutFilterEnabled True
-DspLowCutFrequency 0.1
-DspLowCutNumTaps 0
-DspLowCutFilterType DCO
-DSPHighCutFilterEnabled True
-DspHighCutFrequency 1000
-DspHighCutNumTaps 256
-DspHighCutFilterType FIR
-DspDelayCompensation Enabled
-DspFilterDelay_µs 3891

## Original info
-OriginalFileName "D:\PegasusData\pat_03249_1711\2023-01-04_13-12-05\AmT2_1.ncs"
-TimeCreated 2023/01/04 13:12:30
-TimeClosed 2023/01/04 15:52:25

Second case:

This is coming from very old Neuralynx recordings:
 ---> 64 neo_reader.parse_header()

Full error trace:
 File c:\Users\agnes.kandracs\AppData\Local\anaconda3\envs\si_env\lib\site-packages\neo\rawio\neuralynxrawio\neuralynxrawio.py:189, in NeuralynxRawIO._parse_header(self)
    186     continue
    188 # All file have more or less the same header structure
--> 189 info = NlxHeader(filename)
    190 self.file_headers[filename] = info
    191 chan_names = info['channel_names']

File c:\Users\agnes.kandracs\AppData\Local\anaconda3\envs\si_env\lib\site-packages\neo\rawio\neuralynxrawio\nlxheader.py:244, in NlxHeader.__init__(self, filename)
    242 sr = re.search(hpd['datetime1_regex'], txt_header)
    243 if not sr:
--> 244     raise IOError("No matching header open date/time for application {} " +
    245                   "version {}. Please contact developers.".format(an, av))
    246 else:
    247     dt1 = sr.groupdict()

OSError: No matching header open date/time for application {} version Cheetah. Please contact developers.

And here is the associated header:
######## Neuralynx Data File Header
## File Name D:\pat_01866_0276\eeg\01866_2011-11-16_17-57\01866_2011-11-16_17-57_CSC2.ncs
## Time Opened (m/d/y): 11/16/2011  (h:m:s.ms) 17:57:38.925
## Time Closed (m/d/y): 11/16/2011  (h:m:s.ms) 18:34:11.088
-CheetahRev 5.4.0

-AcqEntName CSC2
-FileType: CSC
-RecordSize 1044

-HardwareSubSystemName AcqSystem1
-HardwareSubSystemType DigitalLynx
-SamplingFrequency 32556.000000
-ADMaxValue 32767
-ADBitVolts 0.0000000153

-NumADChannels 1
-ADChannel 1
-InputRange 500
-InputInverted False
-DspLowCutFrequency 0.100000
-DspLowCutNumTaps 0
-DspLowCutFilterType DCO

-DspHighCutFrequency 8000.000000
-DspHighCutNumTaps 256
-DspHighCutFilterType FIR

## File Name C:\CheetahData\2011-11-16_17-21-06\CSC2.ncs
## Time Opened (m/d/y): 11/16/2011  At Time: 17:21:14.359
## Time Closed (m/d/y): 11/16/2011  At Time: 18:34:48.593

If you can potentially add those pattern to neo it would be really appreciated :)
Thank you,
Anthony

Hello all,
I encountered the same issue when working with version Cheetah "6.3.1". I think it's just another pattern for datetime... Can somebody help with that ? @samuelgarcia maybe ?
Thanks in advance,
Aymard

Maybe this can be a possible fix when the end of the time pattern is .000 :
In _strptime.py file on line 351:
if len(data_string) != found.end():
        if data_string[found.end():] == '.000':
            pass
        else:
            raise ValueError("unconverted data remains: %s" %
                            data_string[found.end():])

I am not sure this will not lead to other possible error but it fixes the issue on my side.
Cheers,

Hi,
I think for Pegasus moving from this pattern :
datetimeformat='%Y/%m/%d %H:%M:%S'

To this:
%Y/%m/%d %H:%M:%S.%f

for datetimeformat should help. I don't know from which version of Pegasus this is happening but I cannot open any of my newest data because of that unfortunately...
Cheers,
Anthony

@Antho2422, Hey Anthony,
It seems like the team has been pretty busy. If you feel pretty confident about your change you should definitely contribute to the project. Open a PR with your fix. Only thing we would need is a small test file as well to make sure the fix worked and didn't break any previous behavior. Would you want to give it a try? Maybe @samuelgarcia knows who else might be willing to help. Once the fix is propagated here then it would flow to SpikeInterface.

I would be happy to give a try. I will try to work on it tomorrow. This will be my first PR so I will definitely need some help to make sure everything is fine.
Thank you

No worries at. Happy to help you work on this. Just post any questions or bumps you run into here. Once you open the PR we can test to make sure your changes do not break existing functionality and then we would need a new test file to add to the testing repo to make sure your changes actually work for the new format. So super low stress. The slowest step is actually adding the newest test file ( I haven't set up an account for the test repo yet so I can help try to rope someone else in to help with that later).

@zm711 @samuelgarcia,
I've just created a pull request :  https://github.com/NeuralEnsemble/python-neo/pull/1370#issue-2088301623
Unfortunately I am not able to give you access to any test data at the moment but this can be reproduced with any Pegasus data in version 2.2.2.
Hope everything is okay and will be included soon !
Cheers,
Anthony

#1370


Final word in help text is not wrapped

Affected Version of clap
Tested with latest master, d29d7cc.
Expected Behavior Summary
The final word in the help text should be wrapped.
Actual Behavior Summary
The final word remain with the next to final word, that is, the last space is ignored.
Steps to Reproduce the issue
#[test]
fn final_word_wrapping() {
    let app = App::new("ctest").version("0.1").set_term_width(24);
    assert!(test::compare_output(app, "ctest --help", FINAL_WORD_WRAPPING, false));
}

static FINAL_WORD_WRAPPING: &'static str = "ctest 0.1

USAGE:
    ctest

FLAGS:
    -h, --help
            Prints help
            information
    -V, --version
            Prints
            version information
";

The "version information" line is longer than 12 characters, despite information being only 11 characters wide.
I have a fix for this that I'll post next.

Interesting, I wonder if it's counting the space? Could you post a gist of the output when you compile clap with features = ["debug"] and run the test program?

I've put up a PR for fixing this and a small related bug. As explained, I hope to do a drop-in replacement later with my textwrap crate, but wanted to make sure the output didn't change first. So I compared the existing code with the wrapping done by that crate and found small problems.


fix: dfx new --no-frontend should not create an assets container

Description
recreates PR #2020 from an internal branch
Checklist:

[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[x] I have made corresponding changes to the documentation.

We're going to need to install node for some of these because the frontend canister is no longer getting built.

Adam is reworking dfx new, no need for this PR anymore


Major api update

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Can't turn off CredentialHelper

I'm trying to set credentialHelper to NONE in my uiConfig variable:
credentialHelper:  firebaseui.auth.CredentialHelper.NONE
But the component won't mount.  The error is:
'firebaseui' is not defined  no-undef
It appears that I don't have access to the 'firebaseui' object, so it can't evaluate the value of firebaseui.auth.CredentialHelper.NONE.

You'll need a reference to the firebaseui module. You can get that by doing:
import firebaseui from 'firebaseui';

or
const firebaseui = require('firebaseui');

and you also probably need to install the library: npm install --save firebaseui


Fix typo in thrown error

Fixes a typo in the error message when declaring both integration: true and needs in the same module

Nope, not a bug.  We are just ahead of the rest of the english language speakers.

:smile:

Build is failing on ember-canary but I don't think it's at all related to this change

Confirm, not related. I did rerun just to see if it is a transient failure.  Would you mind opening an issue so we can track it down after merging this?


Add deprecation notice to package description

This builds on #1070 and changes the package description to make it clear that vim-mode-plus should be installed instead.  After this is merged a new patch should be released.
We should also consider moving this repository to the atom-archive organization.
/cc @maxbrunsfeld @t9md

Thanks @50Wliu!


Display raw platform name instead of NULL for unknown devices

Fixes issue with NULL for unknown devices

@numen31337 Thank you!


Investigate restrict-size performance issue

Describe the bug

restrict-size is used in libp2p-mplex, it's pretty simple but takes 4.6% of cpu time

0318_contabo_20.cpuprofile.zip
Expected behavior
Should be <1%

reattaching https://github.com/ChainSafe/lodestar/issues/4603#issuecomment-1263095995 (after libp2p is updated to 0.39.2), this time it shows restrictSize() takes 4%

Detail

I disabled async stack trace so there is no async hook there @wemeetagain

0930_multilined_fastMsgId_lg1k_1.2.0.cpuprofile.zip

tracked in https://github.com/libp2p/js-libp2p-mplex/issues/205

Upstream issue fixed https://github.com/libp2p/js-libp2p-mplex/issues/205


tetragon: docs, trnasform events into single page

I don't like these multi page click icons lets just build a single page for events.

thanks Mahe I think I resolved your comments.


[Event Request] Codeunit 5720 "Item Reference Management" - OnValidateSalesReferenceNoOnBeforeAssignNo (BaseApp_22.0)

Please add VAR to ReturnedItemReference in procedure OnValidateSalesReferenceNoOnBeforeAssignNo in Codeunit 5720 "Item Reference Management"
Change:
    [IntegrationEvent(false, false)]
-   local procedure OnValidateSalesReferenceNoOnBeforeAssignNo(var SalesLine: Record "Sales Line"; ReturnedItemReference: Record "Item Reference");
+   local procedure OnValidateSalesReferenceNoOnBeforeAssignNo(var SalesLine: Record "Sales Line"; var ReturnedItemReference: Record "Item Reference");
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 11238.


use local cached file and do etag checks to prevent unnecessary downloads

As discussed in Slack channel - this PR causes the EAs to cache the file locally along with an eTag file, which is used to compare local to online, so the full file is only downloaded when a change is made.
I chose /private/tmp/sofa as the cache folder, but this could be altered to something more permanent.

Thanks!


Adopt row major conv output in unet

This has been done on perf branch nsmith/unet_test_perf2_groups, the whole branch will be merged into main as part of #6670.  Tracking for this issue to move there.


[doc] add sphinx doc CI workflow

@jschenxiaoyu @robin1001

@jschenxiaoyu
When this is merged, please set gh-pages branch as the default GitHub Pages branch at:
https://github.com/mobvoi/wenet/settings
Then the document will be hosted at https://mobvoi.github.io/wenet


Convert input text to Memrise output

According to http://feedback.memrise.com/knowledgebase/articles/525095-add-words-to-my-course-or-upload-words-from-a-spre ,

there is a limit of 1000 words.
tab/comma delimiting can be used for words
new line/semicolon delimiting can be used for lines

I think the only difference between outputting for Anki and Memrise is in the DeckFactory.java deckGenerator code: "getPinyinWithHTML" should be just "getPinyin".
In terms of the UI, having a radio button to choose between Anki and Memrise would do the trick in a clean manner. If we have more outputs later on, we could use a drop-down menu.

I think another difference will be that the only columns (At least for now) that we want will be chinese name and definition (maybe pinyin too, though I guess we'd want to merge this with definition?). I am unsure whether memrise lets you define your own columns.

I think you're right, Daniel. Perhaps we can use a new line to separate definition&pinyin, whilst keeping them in the same field (may need semicolon delimiting then). We'll just have to experiment and see what works!

Haven't used Memrise for years; can look back into this if it comes up organically


Add unit test for GMAO

Need to add unti test for GMAO historic and current day small region

This would have caught issue of our last commit for both scenarios to introduce a bug.

opendap does not have ds.close() capability.
the clearing of the variable is not indent correctly for the historic data

closing this as has been solved by PR #285.