[Bug]: my vllm phi-3-vision server runs one request correctly then returns an error for the same request stating 2509 image tokens to 0 placeholders

🐛 Describe the bug
I deployed a basic vllm multi-modal server with phi-3-vision, that seems to run well at first, i.e my first request is correctly executed, but at the very moment i send a second request i get an internal server error 500, and this output ERROR:__main__:Error during generation: Attempted to assign 1 x 2509 = 2509 image tokens to 0 placeholders while sending the same request a second time.

Your current environment
PyTorch version: 2.3.1+cu121
OS: Ubuntu 22.04.4 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Versions of relevant libraries:
[pip3] numpy==1.24.4
[pip3] nvidia-cublas-cu12==<IP_ADDRESS>
[pip3] nvidia-cuda-cupti-cu12==12.1.105
[pip3] nvidia-cuda-nvrtc-cu12==12.1.105
[pip3] nvidia-cuda-runtime-cu12==12.1.105
[pip3] nvidia-cudnn-cu12==<IP_ADDRESS>
[pip3] nvidia-cufft-cu12==<IP_ADDRESS>
[pip3] nvidia-curand-cu12==<IP_ADDRESS>
[pip3] nvidia-cusolver-cu12==<IP_ADDRESS>
[pip3] nvidia-cusparse-cu12==<IP_ADDRESS>
[pip3] nvidia-ml-py==12.555.43
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] nvidia-nvjitlink-cu12==12.5.82
[pip3] nvidia-nvtx-cu12==12.1.105
[pip3] pyzmq==26.0.3
[pip3] torch==2.3.1
[pip3] torchvision==0.18.1
[pip3] transformers==4.43.2
[pip3] triton==2.3.1
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.3.post1@38c4b7e863570a045308af814c72f4504297222e
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	GPU1	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	PHB	0-47	0		N/A
GPU1	PHB	 X 	0-47	0		N/A

Please update your vLLM version as it fixes a number of bugs relating to it.

I have experienced this before. IIRC, for me, this occurred only when I tried to use ENABLE_PREFIX_CACHING=true with Phi-3-vision. Try turning this flag to false.

Thank you for your replies, updating vllm and disabling ENABLE_PREFIX_CACHING, worked for me.


Pressing volume key doesn't open peek

iOS: 11.2.6
Xcode: 9.2
deploy: iOS 9
device: SE
swift: 4.0
program: example (pull date 23march 18) branch master

I think the example project uses a shake. It’s configured in AppDelegate.swift. But totally agree I should change that.
Thanks! Hoe you like the tool.

Fixed :)

thank you


Service Bus: Added 1 new command under Queue and Topic each

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

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

@MyronFanQiu can you please review the PR

@v-Ajnava please take a look at @MyronFanQiu 's comment and improve your code.

@v-Ajnava Since this PR is out of date, I will close it first. Please feel free to open a new PR.


Allow reverts when checking versions

If a new version is lower than the current checked-in version, and the
transition from the new version to the old version is valid (indicating
that it could have been the previous version), allow the change.
This prevents the version check from failing when reverting a PR.
This is not fool-proof (e.g., it would allow a revert of an
already-published PR); if we have issues in practice we can further
restrict the check (by checking that the new version is the current pub
version, for instance).
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format. See plugin_tool format)
[x] I signed the CLA.
[ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

There are a few extra things in the CHANGELOG that we forgot to list as the changes went in, so I filled them in retroactively.


The "path" argument must be of type string. Received type undefined

/lib/middleware.js
if config.cache.useHostPrefix === true
context.req.hostname is undefined

@panchenyu0928 I guess you're not using express, and hostname remains undefined.
express uses 'X-Forwarded-Host' header field (if configured to trust proxy) to provide that field.
I have fixed it to use req.host if req.hostname is not provided, but if you're behind reverse proxy, It won't have a meaning value.

Also if you're not serving multiple domains, disabling useHostPrefix saves you some memory.

@arash16 Thank you！😊


[SQL][Minor] Fix foreachUp of treenode

foreachUp should runs the given function recursively on [[children]] then on this node(just like transformUp). The current implementation does not follow this.
This will leads to checkanalysis do not check from bottom of logical tree.

Test build #30318 has started for   PR 5518 at commit 1ccbfa8.

Test build #30318 has finished for   PR 5518 at commit 1ccbfa8.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

Please add a test in TreeNodeSuite.

@marmbrus this will fail mima test, should i ignore tree node in MimaExcludes ?

Test build #30381 has started for   PR 5518 at commit 18e28b2.

Test build #30381 has finished for   PR 5518 at commit 18e28b2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

@marmbrus  test case added

Thanks!  Merged to master.


XACML temples are not visible to tenants

XACML templates are not visible for tenants. They are visible only for the super tenant.

Duplicate of https://github.com/wso2/product-is/issues/6294


How to install ray in this project ?

From the readme, I find it needs ray[rllib]==1.8.0, but when I tried to install ray[rillib], I find I can't install it. How to install ray with python3.6?

do
pip install ray[rllib]==1.8.0


2/3 Vote Block

From Grad School:

The ETD system currently allows students' work to move from "final pending" to "final submitted" with only  2/3 approved vote. This results in work not being reviewed by the entire committee. We would like to put a stop to this, so that only students who have received votes from every one of their committee members move on to "final submission is submitted". This way, if a member doesn't vote, an OTD staff person could simply log that the committee member "did not vote" which would move the student's work to "final submission is submitted". We would also like a new communication to be sent where if ETD identifies that if 2/3 of the committee has approved, ETD will automatically email any committee member who has not voted.

email:
Dear Professor,
We are writing to inform you that 2/3 of the committee has voted on this student’s final work. Unfortunately, a vote has not been entered for you at this time. We ask that you enter a vote within 7 days so that your name will appear in the student’s final work. If you do not respond within the 7-day period, your name will not appear in the student’s final work.
Thank You,
Office of Theses and Dissertations

We will have to change the seventh day evaluation worker to not update the submission status and change what emails go out to dissertation committees.  Right now, an email will go out to the committee chair and author for dissertations.  Do we still want this to happen?  Or should we just send emails out to committee members that need to vote?

To clarify - 7 days after the email above goes out, the work should move to final submitted.

Do we need another notification after the second 7 days?  Or just automatically update to "Did Not Vote"?

Got clarification from Fawn- we do not need to notify OTD; they are already monitoring. They would like us to still automatically update to "Did Not Vote" though

Update on preferred wording from Fawn:
Dear Dr. ,
This will be your final opportunity to vote on 's final work. If you do not vote within seven days, your name will be removed from the student’s work.
Add link to vote
Thank You,
Office of Theses and Dissertations


Reconsider Status Codes for HealthChecks middleware

From @anurse on Tuesday, October 10, 2017 4:14:41 PM
@Drawaes suggested using 429 instead of 503 for our Unhealthy status in the health checks middleware. At the very least, we should consider making it configurable. I've seen systems that use 503, but consul uses 429 (and they are a major use-case for this component).
We should re-examine this before release.
Copied from original issue: aspnet/Diagnostics#409

The set of status codes for health checks are customizable now 👍


add springboot auto complete

When you add an integration with springboot, the redisson configuration item is automatically prompted in the configuration file

@ConfigurationProperties annotation can't be applied to Redisson Config object

Yes that's right, @ConfigurationProperties applies to read Redisson configuration file, but in Intellij editor application.yml file does not automatically prompt for spring.redis.redisson.config the key, to join the above configuration, can automatically prompt. This configuration has been added to the official spring integration, and I hope that redisson-spring-boot-starter can be added as well.


Inputting survey data

Finding convenient way for user to input data from survey data based on the name of the roads or latitude langitude into the traffic network's edges.
Possible solution proposed by @mochrifzal is to convert lat-lang or name into OSMid then input the data.

It's manually inputted


Ability to revert drag/sort if desired from within sortupdate callback

Intercepting "sortupdate" is a great feature if you use it to update a dataset/datastore or do some other permanent/persistent action.  However if you want conditionalize the "finality" of the element placement you can't.  It would be nice to be able to revert the drag and sort from within the "sortupdate" callback if desired.
In my case I want to be able to open a dialog and accept data entry that is contextually specific to the drag/drop/sort operation.  If the user cancels that data entry I want to revert the dragged element, and if they submit the data, persist the elements new location and it's accompanying data.
I can accomplish this "intercept and revert" using jQueryUI's "drop" event handler but only on "draggable/droppable" elements and not the combined "draggable/droppable/sortable" that this plugin provides as I'd like to be able to drag and drop and sort between multiple lists.

Could you not store the original position of the element and the original parent and on a canceled event, just move it back?
I think remembering the original position and parent until the page is refreshed is something we could definitely consider, however at the moment I feel the whole reverting is a little "out of scope" for the plugin.
If you have a solution which does not introduce to much complexity I will gladly see if it can be merged. Should you decide to send a PR, please write a separate function which can be called, as I will be moving towards refactoring the whole plugin to be split into functions for easy testing, extending and refactoring.

@coreypmurphy Actually this should currently be possible this is the current data for sortupdate:
item: dragging,
oldindex: index,
startparent: startParent,
endparent: newParent

You could just bind an event to sortupdate and pull up a dialog, of the user cancels you could use oldindex and startparent to revert the item. Would that work for you or am I missing something here?

Lukas,
Thanks for all your hard work recently. You appear to be really getting after it with this project and it's cool to see that. I'll pull down the latest and see if I can accomplish what you suggested and let you know.
Thanks again,
Corey P. Murphy

On May 20, 2015, at 9:13 AM, Lukas Oppermann<EMAIL_ADDRESS>wrote:
@coreypmurphy Actually this should currently be possible this is the current data for sortupdate:
item: dragging,
oldindex: index,
startparent: startParent,
endparent: newParent
You could just bind an event to sortupdate and pull up a dialog, of the user cancels you could use oldindex and startparent to revert the item. Would that work for you or am I missing something here?
—
Reply to this email directly or view it on GitHub.

@coreypmurphy perfect, thanks. If this can not be accomplished, I would welcome a PR, but please make sure to implement it in a way that will have the least performance influence possible. :)

Hey @coreypmurphy any news on this? Just letting you know, that the api for sortupdate will change a bit once this PR is merged: https://github.com/voidberg/html5sortable/pull/139

@lukasoppermann I've pulled down the latest but I'm still having an issue.  Probably just fried at the end of the day at work but I'm not able to revert it still.  I've set the "index" of the item in question to the "oldindex" and then tried to reload the sortable but it's not reloading it.  Here's the code (minus the markup):
var $item;
var $itemDragged;
var origin;
var destination;
$('.connected').sortable({
connectWith: '.connected'
}).bind('sortupdate', function(e, ui) {
$item = ui;
$itemDragged = ui.item;
origin = ui.startparent[0].id;
destination = this.id;
$("#form_to_complete").show('fade');
});
$("#save").on("click", function() {
doSomeDBStuff(origin,destination,$itemDragged);
$("#form_to_complete").hide();
});
$(".close").on("click", function(){
$item.item.index = $item.oldindex;
$('.connected').sortable();
$("#form_to_complete").hide();
});
Thanks again for all the work and help.  Thoughts?

