[9.x] Removed class exists check for make:migration command

Since Laravel 9.x we are using anonymous classes for migrations, thus this check can be removed.
And it is also a nice-to-have for other Laravel projects that migrated to Laravel 9.x:

We can quickly create a migration file with a simple name (eg php artisan make:migration UpdateUsersTable) even the class already exists.
We don't have to think for a long long name, just create the migration and start the development quick.

This improvement can be considered as bug fixing.

There are 3rd party package which are still using class names in migrations.

@ankurk91 would be out-of-scope of this improvement PR since I didn't touch anything related to the migration core, only the make:migration command 😄

How does this affect older projects that were upgraded to Laravel 9 and don't (exclusively) use anonymous migration classes?

How does this affect older projects that were upgraded to Laravel 9 and don't (exclusively) use anonymous migration classes?

Since Laravel 9, make:migration will use anonymous classes (you can check the stub files)
But if you intend to change the your new migration class back to the old way then I guess it isn't recommended to do to so?

This will break all existing apps that still use non-anonymous classes. We specifically let this in to allow for an easy upgrade path.

This will break all existing apps that still use non-anonymous classes. We specifically let this in to allow for an easy upgrade path.

@driesvints yes but the thing is, make:migration will create an anon class so the check is not relevant at the creation time?
I mean after the creation, people can change it back to the old way and if they did, that's another story and it is their fault - make:migration here did nothing wrong, did it?
Additional point: I already had the UpdateUsersTable class, now I want to hit the php artisan make:migration UpdateUsersTable again => class existed => can't create a migration (even anon class migration) so it is like a small blocker/pain here?


I Can't Understand What Is Order Here. Could You Explain It In A Simple Way?

https://github.com/Leaflet/Leaflet/blob/00e0534cd9aa723d10a652146311efd9ce990b46/src/layer/vector/Canvas.js#L263

It's an iterator.


improved table list filter

Please also remove the now unused model "tableName" in tablelist controller.

After that: Ack ;)


DOCS-5515 Adds Create a Log Integration Documentation

What does this PR do?

Adds a previous FAQ as a new doc page in the Integrations section of the Developers doc set.
Motivation

DOCS-5515
Additional Notes

Preview
https://docs-staging.datadoghq.com/alai97/log-integration-guide-to-page/developers/integrations/log_integration/
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Created DOCS-5564 for the docs editorial review.


Highlight create here text in login activity and sign in here text in sign up activity

Issues solve #50
Description
Highlight create here text in login activity and sign in here text in sign up activity
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Thanks @amitthecoder , LGTM.


Fix get_scale().

Scale is applied before rotation in Basis since #7438. Basis::get_scale() is one of the functions that wasn't correctly updated back then.
Let's see if this is the last of them :(

Hold on, sorry, after double-checking it, #7438 didn't miss it, actually it's correct. Basis is split as B = R.S, in which case you'd need to check the length of columns, not rows.
I was misled to think that way after seeing a piece of code in bullet physics PR.


DEV: Add tests for onebox rendering within [chat]

This is a companion commit to https://github.com/discourse/discourse/pull/16050
to add tests for the fix there, which was that we weren't hoisting
the html_raw tokens from the custom markdown engine properly, which
caused issues with onebox rendering.

This will not pass until https://github.com/discourse/discourse/pull/16050 is merged


Fix HtmlString

See #32286, #32285

As I said before, the constructor insists on you passing a string. If you do not pass a string, then the behaviour is formally undefined.

You can use static analysis tooling to make sure you are not calling this incorrectly. Laravel does not perform a typecheck at runtime here for performance reasons.

@GrahamCampbell Converting to string is done in several places, ie:
https://github.com/laravel/framework/blob/ff1a13ecf09d978575148c5095b87fa66187aad6/src/Illuminate/View/ComponentAttributeBag.php#L147-L166

That is totally different. That is getting the string representation of an htmlstring.

It is not that class implements the Illuminate\Contracts\Support\Htmlable contract just like HtmlString does and it needs to return a string in all circumstances.

Only in the circumstances where it is constructed correctly. Otherwise it has undefined behaviour.

Secondly see this part of code of the Illuminate\Support\Stringable class
https://github.com/laravel/framework/blob/ff1a13ecf09d978575148c5095b87fa66187aad6/src/Illuminate/Support/Stringable.php#L25-L28
It accepts a default value of an empty string. It casts $value to string.

Only in the circumstances where it is constructed correctly. Otherwise it has undefined behaviour.

You dont know if it is constructed correctly.

You are not allowed top pass any value. This is always the convention when code does not have type checks. The implementer trusts that you call it correctly.

You should have type checks in my opinion, that's another debate ;-)


Fix the 'Unresolved Symbols' error caused by commit 21e692b

The wke\wkeGlobalVar.cpp for global variables is not added into the VS2015 project files.
The namespace is missing for g_isDecodeUrlRequest in WebURLLoaderManager.cpp file.

Thanks，I‘ll recommit again

OK. I'll just close this PR and wait for the updates.

There is a compilation error in 'Release' configuration.  I submitted a fix in my branch. https://github.com/fantasydr/miniblink49/commit/ebac8d4a0fda0bbc17c3e7331673457af41a590e
The lib is much faster in 'Release' mode and it works as expected. Thanks for providing the source code, it helps me lot 👍
PS, since the owner prefer to recommit rather than merge, I didn't create new PR but only leave a comment instead.

Hi， I ‘m ’curious to know what is your product use?

I didn't use it in any product. I'm trying to understand how to reduce the size of chromium.


Telegraph::chatAction ????

chatAction method not exists!
Thank you

Telegraph::chatAction(ChatActions::TYPING)->send();


Profile picture is not loading in Profile section

profile picture is not loading while it is updated.

Profile picture is loading now. This issue is no more.


Add Kyiv National Linguistics University

https://knlu.edu.ua/en/
Kyiv National Linguistic University is the successor of the Kyiv State Pedagogical Institute of Foreign Languages, established by the Ministry of Education of the USSR on March 30, 1948. By decree of the Cabinet of Ministers of Ukraine of August 29, 1994, the Kyiv State Pedagogical Institute of Foreign Languages was renamed Kyiv State Linguistic University. Since its founding, the university has made significant achievements in the field of foreign linguistics.

@daandriichuk There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.


Use exec on Unix to propagate signals

This is only possible when telemetry is disabled, but almost all of the time it
is! This should help fix lots of common rustup-related issues related to signals
and such.
Closes #31
Closes #806

@bors: r+

:pushpin: Commit e13247f has been approved by alexcrichton

:hourglass: Testing commit e13247f01d34a206eed4594c28cc17d82ffa9894 with merge 9f70345c9cce966c39e6e000093296c9e8e7a62d...

:broken_heart: Test failed - status-appveyor

:pushpin: Commit 394b6e4 has been approved by alexcrichton

@bors: r+

:hourglass: Testing commit 394b6e435a393c2537683730a7a5367b30c358eb with merge b33d20a91dc189d99f3e21e3a4762d07c57801da...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing b33d20a91dc189d99f3e21e3a4762d07c57801da to master...

Oh yay thank you!!


Run all tests of the currently opened file

Create a command that opens vs-code terminal and run a mix test "path-of-the-current-file" on the root folder

I can do this one!


Enhanced the configuration of weekly-report component

What is the purpose of the change

After the weekly-report component generates a report, it will create an issue and send it to slack at the same time. The component is not flexible enough.
Brief changelog

Change weekly-report configuration, add receiver field. There are two types of receivers, issue, and slack by default. Users can configure to send to slack or issue or send to both slack and issue.

/approve


Updated renovate config to update the following:

NODE_VERSION and NPM_VERSION baked into Dockerfiles using ENVs or ARGs
Update balena blocks images
Change-type: patch
Signed-off-by: Olamide Omolola<EMAIL_ADDRESS>

@ab77, please can you review and certify?

please can you review and certify?

@olamiko could you please rebase it first. There should be only one commit.


Feature/CCE 361

