Please send pull requests to Crystal :-)

Hi!
The code you are writing in Crystal is quite amazing! Mongo, AMQP, and now correct and complete bindings for OpenSSL.
The OpenSSL bindings in Crystal are poor/incomplete because we didn't have time to finish them, and we also lack a total understanding of its API (which is, again, just lack of time). If you can, please submit pull requests with your additions/changes, we would be more than happy to merge them :-)
I also really like that you wrote specs for this.
Our idea of Crystal's standard library is that it has support for most of the standard libraries and protocols out there. OpenSSL is very common so it would be nice to have excellent support for it out of the box.

Yes, my plan was to create a pull request eventually. At this point I'd like to have a separate repository as a playground where I can add features/tests without constantly making pull requests. But definitely I do like to see it in a standard library. Thanks for the kind words!

+1 for PR to Crystal, I've been playing around with adding missing things to the OpenSSL lib also, you can see here: https://github.com/manastech/crystal/pull/669
If you want I can also help you test and open issues until you feel your lib is stable enough to be added into the standard library.

@bararchy absolutely. any help is appreciated.

@datanoise @bararchy Of course, take your time, there's absolutely no rush in any of this. And big thanks for the help! :-)


gnu-efi: 3.0.15 -> 3.0.17

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gnu-efi is: GNU EFI development toolchain
meta.homepage for gnu-efi is: https://sourceforge.net/projects/gnu-efi/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.17 with grep in /nix/store/ik3i4chh4bdld9kj0a80xw8pg39w5nz6-gnu-efi-3.0.17
found 3.0.17 in filename of file in /nix/store/ik3i4chh4bdld9kj0a80xw8pg39w5nz6-gnu-efi-3.0.17

Rebuild report (if merged into master) (click to expand)

17253 total rebuild path(s)

17253 package rebuild(s)

First fifty rebuilds by attrpath

AusweisApp2
CHOWTapeModel
ChowKick
ChowPhaser
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
EBTKS
EmptyEpsilon
HentaiAtHome
MIDIVisualizer
OSCAR
OVMF
OVMFFull
PageEdit
R
SDL2
SDL2_gfx
SDL2_image
SDL2_image_2_0_5
SDL2_mixer
SDL2_mixer_2_0
SDL2_net
SDL2_sound
SDL2_ttf
SDL2_ttf_2_0_15
SDL_audiolib
SDL_compat
SDL_gpu

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ik3i4chh4bdld9kj0a80xw8pg39w5nz6-gnu-efi-3.0.17 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gnu-efi https://github.com/r-ryantm/nixpkgs/archive/6efc3981aab2b94545beef6e17a467bedfe748cf.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ik3i4chh4bdld9kj0a80xw8pg39w5nz6-gnu-efi-3.0.17
ls -la /nix/store/ik3i4chh4bdld9kj0a80xw8pg39w5nz6-gnu-efi-3.0.17/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

Broke this one, I think:
https://hydra.nixos.org/build/221624593/nixlog/2/tail

https://sourceforge.net/p/gnu-efi/bugs/37/
Let's revert this PR, then.


feat!: new data proxy activation logic for Prisma 3.15.x

Updates the way the Data Proxy gets activated in the Prisma Client. This will be effective in Prisma 3.15.0.

It looks like const PrismaClientRuntime = params.useDataProxy ? PrismaClientRuntimeProxy : PrismaClientRuntimeLocal came back after the merge.

I wish we had tests on this library :D not yet, but I'm happy the PR comes from basically the most knowledgeable person on 🌍  about Prisma Client :D :D


Add support for CFTimeIndex in get_clean_interp_index

[ ] Closes #xxxx
[x] Tests added
[x] Passes black . && mypy . && flake8
[ ] Fully documented, including whats-new.rst for all changes and api.rst for new API

Related to #3349
As suggested by @spencerkclark, index values are computed as a delta with respect to 1970-01-01.
At the moment, this fails if dates fall outside of the range for nanoseconds timedeltas [ 1678 AD, 2262 AD]. Is this something we can fix ?

Hello @huard! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/tests/test_duck_array_ops.py:

Line 716:1: W391 blank line at end of file


Null pointer exception on actor initialization

Expected Behavior
I have a service that registers an actor. On dapr run the service should start without errors.
Actual Behavior
With the following command:
 dapr run --app-id actors --app-port 3000 dotnet run

A null pointer exception is thrown:
== APP == Unhandled exception. System.ArgumentNullException: Value cannot be null.
This declaration is valid but fails:
abstract Vote: Animal -> Task<Votes>

To workaround this issue,, the method must be declared as:
 abstract Vote: animal: Animal -> Task<Votes>

Thanks for sharing a workaround.
The dapr .net sdk only supports c# currently.

I'm starting to get into F# and would love to use it w/ Dapr. Are there any plans for a F# friendly package?

I'm starting to get into F# and would love to use it w/ Dapr. Are there any plans for a F# friendly package?

Thats would be a good addition, there are no plans to make a F# friendly package as of now, We would appreciate your contribution if you want to do a PR for it.

This seems like a bug that could be fixed though right? Is this a protocol limitation or just a bug in the IL generation code?

This seems like a bug that could be fixed though right? Is this a protocol limitation or just a bug in the IL generation code?

Yes, it can be seen as a bug when it comes to enabling F#. At this point core focus is to enable C# development and fix issues found for it.


Publication Datasets

Encode had dataset structure, which is also linked from publications. We have that field in publication but there is no dataset schema.
Are we going to have datasets, or shall I delete that from publication?

Do I remember correctly that in encode, datasets and experiments are almost matching one-to-one? I don't think we need dataset objects.
And we have a schema for ExperimentSet, right? What is the use case we have in mind for that? Can an experiment be in multiple sets?
I think we can have publications linking an array of experiments or an experiment set.
Does that make sense?


Monitor State sync with each build

Is your feature request related to a problem? Please describe.
State sync is cool, but unreliable.
Describe the solution you'd like
Add a github action that tests state sync with each software change.  Make it a first class citizen.
Describe alternatives you've considered
I have tried many ways to sync nodes, and they are all much too slow and much too stateful.
Additional context
akash-network/node#1366 is a first stab at this.  It is not expected to pass all the time, but instead to give us an idea of how it's going.

Checked with @troian , we still want this except that the cronjob should be somewhere else; not the node repo.

as per Artur during today's sig-support meeting:

state-sync needs to be tested on different stages including the mainnet;
incorporate scripts from Osmosis team;
take existing state of the validator set and then state-sync from that, because state-sync is sensitive to the AppState;

GH action within node repo and can be triggered either manually or on release candidates. Or use a matrix state.


feat: page 支持通过 props 获取 query

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.03% :tada:

Comparison is base (dbfc996) 93.97% compared to head (c1307ff) 94.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   93.97%   94.00%   +0.03%
==========================================
  Files          14       14
  Lines        1594     1602       +8
  Branches      271      273       +2
==========================================
+ Hits         1498     1506       +8
  Misses         96       96

Files Changed
Coverage Δ

src/element.ts
98.02% <100.00%> (+0.02%)
:arrow_up:

src/page.ts
100.00% <100.00%> (ø)

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


Resolved disallowed-html-tag build warnings in ad domain services

Docs Build status updates of commit b2bf5bc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azps-7.5.0/Az.ADDomainServices/Get-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ADDomainServices/Remove-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ADDomainServices/Update-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-8.0.0/Az.ADDomainServices/Get-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-8.0.0)

azps-8.0.0/Az.ADDomainServices/Remove-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-8.0.0)

azps-8.0.0/Az.ADDomainServices/Update-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-8.0.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


replace http parser?

@asvetlov should we replace http parser with https://github.com/MagicStack/httptools?
looks good http://magic.io/blog/uvloop-make-python-networking-great-again/

@danielnelson ^^^

aiohttp is the only viable option for using the new uvtools library but aiohttp's http parser is a serious bottleneck at the moment. How can we help fixing this or possibly swapping out the http parser with httptools?

i will take a look this weekend, should be relative easy to use httptools instead of parser

added basic integration in @6dae1a28c863e2d0e7ce7b8cda7d3e1c2f3eee0d
passes most of tests from test_client_functional_oldstyle.py

Do you see any improvement in performance?

i am not at that stage yet. need to fix more tests.

@fafhrd91 I've also added support for HTTP upgrade in v0.0.7.

i don't like HttpParserUpgrade exception for upgrade, i think it should be should_upgrade() method.

all aiohttp tests pass with new parser

The code is in the httptools-integration branch, right?

yes

i didn't work on any optimization though

Interestingly, I don't see any difference in performance ;(

That means that there is another bottleneck, perhaps even worse than the parser.

I test it on 3.4.3, I see lower performance with http parser. Maybe http tools should do all headers manipulation and return dictionary.
Sent from my iPhone

On May 8, 2016, at 4:13 PM, Yury Selivanov<EMAIL_ADDRESS>wrote:
That means that there is another bottleneck, perhaps even worse than the parser.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

On Python 3.5.1 @ Mac OS X I see roughly equal performance. Doing headers manipulation in httptools will give it a little performance boost, but nothing significant.
A rudimentary asyncio HTTP protocol server does 50,000 requests/s on my machine, and aiohttp only 3,300. While aiohttp does more work (supports timeouts, deflate etc) it shouldn't be 15 times (!) slower.
httptools parser should be faster than any parser implemented in pure-Python, that's just C vs Python story. If the main bottleneck of aiohttp isn't parsing, it should be something else, I suspect it can be the buffering code.

i just tried to find bottleneck. replacing aiohttp.web with aiohttp.server increases performance about 70%,
but what is interesting, everything else gave at max 10% increase. i removed parser, parser buffer, simplified headers processing, removed generators on write path, remove a lot of check during write operation. and still it gave only 10% increase. at this point i think python just slow and in any case speed of web server is useful only for benchmarks.

replacing aiohttp.web with aiohttp.server increases performance about 70%

Do you see any difference of httptools vs pure-Python parser for bare aiohttp.server?

no

Could you please push your code to some experimental branch?

BTW, did you try to disable aiohttp flow control (CORK)?

added to experimental-pertest
replacing FlowControlDataQueue with simple DataQueue gives ~30% more

i also added simple bench.py script

