feedback: Open Chat in a new tab by default

Version
1.92.1
Areas for Improvement

[X] UI/UX
[X] Onboarding
[ ] Docs
[X] Chat
[ ] Commands
[ ] Context
[ ] Response Quality
[ ] Other

What needs to be improved? Please describe how this affects the user experience and include a screenshot.
Currently, I need to manually move to chat window to a new tab manually.
Its a bit annoying, to always have this to do by hand.
Also, many people probably dont know, they can do that at all.
Having the chat window as it is now by default, takes a lot of screen real espace.
Especially on a notebook can this be limiting.
Describe the solution you'd like to see
I like that every new chat windows opens in a new tab by default.
Describe any alternatives that could be considered
No response
Additional context
No response

Hey! Can you give me instructions (screenshots/video) on how to move Cody in VSCode to another window?
I want to move my Cody Chat extension to another monitor while it still has context on my project opened in original VSCode window. As I understood from this, it's possible to achieve it. But I don't know how.
Please help. Thanks!

Could we put this up for discussion at the next triage, please.


remove legacy update marker code

Fix #148
The problem was a reference to old code. Changing the context name should now be possible without an error.
Please review @eScienceCenter/spacialists

Please review again @derntl


projectatomic PPA repository has no Release file!

/kind bug
Description
Cant add the repo projectatomic to install podman on ubuntu18.10
Steps to reproduce the issue:
$ sudo add-apt-repository -y ppa:projectatomic/ppa

Describe the results you received:
E: The repository 'http://ppa.launchpad.net/projectatomic/ppa/ubuntu cosmic Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Distro info
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.10
Release:	18.10
Codename:	cosmic

cosmic is EOL'd already, so I didn't build for it. Might wanna uprade to disco and retry, or wait a while and give OBS repos a try ( not ready yet but will be soon).
See: https://github.com/containers/libpod/issues/1742#issuecomment-555061785

@mido3ds @lsm5 Can we close this issue?

Yes, sure. Sorry didn't have time to reply and close.
On Thu, Dec 19, 2019, 3:36 PM Daniel J Walsh<EMAIL_ADDRESS>wrote:

@mido3ds https://github.com/mido3ds @lsm5 https://github.com/lsm5 Can
we close this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/issues/4520?email_source=notifications&email_token=AEWP6OAYGD32S2XYY6YA53TQZN2FXA5CNFSM4JNXZHS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHJT6ZY#issuecomment-567492455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWP6OE3MMG6GGUEKI7ZUCLQZN2FXANCNFSM4JNXZHSQ
.


serde and network packets

A while back, I tried to write a serde Deserializer and now that I have a lot more experience with Rust, I wanted to talk about a specific kind of use case that I'm not sure there's a good solution for in serde.
I've looked at the JSON and bincode serde implementations and so far what I see is that the JSON implementation is able to pretty easily look at the serialized data to deserialize it appropriately. bincode is binary data so it doesn't have the same flexibility to guess the type from the data itself from what I can see in the documentation, but it does have one notable difference from the format I'm trying to work with: the length of variably sized types like strings and byte buffers appears to be encoded directly before the actual data itself.
Personally, I would love to migrate over to serde because it would be great to be able to interchangeably serialize to JSON and my binary representation for example, but I've bumped into some of the same problems that I did last time and wanted to see if we can work to find a solution now that I can explain it in a little bit more depth.
The problem
My binary format is packet based and has a lot of nested headers. Sometimes these headers encode a length and other times they do not. They all really need to be represented as structs. If there is a length associated with the packet header, it is always at the beginning of the packet header and the payload is at the end. I'm not really finding a way to associate the length as the first field with the size of the byte buffer, string, or other data type that's deserialized as the payload or last field.
My understanding of serde's model
I've been doing a lot of digging into the architecture, and from what I've seen so far, it appears that the Deserialize implementation drives the Deserializer to expect the correct types for deserializers that need this, the Deserializer is responsible for actually reading and parsing the data and passing it to the visitor, and the Visitor is responsible for taking that data presented to the visitor from the Deserializer (whether this is a struct handler or just a simple u32) and generating the final representation expected by the Deserialize method. Because visitors are used across Deserializer implementations, they should not be involved in actually parsing any data. Rather, it is on the Deserializer to handle the parsing and provide data structures that implement traits like SeqAccess to further handle deserialization from the visitors.
Generic deserializer problem
I've tried DeserializeSeed as this appears to be the way to implement stateful deserialization but the problem here is that it's implemented for a generic deserializer. My byte buffer for the entire packet needs to be stored internally in the deserializer and therefore, if I pass a length into DeserializeSeed, I have no way to set that length in my deserializer for the next payload deserialization. I can't access the byte buffer to slice it appropriately, I can't call any methods on it to set the length as they won't be recognized given trait bounds on the type parameter, and this generally doesn't seem to be the use case for DeserializeSeed. I've also considered the deserialize_with field attribute, but this obviously has the same generic deserializer problem. Even if I could provide a simple boolean for each sub-header that's being deserialized, it would be pretty trivial to make that true if the first field was a length and false if not, but because the deserializer is provided at the top level when Deserialize::deserialize is called and doesn't really seem to be accessible at lower levels of struct parsing, I'm not entirely sure how I would go about providing that on a per-struct basis.
What it boils down to
I considered what would be most helpful here and I think the fundamental problem is that serde deserializes all structs using the same method on the deserializer. To internally handle the length currently, I can't keep the length in its original u32 format because there is no way to differentiate between a u32 that has no side effect on the deserializer and a u32 that is a length and does have a side effect on the deserializer. Wrapping the length in a newtype struct also seems challenging because I could potentially special case the newtype struct handler to recognize a type name associated with lengths, but this also seems fraught because it involves string comparison for determining the type which I generally would prefer to avoid and I wouldn't know what the internal type of the length is from the newtype struct handler alone (u16 or u32) as it only accepts a name and visitor. Perhaps this could be worked around by passing a PhantomData marker to the visitor in the deserialize method, but I'm still not sure that this is the recommended way to do so. What are your thoughts? Is there a way to pass additional information to the Deserializer struct to provide more context for the parsing step that I'm missing?

Sorry that no one was able to provide guidance here! If you are still interested in talking about this use case, please take this to one of the resources listed in https://github.com/serde-rs/serde/tree/v1.0.135#getting-help.


Decorator to initialize resources

We need some decorator to initialize resources, for instance create files in a tempdir or populate a database.

On a second thought, this should probably be a decorator of the resource or the Fragile that provides the resource

done


SWARM-1692: upgrade wildfly-nosql + nosql config-api variants and add test case.

Resolves WFNOSQL-21 'IllegalArgumentException: WFLYNOSQL0003: @Named annotations must provide a value' deployment failure and also brings in an OrientDB subsystem configuration improvement.

I'm not sure why Windows failed, retest this please

Probably disk space issues.  S’happens.
-Bob
On Wed, Nov 22, 2017 at 10:17 AM, Scott Marlow<EMAIL_ADDRESS>wrote:

I'm not sure why Windows failed, retest this please
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wildfly-swarm/wildfly-swarm/pull/704#issuecomment-346380762,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA-T9Q1zE7gOO7CoEkVzbVUayGHwo9aks5s5DsbgaJpZM4QmS1Z
.

retest this please


How to be part of any open source like ssoc.

i want to be part of an ooen source program like ssoc, those who have any knowledge about that please guide me:)

Hey thanks for this wonderful query. There any many paid and unpaid open source programme.
Paid:
GsOC,Outreachy, Lfx, Xros, GSOD, Summer of Bitcoin and many more.
Unpaid.
Ssoc, GSSOC, KHARAGPUR WINTER OF CODE, OSS, HACKTOBERFEST and many more.
Now ssoc is over and around NOV SWOC AND GWOC will open
For ssoc and SWOC follow their linkedin page connect with me on linkedin. You can also Check Hack2skill website for SSOC
AND same follows for the GSSOC AND MANY OTHER PROGRAMME.
Let me know if u have any other  doubt.

thnks for this


Loading a local NER model but has no embeddings

Hi! I have a local trained model which only has NER in its pipeline and as soon as I try to add the concise-concepts data it returns
Exception: Choose a model with internal embeddings i.e. md or lg.

How can I train my model to have the necessary embeddings to work out with concise-concepts?

I could be that you don't have embeddings within the spaCy model. Try out concise-concepts==0.3.0 for using custom Gensim embeddings.

https://github.com/Pandora-Intelligence/concise-concepts#use-gensimword2vec-model-from-pre-trained-gensim-or-custom-model-path

@marcossilva I recommend looking at the spaCy guide on embeddings or training . Alternatively, there is Gensim, you could also try to use a FastText  model from Gensim to deal with OOV words for your specific dataset, or even better, just train a custom word2vec model for your data.


Add X-axis and Y-axis labels in all charts in the dashboard.

This will be helpful for users to analyze the data.

Fixed in PR #286


Added check if a connected graph is bipartite or not

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
Check if a connected graph is bipartite or not #2680
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#2680

Duplicate #1510


Number of phases changing when charging stopped and started again

I have a 35A single phase mains.
I supply mains data via API with arguments L1=value, L2=0,L3=0
If I leave out L2 and L3 it will say check data connection.
After powering up and start charging the first time it will say SMART1F.
If smart charging stops because of no power and restarts again it shows SMART3F.

It seems kind of random or based on the current?
How does it determine it?
Many times 6-16A is SMART3F. Amps above this will be shown as SMART1F.
I opt for a device setting where you simply specify 1/2/3 phase operation.

@Imaginous did you try the version 1.80? Think it solves your issue.

I'm using the version of @dingo35, which solves the external P1 step up only occurring when an external package has been received from the API.
I understood this fix is not in V1.8 yet.


Inappropriate behaviour in high speed automatic driving

Hi everyone,
I have a problem at running automatic_control.py.
The first thing to say, I use CARLA version 0.9.7 but not the latest commit. I got the source zip file from release page. Also, have to say that I built it on Windows 10 via VS2019 (made some changes in cmake files to change VS2017 to VS2019 by myself) and running with unreal engine 4.22.
Now let me describe what is the problem.
I need to check autonomous driving vehicles behaviour on speeds more than 50 km/h, then I changed target_speed to 50 in file basic_agent.py and get automatic_control.py running again. I found my vehicle driving like a drunk guy!!

