Adding elements to the CodeEdit arrays doesn't work

Godot version
4.0.stable
System information
Windows 10 ; Nvidea RTX 3060 Ti ; Intel Core i7 10700k
Issue description
When trying to either add an element to the delimiter_strings, delimiter_comments and code_completion_prefixes in the CodeEdit Node, it does nothing. Except for the CodeCompletion Prefix, where if you  try to add an element, you get an error message : "Code completion prefix cannot be empty", although I am trying to add an element. ( Yes I have enabled code completion ). It still does nothing though.
Steps to reproduce

Create new Project with Control Node as Parent and add CodeEdit as child.
Click on either Delimiter Strings, Delimiter Comments or Code Completion Prefixes ( Where you have to enable it first ) and try to add an element.

Minimal reproduction project
Unneccesarry

I've been taking a look and it seems there are many issues contributing to this problem.
The main one is that, when you click the Add Element button on any of the fields mentioned, an entry gets added with an empty String as a value. Because of that, functions like CodeEdit::set_code_completion_prefixes and CodeEdit::_set_delimiters get called with that empty string as a parameter, and both of those perform an assertion on that String value not being empty: ERR_CONTINUE_MSG(prefix.is_empty(), "..."); . This assertion prevents the element from being added and produces the error shown in the issue.
Replacing this empty string with a "null" string value inside CodeEdit::_set_delimiters:
if (key.is_empty()) {
	_add_delimiter("null", "", true, p_type);
	continue;
}

Allows adding new elements, and they get the "null" default value:

Having "null" as the default value seems to have been the intended idea, since in the CodeEdit::_add_delimiter function the next comment is present:
// If we are the editor allow "null" as a valid start key, otherwise users cannot add delimiters via the inspector.
	if (!(Engine::get_singleton()->is_editor_hint() && p_start_key == "null")) {

Even so, I'm not sure if it's the best idea to fix it.
Being able to add new elements with default "null" values, a new issue appears. Since the delimiters are Strings, their value gets updated as you're typing them. (See Note1 below) .This causes the set_delimiter_strings event to get fired on every change you do to a delimiter, making it imposible to set the end symbol for them:

Also, as it can be seen in the .gif, the array of delimiters gets reversed every time a delimiter is added. Making it even harder to add them.
I've shown the issues with the delimiter_strings, but the delimiter_comments and code_completion_prefixes's issues are caused by the same reasons. Not sure on how to proceed on fixing this issue, since it seems it'd need a change to the behaviour of how strings are modified in the editor.
Note1:
Triggering events as string typing occurs seems to be the intended behaviour, as explained in EditorInspector::_property_changed
void EditorInspector::_property_changed(const String &p_path, const Variant &p_value, const String &p_name, bool p_changing, bool p_update_all) {
	// The "changing" variable must be true for properties that trigger events as typing occurs,
	// like "text_changed" signal. E.g. text property of Label, Button, RichTextLabel, etc.

Perhaps running the is_empty() check only when input is confirmed would work, but I don't know how to do that in code, but it might look like this :
if input-confirmed, then if entered is-empty, then push error and return, else push delimiter into array.
This should also work when modifying a part in the array as far as I'm concerned.
Hope this helps

I wasn't even able to add comment delimiters manually to the *.tscn file.
Something like
delimiter_comments = Array[String](["--[[ ]]--"])

will just be ignored and doesn't show up in the inspector or work in the project.

recently gone through code_edit.cpp  file where this issue is based and also came across this issue, I see 3 possible solutions:

Have a Copy of the PackedStringArrays related to the issue one with out empty arrays
Have a private C++  function to generate a copy PackedStringArrays  at run time
Just check for if is an empty array at each point of use of related PackedStringArrays and allow to empty string entries

3 is kind of better, but may result in another bugs if something is missed


Bad documentation

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
The documentation says to use Kubernetes labels to disable sidecar injection. However, this has no effect on pods. Instead, the user must configure Kubernetes annotations.
[x] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade

Secondary problem, do not know how to fix:
Applying this annotation to Jenkins worker pods creates an infinite loop. This prevents the job from ever fully initializing. It gets stuck there forever.

Can you provide some more information? Note the labels part is new and applies only to Istio 1.10. If you are on an older versions you should find the appropriate page on https://istio.io/archive/


In detached mode when there are no items, the dropdown still appears

Description
I want to display the items only when user has typed at least three characters.
Currently this results in the bug shown on the screenshot:

As you there is some strange white space below the input.
Reproduction
Steps

Open this CodeSandbox https://codesandbox.io/s/dont-open-dropdown-g4gdcm?file=/app.tsx
Click on Autocomplete and type "a".
As you can see there a strange white space below the input.

Expected behavior
When no items are available, the dropdown should not open and this strange white space below the input should not be present.
Environment

OS: macOS 13.3.1
Browser: Brave Version 1.57.62 Chromium: 116.0.5845.180 (Official Build) (x86_64)
Autocomplete version: 1.11.0

Hi @Mikilll94 , thanks for your report. We've updated the classic theme to hide the layout when it is empty. This will be released next week. In the meantime, you can apply the following CSS rule to have the same outcome:
.aa-DetachedContainer--modal .aa-PanelLayout:empty {
    display: none;
}

@dhayab
When this fix is going to be released?

This fix will be part of the next release, which should happen next week on Tuesday.

Hi @Mikilll94 , you can now update to<EMAIL_ADDRESS>to benefit from this and other bugfixes: https://github.com/algolia/autocomplete/releases/tag/v1.11.1.


Roll Flutter Engine from b149eea0a3c2 to d8928a0b66b8 (2 revisions)

https://github.com/flutter/engine/compare/b149eea0a3c2...d8928a0b66b8
2022-09-01<EMAIL_ADDRESS>[Impeller] More aggressively downscale gaussian blur textures (flutter/engine#35853)
2022-09-01<EMAIL_ADDRESS>Ignore all translations in Matrix::TransformDirection (flutter/engine#35854)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Closing this roll to trigger a new roll that includes the canvaskit revert.

Commit queue failed; closing this roll.


Fix settings config loading

Fix usage of config.action_mailer.<mailer>_settings by adding delivery method after initializing railties.

@admtnnr

LGTM. Sorry about that!

No worries.. doesn't hurt to double check. I also just added some railtie tests.

Yep, looks good. Was just about to ask if we could add something to the dummy app to catch this case when I saw your second commit come through. 👍


⚠ Issue Doble quotation marks in spanish lesson.

Inside the tutorial area, part 5 , lesson "Create a blog post archive". When you give us the code for the post-4.md.
In spanish version you give a code with a problem in the doble quotation marks of property alt of the image.
image:   url: "https://docs.astro.build/default-og-image.png"   alt: "La palabra "astro" contra una ilustración de planetas y estrellas."
The word astro is surrounded by doble quotation marks and broke the sentence. This generate a problem when i try to display links in the blog page and with dynamic tags after.
I only change this doble quotation marks for single like this and all works perfectly.
image:   url: "https://docs.astro.build/default-og-image.png"   alt: "La palabra 'astro' contra una ilustración de planetas y estrellas."

Hello! In this case, it is a typo. If you feel like it, you can submit a PR to correct it. If you open a PR, be sure to link this issue in the description of the PR. 🤠

Since #4877 has been merged, I think this issue is safe to close now!

Thanks to all for the help. I hope to collaborate again in the future.


remove DNS instructions for solr, activemq, and blazegraph

Since these three services are not exposed via traefik they don't need to have their DNS records set

I'm in favor of removing the config and documenting how to use SSH port forwarding to access these services if/when needed and/or having traefik limit what IP ranges can access the services. Leaving these services open to the WWW in production is a security risk, and we shouldn't have the recommended solution for needing to access the UIs for these services be opening them up world wide.

We just discussed this at the tech call and everyone is in favour of this. I will push another update to make those changes.


Configure dependabot for Live.Avalonia

We could incorporate the setup from https://github.com/AvaloniaUI/Avalonia.NameGenerator/blob/main/.github/dependabot.yml
This would allow keeping Live.Avalonia dependencies always up to date.

Should be resolved by https://github.com/AvaloniaCommunity/Live.Avalonia/commit/1364d29b128472e90c9655afed101c5c66c6357c


Update the Signing Certificate

Previous cert was SHA1 and SHA256. Going forward MicroBuild will support SHA256 exclusively. So changing cert.

LGTM. That's the one the runtime uses via Arcade.


how to build the nexmo script

how to build the node-nexmo library to single JavaScript file it's possible or not

yes, it is
npm run compile should help


Prevent adding duplicate layers to a chapter

If a user adds a layer that has already been added within a chapter, the application throws an error.
"Error: Cannot add item: item with id ... already exists"
When a user adds a layer, we need to check the chapter config to see if the layer has already been added. If it has, the user should not be able to add the layer, and a message should be displayed in the UI to let them know they have attempted to add a duplicate layer to the chapter.

Alternatively, we could simply not show the user the layer in their search results if it's already been added.


emoji_picker: Migrating to TypeScript

Fixes:
In this commit, emoji_picker.js file has been migrated to typescript. web/shared/src/typeahead.ts was modified to update the Emoji type. tests-js-with-node was also modified to add the file for tests.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@Wr4th100 thanks for working on this! . But there are a few things that can be corrected for this to be mergeable pull request.

Have you created a merge commit , instead of rebasing your branch ?..The Commit structure is not upto zulip's standard. Refer to commit discipline docs to more details.
The PR discription needs some work too. You can look at the PR discriptions of other merged PR's for reference.

@Wr4th100 thanks for working on this! . But there are a few things that can be corrected for this to be mergeable pull request.

Have you created a merge commit , instead of rebasing your branch ?..The Commit structure is not upto zulip's standard. Refer to commit discipline docs to more details.
The PR discription needs some work too. You can look at the PR discriptions of other merged PR's for reference.

Ok sure, I will make the necessary changes!

@Wr4th100 are you still working on this? It'd be great to keep this PR moving if so.

@Wr4th100 are you still working on this? It'd be great to keep this PR moving if so.

Yes, apologies for not updating sooner. I have made the required changes, please have a look!
Also, should I make a prep commit for changing the type Emoji in typeahead.ts?

Also, should I make a prep commit for changing the type Emoji in typeahead.ts?

Yes changing an existing type should have its own commit with an explanatory commit message.

Yes changing an existing type should have its own commit with an explanatory commit message as to what was wrong with the original type... and some analysis to show it's correct. I'm a little skeptical of the claim that we created those types with several missing fields; it seems more likely that the typeahead objects have that type, and maybe you're importing the wrong type, and want something from emoji.ts instead?

Yes, after consideration I have now used both the Emoji type from typeahead.ts and EmojiDict type from emoji.ts.
Earlier when I used either one of the types, the other would have required changes. Especially when using the typeahead.Emoji  type. But I ran into a very tiresome type error when I used the emoji.EmojiDict which made me go with the first option.
After checking again, only search_results is dependent on the typeahead.Emoji type, while the rest are fine with emoji.EmojiDict. Therefore, I made the changes accordingly.
Thank you for your insights, please take a look and let me if know if there are any changes to be made!

Thank you for the suggestions @LaPulgaaa, I have made the required changes now.

Hey @Wr4th100, are you planning to complete this one?


Check_Kyma_Resource_Deleted executed but not completed retriggers instance deletion for ever

Removal of kyma resource in  is done on the best effort basis in Delete_Kyma_Resouce_Step so we try to remove, but if attempt fails we continue deprovisioning.
We execute subsequently Check_Kyma_Resource_Deleted step to allow some time to pass before we check if removal was completed. If resource was not removed eventually (in configured timeframe) we mark the step as executed but not completed and continue deprovisioning.
In such scenario deprovision_retrigger_job tries to delete this cluster again by issuing DELETE operation on the instance.
This attempt fails since there is no cluster any more, kyma resource is in error state (kubeconfig secret is removed) and kyma resource finalizer is preventing kyma resource from deletion.
  finalizers:
  - operator.kyma-project.io/Kyma

...
status:
  activeChannel: regular
  conditions:
  - lastTransitionTime: "2023-03-24T01:26:01Z"
    message: all modules are in ready state
    observedGeneration: 1
    reason: ModulesAreReady
    status: "True"
    type: Ready
  - lastTransitionTime: "2023-04-03T12:20:09Z"
    message: not all modules are in ready state
    observedGeneration: 2
    reason: Ready
    status: "False"
    type: Modules
  - lastTransitionTime: "2023-04-03T12:20:09Z"
    message: module catalog is out of sync and needs to be resynchronized
    observedGeneration: 2
    reason: Ready
    status: "False"
    type: ModuleCatalog
  lastOperation:
    lastUpdateTime: "2023-04-12T07:57:38Z"
    operation: 'initializing sync context failed: secret with label operator.kyma-project.io/kyma-name
      "e4768f87-83a0-4339-992d-8d204469b85d" not found'
  state: Error

Notes:

discussion with Jellyfish team is required to raise awareness of the issue (Lifecycle Manager is responsible for removing the finalizer in Kyma CR),
After discussion with Jellyfish we can add label/annotation to distinguish orphaned instances (Kyma CR without associated cluster) if required.

Resolved by:
https://github.com/kyma-project/kyma-environment-broker/pull/427
https://github.com/kyma-project/kyma-environment-broker/pull/432


Upgrading jQuery to 3.6.0

Changes proposed in this pull request:

Upgrade jQuery and jQuery Migrate.

The jQuery migration from 1.12.4 to the latest (3.6.0) version required many fixes due to deprecated and removed methods, and also some behavior changes.
Luckily, all the work was able to be done incrementally since all of the fixes were compatible with the old version (1.12.4).
All of the Cypress tests are passing, and I have done some additional manual testing.
Summary of Changes:

Replaced usages of jQuery functions removed in 3.6.0 or earlier. (Ref: https://api.jquery.com/category/removed/).
Replaced usages of jQuery functions deprecated in 3.6.0 or earlier. (Ref:  https://api.jquery.com/category/deprecated/).
Fix broken code due to jQuery no longer auto-inserting tbody tags.
Correct  to  becuase latest jQuery did not like self-closing .
Fix deprecations in external dependencies jquery.imgareaselect.js and suggest-4_3a.js.
Due to a change in behavior in jQuery $.post(), calls to /command/core/toggle-starred-expression would fail if a "json" is expected, but an empty response is returned. By omitting the expected content type (""), then the content-type is set by the server response. (Ref: https://github.com/jquery/jquery/issues/3973).
Updated select2 package which fixed jQuery deprecations.

Pending:

The tablesorter dependency has an open PR to remove calls to deprecated functions. (Ref: https://github.com/Mottie/tablesorter/pull/1786). The current tablesorter does work with jQuery 3.6.0, so it's still okay to go ahead.
May want to remove jquery-migrate.  They don't recommend you have it enabled in production.  We should just add it in dev builds when we are doing migrations from one jQuery version to the next.

Associated PRs:

https://github.com/OpenRefine/OpenRefine/pull/4886
https://github.com/OpenRefine/OpenRefine/pull/4882
https://github.com/OpenRefine/OpenRefine/pull/4874
https://github.com/OpenRefine/OpenRefine/pull/4872
https://github.com/OpenRefine/OpenRefine/pull/4862
https://github.com/OpenRefine/OpenRefine/pull/4861
https://github.com/OpenRefine/OpenRefine/pull/4853
https://github.com/OpenRefine/OpenRefine/pull/4852
https://github.com/OpenRefine/OpenRefine/pull/4851
https://github.com/OpenRefine/OpenRefine/pull/4840
https://github.com/OpenRefine/OpenRefine/pull/4837
https://github.com/OpenRefine/OpenRefine/pull/4827
https://github.com/OpenRefine/OpenRefine/pull/4822
https://github.com/OpenRefine/OpenRefine/pull/4817
https://github.com/OpenRefine/OpenRefine/pull/4811

Very nice, thank you for the amazing work!
In terms of schedule, I wonder if we want to have this in 3.6. This is going to be a breaking change for some extensions, I suspect. I guess we could decide to put this in 4.0 only and keep the 3.x series as compatible as possible. But then jQuery 1.12.4 does have some vulnerabilities so I guess there is also a case for migrating it in 3.x directly.

Is OpenRefine 3.6 going to be released soon?
In the OpenRefine UI code, I only found 10 actual issues which broke either the code or the cypress tests, which resulted in 53 tests failing.  In some cases, one bug broke multiple tests.
A lot of the changes were migrating deprecated functions, but of course that isn't required.  JQuery has deprecated many functions, but they have been very slow to remove them. If you look at the list, they have only removed functions in early jQuery 1.3-1.10.   (Ref: https://api.jquery.com/category/removed/).  There were only a handful of places OpenRefine was using removed jQuery functions.
For extensions, I would expect there would be few issues. If there are popular extensions outside of this repo, link them here and I can try them out with OpenRefine 3.6 and jQuery 3.6.00 to see if they work.
And yes, the upgrade does resolve three security issues. Not sure how critical that is if OpenRefine is mainly used locally or within a private network. I've added info to the PR for that.

Yes I would like to release 3.6 soon.
We have a list of popular extensions here: https://openrefine.org/download.html

I looked at the list of extensions but most were not active and haven't been updated to work with OpenRefine 3.5.x.
If we postpone upgrading jQuery, that means any new extensions on 3.6.x will be using jQuery 1.12.4, and may break when jQuery gets upgraded in the next OpenRefine version.


EdX Courses Analysis

Pull Request for ML-Crate 💡
Issue Title : EdX Courses Analysis

Info about the related issue (Aim of the project) : To analyse the EdX course data
Name: Lahari Boni
GitHub ID: https://github.com/LahariBoni
Idenitfy yourself: HSOC Participant

Closes: #143
Describe the add-ons or changes you've made 📃
Initially, I  explored the data a bit and proceeded to clean the data to remove all the duplicate values, numbers, punctuations, and stop words for better understanding and gradually performed EDA with appropriate charts, and word clouds to analyze the data for insights
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
The files are run multiple times and also on a different local machine
Describe how have you verified the changes made
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Re-opening PR as it seems to happen by mistake. I sincerely apologise for this.

It's picture perfect! Great representation of the project, everything looks super cool 💯 @LahariBoni

It's picture perfect! Great representation of the project, everything looks super cool 💯 @LahariBoni Your PR is being upgraded to Level 3 🎉

Thank you @abhisheks008 😄


Minor readability issue with filenames of test resources

The read*, update and delete tests use test.txt when setting up turtle resources. Elsewhere it is test.ttl.
It doesn't affect whether or not the tests work but might help readability if the filenames were consistent with the content types.

We may consider that these tests are doing 2 things at the same time and that it is OK
or relabel them to explicitly state that they update/delete resources with filenames ending with a different media type  then the content-type,
or add tests so that they do only one thing at a time

Personally, I think for traceability reasons a test should focus on one thing at a time. I suspect the filename difference was a simple copy & paste oversight in this case not a deliberate purpose of the test. Each test should be clear about its purpose.


Enhance Alpine package scan results

Alpine packages lack some important info like copyrights or where the source code is located. This info can't be gathered from the packages themselves as its just not there. To get this info we need to:
download aports repo & for each pkg check it out on commit specific to alpine package (via fetchcode)
parse APKBUILD nexB/scancode-toolkit#2541
download package sources (fetchcode) & amend new info to package's scan results
Discussed a bit with @pombredanne
Most likely @quepop will PR it
The question is- should it be standard behavior when alpine based docker is being scanned or should it be a separate pipeline?

Related: https://github.com/nexB/scancode-toolkit/issues/2541#issuecomment-856904754

@pombredanne should I PR it?

@quepop please do! that's much easier for review
Of note:

I am not super comfy (that's an understatement) with sourcing arbitrary shell scripts at https://github.com/quepop/scancode.io/commit/2f59440b97cfc89e4bc8291d7839188d25e0fb34#diff-a6e238494a11c38a54d401da6211e8e3461898f10e22ff6673a7896ffedbbd3eR43
I am not sure we want the alpine package getter to always fetch packages. If anything I know several users run in an airgap'ed environment and they surely will not want (and will not be able anyway) to fetch things from the outside world

@pombredanne I'dont think there is any other way. In order to extract source variable value (or any other value) we need to run the script (APKBUILD file), because in some of the alpine packages, source variable is built in a for loop (not to mention that sometimes bash string formatting is used)
From<EMAIL_ADDRESS>APKBUILD file:
`_arch_keys="
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>    "

for _i in $_arch_keys; do
source="$source ${_i#*:}"
done
`

@quepop good point, but do we need these (e.g. these _arch_keys) ?

@pombredanne We need the source variable to be complete (to be able to download sources and scan them for copyrights). I tried to emphasize (by giving an example) that the only way we get a complete source variable every time is by running the script.

@pombredanne Regarding https://github.com/nexB/scancode-toolkit/issues/2541#issuecomment-858408580, how should I name these new files (pipe, pipeline)? My idea is /pipes/alpine_helper.py and /pipelines/alpine_complement.py

@quepop There's already a pipes/alpine.py pipes module in which you can add the new functions.
For the pipeline, what about "alpine_packages.py", unless it's too specific?

In the case of multi-image projects we cannot match DiscoveredPackage from the database to the image (and alpine version) they are from - db entry for DiscoveredPackage only include project_id (no image_id or image_index field). Alpine version is only present in project.extra_data.images array. It's a problem because in the aports repository every alpine version is a certain branch and we need to pull them and make checkouts (branch and then commit-id) to be able to extract correct APKBUILDs.
My solution is to add a new database field to the DiscoveredPackage class named image_index which would store an index of the image from the project.extra_data.images array it is from.

@quepop what about adding the extra_data field on the DiscoveredPackage model, for consistency with Project and CodebaseResource models?
Let me know if that would work for you and I'll make that change upstream.

Repasting from https://github.com/quepop/scancode.io/commit/2f59440b97cfc89e4bc8291d7839188d25e0fb34#r52902214 for reference :
@quepop
I have been thinking more about this for nexB#191 and sourcing arbitrary bash script this way is too much of a security concern.
An alternative could include:

using a container, jail or some sorts of chroot to try to minimize the risk a little
implement a simple bash/shell script parser and perform parameter expansion

I think 2. is best and much less involved than having a container depdendency.
Furthermore, there could be other variables of interest in an APKBUILD and we need eventually to parse other shell-based manifests to extract metadata such as PKGBUILD (Arch), ebuild (Gentoo) m4 (Autotools) and a few more.
Therefore I am implementing this that can then be reused here:

using a simplified Pygments shell lexer derived from https://github.com/pygments/pygments/blob/master/pygments/lexers/shell.py
adding lightweight parsing to recognize variables declaration possibly based on https://github.com/nexB/pygmars/ ... or just hand crafted as this is rather simple once the harder lexing is done above
doing parameter expansion with @kojiromike https://github.com/kojiromike/parameter-expansion/ and this PR I pushed yesterday kojiromike/parameter-expansion#9 to address some bash'ism used and needed for APKGBUILD handling

@quepop what about adding the extra_data field on the DiscoveredPackage model, for consistency with Project and CodebaseResource models?
Let me know if that would work for you and I'll make that change upstream.

:+1:  ... this is being added too to SCTK FWIW

@pombredanne sounds great but what about APKBUILDs like these: https://github.com/nexB/scancode.io/issues/191#issuecomment-859129456?

@quepop extra_data added on the DiscoveredPackage model https://github.com/nexB/scancode.io/pull/222
Make sure to use the DiscoveredPackage.update_extra_data() API, see https://github.com/nexB/scancode.io/blob/main/scanpipe/models.py#L197

@tdruez @pombredanne
There is another issue that i found looking at my database. Some alpine images like docker://alpine:3.13 have subpackages installed from a parent package that is not installed (libcrypto, libssl from openssl). Handling cases like these isn't clear for me (because running the scan on the parent still leaves a problem of deciding what subset of the scan's output is applicable to the parent's children)
I've decided that in cases where a parent package is present, my code will update only its missing information (not its chilrdren).

@quepop not sure what to do wrt. subpackages... can you provide a concrete example with links and may sample APKBUILD snippets?

re:

@quepop not sure what to do wrt. subpackages... can you provide a concrete example with links and may sample APKBUILD snippets?

Actually the new approach in https://github.com/nexB/scancode-toolkit/blob/b7d070fc788f4c233df1f31d735e1ac5b3aa2d29/src/packagedcode/alpine.py#L144 could be enhanced to also collect these.

collect the variable and resolve that lists all the subpackages such as  https://gitlab.alpinelinux.org/alpine/aports/-/blob/9238f96475ab45281ce3d0cdfbf37efcbdeb8888/main/openssl/APKBUILD#L14
for each of these, parse the corresponding bash function with the same name and extract the variables defined inside a back function

Then merge subpackages variables with the base variables.
This way declared licensed, description and other sub-package-specific metadata could be collected.
Note that this would work whereas the approach to just eval the APKBUILD with bash would not be able to collect variables defined inside a function, as this would require to run the function, which is impractical since the function names are not known (unless parsed ...) .

See https://github.com/nexB/scancode.io/pull/272#issuecomment-1964170077
The feature will be implemented instead in PurlDB ...
See https://github.com/nexB/purldb/issues/95 and https://github.com/nexB/purldb/issues/237

Moving this to PurlDB


[Gekidou] Handles GroupMember Add/Delete WS Event

Summary
Handles a user being added or removed from a Group (GroupMembership event).
NOTE: Waiting https://github.com/mattermost/mattermost-mobile/tree/MM-45530-Gekidou-Group-Websocket to be merged in first.
Ticket Link
https://mattermost.atlassian.net/browse/MM-45531
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:
Screenshots

Release Note

@shazm you have conflicts here

@shazm you have conflicts here

Ah darn -  I'll fix up; I'm not sure if there's been a change in git's handling of PR's but basing a branch off a branch that gets merged used to be straightforward - now it always throws up conflicts :(


await userType so routing will work properly after login

awaiting userType so that the correct route is pushed after login

Correct. You did add await userType
...still not sure what to put for reviews


powerplatform_data_loss_prevention_policy added additional validation for default_action_rule_behavior and action_rules when both are empty

This pull request includes several updates and improvements to the Data Loss Prevention Policy functionality, primarily focusing on schema enhancements and validation logic. The most important changes include adding additional validation for default_action_rule_behavior and action_rules, updating the Go module version, and refining the schema attributes by removing redundant descriptions.
Validation Enhancements:

Added validation logic to ensure default_action_rule_behavior and action_rules are correctly configured when both are empty. (internal/services/dlp_policy/resource_dlp_policy.go)

Schema Attribute Refinements:

Removed redundant Description fields from schema attributes in DataLossPreventionPolicyDataSource and DataLossPreventionPolicyResource. (internal/services/dlp_policy/datasource_dlp_policy.go) [1] [2] [3] [4] [5]
Added MarkdownDescription to provide more detailed information and links to relevant documentation. (internal/services/dlp_policy/datasource_dlp_policy.go, internal/services/dlp_policy/resource_dlp_policy.go) [1] [2]

Module and Dependency Updates:

Updated Go module version to 1.22.9 in go.mod file. (go.mod)

New Helper Functions:

Introduced getConnectorGroup function to handle conversion of connector attributes. (internal/services/dlp_policy/helpers.go)

Configuration Validation:

Implemented ValidateConfig method in DataLossPreventionPolicyResource to ensure proper configuration of connectors. (internal/services/dlp_policy/resource_dlp_policy.go)

Codecov Report
Attention: Patch coverage is 67.53247% with 25 lines in your changes missing coverage. Please review.

Project coverage is 73.35%. Comparing base (1f1b233) to head (8952d67).

Files with missing lines
Patch %
Lines

...nternal/services/dlp_policy/resource_dlp_policy.go
52.50%
14 Missing and 5 partials :warning:

internal/services/dlp_policy/helpers.go
90.00%
2 Missing and 1 partial :warning:

internal/services/environment_settings/models.go
0.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #524      +/-   ##
==========================================
- Coverage   73.46%   73.35%   -0.12%
==========================================
  Files         105      105
  Lines       14949    14957       +8
==========================================
- Hits        10983    10972      -11
- Misses       3292     3308      +16
- Partials      674      677       +3

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


Fix U/I128 and U/I256 number representation for consistent sorting

See https://github.com/stellar/rs-soroban-env/issues/743#issuecomment-1499950221 for details.

Thanks @jayz22 !


Create cafebiiling.c

_Please help in in hacktoberfest to make my first pull request_

Please merge it

Thanks bro

Thanks
On Thu, 6 Oct, 2022, 1:09 pm Jara, @.***> wrote:

Merged #9 https://github.com/Jara18/hacktoberfest-2022/pull/9 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/Jara18/hacktoberfest-2022/pull/9#event-7531438997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU2VZEXH4Y5DQMFZQP5KNM3WBZ63DANCNFSM6AAAAAAQ2ZXGFA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Iptables for Docker

Hi,
I´ve installed my document and community server at debian 9 and struggling with my local iptables. Can anybody tell me, which docker iptable routing is necessary for onlyoffice? As I´m running the community server on localhost and I have to bind my database externally (my mysql server is running on all interfaces and onlyoffice can´t connect with localhost host setting), it seems that there is also a local iptables problems.
How does the docker onlyoffice network work?
Also I´m wondering, how I can set ip address filter with the docker image - example:
"ipfilter": {
           "rules": [{"address": "*", "allowed": true}],
           "useforrequest": false,
           "errorcode": 401

Regards,

Hello @benhartwich can provide the errors you get? And how you run the containers? See this guide?

Closed for a reason - there has been no activity for a long time.


:bug: Use correct case on verb for metrics-viewer ClusterRole

Summary
Some things take you much longer to find than you'd be willing to admit - This is one of them. I was trying to get metrics from /metrics with the metrics-viewer user we added in #3064, and it just. wasn't. working. I banged my head against that wall for several days.
Until I read the upstream documentation and realised the difference to our ClusterRole: The verb was all caps in ours. And - obviously, some might say - verbs in ClusterRoles are case-sensitive! So GET != get. This confusion is compounded by the fact that the HTTP verb, in fact, is GET. Kubernetes still seems to expect get here.
Binding to this ClusterRole gave me two different outcomes depending on which verb I had in there:
# this is with the current ClusterRole
$ kubectl http /metrics -v
GET /clusters/root/metrics HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Authorization: Bearer <token>
Connection: keep-alive
Host: <host>:6443
User-Agent: HTTPie/3.2.2

HTTP/1.1 403 Forbidden
Audit-Id: ef0fa904-3f29-4465-af42-302d67605742
Cache-Control: no-cache, private
Content-Length: 195
Content-Type: application/json
Date: Tue, 30 Jan 2024 15:32:56 GMT
X-Content-Type-Options: nosniff

{
    "apiVersion": "v1",
    "code": 403,
    "details": {},
    "kind": "Status",
    "message": "forbidden: User \"user\" cannot get path \"/metrics\": access denied",
    "metadata": {},
    "reason": "Forbidden",
    "status": "Failure"
}

# this is with `get` in the ClusterRole
 $ kubectl http /metrics -v
GET /clusters/root/metrics HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Authorization: Bearer <token>
Connection: keep-alive
Host: <host>:6443
User-Agent: HTTPie/3.2.2

HTTP/1.1 200 OK
Audit-Id: f9138fb7-7757-474f-a4b3-27b4d6a1c7b9
Cache-Control: no-cache, private
Content-Encoding: gzip
Content-Type: text/plain; version=0.0.4; charset=utf-8
Date: Tue, 30 Jan 2024 15:30:33 GMT
Process-Start-Time-Unix:<PHONE_NUMBER>
Transfer-Encoding: chunked

< ... metrics here ...>

Related issue(s)
Fixes #
Release Notes

Use correct verb in metrics-viewer ClusterRole to give access to `/metrics`

/lgtm
/approve

@stevekuznetsov long time no see! :D


Package request: edex-ui

Project description
eDEX-UI is a fullscreen, cross-platform terminal emulator and system monitor that looks and feels like a sci-fi computer interface.
Metadata

homepage URL: https://github.com/GitSquared/edex-ui
source URL: https://github.com/GitSquared/edex-ui
license: gpl3-3.0
platforms: unix, linux, macos, windows

Add a :+1: [reaction] to [issues you find important].

Archived three years ago?
Javascript?
No way...

The newest fork I found was that:
https://github.com/novaTopFlex/edex-ui-nokbd


Blocked content handling PoC

As an editor I don't want users to be able to see content that I decided should be blocked (legal, commercial, geoblocking). As a developer I would like to have an implementation strategy that matches business needs.
Acceptance criteria

Legal, commercial, etc. content is blocked.
Geoblocked content is blocked outside Switzerland.

Hints

On Apple platforms the following works more or less (some images can be seen when seeking is smooth):
queuePlayer.smoothCurrentTimePublisher(interval: .init(value: 1, timescale: 10), queue: .main)
      .receiveOnMainThread()
      .sink { [queuePlayer] time in
          if time.seconds > 120 && time.seconds < 240 {
              queuePlayer.seek(to: .init(value: 240, timescale: 1))
          }
      }
      .store(in: &cancellables)

On Apple platforms we could implement a smooth boundary time publisher to implement this mechanism, whether a forbidden time is crossed during normal playback or reached through seeking (we could combine traditional boundary time observation with our seekTimePublisher, e.g.). We should update (or implement) the existing issue we already have if this makes sense.

We probably should use a PlayerItemTracker to implement blocked segment tracking. Note that this requires us to implement selective tracker selection, since we don't want such trackers to be disabled when playing SRG SSR content. Maybe having two tracker lists (optional and mandatory) would provide a simpler API (to be discussed).

Maybe we should investigate how we can associated metadata (and notify the surrounding app about its changes) with blocked intervals. An app should be able to observe blocked information and respond appropriately (e.g. display a corresponding message).

Remark
We might have issues with standalone mode and geoblocking. In the following example, no blocked segments are returned for the full-length, which makes the content incorrectly playable. This should be reported to the backend team (Letterbox and Pillarbox players are all affected).
Tasks

[ ] Investigate possible approaches.
[ ] Implement the approach that best makes sense.
[ ] Report geoblocking issues with standalone playback to the backend team.

We implemented two PoCs with various tradeoffs. In the two cases we first extracted segments for the current chapter in the media composition.
First PoC: Player item tracker
We can use a PlayerItemTracker implementing a periodic time publisher to detect when a forbidden time range is reached.
Pros

No API changes made in PillarboxPlayer. Our business need of handling blocked time ranges has no visible API impact on the player.

Cons

Some forbiddden images can be briefly seen when seeking smoothly into forbidden ranges. This is due to the fact that we cannot change a seek target time from a player item tracker. Once a seek request has been made, and even if we detect we must seek somewhere else immediately, images from the first seek might still be seen.
Cannot be integrated as interstitials on tvOS.

Remaining work

We need to have a way to avoid such trackers to be disabled with isTrackingEnabled, as they are mandatory.

Second PoC: Support in player metadata
We can add forbidden time ranges to PlayerMetadata. This way we can observe time to detect when forbidden ranges are entered. Moreover we can have seeks directly prevent target times in forbidden ranges. For a better result the forbidden ranges must be provided to the QueuePlayer so that low-level seeks, no matter where they are made from, prevent access to forbidden regions.
Pros

No forbidden images can be briefly seen. The only exception is when casting over AirPlay. In such cases we can still briefly seek into forbidden regions.
Can be integrated as interstitials on tvOS (but not when casting over AirPlay).

Cons

The PillarboxPlayer API must be changed to address a business use case that maybe makes sense only to us.

After investigating how #845 could be implemented it appears that the second strategy offers a common formalism to handle credits as well as blocked intervals.
For this reason the business discussion about briefly seen forbidden frames is irrelevant. Still remains the discussion about how packaging could avoid forbidden images to be delivered entirely.


Getting stuck when starting to debug a notebook cell

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Debugging of a notebook cell is stuck forever when started.
This is related to issue 13349.
MWE (debug random python function in cell):
slice()

VS Code Version
Version: 1.85.1 Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2 Date: 2023-12-13T09:49:37.021Z Electron: 25.9.7 ElectronBuildId: 25551756 Chromium: 114.0.5735.289 Node.js: 18.15.0 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.19044
Jupyter Extension Version
v2023.11.1003402403
Jupyter logs
No response
Coding Language and Runtime Version
Python v3.10.12
Language Extension Version (if applicable)
Pylance v2023.12.1
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

@moesphere sorry you are running into this issue.
Please can you share more information than this is related toxxx

Please can you providce steps to repro the issue
Please can you share the logs as follows

Please could you enable logging as follows:

Open settings & go into Jupyter->Logging
Change the value to verbose
Reload VS Code,
Attempt to repro the issue & then copy the output from the Jupyter output panel.

@DonJayamanne this is likely a duplicate of https://github.com/microsoft/vscode-jupyter/issues/14958

Yes, it is a duplicate of #14958.
I am experiencing the same as mentioned in the issue: If I switch to a different python version, it works.
I will comment in the original issue.


SSR web worker build with noExternal: true omit dynamic imports

Describe the bug
I am trying to update a SSR Vue app to<EMAIL_ADDRESS>and deploy to a Cloudflare Worker.
On Cloudflare I'm getting a warn:
"[Vue warn]: Component <Anonymous> is missing template or render function."
On client side I'm getting an hydration error.
Works properly on dev mode.
As far as I understand the issue is related to missing dynamic imports in the server bundle.
Cloudflare Workers don't support dynamic imports. 😕
I reproduce the behavior on StackBlitz.

Am I doing something wrong?
Reproduction
https://stackblitz.com/edit/node-tmtkzw
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    @vitejs/plugin-vue: ^3.0.1 => 3.0.1
    vite: ^3.0.3 => 3.0.3

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

You are importing App.vue and mounting that. But that's an empty file.
This is the reason why it's not working.

Hi @sapphi-red
I forgot to add the router-view to App.vue.
I do it and I still get the dynamic imports out of the SSR bundle.
As I understand noExternal: true should embed all modules in the bundle.
With<EMAIL_ADDRESS>works
https://stackblitz.com/edit/node-hq9vb8

Ah, sorry this condition was wrong.
https://github.com/vitejs/vite/commit/3b41a8eb1006ae636e2b01b8416327e001625df4#diff-aa53520bfd53e6c24220c44494457cc66370fd2bee513c15f9be7eb537a363e7R450


奇怪的字符串定界符

我想要一个包含双引号的字符串，按照Python的习惯我写了 'a"b"c"' 结果报错。
然后我发现可以这样写：
Shell.println("abc');

是不是需要"修复"一下呢？

Because kula-lang is a real-time compiled language, we choose to ignore this problem of high cost and low benefit. (kusa)
Kula is not designed to distinguish between single and double quotation marks, that is, you can use single quotation marks to close a double quotation mark, and vice versa. This is obviously a bad design, but in fact, it doesn't affect anything except driving developers crazy.
But kula itself supports escape characters. You can still use  \ " to escape double quotes like other languages to create a string containing them.

Because kula-lang is a real-time compiled language, we choose to ignore this problem of high cost and low benefit. ~(kusa)~
Kula is not designed to distinguish between single and double quotation marks, that is, you can use single quotation marks to close a double quotation mark, and vice versa. This is obviously a bad design, but in fact, it doesn't affect anything except driving developers crazy.
But kula itself supports escape characters. You can still use  \ " to escape double quotes like other languages to create a string containing them.

Translation: The author is so lazy that she do not want to fix this bug at all! Oh, no! In other words, it isn't not a bug... We can call it a 'feature' of kula-lang instead!
Kula-lang is such a perfect language that it hasn't any bugs! Yeah! What a good deal made in China!


Implement the HTMLTableElement

That was a beast. I hope I implemented everything according to the spec

Hey @speich, thanks so much for your contribution. This looks great - a big piece of functionality with some good tests. I'll checkout on my computer later and run the tests and do some debugging. We're getting there with the final HTMLElements!

I just started to implement the HTMLTableColElement which has the attribute span. Its value is required to be a non-negative integer clamped to [1, 1000]
The spec lists rules for parsing different data types https://html.spec.whatwg.org/multipage/common-microsyntaxes.html#rules-for-parsing-non-negative-integers
Is there already a class in the facade, that already implements these rules and that I can use ?  If not, what do suggest I'll do ?

I've decided to set my IDE to display PHPStan level 6 warnings by default. It's overkill, but it helps me better understand what the code's doing, which I suppose is exactly the point.
I went through and added extra type hint comments where necessary to suppress PHPStan's warnings, and in the process it allowed me to understand what the tests and functionality is doing in your branch.
Thanks for your work on this. I'll spend a little bit more time reading through it and merge it later if I don't have any questions.

Thanks again @speich . One thing I'm looking into that you might be able to help complete is two lines of untested functionality:
https://github.com/PhpGt/Dom/blob/5f478883175a674c525ff6d4f4709500706cc5e5/src/HTMLElement/HTMLTableElement.php#L311
and
https://github.com/PhpGt/Dom/blob/5f478883175a674c525ff6d4f4709500706cc5e5/src/HTMLElement/HTMLTableElement.php#L445
I'm taking a look if I can create some unit tests to test these lines, but I think they might actually be obsolete and never end up being executed.
Will report back if I find anything.

Good catch!
https://github.com/PhpGt/Dom/blob/5f478883175a674c525ff6d4f4709500706cc5e5/src/HTMLElement/HTMLTableElement.php#L310-L312
The idea was to filter out rows of tables inside the table and that are direct children of the table (don't have a tbody as parent). But this is already taken care of by (I don't need to compare table with table, null will do too)
https://github.com/PhpGt/Dom/blob/5f478883175a674c525ff6d4f4709500706cc5e5/src/HTMLElement/HTMLTableElement.php#L313-L315
Also the test
https://github.com/PhpGt/Dom/blob/5f478883175a674c525ff6d4f4709500706cc5e5/test/phpunit/HTMLElement/HTMLTableElementTest.php#L381-L396
never creates a tr that is an immediate child of the table, because insertRow() automatically inserts a tbody first. Test should be modified accordingly to create a tr directly and append it.
https://github.com/PhpGt/Dom/blob/5f478883175a674c525ff6d4f4709500706cc5e5/src/HTMLElement/HTMLTableElement.php#L444-L446
I added this probably for completeness, but as you spotted, it is never used.

Thanks for the explanation. It's all very readable in code too. I'll make any adjustments while I go through some more HTMLElement classes and get closer to full implementation.

Follow-on PR: #273


ClusterQueue update events continue firing forever

What happened:
ClusterQueueStatus.flavorsUsage.resources with random order continues to fire ClusterQueue update events forever, and the number of reconciles will explode.
Therefore kueue-controller-manager much increases the load of kube-apiserver and etcd.
What you expected to happen:
ClusterQueues aren't updated if the old and the new ones have the same usage.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version): v1.25
Kueue version (use git describe --tags --dirty --always): v0.3.2
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign

The problem part: https://github.com/kubernetes-sigs/kueue/blob/b4635731cad8ec334d622ff4db60bfe1fbc13f77/pkg/cache/cache.go#L867-L881

Are we triggering a reconcile even if the usage didn't change?

Are we triggering a reconcile even if the usage didn't change?

Yes.
if ClusterQueueStatus.flavorsUsage.resources are changes, it fires update event.
For example, in the following diffs fire update event:
status:
...
  flavorsUsage:
  - name: foo
    resources:
    - borrowed: "0"
      name: "cpu"
      total: "4"
    - borrowed: "0"
      name: "memory"
      total: "4"

status:
...
  flavorsUsage:
  - name: foo
    resources:
    - borrowed: "0"
      name: "memory"
      total: "4"
    - borrowed: "0"
      name: "cpu"
      total: "4"

This issue will often happen in the HPC cluster. Because in HPC cluster, pods have multiple SR-IOV Virtual functions in resources.Resources like this:
resources:
  requests:
    cpu: 256
    memory: 512Gi
    example.com/gpu: 8
    example.com/vf-0: 1
    example.com/vf-1: 1
    example.com/vf-2: 1
    example.com/vf-3: 1
    example.com/vf-4: 1
    example.com/vf-5: 1
    example.com/vf-6: 1
    example.com/vf-7: 1

ah I see. we should probably do an internal alphabetical sort

ah I see. we should probably do an internal alphabetical sort

Right. I will sort outFlvUsage.Resources according to resourceName in https://github.com/kubernetes-sigs/kueue/blob/b4635731cad8ec334d622ff4db60bfe1fbc13f77/pkg/cache/cache.go#L850.
Also, I will apply this way in the localQueue usage as well.


Tor support

Can you add tor support for ssh connections? That would be awesome!

Hello @P1xelfehler 👋
Thank you a lot for your issue 🙏
I'm closing this in favor of https://github.com/isontheline/pro.webssh.net/issues/362


Css loaded after js

Hey, I'm having a problem loading css correctly.
This i the situation:
I'm using DateRangePicker
I load the css in my base.scss class (the first "scss" node in tree) the following:
@import '~react-dates/css/variables.scss';
@import '~react-dates/css/styles.scss';

And I can't figure out why is this happening or how to solve it.
it happens in every refresh

This is the link for "react-dates" issue I opened maybe the comment can help, they think its because the js/css loading isn't right.

This problem was also confused me for a long time.
Finally, I found it to be solved after building and dispatching. So I don't need to worry about it.


test(composer): tx integrity test

Summary
This adds a test to verify that that payload of the sequencer transaction submitted by the Searcher does not change the rollup transaction that it collected.
Testing

Added the single_rollup_tx_payload_integrity test that verifies that a transaction with a single payload contains a payload that was serialized properly.

When we support packing multiple rollup transactions into a single sequencer transaction in the composer we will need to change this test to encode/decode a Vec<Transaction> instead of a single one.

Related Issues
closes #213

Undid the changes to rollup transaction encoding that were previously part of this PR and cleaned up the tests a bit. The test now compared the RLP encoding of the expected transaction to the one inside the sequencer transaction that the composer submits to verify that the transaction was encoded as expected.


Mixed Bilinear Form Integrator for Strong Laplacian in IGA

As far as I saw there is now implementation of bilinear forms involving strong Laplacians.
In the IGA setting, this would make sense for C^1 elements and might be beneficial for least squares problems or the biharmonic equation.
 I would volunteer to implement them, but I might need some help from more advanced MFEM developers on the way. (I have already tweeked MFEM here and there, but never pushed something)
Greetings,
Michael

I have started implementing this as a "mfem::MixedDiffusionDiffusion" integrator.
Right now I encountered two things, where I would need help:
-) I have read the "CONTRIBUTING.md" file and proceeded as stated there. But I cannot push a development branch to the server, as I am only in the users group. What do I need to do to contribute.
-) I did not find much about IGA on the website. Is there a resource, to find out how the "mesh" input files are structured and how MFEM does handly NURBS geometries?
Thanks and greetings,
Michael

Regarding the mesh documentation I was doubting what to put on the website.
Cox de Boor algorithm and such is a lot of effort to put on the site but not useful.
I did not think about the mesh format... but that is an obvious one.
I will try to write something, perhaps you (@michi002) can help me by judging whether it is clear and useful.

Hi Michael,
You should be able to contribute now.
Best regards, Boyan

I opened a related issue #4238 related to the Laplace terms. I have also started a NURBS section on the website, related to the NURBS H(div) and H(curl) PR.
I have implemented the terms as mention in the issue. I called itLaplaceLaplaceIntegrator and BiHarmIntegratoras alias.

@IdoAkkerman
Perfect. I saw your post today, but was on the mobile, so you were first.
Did you already implement this bilinear forms? If yes on which branch?
I would happly help you in improving the documentation, as well as coding the bilinear forms, if it did not happen yet.
Some questions arised in your post:
Do you want to have this operators for H^2 conforming functions (that is what I need), implying that we need C^1 ansatz functions. Do you also want to consider instead of the laplacian operators of the form
$Tu = \nabla \cdot \left( A \nabla u\right)$
wher $A$ is a matrix coefficient with the bilinear form
$a(u,v) = \langle Tu, Tv\rangle$?
This would imply, that we somehow would have to deal with the chain rule for non-constant coefficients.
Greetings,
Michael

I have not commit yet. I can do that tomorrow after noon.

I have committed my stuff in  the branch dev-stab-mini .
The miniapps in miniapps/stabilized are still very rough. But things seem to converge:
For instance:

./laplace -b -o 3  shows the convergence of the Galerkin form for the biharmonic problem. With essential BCs on the solution.
./ex_condif -s 1 -o 3 shows the convergence of the SUPS formulation for convection diffusion.

@IdoAkkerman did push it to the general mfem repository? I cannot find that branch (also not on the Website).

And thank you for sharing :)


Add get-data view and button

It implements a function that allow an user to download the data about what was already done. One can download only cities that have already been mapped or an user can download a list with all cities and those without any kind of information will come with empty columns.
NOTE: the way it's implemented, cities that starts with an accent on the first letter will be in end of the file.

@sergiomario fiz as alteracoes e arrumei o sort antes de exportar.


DNA barcode sequence field in BOLD tsv

Using the latest BOLD tsv released on April 5th 2024, the field indicating DNA nucleotide barcode sequence is nucraw.
This is not compatible with the code.

Is it possible that this is the same issue as #88, which @AnnemiekeSchonthaler has been working on?

Edited the code so it can take both nuc and nucraw as column names now

Is this throughout the code base? It looks like nucraw is still being referenced in various locations: https://github.com/search?q=repo%3Anaturalis%2Fbarcode-constrained-phylogeny nucraw&type=code

Replaced in all occurrences.


Add UG, my PPP and update AboutMe

User Guide was originally from our initial draft last team meeting. I made some minor grammatical/formatting changes.
Added myself into AboutMe.
Feel free to use the overall flow/template/guide for my PPP for your own PPPs as well

Codecov Report

Merging #41 (90dfe47) into master (6b7e4ad) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 6b7e4ad...90dfe47. Read the comment docs.

Related: #49


Make notes on Femnist dataset

• Federated Extended MNIST (FEMNIST), which serves as a similar (and yet more challenging)
benchmark to the popular MNIST [13] dataset. It is built by partitioning the data in Extended
MNIST [5] based on the writer of the digit/character

raw_data
$ ls data/raw_data/
by_class  by_write

by_class
ls data/raw_data/by_class/
30  33  36  39  43  46  49  4c  4f  52  55  58  61  64  67  6a  6d  70  73  76  79
31  34  37  41  44  47  4a  4d  50  53  56  59  62  65  68  6b  6e  71  74  77  7a
32  35  38  42  45  48  4b  4e  51  54  57  5a  63  66  69  6c  6f  72  75  78

30-39 is numbers 0-9
4a (or 65) - 5a(90) is small a-z
and above it is big A-Z
by_write
$ ls data/raw_data/by_write/hsf_0/
f0000_14  f0046_20  f0092_16  f0138_30  f0184_49  f0230_04  f0276_07  f0322_19  f0368_39  f0414_22  f0460_02
f0001_41  f0047_00  f0093_24  f0139_23  f0185_23  f0231_43  f0277_33  f0323_17  f0369_34  f0415_28  f0461_47 ....

A total of 500 such directories.
Each of these directories has 4 sub-folders.
$ ls data/raw_data/by_write/hsf_0/f0001_41/
c0001_41  d0001_41  l0001_41  u0001_41

Under each of these folders are image files.

On displaying files under one of these folders, I can see that these are different hand-written characters.
from IPython.display import Image, display

base_dir = "/home/jupyter/leaf/data/femnist"

image_dir = os.path.join(base_dir, "data/raw_data/by_write/hsf_0/f0000_14/c0000_14")

import os

files = []

for r, d, f in os.walk(image_dir):
    for file in f:
        if '.png' in file:
            files.append(os.path.join(r, file))

for imageName in files:
    print(imageName)
    display(Image(filename=imageName))

hsf_0/f0000_14/c0000_14/c0000_14_00050.png --> e
hsf_0/f0000_14/c0000_14/c0000_14_00129.png ---> w
f0000_14/c0000_14/c0000_14_00042.png --> e
... and so on

TODO: I am not sure how does the file names correspond to writes at the moment.

$ for i in $(ls data/raw_data/by_write/); do echo $i - $(ls data/raw_data/by_write/$i | wc -l); done
hsf_0 - 500
hsf_1 - 500
hsf_2 - 500
hsf_3 - 598
hsf_4 - 500
hsf_6 - 499
hsf_7 - 500

The above shows number of users per top level folder in by_write/* .
I am not sure why users have been clubbed together.
One possibility is that each of these hsf_X corresponds to a node in fed-learn algo. So we balance the number of files to some extent.
hsf_0 - 174415
hsf_1 - 162723
hsf_2 - 135783
hsf_3 - 89210
hsf_4 - 82587
hsf_6 - 85778
hsf_7 - 83759

Though I can see that these are not evenly balanced. There is a decreasing number of samples per node above (ie. if they are nodes).

Also from file, get_file_dirs.py, which generates some intermediate files from the raw data.
** Note: I am interested in by_write below
Creates .pkl files for:
1. list of directories of every image in 'by_class'
2. list of directories of every image in 'by_write'
the hierarchal structure of the data is as follows:
- by_class -> classes -> folders containing images -> images
- by_write -> folders containing writers -> writer -> types of images -> images
the directories written into the files are of the form 'raw_data/...'

to be continued...
The end goal is to understand data input for leaf.


I doesn't search git gutter plugin?

VSCode Version:1.11.1
OS Version:10.12.4

Steps to Reproduce:
1.I hope to find a git plugin to manage my github project.So I find git gutter,
2.But I search my vscode's  extension,doesn't find it

There are several extensions helping with git and github: https://marketplace.visualstudio.com/search?term=github&target=VSCode&category=All categories&sortBy=Relevance


indianajones pinball adventure

wrong link for table goes to b2s.  rom volume near silent.

Switched the correct links.  Volume issue? You need a keyboard and have to do the open coin door volume fix. https://vpuniverse.com/vpuwiki/mjr_wiki/audio-sounds/how-to-adjust-the-sound-volume-of-roms-r9/


Add file path autocompletion.

I write a lot of js code with require statements in them. We have files all over the place, and it's hard to remember where everything is. It would be great if I could autocomplete file paths (relative to the current file). What would it take to implement something like this?
For reference, neocomplete in vim does a great job of this: https://github.com/Shougo/neocomplete.vim

This package is deprecated and no longer shipped with Atom. If this is still an issue using the autocomplete-plus package, please open up a new issue on that repository, thanks.


Unknown choices in  Modelica/Modelica/Fluid/Fittings.mo

In several functions in Modelica/Modelica/Fluid/Fittings.mo the names that appear in choices annotations refer to classes that do not exist. Code block starting on line 1298:
      choices(
      choice=Modelica.Fluid.Fittings.Utilities.QuadraticTurbulent.LossFactorData.wallFriction(),
      choice=Modelica.Fluid.Fittings.Utilities.QuadraticTurbulent.LossFactorData.suddenExpansion(),
      choice=Modelica.Fluid.Fittings.Utilities.QuadraticTurbulent.LossFactorData.sharpEdgedOrifice()));

There is no package Utilities in Modelica.Fluid.Fittings. Perhaps it is supposed to be BaseClasses instead of Utilities.
Another block with unknown names is:
      choices(
      choice=BaseClasses.PressureLosses.QuadraticTurbulent.LossFactorData.wallFriction(),
      choice=BaseClasses.PressureLosses.QuadraticTurbulent.LossFactorData.suddenExpansion(),
      choice=BaseClasses.PressureLosses.QuadraticTurbulent.LossFactorData.sharpEdgedOrifice()));

It appears on lines 1340, 1438, 1480, and 2033. There is no package PressureLosses in BaseClasses. It should be BaseClasses.QuadraticTurbulent.LossFactorData....
The problem appears in the master branch, and in maint/3.2.2. The line numbers are the same in both branches.

Resolved in master and maint/3.2.2 branches.


Type of fields on non-model classes resolves to return type

Fields on non-model classes are resolving to the return type as if they were on a model.  Use case is binding the field with other data in order to access its field name and to_python methods while doing data collection.
from typing import Generic, Text, TypeVar
from django.db import models

A = TypeVar('A')
B = TypeVar('B')

def field_name(field: models.Field[A, B]) -> Text:
    reveal_type(field)  # Revealed type is 'django.db.models.fields.Field[A`-1, B`-2]
    return field.name

class BoundWithField(Generic[A, B]):
    field: models.Field[A, B]
    def field_name(self) -> Text:
        reveal_type(self.field)  # Revealed type is 'B`2
        return self.field.name  # "B" has no attribute "name"

At the moment, the stubs represent the most common scenario of fields being instantiated and assigned to attributes inside of model classes.
To make that work, Field in the stubs implements the descriptor protocol (e.g. has a __get__ method).
It's not exactly the truth of how it works in Django (after some poking through the code). Field in Django itself isn't a descriptor.
When a model is instantiated, Django's ModelBase metaclass actually adds the field to the class wrapped in a django.db.models.query_utils.DeferredAttribute, which is a descriptor.
We could choose to represent this in the stubs, and make them "less-usable" in the model field definition scenario if the stubs are used without the mypy django plugin (e.g. with another type-checker). In that case, the mypy django plugin could handle the job of wrapping the attribute in the descriptor on the model when it is defined.
See also https://github.com/python/typing/issues/399

I've fixed it in https://github.com/typeddjango/django-stubs/pull/109/commits/ad44a4035365e0869780143624080e1545c6ab62


CocoaPods

I'd be cool to also add Cocoapods support. I done this implementation myself last week for my library. Just follow this post and this checklist.

While I agree that the more options the better, I don't agree with CocoaPods way of thinking on the dependency management subject. It is my personal opinion that it obscures more than it helps for beginners, and makes things more complicated for experienced developers.
That's not to say I won't add support for it. That's to say I'll need more demand first. (:

@lsunsi I understand your opinion and I read about all the pros and cons for CocoaPods and Carthage. I've tried both and CocoaPods currently fits my needs.
CocoaPods is way more than just the installing libraries. Each CocoaPods library also has a CocoaDocs page on which the user can easily read the documentation.
CocoaPods also provides a quality index based on tests, documentation, stars... And we, from AwesomeiOS, will start to use this index to do the ranking of our libraries.
If you just don't wanna do it, I could do it for you and make sure it is updated every time there's a new version of ReduxKit.
Thank you for considering the option.

@lfarah I definitely agree about the index and ratings being a strong point for CocoaPods. I see your point now. I will bring support for CocoaPods in future updates of this library, you don't need to worry. I'll just reopen this issue so I don't forget. (:
Thanks man!


Improve RuboCop internal error message

In https://github.com/Shopify/ruby-lsp/issues/2992, the error is likely from rubocop-rails rather than RuboCop itself.

I think this is a good improvement, but we should probably print the exception to OUTPUT to make it easier to debug too?
This should work:
diff --git a/lib/ruby_lsp/server.rb b/lib/ruby_lsp/server.rb
index f975e065..34c15fcc 100644
--- a/lib/ruby_lsp/server.rb
+++ b/lib/ruby_lsp/server.rb
@@ -651,6 +651,8 @@ module RubyLsp
       ))
       send_empty_response(message[:id])
     rescue StandardError, LoadError => error
+      send_log_message(error.full_message)
+
       send_message(Notification.window_show_message(
         "Formatting error: #{error.message}",
         type: Constant::MessageType::ERROR,


Fix/#298 sort by price

Use a column renderer for total price formatting
Fixes unexpected ordering of prices which was caused by ordering formatted Strings

 Sorting by price not working as expected #298

Neat fix, good work!
Nevertheless, I think there are some more grids that need to have the ordering fixed.
This might be the grid

in the MaintainProducts section
in the CreateOffer section (the product grids, the overview grids)

Thanks for the input! I fixed it in the MaintainProducts section and noticed someone else already implemented it correctly in the CreateOffer section. It should work everywhere now!

I think this PR can now be merged into development 🚀

please merge development into this branch with the new alpha version in the CL. Then you can update the CL accordingly :)


Problem with libcurl3!

Hi. During installation(debian 10) an error comes out because in debian 10 it is no longer present in the libcurl3 library, but instead is present only in version 4, that is libcurl4. How to solve?

There's a work around here:
https://github.com/adobe/brackets/issues/14171

I've already tried.  but it does not work!