Regression in entryclone regexp handling

Consider
\documentclass{article}
\usepackage[utf8]{inputenc}
\usepackage[T1]{fontenc}
\usepackage[style=numeric,defernumbers,sortcites]{biblatex}
\DeclareBibliographyCategory{related}
\DeclareBibliographyCategory{main}

\DeclareCiteCommand{\cite}[\mkbibbrackets]
  {\usebibmacro{prenote}}
  {\addtocategory{main}{\thefield{entrykey}}%
   \usebibmacro{citeindex}%
   \usebibmacro{cite}}
  {\multicitedelim}
  {\usebibmacro{postnote}}

\DeclareCiteCommand{\citer}[\mkbibparens]
  {\usebibmacro{prenote}}
  {\addtocategory{related}{\thefield{entrykey}}%
   \usebibmacro{citeindex}%
   \usebibmacro{cite}}
  {\multicitedelim}
  {\usebibmacro{postnote}}

\addbibresource{biblatex-examples.bib}

\DeclareSourcemap{
  \maps[datatype=bibtex]{
    \map{
      \perdatasource{biblatex-examples.bib}
      \step[fieldsource=entrykey, match=\regexp{(.*)}, final]
      \step[entryclone={rel-$1}]
    }
  }
}

\begin{document}\nocite{*}
\cite{shore,sarfraz,spiegelberg}

\citer{rel-spiegelberg,rel-springer}

\printbibliography[category=main,title=Main List,resetnumbers]

\DeclareFieldFormat{labelnumberwidth}{\mkbibparens{#1}}
\printbibliography[category=related,resetnumbers,title=Related Documents]

\end{document}

from https://github.com/plk/biblatex/issues/297. With biblatex 3.12/Biber 2.12 the document gives the expected output.
But when run with the current dev versions from Sourceforge the rel- entries are not produced. The .bcfs look similar enough, but the .bbl from the dev versions does not contain the cloned entries.
The simpler example by David Purton from https://tex.stackexchange.com/q/481491/35864
\documentclass{article}
\usepackage[defernumbers=true,backend=biber,style=chem-acs,articletitle=true,doi=true]{biblatex}
\usepackage{filecontents}
\begin{filecontents}{\jobname.bib}
@article{citea,
author = {McClane, John},
journaltitle = {20th Century Fox},
pages = {1--10},
title = {{Die Hard}},
volume = {1},
date = {1988}
}
\end{filecontents}
\addbibresource{\jobname.bib}
\DeclareSourcemap{
  \maps[datatype=bibtex, overwrite]{
    \map{
      \step[fieldsource=entrykey, match=citea, final]
      \step[entryclone={citeb}]
      \step[fieldset=options, fieldvalue={dataonly}]
    }
  }
}
\begin{document}
\nocite{citea}
Yippee Ki Yay, motherf****r.\cite{citeb}
\printbibliography
\end{document}

works fine.

The issue here is the \perdatasource{biblatex-examples.bib}. Since the \addbibresource{biblatex-examples.bib} is not found as an absolute or relative path, it's resolved by kpsewhich. This results in an absolute path but then this is compared to the string biblatex-examples.bib which fails. This could be considered a bug but did it work with the previous version or is it perhaps an anomaly of biblatex-examples.bib being found by path search in previous attempts?

Oh I never thought it would be that...
Indeed, in my tests
\documentclass{article}
\usepackage[utf8]{inputenc}
\usepackage[T1]{fontenc}
\usepackage[style=numeric]{biblatex}

\addbibresource{biblatex-examples.bib}

\DeclareSourcemap{
  \maps[datatype=bibtex]{
    \map{
      \perdatasource{biblatex-examples.bib}
      \step[fieldset=note, fieldvalue={Hello}]
    }
  }
}

\begin{document}
\cite{sigfridsson}

\printbibliography
\end{document}

works with Biber 2.12 but not with 3.12 with an otherwise similar set-up.
I'm in two minds about this. Obviously it would be more convenient for me when I'm answering TeX.SX questions if kpsewhich files would still match \perdatasource. And since it worked before we risk breaking people's documents. On the other hand, I'm not sure how important this is and how complex it would be to fix this on the Biber side.

I suppose there is a chance that there would be two files with the same name in different places with different content ... this is more of an edge case that the example in question though so this is now fixed in 2.13.

Yes, that occurred to me as well. One might have something like \addbibresource{chapter-one/bib.bib} and \addbibresource{chapter-two/bib.bib}. I'll compare with Biber 2.12 tomorrow if I can come up with a useful test.

Fixed in latest 2.13 release.


Why is pam_unix required, even if unixAuth = false

This is more of a proposal/question than an actual pull request. Not sure how to actually test it!
Motivation for this change
I was unable to set security.pam.services.sshd.unixAuth to false, and was having issues with SDDM/SSSD previously #94744 then I noticed no matter what, pam_unix.so was marked as required. Even if unixAuth = false.
Things done
Wrapped all * required pam_unix.so in optionalString's based on other lines I saw in this file. Not sure if that would fix my problem or not, hoping someone who knows more about NixOS could weigh in. I'd also like to note that SSSD worked with SSH/Console log in out of the box when I opened #94744 and are now both borked for some reason.

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Our whole PAM situation is in dire need of a refactor, and much more tests.
See https://github.com/NixOS/nixpkgs/pull/90490#discussion_r441150921 for pointers and https://github.com/NixOS/nixpkgs/issues/90640 for details.
I'm pretty sure there are some interesting corner cases not handled properly.

@flokli what would you suggest? I'm not sure how exactly to help, as it apparently the tests didn't like what I tried to do

I'll close this since @Rudi9719 seems to be mostly inactive. Feel free to reopen it or do a new one as soon as you have to again :D
Still thank you for your contribution

I had to switch to a more stable distro and haven't had time to circle back and test updates- this branch is probably outdated

as I said feel free to reopen as soon as you have time to work on it again :D


[RMP] KDD Tutorial (Merlin Models—Retrieval, Ranking, and Negative Sampling)

Problem:
We have an accepted KDD tutorial (outline can be found here). But Merlin Model does not include all the necessary components to build the outline examples. We also don't have all the experiment results that allow us to tell the desired story, i.e.: Two-tower is better than MF for the retrieval stage  & a two-stage model (Retrieval+Ranking) leads to a better score than Retrieval only.
Goal:

Implement the missing blocks in Merlin Models to support the KDD tutorial notebooks.

[x] Negative sampling for ranking

[x] https://github.com/NVIDIA-Merlin/models/pull/551

[x] https://github.com/NVIDIA-Merlin/models/pull/552

[x] https://github.com/NVIDIA-Merlin/models/pull/583

[ ] https://github.com/NVIDIA-Merlin/models/issues/596
Note: Marc / Oliver to add the PR here

Note: Negative sampling refactoring for retrieval is not in scope for KDD tutorial. We will re- visit the refactoring in 22.08. Marc to add a roadmap ticket for this.

[ ] Support batch-predict for top-k recommender
https://github.com/NVIDIA-Merlin/models/issues/499 (this is not a blocker for KDD tutorial)
Note: Batch predict is a P2

[ ] Code for generating local-prediction step of two-stage model

[ ] https://github.com/NVIDIA-Merlin/models/issues/498 (this is not a blocker for KDD tutorial)

Run (Hyper Parameter Optimization) HPO experiments for retrieval and ranking steps:

[x] Preprocessing of REES46 commerce dataset
[x] Script for Retrieval models with Two-tower and MF
[x] https://github.com/NVIDIA-Merlin/models/issues/544
[x ] Script for Ranking models with DLRM and DCN
[ ] https://github.com/NVIDIA-Merlin/models/issues/545
Note: 22.07 will focus on retrieval. Gabriel to update the status here . These are related to integration tests which are focussed on retrieval.
[x] Compute NDCG for ranking models (grouping preds by example id and converting preds to TF tensors to use NDCG metric implementation) --> we have a custom function implementation for that task.

For 22.08 - we will refactor code for negative sampling for retrieval. This may not be in time for KDD tutorial.
Experiments for ranking will have to be done for KDD tutorial.

Evaluation pipeline of the 2-stage model
Note: Ronay to discuss with Sara and Gabriel and update these tasks. This would be mostly be a custom functionality for the KDD tutorial.

Constraints:

REES46  has only positive interactions, we need to generate negative samples for training the ranking model
Get HPO results that show the Two-tower model is better than MF, the current experiments are showing the opposite.
Get results of the ranking models to update the outline accordingly
Save and Load back a two-tower model : https://github.com/NVIDIA-Merlin/models/issues/498

Starting Point:
Proposal: https://docs.google.com/document/d/1NtuE9dKV3q7Amg6RMJbiFDidP486jW4RACJZ4V8ED80
New content doc: https://docs.google.com/document/d/1mPT0QVjdZKh7Pe4O6hQOMljTfWMBLYoSRCxDUPI2Mhs/edit
The proposed outline is:
* Kickoff: 5 min
* Presentation: 25 min
      Introduction to two-stage recommender systems
* Hands-on Lab
     **Retrieval model : 50 min
             Introduction to candidate generation and negative sampling techniques
             GPU-accelerated ETL with NVTabular
                     Introduction to NVTabular
              Training a MF model with TensorFlow using Merlin Models
              Training a Two-Tower model with TensorFlow using Merlin Models
                    Generating more features with NVTabular
       Break: 10 min
       ** Two Stage Recommender Models: 45 min
              Training ranking models using Merlin Models
       ** Model evaluation: 30 min
               Evaluation of  two-stage recommender systems
Wrap up and Q&A: 15 min

Some context:
How do we train a DLRM model without negative interactions, i.e. explicit binary target column?

We would need a candidate retrieval stage to generate the candidates, and then train the ranking model with these
candidates. In that case, the ranking objective will be a binary classification task, and the loss is binary cross entropy. For that we need to generate our own binary target column based on the output from candidate retrieval stage.

Do we have an integrated way to feed the candidates from the retrieval model to ranking model? How would that work?

We will need to implement an additional step to generate candidates, save the data and load it back for the ranking step. Currently we are not supporting this functionality.

@bschifferer , please fill up the problem, scope and starting point section above. It may be listed in the comments section. Please provide them above for clarity. Let me know if you need any help.

@viswa-nvidia  @karlhigley  @sararb  for viz. Some updates:

currently HPO results on retrieval model with Ecom-REES dataset does not show strong justification that TT model is always better than MF. Thus, we will not move with this motivation, instead we will showcase with different hyper-parameters we can get better results with TT.
we are not blocked by #499 for now.
We did not run HPO experiments for ranking model yet, it is waiting for negative sampling api from Marc. But I will create the pipeline asap, and run some HPO based on manual negative sampling function that we created.
Saving and loading back a TT model is still required, and we still get an error there.

This ticket is not in POR. Need to refactor this ticket. there is another ticket covering negative sampling work [RMP] Improve negative sampling for retrieval

@rnyak , please check off the done work and close this ticket https://github.com/NVIDIA-Merlin/Merlin/issues/269

[ ]  [BUG] The negative sampling for ranking gives AUC 0.00 value when we add sampling layer  to the Model class models#596
Note: Marc / Oliver to add the PR here

Let's keep it open due to https://github.com/NVIDIA-Merlin/models/issues/596

Closing as bug is tracked seperately


There is no data to drill down，Request to add

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
Expected results
Actual results
Steps to reproduce

The issue template should be filled not left empty :)

There is no data to drill down，Request to add

Notice: this issue has been closed because it has been inactive for 227 days. Feel free to comment and request for this issue to be reopened.


When creating/saving a GKE cluster validation for nodepools with same name does not happen

If this is a UI issue, please file an issue at https://github.com/rancher/dashboard/
Rancher Server Setup

Rancher version: 2.6-head commit id: 0ab00d7
Installation option (Docker install/Helm Chart): Helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1

Proxy/Cert Details: Rancher generated

Information about the Cluster

Kubernetes version: 1.20
Cluster Type (Local/Downstream): Downstream - GKE v2

Describe the bug
When creating/saving a GKE cluster validation for nodepools with same name does not happen
To Reproduce

Deploy a GKE v2 hosted cluster.
Have two nodepools with same name.
Create/Save cluster details
No error is seen during creation.
Error seen on cluster after deployment: Cluster [c-mf29m] cannot have multiple nodepools with name <nane>
Edit cluster, user does not have access to edit the nodepool name, but has to delete one and add another one.

Expected Result
When creating with same nodepool names, Error should be seen and not allow user to save details

This is the same as another issue around allowing node pools with the same name - note that this is in the Ember UI

Still reproducing the original issue in v2.6-head https://github.com/rancher/rancher/commit/c47178eb1457a7ff1aac135c86d3627dc971b704.
ENVIRONMENT DETAILS

Rancher Install: Helm
Rancher version: v2.6-head https://github.com/rancher/rancher/commit/c47178eb1457a7ff1aac135c86d3627dc971b704
Browser: Chrome

TEST RESULT
FAIL
REPRODUCTION STEPS

Setup Rancher and navigate to the UI in a browser.
Provisioned a downstream RKE1 node driver cluster.
Added a new node template and ensured that both name prefixes are the exact same; verified that an error does occur:

Exit out of RKE1 cluster provisioning and provision a hosted GKE cluster.
Filled out the options and when I got to the Node Pools section, added a new node pool with the same name as the first one; there is still no validation that occurs and the cluster creates.
After some time, I received the following error message:

Removing the additional node pool and attempting provisioning again, I am able to successfully provision the hosted GKE cluster:

While this issue awaits a fix, here is the QA test plan:
TEST PLAN

Setup Rancher and navigate to the UI in a browser.
Provision a downstream RKE1 node driver cluster.
Add a new node template and ensure that both name prefixes are the exact same; verify if an error occurs.
Exit out of RKE1 cluster provisioning and provision a hosted GKE cluster.
Fill out the options and when you get to the Node Pools section, add a new node pool with the same name as the first one; verify if there is any validation that occurs.
Create the GKE cluster and validate if the cluster successfully provisions.

This issue was tested on #9987


fix: swap survey editor

Swapping editor with survey editor. However implementation doesn't seem to work for me also it has couple usability quirks.
I would recommend to stay with original editor if possible.

CC @craicoverflow


Implement calibration module using argmin crate.

All pricing models should be able to be calibrated to market prices.
A nice Crate called argmin available here looks like it can achieve this, using Brent method or something similar.
Can also perform calibration using RustQuant::autodiff possibly ?
This requires the ability to read in price data from .csv or .json, for example. See related issue #23

Could we start first with grouping the pricing models inside the modules to have a better structure? lets say
-instruments
- general_pricing_models
- options
- specific pricing models for options
..
..
..

Are you referring to the models contained in option_models.rs ?

The instruments crate will just be reserved for instrument definitions and impls.
Models go in the models crates (the ones contained in instruments/options/option_models.rs will be moved there shortly).

Ok, I understand thanks for the information! I will remain subscribed till then :)