Forgot to mention that in the code above I was assuming that start and end parent are the same for testing purposes.

Hey @coreypmurphy I did not have time to check it out yet. I will hopefully get to it this weekend.

Hey @coreypmurphy, the sortable is not as advanced as you think. :) You can not just set the index of an item and reload. You would need to do some jQuery stuff:
$('.connected').sortable({
    connectWith: '.connected'
  }).bind('sortupdate', function(e, ui) {
    $item = ui;
    $itemDragged = ui.item;
    origin = ui.startparent;
    $("#form_to_complete").show('fade');
  });
$(".close").on("click", function(){
    // this code is untested, but you will get the idea.
    origin.children(origin.data('items')).get($item.item.oldIndex - 1).after($item);
    $("#form_to_complete").hide();
});

Ah.  Sorry.  I guess I figured it behaved similar to the jQuery "revert" but I guess in this case the element has already beed dropped and the list sorted so I'll have to perform the "revert" manually.  Cool.  I greatly appreciate your time.  Awesome to see you taking ownership of this project.

No worries, if you could share a version of your revert function via jsfiddle, I would greatly appreciate it.

Sure thing. Would be good to see it incorporated as a method into the plugin. I'd gladly send a pull request but I'm extremely swamped.
Corey P. Murphy

On Jun 1, 2015, at 10:11 AM, Lukas Oppermann<EMAIL_ADDRESS>wrote:
No worries, if you could share a version of your revert function via jsfiddle, I would greatly appreciate it.
—
Reply to this email directly or view it on GitHub.

Sure that would be awesome.


Support for UVC on iPhone

According to the documentation (the README.md) the app supports UVC (USB-C) cameras on iPad.
Does it support also UVC (USB-C) on iPhones with USB C?(iPhone 15, 15 Pro, ..).
Is there an Android version/Do you happen to know an alternative to Moblin with UVC support and Network Bonding on Android?
Thank you for the amazing app :)

iPhone do not support USB cameras. Apple limitation. At least that's what I've heard.
The Android app IRL Pro has bonding.

Thank you, that answers my question.


[CU-5wgufp] haproxy

Update ConfigWatcher to allow it to watch for secrets and configmaps with any listOptions filter (label selector, filter selector, etc.)
Add watcher for HAProxy TCP configmap
Add watcher for TLS certificate secrets
Structs for parsing and generating HAProxy config files, loading/validating X509 certificates, and writing files to disk
Watch for update signals to trigger HAProxy soft reload

Task linked: CU-5wgufp standalone dask cluster support


can not translate a  long paragraph

 translator = Translator()
 string ='''Architecturally, the school has a Catholic character. Atop the Main Building's gold dome is a golden statue of the Virgin Mary. Immediately in front of the Main Building and facing it, is a copper statue of Christ with arms upraised with the legend "Venite Ad Me Omnes". Next to the Main Building is the Basilica of the Sacred Heart. Immediately behind the basilica is the Grotto, a Marian place of prayer and reflection. It is a replica of the grotto at Lourdes, France where the Virgin Mary reputedly appeared to Saint Bernadette Soubirous in 1858. At the end of the main drive (and in a direct line that connects through 3 statues and the Gold Dome), is a simple, modern stone statue of Mary'''
 print(string)
  value =translator.translate(string,src='en', dest='zh-CN')
 print(value.text)
The return result is very small:  在建筑上，学校有天主教的品格。
Which is not right. only the first sentence is translated.
Thanks

I guess this issue is related to the existing one #10, currently investigating.

I've figured out the reason. It was because that the result is separated into list, each with a block of sentence. Here's the result for the text above:
[[["在建筑上，学校有天主教的品格。","Architecturally, the school has a Catholic character.",,,3],["在主楼的金色圆顶上是圣母玛利亚的金色雕像。","Atop the Main Building's gold dome is a golden statue of the Virgin Mary.",,,3],["立即在主楼前面，面对它，是一个铜雕像的基督与武器提升与传说“火毒ite Me Me”。","Immediately in front of the Main Building and facing it, is a copper statue of Christ with arms upraised with the legend \"Venite Ad Me Omnes\".",,,3],["主楼旁边是圣心大教堂。","Next to the Main Building is the Basilica of the Sacred Heart.",,,3],["在大教堂后面是石窟，一个玛丽亚的祷告和反思的地方。","Immediately behind the basilica is the Grotto, a Marian place of prayer and reflection.",,,3],["它是法国卢尔德的石窟复制品，圣母玛利亚在1858年被认为是圣伯纳迪特·苏比诺（Saint Bernadette Soubirous）。在主驱动结束时（通过3个雕像和金圆顶连接的直线）是","It is a replica of the grotto at Lourdes, France where the Virgin Mary reputedly appeared to Saint Bernadette Soubirous in 1858. At the end of the main drive (and in a direct line that connects through 3 statues and the Gold Dome), is a",,,3],["简单，现代的玛丽石雕像","simple, modern stone statue of Mary",,,3],[,,"Zài jiànzhú shàng, xuéxiào yǒu tiānzhǔjiào de pǐngé. Zài zhǔlóu de jīnsè yuán dǐng shàng shì shèngmǔ mǎlì yǎ de jīnsè diāoxiàng. Lìjí zài zhǔlóu qiánmiàn, miàn duì tā, shì yīgè tóng diāoxiàng de jīdū yǔ wǔqì tíshēng yǔ chuánshuō “huǒ dú ite Me Me”. Zhǔlóu pángbiān shì shèng xīn dà jiàotáng. Zài dà jiàotáng hòumiàn shì shíkū, yīgè mǎlìyà de dǎogào hé fǎnsī dì dìfāng. Tā shì fàguó lú'ěrdé de shíkū fùzhì pǐn, shèngmǔ mǎ lì yǎ zài 1858 nián bèi rènwéi shì shèng bó nà dí tè·sū bǐ nuò (Saint Bernadette Soubirous). Zài zhǔ qūdòng jiéshù shí (tōngguò 3 gè diāoxiàng hé jīn yuán dǐng liánjiē de zhíxiàn) shì jiǎndān, xiàndài de mǎ lì shí diāoxiàng"]],,"en",,,[["Architecturally, the school has a Catholic character.",,[["在建筑上，学校有天主教的品格。",0,true,false],["在结构上，学校有一个天主教的性格。",0,true,false]],[[0,53]],"Architecturally, the school has a Catholic character.",0,0],["Atop the Main Building's gold dome is a golden statue of the Virgin Mary.",,[["在主楼的金色圆顶上是圣母玛利亚的金色雕像。",0,true,false],["头顶上的主楼的黄金圆顶的圣母玛利亚的金身。",0,true,false]],[[0,73]],"Atop the Main Building's gold dome is a golden statue of the Virgin Mary.",0,0],["Immediately in front of the Main Building and facing it, is a copper statue of Christ with arms upraised with the legend \"Venite Ad Me Omnes\".",,[["立即在主楼前面，面对它，是一个铜雕像的基督与武器提升与传说“火毒ite Me Me”。",0,true,false],["立即在主楼前，它面对的是基督的铜像与传奇“Venite广告我Omnes”伸不缩武器。",0,true,false]],[[0,142]],"Immediately in front of the Main Building and facing it, is a copper statue of Christ with arms upraised with the legend \"Venite Ad Me Omnes\".",0,0],["Next to the Main Building is the Basilica of the Sacred Heart.",,[["主楼旁边是圣心大教堂。",0,true,false],["旁边的主楼是圣心大教堂。",0,true,false]],[[0,62]],"Next to the Main Building is the Basilica of the Sacred Heart.",0,0],["Immediately behind the basilica is the Grotto, a Marian place of prayer and reflection.",,[["在大教堂后面是石窟，一个玛丽亚的祷告和反思的地方。",0,true,false],["立即大殿后面是石窟，祈祷和反思的地方玛丽安。",0,true,false]],[[0,87]],"Immediately behind the basilica is the Grotto, a Marian place of prayer and reflection.",0,0],["It is a replica of the grotto at Lourdes, France where the Virgin Mary reputedly appeared to Saint Bernadette Soubirous in 1858. At the end of the main drive (and in a direct line that connects through 3 statues and the Gold Dome), is a",,[["它是法国卢尔德的石窟复制品，圣母玛利亚在1858年被认为是圣伯纳迪特·苏比诺（Saint Bernadette Soubirous）。在主驱动结束时（通过3个雕像和金圆顶连接的直线）是",0,true,false],["这是法国卢尔德的石窟圣母玛丽亚据说出现了圣伯纳迪特·索比劳斯于1858年在主驱动器的末尾的一个副本（在通过3个塑像和黄金圆顶连接直通线），是一个",0,true,false]],[[0,236]],"It is a replica of the grotto at Lourdes, France where the Virgin Mary reputedly appeared to Saint Bernadette Soubirous in 1858. At the end of the main drive (and in a direct line that connects through 3 statues and the Gold Dome), is a",0,0],["simple, modern stone statue of Mary",,[["简单，现代的玛丽石雕像",0,true,false],["玛丽的简单，现代的石像",0,true,false]],[[0,35]],"simple, modern stone statue of Mary",0,0]],0.99607933,,[["en"],,[0.99607933],["en"]]]

I will fix this soon with a new release.


improvement should send query timeout status to be, instead of internal error

Proposed changes
If a query is cancelled, the reason is very unclear and we do not know the call stack.
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Trackpad: Tab switch on mouse scroll: horizontal scrolling sensitivity too high

Refs: https://github.com/microsoft/vscode/issues/95952

"workbench.editor.scrollToSwitchTabs": natural
Have a mouse with horizontal press
Horizontal scroll is too senstivie in the tab area and seems pretty useless due to that

Vertical scroll works just fine. And horizontal scroll in the editor has good sensitivity so maybe you could check with Alex how he handles the horizontal scroll.

@isidorn I assume you tested this from the balcony where you have no external mouse attached but the macOS touchpad?

@bpasero no it is raining, and I am on the balcony only for the standup.
This is with an external mouse. The MS one. The small scroll can be pressed left and righ.

Up for help: the code that checks for the wheel event is here:
https://github.com/microsoft/vscode/blob/ca0af15b543f83f48a80f2644272718d5a327542/src/vs/workbench/browser/parts/editor/tabsTitleControl.ts#L325-L358

To add my specifics here from #97265, I am on a Mac and using a Mac "Magic Mouse" that mimics a trackpad for horizontal and vertical scrolling.  Both of those work as expected within an editor window, but when using horizontal scroll in the tabs, it shoots instantly to the last tab when scrolled right and instantly to the first tab when scrolling left.

@bpasero Maybe it's obvious but the issue looks like a tab change is triggered every single scroll event. It just needs throttled with at least 100 ms threshold. There's already a decorator in common:
https://github.com/microsoft/vscode/blob/e0639de66595b5e4e41c9b3d0116f865237ff8a5/src/vs/base/test/common/decorators.test.ts#L167-L174

@texastoland happy to test this out in a PR. I gave debouncing a very brief try and felt it was not working, but I was not using that annotation but something else.

