Orientation problem

When I load a PDF for example this
https://schiaffonati.faculty.polimi.it/CE/normative ethics 4.pdf
I have a problem with orientation because they need to be horizontal.
With iOS and webview it works fine as you can see.

But with pdf_viewer it's vertical.

Thank you 👍🏻

me too

Did anyone have a solution for this bug? It seems like it only happens on ios, android is just fine.


Miscellaneous - Submissions, meeting report etc

https://docs.google.com/document/d/1uIExM0Il3zoah6V1oFY1NkbuuTlibsab/edit

This was done and we have submitted the report 🎉


HCS12 / HC12 CALL Instruction issue

Describe the bug
For Motorola Processors in the HC12 / HCS12 family, CALL instructions are not correctly decoded in the Listing window. For processor HC12, opcode 4a is not registered as a CALL instruction to begin with in newest development build as of 7/25/22. For HCS12, PPAGE immediate values are not included in the addressing. Example:
      008004 4a  80  59  00    CALL       0x8059 , 0x0 =>LAB_000057+2

This instruction should point to a function at memory location 0x8059.
Example 2:
      01a4c5 4a  a5  38  01    CALL       0xa538 , 0x1 =>LAB_00a4c9

This instruction should point to a function at 0x1a538
To Reproduce
Steps to reproduce the behavior:

Load given binary
Apply either HCS12 or HC12 processor languages
Go to memory location 0x8004
See error

Expected behavior
CALL Instructions should have the IMM PPAGE value loaded into the function address.
Screenshots

Example File
Example issue.zip
Environment (please complete t
he following information):

OS: Windows 10
Java Version:  "18.0.2" 2022-07-19
Ghidra Version: 10.1.4 , 10.2 DEV_BUILD
Ghidra Origin: Github, BUILT

Agree the mapping is suspect in your example.  We have no raw firmware examples, other than sample project .elf and hex format for this and the manual is rather vague as to where the pages are actually laid out in physical memory.
Was hoping someone would come up with a real world example.  Is your example produced from GCC or from CodeWarrior.  We based the mapping on CodeWarrior.
Previously the HC12 processor only supported the HCS-12X variant.  They needed to be separated into the HC-12, HCS-12, and HCS-12X variants because the paging is different on each one.
There is actually a very complicated page mapping going on in the HCS-12X with many paged areas and somewhat in the HCS-12 with PPAGE.  They just aren't mapping to the values you expect, so we need to take a look at it and understand why.
This document goes into alot more detail on the HCS-12 and HCS-12X paging schemes.
https://www.nxp.com/docs/en/application-note/AN3784.pdf
In particular, from the document:
• In the HCS12 Family, pages are chosen in sequential order in such a way that the last page of
memory is given the number 0x3F.
 For example: If your S12 device has 32 kB of Flash, the Flash will be conceptually divided into
two 16kB pages with numbers 0x3E and 0x3F. If it has 48 kB of Flash, 3 Flash pages will be
defined, with numbers 0x3D, 0x3E and 0x3F,

 In the HCS12X (typo?) Family, pages are chosen in sequential order in such a way that the last page of
memory is given the number 0xFF.
 For example: If your S12 device has 8 kB of RAM, the RAM will be conceptually divided into
two 4kB pages with numbers 0xFE and 0xFF. If it has 16 kB of RAM, 4 RAM pages will be
defined, with numbers 0xFC,0xFD,0xFE and 0xFF

I don't believe the CALL with the page instruction is supported on the HC-12.  Nor is bkgnd.  We could support this call variant even in the HC-12 and just remove the paging, or have it act as a simple flat memory.

The target MCU that the example flash was pulled off of is defined by this datasheet:
https://www.nxp.com/docs/en/data-sheet/MC68HC912DG128.pdf with an IC Mask of 0K50E
I have no idea how it was programmed, only the flash that was pulled off of it by an XPROG unit. It is an HC12 chip, not HCS12 or HCS12X
The paged memory in the flash pulled from the IC is located in offsets of 0x018000, 0x028000, etc, corresponding to memory pages 1, 2 etc. The following page from the above listed datasheet describes how I believe this flash is laid out.

Supporting the CALL instruction with paging would be fantastic for HC12 variants. I'm not entirely sure the best way to then load the Memory Map so that the CALL instruction with paging will correctly point to the right memory section, as currently in the newest development build only only files up to 0xFFFF can be loaded, whereas the target MCUs flash extended to 0x7FFFF.
I was loading the flash into various segments, however that was not effective.


Remove String::erase method declaration

It's just an old leftover, as discussed in the rocket chat.

Let's wait for the CI backlog to be somewhat reduced before merging as this will require a good hour to rebuild all platforms and update the master cache for future PRs/merges.


Reduce amount of memcpy()

Node's Buffer interface supports a way to declare an existing block of memory as a buffer, but it requires a free callback. This may be tricky to implement.

now tracking at #319


Fix stops the background music (updated)

Fix this issue https://github.com/capacitor-community/native-audio/issues/40
This fix has been published on NPM until it's merged:
npm i @forgr/native-audio
By default, audio is not mixed on android and iOS, if you need to mix it just do :
NativeAudio.configure({focus: false});

iOS explainations :
For playback:
By default, using this category implies that your app’s audio is nonmixable—activating your session will interrupt any other audio sessions which are also nonmixable.
https://developer.apple.com/documentation/avfaudio/avaudiosession/category/1616509-playback
For ambient:
When you use this category, audio from other apps mixes with your audio.
https://developer.apple.com/documentation/avfaudio/avaudiosession/category/1616560-ambient
PS:
this is a rework of the PR https://github.com/capacitor-community/native-audio/pull/42 but where the old behaviour is maintained.

@bazuka5801 if you can have a look ?

Would be great to get this merged in if all looks OK? I have an issue at the moment (in that the app steals focus when I boot it) and it appears this PR would fix that.

@bazuka5801 my fork is used in production since 2 month without issue i think it's safe to merge :)

Hi, could you please merge it into official master ?

@riderx Thanks for contributing :100:
Sorry for the long reply, I needed to make sure everything was working properly.

Thanks, i totally understand !


Highlight potential misuse of PlayerArgument and OfflinePlayerArgument

Documentation URL
https://commandapi.jorel.dev/9.5.0/argument_entities.html
What the documentation currently says
Player argument
The PlayerArgument class is very similar (almost identical) to EntitySelectorArgument.OnePlayer. It returns a Player object and requires the player to be online.
Developer's Note:
The PlayerArgument internally uses the GameProfile class from Mojang's authlib, which means that this argument has a slight performance overhead compared to using EntitySelectorArgument.OnePlayer
OfflinePlayer argument
The OfflinePlayerArgument class is identical to the PlayerArgument class, but instead of returning a Player object, it returns an OfflinePlayer object. Internally, this argument makes calls to Mojang servers (via Mojang's authlib), meaning it can be slightly slower than alternative methods (such as using a StringArgument and suggesting a list of existing offline players).
The OfflinePlayerArgument should be able to retrieve players that have never joined the server before.
Proposed fix
Relating to this discussion on Discord
Although the performance impact is small individually, it can be exploited by malicious players to stall a server by spamming it repeatedly. This vulnerability should be made known so that appropriate permissions can be set. Suggest using red block text instead of grey.

Would also be nice to have this brought to developer's attention in the upcoming release notes as well as documentation changelog.


nixos/xserver: clean up the useXFS option

Motivation for this change
The useXFS option has confusing semantics, being somewhere between a boolean option and
a FontPath specification. Introduce fontPath to replace it and mark the old option as removed.
I hope using mkRemovedOptionModule is the right thing to do here. I am not that deep into the module system.

@cole-h In your opinion is this fine now? Is there someone else I should ask for an additional review / opinion?

Diff LGTM. However, I can't test this (I don't use X).

I did some basic testing now. It throws the expected error message when you eval with useXFS set, and the new fontPath option does work (with a local path, but it should work at least as much as it used to).

By the way, for cross reference: This addresses one more item of https://github.com/NixOS/nixpkgs/issues/76184

/marvin opt-in

/marvin needs_merger

I think this is good to go..

/status needs_merger

already has an external review, but all right..

/status needs_reviewer

Thank you! Sorry it took so long.


Add gh cert in optional dev container env args

Adds a fix for the devcontainer make commands treating the pem cert as multiple env vars
Also makes app cert optional in line with potential customers only deploying transform / serve without apps

/test-force-approve


Lorem is a true true fan and a friend and is a friend

I miss the way we were you we miss me  hellofdsfsd ss

Lorem ipsum test label

MIKI

test


Add bar chart bottom tooltip for fromY value

Related issue #1389

Please read the contributing guideline

This issue is closed due to the inactivity


Change the reference of autoware_auto_msgs

https://github.com/autowarefoundation/autoware_common/issues/4#issuecomment-987946881

[ ] Move Tier IV's fork to AWF org?
[ ] Consider renaming repository name as autoware_msgs

Because the package names might change in the future.

Currently, the reference is tier4 org.
https://github.com/autowarefoundation/autoware.universe/blob/c2e79c3a56956f867030b2cbc9675e2e7c7c1339/build_depends.repos#L10-L13
https://github.com/tier4/autoware_auto_msgs

Move to https://github.com/autowarefoundation/autoware_msgs/issues/2.


Proper login error handling for deactivated users

I want to merge this change because error message and code should be proper for users that are not yet activated by email and deactivated.

Port from #8250
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4583 INVALID_CREDENTIALS instead of ACCOUNT_NOT_CONFIRMED error on login


MRG: Fix hash

Closes #4796.
I can't reproduce this locally. Maybe it is NumPy version dependent. We'll see if the CIs agree there is no bug :)

Apologies for the slow reply, yes the original issue is resolved.
Many thanks.

Tom

mne.sys_info()
Platform:
Linux-3.10.0-514.6.2.el7.x86_64-x86_64-with-centos-7.3.1611-Core
Python:        3.6.0 |Anaconda custom (64-bit)| (default, Dec 23 2016,
12:22:00)  [GCC 4.4.7 20120313 (Red Hat 4.4.7-1)]
Executable:    /usr/local/anaconda3/bin/python
CPU:           x86_64: 48 cores
Memory:        440.6 GB

mne:           0.16.dev0
numpy:         1.14.0 {blas=openblas, lapack=openblas}
scipy:         1.0.0
matplotlib:    2.0.0 {backend=Qt5Agg}
sklearn:       0.18.1
nibabel:       Not found
mayavi:        Not found
pycuda:        2017.1.1
skcuda:        0.5.1
pandas:        0.20.1

