get types can match incorrectly at times

The type overloading for get, which allows both the primary index type and an object with filter keys, can lead to cases where an index type is attempted to be used incorrectly. In particular, because of TypeScript's design, {[key:string]:any} effectively matches any object, including arrays.
Here are the overloads in question:
https://github.com/dfahlander/Dexie.js/blob/b6a1236b17324fc53fe8440b3f3de50f7347916a/src/public/types/table.d.ts#L19
https://github.com/dfahlander/Dexie.js/blob/b6a1236b17324fc53fe8440b3f3de50f7347916a/src/public/types/table.d.ts#L21
Here's an example of where this falls over:
arrayIndexTable: Dexie.Table<{ firstName: string, lastName: string }, number> // primary index is set to `number`

// initialized as:
//         "&[firstName+lastName],firstName,lastName",

// TypeScript is a-okay with this!
const existing = arrayIndexTable.get(["Shadow", "Fiend"])

Ideally, the above would fail at the type level before hitting a confusing lack of results. Unfortunately, it doesn't, because an array matches {[key:string]:any}. An easy fix for the particular problem is:
get(equalityCriterias: {[key:string]:unknown}): PromiseExtended<T | undefined>;

However, I don't have the depth here to understand if this would yield too much pain internally to the implementations of these functions, but I believe it's effectively saying the same thing while disallowing incompatible types like arrays.

Thanks. I will look into it. The internal implementation wouldn't be harmed by changing the type - so it would just be to change it and verify that it still compiles.
Side note: "&[firstName+lastName],firstName,lastName" would actually mean that the primary key is an array of the two items [firstName, lastName]. Adding a leading comma would make the primary key outbound and allow a non-property as key.

Got it! Making the array the primary key was my goal with the example here---with the type of the primary key set to number despite being a [string, string]. Not sure if I messed that up though? 😅


Examples and readme rework

I've moved examples into a separate repository and reworked the README.md to provide a better experience for new users. It relies on hover to do the heavy work. The original instructions are moved to a wiki page, which is also linked from the README.md.

@GeertJohan awesome !


tailwindcss lsp in astro file

how to run tailwindcss lsp in astro file?

A new version of LSP-tailwindcss is released, it will be available soon on Package Control.


Notmuch -- make +afew feature and  fix +notmuch/update buffer kill

Addressing #3188
Using compile approach similar to doom sync. It runs in a buffer and
deletes the buffer when complete.
afew is and should be optional. Putting it behind a feature flag so
that if you don't have afew installed, it won't be run. It's easily
available for those that want it, and easy to ignore for everyone else.
added documentation.

Some more work was needed to complete notmuch's readme, which I've collect into 494d87d. Thanks for your help and bringing it to my attention!


fix: check if tsconfig file exist

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug Report] Mac, CMD+Q makes app hang

To create a microtask in dispose in which we dispose of player and controller is not sufficient. To do it this way makes the app stop responding when closed. CMD+Q triggers this every time, exit triggers this sometimes, I guess that sometimes dispose has enough time to run. I have not found a way to resolve this issue.

I can't reproduce, either with my app or media_kit_test, if I use cmd + q while a video is playing the app closes without any problem, in release or dev mode.
Can you share a minimal example?

Wait, sorry, I can reproduce with media_kit_test, it happens when the video playback is finished. I'm on it.

As suggested by @alexmercerind adding media_kit_native_event_loop fixes this issue.

Let's close this one as there's already a recommended solution.

Thanks,
Maybe this should be stated in the README.
media_kit_native_event_loop has MINIMUM_TARGET=11.0, can this be lowered?

@alexmercerind I will check if I can fix this bug without media_kit_native_event_loop. This may lead to more issues in the future.

@alexmercerind The following patch fixes this issue without using media_kit_native_event_loop, this comes from the timeout of mpv_wait_event being infinite in release mode.
Using the same timeout in dev and release mode seems to have few impact on the cpu when the player is idle, about +0.5% on my old Intel cpu.
Since the use of media_kit_native_event_loop is recommended, I think this overhead is negligible and will prevent this kind of issue from occurring again.
diff --git a/media_kit/lib/src/libmpv/core/initializer_isolate.dart b/media_kit/lib/src/libmpv/core/initializer_isolate.dart
index 1ad4fbb..d9a1208 100644
--- a/media_kit/lib/src/libmpv/core/initializer_isolate.dart
+++ b/media_kit/lib/src/libmpv/core/initializer_isolate.dart
@@ -90,9 +90,6 @@ abstract class InitializerIsolate {
     }
   }

-  /// Detect if app running in release mode.
-  static const _isReleaseMode = bool.fromEnvironment('dart.vm.product');
-
   /// Runs on separate isolate.
   /// Calls [MPV.mpv_create] & [MPV.mpv_initialize] to create a new [mpv_handle].
   /// Uses [MPV.mpv_wait_event] to wait for the next event & notifies through the passed [SendPort] as the argument.