Getting crazy

 All committers of the pull request should sign our Contributor License Agreement in order to get your pull request merged.1 out of 2 committers have signed the CLA.:white_check_mark: saadmhybris:x: Jens RusitschkaJens Rusitschka seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account ([for further information, click here] (https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/#commits-are-not-linked-to-any-user)).


Improve facetview UI

facetview has been demonstrated displaying a collection. This allows for embed of a collection search in any web page. Imrpove the UI on this, and test on multiple browsers.

Is this local facetview or https://github.com/okfn/facetview

Improve in the repo and port into bibserver

INVALID. See FacetView repo. Embedding support in BibServer should be a separate issue.


Include basic column statistics in column information view

(Please excuse me if this feature request has already been discussed)
When exploring new databases and performing basic content checks I regularly evaluate database columns for "completeness" (how many fields per column are non-null/non-empty) and the most frequent values (usually the most frequent 3 values).
The background for this inspection is that many databases schemas have unused or overlooked fields that are intended to be populated with meaningful values but are neglected by users so that they will largely contain default or NULL values. Performing such a simple analysis requires some dynamic programming (e.g. adjusting frequnency counts for column names and data types) which feels like repeating a lot of work that has already been done under the hood of dbeaver.
The table/column information view quite already provides basic schema data, and lists data types, constraints, comments and others. It would be great to have a mechanism to extend this view with column statistics, i.e. trigger a computation of the share of non-null field values or the computation of the most frequent values.

Thank you for your suggestion. What database do you use?

I usually work with Postgresql, SQLite, MS SQL Server, Oracle, and (unfortunately) MS Access databases

Different databases have different system metadata columns, many of them do not have table/column statistics at all. So we will need to do it on client-side. Although it can be quite expensive.
We can make new Tool, like "Calculate column statistics".

Most frequent values can be calculated by simple GROUP BY+ORDER.
Null/Not Null is also simple.

@matthias-mueller Do you have any other statistic parameters on mind?

@serge-rider I think that's all I had in mind. Two remarks:

It probably makes sense to add a LIMIT parameter to the "most frequent values" query
The frequencies (absolute, relative in %) of each distinct value category (i.e. null,not null, and each of the most frequent values) are somewhat essential


TextEditor.displayBuffer is deprecated.

TextEditor.prototype.displayBuffer has always been private, but now
it is gone. Reading the displayBuffer property now returns a reference
to the containing TextEditor, which now provides some of the API of
the defunct DisplayBuffer class.
TextEditor.displayBuffer (<embedded>:6:61)
<unknown> (C:\Users\Wanty\.atom\packages\fold-comments\lib\fold-comments.coffee:32:27)
Workspace.observeTextEditors (<embedded>:11:428905)
Object.activate (C:\Users\Wanty\.atom\packages\fold-comments\lib\fold-comments.coffee:30:31)
Package.activateNow (<embedded>:11:3288314)
<unknown> (<embedded>:14:1879)

Dupe #27


disasm: Names for Tuple Components are not set

Problem
When inferring ABIs from bytecode, WhatsABI currently omits names for tuple components that are likely derived from Solidity structs. While the ABI specification technically allows unnamed tuple components, Solidity always generates names for struct members when creating the ABI. This discrepancy causes compatibility issues with libraries like go-ethereum that expect these names to be present when working with structs (See https://github.com/ethereum/go-ethereum/blob/v1.14.11/accounts/abi/type.go#L181).
Example contract:
// SPDX-License-Identifier: MIT

pragma solidity ^0.8.0;

contract TestC {

    struct MyTuple {
        uint256 num;
        string str;
        bool flag;
    }

    function getTuple() external pure returns (MyTuple memory) {
        return MyTuple(123, "hello", true);
    }

    function setTuple(MyTuple calldata inputTuple) external pure returns (uint256, string memory, bool) {
        return (inputTuple.num, inputTuple.str, inputTuple.flag);
    }
}

ABI Generated by Solidity Compiler:
{
  "contracts": {
    "test.sol:TestC": {
      "abi": [
        {
          "inputs": [],
          "name": "getTuple",
          "outputs": [
            {
              "components": [
                {
                  "internalType": "uint256",
                  "name": "num",
                  "type": "uint256"
                },
                {
                  "internalType": "string",
                  "name": "str",
                  "type": "string"
                },
                {
                  "internalType": "bool",
                  "name": "flag",
                  "type": "bool"
                }
              ],
              "internalType": "struct TestC.MyTuple",
              "name": "",
              "type": "tuple"
            }
          ],
          "stateMutability": "pure",
          "type": "function"
        },
        {
          "inputs": [
            {
              "components": [
                {
                  "internalType": "uint256",
                  "name": "num",
                  "type": "uint256"
                },
                {
                  "internalType": "string",
                  "name": "str",
                  "type": "string"
                },
                {
                  "internalType": "bool",
                  "name": "flag",
                  "type": "bool"
                }
              ],
              "internalType": "struct TestC.MyTuple",
              "name": "inputTuple",
              "type": "tuple"
            }
          ],
          "name": "setTuple",
          "outputs": [
            {
              "internalType": "uint256",
              "name": "",
              "type": "uint256"
            },
            {
              "internalType": "string",
              "name": "",
              "type": "string"
            },
            {
              "internalType": "bool",
              "name": "",
              "type": "bool"
            }
          ],
          "stateMutability": "pure",
          "type": "function"
        }
      ]
    }
  },
  "version": "0.8.25+commit.b61c2a91.Darwin.appleclang"
}

Proposed solution
~Fill in the component names with _n when they're empty.~
Fill in the component names with field${n} when they're empty.
Example:
{
   "type": "function",
   "selector": "0x95d376d7",
   "payable": false,
   "stateMutability": "payable",
   "inputs": [
     {
       "type": "tuple",
       "name": "",
       "components": [
         { "type": "uint32", "name": "" },
         { "type": "bytes", "name": "" },
         { "type": "bytes32", "name": "" },
         { "type": "uint64", "name": "" },
         { "type": "address", "name": "" }
       ]
     },
     { "type": "bytes", "name": "" }
   ],
   "sig": "assignJob((uint32,bytes,bytes32,uint64,address),bytes)",
   "name": "assignJob",
   "constant": false
}

To be:
{
  "type": "function",
  "selector": "0x95d376d7",
  "payable": false,
  "stateMutability": "payable",
  "inputs": [
    {
      "type": "tuple",
      "name": "",
      "components": [
        { "type": "uint32", "name": "field0" },
        { "type": "bytes", "name": "field1" },
        { "type": "bytes32", "name": "field2" },
        { "type": "uint64", "name": "field3" },
        { "type": "address", "name": "field4" }
      ]
    },
    { "type": "bytes", "name": "" }
  ],
  "sig": "assignJob((uint32,bytes,bytes32,uint64,address),bytes)",
  "name": "assignJob",
  "constant": false
}

EDIT: Adjusted the placeholder format that should be supported by geth.

HMMM! That is annoying behaviour on go-ethereum's part!
I'm generally hesitant to add made up results that WhatsABI did not actually extract. It would make it harder to distinguish if the ABI actually has a field named _0 vs the disassembler just assigning it that name.
I guess some options:

Add a helper for augmenting any ABI with missing names into something go-ethereum-friendly.
Add an optional field to autoload to use helper from #1. Not sure if this should be the default? Technically a breaking change, I guess.
Just do it by default and abstract away from the user.

What do you think?

@shazow I understand your concern; that makes a lot of sense. From a user’s perspective, I think having a Geth-friendly ABI generated by default might be preferable.

@yohamta I'm hesitant to make it default but I'll consider it. Would you be interested in contributing this? Could start with a standalone helper like abiFillEmptyNames that takes an ABI and fills in any empty names, then we can figure out where to insert it for autoload?

That sounds good to me! I'll open a draft PR later so we can discuss further.

@yohamta Do you have a deployed contract/whatsabi code to reproduce this issue by the way?

@shazow Yes, here is the contract that reproduces the issue:
https://mantlescan.xyz/address/0x806feB7478b61bf068aA12C9413C6Ad7DD12232a


Trying to pull linux image for windows host and container

Error output
build_windows            starting local_image_windows:latest (build)Error: Command failed with exit code 1: docker create --volume=gcl-build_windows-955953-build:/gcl-builds --volume=gcl-build_windows-955953-tmp:/tmp/gitlab-ci-local-file-variables-op-thz-dev-thzcam-ino_cameraapi-955953 docker.io/firecow/gitlab-ci-local-util bash -c chown 0:0 -R /gcl-builds/ && chmod a+rw -R /gcl-builds/ && chmod a+rw -R /tmp/
Unable to find image 'firecow/gitlab-ci-local-util:latest' locally
latest: Pulling from firecow/gitlab-ci-local-util
image operating system "linux" cannot be used on this platform
    at makeError (C:\snapshot\firecow-gitlab-ci-local\node_modules\execa\lib\error.js:60:11)
    at handlePromise (C:\snapshot\firecow-gitlab-ci-local\node_modules\execa\index.js:118:26)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Job.start (C:\snapshot\firecow-gitlab-ci-local\src\job.ts:355:43)

Minimal .gitlab-ci.yml illustrating the issue
---
job:
  image: local_image_windows:latest
  script:
    - echo "Heya"

Expected behavior
Build is started
Host information
Windows 10
gitlab-ci-local 4.37.0
Additional context
I am a docker newbie, so it may be an issue on my side
my docker is set to run windows container, so it cannot pull your linux image, and if I switch to linux container, first step is OK, but then it cannot use the windows container to actually build

GCL doesn't support docker in "windows" image mode.
You can create a feature request or rename this issue.


Update dexscreener/pairs query

The pairs query was not setting the pairId correctly. Here's the new query:
pairs.sql.zip

Updated query with blockchainType and exchangeId set
pairs_v2.sql.zip


Add field constraints in content page

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


Recomputation fixes with native fp8

Change fp8 cast to be inplace so that its reflected in original fp8 tensor, not Float8 copy
Change so that fp8 and fp8_t cast occurs in the forward-forward pass when doing recomputation

Fixes #539

/te-ci pytorch


Moderator feedback to users

Summary
Discussed the possibility of users "spamming" moderators by clicking edit on a rejected story and then saving the form which counts as an edit but has made no change to the story.
Proposed solution/prevention. Allow moderators to provide specific feedback to the users about why a story was rejected. For example "Sentence starting 'The quick brown fox...' contravenes rule 1.23 in the code of conduct"
Technical implementation: Messaging system needs to be implemented? Add a comment under the story when being viewed or edited?
Moderation implementation: Rules and example responses to be developed
Deadline
June 2023
What needs to be done?

[ ] Decide on best way to communicate with story writers
[ ] Decide rules to limit "personal" interactions between story writers and moderators
[ ] Expand ExperienceHistory or make similar for writer-viewable feedback

Who can help?

@GeorgiaHCA

Updates
4th May 2023
Some selected relevant comments from 4th May meet up (link to Google doc)

Suggested change is to add a comment to the author - e.g. moderate language in first paragraph
[with current implementation] It is hard to distinguish which comment is going to author, and which is being made private
Can we possibly have text boxes asking specifically for the moderator to fill out, including the part of the CoC which is contravened and what part of the experience contravenes it (can be multiple);
Might also be nice if there was a link to the COC in the “reply”

Shall we merge #494 as is and use that as the backbone to improve upon based on today's meeting?
I feel like the minimal change would just be to change the labels to specifically instruct moderators to use the new textbox to write:

which sections of the experience are problematic and,
which parts of the CoC are not followed by them (with link to it)?

I guess the question I'd been keen to know the answer to is whether the structure that was discussed during the meeting will ultimately need to be reflected in the database structure. For example, at the moment it's just a single text field for the reply, but maybe it should be a list of CoC references + strings (or CoC refs + "(start, end)" indices into the story)?
If the plan is ultimately to go with this more structured approach, then it would likely need a new table in the database, and merging the current changes might end up causing more mess than benefit.
If we have a clear idea about the end-goal, then I'd personally favour getting the database structure/models corect now, but refining the user interface gradually. If we don't really have that clarity, then I'd go with the changes you suggest @gedankenstuecke. These are just my thoughts on the matter.

I think that's a very good point @llewelld! I guess it also depends a bit on what we want to use the answers for?
I guess in the current form I would envision those replies being sent through the oh_member.send_message() interface to inform people about the reply/moderation being done on their experience. In that case ultimately the responses would end up being purely text again. If we wanted to also be able to do something like statistical analyses or automate responses based etc then a more structured approach would be necessary.
The main issue I see with the structured approach is that it requires a lot of thinking about potential edge cases. E.g. what happens if one section of an experience breaks multiple sections of our CoC? What if there's multiple sections that break different sections etc? It quickly would become a matter of effectively becoming a "leaving comments on a Google Doc"-type of thing.
I guess by just storing the "final" text in the DB/model we still remain flexible with respect to that if we forgo deeper (automated) analyses of these rejections/comments, as we can tweak the form and form-validation to still have a deeper/complex system if we want, as long as we can concatenate it into a string at the end?

Thank you @gedankenstuecke for flagging this issue! And I'm so excited that the project is looking into operationalising these decisions!! So amazing ☀☀☀
Before I went on maternity leave I had always advocated for communications to happen through a different platform. A ticketing system like Jira, or even just email.
The reason being that I don't actually think the edge cases will be that common at the start and I was always focused on getting us going. The implementation of how it would be stored in the database was - in my mind - not part of an MVP.
Has the thinking moved on since then? What would not implementing this change mean?
We WOULD need a process document for how the emails were sent on a consistent basis etc. So this thinking is absolutely valuable, it's just whether comms happen through the app or not is the open question in my mind 💌

Thanks @KirstieJane!
One big challenge with the email or Jira approach would be that people need to re-identify themselves outside AutSPACEs to use those, while AutSPACEs has the deidentified communication baked in through the OH API as we can send messages through the API to update the participant on moderation decisions.
The current MVP implementation that @llewelld already made in #494 offers storing the moderator-responses as free-text in the AutSPACEs backend and could quite easily be expanded for the "send a message through OH".
But I would agree on maybe not worrying too much about the edge cases. I think merging the existing work with minor front-end tweaks as outlined above might be a good way to have a record of moderation decisions and way for moderators to give feedback!

I've made some changes to PR #494. It's really a prototype to see what you think. The changes make the moderator's reply more structured, but the end result does just end up in the same text field as you suggested @gedankenstuecke (stored as a JSON string). I agree keeping it as a text field in the database allows for more flexibility for the future.
In retrospect I should have gone with your judgement @gedankenstuecke and merged the previous version in first. I'm afraid I was too focused on making these additional changes.

Wow! First of all let me say that this is a really cool use of JS to make the selection, have the hover with the dropdown appear etc and I'll definitely have a closer look at how you did that in case I ever want to do something similar! 😄
I fear I have a few concerns about how practical it would be for the moderation approach once we'd move beyond prototype stage:

We would have to encode all content moderation rules into the dropdown approach. The question would then be how granular would we implement those? High-level rules or more granular approaches?
If we go with very granular approaches, how unwieldy will the dropdown become?
If we ever decide to adapt/change the moderation rules we will have to adapt the dropdown items accordingly. I think technically that's definitely doable with the current implementation, but the free-text alternative wouldn't require this. Are we okay with that overhead?

We can have a chat about it during today's co-working slot and see if we have a consensus between us on the issue. And otherwise we can maybe use the next community meet-up to present a pro/con list of both approaches!

As discussed yesterday, there are now two versions on separate branches:

#509
#494

That's amazing, thanks so much @llewelld, I think with that we can easily compare and contrast during the community meet-up and ask for feedback. I think it'll also be good to outline the potential pros & cons of both approaches during that meetup!

Thanks @gedankenstuecke. Excellent idea to outline the pros and cons; I'll prepare and share with you a document on them in advance.

That would be amazing, thanks so much! Otherwise we also have already a few notes here in the issue

Can this now be closed, or is there more to be done on this task?

I think this can be closed as #523 is covering the remaining part of this catch all! 👍


Allow multiple attributes in DOM inspector

Thanks for making react-inspector! I noticed that the DOMInspector component doesn't seem to support multiple DOM attributes, this PR should fix that.
Let me know if there's anything I can do to help get this merged!

Hey Tim, thanks for the PR! I've merged it.


Allow to match Copyright garbage by Regex patterns

Please have a look at the individual commit messages for the details.

It would super useful and likely more efficient if these garbage copyright could be reported upstream as bugs to ScanCode Toolkit where these will be fixed forever and everyone ;)

We had this discussion internally already @pombredanne, and I actually believe the copyright garbage feature in ORT would not really be required all that much anymore with recent version of ScanCode.  But not on the critical path of an compliance scan. This implies to me that ORT must provide means to make short term fixes on that critical path, so that issues can be fixed timely. While contributing back can be done when time permits. So, I disagree that this feature can be dropped as it provide means to zap out copyright statements, which is needed even though it's use becomes less and less frequent.

Our garbage file was filled with findings by older scancode versions, would these help when used with newer scancodes?

would these help when used with newer scancodes?

Very likely not. Only those garbage findings that are still valid for the current ScanCode version would be helpful. Actually, to remove unneeded garbage entries, I personally think it makes sense to start from scratch with an empty file and add entries anew as needed.

Our garbage file was filled with findings by older scancode versions, would these help when used with newer scancodes?

@pombredanne I believe this is for you. I presumed you could just put all garbage entries into a plain text file, run latest ScanCode against it and see which are considered as copyrights statements. Which would be your list of bugs then. Does this make sense?

Very likely not. Only those garbage findings that are still valid for the current ScanCode version would be helpful. Actually, to remove unneeded garbage entries, I personally think it makes sense to start from scratch with an empty file and add entries anew as needed.

Maybe we shouldn't assume that all users are on latest ScanCode version. As a compromise how about limiting it to
a min ScanCode version. E.g. add all garbage entries for ScanCode 3.x + but not for 2.x ?

Maybe we shouldn't assume that all users are on latest ScanCode version.

That's not what I'm assuming, but I am assuming that @pombredanne is only interested in issue reports against the latest ScanCode version; otherwise he'd first need to verify for each report if there's still something to fix with the latest ScanCode version.

That's not what I'm assuming, but I am assuming that @pombredanne is only interested in issue reports against the latest ScanCode version;

I was proposing to contribute the copyright garbage to ort-config so these can be used to filter out copyright garbage by any ORT user. Apart from that, as I outlined here above [1] it may be possible to filter the entries for the latest scancode version, so that the whole list can be indeed useful for @pombredanne  as input.
[1] https://github.com/oss-review-toolkit/ort/pull/6592#issuecomment-1473596956

Ok guys, this discussion is getting too off-topic now from the topic of the original PR. Please consider continuing in our discussions forum.


Reduce bundle size (Gatsby)

Bug Report
Hi! I'm using SUIR with Gatsby. My gatsby build has semantics components and comments all over the place. In js files, css and index.html. How to strip not needed components(tree shaking)? Should I use webpack, babel? Not quite sure how though. Thank you.
Expected Result
Reduced bundle size
Actual Result

I'm also interested in this answer. @dankarimov did you ever figure this out?

Hi! Webpack does tree shaking for javascript by default, if you use ES2015 module syntax (import/export). And for css, I just made a local copy of semantic styles and removed all unused css rules. You can do it with tools like PurgeCSS or by hand. Hope that helps!


Trouble using Bitbucket repo from azure pipeline

Our version control (currently) is in BitBucket, currently I've been unsuccesful in getting this to work.
Almost all attempts end in: /home/dependabot/dependabot-script/vendor/ruby/2.7.0/gems/dependabot-common-0.169.6/lib/dependabot/file_fetchers/base.rb:471:in rescue in default_branch_for_repo': Dependabot::RepoNotFound (Dependabot::RepoNotFound)`
Is it just not possible to use this with bitbucket repo's?
If it is, could you please help me out a bit, or help me find out which parameters are important?
It it is not, and you're open to extending this project with support for it, would you be interested in providing a small amount of guidance such that I'd be able to open a PR for this?

Hi @MidasLamb at this time, this only supports repositories in hosted on Azure DevOps otherwise referred to as Azure Repos.


Execution spinner hung - Native notebooks

VSCode Version:
OS Version:

Steps to Reproduce:

ran all cells, only first 3 cells show running state
spinner keeps spinning but timer is not increasing
restart kernel was not available. Happened again after interrupting kernel through command palette

Does this issue occur when all extensions are disabled?: Yes/No

Issues like this should go to the python extension first, because it is in control of how the cell execution state appears.


Tests: Nonbonded energies

@mrshirts says: Use a hard (simple, discontinuous) cut-off for electrostatics if it's implemented in the engine. Periodicity doesn't matter, if that's an issue with engines' implementations. InterMol did this for its testing. PME is messy, avoid that if possible for testing.

Will require forcing OpenMM objects into CutoffPeriodic/CutoffNonperiodic since the toolkit does not support it. GROMACS is more straightforward.

Might still want to consider other approaches (PME with non-default accuracy settings) and/or checking against multiple engines (currently only OpenMM and GROMACS).

Somewhat resolved now (see #212), will open other issues to resolve other details.


Added .tex to .pdf listener

Codecov Report

Merging #26 (86e101a) into master (df3d4cf) will decrease coverage by 4.27%.
The diff coverage is 53.65%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   87.70%   83.43%   -4.28%
==========================================
  Files          10       11       +1
  Lines         309      338      +29
==========================================
+ Hits          271      282      +11
- Misses         38       56      +18

Impacted Files
Coverage Δ

eds4jinja2/builders/report_builder_actions.py
40.00% <40.00%> (ø)

eds4jinja2/entrypoints/cli/main.py
88.88% <88.88%> (-1.12%)
:arrow_down:

eds4jinja2/builders/report_builder.py
97.77% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Griddle heading ascending/descending classnames not generated

I'm using Griddle with pagination, filter and local plugin, with useGriddleStyles = false so that I can use custom styles defined externally.  I'm trying to style the ascending/descending icons of the header to those from fontawesome.  I'm doing this by overriding the griddle-heading-ascending and griddle-heading-descending classes like this:
.griddle-heading-ascending:before {
   content: "\f15d";
   font-family: FontAwesome;
}

However griddle just uses the built-in up/down arrow icons.  Looking at the elements via Chrome, it seems that there are no griddle-heading-ascending and griddle-heading-descending classes generated.  It just creates an inline <span> inside the <th> with the griddle-table-heading-cell classname:

Is there supposed to be griddle-heading-ascending and griddle-heading-descending span classes generated?
BTW using Griddle 1.6.0

Not sure about "supposed to be", but you rightly observe that there are not (code). However, you can replace those icons with styleConfig:
<Griddle plugins={[LocalPlugin]}
  styleConfig={{
    icons: {
      TableHeadingCell: {
        sortDescendingIcon: <i className="fa fa-caret-square-o-up" />,
        sortAscendingIcon: <i className="fa fa-caret-square-o-down" />,
      },
    },
  }}
  />

@dahlbyk In the docs the ascending/descending classnames were listed as supposed to be there--well, at least from my own understanding =)  This is what is described in the styles section: https://griddlegriddle.github.io/Griddle/docs/styles/
I could use styleConfig but I would prefer styling to be externalized in it's own css (actually we use sass).  Every other classname in that link above seems to exist (we were able to stylize just fine externally) except for the ascending/descending ones.

@moraleslos interesting... the classNames entries exist, we just don't use them. I'll see if I can get that fixed.

@dahlbyk thanks!


added carousel component for stories

Fixes #284

I have modified the stories display into a slideshow format to improve the overall look of the page

I have also added next and previous navigation icons

Before

After

update your code with current changes


Deck Calibration Check: Determine a Move That Would Result in a Crash or Limit Switch Error

No longer relevant


Fully move zulip_bots to Python 3.

Fixing issue #187

So, it failed the linting test... Have I modified the rule correctly?

Sorry - This has been merged/rebased incorrectly, so some changes which weren't intended have been made.
I'll fix it tomorrow


Allow admin role to patch contributor, researcher and user_manager role, return these roles in get user response

Pull Request Overview

Admin user can patch researcher, contributor and user_manger role
Roles in user get response changed, staff dropped as it's not something that we expect to be seen at the API, contributor, researcher and user manager added

/rebase


Dynamic findbugs plugin version, restoring support for java 6 build

jmxtrans project » embedded-jmxtrans #108 SUCCESS
This pull request looks good
(what's this?)

Thanks @YannRobert, don't we have to specify a compiler target 1.6 in pom.xml?

I don't understand your question.
The maven-compiler-plugin source and target are already set to 1.6 in pom.xml

oops, you are right @YannRobert


Add emitter types

Dublicate of #33

Dublicate of #33

ouch... I am amazed how similar the solutions are though 🤓
May the better one win ✌


Compare against transition IDs instead of names

Summary of PR changes
Small bug where a transition name could have an alias.  If an issue is in that alias status, it will not skip but attempt to transition to the same status - producing an error.
PR Requirements

[x] For major, minor, or breaking changes, at least one of the commit messages contains the appropriate +semver: keywords.

See the Incrementing the Version section of the repository's README.md for more details.

[x] The action code does not contain sensitive information.

NOTE: If the repo's workflow could not automatically update the README.md, it should be updated manually with the next version.  For javascript actions, if the repo's workflow could not automatically recompile the action it should also be updated manually as part of the PR.

Tested in action: https://github.com/im-enrollment/ifp-team/actions/runs/5392985039/jobs/9792165451


error Command failed with exit code 1

Describe the bug
I have this Github repo
What I did was to just import it to the StackBlitz. Dependencies started installing, but then they exited.
Link to the blitz that caused the error
https://stackblitz.com/github/Rolanddoda/vue-i18n-next-issue
Steps to reproduce

Goto https://stackblitz.com/github/Rolanddoda/vue-i18n-next-issue
Wait for the terminal to do the installation
Observe the error

Expected behavior
This was supposed to work
Screenshots

Platform
Browser name  = Chrome
Full version  = 96.0.4664.93
Major version = 96
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.93 Safari/537.36 OPR/82.0.4227.33
performance.memory = {
  "totalJSHeapSize": 84095440,
  "usedJSHeapSize": 75297888,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = bc2b78560fa0e6097e7f69d4e820659cab64d2c5

Additional context
No response

Thanks for reporting this @Rolanddoda - I just repro'd the issue on my machine as well. Seems like every single dependency tarball is giving that same error, but when I try installing the same dependency in fresh node.new project it works fine 🤔
Our team is taking a deeper look and will circle back once we've gotten to the bottom of it!

After some investigation I noticed that the dependencies are being resolved against https://registry.npmmirror.com. I believe we see this as a custom registry and requires auth tokens in order to work. If you remove your package-lock.json and run npm install to generate a new one, it will work.
Due to the holidays almost everyone is out, so tagging the correct people so we can come back to you in the beginning of January.
cc @jrvidal

@Rolanddoda This should be fixed now, thanks for the report :tada:


Bug(server): Unable to access /docs/index.html endpoint (404)

What happened?
Unable to access /docs/index.html endpoint.

What did you expect to happen?
Can be visited normally.
How can we reproduce it (as minimally and precisely as possible)?
Access https://karpor-demo.kusionstack.io/docs/index.html in browser
Anything else we need to know?
No response
Karpor version

$ karpor -V
# paste output here

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

I want to claim this task.

I want to claim this task.

@solarhell Welcome! 👋 I have assigned this issue to you. If you need to provide other information or help, you can @me any time.

@solarhell Hi 👋 Is there any problem?

@elliotxx It's been a little busy the last couple days but I'm still working on it

@solarhell Sure, busy you first. If you've got any puzzling queries, feel free to have a chat in this issue.

It seems that doc works fine when i use helm install. Maybe the helm package version used in demo env is old.

@solarhell Let me confirm again. Actually, the latest version is also used in LiveDemo.

@solarhell Yes, it's really okay. Sorry, take up your time.


security system

still Requires a bunch of work, but this is so you can see the code, most of it is kindly borrowed from robust toolbox

Test build of the flatpak is going now, once thats done the flathub bot should post it over here if anyone else would like to test: https://github.com/flathub/org.unitystation.StationHub/pull/7


C:/HaxeToolkit\haxe\std/cpp/_std/sys/io/File.hx (Line 30)

Describe your bug here.
Game Crash when loading speaker-pico into song as girlfr

iend
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I have no idea how to change the source code, so it should be all intact

Missing a file

pico gf relies on the picospeaker.json file, if that file is missing, then you can't use him


Extract rpc-filter crate

Problem
solana-rpc-client-api is too big and filter.rs is easy to pull out
Summary of Changes
pull it out and re-export with a deprecation notice

Sorry, I'm going to need more detail on the problem: by what metric is solana-rpc-client-api "too big"?

A release build takes 70 seconds on my machine and pulls in a long list of heavy dependencies that many use cases don't require, such as spl-token-2022, solana-transaction-status, solana-program-runtime, solana-zk-token-sdk and reqwest

A release build takes 70 seconds on my machine and pulls in a long list of heavy dependencies that many use cases don't require, such as spl-token-2022, solana-transaction-status, solana-program-runtime, solana-zk-token-sdk and reqwest

Sorry, I think I'm going to need to see the bigger picture before jumping on board with this PR or https://github.com/anza-xyz/agave/pull/2084
It doesn't necessarily make sense to me to make every module in the repo its own crate. When would you use RpcFilterType, for instance, without one of the config types?

But I'd be open to starting by just making a crate that houses both config.rs and filter.rs

Closing in favour of #4038


Addon doesn't remove utm in hash tags

Like this: http://bigbrowser.blog.lemonde.fr/2012/09/05/langue-morte-comment-dit-on-minijupe-en-latin/#xtor=RSS-32280322?utm_source=twitterfeed&utm_medium=twitter

I will just confirm that here... https://www.washingtonpost.com/news/wonk/wp/2016/12/21/the-real-reason-you-cant-buy-whipped-cream-this-christmas/?utm_term=.0f990b128c9a
IT didn't remove this "UTM" from the URL

This should now work in the new version of the extension (1.1.0)

I just tested this with 1.1.0 and I ended up with http://bigbrowser.blog.lemonde.fr/2012/09/05/langue-morte-comment-dit-on-minijupe-en-latin/#xtor=RSS-32280322%3Futm_source%3Dtwitterfeed.


dbeaver-ce snap is not using the snapcraft.yaml of this repo

The current snap does not seem to be using the snapcraft.yaml file of this repo. Are there plans to migrate towards the snapcraft.yaml of this repo?

The current snap uses snapcraft.yaml from this repository. But there is a build problem via Jenkins using snapcraft.yaml from this repository, specifically in the cleanup part.
The Jenkins build crashes with an error:
Priming cleanup
+ cd /snap/core18/current
+ find . -type f,l -exec rm -f /home/jenkins/workspace/DBeaverDesktop/snapcraft-dbeaver-ce/prime/{} ;
find: ‘./var/cache/ldconfig’: Permission denied
find: ‘./var/lib/snapd/void’: Permission denied
find: ‘./var/lib/private’: Permission denied
find: ‘./root’: Permission denied
find: ‘./etc/ssl/private’: Permission denied
Downloading 'dbeaver-ce_latest_amd64.deb'   0%
Downloading 'dbeaver-ce_latest_amd64.deb' 100%
Failed to run 'override-prime': Exit code was 1.

But manually, everything is building successfully, with the current snapcraft.yaml .
Please tell us if you know how to fix this problem with snap permissions.


Request - BG lang, Servers

Can you please add Bulgarian servers, so I will be able to involve myself.
I have tried but looks like my code reading is not so good.
Thanks.

Hi @Fauster,
The server URL must be added to this list.
How does the URL look like if you are logged in?

Hi @Fauster,
The server URL must be added to this list. How does the URL look like if you are logged in?

Hey,
I have tried but it is not working for some reason.
Even had added a USA and a German account to test the bot - still no success.
Either way : It is a subdomain as follows -  bg.molehillempire.com
Trying  over Debian 10/11/12 script never starts successfuly.

Hi,
do you use the Portal Login? Because i think in the login process is a bug and maybe thats why you cant login...

Hi, do you use the Portal Login? Because i think in the login process is a bug and maybe thats why you cant login...

Hi Cl0ckm4n,
yes my accounts are Portal Login.
Also noted that LOG-s are showing only "Fail" to login. Nothing else -> where breaks.

Ok, i will push a fix for the portal login tomorrow. I still have to test a little bit...

@Fauster Can you test it again?
Cl0ckm4n has fixed the portal login (https://github.com/MasterZydra/WurzelimperiumBot/commit/39888552d9ae524af1720b7e848106b314ec2fbf) and I added the URL (https://github.com/MasterZydra/WurzelimperiumBot/commit/a675af77e40de71c401f1dfdc9c7b553e6040e34)

LOG:
2024-01-05 16:51:38,177 - -------------------------------------------
2024-01-05 16:51:38,177 - Starting Wurzelbot for User ****** on Server No. *
2024-01-05 16:51:38,193 - An error occured while starting Wurzelbot.
Isn't it necessary to have *.lang.yaml as well?

Ok lets see if we can get this :)
This error means that the login itself fails...
1.) you pulled the recent changes from the repo, right?
2.) did you fill your loginData correctly, like:
user = 'Your_Username'
pw = 'Your_Password'
server = Your_Servernumber as integer
lang = 'bg'
portalacc = True
3.) i maybe found another little bug in the portallogin --> see branch "fix_portallogin" in my fork
adjust your code and test if it works now
4.) if not working to this point open the developer-tools in your browser (with F12) when logging in and send me the URL that look like this:
GET | https://sxx1xx.wurzelimperium.de/logw.php?port=1&unr=xx7xx&portunr=xx6xx&hash=xx32xx&sno=xx1xx
!!! Please remove your private data, like unr, portunr, hash BUT tell me the count of characters for each variable (like i did xx6xx stands for 6 characters...)