Send image to telegram group

Discussed in https://github.com/photobooth-app/photobooth-app/discussions/275

Originally posted by baumrasen May 26, 2024
Currently I use in the printing configuration a batchscript, that sends the image to a telegram group. This is working fine so far.
The popup message "Image printed successfull" (or something like that) after successfull send the image is not so nice for that. May be it is possible to change the text for successfull "printing" so it could be used more flexible?
Is it possible, to print an image or an collage automatically? In this way all images/collages could be send to the telegram group.

continue in #288


Nanopolish call-methylation Processing Speed

Hello, I'm using nanopolish call-methylation on an Ontario long read data with an average read depth of 35. I'm using 40 processors with 4 GB of memory per processor. I'm using the standard parameters (-t, -r, -b, -g). The job has been running for 12 days, and it has only completed processing chromosomes 1-4 and 10-22 so far. I'm wondering if there are ways to speed up the process, or if this long processing time is expected for this tool?

Likely to be the fast5 IO  bottleneck
You may try https://github.com/hasindu2008/f5c/ with the --iop option to spawn parallel processes for IO. F5c should give same output as nanopolish.
To go even more faster, the best solution is to convert your fast5 to blow5 using slow5tools and then run nanopolish or f5c on it. Instructions are at https://hasindu2008.github.io/slow5tools/workflows.html
.

Thank you for the quick and helpful response. I will try your suggestions. I am wondering if it is possible to use intervals (one Chr per job) when running the Nanopolish call-methylation and the calculate_methylation_frequency.py, and at the end concatenate the methylation_frequency.tsv files from each chromosome into a single file?


[arm-botservice] When to support AzureBot resource rather than Bot Channels Registration?

Is your feature request related to a problem? Please describe.
The Bot Channels Registration will be deprecated on September 1, 2021,

But currently the arm-botservice api botclient.bots.create( ) can only support creating bot channels registration, when will support to create Azure Bot resource?
Describe the solution you'd like
Provide api to create Azure Bot resource.
Additional context
It is an urgent issue!

@YitongFeng-git which package version are you using ? Based on the description, bots.create https://github.com/Azure/azure-rest-api-specs/blob/master/specification/botservice/resource-manager/Microsoft.BotService/preview/2021-05-01-preview/botservice.json#L35  should be the method to create a Bot Service and channels.create https://github.com/Azure/azure-rest-api-specs/blob/master/specification/botservice/resource-manager/Microsoft.BotService/preview/2021-05-01-preview/botservice.json#L309 should be the method to create a Channel Registration for an existing Bot Service.

@qiaozha I am using the arm-botservice sdk: https://github.com/Azure/azure-sdk-for-js/blob/cfa456f927ba1f23a5182b9308c386a2aece7a59/sdk/botservice/arm-botservice/src/operations/bots.ts#L37
I tried the latest sdk version, it still can only create bot channels registration, will this sdk support Azure Bot resource ?

@YitongFeng-git
We can create it successfully by below code ,could you test it again?

Hi, @colawwj , I tried again, it created an "Bot Channels Registration", rather than "Azure Bot"

Here is my code, the difference is just less properties:

I tried the version "@azure/arm-botservice" 2.0.0 and the latest released version "@azure/arm-botservice" 2.1.0, both created "Bot channels registration" rather than "Azure Bot".
@colawwj Did you check the result on Azure portal, is it an Azure Bot?

interesting. The resource type in properties show it's indeed Microsoft.BotService/botServices. @YitongFeng-git  Could you try set the kind value azurebot ?

@qiaozha I can not, for in the sdk, the kind can only be the following values:

@qiaozha let me first introduce the background. From Azure Portal, we find that "Bot Channels Registration will be deprecated on Sept 1st, 2021" and the official recommendation is to use Azure Bot instead. Since our product uses Azure management SDK to provision Azure resource, so we want to know which SDK package and API is provided to create an Azure Bot resource?
@YitongFeng-git has tried the latest "@azure/arm-botservice" v2.1.0 but the created resource is still deprecated Bot Channel Registration instead of Azure Bot. Is that the correct sdk package for us to use?

@eriolchan just forward you an email. We should get service team's confirm to release a new SDK to resolve this issue but didn't have any luck yet.

@YitongFeng-git @eriolchan The latest @azure/arm-botservice<EMAIL_ADDRESS>has been released to resolve this issue. Could you have a try and let us know if you have any more questions? Thanks

@qiaozha thanks! How to use the sdk to deploy an Azure Bot?

@YitongFeng-git  if by deploy you mean publish to Azure, based on their rest api specs https://github.com/Azure/azure-rest-api-specs/blob/master/specification/botservice/resource-manager/Microsoft.BotService/stable/2021-03-01/botservice.json, I don't think this is support with rest api. If you've ever seen such kind of usage in other language SDK or rest api, show me the link, I am happy to find out how.
Still, it needs service team's confirmation.

Hi, @qiaozha You said<EMAIL_ADDRESS>resolved the issue, I upgraded the package and tried, but it still create an Bot Channels Registration, how to create an Azure Bot? How to modify my code? Any documents?

The following is Azure Bot resource, my question is how to create an Azure Bot resource by @azure/arm-botservice sdk?

Did you change the kind value as azurebot ?

Hi, @qiaozha thanks for the updates, but I found if I created an Azure Bot on the portal manually, it will create two resources, it will create a key vault with the Azure Bot, but if using sdk there is no keyvault, currently I haven't know whether this different behavior matters:

This is because you have choose to create a new Microsoft App ID in the portal and the client secret for that app registration is stored in the created keyvault. SDK doesn't have such kind of automatic logic.