@@ -161,7 +158,7 @@ abstract class InitializerIsolate {
     // Ensuring the successful sending of the last event before moving to next [MPV.mpv_wait_event].
     while (true) {
       completer = Completer();
-      final event = mpv.mpv_wait_event(handle, _isReleaseMode ? -1 : 0.1);
+      final event = mpv.mpv_wait_event(handle, 0.1);
       // Sending raw address of [mpv_event].
       port.send(event.address);
       // Ensuring that the last [mpv_event] (which is at the same address) is NOT reset to [mpv_event_id.MPV_EVENT_NONE] after next [MPV.mpv_wait_event] in the loop.

I published few updates. The media_kit_native_event_loop is also updated.
Your code may require few changes; the API has been improved. I guarantee this is last breaking change. Apologies.


Multi arch builds fail

When trying multi-arch build, it's now failing, unsure on why?
Drone pipeline:
kind: pipeline
type: docker
name: deploy

platform:
  os: linux
  arch: amd64

steps:
- name: latest
  image: thegeeklab/drone-docker-buildx
  privileged: true
  settings:
    repo: modem7/dhcprelay
    purge: true
    compress: true
    cache_from: "modem7/dhcprelay:latest"
    platforms: linux/amd64,linux/arm/v6,linux/arm/v7,linux/arm64/v8
    username:
      from_secret: docker_username
    password:
      from_secret: docker_password
    tags: latest

Error:
https://drone.modem7.com/modem7/DHCP-Relay/19
The last time this worked was about 12 days ago (https://drone.modem7.com/modem7/DHCP-Relay/10/)
It could be an ID10T issue, so apologies if so, still quite new to drone.
The build works with just linux/amd64 selected.

Thanks for your report. Looks like a QEMU emulator issue to me, see https://github.com/docker/buildx/issues/493. But I'm also just guessing I have never seen this error before, sorry.

Ahhah,
What I'll do is post on the buildx git if you think that might be the case, and will comment back here with relevant issue link

OK, I seem to have resolved it.
Need to run "docker run --privileged --rm tonistiigi/binfmt --install all" for everything to work.
Info from https://github.com/docker/buildx#building-multi-platform-images

Glad to hear it works now, but I'm a bit confused how it was able to work "about 12 days ago" without a proper installed qemu emulator :thinking:

I suspect I had already done it (I remember doing it before), but docker
recently updated on the system, so there's a big chance that it needs to be
done each update if that's the case
On Sun, 15 Aug 2021, 12:22 Robert Kaussow, @.***> wrote:

Glad to hear it works now, but I'm a bit confused how it was able to work
"about 12 days ago" without a proper installed qemu emulator 🤔
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/thegeeklab/drone-docker-buildx/issues/41#issuecomment-899035500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBGACUW7TMDLOOJ6KGDBOTT46PQ7ANCNFSM5CCTCSHQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Could not parse RFC3339 datestring

File "/usr/local/google/home/gowoon/.local/lib/python2.7/site-packages/dsub/providers/google_base.py", line 359, in parse_rfc3339_utc_string
rfc3339_utc_string, e)
AssertionError: Could not parse RFC3339 datestring: 2019-10-08T12:11:24.999999594Z exception: microsecond must be in 0..999999
As said in the error, parse_rfc3339_utc_string() cannot parse '999999594' in the string '2019-10-08T12:11:24.999999594Z'

Hi @gowoonc , thanks for the report.
I'm a bit surprised by this error, as parse_rfc3339_utc_string explicitly handles this case.
Can you tell me what version of pytz you have installed? You can find it by running pip freeze | grep pytz

Could you also clarify what version of dsub you are running? dsub --version

It's pytz==2019.2, dsub version: 0.3.3

Thank you @gowoonc , I believe I have found the cause of the issue. We do a division that has different behavior between python2 and python3. I believe this is causing an issue with Python3.
Can you please confirm that you are using Python3?
I will be pushing out a fix that will be available in the next release.

This is fixed in 0.3.5


How to restrict the review to special user?

In enterprise environment with a team developing its own app around a LUIS App, you don't want anybody to be able to edit the productive LUIS app, so you need a special tenant for reviewing the utterance.
How shall we deal with that challenge?
Currently, the web interface luis.ai only offers 'Admin' and 'Collaborators' roles, without restriction.
We were thinking about exporting part of the app (via LUIS v2 Api or botbuilder-tools) and process the review externally and then reimport some part but it looks quite complicated and we then can't benefit from the nice integration of the web interface shown in this article.
More generally, using a LUIS App in enterprise environment where you separate DEV, TEST and PROD is quite challenging…

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

ID: dcf64d58-528f-f633-374b-040590148aeb
Version Independent ID: d48055e5-e1f7-43d7-f2ae-65358329aa30
Content: Label suggested utterances with LUIS
Content Source: articles/cognitive-services/LUIS/luis-how-to-review-endoint-utt.md
Service: cognitive-services
GitHub Login: @diberry
Microsoft Alias: diberry

Hi @ericbl, thank you for taking the time to ask this question of how to manage a LUIS app in the development process. If I understand you correctly, you would like to separate out parts of the model for certain people to work on while securing part of the model so that part can't be changed. Is that correct?
LUIS doesn't do this easily yet. Have you looked at breaking your app up to use the dispatch model? That would be separate LUIS apps controlled by a single parent LUIS app. If that doesn't meet your needs, can you elaborate on the issues so I can continue to investigate a solution.
Thank you,
Dina
#in-progress.

Thanks for the quick reply.
Yes, we would like to have one team member working on the review, but without the risk that he can edit  the rest of the model.
A sub kind of Collaborator role with very restricted right would be the easiest way.
We are also evaluating the Dispatch Tool, but we don't see it as an answer for the current issue, since the reviewer would still be able to damage a sub part of the model and possibly cause the all application to crash.
Is there any plan to bring the review process on a Enterprise level?
What do you think about the above suggestion with "Reviewer" role?
Is there a Roadmap about the next features in LUIS?
Similarly, we wish also more advanced concepts for the productive deployment of a LUIS App, e.g. be able to link our model to our source control; separate more the development, testing and productive environments maybe creating 3 different app, although the reviewed utterances on Prod should be merged back to Dev!

Hi @ericbl, Thank you for elaborating on your needs. An internal documentation ticket has been created for both issues (enterprise level roles & development pipeline). You should be notified when each is completed. There is no current guidance so please be patient.


[DOCS] missing crun instructions in https://wasmedge.org/docs/develop/deploy/podman/#prerequisites

Description
Summary
Step 3 of https://wasmedge.org/docs/develop/deploy/podman/#prerequisites: It seems like some instructions are missing.
Build and configure crun with WasmEdge support
Next, configure and build a crun binary with WasmEdge support.
Maybe there should be a git clone, similar to https://wasmedge.org/docs/develop/deploy/oci-runtime/crun/#prerequisites?
Issue refers to:
https://github.com/WasmEdge/WasmEdge/issues/3028
Screenshots
No response
Additional information
No response

I'll work on this


Refactoring: put as many objects as possible in abstraction

GitHub can't show diffs on the main AMXD file, because it is binary. Create a new abstraction "main" that contains as many objects from the AMXD file as possible, so that changes can be properly reviewed in pull requests.

Included in version v1.3


refactor(validations): Add URL-based Routing to Dataset Validations Tab

Summary
In this PR, we introduce URL-based routing to our dataset validations tab. This enables us to directly link to assertions, tests, and any tabs that are added in the future.
To do so, we have to refactor the validations tab, and also change how we are determining the currently selected EntityTab.
Status
Ready for review
Screenshots

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@jjoyce0510 There are failing tests here

Fixed failing tests. Let's wait til green!

Failing test unrelated to this change. Merging.


Create the Login functionality for the project.

Like every other application, this also requires login functionality. The basic process of this functionality for a REST API is to first check the user credentials and return an error response if they are wrong. On successful verification of the user's credentials, the server should return a secret (JWT token in this case) to the User. The same secret will same be used in the future to verify which user is raising a request to the server.
The Task
The Login functionality will be written in a function in the file controllers > auth.controller.js. The function attributes are the default attributes of a request handling function, i.e., (req, res).
Request Parameters:
{
  "email"<EMAIL_ADDRESS>  "password": "mY-pAsSwOrD",
  "role": "Student"
}

Response Data:
{
  "name": "User Name",
  "email"<EMAIL_ADDRESS>  "role": "Student",
  "secret": "JWT Token"
}

For checking the password, you will have to first convert the password into hash, the hash_password function present in the utils > password.js should help in converting password into hash. Now you will have to match this hash to the password stored in the database.
Once the user is successfully verified, you will have to create a User Secret, which is actually a JWT token. For creating this JWT token, you have to use the getJwt function present in the utils > password.js
To make returning a response easier, some pre-defined response functions according to the status code of the response are written in utils > responseCodes.js. It is recommended to use them for returning successful or error response.
Make sure to pull before starting your work.
For any doubts or difficulty, you can ping me on discord, I'll try to help you out as soon as possible.
All the Best!!

claim

claim

you need to register before claiming the task.

claim

Assigned to @priyankasharma2823, congratulations!!
You have 48 hours to complete this task.


found a integer overflow leads to stack_overflow

in parser.c
k seems like the size of the rest of buffer on stack(var 'buf' which type is char[]).
n is memcpy size as 3rd  para.
while condition k>0 to keep buffer is not full.
if enter in if(k<n) branch n=k,var n is assigned the value of k
then k -=n ,var k is assigned the value 0
then --k, var k is assigned the value 0xffffffffffffffff while var k's type is size_t.
so the while conditionk > 0 not make sense.
next memcpy will lead to stack overflow
void error_ref_sym(fb_parser_t *P, fb_ref_t *ref, const char *msg, fb_symbol_t *s2)
{
    fb_ref_t *p;
    char buf[FLATCC_MAX_IDENT_SHOW + 1];
    size_t k = FLATCC_MAX_IDENT_SHOW;
    size_t n = 0;
    size_t n0 = 0;
    p = ref;
    while (p && k > 0) {
        n = (size_t)p->ident->len;
        if (k < n) {
            n = k;
        }
        memcpy(buf + n0, p->ident->text, n);
        k -= n;
        n0 += n;
        buf[n0] = '.';
        --k; //overflow here when k = 0
        ++n0;
        p = p->link;
    }
    buf[n0] = '\0';
    if (n0 > 0) {
        --n0;
    }
    if (k <= 0) {
        memcpy(buf + FLATCC_MAX_IDENT_SHOW + 1 - 4, "...\0", 4);
        n0 = FLATCC_MAX_IDENT_SHOW;
    }
    error_report(P, ref->ident, msg, s2 ? s2->ident : 0, buf, n0);
}

poc:
➜  fuzz_test cat test9
struct tsj{
        damage:shortllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllllve.tlllllllllllllllllllllllllllllllllllllll;
}
➜  fuzz_test ../flatcc/bin/flatcc test9
test9:2:9: error: 'shortlllllllllllllllllllllllllllllllllllllllllllll.tlllllllllllllllllllllllllllllllllllllll': unknown type reference used with struct field: test9:2:2: 'damage'
*** stack smashing detected ***: terminated
[1]    3685 abort (core dumped)  ../flatcc/bin/flatcc test9
➜  fuzz_test

➜  fuzz_test gdb ../flatcc/bin/flatcc
GNU gdb (Ubuntu 9.2-0ubuntu1~20.04) 9.2
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word"...
pwndbg: loaded 196 commands. Type pwndbg [filter] for a list.
pwndbg: created $rebase, $ida gdb functions (can be used with print/break)
Reading symbols from ../flatcc/bin/flatcc...
pwndbg> set args test9
pwndbg> r
Starting program: /mnt/c/Users/tsj/Desktop/flatcc/bin/flatcc test9
test9:2:9: error: 'shortlllllllllllllllllllllllllllllllllllllllllllll.tlllllllllllllllllllllllllllllllllllllll': unknown type reference used with struct field: test9:2:2: 'damage'
*** stack smashing detected ***: terminated

Program received signal SIGABRT, Aborted.
__GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
50      ../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
 I wonder if --k has any purpose at all, even without hitting overflow.

I think there are more problems. The assignment of '.' might be outside the buffer. --k accounts for the ..

yes, i agree --k accounts for the .

Please review last commit on master: 8a90c92

BTW: it is rather strange that the compiler did not warn about this.

it seems not solve memcpy's copy  overflow
actually the stack overflow happens on the next loop after truncated
in my opinion
just mod   while (p && k>0) to
while(p && k>0 && k<0x8000000000000000

compiler did not warn because it don't know the value of  n0, n on the runtime
memcpy(buf + n0, p->ident->text, n);

Sorry, but your suggestion is not good because it depends on the integer size.
The following was written before your last EDIT, FYI. So I will close here.
Why do you think an overflow still happens? Does your test still fail?
The invariant is that k is what is left in the buffer.
In the new version, k > 0 on entry. If '.' is added, k could become zero.
However, n will not be larger than k when entering memcpy, so worst case, 0 bytes are copied, which will not result in a buffer overrun. At least as I understand it.
The compiler could have warned that the test <= 0 does not make sense. There are many other places where the compiler warns even if the code actually makes sense.
It's getting late, so I'll continue tomorrow if you have some suggestions.

Thanks for reporting.


Automerge major dependency updates to reduce traffic

It’s important that we automerge major updates as they are the only ones that will be opened in most of our repos because we do not use lockfiles.
Let’s trust our tests to detect any potential breakage.

Checklist

[ ] run npm run test and npm run benchmark
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

cc @Fdawgs @Eomm

I think this change hurt us more than the benefits it takes: mainly the plugins forward some options to external modules.
We had some issues because of that.
Since we are not testing all the external's module options, the CI could be green and merge it.
This means that in some cases we should release the plugin as a major version because the options used by the dev through the fastify's plugin could be changed.
In both cases (with or without the automerge), a fastify maintainer needs to check the external dep's release to decide if it should cause a major or a minor release.
A blocked dependabot's PR is easier to manage than reading the merged PRs and going to check them.
For sure and at least, the dependabot configuration should not call these kinds of PRs as chore.
ref https://github.com/fastify/fastify/pull/3716

Here are some examples:

https://github.com/fastify/fastify-websocket/pull/145
https://github.com/fastify/benchmarks/pull/228
https://github.com/fastify/github-action-merge-dependabot/pull/74

We have more than 50 modules in this or, mostly they do not have lock files and they only receive dependency updates with a semver-major. Dealing with a few  mistakes is significantly less work than manually checking all dependency updates across the full org.
Let's make it this way: I am the one that have historically done most of the releases across the org.
This changes increases the time it takes to release a module as it requires more manual steps and wait for CI runs.
I will have to further reduce the amount of releases I can do (I already have, partially because of this).
Are others willing to pick up the work? I would be happy to let other take the load. I get upset when others increase my workload without giving me more help.

Dealing with a few mistakes is significantly less work than manually checking all dependency updates across the full org.

Indeed, I think my approach relies less on the final users (that should open an issue) and moves the burden to the maintainer that publishes the release.

Are others willing to pick up the work?

Yes, I'm. I think to have done it a few times already but I can do more for sure.
In any case, I would like to hear some others @fastify/plugins opinions.

Indeed, I think my approach relies less on the final users (that should open an issue) and moves the burden to the maintainer that publishes the release.

Exactly. My goal is to push as much work as possible on the users. They are not paying for it, doing QA and opening an issue is the minimum they can do.

Sounds good! I agree, users should be testing updates to plugins in their applications/services using CI themselves.
If a dependency breaks a plugin, and it wasn't caught by our own CI, then hopefully users will report this back if it breaks their apps.

I would propose a parameter instead of removing the target option.
I agree with you at some point, but I would like to define this kind of usage:

plugin that does not forward configuration to dependencies: target: major
plugin that does not forward configuration to dependencies AND have a low download rate: target: major
plugin that forward configuration to dependencies AND have a high download rate target: minor

Adding the parameter would let us apply this setup in both ways reducing the burden.
Notice that this workflow is used by 2 plugins only, I was planning to spread it during the fastify@v4 upgrade

high download rate = 10k/week

How about setting the exclude  settings of the automerge action? We can just skip the critical dependencies whose options are exposed to the end users.

Notice that this workflow is used by 2 plugins only, I was planning to spread it during the fastify@v4 upgrade

I have been planning on doing it, was just waiting on a general agreement on script naming in https://github.com/fastify/workflows/issues/9.

@Fdawgs do you think we could add a workflow input to pass exclude to the automerge action?

@Fdawgs do you think we could add a workflow input to pass exclude to the automerge action?

Sure! Should have time to add it this evening.


Enh/adamji/td 19768 support reqid

Codecov Report

Merging #147 (9bab41f) into main (f41cc60) will increase coverage by 1.08%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
+ Coverage   74.67%   75.76%   +1.08%
==========================================
  Files          19       20       +1
  Lines        2219     2335     +116
==========================================
+ Hits         1657     1769     +112
- Misses        562      566       +4

Impacted Files
Coverage Δ

taos/__init__.py
100.00% <ø> (ø)

taos/connection.py
88.67% <33.33%> (ø)

taos/cinterface.py
72.53% <90.90%> (+1.46%)
:arrow_up:

taos/utils.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove DataEntry class

As discussed in #159 , the DataEntry class and DataCatalogEntry can be removed since are replaced by Asset and DataAddress respectively.
This should be done after #159 gets merged

Totatlly agree with doing this, though I highly recommend doing this when there are only few other PRs open as the potential for a "merge hell" is quite high due to the widespread use of DataEntry :)


Docs should say why file-backed maps are unsafe

In contrast to the functions to create anonymous maps, all the file-backed ones are marked unsafe.  However, the docs don't say why this is.
What I can glean from #25 is that while memmap would have one believe that one gets an immutable &[u8] out of a map, in reality the file can change underneath and one may need to take extra measures like file locking and whatnot.
Is this accurate?  If so, I can submit a PR to update the docs to that effect.

The actual story here is quite complex. There was a recent thread on urlo, and as far as I can tell, there is no obvious consensus on how to document this. https://users.rust-lang.org/t/how-unsafe-is-mmap/19635
It would be good to point out this issue in the documentation however, but I don't think we can write anything authoritatively yet.

Thanks, @BurntSushi :smiley:

There are two reasons:

Because you can get multiple maps to the same region in the same file, both mutable and immutable at the same time.
The memory (even the immutable one) is file-backed. If the file changes (even if it changes from outside the process), the memory contents will change with it. Therefore immutable memory may change randomly which is highly unsafe.

#66 landed with improved descriptions of why unsafe is necessary.  If you still feel the documentation is lacking please feel free to open a new issue/PR with a specific proposal.  Thanks!


docs: content enhancement

Find some errors in the book content and use this change to improve them.

@ratankaliani Please take a look and help review.


How to use Access Token in Down Stream API calls?

Blazorade-MSAL has made authenticating super easy - so thank you for creating this great library.
However, I'm wondering what you would suggest in being able to use the Access Token for downstream API requests.
(I am using Blazor Server w/ .net6). My API requests live in a Scoped service and and I seem to loose all session state when I navigate to another page, (and I have to use the "force" flag to even get it to navigate to the page). This leaves me with using ProtectedLocalStorage where I can get the auth information.
I have tried everything outlined ASP.NET Core Blazor authentication and authorization, and then  the Blazor Server specific stuff at Secure ASP.NET Core Blazor Server apps, but these approaches didn't  even work for me when I was using the project template to generate a project using Microsoft Identity, (not using Blazorade-MSAL).
Just wondering if you, or anyone else can suggest the proper technique.
I have studied https://docs.microsoft.com/en-gb/aspnet/core/blazor/state-management?view=aspnetcore-6.0&pivots=server and it seems I that since upon every page navigation that a new session is being created, I have to rely on persisting in browser local storage.
Am I missing something?

I thought I'd update the Blazorade MSAL library and make sure it works with .NET6 and MAUI. I could at the same time add an example of how you can acquire an access token for Microsoft Graph and use that for accessing it.
I guess that would be adequate for your purposes?

@MikaBerglund  yes, I think that would indeed help, and would be greatly appreciated.

or more specifically.. .an update to .net6 and sample that illustrates how we can use <CascadingAuthenticationState> (if this would still make sense when authenticating with Blazorade rather than when using the MS ID scaffolding) to share the authentication token for downstream API requests.
Obviously the technique outlined Pass tokens to a Blazor Server app doesn't work... so what is the alternative when using Blazorade-msal?

The Graph Client sample is now updated to use the new BlazoradeRequestFactory that I've added in #20.
I'll close the issue for now, but feel free to open it should you have any further questions in the subject.


dbrouter shortcut function for model lookup by contenttype

Testing

Ensure you have the legacy db setup with a dump from content
Clone knowledgebase app as a sibling to cfgov-refresh
build frontend assets ./setup.sh
cd back into cfgov-refresh
pip install -e ../knowledgebase
pip install simplejson
./cfgov/manage.py update_index knowledgebase
Enter django admin and select a spanish answer
Then click view on site site and see then site should render

@rosskarchner
@kurtw

Nice 💯

👍


Refactored SVNUtil and moved Exceptions to common package

Refactored SVNUtil and moved Exceptions to common
Type of request

[ ] 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 change)
[x] Refactoring
[ ] Documentation or documentation changes

Related Issue(s)
#19
#16
Concept
Refactoring and code cleanup for SVNUtil class.
Moved Exceptions to a common package org.aposin.mergeprocessor.exception
Estimated investment / approximate investment
1 day.
Initiator
Author : Antariksh
Technical information
How has this Tested?
Unit tests.
Checklist:

[x] All the commits are signed.
[x] My code follows the code style of this project.
[x] I agree with die CLA.
[x] I have read the CONTRIBUTING docs.
[x] I have added/updated necessary documentation (if appropriate).

Due to no response on #37 , I took the commits, squashed them and created a new PR.


Remove Python 3 specific init code

The Python 3 specific code here doesn't seem necessary?

Thanks @tavinathanson - LGTM! Didn't really understand what was going on when I wrote this. Appreciate your help!

Coverage decreased (-0.5%) to 80.0% when pulling f3afc75c60cbde4ad92e0791c0a53dc42cac9433 on remove_excess into 233816fbbcc8c48da0826b903cc41725aed644de on master.

Coverage decreased (-0.5%) to 80.0% when pulling f3afc75c60cbde4ad92e0791c0a53dc42cac9433 on remove_excess into 233816fbbcc8c48da0826b903cc41725aed644de on master.

Coverage decreased (-0.5%) to 80.0% when pulling f3afc75c60cbde4ad92e0791c0a53dc42cac9433 on remove_excess into 233816fbbcc8c48da0826b903cc41725aed644de on master.

Coverage decreased (-0.5%) to 80.0% when pulling f3afc75c60cbde4ad92e0791c0a53dc42cac9433 on remove_excess into 233816fbbcc8c48da0826b903cc41725aed644de on master.


Resulting types of withStyles are different than input types and are missing props

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
Given something along those lines:
import { Link, withStyles } from '@material-ui/core';
const StyledLink = withStyles({ /* ... */ })(Link);

These 2 things should both compile fine:
<Link component="button">...</Link>
<StyledLink component="button">...</StyledLink>

This was working fine in Material UI v3.x, and we started seeing this in v4.x.
Current Behavior 😯
<Link component="button"> compiles fine, but <StyledLink component="button"> does not. withStyles loses some typing information:

This is the compilation error that shows up on the StyledLink component:
ERROR in [at-loader] ./src/index.tsx:20:8
    TS2322: Type '{ children: string; component: string; }' is not assignable to type 'IntrinsicAttributes & Pick<DefaultComponentProps<{ props: AnchorHTMLAttributes<HTMLAnchorElement> & Pick<TypographyProps, "color" | "display" | "hidden" | "style" | "title" | "ref" | "classes" | ... 255 more ... | "variantMapping"> & { ...; }; defaultComponent: "a"; classKey: LinkClassKey; }>, "color" | ... 272 more...'.
  Property 'component' does not exist on type 'IntrinsicAttributes & Pick<DefaultComponentProps<{ props: AnchorHTMLAttributes<HTMLAnchorElement> & Pick<TypographyProps, "color" | "display" | "hidden" | "style" | "title" | "ref" | "classes" | ... 255 more ... | "variantMapping"> & { ...; }; defaultComponent: "a"; classKey: LinkClassKey; }>, "color" | ... 272 more...'.

As a temporary workaround, replacing const StyledLink = withStyles(styles)(Link); with const StyledLink = withStyles(styles)(Link) as typeof Link; lets us compile our components.
Steps to Reproduce 🕹
I built a "minimal" repo to reproduce the issue at https://github.com/astorije/material-v4-withStyles-issue.
Install the dependencies with yarn install, and compile/serve with yarn start. The actual issue can be found here.

Greatly appreciate the detailed error report :+1:
Unfortunately this is a tradeoff we took in order to fully support polymorphic components. While the base component has "better" types the resulting types from higher-order components are worse with regards to polymorphism.
While it's not strictly a duplicate I'll still close this in favor of #15827. The underlying issue is the same.
In the meantime I suggest using const StyledLink = withStyles({ /* ... */ })(Link) as Link; or something similar.

Thanks for the reply, @eps1lon!
That's good to know. We are moving away from withStyles right now, so hopefully we can soonn get rid of these as ... that we have everywhere.


Port str trim

Larger PR due to many test cases. Feedback is welcomed!

I'm going to go ahead and land since I think it's good. Just saw the Arc and wondered if we needed it.


[MI200][Reduction] Weird behavior when using int8 data type

When I'm using miopenInt8 for source tensor and miopenFloat for destination tensor and miopenInt8 as a comptype I get correct result however, when I use miopenInt32 or miopenFloat as comptype I get incorrect result. The only difference between correct and incorrect results is that incorrect result has zeroes instead of negative values.
As far as I understand comptype is an accumulation type, if so then I would expect both miopenInt32 and miopenFloat should work here. Can you please clarify the behavior?

Reproducer (clickable)
#include <functional>
#include <fstream>
#include <numeric>
#include <iostream>
#include <random>

#include "miopen/miopen.h"

#define CHECK_MIOPEN(f, msg) \
    do { \
        const auto ___status___ = f; \
        if (___status___ != miopenStatusSuccess) { \
            throw std::runtime_error(msg); \
        } \
    } while (0)
#define CHECK_HIP(f, msg) \
    do { \
        const auto ___status___ = f; \
        if (___status___ != hipSuccess) { throw std::runtime_error(msg); } \
    } while (0)

#define DTYPE1 int8_t
#define DTYPE2 float

int main() {
    auto status_miopen = miopenStatusSuccess;
    auto status_hip = hipSuccess;

    const int ndims = 2;
    int src_dims[ndims] = {5, 250};
    int src_strides[ndims] = {250, 1};

    int dst_dims[ndims] = {1, 250};
    int dst_strides[ndims] = {250, 1};

    float beta = 0.0f;
    float alpha = 1.0f;

    auto nan_prop_mode = miopenNanPropagation_t::MIOPEN_NOT_PROPAGATE_NAN;
    auto indices_mode
            = miopenReduceTensorIndices_t::MIOPEN_REDUCE_TENSOR_NO_INDICES;
    auto indices_type = miopenIndicesType_t::MIOPEN_32BIT_INDICES;
    auto op_kind = miopenReduceTensorOp_t::MIOPEN_REDUCE_TENSOR_MAX;

    auto data_type1 = miopenInt8;
    auto data_type2 = miopenFloat;
    auto comp_type = miopenInt8; //miopenInt32

    void *workspace = nullptr;
    size_t workspace_size = 0;

    // Create handle
    miopenHandle_t handle {};
    status_miopen = miopenCreate(&handle);
    CHECK_MIOPEN(status_miopen, "Couldn't create handle.");

    // Create Tensor descriptor
    miopenTensorDescriptor_t src_desc;
    status_miopen = miopenCreateTensorDescriptor(&src_desc);
    CHECK_MIOPEN(status_miopen, "Couldn't create src desc.");
    miopenTensorDescriptor_t dst_desc;
    status_miopen = miopenCreateTensorDescriptor(&dst_desc);
    CHECK_MIOPEN(status_miopen, "Couldn't create dst desc.");
    miopenReduceTensorDescriptor_t reduce_desc_;
    status_miopen = miopenCreateReduceTensorDescriptor(&reduce_desc_);
    CHECK_MIOPEN(status_miopen, "Couldn't create reduce desc.");

       But I could not reproduce the issue using our MIOpenDriver since it does not support using different types for in and out tensor

Sorry, I fixed it, can you please try it again?

From the testing, I think the result of using either int8_t or int32_t as DstDataType are wrong, and the result of using float as DstDataType seems correct.
In the original MIOpen development,   only fp16, fp32 and fp64 input data types are supported, and the fact can be checked from our cover testing in files driver/driver.hpp and test/reduce_test.hpp.   MIOpen is mainly for deep learning training, so no strong reason that it should support int8
We have other project which support reduction on all data types of fp64, fp32, fp16, bf16, int8 and int4

Thanks for the clarification. What is the project you are referring to?

Thanks for the clarification. What is the project you are referring to?

We do have project that is not publicly accessible at present.

Ok, so the outcome is that the reduction operation doesn't support int8 data type. Thanks for the answers!


change parallel processing to batch by SCI product

a new parallel processing grouping:
Groups all files into parallel processes by product type if daily products. For by request products it is also grouped by type but additional into batches of a given size.
In L1 processing: also a simple priority heuristic starts with ql data as this takes longest.
takes down:
L0 processing from 48h to 9h
L1 processing from 55min to 42min
L2 processing already implemented but needs measurements and maybe adjustments

Codecov Report

Merging #259 (52d22ef) into master (93879f2) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #259      +/-   ##
==========================================
+ Coverage   80.71%   80.80%   +0.09%
==========================================
  Files          60       60
  Lines        5402     5428      +26
==========================================
+ Hits         4360     4386      +26
  Misses       1042     1042

Impacted Files
Coverage Δ

stixcore/processing/L0toL1.py
71.59% <100.00%> (+5.37%)
:arrow_up:

stixcore/processing/L1toL2.py
68.96% <100.00%> (+1.89%)
:arrow_up:

stixcore/processing/LBtoL0.py
68.51% <100.00%> (+2.18%)
:arrow_up:

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


Need a Zip script for UMD country data

Design script to zip raster tiffs or grid files in Amazon S3
Add script to GFW-Sync
Run from DM server

Fixed in GFW-Sync2

Here?
https://github.com/wri/gfw-sync2/blob/master/raster_layer.py

Yes,
the RasterLayer object inherits from the Layer object.
The archive methods is defined there


Add one more 'need root' phrase

'must run as root' -- used by, at least, add-apt-repository

Thanks!


fail better when node version loaded in browser

so we don't explode when loaded in some weird enviroment

how would this happen though?

see #69 where react-native is loading the index instead of the browser

Why isn't he using require('pbkdf2/browser')?

HE isn't including anything, he's getting crypto, the issue is that that
file runs even if you don't try to use pbkdf2 specifically
On Thu, Jun 22, 2017 at 11:12 AM Daniel Cousens<EMAIL_ADDRESS>wrote:

Why isn't he using require('pbkdf2/browser')?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/crypto-browserify/pbkdf2/pull/71#issuecomment-310410373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABE4n3UIBAoF221Xi-fVXs6o2AEaf_Efks5sGoRrgaJpZM4OA3H8
.

@calvinmetcalf @dcousens
I use some libs and these lib use pbkdf2.  When can this PR be merged?
One more point, in react-native if (crypto.pbkdf2Sync.toString().indexOf('keylen, digest') === -1) {   throw new Error('Unsupported crypto version') }
The 'keylen, digest' will be minified to sth like a,b so it will always throw error.


Update setup.py

Adding description to setup.py

Surprisingly, You are really fast to send PR!! thanks :)


New sample: ShowLabelsOnLayer

New sample for .NET: ShowLabelsOnLayer
This is not the 1st implementation of the sample - meaning the design/layout etc. was previously dictated. The initial implementation was done on Java.

@nCastle1 Can you please do the .NET code review for the various platforms (WPF, UWP, Xamarin.Forms, iOS, Android)?


Adding Different Visual for Optional Commands

Description
Added a placeholder entry for the optional commands.
Linked Items:
Closes #280
Added

Added some placeholder text to be displayed when an optional command is presented.

Checklist:

[x] I updated project documentation to reflect my changes.
[ ] My changes generate no new warnings.
[x] I updated new and existing unit tests to account for my changes.
[x] I linked the associated item(s) to be closed.
[x] I bumped the version.
[x] I added the labels corresponding to my changes.

Still need to get the new screenshot, but GitPod is being difficult and I can't get the extension to work locally on a personal machine. So, need help with that portion to fully close out this task.

Went to migrate this over to the VSCode repo where it should properly belong now, but I don't have write permissions to do so. I have a local branch holding this, and adding the missing workflow and template files as discussed in today's technical deep dive of the new API branch for the new VSCode Extension repo.
Till I can get write permissions this PR will have to remain active.

This PR has been replicated within the VSCode Repo, so closing it out here and deleting the branch. The replicated PR also includes the migration of pertinent GitHub components for establishing pipeline, templates, and the move of the VSCode Extension directory from AaC core. It can be found here


abandoning-the-azure-ad-connect-sync-encryption-key for AD Connect newer than 1.1.443.0

https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-sync-change-serviceacct-pass#abandoning-the-azure-ad-connect-sync-encryption-key
Says:
Important!
The following procedures only apply to Azure AD Connect build 1.1.443.0 or older.
Where is the procedure for AD Connect newer than 1.1.443.0?

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

ID: 5a6ac21b-3096-44b3-8099-f029a2bcf718
Version Independent ID: 30085cd3-7d19-e88b-d4c6-14e9524ec046
Content: Azure AD Connect sync:  Changing the Azure AD Connect Sync service account
Content Source: articles/active-directory/hybrid/how-to-connect-sync-change-serviceacct-pass.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

Thanks for the feedback! We are currently investigating and will update you shortly.

@aleksbor For later versions you can change the password on the service account and this will destroy the encryption key. For earlier versions it was unsupported to change that password. https://docs.microsoft.com/en-us/azure/active-directory/hybrid/reference-connect-accounts-permissions

Closing this out for now but if you have further questions feel free to leave a comment and I will gladly continue the discussion.


OutOfRangeError (see above for traceback): End of sequence  IteratorGetNext = IteratorGetNext

I am trying to run this on my own custom dataset. I have followed the steps and created tfrecords. When I run the training after about ten minutes I get the following error:
 File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1135, in _run
    feed_dict_tensor, options, run_metadata)
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1316, in _do_run
    run_metadata)
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1335, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.OutOfRangeError: End of sequence
         [[Node: IteratorGetNext = IteratorGetNext[_class=["loc:@map/TensorArrayUnstack/TensorArrayScatter/TensorArrayScatterV3"], output_shapes=[[?,513,513,3], [?,513,513], [?], [?]], output_types=[DT_FLOAT, DT_INT32, DT_INT32, DT_INT32], _device="/job:localhost/replica:0/task:0/device:CPU:0"](IteratorFromStringHandle)]]