@Cl0ckm4n  So, I got yours : https://github.com/Cl0ckm4n/WurzelimperiumBot/tree/fix_portallogin    ,
setted the details and ....

Only unr URL that I can see is : https://www.bg.molehillempire.com/login.php?logout=1&lastServer=1&unr=*8 on the logout. Do not know how to use "Inspect" at Chrome to look for your request.

ok i have another idea...
go to HTTPCommunication.py and edit the serverurl for bg to this:
SERVER_URLS = {
'de': '.wurzelimperium.de/',
'bg': '.bg.molehillempire.com/',
'en': '.molehillempire.com/',
'us': '.molehillempire.com/',
'ru': '.sadowajaimperija.ru/'
}

ok i have another idea...
go to HTTPCommunication.py and edit the serverurl for bg to this:
SERVER_URLS = { 'de': '.wurzelimperium.de/', 'bg': '.bg.molehillempire.com/', 'en': '.molehillempire.com/', 'us': '.molehillempire.com/', 'ru': '.sadowajaimperija.ru/' }

I had the same idea couple of days ago. I had tried that already. Even going further as "hard wire" the domain / with without subdomain etc...

please try this version: https://github.com/Cl0ckm4n/WurzelimperiumBot/tree/bg-login

please try this version: https://github.com/Cl0ckm4n/WurzelimperiumBot/tree/bg-login

