Can't find element with protractor API after login

I'm getting a weird behaviour when trying to locate an element after a login.
I can see on my browser that the login is successfull and that the app display the homepage. However, when I'm trying to access an element on that page, the test fails.
The thing is, if I use the Webdriver API instead of protractor, the element is found.
it('should redirect to home page when logged', function() {
      var username = ptor.findElement(protractor.By.input('user.name'));
      username.clear();
      username.sendKeys('admin');

      var password = ptor.findElement(protractor.By.input('user.password'));
      password.clear();
      password.sendKeys('admin');

      var submitButton = ptor.findElement(protractor.By.css('.loginbutton'));
      submitButton.click();

//      ptor.driver.findElement(protractor.By.css('.logout-button')).click();//works
      ptor.findElement(protractor.By.css('.logout-button')).click();//doesnt work

    });

I have a similar issue, there is an element that can be found by css and xpath locators in chrome:
http://prntscr.com/ck5t70
http://prntscr.com/ck5tqb
but it doesn't work with protractor:
this.ui.contactsList = $(by.xpath("//div[@class='list-group ng-scope']"));
this.ui.contactsList = $("div.list-group");

Output:

NoSuchElementError: No element found using locator: By(xpath, //div[@class='list-group ng-scope'])
NoSuchElementError: No element found using locator: By(css selector, div.list-group)

Hi there,
I am facing same issue after logging into the application, the element is not identified.
[11:53:47] W/driverProviders - Using driver provider directConnect, but also found extra driver provider parameter(s): seleniumAddress
[11:53:47] I/launcher - Running 1 instances of WebDriver
[11:53:47] I/direct - Using ChromeDriver directly...
Hello
Started
F
Failures:

it should pass it should validate
Message:
Failed: script timeout: result was not received in 11 seconds
(Session info: chrome=74.0.3729.169)
(Driver info: chromedriver=74.0.3729.6 (255758eccf3d244491b8a1317aa76e1ce10d57e9-refs/branch-heads/3729@{#29}),platform=Mac OS X 10.14.3 x86_64)
Stack:
ScriptTimeoutError: script timeout: result was not received in 11 seconds
(Session info: chrome=74.0.3729.169)
(Driver info: chromedriver=74.0.3729.6 (255758eccf3d244491b8a1317aa76e1ce10d57e9-refs/branch-heads/3729@{#29}),platform=Mac OS X 10.14.3 x86_64)
at Object.checkLegacyResponse (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/error.js:546:15)
at parseHttpResponse (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/http.js:509:13)
at /usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/http.js:441:30
at processTicksAndRejections (internal/process/task_queues.js:89:5)
From: Task: Protractor.waitForAngular() - Locator: By(css selector, ..kuiSize)
at Driver.schedule (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver.js:807:17)
at ProtractorBrowser.executeAsyncScript_ (/usr/local/lib/node_modules/protractor/built/browser.js:425:28)
at /usr/local/lib/node_modules/protractor/built/browser.js:456:33
at ManagedPromise.invokeCallback_ (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:1376:14)
at TaskQueue.execute_ (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:3084:14)
at TaskQueue.executeNext_ (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:3067:27)
at /usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:2927:27
at /usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:668:7
at processTicksAndRejections (internal/process/task_queues.js:89:5)Error
at ElementArrayFinder.applyAction_ (/usr/local/lib/node_modules/protractor/built/element.js:459:27)
at ElementArrayFinder. [as isDisplayed] (/usr/local/lib/node_modules/protractor/built/element.js:91:29)
at ElementFinder. [as isDisplayed] (/usr/local/lib/node_modules/protractor/built/element.js:831:22)
at UserContext. (/usr/local/lib/ProDemo/test.js:18:69)
at /usr/local/lib/node_modules/protractor/node_modules/jasminewd2/index.js:112:25
at new ManagedPromise (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:1077:7)
at ControlFlow.promise (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:2505:12)
at schedulerExecute (/usr/local/lib/node_modules/protractor/node_modules/jasminewd2/index.js:95:18)
at TaskQueue.execute_ (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:3084:14)
at TaskQueue.executeNext_ (/usr/local/lib/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:3067:27)
From: Task: Run it("it should validate") in control flow
at UserContext. (/usr/local/lib/node_modules/protractor/node_modules/jasminewd2/index.js:94:19)
From asynchronous test:
Error
at Suite. (/usr/local/lib/ProDemo/test.js:6:5)
at Object. (/usr/local/lib/ProDemo/test.js:5:1)
at Module._compile (internal/modules/cjs/loader.js:774:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:785:10)
at Module.load (internal/modules/cjs/loader.js:641:32)
at Function.Module._load (internal/modules/cjs/loader.js:556:12)

1 spec, 1 failure
Finished in 18.421 seconds
[11:54:07] I/launcher - 0 instance(s) of WebDriver still running
[11:54:07] I/launcher - chrome #01 failed 1 test(s)
[11:54:07] I/launcher - overall: 1 failed spec(s)
[11:54:07] E/launcher - Process exited with error code 1
npm ERR! Test failed.  See above for more details.
suneels-MacBook-Pro:ProDemo suneelyadava$

"use strict";
exports.__esModule = true;
var protractor_1 = require("protractor");
console.log("Hello");
describe("it should pass", function () {
it("it should validate", function () {
protractor_1.browser.get('https://<IP_ADDRESS>:32666/logon');
protractor_1.browser.manage().window().maximize();
protractor_1.element(protractor_1.by.xpath('html/body/hzn-root/hzn-logon/div/hzn-enter-username/section/kui-input-box/form/kui-input/div/input')).sendKeys('admin');
protractor_1.element(protractor_1.by.name('username')).click();
protractor_1.element(protractor_1.by.tagName('input')).isElementPresent(protractor_1.by.tagName('name'));
protractor_1.element(protractor_1.by.tagName('input')).sendKeys('admin');
protractor_1.element(protractor_1.by.name('password')).click(); //Complete Login
//browser.refresh();
//element(by.className('time-zone')).click()
//element(by.xpath("//kui-icon[@kuiicon='user-profile']")).click();
protractor_1.element(protractor_1.By.id("Light")).click();
protractor_1.element(protractor_1.By.className(".kuiSize")).isDisplayed();
var a = true;
expect(a).toBe(true);
});
});

Please help if someone has solved this.

When I run same stuff with selenium, this works well using java


Mount name optional

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #2420
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/test fluid-e2e


Feature/languages

Codecov Report

Merging #19 (f859870) into develop (cbc1e8b) will increase coverage by 0.49%.
Report is 1 commits behind head on develop.
The diff coverage is 86.36%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           develop      #19      +/-   ##
===========================================
+ Coverage    90.56%   91.05%   +0.49%
===========================================
  Files           74       76       +2
  Lines         1335     1353      +18
  Branches       262      262
===========================================
+ Hits          1209     1232      +23
+ Misses         117      114       -3
+ Partials         9        7       -2

Files Changed
Coverage Δ

src/interfaces/Schema.ts
100.00% <ø> (ø)

src/interfaces/Seo.ts
100.00% <ø> (ø)

src/services/api/modules/orderStatuses.ts
100.00% <ø> (ø)

src/services/api/modules/pages.ts
100.00% <ø> (ø)

src/services/api/modules/productSets/index.ts
100.00% <ø> (ø)

src/services/api/modules/products/index.ts
70.00% <ø> (ø)

src/services/api/modules/schema.ts
100.00% <ø> (ø)

src/services/api/modules/globalSeo.ts
66.66% <57.14%> (ø)

src/interfaces/index.ts
100.00% <100.00%> (ø)

src/interfaces/languages/index.ts
100.00% <100.00%> (ø)

... and 3 more

... and 2 files with indirect coverage changes


Open url improvement

It should not use the password from a host lookup that stored a password in case of a rogue server is being used

Done


Obs submission not working

It keeps spinning on submission page with Cancel submission button and spinny wheel. If I navigate away from that tab and come back to it later, the state doesnt seem to change.  (I can also edit fields in the submission which I probably shouldn't be able to do?)
if I click the back button next to "submit observation" header at the top of page that takes me back to the main obs screen and if I go to submit a new obs my previous one appears to have been lost.

You get no toasts showing up, right?

correct, no other screens just the spinning symbol on the cancel button

(I can also edit fields in the submission which I probably shouldn't be able to do?)
I created #428 to track this separately

The bad submission behavior should be fixed in #424 . I suspect the base issue here was that we weren't supplying the new obs_source param which became required.


Race condition signal: killed

Possibly duplicate of https://github.com/roboll/helmfile/issues/1166
I have a k8s cluster that runs CI for several components and there can be up to 50 parallel helmfile commands running across 8 nodes. Here's the failure I'm seeing during the execution:
FAILED RELEASES:
NAME
mkt
in product/helmfile.yaml: failed processing release mkt: command "/usr/bin/helm" exited with non-zero status:

PATH:
  /usr/bin/helm

ARGS:
  0: helm (4 bytes)
  1: upgrade (7 bytes)
  2: --install (9 bytes)
  3: --reset-values (14 bytes)
  4: mkt (3 bytes)
  5: stable/mkt (10 bytes)
  6: --wait (6 bytes)
  7: --timeout (9 bytes)
  8: 1500s (5 bytes)
  9: --create-namespace (18 bytes)
  10: --namespace (11 bytes)
  11: mkt-60-4b7 (10 bytes)
  12: --values (8 bytes)
  13: /tmp/values386558221 (20 bytes)
  14: --values (8 bytes)
  15: /tmp/values491133192 (20 bytes)
  16: --values (8 bytes)
  17: /tmp/values623731911 (20 bytes)
  18: --history-max (13 bytes)
  19: 10 (2 bytes)

ERROR:
  signal: killed

EXIT STATUS
  -1

I should stress that each container has limits of 3 vcpu and 2gb memory. The helmfile that I use has a dependency chain and installs in 5 phases: 2 charts, 3 charts, 2 charts, 3 charts, then finally 7 charts. this "mkt" chart is number 6 out of 7 of the last group.
Additionally there are no process limits and max open files is 1048576.
I'm running a relatively old version of helmfile. I'll try an upgrade to see if anything changes
helmfile v0.130.0
helm v3.5.2

@mumoshu Any chance you can help out here?

@mattymo Hey! Sorry I had been blown away by the big waves of github notifications.
First of all, is this caused by a race condition? At glance it looks like your helm process is getting OOM killed due to helmfile trying to run dozens of helm processes in parallel. If that's your issue, you can just set lower concurrency like helmfile --concurrency 5 apply

Lowering concurrency helped

Not sure why I didn't come up with this idea but I believe we'd better default to a smaller concurrency, perhaps to the number of cores, rather than the number of releases defined in a said helmfile.yaml which can be extremely high.

Hello! Is it possible to re-open this issue?  We're encountering the same problem.  We have 121 releases (some with deps on others), being executed on a 16-core / 64 GB RAM VM, with --concurrency 16.  It seems to happen almost always 3 minutes into execution.  But we are able to get a successful run after enough retries.  The behavior feels like a race condition, as the author of this issue alluded to.  Would you be able to take a further look at things @mumoshu ?

Please post a new issue in helmfile/helmfile


Update GB (+44) carrier data from Ofcom

Uses the most recent carrier data from Ofcom, with carrier names being changed to friendlier versions (brand/trading names).
Data has also been compressed to reduce duplicate prefixes, although the script has compressed it further than it used to be.
Generated using https://github.com/giggsey/GBCarrierGenerator (so can be run again).
A few questions about the company names:

[ ] The current data converts Hutchison 3G UK Ltd to Hutchison. Their commonly used brand name is 3/Three.
[ ] Vectone Mobile Limited is called Mundio in your data (from Wikipedia: Vectone Mobile is operated by Mundio Mobile Ltd). However, their website just calls themselves Vectone Mobile
[ ] M P Tanner Limited t/a FIO Telecom has been translated to their trading name FIO Telecom

Thanks for this!
a) Please do change Hutchison to Three everywhere. or 3. Um. not sure which is best. Probably "Three". They say "About Three" on their website.
b) Fine to change Mundio to Vectone Mobile
c) Not sure what you think we should do for the last one?
In the file, can you add comments at the top for any translations you do from the data, and update the link to ofcom data if it has changed? A note about the compression technique used would be also helpful. We actually have an online app somewhere that can be used, but yours looks sane.
Also delete the empty blank line at the bottom thanks :)

Regarding item 3, it's more of a note instead of a question.
I'll make the changes, and update shortly (might be tomorrow).
As for the compression, do you have a link to that online tool? I'm slightly concerned that I compressed too hard. I sanity checked some ranges by hand, and they appear okay.

It looked okay to me :)
We have:
http://libphonenumber-combine-geodata.appspot.com/
I think it's even more aggressive than what you have though, there are definitely known issues about using it...

It's too aggressive, we don't even use it for changes we make - and the last time I wished it was better was actually for this same file :)
E.g. it compresses
331|Paris
334|Paris

to
3|Paris

Updated Hutchison to Three and added a comment about how the data was generated.
I've also rebased to upstream's master.

Last comments addressed.


Dynamic CLR

Hello
How are you?
Sorry to emit this issue here.
I read your blog "Exploring Super-Convergence" which explained an auto dynamic CLR method.
I could NOT find any source code for this method.
Could u share the code?
Thanks

Hello,
Sure, there is a public colab notebook available. https://colab.research.google.com/github/deepmipt/DeepPavlov/blob/master/examples/super_convergence_tutorial.ipynb


Basic http auth

PR for issue #11.

I switched the main branch to develop, I will stop pushing directly to master as well.
I'll only merge to master from develop when things are "stable".

😂

edit code
run the app
test what you changed
commit

Alright, I fixed and cleaned up a little bit.
As a suggestion, digging around the Sinatra Docs and how to work with modular apps is gonna help you understand better how to use it.
When it comes to commits, there were a couple of inconsistencies on structure and logic that I think should be easy to realize if you spend more time actually testing what you code before you commit.
I don't really mind correcting this issues, I just say that so you get a better understanding and don't think that I just wanna go over your commits.

It worked on my side though.
But thanks for your advices. As you can see I'm still a beginner when it comes to the points you mentioned. So you offering me an opportunity to learn more is the more I could have hoped for.
I don't mind my code being fixed by someone else. I know I have flaws. I'll try to slow down a bit so I can fix them.

Sure, I'm a coach IRL so no worries, I just don't want you to lose motivation :D
I forgot to mention, there's no point in adding a "web only" mode, because the core idea of this project is to use a discord bot.
Evidently, in the future the web UI is gonna completely required, but for development purposes of pure bot features, is better to disable the web app, on the other hand if you disable the bot, the web is completely useless.

I did that so I can test it without running the bot. Of course I could just run Sinatra in Pry.

Ah, in that case you can just comment the bot line :D

I did that at first ;) I use Pry now I should not have commited that.


Interact in main pane?

Another question, @gencay: Interacting with the extension in the left side pane feels awfully crammed. Can I have the extension interact in the main pane?

Hi @ymyke, thanks for the suggestion. However, it's not an option at the moment since creating a conversation window on a file tab wouldn't be as reliable as a reserved sidebar. Also the UX is heavily optimized for a sidebar at the moment.
Why would you want to interact with it on a main pane? I believe for the purpose of vs-code and having the conversation about your code, where both are visible at the same time is more valuable compared to switching back and forth between tabs on the main pane. If you could elaborate a bit more on the need and the use cases, I may consider looking into it in the future.

I work on a rather small screen and it's hard to interact with the small sidebar.
See here:

Also, I wouldn't want to make the sidebar larger because then I'd have to constantly resize it when using different content such as file explorer or git.
When I work with Jupyter / Interactive Windows, my main pane gets split in half and I have code and interactive window side by side. I like that way of working. Maybe an option for your extension too?

It's not viable to keep a conversation on a dynamic window aka editor. User may close it unintentionally and thus requiring them to re-login. One thing I am thinking that may improve your experience would be moving the copy/insert/new buttons above the code blocks just like the chatgpt web. Would that address your issue? I don't have any plans to move the conversation window to an editor for a full-page experience. I'd rather have a static and more reliable real estate for the extension.
Let me know your thoughts about the copy/insert/new buttons moving above code-blocks. That may be something I'd consider making customizable.

I see your reasoning. I would leave the buttons where they are, that doesn't make much of a difference for me.

@ymyke Check out the newest version on marketplace and see if it looks better. I made some UX improvements, moving the code action (copy, insert, new) buttons to above code block for better visibility.

Great improvement, thanks @gencay !


Not informative output if the test is broken

Conversion completed successfully
One or more go tests failed. exiting with error 1

Not confirmed


WSL command history location?

Had a query with regards to locating the WSL command history file, similar to how Bash and PowerShell entries can be found at %USERPROFILE%\.bash_history and %APPDATA%\Microsoft\Windows\PowerShell\PSReadLine\ConsoleHost_history.txt respectively. My configs are as follows:
> wsl -v
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.19045.5011
> wsl -l
Windows Subsystem for Linux Distributions:
docker-desktop (Default)

history lists previously used commands in the following format:
:/mnt/host/c/WINDOWS/system32# history
   0 cat
   1 exit
   2 help
   3 history
   ...

I've surfed over the official WSL docs and net in general but to no avail. Also skimmed through and searched for issues to see whether the same question was already asked or mention has been made about it being stored within the Docker volume directories (\\wsl$\docker-desktop\...).

/question

@m1younis keep in mind that history is a function of the shell

@elsaco any idea where it executes though?

Dug into the distro and found a file named .ash_history (as described in the opening comment) within \root\ and \mnt\host\wslg\distro\root\, hope those with the same question can benefit from this.


Support concrete IEEE754 encoding.

We finally added concrete IEEE754 encoding to check the performance degradation compared to abstract FP encoding.
This option can be set by giving --use-concrete-fp to mlir-tv

Thank you!
I will review this patch today night.


Update Handle Version to 6.2.5_03 for DSpace 6.x

References

Fixes #8557

Description
Changes the Handle version to the updated version for JDK 1.8
Instructions for Reviewers
Please add a more detailed description of the changes made by your PR. At a minimum, providing a bulleted list of changes in your PR is helpful to reviewers.
List of changes in this PR:

Changes Handle version from 6.2 to 6.2.5_03

Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[ ] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[ ] My PR passes Checkstyle validation based on the Code Style Guide.
[ ] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[ ] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[ ] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[ ] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[ ] If my PR fixes an issue ticket, I've linked them together.

Thanks @misilot for creating this PR as a "template" for those running DSpace 6.x and requiring a patch for issue #8557.
As noted in the latest comment in that ticket, this patch can be easily applied manually.
Closing this PR as we will not merge it.  However, the code in this PR can be manually applied for sites that need it.  Several institutions have verified that this PR fixes issue #8557 for any 6.x affected sites.


[LS] New import syntax fails

Possible regression.
Importing with the new import syntax in scripts has some issues.
Here's a code example to reproduce:
import "Bar"

pub fun main(a: Int, b: Int): String {
    return Bar.X
}

The output looks like:

And the error is:
checking of imported "pub" program failed.

Not valid, vscode was opened in a folder that was not the same as the root of the project.


docs: alternative for alpakka-samples links

https://akka.io/alpakka-samples/ -- we don't have a pekko equivalent yet

https://github.com/apache/incubator-pekko-connectors-samples needs work
We might want to just change the link text to acknowledge that the alpakka-samples are useful for now (as they show the concepts and migrating to pekko shouldn't be hard)
In a few weeks, we might have the pekko-samples anyway

I can't find any links in the repo that are affected


Github Issues connector

Added an Issues connector to the github community connector! This also includes a large amount of generic code related to having multiple types of data in a single connector with a single config (see also https://github.com/Lucretiel/datastudio/tree/connectorHelper for more work along this line).
Note to reviewers while I did my best to resuse existing Stars Connector code, this is functionally a total rewrite of the github connector, so the diff isn't going to be much help.

Two main things:

:fire: The documentation is excellent
:clock1: It's probably worth looking over some of this keeping in mind that all the code has to get run for every getDataCall. Some of the abstraction might negatively affect performance.

This solves #24


the performance of your released occupancy model tpv04_occupancy

hello, i run the inference of tpv04_occupancy with command, the gpu i used is v100
python eval.py --py-config config/tpv04_occupancy.py --ckpts ckpt/tpv04_occupancy_v2.pth
the performance on nuscenes is shown in the picture.
the performance is not matched with paper released and could you please explain the evaluation metrics of miou vox/pts, i'm confused about it.

We use different supervision signals for lidar segmentation and occupancy prediction, and thus yielding different model weights for these two tasks. For lidar segmentation, lovasz softmax loss supervises point predictions and crossentropy loss supervises voxel predictions. While for occupancy prediction, both of them supervise voxel predictions.


VirtualBox requires WHQL drivers for VMSVGA so the old drivers must be used on Windows 8.1 Host

Does VMSVGA10 actually use DX12 so there is no luck of getting it working on Windows 8.1 with DX11? I installed Windows ME on the latest version of VirtualBox 7.0 but stuck with llvmlipe. My GPU is GTX 1650S, CPU is i5-12400F.

Seems like I can't even get VMSVGA9 working. For some reason I'm still stuck with llvmlipe...

VBox.log

From looking into log it definitely denies loading nvidia driver, and says no 3d acceleration.

Installing the last available WHQL Nvidia driver (dated 2021, instead of the new cringey unsigned ones) helped.

There are three ways of fixing the problem:

Compile VirtualBox without HARDENING (very difficult and requires TestSigning/Disable Driver Signature boot options, both can break different software like EAC on the Host machine).
Use WHQL drivers (the old ones from 2021, as the new ones are unsigned; Microsoft calls Windows 7 and Windows 8.1 EOL systems and doesn't allow to sign new drivers for them).
Use Windows 10 or Windows 11.
The 3rd solution is probably the most obvious and easiest, but I personally call the 2nd one best though "dangerous". Anyway, its never safe to use an EOL system, so... its better stick to a modern OS if its possible. VBox would be much better without its Hardening though. Unfortunately, the devs will never let us to turn it off officially.


fix: guard for xrandr not available

use which to check if xrandr is available.

this was tested with current unstable package of mdd and works. To test uninstall xorg-xrandr from your system.


Broken image

I have just checked the live demo page: http://castaway.xesf.net/#entry=MJFISH.TTM
But it has the same issue.
Were you tried in your own install?

I think we do have some issues with contexts between scenes causing this.
The latest version is not the same as the live demo version so I am not sure this still remains.
The whole scene context and ads player needs to be rebuilt and simplified, there are a lot of issues with the way I have shared the objects between the calls.
There is actually a better implementation in C of the player here: https://github.com/jno6809/jc_reborn

Thanks! Unfortunately I am on macos and I cannot compile it. This is why it was so cool to make it on a webserver on a linux box and publish.
I have tried to use the latest code, and it is partially working, but buggy as you said.
I wish you have time to one day refresh the project again. I will watch for sure.
Thanks!

Ohh! If you have compiled version of jc_reborn for mac can you share somehow?
I am glad that you want to work on this web implementation. I can see also a better choice this one.

It is not a packaged version so not sure you will need some dependencies, but I can check what I have later and send it to you. I don't have my laptop with me know where I have those.


Haskell backend is very slow to take steps on C semantics

I am working on getting symbolic execution working for the C semantics, but progress is very slow because symbolic execution is incredibly slow, to the point of taking hours on even incredibly basic C programs.
I am sharing two C programs for reference in the attachments, in the form of bug reports containing the definition.kore and initial kore configurations. The first is simply int main() { return 0; } and takes roughly 10 minutes to execute, despite only having roughly 100 steps of execution. The second is slightly more complicated:
#include <rvmatch/symbolic.h>

int main() {

  int x;
  if (symbolic(int) >= 2)
    x = 1;

  x; // maybe uninitialized

  return 0;
}

symbolic is a macro which is implemented in such a way that symbolic(int) returns an rvalue of type int whose value is a variable. This program is also highly simple, but it will run for multiple days without finishing. For comparison, both programs run on the llvm backend in less than a second.
Even though I shared the bug reports with definition.kore, I strongly suggest running kore-exec --serialize and using the serialized definition before actually profiling them. The placement of the kore definition and other initial information into a compact region has significant impact on garbage collection throughput beyond simply the additional ~40s of bootup time it takes to construct the initial definition from scratch.
Any insight into where time is being spent would be valuable, as I am having trouble figuring out how to optimize these programs further.
smoke.tar.gz
maybe-uninitialized.tar.gz

Some leads have already been identified for why the programs are so slow, see the related Slack discussion.
Key postings:

Program takes 30 seconds with Ceil simplification forcefuly disabled.
Issue #3136 was created to address this.
A possible short term solution is to have a front-end flag that omits all rules relating to Ceil simplification from definitions.kore.
https://github.com/runtimeverification/k/pull/2768 implements this as a proposal.
MultiAnd.filterAnd appears to be another bottleneck, This is being addressed in #3099

Note: there are two programs here. I have not tested the second one with ceil simplification fully disabled; but the short term fix pr I created only optimizes the second program up to about 100 steps; after that, each step starts taking over a minute, even with no ceil simplification axioms.

Also note that the flag --disable-ceil-simplification-rules does not work for any non-trivial proofs, so it's not a long term fix and I don't think is even worth merging actually. It's good to have for experiments.
As far as I understand it, the problem is this (@dwightguth please confirm):

Simplification is called on some term, which triggers the equation application.
Equation application triggers a call to the Ceil simplifier.
The Ceil simplifier searches for user-supplied rules with #Ceil at the top, and tries to apply them, using the same equation applicaiton procedure as (1). It's this recursive call to equation application procedure that is slow, because it can retrigger this entire process.

Maybe a better fix is to make a specific "Ceil equation application procedure", which does not have the full generality of the normal equation application. In particular, it should never recurse back into the ceil simplifier, it should just do one level of applying the user-supplied #Ceil equations and then quit. This should allow users to supply Ceil rules (needed for any non-trivial proofs), but stop the bad performance case here.

As discussed, profiles for the programs here show that much time is spent in hashWithSalt @ TermLike.
An initial experiment with a SimplifierCache that used a simple (Ord-based) Map still showed hashWithSalt @ TermLike as the biggest eater in the profile (smoke program), most likely due to SideCondition whose contents could not be converted easily to use Map/Set instead of HashMap/HashSet.
Following some discussions on slack and in person, I am attempting to cache hashes for all TermLike data to avoid recomputation, in https://github.com/runtimeverification/haskell-backend/compare/jb/caching-termlike-hashes (WIP, failing tests at the moment)

Are we able to delay simplifying the generated #Ceil conditions, instead of immediately discharging them? Could we store them in a list, and then cycle over them all afterwards one at a time separately? Then we could have a simpler rewriter used for that last pass of simplification of the Ceil conditions, which would probably be enough for a lot of cases.


receptive field  of the generator

Though search in the internet, I am still confused about the computation of receptive field of convs + deconvs network. Could someone explain how to compute the receptive field of convs+deconvs network. Or just explain the receptive field of the Generator of Hifigan.

According to this paper， https://arxiv.org/abs/1705.07049 ， receptive field of the generator can be computed.


payback loan - allow payment of more than the outstanding amount

What would you like to be added:
In some cases, interest might kick in after payment of full outstanding balance. This will cause an infinite loop where users cannot pay a small amount of their outstanding amount.
Must allow users to be able to pay more than their outstanding balance. Need a warning message to display in this scenario. Excess payment will be returned to their wallet after the transaction has been done.
Why is this needed:

Linked to https://github.com/DeFiCh/wallet/issues/1663


Add CODE_OF_CONDUCT.md file

Hi there!
I recommend adding a CODE_OF_CONDUCT.md file to your repository. This file would serve as a guide for potential contributors, providing them with a clear set of expectations, guidelines, and behavior standards for participating in your project. It plays an essential role in creating a welcoming and inclusive environment for everyone involved.
I would like to work on this issue. Please assign it to me under GSSoC'23.
Thank you!

Assigned this to you 🚀
Specify the less obvious things rather than common decent human values that mostly everyone does anyway.


Evaluate AWSGraphQLSubscriptionOperationCancelTests for cause of sporadic failures in dry runs

Describe the bug
These tests have failed multiple times recently possibly due to a race condition. Temporarily the timeout could be increased but if it is a race condition it is possible even more time would still result in tests failing. These tests will need to be evaluated to reproduce these sporadic failures locally to observe the stack frames when the tests fail and determine what can be done to fix the problem.
To Reproduce
Steps to reproduce the behavior:

Run these tests repeatedly (use performance loop features of XCTest)
Run until tests fail like in dry runs
Analyze the stack frames to see where threads are blocked
Determine how to eliminate the race condition

Expected behavior
These tests use expectations which are fulfilled so they should not fail based on timing. They should run quickly and fulfill the expections before the timeout expires.
Screenshots
Environment(please complete the following information):

Amplify Framework Version: [e.g. 1.5.3]
Amplify Categories:

[ ] Analytics
[ ] API
[ ] Auth
[x] DataStore
[ ] Predictions
[ ] Storage

Dependency Manager: [e.g. SwiftPM, Cocoapods]
Swift Version: [e.g. 5.0]
CLI Version: [e.g. 4.41.2, Run amplify version]
Xcode Version: [e.g. 12.4 (12D4e), Run xcodebuild -version]
Include any relevant log output under ~/.amplify/logs/amplify-cli-<issue-date>.log

**Is this a regression (i.e. was this working before a version upgrade?)
If applicable provide the versions used before and after the upgrade.
Device Information (please complete the following information):

Device: [e.g. iPhone6, Simulator]
iOS Version: [e.g. iOS 11]
Specific to simulators:

Additional context
Add any other context about the problem here.

Failing dry run

testCancelSendsCompletion
> Asynchronous wait failed: Exceeded timeout of 0.3 seconds, with unfulfilled expectations: "Received value for disconnected".

I haven't been able to reproduce this failure locally or on CircleCI.  I added a performance measure block to the test and ran it several thousand times locally.  I also pushed a branch with the modified test and it ran successfully 1,000 times on CircleCI.  https://github.com/aws-amplify/amplify-ios/blob/chore/testAWSGraphQLSubscriptionOperationCancelTests/AmplifyPlugins/API/AWSAPICategoryPluginTests/Operation/AWSGraphQLSubscriptionOperationCancelTests.swift
Looking into this a little more, I noted that a subset of CircleCI's macOS fleet was undergoing emergency maintenance at the time of the failed test.  It's possible that this contributed to the failure:
https://status.circleci.com/incidents/p6km16s326m5
This emergency maintenance occurred multiple times from 28-30 June.
https://status.circleci.com/history
Closing this issue for now.  We should re-open if these failures happen again.


Introduce 'Nancy' as core for APIv2

Using 'Nancy' for best cross-compatible. Also introduce easier route making for API. Currently port is set to 1234, but eventually will be change. This commit don't break anything, and code that is used as base for current API was untouched. This is push-ed now to avoid future compatibility breaks that could occur if someone would mess with core-api objects.
This is a introduce for APIv2

Fix'd against conflict

pushing to separate branch


Do not use logging.basicConfig and instead use NullHandler for logging in the library.

This supercedes https://github.com/fgimian/easysnmp/pull/11.

Coverage increased (+0.45%) to 97.77% when pulling 4af340602da2805b55269fb299c332591e0ffd71 on nnathan:fix_logging into 08b221b46d3629af5f4cdeb30532d5a9b67bf0e8 on fgimian:master.

Thanks mate :smile:


The app is crashed on disconnecting WC and switching account in MM

Description
The app is crashed with the error on screenshot
Environment

Browser: Chrome
Wallet: MetaMask mobile app connected via WC and MM extation in chrome
Safe: 0x9DF58F04F3e914ED8D972146401C85313A53FecA
Environment:

production (xDai)

Steps to reproduce

Connect MM mobile app via wallet connect to the safe on xDai
2. TBD ( some extra steps here that is the main reason of crash)
disconnect WC
go to the MM extatntion and change account there without connecting to safe ( may it be because the same account was in mobile MM and in extantion MM?)

Expected result
No crash
Obtained result
The app is crashed
Screenshots

Might be fixed by the recent onboard update? Have you checked on the l2-ux branch?

Tried to reproduce this issue in the PR https://github.com/gnosis/safe-react/pull/2939 that has un update for the onboardjs.
The issue is not happening anymore for me


Add "video" to attachment list

This switch statement didn't account for "video" attachments. See this gist for a full video object returned from the Facebook API: https://gist.github.com/bbatliner/c5c702b9dc9c07c080c3. Let me know if you need/don't need any of the properties.

Thanks, @bsansouci just added b177725 to match this on the sendMessage side.


Sungrid input from command file instead of command line

Signed-off-by: Sharon Topaz<EMAIL_ADDRESS>Updating SGE to use a script that contain the execute command lines as input.
Before SGE (qsub) got directly the execute command line with all the arguments on the same line.
There is a limit of 4096 characters in system buffer size. If the command line contains more than 4096 characters there is an error message:
Error : Unable to run job: a path or filename may not exceed 4096 characters\nExiting.\n'
Updated the code so all our commands will be passed by script to qsub command.

Another small update so the log will be created correctly.

@matutem Can you please review ok and merge if it ok?

@matutem Reminder:)

Thanks @matutem. Can you please help me and merge? I don't permissions to merge.

@weicaiyang @msfschaffner Maybe you can help to merge please?


new skipLabels option doesn't work with labels with spaces

Describe the bug
I used to have a setting of skipLabel: 'no changelog entry needed' in my config file. I cannot upgrade to v1.6.0 because the new version parses a setting of skipLabels: 'no changelog entry needed' as Skip Labels: no,changelog,entry,needed (as per the job log).
To Reproduce

Create a label "no changelog entry needed"
Setup a GitHub action like this:

name: Changelog Check

on:
  pull_request:
    branches: [ master ]
    types: [ opened, synchronize, reopened, ready_for_review, labeled, unlabeled ]

jobs:
  check-changelog:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2

      # Gives an error if there's no change in the changelog (except using label)
      - name: Changelog check
        uses<EMAIL_ADDRESS>        with:
          changeLogPath: 'CHANGELOG.md'
          skipLabels: 'no changelog entry needed'

Result: PRs with the "no changelog entry needed" label still don't pass CI.

Expected behavior
I thought the new behaviour was to only split on comma, not on spaces too.
A link to or sample of your workflow
https://github.com/robotty/simple-process-stats/blob/7ed34ca9718b82e942e4a6483b20fd3a5faa1332/.github/workflows/changelog-check.yml
https://github.com/robotty/simple-process-stats/pull/9
https://github.com/robotty/simple-process-stats/pull/9/checks?check_run_id=1563404236

Perhaps the skipLabels input should be a yaml list, not a string?

👋🏼  Hi Ruben.
Thank you for your thorough report on the bug and sorry for its existence. Spaces within a label should be supported and I'll get to working on this right away.
As far as passing a list as a yaml list, I believe GH Actions only support string key/value pairs as inputs. This is based on the reading in this community thread and how inputs as passed themselves as env variables.

I was about to file an issue about his, but @RAnders00 beat me. Once you fix this, some of my users will be happy: They've wanted me to expand the labels that don't need a changelog change because the enforcer is annoying them. (But, well, that's the point is what I say 😄 )

@RAnders00 I believe I have fixed this in the PR linked above. If you would like, you can test using the version with the fix by changing your workflow to use it explicitly by doing
uses: dangoslen/changelog-enforcer@issue-58
If you could try that out and let me know that would great!

@dangoslen Worked on my test: https://github.com/GEOS-ESM/GMAO_Shared/pull/147/checks?check_run_id=1566548789
We'll see if @RAnders00 sees the same. Thanks!

Works for me too! 👍🏻

Thank you both for checking! I'll merge that PR and will create a new v1.6.1 version


Mostly cleanup patches

Noticed some of these while doing other stuff and a lot more after finding that the toolchain had -Wno-unused-variable set.
Last commit fixes the initial flash of uninitialised data after being powered off (by clearing LTDC ram when persist needs reset). It does add a warning about writing out of bounds that I can't seem to avoid.

Thank you!


Update loading of gltf animation.

This is to account for GLTF info being more like instructions on how to build your animation instead of verbose description of each pose.
Fixes #1553

PS: Also added a few comments to explain the animation loading.

@raysan5 This fixes the issue where only 2 frames were switching in the example by sampling a constant of 30 poses per second.

@object71 thanks for the update! yeah, I saw the issue! thanks!


[Build logic] Relying on Gradle script to generate plugins is slowing out the build

Summary
This project is using <plugin id>.build.gradle.kts in a module to let Gradle generate those convention plugins.
This process is really slow and explicitly and manually writing those plugins should be done instead.
Profiling
cf #38 for the before and after using
./gradlew clean && ./gradlew app:assDebug --profile --offline --rerun-tasks

generated plugins
explicit plugins

Total Build Time
25.373s
11.205s

Startup
0.263s
0.265s

Settings and buildSrc
0.003s
0.003s

Loading Projects
0.009s
0.007s

Configuring Projects
12.724s
0.765s

Artifact Transforms
0.562s
0.064s

Task Execution
42.970s
34.711s

(On a bigger app at my previous company, the configuration was around 1 minute long with scripts vs a few seconds, unfortunately I don't have access to the benchmarks I made anymore)
Here's the profiles:
profile.zip

Lovely! I'd incorporate this in one of my projects. Cool!


Problematic URL encoding for query string params with multiple values

Issue summary
The common way to pass multiple values under one query string parameter is the following:
http://www.example.org?param[]=foo&param[]=bar
When I store such URL in Schemes\Http, [] part of the query string undergoes normalization (RFC3986) and is url encoded.
[ -> %5B
] -> %5D
Is that expected?
System informations
Irrelevant.
Standalone code, or other way to reproduce the problem
$uri = League\Uri\Schemes\Http::createFromString("http://www.example.org?param[]=foo&param[]=bar");
var_dump($uri);

Expected result
class League\Uri\Schemes\Http#3 (1) {
  public $uri =>
  string(54) "http://www.example.org?param[]=foo&param[]=bar"
}

Actual result
class League\Uri\Schemes\Http#3 (1) {
  public $uri =>
  string(54) "http://www.example.org?param%5B%5D=foo&param%5B%5D=bar"
}

@mirfilip this is expected behavior  the [, ] characters are reserved characters  according to RFC3986 they must be encoded whenever you encounters them in a URI except for specific usage.

Fair enough @nyamsprod. Thx


Shard the STRINGS in the intern cache for better parallel scalability

(This is an improvement to the approach in #162 that is, however, probably not yet ready to merge. Posting it for discussion.)
The 20% of runtime in egg::util::intern with #162 is still pretty high. This PR instead shards the cache to reduce mutex contention in the presence of multithreading. With it, in the context of my application, it further reduces the percentage of time in intern() by a factor of 4 with a corresponding benefit to overall performance:

As it stands currently, this PR unfortunately increases the size of Symbol to 8 bytes, which is probably not desirable. One option might be to hijack the high bits of the value in Symbol to store the shard index, if that would be acceptable.

(Another issue, perhaps, is the use of the same hash function both to choose a shard and then for inserting strings to the hash table. That could presumably potentially lead to an unexpectedly high number of hash collisions in a worst case scenario.)

I agree, I'd rather not increase the size of Symbol. If this is a continued perf bottleneck, we could consider switching to a purpose-built interning solution (many exist in Rust, but I'm not familiar with all of them).

Just leaving an idea here for now: You could "inline" < 3 byte strings into symbol pretty easily, which would require no synchronization for most variables (since most are very short).
Redefine Symbol as a i32 newtype. If the sign bit is set, then just read the bottom three bits as the string. I guess the string would be 0 terminated, or you could stash the length in the top byte as well.

Nice! I just got around to trying this with my workload; to my surprise it doesn't seem to give a performance benefit, FWIW.

Ah, yes-for the example I was testing with, it happened that they were all >4 characters. Trying another one that has a mix of short ones and a few longer ones, I see a 10% reduction in overall wallclock time, so this is a definite win for that case, since it does a bunch of other stuff besides call into egg.

Great! The latest commit on tiny-symbols now implements both of these techniques, and it keeps the Symbol size to 4 bytes :). Barring any crazy results from you, I'll merge that and close this PR.

Hi Matt! I've taken this idea and combined it with something inspired by this post and made the symbol_table crate.
Commit ed1faf143b20ad93d0d266e88c94abf38ce6022c (and the upcoming 0.8 release) will use this for string interning, which should solve your perf issues once and for all!


Option to get currently loaded Grafana theme

Is there a way to get information about the currently loaded Grafana theme using this plugin?
I have looked at 'option' object but it seems this option is not available there.
I want to dynamically change the color of the line / path element based on currently loaded Grafana theme.

I think there might be a way to achieve this! I'll look into it and keep you posted.

@LarsStegman yes, sorry for the delays. I've figured out how to achieve this and I'll be implementing it in the next minor version of this plugin (0.2.0). I'll keep you all posted through development.


can i fill child on touch..?

can i fill child on touch..?

Now we can't do this, but its good suggestion i will try to implement.

Any update for this ? :)


[Power Lora Loader] Assorted Proposed Changes

Hey @rgthree,
Wanted to bring some PR proposals to your attention
Simple QOL
#267
Context menu option to load all of the Civitai data for the LoRAs in the current Power Lora Loader node
#264
Allows for adding several LoRAs at a time by holding shift
Much harder and I'm not sure about
#263
I want to get the benefit of the Power Lora Loader with some of the convenience and portability of a LORA_STACK. I've got something that works, but it also has some nasty edge cases.
Thanks 😄

I have the ones that aren't the stack in here if you want to try them out: https://github.com/rgthree/rgthree-comfy/compare/main...DrJKL:rgthree-comfy:drjkl/pll-civitai-name


Add unit tests for censors

Improvements to censors testing (this got lost in work a few months ago)

Looks like we need to fix these tests

Looks like we need to fix these tests

Works on my machine, even with a clean install...


Replace $JOB_NAME with a hand-made name

Jenkins set some variables to identify the name of the test that
is running.  We name the tempest run using $JOB_NAME, but in this
Jenkins instance is always 'openstack-mkcloud'
Instead of testing with $BUILD_ID, we will use a hand-name name to
identify the tests.

Kind of +1, but unsure if the current value of $name is good enough -- there are so many variables we could include in there...

According to the documentation, this is the list of variables that we have:

hacloud
clusterconfig
upgrade_cloudsource
TESTHEAD
cephvolumenumber
cephvolume_hdd_size
controller_ceph_hdd_size
nodenumber
nodenumberlonelynode
nodenumbercompute
vcpus
adminvcpus
admin_node_memory
controller_node_memory
compute_node_memory
drbd_hdd_size
drbd_database_size
drbd_rabbitmq_size
networkingplugin
networkingmode
keep_existing_hostname
debug_mkcloud
debug_qa_crowbarsetup
debug_openstack
user_keyfile
want_neutronsles12
want_sles12
want_dvr
want_docker
install_chef_suse_override
vlan_public
tempestoptions
rally_server

Maybe we can add to the name this:

cloudsource
hacloud
clusterconfig
upgrade_cloudsource
TESTHEAD
cephvolumenumber
drbd_hdd_size  (not sure that this means DRBD usage)
networkingplugin
networkingmode
want_neutronsles12
want_sles12
want_dvr
want_docker

Added a longer selection of variables to build the name

r? @vuntz @bmwiedemann @jdsn

Heh, nice. Now I worry it might get too long ;-)
Anyway, still +1 as it can't be worse than what we have.

For the jenkins test this produced
develcloud5/head:1/ceph:1/cinder:local/drbd:0/net:openvswitch/netmode:gre/netsles12:1
In the jenkins context netsles12 is a constant.
and the difference of head=1 should be small and temporary

Replace echo / exit with complain

Drop netsles12 and head variables


(chore): Added Hero Section Component for the Third theme

Signed-off-by: Abhijay Jain<EMAIL_ADDRESS>Description
Added Hero Section Component for the Third theme
Resolves #305
Screenshots (In case of UI changes):

Desktop  View

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests have passed and it does not give any unexpected error for the same.

@dinithminura can you please review this PR :)