Caused by op 'IteratorGetNext', defined at:
  File "train.py", line 95, in <module>
    batch_images_tf, batch_labels_tf, _ = iterator.get_next()
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/data/ops/iterator_ops.py", line 370, in get_next
    name=name)), self._output_types,
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/ops/gen_dataset_ops.py", line 1466, in iterator_get_next
    output_shapes=output_shapes, name=name)
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3392, in create_op
    op_def=op_def)
  File "/home/ubuntu/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1718, in __init__
    self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access

OutOfRangeError (see above for traceback): End of sequence
         [[Node: IteratorGetNext = IteratorGetNext[_class=["loc:@map/TensorArrayUnstack/TensorArrayScatter/TensorArrayScatterV3"], output_shapes=[[?,513,513,3], [?,513,513], [?], [?]], output_types=[DT_FLOAT, DT_INT32, DT_INT32, DT_INT32], _device="/job:localhost/replica:0/task:0/device:CPU:0"](IteratorFromStringHandle)]]

Any idea what would cause this?
Thanks

Hello,
Based on the stack trace, I can't say much. Just make sure the TF record contains the necessary data. It needs Input image (RGB), output mask (single channel), height, and width. Also, be aware that crops of size 513x513 will be performed.


Local edge repository for docker images - ICRA 2024 Workshop

