An idea to combat model pollution

Problem
has_paper_trail adds too many methods to the ActiveRecord model, as described in https://github.com/airblade/paper_trail/issues/703, and others.

If I'm counting correctly, installing the paper_trail gem adds 36 instance
methods and 10 class methods to all of your active record models. Of those
46, 13 have a prefix, either "pt_" or "paper_trail_". I don't know what the
best way to deal with this is. Ideally, we'd add far fewer methods to
people's models. If we were able to add fewer methods to models, then I
wouldn't mind prefixing all of them.
https://github.com/airblade/paper_trail/issues/703

Solution
Add only two methods to the AR model.

An instance method #paper_trail
A class method .paper_trail

The instance method would return a RecordTrail and the class method would
return a ClassTrail. Those names are totally up for debate.
Advantages

Plain ruby, easy to understand
Adding new methods to e.g. the RecordTrail does not add any methods to
the AR model.
Method privacy is more strongly enforced.
Enables isolated testing of e.g. RecordTrail; it can be constructed with a
mock AR instance.

Disadvantages

Two new classes, though they are simple.

I think I'd prefer version over paper_trail, as it's easier to type and mirrors the versions association. So you could do:
record.version.live?
record.version.originator
record.version.source
record.version.switched_on?
record.version.save?

I think I'd prefer version over paper_trail, as it's easier to type and mirrors the versions association.

Cool, I'm glad you like the general idea. I'm not opposed to the word version, but I am concerned it's bit of a common word. Any other ideas? "trail" is uncommon and easy to type.

You can get the best of both worlds by making it configurable.

You can get the best of both worlds by making it configurable.

That's true, we could pass the name(s) to has_paper_trail. However, configuration means a more complicated implementation, so if we can choose a name that is likely to satisfy 99% of people, or at least not cause a name conflict for 99% of people, then that would be my first choice.

@jaredbeck, apologies for the delay in reviewing this. This seems like a sensible refactor to me, since requests tend to come in pretty consistently to rename method names provided to models that are causing namespace collision in ActiveRecord. The questions is does it make sense for all instance methods (and possibly even class methods) provided by PaperTrail to be 'hidden' / 'extracted' behind a barrier such as this.
I'm not sure what the answer is, but it might be sensible to hold off on this until PaperTrail 5 if the answer is to indeed extract all of them, since it will cause breakage. Or maybe we do the transition in PaperTrail 4 but have the old methods show a deprecation warning, and then remove the old methods once PaperTrail 5 rolls out.

This seems like a sensible refactor to me, since requests tend to come in pretty consistently to rename method names provided to models that are causing namespace collision in ActiveRecord. The questions is does it make sense for all instance methods (and possibly even class methods) provided by PaperTrail to be 'hidden' / 'extracted' behind a barrier such as this.

That's a good question. PT adds at least 36 instance methods by my count. This PR only extracts one of them. Maybe the next step is to try extracting a dozen of them and see if we're still happy with this idea. I'm happy to move slowly on this, as it would be a big change to the public API and I want to get it right.

.. it might be sensible to hold off on this until PaperTrail 5 if the answer is to indeed extract all of them, since it will cause breakage. Or maybe we do the transition in PaperTrail 4 but have the old methods show a deprecation warning, and then remove the old methods once PaperTrail 5 rolls out.

This is a big change, and I don't want it to hold up PT 5.  I'm happy if this doesn't happen until PT 6. We could do a 5.99 release that deprecates the old polluting API.

OK, I've taken this from an idea to a proof of concept. I'd like your feedback again, please, to know if I should continue.
Things I like so far:

No change to our public API.
Private methods! We'll be able to break up big methods like setup_model_for_paper_trail (currently the largest method in the library) without adding any methods to our public API. :scissors:
The Code Climate GPA has increased by 0.24 so far, partly due to the new private methods. :mortar_board:

Things I don't like:

The two lambdas defined in ModelConfig are harder to understand. In addition to having the same "execution scope" as before (the model class) they now also have the lexical scope of ModelConfig.
I had to use send in a few places to call private methods on the model from ModelConfig. For example, attr_accessor.

Next Steps
Sean and Ben, please take another look and let me know your concerns. If we do decide to continue with this, we could then discuss the name of the two important new methods currently named "paper_trail". Thanks for your help!

Status: class methods 63% done, instance methods 28% done. Deprecation scheduled for 5.2, removal in 6.0.

Status: class methods 100% done, instance methods 60% done.

Whew. This is complete, and ready for final review. 7 class methods and 30 instance methods are deprecated, and a handful of private methods are moved without deprecation.
Here are the things I'd like feedback on:

Are #paper_trail and .paper_trail the right names for the two most important methods? As Sean pointed out, it's a lot to type. I'm tempted to suggest the more concise #trail, which I think we could get away with. However, #paper_trail is easier to remember.
Is ::PaperTrail::RecordTrail a good name? Again, I'm tempted to suggest the simpler ::PaperTrail::Trail.
Is ::PaperTrail::ModelConfig a good name? ModelSetup might be better?
Is everyone OK with deprecating in 5.2 and dropping in 6.0?

Thanks for the review!

@batter @seanlinsley any feedback? objections?

@jaredbeck sorry, I'll try responding in the next few days.

Sean and Ben, I'll be releasing this, as 5.2.0, this weekend, unless I hear any objections. If you want another week to review it before it's released, please let me know, thanks.


Unexpected output path when generating docs for scheme "ParseSwift (iOS)"

When generating docs in the Swift Package Index for an iOS package, we're running the following set of commands:
git clone https://github.com/parse-community/Parse-Swift.git
cd Parse-Swift

env DEVELOPER_DIR="/Applications/Xcode-13.3.0.app" xcrun xcodebuild -derivedDataPath "$PWD/.derivedData" docbuild -scheme "ParseSwift (iOS)" -destination 'generic/platform=ios'

mkdir -p .docs/parse-community/Parse-Swift

env DEVELOPER_DIR="/Applications/Xcode-13.3.0.app" xcrun docc process-archive transform-for-static-hosting $(find "$PWD/.derivedData" -type d -name "*.doccarchive") --output-path ".docs/parse-community/Parse-Swift/main" --hosting-base-path "parse-community/Parse-Swift/main"

The output path we end up with here is:
❯ ls .docs/parse-community/Parse-Swift/main/documentation/
parseswift/

The problem is that we're trying to derive the hosting URL from the scheme the users have set ("ParseSwift (iOS)"), which we thought was just going to be scheme.lowercased(), i.e. parseswift (ios). It appears that it's also stripping the  (iOS) part.
The issue – if it is actually an issue – seems to the the output of the docbuild command. We are discovering it via the find "$PWD/.derivedData" -type d -name "*.doccarchive" and if I run this it results in:
❯ find ".derivedData" -type d -name "*.doccarchive"
.derivedData/Build/Products/Debug-iphoneos/ParseSwift.doccarchive

I'm guessing it's because ParseSwift is the actual framework/product/module name and this is what the archive is named after.
I think this is tripping us up when using xcodebuild to generate docs, because in Xcode you can end up with a scheme name that's disconnected from a product/module name whereas when building with swift package generate-documentation --target you can't really end up in that situation.
I'm pretty sure we can find a way to work around for this without deriving the resulting URL path ourselves but I'm raising this issue here as requested by @franklinsch in https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/issues/1770 just in case this is actually unintended behaviour.

It does, thank you! The only thing that's a bit odd is that the project doesn't have a target ParseSwift:

So I'm guessing it must be the product that determines the docarchive's name? I was exploring this because I was hoping that I could introduce a temporary workaround by using xcodebuild's -target parameter and changing the requested documentation target's name to ParseSwift.
❯ xcodebuild docbuild -target "ParseSwift" -destination 'generic/platform=ios'
2022-05-31 14:22:45.902 xcodebuild[38458:13144183] Requested but did not find extension point with identifier Xcode.IDEKit.ExtensionSentinelHostApplications for extension Xcode.DebuggerFoundation.AppExtensionHosts.watchOS of plug-in com.apple.dt.IDEWatchSupportCore
2022-05-31 14:22:45.902 xcodebuild[38458:13144183] Requested but did not find extension point with identifier Xcode.IDEKit.ExtensionPointIdentifierToBundleIdentifier for extension Xcode.DebuggerFoundation.AppExtensionToBundleIdentifierMap.watchOS of plug-in com.apple.dt.IDEWatchSupportCore
Command line invocation:
    /Applications/Xcode-13.4.0.app/Contents/Developer/usr/bin/xcodebuild docbuild -target ParseSwift -destination generic/platform=ios

User defaults from command line:
    IDEPackageSupportUseBuiltinSCM = YES

xcodebuild: error: The project 'ParseSwift.xcodeproj' does not contain a target named 'ParseSwift'.

Anyway, please feel free to close this, it seems like it's on us to adjust our assumptions!

I see, the DocC archive is be named after the target's PRODUCT_NAME ("Product Name" in build settings) I believe (@d-ronnqvist can confirm), which is not the name value you pass in the -target invocation.


fix(zendesk): only fetch comment authors

Description
We were trying to fetch all users in the workspace every single time, which was taking a very long time (never really completing from what I can tell)
With this change, we're only fetching the users that commented on the tickets, which is usually very quick.
Risk
N/A
Deploy Plan
Deploy connectors and resume the connectors

LGTM 🙏


Update of Lua source to patch level 11

Lua 5.1.4 interpreter release has had about 11 bug fix patches for problems of varying severity and one language bug involving __newindex.

Let's wait for Lua 5.1.5 release (this is a RC1).