I'll give it a shot 🤙🏽

I'm having the opposite problem when using the mouse scroll wheel. When I scroll, there's a delay before it switches. In other applications like browsers it switches immediately. And even if I use keyboard shortcuts in vscode to switch tabs they switch immediately, so something is different about the scroll behavior.

Hi, I would like to try solving this problem. I am from Grace Hopper OSD

/verified 👏🏼

@texastoland thanks


feat: add redshift driver

Related SQLx fix for Redshift support: https://github.com/launchbadge/sqlx/pull/2832


cant set new datasource to Graph Database Support

after instalation (and restarting the ide 'datagrip')
im trying to create new datasource but there is not the "Graph Database" driver on the  drivers list

i got it can close

@yaronmil
Hi. Looks like that you already found it :)
Just to clarify - IntelliJ platform does not allow to extend their default database support functionality.
Therefore I was forced to create my own from scratch.
Feel free to drop any questions/comments/bug report if you found them! :)


going back to s3-sync-action

@svengabr
Hey Sven, we saw many PRs regarding the Deploy mechanism, could you please elaborate a bit why you've done and why?
To me it seems like deployment performance was improved with these changes, or?


How to export model for another look?

Hello,
I have used MLTD tools and it is very helpful for me.
I tried to export Mizuki's SSR005 another look but received only error message such as "Cannot load body avatar."
I used Head and Body as below and faced error message.
Head: ch_ss105_044miz.unity3d
Body: cb_ss105_044miz.unity3d
When I used Head and Body as below, no error message and can load nicely exported model.
Head: ch_ss005_044miz.unity3d
Body: cb_ss005_044miz.unity3d

Does anyone know how to export its another look model?
I really appreciate if anyone can give me guidance.

this is because the files with "ss105" are just an image file which is the recolor outfit (it does not contain any model inside it, thats why you get error when converting), you can extract it with AssetStudioGUI and replace the textures exported with the "ss005" files in the "tex" folder

this is because the files with "ss105" are just an image file which is the recolor outfit (it does not contain any model inside it, thats why you get error when converting), you can extract it with AssetStudioGUI and replace the textures exported with the "ss005" files in the "tex" folder

Thank you for reply.
Where can I get AssetStudioGUI ??
And is there any manual for it? if so, it will be very helpful.(cause I don't have any knowledge for unity system)


[Inference/Kernel] refactor kvcache manager and rotary_embedding and kvcache_memcpy oper…

…ator
📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[ ] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ ] I have added relevant tags if possible for us to better distinguish different PRs
[ ] I have installed pre-commit: pip install pre-commit && pre-commit install

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[ ] I have performed a self-review of my code
[ ] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[ ] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

Since manual construct Alibi PE will introduce unnegligible precision error, we choose to pass randomly generated ones in testcases, and you may refer to some Alibi-related discussions
https://github.com/vllm-project/vllm/issues/1009
https://www.infoq.cn/article/ocjyhximvswg4o5rbodb
https://github.com/vllm-project/vllm/pull/945
https://github.com/vllm-project/vllm/pull/1022

May be a end2end cost time comparison should be given to prove that new layout and decoding can improve the performance in case of having a negative impact on other kernels caused by new layout


Integrate the sentry for the error logging.

Until we aren't doing OSS we can integrate sentry for the backend application.

Done in https://github.com/chaos-genius/chaos_genius/pull/357


Impossible to override WebClient.clientConnector if ReactiveOptions not null

Bug
Since 3.0.0 it become impossible to override clientConnector in WebClient builder if ReactiveOptions are exists. At least I couldn't find way to do it, will be happy to be wrong.
Example:
@ReactiveFeignClient(
    value = "some-api",
    configuration = [TestConfigurationWithConnector::class]
)
interface TestClient {

    @GetMapping("/")
    fun getSomeInfo(): Mono<Any>
}

class TestConfigurationWithConnector {
    @Bean
    fun webClientCustomizer(): WebClientFeignCustomizer {
        val insecureSslContext = SslContextBuilder.forClient()
            .trustManager(InsecureTrustManagerFactory.INSTANCE)
            .build()

        val httpClient = HttpClient.create().secure { it.sslContext(insecureSslContext) }

        return WebClientFeignCustomizer { builder ->
            builder.clientConnector(ReactorClientHttpConnector(httpClient))
        }
    }
}

What happening under the hood during bean initialization:

Invoked WebReactiveFeign.builder(WebClient.Builder webClientBuilder,  webClientCustomizer), in result - we have WebClient with our connector. At this moment everything is ok.
Invoked ReactiveFeignBasicConfigurator.configureUsingProperties, at the start of this method - ReactiveFeignBuilder still contains out HttpConnector.
In ReactiveFeignBasicConfigurator.configureUsingProperties method, if optionsBuilder != null - invoked resultBuilder.options(optionsBuilder.build()) function:

ReactiveOptions.Builder optionsBuilder = config.getOptions();
if(optionsBuilder != null){
	resultBuilder = resultBuilder.options(optionsBuilder.build());
}

Inside of WebReactiveFeign.options(..) method - we have:
  @Override
  public Builder<T> options(ReactiveOptions options) {
      webClientBuilder.clientConnector(
              buildNettyClientHttpConnector((WebReactiveOptions)options));
      updateClientFactory();
      return this;
  }

Here we again set clientConnector to webClientBuilder and override the previous value.
Disclaimer:
I know, that since 3.2.3 it become possible to add flag disableSslValidation to allow insecure SSL context, but this bug also affects all other params, which can be set in clientConnector.
Hoping that i wrong and there is still some way to provide clientConnector, if so - please advise me.
Thank you.

try to use WebReactiveFeign.Builder.httpClient(HttpClient httpClient)


More world data

Could we add all the game rules and available world data into ServerTap? Routes to update game rules would be nice too.
https://nodecraft.com/support/games/minecraft/using-gamerules-in-minecraft

Sounds great! I'll be around, so I can totally review your code. Looking forward to it!

@srmullaney Any updates on this?

Going to close this since it's over 2y old!


[Bug]: Cant get return out of page.on, returns undefined

Bug description
Steps to reproduce the problem:
page.on return not works :c
import puppeteer from "puppeteer-extra";
import StealthPlugin from 'puppeteer-extra-plugin-stealth'

async function ProductAPI() {
        puppeteer.use(StealthPlugin())
        const browser = await puppeteer.launch({headless: true});
        const page = await browser.newPage();

        page.on('response', async (response) => {
            if(response.url().includes('some_link_here')){
            console.log('XHR response received');
            const HTMLdata = await response.text()
            console.log(HTMLdata) // If you just try to display the log, then everything works, but return returns undefined
            return "fine!" // Not works!

        };});

        await page.goto('some_link_here');

}
await ProductAPI().then((data) => {
    console.log(data)});

console.log(data) gives undefined, instead of "fine!", I tried different options, but I can't get the necessary return.
Puppeteer version
3.2.2
Node.js version
16.18.0
npm version
8.19.2
What operating system are you seeing the problem on?
Windows
Relevant log output
No response

You need to use a promise that resolves when the page response is fine. You need to wrap it with a promise constructor like so:
import puppeteer from 'puppeteer-extra';
import StealthPlugin from 'puppeteer-extra-plugin-stealth';

async function ProductAPI() {
  puppeteer.use(StealthPlugin());
  const browser = await puppeteer.launch({headless: true});
  const page = await browser.newPage();

  return new Promise(resolve => {
    page.on('response', async response => {
      if (response.url().includes('some_link_here')) {
        console.log('XHR response received');
        const HTMLdata = await response.text();
        console.log(HTMLdata); // If you just try to display the log, then everything works, but return returns undefined
        resolve('fine!');
      }
    });

    page.goto('some_link_here');
  });
}
await ProductAPI().then(data => {
  console.log(data);
});

You need to use a promise that resolves when the page response is fine. You need to wrap it with a promise constructor like so:
import puppeteer from 'puppeteer-extra';
import StealthPlugin from 'puppeteer-extra-plugin-stealth';

async function ProductAPI() {
  puppeteer.use(StealthPlugin());
  const browser = await puppeteer.launch({headless: true});
  const page = await browser.newPage();

  return new Promise(resolve => {
    page.on('response', async response => {
      if (response.url().includes('some_link_here')) {
        console.log('XHR response received');
        const HTMLdata = await response.text();
        console.log(HTMLdata); // If you just try to display the log, then everything works, but return returns undefined
        resolve('fine!');
      }
    });

    page.goto('some_link_here');
  });
}
await ProductAPI().then(data => {
  console.log(data);
});

Thank you very much!


SSI - Multi Wallet Support (ext: CXAR-1048)

+Benefit Hypothesis+
Concept and reference implementation needed to enable muti-wallet support.
This means that inside the network "bring your own wallet" need to get enabled. The wallets can be own hosted but also managed.
It needs to be ensured that different wallet "types" are supported (onboarding, maintenance, issue credentials, revert credentials, etc....)

+Dependencies+
This feature depends on the SSI strategy which is currently getting implemented/designed as part of PI10 and PI11.

+Beneficiaries+
< Who will benefit for that feature>

Components: CPLP, EDC, MIW
Sprints: N/A
Fix Versions: N/A
StoryPoints: N/A
Attachments: N/A

closed - tickets are managed and created on sig-release
closing this ticket


Error status message

Running several tests (locally) I've come up with several different errors in the console, but the message on the UI seems to always be the same : "Invalid website".
I think it should reflect the actual error thrown. For example:
Traceback (most recent call last):
  File "worker_predict.py", line 78, in <module>
    work(args.baseline_dir, args.updated_dir, args.prefix)
  File "worker_predict.py", line 54, in work
    updated_network_log_file)
  File "/Users/sdumitre/work/frontend-regression-validator/fred/process/process.py", line 8, in __init__
    self.baseline_js_logs = JavascriptLogPreprocessor(baseline_js_log_file)
  File "/Users/sdumitre/work/frontend-regression-validator/fred/data/log_preprocess.py", line 7, in __init__
    self.logs = json.load(open(log_file, 'r'))
FileNotFoundError: [Errno 2] No such file or directory: './tmp/1_baseline/buttons_1_js_log.json'

gives the Status: Invalid website message. Guess this is a low priority issue, but it for non-coder users this error message is confusing.

no longer relevant, we're updating soon to a v2, closing.


Don't require passing concrete HashMap type

Describe the bug
ApiClient#configureApiKeys should accept a Map instead of requiring HashMap.
Environment and Versions (please complete the following information):
A clear and precise description of your setup:

Version: 1.0.0-beta.7

I cannot follow the instruction "Add severity/ label." as the labels are not editable.

Hey @rehevkor5 thanks for the issue report. I've opened a PR upstream to the client generator to switch to allowing this method to accept the Map interface. https://github.com/OpenAPITools/openapi-generator/pull/7982

@rehevkor5 In the meantime, I've updated the client directly to no longer require the concrete Map type https://github.com/DataDog/datadog-api-client-java/pull/540
I'll go ahead and close this issue out, but let us know if you experience any further issues here.


Reuse existing ES connection from Enhancement