YOU MY MAN ARE MAGICIAN!
IT DOES WORK!
May need some touches but it does work. Login flawless. Command output exists, even cyrilic is visual.
Can you please explain what was the issue? Do not tell me it was the *.yml ....

Thats awesome :)
No the *.yml wasn't the issue. I just added it for your convenience and for further translations.
The main issue was:
The login process expected a fixed character length of 7 for the unr. But the bg-unr had 8 characters so it couldn't be resolved.

Thats awesome :)
No the *.yml wasn't the issue. I just added it for your convenience and for further translations.
The main issue was: The login process expected a fixed character length of 7 for the unr. But the bg-unr had 8 characters so it couldn't be resolved.

In general it was something that I could not solve myself. Phewww .... Now I will have to sort translation.
Can I please request a "Linux installation" as well?

Thats awesome :)
No the *.yml wasn't the issue. I just added it for your convenience and for further translations.
The main issue was: The login process expected a fixed character length of 7 for the unr. But the bg-unr had 8 characters so it couldn't be resolved.

In general it was something that I could not solve myself. Phewww .... Now I will have to sort translation. Can I please request a "Linux installation" as well?

The reason for the problem was that the bot was only tested for german servers. Yeah it was a bit tricky ^^
Linux... that already exists: https://github.com/MasterZydra/WurzelimperiumBot#installation-script-for-linux