Error executing lua, attempt to call field _trim (a nil value)

Description

When trying to use some command of lsp saga, I'm getting this error.

For now, as workaround I'm using enhance-window branch instead of master, which at least only give this error in LspInfo, but in master everything gives same error.
This is happening from around 1 month, after I updated lspsaga.

Environment

nvim --version output:

NVIM v0.5.0-dev+1178-gf0ace6d41
Build type: Release
LuaJIT 2.1.0-beta3

Operating system: macOS 11.2.1 20D75 x86_64

It's fixed updating neovim version

What version are you using? Are you using brew? I just updated lspsaga and I'm getting these errors.

What version are you using? Are you using brew? I just updated lspsaga and I'm getting these errors.

I'm using NVIM v0.5.0-dev+1428-g997a9c879

I currently have this problem and the version of neovim is 0.10 on fedora workstation 40.


"Requested environment is not available"

Hi!
When initializing my test I am getting the following error:
Selenium::WebDriver::Error::InvalidArgumentError:
Requested environment is not available
I am using the docker setup, and started both the Hub and Selenoid UI. I am using Macos when an Intel chip.
I can access the Hub and UI without any problem, the /status endpoint also gives me json about the browsers so that should all be good. I am using ruby as language binding.

I started both the Hub and UI with the quickstart:
./cm selenoid start --vnc
./cm selenoid-ui start
Any help would be greatly appreciated!