mne.utils.object_hash(np.int64(0))
220659586212589107752456316032363055450

On Thu, Apr 5, 2018 at 12:11 AM, Alexandre Gramfort <
<EMAIL_ADDRESS>wrote:

I confirm it fixes the pb on my old py35 env.
merging.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/pull/5099#issuecomment-378841382,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGM9yNys3F1XEk7fxXamw7c0tSZ7fX2Tks5tlcO9gaJpZM4THdHO
.


Allow alert height to auto-adjust based on text length/size

:+1: thanks for this.

Sure thing :)


Long int fix

Description
Data Prepper should support long integer literals . Without this support conditionals like /durationInNanos >=100000000 are failing with the error
2023-02-10T23:04:34,582 [span-pipeline-processor-worker-5-thread-1] ERROR org.opensearch.dataprepper.expression.ParseTreeEvaluator - Unable to evaluate event
org.opensearch.dataprepper.expression.ExpressionCoercionException: Unsupported type for value 530159171
        at org.opensearch.dataprepper.expression.ParseTreeCoercionService.resolveJsonPointerValue(ParseTreeCoercionService.java:67) ~[data-prepper-expression-2.1.0-SN
APSHOT.jar:?]
        at org.opensearch.dataprepper.expression.ParseTreeCoercionService.coercePrimaryTerminalNode(ParseTreeCoercionService.java:35) ~[data-prepper-expression-2.1.0-
SNAPSHOT.jar:?]

Issues Resolved
Check List

[ X] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

Codecov Report

Merging #2265 (dfc263b) into main (c797d5c) will increase coverage by 0.25%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2265      +/-   ##
============================================
+ Coverage     93.66%   93.91%   +0.25%
- Complexity     1825     1926     +101
============================================
  Files           224      224
  Lines          5284     5375      +91
  Branches        426      427       +1
============================================
+ Hits           4949     5048      +99
+ Misses          233      222      -11
- Partials        102      105       +3

Impacted Files
Coverage Δ

...xpression/LiteralTypeConversionsConfiguration.java
100.00% <100.00%> (ø)

.../dataprepper/expression/OperatorConfiguration.java
100.00% <100.00%> (ø)

...taprepper/expression/ParseTreeCoercionService.java
100.00% <100.00%> (ø)

...opensearch/dataprepper/pipeline/ProcessWorker.java
88.46% <0.00%> (+15.38%)
:arrow_up:

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

@dlvenable added tests to ConditionalExpressionEvaluatorIT to test long integer behavior.


Error: Error: Results must be all atomic, or all data frames

In the mutations_heatmap function, subfunctions plot_bar an error occurs when processing a large data sets.
For example running the code with a recurrence cutoff of 20 seems to work fine however a recurrence cutoff of 0 will produce the error. needs to be looked into further (test file is bcla maf file).

Upon further review this seems to be a bug in plyr : http://stackoverflow.com/questions/30520575/intermittent-errors-with-ggplot2-segfault-memory-not-mapped

This appears to be fixed in the latest developmental version of plyr <IP_ADDRESS>, added this version or greater as a dependency


Add Tidal

No direct version installer as far as i know..
Microsoft Reviewers: Open in CodeFlow

no idea what's failing, cba troubleshooting since it works locally, also seems like Tidal have moved to the Microsoft Store because that version is 2.17

/AzurePipelines run


URL validation is minimal at best

FILTER_VALIDATE_URL is a minimal validation filter, as documented in the PHP docs:

[FILTER_VALIDATE_URL] Validates value as URL (according to » http://www.faqs.org/rfcs/rfc2396), optionally with required components. Beware a valid URL may not specify the HTTP protocol http:// so further validation may be required to determine the URL uses an expected protocol, e.g. ssh:// or mailto:. Note that the function will only find ASCII URLs to be valid; internationalized domain names (containing non-ASCII characters) will fail.
@sauce

At a minimum, I think a secondary validation should be made to detect the presense of a URL protocol:
<?php
    // update to the /src/Rules/Url.php file
    public function check($value)
    {
        return filter_var($value, FILTER_VALIDATE_URL) !== false &&
            preg_match('/^[\w]{2,6}:\/\//', $value);
    }

I may look into issuing a PR later this evening/week when I can get to it but I've already tested and validated the REGEX provided works just fine.

How about mailto? I think it is better to provide parameter for URL scheme. So we can do something like 'my_url' => 'url:http,https' to allows URL with scheme http or https.

I can't think of a single instance where I would be validating a user submitted mailto link, since I would rather accept and validate an email and compose the mailto in my app, but my suggested regex would encompass a mailto: prefix just fine. Though it only validates the presence of a protocol string of 2 to 6 characters. It does not validate if the protocol prefix is a known protocol or not,

Sorry for inactivity, I was very busy a week ago.
Yes, you're right. But I think someone may just need URL contains http or https, so it must be invalid if user input url with protocol ftp://, file://, etc.

The regex I wrote is based on this list detects the presence of a protocol string based on this list of publicly used protocols (http://www.realifewebdesigns.com/web-resources/web-protocols.html)
So ftp:, file:, arp:, dhcp: etc would be validated accordingly. This was the most effective way to allow for new or non-public/proprietary protocols to be utilized without users experiencing false negatives.
My regex only checks for protocolstrings of length 2-6, so if future protocols are shorter/longer, it should be easy to resolve going forward.

Sorry for ignoring this so long. I just found out that by default FILTER_VALIDATE_URL also checking scheme by enabling FILTER_FLAG_SCHEME_REQUIRED.
But I've just add capability to specify URL scheme like url:http to only allow http URL, or url:http,https to allows http or https, and so on. So I will close this. Thanks for your information btw. :smile:


Add initial test for gcp pubsub indexer

Description
PR is still in draft, few things I need to do before review
Before adding new functionality to gcp pubsub indexer (at_least_once support), we want to be sure that basics functionality are working.
As suggested in https://github.com/quickwit-oss/quickwit/pull/3720#issuecomment-1675038458 this PR is adding basic integration test.

[x] test: client return error if subscription does not exist
[x] test: ensure we process message and exit when backfill_mode

We are also doing small improvement: upgrade the dependency, better support for GCP auth.
How was this PR tested?
This pr is adding integration test :)

You're almost there, so I will approve this PR, assuming you fix the little things here and there. Worst case, we'll take care of them in the next PRs. Thanks for your great work!

yeah I am fixing the PR now and I will likely add an assert for the checkpoint delta. likely tomorrow. Thank you for your details review !

I wanted to quickly do https://github.com/quickwit-oss/quickwit/pull/3744#discussion_r1294665268 but it is actually a bit more tricky because almost everything is random for now:

the partition_id
the message_id
the timestamp

I will check more tomorrow, but does there is an easy way to assert any string, any timestamp greater than X etc inside a struct or I need to write a custom function that will do that ?
Anyway I put the PR in ready, I also had to revert a suggested change https://github.com/quickwit-oss/quickwit/pull/3744/commits/c524b6d3ce3b81572868117a7a427c565301a457 because we likely don't want to important google_auth in this package just for the link ? or is it worth it ?

@guilload Also I wanted to ask, isn't better we don't add parallelism in the gcp indexer, but just adding max_num_pipelines_per_indexer  and desired_num_pipelines ?

It depends on the throughput delivered by the source with no parallelism. We need to achieve a throughput of at least 20-30MB/s to saturate the indexing pipeline, after which we can start spawning more indexing pipelines.

It depends on the throughput delivered by the source with no parallelism. We need to achieve a throughput of at least 20-30MB/s to saturate the indexing pipeline, after which we can start spawning more indexing pipelines.

Okay sounds good ! Then next PR will focus on at_least_once. Then we can test / benchmark. Then optimize / add parallelism if needed ?
Also in parallel I am working with @sravan-s on a Nats implementation. https://github.com/AyWa/quickwit/pull/2
We are planing to follow similar step as what we do for gcp pubsub :)


protoc-gen-twirp generates non stable imports list

If the protobuf definition has more than one imports, the generated code may change due to iterate over a map https://github.com/twitchtv/twirp/blob/master/protoc-gen-twirp/generator.go#L340-L342 .
This is an issue because we check in generated codes, and it keeps changing making our CI think someone forgot to check in the generated codes.

I think you refer to the order of imports being unreliable each time that the code is generated right?
That could be fixed by converting the map into a list and sorting it. What do you think?

Thanks for the reply. Yes, non-stable order of imports is what I meant. I actually forked and patched using the same way as your suggestion
diff --git a/protoc-gen-twirp/generator.go b/protoc-gen-twirp/generator.go
index f331f65..4e42030 100644
--- a/protoc-gen-twirp/generator.go
+++ b/protoc-gen-twirp/generator.go
@@ -22,6 +22,7 @@ import (
        "go/printer"
        "go/token"
        "path"
+       "sort"
        "strconv"
        "strings"

@@ -337,8 +338,13 @@ func (t *twirp) generateImports(file *descriptor.FileDescriptorProto) {
                        }
                }
        }
-       for pkg, importPath := range deps {
-               t.P(`import `, pkg, ` `, importPath)
+       pkgs := make([]string, 0, len(deps))
+       for pkg := range deps {
+               pkgs = append(pkgs, pkg)
+       }
+       sort.Strings(pkgs)
+       for _, pkg := range pkgs {
+               t.P(`import `, pkg, ` `, deps[pkg])
        }
        if len(deps) > 0 {
                t.P()

, but I have some trouble to come up a test case for generateImports function.

Fixed by #312


Redmi 4x

There is no kali nethunter or external wifi card support for redmi 4x in pie plz do something. I am using Aex 6.2 pie custom rom. If you will patch the kernel with wifi drivers than plz i have the drivers of an adapter which i bought for redmi 4x in the hope that it will work but not detected by the device , i will send those drivers to you please also be patch them i will never forget your's this great work. Thank you.

This module only adds the drivers for wireless adapters. The kernel must still support it.
Sorry, I don't compile kernels, only add the required files. Try XDA or the the maintainer for that ROM to include the patches.


Licence?

I may want to submodule this in the future, clear licensing would be helpful in that regard :)

Good idea. How about CC-BY or unlicense?

Looks like either is fine with me :)

Fixed in 6f074d807fe16


Give pointers on how to use git / github

Not doing this after all, instead do #150 for non developers.
(we assume developers will know how to use github)


self-document the benchmark's setup

