Page refreshes unexpectedly on form submission instead of loading dynamic results

Running the following playwright test works as expected and passes:
import asyncio
from playwright.async_api import Playwright, async_playwright, expect

async def run(playwright: Playwright) -> None:
    browser = await playwright.chromium.launch(headless=True)
    context = await browser.new_context()
    page = await context.new_page()
    await page.goto("https://www.vmi.lt/evmi/metines-gyventojo-seimos-turto-deklaracijos-duomenu-israsai?lang=en")
    await page.frame_locator("iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator("#ddMetai").select_option("2022")
    await page.frame_locator("iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator("#ddPareigGr").select_option("501")
    await page.frame_locator("iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").get_by_role("button", name="Ieškoti").click()
    text = await page.frame_locator("iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator("#ListViewAsm_ctrl0_Label1").inner_text()
    print(text)
    await expect(page.frame_locator("iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator("#ListViewAsm_ctrl0_Label1")).to_contain_text("ALMINAS MAČIULIS")

    # ---------------------
    await context.close()
    await browser.close()

async def main():
    async with async_playwright() as playwright:
        await run(playwright)

asyncio.run(main())

Two drop-down selectors are changed, the submit button is clicked, and the correct results are verified after loading.
However, running the identical code in a scrapy spider with scrapy-playwright results in a pause after the form is submitted, and then an apparent page reload returning the page to its original state instead of loading results. The script then hangs waiting for the "#ListViewAsm_ctrl0_Label1" element. Adding error handling with screenshots confirms that the page has been reloaded.
import scrapy

class LithuanianspiderSpider(scrapy.Spider):
    name = "lithuanianspider"
    allowed_domains = ["www.vmi.lt"]
    start_urls = ["https://www.vmi.lt/evmi/metines-gyventojo-seimos-turto-deklaracijos-duomenu-israsai?lang=en"]

    def start_requests(self):
        self.logger.info("Running requests...")
        yield scrapy.Request(
            self.start_urls[0],
            meta=dict(
                playwright=True,
                playwright_include_page=True,
            ),
            callback=self.parse_form,
        )

    async def parse_form(self, response):
        page = response.meta["playwright_page"]
        await page.frame_locator(
            "iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator(
            "#ddMetai").select_option("2022")
        await page.frame_locator(
            "iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator(
            "#ddPareigGr").select_option("501")
        await page.frame_locator(
            "iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").get_by_role(
            "button", name="Ieškoti").click()
        self.logger.info("========== WAITING FOR RESULTS ==========")
        try:
            text = await page.frame_locator(
                "iframe[name=\"_com_liferay_iframe_web_portlet_IFramePortlet_INSTANCE_nv94dDWrzyD2_iframe\"]").locator(
                "#ListViewAsm_ctrl0_Label1").inner_text()
            await page.screenshot(path="screenshot.png")
            self.logger.info("==========>>>> RETRIEVED CONTENT: {}".format(text))
        except Exception as e:
            self.logger.info("========== TIMED OUT ==========")
            await page.screenshot(path="timeout.png")

        await page.close()

Seems to be working correctly for me with PLAYWRIGHT_PROCESS_REQUEST_HEADERS=None, i.e. giving complete control over headers processing to Playwright. The default behavior is to somehow emulate how Scrapy handles headers, but it doesn't work for all cases.

@elacuesta that was it. thank you!

Glad you got it working, closing the issue now.


Connection String elements on config files parsed incorrectly

Publish Azure Data Factory - is adding extra chars onto connection strings when parsing the config file.
Example:
"Integrated Security" becomes "integratedÂ security".
It always seems to be the same char that gets added.
I have checked the strings for unprintable unicode chars and there are none present.
Obviously this breaks the deployment as the linked services will not work until manually corrected after release.
As the incorrectly parsed string doesn't fit into any of the input boxes, it adds them as "Additional Connection Properties".

Please double-check whether your config file is saved with UTF-8 encoding.

Hi Kamil, I tried it both ways and still the same issue.
Which is the correct way?
Cheers,
Dave
Sent from my iPhone
On 21 Dec 2021, at 12:35, Kamil Nowinski @.***> wrote:

Please double-check whether your config file is saved with UTF-8 encoding.
—
Reply to this email directly, view it on GitHubhttps://github.com/SQLPlayer/azure.datafactory.devops/issues/79#issuecomment-998743632, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW7UOXFJL2PMWICYHC6MJD3USBYCXANCNFSM5KNQU5DQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Hi just to update this one. The extra chars have stopped appearing now I stopped using UTF-8 encoding on csv file. The values still aren’t aren’t landing in the correct fields and are dropping into additional connection properties.
It’s only on SQL server Type linked services and only on the connection string.
Cheers,
Dave
Sent from my iPhone
On 21 Dec 2021, at 13:47, David Edmonds @.***> wrote:
 Hi Kamil, I tried it both ways and still the same issue.
Which is the correct way?
Cheers,
Dave
Sent from my iPhone
On 21 Dec 2021, at 12:35, Kamil Nowinski @.***> wrote:

Please double-check whether your config file is saved with UTF-8 encoding.
—
Reply to this email directly, view it on GitHubhttps://github.com/SQLPlayer/azure.datafactory.devops/issues/79#issuecomment-998743632, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AW7UOXFJL2PMWICYHC6MJD3USBYCXANCNFSM5KNQU5DQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Please attach at least two files: config files and Linked Service causes the problem.

Config-UAT.csv
onpremiseSQLServer.txt
Here you go.
I have removed sensitive info from connection strings but the format is identical.

Try to reproduce this issue on your local PC with PS module first.
You should be able to check substituted values in new file ~onpremiseSQLServer.txt

Also, try to encapsulate values (in CSV config file) in quotes:

Ok I will give it a try, I didn't use quotes for somethings as I copied them out of the JSON view of the LS and noticed that only some things had quotes.
Thanks for the help

Double quotes around the values has fixed it.
Many Many thanks for the great product and excellent speedy assistance.
Best Wishes and Happy Christmas,
Dave

Thanks. The same to you!


Removed needless extra try in run_forever

I'm sorry, I don't find that better or clearer. It's only the _run_once() call that can raise _StopError.

@gvanrossum OTOH it's only while True: _run_once() code in that try block...


Only show black background. Video not play.

I using example code.

Please search GitHub for example usages of react-native-video and consider opening a question on Stack Overflow if you need further support. Thanks!

check that video is on https

@jhabdas I'm getting this same black window even while trying to test the example within this official repo.
Also, just my humble opinion, this particular package isn't popular enough to have support on SO.
Thanks.


Mistake in diagram in "Private Endpoint and Private DNS configurations" regarding scenario "Virtual network and on-premises workloads using a DNS forwarder"

Hi all,
I found a trivial mistake in a diagram in this page:   https://learn.microsoft.com/en-us/azure/private-link/private-endpoint-dns#virtual-network-and-on-premises-workloads-using-a-dns-forwarder    regarding Private Endpoint and Private DNS configurations in this particular scenario "Virtual network and on-premises workloads using a DNS forwarder":

As you can see in the attached image the On-Premises Network and VNet-spoke-eastus-001 IP addressing are overlapping and it could be a potential source of misunderstanding and confusion.
Can you please review the diagram and fix it accordingly (it's enough to define VNet-spoke-eastus-001 with IP addressing <IP_ADDRESS>/24 as in a previous example in the same page).
Thanks for you kind cooperation,
Luca Bovo - beanTech

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

ID: a88af182-32aa-a8f3-359d-b92ae1cfb8c7
Version Independent ID: 01ce07fc-4bc2-def5-9ecd-a80330ad9488
Content: Azure Private Endpoint DNS configuration
Content Source: articles/private-link/private-endpoint-dns.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@lucabovo Thanks for your feedback.
I shall assign this thread to the author @asudbring for further review

@lucabovo Thanks for the feedback! We have created an item in our backlog and will update the article if appropriate.
#please-close


Support active_low configuration

@nastevens or @posborne can you give this a final look at merge if it's OK? Adam has been testing with this.

There's CI problems but those can be fixed separately.  Passing on stable on Linux and the OSX failure seems superficial (we just have a stub for OSX).


Python 3.10 with Pylance produces popups about Python language not supported version

This issue has been moved from a ticket on Developer Community.

[severity:It's more difficult to complete my work]
The problem occurs under Windows Server 2019 Standard version 1809. The server is practically "empty" except for an Office installation. VS 2022 v17.02, downloaded and installed for Python usage only.
One solution with one project with one module with 2 lines of code.
When opening the solution two things happen:

The top message "Your're using Python 0.0. Some features..." The More info link does not open
A popup "Pylance language server does not suppurt Python 2.x Type checking ..." appears.

Once the project gets larger (still very small, y 5 files) with imports of installed libs like pandas the number of popups grows to between 5 and 50. If they are clicked off, the work can continue. The popups come up when saving a file, starting the app, opening a file for editing.
The same behavior on different servers, different code. I tried various combination of with / without toml. pysetup.toml yes/no, wheel installed yes/no etc. No configuration seems to be working. Should be straight forward to reproduce with any code / project with some files.
The popup and the top-row message are in the screenshot attached.
Thank you and kind regards

Original Comments
Feedback Bot on 12/6/2021, 00:02 AM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

Dupe of #6795


Add Custom Crop ViewController

In order to enable for custom aspect ratio when cropping.
Can't make tests pass on Travis though, due to the error message:
“Use Legacy Swift Language Version” (SWIFT_VERSION) is required to be configured correctly for targets which use Swift. Use the [Edit > Convert > To Current Swift Syntax…] menu to choose a Swift version or use the Build Settings editor to configure the build setting directly.
I believe this is an error on xcbuild or something must be outdated on Travis CI server side. Also, didn't check the .travis.yml, something might be wrong there.

Excellent. Very nice, thank you for the contribution.
Has anybody else reviewed this yet?

No news on this? Ill might just create a new pod with this functionality based on yours. Is it ok @fulldecent ?

That is okay but also I am happy to merge here. Would you please be able to attach a few screenshots illustrating the new feature?
Sorry, I know Travis can be a pain, I hope that will sort itself out.


[0.5] Implement Descriptor Indexing

This implements the gfx-hal component of supporting descriptor indexing.
Main actions:

Add features to hal
Enable appropriate features on metal
Enable features on DX12 with additional compiler flag if needed
Wire up enabling extensions to Vulkan.

Remaining Questions:

This PR isn't finished, but my main concern is how to get the struct of function pointers from Instance::create, through PhysicalDevice so it can get to PhysicalDevice::features where it needs to be. I sorta just stuck it in the structs, but if you have a better idea, please do suggest.
Any other vulkan advice/tips appreciated.
I need to properly detect when optional instance extensions aren't supported.
Remove the horrible duplication of the extension name... somehow.

I believe that is everything addressed

Everything should be bumped and all, I'll work on forward-porting this to master.

bors r+

bors r+

bors retry


The red border still shows after removing the conflict metadata for blobs

Storage Explorer Version: 1.4.0
Platform/OS Version: Windows 10/MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32/x64
Build Number: 20180727.7
Commit: 3fd8ad34
Regression From: Not a regression
Steps to Reproduce:

Open one blob container which contains one blob at least.
Open the 'Properties' dialog for one blob -> Add two metadata with the same name -> Remove one metadata then check the rest metadata.

Expected Experience:
No red border displays after removing the conflict metadata.
Actual Experience:
Display a red border after removing the conflict metadata.

Duplicate of #368.


how to use moment() in controllers or directives

Thanks!
Apparently, you only need to inject moment, no need for $window. Can you please update the PR?


Tune CI integration with GitHub Actions

CI is finished #9 . Need to:

[x] #10
[x] - Integrate with dagger cloud
[x] - Decide if CI is run on GitHub Actions Runners or any other

Workflow created and tested, fully integrated with dagger cloud and no need to run CI on dedicated GitHub actions runners


Error in running Amp-pwa sample

Could not find a required file.
Name: index.html
Searched in: /home/user_name/Desktop/web/samples/amp-pwa/public
npm ERR! Linux 4.15.0-43-generic
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "start"
npm ERR! node v8.10.0
npm ERR! npm  v3.5.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'react-scripts start'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the amp-pwa package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     react-scripts start
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs amp-pwa
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls amp-pwa
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/user_name/Desktop/web/samples/amp-pwa/npm-debug.log

I found you can solve this by creating a /public folder and moving the index.html from the root into that newly created folder. But then you'll get a new series of errors because this sample is relying on older packages.

Bump, I am having this same error.

@maddoxnelson Not sure if this helps, but after trying a number of AMP/React integrations, the one that is working best for me is https://github.com/zeit/next.js/tree/canary/examples/with-amp.

Hi
Having the same issue while tried to sping up with npm start
Any updates on this issue?

The original repository have the dependencies correctly pin and still work
https://github.com/choumx/amp-pwa


Doesn't work from es-ES to it-IT

Hi Robin,
I'm trying to run the script to translate from spanish to italian, but it gives to me this error:

<EMAIL_ADDRESS>gpt-translate-json
gpt-translate-json --apiKey=myapikey --model=gpt-3.5-turbo --maxTokens=3500 --langs=it-IT --originalLang=es-ES
GPT Translate Json
translating files...
file:///C:/Users/myuserfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:152
throw new Error("Original asset not found");
^
Error: Original asset not found
at C (file:///C:/Users/myuserfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:152:13)
at et (file:///C:/Users/myuserfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:176:31)

The current code line on package.json is:
"gpt-translate-json": "gpt-translate-json --apiKey=myapikey --model=gpt-3.5-turbo --maxTokens=3500 --langs=it-IT --originalLang=es-ES"  },
And finally, in i18n folder, I have created the es-ES subfolder, so there are two: en-US (already there) and the new one, es-ES, with this app.json file inside:
{   "app": {     "dates": "Fechas y tiempo relativo",     "devs": {       "one": "{{ value }} desarrollador de software",       "other": "{{ value }} desarrolladores de software"     },     "greeting": "¡Hola! Soy {{name}}",     "list": [       {         "value": "Primer punto"       },       {         "value": "Segundo punto"       }     ],     "title": "
Biblioteca para traducir JSON usando GPT
"   } }
Do you know what is wrong??
Thanks for your time,

Hi Robins, thank you for your reply.
With that modification it works perfectly on the sample file, but now I'm getting other errors on another json larger file. The large file I want to translate is a one-line 43.000 characters, and already named app.json (I have replaced it and deleted the it-IT other one generated with the sample app.json, just to avoid replacement problems).
This is what I get:
GPT Translate Json
translating files...
Error: app.json: OpenAI API - Bad Request
at L (file:///C:/Users/userfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:133:13)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async Promise.allSettled (index 0)
at async b (file:///C:/Users/userfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:140:15)
at async Promise.all (index 0)
at async C (file:///C:/Users/userfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:150:7)
at async et (file:///C:/Users/userfolder/Documents/GitHub/gpt-translate-json/node_modules/gpt-translate-json/lib/index.js:176:25)
i18n.metadata\translated.json
i18n.metadata\translated-langs.json
Total tokens: 0

Could you post the command you are using (without apiKey)?
Try reducing maxTokens. As in doc: the command splits the files into multiple requests to respect the maximum number of tokens in each request based on the English language, but you are using ES

Hi Robin, thank you again for your help.
I've changed the max characters from 3500 to 3000. After that, I've run again the command to reinstall
npm install gpt-translate-json
and then, the command to run it:
npm run gpt-translate-json
Here is an screenshot with the full process, in case you want to see it: https://prnt.sc/KcWn5bpHFr3I
And the tree /f result for the "i18n" folder is:
├───.metadata
│       translated-langs.json
│       translated.json
│
├───en-US
│       app.json
│
└───es-ES
app.json

Usually the error Bad Request happens when you exceed the limit of character allowed by the API: see for example this thread: https://community.openai.com/t/error-retrieving-completions-400-bad-request/34004
You should try to still reduce the value of -maxToken (for example to 1000) and try again.
But be careful: a 43000 character file is split into multiple requests to respect the token limit. If you are using OpenAI's free plan, you would still exceed the 3 RPM limit.
Alternatively you should debug here, and see if the error offers finer details: https://github.com/robisim74/gpt-translate-json/blob/main/src/index.ts#L205

Closing due to lack of feedback


Use Uuid to name JSON spec files

Tasks/Steps/Elements
https://docs.python.org/3/library/uuid.html
Versions checklist

Spyder version: >= 5
Python version: 3.7, 3.8
Qt version: all
PyQt version: all
Operating System name/version:

server : Ubuntu 20.04 Linux (aarch64 on RPI)
client : Windows/Linux/macOS

Hi @ccordoba12, thanks for the suggestion. If you have more ideas/feedback, please leave them here.
Cheers!

Fixed by #5c4d702


Corrupted Software after update

Does this issue occur when all extensions are disabled?: Can't Test

VS Code Version: Latest
OS Version: Windows 10 21H2

Steps to Reproduce:

Close VSCode

For a few weeks, when closing visual studio code, I have an up-to-date window appearing randomly.
When it appears, the update does not complete (error) and I can no longer start VSCode. I have to reinstall it.
Error after update :

Uptodate popup :

/duplicate #47841
As a workaround, please disable Windows background updates by setting update.enableWindowsBackgroundUpdates to false.


Add cache eviction test. Clean up some other tests.

cc @chiragsalian @madmax330 might be interested in the conversation here
Doesn't seem like there is a very clear consensus yet on whether we need to fix the existing cache eviction strategy. But the concerns in the Slack thread above make a lot of sense. I have also started to see storage errors popping up more and more frequently while testing Expensify.cash via the web proxy instead of local development. I expect that other engineers will see the same if they switch to testing via the proxy.
For now, this PR just adds tests for the existing cache eviction system since none were written when it was conceived. Eventually if/when we re-work cache eviction it will be good to have a test set up to build off of. Additionally, it seems like a good idea to get some of this stuff under test.

Not sure why but this PR didn't have GH actions run on it, meaning there was some lint errors that snuck in 🤔

When did we add GH actions for this repo?

I can't recall ever seeing them run, but maybe they were at one point.


Fix Jenkins presubmits by reverting #3255

Currently, presubmits at head are broken with the following error:
[EnvInject] - Variables injected successfully.
Parsing POMs
Downloaded artifact http://repo.maven.apache.org/maven2/org/apache/apache/18/apache-18.pom
ERROR: Failed to parse POMs
java.io.IOException: remote file operation failed: /home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_MavenInstall@2 at hudson.remoting.Channel@7b1b4155:beam7: hudson.remoting.ProxyException: hudson.maven.MavenModuleSetBuild$MavenExecutionException: org.apache.maven.project.ProjectBuildingException: Some problems were encountered while processing the POMs:
[ERROR] 'dependencies.dependency.version' for org.apache.beam:beam-runners-flink_2.10:jar is missing. @ line 68, column 21
[WARNING] 'artifactId' contains an expression but should be a constant. @ org.apache.beam:beam-runners-flink_${flink.scala.version}:2.2.0-SNAPSHOT<EMAIL_ADDRESS>line 29, column 15

	at hudson.FilePath.act(FilePath.java:993)
	at hudson.FilePath.act(FilePath.java:975)
	at hudson.maven.MavenModuleSetBuild$MavenModuleSetBuildExecution.parsePoms(MavenModuleSetBuild.java:985)
	at hudson.maven.MavenModuleSetBuild$MavenModuleSetBuildExecution.doRun(MavenModuleSetBuild.java:690)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:490)
	at hudson.model.Run.execute(Run.java:1735)
	at hudson.maven.MavenModuleSetBuild.run(MavenModuleSetBuild.java:542)
	at hudson.model.ResourceController.execute(ResourceController.java:97)
	at hudson.model.Executor.run(Executor.java:405)
Caused by: hudson.remoting.ProxyException: hudson.maven.MavenModuleSetBuild$MavenExecutionException: org.apache.maven.project.ProjectBuildingException: Some problems were encountered while processing the POMs:
[...]

https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/14503/console
This change reverts the suspected cause #3255 to unbreak development at head.

R: @robertwb @aljoscha @neurolabs

Coverage increased (+0.006%) to 69.553% when pulling 66dca6c33d5a4100a4aa4af1101d74902e347671 on charlesccychen:fix-master into 59542c3cc796ac2aa2976ea17d385b141c1b0b38 on apache:master.

Fixes jenkins, LGTM. Perhaps the tests were out of date on the original PR?

Hi, very sorry for the inconvenience. The problem was caused because the flink.scala.version variable was not set in the nexmark pom, which came in after the original PR for changing the Flink Scala version.
I'll open a new PR with a fix for that.


Cannot use task ifeval

I think this might be the reason -- working on it rn:
>>> import lm_eval
>>> from lm_eval.tasks.ifeval import instructions_util
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/pasquale/anaconda3/envs/leaderboard/lib/python3.10/site-packages/lm_eval/tasks/ifeval/instructions_util.py", line 22, in <module>
    import immutabledict
ModuleNotFoundError: No module named 'immutabledict'

just found that the missing dependencies can be installed with ".[ifeval]"


refactor: Make get notes return all notes at beginning of array #4991

Resolves #4991

Benchmark results
Metrics with a significant change:

note_history_successful_decrypting_time_in_ms (5): 1,455 (+19%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 253d0022 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,668
18,820
36,356

l1_rollup_calldata_gas
66,328
239,140
469,712

l1_rollup_execution_gas
647,426
929,498
1,305,892

l2_block_processing_time_in_ms
1,291 (-1%)
4,762 (-1%)
9,405 (-2%)

note_successful_decrypting_time_in_ms
173 (-14%)
522 (-4%)
1,016 (-3%)

note_trial_decrypting_time_in_ms
76.7 (-27%)
37.5 (-39%)
47.6 (-19%)

l2_block_building_time_in_ms
18,057 (-2%)
68,594 (-1%)
135,994 (-1%)

l2_block_rollup_simulation_time_in_ms
8,041 (-2%)
28,732 (-2%)
56,662 (-1%)

l2_block_public_tx_process_time_in_ms
9,995 (-2%)
39,809 (-1%)
79,234 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,849 (+6%)
26,525 (-2%)

note_history_successful_decrypting_time_in_ms
:warning: 1,455 (+19%)
2,616 (+5%)

note_history_trial_decrypting_time_in_ms
113 (+72%)
179 (+5%)

node_database_size_in_bytes
18,628,688
35,024,976

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
280
44,366
28,244

private-kernel-ordering
215
52,868
14,326

base-parity
1,872 (-2%)
128
311

base-rollup
721
165,787
925

root-parity
1,589 (+13%)
1,244
311

root-rollup
52.4 (-2%)
4,487
725

private-kernel-inner
642
73,771
28,244

public-kernel-app-logic
447
35,260
28,215

public-kernel-tail
176 (+2%)
40,926
28,215

merge-rollup
8.42 (+1%)
2,696
925

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (-3%)
16.0 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.584 (-3%)
0.495 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.3 (-2%)
72.7
230 (-1%)
448 (+1%)
876 (-1%)
1,721 (-2%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.475 (-2%)
0.448
0.419 (-1%)
0.418 (+1%)
0.416
0.412 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
55.0 (+1%)
108
336 (-1%)
668 (+1%)
1,313 (-1%)
2,625
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
104
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.489 (+1%)
0.484
0.457 (-1%)
0.458
0.455
0.454
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.6

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.537

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes

tx_size_in_bytes
22,012

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
3,178 (-2%)
1,748

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
12.1 (+2%)
1,250

I'd strongly recommend holding this off until we can test this from noir directly. There we'd simply mock the oracle response and test the noir return value from noir itself. The end to end approach has a million in-between layers that make the test a) way harder to write b) way harder to read and ensure correctness c) way slower.


Add SambaNova Provider

I created a PR to add SambaNova as a provider, would appreciate help merging this! @rohitprasad15 @ksolo  @standsleeping  @andrewyng

@ksolo @rohitprasad15 I have addressed review, let me know if I can help with anything else before merging! 🙇

@ksolo  @rohitprasad15 I have addressed review, let me know if I can help with anything else before merging! 🙇

@snova-zoltanc  - we will just test it today on our end to see that it works, and then we can merge. Thanks for the PR.

today on our end to see that it works, and then we can merge. Thanks for the PR.

Sounds great, thank you!

Tested. It works. Thanks.


Display "dirty" status with Git: index / working dir

It would be useful to have an indicator next to the branch name that would indicate if there are changes in the index and/or working copy.
This should be done through fugitive probably.
See https://github.com/tpope/vim-fugitive/pull/219 for an older PR and https://github.com/tpope/vim-fugitive/issues/206 about the issue in general.
This can be useful to see if there are any changes in the index already (before :Gw).
I usually fall back to Ctrl-Z and then my prompt has these indicators.

Interesting. If this feature is ever supported by fugitive, please reopen. Closing until then.


boards: arm: Add RTC clock source for 96b_aerocore2 and az3166_iotdevkit

The 96b_aerocore2  and az3166_iotdevkit boards doesn't have its RTC node enabled, and they are failing the following test:
tests/benchmarks/footprints/benchmark.kernel.footprints.pm
--> Enabled RTC node for that platform.
This PR completes the https://github.com/zephyrproject-rtos/zephyr/pull/66359
to fix https://github.com/zephyrproject-rtos/zephyr/issues/66360

Added Hotfix as this is blocking #63973


[DOCFIX] Fix EN docs menu

update the docs menu based on current layout of md files in docs/en/

alluxio-bot, merge this please


Incompatible with Mongoid ORM

Devise.setup do |config|
  config.mailer_sender =<EMAIL_ADDRESS>  require 'devise/orm/mongoid'
  config.case_insensitive_keys = [:email]
  config.strip_whitespace_keys = [:email]
  config.skip_session_storage = [:http_auth]
  config.reconfirmable = true
  config.expire_all_remember_me_on_sign_out = true
  config.responder.error_status = :unprocessable_entity
  config.responder.redirect_status = :see_other
end

ActiveRecord::ConnectionNotEstablished at /api/v1/users/tokens/sign_in
No connection pool for 'ActiveRecord::Base' found.

To access an interactive console with this error, point your browser to: /__better_errors

activerecord (7.0.8) lib/active_record/connection_adapters/abstract/connection_handler.rb, line 208
  203               message = "No connection pool for '#{spec_name}' found for the '#{role}' role."
  204             else
  205               message = "No connection pool for '#{spec_name}' found."
   This is for Rails 8


fix bug in list action

the list action displays the following:

Expected result is:

Hypothesis: Ibang handler yung naghahandle ng transaction

Possible solution:

[ ] create a separate folder for bci and migrate xo files to bci
[ ] build a local copy of rest-api and validator and observe wheh

Possible solution:

[ ]  create a separate folder for bci and migrate xo files to bci
[x]  build a local copy of rest-api and validator and observe whether this solves the problem

solution 2 result:

Possible solution:

[X]   create a separate folder for bci and migrate xo files to bci
[x]   build a local copy of rest-api and validator and observe whether this solves the problem

solution 2 result:

solution 1 result:


Clamp mouse wheel values

Browsers return some abstract value and we need to limit it to 1.0.

The value returned by browsers isn't actually "abstract". Browsers return the amount of pixels that should be scrolled on the Y axis, but since this is zoom we're talking about, there aren't any pixels to speak of, only scale increments.
I'm not that happy about just clamping the values. macOS for instance will "accelerate" scrolling: the amount scrolled will be greater with each step of the scroll wheel. I'd rather keep that behavior, and supporting that requires us to not clamp the value, as it can accelerate past 1.0.

There is only one problem - scale. What scale are you going to use for this when almost every browser gives some different value? For example, in my case on Chrome this value is around 200, but on Firefox this value already exceeds 1000. Making a scale for this will be hard.

Yeah, I know that. I wonder if winit exposes this scale somehow. Looking at browser APIs though, there doesn't seem to be a way of checking how the values are scaled, which is a shame.
I propose that we postpone this until an official wasm version is implemented. At that point we can add a #[cfg] checking if we're compiling for wasm, clamping the value then.

Makes sense, I think.

Looks OK I guess. Merging.


Add acknowledgements

Currently, the developer guide does not include acknowledgements.
Let's add them to make the developer guide complete

Codecov Report
Base: 71.54% // Head: 71.54% // No change to project coverage :thumbsup:

Coverage data is based on head (8dd6c6a) compared to base (def84b6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #234   +/-   ##
=========================================
  Coverage     71.54%   71.54%
  Complexity      659      659
=========================================
  Files           106      106
  Lines          2035     2035
  Branches        242      242
=========================================
  Hits           1456     1456
  Misses          482      482
  Partials         97       97

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #44.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #44

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.

For the host section of the recipe, you should usually use python {{ python_min }} for the python entry.
For the run section of the recipe, you should usually use python >={{ python_min }} for the python entry.
For the test.requires section of the recipe, you should usually use python {{ python_min }} for the python entry.
If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12237782122. Examine the logs at this URL for more detail.


prevent tcgc versioning projection

since autorest emitter needs to generate multiple version swagger, need to pass all to tcgc to prevent tcgc do the versioning projection to latest version.

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-autorest

Show changes
@azure-tools/typespec-autorest - fix ✏

prevent tcgc versioning projection

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/745/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/745/


select options - RuntimeWarning: divide by zero encountered in log

The bug
When using the select option specifying a list of options with 3+ elements in which two have a very similar encoding, at line 146 of guidance/select/_select.py, computing the logarithm of zero raises an error.
To Reproduce
transformers.version == 4.29.2
guidance.version == 0.0.64
from transformers import AutoTokenizer, AutoModelForCausalLM
import guidance

def main():
    tokenizer = AutoTokenizer.from_pretrained('gpt2')
    model = AutoModelForCausalLM.from_pretrained('gpt2')

    gpt2 = guidance.llms.Transformers(model=model, tokenizer=tokenizer)
    guidance.llm = gpt2

    valid_options = ["HTR2C", "NR3C1", "HTR2A"]
    program = guidance("""The best option is: {{select 'acronyms' options=valid_options}}""")
    executed_program = program(valid_options=valid_options)

if __name__ == '__main__':
    main()

PYTHON_PATH/site-packages/guidance/library/_select.py:146: RuntimeWarning: divide by zero encountered in log
  logprobs_out[k] = np.log(or_prob)

System info (please complete the following information):

OS: Ubuntu 20.04.5 LTS
Guidance Version: 0.0.64

Additional Notes
Trying to debug the issue I have noticed a couple of things, I hope they can help:

The warning is not raised if the list of valid_options is ["HTR2C", "HTR2A”], i.e., if the list is made up of only the elements with similar encodings, though these are the problem (see next point)
The issue is caused by sub_logprobs, which after the recursive call of recursive_select is defined as

{
(464, 1266, 3038, 318, 25, 367, 5446, 17, 34, 50256): -1000,

(464, 1266, 3038, 318, 25, 367, 5446, 17, 32, 50256): -0.4258567988872528
}

I believe the bug lies in the fact that the first element of the above dictionary should not be there as the token 32 was selected as the best for prefix=[464, 1266, 3038, 318, 25, 367, 5446, 17], so that the other option should have been discarded. However, I couldn’t understand why that first element appears in sub_logprobs (maybe is an effect of async?)

I'm also having this problem with the sample program shown on the readme page.

Hopefully this is fixed in the new release, where we fixed a ton of select bugs. Sorry it took us so long to get to it!


About the usage of the dataset.

Hi, @zzuxzt!
Great work, thanks for sharinig it with the community! I am now working on Social-Nav and I am interested in occupancy grid prediction and planning. I noticed your dataset comes from the SCAND dataset but I am little confusing about how to use the npy files provided by you especially the "scan" part. Did you capture the ranges in the ROS  bag files directly into your npy files? And I guess the max range is 30 meter?
Thank you

Thanks for your interest in our work. The "lidar scan .npy" file is a numpy array of scan range values. You can directly use python np.load(".npy") to read the npy file and obtain the scanning range values, and you can find the scanning bearing values (the angle range and resolution) based on the specific lidar sensor installed on the robot. The maximum range is also based on the specific lidar sensor, but I manually cut it off to 30 meters when I used it.

@zzuxzt
Hi, thanks for your reply, in your settings described in your paper, the size of the local map is (0,6.4)m on x axis and (-3.2, 3.2)m on y axis. But I see that in your code to validate a scan is in the local discrete coordinate or not, you write:
flag_v = (x_r < self.x_max) & (y_c < self.y_max) & (x_r >= 0) & (y_c >= 0) in a member function of GridMap. Hrere (y_c > 0), why not (y_c >= -3.2), -3.2 is the y_lim fo the GridMap.

Note that x_r and y_c are the grid indices of the OGM (x_row, y_column) rather than the physical coordinates. The range of x_r and y_c is [0, 64] in our paper.

@zzuxzt
Ok, my bad. I still have a problem understanding your code.
In the motion compensation, when calculating the coord transformation of past seq_len frames pos relative to the predicted frame  pos_origin, why the inverse rotation matrix are all the same?

This is because we use the predicted coordinate frame as the reference frame and need to transfer the coordinates of all the past lidar sequences to the predicted coordinate frame. The predicted coordinate frame (future pose of the robot) is the same for all the past lidar sequences.

@zzuxzt
Sorry, I still have question understranding the transformations.
It looks like the "positions" data (pose) in the dataset is under the odom frame. And the member function robot_coordinate_transform of LocalMap is to transforme the past poses to the sampled (predicted)  reference frame (t+1). I think it should be impolemented as transform the t+1 frame to the spatial frame multiplied with the transformation from spatial frame to the past frames as:

In your code, of course the rotation is correct, you just simply calculated the translation with respect to the spatial frame:

@zzuxzt
Sorry, I still have problem understranding the transformations of motion compesation.
It looks like the "positions" data (pose) in the dataset is extracted from the odom. The member function robot_coordinate_transform of class LocalMap is to transform the past poses to the sampled (predicted)  reference frame at t+1. I think it should be implemented as multipliy the transformation of the spatial frame relative to the t+1 frame to the spatial with the transformation from spatial frame to the past frames as:

$$
v_{t+1}=(T_{t+1, s}\cdot T_{s,t-\tau:t})\cdot v_{t-\tau:t}
$$
In your code, of course the rotation is correct, you just simply calculated the translation with respect to the spatial frame:

Thank you for answering my questions.


_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 20s
master passed

+1 :green_heart:
compile
0m 25s
master passed

+1 :green_heart:
checkstyle
0m 10s
master passed

+1 :green_heart:
spotless
0m 47s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 29s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 45s
the patch passed

+1 :green_heart:
compile
0m 23s
the patch passed

+1 :green_heart:
javac
0m 23s
hbase-procedure generated 0 new + 43 unchanged - 1 fixed = 43 total (was 44)

+1 :green_heart:
checkstyle
0m 10s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
11m 36s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

+1 :green_heart:
spotless
0m 47s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 33s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

30m  3s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5378/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5378

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 4330ddc34a56 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 5527dd9453

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
79 (vs. ulimit of 30000)

modules
C: hbase-procedure U: hbase-procedure

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5378/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Link not working

Description:
Url "Configuring API Manager for SSO" in the doc https://apim.docs.wso2.com/en/3.1.0/learn/api-security/oauth2/grant-types/saml-extension-grant/ is not working.

Also command line tool in the Download the command-line tool from here and extract the ZIP file. is not working as well

Also command line tool in the Download the command-line tool from here and extract the ZIP file. is not working as well

Fixed in https://github.com/wso2/docs-apim/pull/1048

Description:
Url "Configuring API Manager for SSO" in the doc https://apim.docs.wso2.com/en/3.1.0/learn/api-security/oauth2/grant-types/saml-extension-grant/ is not working.

Fixed in -

https://github.com/wso2/docs-apim/pull/1218
master

https://github.com/wso2/docs-apim/pull/1216
3.1.0

https://github.com/wso2/docs-apim/pull/1217
3.0.0


Search field - add disabled state and stretch sizing

Summary
This fixes #953.  This allows for the search field to be disabled and have it stretch to 100% of its containers width.
Additional Details
Two new props have been added.  isDisabled is for disabling the field and button so you cannot click on it or tab to it.  sizing has been added that will allow you to pass in 'fixed' or 'stretch' for it to work the current way (fixed) or to have it fit its container (stretch).

@emilyrohrbough updated to just use a boolean prop for the stretching, I used the string prop before in case in the future we needed another sizing attribute as discussed in tech design, but agree there should not be any other cases

mjhenkes 39 minutes ago  Owner
What do folks think about changing this to isBlock to be more consistent with other apis, namely button. Would describing this behavior as block be an abuse of the term?
@mjhenkes isBlock would make me believe how it will display next to its peers, currently it is display: inline-flex, which makes it atomic to its peers and always displays inline
I have the width set to 100% to match with the select stying; however, setting it to be display: flex allows it to fill its container as well
If changing the display attribute to be flex, then I think isBlock makes sense as the 'flex' style makes it a block-level element
After thinking more on this, I am more in favor of using isBlock and using display: flex and let it size itself next to its peers rather than always be 100% that maybe could cause display issues that I have been experiencing with the collapsible menu view.

@kschuste I prefer isBlock as well

Functional issue complete.  No issues identified.

Functional verification completed. The search examples display and function as expected, including the new properties.

+1 : Marking as Verified, ready for Release.

Disabled search has funny very minor space/styling  issue to watch for in the future, but all looks great otherwise.

JIRA created


Using the n_of_n_taproot_public_key as the internal taproot key

Why do some of the connectors, such as Connector1, use the n_of_n_taproot_public_key as the internal key?
I thought it could have to do with the optimization mentioned in the third bullet here, but this does not seem to apply here. No leaf allows this key to spend the input by itself, but always in conjunction with some some other condition(s) (e.g., a relative timelock). Therefore setting the internal key to the signers' key gives them more power than they need.
Should a nothing-up-my-sleeve key be used instead, as mentioned on the same link?

Concrete examples:

Connector 1
Connector 2

Looks like that is used to be able to skip the light client phase for testing purposes (as the light client design is not completed yet).

Therefore setting the internal key to the signers' key gives them more power than they need.

I don't think it does. The whole system relies on the assumption that one of the n parties delete their private key as otherwise all n parties can collude to directly spend the locked funds.
Nevertheless, you are right that we could and should use an unspendable internal key as soon as the light client is implemented (so we can mock it up another way) to make sure no one accidentally pre-signs the key spend path.


webpack 4 - Adding minChunks to optimization.splitChunks breaks common chunk

Do you want to request a feature or report a bug?
BUG

What is the current behavior?
When adding the minChunks property to optimization.splitChunks.cacheGroups.X, the common chunk breaks, e.g. either isn't emitted at all, or contains ALL the code of all entries.
If the current behavior is a bug, please provide the steps to reproduce.
This configuration doesn't produce a commons.js file at all, although there are multiple entry points with the same node_modules dependencies (adding test doesn't change anything either).
  optimization: {
    splitChunks: {
      cacheGroups: {
        'commons': {
          minChunks: 2,
          chunks: 'all',
          name: 'commons',
          priority: 10,
          enforce: true,
        },
      },
    },
  },

This emits a common.js that includes ALL the source code. All the other chunks are basically empty.
  optimization: {
    splitChunks: {
      minChunks: 2,
      cacheGroups: {
        'c4.commons': {
          chunks: 'all',
          name: 'c4.commons',
          priority: 10,
          enforce: true,
        },
      },
    },
  },

What is the expected behavior?
I expect that there'd be a common.js that has all the chunks that are used more than once across all entry points.
If this is a feature request, what is motivation or use case for changing the behavior?

Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
node v9.5.0
yarn 1.5.1
Window 10 r1709

    "webpack": "^4.5.0",
    "webpack-cli": "^2.0.14",

Can you create minimal repo, which reproduces your issue. Thanks.

So actually I came across a very strange behaviour while creating the minimal repo. Using 2 simple index.js files with a node_module import worked just fine with the parameters provided in my initial message. Then I realized that it probably has to do with one of the node_modules I'm using (my own in fact). Instead of updating with npm directly, I created symlinks in the node_modules folders of the project.
Before I added the dependency to the symlinked project this was emitted:
commons.js  90.1 KiB       0  [emitted]  commons
 entry2.js  1.66 KiB       1  [emitted]  entry2
 entry1.js  1.66 KiB       2  [emitted]  entry1

and afterwards:
commons.js   180 KiB       0  [emitted]  commons
 entry2.js  1.67 KiB       1  [emitted]  entry2
 entry1.js  1.67 KiB       2  [emitted]  entry1

In my case, entry1 and entry2 both have a dependency to Vue, so it's extracted into the commons.js file. The other (symlinked) node_module also has a dependency to Vue, which is included in the emitted build.js of the node_module. With the node_module dependency added to entry1 and entry2, the commons.js build now includes everything from the build.js and the Vue dependency. So now Vue is included twice, as indicated by the file sizes.
I suspect that this is the problem that also leads to the chunk splitting not working in my (much bigger) project.
You can find the repo here: https://github.com/johannes-z/webpack-minChunks
For my testcases I ran these commands on windows inside the project folder:
mklink /J "./src/app1/node_modules/external" "external"
./src/app1/node_modules/external <<===>> external

mklink /J "./src/app2/node_modules/external" "external"
./src/app2/node_modules/external <<===>> external

mklink /J creates a "Hard Link". This is NOT a symlink. webpack doesn't see that app1/node_modules/external and app2/node_modules/external resolve to the same directory.
You should use mklink /D instead, which creates a "Soft Link" (symlink).

And the first splitChunks snippet is correct.

Ok I will try using /D. Also, I tried finding the new docs for chunks: '...', describing what the difference for all the options were, but couldn't find it. Can you point me to it?

So I don't know if this is relevant but in my real life application case app1

Thanks @sokra!
So I don't know if this is relevant, but in my real-life application, app1 and app2 both had external in their node_modules, rather than the umbrella project. I removed it from there and explicitly added it to the umbrella project, and now the chunks are extracted correctly.


Support for default configuration

Description
I believe there should be the ability for a CLI flag to enable a default config mode (create the config file) instead of prompting, and also support via ENV.
I am setting up new testing renter infra for lume and have found in the process of using the docker image:

It did a constant reboot asking for an API password
Even if it got passed that, it would not generate a seed automatically?

I know that a lot of the current config options will be used internally for the desktop wrapper, so to support more server use cases or even just spinning up for testing in container setups, I would like the behavior to generate a config with:

Sane defaults
API key auto-generation
Seed auto-generation

And put it all behind a flag/env var to be opt-in. After, update the docker entrypoint script to use this.
It makes sense for things to be interactive for hobbyist power users, but this does not work well with a docker setup with no pre-existing config created.
Version
n/a
What operating system are you running (e.g. Ubuntu 22.04, macOS, Windows 11)?
Ubuntu 22.04
Anything else?
No response

Is this somehow different from renterd config? The default config already has relatively sane values. You can even generate the config using a docker container before starting the actual service container:
docker run --rm -it -v $(PWD):/data -w /data ghcr.io/siafoundation/renterd:latest config

If you mean completely user input free seed generation, that's a non-starter for any of our apps, imo. Generating a seed with no user input/verification, saving it to a config file somewhere, and running the service is an accident waiting to happen. If you're setting up renterd as infra, it's not too bad to generate a config somewhere and copy it wherever necessary.

Understood. I did not view this from a security POV on the secrets.
I think the real crux is that I'm using docker-compose right now on a new VPS, and just launching the containers it has assumed user interaction. That and I wasn't aware of the config scaffold command. I do think it might make sense still to have some sort of empty config template be created on the docker setup, excluding any secret/password generation, or at a minimum have skunk_ink add the docker commands to easily reference for getting things configured.
Overall... I get your point, thanks.

Cool, I think the best fix is adding docker/compose setup guides to our docs -- @skunk-ink.
@ChrisSchinnerl -- hostd has an -env flag that disables stdin and shows an error message to create a config file specifically for Docker. I thought we had one in renterd too, but maybe it got lost?

I'm not sure we ever had that in renterd but we can add it.
https://github.com/SiaFoundation/renterd/pull/1139


Add a new block indent style "TabbedTwice"

Most useful for use with 'fn_arg_indent = "TabbedTwice"'
Fixes #957
Ideally we'd have something like TabbedN(u32) or similar, but the configuration_option_enum! doesn't appear to support that yet.

Thanks for the PR! The code looks good, it just needs some tests. Could you add these as well?

@marcusklaas can you give me some pointers on what type of tests you're looking for?
Would just dropping a source file in tests/source and tests/target that shows application of fn_arg_indent = "TabbedTwice" be sufficient? Or there something else you're after?

That'd be great, yup!

This branch is a decent bit behind master, and it seems like the mechanism for adding a new indentation type significantly changed since this PR was authored (or I personally just can't follow the codebase well).
Regardless, here are the files you can use for testing:
source:
// rustfmt-fn_arg_indent: TabbedTwice

fn foo(a: Aaaaaaaaaaaaaaa, b: Bbbbbbbbbbbbbbbb, c: Ccccccccccccccccc, d: Ddddddddddddddddddddddddd, e: Eeeeeeeeeeeeeeeeeee) {
    foo();
}

target:
// rustfmt-fn_arg_indent: TabbedTwice

fn foo(a: Aaaaaaaaaaaaaaa,
        b: Bbbbbbbbbbbbbbbb,
        c: Ccccccccccccccccc,
        d: Ddddddddddddddddddddddddd,
        e: Eeeeeeeeeeeeeeeeeee) {
    foo();
}

That target file doesn't seem quite right.. Shouldn't the first argument be on a new line as well? That way all the arguments will always be aligned.

@marcusklaas only if you enable fn_args_layout = Block. And there is a bug there (first arg is only indented once), which is why I'd held off on completing this.

Closing due to lack of activity, feel free to re-open with a rebase.


Writing signatures honors subkey revocation

Previously we would sign with revoked subkeys
Add a test case
Workaround bug in GPG which doesn't appear to export subkey revocations
along with secret keys in gpg --export-secret-subkey

Plenty of Yaks getting shaven. r? @oconnor663

Jack, can you hold off on this one?  I need to fix some broken stuff here.

Closing this one and reopening a new one.


Remove duplication in comment

Looks like there was a duplication in comment relating to preventing overflow in all browsers - so I removed it for you.

Great catch, @06b. Thanks.


presto-hive: add posibility to specify storage format using hive storage handler class name

format table property accepts predefined literals, like 'ORC', 'TEXTFILE', etc..
Now that property also accepts fully specified storage handler class name.
Specified class must implement org.apache.hadoop.hive.ql.metadata.HiveStorageHandler.
In order to use it you have to put the jar with custom storage handler classes into hive-connector class-path.
Testing: unit tests, integration tests

Replacement for https://github.com/facebook/presto/pull/3094

Replacement for #3094

Closing based on our recent conversation.


Update package.json for saucelabs

https-proxy-agent has a vulnerability. Saucelabs has taken it out of their package, but the version of saucelabs you're using has not removed it yet. If you upgrade your saucelabs package dependency it will eliminate the need for https-proxy-agent and fix the vulnerability.

@googlebot i signed it!

@googlebot i signed it!

This seems to be necessary also for other reasons as saucelabs included in protractor is really outdated: https://github.com/angular/protractor/issues/5419


add module Data.ObjectMap.ST

Hi,
Building an ObjectMap in a purely functional way can be very expensive since insertions take O(n) time.
So, I added a module to be able to manipulate mutable ObjectMaps via the ST monad.
I have also removed non exported modules in test/Main.purs, created a file test.dhall and removed libraries from spago.dhall that only concern tests.
You can run test via spago test -x test.dhall

I have moved ObjectMap definition and related functions to Data.ObjectMap.Internal
So, now, the constructor of ObjectMap is hidden in Data.ObjectMap.
It was necessary to move related functions to avoid class orphans for Index and At.
I have updated README.
Feel free to rewrite it, I am a poor documentation writer.

Great, I added some small fixes, they're in the pr/1 branch right now. If you rebase or merge them into your branch, I'll merge to main and release the next version of the package to the registry.
Unrelated:
While looking at my code again I saw that in the future I may want to get rid of the EncodeJson instance. Opened an issue for this, maybe you have some feedback on this.


Add OpenShift driver

Add a driver for running containers (=notebooks) on OpenShift. This will allow replacing the current self written pool mechanism in docker_driver.
Initial thoughts on design:

interaction through the REST API (https://docs.openshift.org/latest/rest_api/index.html)
create a project per user for isolation
create a robot account in each project
launch a pod per instance, create a route per pod

Open issues:

persistent storage could be provided by claiming a volume per project

this will require persistent projects, too (and of course a lot of administrative stuff)

automate the building of notebook images by doing it from source on demand

this could make the start-up too slow, because not every image is cached on every node beforehand

should we use run once pods with their build in time limits?
can we use egress policies for outgoing traffic and a single project per user group?

for notebooks and with Origin 3.3, yes. If we aim for cluster instances (=Spark), then probably not

@tourunen can we close this or do you want to implement an initial cleanup routine before closing this? Or is this some kind of umbrella ticket for the OS driver and other things?

We can close this and add new ones for feature requests and bugs


V2D enhancements

This is an attempt to collect all the Genetics Variant To Disease Pipeline steps to have an overall description of how it works and to identify points to work on to improve the overall functioning of the pipeline.
2 overall notes:

The pipeline is overall complex in terms that there are many steps involved, when thinking about QC it is important to identify which are the critical points.
The number of outputs is not very big, it is feasible to validate them by defining a JSON schema for each. Is this the right call? @DSuveges

Workflow DAG
Graph of dependencies of the different scripts that are called on the pipeline.

Workflow description
The whole pipeline consists on the creation of 4 files: study, fine mapping, LD and top loci tables.
Top loci table

The result of merging the associations from GWASCat and the summary statistics (merge_gwascat_and_sumstat_toploci).

GWASCat:

Download GWAS Catalogue associations (All associations v1.0.2 - with added ontology annotations, GWAS Catalog study accession numbers and genotyping technology)
Filter down the variant index to only include variants collected in GWAS Cat.

(Problem: this is parsed and joined by reading inputs line by line)
~Bug: cannot run the code without incurring in encoding issue: ValueError: invalid literal for int() with base 10: '\ufeff79254949'~ Fixed by @Jeremy37

Populate variant annotation on the GWAS Catalogue data.
Join GWASCatalogue studies with a table of new studies and format.

Summary statistics:

Associations are coming from the output of the finemapping pipeline

Problem: this file is encoded in the pipeline config to be using 'gs://genetics-portal-dev-staging/finemapping/merged_210515/top_loci.json.gz', is this the most up to date data? Yes, as noted by @Jeremy37, config is updated per release.

Study table - TBC

From independently generating 3 tables: GWASCat, UKBB, Finngen.

GWASCat:

Download GWAS Catalogue studies (All studies v1.0.2 - with added ontology annotations, GWAS Catalog study accession numbers and genotyping technology).
Split studies with multiple traits and merge this table with:

The table containing GWAS assocs enriched with variant information (almost top loci)
Ancestry annotation

Fine mapping table

Download credible set from the fine mapping pipeline.

Problem: this file is encoded in the pipeline config to be using 'gs://genetics-portal-dev-staging/finemapping/merged_210515/credset/_SUCCESS', is this the most up to date data? Yes, as noted by @Jeremy37, config is updated per release.
Enhancement: download_credible_set_directory has some weird logic to download the credible set, apparently the problem is that it is a directory. This should be handled with no problem.
Format fine mapping table.

LD table

Create study/variant/population LUT: The result of inner joining study, top loci and the population map (to map from GWAS ancestry to 1000G superpopulation)
Create text file with all variants contained in the above LUT.

Issue: Instead of using bash to get the list of variant ids in chrom:pos:ref:alt form, implement this step in calculate_r_using_plink and use Spark.

Calculate LD for the variant IDs in the LD lut. This process is paralleled with up to 300 threads.

Problem: PLINK is called once per variant. Is it possible to input a file as explained here?

Generation of LD dataset:

The above LUT is joined with the results of the LD analysis.
Study/population weighted LD is calculated.
Conducts credible set analysis with PICS using the toploci table. This is joined with the above table (LUT + LD)
Output is a table with the studies enriched with the LD and PICS results.

@DSuveges shall we close this one?

Not the entire v2d is implemented so far, but we are so out of scope of this ticket, we should close.


Repository Configuration: Dependency Management

Dependency Management

Splitting apart standalone Jetty dependencies from the war file dependencies in PR: https://github.com/opendcs/rest_api/pull/46

Setting up separate deployments and artifacts for CWMS Oracle vs Postgresql will be handled under the issue: https://github.com/opendcs/rest_api/issues/5


Indicator aiming at the floor

Hi there,
I want to use this tool to learn line-ups in custom games, but it's not properly working for me. I've attached a video with what's happening. Hopefully you know what's wrong. It's telling me to aim at the ground, instead of the sky.
https://streamable.com/jxwbbz

I'm sorry, I cannot access the video. Can you try to send the video again, or using a different method?

I'm sorry, I cannot access the video. Can you try to send the video again, or using a different method?

https://streamable.com/jxwbbz
Sorry, the old link expired.

the same problem, the sensitivity is set in the settings (maybe this happens when you try to throw a lineup from the defense side)

Have you added your own Valorant sensitivity to settings.json? It is by default set to 0.4.

Yes, all the settings are correct. Mouse acceleration is off too. My sens is 0.35 and I changed it in the settings.json.

When you click the right arrow, does the distance in the console match the distance to the spike in-game?

the same problem, the correct sensitivity is set in the settings (maybe this happens when you try to throw a lineup from the defense side)

I believe the issue is the overlay moving down too quickly. I think it might be caused by a mouse tracking issue, and/or a Valorant sensitivity discrepancy.

When you click the right arrow, does the distance in the console match the distance to the spike in-game?

It varies. It's between 0.8 and 1.2 meters off.

Also, try to tab out of the Valorant window while keeping the mini-map and the spike visible. See the explanation video in the repo for reference. Is there any obvious differences between my video and your screen?

The only thing I've noticed is that when I press the right arrow my DPI doesn't change, which could be the issue.

The only thing I've noticed is that when I press the right arrow my DPI doesn't change, which could be the issue.

This definitely sounds like the issue. If the mouse doesn't slow down, it ruins the sync between game and mouse.
Inside the Python file, there is a change speed function.
Inside a new file, could you test the function to maybe find a fix?

Well, never mind. I got permanently banned for trying to use it in custom games.

Well, never mind. I got permanently banned for trying to use it in custom games.

I'm terribly sorry to hear.
I have recently spent many hours testing the tool in customs, and many more for development so this is very shocking.


marker get smaller

Hello,
I am encountering an issue in the usage of the plugin.
I have a map where I have added a marker :

Im my app if I change the marker color and add a circle with the following functions :
await controller.setMarkerIcon(geopoint, blackIcon);
await controller.drawCircle(CircleOSM(
key: "BlackPoint",
centerPoint: geopoint,
radius: 30.0,
color: Colors.white.withOpacity(0.5),
borderColor: Colors.white,
strokeWidth: 0,
));
I have may Marker size being changed (looks like divided by 2) :

Anything I should do ?
Many thanks for your help.

on which platform you faced the issue ?

no issue on android. not tested on apple.
web only.

is the issue happen when you draw the circle or when you set the marker icon ?

another unrelated question are you happy with result from osrm because im working on adding other services like valhala or openroute ?

is the issue happen when you draw the circle or when you set the marker icon ?
if i change a markerIcon with no circle it is working. It looks it is adding the circle that create the image reduction.
another unrelated question are you happy with result from osrm because im working on adding other services like valhala or openroute ?
I have to say osrm is not the greatest but it is enough for our usage.

thnx for your feedback
i will make fix for the bug
and for routing like i said im working on adding more routing api in our routing client

new version has being published 1.3.1

Thanks for your fix.
I was not able to test as I get the following error when upgrading to 1.3.0 :

I have also the same issue in 1.3.1

iwill back home soon i will check it and do fix
sorry for that

technically no i need to fix migration to web package the issue that response from js not what we expect so i will setup https for localhost that i can test getcurrentposition correctly

new version has been published 1.3.2

can you show me more of error trace ?

new version is already published 1.3.3

new version is already published 1.3.3

I will test it today. Many thanks

All good ! It is working and no more issues.


SuiteCRM Version 8.0.0 compatibility

Is this module already compatible with SuiteCRM Version 8.0.0 ?
Thank you!

Hi, there is a kanban item for modules in the description of the SuiteCRM 8 Rodmap, and there will be no sense in this package, so at the moment there are no plans to transfer it to SuiteCRM 8.

Hello there. I updated to 8.0.. 8.1 and Kanban is not present. I read again the suitecrm roadmap and there is no more kanban there!
Any chance to revive this project and make it compatible with SuiteCRM 8+?
Regards!


Include JWForever5504 in manifest.json metadata

I don't know if this is something you want, but I figured I'd check with you. In the manifest, I added my username, since I contributed to this repository. I thought that might be a good form of credit for anyone who contributes. If you don't want this, I fully understand.

yep, its fine


Get extension fails when username in windows contains "ilegal characters"

When attempting to create local container using .ps1 file it fails when username in windows contains "ilegal character: ñ"
My lastname has a "ñ" and it is failing when script attempts to create a container.
Error: Exception calling "GetExtension" with "1" argument(s): "Illegal characters in path." Stacktrace: at New-BcContainer, C:\Users\DanielMagañ\AppData\Local\Temp\f119c7d6-251e-4a52-a8b5-53805a3d6aa5\BcContainerHelper\ContainerHandling\New-NavContainer.ps1: line 1492 at <ScriptBlock>, C:\Users\DanielMagañ\AppData\Local\Temp\f119c7d6-251e-4a52-a8b5-53805a3d6aa5\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 647 at <ScriptBlock>, C:\Users\DanielMagañ\AppData\Local\Temp\f119c7d6-251e-4a52-a8b5-53805a3d6aa5\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 840 at <ScriptBlock>, C:\Users\DanielMagañ\AppData\Local\Temp\f119c7d6-251e-4a52-a8b5-53805a3d6aa5\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 783 at <ScriptBlock>, C:\Users\DanielMagañ\AppData\Local\Temp\f119c7d6-251e-4a52-a8b5-53805a3d6aa5\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 766 at Run-AlPipeline, C:\Users\DanielMagañ\AppData\Local\Temp\f119c7d6-251e-4a52-a8b5-53805a3d6aa5\BcContainerHelper\AppHandling\Run-AlPipeline.ps1: line 731 at CreateDevEnv, C:\Users\DanielMagañ\AppData\Local\Temp\tmpD594.tmp.ps1: line 1598 at <ScriptBlock>, C:\DEVELOPMENT\AL\Dataverse-Sales-Integration-Mexico\.AL-Go\localDevEnv.ps1: line 123 at <ScriptBlock>, <No file>: line 1

Is this the latest version of AL-Go - did you just create this repo?

Hello @freddydk . Thanks for checking in. Yes, its the most recent one released. I just created it this weekend.

Thanks - I did go through everything with special characters a few releases ago - but apparently I missed something:-(
Will check this out.

Hello @freddydk. Apparently, this issue was in my end. I made a clean installation of all components, and it resolved my issue.
Sorry for the distraction. And thank you very much.

Hello @freddydk. Apparently, this issue was in my end. I made a clean installation of all components, and it resolved my issue.
Sorry for the distraction. And thank you very much.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in 3875e6d.


Flags and better luck on first click

Bombs now are been placed after the first player interaction so the first click will always be or an empty space or (only when there are too many bombs) at least an number. Just like the original game for windows.
Also, flags are a thing now.

Great stuff! I will look into it in a bit and merge it. Thanks!

I merged your work into my branch. I did changes though. The unit tests were failing too. I saw you didn't write tests for your code. So I wrote some TODO points in the README including other points that could use improvement:

Create tests for flagging

Create tests for not adding mines on surrounded squares of a position to avoid

Improve code for position avoid regarding surrounding positions. See Grid@createRandomPosition

Prevent infinite loop when creating a grid with more mines

Nice, I will look into those issues when i get time.


Dataset Viewer issue for shainaraza/clinical_bias

Link
https://huggingface.co/datasets/shainaraza/clinical_bias
Description
The dataset viewer is not working for dataset shainaraza/clinical_bias.
Error details:
Error code:   ResponseNotReady

I cannot access https://huggingface.co/datasets/shainaraza/clinical_bias anymore. Did you turn it private or delete it?

The dataset is private and the dataset viewer does not support private datasets.

The dataset is public now and the viewer works as expected.

I'm closing this issue.


Direct support for Direct2D

Currently, the API calls PA_CreateNativePictureForScreen and PA_CreateNativePictureForPrinting return a GDI+ Bitmap or Metafile.
Please add support for Direct2D to these calls. (But do not remove GDI+ support).

Why you still need GDI+ Support? For me, 4D can remove these entry points and provide new D2D-entry points. I don't need GDI+ anymore.

Because I need to support both GDI+ and Direct2D in the same codebase.
There are some things that Direct2D does not do. For example creating a picture with vector data, printing of the plugin area.
 And it is a shame it does not support all the features of GDI+.

There are some things that Direct2D does not do. For example creating a picture with vector data, printing of the plugin area.

All these things i do with D2D. See hmReports. It’s completely Direct2D, also printing and saving to picture files.
Of course, D2D doesn’t support vector images like GDI Metafiles, but for this i export SVG.
Grüße/Regards,
[heubach-media] | Christian Sakowski
<EMAIL_ADDRESS>Tel: +49/(0)40/41 455 455

Am 09.05.2019 um 13:58 schrieb Rob Laveaux<EMAIL_ADDRESS>Because I need to support both GDI+ and Direct2D in the same codebase.
There are some things that Direct2D does not do. For example creating a picture with vector data, printing of the plugin area.
 And it is a shame it does not support all the features of GDI+.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

--
heubach media
Osterfeldstr. 12-14 | Haus 1 | Eingang Nord
22529 Hamburg
tel: 040 / 52 10 59 - 10 | fax: -99
mail<EMAIL_ADDRESS>home: www.heubach-media.de
Geschäftsführer|CEO: Matthias Heubach
Mieten Sie Ihre Computer, iPads & Drucker für Ihre Events bei:
http://www.milo-rental.com
Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen.
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten haben,
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.

This e-mail may contain confidential and/or privileged information.
If you are not the intended recipient (or have received this e-mail in error)
please notify the sender immediately and destroy this e-mail.
Any unauthorized copying, disclosure or distribution of the
material in this e-mail is strictly forbidden.