Prepare an edge device with local repository for attendance to ICRA 2024 workshop to download docker images.
-Research software solution.
-Test implementation.
-Final deployment in edge laptop with GPU

-Add security with duck DNS.


Fixed gap after footer

This fixes #1749. The screenshot in the expected section actually uses this fix.
The problem was that the margin of the last <p> went outside the footer.

This PR adds an invisible pseudo-element to fix that.

The invisible pseudo element really feels like a hack to me... CSS is awful.


Request: add telepon.cyou to the filter

Evidence: https://telepon.cyou/view-https-stackoverflow.com/questions/3437059/does-python-have-a-string-contains-substring-method
Original: https://stackoverflow.com/questions/3437059/does-python-have-a-string-contains-substring-method

Thanks a lot!


Drop Get from the function names in the security-proxy-setup project

There are a couple of functions with Get-* prefix that return values such as  GetProxyServerName(). It is suggested to drop Get from the function names.

Note that whether these interfaces should be used at all should be a topic of discussion for this week's WG meeting

This will be resolved when the new PR is created from the new feature branch. All of these methods and their associated interfaces have been dropped.

This has been addressed & security-proxy-setup has been merged in the edgex-go, hence close the issue.


Release 0.5.1

We should really find a way to update the version number fewer places :)

It's done with a script (bump2version), but had to change a few places where it changed unrelated things.