Thats awesome :)
No the *.yml wasn't the issue. I just added it for your convenience and for further translations.
The main issue was: The login process expected a fixed character length of 7 for the unr. But the bg-unr had 8 characters so it couldn't be resolved.

In general it was something that I could not solve myself. Phewww .... Now I will have to sort translation. Can I please request a "Linux installation" as well?

The reason for the problem was that the bot was only tested for german servers. Yeah it was a bit tricky ^^
Linux... that already exists: https://github.com/MasterZydra/WurzelimperiumBot#installation-script-for-linux

Will the "BG patch" merged or shall I smash fork?

Thats awesome :)
No the *.yml wasn't the issue. I just added it for your convenience and for further translations.
The main issue was: The login process expected a fixed character length of 7 for the unr. But the bg-unr had 8 characters so it couldn't be resolved.

In general it was something that I could not solve myself. Phewww .... Now I will have to sort translation. Can I please request a "Linux installation" as well?

The reason for the problem was that the bot was only tested for german servers. Yeah it was a bit tricky ^^
Linux... that already exists: https://github.com/MasterZydra/WurzelimperiumBot#installation-script-for-linux

Will the "BG patch" merged or shall I smash fork?

i only can create a pull request. The merging has to be done from @MasterZydra
Lets wait for his answer :)
But, of course you can make a fork in the meantime.

Thats awesome :)
No the *.yml wasn't the issue. I just added it for your convenience and for further translations.
The main issue was: The login process expected a fixed character length of 7 for the unr. But the bg-unr had 8 characters so it couldn't be resolved.

In general it was something that I could not solve myself. Phewww .... Now I will have to sort translation. Can I please request a "Linux installation" as well?

The reason for the problem was that the bot was only tested for german servers. Yeah it was a bit tricky ^^
Linux... that already exists: https://github.com/MasterZydra/WurzelimperiumBot#installation-script-for-linux

Will the "BG patch" merged or shall I smash fork?

i only can create a pull request. The merging has to be done from @MasterZydra Lets wait for his answer :) But, of course you can make a fork in the meantime.

Smashing! Already working over translation. Amazing! Thank you so much!


install dgl=0.5.2

Hi,
I can't find the dgl 0.5.2 for installation, could you please provide some more info in this regard?
Best Regards
Anahita

You can check this PR for a containerised approach to avoid dependency hell ;)

Thanks for the reply, I have mac setup, and I need to make it work on mps backend, probably will need to make a branch of your work for that.
Sent from Yahoo Mail for iPhone
On Thursday, October 3, 2024, 11:20 PM, Lucas Hänke de Cansino @.***> wrote:
You can check this PR for a containerised approach to avoid dependency hell ;)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[MB-15674] Non-PPM shipments are not visible if the move does not have pro-gear

Jira ticket for this change
Summary
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the UI locally.
make client_run

Terminal 2
Start the Go server locally.
make server_run

Test plan

As a services counselor, open a move with hhg and ppm shipments w/o progear in the PPM closeout menu
Click Review Weights

[ ] Non-HHG shipments are shown on the page

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-15674

  Generated by :no_entry_sign: dangerJS against d29bc1bc96328dd98563126731e4f57f121f480c


Update the C# extension publisher name

Hi,
I work on the C# extension for VS Code. We see that this extension has a dependency on our C# extension. We wanted to inform you of our intent to change the account we publish the extension under from ‘ms-vscode’ to ‘ms-dotnettools’ on March 2nd.
Extensions referencing our extension using the ‘ms-vscode’ publisher should still continue to work after the transition. However, it would be best to publish an update to your extension where you reference us with the new ‘ms-dotnettools’ publisher name after that time
Thanks
-Joey

@JoeRobich apologies for not seeing this until now.  We have just been alerted to the fact that this:

Extensions referencing our extension using the ‘ms-vscode’ publisher should still continue to work after the transition

Isn't actually working, and the name change seems to be causing problems.  We are working on publishing a new release of our extension just now.

Relates to #329

@JoeRobich your changes have been merged, thanks for your contribution 👍


slugify() got an unexpected keyword argument 'ok'

Describe the bug
The slugify dependency throws.

TypeError: slugify() got an unexpected keyword argument 'ok'
TypeError: slugify() got an unexpected keyword argument 'only_ascii'