Hi, @qiaozha to create a bot with key vault, it should contains the following properties:
"openWithHint": "bfcomposer://",
"appPasswordHint": "/subscriptions/1756abc0-3554-4341-8d6a-46674962ea19/resourceGroups/yifetest726-rg/providers/Microsoft.KeyVault/vaults/bot-secrets-xgwbfu/secrets/bot-yifeABot-pwd-y3qv35",

But currently your sdk doesn't support.

@YitongFeng-git Since this is no longer a SDK related issue, could you ask your question in https://docs.microsoft.com/en-us/answers/topics/azure-bot-service.html to get a better support from the service team directly ? Thanks


feat: push unpushed changes to origin

I'm not entirely sure this is the solution - the problem is that maybe the person isn't ready to push to origin because of known conflicts etc. The only way to be sure is to basically push this also to an autosave branch IMO.

I see. I thought that there is only one branch moving forward and did not know that there might be possible conflicts.
The autosave branch is already pushing un-pushed changes to its own remote since it is branching from the top. Isn't this enough then?

ah wait - you're right! Of course...
there is this related issue: https://github.com/SwissDataScienceCenter/renku-notebooks/issues/138
The "isolation" branches are there for renku executions that run in parallel. Currently we are not doing anything to save those.

sorry this isn't quite done. In the case where I commit some changes and my notebook server gets terminated, the unpushed changes don't go to the autosave branch because of https://github.com/SwissDataScienceCenter/renku-notebooks/blob/597b3ba19cf6fe6280a2a681d46e589d3b13a89a/helm-chart/renku-notebooks/templates/configmap.yaml#L43-L45
I think we should change this behavior to check if the current HEAD is different than the value of CI_GIT_COMMIT_SHA environment variable. If not, then we exit, if yes then we push to the autosave branch.


Stuck on "Loading..."

I just tried the latest commit and the Game and Admin pages both get stuck on "Loading..."

Check out this message thread.
https://github.com/joshzcold/Cold-Family-Feud/issues/24
Basically branch room_codes should have enough functionality to use.
Ill try branch master and check the problem. Might be worth it to try a tagged release if I broke master.

Seems to be working for me

Got a list of steps to take to make the game data invalid?
Server output?

I think it might be a dependency issue with npm... Clean checkout into a new directory fixed it. After npm install react-i18next it builds and runs normally. In my old working directory I get
$ npm run start

><EMAIL_ADDRESS>start
> NODE_ENV=production node server.js

info  - Using webpack 5. Reason: no next.config.js https://nextjs.org/docs/messages/webpack5
> Ready on http://localhost:3000

where that line about webpack is the only one that differs from the output of the working version.
No further steps needed, game is broken from the get go.
I assume that I'm missing some Node knowledge here, perhaps you could include instructions for how to get the dependencies sorted out in the readme?

react-i18next is located in package.json
    "react-dom": "^17.0.1",
    "react-i18next": "^11.8.13",
    "react-router-dom": "^5.2.0",

Any time I add a new dependency you gotta run npm install to get it.
im guess you got "loading..." because the server couldn't import i18next and just ran into an error.

Im thinking there could be a better way to package up the app. Might be a good reason to package it up in docker or electron.

Just to clarify, I had installed react-i18next in my previous working directory (where the problem occurred) as well, my guess is something failed to get updated or some leftover config file screwed everything up. I was just wondering if there's a better way than starting over with a clean working directory...

removing node_modules and doing npm install should be enough.

Im going to go ahead and close this since its not a code issue.
I added the "Update via git" helper snippet in the readme.


dependencies task support for configuration cache

Expected Behavior
Running ./gradlew :subproject:dependencies && ./gradlew :subproject:dependencies should succeed even if configuration caching is enabled
Current Behavior
Running ./gradlew :subproject:dependencies && ./gradlew :subproject:dependencies fails because the set of dependencies has been discarded
frameworks/support$ ./gradlew viewpager2:viewpager2:dependencies
Configuration cache is an incubating feature.
Reusing configuration cache.
> Task :viewpager2:viewpager2:dependencies FAILED

1 problem was found reusing the configuration cache.
- Task `:viewpager2:viewpager2:dependencies` of type `org.gradle.api.tasks.diagnostics.DependencyReportTask`: cannot deserialize object of type 'org.gradle.api.artifacts.Configuration' as these are not supported with the configuration cache.
  See https://docs.gradle.org/7.2/userguide/configuration_cache.html#config_cache:requirements:disallowed_types

See the complete report at file:///Users/jellefresen/workspace/android/androidx-compose/frameworks/support/build/reports/configuration-cache/9y7v26ujv7wt5ov514l2f5976/4c6ykuuvrws934qp4ff0uw25c/configuration-cache-report.html

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':viewpager2:viewpager2:dependencies'.
> The value of this property has been discarded during serialization.

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

* Get more help at https://help.gradle.org

BUILD FAILED in 892ms
1 actionable task: 1 executed
Configuration cache entry reused with 1 problem.

Thanks @mathjeff
Closing as a duplicate of https://github.com/gradle/gradle/issues/17470


add total SU and CU amount to explorer

use new CU and SU formulas and show total grid CU and SU amounts

These number doesn't make sense when applied on a total number of resource units


Name lookup for names under auction

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

What is the current behavior?
currently the endpoint /middleware/names/NAME will return  a view of names that have been claimed, but with the auction mechanism,  the presence of a claim transaction does not guarantee that the name has been claimed

What is the expected behavior?
not sure, either not display the records under auction or add a field to identify the names currently under auction

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. forum, telegram, etc)

I need to rework the name auction part, to take account of the new behaviour. In general we will need

a parameter LIMA_FORK_HEIGHT which maybe defaults to mainnet's Lima height?
and then old behaviour under this height, and new behaviour over it.

I will do this tomorrow :)


Confusion around Counter

Hey there, currently I have the following module,
defmodule MyAppWeb.SafeResolverMiddleware do
  # ...
  @impl Absinthe.Middleware
  def call(resolution, resolver) do
    op = Enum.find(resolution.path, &current_operation?/1)

    span_nometa(
      [:safe_resolver_middleware],
      %{operation_name: operation_name(op), operation_type: operation_type(op)},
      fn -> Resolution.call(resolution, resolver) end
    )
  rescue
    exception ->
      # ....
  end

  # ... in another module
  def span_nometa(event, start_and_stop_metadata, fun) do
    # more and less what we have today
    :telemetry.span([:myapp] ++ event, start_and_stop_metadata, fn ->
      {fun.(), start_and_stop_metadata}
    end)
  end
end

Which I would like to have a counter around the exception,
counter(
  "myapp.safe_resolver_middleware.exception.duration",
  tags: [:operation_name, :operation_type]
)

Readind the following, https://github.com/beam-telemetry/telemetry_metrics/blob/ab8616480cc78bd2377c390f07261da6f37a7401/lib/telemetry_metrics.ex#L15-L18

You could define a counter metric, which counts how many HTTP requests were
completed:
Telemetry.Metrics.counter("http.request.stop.duration")