I have converted my python program to ElastAlert Enhancement module successfully.
It enriches every match with additional information from other events stored in ES by creating a new connection and querying the cluster.
I was thinking if there is a way to leverage already existing ES connection from the ElastAlert in the enhancement?

Hi @admlko, unfortunately not. The connection object is basically just a bunch of configuration though, so it's not any less efficient to create a new Elasticsearch client instance.
You probably are already, but if you aren't, you should use elastalert.util.elasticsearch_client, which should make it super simple to create a new client instance.


Flatten exports from CAIP packages

Overview
Use flattened exports from the outcome of https://github.com/shapeshift/lib/issues/548
References and additional details
Like https://github.com/shapeshift/web/pull/1591, but for unchained.
Acceptance Criteria
Functions like toCAIP19 or fromCAIP19 are imported directly from the CAIP packages.
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Now tracked in lib: https://github.com/shapeshift/lib/issues/619


ui-payment > build > error

cd ui-payment
yarn install

[1/4] Resolving packages...
............................

$ husky install
husky - .git can't be found (see https://typicode.github.io/husky/#/?id=custom-directory)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Hey! Could you describe the use-case for building custom frontend?

Does it matter?

Yep, we want to improve developer experience by collecting use-cases outside of default setup (i.e. running via published docker image).
Regarding your question, you can follow gh action script to reproduce the same build:

Put desired vars in .env
Run npm ci --ignore-scripts
Run npx tsc && npx vite build --base='/p/'

 ▸ ~/Code/o2pay/oxygen/ui-dashboard ▸ git:develop ▸ nvm use
Found '/Users/abc/Code/o2pay/oxygen/ui-dashboard/.nvmrc' with version <16.13.0>
Now using node v16.13.0 (npm v8.1.0)
 ▸ ~/Code/o2pay/oxygen/ui-dashboard ▸ git:develop ▸ npm ci --ignore-scripts

added 548 packages, and audited 549 packages in 31s

137 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
npm notice
npm notice New major version of npm available! 8.1.0 -> 10.1.0
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.1.0
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
 ▸ ~/Code/o2pay/oxygen/ui-dashboard ▸ git:develop ▸ npx tsc && npx vite build --base='/p/'
vite v4.4.7 building for production...
✓ 6482 modules transformed.
dist/assets/tron-5b6b6483.svg            0.43 kB │ gzip:   0.31 kB
dist/assets/eth-1f94df85.svg             0.53 kB │ gzip:   0.28 kB
dist/assets/arrow-up-d1a531a7.svg        0.76 kB │ gzip:   0.39 kB
dist/assets/arrow-down-ea43cf9f.svg      0.77 kB │ gzip:   0.39 kB
dist/assets/busd-a6ab749c.svg            0.86 kB │ gzip:   0.47 kB
dist/index.html                          0.90 kB │ gzip:   0.43 kB
dist/assets/matic-479d03ee.svg           1.02 kB │ gzip:   0.48 kB
dist/assets/usdt-337d36c2.svg            1.07 kB │ gzip:   0.52 kB
dist/assets/bnb-d068cbf5.svg             1.61 kB │ gzip:   0.73 kB
dist/assets/usdc-913adf09.svg            1.70 kB │ gzip:   0.70 kB
dist/assets/index-7de3f4c4.css           4.89 kB │ gzip:   1.41 kB
dist/assets/tron-167e02ed.js             0.62 kB │ gzip:   0.43 kB
dist/assets/eth-59de992b.js              0.81 kB │ gzip:   0.40 kB
dist/assets/arrow-up-49a1cfc5.js         0.92 kB │ gzip:   0.52 kB
dist/assets/arrow-down-ba5fc5fb.js       0.94 kB │ gzip:   0.51 kB
dist/assets/busd-88859388.js             1.02 kB │ gzip:   0.55 kB
dist/assets/matic-7f5c3fd9.js            1.21 kB │ gzip:   0.60 kB
dist/assets/usdt-a3321d3c.js             1.31 kB │ gzip:   0.64 kB
dist/assets/bnb-cb17ca65.js              1.69 kB │ gzip:   0.74 kB
dist/assets/usdc-6c2da4e2.js             1.89 kB │ gzip:   0.83 kB
dist/assets/index-78303a53.js        1,565.97 kB │ gzip: 489.28 kB

(!) Some chunks are larger than 500 kBs after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/configuration-options/#output-manualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.
✓ built in 7.61s
 ▸ ~/Code/o2pay/oxygen/ui-dashboard ▸ git:develop ▸

@naddika I want to warn that you attitude is a bit disrespectful.


Resolve responsiveness issues

![Uploading Screen Shot 2016-11-21 at 10.54.49 AM.png…]()

addressed by the redesign.


Advanced REST client file upload issue

File data is not posting to the server from the latest update, this bug is not there in old version.
Expected outcome
File should be posted to the server url like as form submit.
Actual outcome
File data is not posting to the server in Content-Type: multipart/form-data
Versions
App: <IP_ADDRESS>5-stable
Chrome: 56.0.2924.87
Platform: Windows (7)
Steps to reproduce

Use post while submitting form.
Use enctype="multipart/form-data"

Hi,
Thank you for issue report.
It is not issue with the app but with Chrome.
Please refer to #945 for more information.
I am closing this as a duplicate.


Filter Icon

How can I change that Filter Icon on the right-hand side? And I want to remove some Items in the last filter. Is it possible in MUI Datatable??

Tech
Version

mui-datatables

node
8

OS
Mac

There's no option as of today to change the icons. That would be a feature request. Let me look into that.
Second issue, removing some items in the last filter. Can you show me a screenshot of what you're talking about ?

Please see the screenshot. I want to remove salary from the filter. But I need that salary on the table. Not in the filters. Is there any chance???

Ok I think this is a great idea. It goes along with some other changes users have requested. Currently I'm on vacation so I will not be able to get to this rigjt away but I will look to add this feature for more customization.

So I have some initial work done that will allow for customization of columns. It'll also address #3 when it's complete. As it stands now you'll be able to define which columns you want filtering for or not.  But, as far as the icon replacement here's what I'm proposing: You'll be able to swap in custom icons but they must be React Components that return SVG much like how material-ui-icons works currently

@krishnaUIDev In version 1.0.8 I've added the ability to customize columns. You can turn the filter on/off per column basis

Thanks @gregnb. And if you have time can you look at removing columns from filters.

@krishnaUIDev yea that's what I just posted lol. You can remove them now in version 1.0.8. Take a look at the new examples folder

When click on Filter Icon or view column icon its getting error @gregnb

@krishnaUIDev looks like a warning. I think the current PR fixes it when we merge it in. But nothing is broken, correct ?

Ya Nothing is broken and can you try to add functions to the rows (onClick events). Is there any chance to add onclick events to that rows? ?

Ok, but I want to learn more about why. What's the use case first to binding an onClick event to the rows? Are you tracking something? Just curious

I am trying to create a data table site. At that place, i need to click handlers for rows to show that row data.

@krishnaUIDev Ok, I will try to work on expanding more options into this package. I'm still on vacation so nothing will get done right away so bare with me. If it makes sense after I look into it some more I'll put it on the feature board.  Some people have also asked for ability for selectable rows which sounds close to what you're asking for

Selectable rows have been added in the latest version. It has two event callbacks that should help with this ticket.

Is there any new features??

Yes, in the latest version there are selectable rows and two callbacks you can read the README for more information

Thanks. And is there any chance to change font icons??

@krishnaUIDev I could do something on the column options. A property called 'icon' so the options available would look like:
const columns = [
 {
  name: "Name",
  options: {
   filter: true,
   sort: false
   icon: <some icon from material-ui-icons only>
  }
 },
 ...
];

But again, I would only allow icons from material-ui-icons

Nevermind that, I wrote about the wrong section this is related to toolbar.  Let me think about this some more

So I'm trying to find a good reason to do this but I'm struggling. The only icons I would allow are ones to be loaded from material-ui-icons package.  Take for example print, if you search https://material.io/icons/ there are only two available.  Were you going to try to load in your own SVG's?

hi @gregnb.
First of all, thank you very much for the component. I am trying to change the search icon. I am trying to load my own SVG.
Is there any functionality to do this?
regards
luis

Did any of you guys have any luck by changing the icons svg?

hi @gregnb.
First of all, thank you very much for the component. I am trying to change the search icon. I am trying to load my own SVG.
Is there any functionality to do this?
regards
luis

Is it still true that you can't conveniently change the filter button?
If it's helpful at all, I've managed to add text with some brutal CSS along the lines of
[data-testid^="Filters"]::after {
  content: "Filters"
}

And you could even hide the SVG and replace it with a background-image.


About the camera map

Hello. I feel a little concerned about your "camera map".
As illustrated in your paper, the camera map contains the 3-dim camera parameters (s, tx , ty ) that describe the 2D scale s and translation t = (tx , ty ) of the person in the image. The scale s reflects the body size and the depth to some extent. tx and ty , ranging in (−1, 1), reflect the normalized translation of the human body relative to the image center on the x and y axis, respectively.
From my personal perspective, the camera map may contain 6 parameters: rotation and translation of 3 axis, which comprises the extrinsic parameters of the camera position. Maybe I may recognise the parameter s as the translation of z dimension, but I am wondering that how can we obtain the rotation of the xyz dimension?

For ROMP, we integrate the absolute rotation of camera and the absolute rotation of each person into the relative 3D rotation of person root  to the camera. The estimated 'global_orent' is the the relative 3D rotation of person to the camera. You may assume that the camera is always pointing to the negative direction of z axis, while the person rotates.

Thanks for your quick response. I may study deeper to understand it..
I saw that the output of the model contains a keyword "joints_smpl24". But it contains a tensor with a size of 453 instead of 243, which is different from other work. May I ask what's the meaning of the remaining 21*3 vector?

Sorry for the delay. The rest 21 joints are 5 face joints + 5 right hand joints + 5 left hand joints + 3 left foot joint + 3 right foot joints. Please refer to this dict for the details

That's all right. Thanks for your explanation!


[DO NOT MERGE] Test PR

Description of your changes

Fixes #
I have:

[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

/test-examples="examples/eks/v1beta1/cluster.yaml"

/test-examples="examples/eks/v1beta1/cluster.yaml"


Resolves #124 by updating class search for enum constants

This PR resolves #124 by updating class search algorithm with prioritizing enum constants while doing a tree traversal. This will help injector to assign correct flat names to classes in a tree by visiting enum constants before other existing anonymous classes.

@lazaroclapp Please let me know if I should land this, thank you very much.

@nimakarimipour you can go ahead and land this now, I addressed @lazaroclapp's comment


Regression 3.4.0+: scenario where Tuple.InverseMap does not compute the expected type

Compiler version
3.4.0 and higher
Minimized code
//> using scala 3.4.0

trait Column:
  type T
  type F[X]
  type Q = F[T]

class varchar extends Column:
  type T = String

trait notnull extends Column:
  type F[X] = X

object Error:

  val firstName = new varchar with notnull
  val lastName = new varchar with notnull

  val relation = (firstName, lastName)

  type RelationTypes = Tuple.InverseMap[relation.type, [X] =>> Column { type Q = X }]

  summon[RelationTypes =:= (String, String)]

object Works:

  object firstName extends varchar with notnull
  object lastName extends varchar with notnull

  val relation = (firstName, lastName)

  type RelationTypes = Tuple.InverseMap[relation.type, [X] =>> Column { type Q = X }]

  summon[RelationTypes =:= (String, String)]

Output
Compiling project (Scala 3.4.0, JVM (8))
[error] .\test.scala:23:45
[error] Cannot prove that Error.RelationTypes =:= (String, String).
[error]   summon[RelationTypes =:= (String, String)]
[error]                                             ^
Error compiling project (Scala 3.4.0, JVM (8))
Compilation failed

Expectation
Successful compilation.

Regression happens in 3.4.0-RC1-bin-20231218-ec2b8bc-NIGHTLY:
Compiling project (Scala 3.4.0-RC1-bin-20231218-ec2b8bc-NIGHTLY, JVM (8))
[error] .\test.scala:23:45
[error] Cannot prove that Error.RelationTypes =:= (String, String).
[error]   summon[RelationTypes =:= (String, String)]
[error]                                             ^
Error compiling project (Scala 3.4.0-RC1-bin-20231218-ec2b8bc-NIGHTLY, JVM (8))
Compiling project (Scala 3.4.0-RC1-bin-20231214-423cd6c-NIGHTLY, JVM (8))
Compiled project (Scala 3.4.0-RC1-bin-20231214-423cd6c-NIGHTLY, JVM (8))

Minimized to:
trait A:
  type T
  type U = T

trait B extends A:
  type T = String

object C extends B

type F[X] = A { type U = X }

type InvF[Y] = Y match
  case F[x] => x

object Test:
  summon[InvF[C.type] =:= String] // ok
  summon[InvF[B] =:= String] // error: selector B does not uniquely determine parameter x

although I'm not 100% sure this an incorrect error.

After investigating #17222, these two issues are probably unrelated

This issue is fixed by #20161: adding import language.experimental.betterMatchTypeExtractors enabled to compile both the @EugeneFlesselle's minimization @kavedaa's original snippet.
Adding the import does not fix #17222.
@smarter would you have any insights on when the amendment will be stabilized?
Should I just add recession tests and mark this issue as solved?

It needs a minor release so it should be in 3.6.0

Wonderful, so this means these examples will compile just fine in 3.6!
I suggest we add regressions tests with import language.experimental.betterMatchTypeExtractors and close this issue already. When 3.6 is released, we can remove the imports in the tests. Sound okay?

I suggest we add regressions tests with import language.experimental.betterMatchTypeExtractors

IIRC tests usually have scala-cli directives for experimental options but sounds fine to me either way


Refactor Code

Sorry @Maan2003 , I can't merge this pull request because I was recently working on changes to the code. Thanks for it, though.


put_assoc works with whatever Ecto.Changeset passed

Environment

Elixir version (elixir -v): Elixir 1.4.5
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): psql 9.6.3
Ecto version (mix deps): 2.1.4
Database adapter and version (mix deps): postgrex 0.13.3
Operating system: Ubuntu 17.04

Current behavior
I have an Event schema:
defmodule Tickets.Ticket do
  schema "tickets_events" do
    field :name, :string
    timestamps()
    belongs_to :creator, {"tickets_users", User}
  end

  def changeset(%Event{} = event, attrs) do
    event
    |> Repo.preload(:creator)
    |> cast(attrs, [:name])
    |> validate_required([:name])
    |> put_assoc(:creator, attrs.creator)
  end

and  an User schema which should be an creator:
defmodule Tickets.User do
  schema "tickets_users" do
    field :nickname, :string
    timestamps()
    has_many :created_events, {"tickets_events", Event}, foreign_key: :creator_id
  end

so associating a User to a new Event works right
attrs = %{name: "some_name", creator: get_user!(1)}
create_event(attrs)
# =>
# {:ok,
#  %Tickets.Event{
#    __meta__: #Ecto.Schema.Metadata<:loaded, "tickets_events">,
#    id: 1,
#    creator: %Tickets.User{__meta__: #Ecto.Schema.Metadata<:loaded, "tickets_users">,
#     ...}}

but this also works
attrs = %{name: "some_name", creator: get_event!(1)}
create_event(attrs)    # passed a %Event{} as a :user
# =>
# {:ok,
#  %Tickets.Event{
#    __meta__: #Ecto.Schema.Metadata<:loaded, "tickets_events">,
#    id: 2,
#    creator: %Tickets.Event{__meta__: #Ecto.Schema.Metadata<:loaded, "tickets_events">,
#     ...}}

The created Event's creator will be User(id=1), which seems to be just taking the id from the given %Tickets.Event{} .
Expected behavior
Shouldn't put_assoc return an error?
attrs = %{name: "some_name", creator: get_event!(1)}
create_event(attrs)    # passed a %Event{} as a :user
# => {:error, #Ecto.Changeset<errors: [creator: {"type is wrong", [validation: :assoc]}]...>}

We should probably raise instead of returning a validation error, though. I can't imagine a situation when reaching this would be anything else than a programmer error.


experiment with minor change to CommGroup definition to see if it speeds stuff up

Probably not a route we want to go down, but worth looking at. It's a 1% reduction in instructions, with massive speedups and slowdowns in lots of places.

!bench

!bench

!bench

Note: I have pushed an update to the lean toolchain because this PR was on a buggy version of the toolchain. WARNING: checking out old commits of this PR using v4.2.0-rc2 or v4.2.0-rc3 can cause lake clean to delete your mathlib folder! If you need to do so, make sure to delete lakefile.olean manually before running any lake commands.

This PR has quite a few merge conflicts now. Is it still relevant, or should we close it?

It's no longer relevant


keosd process exits if failed to bind to port #3314

Be aware this causes a change in behavior to nodeos too -- it will shutdown on http/https start failure. This seems... okay enough to change at this point.


up/down/forward/back

function([Blocks=1])
Returns:  true - if it goes all the way.
false - if Blocks is not a number.
- if it couldn't complete all the moves.
-it should return nil if Blocks is not a number.

Resolved


serving/xrefs: short-circuit return when no serving data is found [phab:D1922]

Created by schroederc at 2017-09-05 16:15:17:

zarko accepted this revision at 2017-09-05 16:18:12

schroederc closed via commit 60f264291477b1e750e4b5d11cb4935c1bf740e1 at 2017-09-05 16:20:41


Moving a package declaration from straight.el recipe pointing to repo to pull from MELPA

Is there a documented way to adjust a package when it moves from git repo hosting to being listed on MELPA?
Specifically, I have a straight.el recipe which installed org-roam from GH repo when that was the only package hosting available. It has since moved to being part of MELPA and I'd now like to adjust my package declaration to point to MELPA however I don't see any documentation as to whether or not this would require any adjustments to striaght.el and/or the build cache beyond just removing the recipe and using the use-package integration.
Thx.

 It sounds like you already knew this though; were you concerned there might be additional steps required?

Not at all. The above was helpful. thx

Do you have any suggestions about what could be added or changed in the documentation of straight.el so that the answer to this question would have been easier to find?

Thanks for asking.
I was working from the README in the root of the repository. It's quite comprehensive.
I doubt that everyone learns/works the same way but over the past few years what I've noticed is that I learn fastest and get to the point of self-sufficiency fastest when I first view "getting started" videos on a topic and then start to look at the documentation after my usage has gone beyond beginner phase. I call this "fractal learning" for lack of a better term. straight.el mostly just works (I use the use-package integration) and thus I've never invested in reading through the docs for depth as opposed to just trying to find answers to very specific questions.
So maybe as penance for asking too many questions ( ;) ), I should create some "getting started" videos on straight.el usage a la:

https://www.youtube.com/watch?v=IofHvutUWV0
https://www.youtube.com/watch?v=Gk9-q8tXbMs

This thread is being closed automatically by Tidier because it is labeled with "waiting on response" and has not seen any activity for 90 days. But don't worry—if you have any information that might advance the discussion, leave a comment and I will be happy to reopen the thread :)


String columns that start with NaN are shown as all NaNs

String columns in a Pandas dataframe are shown correctly when the first row's value is a string, rather than a NaN. When the first value is NaN, all subsequent values are shown as NaNs. Each statement below is an observation that's either shown as table output or as text from a print statement. These observations seem to show the rich table output is the culprit, and the problem only affects string columns.
good = pd.DataFrame({'a': ['A',np.nan]})
good # correct
good['a'] # correct
print good # correct

bad =  pd.DataFrame({'a': [np.nan,'A']})
bad # both values are NaNs
bad['a'] # correct (showing a series, rather than a dataframe, seems to work)
print bad # correct (print statements also work)

both = pd.DataFrame({'a': ['A',np.nan], 'b': [np.nan, 'A']})
both # column that starts with NaN shows all Nans
print both # correct

numeric = pd.DataFrame({'a': [10,np.nan], 'b': [np.nan, 10]})
numeric # correct (numeric data works fine)

confirmed: https://pub.beakernotebook.com/#/publications/562ebfec-8ae3-49d3-979c-31f22588abf4

You're absurdly fast. Thanks for being so responsive Scott.

:) thanks -- fast at confirming bugs, that is :P


Feature: warn if env variables assignment is skipped due to process level env

Is your feature request related to a problem? Please describe.
Related to https://github.com/netlify/cli/issues/1596
When env variables assignment is skipped due to process level env, it would be nice to add a log message telling the user that.
Describe the solution you'd like
A log warning when dotenv/site level/addons envs are not assigned due to process level envs.
Describe alternatives you've considered
Manually verify the process level envs don't overlap with other ones
Additional Context
The place to implement this is https://github.com/netlify/cli/blob/d8ff3b77d9c1a5f153a956b382d3a3ebaf7fabee/src/utils/dev.js#L109
Can you submit a pull request?
Yes

If I understand correctly, this is the order of precedence for environment variables:

Process level
.env files
Site environment
Addons environment
Team environment

In this issue we're proposing to warn the user whenever a variable found in 1 overrides a variable with the same name found in any other stage. I was thinking we could expand this and warn the user whenever a variable is overridden at any stage.
For example, take a variable defined at the team and site levels. Currently, we simply tell the user that we inject the site-level environment, omitting the fact that it took precedence over the team one. I suppose this might be confusing to some people.
We could show something like:
◈ Skipped shared build setting env var: NETLIFY_TEST_VAR (defined in build setting)
◈ Injected build setting env var: NETLIFY_TEST_VAR

As for the specific case described in the issue, it would look something like:
◈ Skipped shared build setting env var: NETLIFY_TEST_VAR (defined at process level)

@erezrokah do you think this would be useful?

If I understand correctly, this is the order of precedence for environment variables:

Process level
.env files
Site environment
Addons environment
Team environment

In this issue we're proposing to warn the user whenever a variable found in 1 overrides a variable with the same name found in any other stage. I was thinking we could expand this and warn the user whenever a variable is overridden at any stage.
For example, take a variable defined at the team and site levels. Currently, we simply tell the user that we inject the site-level environment, omitting the fact that it took precedence over the team one. I suppose this might be confusing to some people.
We could show something like:
◈ Skipped shared build setting env var: NETLIFY_TEST_VAR (defined in build setting)
◈ Injected build setting env var: NETLIFY_TEST_VAR

