[Relax] Allow composition of DFPattern replacements

The rewrite_call function accepts a DFPattern, and a function to
rewrite expressions matching that pattern.  Often, the rewriting
function will perform additional validation that cannot be expressed
within the DFPattern itself.  If this additional validation fails,
the rewriter function will return the matched expression unmodified.
Prior to this commit, an OrPattern that matches on the first branch,
but whose rewriter function does not apply a modification, would
prevent the second branch from being checked.  This commit updates the
ExprPatternRewriter to check both branches of a OrPattern, if the
rewriter function of the first branch does not modify the result.

This PR is currently marked as a draft, because it depends on the refactor done in https://github.com/apache/tvm/pull/16730.

The pre-requisite PR https://github.com/apache/tvm/pull/16730 has landed, so this PR is now rebased on top of main and marked as ready.  Thank you @slyubomirsky for the review, and so I think it's just waiting on CI now.

Hi @Lunderberg, it seems that this PR introduces new regression. The original issue came from CI in MLC-LLM https://ci.mlc.ai/blue/organizations/jenkins/mlc-llm/detail/PR-2068/1/pipeline/. And I spent some time adapting your test test_backtrack_if_rewriter_returns_no_op and reducing it to the minimal I can get.
Running the test below
def test_backtrack_if_rewriter_returns_no_op():
    pat_match_no_rewrite = is_op("relax.add")(wildcard(), wildcard())

    pat_arg = wildcard()
    pat_zeros = is_op("relax.zeros")(wildcard())
    pat_add = is_op("relax.add")(pat_arg, pat_zeros)

    pat = pat_match_no_rewrite | pat_add

    def rewriter(expr, matches):
        print(f"matching {pat} to {matches[pat]}")
        assert isinstance(matches[pat], rx.Call)
        if pat_match_no_rewrite in matches:
            return expr   ## <<== return the expr
        elif pat_add in matches:
            return expr   ## <<== also return the expr
        else:
            raise RuntimeError("Pattern matched, but neither branch matched")

    @R.function(private=True)
    def before():
        with R.dataflow():
            A = R.ones([64, 128], "int32")
            B = R.zeros([64, 128], "int32")
            C = R.add(A, B)

            R.output(C)
        return C

    after = rewrite_call(pat, rewriter, before)

will yield the following output:
matching OrPattern(Op(relax.add)(*, *) | Op(relax.add)(*, Op(relax.zeros)(*))) to R.add(A, B)
matching OrPattern(Op(relax.add)(*, *) | Op(relax.add)(*, Op(relax.zeros)(*))) to R.add(A, B)
matching OrPattern(Op(relax.add)(*, *) | Op(relax.add)(*, Op(relax.zeros)(*))) to C
[22:36:29] /home/ruihang/Workspace/tvm/src/relax/ir/block_builder.cc:65: Warning: BlockBuilder destroyed with remaining blocks!
Traceback (most recent call last):
  File "/home/ruihang/Workspace/tvm/tests/python/relax/test_dataflow_pattern.py", line 1949, in <module>
    test_backtrack_if_rewriter_returns_no_op()
  File "/home/ruihang/Workspace/tvm/tests/python/relax/test_dataflow_pattern.py", line 1945, in test_backtrack_if_rewriter_returns_no_op
    after = rewrite_call(pat, rewriter, before)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ruihang/Workspace/tvm/python/tvm/relax/dpl/rewrite.py", line 59, in rewrite_call
    return ffi.rewrite_call(pattern, rewriter, func)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "tvm/_ffi/_cython/./packed_func.pxi", line 332, in tvm._ffi._cy3.core.PackedFuncBase.__call__
  File "tvm/_ffi/_cython/./packed_func.pxi", line 263, in tvm._ffi._cy3.core.FuncCall
  File "tvm/_ffi/_cython/./packed_func.pxi", line 252, in tvm._ffi._cy3.core.FuncCall3
  File "tvm/_ffi/_cython/./base.pxi", line 182, in tvm._ffi._cy3.core.CHECK_CALL
  File "/home/ruihang/Workspace/tvm/python/tvm/_ffi/base.py", line 481, in raise_last_ffi_error
    raise py_err
  File "tvm/_ffi/_cython/./packed_func.pxi", line 56, in tvm._ffi._cy3.core.tvm_callback
  File "/home/ruihang/Workspace/tvm/tests/python/relax/test_dataflow_pattern.py", line 1921, in rewriter
    assert isinstance(matches[pat], rx.Call)
AssertionError

As you can see in the error message, with this PR, the pattern matcher mapped an add to a single Var. This breaks our assumption that what matched is always an add. Would you mind helping dig and fix this issue? Thanks in ahead.

The particular rewrite_call that fails in MLC-LLM CI is this one https://github.com/mlc-ai/mlc-llm/blob/main/python/mlc_llm/compiler_pass/fuse_ft_dequantize_matmul_epilogue.py#L213-L225

Ooh, that's a weird one, and thank you for the test case.  I can reproduce it on my side, and will look in more detail tomorrow.  For now, the first thing I'm noticing is that the failure occurs on the third time that rewriter is called, where I'd expect it to only be called twice (once for each branch).
matches[pat_match_no_rewrite] is filled during the first call, and matches[pat_add] is filled during the second call.  During the third call, which fails the assert, matches[pat_match_no_rewrite] is filled again, which seems rather odd.

This is an interesting one.

The DFPatternMatcher will unwrap variables when attempting to find a match (link), as part of ExtractMatchedExpr.
The ExprPatternMatcher will save the value of each variable binding is saved in order to (link).
The ExprPatternMatcher checks for a match in the override for Expr VisitExpr(const Expr&).

So the call to rewriter with the assert fails is made when ExprPatternMatcher is checking the variable usage in the body of the SeqExpr.  This gets unwrapped by DFPatternMatcher, and identified as an expression whose value is equal to the match itself.
This potential bug existed all the way back to https://github.com/apache/tvm/pull/15578.  However, until https://github.com/apache/tvm/pull/16732, the relax::Var body of a SeqExpr would never be the first match encountered.  Any time a relax::Var would have matched, the expression to which it was bound would have matched first, hiding the later match.
I think the resolution is to update condition (1).  Where currently, we unwrap all variable bindings as part of a match, we should instead unwrap all variable bindings except for the top-level of the match.  This could be most easily implemented by having the ExprPatternMatcher::VisitExpr skip the call to TryRewrite for relax::Var.  I'll try this out and see if it resolves the issue.

@Lunderberg Thanks for the swift fix! It works for e2e compilation now.


Snowflake: remote file not found when using parallel-isolated decomposition in airflow

dlt version
1.4.1
Describe the problem
When using the parallel-isolated decomposition in airflow with snowflake as destination, using an external GCS stage, there seems to be a race condition somewhere and sometimes tasks fail when trying to load the state file because it has already been deleted (or possibly never been created in the first place).
snowflake.connector.errors.ProgrammingError: 091016 (22000): 01b9001e-0203-c615-0003-0f22001f45b2: Remote file 'gcs://composer-prod/data/dlt/datahub/_dlt_pipeline_state/1734084368.7914042.5748b20734.jsonl' was not found. If you are running a copy command, please make sure files are not deleted when they are being loaded or files are not being loaded into two different tables concurrently with auto purge option.

It does not happen when using the serialize decomposition.
Expected behavior
State file should not be deleted before it is uploaded to Snowflake.
Steps to reproduce
Set up Snowflake as destination with an external stage, and try to load a few resources with the parallel-isolated decomposition strategy.
Operating system
Linux
Runtime environment
Google Cloud Composer
Python version
3.11
dlt data source
SQLAlchemy (Snowflake)
dlt destination
Snowflake
Other deployment details
No response
Additional information
No response

@julesmga path to state looks weird to me. when using parallel-isolated each resource gets its own task that is executed in separate pipeline name. Also the working folder should be random. It does not look like that in your case:
snowflake.connector.errors.ProgrammingError: 091016 (22000): 01b9001e-0203-c615-0003-0f22001f45b2: Remote file 'gcs://composer-prod/data/dlt/datahub/_dlt_pipeline_state/1734084368.7914042.5748b20734.jsonl' was not found. If you are running a copy command, please make sure files are not deleted when they are being loaded or files are not being loaded into two different tables concurrently with auto purge option.

Could you paste your DAG? are you forcing working folder with environment variable maybe? do you see a correct DAG in Airflow (each resource in separate task)

Thanks for the quick reply, here are my env variables:
        "DESTINATION__STAGE_NAME"  = "public.composer"
        "DESTINATION__FILESYSTEM__BUCKET_URL" = "gs://composer-prod/data/dlt"
        "RUNTIME__LOG_LEVEL" = "INFO"
        "DATA_WRITER__BUFFER_MAX_ITEMS" = "1000"
        "DATA_WRITER__FILE_MAX_ITEMS" = "1000000"
        "LOAD__DELETE_COMPLETED_JOBS" = "true"
        "LOAD__TRUNCATE_STAGING_DATASET" = "true"
        "EXTRACT__WORKERS" = "4"
        "LOAD__WORKERS" = "4"

DAGs are split into a common file, and a separate DAG per source, tasks are correctly split by resource in Airflow in both serialize and parallel-isolated scenarios.
common.py: https://paste.xinu.at/YoUwAjoJntv6LtoH/
datahub.py: https://paste.xinu.at/Bu1sj2xCPi4s6RAT/


Feature/60 containerize application backend

This pull request adds:

Conditional compilation in the game
Docker container build for the server side
CI to build and release the docker container on ghrc.io

Forced push because I rebased this branch on develop so that all tests could pass :innocent:


unable to scan

Script failed to scan birthdays.
Error message: eyJ0eXBlIjoiU0NBTl9FUlJPUl9USU1FT1VUIn0=

The issue is solved in https://github.com/zagushka/birthday-calendar-extractor/pull/94


cursors not immediately closed

There are cases where mongodb returns a cursor_id (in _parse_response) but that cursor is not closed explicitly.
testing for this should follow the pymongo test_kill_cursors tests.

this is a dupe of #26


MAJOR Update for FRENCH translation for Trader ext + Add missing language keys