We should really find a way to update the version number fewer places :)

It's done with a script (bump2version), but had to change a few places where it changed unrelated things.


Standardize StringUtils usages in NetworkModelImpl

Description
The CloudStack coding conventions specify org.apache.commons.lang3.StringUtils as the default alternative for string operations, and com.cloud.utils.StringUtils for operations not covered by the former.
Since org.apache.commons.lang3.StringUtils is seen as the default alternative and is used more often in NetworkModelImpl.java, this PR inverts how these two classes are referenced in this file in order to standardize it.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[X] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[X] Minor

Screenshots (if appropriate):
How Has This Been Tested?

@winterhazel , I think the default should remain com.cloud.utils.StringUtils or be named org.apache.cloudstack.utils.StringUtils . We should maintain a proxy to all external libraries to reduce maintenance risk and ease migration.
org.apache.commons.lang3.StringUtils is not a default but just the most convenient one. We should not put all our egs in that basket.

@winterhazel , I think the default should remain com.cloud.utils.StringUtils or be named org.apache.cloudstack.utils.StringUtils . We should maintain a proxy to all external libraries to reduce maintenance risk and ease migration. org.apache.commons.lang3.StringUtils is not a default but just the most convenient one. We should not put all our egs in that basket.

@DaanHoogland, by the discussion and vote threads we had in the dev mailing list (which led to the creation of session Libraries in our coding conventions), com.cloud.utils.StringUtils should only be used when org.apache.commons.lang3.StringUtils does not cover the operation.
Anyways, this PR is not changing the libraries that are being used, the changes were already made by PR #5386. :smile:
This PR is just inverting the references in class NetworkModelImpl, as we have more calls to org.apache.commons.lang3.StringUtils than com.cloud.utils.StringUtils. This makes the code more readable, IMHO.

@GutoVeronezi I am not against making it standard. I am against not proxying it. In a codebase as big as this one we need to be aware of the maintenance costs of 3rd library updates. We should proxy all important libraries, even if only passing through. This is also why I think the log4j change is half done. It is a best practice for using 3rd party libraries that we only half implement in our project (i.e. hypervisor plugins) and should be more vigilent about. Strings are not the most complex 3rd party library but still...
anyway, not -1íng just warning for a breaking with best practices here.

@blueorangutan package

@shwstppr a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7429

@blueorangutan package

@shwstppr a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7431

@blueorangutan test

@shwstppr a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan package

@shwstppr a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7577

@shwstppr a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-8178)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47704 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7980-t8178-kvm-centos7.zip
Smoke tests completed. 114 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_02_upgrade_kubernetes_cluster
Failure
607.60
test_kubernetes_clusters.py


feat: add oauth

Warning: Login doesn't work in preview builds because... random URL every time and Discord requiring a static one. Works locally, and I have set the production environment variables in CF for a successful deploy of this feature.

TODO

[x] Setup environment variables in server.
[x] Stylize login button.
[x] Stylize logout button.
[x] Fix lack of redirection in callback route.
[x] CSP based on helmet's values.
[x] CORS?
[x] Support custom redirectUri (for development) with forced origin (production).
[x] Decrease amount of requests (callback and logout should update session).
[x] Fix session not refreshing unless the entire page is reloaded.

A PR for the yarn patch on the upstream library has been created: https://github.com/Baroshem/nuxt-security/pull/157


Bug in your script

In the script to create the disk stripe, you obtain the number of disks "$PoolCount = Get-PhysicalDisk -CanPool $True", then fail to use that value in your subsequent command: "New-VirtualDisk ... -NumberOfColumns 2 ...".
This snippet will always create 2 column stripe set.

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

ID: 139dc620-240c-779f-1eed-c12accfe2ef6
Version Independent ID: a6deda7b-8361-2e2c-808a-48c047bae7a1
Content: Performance guidelines for SQL Server in Azure - Azure SQL Server VM
Content Source: articles/virtual-machines/windows/sql/virtual-machines-windows-sql-performance.md
Service: virtual-machines-sql
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hi @JohnDelisle , thanks for bringing this to our attention! Let me look into this :)
Thanks!
Masha from the SQL Docs team

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Hi @JohnDelisle , thank  you so much for your patience and for bringing this to our attention! I've processed a change to update the script and it should be live later today.
Please feel free to comment should you need anything further.
Thanks and hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close


Pass all tests on travis.

I really want to get everything in a state where all the tests pass on travis. This requires fixing the few tests that fail on CPS.

@baxtersa Can you fix this issue before moving on to implementing the new transform? Once you get started with the new transform, it is unlikely that you will have the time to come back to this.

Fixed by #37


this not working in version 2.1.0 of the ionic v1

I put all the procedures can not run this version, what to do?

hi @Dougbh ,
what you mean with version 2.1.0 of the ionic v1?
Latest v1 is 1.3.x as of now
Am I missing something?
regards
Mauro


Console not colored when using Console.Error

When making Console.Error calls, the console is not colored.
ie Console.Error.WriteLine("An Error Has Occurred", Color.Red) is the default color scheme. but  Console.WriteLine("An Error Has Occurred", Color.Red) is red.

Thanks for the report.  Calls to Colorful.Console.Error are currently passed to System.Console.Error (https://github.com/tomakita/Colorful.Console/blob/master/src/Colorful.Console/ColorfulConsoleFront.cs#L59), so text written to Colorful.Console.Error won't be colored.  It would be nice if this were supported, though, so I've put it on my list of things to work on.


Android Platform Fuzz crash show "If 32bit fuzz target in aarch64 system, try ARM 32bit build"

Crash show： “If 32bit fuzz target in aarch64 system, try ARM 32bit build”， but my platform is Arm64, my bin/so also is 64-bit.
There is nothing in SIGABRT.xxx.txt，except crash input.
=====================================================================
FUZZER ARGS:
 mutationsPerRun : 5
 externalCmd     : NULL
 fuzzStdin       : FALSE
 timeout         : 1 (sec)
 ignoreAddr      : 0x0
 ASLimit         : 0 (MiB)
 RSSLimit        : 0 (MiB)
 DATALimit       : 0 (MiB)
 wordlistFile    : NULL
 dynFileMethod   :
 fuzzTarget      : ./bin/fuzzer ___FILE___
CRASH:
DESCRIPTION:
ORIG_FNAME:<PHONE_NUMBER>0000000000000000000000.00000000.honggfuzz.cov
FUZZ_FNAME: out//SIGABRT.PC.7fb68f84fc.STACK.0.CODE.-1.ADDR.0.INSTR.[NOT_MMAPED].2020-08-07.19:40:32.13891.txt
PID: 13891
SIGNAL: SIGABRT (6)
PC: 0x7fb68f84fc
FAULT ADDRESS: 0x0
INSTRUCTION: [NOT_MMAPED]
STACK HASH:<PHONE_NUMBER>000000
STACK:
 !ERROR: If 32bit fuzz target in aarch64 system, try ARM 32bit build
=====================================================================

Please re-open if it's still happening to you.


Need better installation documentation

With particular focus on YUM and APT repository setup using bintray's repositories. Would be nice to have better documentation around other ways as well (Windows step-by-step and Mac).
Also, this documentation should be linked to, from the Go download page.

I'm going to fix this

:+1:


Unclear instructions in README

Hi :wave:
In the "beginner" section of the README, the links for requesting activity data, for Garmin and Strava, are the same and redirect to Strava support. So I think the link for Garmin might be this one: https://support.garmin.com/en-US/?faq=W1TvTPW8JZ6LfJSfK512Q8&productID=73207&searchQuery=export data&tab=topics
But after requesting my Garmin data, the project doesn't work and no FIT, TCX, GPX files are present in the exports.
Am I doing it right?

You're right, fixed that incorrect link!
Can you share the command you're using that passes in the downloaded Garmin ZIP file?

@NathanBaulch Thank you for your reply !
I retry and it's works but not when the zip file is in a directory, or when I use an absolute path to the zip file
$ go/bin/rainbow-roads export.zip
files:         5,059
activities:    99
records:       101,170
...
$ go/bin/rainbow-roads ~/export.zip
Error: open /home/phlowrient/export.zip: invalid argument
$ mkdir tmp
$ cp export.zip tmp/
$ ls tmp/
export.zip
$ go/bin/rainbow-roads tmp/export.zip
Error: open tmp/export.zip: no such file or directory
$ go/bin/rainbow-roads ~/tmp/export.zip
Error: open /home/phlowrient/tmp/export.zip: invalid argument

I'm on Manjaro, with go 1.22.0 installed


UIButton Text Color

I am attempting to use BonMot to style the titleLabel's of UIButtons with a custom font and colors.
What I expect to happen
When setting the attributedText value for an @IBDesignable UIButton's titleLabel with a string styled by BonMot. It should correctly render in interface builder and it should correctly render when running in the simulator.
What actually happens
The button is rendered correctly in interface builder, but defaults to the system font color for the default button state in the simulator.
The custom button for this example project is:
import Foundation
import UIKit
import BonMot

@IBDesignable class CustomButton: UIButton{

  override public init(frame: CGRect) {
    super.init(frame: frame)
    self.customize()
  }

  required public init?(coder aDecoder: NSCoder) {
    super.init(coder: aDecoder)
    self.customize()
  }

  override public func prepareForInterfaceBuilder() {
    self.customize()
  }

  func customize(){

  }
}

@IBDesignable class CallToActionButton: CustomButton{

  override func customize(){
    // MARK: - Colors
    self.backgroundColor = UIColor.black

    //  MARK: - Type
    let style: StringStyle = StringStyle(
      .adapt(.control),
      .font(UIFont(name: "Courier", size: 22.0)!),
      .alignment(.center),
      .color(UIColor.yellow)
    )

    self.titleLabel?.attributedText = titleLabel?.text?.styled(with: style)
  }

}

What I see in interface builder:

What I see in the simulator:

Setting the UIButton's type to "Custom" instead of "System" results in the default white text color being used (or whichever text color is selected in interface builder.)

After typing this up, I realized that I'm taking care of the custom button being displayed in interface builder by calling prepareForInterfaceBuilder in my custom button class, but not accounting for when the button is actually drawn on screen.
This ends up solving the problem, although I am not sure if this is the correct step in the drawing lifecycle in which to add the customizations.
  override public func layoutSubviews() {
    super.layoutSubviews()
    self.customize()
  }

Hope this is helpful for anyone else that runs into the same problem. :)