Checking the slugify source code I indeed don't see those as possible arguments.
To Reproduce
Command to reproduce the behavior:
bandcamp-dl https://virtualmage.bandcamp.com/album/strange-landscape-ep
Expected behavior
Not crash :joy:
Logs
Traceback (most recent call last):
  File "/bin/bandcamp-dl", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/lib/python3.12/site-packages/bandcamp_dl/__main__.py", line 132, in main
    bandcamp_downloader.start(album)
  File "/usr/lib/python3.12/site-packages/bandcamp_dl/bandcampdownloader.py", line 52, in start
    self.download_album(album)
  File "/usr/lib/python3.12/site-packages/bandcamp_dl/bandcampdownloader.py", line 151, in download_album
    filepath = self.template_to_path(track_meta, self.config.ascii_only,
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/bandcamp_dl/bandcampdownloader.py", line 91, in template_to_path
    self.logger.debug(f'Track artist is None, replacing with {slugify_preset(track["albumartist"])}')
                                                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/bandcamp_dl/bandcampdownloader.py", line 71, in slugify_preset
    slugged = slugify.slugify(content, ok=ok_chars, only_ascii=ascii_only,
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: slugify() got an unexpected keyword argument 'ok'

Desktop (please complete the following information):

OS: Arch Linux
Version: latest git commit (v0.0.16.r0.gcdf24c0)
Python Version: 3.12.5
slugify version: 8.0.4

Known fix
Pass --no-slugify and download works.

Probably related to recent changes in arg handling, I'll check it out soon unless someone else does, just not a lot of free time for the next few days
Good that it's something that can be worked around for now but I imagine it must suck for those wanting unaltered filenames.

Update: Not happening for me under the current codebase, likely due to you somehow ending up with slugify and not unicode-slugify (which is specified in the pyproject.toml as well as pulled in by the build script on the AUR)

Can confirm.
Uninstalling slugify and replacing it with python-unicode-slugify 0.1.5 worked.


Email Verification

When a user's email address gets verified by them following a link sent to them in an email, we need to somehow convey to the user that the email has been verified successfully.

Perhaps using alerts.html?
This is a bottom fixed bar that appears on the bottom of the screen.
It is orange when you click 'Get Help'
Perhaps it's teal and reads "Your email has been successfully verified." ?

@micchickenburger  bumped. Thoughts?

Sounds fine to me.  When I revisit the accounts process I'll let you know if I need anymore details.


Error installing OpenEXR with pip

I was excited to see the new python bindings but I'm having some trouble on macos Sonoma 14.5.
I'm not sure if this is an issue on my end or if something needs adjusting.
Log from pip attached:
cmd_2024-10-01_17.00.40_pip_errors.log

That looks like a problem with the distribution. It's incorrectly looking for a test directory that is not included with the sdist. I'll investigate further.
It's a workaround for now, but you might try cloning the repo and building from source.

Thanks Cary! No rush on my side really, let me know if it's handy for me to test anything

I have the same issue. As far as I know OpenEXR usually also ships pre-built binaries but for some reason version 3.3.0 does not include files for Python 3.12. Could this be part of the issue?
P.S. Version 3.2.4 does include builds for 3.12 though.

I noticed that the GH workflow for building Python wheels does not include Python 3.12. Happy to create a PR if this should be added :slightly_smiling_face:
P.S. The Python 3.13 release is also around the corner :snake:

Hmm, I had removed python 3.12 at one point during development because the build failed, but trying it again now, it does seem to work. Please do submit a PR, thanks!
#1857 should fix the sdist/macOS 14.5 problem, we'll make a patch shortly.

Hmm, I had removed python 3.12 at one point during development because the build failed, but trying it again now, it does seem to work. Please do submit a PR, thanks!

I create a PR: #1861 :slightly_smiling_face:

I'm not sure if this is an issue on my end or if something needs adjusting.

Not just you.
Here's a minimal test case to reproduce the issue:
docker run -it python:3.11-bookworm python -m pip install openexr==3.3.0

I've tried openexr==3.2.3, …==3.2.4, and …==3.3.0 all to the same effect.
I've also tried python 3.10, and bullseye instead of bookworm.

This should be fixed in v3.3.1, released just now. Give the wheel an hour or so to get built.

Thank you a 2^20, @cary-ilm ! I have confirmed v3.3.1 works for me.

Glad to hear it, thanks!

Also now works for me!
Seeing as there's new issues for taking a look at pip install without a binary, I think this can be closed.
Thanks for the quick fix!


fix issue#504

fix https://github.com/gojue/ecapture/issues/504

I have consulted a lot of information and I am not sure if these errors can be skipped. However, many open-source projects also choose to skip errors and continue processing. So, I agree with your modification. If anyone else knows about this issue, please answer me as well.
ref:

https://github.com/DataDog/datadog-agent/blob/85871eadd769876875e715f71634ff638caae649/pkg/network/go/asmscan/scan.go#L118
https://github.com/golang/go/blob/69583738eb73ef928a07a1e215b719823fd27aa9/src/cmd/internal/objfile/disasm.go#L357
https://github.com/grafana/beyla/blob/490a82c1af954babd3188af6aff011f0fda70401/pkg/internal/goexec/instructions_arm64.go#L15

However, I suggest adding comments to facilitate others' understanding of the reasons for the modifications.

Can you share your experience on how to find these references quickly?

Search arm64asm.Decode in github


Add deploy and codeql badge

Adds a badge to the readme for deployments and codeql
Rendered:

Rendered:

Second Option would be:

I don't have strong attachment to either version so happy to go with either.

updated this to use the second option as I think it looks a little cleaner

.deploy


Update default covariance type for GaussianCluster

Reference Issues/PRs
None
What does this implement/fix? Explain your changes.
Changes the default covariance_type for GaussianCluster from "full" to "all"
I find I almost always change to "all" for real data applications so it might as well be the default

Codecov Report

Merging #307 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #307   +/-   ##
=======================================
  Coverage   88.36%   88.36%
=======================================
  Files          39       39
  Lines        2467     2467
=======================================
  Hits         2180     2180
  Misses        287      287

Impacted Files
Coverage Δ

graspy/cluster/gclust.py
93.61% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


How to run multiple rancher server clusters in one k8s cluster?

I like to run mulitple rancher server clusters in one k8s cluster.
The problem is that they share the same etcd database, is it possible to make this multi-tenant or to use an external etcd database for each rancher server cluster?
Thanks!

@superseb  this is not working for rancher server HA clusters in one cluster right?


new(modern_probe): Add mlock, mlock2, munlock, mlockall, munlockall

Signed-off-by: David Windsor<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Which issue(s) this PR fixes:

#723
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dwindsor
Once this PR has been reviewed and has the lgtm label, please assign gnosek for approval by writing /assign @gnosek in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

ei @dwindsor thank you for all of them i will take a look ASAP :rocket:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dwindsor, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 679cb6085b53cf91f5d73a56097777b35eb12f27

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dwindsor, FedeDP, hbrueckner
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Omit identifiers for un-referenced properties

Properties have the useful feature that they can be referenced uniquely. However, the mechanism by which this is done results in an excess of triples which could otherwise be omitted.
Requirements

Must preserve a means of retrieving all properties of a given type.
Should only store a single triple for a property with a set value unless some other object references it.

OBE due to recent merge to dev of changes to properties - removing the property object


obj to doc update

closes #267

@GavinMendelGleason you can check again

See https://github.com/terminusdb/terminusdb-client-python/issues/267#issuecomment-983383998.

@GavinMendelGleason @spl all added, could you please check?


Figure out a way to split the monolithic JAR into smaller modules

Very nice work - one concern though - to quote

Note that the jar file is huge, due to the large number of classes

Indeed, we are currently looking at ~18 MB. While there are solutions that would allow us to extract only the classes we actually need (+ their dependencies), I wonder if it would possible to have several Maven artifacts - e.g. by "subject":

fastutil-core - code that is required by the other artifacts
fastutil-collections
fastutil-maps
fastutil-arrays
fastutil-io
etc.

Note that it is quite acceptable to have some dependencies "tree" between the artifcats - e.g., perhaps fastutil-collections requires fastutil-arrays- of course, without cyclic dependencies ....

Perhaps fastutil could provide a mechanism to export an implementation as a single java file?  In my application, I only want to use LongOpenHashSet and plan to copy it into my repo.

I cannot speak for the developers of this project, but I believe that providing a mechanism for extracting specific classes (and the code that supports them) is not within the scope of this project as it is a rather complex/tricky/delicate (choose your adjective ...) issue. That is why when I suggested this issue I proposed a "gross" division as it seems more feasible. That being said, IMO the more granularity the better - e.g., perhaps fastutil-collections could be sub-divided into:

fastutil-collections - common collection support code
fastutil-lists - only List-s related code
fastutil-sets - onle Set-s related code

@gaul You might want to look into proguard. By -keeping specific classes you can easily create a shrinked jar - you may also let proguard figure out which classes of fastutil your application needs and create one shaded jar. I'm using this in a project to shrink the resulting jar from ~25mB (mostly by fastutil) to ~2mB (fastutils share are a few hundred kB)
@lgoldstein The staggering size of fastutil mostly is a result of all primitives being included and thus all their combinations in the binary objects, like maps. If one only includes the basic primitives int, long, and double (did I forget one?) the size already goes down to ~2mb. I remember having a "fastutil-min" target, which did exactly that, but I can't find the code - I can search it . I do think this is would be the right way to go - maybe have two independent, self-contained artifacts.
I think there is little merit from having a finer granularity in this particular case. Quite a lot of larger utility libraries seem to follow this philosophy, for example, guava is also shipping one big jar (yes, there are also some which split their stuff). Storage is incredibly cheap and maintaining granularity is a real pain, especially with the infrastructure of fastutil. When you are developing for a somehow limited platform (Android), you should probably use a minimizer anyway.

I would like to see fastutil splitted into more modular artifacts in order to reduce the library size. I my case I'm using tablesaw-core library which is about 500KB in size and it's size is bloated because of fastutil beeing such a big dependency (~18MB).
In the meantime proguard could be a feasible solution.
Please see my ticket in the tablesaw issue-tracker:

https://github.com/jtablesaw/tablesaw/issues/537#issue-455223719


raptor - obedience

no discard because obedience resolve before: http://www.vekn.net/forum/rules-questions/75512-raptor-obedience?start=0

Wording has been changed ro "Only usable if this vampire is about to enter combat with an acting younger vampire."
I'll include it as a complete of a ruling already listed, I think "combat does not happen" can be enough.


Provide the ability to get remote static public key

It's the application's responsibility to determine whether the remote party's static public key is acceptable, as the security consideration section stated in The Noise Protocol Framework. However, the current implementation does not provide the opportunity for application to examine the remote party's static public key.
Once the handshake process is done, the HandshakeState object is deleted, and this is also the recommended behavior. Thus, in this PR, I try to reuse the keypairs field in NoiseProtocol object to record the keypairs which will be encountered during the handshake process, and provide a get_public_bytes interface to retrieve these keys.
The test for this new API is also provided in this PR.

Hi @plizonczyk , is there something I need to do for the CI flow? :D


Do not use script-wide use strict statement

You are using a script-wide form of use strict statement. This may cause problems with minimizing. You can use a function form instead:
(function(document, window, ...) {
    "use strict";
})(document, window, ...);

There is more explanation on the following stackoverflow-thread: stackoverflow link

thanks, looking into this

That seems to be automatically added by one of the preprocessors as it isn't present in the actual _wrapper.js file. I am trying to track down which one is doing it, and I think 6to5 is likely the culprit. Since 6to5 is now Babel, it may be worthwhile to migrate over to Babel to get the latest updates.

@shshaw This is somewhat of a big issue. I've problems with including different libraries when they're included after Cash. If I include those libraries before Cash everything works. Wondering that not more people have found this issue. For now I can only include Cash as the last script.

Yeah, I want to fix this as well.
It's tied into #93 ( and #90 / #92 ), which is a big change to the build process. @simeydotme was helping with this, but I'm not sure if he's made any more progress. If you have Babel experience or can help with the build process, then please contribute.

Unfortunately I don't have experience with Babel nor 6to5.

Unfortunately I don't either 😄  I've stumbled around with it a bit, but their documentation is quite cryptic, so I haven't wanted to deploy anything without someone experienced taking a look.

Took another stab at this, and managed a gulp-free build process using babel and uglify.
Pushed to the build-revision branch.

Works for me!
Thanks for preparing this build, it took me some time to land here; when I did, problem solved :)

This will be addressed in the next milestone.

Fixed in v2.1.3.
It was surprisingly tricky to implement, the minifiers kind of got in the way.


PageStorage: Fix entry.tag after full gc && add more debug message

What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tiflash/issues/5093, ref https://github.com/pingcap/tiflash/issues/5076
Problem Summary: The entry.tag is incorrect after full gc, this will lead to more IO caused by RegionPersister::doPersist
https://github.com/pingcap/tiflash/blob/5b61ae70550624d3bf0b5ca6bac89013ed5a6a4b/dbms/src/Storages/Transaction/RegionPersister.cpp#L111-L115
What is changed and how it works?

Copy the old entry and upodate the fields of new entry
Add some error message when exception is thrown while reading (for #5076 )

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

/merge


[WIP] Drop ROOT files in tests/samples in favor of scikit-hep-testdata

Most files should come from a version dependence on scikit-hep-testdata, but some, like Event.root, has to come from an HTTP request. (tests_require includes requests, so uproot.open("http://...") will work.)

We're going to need

issue187.root
issue213.root
issue232.root

and it would be really nice to find a more consistent solution to Event.root, but I don't think it can be put in any PyPI repository.

I think this test failure is due to scikit-hep-testdata. Posting as issue scikit-hep/scikit-hep-testdata#14.

Many bugs have been fixed, but ultimately, this will have to wait for scikit-hep/scikit-hep-testdata#14.

This is happening in uproot4.

Ah nice, I didn’t know about this test-data repo. I should use it for UnROOT.jl

That way, we don't have to all have the same test files clogging up our git repos.
I also found uses for it in writing tutorials. When I needed an example of complex data, there was uproot-issue399.root with a jagged array of histograms!
Also very useful in developing a ROOT-file reader: user-submitted issues are the only examples of files I have with TBaskets embedded inside the TTree. It's a situation I haven't been able to cause—I don't know how the files ended up this way. But the following "issues files" have good examples:

uproot-issue21.root has flat arrays (e.g. nllscan/mH)
uproot-issue327.root has jagged arrays (e.g. DstTree/fTracks.fCharge)
uproot-issue232.root and uproot-issue187.root have jagged arrays (e.g. fTreeV0/V0s.fV0pt and MCparticles.nbodies)
uproot-from-geant4.root has jagged arrays (e.g. Details/numgood and TrackedRays/Event and phi)

The way jaggedness (offsets, the fNevBuffer) is encoded is different in embedded baskets than free baskets, so it's important to have these tests. As for branches, I tend to go for the integer ones because small (< 256) integers are easy to recognize in a byte stream. Also, the files made by Geant4 are all a little weird—they have obscure class versions because the Geant4 file-writer is an independent implementation, not ROOT.

Yes I see, thanks for the explanations! It's indeed extremely useful.
The jaggedness is still something I struggle with, so it’s very helpful to have these examples consolidated.

This may be helpful:
https://github.com/scikit-hep/uproot4/blob/9f2c50466fb781f6f0a1fff9ed1b1a30e360c5c6/uproot4/models/TBasket.py#L30-L103
In Uproot 1‒3, basket handling was distributed in many places because I was still figuring it out. This time, I managed to consolidate it. Strictly speaking, this is both the TBasket and its TKey, but it makes sense to handle this TKey in a special way, mixed with the TBasket handling.
Even though I could consolidate it, the embedded/non-embedded cases have to be partially split in a big if/else clause. After the header, they're just different.
Once you've identified what I call the data (array of uint8) and the byte_offsets (array of int32, specifying byte positions), you're ready to move on. These are the only two things you need to know from a TBasket. Even if you're struggling with interpreting bytes within an entry, knowing absolutely where the entry starts is a big advantage.

I got it thanks, that looks quite consise! Yes, so currently the data and the byte_offsets is something which I already got working for many cases (not the TTree Baskets but I put that on low priority for now). The interpretation is really what's missing but for that I need to go through the streamer interpretation process first. It's still quite foggy how to see if something is jagged or not, but I got a few clues, like the classname (when it includes std::vector) or a specific kConst value etc.
I'll see how far I get next week. So far I am happy that I can do my physics analysis for my PhD with UnROOT.jl already, so I try to focus on some results first, I have to finish until October ;)


ShortcutsModifier - pressedKeys stuck when a child window is opened from a shortcut handler

If a keyboard shortcut is assigned to an AppWindow and its handler leads to opening of another window, the pressed key stays in the ShortcutsInstance's pressedKeys set.
import androidx.compose.desktop.AppWindow
import androidx.compose.runtime.getValue
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.runtime.setValue
import androidx.compose.ui.input.key.Key
import androidx.compose.ui.window.Dialog
import androidx.compose.ui.window.DialogProperties
import javax.swing.SwingUtilities

fun main() {

    SwingUtilities.invokeLater {

        var aIsPressed by mutableStateOf<Boolean>(false)

        AppWindow(title = "Parent").apply {

            show {
                var dialogShown by remember { mutableStateOf(false) }

                if (aIsPressed) {
                    dialogShown = true
                    aIsPressed = false
                }

                if (dialogShown) {
                    Dialog({ dialogShown = false }, DialogProperties(title = "Child")) { }
                }
            }

            keyboard.setShortcut(Key.A) {
                println("A was pressed.")
                aIsPressed = true
            }
        }
    }
}

In the Parent window, press A.
The Child window is opened.
Close the Child window.
Click the Parent window so it gets the focus.
Press any key other than A. The keyboard shortcut handler is erroneously fired which leads to another opening of the Child window.

There is a clumsy workaround for this bug: delay the opening of the Child window for a sufficient (100 ms) amount of time:
                val composeScope = rememberCoroutineScope()
                ...

                if (aIsPressed) {
                    aIsPressed = false
                    composeScope.launch {
                        delay(100)
                        dialogShown = true
                    }
                }

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Enhanced cookies syntax options

closes #221
This PR enhances cookies syntax options.
Now cookies are activated with this syntax:
cookies true
# or
cookies max_age: 300

@AlfonsoUceda This looks great! :clap: Merged. :+1:


Closes #287 by adding the first-timers-friendly badge to README

Closes #287
This PR adds the first-timers-friendly badge to README

✅ By submitting this PR, I have verified the following

[x] Checked to see if a similar PR has already been opened 🤔
[x] Reviewed the contributing guidelines 🔍
[x] Added my name to the bottom of the list under the Contributors section in the README.md with a link to my personal website or GitHub profile 👥

Hi @fvcproductions, it would be great if you could review the PR and let me of any changes to be done to proceed with it.

I have resolved the placement and format issue and have merged the conflicts as well due to multiple contributors changing the README file.

Thanks @sonali9696 - much appreciated! 💯

Loved working on the issues and would look forward to making more contributions to your website. :)


ERROR: Signature extraction failed: Traceback (most recent call last):

Checklist

[x] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2021.06.06
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE

ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users...\youtube_dl\extractor\youtube.py", line 1348, in _decrypt_signature
func = self._extract_signature_function(
File "C:\Users...\youtube_dl\extractor\youtube.py", line 1262, in _extract_signature_function
res = self._parse_sig_js(code)
File "C:\Users...\youtube_dl\extractor\youtube.py", line 1331, in _parse_sig_js
initial_function = jsi.extract_function(funcname)
File "C:\Users...\youtube_dl\jsinterp.py", line 245, in extract_function
raise ExtractorError('Could not find JS function %r' % funcname)
youtube_dl.utils.ExtractorError: Could not find JS function 'na'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError("Could not find JS function 'na'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.")); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

This problem has been fixed in version 2021.12.17.
Please close this.


Sanitize lockfile

Context
Reviewing https://github.com/bundler/bundler/pull/4107 , I agreed with @nemski's comments about separate examples.
I also felt it would be useful to give an example when there were no credentials in the URI
Change
Rather than comment on the PR, I figured I'd PR against it.

:+1:


glyph font based on aristo 2 theme icon set

just created a font based on aristo 2 theme icon set
aristo 2 was released by "madebysofa" under http://creativecommons.org/licenses/by-sa/3.0/us/ for the cappuccino.org framework

If this included more icons, I'd be all for including it, but alas, I'd rather have the breadth than the icon font at this point.
I'll continue to keep this in our back pocket though—awesome work!


Performance tuning

How much does this patch improve performance? What can we expect here?

All of this gave me about ~2% speedup for this benchmark: https://github.com/STEllAR-GROUP/hpx/blob/master/tests/performance/local/skynet.cpp. Granted not much, could even be in the measurement noise. I just thought we shouldn't loose it.

OK. The changes look good!


Toggleable config button in options menu (#39)

Implement https://github.com/AsoDesu/IslandUtils/issues/39 by adding a new config option in misc. settings

Note:
I added a Finnish translation for this because I speak Finnish!

LGTM 👍


additional material component LESQ-237

Description

Updates Lesson Overview mv to 8, tests files, fixtures, schemas and query
adds expandable container and renders conditional material component
Updates types on lesson page, components and avo

Issue(s)
Fixes # LESQ-237
How to test
N.B. Worksheets only load when logged into google as assets are not public currently

Go to https://deploy-preview-1816--oak-web-application.netlify.thenational.academy/beta/teachers-2023/programmes/combined-science-secondary-ks4-foundation-aqa/units/measuring-waves/lessons/lesson-1-in-measuring-waves#additionalMaterial
Click on _______
You should see _______

Screenshots
How it used to look (delete if n/a):

How it should now look:

Checklist

[x] Added or updated tests where appropriate
[x] Manually tested across browsers / devices
[x] Considered impact on accessibility
[x] Design sign-off
[x] Approved by product owner

:tada: This PR is included in version 1.229.12 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to get response code when service.events().list

I build one booking website using the Google Calendar API, i have one autoload.py will load events every 10 mins, everything oke if i have permission to acces calendarId, but If i dont have permission calendarId, my python file autoload.py stop run. So i need to know the way to get respose status_code for this function.
events_result = service.events().list(calendarId=calendarId,                                           timeMax=now,                                           alwaysIncludeEmail=True,                                           maxResults=100, singleEvents=True,                                           orderBy='startTime').execute()
I want to know how to get respose status_code

If i have permission to acces calendarId, everything is ok.
If i dont have permission calendarId, my python stop running.
  File "/Users/nguyenngoclinh/.conda/envs/1z_vietnam/lib/python3.7/site-packages/googleapiclient/http.py", line 907, in execute  raise HttpError(resp, content, uri=self.uri) googleapiclient.errors.HttpError: <HttpError 404 when requesting https://www.googleapis.com/calendar/v3/calendars/pia9kapki9l4r9uejgf7r3leo4%40group.calendar.google.com/events?timeMax=2020-08-19T05%3A38%3A17.101875Z&alwaysIncludeEmail=true&maxResults=100&singleEvents=true&orderBy=startTime&alt=json returned "Not Found">
So the way to get response status_code

Specifications

Python version (python --version)
OS (Mac/Linux/Windows)

See https://stackoverflow.com/questions/23945784/how-to-manage-google-api-errors-in-python


Fix 64 bit event parsing

This fixes some events that previously could not be parsed.

Neat!
I've invited you as a collaborator, in case you wanted to iterate a bit more rapidly w/o having to funnel all of it through PRs. Of course I'm still happy to review if you want me to :)
Changes took good to me, feel free to merge this in yourself!

Awesome! 🍻


How to upgrade to v13 beta

I'd like to upgrade my deployment to v13 beta, is it possible with this helm chart?

Set the image tag while installing/upgrading helm chart release.
https://helm.erpnext.com/faq.html#how-do-i-customize-values-for-the-erpnext-helm-chart

Is there anything else that needs to be done to upgrade other then change the image tag?
Thanks

https://helm.erpnext.com/faq.html#how-do-i-upgrade-and-migrate-to-the-latest-helm-release

set the MariaDB host and storageClass.
set the additional tags
set migrateJob.enable


Discussion: Add button to "split" technologies; replace concurrency

As suggested here, I'd like to propose a "Split" button as a replacement for the concurrency options we have today.
The idea is that any technology with two or more units can be split into two separate technologies, with half the number of units each.
For example, if we have a Household with 4 units:

... the visitor may hit "Split", which will result in two households each containing two units:

If that is not enough, the visitor can then click "Split" on those two, ending up with four separate technologies, each with one unit:

The "Split" button would be greyed out and disabled if the technology has only one unit installed.
--
Things we might consider:

Should the split technologies both use the same profile, or should we auto-select a different profile for the new technology?

Should we support splitting buffers and all their technologies? (I think for simplicity, no).

Nice-to-have? Should we allow a user customise how many technologies are created? Perhaps they want to split a units=50 technology into five separate technologies of 10 units? That could be a "..." option which the user clicks, and enters how many techs they'd like (up to, but not exceeding, the number of units already installed):

I'm on the fence about this, I would like to avoid adding a "..." button – at least at first – and see if a simple "Split" button will suffice. We could add "..." if necessary later.

Motivation

Splitting technologies using the current "min concurrency" option is like using a sledgehammer. Maybe the user only wants to split that solar array of 500 units into two or three separate technologies, not 500?

Discouraging huge numbers of units=1 technologies will result in faster calculations, and a more responsive technology form.

Removing "max concurrency" eliminates a category of bugs where merging technologies would lose any custom settings entered by the visitor. "Add 2 P2P batteries and set a custom capacity on one. Apply the concurrency settings. You either end up with two custom-capacity batteries, or two default-capacity batteries (depending on which order they appeared on the form)."

We could delete a lot of code relating to concurrency.

:space_invader: Bonus Round :space_invader:
When saving an LES, we could de-duplicate technologies which have a common parent (heating techs which belong to the same buffer, or non-heating techs which belong to the same endpoint). Two identical units=1 solar arrays (all attributes are exactly the same) could be merged into one technology with units=2.
--
Pinging @ChaelKruip, @dennisschoenmakers, and @grdw for feedback.

I'm on the fence about this, I would like to avoid adding a "..." button – at least at first – and see if a simple "Split" button will suffice. We could add "..." if necessary later.

I agree about that '...' button which is mysterious. Why not use one button that reads: Split, then you get that dialogue:

Default should be 2 (what if it is not divisible by 2? 😒 ) or the last chosen number. Disadvantage is a standard 'two click', but I wouldn't be bothered too much about that.

(what if it is not divisible by 2?)

Round one up, round one down. units=3 => units=2 + units=1.

I ❤  the ideas in this post!

Should the split technologies both use the same profile, or should we auto-select a different profile for the new technology?

Auto-select a new one. This is the reason why people might want to split anyway.

Should we support splitting buffers and all their technologies? (I think for simplicity, no).

I am not sure about this one. Perhaps not at first but heating is an important part of the load and being able to change concurrency of that is worth some effort IMO.

Nice-to-have? Should we allow a user customise how many technologies are created?

Yes. I think that we need to have an option to split into more groups than two. I like @dennisschoenmakers suggestion.

The split does need to be limited to [technology_units, number_of_profiles].min. Also - as discussed with @ChaelKruip yesterday the first approach shall be with non-buffer technologies. The technologies will automatically-assign a profile.
What I'm thinking about though. When clicking the split button it should only affect the technology to which the split button is attached.
Some code about how I think it might work for a single technology:
$("a.btn.split").each(function () {
    $(this).off('click').on('click', Splitter.split.bind(this));
});

/*contents of Splitter*/
var Splitter = (function(){
    'use strict';

    return {
        split: function () {
            // Do an Ajax request to the server with the data of the technology
            // The Ajax request should grab the data and the ID of the technology.
            // This way  the Ajax request knows which technology field to update
            //
            // {
            //    technology_data: $(this).parents(".technology").data(),
            //    technology_id:   $(this).parents(".technology").attr("id"),
            //    units:           $(this).data('max_units') || -> fetch content from input?
            // }
        }
    };
}());

Imagined contents of the def split action in testing_grounds_controller.rb
def split
  @technology_id = params_for_splitting.fetch(:technology_id)
  @splitted_technologies, @technology_id = TestingGround::Splitter.new(params_for_splitting).split
end
...

private

def params_for_splitting
  params.require(:split).permit(:technology_data, :technology_id, :units)
end

Imagined contents of the splitter.rb or something in those lines.
class Technologies::Splitter
   def initialize(technology_data:, units:, **)
       @technology_data = InstalledTechnology.new(technology_data)
       @units = units
   end

   def split
      # do the splitting magic :)
   end
end

Imagined contents of the split.js.erb
<% @splitted_technologies.each do |technology| %>
  $("#<%= @technology_id %>").append("<%= render 'technology_template' %>");
<% end %>

$("#<%= @technology_id %>").remove();

Some code about how I think it might work for a single technology:

I'm not sure I follow the need for any Ruby, or the round-trip to the server. Can't this be done all in JS?
I was expecting that we might be able to reuse/adapt the "Add technology" code in the tech form, except that "Split" would:

Add a new technology immediately below the one being split (the "source"), instead of at the top of the list.
Take the attributes from the source technology instead of using the default attributes.
Assign a new profile.
Set the number of units on both the new and source technologies.

Can't this be done all in JS?

It could I guess. 👍  I do need to draw up some different pseudo code than. You're take on the word 'split' would be cut in half I guess. So hit the split button, get two technologies with the units divided by two and assign a new profile for the technology below that.
However the BONUS ROUND should be done at the server end I guess. That makes a bit more sense to me.

Take the attributes from the source technology instead of using the default attributes.

This would probably be the main reason why an all js approach would be better 👍

Pure JS solution would be something like:
$("a.btn.split").off('click').on('click', Splitter.split.bind(this));

/*contents of Splitter*/
var Splitter = (function(){
    'use strict';

    return {
        split: function () {
            var tech = $(this).parents('.technology');

             for(var i = 0; i < units; i++) {
                var clone = tech.clone(true, true);
                tech.after(clone);
            }

            // set profiles
            // set units
        }
    };
}());