Typo for result object in rule "Namespaces should have quotas assigned"

Brief Background:
There is a hardeneks rule currently present named "Namespaces should have quotas assigned". This can be seen in the code snippet here in the class ensure_namespace_quotas_exist.
Problem:
There is a typo on Line 21 in the result object where Namespace is spelled wrong.
Solution:
Fix typo

Fixed with #39


feature request: Status icons (git, lsp, file modifications)

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
Is it possible to add column for statuc icons? For example [+] for modified files, some other icons for git and maybe errors or diagnostics from lsp
Provide background
I really like the plugin but without these icons it can't replace my file tree plugin i've been using (NeoTree). Quickly looking for changes or errors is a lot harder
What is the significance of this feature?
strongly desired
Additional details
No response

There is an existing third-party plugin for git status https://github.com/refractalize/oil-git-status.nvim

Wow. Thank you very much. It is exactly what i was looking for.


Use I18n for error messages

There are anothers messages that should be placed in internationalization file:
https://github.com/lynndylanhurley/devise_token_auth/blob/master/app/controllers/devise_token_auth/passwords_controller.rb
https://github.com/lynndylanhurley/devise_token_auth/blob/master/app/controllers/devise_token_auth/registrations_controller.rb
https://github.com/lynndylanhurley/devise_token_auth/blob/master/app/controllers/devise_token_auth/sessions_controller.rb
https://github.com/lynndylanhurley/devise_token_auth/blob/master/app/controllers/devise_token_auth/token_validations_controller.rb

Indeed, thanks for pointing that out. I'll do it.

Thanks @lukaselmer! Let me know when those other files are updated and I'll merge ASAP!

Will the messages in the files pointed by @felipegs be transfered to the internationalization file? If not, is there another way to translate them based in the user's language?

+1, I need i18n, it's great if this gem is able to support this function.

+1 this would be very useful to have completed.

Closing in favor of #283


Archiving hard link

Fix issue #119
I have not tested the code in redox and windows.
There is a little bit code duplication in append_fs but I cannot find a good solution.  Any advice would be welcome.

I have fixed the warings in windows build.

Thanks for the PR! It looks like there's a fair bit of code movement here, could the code stay where it was primarily, outside the generic functions? Additionally could some restructuring perhaps help the amount of #[cfg] traffic?

There is quite a lot of code movement because a HashMap has to be accessed to find a hard link.  I moved those functions to methods so that the HashMap need not be passed around.  That makes the argument list shorter.  If that is not reasonable I can move them back.
As for the #[cfg], the difficulty is I cannot find an open source implementation of tar on windows supporting hardlink.  I am not experienced on windows either.  Any suggestion will be appreciated!

Hm ok I definitely want to keep the implementations here outside of the generic functions, that may require some refactoring though to avoid passing too many parameters everywhere.
For Windows I do believe there's the basic equivalent of inodes. The same-file crate may be useful here as well

@alexcrichton Thanks for the response.  I will move the implementations away, perhaps later this week.
As for windows, to my understanding using same-file may not be a good approach.  In unix we only record and find hard link when hard link count > 1, which save lots of memory and time.  And accessing HashMap is quite fast.  But to use same-file we have to record every file, and call is_same_file for every (recorded file, currentfile) pair, so it is O(n^2) for n = number of files.  Again I have no experience in windows API so I cannot think out a better approach.  Any expert here?

The requirements are:

append_fs, append_path, etc, have to be implemented outside impl Buider.
reduce number of cfg.
(my own requirement) not perform poorly when there are many files.

And in current situation I cannot find a way to meet all of them simultaneously.  To achieve [2] we can consider using same-file crate, but then [3] cannot be achieved, as mentioned above.  In the current state, trying to achieve [1] will violate [2] because I need to make version in of append_fs which takes a HashMap, and windows version does not take HashMap.
So I am closing this PR.  If anyone is interested feel free to use my commits.


Add Swift 5.7 support to legacy release/0.x branch.

This adds support for Swift 5.7, but only to the legacy version.
Motivation:
Some projects are still using the legacy version but would like to use Swift 5.7. The dependencies of this project require updating for each Swift version.
Modifications:

Updated dependencies for Swift 5.7.
Worked around an issue in swift-metrics, but only did so for Swift 5.7 to keep backwards compatibility.

Result:
Swift 5.7 is now supported.

@swift-server-bot test this please

Thanks! Hopefully won't need it much longer. A new tag/release would be great after you merge, if you don't mind.

Yup, I'll cut a release right away for y'all :)


Search component doesn't accept iconPosition prop

While most of the props are spread onto Input component, including being able to pass a custom Icon - all of these making Search component very extensible, it doesn't seem to be possible to change the positioning of icon to appear on the left side instead of the right.
Is that something that could be supported? Or is there any other way to achieve that?

Search component fully supports Input shorthand:
<Search input={{ icon: 'user', iconPosition: 'left' }} />

<Search input={{ icon: 'user', iconPosition: 'right' }} />

However, it will be awesome to get this in docs.

I very much agree with this.


CI: update macos version

macos-12 has been removed, this should fix CI for that job

Oooh great, thanks @bsipocz!


Add support for RGBA

Ok so basically i'm trying to turn this:
{
"sRed": 88,
"sGreen": 65,
"sBlue": 126,
"sAlpha": 192,
"eRed": 38,
"eGreen": 76,
"eBlue": 200,
"eAlpha": 208
}
so that it works with blur but i'm not sure how to do it please help

this is what i've gotten so far but right now it's crashing the game when i use it
# The end color of the background gradient. Given in ARGB hex. [default: 75000000]
S:gradientEndColor=C058417E
# The start color of the background gradient. Given in ARGB hex. [default: 75000000]
S:gradientStartColor=D0264CC8

ok turns out it was another mod that was the problem


Introduce IRQ state management API and disable IRQs during task switches.

This PR introduces helper functions and guards for managing IRQ state during code execution in the COCONUT kernel.
It also introduces IRQ-safe versions on SpinLock and RWLock.

Pushed a new version which addresses all review comments.

@joergroedel If you go into the "Files Changed" tab, there should be a "Add to batch" button next to suggestions. This allows you to add all suggestions to a batch and apply them with a single commit.

@joergroedel If you go into the "Files Changed" tab, there should be a "Add to batch" button next to suggestions. This allows you to add all suggestions to a batch and apply them with a single commit.

Thanks, but I need them as single commits to back-merge them into the respective commits of the series. I will of course add Co-authored tags.


Captcha

Optional (config) captcha for logins, registrations, new thread and posts.

Implemented for login and registration. New thread and posts is not finished yet.

Removed from login to prevent captcha sign in each login.


Fixed YAML Formatting of Policies

The YAML formatting for both policies was not in proper K8s format and would give errors, this was now fixed.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

hey there @animetauren ... just checking in to see if you wanted to fix up those wrappers?


DOC: Last few footbibs

closes #8923
closes #8919