@sealz I'm glad you figured it out! It's kind of weird to call customize() in layoutSubviews(), since I don't think you should have to. Since you're loading from IB, init?(coder:) should be sufficient. But maybe awakeFromNib() is a better place for it? I don't use IB much any more, so I'm a little rusty. Feel free to post back here if you find out anything more. Definitely valuable to have this issue logged in case others have the same problem!


Revert "Remove the GRIST_ALLOWED_HOSTS environment variable (#899)"

This reverts commit b054810fd1e5b676bbd9896d189572948d26fce4.
We reintroduce GRIST_ALLOWED_HOST without documenting it if #915 cannot be merged at time for the next release.
It's still acceptable for us to just build our own docker image instead if you're are not comfortable with this PR, in such case we may drop this PR.

#915 has been merged before v1.1.14, so this PR is no longer necessary.


First ForgeBots Push

Initial push including ForgeBot theme and style.

LOL!!! I'm am brand new with Git and just realized I sent this to your repo. Please forgive me.


Data mining tool to collect contest results

I have been working on a strategy for collecting/logging the hackathon results, but could use some help.
The basic idea is to use the GitHub API to do advanced searches for the [unitaryhack] string in the title of PRs on participating repos. I can do this on the advanced search page manually but I would love some automation for this via say a Python script, with a stretch goal to embed this on the event website.
Some resources I was looking at:

https://codesnippet.io/github-api-tutorial/
https://docs.microsoft.com/en-us/azure/azure-functions/create-first-function-vs-code-python

Thanks to @yusharth for volunteering to help out with this!! 🎉 Please drop any questions/ideas/discussion here as needed :)

@yusharth, do you have any updates/are you stuck on anything?

Still working on it. Sorry, I wasn't able give time in past few days.
On Tue 4 May, 2021, 12:19 Sarah Kaiser, @.***> wrote:

@yusharth https://github.com/yusharth, do you have any updates/are you
stuck on anything?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unitaryfund/unitaryhack/issues/23#issuecomment-831719590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANUI75MHYFKPROL7AJYMHVTTL6KG3ANCNFSM43CD4VQA
.

Resolved with the new GH action on this repo.


Introspection crash when relation name is only used on one side.

The handwritten schema is detected as valid by the schema validation in VS Code, but crashes later. Trigger is the usage of the optional relation name on one side but not the other.
We successfully received the error report id: 12442

Related to https://github.com/prisma/prisma-engines/issues/1537

Related to https://github.com/prisma/prisma-engines/issues/1537

This should be solved by us splitting the parsing and formatting step here https://github.com/prisma/prisma-engines/pull/1662


mkKdeDerivation: set the expected UTF-8 locale

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

I was kind of hoping to push that up into stdenv, but maybe we can keep it here for now.

I know, it's 2024, we really should be expecting utf everywhere. You could also argue that this should be part of the qt6 build infra as all qt6 applications require this. But adding it here will at least shut up a lot of the builds.

Setting it here's an incremental improvement and easier to test.
I'd also be fine with adding it for all Qt6 builds.
Could add a comment noting this can go away once the default is fixed more generally.

All right.


feat: preserve some values when regenerating BUILD.bazel

In this (surprisingly large) PR I'm adding some code to preserve values of some manually edited attributes in the generated BUILD.bazel files.
The initial generation of build files is not changed, but if a file exists, I use buildozer (which is an official tool for manipulating BUILD.bazel files and has a proper syntax parser inside) and save the values we want to preserve. When the file is regenerated, I use buildozer again to set those preserved values.
List of preserved values:

name of all the *_gapic_assembly_pkg rules (which is useful for non-cloud library owners who want to remove cloud from the rule name; also, this is the only user-facing rules so making it possible to change their names make sense;
the following parameters of all the *_gapic_library rules:

extra_protoc_parameters
extra_protoc_file_parameters
package_name
main_service
bundle_config
iam_service
(the last four make sense for TypeScript; other languages can add their parameters-to-be-preserved to the list)

Now this all looks wonderful, until we realize that buildozer is written in Go and does not have any API other than its CLI, so to make it all bazel friendly, apparently the only way is to include the buildozer binary dependency to the .jar as a resource. This indeed makes .jar non-portable (because it would build buildozer for the current platform only), but since we only use this build file generator as bazel run, it should be fine. If you bazel run on Linux, your .jar will contain a Linux buildozer binary; if you're on Mac, it will be a Mac binary; if you're on Windows, it's very unlikely it will work at all but who cares.
This is my second Java PR in a few years, I have no idea how to write Java code properly, so please forgive me :)

Updates based on feedback from @noahdietz (thank you!):

--overwrite can be passed to disable the feature;
Upgraded Go rules to v0.23.3 (this also required upgrading Skylib, I'm now using the same version as we use in googleapis)
Misc fixes
--help option is now supported :)

LGTM, but I'll let @vam-google approve.
Thanks Alex, this is great.

@vam-google Can you please take a look once more? I resolved most of the comments. Let me know if it looks good.


Gateway Timeout (Code 504)

Hi Nick,
I've just found your app, so this is my first attempt to grab my Fitbit data.  I have roughly three years of data to pull, if memory serves me right.  I'm receiving the error message below, as tested on OS X Yosemite via both Safari 9.1.3 and Firefox 48.0.2.  Any ideas?
Thanks,
Mitchell
{
  "message": "Unexpected token < in JSON at position 0 in \"https://api.fitbit.com/1/user/-/activities/activityCalories/date/today/max.json\": \n<!DOCTYPE html>\n<!--[if lt IE 7]> <html class=\"no-js ie6 oldie\" lang=\"en-US\">...",
  "host": "api.fitbit.com",
  "hostname": "api.fitbit.com",
  "method": "GET",
  "path": "/1/user/-/activities/activityCalories/date/today/max.json",
  "statusCode": 504,
  "statusMessage": "Gateway Timeout",
  "requestUrl": "https://api.fitbit.com/1/user/-/activities/activityCalories/date/today/max.json"
}

ParseError: Unexpected token < in JSON at position 0 in "https://api.fitbit.com/1/user/-/activities/activityCalories/date/today/max.json":
<!DOCTYPE html>
<!--[if lt IE 7]> <html class="no-js ie6 oldie" lang="en-US">...
    at stream.catch.then.e (\\<IP_ADDRESS>\volume-3-default\71e539140aab05178a3f\76b230477a3846258950e7edbb2d6695\site\wwwroot\node_modules\got\index.js:109:14)
    at wrappedPromise.linkTransaction (\\<IP_ADDRESS>\volume-3-default\71e539140aab05178a3f\76b230477a3846258950e7edbb2d6695\site\wwwroot\node_modules\newrelic\lib\instrumentation\core\globals.js:168:23)
    at wrappedPromise.wrapped [as __NR_wrapper] (\\<IP_ADDRESS>\volume-3-default\71e539140aab05178a3f\76b230477a3846258950e7edbb2d6695\site\wwwroot\node_modules\newrelic\lib\transaction\tracer\index.js:155:28)
    at wrappedHandler (\\<IP_ADDRESS>\volume-3-default\71e539140aab05178a3f\76b230477a3846258950e7edbb2d6695\site\wwwroot\node_modules\newrelic\lib\instrumentation\core\globals.js:202:26)
    at process._tickCallback (internal/process/next_tick.js:103:7)

For some reason, the activityCalories endpoint appears to be bombing out. The others work fine. I've followed up with Fitbit.

Well, I "fixed" this by just removing the activityCalories endpoint in 9d2398b72abc11d96d54b867b7f46a6a288395b7. You should be able to get your data now.

Thank you, Nick!  Problem solved.


Docuementation Site down

Observed behaviour
Visit https://relishapp.com/rspec/rspec-core/docs -> see Heroku error

Expected behaviour
Expect to see rspec documentation.

Hello
Yes. This is a general error with relishapp.
See: https://github.com/rspec/doc-site/issues/5#issuecomment-1446152910

Closing for the moment because it is not related to rspec-rails itself.


CUDA Graphs + ZeRO-3 / TP+PP

Are there examples of using CUDA Graphs with a) ZeRO-3 / ZeRO-3++ or b) MP / PP?
Not sure if the first case would be beneficial or even possible given device syncs from parameter sharding?

Hi @jeromeku, CUDA Graphs require using the same memory region for every run. As ZeRO3 repeats gathering and discarding memory for sharded parameters, it cannot run with CUDA graphs.
It might be possible to run PP with CUDA graphs in theory, but we heavily rely on the dynamic memory allocation by PyTorch. I don't think it is realistic to enable CUDA graphs with our PP engine.

