Update to pandas 2.1.2

Main objective of this PR is to update the pandas requirement to 2.1.2.
Unfortunately, the CI failed initially. It seems that some functions within pandas 2.1.2 use in-place modification. When called on a DataSet, it raises an error. For more details, see for example this run.
The solution I take here is to automatically convert a DataSet to DataFrame when calling a DataFrame function, such that the immutable option does not become a problem. Shouldn't be a problem, since all DataFrame functions called on a DataSet would convert the DataSet to a DataFrame anyway. Now we just do it upfront.

Main objective of this PR is to update the pandas requirement to 2.1.2.
Unfortunately, the CI failed initially. It seems that some functions within pandas 2.1.2 use in-place modification. When called on a DataSet, it raises an error. For more details, see for example this run.

Did you file an issue with this to Pandas? Seems bad that this should happen with just a patch version increase (or at all really pandas shouldn't be doing anything in place unless you explicitly ask any more). Note that I didn't actually look at the code for why it failed I am just commenting on your comment here.

The solution I take here is to automatically convert a DataSet to DataFrame when calling a DataFrame function, such that the immutable option does not become a problem. Shouldn't be a problem, since all DataFrame functions called on a DataSet would convert the DataSet to a DataFrame anyway. Now we just do it upfront.

Did you file an issue with this to Pandas? Seems bad that this should happen with just a patch version increase (or at all really pandas shouldn't be doing anything in place unless you explicitly ask any more).

Thanks, good point! I did some debugging. The following raises an error (in pandas 2.1.2, prior to this PR):
import pandas as pd
from strictly_typed_pandas import DataSet

class B:
    id: int
    age: int

df1 = pd.DataFrame(
    dict(
        id=[1, 2, 3],
        name=['a', 'b', 'c'],
    )
)

df2 = DataSet[B](
    pd.DataFrame(
        dict(
            id=[1, 2, 3],
            age=[10, 20, 30],
        ),
    ).set_index('age')
)

df3 = df1.merge(df2, on='id')

Error:
File ~/.pyenv/versions/3.11.2/envs/strictly_typed_pandas/lib/python3.11/site-packages/pandas/core/frame.py:10487, in DataFrame.merge(self, right, how, on, left_on, right_on, left_index, right_index, sort, suffixes, copy, indicator, validate)
  10468 @Substitution(\"\")
  10469 @Appender(_merge_doc, indents=2)
  10470 def merge(
   (...)
  10483     validate: MergeValidate | None = None,
  10484 ) -> DataFrame:
  10485     from pandas.core.reshape.merge import merge
> 10487     return merge(
  10488         self,
  10489         right,
  10490         how=how,
  10491         on=on,
  10492         left_on=left_on,
  10493         right_on=right_on,
  10494         left_index=left_index,
  10495         right_index=right_index,
  10496         sort=sort,
  10497         suffixes=suffixes,
  10498         copy=copy,
  10499         indicator=indicator,
  10500         validate=validate,
  10501     )

File ~/.pyenv/versions/3.11.2/envs/strictly_typed_pandas/lib/python3.11/site-packages/pandas/core/reshape/merge.py:169, in merge(left, right, how, on, left_on, right_on, left_index, right_index, sort, suffixes, copy, indicator, validate)
    154     return _cross_merge(
    155         left_df,
    156         right_df,
   (...)
    166         copy=copy,
    167     )
    168 else:
--> 169     op = _MergeOperation(
    170         left_df,
    171         right_df,
    172         how=how,
    173         on=on,
    174         left_on=left_on,
    175         right_on=right_on,
    176         left_index=left_index,
    177         right_index=right_index,
    178         sort=sort,
    179         suffixes=suffixes,
    180         indicator=indicator,
    181         validate=validate,
    182     )
    183     return op.get_result(copy=copy)

File ~/.pyenv/versions/3.11.2/envs/strictly_typed_pandas/lib/python3.11/site-packages/pandas/core/reshape/merge.py:797, in _MergeOperation.__init__(self, left, right, how, on, left_on, right_on, left_index, right_index, sort, suffixes, indicator, validate)
    794     self.left = self.left._drop_labels_or_levels(left_drop)
    796 if right_drop:
--> 797     self.right = self.right._drop_labels_or_levels(right_drop)
    799 self._maybe_require_matching_dtypes(self.left_join_keys, self.right_join_keys)
    800 self._validate_tolerance(self.left_join_keys)

File ~/Documents/projects/2022/strictly_typed_pandas/strictly_typed_pandas/immutable.py:43, in inplace_argument_interceptor.<locals>.func(*args, **kwargs)
     40 if \"inplace\" in kwargs and kwargs[\"inplace\"]:
     41     raise NotImplementedError(immutable_error_msg)
---> 43 return call(*args, **kwargs)

File ~/.pyenv/versions/3.11.2/envs/strictly_typed_pandas/lib/python3.11/site-packages/pandas/core/generic.py:1922, in NDFrame._drop_labels_or_levels(self, keys, axis)
   1920     # Handle dropping columns labels
   1921     if labels_to_drop:
-> 1922         dropped.drop(labels_to_drop, axis=1, inplace=True)
   1923 else:
   1924     # Handle dropping column levels
   1925     if levels_to_drop:

File ~/Documents/projects/2022/strictly_typed_pandas/strictly_typed_pandas/immutable.py:41, in inplace_argument_interceptor.<locals>.func(*args, **kwargs)
     38     raise NotImplementedError(immutable_error_msg)
     40 if \"inplace\" in kwargs and kwargs[\"inplace\"]:
---> 41     raise NotImplementedError(immutable_error_msg)
     43 return call(*args, **kwargs)

NotImplementedError: To ensure that the DataSet adheres to its schema, you cannot perform inplace modifications. You can either use dataset.to_dataframe() to cast the DataSet to a DataFrame, or use operations that return a DataFrame, e.g. df = df.assign(...)."
}

Let's reproduce that without stp. And let's see if there are any side-effects (due to in-place modifications) in df2.
import pandas as pd

df1 = pd.DataFrame(
    dict(
        id=[1, 2, 3],
        name=['a', 'b', 'c'],
    )
)

df2 = pd.DataFrame(
    dict(
        id=[1, 2, 3],
        age=[10, 20, 30],
    ),
).set_index('age')

df3 = df1.merge(df2, on='id')

df2

Output:

Appears like there are no side-effects.
So I guess pandas does an inplace modification of a copy of the DataFrame. And since we subclass the DataFrame into a DataSet (where we forbid inplace modifications) this results in an error. I could report it with them, but I don't think they'd consider this part of their public interface.
Anyhow, it seems like we got a workaround. I'll merge this now!


Stop catching i.stack.imgur.com

"Misleading link" linking to i.stack.imgur.com is too falsey.
See the MS search.
+Another minor tweak.

Coverage remained the same at 65.71% when pulling 1372ff01d7ceb107081139c3eaf33ca4d0929a06 on iBug:patch-4 into 3c8b8d092d300189d06ec5bc8692f31bdeea2d57 on Charcoal-SE:master.

FWIW, the cases that resulted in the original misleading link filter were a user pretending to link to documentation/JSFiddle, but the links actually went to porn on stack.imgur.
I’ve thought about whitelisting imgur if the original link looks like a photo (maybe if the URL contains photo, `png, etc?).  We could also try sending a request to the URL in the text and checking if it’s a photo.

@NobodyNada We're going to hit a lot of documentations and code segments and GitHub repos if we try to determine the URL in link text.

@iBug But how many of those 1) link to imgur and 2) already trip the misleading link filter?

@NobodyNada A few every day.


Add check for 'onboarding' on aws and azure in the future to ensure that the users have a fixed ip range before we create a whitelist on MC API

Add check for 'onboarding' on aws and azure in the future to ensure that the users have a fixed ip range before we create a whitelist on MC API.

Done. We have a check list: https://docs.google.com/document/d/1e4GNMMHMhE7U8Y4rzYBA2azuMrt-Bj9d6FHj2_FWu2c/edit .


presence: Document that a user can disable their presence updates.

This is a crucial detail, as it makes UserPresence an unreliable
indicator of when users were recently active. It should be documented
more clearly.

I'll add a docstring to UserActivity and a bit more text and then merge this.  Thanks for doing this @mateuszmandera!


Hotfix package objects

@bherd-rb this PR should fix #19.  I have moved Tradable as well as some other objects that I think should clearly be top level in their respective packages out of the package object.  I have left certain classes inside the package object.

@bherd-rb Do you think that Price and Quantity should be moved out of the package object as well? Have you encountered issues in accessing these classes from Java?  There is not any reason why a user should want to extend either Price or Quantity.

@bherd-rb Upon reflection I have moved the Price and Quantity case classes outside of the auctions package object. This leaves only the type alias Currency in the package object (which is where type aliases that need to be accessible throughout a package must live).

@davidrpugh Yes, I also had some problems accessing Price and Quantity. I moved them to a separate file for testing purposes but what you've done now is clearly a better solution.


Game crash Tweak XL

Tweak XL is not functional for me. I've tried install Vortex, Github, newest- older versions, have tried disabling redmod, have tried on steam and used separate file (Backup file location). I've even tried putting files into other locations(shot in the dark), but nothing seem to work except disabling the mod. Tips?

This is not an issue. The game was updated again, and requires another update. Please be patient for psi to update the mods. You may follow updates on the cyberpunk modding discord.

This is not an issue.
requires another update.


Feedback data section

[x] Define the abbreviation SNR = "Supernova Remnant"
[x] For gamma-cat I'd prefer you link to the Github repo. I don't know yet if we'll keep showing a catalog at gammapy.github.io/gamma-cat .
[x] The URLs for SNRcat and gamma-cat are in Table 2. Suggest to remove them from the main text to avoid the duplication and save space
[x] Suggest to remove the sentence "The university of Manitoba maintains ...". Samar and Gilles maintain it, and I think Gilles has already moved to another university.
[x] I see you talk about SESAME now. I think I would just move this last paragraph to the map / Aladin Lite features section. If you consider the databases that back up SESAME to be part of the "data" that needs to be described for gamma-sky.net, OK. I'd still suggest you mention SESAME in the maps features though.

I would remove the last sentence that some images are currently fetched from 3FGL-table.
That was just a temp solution to have something to show in the group meeting, and it's not live at gamma-sky.net.
They don't even have 2FHL plots there, we'll just write a script to make both 2FHL and 3FGL plots for gamma-sky.net and serve the PNGs ourselves.
(see https://gammapy.readthedocs.io/en/latest/spectrum/plotting_fermi_spectra.html)
It'll also be good to have this single-sentence paragraph on the next page gone for paper structure.
I do think we should link to "3FGL-table" somewhere, like we do from
https://github.com/gammapy/gamma-sky/wiki/Other-projects
In the paper this could either be in the intro, in a section mentioning "other projects" like 3FGL-table or WWT or Google Sky or TeVCat and why we still decided to do a new webpage.
Or you could mention "3FGL-table" in the outlook section when discussing features we want to add, e.g. selecting sub-classes of sources like "SNR" or "Galactic" or "blazar" or "UN-ID".

See fixes here.
We mention that the SNRcat database is maintained by Gilles and Samar - should we say the same for gamma-cat, that you maintain it? Or did you want me to remove this "database is maintained by..." sentence altogether?
I didn't make an "other projects" paragraph in the Intro like you suggested. I'll open an issue in case you still want to do so.

Thanks for implementing all the suggestions.

We mention that the SNRcat database is maintained by Gilles and Samar - should we say the same for gamma-cat, that you maintain it? Or did you want me to remove this "database is maintained by..." sentence altogether?

Keep as-is now, or remove the sentence who maintains it.
I just found it weird to say that the university of Manitoba maintains it.
No need to mention who maintains gamma-cat.


Cut v0.19.0

Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

I did not, what are you a suggestion? (:
I can cherry-pick that, but given Rule API is experimental I would continue with v0.19.0 release WDYT?

@kakkoyun https://github.com/thanos-io/thanos/pull/3999


Text Extractor should add to clipboard

Description of the new feature / enhancement
Text Extractor (OCR) is a great PowerToy but please let it add the extracted text to clipboard.
Scenario when this would be used?
This allows easily accessing and reusing the copied text.
Supporting information
No response

Text Extractor does exactly this. Does it not work for you?

I am able to paste after making a picture snippet.
But it doesnt add to Windows clipboard when you press Win+V

It shows up in my clipboard history, (Win+V).  I just tested.  As of now i can't repro this.
can you please create maybe a
step 1. Do X
step 2. do y ....
/needinfo

I restarted my PC and it works now. Thanks.
Powertoys Rock!


Ex 50 requires '!s' in std.debug.print

_Compiling 050_no_value.zig...
/snap/zig/5567/lib/std/fmt.zig:508:17: error: cannot format error union without a specifier (i.e. {!} or {any})
@compileError("cannot format error union without a specifier (i.e. {!} or {any})");
^
/snap/zig/5567/lib/std/fmt.zig:183:23: note: called from here
try formatType(
^
/snap/zig/5567/lib/std/io/writer.zig:28:34: note: called from here
return std.fmt.format(self, format, args);
^
/snap/zig/5567/lib/std/debug.zig:93:27: note: called from here
nosuspend stderr.print(fmt, args) catch return;
^
./exercises/050_no_value.zig:74:20: note: called from here
std.debug.print("{s} {Err!s} / ", .{ first_line1, first_line2 });
^
./exercises/050_no_value.zig:67:20: note: called from here
pub fn main() void {
^
050_no_value.zig: The following command exited with error code 1:
/snap/zig/5567/zig build-exe /home/timos/Projects/zg/ziglings/exercises/050_no_value.zig --cache-dir /home/timos/Projects/zg/ziglings/zig-cache --enable-cache
Edit exercises/050_no_value.zig and run this again.
To continue from this zigling, use this command:
zig build 50_
Line 74
std.debug.print("{s} {!s} / ", .{ first_line1, first_line2 });

@timos-komni
Yup! This has been fixed: https://github.com/ratfactor/ziglings/pull/116
Please let me know if you still have issues after updating or any other problem. Thanks!


ValueSet expansion does not contain ValueSet.expansion.contains.display element

The current implementation of the ValueSet expand operation (ValueSetSupport.expand(ValueSet)) does not include the ValueSet.expansion.contains.display element even when it is available in the ValueSet.compose.concept.display element or the CodeSystem.concept.display element.
The display element is cases and since we are doing set operations (intersection, set difference, etc.) with ValueSet.Expansion.Contains instances we rely heavily on the hashCode and equals methods which includes use all sub elements (e.g. system, code, display, etc.).
In order to get this to work, we would either need to a) use TreeSet with a custom Comparator implementation (with the tradeoff being performance O(1) versus O(log(n))) or b) use a wrapper class that implements the hashCode and equals methods differently (i.e. on system and code sub elements only).
Option a) versus option b) is a space/time tradeoff.

To verify, I deployed with the fhir-us-core jar and hit the following endpoint: GET [base]/ValueSet/$expand?url=http://hl7.org/fhir/us/core/ValueSet/detailed-ethnicity
I see display values in the expansion and so I think its working as expected:
        "contains": [
            {
                "system": "urn:oid:2.16.840.1.113883.6.238",
                "version": "3.1.1",
                "code": "2133-7",
                "display": "Ethnicity"
            },

To verify, I deployed with the fhir-us-core jar and hit the following endpoint: GET [base]/ValueSet/$expand?url=http://hl7.org/fhir/us/core/ValueSet/detailed-ethnicity
I see display values in the expansion and so I think its working as expected:
        "contains": [
            {
                "system": "urn:oid:2.16.840.1.113883.6.238",
                "version": "3.1.1",
                "code": "2133-7",
                "display": "Ethnicity"
            },


TLS ECDH 2a: server-side ECDHE-ECDSA and ECDHE-RSA (1.2)

Description
Implement the ECDHE part of the ECDHE-ECDSA and ECDHE-RSA key exchanges in (D)TLS 1.2 based on PSA when MBEDTLS_USE_PSA_CRYPTO is defined.
Status
READY
Requires Backporting
NO
Which branch?
Migrations
NO
Additional comments

@superna9999 Thanks for reviewing.
Got general problem with this change as one ssl-opt test is failing (and few compat tests) in full config.
Failing ssl-opt test is keyUsage srv: ECDSA, keyAgreement -> ECDH-. This test loads key from file and because of this psa code that initializes handshake->ecdh_psa_privkey is not executed. As a result psa_raw_key_agreement() call fails.
@mpg Can you help here and give some input on this matter ?

@mprse I did an implementation of ECDH key loading for #5320, it should probably be implemented as part of this PR. I can push it on my fork on Monday if it can help

@mprse I haven't looked closely yet, but as noted in the description #5317 (last paragraph), the scope of the task is only the ephemeral key exchanges ECDHE-ECDSA and ECDHE-RSA, but not the static key exchanges ECDH-ECDSA and ECDH-RSA (notice the missing "E" before the "-") which are covered by #5320.
The test you're mentioning should result in a static ECDH- key exchange being selected, which (as far as this task is concerned) should keep using the non-PSA path (until #5320).
I assume you need a more precise runtime test for selecting the PSA case - client-side has
#if defined(MBEDTLS_USE_PSA_CRYPTO) &&                           \
        ( defined(MBEDTLS_KEY_EXCHANGE_ECDHE_RSA_ENABLED) ||     \
          defined(MBEDTLS_KEY_EXCHANGE_ECDHE_ECDSA_ENABLED) )
    if( ciphersuite_info->key_exchange == MBEDTLS_KEY_EXCHANGE_ECDHE_RSA ||
        ciphersuite_info->key_exchange == MBEDTLS_KEY_EXCHANGE_ECDHE_ECDSA )
    {

and that's probably what you want in all the relevant places too.
If that's not the cause of the issue, ping me again and I'll have a closer look.
Note: when splitting tasks I took care to document dependencies; in this case the description of 5320 has "Depends on: 5317", not the other way round, so the fact that code written for 5320 would look useful here should raise suspicion and be a hint to double-check what the intended scope was - of course it will also happen that I've missed something (for example while doing the PK tasks it became apparent that a common preliminary PR was needed), but hopefully I haven't got a dependency completely reversed too often :)

Thanks @mpg @superna9999 . Need to take a closer look into the key exchange types. It looks like this missing conditions should help with this test.

To be fair, key exchanges are much cleaner in TLS 1.3, in 1.2 it's quite messy. Don't even get me started on the fact that "ECDSA" means two completely different things in "ECDHE-ECDSA" and in "ECDH-ECDSA".


Renaming device throws error

Traceback (most recent call last):   File "C:\Users\izzy\Git\svd_editor\modules\devview.py", line 38, in SetValue     wx.PostEvent(self.parent.GetGrandParent(), my.Evt(my.EVTID_TREE_RITEM)) AttributeError: 'module' object has no attribute 'EVTID_TREE_RITEM'
Saving/reloading the file shows the correct name at the root of the tree, but it does not update until you do.

Also, maybe not the place for it, but thank you for this tool. It's already saved me a good three days worth of typing, probably, this afternoon alone.

Thank you for your contribution.


netatalk: 3.1.0 -> 3.1.7 with ehmry's service module and bug fixes

I needed Netatalk in order to use my NixOS NAS as a Time Machine server, so I updated the netatalk package to 3.1.7, fixed some issues, and included the service module and configuration patch written by @ehmry with some bug fixes. Some of the improvements I made are:

ACL support, which is required for the current version to build
PAM authentication
The service module now opens the proper TCP port
services.netatalk.extraConfig now works as it should
The required directory /var/lib/netatalk/CNID is now created if it doesn't exist

Now the service runs without logging any CNID or authentication errors. I've tested it using my NixOS NAS running on the unstable channel with these changes merged, serving files to my Mac running OS X Yosemite with no issues. Time Machine backups seem to be working just fine as well.
@jcumming

:+1:  thanks for picking it up

No problem at all! The hard work was all yours - it just needed a little polish and updating.
That Travis CI error is looking strange to me, though. It seems like the build system can't find OpenSSL des.h... Unless that's a known quirk or similar, I may have to play with this patch a little more.

Looks like the changes in this pull request still work with nixpkgs HEAD; I can't reproduce the failing Travis CI check locally. Is there anything more I should try to fix the failing check, or is this pull request alright to eventually merge?

If you get a chance, it would be nice if the volumes were an attribute set, rather than a list. This is because samba and rsyncd use that format, so you can do things like:
let
  shares =
    { foo = { path = "/srv/foo"; };
      bar = { path = "/srv/bar"; };
    };
in
{
  services.rsyncd.modules   = shares;
  services.samba.shares     = shares;
  services.netatalk.volumes = shares;
}

https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/network-filesystems/rsyncd.nix#L71

Good thinking! I'll implement and test the changes sometime this week, when my own NixOS machines aren't doing esoteric (work-related) things :)

I'm surprised that Travis broke on this. I checked out the branch and this compiles cleanly for me ...

Hmm. Also compiles cleanly for me when merging into master. Huh.

For a personal machine of mine I made some modifications to the volumes config based on the Samba module, and I dropped the bit about opening the firewall automatically.
9fbd39949533cff9865655fbac761444234fd717

Thanks for cleaning this up and committing it! Work never did quite let up once deadlines got closer. I do want to ask for the future though: aren't services meant to open their own ports on the firewall once that service is enabled? For example, once you enable SSH it opens TCP port 22 on the firewall without any need for the administrator to change another part of their system configuration. I was actually very surprised when I first noticed netatalk didn't do this on its own, and once I realized what was going on it took a trip to Google to figure out which ports I needed to open. What's the recommended style for future service declarations: opening ports for servers, or leaving it to the admin?

@unjordy I used to think so, but in the case of netatalk I have the interface that netatalk listens on in my trustedInterfaces anyway, because open-by-default is not consistent among the other local services. And protecting ports that have services listening on them are more important than closed ports anyway.
I think the best thing to do is to put a port option on every service where it is practical so that they can be referenced by the firewall configuration, like this:
firewall.allowedTCPPorts = with config.services; [ netatalk.port rsyncd.port ];

Closed at a2b8cc0aaf74f4a98b820925f96a33123fbcd1c4


reference local copies of action instead of by repo name

Current dependencies on/for this PR:

main

PR #4   👈

This comment was auto-generated by Graphite.


mono:slim?

In the interest of slimming down the image sizes, would you be open to a PR that migrates all the FROM mono to FROM mono:slim? Doing so results in an image that's roughly 463 MB smaller.
The only package that I saw that needed to be manually installed after migrating to mono:slim is curl.

Good idea, I will try this on my end and see what I can find out

@bskrtich - If you're looking to implement it on your end, tshock also required nuget (or a package it brings) to be installed: https://github.com/beardedio/terraria/pull/6#issuecomment-630399703.

I'm not sure what the actual packages required were.

This was handled.


Getting more done in GitHub with ZenHub

Hola! @sabinmarcu has created a ZenHub account for the jwagner organization. ZenHub is the leading team collaboration and project management solution built for GitHub.

How do I use ZenHub?
To get set up with ZenHub, all you have to do is download the browser extension and log in with your GitHub account. Once you do, you’ll get access to ZenHub’s complete feature-set immediately.
What can ZenHub do?
ZenHub adds a series of enhancements directly inside the GitHub UI:

Real-time, customizable task boards for GitHub issues;
Burndown charts, estimates, and velocity tracking based on GitHub Milestones;
Personal to-do lists and task prioritization;
“+1” button for GitHub issues and comments;
Drag-and-drop file sharing;
Time-saving shortcuts like a quick repo switcher.

Add ZenHub to GitHub
Still curious? See more ZenHub features or read user reviews. This issue was written by your friendly ZenHub bot, posted by request from @sabinmarcu.

Spam spam spam.


#155148070 redesign user interface

What does this PR do?
redesign the user interface
Description of Task to be completed?

redesign the user interface

How should this be manually tested?

clone the branch
register a new user and sign in
visit localhost: 8000

Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#155148070 redesign the user interface
Questions:
N/A

Coverage remained the same at 86.592% when pulling 20f6b8cad21a1dea02d9889adcb0298ceff284f4 on feature/#155148070/redesign-user-interface into 7f6b90d86c47bf8de8d4fdca7a4b15e4899d6fa8 on develope.


Add RAG using LangChain with Azure OpenAI and Cosmos DB for MongoDB vCore Template

Your template repository
Place your template repository link here:
https://github.com/Azure-Samples/Cosmic-Food-RAG-app

[x] Added an entry to https://github.com/Azure/awesome-azd/blob/main/website/src/data/users.tsx that includes:

[x] Template title - A short title that reflects the local application stack that someone could use to get their application on Azure (e.g. "Containerized React Web App with Java API and MongoDB")

[x] Description - 1-2 sentence description of the architecture (e.g. Azure services) or solution that is defined by the template.

[x] Architecture Diagram or Application Screenshot - The image should include all services and their connections (example). You should add the image to the website/src/data/images/.

[x] Link to Author's GitHub or other relevant website - Used for attribution

[x] Author's Name -  Name to credit on the gallery card

[x] Link to template source - Link to the template GitHub repo

[x] Tags - Specify tags to represent the template. If you don't see a relevant tag for your template? Feel free to add one!
Required tags:

[x] Tag your template as Micrsoft-authored ("msft") or Community-authored ("community")
[x] Tag the IaC provider ("bicep" or "terraform")
[x] At least 1 tag for programming language used
[x] At least 1 tag for Azure services integrated
[x] Add the "new" tag for any newly authored templates

[-] In the PR comment, if you can also add a link to the PR where you made your repo azd compatible this will allow us to provide feedback on your template and speed up the review process.

[-] If the template is Microsoft-authored, we encourage you to also publish it to learn.microsoft.com/samples.

@v-xuto Could you please test this template?

@v-xuto Could you please test this template?

@john0isaac, @gkulin  We have finished this template test, and we filed one issue. Please review.

https://github.com/Azure-Samples/Cosmic-Food-RAG-app/issues/99

@v-xuto thank you for testing, I did leave a resolving comment on the issue.
if you are okay with it I will resolve the issue.

We retested it and the issue was fixed now. Everything is working fine.


Keep the program running while monitor is turned off

[x ] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
the screen doesn't change if I turn off monitor.
Describe the solution you'd like
A clear and concise description of what you want to happen.
the program should continue to run and not have the screen freeze
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
none
Additional context
Add any other context or screenshots about the feature request here.

When you turn off the computer monitor or the device screen? In both case, the program continues running.

when i turn my pc  monitor on again the screen is black. is this supposed to be the case or is it because the screen auto locked itself

It is not supposed to be the case. Is the device screen on?
Probably a graphics driver issue.


[CMake] Use correct compiler flag for unoptimized MSVC/clang-cl builds

@swift-ci please clean smoke test and merge


Partial text on loading screen

When starting a game, there is some text shown below the level loading progress bar. Usually it's just a single letter, somewhat off to the left (the exact position differs from one instance to the next, as if it's the first letter of some center-justified text of varying lengths.) Once in a while instead of a single letter it will be a longer string.
This happens whether I'm loading the tutorial, a new game, or a saved game.

I thought it only happened in debug builds but it's not.
Anyway bisected and found the bad commit: e6f8862


[Icon Request] Support for hardhat config file

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: filename
Icon Name: hardhat
Sample original Icon:
Filenames: hardhat.config.js, hardhat.config.ts

More info: From the about section of their repository

Hardhat is a development environment to compile, deploy, test, and debug your Ethereum software. Get Solidity stack traces & console.log.

Could you source an svg icon for hardhat? Currently vscode-icons is only accepting SVG icons.

Updated!


Client side indexes?

Hi jagi,
do I assume rightly, that an index I define with astronomy (http://astronomy.jagi.io/indexes) only uses/implements server side mongo indexes? Or does astronomy also maintain client side indexes?
Thanks, ayalpani

Hi, there is no index feature in MiniMongo and that's what is used on the client in Meteor and Astronomy. Astronomy is not changing here anything. So yes indexes are only implemented on server. MDG decided not to implement indexes on the client as user is mostly subscribed to small amount of documents and sorting/filtering/searching is quite fast for small sets.

Hi jagi, this was expected. So I need to create my own custom indexes, as my use case demands so - I need to support thousands of nested sets in the client. And thank you for astronomy, I just started using it today and it's really great :)

You may try extending MiniMongo and publishing a package. It would be great to have one.
I'm currently developing Astronomy 2.0 and it will be much better than 1.0 :)


fix search in marketplace

Coverage decreased (-7.8%) to 64.354% when pulling 4276c01b4776076b5433ade676ca9d3066649d31 on qixcheng/webportal/marketplace into 0922795c2e6c5c58e00a5e697e24f7c209525fbf on paishare.


(Frontend) Footer overlaps (version and copyright)

Removed "version" in string, now it's narrow enough to fit properly even on mobile screens.


Short Interruption of Internet Connection Stops Price Retrieval

Hi,
whenever after 24 hours my internet provider makes a forced separation of my line, afterwards no price data is retrieved anymore.
I thought that the last statement in https://github.com/ftomassetti/backtrader-oandav20/issues/10 meant, that with
stream_timeout=None,
poll_timeout=none,

a short interruption of my internet connection will not harm the processing anymore.
Did I understand it wrong?
Thanks in advance!

The reconnection will only be done on initialization, when the connection goes away while running, the oanda store will stop working. I will take a look about reconnection after initialization. The problem here is, that if the connection goes away, data for feed will be missing.

In my last statement, there is a error. It is not only connecting on initialization, but the 2 values for timeout will be pushed to v20 api wrapper. Mixed it with an other project. Sorry.
It will still stop when there is a connection issue. I will have a look into that. For now you could set a cron job to restart Backtrader after 23:59

thanks for taking care!


Copy-on-write for transition dispatches.

Most of the time the dispatches are the same for all nodes in a selection, so it’s a shame that we create a new one for each node. I bet there’s a simple way to use copy-on-write so that in the common case selected nodes can share a single dispatch object. For example, iterating over the nodes, and if the next node’s dispatch is the same as the previous node’s, then we already updated the dispatch and we don’t need to do anything.

This is actually quite a bit harder than it seemed because dispatches can be modified while they are dispatching events. So, if you take a copy of a dispatch before modifying it, and that dispatch is currently dispatching an event, the current event will not be affected by the modification (which is a bug). And the solutions I considered all seemed quite complicated; for example, if you do need to modify the currently-dispatch dispatch in-place, then how do you not also modify any other nodes that share this dispatch, but should not be affected by the modification? Ack.

Fixed.


[QUESTION]rocksdb is stored in key order, supports range api, redis does not support

rocksdb is stored in key order, supports range api, redis does not support

Kvrocks supports rich Redis data types, some of which support range related APIs, like zrange, lrange etc, which you can find here. Kvrocks implements the above APIs based on the rocksdb range APIs.

Closed as stale. You can ask questions at the Discussions forum.


Actualize notebooks: load datasets using TopicNet

Seems that comment in demos' README about datasets in quite enough


Fix value out of range bug

re: https://github.com/dolthub/dolt/issues/5642

@zachmu I rewrote this with a new Type.Convert interface to indicate out of bounds Convert(interface{}) (interface{}, bool, error), which is ignored in most cases. Carve outs for numeric types avoid breaking the zero'ing behavior of other types during column default checks and insert/update error ignoring.


Switch hints to use double quotes everywhere

Also, have Battle#hint take Side per @urkerab's suggestion.
Followup to #5258, no functionality changes.

The rule I have in mind is:
foo(bar, baz);
// good

foo(bar,
  baz);
// good

foo(
  bar,
  baz
);
// good

foo(bar,
  baz
);
// bad

foo(
  bar,
  baz);
// bad

I agree, that seems like a good style. TBH though, I think we should be using an automatic formatter (prettier, clang-format, etc, take your pick) and if we can't configure it with the style we want completely (the string quoting rules spring to mind) we should just accept it because not having to GAF about formatting issues and having an automatically uniform codebase far outweighs trying to manually achieve 'perfection' but having it poorly done (humans are fallible). I go into more details about that in the doc I'm writing reviewing PS with suggestions, but because its came up twice recently I figured I might as well float the idea your way earlier.

Hm. I've always thought prettier output was uglier than the manual stuff we've been doing. I don't really mind everything being somewhat inconsistent, in exchange for being able to convey information with code structure.

I think the current tradeoff is good enough; we leave some formatting decisions in the hands of coders (and accept that they might be wrong sometimes), in exchange for having some flexibility in how you want the code to look.

Hm. I've always thought prettier output was uglier than the manual stuff we've been doing. I don't really mind everything being somewhat inconsistent, in exchange for being able to convey information with code structure.

I haven't used prettier, I'm using a clang-format based solution that we use at work (I suggested prettier because its more popular). Its not perfect (I really dont like that it makes type A = 'foo' | 'bar' | ... into type A = 'foo'|'bar'|... for example), but I've stopped fighting it after having used it for several years (and you can always selectively turn it off for sections where it does something stupid. I might even be able to configure, I havent bothered looking into it).
I really like the style of the new PS (and Preact PS client) code. I think before suggesting a move to an automatic formatter more formally I'll need to try setting one up and seeing concretely what changes it would make. I don't know about prettier, but I think clang-format comes pretty close to what we're doing anyway (minus the "/`/' stuff)


CWL

i have merged the taverna-3 branch to master branch

In incubator-taverna-common-activities/taverna-cwl-activity/src/main/java/org/apache/taverna/cwl/ is it possible to rename CwlDumyActivity.java to CwlDummyActivity.java?

Thanks!
Some licensing things.. (sorry):
You add cwl-logo-header.png  - we talked about this coming from commonwl.org repository which is now Apache licensed
This simplifies things for us (compared to Creative Commons), but we still need to respect these elements of Clause 4:

(b) You must cause any modified files to carry prominent notices stating that You changed the files; and

(c) You must retain, in the Source form of any Derivative Works that You distribute, all copyright, patent, trademark, and attribution notices from the Source form of the Work, excluding those notices that do not pertain to any part of the Derivative Works; and

Clause a) is obvious as we are also Apache 2.0-licensed. As there is no NOTICE file in https://github.com/common-workflow-language/common-workflow-language we don't need to comply with clause d).
I think you have scaled and cropped the image, so we need to add the b) notice. Obviously this is a bit tricky in PNG - PNG metadata is not 'prominent', so perhaps adding a cwl-logo-header.txt would be easiest?
In cwl-logo-header.txt we can also for c) copy the attribution from the CWL contributor list - we don't know which of these have contributed to the logo itself, but we can ask @tetron and @mr-c.
So @ThilinaManamgoda - would you be able to add such a text file to neighbour the PNG? Something like:

cwl-logo-header.png adapted by Thilina Manamgoda, original from
https://github.com/common-workflow-language/common-workflow-language/blob/master/site/CWL-Logo-Header.png
which was contributed by
Peter Amstutz,
John Doe
etc.  (we can shrink the big list once we get info about who made the logo)

Also could you remove the potatoman exampleIcon.png as we now believe we do not have permission to redistribute this :-(

The logo is under the following license: https://github.com/common-workflow-language/logo/blob/master/LICENSE.md
The logo was done as a work for hire by @stefanmalacu

Thanks, @mr-c! But does this not mean that https://github.com/common-workflow-language/common-workflow-language/blob/master/site/CWL-Logo-Header.png is a redistribution is a violation of both those licenses and also misleading as its top-level repository LICENSE claim it is Apache license?

@stain Indeed; I've created a pull request to clarify the situation. https://github.com/common-workflow-language/common-workflow-language/pull/288

@ThilinaManamgoda if the https://github.com/common-workflow-language/common-workflow-language/blob/master/site/CWL-Logo-Header.png logo is truly still under LGPL and CC-BY-SA-3.0 (which I think @mr-c would need to clarify), then we can't redistribute your shrunk PNG version, as it then needs to be unmodified.
Of course you would be free to contribute the modified shrunk version back to  https://github.com/common-workflow-language/logo and then we use that binary "unmodified". But still being under CC-BY-SA 3.0  it means you would need to modify this top-level LICENSE and README, e.g. as in https://github.com/apache/incubator-taverna-language/blob/master/LICENSE#L271

Thanks, @mr-c. Perhaps the simplest then is to drop the logo again.. unless @ThilinaManamgoda would like to have a go at adding those LICENSE and README modificatoins?

I will try to add those license


Updated dependencies to newest version.

Tested on demo site with "start gulp" and "start production". No errors.

master is getting blown away, and replaced with the blendid branch. Deps are pretty up to date there, but i'll do one last upgrade before the final release.


Garnet WithPersistence Failed To Start

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
While I try to extract the test project for garnet from Aspire.Hosting.Tests to the new test project Aspire.Hosting.Garnet.Tests according to https://github.com/dotnet/aspire/issues/4294, running Garnet Container using WithPersistence failed.
Expected Behavior
The container starts without any errors.
Steps To Reproduce
Just add GarnetResource using WithPersistence
var builder = DistributedApplication.CreateBuilder(args);

var garnet = builder.AddGarnet("garnet")
    .WithPersistence();

builder.Build().Run();

Exceptions (if any)
No response
.NET Version info
.NET SDK:
Version:           8.0.303
Commit:            29ab8e3268
Workload version:  8.0.300-manifests.280f8661
MSBuild version:   17.10.4+10fbfbf2e
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.303\
.NET workloads installed:
[android]
Installation Source: SDK 8.0.300, VS 17.10.34916.146
Manifest Version:    34.0.95/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.android\34.0.95\WorkloadManifest.json
Install Type:        FileBased
[aspire]
Installation Source: SDK 8.0.300, VS 17.10.34916.146
Manifest Version:    8.0.1/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.aspire\8.0.1\WorkloadManifest.json
Install Type:        FileBased
[ios]
Installation Source: SDK 8.0.300, VS 17.10.34916.146
Manifest Version:    17.2.8053/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.ios\17.2.8053\WorkloadManifest.json
Install Type:        FileBased
[maccatalyst]
Installation Source: SDK 8.0.300, VS 17.10.34916.146
Manifest Version:    17.2.8053/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.maccatalyst\17.2.8053\WorkloadManifest.json
Install Type:        FileBased
[maui-windows]
Installation Source: SDK 8.0.300, VS 17.10.34916.146
Manifest Version:    8.0.21/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.maui\8.0.21\WorkloadManifest.json
Install Type:        FileBased
Host:
Version:      8.0.7
Architecture: x64
Commit:       2aade6beb0
.NET SDKs installed:
8.0.300 [C:\Program Files\dotnet\sdk]
8.0.303 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.19 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.19 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.30 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.19 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Anything else?
Container Logs:
GarnetServer
Copyright (c) Microsoft Corporation
ERROR(S):
Option 'save' is unknown.
Encountered error(s) while parsing command line arguments.
For detailed usage information run with --help.
09::55::17 info: ArgParser[0] Configuration import from embedded resource succeeded. Path: defaults.conf.
Unable to initialize server due to exception: Encountered an error when initializing Garnet server. Please see log messages above for more details.

WithDataShouldPersistStateBetweenUsages(useVolume: true) work on my local but WithDataShouldPersistStateBetweenUsages(useVolume: false) failed because of #5055.
For now, I disabled this test to discuss it.

I get the same behavior and error when using WithDataBindMount. Without using WithDataBindMount it works fine.

A workaround that worked for me until the linked PR is merged and released:
var cache = builder
    .AddGarnet("cache")
    // TODO: .WithDataVolume("cache-data") // workaround for https://github.com/dotnet/aspire/issues/4870
    .WithVolume("cache-data", "/data")
    .WithArgs("--checkpointdir", "/data/checkpoints", "--recover", "--aof", "--aof-commit-freq", "60000");


Adding Cost Effective Gradient Boosting to LightGBM

S Peter proposes a feature which can be added to LightGBM as a feature. The work has already been done in a fork here- https://github.com/svenpeter42/LightGBM-CEGB
Paper related to the topic- https://papers.nips.cc/paper/6753-cost-efficient-gradient-boosting.pdf

ping @svenpeter42

@svenpeter42 Are you interested into adding Cost Efficient Gradient Boosting to the official LightGBM?

ping @svenpeter42

sorry, i've been pretty busy with research lately. I'll try to eventually get back to this but cannot tell when I'll have enough time :(

@svenpeter42 Can I use your code to integrate it into core LightGBM repository as a PR?

Sure, absolutely!

I have an implementation of this, integrated in the serial tree learner, in https://github.com/rvb/LightGBM/tree/CEGB-rebase .
Would you be interested in a PR based on that branch?
There's a bunch of other stuff on there as well, I'm happy to rebase just the CEGB commits onto current master.

@remcob-gr Go ahead make a PR for the same. I will close my current PR for CEGB.


two classes have not been trained on SemanticKitti

Hello,Thank you for your great work!When I train on SemanticKitti,the motorcyclist and other-ground classes  get very low score,

i have the same issue

The validation sequence of semantic-kitti has only 5 points labeled as motorcyclist, therefore it cannot distinguish them as a motorcyclist, obtaining a 0% IoU score. The same issue happens with the other-ground class.
If you run in on test and upload results to the benchmark server, you will see it indeed learns those classes.

thanks


Add support for TSPLIB

add tsp reader
add comparition with some paper results

Data:
http://elib.zib.de/pub/mp-testdata/tsp/tsplib/tsp/
optimal solutions:
http://comopt.ifi.uni-heidelberg.de/software/TSPLIB95/STSP.html

Will use this doc as reference and only add support for Symmetric TSP; because all the data from DiscreteOptimization course are also symmetric;
https://readthedocs.org/projects/tsplib95/downloads/pdf/latest/


about the final 'io'

Describe the bug
i have check that there is 39 finals in paddlespeech symbols, but i found a unknown 'io' final, i want to know which pinyin can generate this final??
https://github.com/PaddlePaddle/PaddleSpeech/blob/f6ca14c5fadac76833b4d1b020f39cbb0eceb825/paddlespeech/t2s/frontend/generate_lexicon.py#L29-L34

one more question: why add tone 1 in this rule?
https://github.com/PaddlePaddle/PaddleSpeech/blob/f6ca14c5fadac76833b4d1b020f39cbb0eceb825/paddlespeech/t2s/frontend/zh_frontend.py#L245

yo
you can see https://github.com/PaddlePaddle/PaddleSpeech/blob/f6ca14c5fadac76833b4d1b020f39cbb0eceb825/examples/csmsc/tts3/local/simple.lexicon#L181

as for tone 1 in zh_frontend.py line 245, it's for the robot style , see https://github.com/PaddlePaddle/PaddleSpeech/tree/develop/demos/style_fs2

for io, it seems there is no xio, pio like pinyin, right?
yo 哟，this is right.

yes, no xio or pio
io  replace y in yo,
iou replace  y in you,
You can see our design principles in : https://zhuanlan.zhihu.com/p/349600439
https://zhuanlan.zhihu.com/p/349600439

thanks. close this.


Can flash multiple flash buttons in a solo frame.

I'm pretty sure this a bug. If I wanted to be able to hold down multiple buttons I'd just put them inside a normal frame.
to repro:
Create a few scenes, add to a solo frame and make them flash buttons. Assign MIDI or Keyboard shortcuts. Start operator mode and hold down both keys.

The flash button pressed first should disable all other buttons being pressed until it is released.

I've had a long think about how this should behave. Please review my suggestion:
So let's say you have a solo frame with 3 buttons: A , B and C.
Button A and B are "flash" buttons. Here's how I think it should work with examples:
Case 1:  C is active, A is then pressed

Button "C" and its scene should temporarily stopped
While button A is active, only its scene should be active
When button A is released, Button C should become active again

Case 2: B is active, A is then pressed

No change (B remains active)
When B is disabled, A then becomes active if it is still pressed.

I do not feel strongly in what order we think buttons should be allowed to be pressed, only that the current behaviour is counter-intuitive because Solo Frame should mean only 1 of the functions should ever be able to be activated.

I'm just a lowly (and relatively new) user. So take this with the requisite grains of salt.  But both of your examples have what seem like counter-intuitive aspects in my mind.
In Case 1, you want a flash button press to override a toggle button, which I think is probably correct. But then you want the software to remember the history of the button that the flash button(s) overrode, and restore it when you release the flash, and that seems odd to me.  Imagine if this was adopted, and if they were all toggle buttons -- wouldn't you expect for consistency that toggling one button off should then restore the prior button back to on?
In Case 2, now you want a flash button to not override another flash button.  I'd prefer that the last pressed thing should always take over in a soloframe, and not have a special case for flash buttons.
For what little it's worth, if a change was made, I'd suggest that pressing a flash button in a soloframe should probably turn off any other widget active in the same frameset, but then nothing should turn back on when a flash is released.

I tend to agree, my reasoning for the difference in behaviour is due to the difference in actual use of the VC during shows.
I guess this is biased but a lot of us are desperate for a way to override a scene/collection temporarily to strobe or do some other effect, until we release it so to go back to that scene/collection.
I feel this could work with a solo frame with a base scene (C) selected, then being able to switch to A or B by flashing these.
But as I said, totally don't mind what direction is taken as long as we can make solo frames do something for flash buttons.

@yestalgia  Just a couple thoughts, in case there's an existing way to get what you are looking for.
One: (and I"m not sure it actually works this way) - to turn off the fixtures in the base scene (C) while the flash scene is active, could you set the fixtures to use LTP (latest takes precedence) and set those fixtures' dimmer channels to 0 in your flash scene? I'm guessing that they'd come back on automatically once the flash was released. And if it works this way, I'd guess it doesn't matter if the flash button was in a soloframe or not.
Two: does it have to be a flash button that accomplishes this?  Could you put a toggle button in the soloframe for the "flash" scene (A), and then simply click button A and then back to button C when you want to restore the original?

Well, I found this forum post where the same concept was being discussed:
https://www.qlcplus.org/forum/viewtopic.php?f=18&t=13413
I’ll try to summarize briefly what the discussion seemed to lean toward as potential directions to head in.
One is to make a new option, perhaps on a flash button, that can essentially apply a blackout to one or more universes or to the other contents of a frame, and then let a selected function play while all other functions are blacked out.  And release the blackout when the selected “flash” button or function is released.
The other is to create some sort of priority system, where presumably if any high priority function(s) is active, any lower priority functions would be temporarily blacked out or otherwise suspended, and then resumed when the higher priority functions are stopped.


Tag Input Component

Summary
Tag inputs are an input field form that render tags inside the input field, followed by a text input field. Below are examples of implementations in other UI libraries.
https://www.melt-ui.com/docs/builders/tags-input
https://buefy.org/documentation/taginput/
Motivation
Tag inputs are a common and necessary component for most websites. It would be convenient if Flowbite has support for them.

I created an example of what a tags input could look and behave like.

@nash-an I took your example and tried some experiments. here is My Example. It is a Wrap of your example with flowbite-svelte's MultiSelect component. My coding skill is definitely at a noob level so the code is not efficient and cleaner.

@shimulroy842 this is exactly what I was looking for! Do you have it on Github so I can pitch in coding/maintaining it? Thanks!

@shiruken1 No I didn't put it on GitHub but you can put it on GitHub and maintain it yourself. Everything is on stackblitz.

@shiruken1 No I didn't put it on GitHub but you can put it on GitHub and maintain it yourself. Everything is on stackblitz.

I just want to say thank you for this!  I modified it some to work with the data I was expecting, but upgraded some components to work with Svelte5 better.  If it helps anyone, here's a stackblitz example.


Helm package code

Hello there
Actually this ticket is not related to a specific issue but it concerns the source code of the helm package command.
I am currently working on a helm plugin related to the helm package command and when I looked at the code of that command I realized that on the method Run the formal parameter vals map[string]interface{} is declared but never used;
so my question is, have you already spotted this or it has been done in purpose just to satisfy an interface ? here is the code below
// Run executes 'helm package' against the given chart and returns the path to the packaged chart.
func (p *Package) Run(path string, vals map[string]interface{}) (string, error) {
	ch, err := loader.LoadDir(path)
	if err != nil {
		return "", err
	}

	// If version is set, modify the version.
	if p.Version != "" {
		if err := setVersion(ch, p.Version); err != nil {
			return "", err
		}
	}

	if p.AppVersion != "" {
		ch.Metadata.AppVersion = p.AppVersion
	}

	if reqs := ch.Metadata.Dependencies; reqs != nil {
		if err := CheckDependencies(ch, reqs); err != nil {
			return "", err
		}
	}

	var dest string
	if p.Destination == "." {
		// Save to the current working directory.
		dest, err = os.Getwd()
		if err != nil {
			return "", err
		}
	} else {
		// Otherwise save to set destination
		dest = p.Destination
	}

	name, err := chartutil.Save(ch, dest)
	if err != nil {
		return "", errors.Wrap(err, "failed to save")
	}

	if p.Sign {
		err = p.Clearsign(name)
	}

	return name, err
}

thank you for your help.

@thynquest Well spotted. The vals map[string]interface{} parameter is not used as you mentioned. You can push a PR if you would like.

@hickeyma  there is the PR

This breaks the API and in Helm we support backwards compatibility between minor/path releases. Hence, this cannot be  changed until Helm 4.  Sorry @thynquest but this will have to be pushed out until Helm 4 dev commences.

Ah too bad .. but I understand the backward compatibility problem. so we can close this issue

For further clarification, the vals map[string]interface{} is currently unused. However, it was put in place to support #3141 in a backwards-compatible manner. We just don't have a working implementation of that yet.

Thank you @bacongobbler  for the precision.
Speaking of working implementation, why can't we give the opportunity to set values when packaging ? (helm package . --set myprop=myval); suppose that  a dev team work on an app and when they push code, it triggers a pipeline which creates a docker image with a version(eg: v1.2.3), but also in addition to create a docker image with a tag version the dev pipeline triggers another pipeline by sending the image version, which will  package the helm chart using the image version and push it to a chart repository. In that case it could be useful to have the ability to set some property when packaging so we can define, for example the kind of docker image we associate to a given chart (but we can think about something else); without forgetting that we can still override it if necessary during install. sorry if I am not clear enough :sweat_smile: . I could be more specific I you want

Honestly, if the solution for #3141 can be revived once the upstream dependency issues are resolved (I sure hope so), it seems better to keep this in for now.


Set the correct device and state classes for the energy sensors

So they will be recognised as such.

Fixes #49

Is this available now? Still cant find a fix for HA energy tab

Is this available now? Still cant find a fix for HA energy tab

The daily energy sensor should work as a consumption device.
If it's not working correctly, you might want to check under Developer Tools - Statistics and see if it's marked with any issues.

Is this available now? Still cant find a fix for HA energy tab

The daily energy sensor should work as a consumption device. If it's not working correctly, you might want to check under Developer Tools - Statistics and see if it's marked with any issues.

Thanks! seems to work.
I had this still in my customizations:
sensor.panasonic_vloermodel_daily_energy:
state_class: total_increasing
device_class: energy
Can it be removed now?

Any update for this @sockless-coding ? I could take over the integration if you no longer wish to support it.

Can this be merged now? I think the consts may need updating.


feat: Add force close modal function

Proposed Changes
Feature
What is the current behavior?
we cannot force close a prompt. (we had an issue where an undismissed prompt was still showing after the user was kicked out because of the auth token expiration)
What is the new behavior?
It is now possible to force close a modal.
Checklist

[x] Documentation has been added/updated
[ ] Automated Unit / Integration tests for the changes have been added/updated
[x] Contains NO breaking changes
[ ] Updated the Changelog
[ ] Associated with an issue

Other information

ShowMessage already has a CancellationToken parameter. Isn't that enough?


Added Transpose, SplitND and ConcatND to model_spec

Added Layers to the model_spec

Transpose
SplitND
Concat

Superseded by #3111


Add quick fix to add 'void' to Promise resolved without value

This adds a quick fix to add void to a Promise that is resolved without a value, i.e. new Promise(resolve => resolve()).
The quick fix is designed to work both in TypeScript, and a few limited cases of JavaScript as well:
TypeScript:

new Promise(resolve => resolve()) -> new Promise<void>(resolve => resolve())
new Promise<number>(resolve => resolve()) -> new Promise<number | void>(resolve => resolve())
new Promise<number | string>(resolve => resolve()) -> new Promise<number | string | void>(resolve => resolve())
new Promise<{ x: number } & { y: number }>(resolve => resolve()) -> new Promise<({ x: number } & { y: number }) | void>(resolve => resolve())

JavaScript:
As there is no way to specify type arguments directly using JSDoc in a JS file, this works by inserting or modifying an /** @type {} */(...) cast around the call to new Promise(...):

new Promise(resolve => resolve()) -> /** @type {Promise<void>} */(new Promise(resolve => resolve()))
/** @type {Promise<number>} */(new Promise(resolve => resolve())) -> /** @type {Promise<number | void>} */(new Promise(resolve => resolve()))
/** @type {Promise<number | string>} */(new Promise(resolve => resolve())) -> /** @type {Promise<number | string | void>} */(new Promise(resolve => resolve()))
/** @type {Promise<{ x: number } & { y: string }>} */(new Promise(resolve => resolve())) -> /** @type {Promise<({ x: number } & { y: string }) | void>} */(new Promise(resolve => resolve()))

The quick fix is not supported on a new Promise with a contextual type in either TS or JS currently, as that could have further-reaching side-effects

@DanielRosenwasser: @andrewbranch suggested I add a unique error message for this case in checker. The best I've come up with is something like this:
Expected {0} arguments, but got {1}. Did you forget to add a 'void' type argument to your Promise?

Not sure I like the wording though. Any suggestions on a description?

Alternatives:

Did you forget to add a 'void' type argument to your Promise? (above)
Did you forget to declare your 'Promise' as 'Promise<void>' or 'Promise<... | void>'?
Did you forget to include 'void' in the type argument to 'Promise'?

(2) seems a tad repetitive and I'm worried the ... | void might be confusing. (2) seems the simplest of the three.

I went with (3), but we can change it later if we come up with something better.

@DanielRosenwasser: I'd like to get a perf test run in with the new change in checker, but I think this is probably fine to merge. Any perf regression would only be in cases where we are already reporting a diagnostic, so clean code isn't affected.
@typescript-bot perf test


binAvailable does not trigger on copy pasting credit card numbers

General information

SDK version: 3.76.0
Environment: Sandbox/Production
Browser and OS: Chrome 91.0 MacOS Catalina 10.15.4

Issue description
Apologies if this is not an issue but could be a feature request.
The Hosted fields event "binAvailable" does not get fired when a user copy paste's a credit card number into the number field which already has another credit card number in it. The event only gets fired if the field's previous value was empty.
Current behaviour:
credit card number field is empty. User paste's previously copied CC number. "binAvailable" gets triggered.
Expected behaviour:
credit card number field has a value (eg:<PHONE_NUMBER>111111). User paste's another CC number into it (eg:<PHONE_NUMBER>554444). "binAvailable" doesn't get triggered.
This behaviour would help us in fetching new surcharge values every time the card number changes this way.

Thanks for reporting this, we'll look into fixing it.

This is now fixed in v3.78.3


Cleaning FeedAdapter - step 2

Removing DataLoadingSubject from FeedAdapter and moving the logic to the Home and Search ViewModels

@nickbutcher - ready for another 👀


Enhance on-open workflow to run commands

https://github.com/microsoft/vscode/blob/a8444a04712ff1bf9b83d1670f2e78733c327f3b/.github/workflows/on-open.yml#L10-L19
In comparison, on-comment workflow acts on any /commands it finds:
https://github.com/microsoft/vscode/blob/a8444a04712ff1bf9b83d1670f2e78733c327f3b/.github/workflows/on-comment.yml#L11-L25
By changing this the following line in this new issue would get actioned:
/assign @JacksonKearl
As things stand I will need to add it as a comment after creating the issue.

Adding the following command which didn't get detected in the original comment.
/assign @JacksonKearl

@lramos15 if I put /assign @lramos15 or other slash-commands in the text I enter when creating a new issue I don't think it gets actioned by the bot. Instead I have to put them in a comment.

Hmm, I guess we could add it. I think the general idea was to keep the actions as lean as possible to avoid unnecessary compute and since the only people who would ever need a command on open would be for /assign by our small set of triagers it never felt worthwhile

I don't feel strongly about it, so feel free to close.

Closing for now, will revisit if we find this use case being attempted to be used often.


nixos/lxc/container: allow useDhcp with LXC containers

https://github.com/NixOS/nixpkgs/pull/347283 changed the default to exclude non-physical network Kind, but that unfortunately also includes veth which LXC uses for its network interfaces. Re-enable that functionality so users can use networkd with useDHCP.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Wouldn't it be better to add a separate 99-veth-default-dhcp that matches veth interfaces instead?

Wouldn't it be better to add a separate 99-veth-default-dhcp that matches veth interfaces instead?

What makes that better?

Previously the default behavior was to match both physical and virtual ethernet interfaces. I think we should keep it instead of only enabling for veth.

In what scenario would the physical unit work in an LXC container?

Maybe when an interface is passed through (using lxc.net.*.type = phys)?

Sigh. This is now here. https://github.com/NixOS/nixpkgs/pull/358806
I suspect phys users of raw lxc are quite rare, but easy enough to support. :)


Fix data and consent permissions

Removed consent id from consent documents, now everything should depend on deployment related consents
Allowed participants to delete and count data points
Fixes #68 #69 #70

This issue still persists within API, when recreating a flow for https://github.com/cph-cachet/carp-webservices-spring/pull/77
as a participant or researcher getting 403.! + I am gonna review CAWS endpoints and see the DAMAGE regarding not defining PreAuthorization in endpoints...... and either create an issue request
@yuanchen233 @xelahalo @bardram


spec: clarify requirements for duplicate constants in interface-typed map literal keys and switches

The Go spec disallows duplicate constants in map literal keys, and allows compilers to reject duplicate constants in switch cases.
However, the Go spec does not formally allow interface-typed constants, and doesn't mention how to handle constants that are implicitly or explicitly converted to interface type. The existing compilers handle these situations in differing ways:
package p

// #1
var _ = map[interface{}]int{
	0: 0,
	0: 0,
}

// #2
var _ = map[interface{}]int{
	interface{}(0): 0,
	interface{}(0): 0,
}

func _() {
	// #3
	switch interface{}(0) {
	case 0:
	case 0:
	}

	// #4
	switch interface{}(0) {
	case interface{}(0):
	case interface{}(0):
	}
}

cmd/compile rejects 1, 2, and 3.
go/types rejects 1 and 3.
gccgo (8.0) rejects none.
/cc @griesemer @ianlancetaylor

Caveat to gccgo not warning about 1 or 2: gccgo 8.0 apparently doesn't even warn about:
_ = map[int]int{ 0: 0, 0: 0 }

Filed as #28104.


Update metas.yml

Please makes sure the items are checked before submitting your PR, thank you!

[ ] Rebase before creating a PR to keep commit history clear.
[ ] Only One commit
[ ] No eslint errors

It's vux components, should not use vue's v- prefix


Fixed Analytics tab in Configure -> Configuration accordion

While working on #4115 I found a bug in "Analytics" tab in accordion. After selecting "Analytics" tab I received error:
[----] F, [2015-09-08T15:24:34.489082 #21236:2ae6c9edd96c] FATAL -- : Error caught: [ActionController::RoutingError] invalid accordion
/home/rblanco/devel/manageiq/app/controllers/mixins/sandbox.rb:148:in `x_active_accord='
/home/rblanco/devel/manageiq/app/controllers/ops_controller.rb:185:in `accordion_select'

After changes, "Analytics" tab works for me:

@miq-bot add-label ui, bug

Can we have a test for this?

@martinpovolny I fixed the condition other way: it was wrongly changed during Haml conversion

@romanblanco : you changes as well as the previous regression are a proof that we really need at least one test to cover this partial
Other that that, cool ;-)

@romanblanco @martinpovolny can point you to a test that just verifies a 200 response is returned rather than an exception.  Once that is added, we can get this merged.

@chessbyte I'd say it's correct how is it now, because otherwise I'm getting exception:
[----] F, [2015-09-23T10:48:14.300393 #7023:2b2960f11968] FATAL -- : Error caught: [NoMethodError] undefined method `data' for nil:NilClass
/home/rblanco/devel/manageiq/app/views/ops/_analytics_details_tab.html.haml:7:in `_app_views_ops__analytics_details_tab_html_haml___3223770388509526053_47456930025860'


Nx inputs configuration

When we next upgrade Nx, it will come with some new configuration capabilities: https://nx.dev/configuration/packagejson#inputs-&-namedinputs
We should also be able to use this to improve cache hits, eg by making the eslint config only affect lint targets, jest config only affect test, jest tests only affecting test + lint, etc

Should also check over our implicit dependencies - at the very least I think we should add our package-lock


Where's the open source code for Kubevious?

According to the Kubevious website, this is an open source project. However, the website links to this repository, which doesn't actually contain any application source code. Is Kubevious truly open source? 🤔

Hey @pcgeek86 , I believe all available code you're looking for is here.
Moreover, @rubenhak I hope can clarify if you have any additional doubts over this.

Thanks @tanmay-bhat.
@pcgeek86, the project is indeed open-source. Unlike many other "cli tools", it was very hard and impractical to fit all the code into a single repository. There are 4 executables and numerous modules involved. Please see the CONTRIBUTING  guide for the locations of the source code.
Btw, we are looking for maintainers to help driving the project. There are lots of things to do. If you want to learn more feel free to join the weekly community meeting or reach out on slack channel


Fixed malloc.h path on iOS target platform

You need to change this to #if defined(TARGET_OS_IPHONE) && TARGET_OS_IPHONE to support older C implementations.


Please add a license

I can't contribute if I don't know what the license is :-)

Thank you for the issue. I've added MIT license. Good luck!


Smarter deserialize_bytes and deserialize_byte_buf

It seems we can just read all at once rather than reading byte by byte.

This looks really good, thanks!  While you are in the read_vec code, could you make a little change?
let mut buffer = Vec::new();
try!(self.reader.by_ref().take(len as u64).read_to_end(&mut buffer));

could be replaced with
let mut bytes = Vec::with_capacity(len);
unsafe { bytes.set_len(len); }
try!(self.reader.read_exact(&mut bytes[..]));

and you'd probably see even better performance.
(this implementation was ruthlessly stolen from https://github.com/TyOverby/bincode/pull/116/files)

Great suggestion, thanks -- done!

Still not as smart as it could be. I filed #118 to follow up in another PR.

@dtolnay: sounds good!  I'll merge this and start in on the recommendations in #118


[spm] Change README to use tracegen

The README currently suggests running ./otlp_exporter_example.py, why is that?  Can we instead provide an example command of running the tracegen?  Instead of saying "[run] a number of times", tracegen can have an exact command line that runs for a specified period of time with given throughput, which seems more useful for SPM. And IIRC we already use it in the docker-compose file, so I would retire the python script altogether.

I introduced microsim for this purpose of generating traces. I used microsim because it has a relatively complex trace structure with multiple "services" having varying latencies, while also generating a continuous stream of span traffic to give meaningful visualizations of requests and durations.
./otlp_exporter_example.py served two purposes:

A very simple example application to demonstrate OTEL instrumentation, in one file. Python is a popular and approachable language.
An interactive way to sending traces one at a time, with immediate feedback in Jaeger UI.

If we feel ./otlp_exporter_example.py is not useful, I would rather remove it altogether than replace it with tracegen, given microsim serves the use case of generating a meaningful stream of traces, unless if there's another use case?

@albertteoh the README currently talks about two different modes, one is using docker compose that runs microsim, and the other running a python script, which apparently only generates a single trace. I think that script can be replaced by running tracegen.

btw, #4747 is why I prefer to remove python script. Unnecessary maintenance.

btw, #4747 is why I prefer to remove python script. Unnecessary maintenance.

Yup, that's a fair point. I'll have a look into using tracegen.


Possible feature: allow people to do REST publishes with a realtime channel object

The usecase is as a fallback if realtime publish fails.
The point is that if we have some network issues in a region that disrupt the channel in a region, we would like to retry the publish in another region, but the realtime client only does fallback host retries for connect attempts, not for publishes, since publishes just go down the connected websocket. There are usecases where you want publishes to happen with a realtime client normally, but if a publish fails, would want to retry it as a REST publish to another region.
Theoretically we could make the SDK do this automatically, but there are problems; realtime publishes have ordering guarantees which REST publishes don't, and we probably would not want to have 'increased availability at the expense of a violation of ordering guarantees' be a thing that the library does automatically. It should probably be explicit.
Currently the approach is to just instantiate a REST client with the same clientOptions as a realtime client. But this seems silly; a realtime SDK is perfectly capable of making REST requests with its current auth, and does so for history etc.
So my proposal is to have realtimeChannel.publish() take a rest: boolean parameter as one of the options object. (using the existing options object publish() can take per RSL1l -- currently only mandated for REST but should be equally easily supportable for realtime -- in general those options are passed to the server unexamined, but it's easy enough to have the SDK understand some subset of them, eg similar to what we do for untilAttach in a history query).
if the rest option is set to true, the sdk does a rest publish.
this enables patterns like
try {
  await channel.publish(msg);
} catch(err) {
    if (err.statusCode < 500) {
      log('Nonretryable publish error: ' + err);
      return;
    }
    log('retryable publish error; retrying with REST; err: ' + err);
    try {
      await channel.publish(msg, { rest: true });
    } catch(err) {
      log('error from REST publish; err = ' + err);
    }
}

(one question: should the rest publish be done to the same endpoint as the realtime connection? simplicity and consistency with other REST requests done by the realtime lib would suggest yes, but obviously there's an argument that, if the main usecase is there being something transiently wrong with the channel in the currently-connected region, it should start with a fallback region)

Can I suggest a rest field for the realtime channel instead?
await channel.rest.publish(msg);

I don't like using options, because if we introduce another publish option for the realtime channel, it might conflict with this new rest option. I believe having the ability to get a rest channel instance from the realtime channel is more explicit for SDK users.
I would apply this to realtime presence as well, and to the realtime client instance (the client already has a rest field, but it is not provided in the type definitions).


ISSUE-509: Provide a delete API for schema metadata

@raju-saravanan I tried to add test cases for delete api in BasicSchemaRegistryClientOpsTest. I added the deleteSchema call at the end of doTestSchemaOps and after that i put some assertions to make sure no versions, branches, states, etc are returned for that schema using the get* methods on SR client. The issue i ran in to is that on server side deleteSchema only deletes schema metadata row and relies on cascade delete to clean up all other tables. However, the test case uses in memory store which does not do that. As a result when i ask for all versions for that schema using getAllVersions call after deletion, that goes through a different route which does not check for schema metadata entry. It goes by branch and tries to get all versions for that branch. The approach is similar for a bunch of other server side methods as well. If you have any suggestions to quickly fix this please let me know. Otherwise i will suggest merging this. I have already tested using the unsuccessful test cases that i added locally and made sure that SR client calling deleteSchema calls the server side api right. It's just that the assertions for different gets after that are not successful. I have also tested the server side execution using UI and mysql locally.

@priyank5485: I think we need to tweak the in-memory DB to support cascade delete operations. We can come back to the test later, I will raise an issue for this, meanwhile, I will merge this PR. Thanks @priyank5485.


Nonce must be greater than

Using .getBalance method on poloniex receiving nonce error:  Poloniex error 422: . Nonce must be greater than x. You provided x.
(removed numbers for security)

Please generate a new API key and make sure you only using on on application

Thanks for the fast response, that fixed the issue


(tech debt) Remove HEADER_SPAN hack

The markdown renderer has a small hack when rendering headers that allows the "HEADING_SPAN" environment variable to be passed to replace all anchors with <span> tags with data-attribute properties. I think this hack exists so that anchors can be rendered properly on Construct Hub using the prism-react-renderer component.
Once the Construct Hub has switched to rendering API references directly from API.json files, we can remove this hack.

I think we are still blocked here.


Notes links broken

The new notes section is screwing up the URL generation since now some URLs end with a semicolon. Detect this condition and strip it out.

Fixed in latest.


Unable to link to self hosted cache

To Reproduce
Steps to reproduce the behavior:
Host a docker container with simple local storage settings
Try to link & enable cache from client via creating a .turbo/config.json with appropriate fields that include token, teamId and apiurl.
Now run turbo link, and the on the client you get error: could not get team information, while on the server these logs are printed:
2023-02-21T18:16:31.570785728Z {"severity":"INFO","level":30,"time":1677003391570,"pid":7,"hostname":"09921250ff30","reqId":"Oe2v3NkCQsO6f_5DaWa91Q-14","res":{"statusCode":404},"responseTime":0.828801155090332,"message":"request completed"}
2023-02-21T18:26:52.569629705Z {"severity":"INFO","level":30,"time":1677004012568,"pid":7,"hostname":"09921250ff30","reqId":"Oe2v3NkCQsO6f_5DaWa91Q-15","req":{"method":"GET","url":"/v2/teams?limit=100","hostname":"myserver.com:36891","remoteAddress":"<IP_ADDRESS>","remotePort":52646},"message":"incoming request"}
2023-02-21T18:26:52.569669865Z {"severity":"INFO","level":30,"time":1677004012568,"pid":7,"hostname":"09921250ff30","reqId":"Oe2v3NkCQsO6f_5DaWa91Q-15","message":"Route GET:/v2/teams?limit=100 not found"}
2023-02-21T18:26:52.569674505Z {"severity":"INFO","level":30,"time":1677004012569,"pid":7,"hostname":"09921250ff30","reqId":"Oe2v3NkCQsO6f_5DaWa91Q-15","res":{"statusCode":404},"responseTime":0.45916032791137695,"message":"request completed"}

My environment
Arch 6.1.12-arch1-1
<EMAIL_ADDRESS>Edit:
Just for clarification, running turbo with cli flags like --team, --token and  --api reports that it is using remote cache -
("Remote caching enabled").

I''m also getting
2023-02-21T19:31:19.498300890Z {"severity":"INFO","level":30,"time":1677007879498,"pid":7,"hostname":"e7ed71e1d305","reqId":"C83hSXNMSoWAw4bAHmKndA-11","req":{"method":"POST","url":"/v8/artifacts/events?teamId=team_jetblack","hostname":"myserver.com:36891","remoteAddress":"<IP_ADDRESS>","remotePort":40928},"message":"incoming request"}
2023-02-21T19:31:19.500350332Z {"severity":"WARNING","level":40,"time":1677007879498,"pid":7,"hostname":"e7ed71e1d305","reqId":"C83hSXNMSoWAw4bAHmKndA-11","data":null,"isBoom":true,"isServer":false,"output":{"statusCode":401,"payload":{"statusCode":401,"error":"Unauthorized","message":"Invalid authorization token"},"headers":{}},"stack":"Error: Invalid authorization token\n    at Object.<anonymous> (/home/app/node/src/plugins/remote-cache/index.ts:41:25)\n    at hookIterator (/home/app/node/node_modules/.pnpm/fastify@4.12.0/node_modules/fastify/lib/hooks.js:246:10)\n    at next (/home/app/node/node_modules/.pnpm/fastify@4.12.0/node_modules/fastify/lib/hooks.js:174:16)\n    at hookRunner (/home/app/node/node_modules/.pnpm/fastify@4.12.0/node_modules/fastify/lib/hooks.js:196:3)\n    at Object.routeHandler [as handler] (/home/app/node/node_modules/.pnpm/fastify@4.12.0/node_modules/fastify/lib/route.js:469:7)\n    at Router.callHandler (/home/app/node/node_modules/.pnpm/find-my-way@7.4.0/node_modules/find-my-way/index.js:412:14)\n    at Router.lookup (/home/app/node/node_modules/.pnpm/find-my-way@7.4.0/node_modules/find-my-way/index.js:390:17)\n    at Server.preRouting (/home/app/node/node_modules/.pnpm/fastify@4.12.0/node_modules/fastify/fastify.js:773:14)\n    at Server.emit (node:events:513:28)\n    at Server.emit (node:domain:489:12)","type":"Error","message":"Invalid authorization token"}
2023-02-21T19:31:19.501278732Z {"severity":"INFO","level":30,"time":1677007879501,"pid":7,"hostname":"e7ed71e1d305","reqId":"C83hSXNMSoWAw4bAHmKndA-11","res":{"statusCode":401},"responseTime":2.9591617584228516,"message":"request completed"}

Thanks for reporting this!@
Can I ask you to try with client<EMAIL_ADDRESS>

Thanks for reporting this! Can I ask you to try with client<EMAIL_ADDRESS>
Hey there and thanks for getting back to me.
I actually tried this yesterday, thinking the API might've updated on how turbo links up to the remote cache server.
I cloned the code and did some logging of the incoming request, and it turns out even though I had a .turbo/config.json with properly configured credentials, it was using the config defined in ~/.config/turbo/config.json.
I suspect that config was generated when I originally made a connection to the official Cercel remote cache.
I changed the values there and can confirm that the self-hosted remote caching server is properly working without having to use CLI arguments.
The only thing I fail to understand is why it's using the defined config in ~/.config/turbo/config.json instead of  .turbo/config.json at the root of my project.
Anyway, that is all I have for now.

I think I've figured out how this behaves.
Turborepo doesn't enable cache unless I have a ~/.config/turborepo/config.json with a token field.
I also need a project root level .turbo/config.json file for teamid and apiurl.

Vercel changed the name of the teamId variable to teamSlug https://ducktors.github.io/turborepo-remote-cache/custom-remote-caching.html#config-file


Expression inside of parenthesis is not syntax highlighted.

The following sample code doesn't have syntax highlighted of expression inside of parenthesis.
lda #($7E-JOYDEADZONE*2)

new match-parens function for parens


fix in PointsCloudSystem: uniform distribution in triangle

See issue 1 in https://forum.babylonjs.com/t/pointscloudsystem-unequal-distribution-of-points/48981
Playground usage example: #UI95UC#2446

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14900/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14900/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14900/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14900/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14900/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14900/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14900/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14900/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14900/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14900/merge/testResults/webgpuplaywright/index.html


Upgrade Function

Would it be possible to add an 'upgrade' function via pip.

No, it's not possible because that would mean that everyone could push a change to a public repo.


while using yaml input copy_files not working

it gives below error since kitchen looks to the temporary directory that json file generated.
Error copying files: source file not found "/private/var/tmp/js/methods.js".

Sorry for late answer: yeah, you're right. I will fix it.

@kadirerdogan fixed. Will be available in v0.9.48 (~August 6th). Now all relative filenames are relative to original input file - not to converted input file.

fixed now (v0.9.48)


Kernel bug in jessie's kernel prevents clean updates

Kernels < 3.18 (and Jessie has 3.16) have a bug regarding unlinking files and directories in presence of mount namespaces.
In our case, this means that firejail (which uses mount namespaces) might accidentally prevent apt/dpkg from installing new packages.
See upstream note and Debian bug 848003.
I suggest we upgrade to kernels from jessie-backports (4.9.x series, currently) until we can transition to stretch (same current kernel version, will use 4.10 LTS when available).

Deployed on ny1 and sf1.


[cli] Fix parsing of boolean options starting with --no-.

Before this change, it was not possible to declare a boolean option whose name started with --no- (unless it was generated as the negation of an other option).

For an arg --no-<foo>, don't you always want there to be an arg --<foo>? The default can be set to either value right?

Yes, that interpretation is not good. I would say that it would be good to throw an error when trying to manually create an arg starting with --no- that says that the unprefixed one should be created instead.

That's an option, though I just thought of something: one might want to have the positive flag not be a boolean: --foo=bar vs --no-foo. What do you think?

@bennostein Yes, that seems to be the only difference.
There seems to be consensus against standalone --no-foo boolean flags, so I'll make it fail upon trying to register such a flag.


Segment Replication - Recover all translog ops when flipping to able engine

Description
Similar to https://github.com/opensearch-project/OpenSearch/pull/6314, when we flip to a writeable engine with SR we do not want to limit to the global checkpoint and rather recover all ops.
Issues Resolved
Related to flaky tests, these tests are still muted until WAIT_UNTIL issue has been fixed.
https://github.com/opensearch-project/OpenSearch/issues/6065
https://github.com/opensearch-project/OpenSearch/issues/5946
https://github.com/opensearch-project/OpenSearch/issues/5669
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #6352 (391df19) into main (7914c04) will decrease coverage by 0.02%.
The diff coverage is 33.33%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6352      +/-   ##
============================================
- Coverage     70.73%   70.72%   -0.02%
+ Complexity    59024    59021       -3
============================================
  Files          4800     4799       -1
  Lines        282453   282421      -32
  Branches      40718    40717       -1
============================================
- Hits         199799   199731      -68
- Misses        66259    66293      +34
- Partials      16395    16397       +2

Impacted Files
Coverage Δ

...in/java/org/opensearch/index/shard/IndexShard.java
69.57% <0.00%> (-0.59%)
:arrow_down:

...tore/remote/directory/RemoteSnapshotDirectory.java
3.12% <0.00%> (ø)

...arch/index/store/remote/utils/TransferManager.java
3.03% <ø> (ø)

...dex/store/remote/file/OnDemandBlockIndexInput.java
73.01% <100.00%> (ø)

...a/org/opensearch/client/cluster/SniffModeInfo.java
0.00% <0.00%> (-58.83%)
:arrow_down:

...ch/transport/ReceiveTimeoutTransportException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...regations/metrics/AbstractHyperLogLogPlusPlus.java
51.72% <0.00%> (-44.83%)
:arrow_down:

.../java/org/opensearch/search/dfs/AggregatedDfs.java
51.61% <0.00%> (-41.94%)
:arrow_down:

...ava/org/opensearch/search/dfs/DfsSearchResult.java
47.87% <0.00%> (-39.37%)
:arrow_down:

...cluster/coordination/PublishClusterStateStats.java
33.33% <0.00%> (-37.51%)
:arrow_down:

... and 451 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.