Update for FRENCH translation for common.php file: Adding all the missing language key present in the core files of this extension.

Related PR: https://github.com/rfdy/trader/pull/11

Update for this comment: https://github.com/rfdy/trader/pull/10#issuecomment-77813569

Hi @rfdy,
it's OK for me, could you take a look ?

Done, see that: https://github.com/rfdy/trader/commit/5d338160138782ab877efe2cca75e9199410dc80


Background shines through in input-group leaving small spots

Hey! Thanks for the great work on bootstrap :wave:
I've noticed small dark spots in an input field when a background-color is set:

https://jsbin.com/viweracaju/edit?html,output
This seems to be only happening in Safari. Is this a rendering issue in Safari that I should report to them or is there an issue with my markup?
It seems to be related to this setting - disabling the border-bottom-right-radius removes the stains.

Operating system and version
macOS 10.15.7

Browser and version
Safari 14.03. Seems to be fine in Firefox

Definitely seems like a rendering glitch. If this was just in the corners that'd be one thing (and perhaps linked to background clipping), but this seems like something outside our codebase.


Bump doc search to 1.9

Description

Urgency

[ ] Blocker
[ ] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

@renesijnke @judithpatudith merged change. Thanks.


Specify a subpath to build the changelog

Hi,
In the context of a mono-repo, is it possible to specify the repository subpath to build the changelog from ?
Something like git cliff -- subpath/
Thanks.

Hello,
You can do git cliff --repository subpath/ and it will use the git repository in subpath/.
If that's not what you mean, please elaborate.

Hi,
Thanks for the quick answer.
So, I have multiple projects in a mono-repo and I'd like to get the changelog of only one of these projects:
git-repo-root
├── project1
│  └── CHANGELOG.md
└── project2
    └── CHANGELOG.md

For now, I usually do conventional-changelog --commit-path=project1 -t releases/project1/v -o project1/CHANGELOG.md
I think something similar in git-cliff is required to be useful in monorepos :)
Regards,

So, I have multiple projects in a mono-repo and I'd like to get the changelog of only one of these projects:
git-repo-root
├── project1
│  └── CHANGELOG.md
└── project2
    └── CHANGELOG.md

I see. Have you tried --repository option? (e.g. git cliff --repository project1)

Yes, but I have an error:
$ git cliff --repository foundry-extensible-plugin
 ERROR git_cliff > Git error: `could not find repository from 'foundry-extensible-plugin'; class=Repository (6); code=NotFound (-3)`

I think I finally get it 😃
0bb7c91 -> adds --commit-path <PATH> argument for generating a changelog scoped to a specific directory
e.g.
git cliff --commit-path git-cliff-core/ -o git-cliff-core/CHANGELOG.md

This argument will be available with the next release. In the meantime feel free to test it out and report bugs if any 🐻

v0.3.0 is released.


Fix contributed profile icons

Fix  #182303 .
The little bug has returned.

@Tyriar You better  👍 ,I learned.

Wait..There are still some issues.

@tisilent oh? It was working when I tested it out?


remove old ubuntu distros

Ubuntu 15.04 Vivid Vervet
Ubuntu 14.10 Utopic Unicorn
Ubuntu 13.10 Saucy Salamander
Ubuntu 12.10 Quantal Quetzal
These should be removed from /src/ubuntu.pxe as they're no longer supported (looks like 13.04 has either been already removed or never present)

-1
There is value keeping historical releases even though they are EOL via netboot. For example, there are times where I would like to replicate a legacy environment for testing which is an ideal usecase for using a tool like netboot for rapid-er deployments instead of keeping archived ISOs lounging around.

https://github.com/antonym/netboot.xyz/pull/55

:+1: :)


Task :kotlinNpmInstall shows warnings