This PR extends the report to include a header that includes important details about the benchmark so that the results are self-documenting/reproducible.
It also adds an optional --notes cli arg if the experimenter needs to add additional notes to be included in the report.
Now it should be much easier to reproduce the report.
Here is an example:
$ python mm_flops.py -m 1 -n 2048 -k 2048 --output_file=mm_m_range.txt --notes="numa_balancing=0"

Benchmark started on 2024-02-17 02:20:50

** Command line:
/env/lib/conda/py39-pt23/bin/python mm_flops.py -m 1 -n 2048 -k 2048 --output_file=mm_m_range.txt --notes=numa_balancing=0

** Platform:
Linux example 5.15.0-1004- #4-Ubuntu SMP Fri Jan 26 21:18:52 UTC 2024 x86_64 x86_64

** Critical component versions:
torch=2.3.0.dev20240216+cu121, cuda=12.1, nccl=(2, 19, 3)

** Additional notes:
numa_balancing=0

--------------------------------------------------------------------------------

Elapsed time for 1x2048x2048: 0.000
Throughput (in TFLOP/s) for 1x2048x2048: 1.502
--------------------------------------------------------------------------------

You can later add other component versions that you think are important

hmm, probably should try to detect nvidia-smi or rocm-smi and dump those as well, right? as it would also be useful to have in the log file to know exactly what the file was generated on


Adding latin hypercube

This PR adds the latin hypercube sampler and an  example notebook demonstrating what it does. Let me know how it looks.

Looks like a helpful contribution! Please add a test to make sure that the generator replicates scipy behavior and that it creates expected batch sizes when it runs out of samples

Sounds good, can work on that soon

Ok, I think those tests should be in place and it should pass flake8 now. Tests are for number of samples returned as the batches get refreshed and that scipy samples are reproduced.

thanks for the contribution!


x/build/cmd/gomote: instruct gomote users on how to request authentication credentials

Once gomote users can create authentication credentials via the new path, the users should be notified via the following mediums:

[ ] Update the gomote create preamble text to include instructions on how to request credentials.
[x] Update the wiki.
[x] Send out an email.

This is a component of the project to revamp the security model used by gomote #47521
@golang/release

This has been completed.


fix void env supply by yarnScript

The assumption of job-exporter was broken by #2352 , which will cause the job-exporter stop reporting the GPU usage by tasks

Coverage remained the same at 52.924% when pulling df31f4f1522a807868cee12eba302555d4b4bbb4 on dixu/fix-void into 02b9169589562a9570c6e1edf335b0be6830c2c7 on master.


topic improvements

The topic https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/how-to-determine-which-net-framework-updates-are-installed has a very low CSAT. Trying to see if some changes can improve the CSAT here.

Included PowerShell version (@doctordns, @zjalexander do you have any suggestions for me in this PS example?) Ideally I'd just list the KB name like the C#/VB code version.
Simplified the code. Updates don't have descriptions on my machine or different names. Am I cutting/simplifying too much?
A lot of traffic arrives here thinking this is a topic to show to update .NET Framework. Changed title to be more explicit about the content

Internal review URL

Get-ChildItem "HKLM:SOFTWARE\Wow6432Node\Microsoft\Updates\*" -Recurse | Where-Object {$_.name -like "*.NET Framework*"} | Split-Path -Leaf

Output:
KB2468871
KB2468871v2
KB2478063
KB2533523
KB2544514
KB2600211
KB2600217
KB2468871
KB2468871v2
KB2478063
KB2533523
KB2544514
KB2600211
KB2600217

Thanks @zjalexander, but that doesn't break the KBs per Framework version  😢

oh, oops.  let me ask some actual experts then 😅

How about this as a PowerShell example for this question?
$DotNetVersions = Get-ChildItem HKLM:\SOFTWARE\WOW6432Node\Microsoft\Updates

ForEach($Version in $DotNetVersions){

   $Updates = Get-ChildItem $Version.PSPath
    ForEach ($Update in $Updates){
       [pscustomobject]@{Version=$version.PSChildName;Update=$update.PSChildName}
       }
}

Returns
Version                                   Update
-------                                   ------
Microsoft .NET Framework 4 Client Profile KB2468871
Microsoft .NET Framework 4 Client Profile KB2468871v2
Microsoft .NET Framework 4 Client Profile KB2478063
Microsoft .NET Framework 4 Client Profile KB2533523
Microsoft .NET Framework 4 Client Profile KB2544514
Microsoft .NET Framework 4 Client Profile KB2600211
Microsoft .NET Framework 4 Client Profile KB2600217
Microsoft .NET Framework 4 Extended       KB2468871
Microsoft .NET Framework 4 Extended       KB2468871v2
Microsoft .NET Framework 4 Extended       KB2478063
Microsoft .NET Framework 4 Extended       KB2533523
Microsoft .NET Framework 4 Extended       KB2544514
Microsoft .NET Framework 4 Extended       KB2600211
Microsoft .NET Framework 4 Extended       KB2600217

@1RedOne, this is great. It brought me closer to what I wanted. If I modify the script like the following, I get the results I was looking for:
$DotNetVersions = Get-ChildItem HKLM:\SOFTWARE\WOW6432Node\Microsoft\Updates

ForEach($Version in $DotNetVersions){

   $Updates = Get-ChildItem $Version.PSPath
    $Version.PSChildName
    ForEach ($Update in $Updates){
       $Update.PSChildName
       }
}

Thanks for your help!

Awesome, happy to help :)


How to add a separate classification layer

Hi, I am trying to train the model with BDD100k dataset. The dataset contains bounding box data of each object at the same time weather information of the image (i.e. rain, sunny, etc.). I want to add an additional classification layer with the Yolo so that, it can classify the weather of the image as well with the objects.
How can I do it? And how to load the weather data?

Did you have any idea? I am also curious about that.


Verify row index implementation

Not sure whether the existing row index implementation is correct. The documentation is slightly hard to interpret. Particularly these sections from https://orc.apache.org/docs/spec-index.html:

To record positions, each stream needs a sequence of numbers. For uncompressed streams, the position is the byte offset of the RLE run’s start location followed by the number of values that need to be consumed from the run. In compressed streams, the first number is the start of the compression chunk in the stream, followed by the number of decompressed bytes that need to be consumed, and finally the number of values consumed in the RLE.

For columns with multiple streams, the sequences of positions in each stream are concatenated. That was an unfortunate decision on my part that we should fix at some point, because it makes code that uses the indexes error-prone.

I think I am running into a bug due to row indexes.
If I write over 10,000 rows to a single file then athena returns the following error
HIVE_CURSOR_ERROR: index (0) must be less than size (0)

orc-tools doesn't have a problem reading meta or scanning the file though

@scritchley is there any update on the correctness of using the row index implementation?


[Help] ==> Did you mean C:\Users\User.drone\drone-hive\hive\reactivex\rxandroid

λ drone add reactivex/rxandroid app

==> Did you mean
C:\Users\User.drone\drone-hive\hive\reactivex\rxandroid

I don't know, did I?

Hmmm, it looks like you're using windows, I don't own any windows machine, can't replicate it :/
It looks like something is wrong with the path to the hive on windows
I'll put the help wanted tag on this issue.


cleanup: remove unused Asset methods

Feature Request
Currently, the Asset class, that is composed by a set of properties, has some methods that describe some properties that are used only by tests and that could be removed, as, aside id, they are specific to use cases.

getName
getVersion
getContentType

I have a doubt about the getDescription one as it is used by the AssetToIdsResourceTransformer to create an IDS Resource, but I'm not sure the knowledge about the name of the field should stay inside the Asset class, as this is strictly dependent to the IDS protocol.
Which Areas Would Be Affected?
asset, core
Why Is the Feature Desired?
to cleanup our domain classes
Solution Proposal
remove the methods
Type of Issue
cleanup
Checklist

[x] assigned appropriate label?
[x] Do NOT select a milestone or an assignee!

Version we don't need but name and content type we do, particularly for display in a UI.

The getDescription should never have been used to create the IDS resource. However, it might still come in handy for UI purposes.

Since the properties map is free-hand there are no limits on define new fields by the users, but I can't see why they should be in the Asset class as for our core domain model these properties are not useful, they are dependent from the context, so I would remove them and let them be handled e.g. by a UI.

The getDescription should never have been used to create the IDS resource.
what should be the alternative in this case?

I'm fine with requiring the use of extensible properties. However, I believe IDS mandates a name and description. @juliapampus is that the case?

To the best of my knowledge, the description is not mandatory (? @PHochmann). However, it is an attribute that could be mapped from Asset to ids:Resource

dct:title and dct:description are not mandatory [1].
[1]: https://github.com/International-Data-Spaces-Association/InformationModel/blob/15e6d2fff83ba49ebdb7a1cd77418cd19ab28d26/testing/shared/DescribedShape.ttl#L43

Just to recap:

id: it is mandatory as used to reference the asset
name: not mandatory but it should be mapped to ids "dct:title" (currently it's not)
description: not mandatory but it should be mapped to ids "dct:description"
contentType: needed to display in a UI
version: not needed at all

Looking to the ids:Resource type I feel like we added these fields to provide a mapping between EDC and IDS, but since there's the IDS review goin' on, I would say to postpone this discussion. I close this issue for the moment


ONNX export update for YOLOv8

Adds a function that allows a dequantize node to be propagated down from a split node. That allows the dequantize node to be close to other ops that might be quantized after the split, and enable the proper conversion.

LGTM overall - will need to add this to the ONNXToDeepSparse converter as well

Will add ONNXToDeepSparse in a future PR


BrowserWindow.getNativeWindowHandle() doesn't give a valid HWND on Windows

Hi,
I'm using BrowserWindow.getNativeWindowHandle() API (introduced in #4023) as I need to reparent the Chromium window in our application. But it seems that the buffer returned does not contain any valid HWND (I checked with MS tool Spy++). I did a test using the same code than getNativeWindowHandle() and I found that using node::Buffer::New() is wrong as I could not get back the initial value. However, this is working when using node::Buffer::Copy(). I think the same change should be done in the ToBuffer() method used by getNativeWindowHandle()'s implementation. The NodeJS states that node::Buffer::Copy() is doing a copy of source data which is better in this case.

Proposed pull request #4398

Fixed by #4398.


Try to improve hybrid property

This is a try of using the mikes provided code in hybrids but it does not work propertly. Can you take a look @zzzeek ?
Description

Fixes: #171
Checklist

This pull request is:

[ ] A documentation / typographical error fix

Good to go, no issue or tests are needed

[x] A short code fix

please include the issue number, and create an issue if none exists, which
must include a complete example of the issue.  one line code fixes without an
issue and demonstration will not be accepted.
Please include: Fixes: #<issue number> in the commit message
please include tests.   one line code fixes without tests will not be accepted.

[ ] A new feature implementation

please include the issue number, and create an issue if none exists, which must
include a complete example of how the feature would look.
Please include: Fixes: #<issue number> in the commit message
please include tests.

Have a nice day!

it doesnt work, my other recipe must have relied upon the way sqlalchemy-stubs works.   feel free to close if theres no other ideas

There is an issue also with the current version. Like if you try the master branch, with the example in hybrid.py you get this:
obj = SomeModel()
reveal_type(obj.hp)

note: Revealed type is "def (*Any, **Any) -> Any"

this is clearly not correct. I think this may be a mypy issue, since pylance (on vscode) does correctly infer the type (to any in master, to the correct return value with this branch)
I'm not sure if the plugin has something to with this tough

closing since for v2 we went on another direction


Datagrid fixed height gets broken once the user changes page size

Describe the bug
When you have a clarity data grid with fixed height (in pixels or percent) and the datagrid is paged, whenever the user changes the page size the grid resizes to a different height (same as needed to exactly fit the new page size).
From this point on the grid can never get back to it intended fixed height.
How to reproduce
You can see the following stack blitz example: https://stackblitz.com/edit/clarity-datagrid-broken-height

I've set 100 items per page as the default and everything is fine.
Change the page size to 50 - and the layout breaks - the grid expands vertically to fit all 50 rows.
Change the size to 10 - now the grid is smaller than it's original fixed size.

If you change the default page size to 10 you see that there is leftover space beyond the tenth element. If you try switching to 5 elements the grid shrinks in height to fit exactly 5 rows.
Expected behavior
Datagrid height remains fixed no matter what page size the user picks.
Versions
App

Angular: tested with 10.0.11 and 11.0.5
Node: N/A
Clarity: 4.0.8

Device:

Type: PC
OS: Win 10
Browser:

Opera 73.0.3856.329
Chrome 87.0.4280.141
New Edge 87.0.664.75
Firefox 84.0.1

On further digging, the issue seems to be related to the height reset happening on page change. The handler for size change (https://github.com/vmware/clarity/blob/e28c89b74aad11aeb861ca5cd4a569d9953e8b96/packages/angular/projects/clr-angular/src/data/datagrid/render/main-renderer.ts#L71) triggers resetDatagridHeight() (https://github.com/vmware/clarity/blob/e28c89b74aad11aeb861ca5cd4a569d9953e8b96/packages/angular/projects/clr-angular/src/data/datagrid/render/main-renderer.ts#L167)
  private resetDatagridHeight() {
    this.renderer.setStyle(this.el.nativeElement, 'height', '');
    this._heightSet = false;
  }

This is when the size set on the grid is lost. If I replace the empty string with the fixed size as originally intended all is good.

On further digging, the issue seems to be related to the height reset happening on page change. The handler for size change (https://github.com/vmware/clarity/blob/e28c89b74aad11aeb861ca5cd4a569d9953e8b96/packages/angular/projects/clr-angular/src/data/datagrid/render/main-renderer.ts#L71) triggers resetDatagridHeight() (https://github.com/vmware/clarity/blob/e28c89b74aad11aeb861ca5cd4a569d9953e8b96/packages/angular/projects/clr-angular/src/data/datagrid/render/main-renderer.ts#L167)
  private resetDatagridHeight() {
    this.renderer.setStyle(this.el.nativeElement, 'height', '');
    this._heightSet = false;
  }

This is when the size set on the grid is lost. If I replace the empty string with the fixed size as originally intended all is good.

Seems that if I strictly follow the example from the clarity docs and define the fixed height in a class based on the grid's parent container all works as expected: https://stackblitz.com/edit/clarity-datagrid-broken-height-workaround
The specific issue can be considered resolved, but this design is not intuitive and ant the very least has to be called out explicitly in the docs.

Seems that if I strictly follow the example from the clarity docs and define the fixed height in a class based on the grid's parent container all works as expected: https://stackblitz.com/edit/clarity-datagrid-broken-height-workaround
The specific issue can be considered resolved, but this design is not intuitive and ant the very least has to be called out explicitly in the docs.

@tonyganchev thanks for open this issue, we gonna take a look at it ...

@tonyganchev thanks for open this issue, we gonna take a look at it ...


Resource merger doesn't work on generic multifield dialogs

If the attribute "sling:resourceSuperType" is being used within the item dialog, then the properties from the superType are not being merged.

Has been fixed with commit dfd9b5be2e2ad164873422ec32ae607cd2e8bbda.


fix: ip_restriction block

It has been a long time that ip_restriction was a block in app service related resource, not an attribute. Therefore, we cannot assign a value to it using = operator;
If we do so, we will see this error:

This issue was already addressed in the function-app module; So, I took the same structure and re-used it in this PR.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Changes proposed in this pull request

fixes the issue with ip_restriction config assignment

@claranet/fr-azure-reviewers

Hi team
Is it possible to get a release tag on this?
Thanks

Hi @kevin-fitlg
This has been released in the version 7.3.1


Resource History Not Based on Source

It should be, because currently we manually prune blacklisted keys from the metadata, but it would me more consistent to use the source endpoint.
feedback from @bogdanromanx

after some investigation I'll close this issue, as it turns out we want some metadata like the User to know who performed the update, as well as the time. The alternative would be to do two requests per revision, which is a little much.


22 add jwt auth integration

Codecov Report

Merging #29 (eaad2ce) into release-candidate (8da24b4) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head eaad2ce differs from pull request most recent head 1e3260a. Consider uploading reports for the commit 1e3260a to get more accurate results

@@                 Coverage Diff                 @@
##           release-candidate       #29   +/-   ##
===================================================
  Coverage             100.00%   100.00%
===================================================
  Files                      3         7    +4
  Lines                     15        41   +26
  Branches                   0         2    +2
===================================================
+ Hits                      15        41   +26

Impacted Files
Coverage Δ

src/jwt/jwt-auth.guard.ts
100.00% <100.00%> (ø)

src/jwt/jwt-auth.module-definition.ts
100.00% <100.00%> (ø)

src/jwt/jwt-auth.module.ts
100.00% <100.00%> (ø)

src/jwt/jwt-auth.strategy.ts
100.00% <100.00%> (ø)

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


Discussion about Data Labelling in the Vietnamese language

Thank you really much for your grateful project.
I just having confused that,
For instance

Input sentence: “Giao tôi lê_lai phường hai tân_bình hcm”
Value after tokenizer:
{‘input_ids’: [0, 64003, 64003, 17489, 6115, 64139, 64151, 64003, 6446, 64313, 1340, 74780, 2], ‘token_type_ids’: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0], ‘attention_mask’: [1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1]}
Because tokenize of “lê_lai” is [‘lê@@’, ‘l@@’, ‘ai’]; of “tân_bình” is ['tân@@’, ‘bình’]; of “hcm” is [‘h@@’, ‘cm’]
The result I got after all: [‘O’,‘O’,‘B-LOC’,‘I-LOC’,‘I-LOC’,‘I-LOC’, ‘I-LOC’,‘I-LOC’,‘O’,‘I-LOC’,‘I-LOC’, ‘O’]

In fact, their prediction should only have 7 tags for the input tokens, but now it was more than this. Do this project have any strategies for this.
According to HuggingFace's document

Now we arrive at a common obstacle with using pre-trained models for token-level classification: many of the tokens in the W-NUT corpus are not in DistilBert’s vocabulary. Bert and many models like it use a method called WordPiece Tokenization, meaning that single words are split into multiple tokens such that each token is likely to be in the vocabulary. For example, DistilBert’s tokenizer would split the Twitter handle @huggingface into the tokens ['@', 'hugging', '##face']. This is a problem for us because we have exactly one tag per token. If the tokenizer splits a token into multiple sub-tokens, then we will end up with a mismatch between our tokens and our labels.
One way to handle this is to only train on the tag labels for the first subtoken of a split token. We can do this in 🤗 Transformers by setting the labels we wish to ignore to -100. In the example above, if the label for @HuggingFace is 3 (indexing B-corporation), we would set the labels of ['@', 'hugging', '##face'] to [3, -100, -100].
Let’s write a function to do this. This is where we will use the offset_mapping from the tokenizer as mentioned above. For each sub-token returned by the tokenizer, the offset mapping gives us a tuple indicating the sub-token’s start position and end position relative to the original token it was split from. That means that if the first position in the tuple is anything other than 0, we will set its corresponding label to -100. While we’re at it, we can also set labels to -100 if the second position of the offset mapping is 0, since this means it must be a special token like [PAD] or [CLS].

I do appreciate your time and sharing.

Hi @lacls
Sorry for the late reply.
I am a little unsure, what exactly you want me to do? Do you have an  idea? I am very open to a Pull Request on this matter.
Best,
Lars


Read the Network List from configuration

The Network List concept can be defined as the union of these two sets:

the registered networks (returned by the Epoch Subgraph), and
the networks listed in the configuration file.

We already parse the latter, but we don't use them when constructing DataEdge messages.
(The former can be acquired simply by querying the Epoch Subgraph.)

Fixed in https://github.com/graphprotocol/block-oracle/pull/99/commits/c1b9bd0e9748732866985a58c4872f29953a74a5

Fixed in https://github.com/graphprotocol/block-oracle/pull/99/commits/c1b9bd0e9748732866985a58c4872f29953a74a5


Loading an Environment resource from a separate thread blocks the main thread

Godot version:

Latest master commit: hash 9c3ddf05cb9c59817d885e9daca6e8f61c89dc97
OS/device including version:

Fujitsu Lifebook with Intel i5, tested on Windows 7 and the latest version of Lubuntu
Issue description:

I'm calling ResourceInteractiveLoader::poll from a separate thread, and it's blocking the main thread during one poll.  I made a project to debug the issue, and had no noticeable stutter until I added a WorldEnvironment to the loaded scene.  After more testing, it seems loading an Environment resource blocks the main thread, regardless of what thread calls load().
There are ways to work around this for my use-case (i.e. having a global WorldEnvironment and using custom logic to update its parameters during scene transitions), but I'm wondering what the underlying cause of this is and if it can be fixed.
Steps to reproduce:

Load a scene from a separate thread
If it contains a WorldEnvironment, it will block the main thread (it may not be noticeable on high-end hardware)
Deleting the WorldEnvironment will remove the stuttering caused by blocking the main thread.  There may be other nodes which cause this behavior, however.

Alternatively, just load an Environment resource from a separate thread, but then you won't have the comparison.
Minimal reproduction project:

blocking_loader.zip
This project loads one relatively large model (6MB) and then an environment.  The Environment causes a noticeable stutter in the animation, while the model does not (at least not until it's inserted into the scene tree).  You can also modify the loading_test.gd script to load a whole scene, which stutters unless you delete the scene's WorldEnvironment.

The old ResourceInteractive loader wasn't really good. You could split loading into multiple steps, but each step could still block your game (at least for rendering-related resources like textures).
4.0 introduced a new threaded loading functionality, which should solve this problem. AFAIK it's impossible to fix this in 3.x due to different renderer.


Rules review

[ ] Added or updated tests
[ ] Documented user facing changes

I've used Larastan for a while and I love it! This is my first foray into Larastan code, and I started by reviewing the Rules.
I'm opening this as a Draft PR just to get some feedback and see if you're open to these types of changes.
I also wrote a failing test to address one of the todos from #1367 but I'm not quite clear how to get it to pass. Inside isCalledOnModel, the $class is an instance of FullyQualified, but the isSuperTypeOf check is maybe() not yes(). I'll keep digging, but I mention it in case you can point me in the right direction.
Changes

Fixed an incorrect generic type on one rule
I standardized imports of PHPStan classes (I didn't see any detailed style rules, so I kept this in a separate commit in case it's not welcome)
Add a psr-4 mapping for the Tests namespace
Added a failing test for the todo inside the NoModelMakeRule

Breaking changes
None

I standardized imports of PHPStan classes

Thank you for this PR.
FQCN should be used in

docblocks
exceptions
in extends/implements

"Tests\\": "tests/"

It is a big work to make tests directory PSR-4 compliant.
First remove classmap mapping and issue
composer dump-autoload --optimize --strict-psr

Hi,
This PR does too many stuff inside. I'd appreciate if each PR is focused on only one thing. Also, for code style we use StyleCI, so if it doesn't complain we don't need to do manual changes. Hope you understand.
For the NoModelMakeRule change please send a separate PR, that looks interesting!
Thank you.

Sure thing, I'll re-submit after Laracon. Thanks!


unsupported TRACE_EXPORTER:

After updating the gofr version to v1.16.0, we're encountering the following error: unsupported TRACE_EXPORTER:. We're not using any tracing, and the environment variables are empty. Could you please look into this issue?

@surajit-zs - Thanks for reporting this issue.
This was already know, and the PR #901 was created and pending for merge.


pdfsam-basic: 4.3.4 -> 5.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/torakiki/pdfsam/releases.
meta.description for pdfsam-basic is: Multi-platform software designed to extract pages, split, merge, mix and rotate PDF files
meta.homepage for pdfsam-basic is: https://github.com/torakiki/pdfsam
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.0.2 with grep in /nix/store/l1ksf01ndbz9r3d5y4d94xbkbzh0rxxk-pdfsam-basic-5.0.2
found 5.0.2 in filename of file in /nix/store/l1ksf01ndbz9r3d5y4d94xbkbzh0rxxk-pdfsam-basic-5.0.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
pdfsam-basic

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l1ksf01ndbz9r3d5y4d94xbkbzh0rxxk-pdfsam-basic-5.0.2 \
  --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 pdfsam-basic https://github.com/r-ryantm/nixpkgs/archive/cee5c1d478a1ddf8fff99430c7fff532c50c728d.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/l1ksf01ndbz9r3d5y4d94xbkbzh0rxxk-pdfsam-basic-5.0.2
ls -la /nix/store/l1ksf01ndbz9r3d5y4d94xbkbzh0rxxk-pdfsam-basic-5.0.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

pdfsam-basic

Maintainer pings
cc @1000101 for testing.

Result of nixpkgs-review pr 214671 run on aarch64-darwin 1

1 package built:

pdfsam-basic

Requires jdk19.


[DO NOT MERGE] fix(core): make nullable field in selection not be optional

This is a working hotfix for #699 to just make the issue gone.
Because I need to solve the issue to make things could continue first.
@dotansimha  I'm not sure if this fix is sane enough for #699. If this direction is acceptable, I'd go further to fix the tests and other stuff.
The goal of this PR is to differentiate a nullable field is in introspection or in a query.

There are still some cases not covered. I've no idea what to do next ×_×

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, and I think we can merge it. Can you please add some unit tests for it?

There are still some cases not covered. I've no idea what to do next ×_×

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, and I think we can merge it. Can you please add some unit tests for it?

All right, I'll try to complete this PR and check what case is missing later in home.

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, and I think we can merge it. Can you please add some unit tests for it?

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, and I think we can merge it. Can you please add some unit tests for it?

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, what do you think missing?
And can you please add some unit tests for it?

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, what do you think missing?
And can you please add some unit tests for it?

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, what do you think missing?
And can you please add some unit tests for it?

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, what do you think missing?
And can you please add some unit tests for it?

Thanks @frantic1048 ! and sorry for the delay in the review.
It actually looks good, what do you think missing?
And can you please add some unit tests for it?

Thanks @frantic1048 :) What's missing at the moment?

Currently, this fixes the simple demo in #699. With my actual project, there is still some error(not sure if it's caused by #699). However, the PR is for #699's case.

Thanks @frantic1048 . We did some architectural changes in the codegne recently, and some templates has moved. I did a fix for this issue here: https://github.com/dotansimha/graphql-code-generator/pull/855


Lies, all lies!

It uses way too much CPU and RAM (despite of what you are claiming, and you claim whatever you want, that does not make your statements true) and it is launched many, many times for just one instance of Teams.
It is not allowed to run on my systems anymore, so I will have to use the web version of Teams, because you effers do not know how to write good software and need to use even worse garbage, just because it is from the same company (which has known for their garbage since they got a bit more known).

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

ID: a98f4d73-3a87-7836-be1e-d98cf264768c
Version Independent ID: a98f4d73-3a87-7836-be1e-d98cf264768c
Content: WebView2 end-user FAQ - Microsoft Edge Developer documentation
Content Source: microsoft-edge/webview2/concepts/end-user-faq.md
Service: microsoft-edge
Sub-service: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

If you have specific technical feedback about WebView2, I would encourage you to provide it, in a civil and respectful way, with additional details, to the WebView2 team at https://github.com/MicrosoftEdge/WebView2Feedback/issues/new.
I will now close this issue as:

out of scope for this repository
violating the Microsoft open source code of conduct.


[FEATURE] choose which node to run if multiple with the same function are used

this should work:
@op
def a():
  pass
@op
def bar():
	a()
	a()
	a(__tawazi_tag="a3rd")

bar([a, 1])
bar([a])  #raises error
bar(["a3rd"])
bar([bar.get_node(a, 2)])

This is handled a long time ago


Unmatched F-String Fix

Line 64
self.write_to_file(f"------{self.date}------\n{self.username} has {initial["followers"]}\n{self.username} is following {initial["following"]} people\n{self.username} has {initial["posts"]} posts\n{self.username} has the following bio: {initial["bio"]}")
^^^^^^^^^
SyntaxError: f-string: unmatched '['

When I first ran the code, it did not work due to double quotes inside the f-string at line 64. However, after changing the inner double quotes with single quotes, I ran the code just fine.

self.write_to_file(f"------{self.date}------\n{self.username} has {initial['followers']} followers\n{self.username} is following {initial['following']} people\n{self.username} has {initial['posts']} posts\n{self.username} has the following bio: {initial['bio']}")

Could you open a PR for this change so I can merge it?


chore: release  parent 0.1.1

:package: Staging a new release
0.1.1 (2023-07-27)
Bug Fixes

introduce release-please process (#51) (39a21ec)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/carbynestack/castor/releases/tag/parent-v0.1.1 :sunflower:


[WordPress Plugins] ... Add WordPress.org slug

Extension
https://www.raycast.com/edmundcwm/wordpress-plugins
Description
It would be great to also add the WordPress.org slug in the overview so it can be easily copied. The Composer integration is also great as the last part of the URL (eg. Akismet wpackagist-plugin/akismet) is already integrated.
Who will benefit from this feature?
No response
Anything else?
No response

👋 @filip-buroform thanks for the suggestion! Could you provide some use cases where it's beneficial to display the plugin slug?

@edmundcwm I do a lot of WordPress core setups to a Composer driven setup. So by quickly pulling the slug name I can scan the plugin with a custom tool at some security and vulnerability issues. I also have a custom WP CLI command where I can add the slug names required for production and dev.
On the other hand is the slug also easy for the default WordPress plugin API endpoints.


Remove using namespace directives

@leonlan this was way less work than I thought it would be. Let's see if clang also accepts this :)


[beta] Sign in issue with username

Issue Description
I'm trying to sign back in to the beta site for Free Code Camp, but it's not letting me use my username to login.
It won't let me sign in unless I use an email, even if it shows inside the message box as "Invalid username".  First image is me trying to use my username. Second image is what shows up in the error message box.
Browser Information

Browser Name: Chromium. Version: 59.0.3042.0 (Developer Build) (32-bit)
Operating System: Windows 8
Mobile, Desktop, or Tablet: Desktop

Screenshot

You will not be able to log in using the username on beta, and soon this would be replaced with a newer login, these messages that you see would be replaced.
We have fixes planned and on their way.
Thanks for reporting!

The PR was merged, so I'm closing this issue. Happy coding!


ONVIF Device Test Tool

Hi,
I've successfully integrated this library (good work btw) in a small ONVIF device emulator I'm creating, and it's being detected by both ONVIF Device Manager and the free Happytimesoft ONVIF client.
However I'm having problems with the official ONVIF Device Test Tool, the application is not being detected. Has anyone tried this before?

Thanks! Unfortunately, I do not have access to those test tools, but if you figure it out I am willing to accept pull requests.

The ONVIF Device Test Tool doesn't send the action in the request (not in the content type, nor in the soap headers), I ended up having to infer it from the soap body. I'm still making some tests, will create a pull request later.

Could it be related to https://github.com/onvif/testspecs/issues/63? So perhaps the new version of the device test tool will fix this issue?


Datagrid ExpressionChangedAfterItHasBeenCheckedError after re-assigning the selection

We're trying to implement a directive, that stores user selection in-between live data refreshes. Wrapping the assignment in 'preselect-comparator.ts' line 57 into a setTimeout works, but i'd like to avoid using setTimeout.
The plunkr is just a partial implementation for a singular selection (i've removed multi-select branches to make it shorter)
[x] bug
[ ] feature request
[ ] enhancement

https://plnkr.co/edit/WUNt57MsppVCqK18VtH1
Select a row in the grid (any row)
Click REFRESH DATA BUTTON
Expected behavior
Data is reset, shuffled, selection is preserved by our custom preselect-comparator directive
Actual behavior
ExpressionChangedAfterItHasBeenCheckedError is thrown
Reproduction of behavior
https://plnkr.co/edit/WUNt57MsppVCqK18VtH1
Environment details

Angular version: 4.x.x

Clarity version: latest (as in plunkr)

OS and version: latest osx

Browser: all

@colossal-squid: It looks to me that you are updating the array twice. First setting it to empty and then shuffling and resetting it. If I remove the first assignment where you set it to empty, everything works fine and we get the same result. Please see: https://plnkr.co/edit/QyzLJiE5h2PjFRg974pg?p=preview
Does this help?

Yes, this definitely "fixes" it in a plunkr, but even if i remove similar "reset" assignments before http service calls, I'd like to prevent such exceptions occurring in real life environment.
Can anyone explain me if this is a bug, or us using it wrong? And if second - what happens under the hood and why is this illegal?

I believe that you just have to make sure that the bindings are not updated after the view has been updated. If they are you need to manually run change detection again. If you can find this pattern in your use case right now (you could check this by logging into the different lifecycle hooks), then an ExpressionChangedAfterItHasBeenCheckedError error will be thrown. More info can be found here:
https://angular.io/guide/lifecycle-hooks
https://medium.com/@golbie/angular-2-change-detection-what-the-hell-is-has-changed-after-it-was-checked-82007cdfa38a
https://stackoverflow.com/questions/43375532/expressionchangedafterithasbeencheckederror-explained
https://stackoverflow.com/a/34957665/3538394
Let us know if this helps?

There is another one with multiselect, I think I should create a separate ticket for this. Thanks


File to import not found or unreadable in _input.scss imports

Do you want to request a feature or report a bug?
bug
Bug Report
i am using this starter:
https://github.com/qdouble/angular-webpack2-starter/tree/no-universal-support
after run "start" command i got error:
"File to import not found or unreadable<EMAIL_ADDRESS>Parent style sheet<EMAIL_ADDRESS>Which version of Angular and Material, and which browser and OS does this issue affect?
<EMAIL_ADDRESS><EMAIL_ADDRESS>@angular 2.4.1
Other information
i think imports in this file "covalent/src/platform/core/common/styles/_input.scss" should have extra level of folder nesting, like this:
../../../../../node_modules/@angular/material/core/theming/palette
^

Hmm, can you provide more information? We dont get that error in any of our products, and people have been upgrading to 0.10 without that error..
Also, you can try using our pre-compiled platform.css file instead of the platform.scss if thats an issue.

Need some time to figure out scss processing in starter repo.
Thank you, closing issue for now.

You can also try adding the platform.css via index.html file.

I am trying to upgrade from v0.9.1 to v1.0.0-beta.2-1 and I am experiencing this issue as well.

Are you using the platform.css or platform.css?

We are using platform.scss.
Our import statement looks like:
@import '~@covalent/core/common/platform';

And we are building with webpack 1.14.0

I have the same issue when using with angular-cli.
I am using styles/main.scss instead of styles.scss as a starting point in angular cli.
So my styles section under .angular-cli.json looks like this
"styles": [
    "styles/main.scss"
]

Now if I import platform using @import within styles/main.scss, I am getting the exact same error.
I believe this issue is to do with the relative paths ../../


Simplify Atom implementation and remove dependencies

Our Atom has always been based on lenses and functional programming. However, with the advent of Streams in effection, I have been thinking about a simpler implementation of Atom. I realized that we could make an implementation which stacks slices on top of each other, rather than accessing the state directly. Effectively our slice becomes sort of like a lens itself, except with the addition of a stream.
Since there were some chicken-and-the-egg problems with making slices, I introduced Container as a simplified slice, this also makes it easy to attach all of the additional method for slice which are not needed for the traversal.
In my opinion the resulting implementation is easier to understand, and also means we do not need to depend on any external packages in @effection/atom.
The resulting code does make rather judicious use of any to skip around the inherent difficulty in making a type safe lens-like thing in TypeScript. However, since the type safety of Slices is provided by the MakeSlice type, there is no loss in external type safety.

@cowboyd I see your point on this. Composing slices is something I hadn't even thought about, and that definitely sounds like a can of worms that we may not want to eat ourselves.
Let's hold off on this for a while and go with the fp-ts based implementation for now.

I can the see the value of removing the external dependency. How much are we giving up by the liberal use of any inside of the implementation?

There are actually some uses of any in the existing implementation as well, and they work mostly around the same issues. There are some ways to increase the type safety of the makeArrayContainer and makeRecordContainer functions, but I don't actually think that they help that much in practice. Basically instead of using Container<any> we'd use a  Container<Array<any> instead, there's still an any in there but it's slightly better type safety inside, unfortunately this also leads to a ton of casts everywhere, which made the code much harder to read for marginal gain, IMO.


Add setting for seperate vertical and horizontal mouse sensitivity

Not sure if this is the best way to do this, but I modified the setting for mouse sensitivity to allow for horizontal and vertical motion.

@Jasper-J-R
Thanks for the PR! At a quick glance this looks like it makes sense to me. Let me go ahead and test it before I give it the green light.


Remake Notes folders with start_paper

And give each Note it's own issue and branch. Might want to upgrade start_paper to capture metadata first.

OK, start_paper upgrade is done, metadata asked for and then stored is:
{
    "authors": "{{ cookiecutter.authors }}",
    "title": "{{ cookiecutter.title }}",
    "description": "{{ cookiecutter.description }}",
    "short_title": "{{ cookiecutter.short_title }}",
    "serial_number": "{{ cookiecutter.serial_number }}",
    "repo_name": "{{ cookiecutter.repo_name }}",
    "folder_name": "{{ cookiecutter.folder_name }}",
    "default_format": "{{ cookiecutter.default_format }}"
}

This is loooooong overdue. Step one is to trash the existing notes, before starting branches for the existing ones. Looks like no-one has pushed to the notes in master since last fall, so I'll salvage the design note into its new branch, and trash the rest. The reason for working in branches is to be able to use the PR review mechanism for task force review of what we write.


PHP 7.2 compatibility changes

The use of Throwable would imply to drop support for PHP 5.6?

We've decided to go to PHP 7.2 without looking back

Yes, and for the record. We are not updating to a more recent PHP version, to prevent major updates to the Symfony/Doctrine installation. When we are going to update to a modern version of Symfony, we should also be able to bump to PHP 7.3 or even 7.4. Which, hopefully, by then is matured to an extent that we feel comfortable about, running on our platform.

I've updated the .travis.yml config

Dropped the PHP 5.6 builds
Removed 5.6 specific config settings from the before_scripts section

Which is now warranted as we no longer aim to be backward compatible for the first 7.2 release

The functional test breaks on the issue that is resolved in #756

@pablothedude the test is not broken. The template rendered in the error producing Behat test contained the reference to the SP fields that where removed in #755
What would you like to see changed here?

But that didn't result in a broken build, so why does it fail now and not before? As you can see master is also green.
If you can make this build green I'm happy.

As a fix for now, the incriminating tests have been marked with the @WIP annotation. These tests only fail on Travis. Where the test repository is unable to find the SP used in the test. A ticket was opened to fix this permanently.
https://www.pivotaltracker.com/story/show/168698275


Is this compatible with Invoke AI?

Invoke won't let me use safetensors. Do I need to use this code? Or am I just doing something wrong?

It's coming if I read this correctly:
https://github.com/invoke-ai/InvokeAI/pull/1583
Basically this PR needs to be merged:
https://github.com/invoke-ai/InvokeAI/issues/1813
And then it will be useable by default (diffusers works by loading safetensors if it's available both on the repo AND the library being there).
Were you thinking about enforcing the use of the format? If yes, we would need to take it to diffusers to maybe add a flag for this.
diffusers is probably willing to do this, the main thing we try at HF not to do is to do breaking change which would break existing user's code.

a safetensor ckpt is different from safetensor diffuser format though.
I believe most people are looking for the ckpt safetensor support, which will be separate from diffuser models.

a safetensor ckpt is different from safetensor diffuser format though.

You mean StableDiffusion ckpt kind of file ? Then it's already supported by the Webui https://github.com/AUTOMATIC1111/stable-diffusion-webui/blob/master/modules/sd_models.py#L168-L179
Does that help ?

Ya but i mean safetensor ckpt support in InvokeAI.

I think InvokeAI does support single-file safetensor models by now, and this issue can be safely closed.

Thanks @keturn
Closing this then


S3.py use the rangestring without end?

When I have a 30G object, maybe I only want to read front several bytes but I will read the whole object!

Sorry, there's some bad description in my previous comment.
Suppose s3://test/test is a 30G object. Run the code below in IPython.
# ipython
import smart_open
f = smart_open.s3.open("test","test","rb")
f.read(1)

Then I will see that the memory of this process will increase to 30G (if you have) .

Oh, if that is true, then it's a bug. Read should be buffered/stream.
Can you make your example reproducible, using a concrete S3 file?

Hmmmm, it seems that I made a mistake.
When I run these codes, the memory of my proxy to s3 will increase, not these codes.
Sorry, I will close this issue and do more tests for it.


[zh] sync configure-access-multiple-clusters.md

Sync with en.
content/zh-cn/docs/tasks/access-application-cluster/configure-access-multiple-clusters.md

Preview: https://deploy-preview-38531--kubernetes-io-main-staging.netlify.app/zh-cn/docs/tasks/access-application-cluster/configure-access-multiple-clusters/

Need to fix gate issue. Resubmit?

/lgtm
/approve


Addition of border for better distinction

The unTab window, on some websites might not be distinguishable at all from the background. Example given below :+1:

Thus inclusion of a border around the outermost container div would really help distinguish the unTab window from all kinds of background.

Makes sense, I will experiment with this.


Add multiple coins

Added some coins along with tests

I recommend to reject this PR and close. This particular ElectrumX is focused on Bitcoin only, and we should focus on supporting Bitcoin (BTC) further, maintaining and fixing as well as adding new features. Adding altcoins only complicates our job. Any altcoin can fork their own version directly from upstream and support their respective coins but adding altcoins to this particular repo should be discouraged.

@gits7r, since thats the stance, would you be okay with either me or yourself just replying to these and redirecting them to here: https://github.com/electrum-altcoin/electrumx. May aswell isolate all the altcoin PR's into 1 repo that wants them?


Move newton into flutter package

I believe @abarth has this underway.

I have this underway now.  :)


ValueError: wrong dimension in model.predict() input

Hello everyone!
I have to develop an image classifier and I am using Keras. I am using Tensorflow backend, running on CPU, with Python 3 on Windows 10. I am new to Python.
I decided to adapt the program I found at https://blog.keras.io/building-powerful-image-classification-models-using-very-little-data.html as I don't have much data. Both training and validation seem to be good but, when I try to load my model and put a new image in, I have ValueError: Error when checking : expected conv2d_1_input to have 4 dimensions, but got array with shape ().
I have been spending my entire week on it without finding anything that can help me. There have been similar issues posted on GitHub, but I was not able to understand or find an answer in these threads.
Thank you very much in advance for your help!
Here is the model generator code:
import keras
import tensorflow

from keras.preprocessing.image import ImageDataGenerator
from keras.models import Sequential
from keras.layers import Conv2D, MaxPooling2D
from keras.layers import Activation, Dropout, Flatten, Dense
from keras import backend as K

print('keras version:' + keras.__version__)
print('tensorflow version:' + tensorflow.__version__)

# dimensions of our images.

img_width, img_height = 150, 150

train_data_dir = 'C:/data/train'
validation_data_dir = 'C:/data/validation'
nb_train_samples = 1500
nb_validation_samples = 650
epochs = 20
batch_size = 15

if K.image_data_format() == 'channels_first':
    input_shape = (3, img_width, img_height)
else:
    input_shape = (img_width, img_height, 3)

model = Sequential()

model.add(Conv2D(32, (3, 3), input_shape=input_shape))
model.add(Activation('relu'))
model.add(MaxPooling2D(pool_size=(2, 2)))

model.add(Conv2D(32, (3, 3)))
model.add(Activation('relu'))
model.add(MaxPooling2D(pool_size=(2, 2)))

model.add(Conv2D(64, (3, 3)))
model.add(Activation('relu'))
model.add(MaxPooling2D(pool_size=(2, 2)))

model.add(Flatten())
model.add(Dense(64))
model.add(Activation('relu'))
model.add(Dropout(0.5))
model.add(Dense(1))
model.add(Activation('sigmoid'))

model.compile(loss='binary_crossentropy',
              optimizer='rmsprop',
              metrics=['accuracy'])

# this is the augmentation configuration we will use for training

train_datagen = ImageDataGenerator(
    rescale=1. / 255,
    shear_range=0.2,
    zoom_range=0.2,
    horizontal_flip=True)

# this is the augmentation configuration we will use for testing:

# only rescaling

test_datagen = ImageDataGenerator(rescale=1. / 255)

train_generator = train_datagen.flow_from_directory(
    train_data_dir,
    target_size=(img_width, img_height),
    batch_size=batch_size,
    class_mode='binary')

validation_generator = test_datagen.flow_from_directory(
    validation_data_dir,
    target_size=(img_width, img_height),
    batch_size=batch_size,
    class_mode='binary')

model.fit_generator(
    train_generator,
    steps_per_epoch=nb_train_samples // batch_size,
    epochs=epochs,
    validation_data=validation_generator,
    validation_steps=nb_validation_samples // batch_size)

model.save('first_try_model.h5')
model.save_weights('first_try_weights.h5')

And here is the loading and predicting part (the problem) of the code:
import keras
import tensorflow
import cv2
import numpy as np

from keras.preprocessing.image import ImageDataGenerator
from keras.models import Sequential
from keras.models import load_model
from keras.layers import Conv2D, MaxPooling2D
from keras.layers import Activation, Dropout, Flatten, Dense
from keras import backend as K

print('keras version:' + keras.__version__)
print('tensorflow version:' + tensorflow.__version__)

model = load_model('first_try_model.h5')

model.compile(loss='binary_crossentropy',
              optimizer='rmsprop',
              metrics=['accuracy'])

im = np.array('C:/data/testvalidate.jpg')
prediction = model.predict(im)
print(prediction)

ValueError                                Traceback (most recent call last)
<ipython-input-36-45b8b72456bb> in <module>()
     40 #im = np.expand_dims(im, axis=0)
     41
---> 42 predo = model.predict(im)
     43
     44 print(predo)

C:\Users\dev\Anaconda3\lib\site-packages\keras\models.py in predict(self, x, batch_size, verbose)
    889         if self.model is None:
    890             self.build()
--> 891         return self.model.predict(x, batch_size=batch_size, verbose=verbose)
    892
    893     def predict_on_batch(self, x):

C:\Users\dev\Anaconda3\lib\site-packages\keras\engine\training.py in predict(self, x, batch_size, verbose)
   1552         x = _standardize_input_data(x, self._feed_input_names,
   1553                                     self._feed_input_shapes,
-> 1554                                     check_batch_axis=False)
   1555         if self.stateful:
   1556             if x[0].shape[0] > batch_size and x[0].shape[0] % batch_size != 0:

C:\Users\dev\Anaconda3\lib\site-packages\keras\engine\training.py in _standardize_input_data(data, names, shapes, check_batch_axis, exception_prefix)
    119                                  ' to have ' + str(len(shapes[i])) +
    120                                  ' dimensions, but got array with shape ' +
--> 121                                  str(array.shape))
    122             for j, (dim, ref_dim) in enumerate(zip(array.shape, shapes[i])):
    123                 if not j and not check_batch_axis:

ValueError: Error when checking : expected conv2d_1_input to have 4 dimensions, but got array with shape ()

im = np.array('C:/data/testvalidate.jpg')
This isn't loading your image RBG values into a numpy array, which is what you need to do. It's just creating an array with that single string value in it:
>>> import numpy as np
>>> a = np.array('file.jpg')
>>> print a
file.jpg

You need to load the image and convert it to a numpy array the same dimensions as your training data (3, 150, 150). There's an example of this in the blog post you linked:
from keras.preprocessing.image import img_to_array, load_img

img = load_img('data/train/cats/cat.0.jpg')  # this is a PIL image
x = img_to_array(img)  # this is a Numpy array with shape (3, 150, 150)

Thank you very much @benfreeston , problem solved!
Here is the corrected code: I had to add another dimension to the Numpy array once the image was converted, I guess this dimension corresponds to the batch_size, which is not relevant here as I have only one image in input? (Please tell me if I am wrong)
img = load_img('C:/data/testvalidate.jpg') # this is a PIL image
print(img)

array = img_to_array(img)

arrayresized = cv2.resize(array, (150,150))
print(arrayresized) # this is a Numpy array with shape (3, 150, 150)

inputarray = arrayresized[np.newaxis,...] # dimension added to fit input size

prediction = model.predict_proba(inputarray)

print(prediction)


LIbrary causes incorrect Temperature Readings for the Si7021 Temperature Sensor.

Line 116 and line 124 of the .cpp file uses a constant of 175.25. However; I t should use a constant of 175.72 per the Si7021 data sheet.  When this is corrected the Si7021 sensor will give correct temperature readings.

This has been updated. See page 22 of the data sheet for information.
https://cdn.sparkfun.com/datasheets/Sensors/Weather/Si7021.pdf


cleos native contract support

All native system contract features (e.g. actions) should have native cleos support:

create producer
vote for producer
stake/unstake
etc

blocked by pull request #2543

I checked the subcommand one by one and here is the result

newaccount - Pass
regproducer - Pass
unregprod - Pass
voteproducer - Pass
listproducers - Pass
delegatebw - Pass
undelegatebw - Pass
buyram - Pass
sellram - Pass
claimrewards - Pass
regproxy - Fail
unregproxy - Fail
canceldelay - Fail

For regproxy, the thrown error message is

I think it's because we miss the "isproxy" field
For unregproxy, the thrown error message is

I think it's because we don't have unregproxy action inside eosio.system.abi
For canceldelay, the thrown error message is

I think it's because cleos specifies the argument as cancelling_auth while eosio_contract_abi.cpp specifies the abi as canceling_auth (single l) instead

Update:
After I pull the latest one, seems regproducer is also throwing an error

I think it's because there is a new field added to regproducer in eosio.system.abi

PR #3202 added to fix the above errors. I'm going to remove the needs testing - fail label

PR 3202 is fixed and tested


fixes #555 str.slice is not a function

Published as v3.12.2. Now all tests are passing :) Yeah!
It's seems like Meteor 1.3 doesn't like the spread operator.

Woo! Great to hear - so is the regular ssr branch compatible with modules now, or do we still need to use the modules branch?

Yes. It's compatible now.
See: https://github.com/kadirahq/flow-router/issues/552


Store avatars in slug directory rather than user_id directory

Avatars are currently stored in a directory equal to the user_id of the User that owns the Avatar. They should be stored in a directory equal to the slug of the User

It was decided that storing them in a directory named for the user_id is necessary, because the directory name must be constant relative to the user or else avatars break.


Fix typo in avro.md

I found typo in avro.md.
So, I fixed it.
Please confirm this PR.
Thank you.

could you please fill CLA http://druid.io/community/cla.html ?

@b-slim I done it.
See: https://github.com/druid-io/druid/pull/4273

Thanks @b-slim!!


[WIP] Add MapToken database migrations, api endpoints

Overview
This commit adds endpoints to manage map tokens. After this commit
tokens can be created/updated/deleted via API endpoints and should open
up the possibility for wiring up some of the UI components.
Checklist
~- [ ] Styleguide updated, if necessary~

[x] Swagger specification updated, if necessary
[x] Symlinks from new migrations present or corrected for any new migrations

Demo

Testing Instructions

Working on these ...

Closes #1203 and #1204

@notthatbreezy, thanks for your PR! By analyzing the history of the files in this pull request, we identified @Lknechtli, @jisantuc and @ddohler to be potential reviewers.


feat(package:async): migrate the Result class to sealed class

By changing the result class from an abstract class to a sealed class,
it will be possible for users to perform exhaustive checks.
#364

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

It's not a bad idea, but it is also potentially breaking.
I'd do it as part of a larger change that adds class modifiers everywhere where they belong in the package, which may require a major version increment.
(Would probably also make the ValueResult and ErrorResult classes final at that point.)


Add support for Python packages with epoch version

@kohder thanks for PR #4891!
While I was reviewing that PR I noticed we would also need to add epoch support to the requirement_parser, so I went ahead and did that in this branch on top of your changes. I also made a few simplifications to the code and brought in more thorough tests like those in the nuget spec.
Let me know what you think, and I will also get another @dependabot/reviewers to take a look at these changes as well!
Fixes #3165

Thanks to @kohder for the initial fix!


Translate May 1 covid report into Portuguese

Hello @nextstrain/portuguese!
The situation report from 2020-05-01 is ready to be translated to Portuguese. The report focuses on Central & Eastern Europe. As per previous week's, please translate the Twitter post below! It's still a bummer when the translation is great, but people do not know it because I used Google translate for the Twitter post (lol).
Branch to translate on: sit-rep_pt
File to translate: https://github.com/nextstrain/translations/blob/sit-rep_pt/narratives/translations_ncov_sit-rep_pt_2020-05-01.md You can translate directly in this file, but please remember to push often as other people may be working on the translation!
Link to preview: https://nextstrain.org/community/narratives/nextstrain/translations@sit-rep_pt/ncov/sit-rep/pt/2020-05-01
Twitter post to translate:
“In the latest #COVID19 situation report, we explore outbreaks in Central & Eastern Europe. The report is now available in Portuguese. Thank you @, @, and @ for the rapid translation."
Logistics
Please update the translations board as you work on the translation: https://github.com/orgs/nextstrain/projects/7
Please assign yourself to the translation if you’re working on it this week.
When a translation is ready for publication -- at minimum meaning two people have worked on it and both agree that it’s ready -- please create a pull request to merge to master and tag me and the issue in it!
Discussion board for your team is here: https://github.com/orgs/nextstrain/teams/portuguese
Standard rules for translation:
Note that the file to translate is in Markdown. In general, it is important not to change any formatting or links. (Although it is okay to update Wikipedia or CDC links to the appropriate language version.) Comments to help with translation are noted in comment symbols like <! -- Comment -->.
Please add your name under translators: with new line, tab, and dash, formatted just like authors are. Please also add a link to your professional page (can use Twitter) under translatorLinks:.
Note: You may know of other great resources available in Portuguese; please don’t add them to the situation report. We need to keep content the same across all translations, so we can be confident in the accuracy of what we publish. Thanks for understanding.
At minimum, I require two sets of eyes on every translation; one of those people should have a background in phylogenetics. Although if you have more, that’s fine too.
Feel free to use a live, online editor like hackmd (sign in with GitHub) to translate together.
Thank you so very much! As always, it's an honor work with you all to make these analyses accessible. Take care.

Working on it. @cassiawag

My fellow Brazilians @andersonbrito @amadotalita , the narrative is ready for proofreading. Feel free to step in and take a deep look before publishing it. :smiley:

I got it! ;)

I forgot to translate the tweet (@cassiawag). Here it goes:
"No mais recente relatório de situação do #COVID19, falamos sobre os surtos na Europa Central e no Leste Europeu. O relatório já está disponível em Português. Obrigado @glauciomarcos e @amadotalita pela rápida tradução."

@cassiawag I just reviewed the document. For me is ready to go :)

Thanks so much @7glaucio & @amadotalita. It's published and tweeted: https://nextstrain.org/narratives/ncov/sit-rep/pt/2020-05-01?f_region=Europe
Nice work!


fix: fix error during build stage

In cases where there is no kubecontext set (automatic builds) all make commands fail as the cluster variable is defined globally. In any case, no make command should fail in case the cluster variable cannot be discovered. This PR fixes this issue.

Codecov Report

Merging #110 (187188d) into develop (165f518) will increase coverage by 2.37%.
The diff coverage is 94.61%.

@@             Coverage Diff             @@
##           develop     #110      +/-   ##
===========================================
+ Coverage    93.52%   95.89%   +2.37%
===========================================
  Files           15       16       +1
  Lines          633      804     +171
===========================================
+ Hits           592      771     +179
+ Misses          41       33       -8

Impacted Files
Coverage Δ

connaisseur/crypto.py
100.00% <ø> (ø)

connaisseur/util.py
72.72% <50.00%> (-27.28%)
:arrow_down:

connaisseur/notary_api.py
95.65% <55.55%> (-4.35%)
:arrow_down:

connaisseur/flask_server.py
96.82% <91.30%> (+25.39%)
:arrow_up:

connaisseur/validate.py
94.31% <93.47%> (-2.35%)
:arrow_down:

connaisseur/alert.py
100.00% <100.00%> (ø)

connaisseur/exceptions.py
100.00% <100.00%> (ø)

connaisseur/key_store.py
90.32% <100.00%> (ø)

connaisseur/mutate.py
100.00% <100.00%> (+5.55%)
:arrow_up:

connaisseur/policy.py
93.50% <100.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


Upgrade XGBoost to v2.0.2

/kind feature
Describe the solution you'd like
We should upgrade XGBoost to bring all the improvements in recent release: https://github.com/dmlc/xgboost/releases/tag/v2.0.2

This will be fixed by https://github.com/kserve/kserve/pull/3309

/assign @terrytangyuan


[docs] Migration to pigment and NextJS integration docs are conflicting

Related page
https://mui.com/material-ui/migration/migrating-to-pigment-css/#configuring-the-theme
Kind of issue
Missing information
Issue description
NextJS integration documentation tells us to create a theme and override Typography's fontFamily with it
[https://mui.com/material-ui/migration/migrating-to-pigment-css/#configuring-the-theme]
const theme = createTheme({
  typography: {
    fontFamily: roboto.style.fontFamily,
  },
});

While in Pigment migration docs it doesn't mention the need of changing it to be like this:
 import type { Metadata } from 'next';
 import { Inter } from 'next/font/google';

+import<EMAIL_ADDRESS>
 export default function RootLayout(props) {
   return (
     <html lang="en">
       <body className={`${inter.className}`}>
       {/* this one here ^^^^^^^^^^^^^^^^^^ */}
         {props.children}
       </body>
     </html>
   );
 }

I was under impression that the first one was the only way of combining next/font and MUI, so I went by pigment migration docs and imported Roboto from next/font directly into next.config.mjs:
export const roboto = Roboto({
  weight: ['300', '400', '500', '700', '900'],
  display: 'swap',
  subsets: ['latin', 'latin-ext', 'cyrillic', 'cyrillic-ext'],
});

/**
 * @type {import('@pigment-css/nextjs-plugin').PigmentOptions}
 */
const pigmentConfig = {
  transformLibraries: ['@mui/material'],
  theme: createTheme({
    typography: {
      fontFamily: roboto.style.fontFamily,
    },

Obviously it didn't work
Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import '/home/goodwin/flaut/flaut-nextjs/node_modules/next/font/google' is not supported resolving ES modules imported from /home/goodwin/flaut/flaut-nextjs/next.config.mjs
Did you mean to import "next/font/google/index.js"?
    at finalizeResolution (node:internal/modules/esm/resolve:259:11)
    at moduleResolve (node:internal/modules/esm/resolve:933:10)
    at defaultResolve (node:internal/modules/esm/resolve:1157:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:390:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:359:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:234:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:87:39)
    at link (node:internal/modules/esm/module_job:86:36) {
  code: 'ERR_UNSUPPORTED_DIR_IMPORT',
  url: 'file:///home/goodwin/flaut/flaut-nextjs/node_modules/next/font/google'
}

So here's my point. MUI doesn't explicitly tells you which way is preferable and doesn't force you into any of the possible solutions though one of them works with Pigment and the other one works by default. Is there a difference and how would it affect our apps if there is. I believe it would be better if docs were consistent and advised users to use next/font via passing class name into body tag. Or if that's only possible with Pigment, then Pigment migration docs should also highlight this change in the code snippet. Thank you.
Context
No response
Search keywords: pigment, nextjs

Hey, @abriginets! We will add a section to the Pigment CSS migration guide to cover this: https://github.com/mui/material-ui/pull/43631/files#diff-603580bb6644e16532b7ca1d7356ab1aa73aa8f9fa68a13664a6406b792db4e5
If you have some time, may I ask you to try this and confirm that is solves your issue? Thanks in advance.

Hello, @DiegoAndai. I tried following steps from updated guide and everything seems to be easy and straighforward, no errors in the terminal now.


After a few tries, stuck ad reading...

Hello,
After a few tries, I got stuck at reading hetzner_ccm, kured and hetzner_csi.
I may have been ratelimited. Is it like that? I've tried deploying maybe 3-5 times.

@renich I have never seen this issue, please share your full kube.tf, just strip it of comments and sensitive values please.

Hm, if you did many tests it could be that you're running into rate limits by GitHub... I did a small test with running terraform plan twice, with this result:
❯ curl \
  -H "Accept: application/vnd.github+json" \
  https://api.github.com/rate_limit
{
  "resources": {
    "core": {
      "limit": 60,
      "remaining": 45,
      "reset":<PHONE_NUMBER>,
      "used": 15,
      "resource": "core"
    },
    "graphql": {
      "limit": 0,
      "remaining": 0,
      "reset":<PHONE_NUMBER>,
      "used": 0,
      "resource": "graphql"
    },
    "integration_manifest": {
      "limit": 5000,
      "remaining": 5000,
      "reset":<PHONE_NUMBER>,
      "used": 0,
      "resource": "integration_manifest"
    },
    "search": {
      "limit": 10,
      "remaining": 10,
      "reset":<PHONE_NUMBER>,
      "used": 0,
      "resource": "search"
    }
  },
  "rate": {
    "limit": 60,
    "remaining": 45,
    "reset":<PHONE_NUMBER>,
    "used": 15,
    "resource": "core"
  }
}

For unauthenticated requests the limit seems to be 60 requests / hour...
Maybe you can verify this by waiting an hour and retry then?

kube.tf.txt

FWIW, it did work after like 20 minutes (and re-trying).

@renich @ifeulner You are right, it's the github limit! Glad it ended up working for you, next time you can just use the provided variables to set the version of those two packages, that way, it completely bypasses the need to hit the Github API.

Thank you. :)


Add comment for unusual ips using in UTs: <IP_ADDRESS>/3

New issues
Tests seem to expect <IP_ADDRESS> to be a valid IP. If this is the case,
consider documenting the requirements to run the tests somewhere obvious
in the source tree. Trivial issue. New RC not required.

change them to <IP_ADDRESS>/2/3?


feat: update schema to support url field for botd result

Caused by https://github.com/fingerprintjs/fingerprint-pro-server-api-openapi/issues/19

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: optimize api 'RunWithOpts' and disable some output with '--no_sum_check'

…um_check'

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

kpm api
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[x] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

31 of 33 (93.94%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.1%) to 39.573%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/opt/opt.go
10
12
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
1.1%

Covered Lines:
1482

Relevant Lines:
3745

💛  - Coveralls