Please If you know why it should happen, help me.
Thanks for any replies

Hi,
Basic agent is pretty... basic. Probably needs a PID re-tuning, since you change its target velocity. Try the Roaming agent or the Behavior agent

Thanks a lot @jackbart94
But there is again the same problem while I made target_speed 50 km/h in local_planner.py and ran automatic_control.py with Roaming agent.
Also, can I get target_speed dynamically from speed limit signs?

Hi,
You still need to change the PID values once you change the desired velocity.
Also yes, you can, just need to get the current affecting speed limit on your ego vehicle. If you don't know how, check the documentation!

@jackbart94
Thanks again for the very fast response. I found some solutions in your documentation.
I'm going to close this issue.
Have a nice time.
Bye


Setup CodeQL

Related: #103
Setup CodeQL analysis using GitHub Actions:
https://github.com/decline-cookies/anvil-csharp-core/security/code-scanning
Consider running dotnet clean and dotnet restore steps like GitHub's autobuild does.

We can't use autobuild because it doesn't pick up on the core level csproj. It only builds the logging project.

See test implementation at
https://github.com/mbaker3/anvil-csharp-core/blob/main/.github/workflows/codeql-analysis.yml
Make sure Logging and core projects are analyzed using both their dot net core and msbuild versions.

Partially completed by #125 but requires additional work outlined in this comment
https://github.com/decline-cookies/anvil-csharp-core/pull/125#pullrequestreview-1144746780


chore(protobuf): Update protobuf definitions via buf

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


HDDS-5056. Avoid false-positive error messages during pipeline creations

JIRA: https://issues.apache.org/jira/browse/HDDS-5056
What changes were proposed in this pull request?
BackgroundPipelineCreator in SCM tries to pipelines with regular intervals. This check is very simple, if new pipeline can be created, it will create them.
However, if the pipelines couldn't be created (which is very likely) an error message is printed out:
scm_1       | 2021-03-31 15:34:57,004 [RatisPipelineUtilsThread - 0] ERROR pipeline.PipelineManagerV2Impl: Failed to create pipeline of type RATIS and factor THREE. Exception: Pipeline creation failed because nodes are engaged in other pipelines and every node can only be engaged in max 1 pipelines. Required 3. Found 0

Today it's printed out on ERROR level, but this message doesn't provide any new information  I think the PipelineManager.createPipeline shouldn't use ERROR level especially as the exception is re-thrown:
    lock.lock();
   try {
     Pipeline pipeline = pipelineFactory.create(type, factor);
     stateManager.addPipeline(pipeline.getProtobufMessage(
         ClientVersions.CURRENT_VERSION));
     recordMetricsForPipeline(pipeline);
     return pipeline;
   } catch (IOException ex) {
     LOG.error("Failed to create pipeline of type {} and factor {}. " +
         "Exception: {}", type, factor, ex.getMessage());
     metrics.incNumPipelineCreationFailed();
     throw ex;
   } finally {
     lock.unlock();
   }

It should be the responsibility of the caller to log the exception or ignore it (in case of BackgroundPipelineCreator)
How was this patch tested?
Full CI tests.

Thanks the review @bharatviswa504 and @bshashikant. Build become green. I am merging it now..


reproduce #560

Good job!

Seems like an enum problem.
/cc @Desdaemon who implemented this great feature. What do you think?


Investigate status of missing metabase alerts for attestation service on-call

We are missing attestation service alerts in the informational alerts Slack channel.
This ticket should:

check on the status of the metabase/Slack integration bug fix
if it looks like the fix won't happen for a while, set up a Slack alert to remind on-call person to check the dashboards once a shift

duplicate of https://app.zenhub.com/workspaces/identity-backlog-board-60e61301c3dc55000e540b61/issues/celo-org/celo-monorepo/9353


Less function svg-gradient and url-loader conflict

Using the svg-gradient function will transform this:
div {
  background-image: svg-gradient(to right, red, green 30%, blue);
}

to this:
div {
  background-image: url('data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIiA/PjxzdmcgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIiB2ZXJzaW9uPSIxLjEiIHdpZHRoPSIxMDAlIiBoZWlnaHQ9IjEwMCUiIHZpZXdCb3g9IjAgMCAxIDEiIHByZXNlcnZlQXNwZWN0UmF0aW89Im5vbmUiPjxsaW5lYXJHcmFkaWVudCBpZD0iZ3JhZGllbnQiIGdyYWRpZW50VW5pdHM9InVzZXJTcGFjZU9uVXNlIiB4MT0iMCUiIHkxPSIwJSIgeDI9IjEwMCUiIHkyPSIwJSI+PHN0b3Agb2Zmc2V0PSIwJSIgc3RvcC1jb2xvcj0iI2ZmMDAwMCIvPjxzdG9wIG9mZnNldD0iMzAlIiBzdG9wLWNvbG9yPSIjMDA4MDAwIi8+PHN0b3Agb2Zmc2V0PSIxMDAlIiBzdG9wLWNvbG9yPSIjMDAwMGZmIi8+PC9saW5lYXJHcmFkaWVudD48cmVjdCB4PSIwIiB5PSIwIiB3aWR0aD0iMSIgaGVpZ2h0PSIxIiBmaWxsPSJ1cmwoI2dyYWRpZW50KSIgLz48L3N2Zz4=')
}

Which is problematic since webpack will then try and fail to resolve 'file' or 'directory', since no file actually exists.
Looking at the url-loader it may make sense to fix there, but sense my issue is currently tied to less I'm reporting it here.

Thx for reporting anyway.

No problem, it's fixed in less now. Sure was a bugger to track down though.


Update http4s-circe, http4s-dsl to 1.0.0-M43

About this PR
📦 Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl

from 1.0.0-M38 to 1.0.0-M43
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, version-scheme:early-semver, commit-count:1

Superseded by #335.


Zero intermediate files as soon as they are no longer necessary (posttask?)

I would like to be able to zero intermediate files (using the zero_file function in the FAQ), whilst the pipeline is running, to limit the amount of disk space used by the pipeline. A concrete example of this is the GATK pre-processing pipeline, which generates a number of large bam files as the results of its indel realignment and base quality score recalibration.  The machine I am running these pipelines on unfortunately does not have enough disk space available for me to wait for the pipeline to complete before I start zeroing of files. Ideally I would like to be able to zero intermediate files as quickly as possible, which is usually one or two tasks after they were generated.
Is there any way to do so at the moment using the ruffus decorators? Ideally I would like to include the zeroing loosely into the existing pipeline, without having to rewrite intermediate tasks. The task responsible for zeroing files will of course need to know the file names generated by the previous tasks..
(I love the new OO syntax by the way, it makes tasks/pipelines much easier to reuse!)

If you feel that the intermediate files can be zeroed one or two tasks after they were generated, the easiest way to do this would be to

forward these intermediate files for these one or two tasks (not pretty I know). In other words, the files should appear in both input and output. Ruffus should not check the timestamps when that is the case but hopefully should still checksum the files.
It might be clearest to have a dedicated zeroing task. With the new syntax, this could be a reusable function.
The intermediate files after zeroing should have the same time stamp (make sure that this is so!) but a different size. I suspect that there needs to be a Ruffus hook to allow this to happen.
Out of curiousity... will you ever have to update / rerun the pipeline. Will you delete all zeroed files at that point? How will you know which files are zeroed? Do some other files have zero file sizes? Have you considered putting a small marker in these "zeroed" files (so they would just contain e.g. "JULIANDERUITER_ZEROED")

I may not have time to write some dummy pipelines to try this out. Can you keep the newsgroup informed as this sounds like of general utility...
I am of course committed to making whatever changes to Ruffus code to make sure it can work.
Thanks a lot,
Leo
P.S. There is not yet any direct support for discarding intermediate files because

I haven't figured out the syntax. When should the zeroing happen.... (what does gmake do?)
I am not yet satisfied that zeroed intermediate files would be distinguishable from failed jobs. (The checksum database would need to be impregnable whereas at the moment, it will fall back on file times, if, for example, there are synching problems with the file system etc.)
I have not yet sorted out how Ruffus would safely back track up dependencies when it realizes that "completed" but zeroed files actually need to be recreated to complete a task far down the dependency chain.


pull request for e2e test at 11/22/2016 20:56:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ADBDEV-4909-16: Remove redundant scanPlan->flow check for NULL.

Remove redundant scanPlan->flow check for NULL.
At this point scanPlan->flow is always not NULL, so I replaced the redundant
scanPlan->flow check for NULL with an assertion.

Can you add a comment why flow can't be null?


Update textual information on the header section

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1147
Originally created by @gtsiolis on 2019-10-17 16:20:30

Problem
The header section at meltano.com could become more clear on what this project is and what it can offer.
Describing it as a free alternative does not make it clear if the project is an online service, a self-installed package, etc. and this could not be applicable in the future according to monetization plans for the future.

Right now Meltano is open source. In the future we'll introduce proprietary features to have a sustainable business model to do quality control, marketing, security, dependency upgrades, and performance improvements. An example of a proprietary/source available feature is fine grained access controls.

Background

The project is referenced as end-to-end data pipeline and dashboarding tool in the roadmap section.
The project is also referenced as a open source data toolkit in some speedruns.
...

Proposal

Keeping the open source description sounds optimal since the plan is to keep it as open source, at least most of it.
The data toolkit description also balances the clarification and abstraction needed in the long run.

TBD

View 5 previous comments from the original issue on GitLab


Add new event to change the criteria for products before a product export is generated

1. Why is this change necessary?
It's currently not possible to add more data to product exports e.g. for custom associations or associations from the core other than those defined in the generator.
2. What does this change do, exactly?
Introduces a new event and dispatches it before a product export feed is generated
3. Describe each step to reproduce the issue or behaviour.

Create a product export sales channel
Customize the template and try to use the tags e.g.
You can't because the association is not added to the criteria
Be angry at @t2oh4e because he didn't add an event when creating the export generator

