Align default IDs in ReportUtils file

Explanation of Change

Fixed Issues

$ https://github.com/Expensify/App/issues/50360
PROPOSAL: https://github.com/Expensify/App/issues/50360#issuecomment-2405820991
Tests

[x] Verify that no errors appear in the JS console

Test report-related functionality:

Open the app and log in.
Send several messages in workspace/1-1/ self DM chats
Send several messages from Users A and B and back.
Submit expenses: manual/scan/distance
React to the message/update it/delete

Offline tests

Same, as in the Tests section.
QA Steps

[x] Verify that no errors appear in the JS console

Same, as in the Tests section.
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/9f63e3e1-11f6-432b-bde8-5ad8fe4b2808

Android: mWeb Chrome

https://github.com/user-attachments/assets/0d053412-ab71-4377-8023-daa75d459e97

iOS: Native

iOS: mWeb Safari

https://github.com/user-attachments/assets/ea681b57-193d-49fd-92e6-d98c1beb0afa

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/18304a9c-4dde-42b8-8513-932650085a27

MacOS: Desktop

https://github.com/user-attachments/assets/1020368d-1b09-4357-9d2a-a9de9b22ca37

I have some issues with ios build, so I'll attach the recording tomorrow.

@paultsimura I've applied your feedback. I'm going to be OOO 25-29 Dec 🎄 , maybe it would be good not to merge this PR during the time, so I can be around to monitor any regressions.

@paultsimura Conflicts are resolved!


Fix ModelChoicePreference when using with model attribute and not queryset

Fix ModelChoicePreference when model is set instead of queryset.
Also, I add the MissingModel exception when model and queryset attributes are unset.

Thank you, let's merge!


todo: merge updates from https://github.com/Budibase/handlebars-helpers

Looks like they've made some great updates on https://github.com/Budibase/handlebars-helpers. Let's get those changes merged in here!
FWIW, we're slowly getting caught up on issues and would welcome some help from other contributors.

Hey @jonschlinkert - sorry I didn't see this before now, we'd be happy to get it merge ready, a lot of the work we'd done in the fork was to get it ready for rollup with Svelte, we are actively maintaining it now as part of Budibase so can help contribute!

No worries, your fork is fantastic. Really nice work! We'd love to have your contributions, and I'd be happy to add you as a collaborator if you want.

Hi @jonschlinkert - great, I've opened a PR with the current state of our fork https://github.com/helpers/handlebars-helpers/pull/401 - happy to work through any changes needed etc!


not working timezone formate

fg.lastBuildDate(write_date.strftime(timezone_format))
issue is :  this is defualt get timezone formate
https://github.com/lkiesow/python-feedgen/blob/master/feedgen/feed.py#L315
i want this : Tue, 09 December 2014 11:37:23 +0000
but
i got this : Tue, 09 Dec 2014 11:37:23 +0000
TRACEBACK

File "/usr/lib/python2.7/site-packages/feedgen/feed.py", line 462, in lastBuildDate
return self.updated( lastBuildDate )
File "/usr/lib/python2.7/site-packages/feedgen/feed.py", line 435, in updated
updated = dateutil.parser.parse(updated)
File "/usr/lib/python2.7/site-packages/dateutil/parser.py", line 744, in parse
timestr = timestr.decode()
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 9: ordinal not in range(128)
timezone_format = '%a, %d %b %Y %H:%M:%S %z
can return non-ascii characters as it is localized. On my computer (in French)  %b will return Déc for "December".

It should be a valid datetime format according to rfc2822. That is why the short month name Dec instead of December definitely makes sense and should not be changed. The lacalized time is an issue though.
Can you check if the patch in fix-time-formatting fixes that issue?

@lkiesow Thanks for replied me. but i solved this issue on my side using local like  locale.setlocale


Support secondary log directory

It can be useful to configure raft engine on a separate (smaller) disk. And when that disk is full, we should provide a way for raft-engine to locate newly created log files on the main disk volume.

U mean that we need a new configuration such as sub-dir or assist-dir to store logs on different disk?
With this pre-requisite, the main dir, that is, the current configuration dir, is still used as the default main directory to store logs. Only if the main dir is FULL or not accessible, we will use the assist-dir or sub-dir to store the incoming logs.
Meanwhile, about the abnormal circumstance:

sub-dir is None or belongs to the same volumn of disk, if the disk is full, the newly file creation should fail.

/assign @LykxSassinator Let me tackle it.


Allow to add a toolchain only - without updating settings.xml, JAVA_HOME and PATH. Only toolchains.xml must be extended.

Description:
Allow to add another toolchain only by a subsequent setup-java - only toolchains.xml must be updated with the additional JDK setup and the JDK specific JAVA_HOME_<jdk version> must be set.
No update of settings.xml, no add to PATH, no update of JAVA_HOME.
Justification:
In my scenario I run a first setup-java configuring settings.xml and setting up my JAVA_HOME JDK.
Then I determine a probable additionally required toolchain version from the compile target setting of my Maven projects POM.
If this is requiring a different JDK, then I am running another setup-java for this toolchain.
In this case I want to get the toolchains.xml being extended only and to preserve JAVA_HOME, PATH and settings.xml from my first setup-java.
Currently I am working around this requirement by a step preserving JAVA_HOME, PATH and settings.xml before the setup-java step and restoring those after that step. This is working fine so far.
Another config property for setup-java like e.g. add-toolchain-only would save those efforts!
Are you willing to submit a PR?
Yes - cannot promise a timeline yet.

Hi, @mhoffrog 👋 Thank you for the feature request, we will take a look and get back to your with our decision.

I would like this too, use case similar to https://github.com/orgs/community/discussions/73309
Pretty much anyone who provides a composite action that runs a jar or JDK commands is affected. I have no idea how others do it, probably delegate the labor to every user 😞.

@TWiStErRob I did file PR #553 already and did keep it up to date with recent master - s. https://github.com/actions/setup-java/pull/553#issuecomment-1919726093.
Unfortunately I do not get any further feedback nor answer on my comments on this PR - its a bit frustrating just getting no reaction for the time being.
@IvanZosimov Maybe you are all pretty busy - would there be a person in the team to assign for review or to put attention to this item?


Cookie passed as object is not setting for request at all

Hello @tomas! Thanks for your code. I'm kind a newbie for it, but have an issue with cookies.
There is a statement in docs :

HTTP Header options

cookies : Builds and sets a Cookie header from a { key: 'value' } object.

According to docs, I'm trying to set cookies for the request in this way:
let cookies = {sessionId: '12345'};
needle('get', URL, null, {headers: {cookies: cookies}});

So this request is not working, because server does not see any cookies set. I guess, it supposed to be set for request as a string 'sessionId=12345', but it seems that its passing as object.
I came up with a workaround, which works fine.
let c = '';
for (let prop in cookies) {
     c += prop + '=' + cookies[prop]+'; ';
}
return needle('get', URL, null, {headers: {'Cookie': c}});

Is it a bug or I'm doing something wrong?

Ops, this did the trick:
{headers: {cookies: cookies}} -> {cookies: cookies}


Add vdex021 support

Had to make some renaming in 019 files to be able to compile 021 without conflicting duplicates.
This basically reads the extra data added in vdex021's header:
uint32_t bootclasspath_checksums_size_;
uint32_t class_loader_context_size_;
However, currently, the extra data at the end of the file isn't parsed, but at least it allowed the extracting of the contained cdex files, as they don't seem to be directly related to that extra data.

@IgorEisberg thank you for submitting the PR and sorry for sitting on it for such a long time. I'll check the changes in vdex 021 dropped with Android 10 and verify if optimal to merge your patch and add missing changes in parser or refactor a few bits with a different strategy.


multiple ips gateway conflict on ubuntu

Hi,
Booting with one ip seems to work fine, but when I attempt to add a second IP I end up with an invalid /etc/network/interfaces file.
Background:

I have a template that sometimes we just boot with a private IP, sometimes we boot with a private and public IP.
I have two NIC's in the template, eth0 mapped to private, eth1 mapped to public.

I may missing something with multiple IPs on the routing config.
Here's a sample command:
ezmomi clone --template ubuntu-14.04-raw --hostname machine-name --cpus 2 --mem 2 --resource-pool 'pool' --ips <IP_ADDRESS> <IP_ADDRESS>
Relevant network config:
https://gist.github.com/ryanrussell/fb3065d8d6bae6f549c4
This is what I get for a /etc/network/interfaces file:
https://gist.github.com/ryanrussell/8afec84d64c33cb80f3e
I'm trying to automate a solution such as this:
https://www.thomas-krenn.com/en/wiki/Two_Default_Gateways_on_One_System#Solution
Here's a sample functional /etc/network/interfaces file:
https://gist.github.com/ryanrussell/c51688efee1a98e346a1
I can do this with chef post-boot if necessary, but everything about it feels hacky and wrong.
Is there a better way that I'm missing? Or a way to get a valid network interfaces file without manual intervention?
Thanks

Hi @ryanrussell -
I'm not super familiar with the Debian /etc/network/interfaces style config.
Couple questions...What happens when the new VM boots after provisioning?  Any specific errors in the boot logs?  Do eth0 and/or eth1 appear down after boot?  Or are they appear up and its the routing thats screwy?