Noticed this only after upgrading Kotlin (and the multiplatform plugin) to 1.6.10 (and Compose to 1.0.1 but I don't think that matters)
Started getting this:
> Task :kotlinNpmInstall
warning workspace-aggregator-1bb757f5-9f00-4637-b8f2-763422759100 > opengl > webpack-dev-server > url ><EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
warning Ignored scripts due to flag.

and also it unceremoniously dumps a "kotlin-js-store" folder in my root project. I added a .gitignore for that folder but just making sure that was intentional. It seems kind of strange.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.

I have the same problem:
at first:
Execution failed for task ':kotlinStoreYarnLock'.
then

Task :kotlinNpmInstall
warning Ignored scripts due to flag.

Task :kotlinStoreYarnLock FAILED

@toffeantyri Yeah, it turns out that behavior is intentional
https://kotlinlang.org/docs/js-project-setup.html#version-locking-via-kotlin-js-store
You need to run a task which intentionally updates the yarn versions.


Correct field names for Subscription

subscription field corrections.
Subscription_s > Subscription_g
Subscription1_s > Subscription1_g
Subscription2_s > Subscription2_g
This is based on live data as well as documentation here: https://learn.microsoft.com/en-us/azure/network-watcher/traffic-analytics-schema-update as

@halkazwini
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@neu5ron,
Thank you for your contribution! I've just merged your PR.


Props are not setted while mounting in tests

Describe the bug
Most of tests use the following type of mounting:
mount(
    {
      template: '<v-app><AboutView /></v-app>'
    },
    ...
  )

This is the only way to render vuetify components rightly. However in tests for components with obligate props there's an error:
props are not passed in the component itself.
To Reproduce
See tests in components/__tests__/*. Try to change mounting option to one above and encounter vue warnings.
Expected behavior
Props should be setted.

Update
It seems that props are correctly set to component with mounting like:
mount(AboutView,
    {
          props: ...
    }
)

And this art of mounting is specified in original docs. However, due to veutify we should mount it through <v-app> tag, which requires setting props as <AboutView :props=props>, which causes another errors, such as:
[Vue warn]: Property "gene" was accessed during render but is not defined on instance.
TypeError: Cannot convert object to primitive value

Should be fixed now?

Missed your previous comment @holtgrewe . Yes, it's fixed.


Issue By ItsGamingTube - Didn't show any account on Client Area

Describe the bug
Create Account Get Something Went Wrong,Didnt show any account on client area
Screenshots

Can you print the actual error here and give more details? I have no clue what is actually not working.

Maybe the IP address is not whitelisted in MOFH Reseller panel

did you add your client area's server IP address in mofh?

Umm ask imgamingtube,im not imgamingtube

Please do not post an issue you cannot replicate. Either close this issue, or find the details we need.
Thanks

Ask "whoever it is" to create an issue, right? Why are you asking on behalf of "whoever it is"? He would know the issue than you!

Ok ok leave it, itsgamingtube is having issue while using github so thus Imloadinguvu help him, that's it!
Ask itsgamingtube to contact me on 0 whatsapp or Gmail.

What's your discord username? I don't use whatsapp (it's the worst app i ever seen) but i can contact you on mail. Tell me your gmail so i can contact you.

<EMAIL_ADDRESS>


Add test_rename_bind_pat

Fixes #2976

bors r+
This only adds a test for a previously fixed issue.

The test syntax was changed a bit since that issue was opened. It should now be fixed.
bors r+


Feat: extend Expression operations

Description
Extend Expression operations.

Fixed fetch priority operations for expressions operations: case when last brench of AST was not packed to Expression leaf to reorgonized expressions priority
Extended Expressions SturctValues attributes analyzing
Extended Expressions SturctValues tests
Extended Expressions Operations tests-many cases for priority expressions check

Codecov Report
All modified lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/semantic.rs
46.22% <100.00%> (+10.48%)
:arrow_up:

tests/expressions_tests.rs
100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


fix:displayName error displayed in development tool

fix:#1652
fixed an issue with incorrect display in developer tools. But I can't think of a good test case. Can you give me some suggestions

I'm not sure if there's a good test case, since React DevTools gets to it via secret internal stuff that isn't a documented API and may change at any time. I'm not super-worried about testing this specifically.
Thanks for the quick fix!


default datasource

Need to add some more defaults here
@Thanood, if you have any suggestions that will make it easier for your app, then please add comments
Atm I have setArray, this resets all sorting/grouping etc
Was thinking about adding these:

addNew(data?:any) (will add blank to top, so it works well with grouping, it will be ungrouped)
resetArray(array) sets array again, but keeps orderby/grouping/selection, will be useful when doing paging
addToArray(array) adds to bottom, and reruns sorting, and grouping if any
removeCurrent() removes current row and returns the row getting removed  (incase you want to save them for undo etc)
removeRows(rowArray) removes rows and returns rows (in case you want to save them for undo etc)
updateGrid() reruns sort/grouping in case you have save option fo new, that way you get everything in the correct place

Although the data part of my app is really simple (just show data items) I will have a look when I'm back at work. Recently I've tried to introduce a "redux-ish" behaviour to the app which takes a lot of time (it needs "predictable state"). Not sure when that's finished.
One thing about the function names. I think they are a little confusing. Especially the addNew/addToArray pair. How about this (same order as your list)?

unshift(data: any) - no optional data, appends to top (like array unshift)
resetArray(array) - see refresh
push(data: any) - array terminology again
remove(row?: Row | Row[]) - don't take the Row to seriously, it's only there for clarity - essentially remove the current without any parameters, remove a single row when row is not an array, remove the row array if row is an array
refresh(array?: any[]) - I just like "refresh" better but that may be personal taste 😃 - if an array is given, it would behave like the resetArray
addBlankRow() - I think this function should have a specific name and it deserves it

Not sure about unshift and push but I'll keep it there for the sake of discussion. 😃
In general I'm not sure if some of these are too "coupled" to the grid (as opposed to the data). For instance addNew() without any arguments manipulates the grid directly, right? If so, it's not manipulating the data which IMO a datasource should be all about.
removeCurrent() is a similar case. I'd probably split that into a getSelectedRow()/removeRow() pair.

thx, like it, Ill have a new look at these when I start adding more.
Added the addRows atm, didnt see this before now, and added it to the demo, just to see how it behaves/ see bugs better.
No problem changing name, only you that use it so we can break it a lot 😂
addNew would add a empty row in array binded to grid... Im not doing slice when you set array. (maybe I should)
A lot we can do to improve how default datasource works.

These are now added, removed my addRows

unshift(data: any)
refresh(array?: any[])
addBlankRow()
push(data: any)
remove(rows?: rows[]) does not do anything

All of these need some tuning, a lot more testing, refresh etc uses orderby and grouping, but overlook filter atm, same with push and addBankRow/unshift

Note to my sself

you need to check selection better when it makes changes to current entity

added the remove


Add the Octicons font

This adds GitHub's Octicons font to Homebrew, literally, the only font I use which isn't already part of the cask.
To note, it will install off of master, not truly the latest release.
Fixes https://github.com/github/octicons/issues/70.
/cc @cameronmcefee

Could you please squash your commits?

Could you please squash your commits?

Done.


Really confusing for a beginner

>You can't have both the AzureRM and Az modules installed for PowerShell 5.1 for Windows at the same time.
Well, which do i need AzureRM or Az ?- I'm new to powershell on azure
Why did you create 2 modules that are incompatible with each other - causing a complete mess and i'm only at the install stage.

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

ID: 463a0d17-6457-034a-a5f8-efe5e5d336e7
Version Independent ID: bd4fb194-d6af-e442-5ced-793e09c3fa83
Content: Install Azure PowerShell with PowerShellGet
Content Source: docs-conceptual/azps-2.8.0/install-az-ps.md
GitHub Login: @sptramer
Microsoft Alias: sttramer

@John-Blair Thank you for your query.
For Azure PowerShell to have cross-platform support a  new module Az was created rather than make modifications to the existing AzureRM module. Moving forward, all new functionality will be added to the Az module, while AzureRM will only be updated with bug fixes.
If you are starting new, recommedation is to leverage the Az module.
Documentation Reference -
How to migrate from AzureRM to AZ in Azure Powershell
Cross Platform Az Module Replacing AzureRM
Hope this information helps, please revert back if you have any further queries.

The question was:

Why did you create 2 modules that are incompatible with each other - causing a complete mess

@John-Blair Thanks for sharing the feedback, currently loading Az and AzureRM side by side is not a supported scenario. However if you want to have both on one mahcine, you can load AzureRM in Windows Powershell and Az in PowerShell Core.
Hope this information helps, please let us know if you have any further queries. Thanks

@John-Blair Thanks again for your feedback, at this point it appears that there's no further edits or updates needed for the documentation. We would close this issue for now, however if you have any further queries please feel free to tag us in your reply and will be happy to continue the conversation. Thank you.

The question was:
Why did you create 2 modules that are incompatible with each other - causing a complete mess?
You don't even answer the question asked and then close the issue - fantastic service - can I nominate this as the most useless response to a question?

@John-Blair Apologies, if the responses didn't answer your question. We certainly want to provide service/support that upholds our customer expectation.
Inactive issues make it difficult for us to track what is being worked, since I didn’t had further response for my previous update on the thread ended up closing the issue. However that doesn't mean the conversation has to be over, I have reopened the issue for us to continue the conversation.
Thanks again for sharing the feedback, the previous responses has been focused on what is the current supportability and limitation of the product and to support a scenario where you wanted to have both the modules what can be done.
With introduction of PowerShell Core 6.x based on the .NET Standard library the biggest and most important changes is that PowerShell has been a cross-platform product. To bring Azure support to all platforms, which means that the Azure PowerShell modules needed to be updated to use .NET Standard and be compatible with PowerShell Core. Rather than taking the existing AzureRM module and introduce complex changes to add this support, the Az module was created.
The Az module has a compatibility mode to help you use existing scripts while you update to the new syntax. The Enable-AzureRmAlias  cmdlet enables a compatibility mode through aliases, to allow you to use existing scripts with minimal modification while working towards a full migration to Az. Please be aware of the full breaking changes list to find where your scripts may require updates.
Please refer to this additional documentation reference on why a new module was introduced.
I will share your feedback to our internal product and request them to update the thread with any other additional information they can provide to further clarify the query you have.
If you have any other feedback on improving the documentation we have to further help our Azure CLI customers/community please do let us know, will be happy to hear the feedback and see where we can improve. Thank you.

Thanks for answering - feel free to close.

@John-Blair Thank you John.


Feature Request - Turn Off Screen

Hi there. Thanks for this SDK !
Is there a way to turn off the screen with it , like you can do in 'Energy Saving'  Menu where you can just turn the screen black by clicking  on 'Screen Off'
Side Request :
Do you think it's possible to return the Name you set to the TV ? The 'Device Name' The System Info do not return that.
Thanks !

I came here just to write this feature request. Is it possible? Maybe with some indirect script that would manually press correct buttons?

It's definitely possible given that klattimer supports it:

def screenOff(self):

    self.__send_command("request", "ssap://com.webos.service.tvpower/power/turnOffScreen", {"standbyMode": "active"})

def screenOn(self):
    self.__send_command("request", "ssap://com.webos.service.tvpower/power/turnOnScreen", {"standbyMode": "active"})

Hi @jurekmroczkowski, Thank you so much!
I have just created a new PR #68 with the change. Can you folks please test it out?

Checking in to see if anyone wants to test the patch out before I merge it with master. Cheers

Checking in to see if anyone wants to test the patch out before I merge it with master. Cheers

Tested it manually, works perfectly fine.

Thank you so much! I almost forgot about this PR. Merging this in.

Thank you so much! I almost forgot about this PR. Merging this in.

Could you also update the pip package, please? Thanks!

Done. Updated PyPI, the latest version is v0.8.7.


Webex Channel Fix: personId -> roomId

This is a much needed fix because presently we are unable to have the rasa-bot respond back to the specific room that sends the message thus enabling no reply to the user
The fix is also presented at https://github.com/RasaHQ/rasa/pull/5460 but since it will be released at 1.11.0 thenn we need to hotfix it soon.
References RasaHQ/rasa#5460
Proposed changes:

...

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

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

@wochinge kindly help for this MR.

Thanks for submitting a pull request 🚀 @erohmensing will take a look at it as soon as possible ✨

Added the changelog, thanks.

Thanks for the help @wochinge ! 🥇

@wochinge do you have any document you can share on the release strategy employed? I would like to learn from it.

@isaac-philip Thanks for your question! We talked about your question internally, and we fully agree with you that the release process should be documented publicly and transparently. We will keep track of our progress towards this in https://github.com/RasaHQ/rasa/issues/6260 .
Maybe this short info can help you out until #6260 is done:

We release Rasa Open Source minors in fixed 4 week cycles (currently on hold as we are working towards a Rasa Open Source 2.0 major)
Micro (also referred to as patch) releases are cut as required. They can only include bugfixes or documentation improvements, but no new features (following the semantic versioning guidelines)


Change Yaxis direction

Hi! How can I change the Y axis direction? I need minValue={120} maxValue={0} and show something like this. The Y axis must grow down.
Is there any prop I can use?

Currently this seems to be not achievable, but you may try the prop yAxisLabels. Pass it like this-
yAxisLabels={[120,100,80...]}

This prop could help me with the y's label but it wouldn't help me mirroring vertically, as to say, my graphic would still show the curve decreasing. I would like to have it increasing. This way the zero would on the top left.


Custom dialog / panel shortcut

Hi team - Our app actually uses some of the default key combinations and would love an override capability.  Haven't poked the bear enough to check if it's just undocumented or not, hoping it exists and it's just not there.  Would be happy to provide a doc enhancement for it.

Sure, PR Welcome ♥


Test

What version are you using?
Current main

What did you do?
Try to pass CI
What did you expect to see?
passes
What did you see instead?
https://github.com/stellar/stellar-cli/actions/runs/9196175043/job/25293677925#step:9:11

This is the test:
thread 'integration::dotenv::cli_args_have_priority' panicked at cmd/crates/soroban-test/tests/it/integration/util.rs:49:10:
called `Result::unwrap()` on an `Err` value: Install(Rpc(TransactionSubmissionFailed("Some(\n    TransactionResult {\n        fee_charged: 182117,\n        result: TxFailed(\n            VecM(\n                [\n                    OpInner(\n                        InvokeHostFunction(\n                            ResourceLimitExceeded,\n                        ),\n                    ),\n                ],\n            ),\n        ),\n        ext: V0,\n    },\n)")))


Seperate user identity from cluster api

@cmaloney this is ready for review

@cmaloney comments addressed; ready for another pass

:ship:


[T3A2]{T16-A1] Ong Li-Jen

Changed 2 files in order for CommandResult to be hidden and require encapsulation.

Ready for review


TumblThree can't download posts with multiple images o_O  Is it fixable? Need help!

There is very common type of posts with several images inside, for example https://cutest-animals-here.tumblr.com/post/181119782205/source   So, right now TumblThree downloads ONLY first image and rest of the images are not downloaded!
I will be very grateful for any help! I need to download all the pictures from that kind of posts https://friendly-animals.tumblr.com/post/181033027154/source

Lets try http://world-of-cats.tumblr.com - set "Settings" Download from 20181214 to 20181216
TT downloaded all images here: http://world-of-cats.tumblr.com/post/181111695607/my-silly-baby-boy-coco
and only the first image here:
http://world-of-cats.tumblr.com/post/181124322525/took-gary-to-a-local-ice-climbing-area-he-loved
http://world-of-cats.tumblr.com/post/181105965661/fortunatelystuff-comic-printed-hoodies
and completely SKIPPED this posts:
http://world-of-cats.tumblr.com/post/181092603719/source
http://world-of-cats.tumblr.com/post/181096736880/source

I will be very grateful for any help!  I need to download all the pictures from that kind of posts https://friendly-animals.tumblr.com/post/181033027154/source.

Did you check the latest issues before opening yours at all (#328)?
Did you read the latest change/release log before opening this issue?

Thank you a lot!!!!!!   It was <IP_ADDRESS>!!!


Getting the package ready for initial 0.0.1 push to PyPI

Remaining tasks:

[x] Add documentation on how to use with base Django and DRF
[x] Add tests for base Django views
[x] Add tests for DRF views
[x] Review the README (there are definitely some changes needed)
[x] Write package description and add to README
[x] Push to PyPI test

We need to update the following:
https://github.com/uktrade/django-hawk/blob/main/setup.cfg#L3-L5


(Feature Request) Add ability to reload views

It would be a nice feature if there was a way to automatically reload views just like cogs are reloaded. I keep all of my views in a single folder, and import them wherever I need them.
I found a way to do this using the importlib library.  I reload the file in which the view is present, and then also reload the cog. And it seems to work. Would it possible to make it work?

I'll take a look this weekend and see what I can do!


Reland "Migrate to Xcode 11.0 so that the iOS 13 SDK may be used. (#62)"

This reverts commit f53b002167c06751ef7a603d14f5a8d9e44a178c.
See: https://github.com/flutter/engine/pull/15154.

Whoops I accidentally pressed enter when I was trying to change the commit message. @cbracken should I revert and change the commit message?

I wouldn't bother :) It's just added convenience, not the end of the world at all.


delete button should be properly allign  in mobile view in compare page(blisss theme)

Bagisto Version

[ ] master branch

Steps To Reproduce
1  go to compare page in blisss theme
Actual Result

expected result

this is not a issue please check once again.


Various documentation refinements

Various documentation refinements:  formatting, tightening up language.
@beorn7
@brian-brazil
Signed-off-by: Anthony D'Atri<EMAIL_ADDRESS>

@brian-brazil I've pushed an amendment that I believe addresses each of your feedback items.  PTAL.

Thanks!


fix(searchbar): add IE support

remove() as a method on HTMLElements is not supported on IE.
Short description of what this resolves:
Add cross-browser support for searchbar.
This fixes IE error, and windows build of Ionic project.
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x
2.x
Fixes: #
Object doesn't support property or method 'remove'

Already MERGED in develop

Merged in master! thanks a lot for the PR!


Document how to tweak/operationalize spinnaker

Namely has given a great writeup of their Spinnaker journey. In it they cover a lot of configuration changes/tweaks that would be useful to know of without having to read Spinnaker's codebase. We should at the very least document these somewhere in a separate "setup" heading.

So I don't forget, it seems
redis:
  configuration:
    secure: true

Is important in some cases... need to dig into why

Resolving this issue, per #811
See Productionize Spinnaker.


Support 4bits quantization

As title.
Notes:

Require onnxruntime>=1.17.0
4bits currently require CPU AVX512 (MatMulFpQ4)
bnb4 currently only support CPU / CUDA (MatMulBnb4)

Thanks so much! This is amazing! I think it would be good to also name/suffix these models slightly differently, so you can export (and use) multiple versions.

@hans00 Can the bnb4 quantization be used with webgpu backend?

@bil-ash ORT recently added support for MatMulNBit and will be available in v1.17.3. You can already access it with this version.

Also, to help test, would you like to create a demo model on the HF Hub which contains all these different variants?

Here is my convertion:
https://huggingface.co/BricksDisplay/stablelm-2-1_6b-q4
https://huggingface.co/BricksDisplay/stablelm-2-1_6b-bnb4
https://huggingface.co/BricksDisplay/phi-1_5-q4
https://huggingface.co/BricksDisplay/phi-1_5-bnb4
As my test (Windows with DML, CPU EP) using StableLM-2-1.6b
Q8: 0.95 token/s
Q4: 2.46 token/s
BNB4: 0.44 token/s

For benchmark, the quantized all-MiniLM-L6-v2 model is
https://huggingface.co/BricksDisplay/all-MiniLM-L6-v2-q4
https://huggingface.co/BricksDisplay/all-MiniLM-L6-v2-bnb4

Did you mean the model file name like model_quantized_int4.onnx or model_quantized_nf4.onnx ?

Yes, exactly :) I think this PR can help standardize which suffixes we should use (as quantized will be overloaded). It might be worthwhile aligning this with other libraries' naming conversions. Do you have any thoughts on this?
Also, just checking, but the bnb4 version has model sizes which are the same as fp16 (only half the size of the original). Is this correct?

Yes, exactly :) I think this PR can help standardize which suffixes we should use (as quantized will be overloaded). It might be worthwhile aligning this with other libraries' naming conversions. Do you have any thoughts on this?

I have no idea.
My naming will be {orig_name}_quantized_{quant_type}.onnx

Also, just checking, but the bnb4 and q4 versions have model sizes which are the same as fp16 (only half the size of the original). Is this correct?

The dtype is not change, just convert from sentence-transformers/all-MiniLM-L6-v2.
The MatMul op should quantize to INT4 or NF4, others unchanged (maybe FP32)

Thanks for this contribution 🤗! Since 4-bit quantization will only be supported in v3, I've merged it into that branch.

@hans00 Have you by any chance experimented with q4f16 formats (model is and runs in fp16, with int4 block weight quantization)? That's the difference between:

(fp32) https://huggingface.co/Xenova/Phi-3-mini-4k-instruct, and
(fp16) https://huggingface.co/Xenova/Phi-3-mini-4k-instruct_fp16


Use codicons in commands

Fixes microsoft/vscode-internalbacklog#1088

Steps I took to make this change:

Went through every command with an icon.
Looked at the icon, found the identical codicon, and replaced it.
Built and verified that the correct icon was still used.
Searched for icon-name.svg in the project. If not found, I deleted the svg.


Tracking link opening

Is there a way to when the link is opened or clicked? The idea to find out how many users have clicked on the link but not provided the information and an easy way to run and manage the campaign.
One way is to use the Gophish and send the emails via GoPhish and use the Evilginx as LandingPage. Evilginx is not logging the URLs that are used to browse the Phishlet otherwise we can export the logs and correlate the data from GoPhish(goPhish adds a separate URL for each user who has received the email). That way we can find out how many users have opened the link vs how many users ended up giving the credentials.
I saw that GoPhish Get parameter is logged in debug on console but I can't find out a way to export the connection logs to extract the data.
Is there a way to get the logs data? or another way to achieve the same goal?

Hey, any updates/alternatives on this since it's been a year?


VDX-88

plugin.schema.json is supplied as we cannot let it generate one as callbacks are being used

the build failing is because of a test in the ms-authenticator that takes to long to complete


Sign In Feature

Implement Sign-in Feature for the CodeLabz Mobile

[x] SIgn In with email And Password
[x] Sign in with google
[ ] Sign in with twitter
[ ] Sign in with facebook
[ ] Sign in with github

@hiranthaR If other sign-in options are still in open action items then I would love to contribute.

Hi, @hiranthaR I am Archit Jain, want to ask can I contribute to issues of this repository for GSoC 2022?

Is this issue available? If available can you please assign me.

I have created many Flutter apps having

Twitter login
Facebook login
Github login
So, can you assign it to me? :)


Apologies - I supposedly have zcoin in a wallet at this address.  In fact I was able to login here.  But obviously this is not the page I belong on.  Can someone please tell me how to access my wallet?  I also have a note with my info that says you apparently only have phone wallets (this was in 2017 probably), but then why do I have this address for my zcoin?  Thank you for helping, and again, apologies for asking here.

This issue tracker is only for technical issues related to bitcoin-core.
General bitcoin questions and/or support requests and are best directed to the Bitcoin StackExchange.
For reporting security issues, please read instructions at https://bitcoincore.org/en/contact/.
Describe the issue
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:

Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead
Screenshots.
If the issue is related to the GUI, screenshots can be added to this issue via drag & drop.
What version of bitcoin-core are you using?
List the version number/commit ID, and if it is an official binary, self compiled or a distribution package such as PPA.
Machine specs:

OS:
CPU:
RAM:
Disk size:
Disk Type (HD/SDD):

Any extra information that might be useful in the debugging process.
This is normally the contents of a debug.log or config.log file. Raw text or a link to a pastebin type site are preferred.

What address? Which wallet is this? More information is required.

Closing this. Feel free to reopen with new information.


Updated the profiles to reflect categories and labeled missingness

This is a follow-up of https://github.com/frictionlessdata/datapackage/pull/68

@peterdesmet
@khusmann
Can you please take a look?

Thanks a lot @peterdesmet! I wanted to fix it too fast so made quite a lot of inaccuracies but now it's fixed

@khusmann a question regarding alignment with the rest of Data Package: would it be beneficial/clearer if value and label were titled name and title or do we want dedicated terms?

I'm merging to fix the discrepancy between the spec ad the profile. Let's open an additional issue/discussion if needed


Avoid using nearly double the memory needed

We were loading weights to the GPU and then doing another model = model.to("cuda") which resulted in another copy!
While doing this, I also reorganized the code a little bit so it is somewhat simpler to follow.
Test Plan
Install the local distribution and check memory usage via nvtop for the 8B model. Before this PR, it was about 34GB -- after this one it is ~18.4GB

Nice !


Explicitly add Loader to yaml.load

Fixes #675 #676

Coverage remained the same at 64.394% when pulling cf6dd45af0485da27ee169d73c231e736a9d5378 on jainnikhil30:master into 87da51c99d0b5b8f188676855768d7e0802dc988 on ansible:master.

Coverage remained the same at 64.394% when pulling cf6dd45af0485da27ee169d73c231e736a9d5378 on jainnikhil30:master into 87da51c99d0b5b8f188676855768d7e0802dc988 on ansible:master.


feat add insert column left or rightmenu item to Column Menu

example code in add_and_remove_column_row_screen.dart

@Hu-Wentao
Thanks for the code contribution.
I'll add it in the next version.

@Hu-Wentao
This function has been applied to the next version.
However, it is not added to the current column menu and can be customized instead.
Your code hasn't been released, but instead I was able to add new features thanks to you.


last commit broke fzf.vim integration

Checklist

[X] I have read through the manual page (man fzf)
[X] I have searched through the existing issues
[x] For bug reports, I have checked if the bug is reproducible in the latest version of fzf

Output of fzf --version
0.29 (devel)
OS

[X] Linux
[ ] macOS
[ ] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
Was updating my work machine plugins and noticed fzf stopped working. Checked my home machine and same thing after updating.
BREAKS ON:
"fzf": { "branch": "master", "commit": "590060a16b85270c19c605e8758cda16c4517086" },
WORKS AFTER REVERTING TO:
fzf": { "branch": "master", "commit": "124cd70710cb7c94ef2282773494bb0d727598d8" },

0.29 (devel)

You should upgrade your fzf binary. :call fzf#install() will do.


@auth IAM provider populate incorrect lambda execution role on the VTL resolvers

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
14.x
Amplify CLI Version
7.6.23
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No.
Amplify Categories
function, api
Amplify Commands
Not applicable
Describe the bug
When a lambda have a dependency on the graphql api, the auth resolver is expected to add the lambda execution role to the allowed list for the IAM provider. However, the role name is populated incorrectly on the resolver.
Expected behavior
Auth VTL resolver should allow the correct lambda execution role for the IAM provider.
Reproduction steps

Init an amplify project.
Add a graphql api.
Add a function, on the advanced setting, give permission for the graphql api (queries, mutations and subscriptions).
Run amplify api gql-compile and verify the auth.1.req.vtl.

GraphQL schema(s)

type Todo @model
  @auth(rules: [
    { allow: private, provider: iam }
    { allow: public, provider: apiKey }
  ]) {
  id: ID!
  name: String!
  description: String
}

Log output

# Put your logs below this line

Additional information
No response

On further analysis, it was confirmed as not an issue. When lambda assumes the execution role, it contains the lambda name at the end which would satisfy the condition $ctx.identity.userArn.contains($adminRole) in the VTL resolver. Closing the issue.


Add examples

The library documentation is really really scarce and it's super hard understanding what to do with it.

How to retrieve an Identity Token?
How does the cache work?
How to avoid page reloads?

Also, complete examples are much more useful than incomplete snippets of code leaving too much to imagination. You should consider adding a samples folder, with subfolders containing working applications.
Also consider these major contexts:

ReactJs
AngularJs
React Native

Thanks

@dluc We are working on MSAL JS documentation for some of these questions you asked, hopefully with the next release we will make some strides. Adding these questions to the planning doc to ensure we cover them in one form or the other.
Regarding samples, there is a PR #987 which is consolidating all samples in the root folder of the repository. Once merged, we will focus on fine tuning the samples as we go.
For React, currently we have a sample that demonstrates MSAL JS integration. @DarylThayil Can you please comment on Angular JS and React Native?

React: As @sameerag noted, we have a React sample available here. We may provide a wrapper in the future, but that work is not scheduled.
React Native: We currently do not support React Native due to lack of support for things like localStorage. We do have Android and objective-c libraries that we recommend for native mobile apps.
Angular: We have two Angular wrappers, msal-angular (4/5) and msal-angularjs (older angular). We'll be working to update msal-angular to support the current version of msal core and newer versions of Angular soon.

Thanks, really appreciate the update and looking forward to see the new additions. (fixed typo in my original post 'library'=>'library documentation').
I've been looking at the React sample, here's some comments in case you could extend it:

Would be nice to have a Typescript example, ie to better understand the variable types involved. This would help with the library API, e.g. how to find identity tokens.
The example encapsulates in AuthProvide logic that doesn't belong to auth, e.g. readEmail. In real applications this sort of methods is in other classes, who simply need access tokens and id tokens for external API calls. I would suggest to refactor the code, e.g. having a Email class that can invoke MS Graph using the access token provided by AuthProvider. This would be the base for any other class that needs tokens to communicate with external services.

@dluc Appreciate the feedback, I agree a Typescript example would be good to have.
To address your other questions:

An ID token will be returned after logging in. More info available here: https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-implicit-grant-flow#send-the-sign-in-request
Our cache isn't currently documented, as its an implementation detail of the library (you do get to configure options like localStorage verse sessionStorage). We've had requests to allow apps to provide their own cache implementations, which we're considering, however, that work is not planned. We would certainly want to provide more complete documentation if end up opening that up.
We are working on changing how we fetch access to address some of the performance problems that come with loading your app in an iframe or popup. Once that is released we will provide guidance to developers on how to take advantage of that.

Closing this issue, as it's been some time and I'm assuming the issue has been resolved. Feel free to open a new issue if you have further questions.


infinite scroller problem in server side rendering

react-infinite-scroller not working correctly in SSR. data fetching working but scroll sticks to footer.
Any Idea?

solved it?

there is a solution for SSR?


Prepare docs for release

I had started this branch to get the documentation updated to release a v2 :)
There might be something new to add, so maybe it's not ready to merge, but I wanted to open the PR to get some feedback

OK, I should have added all the important changes that will be present in V2!


docs: Add basic usage information

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (83503b1) to head (054b977).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #18   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          161       161
=========================================
  Hits           161       161

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


#if LUA_ALLOWED is messing up the compiling.

Describe your bug here.
I try to compile the ourple source code, but the #if LUA_ALLOWED on line 1207 of the playstate is messing up things as its seen as a blockage. The reason I can't just delete it is because its the thing that makes lua scripts run in data folders. Anyone have a fix for this? I've tried putting it in different areas on the script but it is always seen as a blockage.
Command Prompt/Terminal logs (if existing)
source/PlayState.hx:1207: characters 3-6 : Unclosed conditional compilation block

C:\Users\(my name)\Downloads\Ourple_Source\Ourple Source>

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
It's the ourple guy source code, so it would be edited yes, but theres no other issues with compiling except this one. (I've compiled it after deleting the line but lua scripts didn't work after words)

if it's from another mod, don't make a bug report here about it

valid


Not initializing correctly for typescriptreact

Thanks for adding such awesome support for JSX in vim!
I've noticed that the indentation doesn't always work in typescript files for me. I can reproduce with a minimal vimrc:
set nocompatible
call plug#begin('~/.vim/plugged')
Plug 'MaxMEllon/vim-jsx-pretty'
call plug#end()

If I open a .tsx file using this minimal vimrc, then any line I auto-indent is given 0 indent. If I open a buffer with filetype javascriptreact, the issue goes away. Here's a gif of me reproducing it on my machine:

My vim version is 8.2.500.

@jonsmithers Should be fixed now.

Thanks!!


Construct urls based on Organization or User

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

Demo for this PR
You can test the changes of this PR in this app

Url changes when switching between organizations
after creation of an organization we are navigated to it's profile with the constructed url

Hey @panteliselef - the snapshot version command generated the following package versions:

Package
Version

@clerk/backend
0.26.1-snapshot.043c566

@clerk/chrome-extension
0.3.25-snapshot.043c566

@clerk/clerk-js
4.54.2-snapshot.043c566

eslint-config-custom
0.3.0

@clerk/clerk-expo
0.18.16-snapshot.043c566

@clerk/fastify
0.6.3-snapshot.043c566

gatsby-plugin-clerk
4.4.3-snapshot.043c566

@clerk/localizations
1.23.3-snapshot.043c566

@clerk/nextjs
4.23.1-snapshot.043c566

@clerk/clerk-react
4.23.2-snapshot.043c566

@clerk/remix
2.9.1-snapshot.043c566

@clerk/clerk-sdk-node
4.12.1-snapshot.043c566

@clerk/shared
0.20.0

@clerk/themes
1.7.5

@clerk/types
3.48.2-snapshot.043c566

Tip: use the snippet copy button below to quickly install the required packages.
# @clerk/backend
npm i<EMAIL_ADDRESS>
# @clerk/chrome-extension
npm i<EMAIL_ADDRESS>
# @clerk/clerk-js
npm i<EMAIL_ADDRESS>
# eslint-config-custom
npm i<EMAIL_ADDRESS>
# @clerk/clerk-expo
npm i<EMAIL_ADDRESS>
# @clerk/fastify
npm i<EMAIL_ADDRESS>
# gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>
# @clerk/localizations
npm i<EMAIL_ADDRESS>
# @clerk/nextjs
npm i<EMAIL_ADDRESS>
# @clerk/clerk-react
npm i<EMAIL_ADDRESS>
# @clerk/remix
npm i<EMAIL_ADDRESS>
# @clerk/clerk-sdk-node
npm i<EMAIL_ADDRESS>
# @clerk/shared
npm i<EMAIL_ADDRESS>
# @clerk/themes
npm i<EMAIL_ADDRESS>
# @clerk/types
npm i<EMAIL_ADDRESS>

!snapshot

Hey @panteliselef - the snapshot version command generated the following package versions:

Package
Version

@clerk/backend
0.28.0-snapshot.a1792a4

@clerk/chrome-extension
0.3.27-snapshot.a1792a4

@clerk/clerk-js
4.56.0-snapshot.a1792a4

eslint-config-custom
0.3.0

@clerk/clerk-expo
0.18.18-snapshot.a1792a4

@clerk/fastify
0.6.4-snapshot.a1792a4

gatsby-plugin-clerk
4.4.5-snapshot.a1792a4

@clerk/localizations
1.24.2-snapshot.a1792a4

@clerk/nextjs
4.23.3-snapshot.a1792a4

@clerk/clerk-react
4.23.3-snapshot.a1792a4

@clerk/remix
2.9.2-snapshot.a1792a4

@clerk/clerk-sdk-node
4.12.3-snapshot.a1792a4

@clerk/shared
0.21.0

@clerk/themes
1.7.5

@clerk/types
3.50.0-snapshot.a1792a4

Tip: use the snippet copy button below to quickly install the required packages.
# @clerk/backend
npm i<EMAIL_ADDRESS>
# @clerk/chrome-extension
npm i<EMAIL_ADDRESS>
# @clerk/clerk-js
npm i<EMAIL_ADDRESS>
# eslint-config-custom
npm i<EMAIL_ADDRESS>
# @clerk/clerk-expo
npm i<EMAIL_ADDRESS>
# @clerk/fastify
npm i<EMAIL_ADDRESS>
# gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>
# @clerk/localizations
npm i<EMAIL_ADDRESS>
# @clerk/nextjs
npm i<EMAIL_ADDRESS>
# @clerk/clerk-react
npm i<EMAIL_ADDRESS>
# @clerk/remix
npm i<EMAIL_ADDRESS>
# @clerk/clerk-sdk-node
npm i<EMAIL_ADDRESS>
# @clerk/shared
npm i<EMAIL_ADDRESS>
# @clerk/themes
npm i<EMAIL_ADDRESS>
# @clerk/types
npm i<EMAIL_ADDRESS>

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Feature request: Add Rectangular Corner Radius property to Properties output

Very often developers need to know the corner radius. It would be very helpful to include it into documentation made by sketch measure.
Thanks.

try new version

Sorry, thanks for the update!


s3 read_json doesn't support parameter "lines"

pandas read_json has parameter "lines", which by default is to False.
As I understand aws-sdk-pandas doesn't support this feature, so on lambda I tried to use pandas library, which comes together with sdk layer.
df = pd.read_json(f"s3://{bucket}/{file}", lines=True)
However, I'm facing this error:
[ERROR] ImportError: Missing optional dependency 'fsspec'.  Use pip or conda to install fsspec.

I believe lines=True should already be supported in the wr.s3.read_json method (see example 4)


Tags

Looks like npm knows about a couple versions that GitHub does not. Can push those for our bisecting? :)

Hola

@akiran :+1:

That would be really helpful to find out if the project published new versions and changelogs.


Rotate webview

When i rotate with css transform the content of a webview, the rendering goes haywire. Is ther a way to chage the orientation of the webview?

I can reproduce it in Chromium extensions, I suggest reporting it to https://crbug.com since we can not fix it on our side.
The webview is rendered differently from other elements, if CSS transform doesn't work perfectly I think there is no better way.


[Bug]: Module parse failed: Unexpected character '�'

What happened?
When I try to invoke this node file with import, it complains "To load an ES module, set "type": "module" in the package.json or use the .mjs extension."
import { ChromaClient } from "chromadb";
const chroma = new ChromaClient({ path: "http://localhost:8000" });
When i add the line to my package.json file all works fine.
"type": "module"
However, when I try to load the same in my NextJS app... it's giving me the following error
./node_modules/onnxruntime-node/bin/napi-v3/darwin/arm64/onnxruntime_binding.node
Module parse failed: Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)
Import trace for requested module:
./node_modules/onnxruntime-node/bin/napi-v3/darwin/arm64/onnxruntime_binding.node
./node_modules/onnxruntime-node/bin/napi-v3/ sync ^./././onnxruntime_binding.node$
./node_modules/onnxruntime-node/dist/binding.js
./node_modules/onnxruntime-node/dist/backend.js
./node_modules/onnxruntime-node/dist/index.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./node_modules/chromadb/dist/module/embeddings/WebAIEmbeddingFunction.js
./node_modules/chromadb/dist/module/index.js
./app/api/chat/retrieval/route.ts
Versions
"chromadb": "1.5.11",
Relevant log output
No response

@dsws i tried 1.5.11 and nextjs this morning and it worked for me - is it possible to post a minimal repro? that would help me fix this. thanks!

Hey Jeff, really appreciate your help on this... I pushed the updated repo here
This is the file I modified
https://github.com/dsws/langchain-nextjs-template/blob/main/app/api/chat/retrieval/route.ts
If you want to see original its in the project langchain-nextjs-template
If you run yarn && yarn dev then submit the form here http://localhost:3000/retrieval

@jeffchuber @dsws, I think this issue has been flying around for a week or so where "type": "module" needs to be added to Chroma package.json

@tazarov added this line to node_modules/chromadb/package.json
But still the same issue....
@jeffchuber have u had a chance to run my project?

Actually i removed my webpack config and it started to not throw an error anymore, but the process takes a very long time.
My embedding are tiny from 3 documents
        const vectorStore = await Chroma.fromDocuments(documents, embeddings, {
            collectionName: "a-test-collection",
            url: "http://<IP_ADDRESS>:8000", // Optional, will default to this value
            collectionMetadata: {
                "hnsw:space": "cosine",
            },
        });

The process either takes a very long time, or just hanging...
Thoughts?

"type": "module" does make a difference tho u guys should add it to the repo

After installing encoding it started to give my original error again... so this is still and issue

Error came back after I upadted to latest chroma. Used the next.js primer that @tazarov gave me, but still same error:
<EMAIL_ADDRESS>Module parse failed: Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)
Is onnxruntime something I need to install on the OS?

Hi,
same issue here (chromadb:
⨯ ./node_modules/onnxruntime-node/bin/napi-v3/darwin/arm64/onnxruntime_binding.node
Module parse failed: Unexpected character '�' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
(Source code omitted for this binary file)
Import trace for requested module:
./node_modules/onnxruntime-node/bin/napi-v3/darwin/arm64/onnxruntime_binding.node
./node_modules/onnxruntime-node/bin/napi-v3/ sync ^./././onnxruntime_binding.node$
./node_modules/onnxruntime-node/dist/binding.js
./node_modules/onnxruntime-node/dist/backend.js
./node_modules/onnxruntime-node/dist/index.js
./node_modules/chromadb-default-embed/src/backends/onnx.js
./node_modules/chromadb-default-embed/src/env.js
./node_modules/chromadb-default-embed/src/transformers.js
./node_modules/chromadb/dist/chromadb.mjs
<EMAIL_ADDRESS><EMAIL_ADDRESS>./node_modules/langchain/dist/vectorstores/chroma.js
./node_modules/langchain/vectorstores/chroma.js
"chromadb": "^1.8.1",
"chromadb-default-embed": "^2.13.2",

Hi, running into same issue, any solutions?

@nneven, have a look at this discord convo - https://discord.com/channels/1073293645303795742/1215704892010663936
If you have a nextjs app the error indicates that your browser doesn't know how to read the onnxruntime binary which is necessary for the default embedding function in Chroma JS client.

Okay so I solved the issue on my end thanks to this StackOverflow answer. From what I understand is that Chroma has a dependency (onnxruntime-node) that uses .node files and webpack needs an external tool to load those. However, here they fixed it by just flat out ignoring them (but since the node-loader solution works I'm not trying this other one).
From the solution I had to omit this code or it wouldn't compile.
if (!isServer) {
      // Fixes npm packages that depend on `fs` module
      config.resolve.fallback.fs = false;
    }

Also I had to install the package @xenova/transformers.
And now I could embed stuff into the database! 🥳

I'm experiencing the same issue.
"chromadb": "^1.8.1",
"chromadb-default-embed": "^2.13.2",
"next": "14.2.5",

Node.js v20.12.1.

I've placed this within my next.config.mjs.
  webpack: (config, { isServer }) => {
    if (!isServer) {
      // Fixes npm packages that depend on `fs` module
      config.resolve.fallback.fs = false;
    }

    config.module.rules.push({
      test: /\.node$/,
      use: "node-loader",
    });

    return config;
  },

However, I'm getting the following error even though I've installed chromadb-default-embed.
Error: Please install chromadb-default-embed as a dependency with, e.g. `yarn add chromadb-default-embed`
    at _DefaultEmbeddingFunction.import (webpack-internal:///(rsc)/./node_modules/.pnpm/chromadb@1.8.1_encoding@0.1.13_openai@4.53.2_encoding@0.1.13_/node_modules/chromadb/dist/chromadb.mjs:2450:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async _DefaultEmbeddingFunction.loadClient (webpack-internal:///(rsc)/./node_modules/.pnpm/chromadb@1.8.1_encoding@0.1.13_openai@4.53.2_encoding@0.1.13_/node_modules/chromadb/dist/chromadb.mjs:2434:26)
    at async _DefaultEmbeddingFunction.generate (webpack-internal:///(rsc)/./node_modules/.pnpm/chromadb@1.8.1_encoding@0.1.13_openai@4.53.2_encoding@0.1.13_/node_modules/chromadb/dist/chromadb.mjs:2408:5)
    at async Collection.validate (webpack-internal:///(rsc)/./node_modules/.pnpm/chromadb@1.8.1_encoding@0.1.13_openai@4.53.2_encoding@0.1.13_/node_modules/chromadb/dist/chromadb.mjs:1775:22)
    at async Collection.add (webpack-internal:///(rsc)/./node_modules/.pnpm/chromadb@1.8.1_encoding@0.1.13_openai@4.53.2_encoding@0.1.13_/node_modules/chromadb/dist/chromadb.mjs:1844:73)

I again eencountered this issue when trying to use xenova/transformers.
Solution was: Use npm and NOT pnpm, yarn or some other package manager.

Hi,
same issue here (chromadb: ⨯ ./node_modules/onnxruntime-node/bin/napi-v3/darwin/arm64/onnxruntime_binding.node Module parse failed: Unexpected character '�' (1:0) You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders (Source code omitted for this binary file)
Import trace for requested module: ./node_modules/onnxruntime-node/bin/napi-v3/darwin/arm64/onnxruntime_binding.node ./node_modules/onnxruntime-node/bin/napi-v3/ sync ^./././onnxruntime_binding.node$ ./node_modules/onnxruntime-node/dist/binding.js ./node_modules/onnxruntime-node/dist/backend.js ./node_modules/onnxruntime-node/dist/index.js ./node_modules/chromadb-default-embed/src/backends/onnx.js ./node_modules/chromadb-default-embed/src/env.js ./node_modules/chromadb-default-embed/src/transformers.js ./node_modules/chromadb/dist/chromadb.mjs<EMAIL_ADDRESS><EMAIL_ADDRESS>./node_modules/langchain/dist/vectorstores/chroma.js ./node_modules/langchain/vectorstores/chroma.js
"chromadb": "^1.8.1", "chromadb-default-embed": "^2.13.2",

I again eencountered this issue when trying to use xenova/transformers.
Solution was: Use npm and NOT pnpm, yarn or some other package manager.

Ive traid most of these solutions to no avail. Putting it as module would break everything on my end. Let me know if you manage it guys!


Make the quorum generator & Validator routines @safe and nothrow

For whatever reason the autogenerated opAssign [...]

Maybe because the dtor is @system ?

Added another commit renaming the field as suggested.

Maybe because the dtor is @system ?

Yes that was it. I wish the diagnostic was better.

https://issues.dlang.org/show_bug.cgi?id=21043

Still a draft ?

No, just infinite wait bug.

I'll rebase.

@Geod24 do you want to look into it more, or it looks good to you?

If you want me to squash some stuff let me know. 👋


Different behavior in syscall of execve

Your Windows build number:  10.0.18362.657

What you're doing and what's happening:  refer to https://github.com/libmozart/system/issues/6

What's wrong / what should be happening instead: refer to https://github.com/libmozart/system/issues/6

Strace of the failing command, if applicable:  https://gist.github.com/mikecovlee/778edfe2c10889a257e6c82625134c3c

@imkiva

The strace log is appreciated, but appears to be not threaded. Try:
$ strace -f -o yourprogram.strace yourprogram args...

refer to
refer to

The likelihood of someone from the community entertaining going through an external thread that does not have CLI repro steps is.... small. A repro would look like:
$ # a CLI step
$ # another CLI step
$ # ... as many CLI steps as necessary
$ strace -f -o yourprogram.strace yourprogram args...

That said, I did follow the links take a look at the strace(1) log.
...
clone(child_stack=NULL, flags=CLONE_CHILD_CLEARTID|CLONE_CHILD_SETTID|SIGCHLD, child_tidptr=0x7faba4d00a10) = 2521
close(10)                               = 0
--- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_DUMPED, si_pid=2521, si_uid=1000, si_status=SIGSEGV, si_utime=0, si_stime=0} ---
...
clock_gettime(CLOCK_REALTIME, {tv_sec=1582272082, tv_nsec=942656200}) = 0
write(4, "runtime.info();runtime.delay(500"..., 49) = -1 EPIPE (Broken pipe)
--- SIGPIPE {si_signo=SIGPIPE, si_code=SI_USER, si_pid=2520, si_uid=1000} ---
+++ killed by SIGPIPE +++

One of the clients is dying on a segfault. Which is a bad start. Then the parent dies on a SIGPIPE, which is understandable. Why the segfault I cannot tell you with the information provided. But this is more likely than not a userspace problem.

Thanks for your responding. We have solved our problem now.


Model 703 - Update voltage scale factor description and model length

In model 703 it would be nice to specify that the voltage scale factor is linked to a percentage at least in the comments.
today: Voltage scale factor.
proposal: Voltage percentage scale factor

Update model length.

Add length to all fixed 700 models

Add length to all fixed 700 models


fix(Next): handle logout url in middleware

The logout doesn't work anymore in demo. This PR intends to fix by adding a handler for the logout endpoint.
Changes
Please list / describe the changes in the codebase for the reviewer(s).

Add logout endpoint to middleware matchers.

🙏


Implement hiding row in table (not removing)

How I can hide row at table ?

you can filter the row out. otherwise there is no way.


lib crash

cryptography 1.6 has requirement pyasn1>=0.1.8 but oauth2client 0.4.11 has requirement pyasn1>=0.1.7

Hi, PRs are welcome but keep in mind that is blocked by #70
Thanks

oh no, I just found a way not to check packtpub's free offer every morning


add text "keep exploring and changing"

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you. Please see our Contribution Guidelines for details.
Thank you and happy coding.


[OTHER]  Description needed

What would you like to share or ask?
When you click on the icon near the "Contributors" title, it only shows the title's name.
Proof of issue
https://github.com/rupali-codes/LinksHub/assets/105683440/3f98bcb1-cc36-4100-9cee-8357cea7616f
Solution
I think it would be best to add a brief description of this section. Here's an example:
These are the people that have helped LinksHub come to life! Thank you for your support! Note: M = Maintainer C= Contributors
I'm open to your suggestions @Anmol-Baranwal, @k-deepak04, and @rupali-codes
Benefit
People who come to LinksHub's website can gain a better understanding of our roles in the project.
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

This would definitely be helpful.


No syntax highlighting for large SQL files

Prerequisites

[x] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom?
[x] Did you check the debugging guide?
[ ] Did you check the FAQs on Discuss?
[x] Are you reporting to the correct repository?
[ ] Did you perform a cursory search to see if your bug or enhancement is already reported?

OS: Windows 10
Atom: v1.9.8
@mojombo @Caged @schacon @defunkt @benburkert
Description
No syntax highlighting for large SQL files
Hi guys,
Recently, I've been working quite more with databases and SQL. But I noticed in v1.9.8 and some older versions that when I open an already existing large *.sql file, Atom doesn't highlight the code. I tried it in safe mode but still got the same issue even when I switched syntax themes.
Atom only highlights large SQL code when I create a new untitled file and set the type to SQL and paste or type SQL code into it.
Expected Behavior: Atom highlights all the code in my SQL file.
Actual Behavior: No highlights for the SQL code at all in a file except when in an untitled file.
Some help would be appreciated. Thanks 😄 ✔

I'm going to assume that you're running into #1667 based on your description of "lengthy" files - if that's not the case, please clarify and I'll reopen this issue.

@50Wliu This is more specific to *.sql files only and is not a duplicate of #1667 .
Also, when I create and empty sql file (not untitled) and paste the sql code, it gets highlighted. But when I re-open it i doesn't get highlighted even in v1.9.9. This has been tested on three Windows 10 PCs even in safe mode and on different themes.
The image below is a GIF of an untitled file with language set to SQL and a named stupiddb.sql file.

Thanks. Please re-open.

I suspect that this is a duplicate of https://github.com/atom/atom/issues/11232 because the conditional behavior of it highlighting sometimes but not other times sounds identical to that Issue. Please add the information you have and any files that you can share that reproduce the problem to that Issue.

Thanks @lee-dohm,
It's just as #11232. I couldn't have explored all the issues on the repo.


Bootify spring instrumentation

It would be convenient to have spring boot magically configure rest template and spring MVC filter. For example, spring-boot-starter-web already includes spring-web and spring-web-mvc deps.
One way to do this is to make spring-boot an optional dep of the existing brave-instrumentation-spring-web and brave-instrumentation-spring-web-mvc packages. It could then add an autoconfiguration file which configures the filters (given an bean of type HttpTracing exists). In non-spring boot environments, this class wouldn't be loaded, so wouldn't crash or otherwise matter.
The only downside is that if people have their own auto-configuration, they may need to exclude this. That said, this should be manageable via a @ConditionalOnMissingBean thing..

another option would be to make an autoconfig directory and jars.

I plan to do this some time in the next few weeks if no one beats me to it.
Whoever you are please let us know here if you do start working on it
On Tue, May 9, 2017 at 12:01 AM, Adrian Cole<EMAIL_ADDRESS>wrote:

another option would be to make an autoconfig directory and jars.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave/issues/399#issuecomment-300055295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIiopKaRm30excnPT8xO15-zU14f6Sqks5r3-USgaJpZM4NUxXS
.

cc @TilinC since you are using spring-boot with brave

I am going to try to start working on this this week. I'll link the PR here when I do

This issue can be closed, sleuth now incorporates brave/zipkin natively.

This issue can be closed, sleuth now incorporates brave/zipkin natively.

In the linked pull request there was some discussion about doing this for people using Spring Boot but not using Spring Cloud (Sleuth), or who are stuck on Spring Boot 1.x. That said, unless we're aware of users for this today, I think we're good to close this.


Can‘t watch YoutTube videos in fullscreen (iPad)

Hi, in the DDG app it is currently not possible to watch YouTube (I know it's from Google😅) videos in full screen view. The fullscreen button is greyed out. (As you can see in picture 1)

Since it works fine in Safari (see picture 2), it seems to be a bug in the DDG app.

Device: iPad Pro 12.9" 3rd generation
Software: iPadOS 13.5.1
App-Version: <IP_ADDRESS>

Hi Jonathan,
Thanks for getting in touch!  Sadly, we are aware of this and are looking in to fixing it if we can. In the mean time you can use the browsing menu (The ... button top right) to select "mobile" mode and you should be good from there.
Thanks,
Chris

From: Jonathan Krug<EMAIL_ADDRESS>Sent: 14 July 2020 16:29
To: duckduckgo/iOS<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [duckduckgo/iOS] Can‘t watch YoutTube videos in fullscreen (iPad) (#675)
Hi, in the DDG app it is currently not possible to watch YouTube (I know it's from Google😅) videos in full screen view. The fullscreen button is greyed out. (As you can see in picture 1)
[image]https://user-images.githubusercontent.com/31386997/87444536-33826900-c5f7-11ea-8856-934b166c9a3f.jpeg
Since it works fine in Safari (see picture 2), it seems to be a bug in the DDG app.
[image]https://user-images.githubusercontent.com/31386997/87444582-41d08500-c5f7-11ea-823d-60fcff27e23c.jpeg
Device: iPad Pro 12.9" 3rd generation
Software: iPadOS 13.5.1
App-Version: <IP_ADDRESS>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/duckduckgo/iOS/issues/675, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAT2LKP2MQ5UM65FRDB5ZDR3R2WXANCNFSM4OZUH2JQ.


[GHSA-g8hw-794c-4j9g] Path Traversal in org.springframework:spring-core

Updates

References
Source code location

Comments
Add 7patch commits, of all which the commit messages claim Backport clean duplicate separators in resource URLs Issue: SPR-16616:
https://github.com/spring-projects/spring-framework/commit/91b803a2310344d925e5d4b1709bbcea9037554
https://github.com/spring-projects/spring-framework/commit/f59ea610dfcf55cd0b42f6dd76a9b3dab0218aa
https://github.com/spring-projects/spring-framework/commit/b9ebdaaf3710db473a2e1fec8641c316483a22a
https://github.com/spring-projects/spring-framework/commit/98ad23bef8e2e04143f8f5b201380543a8d8c0c
https://github.com/spring-projects/spring-framework/commit/0e28bee0f155b9bf240b4bafc4646e4810cb23f
https://github.com/spring-projects/spring-framework/commit/13356a7ee2240f740737c5c83bdccdacc30603a
https://github.com/spring-projects/spring-framework/commit/695bf2961feffd35b5560ccc982a2189dcca611

Hey @MarkLee131, you don't have any reference linking the tracking ID SPR-16616 to this advisory do you?

Hi @darakian  the updated link for Spr16616 is https://github.com/spring-projects/spring-framework/issues/21157, since spring migrated its issue from jira to GitHub years ago.


patch 1

refactored Header.js
add lazy loading component
modify header logo to using lazy loading component

fixed dependecies


internal/shaderir: remove ConstType

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
A type information should already be in a constant.Value so ConstType seems redundant.
Why is this needed?
No response

Hm, this seems tougher work than I expected 🤔


Use FastGaussQuadrature instead of GaussQuadrature

FastGaussQuadrature.jl has reached version 1 and seems to be receiving updates and bug fixes, GaussQuadrature hasn't been updated in 4 years.

This was removed at some point


[KYUUBI #1449] Fix when KyuubiServer initialize fail but not exit

Why are the changes needed?

Kyuubi server should exit when initialize fail.
Detail see #1449.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

can you check the jstack to see which non-daemon thread prevents termination when an exception occurs?

can you check the jstack to see which non-daemon thread prevents termination when an exception occurs?

I use Arthas to see still running thread:
"NIOServerCxn.Factory:/<IP_ADDRESS>:2181" Id=21 RUNNABLE (in native)
    at sun.nio.ch.EPollArrayWrapper.epollWait(Native Method)
    at sun.nio.ch.EPollArrayWrapper.poll(EPollArrayWrapper.java:269)
    at sun.nio.ch.EPollSelectorImpl.doSelect(EPollSelectorImpl.java:93)
    at sun.nio.ch.SelectorImpl.lockAndDoSelect(SelectorImpl.java:86)
    at sun.nio.ch.SelectorImpl.select(SelectorImpl.java:97)
    at org.apache.zookeeper.server.NIOServerCnxnFactory.run(NIOServerCnxnFactory.java:203)
    at java.lang.Thread.run(Thread.java:748)

can you check the jstack to see which non-daemon thread prevents termination when an exception occurs?

I use Arthas to see, Non-daemon thread is:
"SessionTracker" Id=22 TIMED_WAITING on org.apache.zookeeper.server.SessionTrackerImpl@5344d655
    at java.lang.Object.wait(Native Method)
    -  waiting on org.apache.zookeeper.server.SessionTrackerImpl@5344d655
    at org.apache.zookeeper.server.SessionTrackerImpl.run(SessionTrackerImpl.java:147)

Thread states are TIMED_WAIT.

Does this issue only occur when using embedded Zookeeper？

can you check the jstack to see which non-daemon thread prevents termination when an exception occurs?

I use Arthas to see, Non-daemon thread is:
"SessionTracker" Id=22 TIMED_WAITING on org.apache.zookeeper.server.SessionTrackerImpl@5344d655
    at java.lang.Object.wait(Native Method)
    -  waiting on org.apache.zookeeper.server.SessionTrackerImpl@5344d655
    at org.apache.zookeeper.server.SessionTrackerImpl.run(SessionTrackerImpl.java:147)

Thread states are TIMED_WAIT.

Does this issue only occur when using embedded Zookeeper？

YES, It causes by zkServer.start(). It has a thread still running when KyuubiServer initialize fail.
To make sure of this, I configure external zookeeper to confirm. It can exit when KyuubiServer initialize fail.
What do you think set embedded Zookeeper run as daemon thread, or do you have a better way to resolve it?

can you check the jstack to see which non-daemon thread prevents termination when an exception occurs?

I use Arthas to see, Non-daemon thread is:
"SessionTracker" Id=22 TIMED_WAITING on org.apache.zookeeper.server.SessionTrackerImpl@5344d655
    at java.lang.Object.wait(Native Method)
    -  waiting on org.apache.zookeeper.server.SessionTrackerImpl@5344d655
    at org.apache.zookeeper.server.SessionTrackerImpl.run(SessionTrackerImpl.java:147)

Thread states are TIMED_WAIT.

Does this issue only occur when using embedded Zookeeper？

YES, It causes by zkServer.start(). It has a thread still running when KyuubiServer initialize fail. To make sure of this, I configure external zookeeper to confirm. It can exit when KyuubiServer initialize fail.
What do you think set embedded Zookeeper run as daemon thread, or do you have a better way to resolve it?

maybe we can stop the zkServer here, if  zkServer.getServiceState == started?

Codecov Report

Merging #1468 (4208663) into master (cc40c87) will decrease coverage by 0.00%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master    #1468      +/-   ##
============================================
- Coverage     59.13%   59.12%   -0.01%
  Complexity      172      172
============================================
  Files           235      235
  Lines         12050    12052       +2
  Branches       1478     1478
============================================
  Hits           7126     7126
- Misses         4319     4321       +2
  Partials        605      605

Impacted Files
Coverage Δ

.../scala/org/apache/kyuubi/server/KyuubiServer.scala
61.03% <33.33%> (-1.63%)
:arrow_down:

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 cc40c87...4208663. Read the comment docs.

thanks, merged to master and v1.4.1


Build process maintenance 3rd

Next of #1031.

Make tests runnable for .NET Framework v4.7.2 with VS2022
In NPOI.Test.sln I got internal errors during test case search with the combination of  VS2022 + NUnit3TestAdapter 4.2.1, then I cannot execute any test cases those are listed in Test Explorer.
To fix the error, update NUnit3TestAdapter version to 4.3.1 .
In NPOI.Core.Test.sln, the problem didn't happen, but I also update NUnit3TestAdapter for consistency.

Update NuGet packages versions for consistency between Core and Framework csprojs
Microsoft.SourceLink.GitHub  --> 1.1.1  (in Directory.Build.props)
NSubstitute  --> 4.2.2
NUnit  --> 3.13.3

LGTM


Supposedly I need a gem for lesson, "Display a Tic Tac Toe Board"

ERROR REPORT TEMPLATE -------------------------------------------------------

What did you do?
I ran the command /usr/local/bin/bundle install

What did you expect to happen?
I expected Bundler to...

What happened instead?
Instead, what actually happened was...

Error details
Errno::EACCES: Permission denied @ dir_s_mkdir - /home/kaycee/.bundle/cache/compact_index/rubygems.org.443.29b0360b937aa4d161703e6160654e47/versions20160717-3214-1st3h5d
  /usr/lib/ruby/2.3.0/tmpdir.rb:86:in `mkdir'
  /usr/lib/ruby/2.3.0/tmpdir.rb:86:in `block in mktmpdir'
  /usr/lib/ruby/2.3.0/tmpdir.rb:130:in `create'
  /usr/lib/ruby/2.3.0/tmpdir.rb:86:in `mktmpdir'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/compact_index_client/lib/compact_index_client/updater.rb:29:in `update'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/compact_index_client/lib/compact_index_client.rb:64:in `update'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/compact_index_client/lib/compact_index_client.rb:55:in `update_and_parse_checksums!'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/fetcher/compact_index.rb:65:in `available?'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/fetcher/compact_index.rb:15:in `call'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/fetcher/compact_index.rb:15:in `block in compact_index_request'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/fetcher.rb:154:in `use_api'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:331:in `block in api_fetchers'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:331:in `select'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:331:in `api_fetchers'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:336:in `block in remote_specs'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/index.rb:10:in `build'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:335:in `remote_specs'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:82:in `specs'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/lazy_specification.rb:56:in `__materialize__'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/spec_set.rb:91:in `block in materialize'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/spec_set.rb:88:in `map!'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/spec_set.rb:88:in `materialize'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/definition.rb:140:in `specs'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/definition.rb:129:in `resolve_remotely!'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/installer.rb:195:in `resolve_if_need'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/installer.rb:70:in `run'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/installer.rb:22:in `install'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/cli/install.rb:102:in `run'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/cli.rb:175:in `install'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/cli.rb:11:in `start'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/exe/bundle:27:in `block in <top (required)>'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/lib/bundler/friendly_errors.rb:98:in `with_friendly_errors'
  /var/lib/gems/2.3.0/gems/bundler-1.12.5/exe/bundle:19:in `<top (required)>'
  /usr/local/bin/bundle:23:in `load'
  /usr/local/bin/bundle:23:in `<main>'

Environment
Bundler   1.12.5
Rubygems  2.5.1
Ruby      2.3.1p112 (2016-04-26 revision 0) [x86_64-linux-gnu]
GEM_HOME  /var/lib/gems/2.3.0
GEM_PATH  /var/lib/gems/2.3.0:/home/kaycee/.gem/ruby/2.3.0:/usr/lib/x86_64-linux-gnu/rubygems-integration/2.3.0:/usr/share/rubygems-integration/2.3.0:/usr/share/rubygems-integration/all
Git       2.7.4
  Bundler settings

orig_path
Set via BUNDLE_ORIG_PATH: "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/home/kaycee/.rvm/bin:/home/kaycee/.rvm/bin"
--- TEMPLATE END ---

Thanks for reporting this! It looks like we need to handle this kind of error better.
@rochesterinnyc, this would be a good bugfix ticket I think--if the default home directory compact index destination isn't writable, we should use a tmp dir instead.

Yup, sounds like a nice GSoC PR opportunity.

I'll work on this!

@kcee403 Would you mind giving me steps to reproduce this issue?

Permission errors will be changing due to https://github.com/bundler/bundler/pull/5007 in 1.14, so I think it's safe to close this (and re-open if needed at a later date)


Question regarding AArch64 Dwarf CFI directives used along Pointer Authentication to enable unwinding

Question goal
Clarification about the current usage of the DWARF CFI directive DW_CFA_val_expression by the project for the purpose of unwinding on AArch64 with Pointer Authentication enabled (i.e. -march=<arch> -mbranch-protection=pac-ret).
References

#3183 at the origin of the GCC ticket.
Bug 104689 opened by @akirilov-arm - aarch64: libgcc: DW_CFA_val_expression is not supported for RA_SIGN_SATE register
#4195 implemented by @akirilov-arm using .cfi_window_save (a.k.a. .cfi_negate_ra_state) instead of DW_CFA_val_expression to set the RA state.

Details
Today, DW_CFA_val_expression is supposedly only used for backtracing in the Cranelift project.
At the end in #4195, @akirilov-arm didn't use DW_CFA_val_expression to set the RA state register.
The code seems to have evolved since then and moved to https://github.com/bytecodealliance/wasmtime/blob/main/crates/fiber/src/unix/aarch64.rs.
This code uses  .cfi_window_save, so no DW_CFA_val_expression are emitted.
Question
As a conclusion, the project doesn't seem to use the exotic setting of the RA state register that was requested in Bug 104689.
Is my understanding correct ?
Is it an issue for you if GCC's AArch64 backend removes the support for setting the RA state register with DW_CFA_val_expression ?

It has been a while since I have worked on Cranelift, but as far as I can tell, the compiler still uses the same DWARF CFI directive when generating function prologues.
The fiber implementation that you have looked at is something completely separate.