Add 0.9.6 windows realease

Recently I build Carla for windows with 0.9.6 version source code and Unreal 4.22.
my experience exists here
if I upload binary files (about 4 gigs) on Google drive, would you put it in release part of the repository for other people use?

This change is

Is it possible to add it in release part?

@nsubiron
if I upload CARLA binary built [0.9.6] for windows, would you place the google drive link of that in release part of the repository?

Could you tell me when will the version of Carla 0.9.6 for windows be available?

I will be uploading files up to the weekend
All the files will be uploaded here:
Awesome-CARLA

Hi @Amin-Tgz
First of all thanks for your PR.
This is really useful information! We'll be glad to merge these content if it is merged in the docs, not just a link to an issue.
Sadly, and as you could understand, we cannot provide these kinds of user-created-download links in our documentation :(


Add Pre & Post Script URLs

Some people may run the build script on an existing image that they've already pre-configured a bit. Others may want to run it on a vanilla version of Linux provided by their cloud provider or something like Vagrant and Docker etc.
Assuming the later case (that it's run on a vanilla version of Linux) there may be some things that users want to run before and after the installation. Some examples include:

Restricting ssh root access
Creating a new users with key
Shutdown/startup service(s) before/after installation
Install other software that they want for their system

Instead of trying to think of all these types of activities, which expands well beyond the goals of this project, giving users the ability to run pre/post install scripts allows them to have re-usable scripts.
I propose we use the same URL concept that we do for the Oracle installation files. This way users can keep the scripts on a cloud drive (Dropbox, AWS, etc) or just copy them to the server before running the build script and reference them using file:///.

Not going to implement as it's out of scope. #68 created as alternative


SQL connection should tolerate being killed by idle too long

What did you do?
Currently when the connection is idle for a long time and be killed, DM will raise error when use it afterwards and triggers an alert
https://asktug.com/t/topic/993973
What did you expect to see?
No response
What did you see instead?
Auto recover from this kind of error
Versions of the cluster
master
current status of DM cluster (execute query-status <task-name> in dmctl)
(paste current status of DM cluster here)

This situation alse exists in the dumping and loading stage

another report https://github.com/pingcap/tiflow/issues/5804


sqlx generator

Prisma schema -> sqlx schema to allow using offline mode without ever connecting to a database.

i don't think this will work since sqlx's offline stuff is only for specific queries, not for arbitrary new ones :(


How to style a slider thumb with styled-components

I am trying to style a slider with styled-components for React, but I do not get how I can style the thumb. I have a CSS that looks like this:
.faderInput::-webkit-slider-thumb {
   -webkit-appearance: none;
    width: 15px;
    height: 15px;
    border:1px solid black;
    ...
}

and my styled component looks like this
const FaderInput = styled.input`
    ...
    ::-webkit-slider-thumb {
        -webkit-appearance: none;
        width: 15px;
        height: 15px;
        border:1px solid black;
        ...
  }
`;

Does anybody know how I can port this class selector to styled-components?

const FaderInput = styled.input`
    &::-webkit-slider-thumb {
        -webkit-appearance: none;
        width: 15px;
        height: 15px;
        border:1px solid black;
  }
`;


Add plugin manager promise

Port of https://github.com/saleor/saleor/pull/11391

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: CLOUD-2286 Replace PluginManager helper with promise of plugin manager


Blog post for Retry Budgets

A blog post for retry budgets.
In addition, I've clarified the instructions on running locally and switched to using an en-dash for dates.

Awesome! Thank you @kevinoliver!

Ok @vkostyukov this has been rebased and squashed. Good to go when you are, thanks.


Split up V8Helpers.cpp/h into multiple files

Description of the problem
Currently, the V8Helpers.h header includes almost all shared helper functions, this is highly unpractical because you can't include only some things and also changing this header causes almost the whole project to recompile.
Desired solution for the problem
Instead of using this giant header file, split it up into more smaller header files.

[x] Split up into multiple files
[x] Move everything into V8Helpers namespace (don't use V8, its easy to be confused with the v8 namespace)
[ ] Move into sub-namespaces (e.g. V8Helpers::Serialization::Serializer)
[ ] Don't include all sub-files in main V8Helpers file

Alternatives you considered
No response
Additional context
No response
Version
No response
Scope
shared

Has since been split up into the shared/helpers folder


fixed for section of developer's document

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 27.733%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
208

Relevant Lines:
750

💛  - Coveralls


Metzmatan's Mark rules?

Are the rules for the Metzmatan's Mark RPG available anywhere online?

Hi; thanks for your interest in this (very much still in-development) system! The test rulebook is available in pdf form on the releases page of this Git repo: https://github.com/Mxblah/metzmatan/releases/tag/v0.1.1 (direct link)
I do intend to update the release process to include the pdf in a more obvious way for future releases.


obsoletions of terms used in extensions

How do I find out about terms used in extensions that have gone obsolete? What happens to these in PomBase?

Can I get a list of all of the annotations (any type, could be GO, modification or expression), that use "GO:0060274   maintenance of stationary phase"
thanks
val

How do I find out about terms used in extensions that have gone obsolete? What happens to these in PomBase?

I think there will be a warning in the log file from the loading of Canto data into Chado.  It will say no such term and refuse to load the annotation, if I remember correctly.

Can I get a list of all of the annotations (any type, could be GO, modification or expression), that use "GO:0060274 maintenance of stationary phase"

I only looked in the GAF file, but I think these are the only annotations:
PomBase SPAC24B11.06c   sty1    is_active_in    GO:0005634      PMID:16141205   IDA             C       MAP kinase Sty1 spc1|phh1       protein taxon:4896      20150127        PomBase existence_overlaps(GO:0060274)
PomBase SPBP8B7.30c     thi5    is_active_in    GO:0005634      PMID:16874521   IDA             C       DNA-binding transcription factor Thi5           protein taxon:4896      20140908     PomBase  existence_overlaps(GO:0060274)

OK, I think there will be more on the expression annotations.

OK, I think there will be more on the expression annotations.

Sorry I didn't read your first comment carefully enough.
This time I did it properly and queried Chado.  I found 74 annotations that use that term in an extension.  70 annotations (all expression) are from the contig files.  For example for SPAC1F8.07c:
FT                   /controlled_curation="term=gene expression, RNA level;
FT                   annotation_extension=during(GO:0060274);
FT                   qualifier=decreased; evidence=ECO:0000231;
FT                   db_xref=PMID:25102102; date=20150929"

There are 4 Canto annotations:
 SPBC28F2.12   | O-phospho-L-threonine [removed_during] maintenance of stationary phase  | 56e7806a8b535223
 SPAC24B11.06c | nucleus [exists_during] maintenance of stationary phase                 | 36867162cb7ee60c
 SPBP8B7.30c   | nucleus [exists_during] maintenance of stationary phase                 | c0b99491ed09bf31
 SPBC28F2.12   | O4'-phospho-L-tyrosine [removed_during] maintenance of stationary phase | 56e7806a8b535223

OK not so many. I can deal with these soon.


Adding Irish Transverse Mercator (EPSG:2157)

Tests failed because IRENET95 not defined. This is the datum given on EPSG.io. I don't know how to find the actual one CoordRefSystems needs...
Sorry!

Thank you @TimG1964 ! This datum is not implemented, but is easy to add.
Here is a definition that you can add to the datums.jl file:
"""
    IRENET95

Ireland 1995 datum.
"""
abstract type IRENET95 <: Datum end

ellipsoid(::Type{IRENET95}) = GRS80🌎

I used the information in the same webpage that you used to find the parameters of the CRS.
Can you please add this definition after the Ire65 datum, and a corresponding test in the test/datums.jl? The tests should pass after that.
Please let us know if you have questions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.31%. Comparing base (5307778) to head (6672e2e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #145   +/-   ##
=======================================
  Coverage   95.30%   95.31%
=======================================
  Files          27       27
  Lines        1193     1195    +2
=======================================
+ Hits         1137     1139    +2
  Misses         56       56

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

Released a patch: https://github.com/JuliaRegistries/General/pull/112365


Docker & Docker Compose support

Before you begin
Check the backlog of issues to reduce the chances of creating duplicates; if an issue already exists, place a +1 (👍) on it.
Checked, no duplicate
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Dockerfile & docker-compose.yml files are extremely common in my work and having syntax support would be great! I would hope for the following features (in order of what I priority I personally would expect)

Syntax checking for errors
Code completion
Code reformatting
Above features depend on version of Dockerfile and docker-compose.yml file (which are contained inside the file itself)

References:
https://docs.docker.com/engine/reference/builder/
https://docs.docker.com/compose/compose-file/
Describe alternatives you've considered
Alternative would be to not have these languages available. This would be possible, but editors commonly support them (idea suite, vscode etc. etc.)
Dockerfile & docker-compose.yml are dialects of Yaml so I would expect the solution to depend on yaml support.
Screenshots

Any plans to add support for it?

This is the most frequent unhighlighted syntax I'm running into these days on Zed.

A IDE needs Docker support


Isolate JuiceShop Instances from each other using NetworkPolicies

Currently a user could use RCE or SSRF vulnerabilities to connect to JuiceShop instances of other users.
This would kind of be a awesome challenge in itself 😅
Like: "Steal the challenge progress from another team"
But as we (currently 😉) don't have the possibility to add new Challenges at run time it would probably be best to prohibit any traffic coming from JuiceShop to other JuiceShop pods via k8s NetworkPolicies. Might even work to prevent any cluster internal traffic from the JuiceShop this would have to be tested to ensure that this doesn't cause troubles with the juice-balancer.

@jonasbg awesome looks good 🙌
Do you know if the dns egress selector is valid for most k8s clusters? or are there setups where this might fail because the pods don't have the label?
podSelector:
  matchLabels:
    k8s-app: kube-dns

Never maintained network policies for open source helm charts only for specific cluster where i could be very sure about the labels :D
if we can I'd like to also add the net policies to the default helm chart. And have them disableable if somebody doesn't want them (for some reason). Some parts might not be doable (e.g. juice-balancer ingress) as it would come from different namespaces / sources depending on the way it's exposed to the outside of the cluster.


Version 23.04.1 (<IP_ADDRESS>) still complains about being unable to upgrade

Upgraded to version 23.04.1 (<IP_ADDRESS>) which seems to be working fine (currently synchronizing block information).
But I still get this message "Crashplan wasn't able to upgrade but will re-attempt in one hour." at the top of the screen.
The About-box reports version <IP_ADDRESS> and I can't see any newer version on the Crashplan download page.
Running on Synology DS1517+ / DSM 7.1.1-42962 Update 5

I think I saw this glitch at the beginning also.  Do you see the message once ?

It was there all of yesterday, but I see it is gone now. All good.


Adding tokens outside of any HTML elements in raw HTML view causes a visual bug

Describe the Bug with repro steps

Add an action with HTML editor (e.g., send an email)
In the HTML editor, click the <> button to switch to raw HTML view
Clear the field
Add to the field

 and then a token of any kind.
Switch back to rich text view and observe that the token is rendered as @{...}. (See GIF)

Leftover issue from: #3849
What type of Logic App Is this happening in?
Consumption (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos

Browser
Chrome 120.X, Windows 11.
Additional context
No response

Happy to take this bug myself once I'm back from my holiday break.

Fixed by #4016.


v2.0.0 Release Notes (release_notes.md) Doc Missing an Intro

Issue
Release Notes is missing an introduction
Suggestion
Add an introduction to summarize what the page is about.

@claireyuw This is another page that misses a summary at the top. I would be taking this up as well.

/assign @Rutam21

@Rutam21 Thank you for your interest in contributing to Milvus documentation! I've added the labels and you can go ahead.


Bump FlatBuffers version to 2.0.0

Release notes for FlatBuffers 2.0.0: https://github.com/google/flatbuffers/releases/tag/v2.0.0
Fixes #2

I've only tested this on my Linux, but it works on my machine.

I'll look at this tonight or tomorrow and prepare a release

Hi @chippers! Did you get a chance to look at this yet?

Hi @chippers! Did you get a chance to look at this yet?

sorry, got a bit distracted with other work. im looking at it now


Add GtkIMMulticontext and GtkIMContext

Add GtkIMMulticontext and GtkIMContext objects.

@daa84 This also need generating InputHints, InputPurpose. See travis log

Hm, looks like cfg(feature = "v3_6") needed for InputHints, but how to set it?

See https://github.com/gtk-rs/gtk/blob/master/Gir.toml#L435-L437

Thanks :+1:

Thanks.
If travis passed, squash all commits please

done.

@GuillaumeGomez 👍 for this and CI passed

Thanks!


Docker remote API with TLS not work  on auto signed CA

I followed the site https://docs.docker.com/articles/https/
but at the end , docker --tlsverify ps  return an erro
An error occurred trying to connect: Get https://xxxxx/v1.20/containers/json: x509: certificate signed by unknown authority

If i test the same URL with CURL:
curl https://xxxxx/images/json   --cert ~/.docker/cert.pem   --key ~/.docker/key.pem   --cacert ~/.docker/ca.pem
it return Json info.
How can i use TLS conf with auto signed CA?

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

What is the exact command you're using for docker ps? I think you'll have to set all options here, e.g. something like;
docker --tlsverify --tlscacert=ca.pem --tlscert=cert.pem --tlskey=key.pem  -H=<your-docker-host-ip>:2376 ps

Or set the environment variables DOCKER_TLS_VERIFY, DOCKER_HOST, DOCKER_CERT_PATH
There may have been made some changes to that document, that are not yet visible on the website, so it may be worth trying the latest version on GitHub; https://github.com/docker/docker/blob/master/docs/articles/https.md
Let me know if that helps, otherwise I'll check the whole procedure to check if there's an issue

I have tested with twice
docker --tlsverify --tlscacert=ca.pem --tlscert=cert.pem --tlskey=key.pem  -H=<your-docker-host-ip>:2376 ps

and DOCKER_TLS_VERIFY, DOCKER_HOST
But my Docker server is under nginx proxy and i hve made the same pem file on my nginx
i followeb this site: http://blog.james-carr.org/2013/10/30/securing-dockers-remote-api/

In order not to complicate things, I really suggest to get it running without using a proxy first, and work from that.
Please keep in mind that the github issue tracker is for reporting bugs, feature requests and pull requests, and not a general support forum. This question is probably better asked in the #docker IRC channel, or the Docker-Users mailing list on Google groups

Closing as not an issue


Data science library 'kernel spec' is not installed in interpreter Python 3.8.2 (64-bit)

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

VS Code V1.45 and Python 3.8.2
Python extension is installed
Right-click on a python file and click on "Run current file in an interactive window"

Actual behavior

Doesn't start the Jupyter notebook
Expected behavior
File runs in an interactiv jupyter notebook.

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running: not started | No Kernel : Not started
Extension version: 2020.05.78807
VS Code version: 1.45
Setting python.jediEnabled: true | false
**Python and/or Anaconda version: ** Python 3.8.2 64bit
OS: Windows
Virtual environment: conda | venv | virtualenv | N/A | ...

Developer Tools Console Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Your current interpreter requires jupyter to be installed and capable of running 'jupyter kernelspec list'. What happens if you click the install button? Does it work after that?

 lines as it attempts to install. And then I get this :

Can you upload your console log? Go to Help | Toggle Developer Tools, click on the console tab, right click, save as and drop it here?
jupyter kernelspec list
is failing to work after installing for some reason.

Originally reported here https://github.com/microsoft/vscode-python/issues/11807

@ajaygujja
Please could you provide the following information:

Screenshot of the error message
Python Version
Are you using Anaconda or another interpreter
What's the currently selected interpreter (displayed on the bottom left of VS Code statusbar)

This will be displayed when u have a python file opened or when you get the kernelspec not installed error.

Output from console window

Open toggle developer tools
Go to console tab and save eveything into a text file and upload here

I have the same problem. Jupyter server running: not started | Python 3.8.2 64-bit: Not Started.

I have the same problem, using VS Code 1.45.1, Python 3.7.7 (miniconda env). For me, this workaround helps: Start VS Code from anaconda prompt with the right environment activated:

open anaconda prompt
type: conda activate your_env_name
type: code

Result: "Run current file in an interactive window" now starts the jupyter kernel without errors.

The same problem here using Python 3.8.3. I was able to solve it by deleting the C:\Users[USERNAME]\AppData\Roaming\Python directory and then reinstall all python extensions in VS.

Looks like there is an easy workaround as mentioned above : delete the Python directory and reinstall python extensions in VS.

@Pylsur

I have the same problem, using VS Code 1.45.1, Python 3.7.7 (miniconda env). For me, this workaround helps: Start VS Code from anaconda prompt with the correct environment activated:

open anaconda prompt
type: conda activate your_env_name
type: code

Result: "Run current file in an interactive window" now starts the jupyter kernel without errors.

This works for me. Thanks :)

Doesn't work for me. Opens a new VS Code window where I get the same error.

I've the same issue on Python 3.8.6 64-bit.

Also having the same issue on Python 3.8.3 64-bit. No issues on 32 bit python.


[COMPILER] Seg fault related to locals in ensure block

Minimal repro:
def test
ensure
  {}.each do |key, value|
    {}[key] = value
  end
end

Failing test: RUBY_ISEQ_DUMP_DEBUG=prism make test/irb/test_locale.rb

The same seems to happen for a rescue block:
begin
rescue
  args.each do |key, value|
    tmp[key] = 1
  end
end

Another example:
begin
ensure
  a do
    b = 1
    c = 2
  end
end

RUBY_ISEQ_DUMP_DEBUG=prism make test/openssl/test_asn1.rb

The first 3 should be fixed by https://github.com/ruby/ruby/pull/9144
The last one looks like a different problem though. I've created https://github.com/ruby/prism/issues/1999

I'm closing this as fixed since #9144 was merged containing the first three test cases (now passing), and the final one has been represented in #1999


[11135] Hotfix: clarify how to open dump file with Wireshark

Signed-off-by: JLBuenoLopez-eProsima<EMAIL_ADDRESS>

Build status


Develop the app for both Android and iOS platforms.

Admins:

POST:   admins/ -> Create a new admin
GET:   admins/  -> Get a list of admins
GET:   admins/{id} -> Get details of a specific admin
PUT:   admins/{id} -> Update details of a specific admin
PATCH:   admins/{id} -> Update specific fields of a specific admin
DELETE:   admins/{id} -> Update details of a specific admin


Bug: Answer codes validation skips validation-schema fix

What is the context of this PR?

As a part of the this fix, the schema tests were failing on runner due to the added validation. This PR updates the schema to fix the failing tests.

How to review

Make sure all schema test passes.

Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Bug: Answer codes validation skips validation when using list collectors


BindOperationsExplicitly is in documentation but doesn't exist in latest versions

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
IFilterInputTypeDescriptor.BindOperationsExplicitly  is only available in the V10 documentation. The interface doesn't have that method in the latest versions, thought.
Steps to reproduce

Check IFilterInputTypeDescriptor for BindOperationsExplicitly

Relevant log output
No response
Additional Context?
No response
Product
Hot Chocolate
Version
12.15.0

It is available... but only in the generic variant.

Closing this issue since it works as specified and is available. The reason that the non-generic version does not have the bindings is that nothing is inferred and everything has to be specified. This is consistent with other descriptors like the IObjectTypeDescriptor.


toDate string shortcut support for timezone

Would be helpful to be able to specify a timezone when using the shortcut methods.

SwiftDate 2 allows to manage easily timezone dates and times.
You can learn more on main readme file.
Thank you for your report.

Awesome, thanks for the info. Keep up the great lib!


New section with clarifications on multithreading, CAS-13342

This is being discussed in CAS-13342. Opening the PR for discussion and review.

Merging for CAS-13342.


Make it composer installable

Even the most simple dependency deserves to be composer installable!

Feel free to make it composer installable.  I've never used composer and don't much work in PHP anymore.

@krewenki Would you like to transfer ownership of this to the DocRaptor owners for maintenance? Otherwise we will probably fork it and deal with this stuff there.

I was just planning on "forking" actually just starting a new project. What ETA do you have @janxious ?

@markushausammann It would probably take me multiple days to get it done since I haven't used composer either. If you're willing to make a PR against a forked project, I am happy to review and merge such a thing at your earliest convenience.

That's a good solution, I will make a PR against your fork, just tell me when it's ready. Using packagist, everything can be setup in a way that every time you tag a release, packagist automatically publishes the new package version. You might want to open a packagist.org account or I'll register it through mine. Packagist is the default package repository for composer, which is what gem is to ruby, just better ;).

Actually you should open your own packagist.org account because you need an API key to set the hook up on github.

Repo is gonna be here: https://github.com/expectedbehavior/php-docraptor

Would you like to make changes before I make the composer PR?

@markushausammann Yeah, I am pulling in some PRs from this repo and making a few changes. Almost there.

@markushausammann I have made all the changes to the EB version of this thing, so PR away!

@janxious can we chat somewhere, I think I'm ready with the PR and I've got a few more PRs planned, if you're interested.

And could you open the issue tracker?

Issue tracker is opened. Let's continue this chat over there.


Stability questions

Starting at "When I upgrade to a new stable version of the compiler...". There are five questions dealing with perceived stability of Rust between versions of the language and crates. I believe in earlier surveys there were some questions about stability across editions.
In the past, Rust had a reputation for instability and we were very concerned with this perception. In my view, I think the opposite is now true. So, I think we should consider if these questions are still optimal. We might well consider them useful for preventing regressions in this area.
To focus the conversation, I have the following questions:

What is the key sentiment we are trying to capture about users perceptions of stability and is that caught by these questions? How do we intend to use this data?
Should we keep each of these five questions in their current state, or drop or change some or all of them?
Should we have questions about stability across the edition change? This could be forward looking, something along the lines of "Do you feel that the 2021 edition will be easy or difficult to change your code for?"

We'll go with the above but change the last two chances to something along the line "the amount and difficulty of changes I need to make to codebases when updating to new versions of the compiler are few and trivial" (definitely not with this wording). Make sure to add a N/A option.


Error: Cannot read properties of null (reading 'length') - callout block

I'm using the obsidian plugin and I get the error when trying to publish.
This is an example of a callout block I have that throws this error...
> [!info] Title
> Request the following three groups, so three separate requests:
> 	`group1`
> 	`group2`
> 	`group3`
>
> [Request Ticket](https://somelink.internal/123)
> ![[Pasted image<PHONE_NUMBER>5212.png|400]]

Adding a single extra line and the error goes away
> [!info] Title
> Request the following three groups, so three separate requests:
> 	`group1`
> 	`group2`
> 	`group3`
>
> [Request Ticket](https://somelink.internal/123)
>
> ![[Pasted image<PHONE_NUMBER>5212.png|400]]

same issue


1 Up Endpoint List Query Fix

Pull requests into this repository require the following checks to be completed by the submitter and reviewers.
Changes in this PR:
Updated the way we query 1Up's endpoint list to use their API since the table on their webpage is paginated
Submitter:

[x] Administrative tasks are complete.

This pull request describes why these changes were made.
The JIRA ticket links to this PR.
The JIRA ticket for this PR is: Lantern 536 JIRA Ticket

[x] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

[x] Code review has been performed.

Code diff has been done and checked to ensure only expected code is being committed.
A linter for the language has been run.

Primary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code is maintainable, reusable, efficient, and correct.
Code accomplishes the tasks purpose.
Code follows style guidance appropriate for the language, including passing linter checks.
Code is well commented (which does not mean verbosely commented).

[ ] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

Merger:
Prior to merging, ensure that all checks have been completed. Delete the branch after the merge
is complete.

Also needs a rebase!


Are results on new versions of GPT not available?

It seems that the only GPT results available here are GPT-3 original (i.e. davinci in the OpenAI API). Are results for newer version, e.g. Codex (code-davinci-002) or GPT-3.5 (text-davinci-003), available?
Multiple papers [1][2] suggest Codex might be the current state-of-the-art model on multiple BIG-bench tasks, and perhaps overall. I think it will be highly valuable to include the results in the official codebase.
[1]: Self-Consistency Improves Chain of Thought Reasoning in Language Models
[2]: Challenging BIG-Bench Tasks and Whether Chain-of-Thought Can Solve Them

I am not familiar with Codex or GPT-3.5, but I know that GPT-4 results will likely not be possible to assess. See footnote 5 from the GPT-4 technical paper:

During our contamination check we discovered that portions of BIG-bench [48] were inadvertently mixed into the training set, and we excluded it from our reported results.


CORE-20359: Fix Windows Liquibase error

The recent Liquibase upgrade has introduced an error to tests where the path to the changelog file is in an invalid format

build windows

We're likely to forget about this. Perhaps we should create the next ticket to upgrade liquibase now and add this information into it?
But also a question, how much effort would it be to move to that liquibase version now instead of this PR? If there are no breaking changes or deprecations and it's just a version bump, that would be easier?

The next version of Liquibase hasn't been released yet. Maybe I should leave a comment next to the liquibase version in libs.versions.toml that we should take a look at these tests?


ReducerFunction Example Error

I'm trying to write a reactivex/rxjs based redux like library for angular2. And this is what I ran into.
I want the ReducerFunction interface to be able to accommodate both the simple reducer functions, as well as an object that is a tree of reducers. This way, my redux store constructor can intelligently combine the reducers depending on the structure of the object that is passed in.
The following does not work:

/** Created by ge on 12/6/15. */
export interface Action {
    type: string;
    [key:string]: any;
}
export interface ReducerFunction<TState> {
    (state:TState, action:Action):TState;
    [key:string]:ReducerFunction;
}

interface State {
    counter:number;
}
var reducer:ReducerFunction<State> = function (state:State, action:Action):State {
      return state;
};

The following works:

/** Created by ge on 12/6/15. */
export interface Action {
    type: string;
    [key:string]: any;
}
export interface ReducerFunction<TState> {
    (state:TState, action:Action):TState;
    [key:string]:any;
}

interface State {
    counter:number;
}
var reducer:ReducerFunction<State> = function (state:State, action:Action):State {
      return state;
};

uhm is this a feature, bug, or why?
Thanks!

You can not reference a generic type without specifying its type arguments. the reference to ReducerFunction in [key:string]:ReducerFunction should have some type for TState, possibly ReducerFunction<TState> or ReducerFunction<any>.
I am not sure i understand the intention, but looks like the interface should not be generic but rather the function, so consider something like:
interface ReducerFunction {
    <TState extends State>(state: TState, action: Action): TState;
    [key: string]: ReducerFunction;
}

From the explanation above, looks like you want them to be either a function or an object, but not both. if this is true, consider re-defining the ReducerFunction as a union type:
type ReducerFunction = {
        <TState extends State>(state: TState, action: Action): TState;
    } | {
        [key: string]: ReducerFunction;
    };

Thanks! Let me play with it a bit more. Will get back to you in the next comment!

Hi @mhegazy
So I want this module to be used in the following way. the problem is that I want the State interface to be defined by the code that imports the library ("./index" in this case). I'm new to type script so not sure what to do in this case ;-/ (and thanks for your help!)
Let's assume that the ReducerFunction is either a function or an object. (I originally wanted other wise, but this might be simpler.)
The  tricky thing is that I also don't know how to write the combineReducers function. It needs to return a function where the state has the type State, but State is not available in the library, it is going to be defined by the script that calls the library. So the following implementation of the combineReducers is also not working because the State interface is not available in this scope:
import {Action, ReducerFunction} from "../interfaces";
// helper function
function pickReducer(reducers: ReducerFunction) {
    return Object.keys(reducers).reduce((result, key):ReducerFunction => {
        if (typeof reducers[key] === 'function') {
            result[key] = reducers[key];
        }
        return result;
    }, {});
}

// mixed reducer type is not supported, but I want to add them later on.
export function combineReducers(reducers:ReducerFunction):ReducerFunction {
    const finalReducers:ReducerFunction = pickReducer(reducers);
    const keys = Object.keys(finalReducers);

    function newReducer <TSate>(state:TSate, action:Action):TSate => {
        return keys.reduce((result, key) => {
            result[key] = finalReducers[key](state[key], action);
            return result;
        }, {});
    };
    return newReducer
};

import {Action, ReducerFunction} from "./index";
interface State {
    counter:number;
}

var reducer:ReducerFunction = function <State>(state:State, action:Action):State {
    if (action.type === "INC") {
        state.counter += 1;
        return state
    } else if (action.type === "DEC") {
        state.counter -= 1;
        return state
    } else {
        return state;
    }
};

describe("interfaces", function () {
    it("Reducer can be a function", function () {
        var state:State = {counter: 0};
        expect(state.counter).toBe(0);
        state = reducer(state, {type: "INC"});
        expect(state.counter).toBe(1);
        state = reducer(state, {type: "DEC"});
        expect(state.counter).toBe(0);
    });
});

//import {combineReducers} from "./index";
//describe("combineReducers", function () {
//    it("should return same reducer", function(){
//
//    })
//})

Okay so right now if I just do the following, I'm getting the following error message:
/components/luna/src/index.spec.ts.ktp.ts(10,15): error TS2339: Property 'counter' does not exist on type 'State'.

// interface.js

/** Created by ge on 12/6/15. */
export interface Action {
    type: string;
    [key:string]: any;
}
export type ReducerFunction = {
    <TState>(state:TState, action:Action):TState;
}|{
    [key:string]:ReducerFunction;
}

/// index.spec.ts file

///<reference path="../typings/jasmine/jasmine.d.ts"/>
/** Created by ge on 12/6/15. */
import {Action, ReducerFunction} from "./index";
interface State {
    counter:number;
}

var reducer:ReducerFunction = function<State> (state:State, action:Action):State {
    if (action.type === "INC") {
        state.counter += 1;
        return state
    } else if (action.type === "DEC") {
        state.counter -= 1;
        return state
    } else {
        return state;
    }
};

describe("interfaces", function () {
    it("Reducer can be a function", function () {
        var state:State = {counter: 0};
        expect(state.counter).toBe(0);
        state = reducer(state, {type: "INC"});
        expect(state.counter).toBe(1);
        state = reducer(state, {type: "DEC"});
        expect(state.counter).toBe(0);
    });
    it("create store", function () {
        var state:State = {counter: 0};
        expect(state.counter).toBe(0);
        state = reducer(state, {type: "INC"});
        expect(state.counter).toBe(1);
        state = reducer(state, {type: "DEC"});
        expect(state.counter).toBe(0);
    });
});

//import {combineReducers} from "./index";
//describe("combineReducers", function () {
//    it("should return same reducer", function(){
//
//    })
//})

State is not a name of the type State, it is a gneric type argument.
function<State> (state:State, action:Action):State { }

this should probably be:
function<T extends State> (state:T, action:Action):T { }

or just loose the generic type parameter altogether:
function (state:State, action:Action):State { }
```

Thanks @mhegazy , so I can just have a typescript file like this?

type function ReducerFunction (state:State, action:Action):State {
}

well, you need to define State in this file, or import it. so something like:

import {Action} from "./index";

interface State {
    counter:number;
}

export function ReducerFunction (state:State, action:Action):State {
    if (action.type === "INC") {
        state.counter += 1;
        return state
    } else if (action.type === "DEC") {
        state.counter -= 1;
        return state
    } else {
        return state;
    }
};

Thanks! I think Im going to move this discussion to gitter.

@mhegazy i found the problem. It looks like generics are not available in enum, so the interface below shouldn't work:
interface ReducerFunction  {
[key:string]:ReducerFunction
}
Is this correct?


remove warnings and exec rights to JSONValue.h

Just for reference, what compiler was giving warnings in the constructors?

I am using GCC but with a custom makefile (made by eclipse).
I am not sure why I get the warning and you don't (with your makefile I get no warnings)
In both case it's with -Wall
I get:
Member 'bool_value' was not initialized in this constructor
Member 'number_value' was not initialized in this constructor
etc...


General atomic element spacing guidlines

I collected some general spacing thoughts for spacing between molecules and organisms that are consistent across page types. I thought I would see if this helped and make sure we were all on the same page with these before we have more discussion
Standard spacing (no rule line)

30px above
This is true of mols/orgs under heroes as well
unless it is a text introduction under a breadcrumb which would be handled like this
60px after
This is true when the mol/org is the last piece to a section. The next element should be a rule line or mol/org of a new section.
This is true for the last mol/org on a page before the footer. (there should always be 60px between the last thing on the page and the footer)
These spacings are not additive! There should not be more than 60px between mol/orgs unless there is a rule line (see below)

Spacing with rule line divider between sections

The rule line acts a mol/org without spacing of it's own
mol/org before rule line has standard 60px after, the rule line, and the mol/org that follows the rule has the standard 30px above.

Spacing between multiple 25/75 molecules

The specs for the 25/75 do not reflect this but we use this molecule commonly without rule lines on top and bottom. The spacing in the spec is still generally correct without the rules.
An update that was made in build was allowing for 30px of space between 25/75 molecules where there were multiples of these in the same section stacked on top of each other.
The last 25/75 molecule should still have 60px of space after.
This spacing should not be used if the multiple 25/75 molecules are not part of the same section or are not intended to be grouped.
I do not think any other molecules abide by this spacing structure.

Other instances

Sidebar mol/orgs abide by the same spacing structure and the slug headings are considered part of a given sidebar mol/org.
Full width text molecules follow this structure when the mol/org  above or below is not another full width text molecule.
When multiple full width text molecules are next to each other the spacing should reflect the web type hierarchy spacing standards

@ajbush and @schaferjh - do these guidelines follow your thinking for our general approach to spacing elements on all page types? Are there things  that are missing or need to be added? Do any of these sound incorrect?

How much spacing should there be for the last molecule on a page? 60px?

Great call, yes 60px after the last molecule on the page. I have updated the original post and added this into the "standard spacing" section.

@duelj yep, this all looks correct to me -- thank you for putting it together!

This is how I've understood it so far but had trouble describing. That's for making it visual.


Global replace of 'ecoregion' to 'climate_region'

It seems like eventually we are going to need to replace the term 'ecoregion' within the code to 'climate_region' or something more intuitive for the new functionality

I agree, let's do it.

@rmscheller  When should we release NECN_Hydro?  This would help @akretchun decide when he should work on this task.

A good question.  End of Q1 2018?  I'll take care of the coding.

Yes, that's a pretty easy deadline.  I hope you will change the formatting of the input files too to reduce confusion, not just the code.  Like ecoregions.txt file should be LandisData ClimateRegions etc.

Also, ecoregions still required by the Core.


ServiceManager's build method question.

Given the decision to stop encouraging the use of the Service Locator in controller/arbitrary contexts, what are the valid use cases for the build method that it provides?
The docs talks about context, but I have no idea how to pass that context in if the locator isn't being passed into one itself, I guess that platform context would be available but that is nothing that we can't do with a standard factory that has access to the config service.
Is there an acceptable way to pass (for instance) a user context into a factory for model creation?

@digitalfu build() is used for the following:

To build the initial instance of a service.
To create new instances of a service marked as not shared
Within plugin managers, which often mark services as unshared, and/or allow passing instantiation options when creating the service; in this latter case, passing options implicitly indicates an unshared service.

The context is only passed when calling build() directly, or using a plugin manager instance.


Missing draggable ghost image in radio and checkbox in chrome 48

Here is a reproduction.
http://codepen.io/scarletsky/pen/dMypvp
When radio / checkbox is dragging, the ghost image miss.
I found that this is related to the following style:
/* _radio-buttons.scss  */
[type="radio"]:not(:checked),
[type="radio"]:checked {
      position: absolute;
      left: -9999px;         /* this line */
      opacity: 0;
}

/* _checkboxes.scss  */
[type="checkbox"]:not(:checked),
[type="checkbox"]:checked {
      position: absolute;
      left: -9999px;         /* this line */
      opacity: 0;
}

I can see the ghost image, in what browser was it tested?
Closed due inactivity, feel free to reopen it if it is still necessary.


Support Request: Custom IR blast not working on Homekit (DIY AC Remote)

Describe Your Problem
Custom IR blast not working on Homekit (DIY AC Remote)
Relevant log output
[26/06/2022, 05:55:25] [SwitchBot] Air Conditioner: WS AC Sending request to SwitchBot API. command: setAll, parameter: [22,2,1,on], commandType: command
[26/06/2022, 05:55:26] [SwitchBot] Air Conditioner: WS AC Command is not supported.

Config for homebridge-switchbot
{
    "name": "SwitchBot",
    "credentials": {
        "openToken": "",
        "notice": "Keep your token a secret!"
    },
    "platform": "SwitchBot"
}

Screenshots
I'm not native English speaker so sorry for my poor English :(
Just setting up Homebridge and connect with my Switchbots, but one of my AC IR Remote setting not working properly
none of options of switchbot's model options working so I custom up AC On and Off on Switchbot app.
Off is working on Homekit, but On is not, Is there any solutions about it? Thank you.
Device & Model
Switchbot Hub mini (JP)
Node.js Version
v16.15.1
NPM Version
v7.22.0
Homebridge Version
v1.5.0
Homebridge SwitchBot Plugin Version
v1.14.0
Homebridge Config UI X Plugin Version
v4.48.1
Operating System
Raspbian GNU/Linux Bullseye (Homebridge Image)

I would need more details on what would need to happen here. I don't use IR devices.

For example? What kind of details do you need?

I guess I don't use Ir devices, so are you wanting an option that you could set a custom On or custom Off Command?

Can you try the latest beta:

Click here to find out how to install Beta

Ok, I installed 2.1.0 release and check some new options for IR devices but still can't turn on my AC.
Is there something manual about IR devices' commands? If so, I want to read it.

@designsforall  what is your config?

{
"deviceId": "deviceId",
"configDeviceName": "WS AC",
"configRemoteType": "DIY Air Conditioner",
"connectionType": "OpenAPI",
"customize": true,
"customOn": "On",
"customOff": "Off",
"irair": {
"pushOn": false
}
as for now, just type in On and Off for custom commands
[14/10/2022, 09:17:55] [SwitchBot] DIY Air Conditioner: WS AC Sending request to SwitchBot API. body: {"command":"setAll","parameter":"25,1,1,on","commandType":"command"},
[14/10/2022, 09:18:01] [SwitchBot] DIY Air Conditioner: WS AC Sending request to SwitchBot API. body: {"command":"SetAll","parameter":"default","commandType":"customize"},
and log from Homebridge, First one (as ON ir blast) is not working, but second one (as OFF) is working.

Thanks first command shouldn’t have sent it that way with customize on. Let me look at the code. Thanks for the logs

@designsforall
Can you try the latest beta:

Click here to find out how to install Beta

Ummm, nope. Now I cant turn off with Homebridge 😂

@designsforall let me add some logging and then see if I can reproduce.

@designsforall
I think you need to set the Push On Changes Config.


Pull Request Tests with [feature] tag in commits should report failed in CI when "Feature" tests fail

I have had 2 pull requests merged where I missed the failing tests due to Travis CI reporting the build passing when tests actually failed. It is partially my fault for not being completely attentive to the test results, but in my defense there are a very large number of feature tests run and it is easy to lose track in the sea of results.
IMO, the CI should report the build failed when when the [feature] tag is present in the commit and any "Feature" test fails. This will make it easier to spot when something goes wrong.
Example PR #4494 Travis CI Passing with failed test https://travis-ci.org/PowerShell/PowerShell/jobs/266123025

This shouldn't be specific to the feature tag.  It indicates there is some bug in Test-PSPesterResults in build.psm1.  I tried a simple repro locally, and couldn't repro the issue.

I had a really fun one #4622. https://travis-ci.org/PowerShell/PowerShell/jobs/266586350#L8280 That one was an unhandled exception in a Job, that killed the calling process, and the pester process, and the test "passed". That's when I started looking very closely at the test results.

This is interesting... pester must write the result as it is going because Test-PSPesterResults will fail if the results file is missing.
I think we will have to switch to running Invoke-Pester with -PassThru  and use those results to determine if the tests passed or failed.

I found one issue... Start-PSPester is not checking the exit code from powershell.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100250

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

TagBot fix

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100612

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


node tests: Cover exports.is_for_stream_id.

@thejwzhang This looks good to me! Approved!

Pinging @showell for final review :)

@showell ready for merge

Merged!  Thanks @thejwzhang @sampritipanda @lonerz


Support merging document forks

There is easy and convenient UI to make experimental copies of a document, and to optionally "land" changes in that copy back on the original. But there isn't yet a convenient way to merge changes if the original diverges.

This is enormously important. I envision this to become the single building block for local-first (offline-first) support in Grist.
This would help enormously with mobile support as mobile connections are very unstable and often not available at all (leaving aside the fact Grist is not yet fully optimized for smartphones :wink:).
Btw. check out https://josephg.com/blog/crdts-go-brrr/ if that'd suit certain merging use cases of Grist.


[typescript] folding regions for multi-line comments

It would if great if typescript would provide folding ranges for

multiline comments (`/* ... */)
sequent, full-line, single line comment

see https://github.com/Microsoft/vscode/blob/1cec53cedd4cbab9f528c788b4cd79e9990991f3/extensions/html/server/src/test/folding.test.ts#L178
as an example

Opened https://github.com/Microsoft/TypeScript/issues/22732 to track this upstream


Automated testing of examples

Motivation

Make it easy to test examples for all distributions supported without manual labor

Benefit

Saves significant time from testing for releases (especially as we add more and more supported distributions) by automagically running the same commands that exist in the READMEs

Rough design/idea
Our example READMEs are already fairly standardly formatted.
Create something that can parse the example READMEs to pull out build commands and commands for every shell and turn them into a bunch of scripts to run. Once scripts are generated, run them as background tasks redirecting stdout/stderr to the appropriate log files and checking for errors. Sleep for X amount of time before running shutdown script to run cluster_shutdown tool and send CTRL-C to various processes.
Can add additional validations for specific cases via grep of log files or querying the cluster if desired.
We might need to tweak the formatting a bit to make the READMEs easier to parse.
Limitations/caveats:

Probably would not automate running kafka/complex examples (only non-kafka/simple examples) initially
not 100% guaranteed to work every time (think 95% solution like orchestration automation)

Future work:

enhance to add functionality to automate testing of examples using kafka and other more complex sources/sinks

Alternatives

No automation - status quo

@dipinhora this effectively exists now, correct? If so, can you close this issue?


ImportError

Hello guys i am facing this issue any ideas ? i have done all the requirements but still dont work !
Traceback (most recent call last):
File "/home/unnamed12/Desktop/SocialFish/SocialFish.py", line 3, in
from flask import Flask, request, render_template, jsonify, redirect, g, flash
File "/usr/local/lib/python3.11/site-packages/flask/init.py", line 19, in
from jinja2 import Markup, escape
ImportError: cannot import name 'Markup' from 'jinja2' (/usr/local/lib/python3.11/site-packages/jinja2/init.py)

Its seems that not all the requirements have been installed.
need to install and to use the same python environment.


Extension issue

Issue Type: Bug
Extension Name: vscode-gradle
Extension Version: 2.4.3
OS Version: Windows_NT x64 10.0.19041
VSCode version: 1.41.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

{
	"messages": [
		{
			"type": 3,
			"message": "Unknown language in `contributes.vscode-gradle.language`. Provided value: kotlinscript",
			"extensionId": {
				"value": "richardwillis.vscode-gradle",
				"_lower": "richardwillis.vscode-gradle"
			},
			"extensionPointId": "snippets"
		}
	],
	"activationTimes": {
		"codeLoadingTime": 41,
		"activateCallTime": 3,
		"activateResolvedTime": 177481,
		"activationReason": {
			"startup": true,
			"extensionId": {
				"value": "richardwillis.vscode-gradle",
				"_lower": "richardwillis.vscode-gradle"
			},
			"activationEvent": "workspaceContains:**/gradlew,**/gradlew.bat"
		}
	},
	"runtimeErrors": [
		{
			"name": "Error",
			"message": "WebSocket was closed before the connection was established"
		}
	]
}

This should be fixed with the latest version. Try update and let know if still a problem, thanks.


Failed to compile llama2-7b model due to "Type must be ranked" issue

I tried to compile llama2-7b model.(models: https://huggingface.co/steerapi/Llama-2-7b-chat-hf-onnx)
But I failed due to Type must be ranked isues.
I am using ONNX-MLIR on x86 prebuild container:
onnxmlirczar/onnx-mlir                                                             latest                 6053c88f8692   10 hours ago    1.9GB
onnx-mlir version 0.4.2, onnx version 1.14.1 (https://github.com/onnx/onnx-mlir.git 2f1003b)
LLVM version 18.0.0git (https://github.com/llvm/llvm-project.git f66cd9e9556a53142a26a5c21a72e21f1579217c)
Optimized build with assertions.

Did you get the model from hugging face (guessed from the model name)?

Yes, from github.
Model Link:
https://huggingface.co/steerapi/Llama-2-7b-chat-hf-onnx/tree/main/onnx

Did you get the model from hugging face (guessed from the model name)? Could you provide a link for me to download the model?

Thanks.
I can reproduce the error.
The error is caused by unranked tensor.
@tungld The error is in the SimplifyShapeRelatedPass. The assertion error came from the incorrect assumption that the type of ReshapeOp output is ranked.  Even if I add check to ignore Unranked tensor, the  SimplifyShapeRelatedPass still failed quietly.
Then I traced the source of unranked tensor. It came from ReshapeOp.  I create  #2539 to fix the shape inference for Reshape Op to avoid that unranked tensor of output.
Now we can run correctly with --EmitONNXIR. But there are still unranked tensor. The source is IfOp in the model.
    %1853 = "onnx.If"(%128) ({
      %4366 = "onnx.Squeeze"(%1851, %1464) {onnx_node_name = "/model/layers.0/self_attn/Squeeze"} : (tensor<1x1x?x128xf16>, tensor<1xi64>) -> tensor<1x?x128xf16>
      onnx.Yield {onnx_node_name = "/model/layers.0/self_attn/Squeeze_onnx.Yield_40"} %4366 : tensor<1x?x128xf16>
    }, {
      onnx.Yield {onnx_node_name = "/model/layers.0/self_attn/rotary_emb/Slice_onnx.Yield_38"} %1851 : tensor<1x1x?x128xf16>
    }) {input_names = [], onnx_node_name = "/model/layers.0/self_attn/If", output_names = ["/model/layers.0/self_attn/Identity_output_0"]} : (tensor<1xi1>) -> tensor<*xf16>

The condition for the IfOP is a constant. We should do constant folding on it.

PR #2544 is created to fold IfOp with constant condition.
With both PR, the error in the code gen:
opt: decoder_model_fp16.unoptimized.bc: error: can't skip to bit<PHONE_NUMBER>76 from 320
opt -O0 --mtriple=x86_64-apple-darwin22.3.0 --code-model small -o decoder_model_fp16.bc decoder_model_fp16.unoptimized.bc
Error message:
Program path: /Users/chentong/Projects/llvm-project/build/bin/opt

I also tried 7b-f32 model from github.com:microsoft/Llama-2-Onnx-7-32.git. Error in SimplifyShapeRelatedOps. At line 332, the size of slicedDims is 0.

@chentong319 is this issue still open?

@chentong319 is this issue still open?

@chentong319 @AlexandreEichenberger @tungld Thank you.
It works for me now after I applied "-store-constants-to-file /external/onnx-models/llama-7b/decoder_model.onnx" option and use float32 model.
podman exec onnx-mlir31025 onnx-mlir -O3 --maccel=NNPA -store-constants-to-file /external/onnx-models/llama-7b/decoder_model.onnx

podman exec onnx-mlir31025 onnx-mlir -O3 --maccel=NNPA -store-constants-to-file /external/onnx-models/llama-7b/decoder_model.onnx

FYI, with new changes in onnx-mlir, you need --mcpu=z16 to trigger NNPA code.

@tung: would it make sense to issue a warning if we have a NNPA access but no mcpu flag?

Yes, it totally makes sense, I often forget --mcpu :)

@AlexandreEichenberger this PR #2589 is to issue a warning.


Nova 4 Support

Any plans to add support for Nova 4.0?

I did not purchase Nova 4 🤣. But you are welcome to contribute PR.

#27


Ensemble Model interpretability

Hello,
I have a doubt regarding the use of SHAP for an ensemble model.
I trained a model on 10-fold cross-validation and I am using the stack of trained models as an ensemble for my target. I use MLPs as the base model. For each fold, I perform standardization of my features and feature selection selecting a subset of 40 features with which I train the fold's MLP. Therefore, at the end of the training, I would have an ensemble model with 400 input features (40 features x10 MLPs). Even though among these features some are repeated (I have checked it and at the end of the day I have around 100 different features) due to the standardization step within the 10-fold cv, a different training set is used on every fold, there is a bit of variability among the features.
To interpret the whole model (ensemble 10 MLPs) I thought about obtaining the explainer of each MLP using its own training set and then obtaining the SHAP values for each of the MLPs of my test set. Then I would sum up the SHAP values of the repeated features, even though these are not exactly the same due to the standardization and I would use as the expected value the average of the sum of the expected values of each MLP. Since for building the ensemble I am just averaging the final results of the MLPs and the SHAP explainer model is linear, I think I am not violating any mathematical conditions.
Would this be rigorous? are the SHAP values of an ensemble model (average) the average of the SHAP values of each model of which it is composed?
I am obtaining the SHAP values using Kernel SHAP.
Thanks for the library and your patience!

Slightly unrelated, but this issue might be of some help: #112
Adding or averaging the shape values of an ensemble of models taking different features and features with different distributions seems unreliable. An approach would be analyzing the shape values of each model of the ensemble separately. This would, of course, make model interpretability harder than having a unique XAI analysis.


Add support for Poetry

After many refactoring/preparation PRs, we're now ready to add support for the package manager Poetry:
https://python-poetry.org
To use Poetry, apps must have a poetry.lock lockfile, which can be created by running poetry lock locally, after adding Poetry config to pyproject.toml (which can be done either manually or by using poetry init).
For now, if a requirements.txt or Pipfile is found it will take precedence over poetry.lock for backwards compatibility (in the future this will become a warning then an error). This means users of the third-party python-poetry-buildpack will need to remove that buildpack in order to use the new native Poetry support, since it exports a requirements.txt file during the build.
Poetry is installed into the build cache rather than the slug, so is not available at run-time (since it's not typically needed at run-time and doing so reduces the slug size). The entrypoints of installed dependencies are available on PATH, so use of poetry run or poetry shell is not required at run-time to use dependencies in the environment.
When using Poetry, pip is not installed since Poetry includes its own internal vendored copy that it will use instead (for the small number of Poetry operations for which it still calls out to pip, such as package uninstalls).
During normal (non-CI) builds, the poetry install --sync command is run using --only main so as to only install the main [tool.poetry.dependencies] dependencies group from pyproject.toml and not any of the app's other dependency groups (such as test/dev/... groups, eg [tool.poetry.group.test.dependencies]).
On Heroku CI, all default Poetry dependency groups are installed (i.e. all groups minus those marked as optional = true).
Relevant Poetry docs:

https://python-poetry.org/docs/cli/#install
https://python-poetry.org/docs/configuration/
https://python-poetry.org/docs/managing-dependencies/#dependency-groups

See also the Python CNB equivalent of this PR:

https://github.com/heroku/buildpacks-python/pull/261

Note: We don't support controlling the Python version via Poetry's tool.poetry.dependencies.python field, since that field typically contains a version range, which is not safe to use. Use the newly added .python-version file support instead. For more on this, see the longer explanation over in the Python CNB repo:
https://github.com/heroku/buildpacks-python/issues/260
Closes #796.
Closes #835.
GUS-W-16810914.

https://devcenter.heroku.com/changelog-items/3050


Allow more general searching

It might be nice to be able to search for the label on an operation. In this case it would probably make sense if the search cycled through possible matches.

ffddeb88f5ff6aa3405e491ec4df2142c997507d added the ability to find operations by name.
The thing left to do is cycle through possible matches.


v2.0.0-pre

Added

Vector subtraction support (#71)
F32 newtype (#72, #75)
num-traits support (#80)
Quaternion::dot and ::inv (#81)
Vector3d ops for Quaternion (#82)
Quaternion::magnitude, ::scale, ::to_array, and ::IDENTITY (#83)
Quaternion::axis_angle (#84)
Quaternion::new (#85)

Changed

Refactor Vector types (#69)
MSRV 1.47+ (#75)
Make Quaternion opaque and module private (#70, #85)

Fixed

acos() behavior for zero/negative (#79)

Codecov Report

Merging #86 (f92e44b) into main (9f2dd2a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   57.42%   57.42%
=======================================
  Files          38       38
  Lines         876      876
=======================================
  Hits          503      503
  Misses        373      373

Impacted Files
Coverage Δ

src/quaternion.rs
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 9f2dd2a...f92e44b. Read the comment docs.


sets: Add utilities for map of sets

This introduces two new helper functions to add and remove to map of
sets. These are simpler to use and help ensure all callers are correct +
efficient.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
227ba2ce1745804c98be7686d9edd372e70389c0
link
true
/test integ-security-multicluster

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

/retest


PSA: "Why my base16 theme doesn't get applied in tmux anymore?"

Because tmux 3.3a ships with option allow-passthrough set to false by default now. Yup, I have no clue what that means, not that well versed on terminal emulation or ANSI escape sequences. Also it prefers to set TERM variable to tmux-256color as opposed to screen-256color now, so it may or may not mess up your other terminal applications dependent on this variable.
To get your stuff operable you would fancy to add these two lines in your tmux.conf:
set -g allow-passthrough on
# if having issues with your other programs dependent on TERM env variable
set -g default-terminal screen-256color

tmux 3.3a discussions: https://github.com/tmux/tmux/issues/3218
Hope it was worth the read.

Another tmux issue I noticed: since the "new" base16-shell writes to a BASE16_THEME environment variable (as opposed to the legacy behavior of writing to a ~/.vimrc_background file), any other existing tmux panes that open vim will launch with the last color scheme known to its environment when that pane was launched, and not use the latest BASE16_THEME variable set from the pane that set the theme.
Steps to reproduce:

Open tmux with 2 panes. The current theme is Theme A.
On Pane 1, set a new base16 theme. The new theme is Theme B.
Switch to Pane 2
On Pane 2 open vim. The vim colorscheme is still Theme A.

Workarounds:

Re-source your shell's rc file
Open a new tmux pane after setting the shell theme

@nbn22385 I'm looking at creating a fix for this atm. Out of interest, what OS and terminal emulator do you use?

MacOS 12.4 + Alacritty for personal use and Windows 10 + Windows Terminal for work.

Just wanted to report the same issue here on macOS 12.5 running Alacritty and Tmux 3.3a. Happy to provide any additional info if needed!

I have the following setup: local machine with tmux, in one window I ssh (or mosh) into another machine and open a nested tmux session there; inside the nested tmux session I ssh into a local VM. In this scenario the theme is not applied correctly (it is if I ssh into the VM directly from a tmux window on my local machine, i.e. not nested).
To fix it I updated my theme at ~/.config/tinted-theming/base16_shell_theme to
if [ -n "$TMUX" ] || [ "${TERM%%[-.]*}" = "tmux" ]; then

because in the remote VM the variable TMUX is not set (it's not running inside the VM, but on the server hosting the VM that I use to access the VM) and TERM is set to tmux-256color.
Maybe it's worth to have the same change in: https://github.com/tinted-theming/base16-shell/blob/56de873b62500f144446a9bb66d541bddd5c38b5/templates/default.mustache#L33

Is there ongoing work to enable support for base16 colors in tmux without requiring the passthrough option to be enabled?

Closing this issue now since the information is included in the repo readme.


update node

For codaco/Network-Canvas#962, so window.require isn't assumed.

Is this development happening separately, or do you want to merge this in?

Sorry, I see this is a different repo. Does it break the existing apps if it were to be pulled in by accident? Steve and I are working on protocol validation issues at the moment, so just wanted to check.


Tooltip goes away even while currently hovering over tooltip trigger

At one point react-tooltips were working as expected, but only recently did we find that even while hovering on the tooltip trigger, the tooltip would open initially, and then precede to close even if the cursor never leaves the tooltip trigger.
After some inspecting, one of the things I was noticing is that sometimes the  hideTooltip method is receiving an undefined event. Any reason for this to happen?
Adding this bit of code fixed the problem for now, but i am wondering what could have caused that to happen.
  if(e == undefined){  return;   }
Any ideas?
(Something to note is that I can see the react state change of the react-tooltip-component, and have completely removed my styles,  and can still reproduce this issue, so i am fairly certain custom styles/animations are not effecting this. )

That's really strange - you could change that return to debugger and try to figure out what the undefined event means....

If anyone else is having this issue, I found that adding this line to the hideTooltip function worked:
if (e === undefined) return
For some reason an undefined event was triggering the close of the tooltip.

I am experiencing the same. I use react-tooltip in a next-js page to show footnotes. They used to work great, but now I get the same thing as @tylergaugler16 .
However, if I move the mouse onto the area, and manage to hold it perfectly still, I get the good old behavior.
This is my code:
class Footnote extends React.Component {
    constructor(props) {
        super(props);
    }

    render() {
        return (
            <React.Fragment>
                <a data-tip
                    data-for={'global_' + this.props.footnote_id}
                    data-event="touchstart focus mouseover"
                    data-event-off="mouseout">
                    <sup>{this.props.footnote_number}</sup>
                </a>
                <Portal selector='#tooltip-root'>
                    <ReactTooltip id={'global_' + this.props.footnote_id}
                        aria-haspopup='true'
                        delayHide={500}
                        delayUpdate={500}
                        effect='solid'
                        globalEventOff='touchstart'
                        className='footnote'>
                        {this.props.children}
                    </ReactTooltip>
                </Portal>
                <style jsx>{`
                    a {
                        cursor: pointer;
                    }
                    sup {
                        color: #060;
                        border: 1px solid #000;
                        padding: 0 1px;
                        margin: 0 3px;
                    }
                    :global(.footnote) {
                        max-width: 50vw;
                    }
                `}</style>
            </React.Fragment>
        );
    }
}

The Portal element is a ReactDOM.createPortal portal, and places the tooltip in a div called tooltip-root in the very beginning of the  element. I think I recall that this was done to make the tooltip work in e.g. modals.
Regardless - I thought I would add my code, perhaps my case is similar to the Tyler's. As I said, it used to work, but does not anymore. Am I using it correctly?
I have not yet tried the fix Tyler proposed.


add support for making a live usb

https://discourse.nixos.org/t/generate-live-usb-with-nixos-flakes/12215

This is possible and would save a lot of time reworking systems or setting up new ones.
Check out https://github.com/bjornfor/nixos-config/blob/master/isos/livecd/default.nix for one idea, though nix-build might do the trick. Either way, write a convenience script to generate the ISO and write to a mounted drive.
looks like this is pretty easy: https://www.github.com/jethrokuan/nix/tree/master/hosts%2Fniximg.nix


Add support for jupyter notebook

This branch is a continuation of PR https://github.com/bndr/pipreqs/pull/404. Copied it to the pipreqsxp fork so that the other members can also contribute.

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (f041de9) 90.07% compared to head (3b16e0b) 90.16%.

Files
Patch %
Lines

pipreqs/pipreqs.py
92.68%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             next     #414      +/-   ##
==========================================
+ Coverage   90.07%   90.16%   +0.09%
==========================================
  Files           2        2
  Lines         262      295      +33
==========================================
+ Hits          236      266      +30
- Misses         26       29       +3

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

@alan-barzilay
We rebased this branch into the next branch to add the poetry changes.
After that, we added a flag --scan-notebooks to give the ability for the user to include jupyter notebooks in the files scanned by pipreqs.
Also, we also did a change in the tests to suppress all the logged warnings and errors during test execution.
At last, we updated the README file explaining how the user can install pipreqs without the dependencies needed for jupyter notebook support.

@alan-barzilay @mateuslatrova Just rebased all the changes, fixing possible cross-contamination errors. Please, feel free to check my changes to make sure everything is ok. Also fixed other changes asked by Alan.

Actually, considering the coverage error was fixed by adding the installation of ipython as a dependency, we can assume that ipython is indeed necessary.


Skeleton3D: Convert 3.x skeletons

This implements conversion of the old 3.x Skeleton format to 4.x upon load (similar to old Mesh format conversion).
Skeletons had a Transform pose property in 3.x, which was broken out in 4.x to

Vector3 pose_position;
Quaternion pose_rotation;
Vector3 pose_scale

Additionally, in 3.x, the pose of the skeleton was relative to the rest position; in 4.x, the poses are absolute.
So, we recompute the poses to be absolute and break the Transform3D into its various components.

This should be fine for the bone properties.
However, there is no animation compatibility, so if you want full compatibility with 3.x Skeleton, you will need to consider that as well; There is never full compatibility possible with animations. Bone animation is now absolute, not relative, and scale has been isolated so that transforms should be different when combined with rotation and scale.

Yes, this is fixed in my next PR. https://github.com/godotengine/godot/pull/87106

Thanks!


Get stalled tasks

Get a list of Job()s that are:

in a given queue or not
with a given name or not
in the doing state for more than X secondes

(we'll need a date column to know when the task was started)


Do not instantiate DWCs in PYNQ shell if not required

The Xilinx AXI stream data width converter IP's raise an error if the widths of the input and output streams are the same. This causes FINN designs that have input or output width equal to the DMA output stream width to fail during PYNQ shell Vivado project generation if e.g. SIMD is set to 16 for the first layer of the tfc_w1a2 test. The shell stitcher script should detect this condition and not instantiate any DWCs in this case.
Note: not having DWCs may raise endianness issues in the generated driver, needs to be checked.

This was addressed in the branch feature/pynqz1_slim_shell which also features a slimmer and simpler shell. This is merged in v0.3b and DWCs are no longer instantiated if not needed.


cronjob advice

I see there's a TODO item in the README for periodic updating of cache_list. Using your gupdatedb advice from brew install findutils, this cron job worked for me to update hourly:
0 * * * * /usr/local/bin/gupdatedb --localpaths=$HOME --prunepaths="/Volumes .*node_modules.*" --output="$HOME/locatedb" > "$HOME/cron-output-loaddb.log" 2>&1

however, is there a reason you recommend gnu locate and updatedb, when equivalent bsd commands are already installed on (most) macs?  There's also mdfind which taps into Spotlight's db (but is supposedly slower than locate).
Not to mention that it's more common on MacOS to use launchctl a .plist file?
All of this automation requires some additional permissions for the scripts to generate files, thanks to SIP. So if someone understands this all better than me, they should consider coming up with a solution that is the least confusing and has troubleshooting steps, such as putil -lint /path/to/.plist, log show, etc.
Otherwise it's possibly not worth adding this todo item other than to say something generic like "Use your favorite scheduler" (and leave it up to the person to figure out).

Forgot to mention that in the above cron /usr/local/bin/gupdatedb is for intel mac's but could be /opt/homebrew/bin/gupdatedb on an m-chip (apple-silicon) Mac. People would have to consult which gupdatedb and update it accordingly and doing anything dynamically in cron is a bit hard to do because of its locked down shell environment.

I personally also prefer the cached_list to load extremely fast, which requires keeping locatedb much smaller.  To do this, I have all my important repos for work/home projects within just $HOME/repos, e.g., $HOME/repos/work  $HOME/repos/personal and only index that:
# Keeping locatedb small (by only loading my $HOME/repos path and ignoring node_modules, makes it load MUCH faster.
# I use `mfind` on the cli to use spotlights more extensive database
alias loaddb="gupdatedb --localpaths=$HOME/repos --prunepaths='.*node_modules.*' --output=$HOME/locatedb"

This keeps my ~/locatedb around 1-2MB instead of 20+ MB and loads instantly from telescope. It also generates the db so fast, that I can run loaddb in less than 2 seconds. So I can more frequently run the cron job to keep it up-to-date:
0/5 * * * * /usr/local/bin/gupdatedb --localpaths=$HOME/repos --prunepaths=".*node_modules.*" --output="$HOME/locatedb" > "$HOME/cron-output-loaddb.log" 2>&1

Thanks for sharing @rkulla!

however, is there a reason you recommend gnu locate and updatedb, when equivalent bsd commands are already installed on (most) macs?

Yes, I don’t have a mac for personal use, so it’s hard to debug the inevitable corner cases that later cause issues :).
I’m happy to update the README with something more generic and to redirect to a GitHub discussion thread for more detailed discussions.


calyptos steps should print out the next step (and possible command) required at the end of the output

After the successful validate command shown below:
'''
[VALIDATION PASSED    ] Cluster one has both an SC and an CC
[VALIDATION PASSED    ] Cluster one has node controllers
'''
It should tell the user what the next step that can be run should be, something like:
'''
Next step: run the prepare step "calyptos prepare -e environment.yml"
'''

Got one step further here but didnt print out the actual command to run:
https://github.com/eucalyptus/calyptos/commit/b459d46a9e25cc2f4c30a640ef59592e991a2469


Concat minified files with source map results in line offset

Hi,
even if i think this repo is not maintained anymore maybe anyone has the same problem and maybe anyone found a solution already:
I am concatenating multiple files, some of them are minified and have links to a sourcemap. Like zxcvbn or jquery.min.js. When I concat them and validate the source-map it have an offset in the lines after the sections with the source-map-comments.
Anybody else has this problem?

I added this line
options: {
    process: src => src.slice(0, -1),
},


trim property

trim
Controls the trim on an armor piece.
If the armor piece already has a pattern and material, you can change only one and keep the other by omitting the one you don't want to change.
Usage

trim:[material=<material>;pattern=<pattern>]
or trim:[material=<material>]
or trim:[pattern=<pattern>]

Only implemented for 1.20+ because it is marked as Experimental in 1.19
Requested by Aviside

Done 👍

Done