After this PR you can use a subscriber to add the necessary data like this:
class ProductCriteriaSubcriber implements EventSubscriberInterface
{
    public static function getSubscribedEvents(): array
    {
        return [
            ProductExportProductCriteriaEvent::class => 'onProductExportProductCriteria',
        ];
    }

    public function onProductExportProductCriteria(ProductExportProductCriteriaEvent $event): void
    {
        $event->getCriteria()->addAssociation('tags');
    }
}

4. Please link to the relevant issues (if any).

5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-15549
Please use this issue to track the state of your pull request.

Can we add a criteria title in the same moment?

Sure, I could do that. What's the policy for changing an already approved and scheduled PR? :D
@shyim @J-Rahe any worries if I add ->setTitle('product-export::products') to the criteria?

@t2oh4e do it :blush: we can update the internal PR :wink:

Title added :)

Any idea why one of the Cypress checks fails? Doesn't look like it's related to my PR 🤔

Happy to see these changes :)

Hey @t2oh4e, the cypress failure doesn't look like it's from your PR, so don't worry about it.
Nonetheless is just wanted to import your PR, and noticed that there is no test for the new event.
Could you please add one to make sure the event wont break silently in the future?

Hey @J-Rahe while adding tests for my new event I realised that the other events in this class weren't tested either. So I added tests for my new event and the already existing events.

Hey @t2oh4e, your PR just got merged : )


🐛 | StatusLed stays off when ShutdownButton LED is activated and shutdown request is aborted by releasing the shutdown button before the hold time expired

Bug
What I did
I activated the shutdown button led and the StatusLed according to gpio configuration and tested it by holding down the shutdown button less than the configured hold time and released the button.
What happened
As long as the shutdown button was pressed the led began to flash as it should.
After the shutdown button was released before the hold time has expired, the LED stopped blinking and stayed off.
I expected this to happen
I expect the LED to stay on when the shutdown is aborted and a StatusLed is confugred to be able to see that the box is on and stays on.
Further information that might help
Software
Base image and version
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"
Branch / Release
master
Installscript
scripts/installscripts/buster-install-default.sh
Hardware
RaspberryPi version
Raspberry Pi 3 Model B Rev 1.2
RFID Reader
16c0:27db HXGCoLtd Keyboard
Soundcard
onboard
Other notable hardware

I have an idea on how to fix it and will create a PR soon.

This seems to be only an issue when the StatusLED and the ShutdownLED use the same GPIO.


Git executable isn't found

I'm using: Windows 7 64-bit, Sublime Text 2.0.2 (64-bit), SublimeGit 1.0.32 (installed using Package Control).
I'm having trouble getting SublimeGit to find the git executable.
Git is in this location on my machine: H:\Program Files (x86)\Git\bin.
This location is in my PATH environment variable, and I can run git from cmd.exe.
When I try to use SublimeGit, I get this message:

I tried all the options suggested in issue #55. Tried setting git_executables["git"] to "H:\Program Files (x86)\Git\bin", with forward slashes instead of backslashes, with short paths (H:/PROGRA~2/...), with/without trailing slash, with/without the executable name appended. Nothing works.
Also tried all of that directly in SublimeGit.sublime-settings instead of the user settings, just to be sure.
Following is my attempt at getting a little more information.
Using Process Monitor, I can see that no matter what I set git_executables to, Sublime only looks for git.exe in this path: C:\Program Files\git\bin\git.exe.
That path doesn't exist, it's not in my PATH, and I never had Git installed there. I also made sure it's nowhere in the Windows registry. So I'm guessing this is hard-coded somewhere in SublimeGit.
I can also see that the path I put in git_executable isn't searched. This is what I currently have in the settings (intentionally using a string that's easy to find in the log :)):
"log_level": "DEBUG",
"git_executables": {
    "git": ["blablabla"],
    "git_flow": ["blablabla"],
    "legit": ["blablabla"],
    "gitk": ["blablabla"]
},

As far as I can see from Process Monitor, Sublime Text never looks for that path.
Here's the console log for trying to run "git log":
[2014-11-15 14:29:41,112 - INFO     - SublimeGit.helpers] get_dir_from_view(view=30): C:\projects\project1
[2014-11-15 14:29:41,112 - INFO     - SublimeGit.helpers] get_repo(window=30, silent=True): C:\projects\project1 (file)
[2014-11-15 14:29:41,128 - DEBUG    - SublimeGit.cmd] cmd: [u'C:\\Program Files\\git\\bin\\git.exe', '--no-pager', '-c', 'color.diff=false', '-c', 'color.status=false', '-c', 'color.branch=false', '-c', 'status.displayCommentPrefix=true', '-c', 'core.commentchar=#', 'symbolic-ref', '-q', 'HEAD']
[2014-11-15 14:29:41,128 - INFO     - SublimeGit.helpers] get_dir_from_view(view=30): C:\projects\project1
[2014-11-15 14:29:41,128 - INFO     - SublimeGit.helpers] get_repo_from_window(window=8, silent=False): C:\projects\project1 (active file)
[2014-11-15 14:29:41,128 - DEBUG    - SublimeGit.cmd] cmd: [u'C:\\Program Files\\git\\bin\\git.exe', '--no-pager', '-c', 'color.diff=false', '-c', 'color.status=false', '-c', 'color.branch=false', '-c', 'status.displayCommentPrefix=true', '-c', 'core.commentchar=#', 'log', '--no-color', '--date=local', '--format=%s%x03%H%x03%an%x03%aE%x03%ad%x03%ar%x04']
error: Executable '['git']' was not found in PATH. Current PATH:

H
\Python27_32bit\;H
\Python27_32bit\Scripts;H
\Windows\system32;H
\Windows;H
\Windows\System32\Wbem;H
\Windows\System32\WindowsPowerShell\v1.0\;H
\Program Files (x86)\QuickTime\QTSystem\;H
\Program Files\nodejs\;H
\Program Files (x86)\SlikSvn\bin;H
\Program Files\Microsoft SQL Server\110\Tools\Binn\;H
\Program Files\FileBot\;H
\Program Files (x86)\nodejs\;H
\Users\yuval\AppData\Roaming\Python\Scripts;H
\Users\yuval\AppData\Roaming\npm;H
\Program Files (x86)\Git\bin

Try adjusting the git_executables['git'] setting.
Traceback (most recent call last):
  File ".\sublime_plugin.py", line 339, in run_
  File "SublimeGit/sgit/log.py", line 32, in run
  File "SublimeGit/sgit/helpers.py", line 395, in get_quick_log
  File "SublimeGit/sgit/cmd.py", line 247, in git_string
  File "SublimeGit/sgit/cmd.py", line 90, in _string
  File "SublimeGit/sgit/cmd.py", line 159, in cmd
sgit2.cmd.SublimeGitException: Could not execute command: [Error 2] The system cannot find the file specified
[2014-11-15 14:29:42,329 - INFO     - SublimeGit.helpers] get_dir_from_view(view=30): C:\projects\project1
Exception in thread Thread-82:
Traceback (most recent call last):
  File ".\threading.py", line 532, in __bootstrap_inner
  File "SublimeGit/sgit/status.py", line 564, in run
  File "SublimeGit/sgit/cmd.py", line 247, in git_string
  File "SublimeGit/sgit/cmd.py", line 90, in _string
ValueError: need more than 2 values to unpack

Also, this message appears in the console whenever I switch between files:
[2014-11-15 14:36:29,138 - INFO     - SublimeGit.helpers] get_dir_from_view(view=71): C:\projects\project1
[2014-11-15 14:36:29,138 - INFO     - SublimeGit.helpers] get_repo(window=71, silent=True): C:\projects\project1 (file)
[2014-11-15 14:36:29,138 - DEBUG    - SublimeGit.cmd] cmd: [u'C:\\Program Files\\git\\bin\\git.exe', '--no-pager', '-c', 'color.diff=false', '-c', 'color.status=false', '-c', 'color.branch=false', '-c', 'status.displayCommentPrefix=true', '-c', 'core.commentchar=#', 'symbolic-ref', '-q', 'HEAD']
Exception in thread Thread-96:
Traceback (most recent call last):
  File ".\threading.py", line 532, in __bootstrap_inner
  File "SublimeGit/sgit/status.py", line 564, in run
  File "SublimeGit/sgit/cmd.py", line 247, in git_string
  File "SublimeGit/sgit/cmd.py", line 90, in _string
ValueError: need more than 2 values to unpack

caused by removing folder with git repo while still having status window opened. Fixed in #259


post-processing on continuous livestreams is wrong

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
yt-dlp used to download continuos livestreams fine with a script I wrote to restart the process when it is done.  Now, however, with its new behavior, it tries to post-process the file, which is horrible, causing a huge interruption in the archived stream and causing a huge amount of inefficient thrashing of the disk.  This is not the right way to download livestreams!  Post-processing 10 gigabytes every 6 hours is a huge interrupt to the streams.  I looked through the manual, but there is no obvious way to turn this atrocious behavior off.  I can do my own postprocessing when I extra what I want from the video.  Maybe it would make sense to postprocess the files to another drive so the disk doesn't thrash seeking back and forth on the same disk.
Verbose log
It said "post processing" and it stopped recording anything, and did not quit the process until I killed it myself.  It would have taken hours to postprocess that file, interrupting my archiving!

Probably --fixup never is what you want. If not, provide verbose logs


'Model' object has no attribute 'logpt'

I'm very new to PYMC and I wanted to test an compute intensive approach (not even sure if it is a good approach) with nutpie.
def get_px_deterministic(data):
    px = []
    for i in range(len(data)):
        beta = pm.Normal(f"beta_{data[i].name}", mu=0, tau=0.001, initval=0)
        alpha = pm.Normal(f"alpha_{data[i].name}", mu=0, tau=0.001, initval=0)
        px.append(pm.Deterministic(f"p_{data[i].name}", 1.0/(1.0 +  at.exp(beta*data[i].data + alpha))))
    return px

def get_p_deterministic(px):
    p=1
    for prop in px:
        p*=prop
    return pm.Deterministic("p", at.pow(p, 1/len(px)))