As for the specific case described in the issue, it would look something like:
◈ Skipped shared build setting env var: NETLIFY_TEST_VAR (defined at process level)

@erezrokah do you think this would be useful?

If I understand correctly, this is the order of precedence for environment variables:

Process level
.env files
Site environment
Addons environment
Team environment

That's correct.

@erezrokah do you think this would be useful?

I think this is a great idea! It will help clear up a lot of confusion we have around env variables handling.

If I understand correctly, this is the order of precedence for environment variables:

Process level
.env files
Site environment
Addons environment
Team environment

That's correct.

@erezrokah do you think this would be useful?

I think this is a great idea! It will help clear up a lot of confusion we have around env variables handling.

What do you think of this? In this case, I've defined NETLIFY_TEST_VAR in four places (.env file, .env.development file, team-level environment, and site-level environment) and TWITTER_SCREEN_NAME in two (site-level environment and process level).
I'm keeping the colour-coding scheme and dimming the "ignored" messages to convey some sense of hierarchy.
Does this look sensible or does it feel too cluttered?
(🔔  @ehmicky for good measure)

What do you think of this? In this case, I've defined NETLIFY_TEST_VAR in four places (.env file, .env.development file, team-level environment, and site-level environment) and TWITTER_SCREEN_NAME in two (site-level environment and process level).
I'm keeping the colour-coding scheme and dimming the "ignored" messages to convey some sense of hierarchy.
Does this look sensible or does it feel too cluttered?
(🔔  @ehmicky for good measure)

That looks very nice! I would like to see it on a white background though, but very nice :)

That looks very nice! I would like to see it on a white background though, but very nice :)

I would like to see it on a white background though

Here it is:

I would like to see it on a white background though

Here it is:

🚀

🚀


Uhhhhh I'm not really sure about what I'm supposed to do here.

Compiling Error that I'm not really sure what to do with.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
The only thing I did was change HScript to TeaScript.

Tried to run setup-windows.bat (or if youre on mac / linux: setup-unix.sh)?
They both can be found in the setup folder


Lots of love from Team Unknown6 (/r/pokemongodev).

Thanks for updating the community @keyphact! We expected it'd contain this, but damn that's a lot of data. Guess they're finally training a model of user behavior to see if we're faking requests.
I'll go over this in more detail now, see if anything needs commenting on, then I'll merge it in. Again, great effort, great work.

Thanks :heart:

@rubenvereecken get on slack or discord, merging this PR.

No thanks. I just care about naming because it'll get renamed afterwards anyway if it's wrong.


JS_NewFloat64 is a misleading name

JS_NewFloat64 tests if its double argument is an int32_t and if so creates a JS_TAG_INT value instead of a JS_TAG_FLOAT64 value.
This is somewhat misleading and prevents creation of actual JS_TAG_FLOAT64 values in places where it would be more appropriate than JS_TAG_INT values, such as Map keys.  There is a __JS_NewFloat64 for this but it is debatable whether it should be documented if published at all.
I propose we add a JS_NewNumber API that keeps the current heuristics, change JS_NewFloat64 to always create JS_TAG_FLOAT64 values and remove __JS_NewFloat64.
Note also that float_is_int32 returns false for INT32_MIN.  I did some benchmarking for various alternatives to implement this test and it seems a much simpler test code is at least as efficient and much more readable, portable and reliable:
static BOOL float_is_int32(double d)
{
    JSFloat64Union t1, t2;

    if (d >= INT32_MIN && d <= INT32_MAX) {
        t1.d = d;
        t2.d = (int32_t)d;
        return t1.u64 == t2.u64;
    } else {
        return FALSE;
    }
}

Sound good. Out of curiosity, why would a float be preferred as map keys?

Because  comparing values with identical numeric values is more efficient if the stored types are the same.
Map hash keys must be identical for JS_TAG_INT values and equivalent JS_TAG_FLOAT64 values.  The current implementation only normalizes -0.0 as 0 keys but it would be more efficient if it also converted integer numeric keys to float64 values so the conversion would only be performed once per lookup and insertion and never needed for rehashing.

Thanks for explaining! Sounds like a good plan to me.

I have a patch almost ready to get rid of __JS_NewFloat64, make JS_NewFloat64 always create a floating point value and add JS_NewDouble to create an integer if possible. I am not satisfied with this name, maybe JS_NewNumber would be better?

JS_NewNumber sounds good to me!

Closed by #319


Possible to define an input field more than once if part of a list

Moved from https://github.com/graphql/graphiql/issues/441
I have an input object that has a list of input objects
mutation createConversation(input: CreateConversationInput!): ConversationMutation

Input Object: CreateConversationInput
messages: [MessageWrapperInput]!

MessageWrapperInput
message: MessageInput
title: TitleInput

Input Object: MessageInput
color: String!
text: String!

Input Object: TitleInput
color: String!
text: String!

The following query is valid according to GraphiQL
mutation {
  createConversation(input: {
  messages: [
    {
      message: {
        text: “a”
      },
      message: {
        text: “b”
      }
    }
  ]}){…
  }
}

But I have redefined the message input object in a single list entry.
Expected:
I should get a warning like: "There can only be one input field named message for each list entry"
The correct query for reference should be something like:
mutation {
  createConversation(input: {
  messages: [
    {
      message: {
        text: “a”
      }
   },
   {
      message: {
        text: “b”
      }
    }
  ]}){…
  }
}

How does graphql handle this? Does it report an issue?

I think this bug has been fixed since the version of 14.2.0 of graphql.


Pre-deprecate include_config_on_deploy

WHY are these changes introduced?
Fixes https://github.com/Shopify/develop-app-inner-loop/issues/2190
The include_config_on_deploy field will be deprecated soon and enabled by default, so we want to push the users to enable it as soon as possible.
Right now, we are showing this message on deploy only once:

WHAT is this pull request doing?
Show this message always, unless it's already enabled:

When using App Management API, if the field is still present in the TOML, it's removed and this message is shown:

How to test your changes?
With different values for include_config_on_deploy in the TOML:

p shopify app deploy
USE_APP_MANAGEMENT_API=1 p shopify app deploy

Post-release steps
Release the documentation PR
Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes

@jessehoek that makes sense to me, thanks! I'll update it.

@jessehoek done! I've updated the screenshots in the description.


Why not use SyncBN

Thanks for your amazing work!It converges fastly and get amazing accuracy on imagenet.
In your pth file, I found args.sync_bn is False.
In your train.py, I found the model  in your code broadcast buffers about bn before each forward instead of SyncBN.
So, I have a small question, why don't you use SyncBN?
Thanks very much!

Good question.
We ignored SyncBN and did not conduct this experiment.
I think it will improve the performance of the model.
I am going to try it now.
Thanks for your question and kind advice.

Thanks for your reply~
hoping SyncBN can achieve higher accuracy


[Android] Fix the warnings when generating java docs

This patch completes the method's descriptions that are missing the
field @param or @return.

@rakuco @xzhan96 @fujunwei Please help to review

lgtm

Remember to mention XWALK-5643.

lgtm


Feature request: Ability to run vue-tsc on a single file and extract the context

I was trying to configure https://github.com/okonet/lint-staged to run vue-tsc on the staged files only in a pre-commit step. But lint-staged passes by default the absolute path of the file under consideration. Example:
vue-tsc --noEmit /Absolute/path/to/project/components/Icon.vue

And when it's run like this, vue-tsc does not get the project's tsconfig info correctly. For example, it "Cannot find module"s configured (with aliases) in compilerOptions.paths, like this:
    "paths": {
      "@/*": ["src/*"]
    },

It would be great if vue-tsc could be configured with some CLI option to specify the root of the project or where the tsconfig file is at. But I could not find a way to do it granularly with the current options.

Hi! We also have this problem in project, because on pre-commit we need to check only staged files. Thanks if someone will answer for this issue

Things to consider:

Ref: https://dev.to/samueldjones/run-a-typescript-type-check-in-your-pre-commit-hook-using-lint-staged-husky-30id

Things to consider:

It is totally possible to do the same with vue-tsc (setting up a Husky hook to run vue-tsc), but the problem remains: it's run for the whole project and not just the staged files. I don't know if I missed something from your point, @wenfangdu.

It simply does not help identify errors in other files.
So, for example, if you change the way an export works in FileA and commit your code, any files importing FileA will likely all be broken, and you will not be notified of the breakage until you deploy it.

@gangsthub It's safer to run vue-tsc on the whole project.

Thanks for your quick answer, @wenfangdu. This is what this issue is about! 😅
I'd like to hear the opinion from the maintainers, though.

Seems like this is an issue with the underlying tsc tool:
https://github.com/microsoft/TypeScript/issues/27379

所以这个无解了吗？🤔

it would be good to have it run on particular files by specifying path

Yep. For my use case I have a project with a LOT of pre-existing type errors - too many to ts-ignore in one go - and I'd like to implement vue-tsc in a Github action so it guards against new errors being introduced via a PR. Hence I need it to ignore everything but the files touched by that PR (which I'd get with a git diff).
Hence an option to specify a list of files would be very useful.


[Question] can I directly modify the golang runtime?

Is there an existing issue for this?

[X] I have searched the existing issues

Question
Excuse me, can I directly modify the golang runtime?
Because this information is available in the runtime, such as goid or more information about GMP
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@oreak  It depends. Unless you know the impact of modifying the runtime.


FIX request url with double slashes

Summary
Fix for: request with double slashes
Description
Wrong request url example
https://api.contentful.com/spaces/SPACE_ID/environments/master//content-types
Solution:
replace Array.join() for path.join() in makeRequest()

Let's try to also account for this probably :thinking: https://github.com/contentful/contentful-migration/issues/189


How to use this bot?

Hello, can you help me with settings?
If I want profit percentage 0.7% should I set 0,07?
And why that bot always create stop limit order and never sell/buy coin?
Pair - USDTBTC

If you want 7% minimum profit, need to set 1.07 for BINANCE_JOBS_SIMPLE_STOP_CHASER_STOP_LOSS_LIMIT_LAST_BUY_PERCENTAGE .
The bot will attempt to create stop-limit order if the buy signal is wait. However, if there is not enough balance to sell, then it won't place an order.
You also can use frontend to update the configuration except for symbols.

Guys, join the Telegram group to solve problems with each other ..
https://t.me/joinchat/z3cKyYFcx0NiOTA8


Require entering project name too during env. delete confirmation

When deleting an environment, you have to enter the environment name as confirmation. However, if you have lots of different projects, entering 'master' doesn't really help confirm you're deleting the right project's master env.
I suggest requiring entering the full name (PROJECT-ENVIRONMENT) when deleting an env.

Well the user has to be on the page of the project to get into the environment before they click delete. It'd probably make more sense to just add the project name to the confirmation box, but not requiring the user to enter it as part of the confirmation.


Merge updated golden image build

