added react-tools to examples/react/package.json

react-tools is required in dependencies for examples/react to run properly.

If you pull jest from master you shouldn't need to use react-tools. react-tools is deprecated. I just recently merged a PR that added react 0.14 to the examples folder but I think we didn't update the website yet.

The examples/react/preprocessor.js uses the module react-tools to transform the JSX, and the example fails to run without first installing it.
Due to react-tools being deprecated though, the example could also be written to use babel, but that's already covered in the react-es6 example. Could the react example just be removed in favor of react-es6 to reduce confusion?

Oh I didn't realize there were two different ones. Yeah, I'm just going to kill the react one and rename the react-es6 to react :) Thanks for bringing this up!

Done in https://github.com/facebook/jest/commit/2597a0b16e5a17640d4455ce36729e462af38365


feat: allow processing placeholders in same directory

Hey, I just made a Pull Request!
This change allows to resolve YAML placeholders that are located in the same directory as the source is. This enables to split entity files, especially very long templates, to multiple parts inside the same directory.
For example in #15241 the $yaml includes can now be done now in local filesystem instead fetching them from the remote. Only limitation is that the included files must be in the same directory (or subdirectory) as the file including those is.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Pulling in resident security expert @Rugvip here too

See also #18297

@drodil yep I see why it could be useful, but we have to implement it in a way that does not compromise security. Do you have any ideas for how we could lock this down enough for it to be safe to ship given our concerns?

@Rugvip Hmm, that's a pretty hard one; your bar with this is surprisingly high given that we are only reading files which most systems can do, and where the OS is the gatekeeper for which files you can actually read. Maybe allow only absolute paths and allow the application to determine those in the config file?


Required validator treats false as "empty" value

In 0.5.0, commit 383f1e11f3a6a59499f750d39de5a4250b13254a, the required validator was changed to treat false as an "empty" value.
My understanding of the "required" validator is that it validates that a field is not empty. Empty to me includes things like null, undefined, empty strings, or empty arrays. false does not mean empty however. It is often a desired valid input, such as for radio yes/no inputs that are v-modeled to a boolean in data. As long as the user selects one option, the field is valid.
I'm curious as to the intention of this change. Not a big deal as I can roll a custom validator for this use case, but I just wanted to see if you could clarify this for me. If this change is to stay, I would suggest updating the documentation for the required validator to include that false is invalid. Happy to PR that.
Here's the reproduction, I'd expect clicking the "false" button to cause $v.$invalid to be set to false. However, the only way to make the form valid is to click "true".
https://jsfiddle.net/su1z053a/
Thanks!

I agree it's not clear, that's an edge case and it needs documentation. Using false as a valid value surely isn't common, especially in comparison to checkboxes.
I'd say there is an additional validator required to fix this, but I'm not yet sure what the configuration would be.
One option would be to:

go with current required
create validator defined = v => v !== undefined && v !== null.

In your case defined would be used for the radio group value.
Second option:

remove false check from required restoring previous behaviour
create validator checked or truthy that would simply be an boolean cast v => !!v

I'm in slight favour of the first option, but it's hard for me to find any specific reason why that would be better. Any argument from you in either direction would be appreciated.

I just ran into this also.  My example of false as a valid value is that I'm using radio options with true and false as the options the API uses.  But either answer is an indication that the question was answered, while it should only give the error if neither was chosen.
Looking forward to a resolution to this.  Until then I am turning off validation for these radios.

My background is from knockout validation, also model based.
There the required validator only checks:

not undefined
not null
not empty string

In vuelidate it checks the three above and:

not invalid date
not empty array
not empty object
not false

I was surprised that it did validate that much, considering that vuelidate is model based. Personally I would have expected more fine grained control. If I want to validate an empty array I would use required and minLength:1 together.
The inclusion of date, empty array and empty object is fine though, I can live with that.
The false check really surprised me. Tons of times I want a boolean, and false is also a valid choice. I think it would be almost equally wrong to make 0 a non-valid value when using the required validator. It is a falsy number, but yet a number. Same with false, it's false but a boolean.
My opinion is to do the following:

Remove false check from required
Add a equal validator that check for strict equality to use when true or false is wanted. Can also be used to check for other values like numbers and object references.

This is like your second option, but using a more generic validator equal that I think has more use cases.

Is there any decision on this?  @mokkabonna makes some very good points and suggestions for how to handle.

I agree, it just needs some time investments to forge it into some code and tests. I'm probably going to do that eventually but it's not the highest priority ATM for me. Any PR is very welcome 🙂

Just encountered this with requiredIf. +1 for allowing false as a valid option.

This seems to have been fixed in #288

Closing as it was merged into Vuelidate.


Wiki filenames for non-latin character are escaped / not UTF8

Description
When creating a Wiki page with non-latin characters in the title (Greek in my case), the resulting MD file is using escaped characters. Please see screenshots bellow.
Seems like Wiki filenames are not using UTF-8 ?
Gitea Version
1.19.3
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
2.38.5
Operating System
Linux gitea-gitea1 4.4.180+ #42962 SMP Sat Apr 8 00:14:24 CST 2023 x86_64 Linux
How are you running Gitea?
Official Docker image from: https://hub.docker.com/r/gitea/gitea
Database
SQLite

That's an legacy problem. In history, the wiki filename system was not well-designed. The current rule is: most chars are encoded to URL parameter format.

The real problem is more complicated than it.
If you have a chance to work on 1.20, you can take a look at my PR for the details:
Make wiki title supports dashes and improve wiki name related features #24143
(the purpose of that PR is just clarifying the details, it doesn't change the legacy behavior, there were already a lot of technical debts)

Note that we can not dump the given title as-is into the file system because because not all strings are valid filenames, like filenames with / or :. URL-Encoding them is the safe option. See here for regex to match for invalid filenames:
https://github.com/sindresorhus/filename-reserved-regex/blob/main/index.js


SQL error

No database

But you can't say there's "no database" if nothing is requiring a database


adjust the solver script to read json format

At the moment the solver script accepts the information in csv format, but it would be nice if this could be adjusted to read json as well

Not terribly needed at the moment. CSV is easier to quick manipulations by hand, and at the point there are no plans to webify this app. Potentially revisit at another date but atm it's not important enough to keep around.


Get layer2 addresses from DHCP, not hardcoded array

Is this a bug report or a feature request?: feature request
What happened: I just wonder if it is possible to assign IP addresses from DHCP, rather than hardcoded array?
What you expected to happen: I would love to have possibility to use config like this:
apiVersion: v1
kind: ConfigMap
metadata:
namespace: metallb-system
name: config
data:
config: |
address-pools:
- name: my-ip-space
protocol: layer2
addresses:
- DHCP
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

MetalLB version:
Kubernetes version:
BGP router type/version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):

Thanks for the report! This is not currently possible, but it's something we might do in the future, and it's tracked in #157 .
Duplicate of #157


Added documentation for windows users

More clarity for build command for small change.
Fixes Issue: #20527

@timmyreilly : Thanks for your contribution! The author, @dominicbetts, has been notified to review your proposed change.

@timmyreilly - many thanks for contributing this clarification.
#sign-off

@TrishaMc : Thanks for your contribution! The author, @dominicbetts, has been notified to review your proposed change.


VS don't get extension logo

Good morning, I'm playing with the AL extension at Visual Studio code and I want to put some example logo at my extension.
The logo help says the following: 'URL to the app package logo'.
I thought that if I put some url of googling image will work, but doesn't works.
Below my app.json code:
{   "id": "c8c89fbb-12e4-4478-b463-e08119392b5c",   "name": "PackagePrueba",   "publisher": "Paperez",   "brief": "",   "description": "",   "version": "<IP_ADDRESS>",   "compatibilityId": "<IP_ADDRESS>",   "privacyStatement": "",   "EULA": "",   "help": "",   "url": "",   "logo": "http://www.staffcreativa.pe/blog/wp-content/uploads/IMAGEN-4.png",   "capabilities": [],   "dependencies": [],   "screenshots": [],   "platform": "<IP_ADDRESS>",   "application": {     "version": "<IP_ADDRESS>",     "locale": "US"   } }
And the error I get is:

error AL1001: Source file 'http://www.staffcreativa.pe/blog/wp-content/uploads/IMAGEN-4.png' could not be found

The url logo could be a local path to the dir of my extension project?

Yes, the url to the logo file needs to be in your extension project directory.
For instance you could have: "media/myLogo.png"

Solved, thanks :)


Sample Code & Animation for Flow Widget

Description
Adds sample code and embeds the animation of the widget in-action for Flow.
Related Issues
Fixes #10146
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[x] No, this is not a breaking change.

Diagram has landed.


Fix typo in continuous summary function

https://github.com/RhoInc/web-codebook/pull/301/files/c8a2c694f71158d6775e7298fad79efed8acb9f6..1f9758c7aedd38af5d49d46e0544399846318e51#diff-1565de4ea8328a1c4725fcc7ba8a8850R12

Looks like this was fixed in 9d1d1e41fff99b687eed4a5c3bc072db9865f743


orginized imports

remove unused imports and organized imports

+1 even so do not see much value :-P


how to react to uploads of not allowed extensions

while it works to restrict a user from uploading files via ngFileSelect when setting the options accordingly i am missing a way to interact with the user and show a message that the extension is now allowed.  Is there a way of doing this?
config:
this.options = {             url: '/api/upload',             filterExtensions: true,             allowedExtensions: ['image/jpg']         };

Hi @schwamster. You mean when you update options?

sry, i guess i wasn't to clear. Here is what i want to be able to do:
Given options.allowedExtensions are limited to image/jpg && options.filterExtensions is true
When a user chooses a file with extension pdf
Then upload does not start
Then a message is displayed to the user: "Only jpg's are suppported"
The part that i am missing is the last "Then" and i have not found a way of doing that in the example. What i could imagine as a way of doing that is another event "onUploadRejected" => for example:
<input type="file" style="display: none;"                            ngFileSelect                            [options]="options"                            (onUploadRejected)="handleUploadRejected($event)"                            (onUpload)="handleUpload($event)" multiple>
Or are there other ways of doing that?

hmmm.. I am not aware of any hardcoded string like Only jpg's are suppported or such, is that maybe a browser specific reply, not sure.

did not really mean a hard coded string. all i want is a way of being notified by your ng2-uploader that the file the user tried to upload does not match any in the configured options

okay, sorry, got it now... can you maybe make a PR for this, I will accept it? thanks.

