Request failed with status code 500 (login)

Preflight Checklist

[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Describe the bug
I can't login
To Reproduce
Steps to reproduce the behavior:

Go to 'mumble.dev'
Type the username and password
Login
See error : Request failed with status code 500

The backend still needs to be fixed! the database is empty and @divanov11  will create that dummy user, after that you will be able to login!


Use Vmware Host raw_connect instead of VimConnectMixin for detect_discovered_hypervisor

Currently Host#detect_discovered_hypervisor uses
with_provider_connection which goes through VimConnectMixin#connect.
Switching to ManageIQ::Providers::Vmware::InfraManager::Host.raw_connect
lets us remove the VimConnectMixin from the base Host class so the mixin
can be moved out of the main repository.
@Fryguy this is the last reference to VimConnectMixin in the main repo so we can finally move the mixin to -providers-vmware

Not sure how this would be hit in the normal course of using the product.  We only connect if the newly discovered hypervisor already has credentials.
I tried:

Discover a host with no hosts in the db already, this didn't call connect because there weren't any credentials
Re-discover an existing host with credentials, this didn't call Host.connect because the type was ManageIQ::Providers::Vmware::HostEsx already from the initial discovery

I had to manually set the type back to Host and re-discover an existing host in order to hit this code path.  Unless I'm missing a way to create an auth record for an entity that doesn't exist yet I think moving detect_discovered_hypervisor to ManageIQ::Providers::Vmware::InfraManager::Host (https://github.com/ManageIQ/manageiq-providers-vmware/pull/106) negated the need to be able to connect in the initial discover because you have to do a discover, add credentials to the discovered hosts, then re-discover the hosts to pick up more information using the credentials.

Closing in favor of https://github.com/ManageIQ/manageiq/pull/16564


repeated variable in pattern matching has weird result

In examples like this:
> [a,a,a] = [1,2,3]
[1,1,1] : List number
> a
1 : number

the variable seems to just bind to the first number and always matches the pattern. What I was hoping for was Erlang-style behavior like this:
6> [A,A,A] = [1,2,3].
** exception error: no match of right hand side value [1,2,3]
7> [A,A,B] = [1,1,3].
[1,1,3]
8> A
8> .
1
9> B.
3

but Haskell behavior might be more consistent with the rest of the language
Prelude> let [a,a,a] = [1,1,1]

<interactive>:3:6:
    Conflicting definitions for ‘a’
    Bound at: <interactive>:3:6
              <interactive>:3:8
              <interactive>:3:10

I second Haskell behavior, that any duplicate pattern (other than underscore) should be a a compile-time error.
That said, it's a fairly easy issue to avoid as a programmer. File under "edge case robustness", "stress test failures", or similar?

I second Haskell behavior, that any duplicate pattern (other than underscore) should be a a compile-time error.
That said, it's a fairly easy issue to avoid as a programmer. File under "edge case robustness", "stress test failures", or similar?

If you come at it with an Erlang background, it's not really an edge case. I could imagine subtle bugs like
fullHouse dice = case List.sort dice of
  [a,a,a,b,b] -> True
  [a,a,b,b,b] -> True
  _ -> False

which would always return True.

Thanks for the report. The new error messages look like this:
---- elm repl 0.15.1 -----------------------------------------------------------
 :help for help, :exit to exit, more at <https://github.com/elm-lang/elm-repl>
-------------------------------------------------------------------------------
> [a,a,a] = [1,2,3]
## ERRORS in repl-temp-000.elm #################################################

-- DUPLICATE DEFINITION -------------------------------------- repl-temp-000.elm

Naming multiple top-level values `a` makes things
ambiguous. When you say `a` which one do you want?

2│ [a,a,a] = [1,2,3]
      ^
Find all the top-level values named `a` and
do some renaming. Make sure the names are distinct!

> let [a,a,a] = [1,2,3] in a + a + a
## ERRORS in repl-temp-000.elm #################################################

-- DUPLICATE DEFINITION -------------------------------------- repl-temp-000.elm

Naming multiple values `a` in a single let-expression makes
things ambiguous. When you say `a` which one do you want?

3│   let [a,a,a] = [1,2,3] in a + a + a
            ^
Find all the values named `a` in this let-expression and
do some renaming. Make sure the names are distinct!

> let f [a,a,a] = a + a + a in f [1,2,3]
## ERRORS in repl-temp-000.elm #################################################

-- BAD PATTERN ----------------------------------------------- repl-temp-000.elm

The free variable `a` appears more than once in this pattern.

3│   let f [a,a,a] = a + a + a in f [1,2,3]
              ^
Rename the variables so there are no duplicates.

In Elm, pattern matching works by binding these free variables to subsections
of the matching value. It does not make sense to have the same name for two
different subsections though! When you say `a` in some code, there
is no way for me to know which subsection you mean!

Would be nice if you only got one error... but hooray for progress!

That is a interaction with 3 separate entries. Each has one error. Not sure what you mean otherwise.

I'm blind (or more likely was up to late), this makes more sense now, thanks.


vmt/vtf named texpath_ not found

Material loader currently cannot find these textures. It needs to be updated to search for these types.
Possibly caused by it searching for a vmt when it should be a vtf instead.

While this hasn't yet been fixed, progress was made with the integration of a dedicated KeyValue parser, as well as FileSystem search improvements. These materials are now found and parsed, but not currently usable due to incomplete KeyValue merging functionality. (these materials make a lot of use of nested include and patch properties


fix(interactive): Resolve issues when finding arrow on some platforms

Manually fix backward support issues for Arrow. Fix #4287

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

Project coverage is 58.78%. Comparing base (0aa119e) to head (0d38674).

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #4290       +/-   ##
===========================================
+ Coverage   36.96%   58.78%   +21.82%
===========================================
  Files         126       12      -114
  Lines       13235      592    -12643
===========================================
- Hits         4892      348     -4544
+ Misses       8343      244     -8099

see 114 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


python311Packages.aio-pika: 9.3.0 -> 9.3.1

Diff: https://github.com/mosquito/aio-pika/compare/refs/tags/9.3.0...9.3.1
Changelog: https://github.com/mosquito/aio-pika/blob/9.3.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 270922 run on x86_64-linux 1

4 packages built:

python310Packages.aio-pika
python310Packages.aio-pika.dist
python311Packages.aio-pika
python311Packages.aio-pika.dist


Enhancement: Better detection of pip-requirement files

The definition in package.json of pip-requirements could be improved and simplified in filenamePatterns.
At the moment, it does not recognize out of the box the following files, which are quite common in Django projects (before the Poetry era though):
requirements/
├── base.txt
├── prod.txt
└── dev.txt

I see that you already have a PR. But we really want to see what the community says on this. The reason we want to wait is if this is not a common pattern or if it can mark files that users did not intend to be marked as pip-requirements file.
We are going to give the community 60 days from when this issue was created to provide 7 👍 upvotes on the opening comment to gauge general interest in this idea. If there's enough upvotes then we will consider this feature request in our future planning. If there's unfortunately not enough upvotes then we will close this issue.

I'm going to skip the voting period based on the fact Twitter implicitly voted enough for this. 😄

Did they? :laughing: Do you have the tweet link close-by?

@batisteo  https://twitter.com/brettsky/status/1491863712721866753

Looks good. All the example files were picked up as pip requirements file for me given the folder structure.


[Feature Request] Ability to replace LineTouchTooltipData widget

Is your feature request relasted to a problem? Please describe.
SideTitles() allows the user to override the titles widget with practically anything they want, making the axis titles very flexible and extendable as needed, but I have some use cases where I would like to stylize the tooltip created by LineTouchTooltipData(). More specifically, I'd like to stylize the container more based on the value and meta.
Describe the solution you'd like
Perhaps getTooltipItems could be refactored to be similar to SideTitles() and have getTooltipItems pass value, meta and expect a widget.

+1, also have a case where I'd like to use a Widget instead of String value

+1

+1 Guys, thanks for the great package. We would also like to see this

I realize this might be a larger task since it looks like the library is rendering the tooltips with the Canvas API, which doesn't directly support rendering widgets (correct me if I am wrong), so this might require a larger refactor to move the tooltips to an OverlayEntry.

I realize this might be a larger task since it looks like the library is rendering the tooltips with the Canvas API, which doesn't directly support rendering widgets (correct me if I am wrong), so this might require a larger refactor to move the tooltips to an OverlayEntry.

Yes, exactly you're correct.
We need to use Overlays API to draw our tooltips, this way we can use widgets.
But it is a huge work, I would be happy if you could contribute to the project.
You can also sponsor this project to help me work more.


Improve tests for Node.js clients and fix hanging requests

Investigating https://github.com/bufbuild/connect-es/issues/463 and https://github.com/bufbuild/connect-es/issues/576, I found situations where both HTTP/1.1 and HTTP/2 client requests on Node.js can hang. This PR fixes them.
The issues were found by adding test coverage. We do not have definitive proof that this fixes #463 because we do not have a reproducible case, but I am fairly confident that it does.

Will this address the issue of explicit cancellations from the client and how the server handles those?

Do you mean https://github.com/bufbuild/connect-es/issues/576? No, it does not.

Will this address the issue of explicit cancellations from the client and how the server handles those?

Do you mean #576? No, it does not.

Ah no, I meant this discussion:  https://github.com/bufbuild/connect-es/discussions/315#discussioncomment-4829888 (I think they're different issues).


Use of attribute/cti

Hi,
I was just wondering why you're using the attribute/cti transform?
Since Tokens Studio adds a type to every token, we decided to no longer use this transform as we can write all our matchers by using the type prop instead of using inferred types from the token name.

You need some kind of name transformer to ensure you don't get token collisions in the final output.
{
  "color": {
    "red": {
      "accent": {
        "value": "#FF0000",
        "type": "color"
      }
    },
    "black": {
      "accent": {
        "value": "#000000",
        "type": "color"
      }
    },
    "white": {
      "accent": {
        "value": "#FFFFFF",
        "type": "color"
      }
    }
  }
}

The above, without such a transformer, gives you a collision between the three tokens, because without a name transformer, the token names would be "accent" for all 3 of them.
It's a name transformer like name/cti/kebab that takes the parent keys and combines them to become:

"color-red-accent"
"color-black-accent"
"color-white-accent"

Here's an example to demonstrate this https://www.style-dictionary-play.dev/#project=pVHLcoMwDPyVjHtjPJhzPiDn3kOmI4xpXBybsU3aToZ/r4V5GG5pfAFJK61W+yDcKGNZBU7k3rRCu/zLGU2O5FHqw6GM9ZIcD2OMGSvqJMYMcC603yQxfQfVC8yW5O10KsIrCU0B/reb6hPLXBziz0BnzkoBb59nLYqXWL+v0ov/aMX3LCt+hoDUA6EBoxv5ubPCmd7ycch53jDLWJb6FsdeaGzoFPjG2Jvb+Med22nyFrRbgOd0bQ03wbiXrBUVVOvWF7q2d1Y08ieKc3WiO/jWS1W/g7/G6hiysADboBqpxJ46uS5CauG81OAlasRRH3ewEqrQmI+C6BaPasBPBw98C3qVsJw+qEk9QAeGPw==
You should see proper style-dictionary output. Now remove the transform in the sd.config.json and note the console warnings you get, and the faulty CSS output (three tokens with the same names).
I hope my explanation helps :)

That explains the use name/cti/kebab, but I was referring to attribute/cti. Or am I missing something here? 😅

Oh true, I hadn't really noticed that one, this is only for putting attributes meta info, but it's not used for creating the CTI name, so it can be removed.
Technically a breaking change, but I'm open to a PR for this to remove that transform from registerTransforms, we'll have to release it as a new minor version.


query: save past queries

Both for use in looking at a transcription again, as well as looking for same thing / similar thing in a different transcription.

Need to add a new savedQueries field to all users via a MongoDB aggregation. This will be an array of savedquery objects. The object will have as keys: "title", "date saved", "queries" (the array of query objects), and "options" (an object).
Make sure that this array gets added to every new user upon user creation as well.
Write server calls to pull down list of saved for each user.
write server calls to push new saved to user's savedquery


Dart format presubmit check prints weird stuff to the console on Windows

PS D:\code\dart\sdk> git cl upload
Using 50% similarity for rename/copy detection. Override with --similarity.
Running presubmit upload checks ...
find: ‘>     bin [’: No such file or directory
find: ‘>     bin [’: No such file or directory

Looks like git has different output on Windows that is confusing the presubmit script.

ping


Minion Experience?

Would it be possible to adjust the experience amount listed for a monster that is marked by a minion? It would really help integrate the mod into the new /award features in 5e.

See response in https://github.com/fantasycalendar/FoundryVTT-MinionManager/issues/5


Fix typo in the assert description in MqttCodecTest

Motivation:
There is a spelling error in MqttCodecTest, where "bout got" shoud be "but got".
Modifications:
Replace the error spelling with correct one.
Result:
Fix typo in the assert description in MqttCodecTest.

Ship it

Am 15.06.2016 um 00:19 schrieb Xiaoyan Lin<EMAIL_ADDRESS>Motivation:
There is a spelling error in MqttCodecTest, where "bout got" shoud be "but got".
Modifications:
Replace the error spelling with correct one.
Result:
Fix typo in the assert description in MqttCodecTest.
You can view, comment on, or merge this pull request online at:
https://github.com/netty/netty/pull/5394
Commit Summary
Fix typo in the assert description in MqttCodecTest
File Changes
M codec-mqtt/src/test/java/io/netty/handler/codec/mqtt/MqttCodecTest.java (20)
Patch Links:
https://github.com/netty/netty/pull/5394.patch
https://github.com/netty/netty/pull/5394.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Cherry-picked into 4.1 (3dbbf06e9bb611db6beb3c65c00204935c7d71fa).


colorized dbt output

TODO:

[x] Test on windows
[x] Add option to turn this off?

:shipit: !!

fixes https://github.com/fishtown-analytics/dbt/issues/357


Linebreaks (CR+LF) for Windows-Users

Would be nice if the lists gets linebreaks (CR+LF) for Windows-Users.
Think, Windows-Users should be able to read the files with 'Editor', too.

How do the NIX systems treat CRLF in the file ?

I don't know!
But you can test it e.g. with this file: https://github.com/Tobias-B-Besemer/hosts

dos2unix && unix2dos ?

No conversion! I guess that the CRLF-Linebreak change nothing for NIX-Users...
...and it makes the file readable for Windows-Users...

Either way is readable by your computer, just not so easily by a human. But you can easily add line breaks by reading it through more or running it through a for loop and then redirecting to a file.

By the way, I just reread the first post here. If it's about reading it in an editor, Wordpad reads NIX files just fine, so this issues is closed. Windows+R and type in either write or wordpad. If you want to keep it that way for notepad or other text editors that get confused, just literally open wordpad and save without doing anything else and it will save it with line breaks and you can open it again with the other text editor.

AFAIK Wordpad will not change the linebreaks by saving.
Why not make the hostsfile with CR+LF that it is readable for everyone in every editor?

^

AFAIK Wordpad will not change the linebreaks by saving.

Whether sadly, that's not the case: I converted my BAT file, originally written using Notepad++ in CR+LF, into the Unix LF, opened in WordPad, added a letter, saved, closed, opened in Notepad++ and the letter was right where I placed it, and it reads Windows (CR LF) in the corner when it should be Unix (LF).
I'm on Window$ 7 Ultimate SP1 x64.

Yeah, it's been doing that for ages. Like I said, the other easy alternative is just Windows+R and type this:
more filenameLF.py > filenameCRLF.py
It will also format it for Windows.

We'll see what Steven decides. If the PR doesn't go through, I'll just make a script for you to automatically pull all the hosts in CRLF. I already do that anyway for the DualServer repo, so  can just make some quick edits for you and I'll drop it in the AutoUpdate repo but won't include it in the zip.

I just opened the latest hosts file using three different editors on Windows.  I can see no problem here.
So the problem clearly isn't on Windows affecting all windows users.  It's particular editors that exhibit this issue.
@Tobias-B-Besemer maybe try using a better editor?  I don't consider Notepad, for example, to be a serious editor.
Closing.

It was just because Notepad.exe (editor.exe) is the default editor in Windows...

I don't care about Microsoft abandonware.

I'm sorry @Tobias-B-Besemer; what I said above ^^^ was too harsh.
Reopening this issue, in light of Pull Request #377.

I made him a normalizer anyway, but just made a repo for it rather than combining it with another one as it could be useful by itself. And if you decide to read the script, I know there are a billion ways it could be faster, but I chose my methods based on both Windows native support and keeping the integrity of the original data.
https://github.com/ScriptTiger/CRLF-Normalizer

Sorry for being OT:
We did an auto-updater, too.
https://github.com/LV-Crew/HostsManager/releases

I like your contact information on the website. Maybe you would be interested in Ricochet IM? You might not have a problem with putting a real account name down for that and everybody on your team could have one, since I know you are all massive Tor supporters.
https://ricochet.im/

@ScriptTiger, thanks for the hint! I made some 'ads' for the project...

@Tobias-B-Besemer, can this issue be closed at this point? There's there's both the Normalizer and the Hosts Packages legacy support repo to address this outside of this particular repo itself.

Thanks for asking! If @StevenBlack don't want it, you can close it...
What is the 'Hosts Packages legacy support repo'?

It's similar to the "non-GitHub mirror" here in that it uses the GitHub Pages web service so it doesn't run into the same errors you get when trying to download the files directly from the git repo pages with some clients, except it hosts both ZIP and raw hosts files with CRLF line endings. It's a compatibility repo for people that need either CRLF and/or ZIP files. After more testing and user feedback, it seems as though the ZIP files were just a misunderstanding of some kind in the first place and never actually needed. I was going to terminate it, but it was getting some traffic. I am not sure which aspect of it people appreciate though, so more feedback on what about it is helpful would be appreciated.

We fixed the CRLF-Problem in our app and the asking for use a 'CRLF-Standard' was more in general...
I would be happy if you give our app a try and maybe fill a/some issues for it...
https://github.com/LV-Crew/HostsManager/releases

I have downloaded your stuff already, but I haven't had time to go through the source yet and build it. Since your application is security related, you should consider its complexity when designing it. The more transparent and easy to follow something is, the more trusting people can be about it. I am not saying I don't trust you, but I have to follow certain rules for security related applications.

Hi @StevenBlack and everyone.
I may have a solution for this issue for you.
I have created a PowerShell module that includes an Install-GCHostsFileBlockList function.
The repository is located here: https://github.com/grantcarthew/GCPowerShell
The module on that page is here: https://github.com/grantcarthew/GCPowerShell#gchostsfile
The module file is here: https://github.com/grantcarthew/GCPowerShell/blob/master/GCHostsFile/Install-GCHostsFileBlockList.ps1
I have published the GCHostsFile module to the PowerShell Gallery. This makes the install very easy.
To install the GCHostsFile module follow these steps on a system with an updated version of PowerShell:

Open PowerShell as administrator.
Install-PackageProvider -Name NuGet -Force (This adds support for the PS Gallery)
Install-Module -Name GCHosts -Force (This installs the module)

Once the above is done the Install-GCHostsFileBlockList function will work.
The function downloads your brilliant hosts file to a temp file then replaces the Windows hosts file fixing all CR-LF issues.
The Install-GCHostsFileBlockList function has a parameter -Extension which takes the following arguments:
'FakeNews',
'Gambling',
'Porn',
'Social',
'FakeNews+Gambling',
'FakeNews+Porn',
'FakeNews+Social',
'Gambling+Porn',
'Gambling+Social',
'Porn+Social',
'FakeNews+Gambling+Porn',
'FakeNews+Gambling+Social',
'FakeNews+Porn+Social',
'Gambling+Porn+Social',
'FakeNews+Gambling+Porn+Social'

If Windows users want to use your hosts file they only need to follow these steps.
I'll add a PR for your README if you like.

I am sure we can all appreciate your willingness to help, but drawing Steven's attention once again to this repo is a sure-fire way to get the issue closed and locked as it's leaning towards "going on and on" again. @Tobias-B-Besemer, make a new issue to raise awareness for your application, but just leave it at a single comment and don't get too spammy about it. After Steven has you on his watch list and sees you're making progress and getting traction, I am sure he wouldn't mind a readme PR for the "Third-Party Hosts Managers" section.

Bump

Thanks for asking! If @StevenBlack don't want it, you can close it...

Thank you, everyone, for input on this matter.

@StevenBlack, he gave you the honors to close it :P

I don't want to close it; it's an outstanding issue.
Right now I'm inclined to fix it in code.

Oh wow, in that case this issue took an unexpected turn. @Tobias-B-Besemer, stay posted.
Would fixing it in code or passively through git or possibly a combination provide the best balance between efficiency and compatibility?

Choose a good text editor. Notepad3 lets you set line endings.

^

Choose a good text editor. Notepad3 lets you set line endings.

Notepad3 is nothing against Notepad++.


Bug: ...Issue Initializing ArchiveBox on SMB-Mounted Directory

Provide a screenshot and describe the bug
I am currently running ArchiveBox version 0.7.2 on my Ubuntu machine and have encountered an issue when initializing ArchiveBox in a directory mounted to an SMB share. The initialization fails with the following error:
archivebox init

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

[i] [2024-12-06 11:02:51] ArchiveBox v0.7.2: archivebox init
    > /home/archivebox/data

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

[+] Initializing a new ArchiveBox v0.7.2 collection...
----------------------------------------------------------------------

[+] Building archive folder structure...
    + ./archive, ./sources, ./logs...
    + ./ArchiveBox.conf...

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

[+] Building main SQL index and running initial migrations...
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/django/db/backends/base/base.py", line 242, in _commit
    return self.connection.commit()
sqlite3.OperationalError: database is locked

...

django.db.migrations.exceptions.MigrationSchemaMissing: Unable to create the django_migrations table (database is locked)

However, when I initialize ArchiveBox in a directory located on the local hard disk of the same machine, it works as expected without any issues.
Here are the key details of my setup:
SMB Share: Mounted to /home/archivebox/data
Local Directory (Working): /home/archivebox/data1
Operating System: Ubuntu 22.04
Steps to reproduce
cd /home/archivebox/data
archivebox init

Logs or errors
archivebox init

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

[i] [2024-12-06 11:02:51] ArchiveBox v0.7.2: archivebox init
    > /home/archivebox/data

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

[+] Initializing a new ArchiveBox v0.7.2 collection...
----------------------------------------------------------------------

[+] Building archive folder structure...
    + ./archive, ./sources, ./logs...
    + ./ArchiveBox.conf...

Command '/usr/bin/chromium-browser' requires the chromium snap to be installed.
Please install it with:

snap install chromium

[+] Building main SQL index and running initial migrations...
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/django/db/backends/base/base.py", line 242, in _commit
    return self.connection.commit()
sqlite3.OperationalError: database is locked

...

django.db.migrations.exceptions.MigrationSchemaMissing: Unable to create the django_migrations table (database is locked)

ArchiveBox Version
0.7.2
ArchiveBox v0.7.2 BUILD_TIME=2024-11-13 09:29:28<PHONE_NUMBER>
IN_DOCKER=False IN_QEMU=False ARCH=x86_64 OS=Linux PLATFORM=Linux-5.15.0-126-generic-x86_64-with-glibc2.35 PYTHON=Cpython
FS_ATOMIC=True FS_REMOTE=True FS_USER=1001:1001 FS_PERMS=644
DEBUG=False IS_TTY=True TZ=UTC SEARCH_BACKEND=ripgrep LDAP=False

How did you install the version of ArchiveBox you are using?
pip
What operating system are you running on?
Linux (Ubuntu/Debian/Arch/Alpine/etc.)
What type of drive are you using to store your ArchiveBox data?

[ ] data/ is on a local SSD or NVMe drive
[ ] data/ is on a spinning hard drive or external USB drive
[x] data/ is on a network mount (e.g. NFS/SMB/CIFS/etc.)
[ ] data/ is on a FUSE mount (e.g. SSHFS/RClone/S3/B2/OneDrive, etc.)

Docker Compose Configuration

ArchiveBox Configuration
[SERVER_CONFIG]
SECRET_KEY = *****************************

You cannot store the index.sqlite3 file on an SMB share as SMB shares do not support the file locking calls needed for SQLite WAL-mode to work.
You can store data/archive on the SMB share, but data/index.sqlite3 must be an a local drive.
https://github.com/ArchiveBox/ArchiveBox/wiki/Setting-Up-Storage#supported-remote-filesystems


On Dreame L20 Ultra Complete setting Cleaning Mode to "sweeping" doesn't work

Describe the bug
I have a Dreame L20 Ultra and when I set the entity Cleaning Mode to "sweeping" I get an error message:
"Cannot set sweeping while mop pads are installed"
Expected behavior
Since the robot automatically leaves the mops in the station if it doesn't need them, this setting should be accessible.
If I set this setting in the dreame home app, the entity correctly updates but if I change the entity to something else again, it won't allow me to change it back to "sweeping". I can only set that value via the dreame home app.
Additional Information (please complete the following information)

Model Name: dreame.vacuum.r2253c (L20 Ultra Complete)
Firmware Version: 4.3.6_1510
Home Assistant Version:

Core: 2023.12.3
Supervisor: 2023.11.6
Operating System: 11.2
Frontend: 20231208.2

Configuration Type: With map support
Errors or warnings shown in the HA logs (if applicable):

Source: custom_components/dreame_vacuum/entity.py:178
Unable to call Cannot set sweeping while mop pads are installed

Source: components/websocket_api/commands.py:238
[545240541376] Cannot set sweeping while mop pads are installed

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 238, in handle_call_service
    response = await hass.services.async_call(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2067, in async_call
    response_data = await coro
                    ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2104, in _execute_service
    return await target(service_call)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 272, in handle_service
    return await service.entity_service_call(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 878, in entity_service_call
    single_response = await _handle_entity_call(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 948, in _handle_entity_call
    result = await task
             ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/select/__init__.py", line 106, in async_select_option
    await entity.async_select_option(option)
  File "/config/custom_components/dreame_vacuum/select.py", line 977, in async_select_option
    await self._try_command(
  File "/config/custom_components/dreame_vacuum/entity.py", line 179, in _try_command
    raise HomeAssistantError(str(exc)) from None
homeassistant.exceptions.HomeAssistantError: Cannot set sweeping while mop pads are installed

Never mind, HACS just offered me an update for the integration and now it works. Should've checked that first, I guess. Sorry


prompt: test function call fix for sarah prompt

Now it overshoots and thinks it is able to execute functions that it can't.
Needs further adjustment


Support latest nodejs versions

The list of nodejs versions tested against was pretty old and needed updating.

@adlawson could you please merge your own PR, since it passed all tests a while ago ?


Merge MC_1.11 dedicated Forge server changes to MC_1.12 branch

Should not cause any new issues for MC_1.12, what do you think?

No, it causes,


Revert "PTX-22246 Fix atupilot tests validation issues (#2067)"

This reverts commit b2e0eae88c836f4d36e70f65d50e01564f8f0d43.

What this PR does / why we need it:

[x] Revert "PTX-22246 Fix atupilot tests validation issues (#2067)" as it's causing pipeline failures in the non-px run  for px-backup pipelines with the below trace:

14:07:54    [PANICKED] in [JustAfterEach] - /usr/local/go/src/runtime/panic.go:261 @ 02/11/24 08:37:54.07
14:07:54    << Timeline
14:07:54
14:07:54    [FAILED] Unexpected error:
14:07:54        <*errors.errorString | 0xc000e9cf40>:
14:07:54        failed to get [autopilot] namespace, Err: the server could not find the requested resource (get storageclusters.core.libopenstorage.org)
14:07:54        {
14:07:54            s: "failed to get [autopilot] namespace, Err: the server could not find the requested resource (get storageclusters.core.libopenstorage.org)",
14:07:54        }
14:07:54    occurred
14:07:54    In [It] at: /go/src/github.com/portworx/torpedo/pkg/log/log.go:334 @ 02/11/24 08:37:32.068
14:07:54
14:07:54    Full Stack Trace
14:07:54      github.com/portworx/torpedo/pkg/log.FailOnError({0x8a77720, 0xc000e9cf40}, {0x80a0dcc?, 0xc002037000?}, {0x0?, 0x6

Which issue(s) this PR fixes (optional)
Closes #PB-5626
Special notes for your reviewer:

Please see this PR #2076, this should fix the issue, also if you need a private image with this fix you can use nikolaypopov/torpedo:PTX-22333

I have tested your fix and commented on the mentioned PR will need to do additional change make sure it check if there is a pxNamespace and then check if autopilot is installed or not. If there is no PX it shouldn't check for autopliot


Ssl issues with Android 7

Hey,
we still have no "ignore SSL certs" option in traccar-manager. You remember? I asked for it a while ago. :)
However:
Until Android 6.0.1 it was possible to "accept" that by importing a root-CA into the android user cert store. By doing so, any app was able to trust what has been signed by that user-driven CA. A good way to use SSL but not any official CA (which is still the plan, I dont want to go to lets-encrypt by reasons). Unfortunaltely, importing a user-cert was "global" to any app.
With android 7 thats not possible anymore with traccar-manager right now - see: https://blog.jeroenhd.nl/article/android-7-nougat-and-certificate-authorities and https://android-developers.googleblog.com/2016/07/changes-to-trusted-certificate.html
Might be a good thing in sense of security, but it would be nice to offer the user at least some ways out if he really wants to.
Could you eventually:

Add an "ignore SSL certs" option as asked a while ago

or -

implement

<network-security-config>
      <base-config>
            <trust-anchors>
                <!-- Trust preinstalled CAs -->
                <certificates src="system" />
                <!-- Additionally trust user added CAs -->
                <certificates src="user" />
           </trust-anchors>
      </base-config>
 </network-security-config>

?
Would that be possible? Or shall I create a PR?
Martin

Please send a pull request with following config:
<?xml version="1.0" encoding="utf-8"?>
<network-security-config>
    <base-config cleartextTrafficPermitted="true">
        <trust-anchors>
            <certificates src="system" />
            <certificates src="user" />
        </trust-anchors>
    </base-config>
</network-security-config>


[WIP] Merge SHUFFLE_AGG/SHUFFLE_JOIN property

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Merge SHUFFLE_AGG/SHUFFLE_JOIN to SHUFFLE,  we has special mark to mark BUCKET_JOIN, don't split shuffle. And that can optimize plan:
      AGG（UPDATE FINALIZE)               AGG（UPDATE FINALIZE)
              |                                   |
            SHUFFLE           ====>               |
              |                                   |
            JOIN                                JOIN
            /   \                               /   \

[FE PR Coverage check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/optimizer/OutputPropertyDeriver.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/base/HashDistributionDesc.java
4
4
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/Utils.java
1
1
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 165 / 178 (92.70%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/optimizer/Utils.java
7
9
77.78%

:large_blue_circle:
com/starrocks/sql/optimizer/JoinHelper.java
57
68
83.82%

:large_blue_circle:
com/starrocks/sql/optimizer/RequiredPropertyDeriver.java
10
10
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/RelationTransformer.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/PushDownJoinOnExpressionToChildProject.java
4
4
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/PushDownPredicateJoinRule.java
12
12
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/join/MultiJoinNode.java
2
2
100.00%

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/PushDownJoinPredicateBase.java
3
3
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/statistics/StatisticsCalculator.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/base/HashDistributionDesc.java
2
2
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/ChildOutputPropertyGuarantor.java
4
4
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/task/EnforceAndCostTask.java
3
3
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/OutputPropertyDeriver.java
5
5
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/join/JoinOrder.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/PushDownJoinOnClauseRule.java
2
2
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/PropertyDeriverBase.java
48
48
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/cost/CostModel.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/rule/implementation/PreAggregateTurnOnRule.java
1
1
100.00%


Proxy List Exception

Hi, thanks for sharing this great project. While I am trying to test it using Tor browser configuration, and setting:
USE_PROXY = False in crawler.py file. I keep getting the following error:
    raise ProxyListException("list is empty") http_request_randomizer.requests.errors.ProxyListException.ProxyListException: list is empty
in the requestProxy.py file. Anything that probably went wrong?
Also, is there any tutorial that covers the configuration when using Tor as a browser to collect data?
Thanks

Hi, sorry for the late replying. You can solve it according to this link and tell me if it works.
About the tutorial, I cant find any as you mentioned, so I just did lots of research by myself to implement these scripts
Btw, you can try the first method, I think it's the easiest way to extract data now


Add hydration and dehydration methods loading and nohydrate mode

Converts poetry to setup.py module definition.
Adds hydration method to be used to restore items to their full value against a base_item in Python. This is used when the conf value nohydrate is used within a pgstac search.
Refactors the existing format_item into a dehydrate method that is compatible with the new hydrate method.
Adds tests for the specific hydration rules

scripts/bin/stageversion is going to need to be updated to not use poetry

Fixed CI. This PR LGTM - @mmcfarland @bitner ready for your review


[81] Production is publicly available

.. to job seekers, not hiring staff.

 Remove basic auth for production and go-live

@despo Done 👍


Python CSS parser

I ported the Rework CSS parser from JS to Python so we could use it in Dataflow.
My idiomatic Python isn't as strong as JS so I might be doing some things poorly, feedback welcome. Works on a few large sample stylesheets.

Drafted a separate PR to integrate with the WIP combined pipeline. See https://github.com/HTTPArchive/data-pipeline/pull/97


Does not work with Remote Development (VS Code Insiders)

Description
When using the preview build of VS Code (https://code.visualstudio.com/docs/remote/ssh) which enables the ability to use SSH and connect to remote machines for code editing. This extension won't enable itself unless it's "installed remotely" and past that point it won't work (because its installed remotely)
I havent looked over the code yet but the VS Code documents talk about how to fix this from an extension state if its installing in the wrong location: https://code.visualstudio.com/api/advanced-topics/remote-extensions
macOS version
v10.14.5 (18F132)
vscode Version
v1.35.0-insider

The workaround for this is to add this to your settings.json
    "remote.extensionKind": {
        "psykosoldi3r.itunes-vscode": "ui"
    }

Fixed and will be available in version 0.10.0


unselect segment item by clicking doesn't works

There is a config method "setReselectionEnabled".
I think this method is used to toggle each segment item's check/uncheck status by clicking on the segment item
But it doesn't work

You can add an addOnSegmentSelectListener with if (isSelected && isReselected){segment.clearSelection(false)


Nested Embedded Broken (postgres)

"stack": "SyntaxError: Unexpected token ')'
at new Function (<anonymous>)
at Function.createFunction (node_modules/@mikro-orm/core/utils/Utils.js:612:52)
at ObjectHydrator.getEntityHydrator (node_modules/@mikro-orm/core/hydration/ObjectHydrator.js:221:40)
at ObjectHydrator.hydrate (node_modules/@mikro-orm/core/hydration/ObjectHydrator.js:23:30)

The snippet (cleansed) being generated is:
 if (data.preferences != null) {
    entity.preferences = Object.create(prototype_preferences);
    if () {
      entity.preferences.review = Object.create(prototype_preferences_review);
    }
    if (data.preferences.review != null) {
      entity.preferences.review = Object.create(prototype_preferences_review);
    }
    if (data.preferences && typeof data.preferences.preference1 !== 'undefined') entity.preferences.preference1 = data.preferences.preference1;
}

@Embeddable()
export class GrandChild {
  @Property()
  gc1 = false;

  @Property()
  gc2 = false;
}

@Embeddable()
export class Child {
  @Embedded(() => GrandChild)
  c1: GrandChild;
}

@Entity()
export class Parent {
  @Embedded(() => Child)
  child: Child;
}

as a note, i'm ripping these nested embeds out the project, but someone else may have use for it
    "@mikro-orm/cli": "^4.5.9",
    "@mikro-orm/core": "^4.5.9",
    "@mikro-orm/migrations": "^4.5.9",
    "@mikro-orm/nestjs": "^4.3.1",
    "@mikro-orm/postgresql": "^4.5.9",
    "@mikro-orm/reflection": "^4.5.9",

Try changing order of discovery, I know multiple people had issues caused by order of embeddables during discovery. Nested embeddables are tested, including such use case, so if that don't help, full reproduction would be needed (ideally as a single file test case).
Also note that unless you use ts-morph, you need to specify types explicitly, type inference don't work with property initializers in reflect metadata:
  @Property()
  gc1: boolean = false;

Lastly let's not ignore issue templates, maintly the part with versions (I do understand you are probably using latest stable, but that can change over time).

yep, using ts-morph. i'll revist with a test case once i get project coverted

Fixed via b095d9e, also part of 4.x branch (so latest 4.x dev version).


feat(#157): junit assertions - fix mistakes and enable tests

junit assertions - fix mistakes and enable tests.
Closes: #157

PR-Codex overview
This PR adds new special assertions and changes the allowed methods from a Set to a Map. It also refactors the code to simplify some methods and improve the readability of the tests.
Detailed summary

Adds new special assertions.
Changes the allowed methods from a Set to a Map.
Refactors the code to simplify methods and improve test readability.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)


Initial svg xml extension definitions

This PR provides new svg xml extension namespace definitions for a proposed rewrite of the SVG save and load functions which will support:

Saving and reloading special operations and
Storing and restoring the associations between elements and operations

I am submitting this PR early and separately so that these extensions can be reviewed and approved before the code is written.
I plan to write the full set of svg load and save enhancements in two stages as separate PRs to this:

A PR against 0.7.x adding support for groups and SVG shapes (which are currently converted to Paths on save). Because svgio.py already supports loading these tags, this should be fully backwards compatible.
A PR against 0.8.x (once the above has been merged also with the 0.8.0 branch) to add the support for the new tags. This needs to be made against 0.8.0 since files written with these tags will not be loadable by 0.7.x versions of MK.

ToDo:

Check Operations attributes which need to be saved
Add DTD definitions.

Proper saving would also save the reference nodes (opnode) values within the operations. Largely this is required to avoid forced classification. So you can load and save what you had before.

I am suggesting doing this the other way around - rather than save a list of the elements against each operation, I am proposing to store a list of operations against each element. There is a reason for this - that elements don't always have an id, whereas operations always have an index. (Its a bit more complicated when you load the file and want to use an existing operation with an existing index - you just need to maintain a mapping between the index of the <op/cmdop> in the file to an actual operation in the tree.

Given the galvo project thing I've been batting around the idea of pens. They are somewhat common in other laser software. And while the operations are 1:many rather than 1:1 like most pen implementations there are some things like hatch in galvo has like 3 different pens which basically makes the hatch operation a glorified set of three pens. There is therefore another reason to consider going with operation index matching for the references and that's because you don't necessarily need to save your operations. I mean clearly you should and that makes sense but if you have elements that say they belong in operation 3 and 7, then you really could just go ahead and put that into operations 3 and 7.
Now clearly there's some obvious non-parallel things with list of operations vs. list of pens like special operations or data-blobs that really make a lot of sense, but you could also do something like mark default operations as if they are a pen, you could assign operations to them with a yet not defined UI.  But, then you could treat your operation list like a pen list and maybe even give a weaker than TREE version that just hides the details from the user where you get a bunch of pens and the elements can be added to these. And if they contain anything they are called an operation and put in the operations code or could be put into operations while empty. But, you could also do things like have the drawing tools add an item into a particular operation when they are created. So select a default-operation for this thing you're drawing and we'll just add it to the operations.
It's a bit sloppy, but I went with the most useful methodology there, but there's clearly a lot to be gained by emulating the way other software works. In this case you'd be able to assign  an element to many different operations or to no operation at all. And if that could be made easier it would reduce the need for classification and let users more easily what their lasers should do. But, also would help to fit into other methodologies. I fully understand the idea of a jack of all trades, master of none sort of thing. But, if somebody loves how whisperer did things it really wouldn't be hard at all to let the give the user a little box with speeds and values that say engrave, cut, and raster. And when they hit the button it searches all the elements and finds anything that is exactly blue stroke and executes an engrave operation on it, by taking that and throwing it into the current device's spooler.
As for 1: Operations attributes are a little unknowable. The extra pages of settings in Operation Properties could well include settings that don't matter to K40 users but could matter a lot to other projects and development. So they could have things like frequency, pwm, wobble, split size, z-bed ticks per pass, angle, shimmy, number_of_unicorns. -- You really just can't know what operations are or could be required. We could if they need to ensure a unique namespace be prefixed as meerk40t:wobble or whatever as needed, but for the most part LaserSettings() as a class and on the whole is going to need to be basically a dictionary of you will not know what. This ambiguity will need to be attached correctly to the cutobjects and to Operations that create cutobjects, and most things that might create cutobjects. Backends are absolutely free to ignore any and all settings they do not support and use their backup global settings or the settings associated with the cutobject. The first page of operations properties is probably going to be universal though, basically power, speed, passes, and what operation you're performing.  But, even that might be assuming too much.

Once again I am having trouble understanding what you are saying. Are you suggesting that Pens are some sort of intermediate mapping from elements to operations? Can you point me to an explanation in help pages for other laser software that would help?

I'm saying that pens are a mapping of 1 to many. You have 1 pen. It can have multiple elements. You have multiple elements they have exactly 1 potential pen. Operations and elements are different they are any number of elements for any number of operations. --- So if we wanted we could implement a similar system to other software as such.
There's bound to be some videos on them, but a lot of laser software uses pens.

So a pen can have 0-n elements and be assigned to 0-n operations? It's effectively a grouping of elements?

Elements can be assigned to 1 and only 1 pen. A pen can have any number of elements. It's effectively an operation without the ability to put the element as relevant to more than one pen. These are all predefined and little color swatches in a toolbar (usually bottom). Then you have your settings programmed into those and they remain the same throughout. When you draw an item you select the pen to use and draw the element and that element classifies into that pen. Then if you select an item and you pick a different pen you classify that element into that operation which also removes it from the previous operation. So if you want to fill something two different ways you need to make a copy of it. You could pick up a free version of rdworks and check how that works.

So - it's like a colour but without giving the user the flexibility to e.g. classify several cyan elements into the same two matching operations and then manually move one of them to a third operation of a different colour? 😉

Yeah, but the interesting method of having them pick like color swatches and each does a different thing.

Note the colors in the operation list and color in the bottom there. You have those you select them and assign a element a color

Ok - so it's a colour plus an operation type combined - so not only can you not put elements of the same colour into different operations, you also can't put elements of the same colour into different types of operation. So it's more restrictive than I first thought.
I think this would not be great when you import an SVG file you created in e.g. Inkscape and then found that it wasn't quite right and you needed to tweak things in MK.
BUT...
I can see the benefit if we create a whole new part of MK focused on drawing and creation - it might well make sense there to enforce a consistent use of colours.

Yeah, or just give them the option to use something like that. What it loses in general power it does make up for being somewhat user-friendly. And we currently have operations that we mark as "default" what if "default" like operations were sort of pens. I mean you could give them a friendly button bar and when you draw something you make it that color and you put it in that operation. There's already some drawing tools and I really am at the point that I could make literally any number of other really awesome and easy drawing tools.

nothing came of this, full save is mostly done in 0.8.x


Code changes to switch vcr helix cluster to DelayedAutoRebalancer.

Switch vcr helix cluster to DelayedAutoRebalancer strategy with OfflineOnline state machine and number of replicas 1.

Codecov Report

Merging #1604 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1604   +/-   ##
=========================================
  Coverage     73.73%   73.73%
- Complexity     8490     8491    +1
=========================================
  Files           595      595
  Lines         46315    46309    -6
  Branches       5881     5881
=========================================
- Hits          34151    34148    -3
- Misses        10406    10409    +3
+ Partials       1758     1752    -6

Impacted Files
Coverage Δ
Complexity Δ

...n/java/com/github/ambry/cloud/HelixVcrCluster.java
75.40% <100.00%> (ø)
11.00 <0.00> (ø)

...ava/com/github/ambry/cloud/HelixVcrStateModel.java
100.00% <100.00%> (+15.38%)
6.00 <3.00> (-2.00)
:arrow_up:

.../github/ambry/clustermap/HelixVcrPopulateTool.java
44.30% <100.00%> (+1.44%)
11.00 <0.00> (ø)

...ub/ambry/network/SocketRequestResponseChannel.java
85.93% <0.00%> (-9.38%)
10.00% <0.00%> (-2.00%)

...a/com/github/ambry/network/SSLBlockingChannel.java
75.67% <0.00%> (-5.41%)
4.00% <0.00%> (ø%)

...va/com/github/ambry/replication/ReplicaThread.java
85.49% <0.00%> (-0.65%)
150.00% <0.00%> (-1.00%)

.../java/com/github/ambry/protocol/AmbryRequests.java
85.60% <0.00%> (-0.57%)
85.00% <0.00%> (ø%)

...a/com/github/ambry/clustermap/ClusterMapUtils.java
88.69% <0.00%> (-0.44%)
61.00% <0.00%> (ø%)

...ain/java/com/github/ambry/router/PutOperation.java
92.98% <0.00%> (+0.16%)
125.00% <0.00%> (+1.00%)

...c/main/java/com/github/ambry/network/Selector.java
79.16% <0.00%> (+0.78%)
79.00% <0.00%> (+1.00%)

... and 4 more

Continue to review full report at Codecov.

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


[frameit] Add font_size parameter

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

frameit scales the keyword and title automatically per screenshot to fit the available size above or below the device. This is a good approach for simple screenshots, but lacks a consistent and unified appearance across the screenshots.
Specifying a fixed font_size for keyword and title (separately) gives users of frameit the ability to fix the font size and keep it consistent across screenshots.
Description

This PR introdocudes a new parameter font_size for the keyword and title parameters. Instead of calculating the actual_font_size from the available space everywhere, it now checks for the font_size and returns it if specified and  font_size  > 0. This way, we keep the old behavior intact.
Testing Steps

Specifiy a font_size in either the default config in your Framefile.json or for a single screenshot config for the title or keyword. Frameit should use this font size when generating the framed screenshots.

Great addition, although a bit scary. Sure this will not change existing behavior?

The code now checks (see below), if a font_size is configured for either keyword or title and only if it does, it returns that. If not, the old implementation continues. So yes, I'm sure this does not change existing behavior.
       font_size = @config[key.to_s]['font_size']
       return font_size if !font_size.nil? && font_size > 0


Remove references to waffle.io

AFAIU waffles.io is not a service anymore.
This PR is for removing the references to this service.
I do not know if you have a replacement tool that you keep track of tasks. If there is one, feel free to add it to this PR.

@bergercookie I had to fix CI, would you mind rebasing this to incorporate the fix and restart travis?

closes #517


Added the missing any and size_t types

The types size_t and any were not defined in the syntax so this patch adds them:

any is not needed imo.
Should be removed from V

I am going to implement the #246  and #247, before doing so I also added the CSpell and fixed two little spellings.

Running cspellchecker is fine but please remove it from the project

Please implement the other things in extra PRs that i can merge this

Sure do


New provider interface in latest ddev v1.17 alpha; ddev config <provider> is gone

It looks like the method of configuring a provider (like Pantheon) has changed?
'ddev pull' gives this message:
Pull files and database using a configured provider plugin.
Running pull will connect to the configured provider and download + import the
latest backups.
Is there documentation on how to configure a pull for Pantheon? I am doing manually using import-db and import-files with downloaded tar.gz's, but I am wondering what the proper way to do 'ddev pull' is.
MacMini M1/ddev version v1.17.0-alpha6

Thanks, happy to continue the conversation here or elsewhere.


Issue #513: Implemented fallback value for color prop of BaseSnackbar…

… and removed the color prop from its consumers.

Issue number
Relevant issue number

Resolves #513

Please check the following

[x] Do the tests still pass? (see Run the Tests)
[x] Is the code formatted properly? (see Linting (Formatting))
For New Features:

[ ] Have tests been added to cover any new features or fixes?
[ ] Has the documentation been updated accordingly?

Please describe additional details for testing this change

Thanks for your feedback! Just pushed another commit.

Done.


RPC feature flags for server and simple network

Motivation
For https://github.com/linera-io/linera-protocol/issues/1609 we will need to disable both these features, as neither will work in the browser.
Proposal
Reorganize the linera-rpc code to draw a cleaner boundary between grpc and simple backends, as well as client and server implementations for each.  Then, add feature flags to enable the latter of each.

Test Plan
CI will test against regressions; feature testing will come with the Web client implementation and tests (future PR).

Release Plan

Need to bump the major/minor version number in the next release of the crates.
Need to update the developer manual.
This PR is adding or removing Cargo features.
Release is blocked and/or tracked by other issues (see links below)

Links

reviewer checklist

Did you mean to include the last bullet of the release plan?

Did you mean to include the last bullet of the release plan?

Actually only two of them are relevant — I forgot to delete the other two.  Thank you :)


Trouble with pip install cirq --pre

In a fresh Colab kernel, I do !pip install cirq --pre, and get the following output:
Collecting cirq
  Downloading cirq-1.5.0.dev20241120233534-py3-none-any.whl.metadata (7.6 kB)
Collecting cirq-aqt==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_aqt-1.5.0.dev20241120233534-py3-none-any.whl.metadata (1.8 kB)
Collecting cirq-core==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_core-1.5.0.dev20241120233534-py3-none-any.whl.metadata (2.0 kB)
Collecting cirq-google==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_google-1.5.0.dev20241120233534-py3-none-any.whl.metadata (2.2 kB)
Collecting cirq-ionq==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_ionq-1.5.0.dev20241120233534-py3-none-any.whl.metadata (1.8 kB)
Collecting cirq-pasqal==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_pasqal-1.5.0.dev20241120233534-py3-none-any.whl.metadata (1.8 kB)
Collecting cirq-rigetti==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_rigetti-1.5.0.dev20241120233534-py3-none-any.whl.metadata (1.9 kB)
Collecting cirq-web==1.5.0.dev20241120233534 (from cirq)
  Downloading cirq_web-1.5.0.dev20241120233534-py3-none-any.whl.metadata (2.8 kB)
Requirement already satisfied: requests~=2.18 in /usr/local/lib/python3.10/dist-packages (from cirq-aqt==1.5.0.dev20241120233534->cirq) (2.32.3)
Requirement already satisfied: attrs>=21.3.0 in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (24.2.0)
Collecting duet>=0.2.8 (from cirq-core==1.5.0.dev20241120233534->cirq)
  Downloading duet-0.3.0.dev20240320150209-py3-none-any.whl.metadata (2.4 kB)
Requirement already satisfied: matplotlib~=3.0 in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (3.8.0)
Requirement already satisfied: networkx>=2.4 in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (3.4.2)
Requirement already satisfied: numpy>=1.24 in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (1.26.4)
Requirement already satisfied: pandas in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (2.2.2)
Collecting sortedcontainers~=2.0 (from cirq-core==1.5.0.dev20241120233534->cirq)
  Downloading sortedcontainers-2.4.0-py2.py3-none-any.whl.metadata (10 kB)
Requirement already satisfied: scipy~=1.8 in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (1.13.1)
Requirement already satisfied: sympy in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (1.13.1)
Requirement already satisfied: typing-extensions>=4.2 in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (4.12.2)
Requirement already satisfied: tqdm in /usr/local/lib/python3.10/dist-packages (from cirq-core==1.5.0.dev20241120233534->cirq) (4.66.6)
Requirement already satisfied: google-api-core>=1.14.0 in /usr/local/lib/python3.10/dist-packages (from google-api-core[grpc]>=1.14.0->cirq-google==1.5.0.dev20241120233534->cirq) (2.19.2)
Requirement already satisfied: proto-plus>=1.20.0 in /usr/local/lib/python3.10/dist-packages (from cirq-google==1.5.0.dev20241120233534->cirq) (1.25.0)
Requirement already satisfied: protobuf>=3.15.0 in /usr/local/lib/python3.10/dist-packages (from cirq-google==1.5.0.dev20241120233534->cirq) (4.25.5)
Collecting typedunits (from cirq-google==1.5.0.dev20241120233534->cirq)
  Downloading typedunits-0.0.1.dev20241120184900-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (4.8 kB)
Collecting pyquil<5.0.0,>=4.14.3 (from cirq-rigetti==1.5.0.dev20241120233534->cirq)
  Downloading pyquil-4.14.3-py3-none-any.whl.metadata (10 kB)
Requirement already satisfied: googleapis-common-protos<2.0.dev0,>=1.56.2 in /usr/local/lib/python3.10/dist-packages (from google-api-core>=1.14.0->google-api-core[grpc]>=1.14.0->cirq-google==1.5.0.dev20241120233534->cirq) (1.66.0)
Requirement already satisfied: google-auth<3.0.dev0,>=2.14.1 in /usr/local/lib/python3.10/dist-packages (from google-api-core>=1.14.0->google-api-core[grpc]>=1.14.0->cirq-google==1.5.0.dev20241120233534->cirq) (2.27.0)
Requirement already satisfied: grpcio<2.0dev,>=1.33.2 in /usr/local/lib/python3.10/dist-packages (from google-api-core[grpc]>=1.14.0->cirq-google==1.5.0.dev20241120233534->cirq) (1.68.0)
Requirement already satisfied: grpcio-status<2.0.dev0,>=1.33.2 in /usr/local/lib/python3.10/dist-packages (from google-api-core[grpc]>=1.14.0->cirq-google==1.5.0.dev20241120233534->cirq) (1.62.3)
Requirement already satisfied: contourpy>=1.0.1 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (1.3.1)
Requirement already satisfied: cycler>=0.10 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (0.12.1)
Requirement already satisfied: fonttools>=4.22.0 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (4.55.0)
Requirement already satisfied: kiwisolver>=1.0.1 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (1.4.7)
Requirement already satisfied: packaging>=20.0 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (24.2)
Requirement already satisfied: pillow>=6.2.0 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (11.0.0)
Requirement already satisfied: pyparsing>=2.3.1 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (3.2.0)
Requirement already satisfied: python-dateutil>=2.7 in /usr/local/lib/python3.10/dist-packages (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq) (2.8.2)
Requirement already satisfied: deprecated<2.0.0,>=1.2.14 in /usr/local/lib/python3.10/dist-packages (from pyquil<5.0.0,>=4.14.3->cirq-rigetti==1.5.0.dev20241120233534->cirq) (1.2.15)
Requirement already satisfied: matplotlib-inline<0.2.0,>=0.1.7 in /usr/local/lib/python3.10/dist-packages (from pyquil<5.0.0,>=4.14.3->cirq-rigetti==1.5.0.dev20241120233534->cirq) (0.1.7)
Collecting packaging>=20.0 (from matplotlib~=3.0->cirq-core==1.5.0.dev20241120233534->cirq)
  Downloading packaging-23.2-py3-none-any.whl.metadata (3.2 kB)
Collecting qcs-sdk-python>=0.20.1 (from pyquil<5.0.0,>=4.14.3->cirq-rigetti==1.5.0.dev20241120233534->cirq)
  Downloading qcs_sdk_python-0.21.4rc0.tar.gz (538 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 538.7/538.7 kB 16.7 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  error: subprocess-exited-with-error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.
  Preparing metadata (pyproject.toml) ... error
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

On the other hand, if I first do !pip install cirq and then !pip install cirq --pre --upgrade, it gives the following error message after doing the installs:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
tensorflow 2.17.1 requires protobuf!=4.21.0,!=4.21.1,!=4.21.2,!=4.21.3,!=4.21.4,!=4.21.5,<5.0.0dev,>=3.20.3, but you have protobuf 5.29.0rc3 which is incompatible.
tensorflow-metadata 1.13.1 requires protobuf<5,>=3.20.3, but you have protobuf 5.29.0rc3 which is incompatible.
Successfully installed cirq-1.5.0.dev20241120233534 cirq-aqt-1.5.0.dev20241120233534 cirq-core-1.5.0.dev20241120233534 cirq-google-1.5.0.dev20241120233534 cirq-ionq-1.5.0.dev20241120233534 cirq-pasqal-1.5.0.dev20241120233534 cirq-rigetti-1.5.0.dev20241120233534 cirq-web-1.5.0.dev20241120233534 grpcio-tools-1.68.0 protobuf-5.29.0rc3 typedunits-0.0.1.dev20241120184900

Please install using
pip install cirq~=1.0.dev

Specifying cirq~=1.0.dev matches the latest development release of cirq-1.*, but only stable versions of its dependencies.
The --pre option should be avoided, because it selects pre-release versions of all cirq dependencies.
In the first example, it made pip install a pre-release of qcs-sdk-python, which is only available as a source archive and needs a rust compiler to install.  rust s not available in the colab, so it failed.
On my side the suggested command still complains about incompatibility with the pre-installed tensorflow-2.17.1 - just as in your second example.  This error can be ignored unless you need tensorflow in the colab.  If so, you can get a complaints-free installation using
pip install cirq~=1.0.dev tensorflow

(this tells pip to consider tensorflow requirements when resolving dependencies, which makes it use a protobuf version that works with both cirq and tensorflow)


test: cache moduleState in integration tests to reduce exec time

As requested, part of the Components.js components manager is cached during integration tests.
This makes tests a couple of seconds faster on my machine.

Super, thanks!

Super, thanks!


Allow specification of a comparator to control sorting of the libraries.

@jgindin thanks

When do you think you'll release a new build?

after i tried everything
perhaps this weekend

@jgindin ok published :D


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 201 ms

Resolved: ClamAV performance has improved in 8af72e4.


Add bundle transmission size limit option

bodyParser has a default limit of 100kb, which is not quite large enough for ambitiously-sized Node.js applications.
(For example, a boilerplate Ember.js application generated with ember-cli is about 325kb.)
This PR adds a configuration option (limit) to control this value, and raises it from the default to 500kb.
I'm adding better error handling to the CLI in a separate PR.
CC: @michelle @slexaxton

lgtm. Could go higher if we wanted since this is hardly public. But this'll do for a long while I hope.

amazing!

I'll prepare a tag release tomorrow evening, thanks!!

Thanks for merging.


[SS-14] creating image deployment flow

this should deploy to our organization's container registry (https://github.com/orgs/Star-Sync/packages), this way we can access the images without the source code
two ways to make a new release

either manual dispatch (github ui)
a merge into release/* (this sounds like a good idea but maybe there is a better way to handle this)

if this works, we copy it 1:1 to the server repo and can wrap up this ticket.

🙌


feat(connector): add connector nmi with card, applepay and googlepay support

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates

Description

Added support for NMI connector that supports the following payment flows:

Verify
Authorize
PSync
Capture
Void
Execute (Refunds)
RSync

Also, apart from Cards, NMI also support payments via GooglePay and ApplePay.
Additional Changes

[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

New connector integration
How did you test it?

update-time: 15 April 2023, Saturday 21:04

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

@pixincreate there are still some open comments for ErrorResponse handling and errors, please work on that too

@pixincreate there are still some open comments for ErrorResponse handling and errors, please work on that too

Could you please highlight those? I had resolved all of them previously.


Sort should be sticky on /metagame

Via cookie?

Esp when switching season or turning tournament-only on and off


Fb python3 integration

Hello Gaetan,
hope you are doing fine :D
I am updating das to be able to use python 3, and still keep compatibility to python 2
as far as the test goes, it works properly until it hits "Unicode", the schema cannot load properly,
because of how python 3 works

Hello Gaetan, Thank you for checking this,
I realize that my code is all over the place, I should have define one python version in one module like how "six" does, and call that module on each module that need the change.
as far as I test it, it works the way how develop branch work,
but, my code is really dirty


bpftrace prints out incorrect values for kernel atomic_long_t

drgn:
$ sudo drgn
drgn 0.0.1 (using Python 3.6.3)
For help, type help(drgn).
>>> import drgn
>>> from drgn import cast, container_of, execscript, NULL, Object, reinterpret
>>> from drgn.helpers.linux import *
>>> find_task(prog, 1).cred.user.locked_vm.counter
(long)14562

bpftrace:
$ cat atomics.bt
#include <linux/cred.h>
#include <linux/sched.h>
#include <linux/uidgid.h>

profile:hz:99
{
        $task = (struct task_struct *)curtask;
        $pid = $task->pid;
        $cred = (struct cred *)$task->cred;
        $user = (struct user_struct *)$cred->user;
        $uid = $user->uid.val;
        $lock_pages = $user->locked_vm.counter;
        if ($pid == 1) {
                printf("uid=%lu, pid=%d,locked_vm: %lld", $uid, $pid, $lock_pages);
                exit();
        }
}

$ sudo bpftrace atomics.bt
Attaching 1 probe...
uid=0, pid=1,locked_vm: 16124

Doesn't look like there's any interesting bit patterns:
$ python3
>>> bin(16124)
'0b11111011111100'
>>> bin(14562)
'0b11100011100010'
>>> bin(16124-14562)
'0b11000011010'
>>> hex(16124-14562)
'0x61a'
>>> hex(16124|14562)
'0x3efe'
>>> bin(16124|14562)
'0b11111011111110'

Spent some more time digging. I looked at the llvm IR and all the offsets seemed reasonable. There's a lot of #ifdef's in struct task_struct so my initial thought was that b/c bpftrace doesn't know the define values it has to guess and maybe offsets were wrong. But fortunately bpftrace has btf support so I tried that too and the results were the same.
Then somehow I had the brilliant idea of slamming up-arrow-enter in drgn while bpftrace was running. When you do that, the values actually line up. I think bpf infrastructure is causing this counter to move. So when bpftrace isn't running, it's one value. But when bpftrace is running, it's another.
Funny how things are interconnected like this.


Crash on Go To Definition (F12)

Using PTVS 3 preview, on VS2017 RC, clicking F12 or Right-click + Go To Definition, results in a crash of visual studio whenever the definition is not in the same file.

Could you open Tools, Python Tools, Diagnostic Info and either post the contents as a gist and link it here, or email as an attachment to<EMAIL_ADDRESS>

Sure: https://gist.github.com/caligula1989/a286f27d07ad9aa2d59d417ca5828be1

Thanks. Looks like Microsoft.PythonTools.Language.EditFilter.GetDefsRefsAndValues needs better protection against incoming null values (and we need to figure out why we're passing in null values).


Modular GW.js Support

This fits https://github.com/thegroundwork/groundwork.js/pull/12 into the cleanup work located at https://github.com/thegroundwork/groundwork.js/pull/13.
Additionally, a few changes:

Renamed dist to lib for idiomatic and clarity reasons: import Events from 'groundwork.js/lib/modules/Events'.
Moved tests in with sources under a src/__test__ folder.
Added new --modules build path in webpack.config
Added a new modules example, which still needs to be built out with a UI. Right now it just outputs the result of a modular instantiation into the console.

To run:

npm install && npm run example:modules

@DarrenN - There might be a few loose details here and there that I need to address but this is largely ready to review.
TODO:

[x] Style modules example
[x] Update Makefile to move lib to latest over s3 rather than in project so that we have better modular imports
[x] Update / cleanup README.md
[x] Flip keys and CDN back to PROD (cdn, cloudfront travis)

I see there is now a React example. Is there still a Non-React example?

For some reason npm watch-dev doesn't start the server. It hangs somewhere during the webpack compile:

You'll notice in the activity monitor that Node isn't doing anything, it looks like the process just stopped somewhere in the pipeline. I blew away node_modules and started over, same thing, it stops right here:
/Users/darren/.nvm/versions/node/v4.4.7/lib/node_modules/groundwork.js -> /Users/darren/github/groundwork.js
NODE_ENV=development node_modules/.bin/webpack -w
Hash: 3bce21322dc300d12243
Version: webpack 1.13.0
Time: 5139ms
                  Asset     Size  Chunks             Chunk Names
          groundwork.js   619 kB       0  [emitted]  index
      groundwork.js.map   744 kB       0  [emitted]  index
             index.html   3.9 kB          [emitted]
    examples/index.html  3.08 kB          [emitted]
     examples/auth.html  8.18 kB          [emitted]
examples/supporter.html  9.17 kB          [emitted]
   [0] multi index 28 bytes {0} [built]
    + 340 hidden modules
Child html-webpack-plugin for "index.html":
        + 4 hidden modules
Child html-webpack-plugin for "examples/index.html":
        + 4 hidden modules
Child html-webpack-plugin for "examples/auth.html":
        + 4 hidden modules
Child html-webpack-plugin for "examples/supporter.html":
        + 4 hidden modules

My env:
$ npm -v // 3.10.5
$ node -v // v4.4.7

When I run the examples (ex: cd examples/modules && npm install && npm start) I didn't get a prompt about ports, my browser was automatically opened to 3000:

And my terminal went to:

There was nothing at http://localhost:3001/
That said, when I popped open console on 3000 I did the see the modular builds messages:

🎉

Q: Why are we running eslint-flowtype in the examples? I ask because it immediately tossed an error in Emacs because I don't have it globally installed.

Q: I'm looking at the compiled modules in /lib while running the examples and the individual modules look crazy large:

Is this just an artifact of the dev builds or is something amiss?

If we're going to drop the tests into the /src folder I think I'd prefer they lived in

Was able to get the non-React examples up and running with npm start. We'll need to adjust those examples to use the newer builds:
Uncaught TypeError: Groundwork is not a constructor

from Line 103 in index.html:
 var gw = new Groundwork();

@DarrenN - Answers inline:

I see there is now a React example. Is there still a Non-React example?

Yup, the "simple" examples are still the same, meant to illustrate using GW.js without needing a full app setup.

For some reason npm watch-dev doesn't start the server. It hangs somewhere during the webpack compile:

This is because it doesn't actually start a server, but a watcher on webpack (not webpack dev server). Why? Because when we're developing examples (which have their own server) all we need is the ability to recompile on the source-code side so that the example picks it up. I will clarify this process today.

When I run the examples (ex: cd examples/modules && npm install && npm start) I didn't get a prompt about ports, my browser was automatically opened to 3000

bleary eyed

Why are we running eslint-flowtype in the examples? I ask because it immediately tossed an error in Emacs because I don't have it globally installed.

This is part of the cool create-react-app util that we're using on the examples side. It shouldn't be asking for anything global; let me research

Q: I'm looking at the compiled modules in /lib while running the examples and the individual modules look crazy large

Will be optimizing this today.

If we're going to drop the tests into the /src folder I think I'd prefer they lived in a test folder as we do in other projects.

Will do. I agree it looks a little cluttered, but was trying to avoid the '../' in the test imports and was curious what you thought. NBD though, will update.

@DarrenN (moved this from slack to here) - can you do me a favor and pull down latest on gw.js and then run make run-examples and see if everything boots ok (should see window pop open at localhost:3000 and  then go into example source and see if any linter errors shown in emacs?

Was able to get the non-React examples up and running with npm start. We'll need to adjust those examples to use the newer builds:

This was due to me exporting as ES6 modules
// Before
export default Groundwork;

export {
  groundworkFactory
};

// Then, in html page
console.log(Groundwork) // => {_esModule: true, default}

Switching back to
module.exports.groundworkFactory = groundworkFactory;
module.exports = Groundwork;

Fixed it.

@damassi make run-examples ran fine and asked me for a new port as I already had 3000 in use. Works.

Awesome, final PR coming up in a sec

Re: flow-type
You misunderstand, because the flowtype plugin is defined somewhere (I'm guessing in an .eslintrc or similar) my local eslint wants to use it, but can't because I need it to be globally installed -> https://github.com/syl20bnr/spacemacs/blob/a298a5a311dfe8b449b5f538740303f1ab3d5695/layers/%2Bframeworks/react/README.org

I thought i disabled eslint via a change i made in package.json -- did you try to see if it still threw a problem?

Alright @DarrenN - this is good to go, and got the module sizes down and README.md updated:

Can you give it one last pass and let me know? (Note that i'll be out till Monday.)

Looks great @damassi - :+1: - lets talk on Monday before merging just to cover everything.


feat: add database dropdown to dashboard import

Currently, when importing a database from a JSON file, the process
looks at the database name from the source (the info is in the file)
and matches the datasources to that name. If no database by that name
exists, it simply fails.
With this PR, we add a database dropdown that allows the user to specify
which databases the datasources should target as the get upserted.
I want to stress that the code in this area is not in a great shape,
and that the challenge of serializing/deser the nested objects is
challenging, but that there should be a much better way to do this.
One of the improvement (out of scope for this PR) that would allow to
simplify those import/export would be to use UUIDs for
importable/exportable objects.
Another identified issue is the indirections between
utils/import_expor_{model}.py on top of {Model}.import_object. Not
addressing that here.
Next topic is the MVC stuff. Decided to stick with it for now as this is
more of a [obious missing feat:] than a rewrite.
SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Do you think that doing this will resolve the issue that if Dashboard to import is using legacy Druid Driver?

@eugeniamz we're more heading towards deprecation at this time, I'd rather help people migrate off the legacy connector and onto SQL druid than investing in that direction.
In theory we could make the dropdown include databases + druid clusters, but that's one more thing to untangle/deprecate...

@dpgaspar @villebro , addressed most comments. Clearly export/import needs more attention, but this tackles important shortcoming of lacking the ability to target a database, and fixes the slug bug.

FYI @mistercrunch this needs a rebase.


Add flatten processor

Description
Adds flatten processor described in #4128.
Issues Resolved
Resolves #4128
Check List

[x] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Doc website issue: https://github.com/opensearch-project/documentation-website/issues/6448


[PluginsPatcher] Doesn't work and obfuscated

Hello, I'm facing the issue when Unity doesn't preload my vulkan plugin in build mode, while it's perfectly working in Editor.
I wanter to try PluginsPatcher on my dll, but it fails;
wh0am15533 Unity Plugin Patcher
Header Valid
Loading ClassData

Unhandled Exception: System.InvalidOperationException: Sequence contains no elements
   at System.Linq.Enumerable.Aggregate[TSource](IEnumerable`1 source, Func`3 func)
   at ..(AssetsFileInstance )
   at PluginsPatcher.Program.(String )
   at ..(String )
   at ..(String[] )

It seems like the patcher code is obfustated so I can't check what's wrong.

Hard to work with things that are obfuscated yeah. Probably why he was banned on UC..


Provide clearer documentation as to the logic-less advantage of Dynamic Names?

Hi all,
After discovering the optional Dynamic Names feature (the README documentation also needs updating, by the way: it currently states that "at present, the only module being described as optional is regarding support for lambdas", which is no longer correct), and reading the documentation in the ~dynamic-names.yml file itself, I came away with the conclusion that it was an unnecessary addition to the language: that you had to munge up your data with extraneous dynamic names for something that already could be solved with if/elses.
However, the biggest pro for Dynamic Names - the one that defeats the above argument, and a con not documented in the if/elses section - is the fact that including a bunch of if/elses inside mustache templates is inherently logic-ful, not logic-less. Since ~dynamic-names.yml only implies this and never mentions it explicitly, I believed that Dynamic Names were a misfeature. However, after reading #54, I came across this comment, which explicitly states the above: one of the advantages of logic-less templating is the lack of logic (duh), and Dynamic Names removes logic from the templates entirely. Interestingly enough, that con is presented for lambdas, but is missing from the if/else section. The pro of Dynamic Names adhering to the logic-less ethos - which as I again believe is the biggest advantage they have over other implementations - is likewise missing from its details.
I think it would be worthwhile to explicitly document the above in the ~dynamic-names.yml file. Can I go ahead and make a PR?

Also considering GraphQL (, and TypeScript), a more real-world example could replace the current one in that file?: instead of dynamic: text|image, it could be __typename:text|image. GraphQL could even be called out by name (e.g.: "... Often, this dynamic name can be a type. This type can be automatically retrieved from many databases. GraphQL, for example...")
Not only is this more real-world, but it makes the example more concrete. The current "dynamic" Dynamic Name example seems purely fabricated in my opinion - as if a developer would have to manually ensure each item in items had that field - but dynamically loading a partial based on an already-preexisting type (such as __typename) could show that that value could likely be auto-populated automatically from a database.

I'm not 100% sure I understand all of your suggestions, but it sounds sensible.
Regardless, everyone is welcome to submit pull requests; permission for that is never needed.
If I understand correctly, the changes you suggested in your second post could exist independently of the changes you suggested in your first post. If that is correct, I encourage you to submit two pull requests, so we can also review the changes independently.

Will do!

@chrisdotcode
I agree the documentation does not make it clear and it took me awhile to see value as well.
I don't know if this helps but the way I see dynamic names when used with partials is you are trying to dispatch based on the shape of the data. In some programming languages this is done with pattern matching (which given your experience with PureScript I think you know).
The problem is most make their data struct like and do not have polymorphic data (e.g. a list of heterogenous types of objects) as having mixed types is tricky for regular JSON serialization.
Your GraphQL example is great example that hopefully spans across other languages without the need to get into category theory.


create cli to create demo lom records

[ ] As in other datamodel modules there has to be a cli to create demo records. (Like https://github.com/tu-graz-library/invenio-records-marc21/blob/c1a0830cb7da756e2284cb7685f86a498428eff5/invenio_records_marc21/cli.py#L144, https://github.com/inveniosoftware/invenio-rdm-records/blob/66e4f03551a7c3a03cb2705bd0c6f417e5ed407d/invenio_rdm_records/cli.py#L31)
[ ] Create the directory "fixtures" and place all what is necessary to create fake records there. (Like: https://github.com/inveniosoftware/invenio-rdm-records/tree/master/invenio_rdm_records/fixtures)
[ ] Write tests to verify that all is working as expected

Metadata creation, CLI, tests and printing created metadata to console were added in #25.
CLI-command was changed to publish to the database instead of printing to console, in #27.
CLI-command was changed to publish linked records of different resource_types (course, unit, file), in #28 .


Adding missing data argument to ios_unit_test

Summary of changes

Patch ios_cc_test and add missing data arguments to the underlying objc library and unit test targets.
Switch to using bazel native-supported ios test runner from GTM which appears to the culprit for preventing accepting data arguments.

Test  & Verification

grpc_bazel_cpp_ios_tests recovers and pass

Closes #28688

Thanks for making this change denny!


add an Observable that knows that global oneSignal has enabled.

I am submitting a

[x] Feature Request

What is the expected behavior?
What is the current behavior?
What are the steps to reproduce?
What is the use-case or motivation for changing an existing behavior?
Which versions are you using for the following packages?
Angular:
Angular CDK:
Angular CLI:
Nx (option):
ngx-onesignal:
Is there anything else we should know?

Support in v8.0.4


Eliminate some obviously useless threading

Problem
pub fn ed25519_verify_disabled(batches: &mut [PacketBatch]) in perf/src/sigverify.rs
does pretty much no work (sets a boolean flag for each Packet in PacketBatch) and yet it creates doubly nested parallel iterators to achieve this results. It further does this on rayon's global thread pool so it is impossible to do accounting on this activity.
Summary of Changes
Changed the function to set the flags in single thread. Unless there are several thousands of packets in those batches, this will be faster.

Sorry I have pushed the wrong diff. Should not make PRs late at night. Especially the "small and trivial" ones.


Harmonize CNA and Expression templates

Neither CNA or expression template headers are inconsistent throughout the database. These will needed to be harmonized to one standard header for use in postgres.
Furthermore, many of the columns in CNA and expression are sparse, redundant or only used for obscure formats. These will be considered to be omitted in the harmonized headers.
ACCEPTANCE CRITERION

[x] CNA data is harmonized
[x] Expression is harmonized
[x] Cyto is harmonized

Get table of cyto to sequence here: http://genome.ucsc.edu/cgi-bin/hgTables

Discussion of staining types: https://github.com/monarch-initiative/dipper/issues/43

Pushing CNA templates. Remove LIH CNA data until it can be further harmonized.
Throughout these transformations I investigated the use of some of the columns in the data. All columns are used, somewhere:
gistic is used in PPTC
picnic is used by charles river
log10r and fold change I could only find in Jax. fold change seems more relevant than log10r, since it's not as commonly used in CNA. Also, if we had log2r it would be easy to resolve log10r.
P-value column was removed. It was not used anywhere and only on 2 sheets. I'm not sure it's intent or use in CNA data. It seems like Z-Score would be more applicable and easier to work with.

Expression data is fine. I had to add column "ncbi_gene_id" to UOM-BC, UOC-BC and NKI expression sheets. CRL had to have "transcript_id" changed to "ensembl_transcript_id"

@done


Doesn't recogonize Apple .pages and some .docx

Hi there, hen I submit an Apple .pages file or certain .docx files, it comes back as application/zip despite including mimemagic/overlay, any ideas why? Attached are the files in question.
unknown_files.zip
edit: also some extra context, the docx file was created by opening up a Google doc and selecting 'download as docx'.

This looks like the same issue as https://github.com/minad/mimemagic/issues/39

Invalid, this should be fixed in the shared-mime-info database upstream.

Invalid, this should be fixed in the shared-mime-info database upstream.


Left border in Zen mode of non-centered layout

Border left too much space in Zen mode of non-centered layout with line numbers off.

----
Version: 1.79.2
Commit: 695af097c7bd098fbf017ce3ac85e09bbc5dda06
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/115.0
Embedder: vscode.dev

Another problem is the Full Screen mode made the Menu Bar unnecessarily disappear (and cannot be re-enabled [1]).
[ [1]
    Precisely, cannot be easily re-enabled:
    As "View: Toggle Menu Bar" only unconditionally toggles "window.menuBarVisibility" between "compact" and "classic". ]

Border left too much space in Zen mode of non-centered layout with line numbers off.

When in Zen/Centered mode, you can resize one sash by click&drag while holding Alt. Also, you can reset the size by double-clicking the sash.

Irrelevant of described problem.
Regardless of in centered layout or not (and actually not Zen mode specific), the left margin with line numbers off appears to be excessive.