Yeah, still, there is a ton of Python code in streams/parsers/etc. While aiohttp architecture is super high-level, flexible and modular, it executes a lot of Python code, and yeah, Python is slow :(. For example, DataQueue uses futures and coroutines and it's only a bridge between parser and the protocol -- it can be removed, or replaced with a simpler code. ServerHttpProtocol.start coroutine can be implemented in ServerHttpProtocol.data_received. I think this complexity can be reduced significantly, albeit that will require a serious rewrite. Anyways, my 2 cents. I'll take a more serious look tomorrow.

Any updates on this, very interesting read?

replacing aiohttp.web with aiohttp.server increases performance about 70%

Does this mean I should try to write all my code just using .server and not using any .web code?

Basically it means you should profile everything before making any performance assumptions

@fafhrd91  did you use some flamegraph technique to find the possible bottleneck ?

@allan-simon no, i didn't. but i don't see reason for doing that, python just slow. any increase of performance in http server code would be neglectable compared to app code.

@fafhrd91  ok as I was going with the impression you were looking for a bottleneck.
Regarding your comment, sure at the end most of the time will be spent in the application code itself, and as a user of the library you should look in your code before trying to switch library. But you also a whole set of microservice that would benefits from high performant http frameworks (for example right now i'm writing one that is streaming a .csv file downloaded form amazon S3 and transforming it on the fly (with some filtering) on a json array (that I can stream too by, one element by line) to assure retro-compatibility, in such case my application code is itself very small and the time spent in the library start to become important.

we at Keepsafe have plenty on streaming and transforming work, we stream terabytes of data in/out os s3 and aiohttp works pretty well on this load. i just say aiohttp on itself is fast enough but if you need to go faster you need to search other options and those options are outside of python.
ps i remember i did a lot of profiling around 0.15 version, and could get 30-50% performance increase in synthetic tests, in there was no any change in performance in real system. (turning off logging gave  more increase than any optimizations)

I see, thanks for the feedback, so your point of view would be that replacing the io loop by uvloop or the http parser is actually not worth the burden ?

uvloop is worse if you want to write some very low level and simple network protocols, otherwise you just won't see any performance benefits. for high level protocol and complex application it doesn't matter. and again, aiohttp is fast enough, so just use it and write network services :)

great, thanks for answering all my questions.

oh just one last question, are you using your applications behind gunicorn  or directly behind nginx (with a supervisor or similar taking care of the service life cycle)?

we use gunicorn and nginx

@1st1 I am working on parser refactoring. You mentioned idea of removing start() and replace with different implementation, could you elaborate on that?

All work is in this pr https://github.com/KeepSafe/aiohttp/pull/1626


Expose mounted property in Ref

There are several issues about how to determine the mounted status of providers after asynchronous operations. I am aware that you do not agree with mounted being a public property, but the solution you recommend is no further than that. As you suggested in this comment (https://github.com/rrousselGit/riverpod/issues/1879#issuecomment-1303189191), the Ref.onDisposed method can be used to determine the mounted state. As I looked under the hood, the mounted property is always consistent with the disposed status and is set to false before the Ref.onDisposed call. I don't understand why it is a problem to make this property public, thus reducing the boilerplate code. If someone writes a variable that is set to true by onDisposed, it is practically the same as reading mounted.
I use it in many places, mainly because a new lint rule (use_build_context_synchronously) keeps warning me about this. Unfortunately, due to Riverpod's implementation, this cannot be done with a simple mixin or extension function, so I have to fill most of my providers with a bool disposed + onDispose(() => mounted = false) combo, which makes it quite dirty. I have now replaced it with this solution and it works without problems. Since only Ref declares this property, it is not available in WidgetRef outside of a provider
https://github.com/rrousselGit/riverpod/issues/1879
https://github.com/rrousselGit/riverpod/issues/1908
https://github.com/rrousselGit/riverpod/issues/1579

Hello!
It isn't visible on purpose. It is destined for internal use only
The linked issues explain why there is no mounted. A boolean doesn't correctly represent the problem. https://github.com/rrousselGit/riverpod/issues/1879#issuecomment-1303189191


[DEPRECATION]: use factoryFor

http://emberjs.com/deprecations/v2.x/#toc_migrating-from-_lookupfactory-to-factoryfor
Fixes::
Ref #72

CI failing for other reasons, this looks good - thanks!
@toranb are we targeting Node 0.12 still or can I pin Travis to >= 4?  ember-cli ecosystem dropped support for 0.12 which is the reason for the CI failure.

@jasonmit great point - hey @snewcomer you mind updating the travis configuration to drop legacy node quick? much appreciated :)
https://github.com/ember-redux/ember-redux/blob/master/.travis.yml#L4
above ^^ is an example of using nodeJS 4 instead of what we have here today

CI failing for reasons unrelated to this PR against beta and canary so I'm going to go ahead and merge.  Thanks!

@jasonmit huh, well normally I'd take the time to look into why the beta is failing / etc :)
@snewcomer would you be up to help nail down what broke in ember 2.12+? If not, I should free up early next month to see what broke and land a PR (mostly curious if you've seen or had issues using this addon with ember 2.12+)

@toranb master is failing with the same reason pre-this-change (it's happening against most of my addons so likely a breaking change).  I was planning to look at it separately.

@snewcomer @jasonmit thank you both for the effort on this! I'm thankful for the community surrounding this addon! To confirm we are up to date now and have no deprecation warnings blowing up the console => I ran the build (using ember-cli v2.10.1) and both ember v2.12 / v2.13.0-beta.1


OPEN_ORDER_LOTSIZE_OUT_OF_RANGE and OPEN_ORDER_PRICE_ZERO for usa500

I am using python for communication with DWX Server and I got strange errors when I want to work with usa500 symbol. I got error OPEN_ORDER_LOTSIZE_OUT_OF_RANGE , but when i disable this check in code i got error OPEN_ORDER_PRICE_ZERO. It looks like that function MarketInfo doesn't work as expectet for this symbol.
Also I have noticed that for symbol ger40 I got error OPEN_ORDER_PRICE_ZERO .
By the way, there is typo error in lines
https://github.com/darwinex/dwxconnect/blob/26a4b38f2aa4237e9e20b506289fd61edfb6c613/mql/DWX_Server_MT4.mq4#L273
https://github.com/darwinex/dwxconnect/blob/26a4b38f2aa4237e9e20b506289fd61edfb6c613/mql/DWX_Server_MT4.mq4#L324
it should be Lot size out of range (min: %f, max: %f): %f  instead Lot size out of range (min: %f, max: %s): %f

Hi,
Thank you for spotting the typo. It is corrected now.
Regarding the error, we can only support Darwinex and not other brokers. On a Darwinex demo accounts it seems to work as expected.
Opening a 0.1 lots SPX500 position was successful. And when I changed the lot sizes to 0.01, it correctly produced the out of range error:

ERROR | OPEN_ORDER_LOTSIZE_OUT_OF_RANGE | Lot size out of range (min: 0.100000, max: 1000.000000): 0.010000

Thank you for great tool that you provided to us !!!


Got an error when using "@"

Here's the relevant line from my JSON:
"records": [{"type": "A", "name": "@", "ttl": 600}]
$ godaddy-dns
[Sat May 21 2016 13:40:35 GMT-0400 (EDT)] Failed request to GoDaddy Api Another record with the same attributes already exists
When I changed "@" to "foo", it created the A record with no problem.
Also noticed that if I change the IP of "foo" on GoDaddy's website to something bogus then re-rerun godaddy-dns, it does not update the IP address and there is no error message.

Hey @nstuyvesant ,
thanks for submitting this issue.
I honestly never tried to use the '@' notation. I assumed it to just work, but maybe there's some protection mechanism from GoDaddy or they simply use something else to reference to the main domain (maybe just an empty string?). I need to investigate more on this side.
For what regards your second point, It's probably the IP cache kicking in and avoiding to send the request to GoDaddy.
So just to make this a little bit cleaner (the documentation maybe goes too short on this part), let me explain what happens exactly behind the scenes:
When you use the command the first time, it detects the public IP of your machine and it stores that into a temp file (you can see the default location of this file by using the --help option). The next time you invoke the command it compares your current public IP with your cached one. If it is the same, it assumes there's no need to update the DNS, so the request to GoDaddy API isn't triggered at all.
At this stage, if you want to force a request anyway, you can either delete the cache file or use the --ipfile option to specify a different cache file.
I should probably change something here to allow the refresh mechanism and the IP cache to be more understandable and easy to use.
I have three options in mind:

Add a --force o --no-cache option that, when set, ignores the IP cache file and always trigger the request to the API server (easy to implement);
Disable the IP cache by default and enables it only when a cache file is explicitly specified through the --ipfile option. (a bit  harder to implement, but more flexible);
Disable the IP cache by default and move the option to the config file itself (as an optional value). The IP caching feature is enabled only when a cache file is explicitly specified. Also add a --force or --no-cache option to override the configuration easily in the case it's needed for a specific run (harder implementation, but definitely the most flexible and probably the easiest to use from a user perspective).

At this stage I am more inclined to implement the third option, but feel free to let me know your thoughts on this!


Emails not signed when queued

I seems that the DKIM signing classes are not used when a mail is dispatched to the queue. Even writing a job that sends an email will not result in signed mail.
How can this plugin be changed so that queued messages are also using the classes that add the DKIM signature?

Whoops, turns out the problem is not with the plugin, but with the referencing of the private key. In .env I used a relative path for the location of the private DKIM key. This worked fine for sending emails directly from the http process. But cron works from a different basepath, so the private key could not be found. Use an absolute path.
Perhaps an idea to put this in the readme.md


uniqueness_error

Hello @mysticaltech
When I run the latest version, I'm having errors about uniqueness.
But I have triple checked everything, there is no duplicate naming used.
I have also reverted back to apply from my local just to verify if the problem was TF cloud, but this happens in both ways.
I have tried with both true and false for below option, same error output.
 placement_group_disable = true

 Error: SSH key with the same fingerprint already exists (uniqueness_error)
│
│   with module.kube-hetzner.hcloud_ssh_key.k3s[0],
│   on .terraform/modules/kube-hetzner/main.tf line 6, in resource "hcloud_ssh_key" "k3s":
│    6: resource "hcloud_ssh_key" "k3s" {
│
╵
╷
│ Error: name is already used (uniqueness_error)
│
│   with module.kube-hetzner.hcloud_placement_group.control_plane[0],
│   on .terraform/modules/kube-hetzner/main.tf line 51, in resource "hcloud_placement_group" "control_plane":
│   51: resource "hcloud_placement_group" "control_plane" {
│
╵
╷
│ Error: name is already used (uniqueness_error)
│
│   with module.kube-hetzner.hcloud_placement_group.agent[0],
│   on .terraform/modules/kube-hetzner/main.tf line 57, in resource "hcloud_placement_group" "agent":
│   57: resource "hcloud_placement_group" "agent" {

this is my naming:
  control_plane_nodepools = [
    {
      name        = "control-plane-fsn1",
      server_type = "cpx31",
      location    = "fsn1",
      labels      = [],
      taints      = [],
      count       = 1
    },
    {
      name        = "control-plane-nbg1",
      server_type = "cpx31",
      location    = "nbg1",
      labels      = [],
      taints      = [],
      count       = 1
    },
    {
      name        = "control-plane-hel1",
      server_type = "cpx31",
      location    = "hel1",
      labels      = [],
      taints      = [],
      count       = 1
    }
  ]

  agent_nodepools = [
    {
      name        = "agent-cpx51-fsn1",
      server_type = "cpx51",
      location    = "fsn1",
      labels      = [],
      taints      = [],
      count       = 1
    },
    {
      name        = "agent-cpx51-nbg1",
      server_type = "cpx51",
      location    = "nbg1",
      labels      = [],
      taints      = [],
      count       = 1
    },
    {
      name        = "agent-cpx51-hel1",
      server_type = "cpx51",
      location    = "hel1",
      labels      = [],
      taints      = [],
      count       = 1
    }
  #  {
  #    name        = "agent-large",
  #    server_type = "cpx21",
  #    location    = "nbg1",
  #    labels      = [],
  #    # To exclude from Longhorn if that is used for instance. Completely optional.
  #    taints = [
  #      "server-usage=storage:NoSchedule"
  #    ],
  #    count       = 1
  #  },
  #  {
  #    name        = "storage",
  #    server_type = "cpx21",
  #    location    = "fsn1",
  #    # Fully optional, just a demo.
  #    labels = [
  #      "node.kubernetes.io/server-usage=storage"
  #    ],
  #    count = 1
      # In the case of using Longhorn, you can use Hetzner volumes instead of using the node's own storage by specifying a value from 10 to 10000 (in GB)
      # It will create one volume per node in the nodepool, and configure Longhorn to use them.
      # Something worth noting is that Volume storage is slower than node storage, which is achieved by not mentioning longhorn_volume_size or setting it to 0.
      # So for something like DBs, you definitely want node storage, for other things like backups, volume storage is fine, and cheaper.
      # longhorn_volume_size = 20
  #  }
  ]

  # * LB location and type, the latter will depend on how much load you want it to handle, see https://www.hetzner.com/cloud/load-balancer
  load_balancer_type     = "lb31"
  load_balancer_location = "nbg1"

  autoscaler_nodepools = [
    {
      name        = "autoscaler-fsn1"
      server_type = "cpx51" # must be same or better than the control_plane server type (regarding disk size)!
      location    = "fsn1"
      min_nodes   = 0
      max_nodes   = 5
    },
    {
      name        = "autoscaler-nbg1"
      server_type = "cpx51" # must be same or better than the control_plane server type (regarding disk size)!
      location    = "nbg1"
      min_nodes   = 0
      max_nodes   = 5
    },
    {
      name        = "autoscaler-hel1"
      server_type = "cpx51" # must be same or better than the control_plane server type (regarding disk size)!
      location    = "hel1"
      min_nodes   = 0
      max_nodes   = 5
    }
  ]

@codeagencybe That just means that your project already has all of these. But I am seeing that the resources that give the error above do not have the cluster_name, so that should be fixed. Will do ASAP, but do not hesitate to PR. What is needed is prefix with the cluster_name as for other ressources.

@codeagencybe Must have been related to the previous error somehow. I tested multiple times, all good now. Please next time do include your kube.tf (just the values without comments, for maximum readability).


Update to reflect currently supported frameworks

@rita3ko could you please resolve the merge conflict?

@rita3ko I merged #772 last week, I think we can keep Jekyll in if we document that requirement. What do you think?


React defaultProps breaks VSCode "Find all references"

Not sure if this bug belongs in TypeScript, VSCode or DefinitelyTyped.
When a component has defaultProps (as advertised in TypeScript 3.0) the "Find all references" command in VSCode doesn't work properly. It returns a reference to the default prop, but none of the JSX instances where the prop is passed.
TypeScript: 3.0.3
@types/react: 16.4.14

I'd try opening an issue on the VSCode repo, too.

https://github.com/Microsoft/vscode/issues/59256


Remove redundant RefreshContractParameters::total_oracles and total_ballots

These are not parameters to the contracts and we already have TokensToMint::oracle_tokens and ballot_tokens.

Done.


Ubuntu 22.04 support

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04 LTS"

Thanks, f8x now supports ubuntu 22.04


An error not by SScript. but a error.

When i tried to compile my Pysch Engine 1.0.0 Prerelease but it said Error: You must have a "project.xml" file or specify another valid project file when using the 'test' command when i ALREADY HAVE THE PROJECT.XML!

Windows 10
CMD lime test windows
Pysch Engine 1.0.0

thats psych engine's fault not sscript, anyway u must be in the directory project.xml is in


[flutter_releases] Flutter beta 3.18.0-0.2.pre Framework Cherrypicks

Flutter beta 3.18.0-0.2.pre Framework
Scheduled Cherrypicks

Submitting as fuchsia failures can be ignored. Tracking issue in https://github.com/flutter/flutter/issues/139998


NETUSD: Deploy Curve Lending to Arbitrum to test launch netUSD on Arbitrum with ETH and WBTC collateral

We have received approval from Curve to fork Curve stablecoin and launch on Arbitrum as netUSD the task are as follows:

Fork Curve stablecoin contract into Volume
Rename Curve Stablecoin to netUSD stablecoin
Deploy Curve Stablecoin Contracts on Arbitrum
Test ability to use Curve Pools on Arbitrum for ETH lending and leverage
Provide update on what we are missing? Oracles etc?

Not needed. Closing.


Add more block import metrics

Add more block import metrics, including gas burnt and gas burn rate.
Drop reporting period from 100 blocks to 1000.
Log one final time at the end of each file.

Signed-off-by: Danno Ferrin<EMAIL_ADDRESS>

Part of my Bonsai Tries work.


Improve README

Like so: https://github.com/patcon/matterbridge-heroku#readme
Will submit PR

Ohai! Just realized you were in Newmarket, Ontario and working with Mozilla...! Is Mozilla using Matterbridge? Really curious about any details that you can share :)

bump

Ohai! Just realized you were in Newmarket, Ontario and working with Mozilla...! Is Mozilla using Matterbridge? Really curious about any details that you can share :)

We used to Mattermost+Matterbridge, but decisions were made somewhere above me to use Slack instead. :(
Feel free to submit that PR and we'll work to get it merged

Going to close this in favour of #5


Methods code

Dear,
Is it possible to share the code of the algorithm?
Cheers

Thank you for your interest in WOODSKE. I am currently collecting data in the field, and I plan to update the project content and the source code of WOODSKE in mid-next month. Please stay tuned for the updates.

Thanks.
Looking forward to your updates！

Hello! The source code of WoodSKE has been uploaded to the Script（in Python) folder.


UF-222 - Remove c:\fakepath\ from the file name text box

+1 to merge.

Thanks.
master: d89ba272b340396cc00c4b143b0b4d2a0d2c7848


send reminder emails for draft versions of deposits

Why was this change made?
implement #589.  from the spec:

"How much time has elapsed before the reminder is issued?  How many reminders are sent before they cease?"
"My inclination is to go with two weeks, and then a month, and then monthly thereafter.  I wouldn't stop as long as the SUNet is valid, but it would be nice if they had an option to request that they stop.  That might be out of scope right now."

This PR as is has a bit of a delta from that description:  I'd originally interpreted as notify at two weeks, then every four weeks after that... but upon re-reading, I think the intent is notify at two weeks, again at four weeks, then every four weeks after that?  If so, it should be easy enough to make that change in this PR or in a follow on (happy to do either).
How was this change tested?
unit tests.  some manual testing locally on rails console and using the mailer preview, e.g.:

Which documentation and/or configurations were updated?
Added the whenever gem and an initial daily cron job to run the new functionality.

@jcoyne i think i've addressed all the suggestions.  had to re-gen .rbi files to get the type-checking to pass after i extracted send_first_draft_reminders and send_first_draft_reminder? to a service class.  sorry for the churn, stuck it in a commit of its own.

After playing with it a bit more and looking up the applicable date functions, I ended up pushing all the selection logic into the query.  For whatever reason, I was thinking it'd be uglier, with more casting or formatting, but the SQL version is actually a lot more concise than the ruby version was.  To support that shift, I indexed works.created_at and works.updated_at.  I left that change in its own commit (only required a very minor test adjustment, since passing the intervals in as Integers started to seem more sensible than using ActiveSupport::Durations, since the SQL wants ints anyway).

I also tacked on a commit that switched back to using created_at, since that was easy to do now that :first_draft is the only state in consideration.  It seemed to make sense given the length of the notification interval.  But it's super easy to revert back to using updated_at, should that be the way to go.

@jmartin-sul You have a small conflict in Gemfile.lock to resolve, then please merge.


Is there a way I can install packages without creating a f environment?

The github repo I downloaded have dumped all the package information in a pipfile. Hence, I have no other option than to use this package. As frustrating it is to use this package, it is also lacks a good documentation.  There isn't anything on the internet regarding this. Is there a way around this or should I have to manually create a requirement.txt file by using Ctrl+C and Ctrl+V?

see here https://pipenv-fork.readthedocs.io/en/latest/advanced.html#generating-a-requirements-txt

You can generate a requirements.txt using https://github.com/frostming/pipfile-requirements


Parallel test runner swallows standard output

The parallel test runner seems to swallow the standard output. See the example below.
test_haas_parallel.py:
import unittest

class TestHaas(unittest.TestCase):

    def test_haas_one(self):
        print('one')

    def test_haas_two(self):
        print('two')

Running haas with the default test runner displays the one and two messages as expected:
$ haas test_haas_parallel.py
one
.two
.
----------------------------------------------------------------------
Ran 2 tests in 0.000s

OK

Running haas with the --runner parallel option makes them disappear.
$ haas --runner parallel test_haas_parallel.py
..
----------------------------------------------------------------------
Ran 2 tests in 0.121s

OK

Python 3.6.5
haas 0.8.0
stevedore 1.29.0

After some investigation on the behaviour of the parallel runner, I found the following:

The stdout and stderr is buffered independently for each test (https://github.com/scalative/haas/blob/master/haas/plugins/parallel_runner.py#L51)
The stdout and stderr content is normally passed to the TestResult through the from_test_case contructor (https://github.com/scalative/haas/blob/master/haas/result.py#L249)
The from_test_case will however ignore the stdout/stderr content if the test did not fail.

So the bug is that the stdout/stderr content is ignored instead of being collected by the parent process. I suspect that the solution to this bug is going to be a little more involved and probably require some small redesign


[Bug]: bad test coverage report minimal reproduction

Version
28.1.3
Steps to reproduce
chris_chiasson@penguin:~/care/jest_coverage$ cat index.js
let flag = true;

exports.exercise1 = function () {
  const result = flag ? 1 : 2; // line 4
  console.log(result); // line 5; definitely "covered"
  return result;
}

exports.exercise2 = function () {
  const result = !flag ? 1 : 2; // line 10
  return result;
}

chris_chiasson@penguin:~/care/jest_coverage$ cat index.test.js
const {exercise1, exercise2} = require("./index.js");
const assert = require("node:assert/strict");

describe("Jest Bug",function () {
  it("should return 1", async function () {
    assert.strictEqual(exercise1(), 1);
  })

  it("should return 2", async function () {
    assert.strictEqual(exercise2(), 2);
  })
})

chris_chiasson@penguin:~/care/jest_coverage$ npx jest --coverage
  console.log
    1

      at log (index.js:5:11)

 PASS  ./index.test.js
  Jest Bug
    ✓ should return 1 (16 ms)
    ✓ should return 2

----------|---------|----------|---------|---------|-------------------
File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------|---------|----------|---------|---------|-------------------
All files |     100 |       50 |     100 |     100 |
 index.js |     100 |       50 |     100 |     100 | 4-10
----------|---------|----------|---------|---------|-------------------
Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   0 total
Time:        0.262 s, estimated 1 s
Ran all test suites.

Test setup is extremely simple, coming from only npm init followed by npm i jest.
chris_chiasson@penguin:~/care/jest_coverage$ ls
coverage  index.js  index.test.js  node_modules  package.json  package-lock.json

chris_chiasson@penguin:~/care/jest_coverage$ cat package.json
{
  "name": "jest_coverage",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "jest": "^28.1.3"
  }
}

Expected behavior
I expect line 5 not to be listed in the uncovered lines, given that its console log statement is being printed during the test and there is only one execution path through that line. After all, it is not wrapped in an if block or other scoping construct that would allow it to be skipped.
Most of this reasoning applies to other lines, but it seems especially egregious for line 5 since it is printed in the output of the test.
Actual behavior
Line 5 is included in the uncovered lines report.
Additional context
I noticed I needed two points in the code that have untaken branches in order to create the invalid span. (Not that I am spending my time creating exotic tests for Jest... just that I decided I didn't believe what it was telling me anymore and decided to prove it.)
Environment
System:
    OS: Linux 5.10 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
    CPU: (4) x64 11th Gen Intel(R) Core(TM) i3-1115G4 @ 3.00GHz
  Binaries:
    Node: 16.14.0 - /usr/local/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.13.2 - /usr/local/bin/npm
  npmPackages:
    jest: ^28.1.3 => 28.1.3

Would you like me to work on this issue? I am new to contributing to this repository, and looking to get involved.

Are you a real person, not a spam bot?

Ok, then you have my permission... not that I have any right to tell you what to do on a public repo that I don't own

Well, I'm having trouble getting started because yarn install is failing. When I run it, I get a huge error log with lots of errors  that say RequestError: getaddrinfo ENOTFOUND registry.yarnpkg.com. I'm not sure what to do about it.


Way to add elements to objects and arrays

Would be nice to make it more complete

same here, i ended up using another component to accomplish addition.


<EMAIL_ADDRESS>decompress error

ERROR Exit code was 2!
Failed to extract files from C:\Users\omriy\scoop\apps\twilio\5.4.1\twilio-win32-x64.tar.
Log file:
~\scoop\apps\twilio\5.4.1\7zip.log

Hi @hamhoum10 ,Could you try installation via scoop in powershell as an admin. Its mentioned in the doc here


METRON-1628 Unable to remove alert from metaalert in Alerts UI

The Alerts UI is unable to remove alerts from an existing metaalert.  This appears to be a regression from #1049 that is similar to #1068.

Testing

Spin-up the development environment and open the Alerts UI.
Create a metaalert containing one or more alerts.
Remove one of the alerts from the meta alert.

Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[ ] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

+1 tested in full dev and the issue is fixed as described.

+1 by inspection

This fix is duplicated in https://github.com/apache/metron/pull/1070.  I verified the fix there also resolves this.

I merged in #1070.  @nickwallen Can you close this PR and the Jira as a dupe?


OMV4 Error. Error setting up entry OMV for openmediavault

OMV4 will throw a python error due to int() in line 198 because of the value 'unknown' van not be converted into an integer.
Proposed change

Type of change

[x] Bugfix
[ ] New feature
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation

Additional information

Checklist

[x] The code change is tested and works locally.
[ ] The code has been formatted using Black.
[ ] Tests have been added to verify that the new code works.
[ ] Documentation added/updated if required.

Does it not return hwinfo version?

On my OMV4 the Integration throws an error stating 'unknown' can not be converted.
The Integration then is inactive with an error in the log:
`Logger: homeassistant.config_entries
Source: custom_components/openmediavault/omv_controller.py:198
Integration: OpenMediaVault (documentation, issues)
First occurred: 11:05:25 (1 occurrences)
Last logged: 11:05:25
Error setting up entry OMV for openmediavault
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 335, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/openmediavault/init.py", line 31, in async_setup_entry
await controller.async_hwinfo_update()
File "/config/custom_components/openmediavault/omv_controller.py", line 132, in async_hwinfo_update
await self.hass.async_add_executor_job(self.get_hwinfo)
File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/openmediavault/omv_controller.py", line 198, in get_hwinfo
if int(self.data["hwinfo"]["version"].split(".")[0]) > 5:
ValueError: invalid literal for int() with base 10: `'unknown'``

so, I checked and it actually had version, but it is capitalized:
https://github.com/openmediavault/openmediavault/blob/4.x/deb/openmediavault/usr/share/openmediavault/engined/rpc/system.inc#L382
According to source code, everything else should be broken too. Does everything work for you?

I only use the OMV filesystem information's.
But it reports all 30 entity's in 4 devices.
I just realized that the Information is lost after 30min when I activate the automatic polling for updates.
If not the Information stays in place.
So it seems the change breaks the automatic update of the information.
Can we change the information coming from OMV so that it is always lower case? Maybe with a lower() when generating the array?
I'll try to modify the file accordingly and try it out.

It seems like it is not as easy after all. It is not only the capitalization, it is also sometimes different wording. At least according to the site you mentioned.
Maybe we should just switch the default to something like "4.0 or lower" then the script runs as normal.
Just mentions instead of unknown "4.0 or lower"
I also found that there is no information on "Version" in "hwstats" in my case. Also no other information except in file systems "fs".
maybe there is another problem with OMV4 but it was working when i installed it the first time. (2020 or so)

well, OMV4 is EOL for over 2 years. underlaying OS is under LTS only for 3 more months. It would be better idea to upgrade instead of spending lot of time to make it work ;)

I just downgraded to v.1.1.7 here the polling works correctly for the file systems.
In regards to the EOL of OMV4 i will retract this request and just have to upgrade to 6 when it is finally released.
There is a lot of plugins missing in OMV5 and 6 though. (like VirtualBox, Docker, etc.)

I dont ever remember VBox to be available on OMV, maybe some custom plugin. But docker is, using omv-extras


etcd-agent: set up directory for etcd logs

use filepath instead, to make it consistent
use fileutil.TouchDirAll when possible
archive log directory when cleaning up (previously just archive etcd.log)

/cc @heyitsanthony @xiang90
After this, I will deploy 5-node etcd cluster.

lgtm

lgtm

I have been running 5-node cluster with this logrotate config:
/home/gyuho/etcd-log/etcd.log {
	size 500k
	daily
	missingok
	notifempty
	rotate 9999999
	copytruncate
}

Works well with crontab.
Thanks.


Grt: improve modeling of instance blockages

Solving Issue #2891

Please run a secure CI with this change

Please run a secure CI with this change

I ran the secure CI:

In the public designs the sky130hd/microwatt design has an metric problem
[ERROR] finish__timing__setup__ws fail test: -2.42731 >= -1.76
The ng45/ariane136 and ng45/bp designs and the private designs are running now, but the previous run had no errors.

Please look at why uW degrades

Please look at why uW degrades

The difference is in their critical path. This has two nets (soc0._331_ and _017339_) that are routed differently.
The figure shows the routing of the net soc0._331_ (In _017339_ you also get a detour around the macro)

This deviation happens in the overflow iterations in GRT, because the resources of each layer change (the GRT resources are increased in some places ).

Layer
Test branch
Master branch
Percentage

met1
1315034
1315034
0 %

met2
1318940
1318743
0.0149 %

met3
910803
910801
0.0002195 %

met4
401758
401758
0 %

met5
215642
215642
0 %

The areas where resources are released are the following: (the yellow circles have a Ggrid that increases their resource and the green circles are all the ggrids that are on the border of the macro)

Increasing the grt resources in those zones causes the routing of the two nets to change, this may be due to other nets using the resources that were not used before or the congestion below is reduced faster (because there is more resources) and choose to route that path.
The following images show the congestion map in both versions:

So we have more routing resources, but we've done a much worse job of global routing a critical net. I just noticed timing driven grt is disabled by default (-critical_nets_percentage), we should either update ORFS to use it or change the default to something more reasonable. gpl looks at the 10% most critical nets by default, so perhaps grt should too.

It suggests that this PR works correctly but just triggers the underlying concern about timing driven global routing.
@eder-matheus do you know why we never set critical_nets_percentage in ORFS?

It suggests that this PR works correctly but just triggers the underlying concern about timing driven global routing.
@eder-matheus do you know why we never set critical_nets_percentage in ORFS?

Not really, but I can fix that. I'll start a secure-ci with this grt option and will update you when it's done. I'll start it with 10%, just as in gpl as @antonblanchard mentioned.

@eder-matheus any update?

@eder-matheus any update?

I've run secure-ci using the 10% critical nets, and some public and private designs have ws failures and drv violations. I think it relates to the change of the topology of other nets that become critical after routing. I considered adding the critical_nets_percentage option after we have the slack calculated inside grt based on the intermediary net topologies generated in the grt flow.

@maliberty Luis and I performed a secure-ci run yesterday, and everything is passing now. The only two failures in the public CI are the same as we have in the master branch, so merging this is safe.


请问哪里可以获取'./iemocap_data_train.csv' 这个文件呢？

请问哪里可以获取'./iemocap_data_train.csv' 这个文件呢？

As mentioned in the readme, we no longer provide the raw data for the four datasets IEMOCAP, MELD, MOSI, and MOSEI, which you can access via this link: https://github.com/LeMei/UniMSE
If it still doesn't work, you can contact me via<EMAIL_ADDRESS>


Book menu incorrectly displayed in iPad

Describe the bug
I have noticed a bug when navigating the following url from an iPad: https://jupyterbook.org/intro.html.
The menu is displayed like grayed/transparent on top of the text, and it doesn't look good at all.
To Reproduce
You may navigate to the url above from an iPad, or just use chrome developer tools and select iPad.
I have copied below a screenshot showcasing the issue and how to reproduce it.

Hope it helps!

My pleasure @choldgraf . Also, please notice that the same behaviour happens with any book produced with the latest version of jupyterbook (I noticed it when trying to create my first online book with your technology).
Please, let me congratulate all of you for your awesome work!

I've played around with this a bit, and I'm really confused about what's going on. In the browser, if I manually resize the window, then it works fine (the widths re-work themselves at the same time and there's no sidebar overlap). However, if I click the ipad button, then the bug mentioned here shows up. Changing the width of the page at all then fixes the problem.
So, not sure what's up, but here's a GIF of what's going on in case anybody has an idea:

Hi @choldgraf , It looks that a Twitter user found a fix!
More info here: https://twitter.com/billgil/status/1267106009778139138?s=20
Copy and paste of his tweet:
In your responsive for iPad you have set the opacity to 0.6

@media
 (min-width: 768px)
.bd-sidebar {
    opacity: 0.6;
}

So remove that and you are done. My I also suggest background-color: rgba(255, 255, 255, 0.8) to get a similar effect without making the content transparent

Another way to solve this issue would be to drop the opacity changing logic entirely -- the theme is equally usable on desktop if we don't set the opacity to 0.6 IMO.

Filed #154 for if we wanna go down that route.


Ionic Push Unable to Save

This is what I see when I attempt to send a push notification. It worked fine last week when testing.

I flushed the cache of my browser, and it stopped doing the above. Apologies. Have a great year ahead!


Anyone else have an issue using --clear-chunk-max-size not working?

I am noticing that the --clear-chunk-max-size flag does not seem to be working since my /tmp/chunks directory is currently at 92G even though I have set --clear-chunk-max-size=10G with my run command. Anyone else seeing this issue?
Currently running the system under 14.04

no problems here, working nicely with --clear-chunk-max-size=10G

@seuffert could you please list the full run command that you use so I can compare it to mine? Do you also use the -t flag when you run plexdrive?

plexdrive -c /home/DIR/config --chunk-size=5M --clear-chunk-age=24h --clear-chunk-max-size=10G -o allow_other,read_only -t /tmp/plexdrive/pd -v 2 /home/DIR/mountpoint

I also have this issue. I am going to be removing all traces to start from scratch here in the next few hours.  I'll let you know how it goes.
For my own, it has not deleted anything on its own since 1.1. I'm currently on 2.1, as 3.0 gave me issues. When I remove everything, I'll start again with 3.0.

There is issues with clear-chunk-max-size. I have done quite a bit of testing on it. 1: It only deletes one chunk for each chunk it downloads. Therefore, the cache will never be smaller than it is when the app is started. 2: There is some leakage so the cache will grow bigger over time and due to #1 never get cleaned up.

No need to rebuild the database. Just a manual delete of the cache/chunk folder.

@PiscisSwimeatus Thanks, I guess I can just have a script running on cron to clear out the cache once a day to prevent my server from filling up in the meantime.

This is what I'm using (ajkis script as the base, but I'd deleting the whole chunk dirs instead):
PD:
--clear-chunk-interval=43800m0s --clear-chunk-age=43800m0s
CRON:
`#!/bin/bash
PLEXDRIVETEMP=/tmp/plexdrive/GoogleAJ/chunks
MINDISKSPACE=50000000
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
echo "MINDISKSPACE=$MINDISKSPACE"
echo "CURDISKSPACE=$CURDISKSPACE"
while [[ $CURDISKSPACE -le $MINDISKSPACE ]]
do
IFS= read -r -d $'\0' line < <(find $PLEXDRIVETEMP -mindepth 1 -maxdepth 1 -type d -printf '%T@ %p\0' 2>/dev/null | sort -z -n)
CHUNKDIR="${line#* }"
echo "Delete: $CHUNKDIR"
rm -rf $CHUNKDIR
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
echo "CURDISKSPACE=$CURDISKSPACE"
done
`

Some have modified a script ajkis wrote.  I added a bit where it will only remove files older than a portion of 1 day.  Here's my contribution:
#!/bin/bash
MTIME=.16666 #any file with a modification time greater than this portion of 1 day will be removed.  .166 is approximately 4 hours.
PLEXDRIVETEMP=/tmp/plexdrive/GoogleDrive1/chunks
MINDISKSPACE=5000000 #5GB
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
echo "MINDISKSPACE=$MINDISKSPACE"
echo "CURDISKSPACE=$CURDISKSPACE"
while [[ $CURDISKSPACE -le $MINDISKSPACE ]]
do
IFS= read -r -d $'\0' line < <(find $PLEXDRIVETEMP -mindepth 1 -maxdepth 1 -type d -printf '%T@ %p\0' 2>/dev/null | sort -z -n)
CHUNKDIR="${line#* }"
echo "Delete: $CHUNKDIR"
find $PLEXDRIVETEMP -type f -mtime $MTIME -delete
find $PLEXDRIVETEMP -type d -mtime $MTIME -empty -delete
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
echo "CURDISKSPACE=$CURDISKSPACE"
done

I removed the previous post since i wanted to polish the script a bit before posting. #1 the scripts need some controls, for example a break for infinate while loop if unable to delete folders. #2 your added find delete should be outside the while loop

Here is the script i'm using:
#!/bin/bash
if pidof -o %PPID -x "$0"; then
echo "Already running, exit"
exit 1
fi
PLEXDRIVETEMP=/tmp/plexdrive/GoogleAJ/chunks
MINDISKSPACE=50000000
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
FORCEMINAGE=120
FORCEMAXAGE=1440
HRDISKSPACE=$(echo "$CURDISKSPACE" | awk '{ sum=$1 ; hum[10242]="Gb";hum[1024]="Mb";hum[0]="Kb"; for (x=10243; x>=1024; x/=1024){ if (sum>=x) { printf "%.2f %s\n",sum/x,hum[x];break } }}')
echo "Starting with $HRDISKSPACE free diskspace"
HRCHUNKSPACE=$(du -s $PLEXDRIVETEMP | awk '{ sum=$1 ; hum[10242]="Gb";hum[1024]="Mb";hum[0]="Kb"; for (x=10243; x>=1024; x/=1024){ if (sum>=x) { printf "%.2f %s\n",sum/x,hum[x];break } }}')
echo "Starting with $HRCHUNKSPACE used chunkspace"
find $PLEXDRIVETEMP -mmin +$FORCEMAXAGE -amin +$FORCEMAXAGE -cmin +$FORCEMAXAGE -type f -delete
while [[ $CURDISKSPACE -le $MINDISKSPACE ]]
do
IFS= read -r -d $'\0' line < <(find $PLEXDRIVETEMP -mmin +$FORCEMINAGE -amin +$FORCEMINAGE -cmin +$FORCEMINAGE -type f -printf '%T@ %p\0' 2>/dev/null | sort -z -n)
CHUNK="${line#* }"
if [[ -z "$CHUNK" ]]; then
break
fi
rm $CHUNK
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
done
find $PLEXDRIVETEMP -mindepth 1 -empty -delete
CURDISKSPACE=$(df -k $PLEXDRIVETEMP | tail -1 | awk '{print $4}')
HRDISKSPACE=$(echo "$CURDISKSPACE" | awk '{ sum=$1 ; hum[10242]="Gb";hum[1024]="Mb";hum[0]="Kb"; for (x=10243; x>=1024; x/=1024){ if (sum>=x) { printf "%.2f %s\n",sum/x,hum[x];break } }}')
echo "Ended with $HRDISKSPACE free diskspace"
HRCHUNKSPACE=$(du -s $PLEXDRIVETEMP | awk '{ sum=$1 ; hum[10242]="Gb";hum[1024]="Mb";hum[0]="Kb"; for (x=10243; x>=1024; x/=1024){ if (sum>=x) { printf "%.2f %s\n",sum/x,hum[x];break } }}')
echo "Ended with $HRCHUNKSPACE used chunkspace"

There are some problems in 3.0.0. They will be tested and fixed in the next release.


Fix deployment missing If condition for tolerations

If you use ArgoCD to deploy this helm chart and leave toleration empty, the helm chart keeps getting reported as not sync

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] PR is based off the current tip of the main branch.
[x] Changes are documented under the "Unreleased" header in CHANGELOG.md
[ ] New or modified sections of values.yaml are documented in the README.md
[x] Commits follow the Kong commit message guidelines

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.António Martins seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@ammartins it looks like your local git installation isn't configured with your email. You'll need to set that to match your GitHub email, edit commits to use your new author info, and then sign the CLA.

@ammartins it looks like your local git installation isn't configured with your email. You'll need to set that to match your GitHub email, edit commits to use your new author info, and then sign the CLA.

Will do.


clojure: <IP_ADDRESS>6 -> <IP_ADDRESS>6

Description of changes

Also update the URL location, fixing issue #249396.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] 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

[x] Fits CONTRIBUTING.md.

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

3 packages built:

babashka
clojure
neil


add docs  & context.data method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 84.444%

Files with Coverage Reduction
New Missed Lines
%

dist/index.umd.js
9
84.44%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
87

Relevant Lines:
96

💛  - Coveralls


utils/findType.js should be capital "F"

Hi There!
First of all, this is a great package, I really appreciate it!
When I tried to use it, I got the following error:
`node:internal/errors:496
ErrorCaptureStackTrace(err);
^
Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/app/node_modules/fetch-appwrite-types/dist/utils/FindType.js' imported from /app/node_modules/fetch-appwrite-types/dist/main.js
at new NodeError (node:internal/errors:405:5)
at finalizeResolution (node:internal/modules/esm/resolve:327:11)
at moduleResolve (node:internal/modules/esm/resolve:946:10)
at defaultResolve (node:internal/modules/esm/resolve:1132:11)
at nextResolve (node:internal/modules/esm/loader:163:28)
at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
at ModuleWrap. (node:internal/modules/esm/module_job:77:40)
at link (node:internal/modules/esm/module_job:76:36) {
code: 'ERR_MODULE_NOT_FOUND'
}
`
The "utils/findType.js" is lowercase on the filesystem, but uppercase in the code.
Thanks!

Hello @kismonguz,
Sorry, was a build issue, that's corrected in 1.8.2.

Thanks!


Add OVO as our user

@hsluoyz  can you help with below error, which I got in above build :
remote: Invalid username or password. fatal: Authentication failed for<EMAIL_ADDRESS>

Merged.


WARN  new NativeEventEmitter() was called with a non-null argument without the required addListener method

Report
Plugin Version
6.4.0
On what Platform are you having the issue?
android
What did you do?
upgrade react-native to latest version 0.66.1
What did you expect to happen?
should not show any warnings
What happened instead?
shows this warning:
 WARN  `new NativeEventEmitter()` was called with a non-null argument without the required `addListener` method.
 WARN  `new NativeEventEmitter()` was called with a non-null argument without the required `removeListeners` method.

Please provide any other relevant information.

@vinayr can you please provide more information about the specific code that causes this warning?

There's no specific code. Just integrate the SDK in the latest version (0.66.1) RN app, and you will see these warnings when app is run.
This might help https://github.com/facebook/react-native/issues/32051

Any update on this issue?

We will check it and let you know. Thank you for informing us.

I am also seeing this issue

This warning will be fixed in the next release. thank you all

issue is fixed in the latest release 6.4.40


Enhancement: Switch message font in Google Inbox

I love Markdown Here. So much so that in Gmail, I changed my Gmail default font to Fixed Width to facilitate writing my emails in Markdown.
But I also love Google Inbox. And sadly, Google Inbox doesn't yet let you change your default font for your compose window. So I'm stuck writing my Markdown in variable-width Sans Serif font :(
It would be great to have an option on the Markdown Here settings that, if checked, will automatically select Fixed Width as my message font in Google Inbox when I compose a message or reply.
This is probably unrealistic, but it sure would be nice!

I hear you, but no. This is out of scope for MDH. It would probably best be done with a dedicated browser extension.


python3Packages.lightparam: fix source and deps

Motivation for this change
noticed it was broken reviewing #97846
source was broken:
hash mismatch in fixed-output derivation '/nix/store/y7x8gnv9b3ydwag4p33n3hz6a3ypin2p-lightparam-0.4.6-py3-none-any.whl':
  wanted: sha256:0cf6962rig1pbghpprg27b78qscycnprpcbdl2vay222a8b319pc
  got:    sha256:0hxchr7gqs09xzblw4l9rr50vcqrr3iclgj4f3gpihf4mhp0nxkr

deps were also broken
ZHF: #97479
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

https://github.com/NixOS/nixpkgs/pull/98065
4 packages built:
python37Packages.lightparam python37Packages.stytra python38Packages.lightparam python38Packages.stytra

Thank you! LGTM

macos 10.14 unhappy:
  ERROR notebook/terminal/tests/test_terminals_api.py::TerminalAPITest::test_create_terminal
  ERROR notebook/terminal/tests/test_terminals_api.py::TerminalAPITest::test_no_terminals
  ERROR notebook/terminal/tests/test_terminals_api.py::TerminalAPITest::test_terminal_handler
  ERROR notebook/terminal/tests/test_terminals_api.py::TerminalAPITest::test_terminal_root_handler
  ERROR notebook/terminal/tests/test_terminals_api.py::TerminalCullingTest::test_culling
  ERROR notebook/tests/test_files.py::FilesTest::test_view_html - AttributeErro...
  ERROR notebook/tests/test_gateway.py::TestGateway::test_gateway_session_lifecycle
  ERROR notebook/tests/test_notebookapp.py::NotebookAppTests::test_list_running_servers
  ERROR notebook/tree/tests/test_tree_handler.py::TreeTest::test_redirect - Att...
  = 132 failed, 130 passed, 8 skipped, 18 deselected, 1 xfailed, 8 warnings, 49 errors in 38.33s =

nixpkgs-review is being annoying and not giving me full logs, let me try and get them...


Fixing Bitbucket cloud optional user properties issue

Issue
Made BitbucketClout.User accountId and nickname optional

Warnings

:warning:

Tests/DangerTests/BitBucketCloudTests.swift#L15 - Collection literals should not have trailing commas. (trailing_comma)

:warning:

Tests/DangerTests/BitBucketCloudTests.swift#L39 - Collection literals should not have trailing commas. (trailing_comma)

:warning:

Tests/DangerTests/BitBucketCloudTests.swift#L59 - Collection literals should not have trailing commas. (trailing_comma)

  Generated by :no_entry_sign: Danger Swift against a6d996ab2f2c433dd6e4581299ab98cee26cfd50


do not create empty folders

force creating by an option

ref https://github.com/keboola/internal/issues/21


Rules Reference does not load

When trying to open the Rules Reference section of Thronesdb, there is an error message:
The thronesdb.com page isn’t working
thronesdb.com is currently unable to handle this request.
500

Came here to post this!


Fix tests/rest/type_generic.js

The rest property has to be the last one otherwise this is invalid syntax.

Wow! Nice find! This is a totally a bug in the Flow parser, that it fails to complain about this! I'm not going to take this PR, but I'm going to fix the parser (and this test). Thanks again for bringing this to our attention!

Looks like this was fixed in https://github.com/facebook/flow/commit/d985b109f6de5a75e6ba96c538ab3847c2b038a1 but this PR didn't get closed. Thanks again for finding this issue!


Azure Oauth 2.0 backend fails to authorize user, email claim not found despite being present

Describe the issue
Oauth2 backend Azure fails to log in the user, gives "error":"failed validating OAuth 2.0 access token: OAuth 2.0 id_token email claim not found" in the Caddy log -- but the scopes are openid email profile, and the Azure authorize request itself is successful.
Azure is configured with access to the relevant scopes, and if it hadn't, I would expect the authorize call to fail. Instead, this is what happens:

I open https://login.example.com and I am presented with the auth portal login screen
I click the Azure button in the auth portal
I am sent to https://login.example.com/oauth2/azure
I am redirected to the authorize endpoint (see full URL below)
I am redirected to https://login.example.com/oauth2/azure/authorization-code-callback with code, state and session_state params.
The authorization-code-callback endpoint returns 401 Unauthorized

Here is the full authorize URL (newlines added by me):
https://login.microsoftonline.com/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/oauth2/v2.0/authorize
  ?client_id=xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
  &nonce=XXXxxxXXXxxxXXXxxxXXXxxx
  &redirect_uri=https%3A%2F%2Flogin.example.com%2Foauth2%2Fazure%2Fauthorization-code-callback
  &response_type=code
  &scope=openid+email+profile
  &state=xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx

Configuration
Paste full Caddyfile below:
{
	log DEBUG
	order authenticate before respond
	order authorize before basicauth
	security {
		authentication portal dev {
			crypto default token name AUTHP_TOKEN
			crypto default token lifetime 864000
			crypto key sign-verify ***********
			cookie domain example.com
			cookie lifetime 2592000 # 30 days
			backends {
				azure_oauth2_backend {
				          method oauth2
				          realm azure
				          provider azure
				          client_id *****
					  tenant_id *****
					  client_secret *******
				          scopes openid email profile
			        }
			}
			ui {
				links {
					"Who am I" /auth/whoami icon "las la-question-circle"
				}
			}
			transform user {
				match realm azure
				action add role authp/user
			}
		}
		authorization policy devpolicy {
			set auth url https://login.example.com/login
			crypto key verify {env.JWT_SHARED_KEY}
			allow roles authp/admin authp/user
			validate bearer header
			inject headers with claims
		}
	}
}

login.example.com {
	authenticate with dev
}

www.example.com {
	authenticate with dev
	authorize with devpolicy
	root * /home/example/wwwroot
	file_server
}

Version Information
Caddy version 2.4.6. These are the non-standard modules:
dns.providers.digitalocean v0.0.0-20210809220558-ac6e4fd9e135
http.authentication.providers.authorizer v1.0.16
http.handlers.authenticator v1.0.16
security v1.0.16

Expected behavior
The user successfully authenticates via Azure AD, and is redirected back to https://login.example.com showing the UI menu with the Who am I link.

@nmenglund , thank you for opening the issue! 👍 I am here to help!
Please remove authenticate with dev from www.example.com:
www.example.com {
	# authenticate with dev
	authorize with devpolicy
	root * /home/example/wwwroot
	file_server
}

Additionally, in your preamble, enable debug:
{
	log DEBUG
        debug
	order authenticate before respond
	order authorize before basicauth

Then, please use trace plugin (https://github.com/greenpau/caddy-trace) on your login.example.com endpoint.
It will help seeing what the webserver receives.
{
	order trace before authenticate
}

login.example.com {
        trace tag="tshoot"
	authenticate with dev
}

Once you do all of the above, you will have more logs related to OAuth exchange.
In particular, below error indicates that it is likely you received ID token, but did not get email claim in it.
"error":"failed validating OAuth 2.0 access token: OAuth 2.0 id_token email claim not found"

Please email me the logs privately to<EMAIL_ADDRESS>
Separate from the above. Please take screenshots from your Azure configuration, hide tenant identifier, and post here to see what is being configured.

the email field was blank -- updating this made the problem go away.

😃 Glad it worked out! 👍

my apologies for the rookie question during the weekend.

No need to apologize! You did well by asking the question 👍

FYI, added disable email claim check configuration option in https://github.com/greenpau/caddy-security/releases/tag/v1.1.2


Adds Space Pirate Ship for Admin Events

About the PR
Adds a medium sized space pirate ship that can hold 9 crew, has onboard atmos, power, and medical services; don't forget the booze! Ready to go and plunder the Spacey Seas for all it's gold and grog!
Screenshots
https://gyazo.com/7d4b0243c37cd3873eec2af0dcffde3d
Changelog

:cl:

add: Added a Space Pirate Ship map for admin events.

I  like this ship more than #7552, because I feel it's more piratey, albeit simpler by comparison. Also to embed your screenshot you might want to add .png to the end of the link there, might embed properly.

not sure how i feel between this and #7552, tbh

Yeah emisse's is better ima take this down


nixos services.xserver.displayManager.session: drop type

For now at least.  I expect someone will find a working type later.
It's incorrect and was causing bad issues.  Example test case:
nix-instantiate nixos/release.nix -A tests.xfce.x86_64-linux --dry-run
This is a partial revert of commit b2d803c from PR #162271.

OMG now... a type-less option is not allowed now?

I was impatient with this.

OMG no... a type-less option is not allowed now?

Glad it's working, that's the goal of https://github.com/NixOS/nixpkgs/pull/162271 :)
This reversal looks good for now. While types.attrs isn't very good and will be deprecated in the future, we can defer the decision for the correct type until then.


6987: Memory leak with WMS time source with reprojection

As discussed, the collection of interimTile for the reproj.Tile was never being cleaned up and thus the references would still hang around. The fix is to perform:
tileImage.js:
newTile.refreshInterimChain();
after the line:
newTile.interimTile = tile
The refreshInterimChain is only done inside the creation of a reproj.Tile (called within getTileInternal) for an ol.ImageTile. It wasn't being done on the created reproj.Tile.

Issue #6987

Thanks @ch08532 for the investigation and fix! Can you please submit a CLA so we can merge this?

@ahocevar I've just submitted the CLA.


har_extractor errors

Hi,
Firstly, I like to say your work on mitmproxy is awesome!  I have a question with the har_extractor for it though.
I'm running a java test application does that has to various things to get specific events to be triggered on multiple platforms (web, native apps, etc)
I then need to capture these network calls and verify the contents. As these calls are https, I'm trying to use mitmdump to dump logs in har format. So my java app can start mitmdump and then read the logs to verify click and tracking events sent in the background.
I was very excited when I saw the har extractor but when I use it, i keep getting errors.
1.) am i using it correctly.
I'm currently doing it this way.
mitmdump -w out/out.dmp -s "out/har_extractor.py --out/har.har"
Then when I load the page i get a variety of errors. and nothing written to my requested output har.har file.
Can you please help me out?
Best,
Nelson
Here are the two errors I"m getting
<IP_ADDRESS> GET https://pagead2.googlesyndication.com/pagead/gen_204?id=ah&v=349&avi=BTJ8BlT74VZurCYuYkgKdzZSYBgAAAAAQATgByAEC4AQCiAWjzJoEoAY-&adk=0&pmd=903&pd=903&fd=1
 << 204 No Content 0B
Script error:
Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/libmproxy/script.py", line 123, in run
    return (True, f(self.ctx, *args, **kwargs))
  File "out/har_extractor.py", line 213, in response
    elif context.HARLog.get_page_ref(flow.request.headers.get('Referer')) is not None:
  File "out/har_extractor.py", line 52, in get_page_ref
    return self.__page_ref__.get(page, None)
TypeError: unhashable type: 'list'

and
<IP_ADDRESS> GET https://pagead2.googlesyndication.com/pagead/gen_204?id=sodar&v=5&t=2&bgai=B_TMolT74VZurCYuYkgKdzZSYBgAAAAA4AeAEAg&bg=!fH9EMCiIOIGddMgCAAAAHVIAAAABKgDrLq8aJ3HDu35G_dL4p9Rd_0UjdyteH5e--Qm2lvQzYas8iMaXH_64LJ-ldQUWyAtqb7L87kMJ7F8OwNDGuxfJ5LoHniT4o1d4YhCeubKVyQghXgsPZkx3ITCtIqNHc28623JmVTuII6AEjs7nb5nn5msqQPVpF_b4x5x3dFf6wzD8U7bWylv6ukc9mEpT3T5l7t3VtPjJK8mKsfjufqEUln5XTVBIjh7RCcecuGUGTNzFMzTOB1299zxo4XRbhKiqMIyP8XTAPt8h72CAV4rRtEkBj29Ju045wrvc6vvQ_I9ZBHgDPFrJa6vSjQ
 << 204 No Content 0B
Script error:
Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/libmproxy/script.py", line 123, in run
    return (True, f(self.ctx, *args, **kwargs))
  File "out/har_extractor.py", line 143, in response
    for k, v in (flow.request.get_cookies() or {}).iteritems()]
AttributeError: 'ODict' object has no attribute 'iteritems'

The first issue you're seeing is caused by the fact that you're using the example from master while you have 0.13 installed.
The second error is probably an issue in the script itself, you should be good by just replacing .iteritems() with .items(). The har extractor has been contributed by @JustusW - maybe he can chip in?
Cheers,
Max

@mhils this error looks as if request.get_cookies suddenly returns an ODict that does not support the full MutableMapping interface anymore?

To the best of my knowledge, ODict never supported the full MutableMapping interface (and also has some slightly different behaviour, i.e. del does not throw an error if the key doesn't exist).

Well, it needs key, value pairs, so items should be fine, it is a bit irritating since either this hasn't worked for quite some time (which would be strange seeing I use this in a production environment on a more or less hourly basis) or there has been a very recent change to either the ODict returned or the return value changed into an odict.

Please mind that this would have to be changed in at least two places where cookies are extracted.


My Game is lauching for a minute and crashes. It's been doing this ever time i open it as well as the game not opening as well.

Please help me with this.

I have a pirated version of the game.
I had the same issue and I had to look around for a while. I saw suggestions to add a line to the bat file for the executable destination but that didn't work. I tried putting the mod engine files in the game directory but that didn't work either.
At the end I did find a solution that worked for me.
My game files were in a folder named "ELDEN RING" and my directory was "D:\Games\ELDEN RING".
Make another folder inside the game directory called "game" and move all the files in the "ELDEN RING" folder into the newly created folder. Your new directory should be "D:\Games\ELDEN RING\game".
Once you've done that rename the modengine2_launcher to "start_protected_game" (rename the original file with the same name to something else).
You should be able to launch the game with this renamed executable now.
If you have the same issue when launching the game the second time, delete the log text file that should have been created "modengine_"
Hope this helps.


How to make the documentation more user-friendly

README.md is addressed to coq developers and other sophisticated insiders.  This was appropriate when this Platform was under initial development, but now we should hope that most readers of the documentation will be ordinary users.  Therefore, I would suggest:

Most users should not clone the repo, but just download a release.

Split README.md into two files, the other being USAGE.md, which explains how to use the Coq Platform.  All the material starting with "Usage of the ..." should be in USAGE.md.  In the README, right before Note on Licenses, there should be:

Suggestion to choose the latest release from github.com/coq/platform/releases
Prominent link to USAGE.md.

The release notes (the "body" text of the release) should have a prominent link to USAGE.md

Proposed edits of USAGE.md are attached
USAGE.txt
(My apologies for doing it this way, instead of more professionally as a pull request.)

It is "brittle" to have the switch-name repeated so many times in USAGE.md.  It would be better to generate USAGE.md from a script that pastes in the result of calculating the name via coq_platform_switch_name.sh.

Suggested addition to the documentation: how to maintain the installation.
In particular, is it sufficient to run opam upgrade once coq-platform is installed successfully.

Sorry for the long delay! The README changed a lot since then and most of the comments are already implemented - notably the file is split into several pieces in the main branch since a while. I took the zip vs git comment into account in PR #160.
The comment from @dnaumann lead to a new expandable section in the main readme file, also in PR #160.


Minor cleanups

Hi,
Thanks for building this nice looking library!
While I was reading the source code, I found a few things could be made "better".
Let me know if there is anything you want to keep as is, e.g. explicit call to generate very http verb
Thanks again
Po

The first version had a lot of macros magic when generating functions so I wanted to keep it as simple as possible, but in this case it is obvious what is being generated.
Thanks!


SIMSBIOHUB-623: Add Site Name and Description When Creating New Sampling Site

Links to Jira Tickets

https://apps.nrs.gov.bc.ca/int/jira/browse/SIMSBIOHUB-623

Description of Changes

Allows a name and description to be added to a sampling site during creation

Testing Notes

Can only name/describe a sampling site during creation when exactly one site is being added

Looks good!
Only minor thing I noticed is the difference of the sampling site description input and the technique description.
Sampling Site description:

Technique description:


perf: use rayon for io-uring loop

Test for seeing if using work stealing is useful in the io-uring loop.

/gcbrun


Serverless testing

This patch syncs the latest JSON files from multiple specifications.  It picks up both:

removal of oppressive language changes
serverless changes

It also adds the actual testing against serverless, which required skipping various tests that are not supported by serverless for one reason or another.
JAVA-4217
JAVA-3987

The serverless test task has failing tests.  They succeeded in the last patch I executed.  Investigating...

Closing, as I can't get the serverless tests to pass consistently. I'm going to open a new PR with just the first two commits from this one.


hi

i test you code and it works but i need feature if you can do it
counting cars and sort them as sedan , trucks ,etc , also if traffic happened or not
if you interested contact e

Hi, that would involve training the model on different image classes of sedans, trucks, etc. Maybe these could be a good starting point: https://github.com/topics/vehicle-detection


[$1000] NEW FEATURE: Don't truncate long room names in room details page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action performed:

Click Global create > new room, and enter a long room name (more than 50 characters).
Notice that the full room name is listed in the welcome message, and truncated in the room header.
Click into room details and notice that the room name in room details is also truncated.

Expected result:
We'd show the full room name in room details, as this should be the source of truth for what the room actually is. If the room name is long, we should just break it into multiple lines in the room details.
Actual result:
We truncate the room name in room details. Meaning you cannot see the full room name there.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @joekaufmanexpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689026712373409
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f72d5afba7910b13
Upwork Job ID:<PHONE_NUMBER>340113920
Last Price Increase: 2023-07-11

@shawnborton let me know if you'd like this as External!

Proposal
Please re-state the problem that we are trying to solve in this issue.
Don't truncate the room name in the details page which help user to see full room name
What is the root cause of that problem?
Currently, we are passing numberOfLines to 1 for showing the title which truncates the text and only shows a single line.
https://github.com/Expensify/App/blob/38890e2e3c32b52dad11a7dfc09494924519946b/src/pages/ReportDetailsPage.js#L153
What changes do you think we should make in order to solve the problem?

We only need to set the numberOfLines to 1 when it is not a chat room. Setting to 0 will result in unsetting this value, which means that no lines restriction will be applied.
numberOfLines={isChatRoom ? 0 : 1}
https://github.com/Expensify/App/blob/38890e2e3c32b52dad11a7dfc09494924519946b/src/pages/ReportDetailsPage.js#L153

To allow it to break in multiple lines we also need to take out the style styles.pre from here.
https://github.com/Expensify/App/blob/38890e2e3c32b52dad11a7dfc09494924519946b/src/pages/ReportDetailsPage.js#L154
isChatRoom ? undefined : styles.pre

Note: It is applied to all platforms, if requires only for small devices then we need to add withWindowDimensions and with isChatRoom condition in above 2 points we also need to add && props.isSmallScreenWidth.

Result

Yup we can do this external, thanks!

Thanks for the proposals folks. For consistency, I think we should focus on solving for all the platforms and not just native apps.
Both the proposals don't fully solve for all the platforms. @dhairyasenjaliya @Pujan92 Can you update your proposals after testing once?  Looking forward to it. Also I would recommend sending out a fresh comment, so that we don't end up in a conflict of which proposal came first. Thank you for your patience on this.

@mananjadhav my proposal is actually a generic one that behaves the same for all platforms.

@Pujan92 still truncated on web using your proposal.

--- a/src/pages/ReportDetailsPage.js
+++ b/src/pages/ReportDetailsPage.js
@@ -150,8 +150,8 @@ function ReportDetailsPage(props) {
                                     fullTitle={ReportUtils.getReportName(props.report)}
                                     displayNamesWithTooltips={displayNamesWithTooltips}
                                     tooltipEnabled
-                                    numberOfLines={1}
-                                    textStyles={[styles.textHeadline, styles.textAlignCenter, styles.pre]}
+                                    numberOfLines={isChatRoom ? 0 : 1}
+                                    textStyles={[styles.textHeadline, styles.textAlignCenter, isChatRoom ? undefined : styles.pre]}
                                     shouldUseFullTitle={isChatRoom || isPolicyExpenseChat || isThread}
                                 />
                             </View>

But the display name component is component for room as well as chats, won't that cause an issue?

Proposal
Please re-state the problem that we are trying to solve in this issue.
Don't truncate the room name in the details page which help user to see full room name
What is the root cause of that problem?
Currently, we are passing numberOfLines to 1 for showing the title which truncates the text and only shows a single line.
https://github.com/Expensify/App/blob/38890e2e3c32b52dad11a7dfc09494924519946b/src/pages/ReportDetailsPage.js#L153
What changes do you think we should make in order to solve the problem?

We only need to set the numberOfLines to 1 when it is not a chat room. Setting to 0 will result in unsetting this value, which means that no lines restriction will be applied.
numberOfLines={isChatRoom ? 0 : 1}
https://github.com/Expensify/App/blob/38890e2e3c32b52dad11a7dfc09494924519946b/src/pages/ReportDetailsPage.js#L153

To allow it to break in multiple lines we also need to take out the style styles.pre from here.
https://github.com/Expensify/App/blob/38890e2e3c32b52dad11a7dfc09494924519946b/src/pages/ReportDetailsPage.js#L154
isChatRoom ? undefined : styles.pre

Update numberOfLines=1 with numberOfLines=this.props.numberOfLines in the DisplayNames to take the dynamic value from parent.
https://github.com/Expensify/App/blob/f2d38d4c42b5e572ad460f53850fb47b67ffc9ef/src/components/DisplayNames/index.js#L85

Result

But the display name component is component for room as well as chats, won't that cause an issue?

Only for Room we are taking out the whiteSpace: pre style so I think it won't create any issue.

Thanks for the comments folks. Yeah I saw that on @Pujan92's proposal but I missed reading that in yours @dhairyasenjaliya.
Considering both the proposals, I think @Pujan92's proposal is better to unset the numberOfLines, instead of hard-coding it.
I am a bit concerned about the comment, // Tokenization of string only support 1 numberOfLines on Web, so we'll have to test that we don't break something and trace it back to the original PR when this comment was added.
🎀 👀 🎀  C+ Reviewed.

@dhairyasenjaliya Your proposal highlights, We need to define max number, where as the other proposal mentions unsetting the value. Obviously for smaller PRs the properties, etc. could be same and I wouldn't call it same proposals. I hope this clarifies.

Ah, Chirag is on leave now - I shall reassign another Engineer!

Hi @Julesssss - just wondering if you could take over for Chirag. We have a proposal here - cheers!

@Julesssss quick bump on this one.

Oop,s missed that last comment. Will review tomorrow

Adding Chirag back so he gets the credit when this is closed.

@mananjadhav shall we hold this issue until this PR gets merged?
Reason - for android resetting the numberOfLines={0} gives an error and the above PR seems to have a patch for numberOfLines. I tested that branch code and seems it is working in Android with it.

I agree let's put this on hold.

@Pujan92 I saw the PR,it allows number | undefined. how about we set this to undefined? instead of 0. Does this still break it?

@mananjadhav I haven't tried undefined yet but as per the react doc it's preferred to pass default value 0.
https://reactnative.dev/docs/text#numberoflines

We just tried in this issue, and it seems to work. https://github.com/Expensify/App/pull/23246#issuecomment-1645573411
Any thoughts @Julesssss ?

My only concern with holding is that the RN bumps sometimes take quite a while.

yeah and we used this.props.numberOfLines || undefined, and it worked fine.

Lets just do this then?
I don't think it's a huge deal either way

Agreed. @Pujan92 ^ ?

Yes, will raise a PR soon!

@mananjadhav seems undefined also giving me an error.

@Pujan92 Please check this PR for reference.

I'm just wondering passing 0 as numberOfLines does make any sense? isn't it weird to understand we're saying 0 lines and it will show all the lines of text? why not just pass undefined or null this makes somewhat sense in my view (came here after seeing PR)?

This is more like a feature request. Hence we don't have any tagged PR. Because we shouldn't end up working on adding the limit again, I think we should add a regression test.
Regression Test:

Open FAB, and click on New Room
Create a room with a very long name (> 100 characters)
Open the room chat
Click on room name in the header to open report details
Verify that room name won't truncate and shows entire room name text

@Julesssss @jliexpensify wdyt?
Also this is eligible for the timeline bonus, because the change was done well within the timeline, but we got blocked by another PR conflict.

Will let @Julesssss weigh in with the final say on timeline bonus - I'll summarise payments once he weighs in!

Regular payment here @jliexpensify

Ok @Julesssss is this correct?
Payments:

Contributor: @Pujan92 $1000
C+: @mananjadhav $1000

Upworks job here

@jliexpensify  It should be $1500.

@jliexpensify It should be $1500, but let's wait for @Julesssss to ack.

Why is that?

@Julesssss I raised this comment here too during the PR. We had the PR ready but had conflicts. The conflict was primarily a regression, which got this issue delayed. I saw your 👍 in the comment as well as  I highlighted the same here too.

Ah yeah, I remember now. 👍

Thanks. Can you comment and confirm it to @jliexpensify so that he can put up a summary on the issue?

@jliexpensify I hereby declare this is worthy of the bonus 😄

Edited

Raised my request on the NewDot.

Pujan paiud, job closed. Working on Regression GH now.

Reviewed details for @mananjadhav. These details are accurate based on summary from Business Reviewer and are now approved for payment in NewDot.


Make internals private?

https://github.com/awslabs/aws-java-nio-spi-for-s3/blob/20c8e9c0647890a0cdaf6a362fb357aacb2b9e58/src/main/java/software/amazon/nio/spi/s3/S3FileSystemProvider.java#L326
Given that when we have a protected member we want to pay extra attention to the stability of the interface, which makes the codebase more difficult to refactor, would it make sense to make this private?

I think it is protected because of a limitation of unit testing that cannot test private methods. It could probably be hidden with a refactoring of the unit test?

Using reflection would be an alternative way to test a private method. If it must be tested.

Yep, despite I agree that testability justifies not optimal design choices, either we are more flexible in compatibility breaking changes or more careful in using it.
Either ways work for me. would you take care of it?

merged into main with PR #133


feat(evm): add query for burner info

Description
Adds a query to fetch burner information for a given address so that the microservices can validate whether a transfer event is a deposit before submitting a ConfirmDeposit request.
Todos

[ ] Unit tests
[ ] Manual tests
[ ] Documentation
[ ] Connect epics/issues
[ ] Tag type of change

Steps to Test
Expected Behaviour
Other Notes

you can close this PR and instead review the one I opened yesterday: https://github.com/axelarnetwork/axelar-core/pull/1241

Duplicate #1241


ImportError: cannot import name 'dataset_pb2' from 'waymo_open_dataset'

Hi,
I ran into this error when I checked out the repo and followed the tutorial.
ImportError: cannot import name 'dataset_pb2' from 'waymo_open_dataset' ([PATH]/waymo_open_dataset/init.py)

I have encountered this before. It might be an installation error. You may try the following steps:

Install the right versions of tensorflow.
Install waymo_open_dataset from source, instead of pip.

I have went to same issue. I made this file manually to get around.
protoc ./waymo_open_dataset/dataset.proto --python_out=.

I have encountered this before. It might be an installation error. You may try the following steps:
1. Install the right versions of tensorflow.

2. Install waymo_open_dataset from source, instead of pip.

Can you please let me know what are the compatible version of TensorFlow and python?
For example available options of TensorFlow for python 3.6, 3.7,3.8.
Thanks in advance for cooperation


[WIP] Watermark reporting - backend support

closing in favor of https://github.com/ManageIQ/manageiq/pulls/5006


Plans to release 2.4.0 as a stable release?

Hello, I was just wondering if a date when 2.4.0 will be released as a stable version is currently available?

As in every other release, we try to have at least 2 RCs before the final release. Each RC is triggered at least a week from the previous one.

RC1 was just released. Depending on amount of cherrypicks and issues discovered, there might be a 2.4 final release around Thanksgiving.

Due to some delays, RC2 will be released this week (likely tomorrow) and final release will come after Thanksgiving.

RC3 was released yesterday. Final release will happen next week if testing does not reveal more issues.

Final release should come this week or the next.

TF 2.4 has been released (forgot to notify the thread at the time). Closing this


Changes in the onboarding tours

We've seen that people don't normally check the "Do not show me this again" thing, and then the onboarding messages keep annoying them forever when they change between tabs inside a layer. To avoid that, lets:

[x] Only show those messages once and never again
[x] Remove the "Do not show me this again" check from all the onboarding messages (since we're only going to show them once, we don't need that check anymore).

in production! 🚀