with pm.Model() as model:
    px = get_px_deterministic(full_training_data)
    p = get_p_deterministic(px)
    observed = pm.Bernoulli("bernoulli_obs",p,observed=D)

In the essence I'm taking the geometric mean of some sigmoid generated probabilities, and try to sample it for a binary decision (the description is probably incorrectly expressed ).
The error I receive during the compilation is that logpt is not yet implemented (?)
  File ".../model.py", line 155, in <module>
    compiled_model = nutpie.compile_pymc_model(model)
  File ".../lib/python3.8/site-packages/nutpie/compile_pymc.py", line 38, in compile_pymc_model
    n_dim, logp_func, expanding_function, shape_info = _make_functions(model)
  File ".../lib/python3.8/site-packages/nutpie/compile_pymc.py", line 96, in _make_functions
    logp = model.logpt()
AttributeError: 'Model' object has no attribute 'logpt'

Could it be implemented?

Thank you for reporting this, seems this was broken a few days ago by this PR: https://github.com/pymc-devs/pymc/pull/5320. nutpie was still using an old API that was removed in this PR.
Should be easy enough to fix on our end.
Somewhat off topic: you can use the shape (or dims) argument to pm.Normal etc to get a vector of number at the same time, so you can avoid the loops.

Just tried to fix it and noticed that apparently I already did, some time ago... :-)
Can you try updating nutpie to see if that fixes it?


[feature request] watch history

This won't be one of the important features until the stable is launched.


Enhance the text that describes nested LRA behaviour

The NESTED LRA annotation element has extra requirements beyond the ones for 'top level' LRAs. To facilitate safe usage of the feature we need to provide clear documentation. This task should at least include the following changes:

Update the javadoc for the @AfterLRA annotation to describe the meaning of the LRA_HTTP_PARENT_CONTEXT_HEADER header
Include a comprehensive example showing best practice when using nested LRAs.
Ensure that the spec examples include nesting.
Review the existing text around nesting to verify its accuracy, clarity and sufficiency.

PR merged.


Add NO_PROXY env var to the lab nodes by default

When http proxy is in place to access internet resources containerlab users typically have to set the http_proxy/https_proxy env vars in their topologies to make the applications running in the containers use the proxy:
name: anysec-macsec

mgmt:
  network: anysec
  ipv4-subnet: <IP_ADDRESS>/24

topology:
  kinds:
    linux:
      image: ghcr.io/srl-labs/network-multitool
      env:
        http_proxy: ${http_proxy}
        https_proxy: ${https_proxy}
        no_proxy: pe1,pe2,p3,p4,ce5,ce6,client7,client8,gnmic,prometheus,grafana,localhost,<IP_ADDRESS>/24

Note, how every lab node, lab mgmt network and localhost has to be called out in the no_proxy env var to ensure that the nodes when communicating with each other will not try to reach the proxy.
Containerlab should set these values automatically as a default env var set:

all node names as provided in the topology
mgmt network
localhost
<IP_ADDRESS>
*.local addresses

Would really love to see this feature implemented!
Subnet ranges do not work with the no_proxy variable. Hostname wildcards can be used, but IP addresses have to be added individually not via CIDR.


bugfix for missing mechanism paths

The mechansims/gpu and mechanisms/multicore paths were not being generated
by CMake during configuration, and modcc was silently ignoring that it was
not able to write to the nonexistant target paths.
I have updated CMake to generate the missing gpu and multicore target paths.

Looks fine to me.


Refactor NCMC code ot eliminate code duplication?

We've moved all of the nonequilibrium switching functionality from perses into NonequilibriumLangevinIntegrat.
All of the nonequilibrium integrators can be moved out of ncmc_switching.py and replaced by calls to openmmtools.integrators.NonequilibriumLangevinIntegrator with the appropriate integrators splittings:

For g-BAOAB, this would be R V O H O V R, where the Hamiltonian update happens in the middle to make the protocol symmetric. We currently recommend this, neglecting shadow work and only including protocol work in the NCMC acceptance probability---this will be the topic of a paper we're trying to tackle ASAP).
For VVVR, this is O V R H R V O
For velocity Verlet, this is V R H R V
For GHMC (Metropolized VVVR) with alchemical updates, you want O { V R H R V } O

@maxentile, @bas-rustenburg, and @patrickgrinaway were integral in putting this together, and can help you if you need more info.