k

(onUploadRejected) not work


ModuleNotFoundError: No module named 'fetch_cord.debugger'

Debug info
OS: Zorin OS 15.3 x86_64
Host: 350V5C/350V5X/350V4C/350V4X/351V5C/351V5X/351V4C/351V4X/3540VC/3540VX/3440VC/3440VX P03AAL.015.CP
Kernel: 5.4.0-45-generic
Uptime: 13 hours, 58 mins
Packages: 2665
Shell: bash 4.4.20
Resolution: 1366x768
DE: GNOME 3.30.2
WM: GNOME Shell
WM Theme: ZorinBlue-Dark
Theme: ZorinBlue-Dark [GTK2/3]
Icons: ZorinBlue-Dark [GTK2/3]
Terminal: gnome-terminal
CPU: Intel i5-3210M (4) @ 3.100GHz
GPU: Intel HD Graphics 4000
Memory: 2551MiB / 3829MiB

Operating system & way of installation
OS: Zorin OS 15.3 x86_64. Fetchcord was installed via pip
Error/issue
Fetchcord won't start because of this error:
Traceback (most recent call last):
  File "/home/alex/.local/bin/fetchcord", line 7, in <module>
    from fetch_cord.run_rpc import main
  File "/home/alex/.local/lib/python3.6/site-packages/fetch_cord/run_rpc.py", line 9, in <module>
    from fetch_cord.testing import gpuid, appid, cpuappid
  File "/home/alex/.local/lib/python3.6/site-packages/fetch_cord/testing.py", line 6, in <module>
    from fetch_cord.debugger import test_debug
ModuleNotFoundError: No module named 'fetch_cord.debugger'

Hello, this error should not exist anymore, I'm thinking of closing this in a week or two if i don't have a response.

try fetchcord 2.7, reopen the issue if the issue still happens


🔧 Update template

This pull request merges the current state of the template used here into this project so that it is up to date.

:tada: This PR is included in version 1.0.3-develop.2 :tada:
The release is available on:

v1.0.3-develop.2
GitHub release
Docker container (sebbo2002/vestaboard2mqtt:1.0.3-develop.2)
v1.0.3-develop.2

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

v2.0.0
GitHub release
Docker container (sebbo2002/vestaboard2mqtt:2.0.0)
v2.0.0

Your semantic-release bot :package::rocket:


Html element title issue

Right now i can see only 'Test' and 'oops' titles on mouseover elements. How can i fix this?
BTW it's a common issue cause i can clearly see it on https://rebilly.github.io/RebillyAPI/ also

Fixed by https://github.com/Rebilly/ReDoc/commit/6b1e8e75b030f5a19b5f8bb90438baac17f696c8
Will go into next release


Thread safety

Hi. I'm considering to use this library in our product. I have read some part of the implementation and I noticed that this library is not thread-safe for now.

First, the implementation of the LoganSquare.TYPE_CONVERTERS field is like the following;
private static final Map<Class, TypeConverter> TYPE_CONVERTERS = new HashMap<Class, TypeConverter>();

Should be;
private static final Map<Class, TypeConverter> TYPE_CONVERTERS = new ConcurrentHashMap<Class, TypeConverter>();

Second, the implementation of the LoganSquare.mapperFor() method is like the following;
public static <E> JsonMapper<E> mapperFor(Class<E> cls) throws NoSuchMapperException {
    JsonMapper<E> mapper = OBJECT_MAPPERS.get(cls);

    if (mapper == null) {
        try {
            Class<?> mapperClass = Class.forName(cls.getName() + Constants.MAPPER_CLASS_SUFFIX);
            mapper = (JsonMapper<E>)mapperClass.newInstance();
            OBJECT_MAPPERS.put(cls, mapper);
        } catch (Exception e) {
            throw new NoSuchMapperException(cls, e);
        }
    }

    return mapper;
}

Should be;
public static <E> JsonMapper<E> mapperFor(Class<E> cls) throws NoSuchMapperException {
    synchronized (OBJECT_MAPPERS) {
        JsonMapper<E> mapper = OBJECT_MAPPERS.get(cls);

        if (mapper == null) {
            try {
                Class<?> mapperClass = Class.forName(cls.getName() + Constants.MAPPER_CLASS_SUFFIX);
                mapper = (JsonMapper<E>)mapperClass.newInstance();
                OBJECT_MAPPERS.put(cls, mapper);
            } catch (Exception e) {
                throw new NoSuchMapperException(cls, e);
            }
        }

        return mapper;
    }
}

Is that all to make this library thread-safe?
Thanks,
Haruki Hasegawa

From what I can tell, there's really no reason to make LoganSquare.TYPE_CONVERTER a ConcurrentHashMap, as all put operations will take place during app initialization. By the time multiple threads are accessing it, there should never be any writes. Accessing a standard HashMap from multiple threads is perfectly safe, as long as one of those threads isn't also writing to it. Am I missing something?
Regarding the LoganSquare.mapperFor() operation, you're totally right. That should be synchronized. I'll do that for the next version.
The rest of the library should be thread safe. We've used it in lots of apps here at BlueLine Labs already and haven't run into a single issue yet.
Thanks!

Thank you for your quick response!

From what I can tell, there's really no reason to make LoganSquare.TYPE_CONVERTER a ConcurrentHashMap, as all put operations will take place during app initialization. By the time multiple threads are accessing it, there should never be any writes. Accessing a standard HashMap from multiple threads is perfectly safe, as long as one of those threads isn't also writing to it. Am I missing something?

Ok, you are right. No need to use ConcurrentHashMap for LoganSquare.TYPE_CONVERTER.

This is an optional suggestion.
Currently, CalendarTypeConverter and CalendarTypeConverter are using DateFormat with synchronization. I think it can be implemented more better ways.

Use ThreadLocal + SimpleDateFormat (no need to use another libraries)
Use Joda-Time's (joda-time-android) DateTimeFormatter class
Use Apache Commons' FastDateFormat class

The second and the third options depend on another (large footprint...) libraries, so it is not proper to use as the default implementation. However, I think it is a good idea that providing them as separated JARs.

Move of these changes (as well as a few others to ensure thread safety) are now available on JCenter as version 1.1.0.

I've got an issue with wrong date parsing results which appears randomly using v1.3.4
Examples :
DefaultDateFormatter: 2016-01-27T16:30:00.000Z=>Wed Jan 27 17:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-28T17:30:00.000Z=>Thu Jan 28 18:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-08T14:35:39.000Z=>Fri Jan 08 15:35:39 GMT+01:00 2016
DefaultDateFormatter: 2016-02-27T14:30:00.000Z=>Sat Feb 27 15:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-02-03T16:00:00.000Z=>Wed Feb 03 17:00:00 GMT+01:00 2016
DefaultDateFormatter: 2016-02-27T16:00:00.000Z=>Sat Feb 27 17:00:00 GMT+01:00 2016
DefaultDateFormatter: 2016-02-03T17:00:00.000Z=>Fri Jan 01 01:00:00 GMT+01:00 2016 <=BAD!
DefaultDateFormatter: 2016-01-27T09:42:47.000Z=>Wed Jan 27 10:42:47 GMT+01:00 2016
DefaultDateFormatter: 2016-01-08T14:35:39.000Z=>Fri Jan 08 15:35:39 GMT+01:00 2016
DefaultDateFormatter: 2016-02-03T17:00:00.000Z=>Wed Feb 03 18:00:00 GMT+01:00 2016
DefaultDateFormatter: 2016-04-20T08:00:00.000Z=>Wed Feb 03 09:00:00 GMT+01:00 2016 <=BAD!
DefaultDateFormatter: 2016-02-03T18:00:00.000Z=>Wed Feb 03 19:00:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-27T14:11:08.000Z=>Wed Apr 27 16:11:08 GMT+02:00 2016
DefaultDateFormatter: 2016-04-20T10:00:00.000Z=>Wed Apr 27 16:11:08 GMT+02:00 2016 <=BAD!
DefaultDateFormatter: 2016-01-12T14:34:20.000Z=>Wed Jan 27 01:34:20 GMT+01:00 2016 <=BAD!
DefaultDateFormatter: 2016-01-27T17:30:00.000Z=>Wed Jan 27 18:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-28T18:30:00.000Z=>Thu Jan 28 19:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-02-27T13:30:00.000Z=>Sat Feb 27 14:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-25T17:49:36.000Z=>Mon Jan 25 18:49:36 GMT+01:00 2016
DefaultDateFormatter: 2016-02-27T15:00:00.000Z=>Sat Feb 27 16:00:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-25T17:30:00.000Z=>Mon Jan 25 18:30:00 GMT+01:00 2016
DefaultDateFormatter: 2016-01-08T14:35:39.000Z=>Fri Jan 08 15:35:39 GMT+01:00 2016
DefaultDateFormatter: 2016-01-25T18:30:00.000Z=>Mon Jan 25 19:30:00 GMT+01:00 2016


Update composer.json

Allow guzzle 7

Thanks for the PR!

v2.0.2 has been tagged and should be available shortly.


Add possibility of changing the input template on T5Ranker