Golden Image build for Debian 11 64b
Versions:
Debian Bullseye 11.7.0
VirtualBox 7.0.6
Packer 1.8.6
bullseye.pkr.hcl
debian-11-preseed.cfg
ssh for post build configuration is functional
run.sh to build.
--
Main fixes
  vboxmanage = [
    [ "modifyvm", "{{.Name}}", "--nat-localhostreachable1", "on" ]
  ]

Addresses an inability to download the preseed.cfg via the local host network even with the Nat configured for local ssh/VDI access.
d-i grub-installer/bootdev  string /dev/sda

Fixed an error where the Grub subsystem wasn't happy without a disk being defined.

I was looking to implement serial ports, and I noticed this for parameterizing the root password

For future code archeologists:
https://github.com/hashicorp/packer-plugin-virtualbox/blob/main/docs/builders/iso.mdx


[TIR][BUILD] Remove buffer params from pass config.

Buffer configurations can be passed during construction
and does not need to be part of the build config.
This is a refactor step to simplify the buildconfig for the PassContext migration.

cc @ZihengJiang @zhiics

I noticed that some buffers lost their name in the IR dump or CUDA dump after merging this PR. For example, x_1 in softmax for CUDA becomes buffer_2. Is is change in purpose?


Stuff

LGTM so far

@ticktackk in https://github.com/Xon/XenForo2-WarningImprovements/pull/45/commits/e7a5e4db0f446794d40a48bf7f7a842871cf3ad2 setFromDefinition() is adding the custom warning definition expiry to all standard warning definitions if they don't have something set.
Little confused on why

I thought custom warning was supposed to act as a parent for every warning definition

Nope, they don't!

I'll remove that part of code

@Xon done with commit : https://github.com/Xon/XenForo2-WarningImprovements/pull/45/commits/b25071c265c1383896686f8f0c50af4ae560de77


Fixes CORS handling