Thanks for responding.. Here's a summary that's better than I could say it:
https://www.thomas-krenn.com/en/wiki/Two_Default_Gateways_on_One_System
You have built two or more network cards into one Linux system and each of these cards has its own default gateway. By default, you can only have one default gateway on a system. The case described would lead to asynchronous routing, whereby the router would reject the packets as appropriate.
On ubuntu, we can only have one default gateway. You have to build policy routes for additional nic traffic to be handled correctly.
There are no visual errors in the log per se, but you will experience seemingly random inabilities to route traffic correctly based on which default gateway gets used(especially when one is private ip space and doesn't publicly resolve). It's a bear to debug from the app layer down to network interfaces.
Happy to elaborate or talk more if it's helpful.


Question: Obtaining the original cause

I have been using this package for a while now without any problems (I love it).  However, today I came across a behaviour in my code that I can't understand when trying compare the result of errors.Cause with a named error. I've read through the documentation, and I came across the following statement:

Depending on the nature of the error it may be necessary to reverse the operation of errors.Wrap to retrieve the original error for inspection.

In which situations is it necessary to reverse the operation of errors.Wrap to inspect the original error?
How does one reverse the operation of errors.Wrap to obtain the original error?

How does one reverse the operation of errors.Wrap to obtain the original error?

https://github.com/pkg/errors#retrieving-the-cause-of-an-error

@pierrre I asked because I had a case where errors.Cause was returning an error which when compared to the original named error in a package did not evaluate as equal.
We've subsequently worked around the issue by comparing the err.Error() string for each instead, but the problem was perplexing.

where errors.Cause was returning an error which when compared to the original named error in a package did not evaluate as equal

Why aren't they equal ?
Did you inspect the error values with fmt.Printf("%#v", err) ?

Why aren't they equal ?

I wish I knew.  From everything that I understand, they should be, but for some unknown reason weren't evaluating as equal.

Did you inspect the error values with fmt.Printf("%#v", err) ?

No, I didn't, but in hindsight I should have.  For logistical reasons it may not be feasible for me to roll back the project to test this out, but I'll do my best to compare them this week.

@johngb Any followup on this ?


feat: Update asserts to use new winter23 Assert class - Security Recipes

What does this PR do?
Update asserts to use new winter23 Assert class. The new Assert class is really elegant compared to the older System methods.
What issues does this PR fix or reference?
#
The PR fulfills these requirements:
[ ] Tests for the proposed changes have been added/updated.
[ ] Code linting and formatting was performed.
Functionality Before
<insert gif and/or summary>
Functionality After
<insert gif and/or summary>

Codecov Report

:exclamation: No coverage uploaded for pull request base (prerelease/winter23@df08cde). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                  Coverage Diff                   @@
##             prerelease/winter23     #358   +/-   ##
======================================================
  Coverage                       ?   94.42%
======================================================
  Files                          ?       58
  Lines                          ?     1974
  Branches                       ?        0
======================================================
  Hits                           ?     1864
  Misses                         ?      110
  Partials                       ?        0

Flag
Coverage Δ

Apex
94.42% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


CASMINST-4488: Linting: remove superfluous trailing whitespace

Summary and Scope
This PR just removes unnecessary whitespace from the ends of lines in markdown documents.
Issues and Related PRs
Testing
N/A
Risks and Mitigations
I shudder to consider the risks.
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[X] License file intact
[X] Target branch correct
[ ] CHANGELOG.md updated
[X] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

/backport release/1.2


The stream option is not working.

When I set stream to True in the invoke API, I am getting the following error :
--------------------------------------------------------------------------- JSONDecodeError                           Traceback (most recent call last) File ~/Documents/grok_projects/.venv/lib/python3.12/site-packages/requests/models.py:974, in Response.json(self, **kwargs)     973 try: --> 974     return complexjson.loads(self.text, **kwargs)     975 except JSONDecodeError as e:     976     # Catch JSON-related errors and raise as requests.JSONDecodeError     977     # This aliases json.JSONDecodeError and simplejson.JSONDecodeError File /opt/homebrew/Cellar/python@3.12/3.12.8/Frameworks/Python.framework/Versions/3.12/lib/python3.12/json/__init__.py:346, in l oads(s, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)     343 if (cls is None and object_hook is None and     344         parse_int is None and parse_float is None and     345         parse_constant is None and object_pairs_hook is None and not kw): --> 346     return _default_decoder.decode(s)     347 if cls is None: File /opt/homebrew/Cellar/python@3.12/3.12.8/Frameworks/Python.framework/Versions/3.12/lib/python3.12/json/decoder.py:338, in JS ONDecoder.decode(self, s, _w)     334 """Return the Python representation of ``s`` (a ``str`` instance     335 containing a JSON document).     336     337 """ --> 338 obj, end = self.raw_decode(s, idx=_w(s, 0).end())     339 end = _w(s, end).end() File /opt/homebrew/Cellar/python@3.12/3.12.8/Frameworks/Python.framework/Versions/3.12/lib/python3.12/json/decoder.py:356, in JS ONDecoder.raw_decode(self, s, idx)     355 except StopIteration as err: --> 356     raise JSONDecodeError("Expecting value", s, err.value) from None     357 return obj, end JSONDecodeError: Expecting value: line 1 column 1 (char 0) During handling of the above exception, another exception occurred: JSONDecodeError                           Traceback (most recent call last) Cell In[10], line 1 ----> 1 response = xai.invoke(       2     messages=[       3         {       4             "role": "user",       5             "content": "What is the capital of France?"       6         }       7     ],       8     stream=True       9 ) File ~/Documents/grok_projects/.venv/lib/python3.12/site-packages/xai_grok_sdk/xai.py:141, in XAI.invoke(self, messages, frequen cy_penalty, logit_bias, logprobs, max_tokens, n, presence_penalty, response_format, seed, stop, stream, stream_options, temperat ure, tool_choice, top_logprobs, top_p, user)     118 payload = ChatCompletionRequest(     119     messages=messages,     120     model=self.model,    (...)     137     user=user,     138 ).__dict__     140 # Make API call --> 141 response_data = self._make_api_call(payload)     143 # Handle tool calls if present     144 if "choices" in response_data and len(response_data["choices"]) > 0: File ~/Documents/grok_projects/.venv/lib/python3.12/site-packages/xai_grok_sdk/xai.py:70, in XAI._make_api_call(self, payload)      66 response = requests.post(      67     f"{self.base_url}/chat/completions", headers=headers, json=payload      68 )      69 response.raise_for_status() ---> 70 return response.json() File ~/Documents/grok_projects/.venv/lib/python3.12/site-packages/requests/models.py:978, in Response.json(self, **kwargs)     974     return complexjson.loads(self.text, **kwargs)     975 except JSONDecodeError as e:     976     # Catch JSON-related errors and raise as requests.JSONDecodeError     977     # This aliases json.JSONDecodeError and simplejson.JSONDecodeError --> 978     raise RequestsJSONDecodeError(e.msg, e.doc, e.pos) JSONDecodeError: Expecting value: line 1 column 1 (char 0)
The error is occurring when the _make_api_call function is called with the payload.

Hello, could you maybe give me a sample payload that you used to get this error so I can reproduce?

Went ahead and simply tried a payload, found the issue. Now streaming is fully supported in v0.0.11, including documentation in the readme! Please let me know if you face any other issues (reopen if you need)

Sure, will try and let you know. Thanks.


Fails to parse JSON, doesn't give us the statusCode

I am attempting to interact with an API that returns 403, a content-type header of application/json, but a response body of Forbidden.
Because the error response Forbidden cannot be parsed it falls into the following case:
try {
  res = new Response(self);
} catch(e) {
  err = new Error('Parser is unable to parse the response');
  err.parse = true;
  err.original = e;
  // issue #675: return the raw response if the response parsing fails
  err.rawResponse = self.xhr && self.xhr.responseText ? self.xhr.responseText : null;
  return self.callback(err);
}

Since we have the xhr object here, could we add the statusCode as well? I need to handle logic for either a 403, 404, 422, and 500 regardless of the response body.

Should be fixed now


Create template page

[x] Create page
[x] Add template sections and text about what should go in each section
[ ] Team approval

Here's the template for making new pages, let me know if there's anything I've missed!


Dependencies for Java related items moving to open JRE/JDK?

This is more of an awareness question.  Have we looked into this as a way of allowing folks not to get hit by any potential licensing issues?

This have not been looked into AFAIK.
However, I did a quick search of the repository now (for all mentions of java, jre or jdk), and didn't find any package that actually have java as its dependency (other than a deprecated java runtime package).
So at least in the repository itself, there is nothing that needs to be looked into.

Yet, its only matter of time when we will have java dependency here.

See also: JohanJanssen/ChocolateyPackages#39

FYI
I switched few of the latest packages to AdoptOpenJDKJRE  package ( by @johanjanssen ) instead of the javaruntime that was previously used as later requires a license for commercial use since 2019.
So far: plantuml, yed and tv-browser all seem to work fine but lets see how it will behave in the wild with this setup.
With tv-browser I intiially used adoptopenjdk11jre which doesn't seem to set registry stuff (even with adequate -ia parameter) so I got it working with it by symlinking jre to its subdir java or by adding registry stuff manually (see #1350). Today I found that jre version 13.3 doesn't require this.

@majkinetor the fact 11 doesn't set registry stuff and 13 does is probably because of the fact that 11 still uses the zip to install and 13 the msi.
If you prefer to use Java 11 as it's an LTS version (13 is not), I can create a new version of the package based on the msi.

I noticed that Java 11 will get an update release in two weeks. I will release a Chocolatey package for that release based on the MSI.

Thx @johanjanssen.
Again, consider migrating it here, especially since on your issue list are some stuff that need 0 attention here.

I'm not completely convinced that it will improve much. But I'm ok if you guys want it.
So what needs to be done?
I know embedding the package is one thing. But also the packages should follow the strict guidelines, where can I find them?

https://github.com/chocolatey-community/chocolatey-coreteampackages/blob/master/CONTRIBUTING.md
Just PR here and we will help.

I know embedding the package is one thing

Embedding is critical in this case - you need to have infinitelly available executable for your project. Even if chocolatey gallery is deprecated one day packages are still available either in your local cache, your intranet cache (artifactory, nexus...), Github releases etc... So it can't get lost either.
However, not sure if embedding is possible due to size limitations.

However, not sure if embedding is possible due to size limitations.

It's growing hard, now on 178 MB. The last two releases added 9MB. So I wouldn't be too surprised if it goes above 200 MB in the (near) future. You still want to embed it and move it to the core packages?

Embedding is probably best left when this limit of 200MB is removed.
As of migration, i think it would be beneficial. If we are all going to use adoptopenjdk as dependency to java projects, its best that most known repository holds it as more people can be involved in enhacing the package, such as adding more parameters, fixing corner bugs or just making it better related to some other package which may require some extra setting and so on.
Ofcourse, its far from mandatory. Feel free to decide any way.

I got one report that package finished using the wrong JRE: https://github.com/majkinetor/au-packages/issues/129
This could ofcourse happen in other similar scenarios (python2 vs 3 as an example etc.). I was taking java executable from the PATH which is a problem in such context. Package should explicitelly try to use adoptopenjdkjre and find it via some method. I am not sure what is the best method to do is - hardcoding a path in script or iterating some registry keys until adopt word is found in case they are handled both in JavaSoft registry section or something else...

I don't know if I understand your remark correctly. You mean that another Java build was used (for instance the Oracle JDK) instead of the AdoptOpenJDK one that you expected?
I wouldn't hardcode dependencies to a specific build of Java. People now have the choice to use OpenJDK, AdoptOpenJDK, Amazon Corretto, Liberica JDK or others. Hardcoding dependencies to one of them basically eliminates the choice for people to choose their own build, or at least makes it a lot harder.

That will probably depend on https://github.com/chocolatey/chocolatey/issues/7/https://github.com/chocolatey/choco/issues/858

I agree with you @johanjanssen, however, virtual packages may be implemented in next decade or never (first ticket is almost decade old). It looks wrong to depend on specific package and use something else once installed.

Yes, but you could also opt to not declare the Java dependency explicitly. Just don't add it as a dependency, then it's up to the users to pick the dependency they want. It's what for instance the Maven package is doing as well.

Majority of users dont care, they just want it to work. That is what package manager promisses too, to not care about complexity of installation. Maven is not great example since it is dev tool. To use TV browser you dont even have to know about java.

I could probably get a pseudo‑virtual package to work by using the chocolateyInstall.ps1 script and an extension to maintain a registry of provided virtual packages.

It can be done, but keep in mind that some solutions will be considered hostile, those that mess up with chocolatey internals such as package files directly etc.
I think it should be done (even in hostile manner) given that choco.exe has almost 0 improvements over years, low number of new features despite the high need, and no new devs (if you don't count trivials) despite general interest.
Anyway, If you start something it would be good to be here or part of coreteam extension.

I will close this as it seems there is nothing more to discuss. I also think this is the best option since Microsoft is one of the sponsors of the project and it looks like they are ramping up their support for java.


feat: update express version to 4.19.2

I'm using this library in a project and Wiz scan is showing that Express 4.18.2 has vulnerability that was fixed in the version 4.19.2
https://github.com/advisories/GHSA-rv95-896h-c2vc
Issue #180

Thanks!!


[Snyk] Upgrade jest from 29.6.0 to 29.7.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade jest from 29.6.0 to 29.7.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 5 versions ahead of your current version.
The recommended version was released a month ago, on 2023-09-12.

Release notes

Package name: jest

29.7.0 - 2023-09-12Features

[create-jest] Add npm init / yarn create initialiser for Jest projects (#14465)
[jest-validate] Allow deprecation warnings for unknown options (#14499)

Fixes

[jest-resolver] Replace unmatched capture groups in moduleNameMapper with empty string instead of undefined (#14507)
[jest-snapshot] Allow for strings as well as template literals in inline snapshots (#14465)
[@ jest/test-sequencer] Calculate test runtime if perStats.duration is missing (#14473)

Performance

[@ jest/create-cache-key-function] Cache access of NODE_ENV and BABEL_ENV (#14455)

Chore & Maintenance

[jest-cli] Move internal config initialisation logic to the create-jest package (#14465)

New Contributors

@ bawjensen made their first contribution in #14465
@ malaviya-parth made their first contribution in #14467
@ niklasholm made their first contribution in #14507

Full Changelog: v29.6.4...v29.7.0

29.6.4 - 2023-08-24Fixes

[jest-core] Fix typo in scheduleAndRun performance marker (#14434)
[jest-environment-node] Make sure atob and btoa are writeable in Node 20 (#14446)
[jest-worker] Additional error wrapper for parentPort.postMessage to fix unhandled DataCloneError. (#14437)

New Contributors

@ stanleyume made their first contribution in #14424
@ dj-stormtrooper made their first contribution in #14437
@ thw0rted made their first contribution in #14444

Full Changelog: v29.6.3...v29.6.4

29.6.3 - 2023-08-21Fixes

[expect, @ jest/expect-utils] ObjectContaining support symbol as key (#14414)
[expect] Remove @ types/node from dependencies (#14385)
[jest-core] Use workers in watch mode by default to avoid crashes (#14059 & #14085).
[jest-reporters] Update istanbul-lib-instrument dependency to v6. (#14401)
[jest-mock] Revert #13692 as it was a breaking change (#14429)
[jest-mock] Revert #13866 as it was a breaking change (#14429)
[jest-mock] Revert #13867 as it was a breaking change (#14429)
[@ jest/reporters] Marks Reporter's hooks as optional (#14433)
[jest-runtime] Fix dynamic ESM import module bug when loaded module through jest.isolateModulesAsync (#14397)

Chore & Maintenance

[jest-changed-files, jest-circus, jest-console, @ jest/core, @ jest/runtime, @ jest/transform] Use invariant and notEmpty from jest-util rather than own internal (#14366)

New Contributors

@ karlnorling made their first contribution in #14401
@ vinicinbgs made their first contribution in #14388
@ panteleevnikita made their first contribution in #14433

Full Changelog: v29.6.2...v29.6.3

29.6.2 - 2023-07-27Fixes

[jest-circus] Fix snapshot matchers in concurrent tests when nr of tests exceeds maxConcurrency (#14335)
[@ jest/core] When running global setup and teardown, do not try to change the message property of the thrown error object when the message property is unwritable (#14113)
[jest-snapshot] Move @ types/prettier from dependencies to devDependencies (#14328)
[jest-snapshot] Throw an explicit error if Prettier v3 is used (#14367)
[jest-reporters] Add "skipped" and "todo" symbols to Github Actions Reporter (#14309)

Chore & Maintenance

[@ jest/core] Use pluralize from jest-util rather than own internal (#14322)

New Contributors

@ Kloen made their first contribution in #14328
@ eryue0220 made their first contribution in #14322
@ david-szabo97 made their first contribution in #14113

Full Changelog: v29.6.1...v29.6.2

29.6.1 - 2023-07-06Fixes

[jest-circus] Revert #14110 as it was a breaking change (#14304)

Full Changelog: v29.6.0...v29.6.1

29.6.0 - 2023-07-04 Read more

    from jest GitHub release notes

Commit messages

Package name: jest

4e56991 v29.7.0
04f8272 chore: update changelog for release
f0cfd50 feat(jest-validate): Allow deprecation warnings for unknown options (#14499)
176b229 chore: use glob@10 in build scripts (#14512)
56793ab fix(@ jest/resolve): replace unmatched capture group with empty string instead of "undefined" (#14507)
b7828e9 chore: remove a couple filters and move to reduce (#14506)
4feb4ba changed-files: simplify Array.from (#14505)
55d6a22 chore: disable automerge of renovate PR
ab1074c Revert "chore(deps): lock file maintenance"
da878b9 chore(deps): lock file maintenance
e821b83 chore: patch psl to avoid deprecation warning (#14502)
a0e067a changed-files: simplify find repos (#14503)
856d6ab docblock: use flatMap instead of map+reduce (#14504)
3738e3f pretty-format: remove needless map (#14494)
9171085 Upgrade Jest website to Docusaurus v3 (#14463)
70138b8 docs: use `caution` over `danger` admonition (#14493)
d724d2b Prepare website docs content for MDX v2 and Docusaurus v3 (#14492)
febaa2f chore: upgrade `github-actions-cpu-cores` (#14491)
0b0cf73 feat(create-jest): Add `npm init` / `yarn create` initialiser (#14453)
008caa9 chore: group tsd dep updates
9183d01 chore: upgrade TypeScript (#14477)
b303ca1 chore(deps): update dependency eslint-config-prettier to v9 (#14419)
9592877 chore(deps): update actions/checkout action to v4 (#14475)
f3ab93a chore(deps): update yarn to v3.6.3 (#14474)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Changing base branch to development.


test-branch/execute-job/wrong-commit-sha-on-pr/9abe3f2e-119d-446a-b4af-93d346b1babb

PR for testing c85d0cabd7d05e8e4220f0f4d145d02db1c8fbdd

/canonical/self-hosted-runners/run-workflows de6a1255006a3461a465650a60ec10869ffc9ea6


nss-mdns: fix build w/musl

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

For possible use with musl-nscd.


Readthedocs and deps fixes

No linked issues found. Please add the corresponding issues in the pull request description.  Use GitHub automation to close the issue when a PR is merged


Adjust Rancher Backups default encryption setting value

Summary
Fixes https://github.com/rancher/backup-restore-operator/issues/554
This change seeks to make the default form value match the one we cite as recommended on the page.
Because Rancher Backups may contain secrets this visual nudge towards encryption reinforces our recommendation.
Many users may be hosting their own S3 they feel is secure enough and can easily opt-out still.
Occurred changes and/or fixed issues
The larger concept here is that our defaults currently opt for ease of use over security.
This minor change aligns more with best-practice will only adding a very small friction for users opting out of higher security via encryption.
Technical notes summary
Any GUI tests that expect useEncryption to be false by default will see a different default option.
Areas or cases that should be tested
Unless already covered by tests, I'm not sure if changing the default needs to be covered by testing.
This isn't seeking to make any functional changes just how we present the default empty form in the UI.
Areas which could experience regressions

I don't believe this is applicable.
Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@richard-cox - Updated now.

Hopefully https://github.com/rancher/dashboard/actions/runs/10292075245/job/29623852918?pr=11609 will be passing after re-run, seemed to fail due to timeout before.

@mallardduck Thanks for checking back. There's a lot in flux atm given the update to Vue3. Might be best to wait a few weeks and then rebase.
I've added myself as reviewer as well.
Would testing be handled via QA responsible for the backup operator (given issue is https://github.com/rancher/backup-restore-operator/issues/554)?

@richard-cox - I think if dashboard QA could handle it that would be best, otherwise we can ask @nickwsuse if they have bandwidth. Maybe @MKlimuszka knows?

@rancher-max Is this something UI QA could pick up in 2.10?

@rancher-max Is this something UI QA could pick up in 2.10?

Short answer: Yes!
Longer answer: Testing seems rather light here, so we should be able to pick it up. That being said, I'd be more concerned with potential regressions in any automation we have today, specifically around https://github.com/rancher/dashboard/blob/master/cypress/e2e/tests/pages/charts/rancher-backup.spec.ts. Is this something that could be briefly reviewed as part of this PR to ensure we still expect success there without changes to the file?
Note the relevant e2e-test run here in CI is a false-positive as it's not actually running anything:
  Running:  pages/charts/rancher-backup.spec.ts                                           (6 of 104)

  0 passing (1ms)

  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        0                                                                                │
  │ Passing:      0                                                                                │
  │ Failing:      0                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  0                                                                                │
  │ Video:        false                                                                            │
  │ Duration:     0 seconds                                                                        │
  │ Spec Ran:     pages/charts/rancher-backup.spec.ts                                              │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘

  (Uploading Results)

  - Nothing to Upload

Thanks Max. I checked the e2e run on this PR and one in master but both ran the backup test. Could you link the one where the test didn't run (it might be incorrectly picked up from another matric run but correctly not running it, the (6 of 104) count seems strange).
To summarise steps then

@mallardduck can you rebase and give it a quick dev test? There's been a bit port from vue2 to vue3 and it would be good to ensure this hasn't broken since. This would also help the failing gate
@mallardduck would you be able to create an issue in rancher/dashoard for this change and link to this issue? That will make it easier for QA our side to track (@rancher-max let us know if this isn't needed)
I'll give it another look before merging

Could you link the one where the test didn't run (it might be incorrectly picked up from another matric run but correctly not running it, the (6 of 104) count seems strange).

Ah that was also pulled from this run, but the part using the standard_user: https://github.com/rancher/dashboard/actions/runs/10292075245/job/29623855217?pr=11609

@mallardduck would you be able to create an issue in rancher/dashoard for this change and link to this issue? That will make it easier for QA our side to track (@rancher-max let us know if this isn't needed)

Not needed! We can just use the currently linked issue to test off of: https://github.com/rancher/backup-restore-operator/issues/554. I've added it to our project board for easier tracking.

Hiya gents, I am back from vacation now this week. I will work on rebasing it to update for Vue migration and testing locally. Once completed I will report back.

@mallardduck There's some strangeness going on with e2e tests. It's been re-run a fair number of times but each time seams different. Gave it one last go, if that fails I'll investigate

@richard-cox - looks like 6th time was the charm

@rancher-max over to you!


Add Apollo Federation @tag directive support

This adds the Apollo Federation @tag directive support to all supported locations (FIELD_DEFINITION, INTERFACE, OBJECT, UNION, ARGUMENT_DEFINITION, SCALAR, ENUM, ENUM_VALUE, INPUT_OBJECT, INPUT_FIELD_DEFINITION) with this, the only piece missing for https://github.com/async-graphql/async-graphql/issues/982 is the @link directive at the top of the schema.

Thanks!❤


Problems compiling with partial call

The following code won't compile:
actor Main
  new create(env: Env) =>
    let b = B

trait A
  fun a1() ?

  fun a2() ? =>
    a1()?

class B is A
  fun a1() =>
    None

The problem is a1()?.  Whether you leave the ? or remove it, you get a compiler error.
Error:
/Users/jtfmumm/projects/pony/test/partial/main.pony:10:9: call is partial but the method is not - this question mark should be removed
    a1()?
        ^
    Info:
    /Users/jtfmumm/projects/pony/test/partial/main.pony:13:10: method is here
      fun a1() =>
             ^
|_|_| jtfmumm:~/projects/pony/test/partial $ lponyc
Building builtin -> /Users/jtfmumm/bin/ponyc-latest/packages/builtin
Building . -> /Users/jtfmumm/projects/pony/test/partial
Error:
/Users/jtfmumm/projects/pony/test/partial/main.pony:10:8: call is not partial but the method is - a question mark is required after this call
    a1()
       ^
    Info:
    /Users/jtfmumm/projects/pony/test/partial/main.pony:7:12: method is here
      fun a1() ?

In src/libponyc/verify/call.c, we have:
  ast_t* method_error = ast_childidx(method_def, 5);
  if(ast_id(method_error) == TK_QUESTION)

I'd think in this case we should be using the signature on the trait (which is partial), but it seems that it looks up the trait signature in the one case and the implementation signature in the other.

The issue is that the traits pass basically copies methods from the trait into the type implementing the trait, and so the verify pass which comes later, sees something like:
actor Main
  new create(env: Env) =>
    let b = B

trait A
  fun a1() ?

  fun a2() ? =>
    a1()?

class B is A
  fun a1() =>
    None

  fun a2() ? =>
    a1()?

I'm not sure if there's a good way to distinguish "trait-copied" methods from "real" methods in the verify pass. Maybe someone on the sync call will have a bright idea about this?

one thing that worries me about us going in and fixing this is issue https://github.com/ponylang/ponyc/issues/1993.
when investigating that issue, we found that regular classes are going through the trait pass so anything we do with traits to fix this issue is going to potentially impact on non-traits until we figure out and fix #1193.

I can think of two possible approaches:

We could include an "original signature" in a method AST when it is copied from a trait. Failure to type check against the real signature could then result in checking against the original signature.
Or, when such a failure to type check occurs, we could walk through the explicit trait/interface list from the is on the type, looking for signature to type check against. Any such signature that we successfully type checked against would be valid.

It turned out that we were already tracking everything we needed - the ast_data for a method was set to the body_donor of the method in the traits pass:
https://github.com/ponylang/ponyc/blob/682b45d3abd7b24381bfc56423da85c3527785c7/src/libponyc/pass/traits.c#L122-L137
Filed #2261 which fixed this pretty simply.

Thanks @jemc!


Allow the tax_percent to be set when the subscription is created.

Although the tax_percent has a default value, the custom_subscription_params method wasn't allowing it to be set when creating a subscription with a non-default tax_rate.

Thank you!


publish doesn't seem to work on your live example

hmm, for some reason, WebSocket handshake goes wrong.
WebSocket connection to 'ws://test.mosquitto.org:8080/mqtt' failed: Error during WebSocket handshake: net::ERR_CONNECTION_RESET


CASSANDRA-18566: Avoid unnecessary deserialization of terminal arguments when executing CQL functions

It seems that the getArgumentDeserializer method is missing for: DoubleType, DecimalType, Int32Type and IntegerType.

Good catch; added.


editor.codeActionsOnSave: wording in hover help

testing #48303:

hover over "editor.codeActionsOnSave" setting

Observe:
hover says "Code actions kinds to be run on save."
the two plurals sound strange to my (foreign language) ears.
Should it be "Code action kinds to be run on save."?

Yes you are correct. The message should be "Code action kinds to be run on save."


SocketAddr  attempt to subtract with overflow

I think there might be an issue here:
    fn address(&self) -> AddressKind<'_> {
        let offset = path_offset(&self.sockaddr);
        let len = self.socklen as usize - offset;
        let path = unsafe { &*(&self.sockaddr.sun_path as *const [libc::c_char] as *const [u8]) };

        // macOS seems to return a len of 16 and a zeroed sun_path for unnamed addresses
        if len == 0
            || (cfg!(not(any(target_os = "linux", target_os = "android")))
                && self.sockaddr.sun_path[0] == 0)
        {
            AddressKind::Unnamed
        } else if self.sockaddr.sun_path[0] == 0 {
            AddressKind::Abstract(&path[1..len])
        } else {
            AddressKind::Pathname(OsStr::from_bytes(&path[..len - 1]).as_ref())
        }
    }

From the linux man pages for "recv":
   recvfrom()
       recvfrom() places the received message into the buffer buf.  The
       caller must specify the size of the buffer in len.

       If src_addr is not NULL, and the underlying protocol provides the
       source address of the message, that source address is placed in the
       buffer pointed to by src_addr.  In this case, addrlen is a value-re‐
       sult argument.  Before the call, it should be initialized to the size
       of the buffer associated with src_addr.  Upon return, addrlen is up‐
       dated to contain the actual size of the source address.  The returned
       address is truncated if the buffer provided is too small; in this
       case, addrlen will return a value greater than was supplied to the
       call.

Specifically this part Upon return, addrlen is updated to contain the actual size of the source address. I believe if the related socket is not bound to an address, i.e. the value for self.socklen would be set to 0, you would get an error, because in the following line you'd be subtracting some positive value (in my case 2) from 0:
        let len = self.socklen as usize - offset;

Resulting in the following runtime error:
thread 'main' panicked at 'attempt to subtract with overflow', /home/ANT.AMAZON.COM/wssmts/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.7.6/src/sys/unix/uds/socketaddr.rs:31:19

I think in SocketAddr::address we would need to check for self.socklen == 0 and return AddressKind::Unnamed instead, or maybe None?
@WalterSmuts do you have a small example we can use as test?

do you have a small example we can use as test?

Here's the git repo.
And here's a summary:
use mio::net::UnixDatagram;
use std::path::Path;

fn main() {
    let mut buff = [0u8; 1024];
    let path = Path::new("/tmp/arbitrary.sock");
    let socket_a = UnixDatagram::bind(path).unwrap();
    let socket_b = UnixDatagram::unbound().unwrap();

    socket_b.connect(path).unwrap();
    socket_b.send_to(&buff, path).unwrap();

    let (_size, addr) = socket_a.recv_from(&mut buff).unwrap();
    let _ = addr.as_pathname();
}

Produces output:
wssmts@laptop:~/Programs/minimal-socket-addr-error-example > cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
     Running `target/debug/minimal-socket-addr-error-example`
thread 'main' panicked at 'attempt to subtract with overflow', /home/ANT.AMAZON.COM/wssmts/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.7.6/src/sys/unix/uds/socketaddr.rs:31:19
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.


Utils tests

Codecov Report

Merging #92 (dc8fb7e) into main (4b4c215) will increase coverage by 0.66%.
The diff coverage is n/a.

:exclamation: Current head dc8fb7e differs from pull request most recent head 286d7c8. Consider uploading reports for the commit 286d7c8 to get more accurate results

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   86.00%   86.66%   +0.66%
==========================================
  Files           9        9
  Lines         150      150
  Branches       41       41
==========================================
+ Hits          129      130       +1
+ Misses         19       18       -1
  Partials        2        2

Impacted Files
Coverage Δ

facade-app/src/Utilities/Utils.ts
77.41% <0.00%> (+3.22%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM, Thank you !!!.


[PE-D][Tester D] UI: Notes truncated

Notes that are too long are truncated, resulting in the user not being able to view the note information

Labels: type.FeatureFlaw severity.Medium
original: dwangwk/ped#4

Long strings with small placeholders has already been covered in your other issue, #123. Will be closing this as duplicate.


Add canonical URL of each page

Which scope/s are relevant/related to the feature request?
astro-angular
Information
After generating a static site, the URLs are visited using query parameters, which are used to track analytics. However, Google is reporting them as separate URLs.
Can a canonical URL element be added to the root URL during the static build? If not, a feature to do so would be of a great addition.
Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

If this is an Astro app using astro-angular (as mentioned above), you can add the canonical URL with your custom logic in Astro in your base layout or header fragment.
If this is an Analog app (and if the above scope is incorrectly set to astro-angular), then this may be blocked due to an upstream issue: https://github.com/unjs/nitro/issues/2052
An unofficial solution would be to modify the document in the constructor, which would be pre-computed for SSG pages at build time. I'm not sure about the SSR pages.

Got it temporarily solved by adding to postRenderer
// Base URL of your website
const baseUrl = 'https://tapbeep.com';
// Generate the canonical URL based on the route
const canonicalUrl = `${baseUrl}${route.route}`;
// Inject the canonical tag into the <head> section
const canonicalTag = `<link rel="canonical" href="${canonicalUrl}">`;
route.contents = route.contents?.replace('</head>', `${canonicalTag}</head>`);

Cool, we'll note the workaround for now


ERROR: tensorflow-2.15.0-cp310-cp310-macosx_11_0_arm64.whl is not a supported wheel on this platform

Issue type
Build/Install
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.15
Custom code
Yes
OS platform and distribution
Mac OS 14.2 - Apple M3 Pro
Mobile device
No response
Python version
3.11
Bazel version
6.1.0
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
ERROR: tensorflow-2.15.0-cp310-cp310-macosx_11_0_arm64.whl is not a supported wheel on this platform
% python3 -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
v2.15.0-rc1-8-g6887368d6d4 2.15.0
Standalone code to reproduce the issue
https://github.com/msusol/jupyter-notebook-on-macos/blob/main/tensorflow.md
Python.org install
https://github.com/msusol/jupyter-notebook-on-macos/blob/main/README.md#install-python
Relevant log output
No response

Do I have the right clang for installing Tensorflow? Seems I need Clang 16?
https://www.tensorflow.org/install/source#install_clang_recommended_linux_only
Clang is a C/C++/Objective-C compiler that is compiled in C++ based on LLVM. It
is the default compiler to build TensorFlow starting with TensorFlow 2.13. The
current supported version is LLVM/Clang 16.
% which python3
/Library/Frameworks/Python.framework/Versions/3.11/bin/python3
% python3
Python 3.11.7 (v3.11.7:fa7a6f2303, Dec  4 2023, 15:22:56) [Clang 13.0.0 (clang-13<IP_ADDRESS>)] on darwin
% /usr/bin/python3
Python 3.9.6 (default, Nov 10 2023, 13:38:27) [Clang 15.0.0 (clang-15<IP_ADDRESS>.5)] on darwin
% /usr/bin/clang -v
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>.5)
Target: arm64-apple-darwin23.2.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
download https://github.com/llvm/llvm-project/releases/tag/llvmorg-16.0.0: clang+llvm-16.0.0-arm64-apple-darwin22.0.tar.xz

Version
Python version
Compiler
Build tools

tensorflow-2.15.0
3.9-3.11
Clang 16.0.0
Bazel 6.1.0

I don't have the right version of Clang:
% /usr/bin/clang -v
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>.5)
Target: arm64-apple-darwin23.2.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Macports: https://www.macports.org/
Installed macports, in order to install Clang 16.0 easily enough. I avoid ports and brew in general, but this is the one example I'm ok with.
~ % source ~/.zprofile   # Macports already added PATH
~ % sudo port install clang-16
~ % port contents clang-16

~ % /opt/local/bin/clang-mp-16 -v
clang version 16.0.6
Target: arm64-apple-darwin23.2.0
Thread model: posix
InstalledDir: /opt/local/libexec/llvm-16/bin

And now update PATH for Clang
~ % echo "" >> ~/.zprofile
~ % echo "# Clang (v16.0) for Tensorflow v2.15" >> ~/.zprofile
~ % echo "PATH=\"/opt/local/libexec/llvm-16/bin:\${PATH}\"" >> ~/.zprofile
~ % echo "export PATH" >> ~/.zprofile
~ % source ~/.zprofile

and now we have what is needed.
~ % clang -v
clang version 16.0.6
Target: arm64-apple-darwin23.2.0
Thread model: posix
InstalledDir: /opt/local/libexec/llvm-16/bin

After getting Clang 16.0 on MacOS 14 when it appears Xcode only provides Clang 15.0, and now everything is working as expected.
tensorflow_io-0.35.0-cp311-cp311-macosx_14_0_universal2.whl
tensorflow-2.15.0-cp311-cp311-macosx_14_0_arm64.whl


Fix the failure for BalanceNumSegmentAssignmentStrategyIntegrationTest

LGTM

I will merge it once it passed the Travis run. But the other PRs might fail
because of this test.
We should enforce people to first pass Travis run, then merge the PR.
On Tue, Jun 14, 2016 at 5:45 PM, Ravi Aringunram<EMAIL_ADDRESS>wrote:

LGTM
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/linkedin/pinot/pull/202#issuecomment-226058644, or mute
the thread
https://github.com/notifications/unsubscribe/AQvgX1UKGOQNJ84dZiB4vdOxvNyJA4zjks5qL0tEgaJpZM4I14sq
.


Removing Signals

That may sound like a terrible idea - maybe it is - but it is puzzling me so I want to hear your opinion on it. Note this is totally uninplementable without giantic refactorings so I'm not asking that at all. Just thoughts, really.
We all know Monads are great for keeping stateful computation isolated from the rest of the program in a purely functional language. They work on Haskell for many reasons. I just suspect those reasons don't apply to Elm, and the FRP's approach could work totally independent of Monads: we could just keep everything at the language level completely pure and allow values to be changed externally from JS. For example, consider the following program:
external time : Double
time = `Date.now()`

ballPosition : Double
ballPosition = sin time * 40

main : Element
main = circle 200 ballPosition blue

Here, time is defined externally by a JavaScript computation. Whenever it is changes, any element depending on it is changed too. Now, it sounds like "time" is stateful and could cause havoc, but it is actually not. For example, referential transparency still holds. Indeed, the language now is actually much more pure than before, since there is no state on the language level at all - not even inside monads. It might help if you think an Elm program as a function which receives its external variables and returns main.
But why would you do that?
Because that keeps our code much simpler and ridiculously easier to newbiews. Since time's type now is just Double, you can use it anywhere without worrying about lifts, fmaps, which I can't overstate how confusing are for newbies. Even for the initiated, that will still make your code much more concise. For example, compare:
external currentDate, url, lastTweet : String

view : HTML
view = div [] [div [] [text currentDate], div [] [text url], div [] [text lastTweet]]

Versus:
currentDate, url, lastTweet : Signal String

view : HTML
view = Signal.map3 (\ currentDate, url, lastTweet →  div [] [div [] [text currentDate], div [] [text url], div [] [text lastTweet]]) currentDate url lastTweet

I doubt anyone would have trouble understanding the first one. Now, the second... well, good luck explaining lifts, maps. As much as Elm has made things easier, there is still "monad understanding" entry wall on the language right now for anyone who wants to use it properly. Trust me - every single time I explain Elm to a friend, I can see him hitting that wall.
The only reason I see not to take this approach is that it sounds scary that "mutable" values aren't tagged on the type level anymore - but if that is what you are thinking, please, just tell me: what is the "Signal" type protecting us from, that this approach would make us vulnerable? Adding a "pure" and an "impure" value together? But we already do that! Signals are already exactly the same thing; they are already stateful. The only difference is that they are awkward to use.

Do you mind retargeting this suggestion to elm-discuss? This is roughly the idea of "automatic lifting" to signals as used in FrTime which comes with it's own set of trade offs. Folks on the list will be able to talk through this in much more detail!
Also, Elm does not have lift or fmap or monadic signals, and I feel somewhat that this suggestion conflates Elm an Haskell in a weird way. Ask the list though, they can give more detail!

No problems... my bad again :(

No worries at all! I should probably try to get folks on the mailing list more inclined towards discussion in issues like this, but the culture is not like that at this moment. Next time I'll tell them to come here instead!

Okay! Done! You're the fastest repo manager ever. Wish I could star more times! :)

Haha, you caught me on a good day, but thanks for saying so! :)
Is it okay to close this for now, potentially reopening after the list discussion?

Of course, delete if possible...?


added filter to convert images from one type to other

Codecov Report

Merging #34 into master will decrease coverage by 1.01%.
The diff coverage is 64.51%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   74.34%   73.33%   -1.02%
==========================================
  Files           9       10       +1
  Lines         269      300      +31
==========================================
+ Hits          200      220      +20
- Misses         46       56      +10
- Partials       23       24       +1

Impacted Files
Coverage Δ

filters/imagefilter.go
83.05% <60%> (-2.14%)
:arrow_down:

filters/convertimagetype.go
65.38% <65.38%> (ø)

Continue to review full report at Codecov.

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


init should populate vendor or just create a manifest

I know we focused on init doing some initial work, including maybe migrations and the like, but after trying to think about how to explain some of the motivations I found it difficult to explain why init doesn't populate vendor after constructing a manifest and lock file.
ATM, I think that init should either:
a. populate vendor after generating a lock + manifest
b. just create a manifest and leave the lock / vendor population to ensure.

I'm in agreement. The fact that I can scarcely even remember our original motivations for the design we chose suggests to me that practical experience is telling us something.
I'd prefer a).

I'd like to help with this one, cloning the repo now

Strongly prefer b) -- #227 adds rm -rf vendor steps to my life.  It would be nice to disable this behaviour on the command line.  For my own interest, are there links to discussions and background for this change?

@chrrrles In what way does a) add rm -rf vendor steps to your life? If there's an existing vendor/ directory, the tool removes it before populating a new one. Unless you're using some workflow where you actively don't want vendor/ to be populated...?

Looks like this was fixed by #227, can this be closed?

ah yes, thank you 😄


[Feature Request] Gaussian Splat labelling workflow

It would be great for (research purposes and others) if this editor could enable users to visualize and edit labelled gaussian point clouds.
Such a feature would require:

Creation or adoption of a standardized label data format for Gaussian Splatting data. Perhaps using an existing label format developed by segments.ai would be a good approach to avoid standard proliferation.
CRUD operations on labeled Gaussian Splat data. To enable users to load, edit, update, apply, and save labels for a .ply file within the editor.
Option to visualize gaussian cloud based on class id. Perhaps by including additional metadata of the class RGB that a point would be rendered as belonging to that class. This would need to be data-visualization friendly by perhaps recommending ordinal data color gradient choices.

I've started the development of such a feature in a fork: supersplat-classifier. Currently, it also includes hot-reloading of .ply files. Ideas and suggestions are welcome :)

Latest work, perhaps someone is interested in helping out!

Used Segments.ai 3D point cloud Segmentation Label
Hot reload of splat and label data from python notebook using WebSocket communications with custom message format.
'Show Classes' option to color the splat centers based on class color.
Annotation manager to highlight, select and hide certain annotations (WIP).

https://github.com/user-attachments/assets/c6985075-313f-429d-bd01-fcbee0011820

OMG this is awesome!

I wonder if this is creeping features into something complicated that might be usable by very few only. I don't see how hot-loading and python notebooking has to be combined with labeling.

@simonbethke while the hot-loading feature is very limited in it's use cases, a wider application would be the implementation of splat streaming to visualize a training process for example. But you are correct, maybe this specific PR should be concerned only with annotation support.

Yea, streaming a training process would be cool.


Azure WebJobs EventGrid Readme issue

Section link:

Suggestion: Update link to https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-event-grid-output?tabs=csharp
@jongio for notification.

/cc @pakrym

There is no functional difference as C# is the first tab in the table.


Async Crypto Endeavour

Seeing these errors:
  1) "before all" hook:
     Error: Uncaught TypeError: Cannot read property 'x-stream-output' of undefined (webpack:///src/request-api.js?0369*****************:30:0 <- test/browser.js:49882)

  2) "after all" hook:
     Error: Uncaught TypeError: Cannot read property 'x-stream-output' of undefined (webpack:///src/request-api.js?0369*****************:30:0 <- test/browser.js:49882)

So far looks good to me

CI is being super weird...

Ok, CI is good now, just fails for the test of adding nested dirs on the browser (reported bug: https://github.com/ipfs/js-ipfs-api/issues/339).
We've already talked about how to act with regards to this in the past and the decision is to continue moving forward (merge and release), while keep tackling that specific test. To make sure there is no confusion, I'm going to add a note to that in the readme.


Implement time functions in Nitro

This PR adds support for time functions (getclockres() and getclocktime()) in Nitro

+1+1 = +1 reached, merging.


Add longer seekbar when layout allows

When the window is large the seekbar doesn't expand to fill, which for super long tracks makes seeking difficult.

It would be nice to have the seekbar expand to fill the available horizontal space for this.

That's a good point, and should be an easy fix! Thanks for the feedback :)

Would it be possible for me to pick up this ticket?


For a patch release

(bug fix) Resolve performance issues in schema, and scitype(::AbstractString) in the MLJ convention (#70)

(bug fix) Resolve problems with tables that are also abstract arrays (#71)

Make conventions a type (#36)

Codecov Report

Merging #75 into dev will decrease coverage by 1.09%.
The diff coverage is 65.21%.

@@            Coverage Diff            @@
##              dev      #75     +/-   ##
=========================================
- Coverage   89.59%   88.49%   -1.1%
=========================================
  Files           8        8
  Lines         221      226      +5
=========================================
+ Hits          198      200      +2
- Misses         23       26      +3

Impacted Files
Coverage Δ

src/autotype.jl
97.91% <100%> (+0.04%)
:arrow_up:

src/conventions/mlj/finite.jl
83.78% <100%> (ø)
:arrow_up:

src/conventions/mlj/images.jl
100% <100%> (ø)
:arrow_up:

src/schema.jl
81.39% <100%> (ø)
:arrow_up:

src/conventions/mlj/mlj.jl
90% <40%> (-3.11%)
:arrow_down:

src/ScientificTypes.jl
85.71% <50%> (-14.29%)
:arrow_down:

src/scitype.jl
75% <55.55%> (+2.77%)
:arrow_up:

Continue to review full report at Codecov.

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


Chore: Keep py-glaredb and glaredb versions in sync

Description

I think the only thing needed to release both at the same time is trigger python releases on 'v*' tags.
Also requires that we update Cargo.toml and py-glaredb/Cargo.toml at the same time.
What This Enables

Impacts

Noted to update both cargos.
@scsmithr will a root cargo build suffice, or do we need a second command for py-glared ?


[All OSes] Powershell Core will be updated to 7.4.x LTS on January, 28

Breaking changes
Powershell Core on all runners will be switched from the 7.2.x LTS branch to the 7.4.x LTS one. The description of the changes can be found on Microsoft learn and in the dev blog.
Target date
The images rollout process will start on January 28 and take 3-4 days.
The motivation for the changes
According to the support lifecycle the 7.2.x branch will have reached its end of life by November, 8 this year and we would like to provide the latest LTS versions.
Possible impact
We do not expect any major breakages, though we highly recommend study all the changelogs from the links above to avoid unexpected behaviour.
Platforms affected

[X] Azure DevOps
[X] GitHub Actions

Runner images affected

[X] Ubuntu 20.04
[X] Ubuntu 22.04
[X] macOS 11
[X] macOS 12
[X] macOS 13
[X] macOS 13 Arm64
[X] Windows Server 2019
[X] Windows Server 2022

Mitigation ways
Here are the runtime downgrade paths if 7.2.x is still needed:
Windows:
        set "extractPath=C:\Program Files\PowerShell\7"
        curl -sLO https://github.com/PowerShell/PowerShell/releases/download/v7.2.17/PowerShell-7.2.17-win-x64.zip
        RMDIR "%extractPath%" /S /Q
        7z x PowerShell-7.2.17-win-x64.zip -o"%extractPath%"
        pwsh --version

Linux:
        sudo apt-get remove powershell
        curl https://packages.microsoft.com/config/ubuntu/22.04/packages-microsoft-prod.deb -o packages-microsoft-prod.deb
        sudo dpkg -i packages-microsoft-prod.deb
        sudo apt-get update
        sudo apt-get install -y powershell=7.2.17-1.deb
        pwsh --version

macOS:
        curl -L -o powershell.pkg https://github.com/PowerShell/PowerShell/releases/download/v7.2.17/powershell-7.2.17-osx-arm64.pkg
           sudo installer -pkg powershell.pkg -target /
           pwsh --version

As part of this work package can I please ask for the following additional enhancements as this would greatly benefit the community and has previously been asked for by myself & generally makes sense to enable this pattern not only for PowerShell but other tooling too to reduce unnecessary and painful custom pre-execution scripts or custom tasks to enable more flexible testing matrixes that are much needed going forward.

[ ] Add a settable specific version option
[ ] Add a use Preview version option
[ ] Add a use Daily build option
[ ] Add a use Stable version option - This is less needed now that we are on an LTS release but will in future be required between this LTS and the next LTS release so makes sense to just add it now and save on the future work needed to add it

Once 7.5 comes out, having the latest stable as an option makes sense, and would be in line with other tools. I would think anything beyond that would be a matter for an tool installer task, no?

Which I expect is already the task used for installing PowerShell on the images in the first place, which as part of this work package would need to be updated anyway, to use the latest version, so I don't see why asking for that task to be updated in such a way that when you then try call PowerShell as an action in your workflow with additional flags asked above is too much of an ask here.
This would allow in my mind for something like the below to be viable & nice and clean in GitHub/Azure DevOps yaml files
GitHub
shell: pwsh
run: Test-Path resultsfile.log | Should -Be $true
version: preview

shell: pwsh
run: Test-Path resultsfile.log | Should -Be $true
version: daily

shell: pwsh
run: Test-Path resultsfile.log | Should -Be $true
version: stable

shell: pwsh
run: Test-Path resultsfile.log | Should -Be $true
version: 7.3.6

Azure DevOps
pwsh: Write-Host Hello $(name)
displayName: Say hello
name: firstStep
workingDirectory: $(build.sourcesDirectory)
failOnStderr: true
env:
    name: Microsoft
version: daily

pwsh: Write-Host Hello $(name)
displayName: Say hello
name: firstStep
workingDirectory: $(build.sourcesDirectory)
failOnStderr: true
env:
    name: Microsoft
version: stable

pwsh: Write-Host Hello $(name)
displayName: Say hello
name: firstStep
workingDirectory: $(build.sourcesDirectory)
failOnStderr: true
env:
    name: Microsoft
version: 7.3.6

and then that way this tells the runner to check for and if not installed, install the specified version to the runner as a setup task which would run prior to running that step and then use the referenced version for that step.
This may increase size of runner images over time as well as in some cases make a build a little slower, but eventually this'll help make management & maintenance easier for both cloud hosted and self hosted runners, even if it is a stretch goal in asking for it, but would be good to see happen

Finally reach point I think, thanks a lot

For anyone hitting the same issue, this upgrade in powershell version brings the following breaking change: https://github.com/PowerShell/PowerShell/issues/20744
The fix (which will work with both old and new pwsh) is to use splatting (see example in that thread).

Deployed


Append import to multiline import

Hi guys!
Congrats for this amazing plugin 🎉
Just spotted an issue when having multiline imports:
Disclaimer: this issue is not linked to React Native, I'm just using it for the purpose of this issue ;)
Initial situation
// Some multiline import
import {
  View,
  StyleSheet,
} from 'react-native';

// ... some other imports

Then the plugin add the import for the Image component from 'react-native'
Current behaviour
import {
  View,
  StyleSheet,
} from 'react-native';

// ... some other imports
import { Image } from 'react-native'; // <= unfortunatly it adds another import from `'react-native'`

Expected behaviour
import {
  View,
  StyleSheet,
  Image, // <= It would be super nice to have it here :D
} from 'react-native';

// ... some other imports

Cheers 🚀

For me it looks like this:
import {     Component, Input, Output, EventEmitter, ViewEncapsulation, HostBinding, OnInit, OnChanges } from '@angular/core';
Adding SimpleChanges:
import { Component, Input, Output, EventEmitter, ViewEncapsulation, HostBinding, OnInit, OnChanges, **SimpleChanges** } from '@angular/core';     Component, Input, Output, EventEmitter, ViewEncapsulation, HostBinding, OnInit, OnChanges } from '@angular/core';
This doesn't compile really well

I will take a look into it.


Modular GCD

Link to problem : https://www.codechef.com/problems/GCDMOD
#1

Pushed the edited README.md.

Thanks! @Shashwat-Garg


Custom question  and matrix

Hi there,
I am using surveyJS (jquery) for psychological testing, in particular matrix question (with strongly disagree- strongly agree).
And I find myself with some troubles :

As a psychometrician I can tell that having different "column width" in a matrix question is an important bias for participant. They would probably be biased with the distance and put a psychological interpretation on it while there is not. In fact "Strongly disagree" is longer than "disagree".

Can you think of a way to force column width to be identical ? (For isntance Strongly disagree would be on two rows).

I did not manage to find how to increase the text size in the questions. AND Could it be possible that the matrix question takes two rows ? (it's more clear for the eyes).

Thanks in advance,
Félix

@felixcuneo You may use the custom css feature. Take a look at the plunker example, I have created: https://plnkr.co/edit/bHuE7VJALQpbygPQyA1Z?p=preview
I have added a matrixtable class:
.matrixtable tr { height: 50px; }
.matrixtable td:first-child {
  width: 3%;
}
.matrixtable td:not(:first-child) {
  width: 1%;
}

and apply it:
var myCss = {
    matrix: {root: "table table-striped matrixtable"},
};
$("#surveyElement").Survey({
    model: survey,
    css: myCss
});

Let me know if it works for you or not?
Thank you,
Andrew

Hi @Andrew, it works fine thank you very much.
For web noobs like me, you can add style="font-size:180%" in your div section to increase question text size.
Thanks !

Okay, the height and alternate shading are great, but how do we set the width of the columns of the matrix question?  There is no example of using the custom css feature to actually change the formatting of anything.

It's can be achieved via the following CSS:
.matrixtable td:nth-child(2) {
  width: 150px;
}

For more details you can check the documentation

Thank you.  Actually, I knew the CSS for the formatting I want, but I can't figure out from the above plnkr, this example or this example how to get a survey to use any formatting that I specify.  I like the matrixtable shading etc, and I just want to add the column widths, but I can't figure out how to get the survey to use that styling.  Actually, I can't even adapt the "CSS classes" example and make the text of my required question red.  My CSS is just ignored no matter how I try to implement it.


Fix email links

adjusting all current email communications to have actual links that point to our site's login page rather than dummy links. tested it works, please test on your end as well, you can simply attempt to login and see if the email you recieve works for you

For organizationSignUp.js and userStudentSignUp.js, you may have to use the
https://knightassist-43ab3aeaada9.herokuapp.com/#/studentemailverified
and
https://knightassist-43ab3aeaada9.herokuapp.com/#/organizationemailverified
links rather than the login page since that is the page where users can verify their emails, other than that looks good

@AnishaRan this was addressed lmk if additional changes are needed, otherwise you can test and delete the branch after. thank you!

Looks good, thanks!


Don't Merge! Regression pull request test

Docs Build status updates of commit e6359ec:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit e6359ec:
:clock10: Full build: the average full build time is 10 min(s) 26 sec(s), this is based on the last 16 build(s) for this repository.

Docs Build status updates of commit e6359ec:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

developer/TestingPullRequest.md
:warning:Warning
View
Details

developer/TestingPullRequest.md

[Warning] Metadata with following name(s) are not allowed to be set in Yaml header, or as file level metadata in docfx.json, or as global metadata in docfx.json: content_type. They are generated by Docs platform, so the values set in these 3 places will be ignored. Please remove them from all 3 places to resolve the warning.
[Warning] the value invalid_type is undefined for metadata content_type
Line 7: [Warning] Invalid file link:(~/../nonexisted1.md).
Line 8: [Warning] Invalid file link:(~/../nonexisted2.md).
Line 9: [Warning] Invalid file link:(~/../nonexisted3.md).

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


ReadOnly DateTimeEditor

Does DateTimeEditor has a readonly problem ?
It allows edit anyway.

nope its working fine bro..have u installed updates n changed Jquiry reference in scriptbundle.json file..
applicable for updated version.

@ANDY7575  Are you sure, i've used this in dialog.ts.
this.form.DataInsercao.readOnly = true;~
And it still edits....

@volkanceylan can you confirm this behaviour please.

I tried both DateTimeEditor and DateEditor with ReadOnly attribute and it doesnt allow editing, so you probably have something misconfigured.

@brunobola  why you are doin this this.form.DataInsercao.readOnly = true;
Apply readonly attribute on  form or Row  unless u have to Apply it on some condition..

@ANDY7575 of course i need a condition it admin then not readonly else readonly.
It works with other editors timedit not, so must be a but i believe.

Since no one give me a solution I have to thnks this is the default behaviour unlike other editors......

@brunobola  sorry brother have no idea. Ask @volkanceylan master...

Who said there is a readOnly property on DateTimeEditor? Using a property that is not there, and you are also getting TypeScript errors probably, and reporting it as a bug??
someEditor.set_readOnly(true)
or
EditorUtils.setReadonly

@volkanceylan you were right, this property exists with other lookup fields but not dateeditor, sorry my bad.
Yout could be more friendly thought, maybe its your english skills, but as you may know I try to help if I can, I don't use only your platform so not a specialist.
And by the way if someone buy the premium package your support will be like this even if user is a bit lazy ?

I might not be so friendly sometimes, but should check if you are friendly enough.
There is nothing wrong with being novice but being lazy and reporting like there is fire somewhere is not something i tolerate, let's check some of your issues, and reporting style, e.g. CAPS and Exclamations and insisting BUGs:
Editor grids won't work as expected (like normal grids) !!!
BUG with Dialog Layout with Maximize and Close buttons - 3.0.6+
Serenity Calendar bug (while its calendar widget and its compatibility to jquery 3.0)

@volkanceylan yes I understand, but I wan't I can check alot of issues that people asked you and you just answer "read the guide".... And worse, no need to prove my point, I never said anything personal to you and by the way most of these posts I give you the reason and know your platform is good.
You didn't answer to my questions if I give you 2000+ $$$ your answers will be the same ?

Hey guys cool down please.
@brunobola  If you need some help ask for help. Do not report bug/ something like that if you are not sure.
@volkanceylan  is working hard in this platform to improve further. He is trying to fix all bugs that are reported. When he see an issue with BUG then he expects that should be a really bug.
So. please don't disturb him.
Thank you.

 We're not living in your dream world, where you have right to be lazy as hell, and person who is working hard has to be nice as angels.  Because you must be pretty sure in bug. When you architect, write and test every little thing by yourself messages like ''BUUUUGGGGG OH SERENE DOESNT WORK'' makes you know. I use sore programming English but if you say its rude, well I dont know how to say reread documents, really.
https://github.com/volkanceylan/Serenity/wiki/Issue-Guidelines

@volkanceylan You take things to much personal and why the use of "BUUUUGGGGG OH SERENE DOESNT WORK" ?
Don't really wan't to enter in that field becasu I always said your platform is good (not perfect) and you are a person like me, the diference is that I know how to make my pride not to cloud my mind and you on the other and seem more like a programmer with no good inter-personal/social skills (you know kindness) only if people give you something in return.
I like your platform but in terms of support you have a lot to learn.
That said always will consider you an excelent master programmer (with web based C#/MVC) and this is a pretty good platform, don't reaaly know you personally so just expressing my opinion based on what i have seen so far (please don't mistake this comments like if I'm attacking you nothing like that, I know you aren't from an english country, but please try to read carefully before responding).
Best regards, and will always help others if/when i can.

Guys stop here. Take a coffee, take a break. Discussing while you are angry does not take you anywhere.
Is a bad world, but here is a SERENE world ;)

@Estrusco WIll do, but i'm not angry :)


Move duplicate rand functionality to test rand

This PR removes duplicate rand functions and has test files use the rand functions declared in traffic_ops_golang/test/rand.go removing the need to import "math/rand" in multiple files.

Which Traffic Control components are affected by this PR?

Traffic Control Cache Config Unit Tests
Traffic Monitor Unit Tests
Traffic Ops Unit Tests
Grove Unit Tests

What is the best way to verify this PR?

Run the unit tests for the components above
PR submission checklist

[x] This PR has tests
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Possible improvement: occurrences of \*test\.Rand are just a bit more common than those of [^*]test\.Rand - should these maybe return values instead of pointers?

We would have to declare variables wherever a pointer is needed because you can't get the address of a function value returned in go. So instead of just test.RandStr() it'd be:
randStr := test.RandStr()
...
Fqdn:            &randStr,
...

And we'd either be pointing to the same random string variable where needed or create multiple random string variables and point to those. Is this what you mean or do you know of another way?

You could make use of the github.com/apache/trafficcontrol/lib/go-util package's functions IntPtr, StrPtr, etc. So it'd be
...
Fqdn:   util.StrPtr(test.RandString())
...

no forward declaration required


Let TaskGroup tasks return a value when joined

There should be a spawn function that allows returning  a value from a TaskGroup task.
https://discord.com/channels/990354215060795454/1060599778410500206/1060603991819571270

Closed by https://github.com/fedimint/fedimint/pull/1591


Dropwizard BOM provides dependency management for modules outside the Dropwizard project

The BOM is setting dependency management for modules that aren't part of Dropwizard. This is not ideal, as it can cause conflicts when other projects import the BOM.
A simple example would be if I had a project using the Dropwizard BOM, and also depending on something that depends on tomcat-jdbc:9.0.8 and tomcat-dbcp:9.0.8. In this case I would get tomcat-jdbc:9.0.22 from Dropwizard, and tomcat-dbcp:9.0.8, which might not work together. This will happen even if I don't use a Dropwizard module that requires tomcat-jdbc.
See a similar discussion here https://github.com/dropwizard/metrics/issues/1336
Ideally the BOM would only set the versions of io.dropwizard modules, and let the consuming project handle resolving any conflicts between transitive dependencies. This is also how several other modular projects do their BOMs (e.g. JUnit, Jackson, Camel)

@dropwizard/committers Any strong opinions on this?
The advantage of having all dependencies in dropwizard-bom is that every project automatically can use the pre-configured (transitive) dependencies without having to declare their versions.
In that way it's similar to org.springframework.boot:spring-boot-dependencies:
https://github.com/spring-projects/spring-boot/blob/7712f9c0e5a84f04a9c93c195b90d05cfcf50a67/spring-boot-project/spring-boot-dependencies/pom.xml
The difference between JUnit, Jackson, Camel, and Dropwizard (or Spring Boot) is that the latter is a framework that basically rules how to build your application while the others are libraries which are usually added to existing applications.

I like having everything self contained

Just to make sure we have a common understanding of what removing the transitive dependencies from the BOM would do: Consuming projects get Dropwizard's transitive dependencies on their classpaths regardless of whether the BOM mentions them or not, the only difference is whether the BOM overrides Maven's normal transitive dependency resolution for these artifacts.
Let me give some examples using tomcat-jdbc (these artifacts don't necessarily make sense together, but illustrate the principle):
Example consumer POM:
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-bom</artifactId>
    <type>pom</type>
    <scope>import</scope>
</dependency>
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-core</artifactId>
</dependency>
<dependency>
    <!-- Uses tomcat-jdbc -->
    <groupId>com.zaxxer</groupId>
    <artifactId>HikariCP</artifactId>
</dependency>

With the current BOM, the tomcat-jdbc version is determined by the BOM. As I'm not using a part of Dropwizard that depends on tomcat-jdbc, this is surprising behavior. I can work around this by declaring my own dependencyManagement section, which overrides the BOM, but this is unnecessary hassle in this case, since there is no reason for Dropwizard to care which version of tomcat-jdbc I'm using here.
With a BOM where transitive dependencies aren't declared, the tomcat-jdbc version is  determined by Maven's ordinary dependency resolution. In this case I would get the version pointed to by HikariCP.
Example 2:
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-bom</artifactId>
    <type>pom</type>
    <scope>import</scope>
</dependency>
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-core</artifactId>
</dependency>
<dependency>
    <!-- Uses tomcat-jdbc -->
    <groupId>com.zaxxer</groupId>
    <artifactId>HikariCP</artifactId>
</dependency>
<dependency>
    <!-- Uses tomcat-jdbc -->
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-db</artifactId>
</dependency>

With the current BOM, I get the version used by Dropwizard. If there's incompatibility with HikariCP, I need to resolve it somehow.
With a BOM where transitive dependencies aren't declared, Maven would determine the tomcat-jdbc version for me. In this case I'd get the Hikari version, as it's declared first. This is exactly the same situation as with the current BOM, since I still need to check that the Dropwizard and Hikari tomcat-jdbc versions are compatible.
Example 3:
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-bom</artifactId>
    <type>pom</type>
    <scope>import</scope>
</dependency>
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-core</artifactId>
</dependency>
<dependency>
    <!-- Uses tomcat-jdbc -->
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-db</artifactId>
</dependency>

With the current BOM, I get the version declared in the BOM.
With a BOM where transitive dependencies aren't declared, I'd get the tomcat-jdbc declared in the dropwizard-db POM. This is the same version as would be in the BOM.
The only case here where having transitive dependencies in the BOM helps is in example 2, if  the Dropwizard tomcat-jdbc version is compatible with the chosen HikariCP version, but not the other way around. Whether this behavior is helpful is up to coincidence, since I could have declared the dependencies in reverse order (Maven would then pick Dropwizard's version) or Hikari's version could be the one that works for both projects.
The downside to declaring transitive dependencies in the BOM is that it expands the number of dependency conflicts the consuming project needs to resolve. If I have a dependency that conflicts with Dropwizard and I'm using that part of Dropwizard, I need to resolve it whether the BOM has transitive declarations or not. When the BOM has transitive declarations, I potentially additionally need to override those declarations, even if I'm not using the part of Dropwizard that has those dependencies.

I don't know if this is related but there is a BOM's pom that's tagged and released as 9.4.20 but mentions jetty-util 9.4.18 and breaks my websockets for org.eclipse.jetty.util.compression.InflaterPool is not present in 9.4.18.
Took me about an hour to figure out where the 9.4.18 util came from.

The dependencies have been reorganized in #2897.
@black-snow Could you please create a new issue for this and add a minimal example to reproduce the problem?

Will most likely be fixed with 2.0.0.


Active Directory Managed Identity authentication requires Principal ID NOT Client ID

In the section "Using Active Directory Managed Identity authentication" that describes using the Active Directory Managed Identity authentication, you should specify that the ID's Principal ID is required, not the Client ID. This is not clear from the docs.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 947b6a2f-66c5-821d-8c58-8798a48a9f3f
Version Independent ID: 35874772-4b2c-7679-7fa3-57e77e02fbb0
Content: Using Azure Active Directory authentication with SqlClient - ADO.NET Provider for SQL Server
Content Source: docs/connect/ado-net/sql/azure-active-directory-authentication.md
Product: sql
Technology: connectivity
GitHub Login: @David-Engel
Microsoft Alias: v-davidengel

The docs are already pretty specific about it:

For a user-assigned managed identity, the client id of the managed identity must be provided when using Microsoft.Data.SqlClient v3.0 or newer. If using Microsoft.Data.SqlClient v2.1, the object id of the managed identity must be provided.

And code examples are provided for both version ranges with appropriately named placeholders.
The Azure Portal even labels the ID when you look at the overview of the Managed Identity: "Object (principal) ID"
How could it be made more clear?
Thanks,
David

@ssemyan -- Scott, thank you for your feedback.
@David-Engel -- David, thank you for clarifying.
I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


deprecate onnx runtime server

/kind bug
What steps did you take and what happened:
onnxruntime has been deprecated https://github.com/microsoft/onnxruntime/pull/7818, we should switch to use triton for serving onnx model instead.
What did you expect to happen:
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Istio Version:
Knative Version:
KFServing Version:
Kubeflow version:
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

switching to use triton inference server to serve ONNX model in https://github.com/kserve/kserve/pull/1926


drivers: counter: esp32: fix missing absolute alarm flag check

Current ESP32 counter driver is not implementing the absolute alarm option.
Closes #50368

I consider this a bug as it won't work when the absolute flag is selected. In case it is a feature, no problem waiting release window to end.


ParsedCommand remove quotes from string argument

Hello.
How to reproduce problem:
Execute command Run "Play('animation')"
Expected: method Run got argument Play('animation')
Got: method Run got argument Play(animation)
Same behaviour with command Run 'Play("animation")'.

I was able to reproduce and fix.
Thanks for your help!