nb: it turns out none of these are actually rendered anywhere (the physionet one is private, the mxne ones are technically public but aren't in python_reference.rst).  I'm inclined to merge it anyway for completeness, and in case one of these becomes public and/or its docstring gets copy/pasted into something that is public.

Agreed, thanks @drammock


Don't require '.json' on the path

The gem is currently appending ".json" to the paths.  That should default to no extension, but be configurable.

    class JsonRequest < Faraday::Middleware
      def call(environment)
        environment[:request_headers]["Accept"] = "application/vnd.api+json,application/json,*/*"
        uri = environment[:url]
        uri.path = uri.path + ".json" unless uri.path.match(/\.json$/)
        @app.call(environment)
      end
    end

fixed

Was the fix merged into the latest gem? How do you control disabling the .json extension?

I don't understand how to disable .json.
So I used this workaround:
  class JsonRequestWithoutFormat < Faraday::Middleware
    def call(environment)
      uri = environment[:url]
      uri.path = uri.path.sub(".json", '')
      @app.call(environment)
    end
  end

  connection.use JsonRequestWithoutFormat


v 0.18 flex 1.0 doesn't work if the height of parent is indirectly determined by parent's siblings

1. issue
The html dom tree is:

The key properties are:

body: flex-direction: column
grandparent: align-items: stretch
parent: flex-direction: column;
me: flex: 1.0
uncle: height: 100px

uncle has a fixed height 100px,  so that the height of grandparent is the same as uncle, and parent should be also be 100px high.
The height of me is expected be 100px, however it's not.
We used only React/Layout.c  to layout v 0.18.
React/Layout.c

Works well in chrome

2. source
<!DOCTYPE html>

                    "flex: 1.0" doesn't work on "me".

                This is uncle

+1

React Native flexbox isn't supposed to work precisely the same as flexbox in the browser. It's operated by css-layout so it's slightly different. We've improved the docs a lot since January, so I think this is more clear now. I'm going to close this issue but if you think there's still an area where the RN behavior doesn't match the docs, then I think it would be great to open a new issue. A reproduction of the problem on rnplay.org is the best way to be helpful for that - that's somewhat more useful than comparison code to run in a browser. Thanks for pointing this issue out.


HoverProvider for object

I know that ordinary suspension prompts can prompt text, and even I can prompt markdown and pictures by looking at some questions. But I see that vscode prompts the structure of an object. And this object can be expanded layer by layer. I want to know how to do this. Can our Monaco do i

t

But I see that vscode prompts the structure of an object. And this object can be expanded layer by layer. I want to know how to do this. Can our Monaco do i !

Are you asking how to show the content of a JavaScript object in a hover or how to create expandable things in a hover?

Yes, when I use "hoverprovider", I can only display some strings, not the object structure as in the screenshot, and it can be clicked. If I don't use "hoverprovider" to build a page with HTML, it will be very complicated (I know Monaco can create a drawing board to display a function similar to pop-up window)

Yes, when I use "hoverprovider", I can only display some strings, not the object structure as in the screenshot, and it can be clicked. If I don't use "hoverprovider" to build a page with HTML, it will be very complicated (I know Monaco can create a drawing board to display a function similar to pop-up window)

It is not possible to render a tree widget using the hover provider. The debugger in vscode hides the markdown hover and implements its own hover for inspecting objects.


MIMXRT106x FlexSPI driver missing

Is there an implementation of MIMXRT10xx FlexSPI driver for reading and writing external flashes. I can see the code for setting flash to XIP, but the driver itself is missing. I would like to mount a filesystem on external QSPI flash connected to FlexSPI peripheral on the MIMXRT1062-EVK board.

You may send the question to<EMAIL_ADDRESS>since many developer don't read github issue daily.

@jturnsek I'm got to FlexSPI driver from @ivanucherdzhiev and I'm clearing it to submit to mailing. Sorry I didn't see this issue before. I agree with @xiaoxiang781216 it is better to submit this question to dev@ mailing list anyway.

Hi, thanks for the reply. Where can I get hold of a driver. I have already started to implement my own, but I would not like to discover hot water.

Hi, thanks for the reply. Where can I get hold of a driver. I have already started to implement my own, but I would not like to discover hot water.

I just sent it to your email. Could you help me clean it up to submit it to the mainline?

Hi,
I am sorry that i didnt had the time to clean it up and upload it my self,
but let me know if i can help with that.
best regards,
Ivan
On Mon, Mar 29, 2021 at 2:09 PM Alan Carvalho de Assis <
@.***> wrote:

Hi, thanks for the reply. Where can I get hold of a driver. I have already
started to implement my own, but I would not like to discover hot water.
I just sent it to your email. Could you help me clean it up to submit it
to the mainline?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-nuttx/issues/3109#issuecomment-809293126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJ6CTLVMRHTLJA55LKY7C3TGBNXFANCNFSM4ZOJ5H6A
.

--
Kind regards,
Ivan Ucherdzhiev
Team Lead @ Barin Sports
Bulgaria
skype: ipy_44
tel: +359888927760


remove duplicate values in redirect settings

These don't look like duplicates to me? These seem to handle both the slash and missing slash?

These don't look like duplicates to me? These seem to handle both the slash and missing slash?

Oh. Didn't notice that. Closing


Add new dataset MMChat

Hi, I am adding a new dataset MMChat.
It seems that all tests are passed

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks ! As for https://github.com/huggingface/datasets/pull/4431 please also update the licensing section in https://huggingface.co/datasets/silver/mmchat ;)
Then if it's fine for you feel free to close this PR


Non-Uniform Representation of Coordinates/Points

Most of the code uses tuples to represent points (example: search.py). The new utils.py treats points as objects as attributes x and y and tends to use a.x,a.y for the coordinates. This can be observed in the distance function.
This was done so as to account for https://www.python.org/dev/peps/pep-3113/ which removes support for parameter unpacking in Python3.
@norvig So a decision needs to be made regarding the representation of points. I feel we can use tuples but manually unpack them at the beginning of the functions. If we do this I can send a PR right away making the required changes.

Good question. It is a shame that Python doesn't have a fixed answer. I see three main choices:
(1) tuple: (x, y) pair; manually unpack.
def Point(x, y): return (x, y)
def X(point): return point[0]
def Y(point): return point[1]
(2) Named tuple:
Point = collections.namedtuple('Point', 'x, y')
(3) Complex numbers -- good for representing points in the (float x float) plane; for example the distance function is just "def distance(A, B): return abs(A - B)". Not so good for ints, though.
I think for simplicity it is probably best to go with (1).  In most places you are right, and we will do:
def f(point):
(x, y) = point
...
But good to have the X and Y functions when needed, like for sort(points, key=Y).

I'll send a PR for making the change and adding all 3 functions Point, X, Y.


Ability to ReOrder the records from listing

I have been using this plugin to create my plugins and I found it very useful indeed.
However, I have one requirement which I would like to implement in one my plugins called as "Slider" and here below I mentioned what I want to deal with.
My Slider plugin simply contains an ability to upload images from admin (backend) and it is showing up front-end of the website.
Currently I have made a database query at front end to either show it in Ascending or Descending order but I want something like this . Client should be able to sort those slider images which he/she has been added from admin and it will show up front based on this updated list.
It is possible as of now or do I need to update plugin or need to find another way to implement this ?
Any help or suggestion will be highly helpful for me.
Thanks

Not sure how you've structured this plugin, but if you've attached the images through the attachMany relation, the sort order can be easily done by rearranging the photos in the upload form.

You need to enable Reorder controller behavior on your slider controllers when you created it.
Check this out:
https://octobercms.com/docs/backend/reorder
Also take note that your slider table should have a sort_order field.
https://octobercms.com/docs/database/traits#sortable
You need to add this link button on your _list_toolbar.htm as it is not added by default.
<a href="<?= Backend::url('bdsoft/team/team/reorder') ?>" class="btn btn-primary oc-icon-sort">Reorder Members
Replace the url to with the correct one.
I hope this helps.

Well I need to reorder by drag an drop and show updated order list something like this https://jqueryui.com/sortable. Not sure your provided links do this.

I gave the link for you to check if your controllers implements reorder behavior, and your models use sortable traits, and your slider table have sort_order field. Without those, reorder by drag and drop won't work, even if you have the button added on your _list_toolbar.htm partial.
Go ahead and check them out first, then add the  tag on your _list_toolbar.htm partial, which I mentioned on my previous reply.

Thank you. I will check and get back to you on this thread by mentioning you if I anything goes wrong or any query.
Thanks. Appreciated.

Now you can use reorder only with reorder trait using /reorder page. Maybe it is better for future implementation to add the functionality to reorder directly from record list and not from a different page.

I agree with you.


Revert "fix kubectl config view to respect serialization formats"

Reverts GoogleCloudPlatform/kubernetes#4660
This changes kubectl config view -o template in a way that breaks e2e. This call used to work and now doesn't, so validate cluster fails.

Self-merging to unbreak e2e.

The gce e2e build looks "fixed" now (only tests are failing, rather than cluster creation).


H

No description provided.

Hi @Arm0222 ,
I noticed the task with issue number #1876 on the project board, and I'm keen to assist. Could you please provide more details about the specific issue you're facing? Understanding the context and requirements will help me investigate and propose an effective solution.

Hi@Arm0222 What could this issues be about there is no description which can explain what the issue is about kindly give details thank you.


Nit: use the standard function div_ceil

Signed-off-by: remzi<EMAIL_ADDRESS>Which issue does this PR close?
None. It is just a small change.
What changes are included in this PR?
We should use the standard function div_ceil instead of implementing our own.
Are there any user-facing changes?
None

Thanks @HaoYang670


Make FixedSizedList Json serializable

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I would like for the arrow-json library to automatically implement the FixedSizedList.
Describe the solution you'd like
It seems like adding FixedSizedList should be trivial, since it would be encoded in the same way as a list, I would be happy to try it myself, I just wanted to start this first to make sure there is nothing I am missing.
Describe alternatives you've considered
The alternative is I continue to implement this manually.

I'd like to work on this issue.

label_issue.py automatically added labels {'arrow'} from #5646


adding meta data to points in plots

Hi,
I have seen in bokeh that you can attach a metadata to plots, such that when you hover over point, there is a window / bubble shown next to it. Is it possible to do the same in Bokeh.jl? It would be great feature for me.
Thanks for an answer.

Hi, @pevnak. It's not possible at the moment.
As you can see from the commit history I've pretty much stopped working on Bokeh.jl so won't be building the feature in the near future. If you would like to implement it I'd be happy to merge a PR.

@samuelcolvin I'm not too familiar with the Julia community, are there good places we could message or promote to to see if there are any folks interested in picking up the torch?

well @spencerlyon2 had taken over but I believe he's not working on Bokeh.jl right now, am I right?
The google groups mailing list would probably be the best place to ask.

Yes, sorry @samuelcolvin @bryevdv I no longer have substantial open source time to allocate here.
I agree that the julia-users google group is probably the best place to look for a maintainer

OK I just sent a message there.


[DRAFT] 0xMakka / #1183 initial implementation of language switcher

Description
DRAFT - Please don't merge as this is a WIP and incomplete.
Related Ticket
Resolves #1183
Checklist

[X] I have run npm run build-all without errors
[X] I have formatted JS and TS files with npm run format-all

@0xMakka @Atmosfearful do we agree this is complete enough that the translators can work off the PR link to finish the translation work?

It's hard to see but the bg color doesnt match the button links

nice catch, I did not see that!

It's hard to see but the bg color doesnt match the button links

Thanks for the feedback @Atmosfearful - eagle eye 😎, great catch I didn't notice this either.
And good to know with using the nextjs built ins, will make the changes to this 👍.


help ERROR:  pllua: cannot call into PostgreSQL with pending errors

i need help. this is my pl/lua code.can be created successfully.
but after i executed the function
an error occurred
cannot call into PostgreSQL with pending errors

What caused this?
How can I solve it?
If this error occurs again in the future, how should I troubleshoot it?
DROP FUNCTION IF EXISTS get_data_by_page(tablename VARCHAR);

CREATE OR REPLACE FUNCTION get_data_by_page(tablename VARCHAR)
  RETURNS TABLE (
		id uuid,
		createduser varchar,
    createdtime bigint,
		updateduser varchar,
		updatedtime BIGINT,
    data jsonb
  ) LANGUAGE pllua stable AS $$
		print(tablename)
  	local r = spi.rows('select id,createduser,createdtime,updateduser,updatedtime,data from $1',tablename)
		for k in r do
				coroutine.yield(k)
		end
$$;

SELECT * from get_data_by_page('client_info');

It looks like this is happening when a PostgreSQL error is raised within a RETURNS TABLE or RETURNS SETOF function, because of an oversight in the error handling path (the code that tries to handle the error ends up calling back into pg too soon). The real error is in your query text - you can't use $1 to replace a table name.
I need to find a way to fix the error handling so that the real error is shown, that may take me a little work.
You should find that the correct error is shown if run in a DO-block or a non-SETOF non-TABLE function.

Hm, I didn't expect that commit to close this. Regardless, I think I have fixed it, and the fixed version is up in the other repo (as linked above) if you want to test it. The fix will be released as 2.0.13 once I've done my own testing.


Weird behaviour when Gzipped files are empty

Hi,
When an empty file (ending with .gz) is created by a process, the following error occurs:
java.io.EOFException

This didn't happen in version 0.8.1 for reference.
In the github actions this also simple creates a filepath without an md5sum. I'm mostly encountering this issue when testing stub runs of certain processes.

Hi @lukfor as @ahvigil pointed out the issue here is that the .gz file isn't actually a gzipped file in the stub section.
I fixed it by changing touch ${prefix}.gz to echo | gzip -n > ${prefix}.vcf.gz

I think it would be best to check explicitly that the provided contents are actually gzipped (starts with GZIPInputStream.GZIP_MAGIC). In fact, maybe this would be more robust than checking the file extension? Though writing uncompressed data to a .gz file seems like bad form to me and probably done accidentally, so I could also see an argument for just erroring out when .gz files don't look like gzip files

Fully agree! Checking the magic byte and throwing an error is definitely the best solution.
We should also mark empty files in snapshots, to spot them quickly.


Fix server-plain unit test

it is nice when mvn test is green for all subproject out of the box.

Thanks! Applied manually via 4bc915531bc83cbfb5b78d06daac315b61a70524


mpv on wayland shows broken cursor, like #181176

Describe the bug
mpv on wayland (gnome, to be precise) shows a broken cursor just like telegram does on #181176
Steps To Reproduce
Steps to reproduce the behavior:

open mpv on gnome+wayland (might even happen on other wayland wms perhaps)
notice the mouse cursor is broken

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

In this image, the cursor appears properly, although it isn't the system cursor. But without the screenshot ui being used, only a small part of the tip of the cursor appears.
Notify maintainers
@AndersonTorres @fpletz @globin @Ma27 @tadeokondrak
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.0.3-zen2, NixOS, 22.11 (Raccoon), 22.11pre420742.f994293d1eb`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - channels(root): `"nixos"`
 - channels(m911t): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

noticed this too

Cannot reproduce this with sway on master.

I can reproduce with gnome on wayland!

Hmmmm. I used home-manager's home.pointerCursor to configure the cursor (found that snipped in your configuration files), and by doing that, mpv respects the cursor properly. For some reason though, by default this is not the case for gnome with default settings. @jtojnar (I don't know who else from the gnome team to ping)


error in loglevel

I found bug. I setup loglevel WARN in tuyagateway in settings, but in log I see DEBUG rows.
mqtthost: ‘mqtt://core-mosquitto’
mqttport: ‘1883’
loglevel: WARN

I am just passing that on to the tuyagateway, I will have to double-check the command line args and see if it is ignoring them.

I need to talk to the developer, as I believe those message are coming from gizmocaster. It doesn't appear that there is a way to control the logging level of gizmocaster from the startup command.


Don't clean up CRDs on cleanup

See https://github.com/istio/istio/issues/22480. We used to always keep
CRDs around, which I think was a good idea as we may wipe out developers
clusters if they run these tests. Deleting the CRDs seems to blow up
Galley for some reason, which should independantly be fixed, but for the
short term we can get tests running.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
de4edacb4c0a9eacff1c2e9514e7d7ffc47d0402
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
infra flake


Wrong  spelling: Bootstrap spelled as Boostrap in the Zephyr code.

Hey, I am learning from your plugin loading code and noticed a spelling mistake..

That's hilarious! Thanks for noticing that. I pushed a fix.


Fix SkeletonEditor's methods to adapt new animation and skeleton system

#53689 and #53765 make big changed the methods of Skeleton and some SkeletonEditor's functions has broken, so I'll fix that.

Thanks!


[SPARK-36852][SQL][TESTS] Test ANSI interval support by the Parquet datasource

What changes were proposed in this pull request?
In the PR, I propose to add new tests to check:

Creating of a table with ANSI intervals using parquet and In-Memory catalog.
Parquet encoding of ANSI interval columns.

Why are the changes needed?
To improve test coverage.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
By running the modified test suites:
$ build/sbt "sql/test:testOnly *ParquetEncodingSuite"
$ build/sbt "sql/test:testOnly *ParquetV1QuerySuite"
$ build/sbt "sql/test:testOnly *ParquetV2QuerySuite"

Test build #143622 has started for PR 34105 at commit 1fc8f60.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48134/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48134/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/48134/

Test build #143622 has finished for PR 34105 at commit 1fc8f60.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/143622/

Merged to master.


Missing App.xaml.cs and Mainpage.xaml.cs in SpeechRecognitionAndSynthesis sample

Which sample are you reporting a bug in?
SpeechRecognitionAndSynthesis
Describe the bug
Missing App.xaml.cs and Mainpage.xaml.cs in SpeechRecognitionAndSynthesis sample. The files are not in repository and VS complain about them missing once i load the solution from the cs folder.
To Reproduce
Steps to reproduce the behavior:

Download the solution or check the repo in GitHub
load the solution in VS 2019

Expected behavior
Solution should load all required files but there are files missing "MainPage.cs" and "App.cs"

Apparently, i should have downloaded the whole repo to get the shared files instead of downloading a specific sample

Yup. Right at the top of the README. You don't have to download the whole repo. There is also a premade zip that contains everything needed for just this sample. See first link below.

Note: This sample is part of a large collection of UWP feature samples. You can download this sample as a standalone ZIP file from docs.microsoft.com, or you can download the entire collection as a single ZIP file, but be sure to unzip everything to access shared dependencies. For more info on working with the ZIP file, the samples collection, and GitHub, see Get the UWP samples from GitHub. For more samples, see the Samples portal on the Windows Dev Center.


starter-kit: v0.6.2

Determines appropriate initialDelaySeconds value for liveness and reqdiness probes based on runtime value

Signed-off-by: Sean Sundberg<EMAIL_ADDRESS>

Fixes github.com/cloud-native-toolkit/planning#863


Add instructions for programming via terminal / command line

It would be nice if someone could add instructions on how to compile, program and upload the config via linux command line /terminal.
I assume the commands are;

Compile firmware and upload
$  pio run -e ttgo-t-beam-v1_2 -t upload

Upload filesystem (build spiffs.bin based on "data" folder and upload)
$ pio run -t uploadfs

Can anyone verify the commands are correct?
Edit: Changed text

Hi ,  I would love to !
but I dont know much "command line" commands for vscode/esptool or the ways to do it.
but I would love to learn and apply this

im adding very soon webflasher with web configuration page to avoid VSCODE and terminal flashing ;)


How i can remove two button Upload All and Remove All in package, you can update with me? <3

I have the same issue, any updates?


Add parallelism for ancestry

Just as it sounds, use multi-threading and have one project ancestry API call per thread. This also means adding locking to the cache.

No longer necessary with CAI.

However, it may be a good idea for resolving project, folder and org names...thinking we could have a queue of "ancestor" trees that all need to be resolved and run them all in parallel. The cache would need to have a lock on in though...not sure how much performance this would give us...possibly worth investigating though.


Rename ConstantPulse to Square or Constant

What is the expected enhancement?
All other pulses in parametric_pulses do not include the terminator word Pulse but ConstantPulse does.

could I try to help you with this issue?

Hello @qanto, @IceKhan13 has already submitted a PR for this #4077 and I had forgotten to assign this. If you would like to contribute please take a look at other open issues labelled with "good first issue" or feel free to ask me to provide some suggestions 😄.


Added if, channel and function examples

Codecov Report

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

@@           Coverage Diff           @@
##           master     #233   +/-   ##
=======================================
  Coverage   84.05%   84.05%
=======================================
  Files          42       42
  Lines        2935     2935
=======================================
  Hits         2467     2467
  Misses        327      327
  Partials      141      141

Continue to review full report at Codecov.

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

Great


Release testing: missing command for vagrant up

#2359
Where: https://wallaroo-docs-rc.netlify.com/book/getting-started/vagrant-setup.html
Existing sections:
Provision the Vagrant Box
What's Included in the Wallaroo Vagrant Box
Shutdown the Vagrant Box
Should be:
Provision the Vagrant Box
Log into the Vagrant Box (tell users about vagrant up)
What's Included in the Wallaroo Vagrant Box
Shutdown the Vagrant Box

@pzel The Provision the Vagrant Box section already tells users to run vagrant up. Can you clarify what you meant by this issue?

Brain hiccup. I meant: "tell users about vagrant ssh". But then, on the other hand, the very next page of the tutorial starts with telling users how to vagrant ssh into their box. So maybe let's close this one, if you agree.


Ignores Apple command keys (and Windows menu keys) because they're us…

…ed for both mouse and keyboard actions


The reason why I actually wanted to be able to customize Command Signature.

I really like SwiftCLI but there's a couple non-trivial features missing in order to make truly complex CLIs.

The ability to group parameters (and options) in groups, and subsequently specify how many members of a group we are expecting (i.e. Exactly one, At least one, At most one, all of them) or alternatively a way to specify that certain parameters or options conflict with each other.

The ability to make parameters that also require a value, like keyed options do.

For example, in my use case, before I began looking into the whole changing the Command Signature issue, I ended up using commands with signature "" and all of their functionality implemented through options and flags., since I found that was the only way I could avoid issues caused by mismatching signature parameter counts.
Although what I did to aliases would probably greatly offset this issue, it's still not really an elegant solution to use extensively.

Could you give an example of 1 and 2? I'm not sure I fully understand what you're describing. Thanks!

I wasn't sure myself that was what was missing until I began reading some documentation on generic argument parsing systems as well as for frameworks written in other languages.
In a nutshell, the second is (I realized while working on it today) a convoluted way to describe options that are not actually optional, or rather parameters that are not positional (can always be placed anywhere in the command but always tied to a value)
As for the first, I mean being able to group flags and keyed options in arbitrary groups. We can then define properties on these groups to make them (and their respective options) behave differently.
(I got an initial implementation done during the morning but haven't committed it yet)
At the moment, I'm working on implementing:

"required" (which I am basically done with), which would specify that AT LEAST one of the options belonging to that group must be present for the syntax of the command to be valid.
and
"conflicting" (still got some work to do on this one), which will specify that AT MOST one  of the options belonging to that group must be present for the syntax of the command to be valid.

I've finished an implementation of the things I was talking about here, as best as I could figure to do it. It's been included in my PR, so you can take a look there.

I appreciate your work on this! Just to understand the problem better, are you essentially trying to make something like this built into SwiftCLI?
public class MyCommand: OptionCommand {

    var silent = false
    var loud = false

    public func setupOptions(options: OptionRegistry) {
        options.add(flags: ["-s", "--silently"]) {
            self.silent = true
        }
        options.add(flags: ["-l", "--loudly"]) {
            self.loud = true
        }
    }

    public func execute(arguments: CommandArguments) throws  {
        if silent && loud {
            throw CLIError.error("Can't be silent and loud")
        }
    }

}

If so, I'm not sure this feature is used widely enough to be included in SwiftCLI. Most terminal programs don't disallow certain options together, but rather just have them override each other. For example, here is a section of the manual entry of ln:
     -f    If the target file already exists, then unlink it so that the link may occur.  (The -f option overrides
           any previous -i options.)
     -i    Cause ln to write a prompt to standard error if the target file exists.  If the response from the stan-
           dard input begins with the character `y' or `Y', then unlink the target file so that the link may occur.
           Otherwise, do not attempt the link.  (The -i option overrides any previous -f options.)

As such, I think it would be keeping with more common practices to write this code like:
    enum Volume {
        case normal
        case silent
        case loud
    }

    var volume = Volume.normal

    public func setupOptions(options: OptionRegistry) {
        options.add(flags: ["-s", "--silently"]) {
            self.volume = .silent
        }
        options.add(flags: ["-l", "--loudly"]) {
            self.volume = .loud
        }
    }

If I'm completely misunderstanding your intention, please correct me.

Yes and no.
My primary intentions were:

Being able to use position-agnostic parameters.
Being able to treat related keys or flags (or combinations of both) as distinct groups.

Indeed, the example you posted would be terrible design, and while what I did would support something like that, it was definitely my reason for doing it.
Although, mind, that example might look something like this in my fork:
public class MyCommand: OptionCommand {

    var verbosity: Int = 2

    public func setupOptions(options: OptionRegistry) {
        options.addGroup(name:"verbosity",required:true,conflicting:true)
        options.add(flags: ["-s", "--silently"], group:"verbosity") {
            self.verbosity = 1
        }
        options.add(flags: ["-informative", "--informative"], group:"verbosity") {
            self.verbosity = 5
        }
        options.add(keys: ["-v", "--verbosity"], usage: "Sets the verbosity level to <value>", valueSignature: "value", group:"verbosity") { [unowned self] (value) in
    self.verbosity = value
}
    }

    public func execute(arguments: CommandArguments) throws  {

        // Do Things. We don't need to worry about incompatible options here, the default parser will handle it and print the appropriate error message.
}

What would happen now would be that SwiftCLI would treat those options (two flags and one key) as a single required entity and thus would not even get to the execute part unless all conditions are satisfied:

One (and only one) of the three is present.
In case of "verbosity" we must supply an appropriate value.

The "conflicting" feature is mostly there to provide a safeguard against unpredictability caused by the order of the factors altering the product. The real beef of the issue is the ability to group keys and flags together and treat them as required parameters without having to worry about their position in the arguments string.

As a whole, options are almost always used as optional things, and options which change the same behavior almost always override earlier options rather than throwing an error. As such, I'm not sure it makes sense to include this specific functionality in SwiftCLI itself. It can be implemented easily enough:
enum Verbosity: Int {
        case none = -1
        case silent = 1
        case errors = 2
        case some = 3
        case most = 4
        case informative = 5
    }

    var verbosity: Verbosity = .none

    public func setupOptions(options: OptionRegistry) {
        options.add(flags: ["-s", "--silently"]) {
            self.verbosity = .silent
        }
        options.add(flags: ["-informative", "--informative"]) {
            self.verbosity = .informative
        }
        options.add(keys: ["-v", "--verbosity"], usage: "Sets the verbosity level to <value>") { (value) in
            self.verbosity = Verbosity(rawValue: Int(value) ?? -1) ?? .none
        }
    }

    public func execute(arguments: CommandArguments) throws  {
        if verbosity == .none {
            throw CLIError.error("Can't be silent and loud")
        }
    }

or if you didn't want overriding options:
    enum Verbosity: Int {
        case error = -2
        case none = -1
        case silent = 1
        case errors = 2
        case some = 3
        case most = 4
        case informative = 5
    }

    var verbosity: Verbosity = .none

    public func setupOptions(options: OptionRegistry) {
        options.add(flags: ["-s", "--silently"]) {
            self.updateVerbosity(.silent)
        }
        options.add(flags: ["-informative", "--informative"]) {
            self.updateVerbosity(.informative)
        }
        options.add(keys: ["-v", "--verbosity"], usage: "Sets the verbosity level to <value>") { (value) in
            self.updateVerbosity(Verbosity(rawValue: Int(value) ?? -1) ?? .none)
        }
    }

    func updateVerbosity(_ val: Verbosity) {
        verbosity = (verbosity == .none ? val : .error)
    }

    public func execute(arguments: CommandArguments) throws  {
        if verbosity == .none || verbosity == .error {
            throw CLIError.error("Error")
        }
    }

Since this functionality goes against general convention yet SwiftCLI still allows for this sort of customization if it's desired, I don't think it makes sense to change substantial sections of SwiftCLI in order to accommodate this feature.

BTW, I made a typo in my comment before,

Indeed, the example you posted would be terrible design, and while what I did would support something like that, it was definitely NOT my reason for doing it.

Basically, what I meant to say is your example is just scratching the surface of what I intended to be able to do.
Stop thinking of options strictly as options for a second.
What I did was add a completely new type of parameter to SwiftCLI; and I implemented it in such a way that options as they are now are a subset of this kind of parameter.
Now, you can make a CLI exactly as you would have done before and the current behavior will not be modified. (i.e. by default OptionCommands will have an OptionGroup called "options" that is neither required nor conflicting, and all options will be assigned to that group unless specified otherwise)
But, users are now also provided with a simple and straight-forward way to implement more complex logic in their OptionCommands.
I was sure I'd included a link to where I got the idea but perhaps I forgot. I was actually looking for alternatives to SwiftCLI, hoping to find something that might a) allow me to do what I wanted out-of-the-box, or b) Give me an insight on how exactly I might make a command signature to satisfy what I needed. I couldn't really find anything suitable to my needs, but I did find this Commandline parser written in C#, and that's where I got the idea (although the logic they implement is MUCH more complex)
https://www.codeproject.com/Articles/19869/Powerful-and-simple-command-line-parsing-in-C#x1-180<IP_ADDRESS>

Ok I understand better what you're getting at with this. Could you provide a non-toy example of when this would be needed? I'm having a hard time thinking of times when an option should be required. By convention, required parameters are positional and optional parameters are named, and by convention conflicting options override each other. While I understand that the functionality you have implemented allows for new possibilities, I think those possibilities go against convention and I'm not sure they have much real utility. A non-toy, non-niche example might persuade me otherwise though, as I'm not set in this line of thinking.

When you have a program where commands might have sub-commands for example; or, when you don't want to implement several commands that end up being virtually the same.
Now, what I don't get is this... the so-called "convention" is flaky at best; the fact of the matter is command-line conventions are about as standard as CSS used to be back in the IE7 days. Being in swift, the primary targets for SwiftCLI would be macOS and probably Linux. Yes? The closest thing to a convention available (to the best of my knowledge) are the POSIX guidelines and the  UNIX conventions to which SwiftCLI doesn't really adhere anyway. Honestly, I'd be hard pressed to think of more than a few examples of utilities that use positional parameters as they're implemented in SwiftCLI. In most cases, they are just there to specify input files.
Furthermore, it's very common to see shell scripts developed in python; ruby, or other languages, most of which have one or more argument parsers, several of which implement features such as I did.
IMO, building a powerful and customizable CLI module, and then purposely leaving out features that are implemented in several other similar modules across many different languages is kind-of self-defeating.

I'll ask again for a non-toy example of when this would be needed. You have not provided any examples of when this kind of feature would be helpful or intuitive to a user.
SwiftCLI certainly was designed with the POSIX guidelines in mind, but you're right that it does not adhere to those guidelines rigorously. A few popular utilities that use positional parameters as they are implemented in SwfitCLI: git, rails, brew. If you could point me to some examples of CLIs which use arguments as you're proposing here, I think that would help me to see the value in them.
I'd argue that simply adopting a feature because other modules do is a silly rationale and a great way to welcome feature-creep. I'm open to this idea, but you have as-yet been unable to give an example of when this functionality would be useful to the user and not counter-intuitive given how other CLIs function.

Okay, let's try this again...

Most commands do not actually use positional arguments except to specify lists of files. The exception being commands (such as in git).

While it's true that many applications can be implemented in just the way you describe, not all of them can and certainly not all of them are. I insist on the fact that the (or at least my idea, for what it's worth) of making a framework is to not arbitrarily push design styles on the user (in this case the user is actually the programmer of the CLI utility, mind you).
What this boils down to, for me, is that (you might disagree) when you are making a tool other people will use to build things with, less is not really more.

My implementation could certainly be improved, I think, by adding additional logic to the way what I already implemented is handled (like that link I shared; i.e. not only being able to XOR option groups)

You must remember that in the end, CLIs are exposed to user-input, and that can often be almost literally anything, not even taking into account simple typos or errors. The way your commands are implemented, built around required/optional positional parameters as the primary component, flexibility of design is very reduced (even if someone wants to work with just options, as many CLIs do, you often will have to trade this for a decent usage-message.), and even then you will often end-up running into the "Too many/too few arguments" errors.

If you really want an example of a popular command that implements similar logic, look no further than tar:
[Koji@Anasas:~/test] $ tar -xf ./foo.tar -cf baz.tar
tar: Can't specify both -c and -x

Ok thanks for the example, I'm seeing the value in it now. I'm improving how options are dealt with in SwiftCLI significantly in the upcoming update (3.0), so while I can't accept your pull request, I will be implementing this feature. Check out https://github.com/jakeheis/SwiftCLI/blob/opt_groups/Sources/OptionRegistry.swift#L74 to see the WIP version, and feel free to open more issues if it fails to do something you're wishing it did.
Thanks for taking the time to talk this feature through with me. I firmly believe every feature should be carefully considered before implementing it, and while I know you have a different opinion on that, I appreciate you taking the time to convince me of the utility of this feature.

I haven't really looked thoroughly at it or tested it yet, I'll try upgrading to 3.0 and see but it looks like it's definitely a more complete implementation than what I did.
One thing I do encourage you to look at again and implement of my PR, though, is what I did to the usage statement and error message generators. It's mostly a cosmetic issue but there were also cases where your code to calculate padding could lead to crashes because of trying to add a negative number of spaces.

Closing this issue, can create a new issue for the message generators if need be


Peek

The peek command shows the album name in brackets. Is it possible to toggle to the artist?

While you're working on /peek, can you restrict to last 10 as well. similar output we had last time.

Done, and done!
Take a look at the new peek.js file, and the "config" field.
Each of the element can be overwritten in the main config.js file. For instance, if I want to show the artist name instead of the album name, I just need to put this as my config.
module.exports = {
username<EMAIL_ADDRESS>// Fill this with your grooveshark email / username
password: 'YOURPASS', // Fill this with your grooveshark password
plugins: {
default: {
peek: {
peekArtist: true
}
}
}
};

In the long terms, any plugins should be self sufficient in terms of default configuration. If you want to let the choice to a user, he can overload one or all the parameters easily.


[noetic-support] remove hardcoded python2 reference

Astra.cfg already used env python, no idea why UVCCamera was listed for python2 then...
I did not verify any functionality with ROS noetic,
but this patch is needed to compile the package at all.

fixed


Increase default font size

Draft because we need to discuss whether we want this

For reference:
Font size 14:

Font size 16:

Font size 18:

Thank you Paul! I would not use a default of 18 but 14 and 16 are both fine for me.

Thank you Paul! I would not use a default of 18 but 14 and 16 are both fine for me.

I agree with @huber-florian, 16 looks nicer IMO. But I'm just nitpicking at this point, I wouldn't be mad if we stay with 14 😄


Remove authorization module from the sdk-all build

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
Remove authorization module from the azure-sdk-all build. The SDK project was removed as part of https://github.com/Azure/azure-sdk-for-java/pull/25518 but this stale link was not removed causing a build break.
All SDK Contribution checklist:

[X] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[X] I have read the contribution guidelines.

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.

/azp run java - core

As a follow-up, we should clean up the Azure Pipelines too for authorization. [java - authorization - ci]

Yes, I am checking with engsys team before deleting it (in case there is a process to follow).


Programmatic Access to Extract Type Information

I'm working on a project where I need to programmatically analyze a Python codebase and extract type information for specific variables. I've been using Pylance in VS Code and I'm interested in leveraging Pylance (or the underlying Pyright) for this task.
Objective:
I would like to programmatically use Pylance or Pyright to analyze Python code and retrieve type information for specific variables.
Approaches Considered:
Language Server Protocol (LSP): I understand that Pylance operates as a language server using the LSP. I've considered setting up an LSP client, starting the Pylance language server, and then sending LSP requests to get type information. However, I'm unsure about the specifics and feasibility of this approach.
Using Pyright Directly: Since Pyright is the core type-checker behind Pylance, I am wondering if there is a way to use it directly as a library to analyze Python code and extract type information.
Questions:

Is there a recommended way to achieve programmatic type extraction using Pylance/Pyright?
Are there any internal APIs or tools within Pyright that could be leveraged for this purpose?

Thank you for your time and for the fantastic work on Pylance and Pyright!

What type information are you interested in? Pyright and Pylance both use LSP. That's the current limit to their extensibility. They don't have any other API to query type information.
However, LSP does give a lot of information about what's under the cursor. I guess it depends upon what information you're looking for.

Ah I see Eric responded to the same request in Pyright:
https://github.com/microsoft/pyright/issues/5734


[Bug]: Time_Series output of forecast recruitment in a seasonal model

Describe the bug
A four season model with recruitment in each season found that the recruitment reported in the Time_Series table was incorrect for seasons > season 1.
Investigation showed that the total recruitment for the year shown in the spawn_recr table is correct.
Also, the numbers_at_age table is correctly showing the detailed numbers broken down by birthseason and season.
The recruitment output for the Time_Series table needs to aggregate across birthseason and was doing it incorrectly for forecast years.  The output for time_series years is correct.
Problem is resolved when a cluase checking on birthseason is removed in ss_benchfore.tpl
To Reproduce
see above
Expected behavior
recruitment output for each relevant season
Screenshots
No response
Which OS are you seeing the problem on?
No response
Which version of SS3 are you seeing the problem on?
<IP_ADDRESS>
Additional Context
No response

demonstration of the fix is in attached excel file
yft_numbers_2.zip


Error in  running movinet-pytorch to onnx converted model

Hi @Atze00

I succesfully transfer trained this repos movinets
But i want to deploy it else where into tensorrt format
So i first converted pytorch model to onnx using torch.onnx.export
I wanted to verify whether model is correctly ported . So I ran it in onnxruntime
It is throwing error like in the screenshot
Suggest any solutions
kindly help me here
let me know if any more screenshots or anything else

hi @Atze00  I did a mistake of saving model as it is in torch.save meaning torch.save(model) instead of recommended way of torch.save(model.state_dict(),PATH) which led to this all nonsense

onnx converted and excuted with out error

I have yet to run sanity tests like onnx output and this repo output is matching on some videos

Now i am proceeding to export this pytorch model to tensorrt , there are some hiccups like oom

Thanks @Atze00  your repo model implementation is giving satisfactory results on our use case


Add table content to README.md

@Isfaqul - you deleted the README.md?

I think I am not entirely sure about github. I probably need to learn about git first and then try again. I am sorry, if I created problems.

I think I am not entirely sure about github. I probably need to learn about git first and then try again. I am sorry, if I created problems. @KendallDoesCoding

No worries. Ik your new to open-source. You will learn.

@KendallDoesCoding I have a python script to do the job for us. But we will have to use github action to run this script every time there is a change in the html file. The script is based on BS4

Sure. Can you make another PR and what the setup has to be done on the gh-action?

I think you will know better about the GH actions. So basically, we will need to install dependecies, which is BS4 and run the script, everytime there is a commit to the index.html file.

@KendallDoesCoding Hi, just made a PR.


Feat/upgrade carousel component to embla carousel

https://www.loom.com/share/af6f8a1853364220ac167697387fb15e
@Arifulislam5577

Dear @shuvo-me,
Your work is very good. You are to be commended for that. But we want to give you some guidelines how to make your work more beautiful and comfortable.
We will use the embla-carousel library for the carousel component.
First we will create a wrapper on top of the embla carousel. From where the items of the carousel can be controlled. And the carousel will look exactly like the embla carousel examples.
For example:

Default Style
Loop
Slides To Scroll
Drag Free
ClassNames
Parallax
Scale
Lazy Load

You can put numbers in the carousel item, or use the pictures we have provided. It is completely up to you. Only the carousel should look like an instance of the embla carousel.
I hope you understand. Best of luck.
Thank you
Md Ariful Islam

Dear @shuvo-me,
Your work is very good. You are to be commended for that. But we want to give you some guidelines how to make your work more beautiful and comfortable.
We will use the embla-carousel library for the carousel component. First we will create a wrapper on top of the embla carousel. From where the items of the carousel can be controlled. And the carousel will look exactly like the embla carousel examples.
For example:

Default Style
Loop
Slides To Scroll
Drag Free
ClassNames
Parallax
Scale
Lazy Load

You can put numbers in the carousel item, or use the pictures we have provided. It is completely up to you. Only the carousel should look like an instance of the embla carousel.
I hope you understand. Best of luck.
Thank you Md Ariful Islam

@Arifulislam5577 Thanks vai for your suggestions. I will update the features as well.


How to use GPU acceleration for LightGBM?

I could not find any documentation on this. Is GPU acceleration always on?

I don't believe GPU binaries are packaged currently, so I don't think it will work out of the box without some packaging changes and possibly code changes

Is there a plan to release it however? The documentation creates the impression that no work needs to be done by developers.
If there's no plan to release it then can you document the steps to leverage it? Should #1062 be merged as well?


Add the ability to validate Tagk/Tagv and Metric strings

No matter what policy's we set for names, people will make mistakes or not follow the policy. Requiring people to pre-register things makes the system too restrictive. So I suggest we add a mechanism that allows some things to be rejected when they don't match rules.
Bosun should be able to aid in enforcing naming conventions on:

Metrics
Possible Tagks
Possible Tag Values Given a Tagk
All Possible Tag Values

My idea on implementing this would be that the user could pass regexs for these things in the configuration (syntax will be more designed if we can agree on this feature).  Although this could have very bad side effects (for example every data point goes through an expensive regex), I'm not sure of anything else that would be flexible enough to be useful.

don't care anymore


Add ppid example for ps

Description
Add an extra example for the ps command
User-Facing Changes
Only adds this example:

Tests + Formatting
N/A
After Submitting
This is related to https://github.com/nushell/nushell.github.io/pull/864

Oops somehow I made the pull request to your fork by accident


Its not clear if auto-grow is only defined at creation time

Hi,
I'm wondering if this setting can be later changed when the server is up, or if this is only set during creation time.
The documentation is not very specific about it. I also could not find any details on az postgres server configuration that could change this later one.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 580efe92-f616-04ca-ec62-12e14e957280
Version Independent ID: b2d3872e-9fec-8bea-c789-cc32b7f201ab
Content: Auto grow storage using the Azure portal in Azure Database for PostgreSQL - Single Server
Content Source: articles/postgresql/howto-auto-grow-storage-portal.md
Service: postgresql
GitHub Login: @ambhatna
Microsoft Alias: ambhatna

@rafa-pizzi Thanks for the valuable feedback. I'll assign this issue to the content author for updating the document appropriately.

@rafa-pizzi Thank you for the feedback. The documentation for CLI support will be updated in few days and the current documentation will be changed to show your suggestion.

Hi, seems this has been updated. can you please close?
https://docs.microsoft.com/en-us/cli/azure/postgres/server?view=azure-cli-latest#az-postgres-server-update

@rafa-pizzi Thanks for the confirmation.
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Collapse causes Problems after update to 4.1.2 (previously 4.1.2)

Made an upgrade to 4.1.2 which results in an error concerning collapsing accordions.

After I made a downgrade it was fully functional again.
Will dig deeper into it and update this issue once I got more information.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via CodePen or JS Bin and report back with your link, Bootstrap version, and specific browser and OS details.

It was my fault but interesting behavior...
Had wrong ID namings (button<> accordion root element) but interesting part I want to share is that it's working with 4.1.0.

yep that's because now we query our elements in Vanilla JS to optimise our performances, in 4.1.0 we queried our elements thanks to jQuery

Thanks for the insight  👍

You just saved me potentially hours of work.  Just what I needed, thanks for posting that!!


Merged master into release 1.17

Merged master into release-1.17 to keep in sync as part of docs release teamwork.

Any idea why there's 175 commits in this PR but the branch says it's 194 commits behind master?
Worst case scenario we can cut (hard reset) from the last master commit before the release.

Hi @VineethReddy02 . Should this pull request remain open?

@VineethReddy02 would you like help getting this merge done?

@sftim @kbhawkey We don't need this anymore. Sorry for the confusion
/close


Cleanup link handler tests

If we're using / for win32 as well then there's no reason to have these checks - it'll be / across all platforms.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.696%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27272

Relevant Lines:
59354

💛  - Coveralls


feat: use a code first approach for the Go SDK

I believe we should remove as much codegen as possible, so here is my attempt of doing that.
 As much as I don't want learn CUE if I just want to run some CI.
So for me, the Go SDK should let you code your logic in Go, and whatever GraphQL internal pipes should be handled automatically. If you get the GraphiQL for free, it's nice, but it shouldn't be required to know how to stitch graphql query in your language.
It's WIP, and we can have nice discussions on the topic here, as needed.

This repository has moved to https://github.com/dagger/dagger/tree/cloak
Please re-open this PR in dagger/dagger:
From the cloak repository:

Fetch dagger/dagger

git remote add dagger git@github.com:dagger/dagger.git
git fetch --all

[optional but recommended] Duplicate your branch

git checkout your-pr-branch
git checkout -b your-pr-branch-copy

Rebase your branch on top of dagger/dagger's cloak branch

NOTE: there might be some commits not authored by you. Delete them.
git rebase -i dagger/cloak

Update your local cloak repo to be downstream of your dagger fork

git remote set-url origin git@github.com:<your_username>/dagger.git
git push --set-upstream origin your-pr-branch-copy

Re-create PR in dagger/dagger

Make sure you have the GitHub CLI installed (on macOS: brew install gh)
gh pr create -R dagger/dagger --base cloak

Close PR in dagger/cloak

I don't know if this still makes sense with all the new API coming.

Closing -- project moved to http://github.com/dagger/dagger/tree/cloak


[NCPI Catalog] Validate custom events are accessible on the GA4 Query Explorer for dev and prod

Need
Custom events are to be included in analytics reports.
Approach
Add a custom dimension in GA for every custom parameter specified in the following tags in the NCPI Catalog Google Tag Manager.
For any parameter that is missing, follow the CC Google Analytics Design Document (specifically the Configuring a Custom Event and Custom Events sections).
Definition of Done
Confirm every custom parameter is visible in analytics reports for NCPI Dataset Catalog - GA4 and NCPI Dataset Catalog CC Staging - GA4. Specifically, document the parameters below with checkboxes to indicate they have been checked:
NCPI Dataset Catalog - GA4 (https://ncpi-data.org)

tag - entity_selected

[x] entity_name

tag - entity_table_sorted

[x] entity_name
[x] column_name
[x] sort_direction

tag - filter_selected

[x] filter_name
[x] filter_value

NCPI Dataset Catalog CC Staging - GA4 (https://ncpi-data.dev.clevercanary.com)

tag - entity_selected

[x] entity_name

tag - entity_table_sorted

[x] entity_name
[x] column_name
[x] sort_direction

tag - filter_selected

[x] filter_name
[x] filter_value

@MillenniumFalconMechanic Should be working on dev and staging now!

Thanks @jpaten!


feat: add feedback bool to tap setting

related to https://github.com/SlimeVR/SlimeVR-Server/issues/422

what is your discord btw so we can add you to the channels we use?

what is your discord btw so we can add you to the channels we use?

WingB#6793


Support Material Theme Dark Mode

Current issue is that the card does not support dark theme mode

I already PR on this issue PR


Install Sentry SDK on consumers

[ ] Install Sentry and enable Sentry.
[ ] Verify exceptions flow through.

So sorry wrong repo :'(


Way to get marker visibility

Hi. Is there a way to detect if marker visible separately or it in cluster?

There should be some actions when you will start that detection. Action, like: click, mouse hover, initial rendering.
In what case you would like to know it?

Sorry to bother you, I solved the problem differently. I took the package supercluster, calculate clusters and points and render custom components without using react-leaflet-markercluster.
The case was in drawing custom markers with additional info in vector layer if it not in cluster

Hi, sorry to hijack this thread (almost a year later though) but I think this is something I need to solve my problem. I'm having problems with draggable markers in react-leaflet-markercluster and the reason is that the markers that are hidden in clusters gets undefined when leaflet tries to access them to make them draggable.
I would like to know how I can detect if a marker is in a cluster, on click event.
Thanks in advance,
Nick


Accessing Request object when using @ApiDecorator

Hello, I have been using @ts-rest with NestJS and enjoying it so far, however, documentation is sparse.
It's extremely common to need access to request headers and metadata in controllers, however, I don't see any way to achieve that when using @ts-rest. The @Api only returns the parsed params, query, and body, but the @Api decorator forces you to use @ApiDecorator and both of those require you to extend the ControllerShape of the Contract.
How can we access the full request in the controller when using @ts-rest with NestJS?

You can still add extra arguments to your request handlers and decorate those with NestJS's own decorators.
async getPost(
  @ApiDecorator() { params: { id } }: RouteShape['getPost'],
  @Req() req: Request
) {}

You also don't need to extend the ControllerShape anymore, see this this documentation https://ts-rest.com/docs/nest#response-return-type-safety.

Ah thanks guys this clears a lot up.


revise Palebuhan.xml

@ekobastiawan : I have just finished a first round of revising Palebuhan.xml.
Could you please look at the argr2ekba comments and take the necessary actions?

@arlogriffiths: Palebuhan.xml has been revised, Mas.


strstr(): Empty needle in pico_search.php on line [Numbers]

Hi,
Using your theme Magazine with Pico 1.0.2, I followed your instruction on how to activate the search functionality. But it gives me error.
Warning: strstr(): Empty needle in C:\xampp\htdocs\plugins\pico_search\pico_search.php on line 37 Warning: strstr(): Empty needle in C:\xampp\htdocs\plugins\pico_search\pico_search.php on line 41 Warning: strstr(): Empty needle in C:\xampp\htdocs\plugins\pico_search\pico_search.php on line 42
The warning still goes on. PHP 5.6
Warm regards,

Hi,
I don't have a running instance of Pico at the moment to test this, but the error means the search query is not passed to the plugin. Make sure your search form sends a GET variable 'q' with the search query to the search results page.

Hey,
It can be fixed if you add @ in front of strstr in pico_search.php


Datepicker

Hi! Please help me! I am using metro version 3 in my mvc project. This is my code:

View:

        <div class="input-control text span2" data-role="datepicker" data-locale="pt" data-
        format="dd/mm/yyyy"<EMAIL_ADDRESS>
                         Date

                 @Html.TextBox("Date", null, new { @readonly = "readonly" })
                 <button class="button" type="button"></button>

Controller:

// GET: /Order/Report
public ActionResult Report()
{
       var model = db.Orders;
       //bug: Add 2 days because metro automatically decreases 2 days
       ViewBag.Date = Convert.ToString(string.Format("{0:yyyy-MM-dd}", DateTime.Today.AddDays(2)));
       return View(model);
}

The problem is this bug, where it is added 2 days to the date informed. This problem can also be seen in the examples of the site.
Is there any fix for this error?

Hi There,
You should use the setDate function to set the date. Here is how i use it in my project. I have a Editor Template for dateTime.  My Datetime picker is a editable one where a user can type in the textbox and also they can select a date from the picker
/Template/
@using SrpWeb.Helpers;
@model DateTime?

  @*This adjusts PICKED date format. Both picked and incoming formats must match for edits to work  day=dddd*@
  @Html.TextBox("", (Model == null) ? String.Empty : ((DateTime)Model).Date.ToString("dd'/'MM'/'yyyy"), new  { @placeholder = "dd/mm/yyyy" })  @*This adjusts INCOMING date format.  Both picked and incoming formats must match for edits to work   day=ddd*@

@using (Html.BeginScripts())
{

}
/************End of Templete/
/************** HTML *************/

        @Html.EditorFor(model => model.StartDate, new { @class = "editor-field info float-left" })
        @Html.ValidationMessageFor(model => model.StartDate)

/**********************/
/Modal****/
[UIHint("_DateTimeDateOnlyShort")]
[DisplayName("Start Date")]
public DateTime? StartDate { get; set; }
/*************************/
Cheers

Solved my problem. Thank you.


[Vue] Fix /match/{matchID} dosent show mapstats correctly sometimes

most lilely this is happening on Builded binary

prob fixed at https://github.com/FlowingSPDG/get5-web-go/commit/c17a168e8cb24f4e88d757f92aa35e123209bde6


WIP: riscv64: Use the zero register directly

This commit is an attempt to use the zero register more often in the riscv64 backend. Currently materializing an iconst 0 value moves the zero register into a general purpose register, and then the general purpose register is used. This means that an extra register is used when it doesn't need to be, for example.
Naively fixing this by returning (zero_reg) during constant emission does not work. One reason this doesn't work is that it introduces vcode aliases to the physical register 0, and during OperandCollector collection the physical register makes its way into normal constraints rather than being a fixed nonallocatable constraint. This is fixable by moving the "resolve alias" phase earlier, for example at the beginning of reg_use rather than lower down in add_operand.
Even after applying such a fix, however, that solution still does not work. This produces situations such as when a 0 constant is returned from a function that the zero register is attempted to be placed into the first return register. These two physical register constraints naturally conflict and nothing is around to generate a move instruction. This happens within OperandCollector meaning it's not easy to generate more instructions at that time.
The "fix", which is incomplete, that this PR implements is to introduce a pseudo-instruction which has a fixed register definition of the physical register zero. This instruction emits no code and is intended to provide a location for regalloc2 to split bundles by automatically inserting a move in the above situation. To get this working however it required removing an assertion in the OperandCollector and ends up violating an invariant of regalloc2 where a virtual register is assigned to a non-allocatable register. This ends up generating instructions which move general purpose registers into the zero register on riscv64, which is unlikely to be what regalloc intended.

I'm opening this after some discussion with @elliottt today about this. We weren't really sure how best to proceed, if at all, with getting this to work with regalloc2. It seems that some sort of new constraint or new logic is required in regalloc2 no matter what, but we weren't quite sure how to modify it. I think @elliottt had a better idea about a possible shape, but he can probably describe it better than I.
@cfallin if you've got time (this isn't urgent by any means) we're curious to get your take on this shape of an approach with a pseudo-instruction that defines a value which we can place a special operand constraint on if necessary. Also cc @afonso360 as you're likely interested in this.
One way I thought of thinking about this problem is that for each platform there are sort of read-only physical registers in addition to the allocatable registers. The zero register on riscv64 is one example but the stack pointer and frame pointer are another on most platforms (since we typically don't want regalloc2 to allocate any virtual registers to those destinations). Something about being able to specify this seems like what we'd want with regalloc2 to (a) allow this in the first place (despite there not actually being a panic today) and (b) prevent merging bundles because bundles shouldn't in general be merged with the pseudo-instruction defining the zero register for example. (@elliottt had more thoughts on this too)

The "fix", which is incomplete, that this PR implements is to introduce a pseudo-instruction which has a fixed register definition of the physical register zero. This instruction emits no code and is intended to provide a location for regalloc2 to split bundles by automatically inserting a move in the above situation. To get this working however it required removing an assertion in the OperandCollector and ends up violating an invariant of regalloc2 where a virtual register is assigned to a non-allocatable register. This ends up generating instructions which move general purpose registers into the zero register on riscv64, which is unlikely to be what regalloc intended.

Eek, yeah, this approach is going to break RA2 completely: there is a strong divide between registers that are allocatable and those that aren't, and using a fixed constraint on a normal vreg will try to put the whole bundle in the fixed register. The zero register can't be used that way, so as you've seen, breakage results.
We actually had a bunch of discussion over in RA2 issues about this a year or so ago; it resulted in a new kind of operand, the "fixed nonallocatable" constraint: you want Operand::fixed_nonallocatable(). That should solve the issue without changing anything about the broader lowering algorithm (and please don't, it could have a bunch of other implications we'd have to think carefully about!).

See e.g. this thread where Amanieu ran into these same problems with the zero register, and reported some nice speedups in his application when using the new constraint he added.

Just wanted to say that this is an amazing coincidence since I was just about to write an issue on this very topic!

Yes sorry to be clear I'm not proposing changing any foundations or anything like that, I'm trying to understand the foundations myself and figure out how best to fit a solution into them.
I'm not sure how to use Operand::fixed_nonallocatable though? I thought that was sort of the first thing I tried which was to use the (zero_reg) directly in lowering (imm _ty 0). That ended up causing issues though where effectively at the end of the function Inst::Rets would create a constarint that a vreg, which was aliased to the zero register, was requierd to be in the first physical return register. This didn't seem resolvable by regalloc2 in talking with @elliottt which is when we came up with the idea of a pseudo-instruction, but with the pseudo-instruction I'm not sure how to use Operand::fixed_nonallocatable. Do you have an idea in mind though of what the register allocation operands would look like for this pseudo-instruction?

The problem was that we wanted to return a value of the zero register for the lowering of a constant, but the fixed nonallocatable constraint only works for uses not defs. The solution we've used in the past is to make special instructions for interacting with nonallocatable registers (MoveToPReg/MoveFromPReg in the x64 backend for example), but those end up turning into the moves that Alex is trying to avoid in the first place.
For argument 0 immediates the use of fixed nonallocatable constraint is great, but when it would be desirable to have that value propagate further is where we just don't have a good story with RA2.

Indeed, it works for uses and not defs -- which implies that we can't in isolation lower an (iconst 0) to a zero reg, but that makes sense because an use of the zero reg isn't actually storing the zero anywhere first. In other words, this decomposition -- "(i) lower an immediate 0, (ii) use it somewhere" is what implies the need for a normal reg.
Now of course it's a standard idiom on ISAs with a zero reg to use it as an arg; so what we need to do is to reflect that structure in the lowering as well, and handle this as part of the arg, when we lower the instruction that uses it.
In other words, we would want to match (e.g.) (iadd x (iconst 0)) and then directly emit an instruction that uses a fixed-nonallocatable Operand in its use-slot. That would imply a new Inst format, because it has different regalloc metadata.

One further thought: I think the clearest argument to me for why the above (do it as part of matching, not as a separate step) is necessary is exactly the case you're running into: returning 0 as a return value needs an instruction to produce the zero value, while using it as an argument does not. In that sense it's more like an immediate (add r1, r2, #2); the zero register is a like a weird encoding for a constant #0. We fundamentally need to "sink" it into consumers to have this distinction and generate an inst in the first case but not the second.

Yeah that works, and it's already what riscv64 and aarch64 are doing I believe. They don't even have special instruction variants, the (zero_reg) is just used in the normal Reg slot as other normal temporaries are used. My hope was that given the commonality of an architecture-specific zero register between these two we wouldn't have to go through all relevant rules and add lowering variants for them. Given how non-special it is in the ISA (e.g. for add it's not a new encoding, it's "just" that the register is set to 0) I would ideally hope that it wouldn't need to be that special in the lowering rules.
That all being said my hope is that there'd be an idea that would be relatively easy/non-invasive to integrate into regalloc2. If there isn't though then alas, but not much can be done about that.


Fix image resolution size

Codecov Report

Merging #234 (95e8e00) into master (24266cd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #234   +/-   ##
=========================================
  Coverage     58.80%   58.80%
  Complexity     1042     1042
=========================================
  Files           161      161
  Lines          3945     3945
  Branches        539      539
=========================================
  Hits           2320     2320
  Misses         1447     1447
  Partials        178      178

Continue to review full report at Codecov.

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


Better error for a missing stations

Error:
> DLY_FLOWS(STATION_NUMBER = "08HA009")
No start and end dates specified. All dates available will be returned.
Error in enc2utf8(col_names(col_labels, sep = sep)) :
  argument is not a character vector

Also results in:
Warning message:
call dbDisconnect() when finished working with a connection


After upgrade from 0.7.5 to 0.7.8 DPOTrainer get RuntimeError: expected scalar type Float but found BFloat16

After upgrade from 0.7.5 to 0.7.8 DPOTrainer get RuntimeError: expected scalar type Float but found BFloat16
I'm using KTO loss, haven't tried other loss yet
Traceback (most recent call last):
  File "cpt-hf/train/train_dpo.py", line 307, in <module>
    trainer.train()
  File "/home/user/.local/lib/python3.8/site-packages/transformers/trainer.py", line 1537, in train
    return inner_training_loop(
  File "/home/user/.local/lib/python3.8/site-packages/transformers/trainer.py", line 1914, in _inner_training_loop
    self._maybe_log_save_evaluate(tr_loss, model, trial, epoch, ignore_keys_for_eval)
  File "/home/user/.local/lib/python3.8/site-packages/transformers/trainer.py", line 2268, in _maybe_log_save_evaluate
    metrics = self.evaluate(ignore_keys=ignore_keys_for_eval)
  File "/home/user/.local/lib/python3.8/site-packages/transformers/trainer.py", line 3019, in evaluate
    output = eval_loop(
  File "/home/user/.local/lib/python3.8/site-packages/trl/trainer/dpo_trainer.py", line 1163, in evaluation_loop
    policy_output_decoded, ref_output_decoded = self.get_batch_samples(self.model, random_batch)
  File "/home/user/.local/lib/python3.8/site-packages/trl/trainer/dpo_trainer.py", line 1056, in get_batch_samples
    policy_output = model.generate(
  File "/usr/local/lib/python3.8/dist-packages/peft/peft_model.py", line 988, in generate
    outputs = self.base_model.generate(**kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/user/.local/lib/python3.8/site-packages/transformers/generation/utils.py", line 1764, in generate
    return self.sample(
  File "/home/user/.local/lib/python3.8/site-packages/transformers/generation/utils.py", line 2861, in sample
    outputs = self(
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/user/.local/lib/python3.8/site-packages/transformers/models/llama/modeling_llama.py", line 1199, in forward
    logits = self.lm_head(hidden_states)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
RuntimeError: expected scalar type Float but found BFloat16

Thanks for reporting! can you share a small snippet to illustrate how you encounter the bug ? cc @kashif

Also can you confirm you don't get the error with trl==0.7.7 ?

Might be related to: https://github.com/huggingface/trl/pull/1143

@younesbelkada
here's the code
def create_and_prepare_model():
    compute_dtype = getattr(torch, "float16")

    bnb_config = BitsAndBytesConfig(
        load_in_4bit=True,
        # load_in_8bit=True,
        bnb_4bit_quant_type="nf4",
        bnb_4bit_compute_dtype=compute_dtype,
        bnb_4bit_use_double_quant=True,
    )

    model = AutoModelForCausalLM.from_pretrained(
        model_name, quantization_config=bnb_config, trust_remote_code=True,
        # device_map='auto',
        device_map={'':torch.cuda.current_device()},
        use_flash_attention_2=True
        )
    print(model)

    target_modules = ["q_proj", "k_proj", "v_proj", "o_proj"]

    peft_config = LoraConfig(
        lora_alpha=alpha,
        lora_dropout=0.1,
        r=rank,
        bias="none",
        task_type="CAUSAL_LM",
        target_modules=target_modules
    )

    tokenizer = AutoTokenizer.from_pretrained(model_name, trust_remote_code=True)
    tokenizer.pad_token = tokenizer.eos_token
    tokenizer.padding_side = 'right'

    return model, peft_config, tokenizer

training_arguments = TrainingArguments(
    output_dir=os.path.join(save_dir, run_name),
    per_device_train_batch_size=batch_size,
    gradient_accumulation_steps=accumlate_steps,
    optim="paged_adamw_32bit",

    # Can't resume training: Error invalid argument at line 393 in file /mmfs1/gscratch/zlab/timdettmers/git/bitsandbytes/csrc/pythonInterface.c
    # optim="paged_adamw_8bit",

    save_steps=500,
    logging_steps=10,
    learning_rate=lr,
    fp16=True,
    max_grad_norm=0.3,
    num_train_epochs=100,
    warmup_ratio=0.03,
    # group_by_length=True,
    lr_scheduler_type="constant",
    run_name=run_name,
    evaluation_strategy="steps",
    eval_steps=200,
    ddp_find_unused_parameters=False,
    gradient_checkpointing=True,
    # weight_decay=0.01,
    # dataloader_num_workers=NUM_PROC//2
)

model, peft_config, tokenizer = create_and_prepare_model()
model.config.use_cache = False # because of gradient checkpointing

trainer = SFTTrainer(
    model=model,
    train_dataset=train_dataset,
    eval_dataset=eval_dataset,
    peft_config=peft_config,
    dataset_text_field="text",
    max_seq_length=length,
    tokenizer=tokenizer,
    args=training_arguments,
    packing=True,
)

trainer.train()

are you sure this is the correct script? In traceback I see lines from the DPOTrainer

@younesbelkada sorry I just changed to the correct one, also tried 0.7.7 works fine

Thanks @Andcircle I managed to reproduce and will provide a fix soon

@younesbelkada thanks for the super fast response

@Andcircle things are fixed now on Pypi, you can use trl==0.7.9 without any problem


Feature request: Monaco Editor integration option?

Tell us how you think we can improve Sandpack
Packages affected

[ ] sandpack-client
[X] sandpack-react

What is this feature?

An option to use VS Code's Monaco editor.
Codesandbox website uses Monaco, so it would be nice to have a similar experience when cross over between the sandpack version and the codesanbox website (when you click the codesandbox logo in sandpack at the bottom right)

How would your idea work?

Ideally the API and setup would not be too different from using the existing @codesanbox/sandpack-react package.
TODO: need to dig into monaco-react package & report API differences & ease of integration

Thoughts regarding monaco-editor
VS Code's team's monaco code editor has lots of nice conveniences built right out of the box for a nice text editing experience that  code mirror doesn't have out of the box & require downloading multiple code-mirror add on packages & composing them together

hover annotations
peek definition (this can potentially be used to peek content of other files that are already in the sandpack sandbox of the user.
option clicking editor brings context menu
command palete
find all references

https://user-images.githubusercontent.com/6743796/149833470-197826fb-e235-46de-952e-442a1fe0782e.mp4
Demo link (thanks to @aboveyunhai from this Github Discussion #237 )

https://playground-brainstorm-git-monaco-editor-react-4ed6d5-anubra266.vercel.app/

Questions
Hi @danilowoz,

Has the team thought about monaco integration in the past?

I'm probably mixing context & the technical tradeoffs that were made to avoiding getting trapped in the same rabbit holes

What area of the codebase would I need to look at to help enable this  feature if I were to start a new feature branch?
What level of difficulty or size would you rate this task for a new contributor in this codebase?

Thanks 🙏
References

monaco-react by @suren-atoyan
code mirror discussion on running a typescript language server in the browser
@typescript/vfs - A map based TypeScript Virtual File System

Run typeScript in the browser
Run virtual TypeScript environments where files on disk aren't the source of truth

Hi @madebysid,
I recently read your forum post in the codemirror discussion forums on how to get typescript's language server to work with codemirror.
I'm curious, are there any pros & con's that come to mind when it comes to choosing codemirror vs monaco-editor?
Was also curious about why you chose codemirror over vscode for https://github.com/prisma/text-editors/ 🙏

@cliffordfajardo thx for the tagging.
Just to add a little bit of context, the current difficulties we had facing is a little bit out of scope, the main trouble we had is the setup of typescript's language server.
The Monaco editor is able to generate autocomplete based on the provided and customized d.ts files relatively easy but not for .tsx | .ts files,
so for those packages written in .tsx by default, I cannot find an easy way to auto generate d.ts files and bundle them for the Monaco editor.
If we look at the node_modules, typescript packages has an auto generated dist folders that contains all the typescript declaration, how to create that declaration folder and import into editor automatically, is something that I was struggling for.
so the lastest update is probably tackling this.
https://github.com/codesandbox/sandpack/discussions/237#discussioncomment-1979100.
In term of UI/UX, as long as we are able to setup the environment and get the hint from the typescript server stably, it's just a matter of time to improve them gradually.
In fact, using CodeMirror in this case leave a lot of flexibilities to users to craft their own hint UI.

Hey all, thanks for raising this issue!
Although I wasn't here when this decision of CodeMirror over Monaco was made, I think I can understand some of the reasons and highlight them here:

Mobile support: Monaco doesn't have a great mobile support, and this is crucial for Sandpack to deliver a uniform experience throughout all types of devices;
Final bundle size: CodeMirror provides an easy way to load only the modules/extensions we want to, which gives us the possibility to lazy-loading and code-splitting the CodeEditor component. Meanwhile, Monaco is a heavy package, and that's why CodeSandbox.io/CodeSandbox Embeds also defaults to CodeMirror in the mobile version (due to the previous reason too);
Extensibility: TBH, I haven't had many experiences with Monaco before, but I can tell what I've heard, and Monaco makes it hard to customize and extend some configuration - and Monaco provides a great API to extensions, which pretty much solved many of our problems;

Plus, Sandpack aims to be designed to be extensible and provide a basic (but powerful) development experience, so the initial goal was never to support any language server by default but to provide ways (API, documentation, support) for the application consumer (you) implement it by yourself.
Typescript integration
So what's the point of the #237 discussion? We want to empower Sandpack users to use their favorite tools and working environment, but by doing it as "extension" and not as a buit-in solution. Similar to the approach we took with Prettier and Eslint, we would like to introduce more guides/recipes of how to do it by yourself and customize it in you favor, based on your needs.
Still want to use Monaco?
In fact, you can easily switch from CodeMirror to Monaco at your end, as Sandpack exposes all the API needed to implement it. Here is an example using Monaco in Sandpack:

Sandbox: https://codesandbox.io/s/sandpack-monaco-integration-citxd?file=/src/App.tsx
Preview: https://citxd.csb.app/

Hope it makes clear this whole situations around the LSP and the editor options. Any further question, I'm more than happy to answer it, otherwise I'd close this issue. 😀

@cliffordfajardo I think Danilo nailed the reasons I also considered when I picked CodeMirror over Monaco.
For me personally, the biggest one was extensibility, because we had very specific interactions in mind, and I wasn't very confident that Monaco would let me build some of them.
This is more of a matter of taste, but I also preferred CodeMirror's API over Monaco's (plus CodeMirror's source code seemed a lot more approachable), so I went with it for prisma/text-editors.

Thanks for the awsome responses with added context 🙏
going to close this issue


Clojure: boot: Add derivation for boot build tooling

This PR adds support for a new clojure build tool, boot (http://boot-clj.com).
This is my first nixpkg contribution so would appreciate any feedback.

@geerds: Updated after your comments. Thanks!

@pSub Updated as requested.

Thanks for reviewing and merging!


Zooming Function

Maybe instead of buttons for zooming discrete amounts we have a zoom bar? Also maybe the function to be able to zoom to where the cursor is located?

There is apparently already some sort of canvas resize function hidden in the project under /public/app/views/ICE/ice.html. It has an id tools_top, disable the hidden display property and add inline width to be about 200 px to see it.

Looks like the canvas is automatically initialized as the view port dimensions and you can use the scroll wheel to change the canvas size or enter numbers.


Lime-35, Adding dashboard overview details for credit lines and savings account

https://linear.app/sublime-finance/issue/LIME-35/dashboard-tiles-subgraph

Review needed @akshay111meher


Fix typo in intercept_channel error message.

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @jinmel isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

@jinmel Please sign the CLA so we can move this PR forward.

I signed the CLA


Fix links in ThirdPartyNotices.txt

change url protocol from http to https
fix broken link http://source.icu-project.org/repos/icu/ to point to Github repository instead
This PR fixes #None

Is this for #132524?

It is not.

ThirdPartyNotices.txt is generated by concatenating licenses from various projects that are in our repository. Unfortunately, we cannot fix their license text. In this case, the root sources for the licenses which you have edited are:

https://github.com/ionic-team/ionic-site/blob/5d3b11a7b5f51193ace741a0e5ce57f646a98f68/LICENSE#L4
https://github.com/microsoft/TypeScript/blob/07fd7bce642b5e3227137672fbaebb7762cfbc63/ThirdPartyNoticeText.txt#L23

Please reach out to those projects directly and then, once we adopt a new version of those projects, we will adopt the new licenses that accompany the new versions.