@tijnster make sure that browser name and version you are requesting are present in /status API output.

Hi @vania-pooh I have it fixed now, thank you. Running into another issue now, if you could give me a pointer on this one that would be great, else I can open another ticket;
Getting the following error when initializing the driver:
1652183218.445][SEVERE]: CreatePlatformSocket() failed: Address family not supported by protocol (97)
--- x11vnc loop: waiting for: 191

@tijnster this is not related. Probably you don't have IPv6 on your machine.


Odd behaviour as editor to a rebase

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Until recently[1], using atom as my VISUAL editor during an interactive rebase would open the git-rebase-todo temporary file in a new tab in Atom and allow me to perform changes before saving and closing.
Now, instead of waiting, it deletes each line in the file one by one, and closes it.  Performing commits this way are unchanged.
Using atom --wait resolves the issue but at the cost of startup time as it opens a new window rather than a new tab.
Using atom --safe has no effect.
atom --clear-window-state had no effect.
Steps to Reproduce

Set atom as your VISUAL.
Perform a git rebase -i with or without --root.

Expected behavior:
A new tab opens in the same project as the directory I'm in containing git-rebase-todo and takes action only when I save and close the file.
I don't know how recently this started happening as it has been a while since I did an interactive rebase.

The --wait option is necessary for things like the rebase command because that is what enables the synchronization of the Atom process with the terminal process. If you don't supply --wait then the terminal process will launch Atom and then continue on and not wait for you to edit and save the git-rebase-todo temporary file. This is the entire purpose of the --wait option.
As far as --wait opening a tab instead of a window, there is an open issue for that already https://github.com/atom/atom/issues/10244. Since the only actionable part of this is duplicated by that issue, I'm going to close this as duplicate. You may want to subscribe to that other issue for updates.
Thanks for contributing and for such a well-written bug report!