The metrics mention that it is counting how many, but it is using duration
measurement, which is confusing to me. Counting "duration" sounds odd.
I tried to change the measurement to .count, but that doesn't exist in
the measurement of the metrics so that it would fail.
measurement = %{monotonic_time: -576460706732624847, duration: 4314}
metadata = %{reason: %RuntimeError{message: "hello, world"}, stacktrace: [], kind: :error, operation_name: "listUsers", operation_type: :query, telemetry_span_context: #Reference<0.3696372234.2707161091.20455>}

I understand that I can avoid using the convention from https://github.com/beam-telemetry/telemetry_metrics/blob/ab8616480cc78bd2377c390f07261da6f37a7401/lib/telemetry_metrics.ex#L47-L48
and I could have more control over the metrics.
Also, I could pass :count as a measurement by using :telemetry.execute/3 but
:telemetry.span/3 does not allow me to pass the measurement, and it is probably
more common to use :telemetry.span/3 than :telemetry.execute/3 I found.
So, I am confused about how to focus on just counting +1.

Because measurements inside a metric may be missing, you always count a measurement. So the docs are correct, you are counting how many metrics are emitted with the duration key.

you are counting how many metrics are emitted with the duration key.

Ooooh, I get it. That would imply that my intent is counter("myapp.safe_resolver_middleware.exception") then?
Except that, I felt discouraged by the following line https://github.com/beam-telemetry/telemetry_metrics/blob/ab8616480cc78bd2377c390f07261da6f37a7401/lib/telemetry_metrics.ex#L556
Since it will require to pass event_name, which is not a big deal, still.
My biggest concern is that if I just do that.
counter(
  "myapp.safe_resolver_middleware.exception",
  event_name: "myapp.safe_resolver_middleware.exception"
)

Then it will skip the metric, as follows:
[Telemetry.Metrics.ConsoleReporter] Got new event!
Event name: myapp.safe_resolver_middleware.exception
All measurements: %{monotonic_time: -576460704105113741, duration: 5968}
All metadata: %{reason: %RuntimeError{message: "..."}, stacktrace: [], kind: :error, operation_name: "getAuctions", operation_type: :query, telemetry_span_context: #Reference<>}

Metric measurement: :exception (counter)
Measurement value missing (metric skipped)

Indeed, there is no exception measurement; I am not trying to count the measurement inside the metric. I am just counting the metrics.
And I kept trying to fix it until I decided I may be doing something wrong.


Remove deprecated message above recommended RPC method

Small 1-liner to remove the deprecated header, must be leftover from a while ago I imagine.

This is important because of recent developments where we want to push devs to use that the async function in order to avoid some of the UX throes of shard congestion.
https://twitter.com/BowenWang18/status/1765956174292394228

hi @mikedotexe
I'm happy to update, but the deprecated tag was added by @telezhnaya for the 1.37 release. Here's the original PR: https://github.com/near/docs/pull/1564
I'd suggest checking with Olga or the nearcore team about it. (tagging @bowenwang1996 just in case he can provide some additional details)

It's on me, I'll review and update this page soon.
It's not a leftover, we do have a new method send_tx.
But I need to think how it's better to promote this new method


Add unit test for setting dropdown value to missing option

https://codedotorg.atlassian.net/browse/STAR-317
Add unit test from PR https://github.com/code-dot-org/blockly/pull/172#issuecomment-484218133.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e4e96c4). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #179   +/-   ##
========================================
  Coverage          ?   59.3%
========================================
  Files             ?      95
  Lines             ?   11516
  Branches          ?       0
========================================
  Hits              ?    6830
  Misses            ?    4686
  Partials          ?       0

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 e4e96c4...4edf296. Read the comment docs.


Fix/newlines in cells

Issue: Newlines are not preserved within a cell's content.
Cause: JSON.stringify escapes \n and you end up \\n instead.
Fix: Replace occurrences of \\n with \n in the stringifiedElement (before the row delimiter).
Includes appropriate tests

Coverage remained the same at 100.0% when pulling c2b0f81a49b591f2201bc693ce43ad1f04607472 on dbuentello:fix/newlines-in-cells into dbdc877bb22216c6a8db05b048760abbc7c35eff on zemirco:master.

resolves #91

Excellent!

It opens correctly in OpenOffice, but in Excel it translates new line as new row. It breaks all displayed data (can't correctly figure out that newline char is within " " and shouldn't create new row). I reverted to v3.4.1 to solve that. Don't know why it was changed. You can't display multi line text within cell (as far I know)

@grimor http://superuser.com/a/73851/209612 are you sure it's not an issue with your version of Excel?

Ok, even if it's possible to create new lines, it doesn't get imported correctly for me (Excel 2010). It converts new lines intro new row (as CSV contains each row in new line). It doesn't parse that it's between double quotes/

@dbuentello what version of Excel do you have?

Excel 2015 for Mac, also tested on Windows with Excel 2016

@grimor can you run code below and upload csv
var json2csv = require('./lib/json2csv');
var fs = require('fs');

var data = [{
  col1: 'This doesnt have newline',
  col2: 'Line1\nLine2',
  col3: 'This doesnt have \nnewline'
},
{
  col1: 'This doesnt have newline',
  col2: 'Line1\nLine2\n\nline 3',
  col3: 'This doesnt have \nnewline'
}]
// console.log(data.col2)
json2csv({ data: data }, function(err, csv) {
  if (err) {
    console.log(err);
  }
  fs.writeFile('file.csv', csv, function(err) {
    if (err) {
      throw err;
    }
    console.log('file saved');
  });
});

Hey, I'm using Excel 2010 and have the same issue. The only solution was to downgrade to 3.4.2
@dbuentello
The result of your script is
"col1","col2","col3"
"This doesnt have newline","Line1
Line2","This doesnt have newline"
"This doesnt have newline","Line1
Line2

line 3","This doesnt have newline"

I tried to generate a multi-line CSV in Excel and I've found out, that Excel uses different characters to determine normal (row) line break and a multi-line cell line break.
Line Breaks in Cell are only determined if they are \n only and all other line breaks are \r or \r\n

Good catch! So the split needs to be more specific it seems.

I've reverted this PR in 3.5.1


Tidy up CalcThermalLoads

The function CalcThermalLoads has some code smells... These are some things that should be fixed:

the name should comply to PEP8 (suggestion: calc_thermal_loads)
the parameter list needs to be shortend -strategies from Martin Fowlers Refactoring:

Replace Parameter with Method (like the InputLocator)
Introduce Parameter Object (create a class to encapsulate the data needed)

keep names for parameters the same throughout the project (why do we call the RadiationLoad Solar, Radiation_Shading2 etc.??)
move to the demand.py script

Progress has been made here on multiple fronts with multiple issues. I think the M1.8 Milestone is appropriate for having this done and over with. The tds variable implements the Introduce Parameter Object refactoring - as does the bpr instance of BuildingPropertiesRow. gv should be expanded on and renamed to context but that will be a separate issue.

@daren-thomas  is this still an active issue?

@JIMENOFONSECA technically it is not finished yet, but I do not want to make changes to demand calculation because @gabriel-happle is making changes. We need to get @gabriel-happle 's changes into master ASAP I think. I have moved this to the next milestone for now.

@daren-thomas is this still an active issue?

@JIMENOFONSECA no. it has been resolved with gabriel's branch.


Ability to change pdf highlights color from the right-click menu.

Describe your request
Is it possible to change the highlight color from the right-click menu?

Will be available in the next release.
https://github.com/RyotaUshio/obsidian-pdf-plus/assets/72342591/4d9e90f0-3877-49d5-b80e-ee41911028fb

Released as 0.37.0-beta.

Thank you soo much for this. Would it be possible to do the same for highlights written natively in a pdf editor?

That's also possible. I will add it in a future release!

For performance reasons, showing context menu items for changing highlight colors is a bit difficult. So I added a dropdown menu in the annotation editing modal instead. It's now available as 0.37.0-beta8.

https://github.com/RyotaUshio/obsidian-pdf-plus/assets/72342591/f259dd31-d567-489a-a60a-0e8fc19ad3a9


Communication channels

Operators need to figure out how to communicate with users and other interested folks. We should add text about this.

We're not really using these issues to track stuff any more.


Support ** completion for fc

This adds support for quick selecting, editing, and re-running history
commands via fc:
$ fc **<TAB>
fc>
 1000/1000
  3645  git st
  3644  vim .
  3643  ls -l
  ...

Upon selecting, e.g., "3644 vim.", the command becomes:
$ fc 3644

Pressing  then allows user's $EDITOR being open to edit that
history command, and re-run the editted command.

Thanks for your interest in this project, but we decided not to add more custom completion functions and features to this repository in order to avoid extra maintenance burden. You might want to define them in your configuration file or create a separate third-party project for serving the functions.


Export-Excel: Spaces not allowed in TableName

Consider the following code:
$ExcelFile = $env:TEMP + '\test.xlsx'
Remove-Item $ExcelFile -EA Ignore

$ExcelParams = @{
    Path          = $ExcelFile
    AutoSize      = $true
    FreezeTopRow  = $true
    TableName     = "Tablename with spaces"
    WorkSheetname = "Test sheet"
}
Get-Process | Export-Excel @ExcelParams

This outputs the following errors:
Exception calling "Add" with "2" argument(s): "Tablename has spaces"
At C:\Program Files\WindowsPowerShell\Modules\ImportExcel\Export-Excel.ps1:231 char:13
+             $tbl = $ws.Tables.Add($targetRange, $TableName)
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : ArgumentException

The property 'TableStyle' cannot be found on this object. Verify that the property exists and can be set.
At C:\Program Files\WindowsPowerShell\Modules\ImportExcel\Export-Excel.ps1:233 char:13
+             $tbl.TableStyle=$TableStyle
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : PropertyNotFound

When using the previous EPPLus.dll this wasn't a problem. But recently I updated the EPPlus.dll with the new one from the master branch and this problem pops up. The spaces in TableName is not a problem in the PowerShell code but more in the EPPlus.dll library I guess.
Thank you for checking this.

This is an exception thrown by the EPPLus library. see line 120 here.:
    private void ValidateTableName(string Name)
    {
        if (string.IsNullOrEmpty(Name))
        {
            throw new ArgumentException("Tablename is null or empty");
        }

        char firstLetterOfName = Name[0];
        if (Char.IsLetter(firstLetterOfName) == false && firstLetterOfName != '_' && firstLetterOfName != '\\')
        {
            throw new ArgumentException("Tablename start with invalid character");
        }

        if (Name.Contains(" "))
        {
            throw new ArgumentException("Tablename has spaces");
        }

    }

The strange thing is with the previous dll they didn't throw this error. Opened a related issue on their GitHub to ask for more information about this change.

I think handling and throwing an error is best. Changing the name of the table from what the user expected will be confusing.

Probably something like
if($TableName -and $TableName.Contains(" ")) {
    throw "TableName [$($TableName)] is not valid, spaces not allowed."
}

Hmmmm... I think it would be better to put that in the parameter validation part. Check my PR, it should solve these issues. I've tried to mimic the original error checking as much as possible.

Fixed with PR #165


S2 | Set up a Mongo Hosting

Data Store
Story

As a developer
I need a scalable datastore for storing users and spots

Acceptance Criteria

Create a "scavanger-hunt" gmail account (do not embed the password anywhere in code!)
Follow http://learnwebtutorials.com/mongodb-tutorial-getting-started-mongolab with that email account (free tier)

The tutorial has you test your new db from the terminal. Please do those steps, and take a screenshot of it, and paste it here

Paste in a ticket comment listing an object from your terminal
Email only the professor the credentials for the gmail account, and the mongo lab credentials at this time.

@CalebSarikey

@esayers85

Gmail and Mongo credentials have been emailed to professor.
Attached are screenshots showing terminal sessions of the following (one from Caleb’s machine and one from Eric’s machine):
-connect to MongoDB scavenger-hunt database

-display the db name using command “db”

-display a list of collections within the db using command “show collections”

-display a list of users using command “show users”

-display documents within the ‘people’ collection using command “db.people.find()”

Nice work


turn_animation_into_updater(anim) calls anim.update_config(**kwargs) which was removed

Description of bug / unexpected behavior

Calling turn_animation_into_updater(some_animation) which is in "mobject/mobject_update_utils.py" will give error because it is calling update_config() method from Animation class but it was removed in previous release
How to reproduce the issue

triangle = Triangle()
anim = Write(triangle, run_time=1)
self.add(triangle)
turn_animation_into_updater(anim)

Logs
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.9/site-packages/manim/__main__.py", line 95, in main
    scene.render()
  File "/home/user/.local/lib/python3.9/site-packages/manim/scene/scene.py", line 165, in render
    self.construct()
  File "scene.py", line 7, in construct
    turn_animation_into_updater(anim)
  File "/home/user/.local/lib/python3.9/site-packages/manim/mobject/mobject_update_utils.py", line 86, in turn_animation_into_updater
    animation.update_config(**kwargs)
AttributeError: 'Write' object has no attribute 'update_config'

Thanks for reporting! This is a consequence of the removal of CONFIG dicts. Were there any other update_config calls, @cobordism @leotrs -- and if so, can you remember what you did with them?

Thanks for reporting! This is a consequence of the removal of CONFIG dicts. Were there any other update_config calls, @cobordism @leotrs -- and if so, can you remember what you did with them?

The method was actually removed in #881. According to this comment, it was only used in one place, but I guess it turns out it wasn't.

The method was actually removed in #881. According to this comment, it was only used in one place, but I guess it turns out it wasn't.

site-packages/manim » grep -r update_config *

animation/transform.py:    def update_config(self, **kwargs: typing.Dict[str, typing.Any]) -> None:

animation/transform.py:        Animation.update_config(self, **kwargs)
grep: animation/__pycache__/transform.cpython-39.pyc: binary file matches

mobject/mobject_update_utils.py:    animation.update_config(**kwargs)
grep: mobject/__pycache__/mobject_update_utils.cpython-39.pyc: binary file matches

There's something in animation/transform.py you might want to look

site-packages/manim » grep -r update_config *

animation/transform.py:    def update_config(self, **kwargs: typing.Dict[str, typing.Any]) -> None:

animation/transform.py:        Animation.update_config(self, **kwargs)
grep: animation/__pycache__/transform.cpython-39.pyc: binary file matches

mobject/mobject_update_utils.py:    animation.update_config(**kwargs)
grep: mobject/__pycache__/mobject_update_utils.cpython-39.pyc: binary file matches

There's something in animation/transform.py you might want to look

Indeed; my bad. In that case, sorry for the wrong ping. Nevertheless, replacing the calls to update_config by something like
for attr, value in kwargs.items():
    setattr(animation, attr, value)

as suggested by @friedkeenan in the comment over at #881 should be a rather straightforward solution for this. We just might need to pull the extra code for "path_arc" from transform.py to animation.py, with an isinstance check or so...

Indeed; my bad. In that case, sorry for the wrong ping. Nevertheless, replacing the calls to update_config by something like
for attr, value in kwargs.items():
    setattr(animation, attr, value)

as suggested by @friedkeenan in the comment over at #881 should be a rather straightforward solution for this. We just might need to pull the extra code for "path_arc" from transform.py to animation.py, with an isinstance check or so...

This bug was fixed in #1241


add basic support for Ubuntu 22

This adds basic support for Ubuntu 22. I have not checked if there are newer (better) supported ciphers. I assume, the slightly older ones are still ok for a first update. Currently this control fails on Ubuntu 22 after hardening with our Ansible Collection is applied.
Signed-off-by: Martin Schurz<EMAIL_ADDRESS>

I just checked the differences for defaults MACs, Kex and Ciphers between Ubuntu 18, 20 and 22.
We should probably add some new variables for the different ssh-versions, add the defaults and then substracts known weak ciphers (if needed). @schurzi what do you think?
ssh-versions:
18: 7.6
20: 8.2
22: 8.9
22
umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1

20
umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1

18
umac-64-etm@openssh.com,umac-128-etm@openssh.com,hmac-sha2-256-etm@openssh.com,hmac-sha2-512-etm@openssh.com,hmac-sha1-etm@openssh.com,umac-64@openssh.com,umac-128@openssh.com,hmac-sha2-256,hmac-sha2-512,hmac-sha1

22
curve25519-sha256,curve25519-sha256@libssh.org,ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521,sntrup761x25519-sha512@openssh.com,diffie-hellman-group-exchange-sha256,diffie-hellman-group16-sha512,diffie-hellman-group18-sha512,diffie-hellman-group14-sha256

20
curve25519-sha256,curve25519-sha256@libssh.org,ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521,diffie-hellman-group-exchange-sha256,diffie-hellman-group16-sha512,diffie-hellman-group18-sha512,diffie-hellman-group14-sha256

18
curve25519-sha256,curve25519-sha256@libssh.org,ecdh-sha2-nistp256,ecdh-sha2-nistp384,ecdh-sha2-nistp521,diffie-hellman-group-exchange-sha256,diffie-hellman-group14-sha1

22
chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>
20
chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>
18
chacha20-poly1305@openssh.com,aes128-ctr,aes192-ctr,aes256-ctr,aes128-gcm<EMAIL_ADDRESS>

Thanks for compiling the lists @rndmh3ro. I have checked them with our Ansible Collection, a Test VM and at CIS Benchmark. If I account for insecure settings, almost all lists are identical.
We currently do our checks based on some specific key versions of OpenSSH, every time there is a substantial new algorithm we want to support we create a new variable and extend our logic in the library. Currently I do not see the need to introduce a new variable for 8.9, the settings from 8.5 still seem applicable.
Regarding your thoughts, I think I am in favor of rewriting our checks to a version based approach. I also think I want to reverse the function logic. But I kind of want to handle this in a separate Issue/PR.

opened new issue #205 for that


v1 Refactor

Codecov Report

Merging #67 (32f48f5) into main (255b975) will increase coverage by 7.0%.
The diff coverage is 99.7%.

@@           Coverage Diff           @@
##            main     #67     +/-   ##
=======================================
+ Coverage   92.8%   99.8%   +7.0%
=======================================
  Files          9      11      +2
  Lines        362     652    +290
=======================================
+ Hits         336     651    +315
+ Misses        26       1     -25

Impacted Files
Coverage Δ

reprexlite/reprexes.py
99.5% <99.5%> (ø)

reprexlite/__init__.py
100.0% <100.0%> (+28.5%)
:arrow_up:

reprexlite/cli.py
100.0% <100.0%> (+10.0%)
:arrow_up:

reprexlite/config.py
100.0% <100.0%> (ø)

reprexlite/exceptions.py
100.0% <100.0%> (ø)

reprexlite/formatting.py
100.0% <100.0%> (+1.8%)
:arrow_up:

reprexlite/ipython.py
100.0% <100.0%> (ø)

reprexlite/parsing.py
100.0% <100.0%> (ø)

reprexlite/session_info.py
100.0% <100.0%> (+2.6%)
:arrow_up:

... and 1 more


fix(types): remove promise from useMutation

To be fair I don’t think this was ever there.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


sheet_to_json not working

Hello I'm trying to convert a sheet to a json object and have ran into an issue where it just returning an empty array each time.
My sheet looks like this:
{
  A1: { t: 's', v: '{"A2":{"t":"n"' },
  B1: { t: 's', v: '"v":187}' },
  C1: { t: 's', v: 'B2":{"t":"s' },
  D1: { t: 's', v: '"v":"rated"}' },
  E1: { t: 's', v: 'C2":{"t":"n' },
  F1: { t: 's', v: '"v":60}' },
  G1: { t: 's', v: 'D2":{"t":"s' },
  H1: { t: 's', v: '"v":"F"}' },
  I1: { t: 's', v: 'A3":{"t":"n' },
  J1: { t: 's', v: '"v":189}' },
  K1: { t: 's', v: 'B3":{"t":"s' },
  L1: { t: 's', v: '"v":"rated"}' },
  M1: { t: 's', v: 'C3":{"t":"n' },
  N1: { t: 's', v: '"v":90}' },
  O1: { t: 's', v: 'D3":{"t":"s' },
  P1: { t: 's', v: '"v":"M"}' },
  Q1: { t: 's', v: 'A1":{"t":"s' },
  R1: { t: 's', v: '"v":"lineId"}' },
  S1: { t: 's', v: 'B1":{"t":"s' },
  T1: { t: 's', v: '"v":"CogsRating"}' },
  U1: { t: 's', v: 'C1":{"t":"s' },
  V1: { t: 's', v: '"v":"rm"}' },
  W1: { t: 's', v: 'D1":{"t":"s' },
  X1: { t: 's', v: '"v":"gender"}' },
  Y1: { t: 's', v: '"!ref":"A1:D3"}' },
  '!ref': 'A1:Y1'
}

and I'm calling the sheet_to_json function like so:
XLSX.utils.sheet_to_json(workbook.Sheets[workbook.SheetNames[0]]);
From what I can tell everything looks right but for some reason it's only returning an empty array. Any and all help is greatly appreciated!

The worksheet has 1 row, which is why you won't see any data (that row is interpreted as the header row).
To get back data, you can pass header: 1 option https://docs.sheetjs.com/#json .
However, given your worksheet, it looks like you had a JSON string of a SheetJS worksheet and tried to read it (the library interpreted it as a CSV).  Check your code.  If I had to guess, you have a line like
    var wb = XLSX.read(json_string);

Instead you want to parse that string:
    var ws = JSON.parse(json_string);
    var wb = XLSX.utils.book_new();
    XLSX.utils.book_append_sheet(wb, ws, "Sheet1");

Okay that makes sense. What I'm trying to do is convert a sheet object into a buffer to to act as the return for a mock of a unit test is there an easy way to do that?
Here is the code that I had:
const testSheetData = [
        {
            'lineId': 187,
            'CogsRating': "rated",
            'rm': 60,
            'gender': 'F'
        },
        {
            'lineId': 189,
            'CogsRating': "rated",
            'rm': 90,
            'gender': 'M'
        }
    ];
    const testExcelSheet = XLSX.utils.json_to_sheet(testSheetData);
    const sheetBuffer = Buffer.from(JSON.stringify(testExcelSheet));

Thanks again for the help

If you're ultimately using XLSX.read, then generate a valid file:
    const testExcelSheet = XLSX.utils.json_to_sheet(testSheetData);

    // create a workbook
    const wb = XLSX.utils.book_new();
    XLSX.utils.book_append_sheet(wb, testExcelSheet, "Sheet1");

    // generate an XLSX file
    const sheetBuffer = XLSX.write(wb, {bookType: "xlsx", type: "buffer"});

You can use "csv" as bookType if you wish.

Awesome thank you!


Revise documentation for cardano-coin-selection library.

Context
As part of the work to create a separate coin selection and fee balancing library, we'd like the library to be documented in a way that's consistent with good practices used by other popular libraries in the Haskell ecosystem.
Decision
Revise the public documentation for the cardano-coin-selection library.
Acceptance Criteria
Source Code Documentation (Haddock)

[x] Every public module should have a clear summary to state its purpose.
[x] Every exported function should have a clear comment to illustrate its purpose.
[x] Every exported type should have a clear comment to illustrate its purpose.
[x] Every exported constructor should have a clear comment to illustrate its purpose.

Other Documentation

[x] The home page for the cardano-coin-selection library must explain the basic purpose of the library. This page can delegate to source code documentation where appropriate, to avoid duplication.

Development

[x] Remove Cardano-specific blockchain limitations from comments. For example:
https://github.com/input-output-hk/cardano-coin-selection/blob/f62d069062f196d3b251ac6e7a039a0030d929d6/src/library/Cardano/CoinSelection/Migration.hs#L19-L21
[x] State what is meant by relatively-balanced in function idealBatchSize. See this comment. (We eventually solved this by referring to idealBatchSize as an automatic way to generate a suitable batch size for selectCoins. See https://input-output-hk.github.io/cardano-coin-selection/haddock/cardano-coin-selection-1.0.0/Cardano-CoinSelection-Algorithm-Migration.html#v:selectCoins.)
[x] Check for obvious grammar and spelling errors.

QA
Section to be added.

@jonathanknowles  haddock doc (https://input-output-hk.github.io/cardano-coin-selection/haddock/) returns 404 for:
Cardano.CoinSelection.Algorithm
Cardano.CoinSelection.Algorithm.LargestFirst
Cardano.CoinSelection.Algorithm.Migration
Cardano.CoinSelection.Algorithm.Random
Cardano.CoinSelection.Fee

Closing, the issue for haddock documentation -> https://github.com/input-output-hk/cardano-coin-selection/issues/60.


[Question] re: API documentation | retrieving JSON or non-PDF formats

Per the API Fetch DMP documentation, it is possible to access the DMP's JSON metadata or retrieve the PDF.
Retrieving the PDF returns a binary file:
~/Documents/projects/dmp-tool$ http https://dmptool.org/api/v2/plans/83794.pdf Authorization:"Bearer $DMPTOOL_ACCESS_TOKEN"
HTTP/1.1 200 OK
Content-Disposition: inline; filename="Test_for_HEAL.pdf"

==other headers elided==

+-----------------------------------------+
| NOTE: binary data not shown in terminal |
+-----------------------------------------+

Is it possible to retrieve the contents of the DMP in non-PDF format via the API?
The Download function provided via the user interface includes several options, some of which actually return the contents of the complete DMP. (The download-as-JSON option still only provides metadata, however.)

Hi @TSSlade it is not at the moment. We've considered different ways of including it in the JSON output but the RDA common metadata standard for DMPs  doesn't align nicely with our current data model.
What its your use case? Do you need the DMP narrative in a structured format, or just a dump of the text (similar to the text file download in your screenshot)?

@briri - the use case is basically parsing the responses to kick off a workflow on our end. Picture something like

"They listed 'intended repository' as 'unknown'. Let's reach out and offer to talk through their options with them."

A JSON format for the content of the DMP (as opposed to the metadata about the DMP) would be ideal. In a pinch we could make do with the text output (slight hassle) or existing PDF (likely a bigger hassle, depending on how it's getting built behind the scenes), but the effort required to build those parsers might be roughly the same as supporting a native JSON export of the content.
Either way it could be a public resource, it's just a matter of hackiness and integration with the broader effort.

thanks for the explanation. The JSON standard we're using has a location to convey some of the narrative elements (e.g. security and privacy statements) but not for everything and they are hard to accurately map since each DMP template has varying structures.
Would be willing to submit a PR that enables this functionality?
The DB structure to get at this information is a bit complicated:

The DMP Template structure is: template -> phases (typically only one) -> sections -> questions
The DMP Plan structure is: plan -> answers
The plan has a reference to the template plans.template_id
The answers have a reference to their corresponding questions answers.question_id

It would make sense to include the phase/section titles and descriptions to help group the questions/answers appropriately. I think something like:
{
  "dmp": {

    /* Other DMP metadata */

    "dmproadmap_narrative": [
      "phases": [
        {
          "title": "Write plan",
          "sections": [
            {
              "title": "Date Preservation",
              "position": 1,
              "description": "Lorem ipsum",
              "questions": [
                {
                  "question": "What is your favorite color?",
                  "position": 1,
                  "answer": "Green"
                }
              ]
            }
          ]
        }
      ]
   }
}

@briri - in principle, yes, I think we'd be open to working on a PR for that. Let me get it verified with the Powers That Be. We might have to sequence it after the current PR that's in progress, but if you're okay with that, I think we'd be okay with that.

Yes, that would be fine and makes sense. thanks @TSSlade

@TSSlade Is the PR currently in progress and mentioned in the comment above related to the repository selector feature?

@mariapraetzellis - the PR is not currently in progress. My question to @briri was more in the vein of "If I were to find someone who could work on this, would it be accepted? Or would it be misaligned with your interests/priorities/things you're willing to support?"
If the latter, I wouldn't bother trying to identify someone to work on it.
The repo selector feature is wholly separate and remains ongoing.


Postcss v7 support?

Hi,
I've just update some of my project dependencies, and got following error.
(rejection id: 3) Unknown error from PostCSS plugin. Your current PostCSS version is 7.0.5, but postcss-each uses 6.0.23. Perhaps this is the source of the error below.
Is there any plans to support PostCSS v7?
Thanks.

I'm quite busy now with my kid and finishing my degree, so I almost have no time for my open source until the next year. So, if you want to make a PR, it's highly welcome =)


A11Y improvements for DropdownNavbarItemDesktop

Pre-flight checklist

[X] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation
The goal of this PR is to improve the usability of the dropdown buttons, for example, the version dropdown:

Improvements

I used a  button instead of a link with an ARIA Role, a button should be usable with the SPACE and ENTER key, it was only working with the ENTER key.

https://www.w3.org/TR/using-aria/#rule1

If you can use a native HTML element [HTML51] or attribute with the semantics and behavior you require already built in, instead of re-purposing an element and adding an ARIA role, state or property to make it accessible, then do so.

Escape key to close the dropdown

I removed aria-haspopup: true: When using this ARIA attributes, the dropdown should match the menu role

A popup element usually appears as a block of content that is on top of other content. Authors MUST ensure that the role of the element that serves as the container for the popup content is menu, listbox, tree, grid, or dialog, and that the value of aria-haspopup matches the role of the popup container.

See also https://adrianroselli.com/2017/10/dont-use-aria-menu-roles-for-site-nav.html

I made sure the onClick expands and collapses the dropdown (For this issue https://github.com/facebook/docusaurus/issues/8478#issuecomment-1763440131)

Test Plan
On https://deploy-preview-9439--docusaurus-2.netlify.app/
Keyboard:

should open with SPACE or ENTER
should close with ESCAPE

Hover

should work with JS disabled
should work with JS enabled
a click on the trigger should expand / close the dropdown

Test links

Deploy preview: https://deploy-preview-9439--docusaurus-2.netlify.app/
Related issues/PRs
https://github.com/facebook/docusaurus/issues/8478

Hi @Josh-Cena @slorber  Still a draft, what do you think of the changes?
I could also make a smaller PR,  that would still fix the issue (https://github.com/facebook/docusaurus/issues/8478). We could keep the link role=button, I think it would be better to use a button, but it has some styling impact.

Hey
Thanks for your work on this
Now is not a good time for me to review 😅 I just had a baby
We'll review this when possible but can't promise it will be soon


CVE bumps: Update go.mod and dependencies

Bump to golang 1.23 and bump oauth2 dependency.

/lgtm

/approve


Principled approach to handling NaNs

In #110 there was an issue that NaNs are raised.
I think this may be a problem of numerical approximations: sometimes we may have $p(x, y)\approx 0$, so that numerically $\log p(x, y) = -\text{inf}$.
If all $\log p(x, y)$, $\log p(x)$, and $\log p(y)$ evaluate to $-\text{inf}$, then PMI evaluates to NaN.
I asked ChatGPT and it sugggested to use the following construction:
import jax.numpy as jnp

def custom_subtract(a, b, c):
    # Calculate the result for a - (b + c)
    result = a - (b + c)

    # Create a mask for the special case when all inputs are -inf
    mask = jnp.logical_and(jnp.logical_and(a == -jnp.inf, b == -jnp.inf), c == -jnp.inf)

    # Return 0 where the mask is True, and the original result otherwise
    return jnp.where(mask, 0.0, result)

# Test
a = jnp.array(-float('inf'))
b = jnp.array(-float('inf'))
c = jnp.array(-float('inf'))

print(custom_subtract(a, b, c))  # Should print 0.0

I think after a minor bug fix in the PMI calculation function, we didn't experience this issue anymore. Closing as such...


Feature : add tokenization api

What problem you’re solving

Enrich SDK for tokenization apis.

Restructure project and refactor code to more generic, reusable. It also changes how to use SDK: each payment acceptance (tokenization, order, ...) now is a service API and each API (pay, query) is a resource -> client need integrating which service will init, instead ZaloPayClient will provide all providers.
Eg:
zlpClient = createClient();
tokenizationAPI = new TokenizationAPI(zlpClient); # service
const request = new AgreementBindRequest();
const response: AgreementBindResponse = await tokenizationAPI.bind(request); # resource

Auto-generate API request, response based on ZLP Open API specification, not need to manually write for each. Also using class instead interface for request, response model.
(Note: Breaking change using naming conversion of spec - snake_case instead camelCase like order section.).

Add test using nock for http api testing.

Add tslint to check code style and fixed 71 complains.

Testing

6 added test cases for tokenization api.

Todos

[x] Code change
[x] Write UTs
[ ] Update README.md (update later after discuss about changes).

@anhldbk , @leonthaibao PTAL

About the name of the properties, the ZOD APIs are using Camel Case currently. So in the future, we will convert them to snake case or keep using camel case?

@leonthaibao
About the name of the properties, the ZOD APIs are using Camel Case currently. So in the future, we will convert them to snake case or keep using camel case?

@noradomi About the name of the properties, the ZOD APIs are using Camel Case currently. So in the future, we will convert them to snake case or keep using camel case?

We will use naming convention defined in API specification (spec using camelCase -> models are camelCase and vice versa) and it is auto-generated. I think we don't need to convert between them.

@leonthaibao
About the name of the properties, the ZOD APIs are using Camel Case currently. So in the future, we will convert them to snake case or keep using camel case?

@noradomi About the name of the properties, the ZOD APIs are using Camel Case currently. So in the future, we will convert them to snake case or keep using camel case?

We will use naming convention defined in API specification (spec using camelCase -> models are camelCase and vice versa) and it is auto-generated. I thinks we don't need to convert between them.

@noradomi We should improve README.md to have tutorials about using the new APIs

@noradomi We should improve README.md to have tutorials about using the new APIs

@anhldbk  Noted in Todos in PR

LGTM, just a note that we need to update the order APIs to follow this new structure

LGTM, just a note that we need to update the order APIs to follow this new structure

@leonthaibao Can you make a PR for it?
@longxice also follow in PR for disbursement.

@anhldbk , @qtdan Any update? If okie,  I'll update README for new structure.

@qtdan Just updated README.md and add CONTRIBUTING.md
PTAL again. If it's ok, pls merge.

@leonthaibao please check if we need any adjustment for publishing npm package

@qtdan I've checked the GitHub workflows, we don't need any adjustment

@leonthaibao Can you make a PR for it?

Okie, I'll handle this


Constant upload is killing household bandwidth

So, I installed this on friends RaspPi Octopi setup, and yesterday he showed me that his internet upload is almost the same as his download.
Compared to all the devices which have a high download, the Pi specifically had an extremely high upload.
The limit isn't the problem, but it's slowing down his other devices that require upload/download.
Is there any solution to this? Or can you make any recommendation's other than disabling the plugin when not in use?

Thanks @starxraider for submitting this issue.
Unfortunately I don't have a good solution to this problem. One workaround is to lower the resolution of the webcam so that less data needs to be transmitted. I set the resolution of my webcam to 1280x720, and the plugin is uploading at the rate lower than 3Mbps. Even though my cable connection only has 10Mbps uplink rate limit, it's not been a problem for me since I don't have a lot of other apps/devices fighting for upload bandwidth. But in case upload bandwidth became a problem to me, I'd probably just change the resolution to 640x360 and the bandwidth usage will be only 1/4 of what it is today.
Do you know what's the uplink rate limit in your friend's house, and the resolution of his webcam?

I have a 10 mb upload via my home cable company and with 4 printers using this, my upload speed is getting crushed.  I have to down the Pi's when not in use

@LilBub Yes I can see how 4 printers will jam all the uplink bandwidth. Will that be an option to reduce the resolution to 640x360?
There are some options to reduce the bandwidth requirement, but none of them can be done without complicating the system by a lot. So I'm hoping that a lower resolution will be a good-enough workaround...

I'd really hate to lower the resolution.  Perhaps an option to only broadcast when a print is running?

That'll be definitely part of the solution. However, I don't think it's a good idea to not show webcam streaming at all when the printer is not printing, as it'll make it as if the printer is offline.
So I guess the final solution should also stream webcam when it's not printing, but at a lower rate. Now the side effect is big gaps between webcam frames and hence long latency for webcam stream to show up on the dashboard upon page load...

My suggestion would be to allow there to be an option (Yes/No) on whether one wants streaming on all the time or only when a print is being made.  That would give those of us with lower bandwidth upstream to use the plugin and down it when not needed.  Users like options :)

Yes, a button for "Start streaming now, and keep streaming for the next 5 minutes" would be nice. There is already the button for start/cancel print, so communicating with the octopi seems to be already possible :)

Not sure how the remote portion works, but would it be possible to just upload a static image at a long interval and then a simple heartbeat every few minutes?  The upload bandwidth would be reduced dramatically this way.
Of course, switch to live when actually printing.

@GrueMaster Thanks for the suggestion. That sounds like solution that won't cost me an arm and a leg to build. Will work on it when I have a chance.

@starxraider @GrueMaster @hakan42 @LilBub I released version 0.6.1 that will slow down the streaming to ~5% of original bandwidth when not printing. Please upgrade and let me know if it solves or alleviate the problem.

@starxraider @GrueMaster @hakan42 @LilBub I released version 0.7.0 that further slows down the upload bandwidth when the webcam feed is not being watched. Please upgrade.

Thanks...I just installed this last night and suspect I have it...didn't notice the whole house lagging from two printers running this time!

I am glad to hear that 😁
On Sat, Jul 14, 2018, 1:44 PM Paul<EMAIL_ADDRESS>wrote:

Thanks...I just installed this last night and suspect I have it...didn't
notice the whole house lagging from two printers running this time!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kennethjiang/OctoPrint-Anywhere/issues/40#issuecomment-405048755,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvmCn8DrrtMpHG9cWVHUnHhSIQHAFm1ks5uGlghgaJpZM4Q4lZZ
.


Generics; ReturnType != ReturnType

TypeScript Version:  Version 2.9.0-dev.20180516

Search Terms: generic function returntype
I couldn't think of a good title for this,
Code
type Delegate<T> = () => T;

function executeGeneric<
    X,
    DelegateT extends Delegate<X>
>(delegate: DelegateT): ReturnType<DelegateT> {
    //Type 'X' is not assignable to type 'ReturnType<DelegateT>'.
    const x: ReturnType<typeof delegate> = delegate();
    //Type 'X' is not assignable to type 'ReturnType<DelegateT>'.
    const y: ReturnType<DelegateT> = delegate();
    //Type 'X' is not assignable to type 'ReturnType<DelegateT>'.
    return delegate();
}

Expected behavior:
Return successfully.
Intuitively, to me,

delegate is of typeDelegateT.
ReturnType<DelegateT> and ReturnType<typeof delegate> should be the same

However,
Actual behavior:
Type 'X' is not assignable to type 'ReturnType<DelegateT>'.
Playground Link: Here
Related Issues:

ReturnType<DelegateT> and ReturnType<typeof delegate> are the same, and both are not assignable from DelegateT, you can not assign a function that returns a string to a string.

Isn't delegate of type DelegateT and delegate() of type ReturnType<DelegateT>?
delegate() is not supposed to be X, it could be X but it could also be something that extends X
But it's saying that delegate() is of type X when it shouldn't be (intuitively, to me, anyway)

ah. sorry about that, did not see the call there.. yes. you are right about that.

I don't know why I didn't think of this shorter example.
function executeGeneric<
    DelegateT extends () => number
>(delegate: DelegateT): ReturnType<DelegateT> {
    //Type 'number' is not assignable to type 'ReturnType<DelegateT>'.
    const x: ReturnType<typeof delegate> = delegate();
    //Type 'number' is not assignable to type 'ReturnType<DelegateT>'.
    const y: ReturnType<DelegateT> = delegate();
    //Type 'number' is not assignable to type 'ReturnType<DelegateT>'.
    return delegate();
}

Playground

This is a suggestion, not a bug.
Direct function calls are evaluated according to their constraints. This is generally desirable:
function invokeAndMultiply<T extends () => number>(a: T, b: T) {
    // OK or no?
    return a() * b();
}

In general you cannot multiply arbitrary types, so this example would be an error if we forced deferral of a()'s return type to ReturnType<T>.
It'd be a pretty substantial complexity trade-off to start deferring the return type evaluation, since anything you do with a generic type constrained to a function type would then need follow-on special handling.

Let's say we defer the evaluation of a()'s return type.
In this example, even if TS hasn't evaluated what ReturnType<T> is,
it should know that ReturnType<T> extends number.
And because of that, it should allow their multiplication, because you can multiply two number types and ReturnType<T> is a subtype of number; the result being just number.

My example is simple. So, deferring the evaluation of the type is also not an issue.
In general, if given this,
function foo<F extends () => T> (f : F) : ReturnType<F> {
    /*
        Current behaviour:
        `value` is of type `T`.

        Proposed behaviour:
        `value` is of type `(T & ReturnType<F>)`,
        where evaluation of `ReturnType<F>` is deferred.
    */
    const value = f();
    //Right now, we can do all things `T` can do because `value` is subtype of `T`.

    /*
        This should now work because `T & ReturnType<F>` extends `ReturnType<F>`.
    */
    return value;
}


Bug with assigning stereochemistry in multiprocessing

Describe the bug
When I assign stereochemistry in the main Python process, or in a single additional process, I get a correct error. However, using 2 or more processes results in another, incorrect result. The same behavior persists with deepcopy of the molecule, but when calling MolFromSmiles(MolToSmiles(mol)), it works properly.
To Reproduce
from copy import deepcopy

from joblib import Parallel, delayed

def calc_num_stereo_centers(mol) -> int:
    from rdkit.Chem.rdMolDescriptors import CalcNumAtomStereoCenters
    from rdkit.Chem.rdmolops import AssignStereochemistry

    mol = deepcopy(mol)
    mol = deepcopy(mol)
    AssignStereochemistry(mol)
    return CalcNumAtomStereoCenters(mol)

if __name__ == '__main__':
    from rdkit.Chem import MolFromSmiles
    from rdkit.Chem.rdMolDescriptors import CalcNumAtomStereoCenters
    from rdkit.Chem.rdmolops import AssignStereochemistry

    smi = "OCC1OCOC2COCOC12"
    mol = MolFromSmiles(smi)
    AssignStereochemistry(mol)
    print("Regular:", CalcNumAtomStereoCenters(mol))

    parallel_results = Parallel(n_jobs=1)(delayed(calc_num_stereo_centers)(m) for m in [mol])
    print("Parallel 1 process:", parallel_results)

    parallel_results = Parallel(n_jobs=2)(delayed(calc_num_stereo_centers)(m) for m in [mol])
    print("Parallel 2 processes:", parallel_results)

This results in:
Regular: 3
Parallel 1 process: [3]
Parallel 2 processes: [0]

However, if I use explicit mol -> SMILES -> mol conversion:
def calc_num_stereo_centers(mol) -> int:
    from rdkit.Chem import MolFromSmiles, MolToSmiles
    from rdkit.Chem.rdMolDescriptors import CalcNumAtomStereoCenters
    from rdkit.Chem.rdmolops import AssignStereochemistry

    mol = MolFromSmiles(MolToSmiles(mol))
    mol = deepcopy(mol)
    AssignStereochemistry(mol)
    return CalcNumAtomStereoCenters(mol)

it results in:
Regular: 3
Parallel 1 process: [3]
Parallel 2 processes: [3]

Expected behavior
Correct output when deepcopy is used.
Configuration (please complete the following information):

RDKit version: 2024.3.3
OS: Ubuntu 24.04
Python version (if relevant): 3.9.20
Are you using conda? no
If you are not using conda: how did you install the RDKit? pip install rdkit

This is happening because deepcopy pickles the molecules to send them between the different processes. The RDKit's default pickling behavior does not preserve all computed properties, so your subprocesses are not getting all of the information they need for this code to work properly (but they are getting some information that tells them that properties don't need to be recomputed).
I will look into fixing this, but in the meantime an easy workaround for you is to add:     Chem.SetDefaultPickleProperties(Chem.PropertyPickleOptions.AllProps) inside your main block.


fix fix bug that failed to get schema of tvf when file is empty

Proposed changes
Issue Number: close #xxx
Problem summary
In previous implementation, when querying tvf, FE will get schema from BE.
And BE will try to open the first file to get its schema info, but for orc or parquet format,
if the file is empty, it will return error.
But even for an empty file, we can still get schema info from file's footer.
So we should handle the empty file to get schema info correctly.
Also modify the catalog doc to add some FAQ.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Pass clerkJSVersion in interstitial

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[x] @clerk/nextjs
[x] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[x] @clerk/backend
[x] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description
Review per commit. (Changes for each package)

[ ] npm test runs as expected.
[ ] npm run build runs as expected.

!snapshot

Hey @panteliselef - the snapshot version command generated the following package versions:

Package
Version

@clerk/backend
0.23.1-snapshot.fe3ca7b

@clerk/chrome-extension
0.3.9

@clerk/clerk-js
4.47.0

eslint-config-custom
0.3.0

@clerk/clerk-expo
0.18.0

@clerk/fastify
0.4.8-snapshot.fe3ca7b

gatsby-plugin-clerk
4.3.10-snapshot.fe3ca7b

@clerk/localizations
1.18.0

@clerk/nextjs
4.21.2-snapshot.fe3ca7b

@clerk/clerk-react
4.20.0

@clerk/remix
2.6.8-snapshot.fe3ca7b

@clerk/clerk-sdk-node
4.10.7-snapshot.fe3ca7b

@clerk/shared
0.19.0

@clerk/themes
1.7.5

@clerk/types
3.42.0

Tip: use the snippet copy button below to quickly install the required packages.
# @clerk/backend
npm i<EMAIL_ADDRESS>
# @clerk/chrome-extension
npm i<EMAIL_ADDRESS>
# @clerk/clerk-js
npm i<EMAIL_ADDRESS>
# eslint-config-custom
npm i<EMAIL_ADDRESS>
# @clerk/clerk-expo
npm i<EMAIL_ADDRESS>
# @clerk/fastify
npm i<EMAIL_ADDRESS>
# gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>
# @clerk/localizations
npm i<EMAIL_ADDRESS>
# @clerk/nextjs
npm i<EMAIL_ADDRESS>
# @clerk/clerk-react
npm i<EMAIL_ADDRESS>
# @clerk/remix
npm i<EMAIL_ADDRESS>
# @clerk/clerk-sdk-node
npm i<EMAIL_ADDRESS>
# @clerk/shared
npm i<EMAIL_ADDRESS>
# @clerk/themes
npm i<EMAIL_ADDRESS>
# @clerk/types
npm i<EMAIL_ADDRESS>

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Issue with custom post type

Similar issue to #2. Impacting code written with a custom post type
Warning: Attempt to read property "post_content" on int in /var/www/html/wp-content/plugins/complete-bootstrap-4-shortcodes/bootstrap-shortcodes.php on line 81
Warning: Attempt to read property "post_content" on int in /var/www/html/wp-content/plugins/complete-bootstrap-4-shortcodes/bootstrap-shortcodes.php on line 100
Warning: Attempt to read property "post_content" on int in /var/www/html/wp-content/plugins/complete-bootstrap-4-shortcodes/bootstrap-shortcodes.php on line 117

Adding isset($post->post_content) && to the ifs on these lines sorts it

Thanks for reporting and fixing this issue. New version 4.6.1 includes your change.


Disallow :{schema} outside of table definitions

I've been bitten by this a few times now (see #75 for an example).
The parser should allow all of these (as it currently does):
(deftable persons:{person})
(defschema wrapper wrapped:object{inner})
(defun foo:object{token-row} () {"balance": 5, "name": "stu"})
But disallow these (which are currently permitted):
(defschema wrapper wrapped:{inner})
(defun foo:{token-row} () {"balance": 5, "name": "stu"})
(defun bar:object{token-row} ()
    (let ((stu:{token-row} {"balance": 5, "name": "stu"}))
      stu))

We should consider what the most natural syntax should be. Should x:{foo} imply an object?
A schema type implies an object intuitively, whereas for a table or binding form it is because it operates on that object/row type. Currently users would never want to specify a table type, namely because to do so implies parameterizing a table var, using a schema-type variable (table:{<a>}) which is not exposed to the user type language yet. Binding forms cannot be variables so that's not an issue.
So, the question becomes is it OK that eventually we will have an inconsistent syntax when parameterizing tables. I think so, since this is essentially some sugar to make the 99.9% case easier to read and code.

Come to think of it, there really isn't any reason to not use schemaName as a bareword representing object{schemaName}. The advantage is better legibility of code and user type presentation. The parser would recognize builtin types and then turn anything unrecognized into a user type, to be approved or rejected by the typechecker.
It also approaches a wider universe of user types, in that the curly-brace syntax is really about parameterizing row types. Curly braces thus specifically indicate row types when specificity is needed.

Joel and I were actually just about to recommend the same thing!
The examples above would then look like:
(defschema person name:string balance:integer)

; here, the table is parameterized by a row type
(deftable persons:{person})

(defun foo:person () {"balance": 5, "name": "stu"})
(defun bar:person ()
    (let ((stu:person {"balance": 5, "name": "stu"}))
      stu))

And over time we could potentially migrate defschema to define real record/row types.

One issue is that we are currently using strings as keys for objects, but in row-typed systems, you typically have statically-known labels as keys. Strings can obviously be concatenated, manipulated, etc at runtime. The current situation would add considerable (IMO unnecessary) complexity to an implementation of "row types", if we could even call it that, at that point.
Perhaps we should move objects to use "labels", and come up with a separate mechanism for true finite maps from arbitrary keys to values. We already have multiple places in our symbolic execution where we throw up our hands if object keys aren't literals. This would also clean things up in that regard.

#47 is probably where we can tackle labels etc

One thing to keep in mind about row types is that we will still not be able to type object merge (+) without sophisticated extensions to the type system. See https://stackoverflow.com/a/35989838/494018


View Only mode

Issue submitter TODO list

[X] I've searched for an already existing issues here
[X] I'm running a supported version of the application which is listed here and the feature is not present there

Is your proposal related to a problem?
I need an option to disable the Produce Message/Remove/Recreate/Clear messages option.
Is there any way to keep only read-only mode for view cluster/topics/search messages?
Describe the feature you're interested in
Read Only mode for non authorised users
Describe alternatives you've considered
No response
Version you're running
56fa824 27.06.2023, 16:36:09
Additional context
No response

@mixMugz I used this tool to monitor my kafka which can provides a read only view : https://zoomphant.github.io/docs/manual/99_plugins/kafka/ by creating a readonly user


Change type of iteration.path to path.Path

Fixes https://github.com/almighty/almighty-core/issues/971
Removed ConvertToLtreeFormat and ConvertFromLtreeFormat from iterations package.
Make use of path.Path for iteration.Path
Tests updated.
No change in representation for database layer. It is still a valid ltree sting like 218172e8_a797_49f8_b3f3_b96e5ac6f0f4.3574fb25_45db_4397_87ca_8690529328be
But iteration.Path is a slice of UUIDs(path.Path) now instead of string.

[test]

Codecov Report

Merging #973 into master will decrease coverage by -0.17%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master     #973      +/-   ##
==========================================
- Coverage   70.41%   70.25%   -0.17%
==========================================
  Files          89       89
  Lines        6947     6929      -18
==========================================
- Hits         4892     4868      -24
- Misses       1615     1621       +6
  Partials      440      440

Impacted Files
Coverage Δ

iteration/iteration.go
55.95% <ø> (-9.61%)
:x:

controller/space_iterations.go
64.61% <0%> (+3.74%)
:white_check_mark:

controller/iteration.go
85.18% <100%> (-2.64%)
:x:

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 2ab8ca3...b751e81. Read the comment docs.


Update syntax for custom query_store_mode example

Missing commas between parameters

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


Bump to v4.11.0

I intend to change the particular version of lean-auto that is imported (there are some subtle differences between lean-auto's v4.11.0 tag and the most recent commit in the duper2 branch that Duper uses), but otherwise, happy to merge this update.


Failing to install after v1.0.1

Hi, this package is now failing to install:
$ npm install evp_bytestokey

><EMAIL_ADDRESS>install /private/tmp/test/node_modules/evp_bytestokey
> node-gyp rebuild

gyp: binding.gyp not found (cwd: /private/tmp/test/node_modules/evp_bytestokey) while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/Users/knightle/.config/fnm/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:305:16)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Darwin 16.4.0
gyp ERR! command "/Users/knightle/.config/fnm/bin/node" "/Users/knightle/.config/fnm/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/tmp/test/node_modules/evp_bytestokey
gyp ERR! node -v v6.10.2
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open '/private/tmp/test/package.json'
npm WARN test No description
npm WARN test No repository field.
npm WARN test No README data
npm WARN test No license field.
npm ERR! Darwin 16.4.0
npm ERR! argv "/Users/knightle/.config/fnm/bin/node" "/Users/knightle/.config/fnm/bin/npm" "install" "evp_bytestokey"
npm ERR! node v6.10.2
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the evp_bytestokey package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs evp_bytestokey
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls evp_bytestokey
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /private/tmp/test/npm-debug.log

Installing version 1.0.0 works:
$ npm install<EMAIL_ADDRESS>/private/tmp/test
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
npm WARN enoent ENOENT: no such file or directory, open '/private/tmp/test/package.json'
npm WARN test No description
npm WARN test No repository field.
npm WARN test No README data
npm WARN test No license field.

I am transitively using this package because browserify-cipher depends on it.

Seeing the same - it worked just fine in the morning.

Investigating

Published v1.0.2 with gypfile: false... the gyp file was for testing only.

Looks good here!

Unpublished v1.0.1.

👍 thanks

Builds are now passing. Thanks!

Thanks for the lightening quick fix!

@dcousens Please make sure to push the commit and tag for 1.0.2. Thanks!

@elyscape thanks for reminder, done!

@dcousens thank you very much！


[ChatGPTUnofficialProxyAPI] ChatGPT Web has switched from SSE to WebSockets.

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
.
Describe the Bug
ChatGPT Web has switched from SSE to WebSockets, as a result, the ChatGPTUnofficialProxyAPI is no longer functional. The SSE-related code needs to be updated.

This project is undergoing a major revamp; closing out old issues as part of the prep process.
The chatgpt package is pretty outdated at this point. I recommend that you use the openai package or the openai-fetch package instead.