Hi guys,
Thanks for your great work!
I'll soon release some MonoT5 rerankers for Portuguese, and I'm heavily considering recommending the rerankers package as the default way of using it, instead of using more complex libraries or writing code from scratch. However, because I used a different template for formatting inputs, I cannot directly use the rerankers package.
I already created a PR (https://github.com/AnswerDotAI/rerankers/pull/16) introducing the required changes. While doing so, I noted that the _get_output_tokens function does not set the token_false to the default value for unknown models, so I included this fix as well.
Can you guys please take a look at my PR?
Thanks!

Hey, I've just merged the PR, thank you 😊  I'll release it on PyPi later today along with some other tweaks

Thanks, @bclavie !


Other attempt to call Facter to find osfamily

in provider/elasticsearch_plugin/plugin.rb
this is to fix the issue @benlangfeld reported here:
https://github.com/elastic/puppet-elasticsearch/commit/cc1e6b961047ba899f153abe30bdef8d8b2d2f6d#commitcomment-16365111
using his suggested fix, and at least does also work for me on OpenBSD.

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

Thanks @buzzdeee

Thanks @buzzdeee, this looks totally safe and technically correct. Did a close/re-open dance to get it into a Travis PR build test, hope it's cool I'm using this as a guinea pig for some CI refactors. :penguin:

Hey @buzzdeee, I think I've worked out the kinks in CI and am ready for this. Can you update this branch onto the latest master and when it passes, we'll be good to go :+1:

long easter weekend (:
rebased as requested,
cheers,
Sebastian


dosen't work correctly if there's a reference in the definition.

Hi,
thanks a lot for  this plugin, it solves an aesthetic problem that bugged me when using definitions with remnote.
However, it doesnt' seem to work properly if the definition includes a reference.

a capture of the plugin working properly :

now i added a reference to another rem in the definition :

it doesn't show the definition as intended, also the definition is not complete, starting from the referenced rem :

any solution? thanks again.

A similar problem occurs if there is no card but only a reference. Then everything after the reference is omitte. only showing the reference.
With the plugin active:

Without the plugin:


⚠ ClamAV has degraded performance

In 4f126b7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 71 ms

Resolved: ClamAV performance has improved in abebe62.


flake: curl to nodeport fails with exit code 7

flake instances

https://github.com/cilium/cilium-cli/runs/2861768000?check_suite_focus=true
https://github.com/cilium/cilium-cli/pull/319/checks?check_run_id=2861173337

symptoms
curl to nodeport (can be either a local or remote nodeport) fails with exit code 7 (Connect failed):
  ❌ command "curl -w %***local_ip***:%***local_port*** -> %***remote_ip***:%***remote_port*** = %***response_code*** --silent --fail --show-error --connect-timeout 5 --output /dev/null http://<IP_ADDRESS>:32193" failed: command terminated with exit code 7

and the subsequent flow validation fails.

Hit in #339: https://github.com/cilium/cilium-cli/actions/runs/950721302


Event Request: OnBeforeExchangeJobPlanningLineAmounts in codeunit 1006 "Copy Job"

Please could we have the following event added:
codeunit 1006 "Copy Job"
[IntegrationEvent(false, false)]
local procedure OnBeforeExchangeJobPlanningLineAmounts(var JobPlanningLine: Record "Job Planning Line"; CurrencyCode: Code[10];var IsHandled: Boolean)
begin
end;

local procedure ExchangeJobPlanningLineAmounts(var JobPlanningLine: Record "Job Planning Line"; CurrencyCode: Code[10])
var
    Job: Record Job;
    CurrExchRate: Record "Currency Exchange Rate";
    Currency: Record Currency;
    IsHandled: Boolean
begin
    IsHandled := false;
    OnBeforeExchangeJobPlanningLineAmounts(JobPlanningLine, CurrencyCode, IsHandled);
    if IsHandled then
        exit;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 53520.


Progress tracking facilities

Naive and maybe sensible implementation would be to add something similar to the task_seg_place map that's already in schedule (sched.agenda.indexed_by_start).
Question then becomes which level to record progress at:

task
or task instance
or task segment

Implementation in PR #29 records progress at task instance level


Bug - Twitter username format

Description
While logged into the Talent Cloud site and viewing my Profile, my Twitter handle is actually my email address. When attempting to update my Twitter handle from my email address to my actual Twitter handle (@nicolsonaitken) and then save, I am presented with an error that reads: The twitter username format is invalid.
When attempting to update my Twitter handle from my email address to my actual Twitter handle (nicolsonaitken) and then save, I do not receive an error but once I save, the Twitter handle reverts back to my email address. In addition, there is an addition error stating: The twitter username may not be greater than 15 characters which is correct but, I didn't enter the email address in, it was accidentally pre-populated.
Technical Info
Device: Laptop
Operating System: MacOs
Browser: Chrome, Version 79.0.3945.79 (Official Build) (64-bit)
Steps to Reproduce
Steps to reproduce the behaviour. Example:

Go to Talent Cloud
Click on About Me
Scroll down to Account Information
Fill in additional details and Save Changes

Portions of this interaction are included within the video attached.
Video
Profile.mp4.zip

[ ] Regression test written.

Although the functionality now works, I think there should either be an example of the format (eg. nicolsonaitken) or the @ should work as well. (@nicolsonaitken) as I think a lot of people would assume that the correct format would include an @.


Confluent support new version 2021-03-01-preview

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[x] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

Hello, could you please provide the link to the Azure Rest APi Specs' PR?

Hello, could you please provide the link to the Azure Rest APi Specs' PR?
Sorry forgot to put the link -
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/confluent/resource-manager/Microsoft.Confluent/preview/2021-03-01-preview/confluent.json


Support SOURCE_DATE_EPOCH in experimental/rpm.bzl; enable file clamping in make_rpm.py

NOTE: this is a review built on top of the changes in #319.  This will be rebased on #319 when necessary.  The part up for review in this PR is 95af33c.

This change allows experimental/rpm.bzl to support setting SOURCE_DATE_EPOCH,
sourced either directly inline in BUILD files or from a rule, as in the existing pkg_rpm rule.
The Starlark code in experimental/rpm.bzl is largely identical to the existing code in rpm.bzl.
Additionally, it was discovered that SOURCE_DATE_EPOCH only affects the RPM's
build time (not the files) unless you explicitly tell it otherwise.  This seems
to be the %clamp_mtime_to_source_date_epoch macro, first available in rpm 4.13+.
It is now set automatically in RPMs when SOURCE_DATE_EPOCH is set in both
pkg_rpm rules.
Tests are also provided for experimental/rpm.bzl only.  Tests for the
existing/legacy pkg_rpm rule can be added if so desired.
Additionally:

Command debugging in make_rpm.py was expanded to include printing out the
environment.

The test helper routines were made even more generic -- now users can specify
the exact CSV manifest formats they want.  This is used to check for mtimes for
this particular use case which are otherwise not checked.

Documentation fixes where noted.  Some of the TODOs will be tagged with
tickets when opportune (soon).

The CI failures are caused by the version of rpmbuild in CentOS 7 being too old to support SOURCE_DATE_PREFIX.  I will take a look into this tomorrow.


Add article on the "compression" of access lists by execution clients

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing draft PRs.
The build passes.
Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

I think this section is written backwards: first, we need to describe the new block format (add a new global_access_list table).
Then we add a retionale for it: we compressed down the gas payment for these fields, but we also need to compress down their data usage

I see the compression part, but not the minimum access list entry cost. Am I missing it somewhere?

I was unsure about that part and extracted it to a separate PR here: https://github.com/eth-infinitism/EIPs/pull/6


Some helpers for imgui (get correct version from other branch, remove mention of deleting font buffer)

This pr is incorrect, i have already changed the imgui version to use the newer system

Thanks. There were some bad commits in this one as I figured it out, did you look at the overall changes of the PR? It was pretty minor in the end
https://github.com/vblanco20-1/vulkan-guide/pull/106/files

DestroyFontUploadObjects  is old and not needed
color_info already exists
draw_imgui contains some of the rendering logic so your explanation is a little hard to read top to bottom


Feature Request: Publishing the analyzer as a standalone JAR

Hi!
Would you consider publishing the analyzer/processor as a standalone JAR, e.g. on Maven Central? That way the project topology would lend itself better to be included as a third party analyser, without the Gradle Plugin infrastructure attached to it.

HI @davidburstromspotify, yes this is an excellent idea, and shouldn't be much work either.

Are there any particular wrappers that you think we could include built-in support for?

Well, the reason I'm asking is because I could use it for a tool that I'm working on, that's eventually going to be open sourced.
It could also make the linter available as an integration in IntelliJ/Eclipse/others.

As a very quick workaround, so you are unblocked, you should be able to depend on the plugin as a jar already.
Add the gradle plugins repository to the parent build.gradle.kts:
subprojects {
    repositories {
        ...
        maven {
            setUrl("https://plugins.gradle.org/m2/")
        }
    }

Depend on the plugin:
implementation("gradle.plugin.com.appmattus:markdown:0.6.0")

Then, to execute, create an instance of RuleProcessor and call process with your required configuration. Take a look at MarkdownLint for the default values that the plugin uses.

Thank you for your suggestion, I'll use that!

In this spirit, also consider uploading com.appmatus.rules into Maven Central so that we can create custom rules.


⚠ ClamAV has degraded performance

In 0c52469, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in dc13eda after 5 minutes.


Fix conversation scrolling for new messages

Before: when the conversation reaches the input box and user enters a new message, it went "under the input box" cf screenshot of issue #1468
Now => It scrolls the last message into view, same behaviour as in chat v1

It is also necessary to update the scrolling when messages are being generated. With multiple messages being generated at the same time, the optimal behaviour is unclear--even further if at the same time multiple users are posting. For now, sticking to 'scroll to the bottom whenever anything happens' (I will soon push the code for that)

For the record: I also experimented with a solution using flex grid, rather than a bottom-fixed input
It caused design issues & complexities so I switched back to this


Error happened after asking for a while

Hey mate, in loveeeee with the plugin. So helpful! I think I'm hitting some limit, getting this error. Any thoughts?
// API ERROR: 14 is greater than the maximum of 1 - 'top_p'

maybe you changed top_p parameter ? plase make screenshot with opened settings panel

Here's my packer:
-- ChatGPT
-- Packer
use({
"jackMort/ChatGPT.nvim",
config = function()
require("chatgpt").setup({
-- optional configuration
})
end,
requires = {
"MunifTanjim/nui.nvim",
"nvim-lua/plenary.nvim",
"nvim-telescope/telescope.nvim"
}
})

Open settings panel using keybinding  and check if top_p param is equal or less 1 if not you can edit it there


server fails in Stuck when using pipeline parallel in multi-nodes

System Info
2 * 4 L40s load llama2-70B, 1 model:  tensorrt_llm.
using image: nvcr.io/nvidia/tritonserver:23.11-trtllm-python-py3
Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
1.python build.py --model_dir xxx --dtype float16 --remove_input_padding --enable_context_fmha --multi_block_mode --use_gemm_plugin float16 --use_gpt_attention_plugin float16 --use_inflight_batching --output_dir ./engine.inflight.tp4pp2.70b --world_size 8 --tp_size 4 --pp_size 2
--multi_block_mode
--max_input_len 8192
--max_output_len 16384
--vocab_size=49954
2.mpirun -np 8 --allow-run-as-root --hostfile myhosts -x NCCL_DEBUG=INFO -x LD_LIBRARY_PATH -x PATH -mca pml ucx trtionserver --model-repository=xxx --disable-auto-complete-config
3.python3 inflight_batcher_llm_client.py -u xxx:8001 --text "Hello, how " –tokenizer-dir=70b -S --request-output-len 40
Expected behavior
load sucessfully & infer successfully
actual behavior
when infering using client, server was stucked(4 GPUs in one nodes is 100%, but the other node's 4 GPU is 0):
WARNING: Logging before InitGoogleLogging() is written to STDERR
I20240227 08:25:18.989490 35588 grpc_server.cc:2495] Started GRPCInferenceService at <IP_ADDRESS>:8001
I20240227 08:25:18.991181 35588 http_server.cc:4997] Started HTTPService at <IP_ADDRESS>:8000
I20240227 08:25:19.032207 35588 http_server.cc:282] Started Metrics Service at <IP_ADDRESS>:8002
dg11-train-prod001-node-10-224-96-171:13983:14088 [0] NCCL INFO Channel 00/1 : 0[0] -> 1[0] [receive] via NET/IBext/0/Shared
dg11-train-prod001-node-10-224-96-171:13983:14088 [0] NCCL INFO Channel 01/1 : 0[0] -> 1[0] [receive] via NET/IBext/1/Shared
dg11-train-prod001-node-10-224-96-171:13985:14089 [2] NCCL INFO Channel 00/1 : 0[2] -> 1[2] [receive] via NET/IBext/2/Shared
dg11-train-prod001-node-10-224-96-171:13985:14089 [2] NCCL INFO Channel 01/1 : 0[2] -> 1[2] [receive] via NET/IBext/3/Shared
dg11-train-prod001-node-10-224-96-171:13984:14090 [1] NCCL INFO Channel 00/1 : 0[1] -> 1[1] [receive] via NET/IBext/1/Shared
dg11-train-prod001-node-10-224-96-171:13984:14090 [1] NCCL INFO Channel 01/1 : 0[1] -> 1[1] [receive] via NET/IBext/0/Shared
dg11-train-prod001-node-10-224-96-171:13986:14091 [3] NCCL INFO Channel 00/1 : 0[3] -> 1[3] [receive] via NET/IBext/3/Shared
dg11-train-prod001-node-10-224-96-171:13986:14091 [3] NCCL INFO Channel 01/1 : 0[3] -> 1[3] [receive] via NET/IBext/2/Shared
dg11-train-prod001-node-10-224-96-174:35588:35724 [0] NCCL INFO Channel 00/1 : 0[0] -> 1[0] [send] via NET/IBext/0/Shared
dg11-train-prod001-node-10-224-96-174:35588:35724 [0] NCCL INFO Channel 01/1 : 0[0] -> 1[0] [send] via NET/IBext/1/Shared
dg11-train-prod001-node-10-224-96-174:35589:35725 [1] NCCL INFO Channel 00/1 : 0[1] -> 1[1] [send] via NET/IBext/1/Shared
dg11-train-prod001-node-10-224-96-174:35589:35725 [1] NCCL INFO Channel 01/1 : 0[1] -> 1[1] [send] via NET/IBext/0/Shared
dg11-train-prod001-node-10-224-96-174:35590:35726 [2] NCCL INFO Channel 00/1 : 0[2] -> 1[2] [send] via NET/IBext/2/Shared
dg11-train-prod001-node-10-224-96-174:35590:35726 [2] NCCL INFO Channel 01/1 : 0[2] -> 1[2] [send] via NET/IBext/3/Shared
dg11-train-prod001-node-10-224-96-174:35591:35727 [3] NCCL INFO Channel 00/1 : 0[3] -> 1[3] [send] via NET/IBext/3/Shared
dg11-train-prod001-node-10-224-96-174:35591:35727 [3] NCCL INFO Channel 01/1 : 0[3] -> 1[3] [send] via NET/IBext/2/Shared
additional notes
when not using pp, server is ok(only using tp=8)

Hi @hezeli123 , you said that when not using pipeline parellism this works for you. I assume you just omitted --pp_size to set it to 1 when you built the engines?
Also, when you ran mpirun, what does your hostfile look like? Just ip addresses of the two nodes?

Hi @hezeli123 , you said that when not using pipeline parellism this works for you. I assume you just omitted --pp_size or set it to 1 when you built the engines?
Also, when you ran mpirun, what does your hostfile look like? Just ip addresses of the two nodes? Do you specify any slots?
Asking because I was looking to do something similar -- ie, no pp (pp_size = 1) and tp=8. My mpirun command looked like this: mpirun -v -wd /workspace --allow-run-as-root --hostfile hostfile -x NCCL_DEBUG=INFO -x LD_LIBRARY_PATH -x PATH -mca pml ucx /opt/tritonserver/bin/tritonserver --model-repository tensorrtllm_backend/all_models/inflight_batcher_llm and the hostfile looked like this:
<IP_ADDRESS> port=2255 slots=1
<IP_ADDRESS> port=2255 slots=1

But I ran into this error: UNAVAILABLE state: Internal: unexpected error when creating modelInstanceState: [TensorRT-LLM][ERROR] Assertion failed: With communicationMode kLEADER, MPI worldSize is expected to be equal to tp*pp when participantIds are not specified (/tmp/tritonbuild/tensorrtllm/tensorrt_llm/cpp/tensorrt_llm/executor/executorImpl.cpp:356)

when using tp8，build engine through --world_size 8 --tp_size 8 , not setting pp
hostfile ：
<IP_ADDRESS> slots=4
<IP_ADDRESS> slots=4


enhance(dataframes): improve performance of groupby_agg on categoricals

Category dtypes in keys of groupby command cause size explosion of grouped dataframe since every combination of categories in keys gets their own group. Using observed solves that (and it is turned on by default).
I've also added caching of population, income groups and countries regions to memory as they never change during execution and it helps a lot with performance.

LGTM


Problem with new update and both button 1 and 2

Hi
I updated yesterday the Pibooth app and I now have a problem with buttons 1 and 2 when printing. I think the new function "Choosing the layout with the buttons 1 and 2" create a conflict with the print function. Both buttons 1 and 2 send a photo to print, whereas before only button 2 allowed printing. Button 1 reset the program and returned to the layout screen.
How can you change the problem so that each button has a specific function?
Thank you for your help!
alex

It's ok ! I unistall and update agin and it work !
thanks !

Hi Alex,
Can you publish your pibooth config?
BR
anxuae

Hi,
Some news about this issue ?
Thanks

Can not reproduce. Without logs, I will close this issue in some days.
Regards.
Anxuae


firewall issues

ever since you migrated from partnership-public-images.jfrog.io  to releases-pts-observability-fluentd.jfrog.io, I have image pull issues due to firewall.
I have been fighting with my network/security team to whitelist all redirects, but this is causing a lot of friction between network/security team and my team. they have the firewall, they have the firewall logs and i dont have access. they say nothing is blocked but it is.
I have managed to whitelist the following:
releases-pts-observability-fluentd.jfrog.io
releases.jfrog.io
endpointdns-prod-use1-lb.jfrog.io
k8s-jfrogsaa-jfrogsaa-fb6f041eda-44b5904933c69ac1.elb.us-east-1.amazonaws.com
jfrog-prod-use1-shared-virginia-main.s3.amazonaws.com
s3-1-w.amazonaws.com
s3-w.us-east-1.amazonaws.com
jfrog-prod-use1-shared-virginia-main.s3.amazonaws.com
s3-1-w.amazonaws.com

but no success... question is, why this image is not on releases-docker.jfrog.io like all the images on the official charts?

relates to https://github.com/jfrog/log-analytics-prometheus/pull/55 and https://github.com/jfrog/log-analytics-prometheus/pull/57

https://hub.docker.com/r/evearakaki/jfrog-fluent i have been puling and pushing ever since

@EvertonSA can you please give me the exact error messages for the pull issues you have. Thanks!

hi @MahithaB, I just tested this morning from the clusters i have the firewall whitelisting and it works 👀
from the cluster I have no firewall whitelisting, I get the following:
Failed to pull image "releases-pts-observability-fluentd.jfrog.io/fluentd:2.1": failed to pull and unpack image "releases-pts-observability-fluentd.jfrog.io/fluentd:2.1": failed to resolve reference "releases-pts-observability-fluentd.jfrog.io/fluentd:2.1": failed to do request: Head "https://releases-pts-observability-fluentd.jfrog.io/v2/fluentd/manifests/2.1": EOF

all these URL's below I was able to get from a tcpdump + wireshark analisys:
releases-pts-observability-fluentd.jfrog.io
releases.jfrog.io
endpointdns-prod-use1-lb.jfrog.io
k8s-jfrogsaa-jfrogsaa-fb6f041eda-44b5904933c69ac1.elb.us-east-1.amazonaws.com
jfrog-prod-use1-shared-virginia-main.s3.amazonaws.com
s3-1-w.amazonaws.com
s3-w.us-east-1.amazonaws.com
jfrog-prod-use1-shared-virginia-main.s3.amazonaws.com
s3-1-w.amazonaws.com

without these (maybe not all of those) firewall whitelisting, the pull of the image fails dramatically.
what I as a paying enterprise + customer would like to see is the move of these official supported images to the official releases-docker.jfrog.io instead of releases-pts-observability-fluentd.jfrog.io. If you need a support ticket let me know I can open that in a few minutes.


John Militer: Watch quicksupplementfact.com

John Militer requests the watch of the watch_keyword quicksupplementfact.com. See the Metasmoke search here

This is already blacklisted through a broad supplement-related regex. There is no point in watching stuff which is already blacklisted.


Fix MODAL_PROFILE bug in set-environment CLI

If you set the profile via MODAL_PROFILE, this would display default instead of the currently-passed profile.
For example:
$ MODAL_PROFILE=modal_labs modal config set-environment prod
New default environment for profile default: prod

Even though it updates the modal_labs profile

@prbot approve


linux: make NumberUtilities_strtod.cpp compile

Two issues here:

wchar_t and WCHAR have different sizes on Linux (32bit and
16bit). This can happen since wchar_t is compiler defined and WCHAR
is defined as an unsigned short. To avoid ambiguities just change
all uses of wchar_t for wchar16 in NumberUtilities*. This should
probably done everywhere, including changing WCHAR to wchar16 too,
but for now this is the minimum patch that makes this file compile
while leaving it in a consistent state.

The way the DECIMAL structure is accessed does not match its
definition in pal/. Change the code to use the macros provided, that
should work everywhere if they are correctly defined.

Hi @xanlpz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

So the macros are not part of the public API. Just change the struct to match its definition in MSDN, that should work.

I'm not sure this is the recommended solution we've been using so far for issues like this. Also I don't think there should be any need to modify the PAL code, since it was already designed to compile cross-platform.
@digitalinfinity can confirm and offer advice if necessary.

For the wchar_t/WCHAR issue I was basically following the advice given by jianchun some time ago, but I'll be happy to do something else if that's preferred. For the record he said:

Currently we are replacing wchar_t with wchar16, and L"..." with CH_WSTR("..."). Will that solve the issues? PAL_STDCPP_COMPAT is a sensitive symbol (not sure what exactly it does either), yes, changing it breaks other things.

As for the tagDEC/DECIMAL structure, basically what's in pal/ does not match this: https://msdn.microsoft.com/en-us/library/windows/desktop/ms221061(v=vs.85).aspx (the struct in pal/ does not have anonymous unions), so the code won't compile. If it was designed to be crossplatform then I guess I'm missing something here.

The change LGTM.
We also found PAL had issues. The case here is either CoreCLR uses custom macros to interact with DECIMAL or the def is not being used at all. @xanlpz 's change conforms to msdn and works better.

@xanlpz change looks good to me too


Urls with subdomain are not matching

IsURL validator is not working  with urls which contains subdomains.
e.g The url "https://s3.amazonaws.com/bucket/5113a82b-06b6-418c-fd34/eae2b303-a7b2-4707-b04a" is not matching with IsURL method.
Any ideas? Maybe include this into URL string which contain url regular expression?
Thanks in advance.

Take a look to this issue: https://github.com/asaskevich/govalidator/issues/30
I think, they're similar.

Yeah! It´s working. Thanks a lot!
2015-03-07 10:22 GMT+01:00 Alex Saskevich<EMAIL_ADDRESS>
Take a look to this issue: #30
https://github.com/asaskevich/govalidator/issues/30
I think, they're similar.
—
Reply to this email directly or view it on GitHub
https://github.com/asaskevich/govalidator/issues/48#issuecomment-77681012
.

--
Raúl.

Okey, I will review IsURL and make its behavior same to IsRequestURL


Added new "current_class" helper prop for menus

Following you lead on li_class I wanted something similar that was easy to apply tailwind classes too. Thought it would be helpful to others.

I can't see any notes, what am I missing? 😄


docs: update doc and some variable description

Description
update doc and some variable description
Types of changes in this PR
Changes that affect the core Terraform module or submodules

[ ] Bug fix
[ ] New feature
[ ] Dependency update

Changes that don't affect the core Terraform module or submodules

[ ] Examples or tests (addition or updates of examples or tests)
[x] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other

Release required?
Identify the type of release. For information about the changes in a semantic versioning release, see Release versioning.

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
update doc
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

/run pipeline

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


Skip bolt files without any points

Minor fix to conversion tool, not production code, merging on green.


remove LibTorch dependencies from lidar_centerpoint

LibTorch (PyTorch C++ API) is useful to deploy the models trained on PyTorch and used for the lidar_centerpoint implementation, but this library has mainly two disadvantages for autoware.

It needs to be built for every OS and CUDA version. (https://pytorch.org/)
It has fuge cuda libraries and they allocate more than 1GB of GPU memory, but most of them aren't used in lidar_centerpoint. (https://discuss.pytorch.org/t/pytorch-1-8-1-cu111-package-size-and-memory-usage-issue/128713)

Therefore, libtorch needs to be removed and the equivalent libraries used for lidar_centerpoint are to be reimplemented.

PR is merged.


Resolve as BigInt3 structure

Currently we use the following logic for resolving BigInt3 structures:
valMemoryValues, err := hinter.GetConsecutiveValues(vm, valAddr, int16(3))
if err != nil {
    return err
}

// [d0, d1, d2]
var valValues [3]*fp.Element

for i := 0; i < 3; i++ {
    valValue, err := valMemoryValues[i].FieldElement()
    if err != nil {
    	return err
    }
    valValues[i] = valValue
}

valValues is the argument that should be passed down to the pack functionality.
As this logic is repeated in the code several times, it would be beneficial to separate it in its own functionality, something like ResolveAsBigInt3 passing the starting address of the structure in memory

Would like to take this on if that's okay.

@cicr99 Is there a specific file where this function should be stored?

Hey @MoigeMatino sure! I just assigned you to it. This functionality should go into the hintrunner/utils folder as it's used by several hints. I would say to add it secp_utils.go file. Wdyt @har777 , since you've been working on that?

Yeah thats the right file imo.

Here's the draft PR: #354


Fix error on docker login secrets

What is the purpose of the change
Fix errors on the Docker login step (as shown in https://github.com/osmosis-labs/osmosis/actions/runs/8414076425/job/23037226970?pr=7824).

This is due to the fact that dependabot does not have access to GitHub secrets by default.
-
        name: Login to DockerHub
        uses: docker/login-action@v3
        with:
          username: ${{ secrets.DOCKERHUB_USERNAME }}   # EMPTY in depandabot
          password: ${{ secrets.DOCKERHUB_TOKEN }}      # EMPTY in depandabot

Normal run:

Solution
Avoid docker login when PR is opened by a depandabot or mergify.
Testing and Verifying

Workflow should run without issues in this PR
Workflow should run in the dependabot PR

Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[x] N/A

Summary by CodeRabbit

Chores

Improved CI workflow by skipping DockerHub login for specific GitHub bot actors.

"Skip the login," it said with a hop,
For bots that we trust, let the redundancy stop.

That's gold


🧑💻: Landing Page Enhancement

:red_circle: Title : Landing Page Enhancement
:red_circle: Enhancement Aim :   Smooth experience
:red_circle: Changes : Smooth scrolling, improved aesthetics, and enhanced user interaction.
Screenshots 📷

N/A

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Akash Choudhury
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.)
GSSoC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Hey, @UTSAVS26! I hope you're doing well! I'm having a bit of trouble finding the index.html file. Would you mind helping me out? I have some advanced Python skills and I'm ready to tackle a few issues I've spotted in the code. Should I jump in and work on this, or would it be better to create a new issue? Thanks a bunch!

Hey, @UTSAVS26! I hope you're doing well! I'm having a bit of trouble finding the index.html file. Would you mind helping me out? I have some advanced Python skills and I'm ready to tackle a few issues I've spotted in the code. Should I jump in and work on this, or would it be better to create a new issue? Thanks a bunch!

Have you joined the discord server?


don't print the api key in error messages

e.g. when a dm push fails, we shouldn't print the api key in the clear

@lukemarsden and @danthebaker
Is this still an open issue? If so, can you add an example.
On a push failure (contrived with invalid API keys) I get
priya@Priyas-MBP:~$ dm push priyahub priya
Pushing admin/priya to priyahub:priya_test/priya
Response '{"jsonrpc":"2.0","error":{"code":-32000,"message":"Unable to connect to any of the addresses attempted: [dothub.com], errs: [Permission denied. Please check that your API key is still valid.]","data":null},"id":8674665223082153551}
' yields error Unable to connect to any of the addresses attempted: [dothub.com], errs: [Permission denied. Please check that your API key is still valid.]

Try it with a valid API key, but a push that fails for some other reason? Eg, try to push to alaric/foo on the dothub, where you're not user alaric, or push to a dot that's got dirty data on the remote?

Needs an attempted push/pull/clone in order to trigger the issue (e.g. failed push).
Deserializing the TransferRequest is the culprit.
API key is printed both in logs and stdout. Stdout is slightly more important to fix.

Done by overriding the toString's of various structs where APIKey is being used, and masking the value for field 'ApiKey" with "****". This has no effect on the JSON serialiser / marshaller.
Some amount of code duplication between the structs the APIKey is defined in both the client and server makes me a bit sad.


ADBDEV-2901 Fix order of restoring metadata

CREATE TABLE mytable (
   id integer PRIMARY KEY,
   name text,
   fromdate timestamp with time zone NOT NULL
);

CREATE OR REPLACE FUNCTION myfunc(arg mytable[]) RETURNS void
   LANGUAGE plpgsql IMMUTABLE STRICT AS
$$DECLARE
   t mytable;
BEGIN
   FOREACH t IN ARRAY arg LOOP
      RAISE NOTICE 'id = %', t.id;
   END LOOP;
END;$$;

If we create a function that recieves a set of rows as parameter, then try to backup and restore it, we can get an error:
LANGUAGE plpgsql NO SQL IMMUTABLE STRICT; Error was: ERROR: type public.mytable[] does not exist (SQLSTATE 42704)

The error says that we can't create a table which recieves a set of rows of an unknown table. That error occurs because gpbackup writes DDL of functions former than that of tables.
Actually, gpbackup tries to resolve dependencies, but this function doesn't depend on the table, it depends on a composite type of a row of this table, which was created automatically when the table was created. This kind of types is not recognized by gpbackup and can't be created separately from a table, so gpbackup doesn't back it up. Also, pg_depend doesn't show that neither a table depends on this type, nor a type depends on a table it was created with, so gpbackup doesn't see any connections between the table and the function that uses this table. It can lead to a situation when a function may be in the queue to be recovered before the table it depends on.
The easiest solution is to sort all relations by oids before dependencies are resolved, because usually the bigger oid is, the later the relation was created, which means that objects with smaller oids shouldn't depends on the ones with bigger oids. However, it's now always true, that's why dependencies should be resolved after sorting by oids, which guarantees correct order of relations.

@dnskvlnk, can you explain, will all tables be reordered by oid or dependency only? Does this patch affect only metadata restoring or data restoring too?
There was out optimization in #9

This patch reorders all relations by oid and then leaves it to dependencies resolver, so all relations will be reordered.
But since metadata restoring and data restoring are two separate operations which rely on different sets of data, data recovery order should not affected by metadata recovery order.
https://github.com/arenadata/gpbackup/blob/b55652d60feee0b9756ac7a419242e4bb8191673/backup/backup.go#L127
This change doesn't affect the patch which reorders tables by their size.

in vanilla postgres this bug does not appear, because vanilla pg_dump puts tables first and functions second, and if table column depends of function, then it puts alter set after functions

We can't create a function, I think
Of course, it's a typo. It's a function who recieves an unknown type.

Did you mean, sort all objects?
Yes, my mistake. We sort all the object of a backup, bot only relations


zh-cn:sync netease en language

Signed-off-by: jacky<EMAIL_ADDRESS>zh-cn:sync netease en language

/approve
I don't care that much about vendor pitches.


Add documentation for new effects

Split effects-palettes.md into separate files.
Add effect flags and parameters.
Document effects added in 0.14.
Update per-segment individual LED control to recommend hex values instead of sub-arrays.
Note: I still have to render the visualizations for the brand new effects.

Wonderful!
Can you add the information that it is possible to upload custom palettes in 0.14 and above (requires restart to apply)?


IndexName variable misspelled in query.go

Hi,
In godynamo/endpoints/query/query.go file at line 51 , the variable Indexname in Query struct is misspelled  .
It should be IndexName .
PS : I was getting error because of this

Sorry. This has been fixed. Thanks!


Add cosign signing step

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

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

/hold

/test all

/test all

/test pre-busola-local

/unhold

/retest


Change text on F2 menu

Please advise on how to update text on the display window to change from F2 to F3?   unable to find in any of the files.

https://github.com/FrazzIe/rp-radio/blob/master/rp-radio/config.lua#L4 change it to "`INPUT_SAVE_REPLAY_CLIP" or the name of the control (I'm assuming 170 for F3) https://docs.fivem.net/docs/game-references/controls/


xplat perf improvements

osx: cache expensive threadId retrieval
pthread_threadid_np(pthread_self(), &tid); call is very expensive. Cache expensive threadId retrieval instead. Same commit also removes unused THREADGetThreadProcessId
xplat: Improve PAL wmemcmp perf.
Since we know the char count, comparing with larger blocks reduces the total cycles to spend.

retargeted to release/1.6

@jianchun Thanks for the review


Update doc for CA1711 to mention new cases

With the PR https://github.com/dotnet/roslyn-analyzers/pull/3360 the rule will now also report on enums whose name ends in Flag or Flags. The rule description needs to be updated accordingly.

dotnet/roslyn-analyzers#3360 is approved but note merged yet. @Evangelink Do you know what analyzer version this will get shipped in?

Next one but I am not sure how things will be done. Tagging @mavasani as he will have the answer (could you proceed with the merge too please?).

Looks like it was merged a while ago. I'm updating the table of Suffixes | Base classes with this entry:
Flag | Flags   System.Enum
Let me know if that seems good. Thanks!

It raises the question whether plural forms of all the entries are caught. Maybe I just need to say "Flag"?

Updating. Doc should go live around noon PST. Please see my question above about plural forms in general. Thanks!

@Mikejo5000 Only Flag and Flags for enums.
Looking at the PR, I can see that we also added a new user option: https://github.com/dotnet/roslyn-analyzers/pull/3360/files#diff-1b5cdc4937a8cdfc79545945ad6c0783R697-R704
Do you want me to create a separate ticket or are you gonna handle it too?

I will add it. Looks like: additional_use_results_methods
Configure a string of allowed suffixes (separated by '|')

Updating. Fix should go live around 5pm PST. Let me know if you see any issues.

@Mikejo5000 There seems to be a visual glitch in the table for the newly added Flag/Flags entry.

Also the configurability section is not following the same structure as for some other rules (e.g. ca1303). Shall I open a different issue?

Good catch. No, I'll make the fix now, and we can discuss here if there are more issues.

On Configurability, do you mean make the phrasing consistent? ca1303 says this instead:
"If you're running this rule from FxCop analyzers (and not with legacy analysis), this rule is configurable."
I will update using this phrasing. Let me know if I'm missing anything here.

I'll merge updates. Should be live around noon.

Sorry I meant that:
1/ The sentence you are mentioning is put only once as a kind of summary.
2/ Each option is put in a headered section:

Yes, that makes sense, of course! Thanks for the clarification.

Should be live around noon.

@Mikejo5000 Thank you! Everything looks good now :)


KAFKA-18084: Added write locks in SharePartition where locks were async calls were being made

About
Added write locks in SharePartition where locks were async calls were being made and no locks were used. This could potentially lead to race condition when updating the state of offsets/batches in SharePartition.
Testing
The code is tested with the help of already present unit and integration tests.

thanks for the approval, can you please merge the PR?

@apoorvmittal10 Do you have any feedback for this PR? Aside from the lock scope, all other changes are ready to be merged.

thanks for the approval, can you please merge the PR?

@apoorvmittal10 Do you have any feedback for this PR? Aside from the lock scope, all other changes are ready to be merged.

Mine is more on the way the locks are acquired and release. There is no point to keep holding the locks when a async API has been called. @adixitconfluent are we saying to have that change in the next PR? If yes, then it seems fine to me.

thanks for the approval, can you please merge the PR?

@apoorvmittal10 Do you have any feedback for this PR? Aside from the lock scope, all other changes are ready to be merged.

Mine is more on the way the locks are acquired and release. There is no point to keep holding the locks when a async API has been called. @adixitconfluent are we saying to have that change in the next PR? If yes, then it seems fine to me.

Hi @apoorvmittal10 , i have addressed your comments where you mentioned to decouple the locks in both maybeInitialize and post persister write state RPC calls. @chia7712 is mentioning about this comment thread https://github.com/apache/kafka/pull/17957#discussion_r1862570482 where we've not reached consensus that we want to separate out the future completion from the lock scope. Other than that, all the changes are done. I would like to merge this PR since other comments have been resolved and it would be good if we can reach consensus on the former thread, so if needed, I can make the changes before the feature freeze.

@chia7712 , can you please review the changes? Thanks!!


refactor(levm): opcodes SMOD, SDIV

abs function can be used to clean up the implementation.
We can early return 0 if the dividend is 0.

Closed in #1398


How can get the link map file while build with gym

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
How can get the link map file while build with gym?
Question Description
I want to copy the link map file to gym's output_directory
fastlane  env
<details><summary>✅ fastlane environment ✅</summary>

### Stack

| Key                         | Value                                       |
| --------------------------- | ------------------------------------------- |
| OS                          | 10.13.4                                     |
| Ruby                        | 2.5.1                                       |
| Bundler?                    | false                                       |
| Git                         | git version 2.17.0                          |
| Installation Source         | /usr/local/bin/fastlane                     |
| Host                        | Mac OS X 10.13.4 (17E199)                   |
| Ruby Lib Dir                | /usr/local/Cellar/ruby/2.5.1/lib            |
| OpenSSL Version             | OpenSSL 1.0.2o  27 Mar 2018                 |
| Is contained                | false                                       |
| Is homebrew                 | false                                       |
| Is installed via Fabric.app | false                                       |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/ |
| Xcode Version               | 9.3                                         |

### System Locale

| Variable | Value       |   |
| -------- | ----------- | - |
| LANG     | en_US.UTF-8 | ✅ |
| LC_ALL   | en_US.UTF-8 | ✅ |
| LANGUAGE |             |   |

### fastlane files:

<details><summary>`./fastlane/Fastfile`</summary>

```ruby
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:ios)

platform :ios do
	desc "Description of what the lane does"
	before_all do
        # 设置代码签名：自动
        enable_automatic_code_signing
        # 跟新构建版本号
        #increment_build_number(
        #    build_number: ENV["build_number"]
        #)
	end

    desc "build a test App "
    lane :beta do
         # 调用 gym 打测试包
         gym(
            configuration:"Debug",
            export_method:"development"
         )
    end

    desc "build a ad Hoc App  "
    lane :adhoc do

     	# 调用 gym 打发布包
     	gym(
     		output_directory: '../sf-tmp',
        	configuration:"Release",
        	#export_method:"ad-hoc"
        	export_options: {
        		method: "ad-hoc", # 这可以不指定
				thinning: "<thin-for-all-variants>" #<none>
			}
        )
    end

    desc "build a release App  "
    lane :release do

     	# 调用 gym 打发布包
     	gym(
        	configuration:"Release",
        	export_method:"app-store"
        )

     # 上传至 iTunes Connect
     #deliver(
        #注意：submit_for_review这个true是提交审核，false不提交
        # skip_binary_upload: true,
        #submit_for_review: true,
        #)
    end

	desc "for my test  "
    lane :mytest do
    	system("bash sf.sh")
    end

    after_all do |lane|
    end

    error do |lane, exception|
    end
end

`./fastlane/Appfile`
# app_identifier("[[APP_IDENTIFIER]]") # The bundle identifier of your app
# apple_id("[[APPLE_ID]]") # Your Apple email address

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.95.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.2.0

slack-notifier
2.3.2

atomos
0.1.2

CFPropertyList
3.0.0

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.5

xcodeproj
1.5.8

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.3.2

terminal-table
1.8.0

plist
3.4.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.62.0

faraday
0.15.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20180417

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.3

gh_inspector
1.1.3

json
2.1.0

mini_magick
4.5.1

multi_json
1.13.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.4.0

bundler
1.16.1

faraday_middleware
0.12.2

naturally
2.1.0

simctl
1.6.4

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.1

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
2.1.0

memoist
0.16.0

os
0.9.6

signet
0.8.1

googleauth
0.6.2

httpclient
2.8.3

google-api-client
0.13.6

emoji_regex
0.1.1

openssl
2.1.0

date
1.0.0

fileutils
1.0.2

ipaddr
1.2.0

generated on: 2018-05-14

https://stackoverflow.com/questions/4926114/xcode-settings-path-to-link-map-file-what-is-it

@zhenian The path to the link map file looks to a a build setting that outputs to the same spot (wether you build with gym or with Xcode directly. Mine was located in the derived data deep down in the Intermediates.noindex directory. gym does not return this path directly itself but you could try to find this path yourself and/or change the build setting on where the linker file gets output 😊


GUI click-tests with TestFX

needs to be done

Next I tried to implement the example code for gui testing, but it didn't work:

I think maven isn't loading the dependencies properly, because as I retried I got the following error:

But we have the same error with AssertJ:

I fixxed the pom with my last commit c3261fd9230fe42c1d4e460414ea0885b9937563 and added a test and a testclass. should work for you too now


[docs] [SPARK-6306] Readme points to dead link

The link to "Specifying the Hadoop Version" currently points to http://spark.apache.org/docs/latest/building-with-maven.html#specifying-the-hadoop-version.
The correct link is: http://spark.apache.org/docs/latest/building-spark.html#specifying-the-hadoop-version

Can one of the admins verify this patch?

Yes, that's fine. There is a long story behind what happened with this link but at this point it is correct to update it.


Add unit tests for grid_warehouse.F90 functions

add unit tests for the constructor and get_grid functions and adapt module to follow MUSICA style

Done by #81.


Login to an OAuth Tableau Servers

Hello,
I have tried to build an app that shows users their VIZs, but after trying to use this library in combination with your REST API, I could not log in to the Tableau Online Servers using a custom view. The library only works when connecting to the public Tableau server, but not to a Tableau server that requires OAuth authentication.
Are there any future ideas or plans to develop this feature or implement a proper SDK for iOS?
@ronzilla @avertin @MadCwik

Hi @Gustavo91,
The sample app supports user authentication inside the webview for the viz. If a user is not authenticated, the webview will automatically display the username password field. Once the user signs in, the webview will be redirected to the viz. No extra api calls or oauth tokens required. At this point we don't have plans to develop this SDK with more authentication support than that.
If you want to use oauth tokens, I can provide a few extra pointers. Are you able to successfully get an OAuth token from the REST API? If so, you need to enable sharing cookies for the webview with the sharedCookiesEnabled prop to make sure it gets passed to the webview context.  Here are two youtube videos explaining authentication with Mobile App Bootstrap in more detail from Tableau Conference.

2018 Tableau Conference
2019 Tableau Conference

Hope this helps!


test: SliverAppBar with shape

wrote test for https://github.com/flutter/flutter/pull/31662#issuecomment-489179743

closed for unique PR with test #31662


Punjabi_Sikh is showing up with an odd URI

http://urigen_local_uri/ae920f6d-eda5-40ab-8ec9-c5c08b7649af/Punjabi_Sikh
https://www.ebi.ac.uk/ols/ontologies/hancestro/terms?iri=http%3A%2F%2Furigen_local_uri%2Fae920f6d-eda5-40ab-8ec9-c5c08b7649af%2FPunjabi_Sikh
Cheers,
Damion

This has been fixed in the latest release of HANCESTRO. The previous term with the bad URI has been deprecated in favour of a new HANCESTRO-compliant one.


[Translation] Russian localization

This is an adapted translation into Russian

ru_ru.zip there is a json file in the archive, unfortunately I can't attach json directly

Works nicely, thanks! I'll add this in the next update.


Validate header size improvement

This small change uses the CSV module rather than a naive join(',') to create the observed vs expected headers in the error message. This will result in better messages if headers include commas.

Thanks! :+1:


0.1.x controller attempts to PATCH SNI object incorrectly

Summary
When updating a hostname to use a new certificate, controller 0.1.x attempts to PATCH an SNI object by its ID. SNI objects do not have IDs on Kong versions supported by 0.1.x
Kong Ingress controller version
0.1.3
Kong or Kong Enterprise version
0.34-1
What happened
The controller found an existing SNI for a hostname when attempting to apply a new certificate to it. This executed this StatusOk case and attempted to execute sni, res = client.SNIs().Patch(sni.ID, sni). Because SNI objects on 0.13-based releases do not have IDs (a complete object looks like {"created_at":1555950056000,"name":"example.com","ssl_certificate_id":"7bbe8a14-6205-11e9-8808-121a6edbaadc"}, the controller attempted to request PATCH /snis/, which is invalid.
On the controller, these log lines appear:
I0725 20:57:57.116118       7 kong.go:1184] updating certificate for host example.com to certificate id 50388d30-969f-11e9-8808-121a6edbaadc
E0725 20:57:57.118111       7 controller.go:131] unexpected failure updating Kong configuration:
patching a Kong certificate: the server does not allow this method on the requested resource (patch snis.meta.k8s.io)

In the admin API pod, this log line (along with a lengthy generic "lapis couldn't satisfy this request" traceback) appears:
content_by_lua(nginx-kong.conf:125):2: in function <content_by_lua(nginx-kong.conf:125):1>, client: <IP_ADDRESS>, server: kong_admin, request: "PATCH /snis HTTP/1.1", host: "localhost:8001"

Expected behvaior
The controller patches the SNI object by name, e.g. PATCH /snis/example.com. The admin API accepts this and updates the SNI to use the new certificate.
sni, res = client.SNIs().Patch(server.Hostname, sni) should accomplish this.
Steps To Reproduce
The following is unverified but should reproduce the issue:

Create an Ingress with a TLS definition.
Update the Ingress to use a different Secret for its TLS definition.
Review error logs

Temporary mitigation: because this only occurs when replacing an existing SNI, forcing the controller to create a new reaches the intended state without issues. To force this, manually delete the offending SNI object from the admin API, e.g.
curl -svX DELETE localhost:8001/snis/example.com

It is very unlikely that we will ever get to this. 0.1 was released a long time ago and the code has matured quite a bit over the last year, including a rewrite. Hence, I'm going to close this.


DateTime android 5-6-7

Hello it seems like there is an issue with the following class, have you encountered the same issues ?
java.lang.NoClassDefFoundError: kotlinx.datetime.Instant
similar to
https://github.com/Kotlin/kotlinx-datetime/issues/97
Edit1:
The fix should be
https://developer.android.com/studio/write/java8-support#library-desugaring
Will try tomorrow, and let you know, it might be good to include it in the readme if it works
@nodh
@JesusMcCloud

I've never tried it on those older Android version, but if the suggested solutions works, I'll add that to our readme.

@nodh
it works, so the fix is basically the following:
compileOpitons
coreLibraryDesugaringEnabled true
dependencies
coreLibraryDesugaring 'com.android.tools:desugar_jdk_libs:1.1.5'


chore: remove search root input

Overview
This change removes the search root experience from modeled data stream search on the dashboard resource explorer. Currently, it is not possible to achieve the desired experience with the queries we can write in TwinMaker Knowledge Graph. To enable the feature in the future, we will prioritize creating a feature request for what is needed.
Additionally, the PR includes:

Refactor of the query statement factory
Improved query statement wildcard handling
Change in validation rules from 3 to 20 characters to 1 to 18 characters (wild cards count as 2 characters)
Lifting useSearch up to the SiteWise query editor to remove the need of useEffect, which was causing infinite loops in certain situations.

Legal
This project is available under the Apache 2.0 License.

@corteggiano Thank you for reviewing!


module 'dash_uploader' has no attribute 'UploadStatus'

The callback function.
@du.callback(
    output= Output("analyze-val", "disabled"),
    id= "dash-uploader",
)
def callback_on_completion(status: du.UploadStatus):
        return False

Error:
def callback_on_completion(status: du.UploadStatus):
AttributeError: module 'dash_uploader' has no attribute 'UploadStatus'

I am new to DU and Plotly dash development in general, have I missed something? This is error is quite unexpected as there's not much that could go wrong with this basic code. Any suggestions? Thank you in advance.

Which version of dash-uploader you are using?

I was about to close the issue because I tried the pre-release 0.7.0a1 and the error is gone now. I was probably using an old version. Maybe I'll switch to 0.6.0 from pre-release for now. Thanks for a quick response!

Yeah the du.UploadStatus was introduced in 0.7.0a1, so that probably explains your error :)


update to 1.19.3?

Could you update to 1.19.3 please?

Could you update to 1.19.3 please?

This was abandoned back in 2019. But someone picked up and has been maintaining it since.
https://builtbybit.com/resources/transport-pipes.22203/
https://github.com/BlackBeltPanda/Transport-Pipes


Implement EvalQueryEngineTool

Description
Notice
I would like input on this PR from the llama-index team. If the team agrees with the need and approach, I will provide unit tests, documentation updates, and Google Colab notebooks as required.
Summary
The reason for this change is to provide a plug-and-play method of using on-the-fly evaluation for tools within an agent. Some areas I would like feedback on:

Handling this on the tool-level makes sense for the use-case
This is a good approach for addressing the use-case, and aligns with the llama-index vision
Anything else you can think of! <3

If the team decides this is a good approach, I will provide unit tests, documentation updates, and Google Colab notebooks as required.
Implements the EvalQueryEngineTool:

Inherits from QueryEngineTool
Uses a given evaluator to evaluate query engine responses
If the response fails evaluation:

The tool output is manipulated to deter the LLM from using it
The evaluator feedback is used as a reason for failure

Type of Change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@d-mariano Yea I think I like this approach. If this was combined with an agent that had the chance to "retry" tool calls, it would be pretty powerful (and also helps avoid tracebacks when calling a tool)

@d-mariano Yea I think I like this approach. If this was combined with an agent that had the chance to "retry" tool calls, it would be pretty powerful (and also helps avoid tracebacks when calling a tool)

@logan-markewich great! And totally agree, that's the idea down the road. I would like to test this with the simple ReAct implementation. Then, after that, test it with more complex use-cases like within a query pipeline.
Also curious what implications this has on a QueryPlan, but that doesn't need to be addressed today.
Okay, great. So with this, I can implement some UTs, of course...should I introduce this in the docs with a notebook example as well? <3

@d-mariano yea may as well make this PR a little more complete 💪

@logan-markewich I've made a few updates so far:

I corrected the change to update ToolOutput.content rather than ToolOutput.raw_output, the agents use content and the entire point of this change is to inform agents on evaluation failures
I've added a test suite

I would like to update the llama-index documentation with a section for EvalQueryEngine. Here I can write up a blurb on the purpose of it and even include a notebook example of it being used with an agent. Are these docs included in this repo? (I haven't looked around, I can also do that at some point.)
I need to get at some other stuff so I'll have this open a bit longer.

@d-mariano yea, there's a docs folder in this repo 💪 it's all markdown.
There should be some existing evaluation sections you can maybe hook into? Or maybe the section on tools would be better


ModuleNotFoundError: No module named 'ollama'

Here is my code:
`from crewai import Agent, Task, Crew, Process
import os
from dotenv import load_dotenv
load_dotenv()
os.environ["OPENAI_API_KEY"] = os.getenv("OPENAI_API_KEY")
import agentops
agentops.init("", tags=["crewai-agent"])
researcher = Agent(
role="Researcher",
goal="Discover new insights",
verbose=True,
backstory=(
"You are a world class researcher working on a major data science company"
),
allow_delegation=False,
)
writer = Agent(
role="Writer",
goal="Create Engaging content",
verbose=True,
backstory=(
"You are a famous technical writer, specialized on writing data related content"
),
allow_delegation=False,
)
task1 = Task(
description=("Investigate the latest AI trends"),
expected_output="Investigate the latest AI trends",
agent=researcher,
)
tast2 = Task(
description=("Write a blog post on AI advancements"),
expected_output="Write a detailed blog post",
agent=writer,
)
crew = Crew(
agents=[researcher, writer],
tasks=[task1, tast2],
verbose=True,
)
result = crew.kickoff()
print("The outputs have been compiled")
print("Result=> ", result)
agentops.end_session("Success")`
It gives me   import ollama
ModuleNotFoundError: No module named 'ollama'
I am not even using ollama.

@areibman hm

any update? Poor docs, no solution.

any update? Poor docs, no solution.

Hey @AIAnytime -- thought we replied. We're taking a look to see what's going on. Can you verify which version of AgentOps you're on?

Please update to agentops v0.2.6 which contains the patch for this issue


Fix spelling issue at doc readme

The link /react-lite-ui/install has npm install react-lite. It should have been npm install react-lite-ui

PR: https://github.com/Codebrahma/react-lite-ui/pull/62


Added rust-like unwrap() for Maybe and Either

Why?
Better escape hatch than .value
Usage
function getUser(id: string): Either<UserError, User>;
const value = getUser(11).unwrap().name;

What else

Fixed compiler and linter errors

Hi @AlexXanderGrib
Thank you so much for this amazing PR. There are a few discussing points, but, anyway, it's so useful.
Waiting for you response, and one more time, thank you a lot ^_^


kam should use Github push API instead of git push command

Is your feature request related to a problem? Please describe.
kam can make use of the github push api to push the files/folders it creates when --push-to-git=true flag is enabled. Right now, User has to add his public key to github account.
https://github.com/redhat-developer/kam/blob/104bc4958d7929c492b2d85272386b63ed6530c3/pkg/pipelines/utils.go#L85
Describe the solution you'd like
Use the Github Push API.
https://docs.github.com/en/free-pro-team@latest/graphql/reference/objects#push
Describe alternatives you've considered
Its not a blocker as user can still get the things done by adding his public key to github account.
Additional context
Issue:
https://github.com/redhat-developer/kam/blob/104bc4958d7929c492b2d85272386b63ed6530c3/pkg/pipelines/utils.go#L85
Solution towards the Issue:
https://docs.github.com/en/free-pro-team@latest/graphql/reference/objects#push

The biggest problem with this, is that we'd need to implement more functionality from git into the kam binary (probably using go-git).
We'd need the equivalent of git push and the various options that uses, and probably functionality around getting to the git hosting service (proxy configuration).

The biggest problem with this, is that we'd need to implement more functionality from git into the kam binary (probably using go-git).
We'd need the equivalent of git push and the various options that uses, and probably functionality around getting to the git hosting service (proxy configuration).

The motive behind raising this issue is to understand , if we can make kam more API friendly.  For example,  instead of using git push can we use the equivalent API from git Providers(Github, BitBucket and more).
Something like this:
curl -i -X PUT -H ‘Authorization: token xxxx’ -d
‘{“message”: “push gitops files”,
“content”:”application/json”
}’ https://api.github.com/repos/gitops/test-appliation

However, the obvious downside is to deal with the different APIs from git providers.

The motive behind raising this issue is to understand , if we can make kam more API friendly.  For example,  instead of using git push can we use the equivalent API from git Providers(Github, BitBucket and more).
Something like this:
curl -i -X PUT -H ‘Authorization: token xxxx’ -d
‘{“message”: “push gitops files”,
“content”:”application/json”
}’ https://api.github.com/repos/gitops/test-appliation

However, the obvious downside is to deal with the different APIs from git providers.

The biggest problem who', is that users would need to replace their use of git with kam for most git operations?
We'd need to implement functionality for at least the following git actions.

git push
git pull
git diff with an upstream branch
git fetch

I'm not sure there's any benefit in using the API, over using something like go-git, but authenticating with a token.

The biggest problem who', is that users would need to replace their use of git with kam for most git operations?
We'd need to implement functionality for at least the following git actions.

git push
git pull
git diff with an upstream branch
git fetch

I'm not sure there's any benefit in using the API, over using something like go-git, but authenticating with a token.

The biggest problem tho', is that users would need to replace their use of git with kam for most git operations?
We'd need to implement functionality for at least the following git actions.

git push
git pull
git diff with an upstream branch
git fetch

I'm not sure there's any benefit in using the API, over using something like go-git, but authenticating with a token.
But, replacing git with kam doesn't really feel like something we should be doing?

@bigkevmcd I was only talking about replacing git push , I agree that we should not replace git with kam for any other use cases.

The biggest problem tho', is that users would need to replace their use of git with kam for most git operations?
We'd need to implement functionality for at least the following git actions.

git push
git pull
git diff with an upstream branch
git fetch

I'm not sure there's any benefit in using the API, over using something like go-git, but authenticating with a token.
But, replacing git with kam doesn't really feel like something we should be doing?

@bigkevmcd I was only talking about replacing git push , I agree that we should not replace git with kam for any other use cases.

But if we only replace git push, what about the other use-cases?
Can you maybe explain the benefits of only replacing git push?

But if we only replace git push, what about the other use-cases?
Can you maybe explain the benefits of only replacing git push?

But if we only replace git push, what about the other use-cases?
Can you maybe explain the benefits of only replacing git push?

I haven't explored the code completely kevin, I will take some time on this today and comment.

But if we only replace git push, what about the other use-cases?
Can you maybe explain the benefits of only replacing git push?

I haven't explored the code completely kevin, I will take some time on this today and comment.

@iam-veeramalla can you maybe update with the results of your explorations?


Add a possibility to set conceal to be enabled by default

Is your feature request related to a problem? Please describe.
Would be nice to have the conceal auto set with the config.
Right now, if I enabled the config, when I close the project and reopen it, I have to call TailwindConcealEnable each time
Describe the solution you'd like
Add an option in the config like enabled = true for the conceal exactly like the document_color.

I try to do something in a PR.
https://github.com/luckasRanarison/tailwind-tools.nvim/pull/11

@luckasRanarison Hi, I just test you merge and it doesn't works.
Here I find why: https://github.com/luckasRanarison/tailwind-tools.nvim/pull/11/commits/540684ed3ab25b1b610f723e69bf8211ea0b1922#r1536926885


Dev adding gh actions

added github-actions for MkDocs deploy

merge 'dev-adding-gh-actions' --> 'dev' @wes-o


this.props.navigation.navigate in componentDidMount and componentWillMount throws setState errors?

Current Behavior
I have a login screen. Prior to the LoginScreen mounting, I'm checking if the user is logged in (user is passed via props) and then I'm checking local storage to see if user has already done onboarding, and then I'm navigating to the appropriate screen (or staying on the login screen if they is no logged in user).
This setup is causing "can not set state in mounting/unmounted component" errors. If I remove the this.props.navigation.navigate call the errors go away... so it must have something to do with navigation. I feel like this setup worked in an earlier version of react-navigation but now I'm on beta9
Has anyone else seen this? Is there a better way to re-route people programmatically before the hit a given screen (in my case, the login screen)?
class LoginScreen extends React.Component {

	state = { stillLoading: true }

	 async componentDidMount(){
		if (this.props.screenProps.data && this.props.screenProps.data.user) {
			let onboardingComplete = await AsyncStorage.getItem('onboardingComplete');
			if (onboardingComplete === 'true') {
				this.setState({ stillLoading: false  });
				return this.props.navigation.navigate('main');
			} else {
				this.setState({ stillLoading: false  });
				return this.props.navigation.navigate('welcome')
			}
		}

		this.setState({ stillLoading: false  });
	}
	render(){
		if (this.state.stillLoading) {
			return <LoadingScreen loadingMessage={''} />;
		}
		return (
			<View style={{flex: 1, justifyContent: 'center', alignItems: 'center'}}>
				<LoginForm {...this.props} />
			</View>
		);
	}
}

Your Environment

{
  "private": true,
  "main": "main.js",
  "dependencies": {
    "antd-mobile": "^1.1.1",
    "apollo-client": "^1.1.1",
    "expo": "16.0.0",
    "graphql-tag": "^2.0.0",
    "lodash": "^4.17.4",
    "meteor-apollo-accounts": "^2.0.1",
    "react": "16.0.0-alpha.6",
    "react-apollo": "^1.1.2",
    "react-native": "https://github.com/expo/react-native/archive/sdk-16.0.0.tar.gz",
    "react-native-elements": "^0.11.2",
    "react-navigation": "git+https://github.com/react-community/react-navigation.git",
    "react-redux": "^5.0.4",
    "redux": "^3.6.0",
    "redux-devtools-extension": "^2.13.0",
    "redux-form": "^6.6.3",
    "redux-persist": "^4.6.0",
    "redux-thunk": "^2.2.0"
  },
  "devDependencies": {
    "babel-plugin-import": "^1.1.1"
  }
}

setState has not completed and you have the navigate action completed . Try adding a setTimeout(() => this.props.navigation.naviate) without any duration. This will cause it to be invoked at the end of the que/.

Great trick thanks @jeffreymendez1993

Alternatively setState() accepts an optional callback that is triggered after setState has been invoked..
this.setState({ stillLoading: false  }, () => this.props.navigation.navigate('main') );
Source


Feature/responsive orc 387

Apply changes for mobile responsive based on designs
https://vizzuality.atlassian.net/browse/ORC-387?atlOrigin=eyJpIjoiMDhjYjI3YjkwMTZjNGRkYzk5ODQwZjdmZjdlYzI2YjgiLCJwIjoiaiJ9

I added the ORC-620 task here too

Visual
 Comment

HomepageOn mobile, the user cannot see that there is multiple testimonies. Do you think we could show a bit of the next one?

PracticesOn mobile and on the details view, the values have very little horizontal space. Do you think we could reduce the space given to the labels?

Practices/NetworkOn mobile, once the user has opened a filter and the keyboard has appeared, the size of the panel never recovers its previous height. Do you think there's something we could do?

DatasetsThe keyword search is counted as one of the filters displayed in the “Filters” button. Could you ignore it?

@Bluesmile82 I'm on Android, Firefox.


Nested Pages (unplugin-vue-router) nest the layout pages

Is this by design?
My layout is:

But when I look at the generated code, the default layout is outside of the yellow layout (I know they're nested routes but this doesn't seem right). The layouts look like:

<template>

Default Layout

    <RouterView></RouterView>

</template>

<template>

Yellow Layout

    <RouterView></RouterView>

</template>

The result is this:

But that doesn't seem right.
Repro:
demo.zip

@shawnwildermuth Yes, it's a design problem.
An easy way to solve this problem is to create pages/customer.vue and set the layout to false
<template>
    <RouterView />
</template>

<route lang="json">
{
    "meta": {
        "layout": false
    }
}
</route>

issue → #134

@JohnCampionJr I think there should be an option to configure whether or not nested routes inherit the default layout.