Currently, we always specify a list of CORS origins in the flask server. This appears to be incorrect (I didn't read the manual).
With this PR, by default, the flask server follows the expected same-origin policy. Only if the user uses the --cors flag is the cors_allowed_origins passed to the server.
I have tested as much as I can on my current local network, which consists of my phone and laptop, hot-spot-ted to my phone. Everything works and I haven't needed to use --cors when accessing the app from laptop(server) or phone.

Seems to do the trick.
@tildebyte @lstein Please test this on your end too. And then approve it. I think we have the solution.

ACK, I can finally access via phone 🎉 - I'll wait for @lstein's ACK. Then whomever gets here first can merge it.

Just folding in the latest image drawer work and then will bring this in.

This will need the front end to be built again.
@lstein I'd recommend with front end changes, you merge them all first and then do a final build version. We could do this once a day at the moment or after major changes --- until we have the automated build bot setup and running properly.


New version tag.

Hello Shawn,
I noticed that we can install this package for laravel 6 but only if we use dev-master.
Could you make a new version release to make sure we can use the latest tag?
Best regards,
Wouter van Marrum.

Any update to this?

No unfortunately not. I am hoping this will be looked at within a month.


Cover images not working

Love the project, it's a perfect visualization tool for someone like me with a ton of games!
However, I'm having trouble getting any of the game cover images to work - none of them are showing in the HTML file because of bad filenames. I'm not sure if this is due to a change in the image APIs or a bug in the rename function or something else.
I cloned the latest repo, I've installed the required Python libraries, and I'm trying to use it with the following commands on elementary OS:
python3 csv_parser.py --image-list
wget -nc -P images -i ./imagelist.txt
python3 csv_parser.py --html5

I also tried --embed with the last one but it changed nothing. I got the output:
Purged 3835 unused images
HTML5 list exported

Meaning that every single one of my downloaded cover images had been deleted so I had to re-download them again. I found that the files wget downloads for me are named e.g.:
0a0adce567d4ca859e21cd9009b4251e7b2648207a67a0c0e644a4d91b5c60cb_glx_vertical_cover.webp?namespace=gamesdb
0a1a41d732c1cfe817e038e3636c1cf61f2aacababd64c0d8d7d6f4b5f24024a_glx_vertical_cover.webp?namespace=gamesdb
0a4de1578573132f7b1be302f9d91b3cf76abdc7c9e38f6546655b4c17585a13_glx_vertical_cover.webp?namespace=gamesdb

The filenames stay the same after running the script if I comment out the deletion line so the script is not renaming them properly, i.e. removing the ?namespace=gamesdb part from the end. I manually renamed a few and the images immediately showed up in the HTML file.

I'm experiencing the same behavior.  All downloaded images in .\images are deleted when running csv_parser --html5

you'd need to rename the images to get rid of ?namespace=gamesdb before running the html5-export.
i.e. perl-rename 's/\?namespace=gamesdb//' images/* on linux machines

Well, it's clear that my fix for these issues clearly isn't working anymore, I'll have to update the CSV exporter and check again where the problem seems to be.

Something worth noting (may not be entirely related, I'm just getting back to looking at this project) is that these images are already cached locally with the exact same filename. Mine are currently stored at "C:\ProgramData\GOG.com\Galaxy\webcache{I think this is a GOG user ID}{platform}".
Perhaps it is worth enumerating all of the webp files in the webcache and doing a lookup for the URL filenames to avoid redundant downloads, and perhaps even consider symlinking to the originals? Might be overkill, but just wanted to bring this to your attention.

I opened a PR aiming to fix that issue.  #6


Update assertion handling to support indexed assertion labels

Right now, we strip the assertion indices so that stds.exif__0, stds.exif__2, stds.exif__5 all get collapsed down to stds.exif in the HashMap. This is resulting in us losing some data, since the assertion index that is last processed will overwrite the previously processed ones.
Some questions:

If there are actions assertions across multiple labels, do we have the SDK iterate over all of them to ensure compliance with editsAndActivity?
Do these need to be sorted before being delivered to the client?

Also worth calling out:

A single instance of an assertion can have an index appended to the label. e.g. A manifest could contain a single actions assertion with the label c2pa.actions__10

@adobe export issue to Jira project CAI


Error Reading Preferences - not valid JSON

I'm trying to change the PHP "executablePath" in order to get PHP Tooling support and I got this error:

Your preferences file is not valid JSON. Please fix the format.

Here is my brackets.json:
 {
    "brackets-eslint.gutterMarks": true,
    "brackets-eslint.useLocalESLint": false,
    "fonts.fontSize": "13px",
    "fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
    "themes.theme": "monokai-plus-theme",
    "livedev.multibrowser": false,
    "useTabChar": false,
    "themes.themeScrollbars": true,
    "extensions.sort": "downloadCount"
    "php": {
		// Default: true
		"enablePhpTooling": true,

		// Default: php
		"executablePath": "C:\xampp\php\php.exe",

		// Default: 4095M
		"memoryLimit": "4095M",

		// Default: false
		"validateOnType": "true"
	},
}

I've tried different things like adding only this line, with/without the comma etc.
"executablePath": "C:\xampp\php\php.exe",
But none of them worked. What could be the problem?
Thank you.

You can't have comments in brackets.json. Removing them should fix it.
You can refer to this in case you have any doubts. https://github.com/adobe/brackets/issues/14783
Let us know if there are still any issues you faith.

Unfortunately, it did not work. Here is the current state of brackets.json:
{
    "brackets-eslint.gutterMarks": true,
    "brackets-eslint.useLocalESLint": false,
    "fonts.fontSize": "13px",
    "fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
    "themes.theme": "monokai-plus-theme",
    "livedev.multibrowser": false,
    "useTabChar": false,
    "themes.themeScrollbars": true,
    "extensions.sort": "downloadCount"
    "php": {
        "enablePhpTooling": true,
        "executablePath": "C:\xampp\php\php.exe",
        "memoryLimit": "4095M",
        "validateOnType": "false"
    }
}

@milkymeda You have to put executablePath like such "C:\\xampp\\php\\php.exe" to escape backspace.

That worked! Thanks a lot!
{
    "brackets-eslint.gutterMarks": true,
    "brackets-eslint.useLocalESLint": false,
    "fonts.fontSize": "13px",
    "fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
    "themes.theme": "monokai-plus-theme",
    "livedev.multibrowser": false,
    "useTabChar": false,
    "themes.themeScrollbars": true,
    "extensions.sort": "downloadCount",
    "php": {
    "executablePath": "C:\\xampp\\php\\php.exe",
    "validateOnType": "true"
    }
}

That worked! Thanks a lot!
{
    "brackets-eslint.gutterMarks": true,
    "brackets-eslint.useLocalESLint": false,
    "fonts.fontSize": "13px",
    "fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
    "themes.theme": "monokai-plus-theme",
    "livedev.multibrowser": false,
    "useTabChar": false,
    "themes.themeScrollbars": true,
    "extensions.sort": "downloadCount",
    "php": {
    "executablePath": "C:\\xampp\\php\\php.exe",
    "validateOnType": "true"
    }
}

I am having same issue here and I added this code but my Brackets still having the issue of resetting itself after lunching.
I still get the default theme and font size whenever I restart.
Secondly, it still takes me to this JSON error page instead of my previous project.
Any help on how to solve this?
Thanks

@marshalnosa Have you used valid JSON?

@marshalnosa Have you used valid JSON? Can you share your preferences.json?

@sprintr, I can't find the preference.json file here.
How do I go about that?

Hi,
Getting "error reading preferences" - "Your preferences file is not valid JSON."
Anyone know what is wrong with it ?
{
"brackets-eslint.gutterMarks": true,
"brackets-eslint.useLocalESLint": false,
"fonts.fontSize": "14px",
"fonts.fontFamily": "hack, 'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
"themes.theme": "os-x-style-flat-dark-theme",
"useTabChar": true,
"tabSize": 3,
"linting.collapsed": true,
"linting.ESLint.collapsed": false,
"linting.JSLint.collapsed": true,
"livedev.multibrowser": true,
}

@saohlsen I think you need to remove the trailing comma from the last line:
"livedev.multibrowser": true

@redmunds @ oh boy! Thank you, it works.

I have a similar problem...
"brackets-eslint.gutterMarks":` true,
"brackets-eslint.useLocalESLint": false,
"fonts.fontSize": "12px",
"fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
"themes.theme": "monokai-darkness-reborn",

I am still getting an error

There is an error in first line...
"brackets-eslint.gutterMarks":` true,
you have to remove that single invited comma in front of true

I have a similar problem...
{
    "brackets-eslint.gutterMarks": true,
    "brackets-eslint.useLocalESLint": false,
    "fonts.fontSize": "12px",
    "fonts.fontFamily": "'SourceCodePro-Medium', ＭＳ ゴシック, 'MS Gothic', monospace",
    "linting.ESLint.collapsed": false,
    "linting.collapsed": true,
    "externalApplications": {
        "jpg": "D:\\Apps\\IrfanView\\i_view64.exe",
        "jpeg": "D:\\Apps\\IrfanView\\i_view64.exe",
        "png": "D:\\Apps\\IrfanView\\i_view64.exe",
        "svg": "C:\\Program Files\\Internet Explorer\\iexplore.exe",
        "psd": "D:\\Apps\\IrfanView\\i_view64.exe",
        "php": {
        "enablePhpTooling": true,
        "executablePath": "C:\\Apps\\scoped_dir3768_943626505\\OpenServer\\modules\\php\\PHP_7.4\\php.exe",
        "memoryLimit": "4095M",
        "validateOnType": "false"
        }
    }
}

but PHP runtime doesn't work.


Exception list

Is there a way to allow certain people to join up on your fire team?

Not currently, no.
I can't promise anything, but it's on my todo list.


update rails 7.0

wip

I've made the gem support Rails 7.0 in the commit https://github.com/abicky/activerecord-debug_errors/commit/419fa94193c1e390d49133bb136244d76104acea.


Needs support for globalgamejam.org pages

Here is an example GGJ game page: http://globalgamejam.org/2015/games/day-world-changed
I have started this in an object called ggjGet here:  https://github.com/AlexBezuska/Community-Game-Jam-Blogpost-Generator/blob/master/index.js#L174

Added this feature!


Add Readme in Hello_World Folder

Description of  the change
Add a Readme file in Hello_World Folder

Domain

[x] Readme

Are you contributing under any open-source program?
HSOC'22

Great idea !
Start working on it ..


Missing "InvalidEmailException" for "resendConfirmationForEmail"

First of all, thank you for your wonderful work and the great library.
I have two suggestions which I will handle in two requests.
1.) Missing "InvalidEmailException" for "resendConfirmationForEmail":
For consistency reasons the method "resendConfirmationForEmail" should check the validity of the email as the other methods that deal with emails do.

You’re right, thank you!


[WIP][SPARK-47975][SQL] Add Collation Support for trim/ltrim/rtrim

What changes were proposed in this pull request?
Why are the changes needed?
Does this PR introduce any user-facing change?
No.
How was this patch tested?

Add new UT.
Pass GA.

Was this patch authored or co-authored using generative AI tooling?
No.

FYI, I think this is already in progress https://issues.apache.org/jira/browse/SPARK-47409
@panbingkun https://issues.apache.org/jira/browse/SPARK-47975 seems like a duplicate ticket

FYI, I think this is already in progress https://issues.apache.org/jira/browse/SPARK-47409
@panbingkun https://issues.apache.org/jira/browse/SPARK-47975 seems like a duplicate ticket

Okay, I will close it, thanks.


Clean up the test harness a bit

Primarily, this gets rid of the test/runtest executable, preventing the need for spawning an additional subprocess for each test. The current approach handles all the things that test/runtest would in a --require and in a loader that switches whether or not to use the IITM loader via the same filename check that was originally done in test/runtest.
One interesting thing that happened while making this change is that we need some way of passing the entrypoint filename to the loader. Since newer versions of Node.js run the loader in a separate thread, process.argv[1] no longer works. The approved alternative is to use inter-thread messages via the register() method, but that's not available on some versions of Node.js that also use loader threads. To get around this, an environment variable is set in the --require that's then propagated to the loader thread. This is a bit hacky, but probably the only thing that will work across the supported version range.
An attempt was made to use the more popular tap test package, rather than imhotap, but there's no version of tap that supports both the entire Node.js version range that IITM supports and also supports ESM. For now, this means sticking with imhotap, but were it not for the incompatibility, tap would be just as suitable here, and is a much more popular test package.

@jsumners-nr

I am concerned this will make debugging more difficult since this introduces a loader on top of the loader being tested.

The wrapper loader used for testing is just a pass-through. It's not chaining loaders or anything like that. It should be completely transparent. What's the concern?

@jsumners-nr

I am concerned this will make debugging more difficult since this introduces a loader on top of the loader being tested.

The wrapper loader used for testing is just a pass-through. It's not chaining loaders or anything like that. It should be completely transparent. What's the concern?

It's not a blocker. I'm just unclear what a debugging session will be like.

@jsumners-nr You make a good point here that the test README should be updated. I'll do that now.


CSS randomly missing from output

Using ~tailwind darkMode~??? breaks CSS with Obfuscator, when CSS is imported in the JS. Or doing slight app changes have the same result.
Expected Behavior

CSS should always be present in output.
Current Behavior

CSS classes are present in the HTML but do not display properly.
This is because the generated CSS is not included in the generated index.html. It is fine without obfuscator.
Steps to Reproduce

When building a simple app using tailwind with darkMode e.g. with the following in tailwind.config.js:
module.exports = {
  darkMode: "class",
  ...
}

Edit: Other updates also break apparently randomly.
And bundling with vite+rollup using vite.config.js:
 build: {
      rollupOptions: {
        output: {
          plugins: [
            obfuscator()
         ]
      }
}

Your Environment

Obfuscator version used:     "javascript-obfuscator": "^4.0.0",     "rollup-plugin-obfuscator": "^0.2.2",

Stack trace

No runtime error, but style does not display.
Minimal working example that will help to reproduce issue

Will try to make one.

Played with it a bit more, this issue is super weird and not related to the darkMode.
For some reason, the CSS is stripped from output most of the time: One commit adding just a single line of darkMode configuration triggered it, but for some reason other commits without darkMode yield the same results.
Has anyone experienced postcss problems before?

Ok, the problem is not related to my project code itself.
I tried to repeatedly build the same project. The size of the built JS changes between builds of the same code:

one run: 970.59 KiB / gzip: 327.36 KiB
another run: 969.64 KiB / gzip: 327.41 KiB

Some builds have the CSS, most don't.
Looks like a timer/async problem somewhere.
Anyone seen this?

Hi,
I've never had this issue, but using a seed 1 value will help you get the same build every time and hopefully help you figure out what's happening.


Enable relative loading of assets for pystage-generated .py files.

Previously, pystage loaded assets from the current directories or the pkg directory. This commit introduces the ability to load assets from any location by dynamically adding search directories relative to the location of the pystage-generated .py file. With this change, pystage-generated .py files can smoothly load assets and run from different directories.
After the merge, errors like this should disappear:
ahlongxp@ZacWorkBook sbs % python move6/move.py
New costume: _1 -> /Users/ahlongxp/coding/pystage/src/pystage/images/zombie_idle.png
New costume: _1_2 -> /Users/ahlongxp/coding/pystage/src/pystage/images/zombie_idle.png
New costume: _2 -> /Users/ahlongxp/coding/pystage/src/pystage/images/zombie_idle.png
Traceback (most recent call last):
  File "/Users/ahlongxp/coding/sbs/move6/move.py", line 18, in <module>
    _1.add_sound('a_bass')
  File "/Users/ahlongxp/coding/pystage/src/pystage/en/sprite.py", line 1752, in add_sound
    return self._core.pystage_addsound(name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/ahlongxp/coding/pystage/src/pystage/core/_sound.py", line 23, in pystage_addsound
    self.sound_manager.add_sound(name)
  File "/Users/ahlongxp/coding/pystage/src/pystage/core/assets.py", line 196, in add_sound
    sound = Sound(self, name)
            ^^^^^^^^^^^^^^^^^
  File "/Users/ahlongxp/coding/pystage/src/pystage/core/assets.py", line 223, in __init__
    if self.file.endswith(".mp3"):
       ^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'endswith'

Thank you for the PR and sorry for the late response :-) This is a good addition, but the check for len > 0 is not needed, sys.argv always contains at least one element. I will modify the PR and merge.


Add alternative movement remapping in example

I would like to use c-j move down c-k move up and c-l select item in the popup list. However im not advanced enough to understand how to do this. Would be nice to add this alternative binding in the readme?

inoremap <expr><C-j>  pumvisible()?  "\<C-n>" : "\<C-j>"
inoremap <expr><C-k>  pumvisible()?  "\<C-p>" : "\<C-k>"
inoremap <expr><C-l>  pumvisible()?  "\<C-y>" : "\<C-l>"


Switch linting to ESLint and Prettier

Close #97.

I'm not sure why Travis is failing now. .eslintignore excludes node_modules/**.

@MoOx How about now?

Looks good!


Project isn't SRP-3 or SRP-6a

SRP-6a is RFC 5054. This implementation uses the older SRP-3 K calculation (RFC 2945). It also uses the alternative M proof which I can't find on either RFC and is citation-needed on Wikipedia.
To comply with SRP-3 it should implement the correct M calculation. I've included an implementation of M1 calculation to the spec. I haven't verified it as the Python library uses the newer K generation algorithm, so the output values do not line up, however, all the input values appear correct.
The only tricky thing is N_sbyte contains a leading zero, this is removed by starting the counter at 1, [1..(N_sbyte.length-1)]. Might not even be necessary to convert the bytes to unsigned, as I did that before noticing the leading zero.
M1Spec: (options) ->
		N = @params.N
		g = @params.g
		s = options.s # Buffer
		I = options.I # Buffer
		A = options.A # Buffer
		B = options.B # Buffer
		K = options.K # Buffer

		N_sbyte = N.toByteArray()
		N_ubyte = []
		for i in [1..(N_sbyte.length-1)]
			N_ubyte[i-1] = N_sbyte[i] & 0xFF

		H_N = crypto
<EMAIL_ADDRESS>			.update(N_ubyte)
			.digest()

		H_g = crypto
<EMAIL_ADDRESS>			.update(g.toByteArray())
			.digest()

		H_I = crypto
<EMAIL_ADDRESS>			.update(I)
			.digest()

 		HN_xor_Hg = []
		for i in [0..(H_I.length-1)]
			HN_xor_Hg[i] = H_N[i] ^ H_g[i]

		result = crypto
<EMAIL_ADDRESS>			.update(HN_xor_Hg)
			.update(H_I)
			.update(s)
			.update(A)
			.update(B)
			.update(K)
			.digest()

		return result

To comply with SRP-6a it should update the K generation, as it is different now. See 2.6. Calculating the Premaster Secret (quoted below). The major change seems to be the introduction of lowercase k, which acts a multiplier for g^x.
The premaster secret is calculated by the client as follows:

        I, P = <read from user>
        N, g, s, B = <read from server>
        a = random()
        A = g^a % N
        u = SHA1(PAD(A) | PAD(B))
        k = SHA1(N | PAD(g))
        x = SHA1(s | SHA1(I | ":" | P))
        <premaster secret> = (B - (k * g^x)) ^ (a + (u * x)) % N

Reference for 6a S calculation. https://github.com/mozilla/node-srp/blob/master/lib/srp.js#L257

Ha, I was tricked by a comment. Implementation looks correct. I was reading the comment though. Values still aren't lining up, I'll try to figure out why.
# Client S calculation, where S = (B - g^x) ^ (a + u * x) % N
	# Returns Buffer
	clientS: (options) ->
		B = options.B # BigInteger
		a = options.a # BigInteger
		u = options.u # BigInteger
		x = options.x # BigInteger

		result = B
			.subtract(
				@k().multiply(
					@params.g.modPow(x, @params.N)
				)
			)
			.modPow(
				a.add(
					u.multiply(x)
				),
				@params.N
			)

		result = transform.pad.toN result, @params
		return result

My best guess is padding out is causing the differences for me. Since that is in a lot of places, I'm going to investigate other JS libraries.

Switched to a node server and this library works great. Closing this since it's functional.