Since we have to worry about external protocol work (external as in work from outside the lambda perturbations) we can't use the NonequilibriumLangevinIntegrat class directly. I think that there were some discussions about modifying the base code so that it can handle the external protocol work case, but it's not implemented as of yet.
My temporary work around for this was to create a class that inherits from the above integrator so that it handles external protocol work. I could look to put it up once we are able to specify the integrator in the refactored code (which I'll open up another issue for).

@jchodera @pgrinaway -- thoughts on Sam's response? I would love to avoid code duplication but I'm not seeing an obvious way to do this here.
And @sgill2 - have you brought in your work-around yet or is that a "to do" item?

My temporary work around for this was to create a class that inherits from the above integrator so that it handles external protocol work

This is what I would do. You can take a look at the ExternalPerturbationLangevinIntegrator if that is helpful. If you have trouble with that, I could help.

Thanks for the input @pgrinaway!
@davidlmobley I haven't brought the integrator into github yet. I'll go about putting up a WIP for it now.

Wait, where is your external work coming from?

The external work should be zero in the case of a rotation/translation of a given whole fully non-interacting ligand.
However, we're also looking to apply other moves, such as rotational side chain sampling, in which the angle/torsion energy changes would need to be accounted for by the external work.

However, we're also looking to apply other moves, such as rotational side chain sampling, in which the angle/torsion energy changes would need to be accounted for by the external work.

One idea would be to use Metropolis Monte Carlo on the lambda = 0 (or softened lambda) state for the sidechain. If you do this, you don't need to include the protocol work contributions since you Metropolize them out. It would also allow you to rapidly decorrelate the sidechain in the softened state. Then you continue your integration back from lambda = 0 -> 1 and just use the total protocol work from NonequilibriumIntegrator for acceptance.

For the BLUES paper, however, you can probably just get away with using NonequilibriumIntegrator throughout.

@jchodera - where is NonequilibriumIntegrator?  Or do you mean NonequilibriumLangevinIntegrator?
@sgill2 - I think a key question is whether we can replace the ncmc_switching.py (with code which we'd then be responsible to maintain which (entirely? partially?) duplicates stuff the Chodera lab has elsewhere) with code directly from openmmtools so we can just introduce that as a dependency.  This would be for the BLUES paper, which is just on rotational moves. Unless there is something we need that is not there, it strikes me that this would be preferable because we want to avoid unnecessary code duplication. Is there any functionality we need that's NOT in openmmtools?
Thanks.

Sam and I just discussed this. We are enthusiastic about the idea of doing Metropolis Monte Carlo on the sidechain (that may make certain other aspects a bit cleaner/easier to deal with as well) as per https://github.com/MobleyLab/blues/issues/50#issuecomment-297127960 so we think we'll proceed in that direction.
That also removes the need for the NonequilibriumExternalLangevinIntegrator that #53 introduces, so we can update that PR to note that this integrator is vestigial/not currently used by the code.
This also means we can do as this issue's title suggests and migrate the code to use NonequilibriumLangevinIntegrator as our default, replacing ncmc_switching.py. @sgill2 remarks that in his tests on lysozyme he's getting similar acceptance to what he was getting before (though presumably acceptance will be better than before with more relaxation) so at least that shouldn't break anything, though we're going to get @nathanmlim to look at it on another test system.
So, long story short, I think we can kill a couple birds with one stone here and move to NonequilibriumLangevinIntegrator to:
(a) move to the better integrator
(b) remove code duplication
And then separately, we'll use MMC for the sidechain moves in the tangential project.

Upon further discussion we think it might still be relevant to test a metropolized and external protocol approach and see how they perform in different cases.
If we consider the sidechain case again, say a side chain rotation is performed at lambda=0 (for the sterics, electrostatics) and that rotamer is 5 kcals/mol more unfavorable. If we metropolized the move we'd often reject it. However if it turns out that at lambda=1 the intermolecular interactions contribute 6 kcals/mol the move would actually be accepted in the 'external protocol work' scheme since the overall energy change would be favorable.

@jchodera - where is NonequilibriumIntegrator? Or do you mean NonequilibriumLangevinIntegrator?

Whoops! Yes, NonequilibriumLangevinIntegrator.

This also means we can do as this issue's title suggests and migrate the code to use NonequilibriumLangevinIntegrator as our default, replacing ncmc_switching.py. @sgill2 remarks that in his tests on lysozyme he's getting similar acceptance to what he was getting before (though presumably acceptance will be better than before with more relaxation) so at least that shouldn't break anything, though we're going to get @nathanmlim to look at it on another test system.

That's surprising!

How long are the NCMC trajectories you were using?
What timestep?
Are you using explicit solvent?
What constraint tolerance for the integrator and PME tolerance?

If we consider the sidechain case again, say a side chain rotation is performed at lambda=0 (for the sterics, electrostatics) and that rotamer is 5 kcals/mol more unfavorable. If we metropolized the move we'd often reject it. However if it turns out that at lambda=1 the intermolecular interactions contribute 6 kcals/mol the move would actually be accepted in the 'external protocol work' scheme since the overall energy change would be favorable.

I believe you can still use NonequilibriumLangevinIntegrator in that case. You can accumulate protocol work associated with the lambda changes during integration, and manually include the protocol work from the sidechain rotamer flip using the initial and final potential energies (from state.getPotentialEnergy()) before and after the Monte Carlo move. NonequilibriumLangevinIntegrator will correctly accumulate the protocol work from the lambda context parameter updates in this case (and even the shadow work too, if desired).

Update: As noted in https://github.com/choderalab/openmmtools/issues/201#issuecomment-302725904, I got the step order wrong, and BAOAB based methods should have the order V R O R V and not the other way around!

For the record, I believe the g-BAOAB scheme noted above is not represented correctly, per John in this thread: https://github.com/choderalab/openmmtools/issues/201#issuecomment-302738799


Add active row highlight for CR timetables and lighten header

Summary of changes
Asana Ticket: 🆕 CR Timetable | Highlight Timetable Row on Hover
Commuter Rail timetable rows now highlight in blue when they are moused over. The header color was changed as well, as per conversation with @ingridpierre.

@beeuhtricks can we disable the highlight on just these first few cells in the corner of the header?

etc.

This should fix the header highlight problem, and Backstop still seems ok.

👍beauteous! lgtm!


Update index mappings

[x] DBNL (+ reindex to ES8)
[x] Dutch Annual Reports
[ ] Dutch Newspapers Public
[ ] ECCO
[ ] Goodreads
[ ] Guardian-Observer
[ ] Jewish Inscriptions
[ ] Periodicals
[ ] Rechtspraak
[ ] Times
[ ] Troonredes

See #1285 - can you please add some context about the how + why? This issue would be impossible to pick up for another developer.

I added the description from #1285 here. I just needed an issue to track the work I'd already been doing.

@Meesch , I just set the updated_highlighting=True as a new global default on the concerning branch (https://github.com/UUDigitalHumanitieslab/I-analyzer/tree/feature/corpus-updates)


Convert Azure managed disks storage from Standard to Premium or Premium to Standard - PowerShell steps have syntax errors

All the PowerShell variables in the examples on the page https://learn.microsoft.com/en-us/azure/virtual-machines/linux/convert-disk-storage are missing the "$".
When filled/declared it should be "$rgName='yourResourceGroup'" and not "rgName='yourResourceGroup'" (without the "$") etc.
Otherwiese the code examples will not work

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

ID: a106b26c-4d9b-9c83-95e6-72f2f13ff39e
Version Independent ID: 8e10d90c-543d-cb32-1098-64dd8504212a
Content: Convert managed disks storage between different disk types using Azure CLI - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/convert-disk-storage.md
Service: storage
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: albecker

@FrankPlawetzki
Thanks for your feedback! We will investigate and update as appropriate.

@FrankPlawetzki
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Update Don't Starve Together Checkboxes

Converted the checkboxes to match the latest format.

I do not believe this one causes any issue for the game server. It is purely to clean up the code.


Create a profile page template for users

We presently don't have a profile page for authors.
Create a template UI with HTML CSS and js.
You can use bootstrap card codes from the internet.

What are the requirements for information, layout, you want to appear on the Profile page?

What are the requirements for information, layout, you want to appear on the Profile page?

It should contain

Profile pic
Full name
Followers and social media icons
About
template post( which will be used in backend to show his posts)

I am sharing some templates you can follow these
https://bootsnipp.com/snippets/K0ZmK
https://www.bootdey.com/snippets/view/user-profile-bio-graph-and-total-sales

This issue is soved by @chaitak-gorai by the commit 7a7af3ac7b29a5ceedac7e7a25468589d249a3f2


[Question] How to report new domain?

macOS user here, how to report new domain from macos system?
how to tell if blocking is working. is can't access adobe websites is a way to tell blocking is working?

HTTP Toolkit, Wireshark, Charles Proxy
Enter ic.adobe.io from your browser. If you see 404 Not Found, you're not blocking correctly. If you get a browser error (ERR_ADDRESS_INVALID) you're blocking correctly.


Change Ubuntu/Debian command to apt (IDFGH-2536)

apt is the preferred installation method, I gather, from all the recent Ubuntu tutorials I've been reading. Moreover apt-get threw a weird error about some libssh package that didn't occur for me under apt.

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

Hi @AndrewQuinn2020 thank you for your contribution.
apt is high level interface to apt-get. If you take a look at the man page of apt then you can see that the install subcommand is further explained in the man page of apt-get.
apt and apt-get both should work.  And they should work the same way since we don't have in or documentation any special argument for install.
Your libssh issue relates probably to your broken apt-get package. You could try to resolve that issue by yourself or report it to Ubuntu.
I don't think that changing apt-get to apt is necessary for our documentation.


[SPARK-35048][INFRA] Distribute GitHub Actions workflows to fork repositories to share the resources

What changes were proposed in this pull request?
This PR proposes to leverage the GitHub Actions resources from the forked repositories instead of using the resources in ASF organisation at GitHub.
This is how it works:

"Build and test" (build_and_test.yml)  triggers a build on any commit on any branch (except branch-*.*), which roughly means:

The original repository will trigger the build on any commits in master branch
The forked repository will trigger the build on any commit in any branch.

The build triggered in the forked repository will checkout the original repository's master branch locally, and merge the branch from the forked repository into the original repository's master branch locally.
Therefore, the tests in the forked repository will run after being sync'ed with the original repository's master branch.
In the original repository, it triggers a workflow that detects the workflow triggered in the forked repository, and add a comment, to the PR, pointing out the workflow in forked repository.

In short, please see this example HyukjinKwon#34

You create a PR and your repository triggers the workflow. Your PR uses the resources allocated to you for testing.
Apache Spark repository finds your workflow, and links it in a comment in your PR

NOTE that we will still run the tests in the original repository for each commit pushed to master branch. This distributes the workflows only in PRs.
Why are the changes needed?
ASF shares the resources across all the ASF projects, which makes the development slow down.
Please see also:

Discussion in the<EMAIL_ADDRESS>mailing list<EMAIL_ADDRESS>Infra ticket: https://issues.apache.org/jira/browse/INFRA-21646

By distributing the workflows to use author's resources, we can get around this issue.
Does this PR introduce any user-facing change?
No, this is a dev-only change.
How was this patch tested?
Manually tested at https://github.com/HyukjinKwon/spark/pull/34 and https://github.com/HyukjinKwon/spark/pull/33.

Test build #137066 has started for PR 32092 at commit 6dabe60.

cc @dongjoon-hyun, @MaxGekk, @maropu, @srowen, @gengliangwang FYI
cc @potiuk too FYI who might be interested in this approach.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41644/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41644/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41644/

Test build #137066 has finished for PR 32092 at commit 6dabe60.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137066/

I was surprised that there is such a solution for the resource-shortage issue, nice.

@HyukjinKwon Thanks for the great effort in testing infra!
+1 for this proposal. I just checked https://github.com/pricing and found that GitHub action is totally free for public repositories. Nice!

It's great. Thank you, @HyukjinKwon !

This is an excellent idea ! I :heart: it.  Are all the pieces already working :) ?

One thing that is missing is how to get back the information about the status of the build in the forked repository. I do not think with the permissions model of GH it is possible to do directly, but I can think of another 'scheduled' workflow that could run in the original repository that could read status from all the PRs that had "forked" builds running.
That would be quite possible. We could even try to utilize "GitHub Check" for that https://docs.github.com/en/rest/reference/checks - I am already using it in Airflow to notify status betwen workflows (I have a 'build image` workflow that notifies the PR that runs a CI workflow this way.
This looks like the regular "yellow/green/red" build status indicator, and it impacts the "mergable" status of the PR.
This way we could have a complete solution, when the PR would become "green" when the forked build succeeds.

One thing that is missing is how to get back the information about the status of the build in the forked repository. I do not think with the permissions model of GH it is possible to do directly, but I can think of another 'scheduled' workflow that could run in the original repository that could read status from all the PRs that had "forked" builds running.

I think it needs a bit complex logic, but I could possibly write a custom GitHub Action for that similar to what I've done with https://github.com/potiuk/cancel-workflow-runs

Thanks @potiuk. Yeah, I think it makes sense to do that seamlessly with updating the status checks too. Borrowing the idea from cancel workflow is a good idea. If we're going this way, I will take a separate look for that.
FWIW, I do think this is still a workaround that disables GitHub Actions to work out of the box. In the mailing list, the permanent solution is being discussed, which I think makes sense and reasonable.

Agree it's a nasty hack and should be out-of-the-box built in GitHub.

I did not have time to take a look yet, and during the week I have a bit limited time but in a week or so I hope to have some POC with the checks in place.

Definitely seems like a possible way to distribute the load and cost.
Are there limitations on the personal repo GitHub actions?
looking at the free pricing for instance: (https://github.com/pricing)

2,000 Actions minutes/month
500MB of Packages storage

How many action minutes would an average spark pr take?  the test build took 1 hours 27 minutes duration but is that action minutes (https://github.com/MaxGekk/spark/actions/runs/728916852) or do you have to add up each of the individual build modules, java 11, etc?
Similar with storage are log files still available, how long or is there any others differences to deal with?
just wondering some of these because say a user runs out of action minutes or there are problems, then what do we do?

@tgravescs : Github Actions and storage are FREE for public repositories. If you look for the smaller footprint in the pricing page (bold is mine):

Unlimited public/private repositories
Unlimited collaborators
Included2,000 Actions minutes/month
Free for public repositories
Included500MB of GitHub Packages storage
Free for public repositories
IncludedCommunity Support

There are some limitations when it comes to number of runners available,

This is ready for a review. I will take a look for https://github.com/apache/spark/pull/32092#issuecomment-816015860 separately.

Test build #137286 has started for PR 32092 at commit 74a7674.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41866/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41866/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41866/

Test build #137286 has finished for PR 32092 at commit 74a7674.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137286/

Thanks @dongjoon-hyun!

Thanks @gengliangwang! (I just fixed a indentation nit)

I am merging it into master.
It's a daytime in my time .. so would likely be able to track it if anything goes wrong.


"buildModelFromXML()" cannot be used to import .xml file for Pinocchino 2.6.4

Hello all! Recently I met a new problem when I use the function "buildModelFromXML()" to import my model from .XML file. Here is my code below:
xml_path = "/home/Tony/RR_robots/robot_description/UR5/UR5.xml" model = pin.buildModelFromXML(xml_path) data = model.createData()
When I run the program, it shows on terminal as
ValueError: The XML stream does not contain a valid URDF model.
For the UR5.xml file, I can make sure there is no problem with that, and I can run it on Mujoco directly. Also, in another project, I can import the XML file through the IKPY library. I don't know what's the problem with this function. Is there something I did wrong or   Pinocchio 2.6.4 didn't support this function?

Can you provide the UR5.xml file for debugging?

Hi, May I send you through email? The file is a little bit large. My email<EMAIL_ADDRESS>Thanks for your help!
Anyone who would like to see the XML file can send email to me and I will send you. Thanks!

Can you provide the UR5.xml file for debugging?

Hi, May I send you through email? The file is a little bit large. My email<EMAIL_ADDRESS>Thanks for your help!
Anyone who would like to see the XML file can send email to me and I will send you. Thanks!

Closing the issue as replied via email - the file in question is a MuJoCo MJCF file and not a URDF. Cf. https://github.com/stack-of-tasks/pinocchio/issues/1680 for MJCF support


[Feature Request] Retrieval results in deployment results

Is your feature request related to a problem? Please describe.
I have generated an evluation set, run Autorag to find best pipeline, great stuff. I would like to use it, and can run the CLI, API and web app. However, these all only seem to reply with the generated text, in many cases it's good to see the retrieval results also to eyeball how well generation is going, but many RAG systems present these to users.
Describe the solution you'd like
It would be great if the CLI and API also returned retrieval results.
Describe alternatives you've considered
Additional context
Thanks!

Little bit duplicate with #493

Anyway @dividor Thanks for your interest and making an issue at AutoRAG.
I have a question. Do you want retrieved_contents at both API server and web???
If so, I think it will be better to change this issue for API server, and for web use #493 issue. Is it okay?

Hi! Sorry for the delay.
API would be fine, thanks.


core dumped

🐛 Bug description
312115 Segmentation fault      (core dumped)
When running:
wasm-pack build --no-typescript --target bundler --out-dir build --out-name index --release
🤔 Expected Behavior
No segfault.
👟 Steps to reproduce
I was trying to build the cloudflare rustwasm template. I placed the current state of my repo in a fork.
In ths case, wasm-pack was being started by cloudflarezs "worker-build" tool, which in turn was started by "wrangler dev".
The wasm-pack command was found via strace.
However, cloning the repo and immediately running wasm-pack still seems to cause the segfault.
git clone git@github.com:jpentland/dadgum.git -b coredump
cd dadgum
wasm-pack build --no-typescript --target bundler --out-dir build --out-name index --release

🌍 Your environment
Include the relevant details of your environment.
wasm-pack version: wasm-pack 0.10.3
rustc version: rustc 1.67.1 (d5a82bbd2 2023-02-07)

I compiled the debug version of wasm-pack and ran with gdb. Results below. It might be an issue with the libs on my local machine. Will look into it more later if I have time.
[Thread debugging using libthread_db enabled]                 Using host libthread_db library "/usr/lib/libthread_db.so.1". [New Thread 0x7ffff72156c0 (LWP 345231)]                      [New Thread 0x7ffff70146c0 (LWP 345232)]                      [Detaching after vfork from child process 345233]
[Thread 0x7ffff70146c0 (LWP 345232) exited]
                                                              Thread 2 "wasm-pack" received signal SIGSEGV, Segmentation fault.                                                           [Switching to Thread 0x7ffff72156c0 (LWP 345231)]             ___pthread_rwlock_wrlock (rwlock=0x35) at pthread_rwlock_wrlock.c:26                                                        26        int result = __pthread_rwlock_wrlock_full64 (rwlock, CLOCK_REALTIME, NULL);                                       (gdb) trace                                                   Tracepoint 1 at 0x7ffff7c71d66: file /usr/src/debug/glibc/glibc/nptl/pthread_rwlock_common.c, line 603.

Here's the full backtrace. Looks like it might not be related to wasm-pack at all.
0x00007ffff7c71d66 in pthread_rwlock_wrlock () from /usr/lib/libc.so.6
(gdb) bt
#0  0x00007ffff7c71d66 in pthread_rwlock_wrlock () from /usr/lib/libc.so.6
#1  0x00007ffff77ca0ee in CRYPTO_THREAD_write_lock () from /usr/lib/libcrypto.so.3
#2  0x00007ffff7837ee5 in ?? () from /usr/lib/libcrypto.so.3
#3  0x00007ffff7837f81 in X509_STORE_add_cert () from /usr/lib/libcrypto.so.3
#4  0x00007ffff781c540 in X509_load_cert_crl_file_ex () from /usr/lib/libcrypto.so.3
#5  0x00007ffff781c696 in ?? () from /usr/lib/libcrypto.so.3
#6  0x00007ffff7837a4a in X509_STORE_load_file_ex () from /usr/lib/libcrypto.so.3
#7  0x00007ffff7f581ab in ?? () from /usr/lib/libcurl.so.4
#8  0x00007ffff7f5858d in ?? () from /usr/lib/libcurl.so.4
#9  0x0000555555b38a5e in bread_conv (bio=<optimized out>, data=<optimized out>, datal=<optimized out>, readbytes=0x7ffff7213380) at crypto/bio/bio_meth.c:123
#10 0x0000555555b378d5 in bio_read_intern (b=0x7ffff00149c0, data=data@entry=0x7ffff001a263, dlen=dlen@entry=5, readbytes=readbytes@entry=0x7ffff7213380) at crypto/bio/bio_lib.c:270
#11 0x0000555555b37ec7 in BIO_read (b=<optimized out>, data=data@entry=0x7ffff001a263, dlen=dlen@entry=5) at crypto/bio/bio_lib.c:296
#12 0x0000555555ae4f6d in ssl3_read_n (s=s@entry=0x7ffff0012860, n=n@entry=5, max=5, extend=extend@entry=0, clearold=clearold@entry=1, readbytes=readbytes@entry=0x7ffff7213448) at ssl/record/rec_layer_s3.c:309
#13 0x0000555555ae8f41 in ssl3_get_record (s=s@entry=0x7ffff0012860) at ssl/record/ssl3_record.c:207
#14 0x0000555555ae6e20 in ssl3_read_bytes (s=<optimized out>, type=22, recvd_type=0x7ffff72135ec, buf=0x7ffff0014d00 "\001", len=4, peek=0, readbytes=0x7ffff72135f0) at ssl/record/rec_layer_s3.c:1320
#15 0x0000555555b16db8 in tls_get_message_header (s=s@entry=0x7ffff0012860, mt=mt@entry=0x7ffff7213664) at ssl/statem/statem_lib.c:1160
#16 0x0000555555b0c398 in read_state_machine (s=0x7ffff0012860) at ssl/statem/statem.c:579
#17 state_machine (s=0x7ffff0012860, server=0) at ssl/statem/statem.c:434
#18 0x00007ffff7f5b03d in ?? () from /usr/lib/libcurl.so.4
#19 0x00007ffff7f60d6c in ?? () from /usr/lib/libcurl.so.4
#20 0x00007ffff7f58803 in ?? () from /usr/lib/libcurl.so.4
#21 0x00007ffff7efe069 in ?? () from /usr/lib/libcurl.so.4
#22 0x00007ffff7f0045a in ?? () from /usr/lib/libcurl.so.4
#23 0x00007ffff7efa69f in ?? () from /usr/lib/libcurl.so.4
#24 0x00007ffff7f35589 in ?? () from /usr/lib/libcurl.so.4
#25 0x00007ffff7f385ce in curl_multi_perform () from /usr/lib/libcurl.so.4
#26 0x00007ffff7f0d2a4 in curl_easy_perform () from /usr/lib/libcurl.so.4
#27 0x0000555555880758 in curl::easy::handler::Easy2<wasm_pack::manifest::Collector>::perform<wasm_pack::manifest::Collector> (self=0x7ffff7213ef8)
    at /home/jp/.cargo/registry/src/github.com-1ecc6299db9ec823/curl-0.4.44/src/easy/handler.rs:3163
#28 0x000055555575a759 in wasm_pack::manifest::Crate::check_wasm_pack_latest_version () at src/manifest/mod.rs:247
#29 0x000055555575a237 in wasm_pack::manifest::Crate::return_latest_wasm_pack_version () at src/manifest/mod.rs:220
#30 0x000055555575979e in wasm_pack::manifest::Crate::return_api_call_result (current_time=<error reading variable: Cannot access memory at address 0x0>) at src/manifest/mod.rs:170
#31 0x0000555555759632 in wasm_pack::manifest::Crate::return_wasm_pack_latest_version () at src/manifest/mod.rs:165
#32 0x0000555555747826 in wasm_pack::build::check_wasm_pack_versions () at src/build/mod.rs:65
#33 0x00005555557354df in wasm_pack::background_check_for_updates::{closure#0} () at src/main.rs:30

I have the same issue.  Here is journalctl trace:
wasm-pack build --target=web --dev --no-default-features --features=hydrate

Stack trace of thread 289095:
#0  0x00007f45a76a1d66 __pthread_rwlock_wrlock (libc.so.6 + 0x8bd66)
#1  0x00007f45a6fca0ee CRYPTO_THREAD_write_lock (libcrypto.so.3 + 0x1ca0ee)
#2  0x00007f45a7037ee5 n/a (libcrypto.so.3 + 0x237ee5)
#3  0x00007f45a7037f81 X509_STORE_add_cert (libcrypto.so.3 + 0x237f81)
#4  0x00007f45a701c540 X509_load_cert_crl_file_ex (libcrypto.so.3 + 0x21c540)
#5  0x00007f45a701c696 n/a (libcrypto.so.3 + 0x21c696)
#6  0x00007f45a7037a4a X509_STORE_load_file_ex (libcrypto.so.3 + 0x237a4a)
#7  0x00007f45a786d1ab n/a (libcurl.so.4 + 0x701ab)
#8  0x00007f45a786d58d n/a (libcurl.so.4 + 0x7058d)
#9  0x0000561f7b434376 n/a (/home/dev/.cargo/bin/wasm-pack + 0x267376)
ELF object binary architecture: AMD x86-64

My environment is also Arch Linux.

I was able to fix this by installing the musl toolchain.
rustup target add x86_64-unknown-linux-musl
On Arch, you must then sudo pacman -S musl
Then in wasm-pack: cargo install --path . --target=x86_64-unknown-linux-musl
Then you should be good to go.

Also on Arch Linux, I worked around the issue by rolling back the version:
cargo install<EMAIL_ADDRESS>


Allow services to be enabled/disabled directly from service list in settings

Feature Description
In order to quickly enable services which are currently disabled
As a user viewing the "Your Services" listing in Settings
I want to click the "power" icon to enable the service directly and not have to go into the service's settings page to use the toggle.
Motivation and Context
Currently, if disabled services are hidden from the sidebar, if I wish to enable a set of services, the workflow is:

Open Settings
Click "Your Services"
Scroll to the service you to be enabled
Click toggle to "Enable Service"
Click "Save Service"
Go to step 3 for each additional service

This can be annoying if you switch services on and off a lot (say, you use a set for "work" and a set for "home").
If the "power" icon were the toggle – for example, went from grey (disabled) to green (enabled) – users could save having to go through 2 extra steps per service.

I was actually going to request the exact same feature.  It would be great (and save clicks) to just be able to either right-click+enable or to use the power icon as a button, so that the service would be enabled from the service list.

@SamVanhoutte Yep, from a UX perspective the power icon "feels" like it should be clickable and should "do the right thing" and enable/disable the service. Then you click it and it takes you to the settings screen and feels "broken".


Parallel branches which "run after" multiple parallel actions render very awkwardly

Describe the Bug with repro steps

Add a trigger
Add an Initialize Variable action
Add a parallel branch from trigger w/ a new Initialize Variable action
Add another parallel branch from trigger w/ a new Initialize Variable action
Add a Current Time action on one of the branches
Add another Current Time action after the one in step 5
Set the Run After configuration for both Current Time actions to be after all 3 Initialize Variable actions succeed

What actually happens:

What I would expect:

What type of Logic App Is this happening in?
Consumption (Portal)
Which operating system are you using?
Windows
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
{
  "definition": {
    "$schema": "https://schema.management.azure.com/providers/Microsoft.Logic/schemas/2016-06-01/workflowdefinition.json#",
    "actions": {
      "Initialize_variable": {
        "type": "InitializeVariable",
        "inputs": {
          "variables": [
            {
              "name": "",
              "type": "boolean"
            }
          ]
        },
        "runAfter": {}
      },
      "Initialize_variable_1": {
        "type": "InitializeVariable",
        "inputs": {
          "variables": [
            {
              "name": "",
              "type": "boolean"
            }
          ]
        },
        "runAfter": {}
      },
      "Initialize_variable_2": {
        "type": "InitializeVariable",
        "inputs": {
          "variables": [
            {
              "name": "",
              "type": "boolean"
            }
          ]
        },
        "runAfter": {}
      },
      "Current_time": {
        "type": "Expression",
        "kind": "CurrentTime",
        "inputs": {},
        "runAfter": {
          "Initialize_variable_2": [
            "SUCCEEDED"
          ],
          "Initialize_variable_1": [
            "SUCCEEDED"
          ],
          "Initialize_variable": [
            "SUCCEEDED"
          ]
        }
      },
      "Current_time-copy": {
        "type": "Expression",
        "kind": "CurrentTime",
        "inputs": {},
        "runAfter": {
          "Initialize_variable_1": [
            "SUCCEEDED"
          ],
          "Initialize_variable": [
            "SUCCEEDED"
          ],
          "Initialize_variable_2": [
            "SUCCEEDED"
          ]
        }
      }
    },
    "contentVersion": "<IP_ADDRESS>",
    "outputs": {},
    "triggers": {
      "Recurrence": {
        "type": "Recurrence",
        "recurrence": {
          "interval": 3,
          "frequency": "Minute"
        }
      }
    }
  },
  "connectionReferences": {},
  "parameters": {}
}

Screenshots or Videos
No response
Browser

Browser: Chrome 128.0.6613.86 x64
OS: Windows 11

Additional context
No response

This sort of behavior is pretty tricky from just first impressions, and it also cuts out a couple places users might want to add nodes.
ie. if you want to add a node between a specific parent and a specific child that isn't possible with your example solution.
I do agree it renders very awkward/bad, but I'm not sure what a better option would be with this configuration of run afters.
I'm not sure this is a very common use-case as well so I'm not sure it's worth investigating for the amount of time/effort it would take.

I think there's bound to be some overlap, even if we changed the line shape there's still edge button/midpoint overlap just because of the layout of the nodes.

Closing this as it is expected behavior, maybe we can start a discussion instead but I'm not sure what can be done here.


Work on an simplified accesible version?

Hi! I recently published an adventure using this template. Along with the exported PDF i used the article class to export my content as a friendlier version for screen readers. But it also implied dropping the DndReadAloud DndSidebar DndComment \DndDropCapLine and of course DndMonster helpers.
I'm considering working on a fork that would render these in simplified ways for screen readers so that all you need to do is change class from dndbook to dndsimplified or even dnddyslexic so, is anyone working on such a thing? would anyone be interested?
The objective of course, is making it easier for we as writers to publish in accessible ways without the trouble of modifying .tex files with each export
Thanks!

Well done, and I am glad you added it to the list of products on our wiki!
I don't know what would be involved in making those functions more accessible to screen readers, but I am interested in hearing about it. Unfortunately I do not have time to help with it at this time. Once you have a firm grasp on what would be involved in this simpler version, we could look at what it would take to merge it into the main class.

Got it. I'll begin checking this as soon as I am able then!
I did had an unrelated question. Is there a preferred way to include the project on credits?

Hm, I had not given that a lot of thought. You might want to take a look at a colophon. O'Reilly reference books have them, and they talk about what tools were used to make the book, what the animal is on the cover, etc.
https://en.wikipedia.org/wiki/Colophon_(publishing)
I don't even do it for my products, but I was thinking about adding it for the hardcover omnibus.

hey there! I'm also interested in figuring out accessibility.  I'm writing one-shot adventures in LaTeX and want them to be used by as many people as possible, including visually impaired GMs. Two notes on this subject:

I tried creating a High Contrast "night mode" version of the package, but I ran into a snag. After copying the folder and re-naming the files, pretty much everything works except the font didn't stick. Any ideas? I can create a separate issue for that if that's best.

Accessibility in LaTeX is a fickle thing. As far as I found, there isn't a solid solution to adding tags for PDF documents for screen readers to use. Many people end up tagging PDFs in a premium version of Adobe after exporting. The best I have found so far is the tagpdf package (https://github.com/u-fischer/tagpdf) but isn't perfect, and adding tags and structure flags in the middle of things like a DND style table sometimes throws errors.

HI! I'm still interested in this. I'm soon to publish my next adventure, I just haven't had time to get around to it (and my tex knowledge itself is limited)
I expect to at least, make some proof of concept during the christmas break.


fints-cli does not work with winston 3.2.1

When trying to run fints-cli with winston 3.2.1 installed it fails because of this error:
TypeError: self._addDefaultMeta is not a function
    at Object.DerivedLogger.(anonymous function) [as verbose] (.../fints/node_modules/winston/lib/winston/create-logger.js:80:14)
    at HttpConnection.<anonymous> (.../fints/dist/http-connection.js:38:22)
    at Generator.next (<anonymous>)
    at .../fints/dist/http-connection.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (.../fints/dist/http-connection.js:3:12)
    at HttpConnection.send (.../fints/dist/http-connection.js:36:16)
    at Dialog.<anonymous> (.../fints/dist/dialog.js:132:52)
    at Generator.next (<anonymous>)
    at .../fints/dist/dialog.js:7:71

It seems this is a Problem in winston that will not get fixed:
https://github.com/winstonjs/winston/issues/1591

Do you jave any clue how to fix this?

@Prior99 This is the only way I could make it run for me https://github.com/x-clone/fints

Getting the same problem. I just did a fresh install of fints-cll inside a docker node container (npm install -g fints-cli), and any command produces this error for me as well
TypeError: self._addDefaultMeta is not a function
    at Object.DerivedLogger.(anonymous function) [as verbose] (/usr/local/lib/node_modules/fints-cli/node_modules/winston/lib/winston/create-logger.js:80:14)
    at HttpConnection.<anonymous> (/usr/local/lib/node_modules/fints-cli/node_modules/fints/dist/http-connection.js:38:22)
    at Generator.next (<anonymous>)
    at /usr/local/lib/node_modules/fints-cli/node_modules/fints/dist/http-connection.js:7:71
    at new Promise (<anonymous>)
    at __awaiter (/usr/local/lib/node_modules/fints-cli/node_modules/fints/dist/http-connection.js:3:12)
    at HttpConnection.send (/usr/local/lib/node_modules/fints-cli/node_modules/fints/dist/http-connection.js:36:16)
    at Dialog.<anonymous> (/usr/local/lib/node_modules/fints-cli/node_modules/fints/dist/dialog.js:132:52)
    at Generator.next (<anonymous>)
    at /usr/local/lib/node_modules/fints-cli/node_modules/fints/dist/dialog.js:7:71

This package is using yarn, so npm install will lead to an incompatible version of winston to be installed. I also find it confusing how a minor upgrade of winston is breaking. I fixed this by changing the way the functions for logging are exported and publish the new versions.


Implement LSP

Do you plan on continue working on that?

I'm currently inclined to extending https://github.com/georgewfraser/java-language-server (my fork is at https://github.com/draftcode/java-language-server with some patches). After I changed my job and the codebase that I work on gets much smaller, this java-language-server is working in a reasonable time. There are still some missing pieces, but my daily use has been shifted to that fork.


(Question) Can the data be used to store in one's own Database?

More of a question really...
I tried looking through their terms of service, but I'm sure it wouldn't be an issue to get the Data and store it into ones own database correct?
I just need a few things about all the players to get something going. Can't imagine that'd be an issue would it?
Sorry to bother you!

Hi,
No I don't think that would be an issue.
From section 7:
"You may access, use, and display the Services and print copies of the NHL Content only for non-commercial, informational, personal use, without modification or alteration in any way, and only so long as you comply with these Terms...
"
Regards

Hi again,
I noticed that there now is a non-documented REST API for Stats at nhl.com. Because this is much better than scraping the page, I created  a new version 4.0 that is simply a wrapper around the API at nhl.com. Much more reliable.
If you like you can of course go to the API directly, e.g.
http://www.nhl.com/stats/rest/grouped/goalies/season/goaliesummary?cayenneExp=seasonId=20142015 and gameTypeId=3
Peter

Oh wow, well look at that!
Thank you @peterstark72 I appreciate it!
That's definitely better/easier than scraping :)


is 0.7.1h that bad

What is your question?
i never seen any mods using 0.7 and above, and alot still use 0.6.x, is 0.7 bad? because i never seen any mod use it yet like i said so i want to know if 0.7 and above is good or bad because idk

migrating to 0.7 from 0.6 is difficult and can be a rocky procress

hscript is almost completely broken: scripts can't use runHaxeCode more than once, for loops don't work, vars don't carry over to other runHaxeCode instances without setVar/getVar, vars could be used without being declared before sscript

3 more reasons and ill close this

I had the time to fix the errors of 0.7.1h with the latest version of the libraries and Haxe.
Now it works correctly.
Soon I will send a repository of each fixed library

I had the time to fix the errors of 0.7.1h with the latest version of the libraries and Haxe. Now it works correctly. Soon I will send a repository of each fixed library

is it the libraries/versions one?

nvm ima just close this


Error: EISDIR keeps on appearing when using downloadDir

I'm using the downloadDir method and every once in a while, for no apparent reason but always on the same directories, I am getting this error: Error: EISDIR, open '/home/imoses/resources/index/FE filters/'.
I tried debugging it but I had no success.
Apparently graceful-fs tries to open a write stream on a directory path and I can't seem to understand why.
This is my code snippet.
    s3client.downloadDir({
        localDir: path.join(process.env.HOME, 'resources'),
        s3Params: {Bucket: '1k-resources', Prefix: 'db/'}
    }).on('end', cb)

Any help would be appreciated.
Running on:
NodeJS v0.10.25
Ubuntu 14.10

Just in case anyone is interested in this, I ended up using getS3Params to check which path is a directory. In case of a file I execute the callback, otherwise I create the directory WITHOUT execute the callback to avoid this issue.
getS3Params: function(localFile, s3Object, callback) {
    if (path.extname(localFile) === '') { mkdirp.sync(localFile); }
    else { callback(null, {}); }
}

This check works for me since I have no files without an extension, but it can be easily modified to accept such cases as well.

I had similar problem. For most of the buckets everything was working fine, but couple of them were failing with the EISDIR error code. I was able to fix it by setting "maxAsyncS3=1".

@iMoses solution did not work for me. I don't like the idea of never calling callbacks in general, so I also tried a version where I call the callback after mkdirp, but with null, null which AFAIK should skip the download.  Still no luck.
@jujokini solution did work, but of course this means a huge reduction in speed.

@jujokini @bobzoller I can confirm that. It works, but the speed reduction is really a downside.
@iMoses I use "/" as delimiter for directories, and from the console output I saw that every directory had the "/" at the end. So I use the getS3Params to only skip the "download" of directories. That solved it for me.
getS3Params: function(localFile, s3Object, callback) {
	console.log("getS3params for: ", localFile);
	if (endsWith(localFile, "/")) {
		callback(null, null); // Skip folders
	} else {
		callback(null, {});
	}
}

function endsWith(str, suffix) {
	return str.indexOf(suffix, str.length - suffix.length) !== -1;
}

@netzwerch do you think the code snippets you posted would go well in the new https://github.com/andrewrk/node-s3-client#examples section?

@faceleg Yes, it can definitely help people to work around this problem.
I assume that the error appears when creating a local writeStream on a "/some/path/" s3-key string. In the local filesystem (non-windows) this will be local folder name, so opening a stream on that causes the error. So maybe it could also be fixed inside of node-s3-client. But then again, why doesn't this happen with maxAsyncS3=1?

my assumption was it was a race condition -- that node-s3-client does the right thing with directory-looking-s3-keys like foo/bar/, but it also handles creating missing directories for file-looking-s3-keys like foo/bar/baz.  if foo/bar/baz completes before foo/bar/, the latter blows up because the directory already exists.

I get the same error, I have tried with this: maxAsyncS3=1, and resolve the problem, but the speed decrease. I have tried with @netzwerch solutions but do the same error. Any other help?

I forgot the "/" at the end of my path, and with the @netzwerch solutions, it works. Thanks


DVN: offchain and contract

Adds the DVN, both its contract and offchain worker.
New Behavior
Adds a couple of things:

A workers/ folder, with a binary to run the DVN offchain worker. Listens to emited events and verifies if the DVN is required.
The folder workers/ is intended as a shared space for the executor (done by @Fly-Style).
The DVN contracts are inside contracts/evm/src/dvn/. This would be the place where I would ask for a more intensive review, since I don't know enough solidity to do something good enough.

Breaking Changes
No observed breaking changes.
Observations

There's some mock data in an .env file to load some credentials for infura, and the addresses of the endpoints and ULN contracts. For now, it uses arbitrum mainnet and a DVN address that appears often, but it will be refactored to add the DVN's addresses in the corresponding chains.
Currently, there's no connectivity with NFFL. Each packet that is requested to be verified, will be verified; but this won't be after connectivity with NFFL is added.
Comments from the PR in the other repo have been addressed here.

@coderabbitai review


ferm syntax

Please include ferm syntax.
https://github.com/cometsong/ferm.vim
https://github.com/MaxKellermann/ferm
http://ferm.foo-projects.org/
https://github.com/cometsong/ferm.vim/issues/3

Please ask each author to send the files to Bram.
Explicit permission from each author is required.

@k-takata - I should send these ferm.vim files to Bram's email directly?
Or send the repo url to 'brammool' git user?

You can send via email or create a PR.


fix: errors related to max prompt size

What this PR does

Show error message when max_tokens is set too high

Show error message when the start prompt has too many tokens

Reduces the parameter max context lines if the amount of tokens exceeds the maximum and send a warning message

Show error message if the prompt size exceeds the maximum even reducing the max_context_lines

Create tokenize function for tests

How it was tested

All automatic tests pass
Streamlit demo works
I've manually checked different configuration and inputs

PR checklist

[X] No API keys or other secrets committed to source?
[X] New functionality is added alongside appropriate tests?
[X]  All source files have the following header?

# Copyright (c) {YEAR} Cohere Inc. and its affiliates.
#
# Licensed under the MIT License (the "License");
# you may not use this file except in compliance with the License.
#
# You may obtain a copy of the License in the LICENSE file at the top
# level of this repository.

Oh actually I think I have one more idea on how we could make one part of this more Pythonic where we loop through the prompt length history; I'll add a specific suggestion soon 😁

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 5 committers have signed the CLA.:white_check_mark: yichern:x: marinamachado:x: lusmoura:x: egrefen:x: madelinehjenkinsYou have signed the CLA already but the status is still pending? Let us recheck it.

@marinamachado I think you're good to re-request review from @yichern whenever you think the PR is ready, and then if he approves you'll be good to merge :)
(No rush on this tho!)


Public GitHub Pages?

Brief Intro

Public GitHub Pages?
More Details

@kelvinlauKL @vincentngo I wonder whether we can let everybody knows the web version on GitHub Pages under the repository description.

Will do. Thanks for the reminder. On the next SAC update blog this month I’ll make a mention.
Thanks!

Cool.


update main README with missing links to components readmes

Closed via: https://github.com/ciena-frost/ember-frost-core/pull/267


Fix typos in chapter2.md

There was a missing space and a missing arrow which are now fixed.

Looks good. Thanks!


Improve Python script to use updated Pulls API to get linked issues

Regarding this comment on https://github.com/OpenRefine/OpenRefine/blob/3df47341164177f6c39891c7b7ab49e5f1261cea/.github/workflows/label_transfer/script.py#L36
Link Relations can now be gathered with the Pulls API from a PR as documented here:
https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#link-relations
Evidentally it looks something like this down in the response?
 "_links": {
      "self": {
        "href": "https://api.github.com/repos/octocat/Hello-World/pulls/1347"
      },
      "html": {
        "href": "https://github.com/octocat/Hello-World/pull/1347"
      },
      "issue": {
        "href": "https://api.github.com/repos/octocat/Hello-World/issues/1347"
      },
      "comments": {
        "href": "https://api.github.com/repos/octocat/Hello-World/issues/1347/comments"
      },
      "review_comments": {
        "href": "https://api.github.com/repos/octocat/Hello-World/pulls/1347/comments"
      },
      "review_comment": {
        "href": "https://api.github.com/repos/octocat/Hello-World/pulls/comments{/number}"
      },
      "commits": {
        "href": "https://api.github.com/repos/octocat/Hello-World/pulls/1347/commits"
      },
      "statuses": {
        "href": "https://api.github.com/repos/octocat/Hello-World/statuses/6dcb09b5b57875f334f61aebed695e2e4193db5e"
      }
    },
    ```

No, that "link relations" field seems to be unrelated to the "linked issues" feature.
But apparently the feature has been added to the API in another way:
https://stackoverflow.com/questions/60717142/getting-linked-issues-and-projects-associated-with-a-pull-request-form-github-ap

Ah not using the REST API but instead their GraphQL API which seems much more powerful indeed.  https://docs.github.com/en/graphql

Removing the "good first issue" label because although it should indeed be relatively simple, it is not about OpenRefine itself and is harder to test for a new contributor.


Alterntive redis fix

We ran this for about 2 weeks without issue. It keeps the group and works completely with whats there currently.
This may be the "optimized" the version you were talking about.
Its major drawback is that is only works correctly in the default set up for redis. the other PR correctly fixes this for all options in redis.

Looks OK. What do you mean by default set up and other options?

Redis out of the box will only consider items with a TTL > 0 for LRU evictions. This works as the group has no TTL. Other modes make redis consider all keys (more similar to memcache)
In the other mode which is more similar to memcache the group can be removed resulting in code which expects objects or arrays to actually get serialised strings. This shows in logs as can not call method x on string.
This is the reason the group needs to go as it can be expired separate from the data.

Well, we'll have to live with that for now. :)

Eh, I just noticed that you're again targeting 3.1-stable instead of develop ... Is this BC-free?

This is BC. It works identically to the current menthod but instead of front loading the group it checks on each use. It's o(1) (and fast) in redis.
The only reason I stopped using this is the issue mentioned above. We dont run redis with the default options