Hi @lee-dohm, do you have any thoughts on why it works fine for commits without --wait but not for rebases, and was working fine for rebases not long ago?

+1 I would love this feature as well

@peterbabic Atom already has a --wait parameter. We aren't going to be enabling this scenario without --wait.


Saraclay patch 1

Docs Build status updates of commit 667ba2a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/ArduinoAndWindows10.md
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/ArduinoWiring.md
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/ArduinoWiringProjectGuide.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


CLEANUP CRASHED! on Ctrl-C - JRuby / Windows environment

Running JRuby 1.7.22 in Windows environment with Reel 0.5.0, get the following crash on a Ctrl+C.
{I, [2015-10-06T13:34:09.304000 #4724]  INFO -- : Celluloid 0.17.2 is running in BACKPORTED mode. [ http://git.io/vJf3J ]
^C
C:\src\ec\rds>E, [2015-10-06T13:35:32.402000 #4724] ERROR -- : CLEANUP CRASHED!
IOError: closed stream
org/nio4r/Nio4r.java:188:in `deregister'
org/nio4r/Nio4r.java:422:in `close'
org/nio4r/Nio4r.java:413:in `close'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-io-0.17.2/lib/celluloid/io/reactor.rb:52:in `wait'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-io-0.17.2/lib/celluloid/io/reactor.rb:21:in `wait_readable'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-io-0.17.2/lib/celluloid/io.rb:53:in `wait_readable'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-io-0.17.2/lib/celluloid/io/tcp_server.rb:19:in `accept'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/reel-0.5.0/lib/reel/server.rb:38:in `run'
org/jruby/RubyKernel.java:1479:in `loop'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/reel-0.5.0/lib/reel/server.rb:38:in `run'
org/jruby/RubyKernel.java:1930:in `public_send'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/calls.rb:28:in `dispatch'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/call/async.rb:7:in `dispatch'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/cell.rb:50:in `dispatch'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/cell.rb:76:in `task'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/actor.rb:339:in `task'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/task.rb:44:in `initialize'
C:/opt/jruby-1.7.22/lib/ruby/gems/shared/gems/celluloid-0.17.2/lib/celluloid/task/fibered.rb:14:in `create'}

Some workarounds have been proposed by @digitalextremist here: https://github.com/kenichi/angelo/issues/56

Solved in celluloid/celluloid-io#155


drivers: gpio: gpio_mcux_lpc: add support for module interrupts

LPC gpio driver has historically used the PINT to route interrupts, which limits interrupt routing to GPIO ports that the PINT supports as inputs. Add the ability to configure the interrupt source for each GPIO driver, so the user can allocate the two interrupts shared between all ports as needed for their application.
Note that this still subject to the limitation in Zephyr that device instances cannot share interrupts. Thus, only two additional GPIO ports can have interrupts enabled using this feature.

@henrikbrixandersen Updated PR to convert the PINT to use an interrupt controller driver, and integrated the GPIO module IRQ support on top of that update


[HUDI-4706] Fix InternalSchemaChangeApplier#applyAddChange error to add nest type

InternalSchemaChangeApplier#applyAddChange forget to remove parent name when calling ColumnAddChange#addColumns
Change Logs
Add removing parent name when calling ColumnAddChange#addColumns in InternalSchemaChangeApplier#applyAddChange
Impact
Error when use InternalSchemaChangeApplier#applyAddChange to add a nest type
Risk level: none | low | medium | high
Choose one. If medium or high, explain what verification was done to mitigate the risks.
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[x] CI passed

@hudi-bot run azure

@xiarixiaoyao Could you please review it? Really thanks!

@wzx140  Thank you for your contribution. Sorry for the late reply.
LGTM ,  could you pls add some test,
of course if  you have no time to add test, i will add later.

@xiarixiaoyao @yihua I found UT not cover InternalSchemaChangeApplier. I will add some tests later.


Get back the :type key for mashes in a forward-compatible way

Relates to #60 and #69.
The issue basically was that the :type method was forcefully removed. Hashie removed :id and :type methods just because they were not part of ruby base anymore, so they were treated as normal key accessors. Removing the method from Mash itself prevented the :type key from being used.
Also, a standard disabling of warning is added to avoid noise for clashing class, first and entries keys:

https://github.com/intridea/hashie/pull/381 (version 3.5.0) — log attempts to override a built-in method and fails to do so. The accessors were not created anyway.
https://github.com/intridea/hashie/pull/395 (version 3.5.2) — add the ability to silence those warning in subclasses.

The corresponding spec fails with current master.

Thanks!

Published as version 1.0.9


Godot script editor is laggy on large monitors

Godot version:
3.3.0 (but tried a few older versions like 3.2.x and they seemed to have the same problem)
OS/device including version:
MacOS 10.15.5
Issue description:
When running Godot full-screen on my 27 inch mac display (basically this https://www.amazon.com/Apple-MC914LL-27-inch-Thunderbolt-Display/dp/B004YLCKYA - though I suspect it happens on other large displays) the Godot script editor is noticeably laggy.
The most obvious problem is that clicking and dragging to select text runs at like 1FPS. It also lags when I hold down a key to repeatedly input it - though I have my repeat rate set really high so this may not happen for everyone.
Steps to reproduce:

Run Godot on a big monitor.
Try to select some text
Notice the text selection is redrawn really slowly

Minimal reproduction project:
None needed?

Can you reproduce this in a brand new project with just a script and nothing else?

Yep.

Also, what graphics card does your device have? On integrated graphics, rendering the editor at a large resolution will be slow, especially if one of the monitors is Retina-enabled (because other monitors will render at double resolution then downscale everything).

It's just the standard MacOS graphics card for my macbook model (mid 2015). "Intel Iris Pro 1536MB"

As a workaround, you can start the editor with the --low-dpi command line argument. Rendering with the lowDPI fallback should be much less demanding on the GPU. See Command line tutorial in the documentation for more informatino.

Yea - maybe Godot should hint me this if I'm running on a big screen.

I can confirm the same problem on a new M1 MacBook Pro.
On the Macbook screen the editor UI is very smooth, as soon as I move the window to my external monitor it feels really slow and laggy.
The --low-dpi option doesn't seem to help.
Interestingly the UI is smooth on the external monitor as long as the godot window isn't bigger that it would be on the MacBook monitor. As soon as I increase the size, the lags start.

Interestingly the UI is smooth on the external monitor as long as the godot window isn't bigger that it would be on the MacBook monitor. As soon as I increase the size, the lags start.

This is expected, as increasing the rendering resolution makes everything slower to render. The M1 GPU isn't that powerful in the first place.


Add SPI enable disable

With the addition of these functions, it becomes quite easy to share an SPI peripheral between multiple devices which require different configuration. You can disable(), free(), then pass ownership between the different devices which can then configure the SPI peripheral.

Could have disable() disassemble the device like free()
On Fri, Aug 14, 2020, 4:15 AM Henrik Böving<EMAIL_ADDRESS>wrote:

@hargoniX requested changes on this pull request.
In src/spi.rs
https://github.com/stm32-rs/stm32h7xx-hal/pull/125#discussion_r470509090
:

@@ -551,6 +551,31 @@ macro_rules! spi {
self.spi.sr.read().ovr().is_overrun()
}

               /// Clears the MODF flag, which indicates that a

               /// mode fault has occurred.

               pub fn clear_modf(&mut self) {

                   self.spi.ifcr.write(|w| w.modfc().clear());

               }

               /// Disables the SPI peripheral. Any SPI operation is

               /// stopped and disabled, the internal state machine is

               /// reset, all the FIFOs content is flushed, the MODF

               /// flag is cleared, the SSI flag is cleared, and the

               /// CRC calculation is re-initialized. Clocks are not

               /// disabled.

               pub fn disable(&mut self) {

I'm assuming that this puts the peripheral in a state where it can no
longer send data since it's disabled? If that is the case with the system
proposed here it would be easily possible to create an SPI device that can
be given to a driver but not actually used by since the peripheral itself
is disabled, I don't think that is a particularly good idea.
If that's the case the correct thing to do would be to represent this in
the type system IMHO, for example via a type state transition or similar.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stm32-rs/stm32h7xx-hal/pull/125#pullrequestreview-467428422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGQJJQQQXU5FS5LUFQA7KLSAT6E5ANCNFSM4P7AMCYQ
.

You could, but in that case your enable() method would need to only take an SPI register struct which means that our API allows to enable a device even if it hasn't been configured before, so that API is unsound as well. I guess a type state pattern would be acceptable to use here but maybe there is other ideas?

I think a type state pattern makes sense here. Then only Spi
implements the SPI hal trait and calling .disable() gives you a
Spi that can't be passed to drivers that need the hal trait.
On Fri, Aug 14, 2020, 2:57 PM Henrik Böving<EMAIL_ADDRESS>wrote:

You could, but in that case your enable() method would need to only take
an SPI register struct which means that our API allows to enable a device
even if it hasn't been configured before, so that API is unsound as well. I
guess a type state pattern would be acceptable to use here but maybe there
is other ideas?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stm32-rs/stm32h7xx-hal/pull/125#issuecomment-674247865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUS5FOARVZBRJE3BMBIK23SAWJJXANCNFSM4P7AMCYQ
.

Great, I like the PR. Agreed the enabled/disabled typestate pattern would really improve this, just to add that we already do this for the ADC, the DAC, and there's another implementation hiding in #114 also. Hopefully that's enough examples to copy.
W.r.t disabling the SPI peripheral in free(), we do this for some other peripherals (SAI..) but it's not consistent yet. I think disabling in the free() method is an improvement (but it's okay for this PR to leave it unchanged)

For my use-case all I really need is to disable the peripheral in free() which I didn't do because I thought it would be inconsistent with other peripherals.
I'll implement the typestate enable/disable for consistency, though.

Most of the SPI functions work whether enabled or disabled, but the typestate helps ensure that you can free() and reconstruct an SPI without problems.
https://github.com/stm32-rs/stm32h7xx-hal/pull/125/files#diff-7ee16ebd4be60979ac97f399eca0cc89R46-R53
It seems that type inference doesn't work through enable() so you have to annotate the word type. If you try to chain the constructor into enable() it loses track of the pac::SPI type as well. If anybody knows a way to tweak the generic impls to solve this I'd love to hear it.

I dont feel good about the fact that the SPI constructor now returns a disabled SPI peripheral which needs to be explicitly enabled afterwards, that is a breaking change to old lib code which we do not have to perform if the SPI constructor returned a default enabled SPI object.
If there is a particular reason we want default disabled here Im missing out on please do explain it to me.

I made it Disabled for consistency with the ADC, but the ADC actually has things which need to be done while disabled so I suppose it's the outlier.

In that case @richardeoin what would be your opinion on consistency with the ADC vs breaking the SPI API beyond the type states?

@hargoniX  I think the initial typestate can be Enabled or Disabled depending on what's obvious for the API. In this case Enabled makes more sense to me (and Disabled making more sense for the ADC is also fine)

bors r+


wechat will close webapi for wechat

https://share.iclient.ifeng.com/shareNews?fromType=vampire&forward=1&aid=38628857#backhead#backhead
Is it realy ?

No, it's not really, it's a RUMOR.
See: https://github.com/Chatie/wechaty/issues/673#issuecomment-347400196

I got this message from a relatively new account when trying to login into wx.qq.com:
1203为了你的帐号安全，此微信号已不允许登录网页微信。你可以使用Windows微信或Mac微信在电脑端登录。Windows微信下载地址：https://pc.weixin.qq.com  Mac微信下载地址：https://mac.weixin.qq.com
That message seems to indicate that this account is permanently blocked from web wechat. But I can login into desktop (mac) wechat client with that account.
What api is desktop client using? not webapi? Can we try to emulate desktop api?

Anyone knows where to buy a wechat account that can be used in wechaty? That maybe the only option left to create new bot.

Infact,I dont know how and where to buy it.

For those developers who want to use Wechaty but be limited by the Web API:
Please have a look at our in-alpha testing Wechaty v0.15, with new Puppet: PuppetPadchat. PuppetPadchat will let you login your bot without any problem.
See: #1296

now almost account will be noticed : 1203为了你的帐号安全，此微信号已不允许登录网页微信。你可以使用Windows微信或Mac微信在电脑端登录。Windows微信下载地址：https://pc.weixin.qq.com Mac微信下载地址：https://mac.weixin.qq.com
web API can't be used any more


[SUGGESTED FEATURE]

What?
Disentangle the clustering function to provide a summary of the centroids (in any suitable format) and approximate (possibly fractional) number of points which are represented by each centroid.
Why?
The above output decouples the act of summarization (e.g., there were 5 well separated clusters in data today, yesterday there was 7) from the act of assigning every data point to the nearest centroid. The latter (assignment to nearest centroid) is not useful in many scenarios -- for example when clustering is used to denoise or reduce data.
Further the decoupling allows for "soft clustering", see https://en.wikipedia.org/wiki/Expectation–maximization_algorithm, and the specific shout out to "The on-line textbook: Information Theory, Inference, and Learning Algorithms" http://www.inference.phy.cam.ac.uk/mackay/itila/
Please check Chapter 20, "An Example Inference Task: Clustering".
In fact in a decoupled setup, both uses of summarization and labeling are covered. Most modern clustering algorithms (circa 1990+) would bypass the labeling step to be more scalable. This is specially true of streaming algorithms, core-sets, etc., etc.

hi, @sudiptoguha  is this the same with https://github.com/opensearch-project/ml-commons/issues/356?

No. I am asking you to change the API for K-Means (and any future clustering, were that to happen). Clustering is not an example of train and predict -- in particular, because clustering is unsupervised and "training" is a complete misnomer in unsupervised context. Now, it seems you can "adapt" classification output to express clustering; but that does not make clustering a supervised task.
I again recommend the textbook The on-line textbook: Information Theory, Inference, and Learning Algorithms" http://www.inference.phy.cam.ac.uk/mackay/itila/
Please check Chapter 20, "An Example Inference Task: Clustering".

Thanks. Agree that "training" maybe not necessary/proper for unsupervised learning. Checked sklearn Kmeans doc https://scikit-learn.org/stable/modules/generated/sklearn.cluster.KMeans.html, seems they provide similar API fit and predict, check their example
>>> X = np.array([[1, 2], [1, 4], [1, 0],
...               [10, 2], [10, 4], [10, 0]])
>>> kmeans = KMeans(n_clusters=2, random_state=0).fit(X)
>>> kmeans.labels_
array([1, 1, 1, 0, 0, 0], dtype=int32)
>>> kmeans.predict([[0, 0], [12, 3]])
array([1, 0], dtype=int32)

And they provide fit_predict API, we have something similar called train_predict

I see. If Scikit-learn APIs are the guiding light then has there been any evaluation if Scikit-learn was sufficient? Or parts of it? Does Scikit-learn have KMeans clustering? That saves significant reimplementation and discussions such as "Scikit-learn does it this way".


SEMANTICS.md example is breadth first not depth

As a follow-up, I added the wikipedia link and a diagram in https://github.com/CosmWasm/cosmwasm/pull/1653. Hope this helps future readers. Thank you for your input!


(chore) make the android unit tests green again

Summary
The android unit tests did not compile. Now they compile and pass again. I added a mock for the repository.
How It Was Tested
I ran the unit tests.
Screenshot/Gif

Before

After

@alisen what is the etiquette for PRs on this repo? Am I expected to merge these myself if they are approved?

@maiatoday That’s a great question! At Droidcon, we usually handled the merging ourselves, but currently, I can confirm that no official rule has been established for this. That’s why I left the merging decision up to you.

@alisen what is the etiquette for PRs on this repo? Am I expected to merge these myself if they are approved?

@maiatoday That’s a great question! At Droidcon, we usually handled the merging ourselves, but currently, I can confirm that no official rule has been established for this. That’s why I left the merging decision up to you.

OK I'll merge these two, that way atleast the dependabot ones will go in

@alisen what is the etiquette for PRs on this repo? Am I expected to merge these myself if they are approved?

@maiatoday That’s a great question! At Droidcon, we usually handled the merging ourselves, but currently, I can confirm that no official rule has been established for this. That’s why I left the merging decision up to you.

~OK I'll merge these two, that way atleast the dependabot ones will go in~ I don't think I can merge them

Thanks for trying to merge the PRs. It seems like our settings might be causing an issue. I'll merge them. We'll also discuss what we can do about this going forward.


1.7.10 - Mekanism V9 - Thermoelectric Boiler & Turbine Render Issue

I've been getting this issue when I log into a server:
Now this only started happening AS I placed a conductive pipe to attach to the thermoelectric boiler, and as soon as I did that I got a render crash, and nothing has seemed to fix it.
And I don't wanna remove Mekanism. D:
This is also only client-side, it doesn't cause a crash on the server side.
Edit: Seem to be getting a render crash when attempting to access the Steam Turbine with a right click while it's spinning, I had to use MER Steam productions tools.
Maybe this is a Java related issue?
Turbine Crash:
crash-2017-02-17_11.09.30-client.txt
Boiler Crash:
crash-2017-02-16_16.01.53-client.txt
---- Minecraft Crash Report ----
// Ooh. Shiny.
Time: 2/16/17 3:36 PM
Description: Rendering Block Entity
java.lang.NullPointerException: Rendering Block Entity
at net.minecraft.client.renderer.RenderBlocks.func_147768_a(RenderBlocks.java:7807)
at mekanism.client.render.MekanismRenderer.renderObject(MekanismRenderer.java:337)
at mekanism.client.render.tileentity.RenderThermoelectricBoiler.getUpperDisplay(RenderThermoelectricBoiler.java:216)
at mekanism.client.render.tileentity.RenderThermoelectricBoiler.renderAModelAt(RenderThermoelectricBoiler.java:121)
at mekanism.client.render.tileentity.RenderThermoelectricBoiler.func_147500_a(RenderThermoelectricBoiler.java:40)
at net.minecraft.client.renderer.tileentity.TileEntityRendererDispatcher.func_147549_a(SourceFile:100)
at net.minecraft.client.renderer.tileentity.TileEntityRendererDispatcher.func_147544_a(SourceFile:92)
at net.minecraft.client.renderer.RenderGlobal.func_147589_a(RenderGlobal.java:781)
at net.minecraft.client.renderer.EntityRenderer.func_78471_a(EntityRenderer.java:1599)
at net.minecraft.client.renderer.EntityRenderer.func_78480_b(EntityRenderer.java:1337)
at net.minecraft.client.Minecraft.func_71411_J(Minecraft.java:1001)
at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:898)
at net.minecraft.client.main.Main.main(SourceFile:148)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.multimc.onesix.OneSixLauncher.launchWithMainClass(OneSixLauncher.java:236)
at org.multimc.onesix.OneSixLauncher.launch(OneSixLauncher.java:297)
at org.multimc.EntryPoint.listen(EntryPoint.java:162)
at org.multimc.EntryPoint.main(EntryPoint.java:53)

does it happen without fastcraft enabled?

I'm not sure, I'll be sure to check, give me a bit and I'll report back.

Yeah, I'm still getting the same crashes regardless of Fast Craft, I've also taken out optifine and tried that too.
I'm still running into the issues, like with the Boiler, I can fully build it, load it with water, but ONCE I connect a thermal pipe to heat it up, and set it to 100,000, the Boiler begins producing, but then I get the render crash.
For the Turbine, I'm fine right clicking it when it's not spinning, but once it hits a certain speed, then I can't right click as it'll make me crash.

Can't reproduce this in 1.11. Calling this fixed until further notice


release: Bump version -> 0.5.0

Preparation for releasing 0.5.0 / incrementing version

Still working on preparing the release notes & video!


Messages arriving early (in previous round) due to time drifts

Since we assume that honest peers might not agree on the current time, it is possible (need to be verified) that an honest peer will receive a round2 message at the end of round1 and therefore ignore it. We first need to verify that this scenario is possible and then think of whether or not it is a problem and fix it.

Shouldn't we have a network delay parameter that also allows minor time drifts between clocks? As I see it, any drift of more than a couple of seconds should be considered malicious anyways


fixinsubquery should always be converted to semi or anti join

Proposed changes
Issue Number: close #xxx
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


Multiple params not allowed errors not returned in response

This issue is a (choose one):

[x] Problem/bug report.
[ ] Feature request.
[ ] Request for support. Note: Please try to avoid submitting issues for support requests. Use Gitter instead.

Checklist before submitting:

[x] I've searched for an existing issue.
[x] I've asked my question on Gitter and have not received a satisfactory answer.
[ ] I've included a complete bug report template. This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.
[x] The feature I'm asking for is compliant with the JSON:API spec.

Description
Choose one section below and delete the other:
Bug reports:
When multiple invalid parameters are provided, only the first error is included in the response. This behavior changed in d177c228510e122ccb069c5f3eb3ad36b2261430: https://github.com/cerebris/jsonapi-resources/blob/a7447ac7d51dcc5c4d5ccc8a59bef5e4d3b744ea/lib/jsonapi/request_parser.rb#L629-L634

This change was made to prevent a DOS attack by providing a request with a million invalid fields. We should allow a configuration option for the maximum number of errors allowed to be returned. If we default to 10, that should cover returning all errors in 99% of cases, while not overloading the server.


set kernel parameters permanently

Hey, I am currently trying out ChimeraOS on my HTPC and like it so far. I only have a small problem, my system regularly hangs which I previously fixed by adding amdgpu.noretry=0 iommu=soft as kernel parameters, I was able to do that as well however I have the feeling that this manual intervention will be necessary on every update. A permanent solution would be appreciated!

after digging into the deployment a bit, I think one can already achieve what I want using a systemd drop-in file:
systemctl edit frzr-autoupdate.service
[Service]
ExecStartPost=grep 'amdgpu.noretry=0' /efi/loader/entries/frzr.conf > /dev/null || ( cp
 /efi/loader/entries/frzr.conf /efi/loader/entries/frzr-bac.conf && sed -e 's/^options .*$/& iommu=soft amdgpu.noretry=0/' -i /efi/loader/entries/frzr.conf )

I'll check whether this will actually work on the next update and close the issue incase it works


Incorrect Content-Length header with multibyte characters with HttpContent.Complete

Hello,
When using HttpContent.Complete with a multibyte character, the calculation for the Content-Length header does not take into account that this is a multibyte character. This is because of the length is calculated with String.length() instead of with String.getBytes.length . Illustration:
"λ".length() // 1
"λ".getBytes.length // 2

With a minimal example:
object Main extends zio.App {
	val app = Http.collect{ case Method.GET -> Root => Response.text("λλ") }
	override def run(args: List[String]): URIO[zio.ZEnv, ExitCode] = Server.start(8080, app).exitCode
}

we can use curl -iv --raw localhost:8080 to see that we get content-length: 2, an "Excess found" of size 2, and only one "λ"
* Excess found in a non pipelined read: excess = 2, size = 2, maxdownload = 2, bytecount = 0
* Connection #0 to host localhost left intact
λ⏎

This is in the latest release <IP_ADDRESS>-RC15 and on main. Let me know if you want me to put together an MR.

@lilatomic can you try the latest snapshot release also? I think it's fixed there.

yep, it looks like it's fixed in the snapshot <IP_ADDRESS>-RC15+31-46c879fd-SNAPSHOT . I guess that's due to the change from HttpContent.Complete(data) to HttpData.CompleteData(Chunk.fromArray(data.getBytes(HTTP_CHARSET))) in ResponseOps.text. That's a bit less ergonomic for creating responses which aren't text or json (I was creating some barebones HTML responses when I noticed this). I think it would be nice to have a simple HttpData.fromString method for that use case (and internally, possibly):
	def fromString(s: String): HttpData[Any, Nothing] = HttpData.CompleteData(Chunk.fromArray(s.getBytes(HTTP_CHARSET)))

anyhow, thanks for your help and the cool library!


ci: only perform downstream testing on PRs into main

@burkedavison This repos should be archived since it has been moved to gapic-generator-java. Why are we making updates to it?

@burkedavison This repos should be archived since it has been moved to gapic-generator-java. Why are we making updates to it?

We are still using this repo for testing the compatibilities of libraries on LTS branches. Graalvm downstream testing was failing yesterday in my testing PR, so Burke was trying to help out. That being said, I don't think we need this PR anymore based on yesterday's discussion in chat.


New version: stevencohn.OneMore version 4.2

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

There's nothing to fix. See https://github.com/microsoft/winget-pkgs/pull/26262, it has the same error but it still got merged because it actually has no errors.


Deploy from smart contracts or pallet

Had a conversation with contracts team, this is possible.
Had a conversation with Sean Tabrizi about logistics.
He suggests registering manually is acceptable and would prevent overengineering.


morphologica at current master only compiles with gcc 9

The code which trips up both the Intel compiler (version <IP_ADDRESS>1) and gcc 7 is in Random.h.

That wasn't the problem. THe problem had to do with morphologica including stale files from my /usr/local/include/morph installation instead of the correct source_dir/morph/ headers. This should be improved with a unified include strategy implemented in commit 73025d5


Download that is stuck will not resume

I had networking issues and the download just stuck - the app wasn't saying anything but new files were not being downloaded.
It would be great to support resume and retry if something bad happens.
BTW One way to reproduce this is simple, start downloading an album - in my case I chose all pictures - stop the internet to the machine (Mac in my case) and resume the internet- the client will stop downloading new content.

updated description with a repro


NOT READY: optimizing _.long.is

Work in progress to find more alternative routines.
Rewritten the performance metrics for is, as underlying bufferTyped.is routine has been optimized
Working upon to write performance table using wStringsExtra

Good. Single test routine? More alternatives?

Good. Single test routine? More alternatives?

Working on it.

Conflict


[NETBEANS-54] Module Review openide.util.*

This branch covers:

openide.util : no changes
openide.util.enumerations : no changes
openide.util.lookup : see patches / wiki
openide.util.ui: see patches / wiki

+1

I merged the PR changes into master.


Add transition between the input videos

Right now, when I execute
vid_transition.py -i ex_vid1.mp4  ex_vid2.mp4 --animation translation --num_frames 15 --max_brightness 1.5
I get two video output vt1_phase.mp4 & vt2_phase.mp4.
But I want a single video let's say output.mp4 that will basically merge the two input videos and will add the transition between the joint of the first and second video.

I will add an option to produce one video transition that contains the two phases, instead of two separate video chunks.

Issue fixed in this merge #11


Query: Add translators for string functions

Resolves #5199
@anpete @divega - Are there anymore translators which we should have?
Currently we have following translators for string
Contains, EndsWith, Equals, StartsWith, Compare, Concat (in relational)
Length, Replace, Substring, ToLower, ToUpper (in sql server)
Adding IsNullOrEmpty in this PR.

@smitpatel IsNullOrWhitespace?

What would IsNullOrWhitespace translate to?

@smitpatel LTRIM(RTRIM( or PATINDEX - see http://stackoverflow.com/questions/350400/testing-for-whitespace-in-sql-server

updated with feedback
functions added are
string.IsNullOrEmpty
string.IsNullOrWhiteSpace
"string".TrimStart
"string".TrimEnd
"string".Trim

:shipit:

Providers beware?

This "Providers-beware" is similarly to the other one. Here we are adding provider-specific method translators. Other providers can write similar but if they don't then it will be evaluated on client. So no breaking change.

@anpete Why "!string.IsNullOrEmpty(r.X)" will translate to "[r].[X] IS NOT NULL AND (([r].[X] <> N'') OR [r].[X] IS NULL)" not"[r].[X] IS NOT NULL AND ([r].[X] <> N'')"?


When using STEAL policy,how can we tell whether a page can be evicted?

As 6.830 didn't mention pinCnt,I'm pretty confused about this

@tli2 @sivaprasadsudhir


Chinese dot character must not be broken off its preceding characters

No line break must be inserted between the 。 character and preceding ideographs.
Bug from user feedback.

Bug in xi-unicode?  The "。" character is U+3002, UAX#14 considers it to be "XB" (no linebreak before).

In the Chinese standard, these symbols are forbidden to be placed at the beginning of a line:
。？！．
，、；：
”’）］〕】》〉」』〗｝
—～
·-/／

(please note that three symbols maybe shared with English and other Latin languages: — - /)
and these symbols are forbidden to be placed at the end of a line:
“‘（［〔【《〈「『〖｛

These two compound symbols can be arranged in any position (please note that no space can be inserted between their two elements, and, as mentioned above, one of their elements, —, can also be used alone.):
…… and ——

There is code to reproduce this issue:
#set page(width: 120pt, height: auto)

This is a text with a formula and Chinese comma $x$，

Also, this is not specific to Chinese punctuations. If set correct margin, we can reproduce this bug in pure English:
#set page(width: 120pt, height: auto, margin: (x: 15pt))
#set text(lang: "zh")

This is a piece of text with a formula and a period $x^2 + 1$.


OBJ importer does not read vertex colors

Godot version
4.0 beta 10
System information
Windows 10 960m
Issue description
While the gltf and dae importers both support vertex colors, the obj importer does not. Blender has supported this feature for the last 3 versions, so it would be good to have this working with Godot 4.
While not strictly part of the obj spec, vertex colors are supported in several applications by having the rgb value after the vertex position as shown by this snippet:
# Blender 3.4.1
# www.blender.org
mtllib vertex_colored_sphere.mtl
o Sphere
v 0.000000 0.831470 -0.555570 0.0000 1.0000 0.0000
v 0.000000 0.555570 -0.831470 1.0000 0.0000 0.0000
v 0.000000 0.195090 -0.980785 1.0000 0.0000 0.0000
v 0.000000 0.000000 -1.000000 1.0000 0.0000 0.0000
v 0.000000 -0.195090 -0.980785 0.4980 0.0000 0.4980
v 0.000000 -0.555570 -0.831470 0.0000 0.0000 1.0000

This is a screenshot of what it should like in blender vs godot currently:

Obj files are still very useful to use for static meshes in Godot since they can be used directly in a mesh instance, so it would be great to have vertex color support.
Steps to reproduce

import an obj with vertex colors
add a material and turn on the vertex color option

Minimal reproduction project
The blend file as well as the godot file with obj export example:
obj_vertex_color.zip

This is a feature proposal as it is not part of the obj spec but is commonly found to be useful. Are you interested in fixing it via a pull request?

@fire I am not a c++ coder so I couldn't code it myself. Should I be moving this to the feature requests section instead?

Duplicate of https://github.com/godotengine/godot-proposals/issues/5973.


Unable to influence \SoapFault::$code from Zend\Soap\Server::fault()

In Zend\Soap\Server::fault (https://github.com/zendframework/zf2/blob/master/library/Zend/Soap/Server.php#L1067) there is the functionality to transform an exception which is preregisterd with Zend\Soap\Server using the method registerFaultException.
Following the logic at line 1072, and $fault is an exception $code becomes the exception error code. The base class exception from PHP libraries defines \Exception::$code as an integer, which is tested in the constructor, and protected with final methods. This has the effect that \Exception::getCode() always returns an integer value.
On line 1086 this value is tested against an array of string values, therefor never making a match, causing the $code variable always being set to 'Receiver' on line 1087.
Original: https://github.com/zendframework/zendframework/issues/6770

You can work around this issue by setting the code in your exceptions construct after initializing the parent exception:
`class Soap_Server_Exception extends Exception {
public function __construct($message = "", $code = 'Sender', Throwable $previous = null) {
	parent::__construct($message, 500, $previous);
	$this->code = $code;
}

}`

This repository has been closed and moved to laminas/laminas-soap; a new issue has been opened at https://github.com/laminas/laminas-soap/issues/19.


Add releasenote

Supplement release note for #48461

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all


Potentially incorrect steps

[Enter feedback here]
Step 7 of "Add the administrative template to Active Directory" does not work for me. "Group Policy Management" is available in administrative tools but not the "Group Policy Management Editor." When I try and access it via MMC --> Add snap in as a work around  it wants to open on a specific existing Group Policy (not a set one, it just won't open without one selected). I can confirm the Edge ADMX files are available once one is selected, it just might be worth a mention or clarification.

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

ID: 8e4fa0af-82d8-c1ca-02b3-fc55eb620ff3
Version Independent ID: bd546e78-3406-0fdd-2aea-d0bd72be9f38
Content: Configure Microsoft Edge for Windows with policy settings
Content Source: edgeenterprise/configure-microsoft-edge.md
Product: microsoft-edge
GitHub Login: @dan-wesley
Microsoft Alias: archandr

@ResearcherAtLarge Since you are not able to find "Group Policy Management Editor" have you tried "Group Policy Management" in Administrative Tools?  Can you share what OS you are using?

Closing as there has not been any recent replies.
If you are still having an issue, please reach out to Support - Microsoft Edge Developer.  They can help investigate your specific scenario.


Fenrus page stuck on "launching xy" splash screen

I have the same problem described in issue#51. I have set the link target to "Open in this tab". When I click on an app and then going back in the browser, I have the "Launching xyz..." splash screen and the page is unusable. A manual refresh of the page is then necessary.
If I click a link with the middle button on the mouse (= open in new tab), I have the app in a new tab and on the Fenrus page again the blocking splash screen.
A way to completely disable this hint window would be very good.
OS: macOS 12.4
Browsers:

Safari 15.5 (176<IP_ADDRESS>.8)
Brave (Version 1.41.96 Chromium: 103.0.5060.114 (Offizieller Build) (arm64))

Not reproducible on:

Firefox (102.0.1 (64-Bit))

fixed in dotnet rewrite


Move Rust and Cargo removal to Warning in Install docs for clarity

LGTM
We have another installation instruction using apt at line 67 $ sudo apt install curl We should make that linux only as this won't work on macOS and windows.

Thanks, Patrick. Let me see if I can rework all of these prereqs.


Modified sample code

The sample code did not work, so we fixed it. Please check it.

Thanks @geeknees !


SDWebImage 3.7.0 download and cache wrong image rarely

hi,
Rarely i see that it loads wrong image eventhought it takes correct url, mainly notice this problem in iphone 6.
Can you please suggest me , which version solved this problem.

#1123  & #1344 seem to be the same issue.

can you reproduce it with the latest version on the demo project?

Yes we can.

i have  the same problem in version 3.7.5

does this occur in v4 as well?

closing this. please reopen if you still face issues


Fix Either constructor when called with object containing a circular reference

Introduced in release 0.9.4 with commit https://github.com/evilsoft/crocks/commit/6bc5377cf4a47a927ff3fa407073d17aa3b0cdbe#diff-d70a7c87b679a4d796e121bd30e12fd1 the inspect function recursively stringifies objects which will blow the callstack when given an object containing a circular reference.
This bug is triggered when constructing an Either due to its inspect method being eagerly evaluated during construction.

Coverage remained the same at 100.0% when pulling cbb54e4cf32ffd45d0a2a2025f0d6783bf47783f on jonwhelan:bugfix-either-constructor into ed99fbf35ff8c693068e3a5bccca13c0e111409b on evilsoft:master.

Coverage remained the same at 100.0% when pulling cbb54e4cf32ffd45d0a2a2025f0d6783bf47783f on jonwhelan:bugfix-either-constructor into ed99fbf35ff8c693068e3a5bccca13c0e111409b on evilsoft:master.

Ohhh this is nice. 👀


Didn't work at all for me.

This didn't at all for me. Had high hopes since i do this regularly. I'm working on a Mac maybe I'm doing something wrong. Great idea in theory.

Do you see any errors? If yes can you mention it here

It can be an activation problem. Sometimes the extension is not activated by vscode.
To manually activate the extension:
Press (Ctrl+Shift+P) to open Command Palette, then type convert px to vw.
If the shortcuts don't work you can use this feature through the command palette.
Steps:

Select the value you want to convert.
Ctrl+Shift+P to open the command palette
Type convert you will see all the options available for conversion.


Added guidelines for assertIs() usage.

I can open a discussion on the mailing list if needed, and possibly also a ticket.
assertTrue is used quite a lot, when it seems that assertIs would have been better because it checks the boolean value of the expression, when assertTrue just checks that the truthiness of the expression.

I would briefly explain why assertIs is preferred (because assertTrue/False may not reveal bugs as you described in the PR description).

@atombrella Thanks :+1: I fixed typos and pushed minor edits.