Closing as we haven't had an activity for a while. Please feel free to reopen when you have something to share.


Page sizes that are multiple of 50 don't render new data BUT pagination works

TanStack Table version
8.20.1
Framework/Library version
React 18.2.0
Describe the bug and the steps to reproduce it
I took your example for uncontrolled pagination and added my data and columns to it.
Everything will work correctly if you use just about any page size, except for 50 (and it's multiples, i.e.: 100, 200).
Load up the provided reproducible example and change page sizes, then change page numbers.
When you use page size of 50, everything else works, EXCEPT it will not render the new page's data.
It will keep repeating the old one.
Your Minimal, Reproducible Example - (Sandbox Highly Recommended)
https://codesandbox.io/p/devbox/dazzling-volhard-lnz4l6
Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Long story short, user error.  The data repeats every 50 rows (or why maybe I shouldn't have trusted the online data generator I used).
Please disregard and continue with your excellent work!
💯 ❤


Use case

Can someone explain me what would be the potential use case of this package? I'm new to this repo and wanted to use this package 🗡
Thanks

The library's use is for validating the request signatures of payloads sent to your interaction endpoint url, when receiving interactions via an outgoing web-hook. See more here. Request signatures are composed of multiple data(s) together, and they're a type of HTTP request header sent.
This won't affect your app's functionality when using slash commands or components, but it's just so you could:

Secure your endpoint from invalid requests,
Have an easier time when handling interaction payloads, this would be true since:

Discord regularly sends HTTP POSTs to your server to make sure it can handle requests - both invalid and valid, as stated in the docs, and,
Discord's API sends a "dummy" POST request to your server upon enlisting the url in your app's home page, and,
This is an easier way of implementing them.

If you want to know how to use this library, you may always see the examples folder, or browser the source files. The official Discord API docs are helpful:

https://discord.com/developers/docs/interactions/slash-commands
https://discord.com/developers/docs/interactions/message-components

TL;DR: this library provides middleware that are helpful when making an HTTP-based server for your app that receives interactions over an outgoing webhook.

Thank @J-Human For all the references and explanation.


Resizing cover

[Resizing feature]
Description
I'm trying to make the cover such that i can overlay it over my taskbar so i can easily change the song, however the square format doesn't allow it. My question is, where are the settings in the mediaplayer so that i can make it a rectangle if possible?

There is no setting to change the cover size as nearly all covers supplied by media players are square. If you still want to proceed, you need to change the width and height values in the Cover.ini file. I will not add a cover size feature as it's really easy to do it yourself with a little bit of work.
You can learn more about customizing skins in the documentation here: https://docs.rainmeter.net/manual-beta/getting-started/customizing/


Add controls to the side on 'large' profile

Change the layout for GW build

according to the latest discussion with Ola, we should display side controls on large profile because we have lots of space that we don't use.
so, when the profile is large, we should stack the hidden dialogs on the right side:
color, find, size for bubble chart and color, find, stack for mountain chart
(the collection may change)
similar to how people are used to have in Gapminder world http://www.gapminder.org/world/

do as much as possible with SCSS and only apply javascript when necessary
@arthurcamara1 please point out which SCSS files to hack and where possible JS would fall appropriate

In order to do this, we need to make buttonlist configurable in UI options of any chart.
Currently, the ui options object for a chart that shows the buttons/dialogs find, color, size, in that order, is:
ui: {
    buttons: [ 'find', 'color', 'size']
}

It will render something like this:

In case we wanted the controls to be on the side (on large profile only), maybe in a different order, we could do it like this:
ui: {
    buttons: [ 'color', 'find', 'size']
    buttons_expand: true
}

And something like this would be rendered on large screen:

And in medium and small screens, the same old buttonlist would prevail.
Now, if we wanted to only expand one of the buttons, we could:
ui: {
    buttons: [ 'find', 'color', 'size']
    buttons_expand: [ 'find' ]
}

How to solve it
Assuming the strategy above, the most straightforward way to solve it is with simple CSS classes.
If you go to the file src/components/_gapminder/buttonlist/buttonlist.js, you will see that each button and each dialog is added dynamically according to the configuration received by options.ui.buttons.
At that point, you can check ui.buttons_expand and add a class to each button and dialog that should be expanded: vzb-dialog-side , for instance.
Then, you edit src/components/_gapminder/buttonlist/buttonlist.scss to hide each button that has that vzb-dialog-side class, and display each dialog that has vzb-dialog-side class with display: block; position: static. It should happen, of course, only in case the screen is large.

Comments and improvements
Initial appearance (urgent by wednesday)
When opening on the large profile, we can see the image like this

but we actually want to have it like this already from the start with no need to click the buttons

i think it is a bug because the ui.buttons_expand is set correctly, as i can see on this breakpoint in buttonlist:

Order of panels (not as urgent)
By clicking the buttons and opening panels i couldn't get panels in the order of this.model.ui.buttons_expand = ["colors", "find", "size"]despite the array is set correctly. The ui.buttons order is prioritised instead. Panels should obey the order in this.model.ui.buttons_expand

Styling (not as urgent)
Continue hacking of the styles. We don't need the impression of panels being separate dialogs, they should rather form a single sidebar.

Start by removing the box-shadow

Change the background to match the button pane. Use color $vzb-color-primary-lightest
Hide OK buttons (we don't want to close the panels ever :-D )
Change title and indicator selector divs to inline. You might need to take indicator selector divs out of "content" container. Just make a new container "inlinable". Put there indicator selectors and search box :)
Put buttonlist below the panels
If we open more dialogs than the ones in the list of buttons_expand, let them open next to the side bar, not inside the side bar
The result picture should look like this

Done. The improvements are taken as new issues #601 #602 #603 #604 #605


Serversideoutput cannot be input, updated, and output in a single callback

I have an issue with the following callback:
# Configure main scenario poi model for control panel selection
app.callback(
    ServersideOutput(id("model-poi-main-scenario"), "data"),
    [
        Trigger(id("compare-view-execute-button"), "n_clicks"),
        Trigger(id("country-execute-button"), "n_clicks"),
        Input(id("model-poi-main-scenario"), "data"),
    ],
    State(id("main-view-poi-dropdown"), "value"),
)(sea_level_rise_storm_surge_poi_update_callback)

This call back is raising an error
AttributeError: 'Output' object has no attribute 'backend'

Which resides on line 513 in enrich.py
The poi model is build in another callback and in this callback I want to update the poi model. I would like to update the store with the updated model. But when I run these callbacks specifically it fails. Seemingly its impossible to update a ServersideOutput with itself as an Input?
There are two ways to go about doing this. Either the cache maintains the same object and the callback doesn't copy the object and just updates in the function or have a function that can take and Input as the serverside object and update the store id in the serverside output.
Wondering if this is an issue or I'm tackling this problem in the wrong way.
The final solution is to make two stores with two models, a store called hidden and then the actual one. And updates the model take the model from the hidden store and update it and then update that with the main model store. See below:
app.callback(
    ServersideOutput(id("model-poi-main-scenario"), "data"),
    [
        Trigger(id("compare-view-execute-button"), "n_clicks"),
        Trigger(id("country-execute-button"), "n_clicks"),
        Input(id("model-poi-main-scenario-hidden"), "data"),
    ],
    State(id("main-view-poi-dropdown"), "value"),
)(sea_level_rise_storm_surge_poi_update_callback)

However this I believe is pretty inelegant and unnecessarily fills cache space. If the first solution worked without throwing an error I would be happy.

I'm using the Dash object which contains all transforms

I am asking because it is not possible to use the same server side output in multiple callbacks, which it is sounds like you might be doing.

It does help with that issue, but only for normal outputs. Server side ouputs are currently not supported.

I suppose shall close the issue then thanks for the help


[dev] development docker persistent environment using volumes

Use named volumes for lua_modules and cpan dependencies.
There is no need to build them into the image becuase they'd be
remounted anyway.
We probably could drop the dockerfile entirely if we would want to.

With this solution, the user is still responsible for keeping the dependencies up-to-date by running make cpan and make dependencies (maybe we should create a new target that installs both types of dependencies). It's the same now, but builds will be faster with this method.

In makefile, we could update development task to
development: build-development ## Run bash inside the development image
    docker run -v $(PWD):/home/centos --rm -it $(DEVEL_IMAGE) make dependencies
    $(DOCKER_COMPOSE) -f $(DEVEL_DOCKER_COMPOSE_FILE) run --rm development

That way, dependencies will always be up to date when running development container.
In addition, make dependencies should include cpan task as dependency
dependencies: $(ROVER) $(S2I_CONTEXT)/Roverfile.lock cpan
  $(ROVER) install --roverfile=$(S2I_CONTEXT)/Roverfile

Also, related to this, I think we should remove script/test. It's no longer needed with the latest changes we made. We can just run make busted and make prove inside the container. It'd be nice to have a target that runs both of them.

:+1:

Few comments

Did not find way to stop containers using makefile. make clean-containers not working.
[minor issue] Coverage error ignored

Could not load stats file /home/centos/luacov.stats.out.
Run your Lua program with -lluacov and then rerun luacov.
make: [busted] Error 1 (ignored)

[minor issue] circle ci error (does not affect at all)

/bin/bash: circleci: command not found (not affected)

@eguzki those issues should be fixed.


docs(embedding_deno): added an overview of all deno crates and what they do.

Closes #470
Adds a new chapter that clarifies what deno crates do and links to them.
Makes it easier to figure out where to even begin constructing your ideal version of Deno.

Crates are documented in their own READMEs in their repos/dirs. Do we want to duplicate those docs in this format? cc @ry @piscisaureus @bartlomieju
A list of all deno related crates are also available in https://crates.io/teams/github:denoland:engineering

I'd argue that it's better to document it in the manual for easier access. No one wants to scavange for this kind of information.

there are other crates as well, like v8 & urlpattern, and probably a bunch more i am forgetting


Increased rainfall and storm frequency in risk mode

Kind of request
Adding new functionality
Enhancement Description
Add increased rainfall and storm frequency in risk mode
Task list

[x] check if edit boxes are provided in GUI
[x] check pydantic model of physical projection

change rainfall in sfincs pre-processing (similar to addition of SLR)

[x] constant (projection only)

[x] shape(projection only)

[x] timeseries (event and projection)

[x] map (event and projection)

[x] change frequency in rp_floodmap_calculation

[x] add tests

Use case
No response
Additional Context
No response

basically done, just needs to be merged


VERBOSE: [FOXDC]: LCM:  [ End    Test     ]  [[xComputer]NewNameAndWorkgroup]  in 0.0160 seconds.
The PowerShell DSC resource C:\Program Files\WindowsPowerShell\Modules\xComputerManagement\DscResources\MSFT_xComputer returned results in a format that is not valid. The results
from running Test-TargetResource must be the boolean value True or False.
+ CategoryInfo          : InvalidResult: (:) [], CimException
+ FullyQualifiedErrorId : TestTargetResourceInvalidResultFormat
+ PSComputerName        : localhost

@1RedOne - I think this one has been fixed. I can't replicate it in the current version.
So I'll close this. Feel free to reopen it if it isn't fixed.


Skip frontend packages install if previously done

We introduce a decorator @cached_procedure for this functionality.  "Procedure" because we are not caching any results, only the inputs to a previous call.  It is up to the decoratee to ensure its args capture the unique call under cache sufficiently.

Is it necessary or desirable to perform real frontend install steps under those conditions?  It's really a case of capturing all such requirements in the cache payload (e.g. by including them as args to the function).

Is it necessary or desirable to perform real frontend install steps under those conditions?

i dont think it's necessary, just a potentially notable change in how those commands work.
i mark approve, i think we should take it; but also note in the docs that if you want to actually reinstall packages to use reflex init first.


Type Error

Umm, I already installed the requirements.txt and also fill the .env file already done in google colab but I'm still getting an error

Ensure that your .env file is filled in correctly


Bootstrap modals for add-to-campaign

Ability to use bootstrap modals within the CMS via a React component. Start by implementing the "Add to Campaign" modal shown from a file.

[x] Create AddToCampaign modal component
[x] Add SingleSelectField component
[x] API endpoints for AddToCampaign in AssetAdmin
[x] AddToCampaign save/error message when submitted

This would also be required for the Insert Media (into TinyMCE) modal so just keep that in mind. I've had a quick chat to @chillu and this should be possible to have React Modal in the Pages section.

See https://github.com/silverstripe/silverstripe-framework/pull/5868/for some necessary enhancements to fom schema.

Chris suggested to include fixing a regression for the "save" progress indicator in "edit file" in here. This was broken by removing the isSaving logic via https://github.com/silverstripe/silverstripe-asset-admin/issues/171

A behaviour check for this:

When the form is saving in the screen designs only the button animates saving, was it only the button that was clicked or all the buttons if you had more than one save button for example?

In case I forget to ask Monday

Pull requests created here:
https://github.com/silverstripe/silverstripe-framework/pull/5895
https://github.com/silverstripe/silverstripe-asset-admin/pull/223

Changes made according to feedback, includes an additional pull request to CMS https://github.com/silverstripe/silverstripe-cms/pull/1587

Merged, but separated out one minor visual bug to fix in the next days: https://github.com/silverstripe/silverstripe-asset-admin/issues/234


causal_conv1d  mamba_ssm

pip install causal_conv1d==1.0.0
pip install mamba_ssm==1.0.1
Hello, I want to ask under the windows system, how to download this package, thank you very much

In ubuntu, my version is:
mamba-ssm==1.1.4
causal-conv1d==1.1.3.post1
cuda==12.2

I can successfully run

@sunkunjie
Hello, currently the causal_conv1d and mamba_ssm libraries do not support the Windows platform. We recommend using the Linux platform, or staying updated on the following repositories:
https://github.com/Dao-AILab/causal-conv1d/releases
https://github.com/state-spaces/mamba/releases


Fix test simple/sb_gb_io.

Commit 24f6b9c341910f6aaca1498872fe2e99ff8210cf broke the simple/sb_gb_io tests. This fixes it by adding proper set_io constraints.

Should be fixed in git head.


Cannot install through the plugin menu

Description
Cannot install napari-allencell-segmenter through the napari plugin menu.
Error message
Looking for: ['napari-allencell-segmenter']

Pinned packages:
  - python 3.8.*
  - napari 0.4.17.*

Encountered problems while solving:
  - package python-3.8.13-hcf16a7b_0_cpython requires openssl >=3.0.2,<4.0a0, but none of the providers can be installed

Expected Behavior
Installation proceeds without error, or gives clear steps on how to proceed.
Reproduction

Open Napari -> Plugins -> Install/Uninstall Plugins
Find napari-allencell-segmenter, and click Install

Environment

Operating system: Windows 10
Python version (ex. 3.8): 3.8.13
napari version (ex. 0.4.1): 0.4.17rc3.dev1+g4f200dfa
How did you install the napari viewer (as a bundled app or through command line with pip)?: conda-forge

Tried on a different Windows machine and I have a similar error:
Looking for: ['napari-allencell-segmenter']
Pinned packages:

python 3.8.*
napari 0.4.17.*

Encountered problems while solving:

package napari-allencell-segmenter-1.1.4-pyhd8ed1ab_0 requires python >=3.7, but none of the providers can be installed

For the above, I downloaded the release bundle from https://github.com/napari/napari/releases/tag/v0.4.17


Migrate non-boolean adjustName values in choice set REs

#11242 changed adjustName to a StrictBooleanField

Do those need special handling, or is Foundry coercing "false" to false?


Delay in Verfication email delivery.

Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a Github issue.

If you are filing an issue about FCM in the background make sure to read #4 and #89 first!

For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: Redmi Note 4
Android OS version: Nougat
Google Play Services version: 3.1.0
Firebase/Play Services SDK version: 11.2.2

Step 3: Describe the problem:
There is subsequent delay in receiving verification email and password reset email when mail is sent using the firebase android sdk for EmailAuthFlow. In some cases it takes hour to receive email. Previously this problem was not there started noticing the delay in past week.
Steps to reproduce:

Register using email and password
Send email verification mail.

Observed Results:

What happened?  This could be a description, logcat output, etc.
Subsequent delay in delivery of mail.

Expected Results:
Since these are user registration mails should be fast enough that user can verify there account as on when created.
Can this happen due to load on firebase email servers? Is there way to send the email using our services.

@jombie thanks for filing this and for taking the time to fill out the entire issue template.
Unfortunately an issue like this is almost definitely unrelated to the code in this quickstart-android repository and probably not even related to the code in the Firebase Android SDK.
Please contact Firebase Support, who can better help you troubleshoot this issue:
https://firebase.google.com/support/

Thanks for the update.


Feat/v4 Twilio iOS v5

Upgrading to use Twilio 5.x iOS SDK.
This is an important update as the Twilio iOS 2.x SDK is unsupported as of January 1st 2020 and they recommend using the iOS 5.x SDK for incoming calls on iOS >= 13.x.
I am not an iOS dev. and I am giving this my best shot. I believe there are bugs, but I wanted to get it out publicly to ask for help from anyone who knows iOS/Objective-C.
EDIT: Here are some links related to the breaking changes:

https://support.twilio.com/hc/en-us/articles/360035005593-iOS-13-Breaking-Changes
https://github.com/twilio/voice-quickstart-swift/issues/251

After some initial testing on my app, it seems that when the call is ended on either end of the call, the call isn't terminated for the other party. Perhaps a bug with the performEndCallAction method on line 543? Will try to investigate further.

Ethan, please make sure you have the latest version updated about 2 hours ago. I believe I found and corrected the issue.
Sent from my iPhone

On Dec 28, 2019, at 11:52 PM, Ethan Naluz<EMAIL_ADDRESS>wrote:

After some initial testing on my app, it seems that when the call is ended on either end of the call, the call isn't terminated for the other party. Perhaps a bug with the performEndCallAction method on line 543? Will try to investigate further.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Put correctly: I found an issue with canceling incoming calls. Make sure you have that fix when testing and let me know if that fixes your issue.
Sent from my iPhone

On Dec 28, 2019, at 11:52 PM, Ethan Naluz<EMAIL_ADDRESS>wrote:

After some initial testing on my app, it seems that when the call is ended on either end of the call, the call isn't terminated for the other party. Perhaps a bug with the performEndCallAction method on line 543? Will try to investigate further.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

That fix resolved the issue! Everything seems to check out on my end so far. The only small issue I found was that at the top of performEndCallActionWithUUID, there used to be a conditional check for the uuid that would just return if uuid == nil. An exception would be raised at the JS layer because that check was removed in your changes, however it wouldn't crash the app.
Not sure how important the uuid is (and in what situations it would be important), but my app is taking calls successfully without having the uuid set. Other than that, everything seems to check out in dev for me!

Upon closer investigation, some of the event listeners are not working in the JS layer. I'll update this comment with more details in a bit.

Fixes #89

After playing around with this a bit, I found a bug when doing back to back calls. The second call doesn't terminate for the callee when the caller terminates the call. In the second call, two logs are missing that showed up during the first call: provider:performEndCallAction and  provider:didDeactivateAudioSession. Seems like an easy fix, will try to fix this on my end today or tomorrow.

The bug I described above was introduced in part, because of self.userInitiatedDisconnect = YES; in line 121 of RNTwilioVoice.m. This change broke for me because in the JS layer I was executing the disconnect method under the connectionDidDisconnect event listener (I don't use a CallUI in my app, I just use the CallKit UI). Although removing the disconnect method in the JS layer would make this PR work for me, I believe this would still break for other people. Not sure what the best move is in this case.
Thoughts @fabriziomoscon @gitstud ?
Happy to clarify if this was a vague or confusing explanation.

I will try to verify this and help during the weekend. Unfortunately my app conversion to RN 0.61 is taking forever, but once I am in a position to make and receive calls I will help 100% with implantation

@enaluz May I ask why you're calling the disconnect method from the connectionDidDisconnect event?
Also, could you please provide the NSLogs so we can see what code path is being taken? I suspect it's from activeCall or activeCallInvites not being cleared at the appropriate time.

I finally finished my dialer yesterday and was able to test all of the events.
@fabriziomoscon what can I do to help get this committed so we can continue to bug fix and add helpful features to this library?

Yeah, that's fare. Sorry about that - the initial implementation was just me copying the updated Twilio QuickStart, which is probably the wrong way to do it.
I'll update it again with just the changes required for updating to the new version of the iOS SDK.