PyO3 extension modules always link in miniz_oxide

🐛 Bug Reports
When compiling the sample module rust2py from
https://pyo3.rs/v0.13.1/module.html#python-modules
cargo-bloat indicates that miniz_oxide symbols are linked in:
 File  .text     Size     Crate Name
 2.2%  13.7%  29.5KiB       std std::backtrace_rs::symbolize::gimli::elf::<impl std::backtrace_rs::symbolize::gimli::Mapping>::new
 1.1%   6.8%  14.7KiB       std addr2line::ResUnit<R>::parse_lines
 0.7%   4.4%   9.3KiB       std std::sys_common::backtrace::_print_fmt::{{closure}}
 0.6%   3.9%   8.3KiB       std miniz_oxide::inflate::core::decompress

I understand that 3.9% of .text is not much, but I wonder what is the zlib compression used for in this trivial module?
🌍 Environment

Your operating system and version: arch linux
Your python version: 3.9 (the module was compiled for abi3-py36)
How did you install python (e.g. apt or pyenv)? Did you use a virtualenv?: no
Your Rust version (rustc --version): 1.49
Your PyO3 version: 0.13.1
Have you tried using latest PyO3 master (replace version = "0.x.y" with git = "https://github.com/PyO3/pyo3")?: no

Thanks for the question.
In the table you show, cargo-bloat shows the crate responsible is std. Indeed, this is what I see in the source for std: https://github.com/rust-lang/rust/blob/b5c496de374407c0937cf5257d89230b5358d053/library/std/Cargo.toml#L25-L28
So my assumption is that these are included via the rust standard library in order to have backtrace support.
Is this creating an issue?

Thanks for the question.
In the table you show, cargo-bloat shows the crate responsible is std. Indeed, this is what I see in the source for std: https://github.com/rust-lang/rust/blob/b5c496de374407c0937cf5257d89230b5358d053/library/std/Cargo.toml#L25-L28
So my assumption is that these are included via the rust standard library in order to have backtrace support.
Is this creating an issue?

 I misinterpreted the cargo-bloat output.
pyo3 also has its own miniz_oxide crate dependency, which confused me.
This zlib inflate code seems to come from the std backtrace generator.
I'm closing this issue.
Thanks for the explanation!

 I misinterpreted the cargo-bloat output.
pyo3 also has its own miniz_oxide crate dependency, which confused me.
This zlib inflate code seems to come from the std backtrace generator.
I'm closing this issue.
Thanks for the explanation!


"Could not be started"

Hey when i try enableing the nodeletemessages plugin it says plugin could not be started any fix?

Same here

use the one in the pull request, that works fine https://github.com/Mega-Mewthree/BetterDiscordTrustedUnofficialPlugins/pull/21 thanks to ShiiroSan for keeping it updated
here direct link to the js file: https://raw.githubusercontent.com/Mega-Mewthree/BetterDiscordTrustedUnofficialPlugins/9a1daa46b315f9d4381ccadbfee49353d69c5a5c/NoDeleteMessages/NoDeleteMessages.plugin.js

Thanks, but you are an hour to late, i already used that >.<


Activation email flags spam systems

Hello everyone!
It seems that the activation email is hitting the spam filters because of a misconfiguration in the rDNS on the mailserver.
outgoing.personalgenomes.org has a PTR record pointing to it, but there is no A record from outgoing.personalgenomes.org to <IP_ADDRESS>.  This, in addition to the fact that activating accounts hits some of the anti-phishing flags, causes it to tip over the spam edge.  Adding an A record should solve that problem.
For reference, the spamassassin debug:
 pts rule name              description
---- ---------------------- --------------------------------------------------
 2.6 TVD_PH_SUBJ_ACCOUNTS_POST TVD_PH_SUBJ_ACCOUNTS_POST
 0.0 HTML_MESSAGE           BODY: HTML included in message
 1.1 MIME_HTML_ONLY         BODY: Message only has text/html MIME parts
 1.3 RDNS_NONE              Delivered to internal network by a host with no rDNS
 0.2 TO_NO_BRKTS_HTML_ONLY  To: misformatted and HTML only
 0.0 TO_NO_BRKTS_NORDNS     To: misformatted and no rDNS

The email headers:
Received: from outgoing.personalgenomes.org (unknown [<IP_ADDRESS>])
        by [snip] (Postfix) with ESMTPS id ECC5516A9B
	for [snip]; Wed, 25 Mar 2015 21:42:29 -0400 (EDT)
Date: Thu, 26 Mar 2015 01:42:28 +0000
From: PGP support<EMAIL_ADDRESS>To: [snip]
Message-ID<EMAIL_ADDRESS>Subject: Please activate your new account
Mime-Version: 1.0
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit
X-detected-operating-system: by outgoing.pgp-hms.org: No matching host in p0f cache. That's all we know.

I think this is one is a Tapestry-specific issue, but because I have access to the personalgenomes.org DNS I went ahead and added the A record. :)

Beat me to it, @beaugunderson.  I just got told that it was in the wrong place, but I didn't have time to move it before you fixed the problem!

I think PGP Harvard isn't supposed to be using a "personalgenomes.org" name anymore? Stuff was converted to "pgp-hms.org".  cc @cure

This was caused by one remaining reference to outgoing.personalgenomes.org instead of outgoing.pgp-hms.org in the mailserver configuration on that machine. I fixed it. The A record for outgoing.pgp-hms.org existed already and is pointed at the correct IP.
@beaugunderson you can remove that outgoing.personalgenomes.org A record, it's no longer needed.

@cure OK, it's been deleted now. :)


Centos 7.5 ninja: build stopped: subcommand failed.

On Centos 7.5 Linux 3.10.0-862.14.4.el7.x86_64, I get the below error when executing
git clone git://github.com/FreeRDP/FreeRDP.git cd FreeRDP cmake -GNinja -DCHANNEL_URBDRC=OFF -DWITH_DSP_FFMPEG=OFF -DWITH_CUPS=OFF -DWITH_PULSE=OFF . cmake --build . cmake --build . --target install
.....
`[424/424] Linking C executable client/X11/xfreerdp
[th6mas@freckle75 FreeRDP]$
[th6mas@freckle75 FreeRDP]$ cmake --build . --target install
[1/1] Install the project...
FAILED: CMakeFiles/install.util
cd /home/th6mas/FreeRDP && /usr/bin/cmake -P cmake_install.cmake
-- Install configuration: "Release"
-- Installing: /usr/local/share/man/man7/wlog.7
CMake Error at winpr/cmake_install.cmake:36 (FILE):
file INSTALL cannot copy file "/home/th6mas/FreeRDP/winpr/wlog.7" to
"/usr/local/share/man/man7/wlog.7".
Call Stack (most recent call first):
cmake_install.cmake:37 (INCLUDE)
ninja: build stopped: subcommand failed.
`

Please use the mailing list or IRC for support questions.
The tracker is reserved for issues.

I subscribed to the mailing list but cannot see how to post or create a thread. Thanks!

Ng0177 write a mail to the list?

Sorry, my mistake: the welcome e-mail went into the wrong folder...


Login form UX

Login form fields should accept Tab click to go to next control.
Also when renew password form shown, the focus should be by default on the first input.

DOne


MH-12613 New WorkflowOperationHandler 'create-event'

This patch adds a new WOH to create new events from an existing one. This is useful when you have a recording with - let's say 3 - different presentations and you want to create 3 events where each event only includes one presentation. With this WOH, you would copy the original event 2 times and cut each of the 3 events differently to only include one presentation, each.
It would be nice to have this functionality in the video editor, but as a first approach, this WOH is a good trade-off between effort and usability. Also, this WOH is useful for creating test data.
This WOH also provides the possibility to select the media package elements to be copied by tags and flavors. Tags can be changed for the newly created media package elements using a "+" and "-" syntax.
A workflow to use/test the WOH is included. It allows you to specify how many copies you would like to create.
This work is sponsored by SWITCH.

Hi Kim,
I totally forgot about this but finally had another look today. It looks mostly good except that you still leave files in the workspace and working file repository, A short background:

The workspace may be local to Opencast nodes and you should always clean up after you are done with an operation.
The working file repository is the central space to store stuff you need during a workflow run. That is why you would leave files there at the end of an operation. A workflow's final cleanup operation will take care of it.

But in this operation you are doing something unusual: You create media packages ouside the context of the workflow since you throw them into the asset manager. For that you also put data into the workspace and the working file repository. Since no one, except your operation, knows about these data, no one will remove them: You have to do that yourself.
Here, you copy the distribution artifacts but leave them in the workspace and the new dublincore catalogs. You should remove both before ending this operation
For the dublincore catalogs, something like this should suffice (untested):
diff --git a/modules/workflow-workflowoperation/src/main/java/org/opencastproject/workflow/handler/workflow/DuplicateEventWorkflowOperationHandler.java b/modules/workflow-workflowoperation/src/main/java/org/opencastproject/workflow/handler/workflow/DuplicateEventWorkflowOperationHandler.java
index b7909d218..d51c4ab5c 100644
--- a/modules/workflow-workflowoperation/src/main/java/org/opencastproject/workflow/handler/workflow/DuplicateEventWorkflowOperationHandler.java
+++ b/modules/workflow-workflowoperation/src/main/java/org/opencastproject/workflow/handler/workflow/DuplicateEventWorkflowOperationHandler.java
@@ -252,6 +252,8 @@ public class DuplicateEventWorkflowOperationHandler extends AbstractWorkflowOper

       // Remove temporary files of new media package
       try {
+        final String episodeDCCatalog = newMp.getCatalogs(MediaPackageElements.EPISODE)[0].getIdentifier();
+        workspace.delete(newMp, episodeDCCatalog);
         workspace.cleanup(newMp.getIdentifier());
       } catch (IOException e) {
         logger.warn("Failed to cleanup the workspace for media package {}", newMp.getIdentifier());

You would need to do something similar for the distribution artifacts you duplicate (though I'm still not sure why they would need to be duplicated at all…)

Hey Lars, thanks for your comments. I added some code to remove these files. However, one file still stays in the workspace:
workspace/http_localhost_8080/assets/assets/9c37eeb6-bea8-4cea-af11-89bc4a2e5994/7125d6d2-75ae-4362-814b-e0e3fab3e4ca/0/xacml.xml
This file is created by the asset manager and I am not sure if this is something to worry about. Do you think I should delete it as well?
In this case, I might have to add some hacky code since the media package only carries version 1 instead of version 0 of this element...

No hack ;-P
Unfortunately, that's broken code elsewhere.
I'll try to have this tested until Monday

Sorry to be annoying but I took a short look at your new patch and it introduces a huge concurrency problem: You introduce a global variable which is possibly used by several operations at once and hence randomly reset. Additionally, operations would delete files of other operations if they are executed simultaneously, leading to arbitrary IO exceptions.

No problem. For some reason I forgot about the fact that the WOH is a service and there will be a shared instance when running concurrently. I changed the variable declaration to be method-local. That should be safe now ;)

Hi Kim,
there are a few remaining issues for which I simply provided a patch:

https://github.com/lkiesow/opencast/commit/3b51cd11891d8716a4127d8f2be9331f010ac5a3

In short this:

Fixes a checkstyle issue
Fixes the dependency declaration
Fixes leaving files in the workspace (you forgot one)

Can you check if that patch is fine for you and include it? Since this pull request is now split across a lot of commits, would you mind squashing them together (feel free to squash my patch as well).

Thanks @lkiesow
I applied your patch and squashed everything into just one commit.

merged!


Exception thrown in drainCollisionEvents callback is silenced

This gave me hard time figuring out what's wrong with my code.
This won't propagate the error to browser context
eventQueue.drainCollisionEvents((handle1, handle2, started) => {
    throw "HELLO";
});

Current workaround
eventQueue.drainCollisionEvents((handle1, handle2, started) => {
   try {
         throw "HELLO";
    } catch (e) {
        console.error(e);
    }
});

Thanks for building rapier!

I see this behavior as well. Took me a while to figure out why I wasn't seeing some errors and made debugging challenging. 😅


Add rule to convert short template notation to bundle notation

Converts short Twig template notation (deprecated in 5.0) for bundle notation with @ prefix.
Needed this rule for my own purposes but tought I'd share for anyone that might need it.

@johanadivare Applied suggestions, waiting for another review!

Please run:
vendor/bin/rector && composer fix-cs

to make CI green.

Looks good, thank you :+1:


Fix a typo in the X509_get0_subject_key_id() documentation

Checklist

[x] documentation is added or updated

Description of change

Fixes a copy&paste error

Pushed. Thanks.


Feature Request: Lint ignore (or fix)

My small-ish app currently has 3,759 flutter lint "hints" in the Dart Analyzer tool, and I think they're all from the generated graphql_codegen dart files.
Some examples:
Annotate overridden members

Prefer is! operator

Private field could be final

Not sure if there's much of a performance hit for the linter checking through all these generated files, but is there a way they could be generated with
// ignore_for_file: type=lint

Since nobody should be fixing lint errors in them
Artemis has something similar ignore_for_file but I'm not sure what the benefit is of linting generated files at all unless you're working on the generator

You can add this in the generatedFileHeader option

Ahhh, perfect, I'll thought that was for something different, I'll update the docs

Okay, that worked for the graphql.dart files but not the graphql.g.dart files

For that it might be easier just to exclude the files: https://dart.dev/guides/language/analysis-options#excluding-files

I don't think generated files need to pass lints instead users should ignore the files.


routes

Hi,
Do I have to add routes manually? because when i visit domain.dev/install this is what i get
404 Not Found
nginx/1.12.0

If yes please share routes so i can use it.
Thanks.

Did you follow the installation instructions thoroughly? Especially the "artisan vendor publish" command? Also are you using Laravel 5.5 or a previous version?

@TechTailor yes i did but somehow it didn't work till i restart my system, now everything is working.
thanks.


Improve Frontend::apply_patch performance

I've been testing this with a patch built from 159 changes. The apply_patch operation was taking ~720ms to finish (which was getting rather noticeable) but with this PR it now takes just ~52ms.
I believe my changeset was rather object heavy (rather than just a text value) which may have been more of an issue for the logic. I have added some benchmarks for the apply_patch operation to cover some of these cases.
From perf I found that lots of time was spent doing malloc and free. Then DHAT helped give a picture of where lots of the allocations were coming from and it turned out to be lots of cloning. So I've been through and tried to reduce the clones by either passing by value (rather than ref) or re-using existing datastructures (in the case of current_objects).
I think there is still room for improvement but wanted to do it bit by bit so it should be easier to check correctness.
Here are the benchmark results compared to main (https://github.com/automerge/automerge-rs/commit/ae56f0d12f1ed599720219b6246c1659d839b486) with the extra benchmarks.
B1.1 Append N characters
                        time:   [823.11 ms 825.63 ms 828.38 ms]
                        change: [-4.6630% -4.2938% -3.8849%] (p = 0.00 < 0.05)
                        Performance has improved.

B1.2 Append string of length N
                        time:   [61.911 ms 62.138 ms 62.317 ms]
                        change: [-19.129% -18.232% -17.388%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 10 measurements (20.00%)
  2 (20.00%) low severe

B1.3 20√N clients concurrently set number in Map
                        time:   [436.28 us 440.28 us 444.61 us]
                        change: [-17.639% -16.373% -15.127%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 10 measurements (10.00%)
  1 (10.00%) high mild

Frontend::apply_patch, small patch
                        time:   [1.5974 ms 1.6159 ms 1.6243 ms]
                        change: [-39.633% -37.435% -35.191%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 10 measurements (20.00%)
  2 (20.00%) low mild

Frontend::apply_patch, small patch, many changes
                        time:   [1.5724 ms 1.5954 ms 1.6098 ms]
                        change: [-39.902% -37.844% -36.178%] (p = 0.00 < 0.05)
                        Performance has improved.

Frontend::apply_patch, nested maps
                        time:   [109.52 ms 110.49 ms 111.75 ms]
                        change: [-83.156% -82.963% -82.744%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 10 measurements (20.00%)
  1 (10.00%) high mild
  1 (10.00%) high severe

Frontend::change insert long string
                        time:   [2.7435 ms 2.7906 ms 2.8168 ms]
                        change: [-17.015% -15.666% -14.331%] (p = 0.00 < 0.05)
                        Performance has improved.

StateTreeValue::apply_diff sequential text inserts across multiple patches
                        time:   [220.76 ms 221.50 ms 222.19 ms]
                        change: [-7.3750% -6.9084% -6.4340%] (p = 0.00 < 0.05)
                        Performance has improved.

StateTreeValue::apply_diff sequential text inserts in a single patch
                        time:   [9.5635 ms 9.6017 ms 9.6321 ms]
                        change: [-36.976% -35.693% -34.027%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 10 measurements (20.00%)
  1 (10.00%) low mild
  1 (10.00%) high severe

I think we could hold off merging this until https://github.com/automerge/automerge-rs/pull/88 is done as it will likely change some of this stuff and best to just focus on the performance of the updated version.

Sorry it took me a while to get to this - can you fix the conflicts so I can merge?

Since I expect Alex's new patch format to change this quite a lot and I don't want to hold that work up should we wait until that is merged, then I can rebase?

Closing this as the other PR has now been merged


Remove TextDecoder polyfill

We have a dependency on
"text-encoding": "^0.6.4",

which results in
warning workspace-aggregator-ff73f3ed-c8fa-408e-9bb6-7dbd250eacf3 > opensphere ><EMAIL_ADDRESS>no longer maintained

It looks like we just need that for src/os/arraybuf.js and src/os/file/mime/text.js (where the later deals with the exception as well). There are also a couple of unit tests.
From https://caniuse.com/#feat=textencoder it looks like the polyfill isn't required for any of the required browsers. Can it just come out?
Edit: Missed that we suggest IE and Edge might be OK. Just catch the exception?

The fact that it isn't maintained does not make the polyfill any less useful. Simply removing it is not a good idea as most of our content type detection depends on it. It would need to be replaced.

Looked on npm, but most of the options are just different packages of the same (dead) upstream.
fast-text-encoding package does work (well, it builds and passes the cypress tests, but since polyfill...) although its a poor replacement since it only decodes to UTF-8 and we need latin2, latin3, latin4, cyrillic and UTF-16.

Are there shortcomings with the current state of the polyfill? It doesn't need to be actively maintained if it already satisfies our needs.

No functional problems (that I've seen). It just offends the security folks.


[Hydration Error with EmptyState]: Nested NoDataEmptyState throws hydration errors on Next JS 15

Package
Carbon for IBM Products
Description
When using NoDataEmptyState as a nested component, a hydration error is thrown

              {contentSwitcherIndex == 2 ? (
                <>
                  {array.length > 0 ? (
                    <Grid>
                      {array.map((s) => (
                        <Column
                          key={"Column" + s.s_name}
                          xlg={5}
                          lg={5}
                          md={4}
                          sm={4}
                        >
                          <ServiceTile
                            key={s.s_name}
                            service={s}
                          />
                        </Column>
                      ))}
                    </Grid>
                  ) : (
                    <NoDataEmptyState
                      style={{ padding: "2rem" }}
                      illustrationDescription="No data available"
                      subtitle="No data available at this time. If you think this is a mistake, please contact admin."
                      title="No data available"
                    />
                  )}
                </>
              ) : null}

Component(s) impacted
NoDataEmptyState

Both CodeSandbox and Stackblitz examples are not working

Browser
Chrome
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
V2.38.0
Suggested Severity
Severity 1 = The design is broken in a critical way that blocks users from completing tasks or damages the brand. Affects major functionality, no workaround.
Product/offering
IBM Research
CodeSandbox or Stackblitz example
Link is not working
Steps to reproduce the issue (if applicable)
Create a Next js react project
Use NoDataEmptyState in a conditional rendering function or directly in the page body
Release date (if applicable)
No response
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current issues for duplicate issues

This is the best I could do to reproduce
https://stackblitz.com/edit/stackblitz-starters-bk9xut?file=app%2FHome%2Fpage.tsx
However, I am not seeing the same error as local

Hi @DoaaRadwan ,
Could you share us more details probably in the Carbon Developer office hours ?

Hi @DoaaRadwan , Could you share us more details probably in the Carbon Developer office hours?

Hi @amal-k-joy, what kind of details do you want ?


FP16 Support for C Predict API

Description
Adds dtype support for C Predict API.
Fixes: https://github.com/apache/incubator-mxnet/issues/14159 and https://github.com/apache/incubator-mxnet/issues/13335
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-label-bot add [pr-work-in-progress]

thank you!


Provide a better way to read the room from the message for slack and other adapters

With the new slack client, the room was always undefined.
Now, the room can be retrieved for slack and other adapters as well.

I've been meaning to do this myself for a while but hadn't gotten around to it, so thanks!


add check for url and do not parse fixes #6

I :heart: that you fixed the indentation.

Yeah I wrote this before I got all my ruby crap set up in vim to be a proper 2-space indent, but worth cleaning up now.

Oops, sorry.  Accidentally clicked on the assign self button.


Udeler video do not dowloader

Hi udemy this morning in the video download process does not download

Its broken.

I have fixed this issue in v1.8.0. You can download the latest version and try again.
Also I have added auto retry option (you can enable it from settings), which will retry the download automatically if it fails.

I'm currently using 1.8 and the problem is still in progress, I've deleted and reinstalled, but no changes

Here You go http://prntscr.com/pxwpoc

or http://prntscr.com/pxwq5f


Fontfamily "Inter" not working

Hi, I've setup a Keycloak instance running on version 9.6.7 of Keycloakify. I tried setting the font-family to the standard "Inter" google font, but can't get it working. Changing the font family to "inter" does nothing to the font and it seems that some sort of default keycloak font is being used.
Here is a picture of the font, which is currently being displayed if I use "Inter" as fontfamily.

Am I missing some sort of import statement or do I need to download the font for it to work?

Hello @fischer-philipp,
Please share how you are importing the fonts. Ther error you're getting in the browser console and how you applies the font.

Ok I see why you got confused.
When I say "out of the box" I mean that it work like in any other app. There's no extra step related to Keycloakify.
I'll reformulate.

Okay perfect, thanks for the quick response!


member() should return True or error, never False

https://docs.opendp.org/en/stable/user/programming-framework/supporting-elements.html#domain
What output can we expect from member()? Going beyond the examples on the page:
f64_atom_domain.member('test')
TypeError: inferred type is String, expected f64. See[ https://github.com/opendp/opendp/discussions/298](https://github.com/opendp/opendp/discussions/298)

So we see three possible outcomes: True, False, or error. @Shoeboxam indicates that the desired behavior from the Rust code is True or error, never False.

@Shoeboxam https://github.com/Shoeboxam  why true vs. error instead of
true vs. false?

Salil Vadhan (he/they)
On Wed, Nov 15, 2023 at 7:59 AM Chuck McCallum @.***>
wrote:

https://docs.opendp.org/en/stable/user/programming-framework/supporting-elements.html#domain
What output can we expect from member()? Going beyond the examples on the
page:
f64_atom_domain.member('test')
TypeError: inferred type is String, expected f64. See https://github.com/opendp/opendp/discussions/298
So we see three possible outcomes: True, False, or error. @Shoeboxam
https://github.com/Shoeboxam indicates that the desired behavior from
the Rust code is True or error, never False.
—
Reply to this email directly, view it on GitHub
https://github.com/opendp/opendp/issues/1007, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB4KIGNQGRYAV7AZ6VHQVO3YETRF3AVCNFSM6AAAAAA7MXEEO2VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE4TKMBWGQ3TKNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I think the API should be consistent: always a bool or always an error.
Errors are preferable over bools because they can explain why the value is not a member of the domain. So this approach would give us both consistency and usability.
If you feel the user should not gain the extra information from an error message, we could instead convert python exceptions to False.

Specifically the Rust return type would be Fallible<()>: either an error, or the unit type.

I guess it depends on what the use cases of the member check are?
I imagined two, but not sure how relevant these are anymore:
One is a row transform map(r) that forces elements of the carrier type to
lie in the domain (similar to clamping, but for arbitrary domains): if
member(r) return r else return domain.default_value
The other is as a debugging diagnostic, e.g. assert(member(r)).
In the latter case, we'd want an informative error message, whereas in the
former we want a boolean value (because the error message.

FWIW, this behavior is different than our member method in python, which returns True or False. Maybe the difference in behavior will be less surprising to Rust users?


Developer vs. business

This PR closes #124.

[x] Redesign header to work with more user-focused links
[x] Redirect to /roles/new after signing up
[x] If a developer, redirect to /developers/new
[x] If a business, redirect to /clients/new

[x] Add business form

[x] Add link to business profile in dropdown
[x] Update avatar image in nav bar
[x] Refactor AvatarComponent naming to be generic to Developer and Business.

Screenshots

Business flow

PR checklist

[x] Your code contains tests relevant for code you modified
[x] All new and existing tests are passing
[x] You have linted the project bundle exec standardrb --fix

This is feature-complete, but I want to sleep on it before merging. My big open question is if this should all be behind a feature flag.


RECEIVE SMS messages using Python SDK?

Is your feature request related to a problem? Please describe.

The Azure Comm Services (ACS) Github repo says the capability to "send and receive SMS messages" exists
But this does not appear to be the case for the Python SDK
REF: https://docs.microsoft.com/en-us/answers/questions/1000280/is-it-possible-to-receive-sms-texts-using-azure-co.html

Describe the solution you'd like
Would like the ACS Python SDK to adopt the ability to RECEIVE SMS texts. This would create a solid use case over Twilio. I want to keep all projects within Azure rather than spin up another vendor.
Describe alternatives you've considered
Twilio
Additional context
https://docs.microsoft.com/en-us/answers/questions/1000280/is-it-possible-to-receive-sms-texts-using-azure-co.html

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Docs:0.14654307,Service Bus:0.08424358,Communication:0.054395504'

Thanks for the feedback @SeaDude, we'll investigate asap. cc @turalf

Answers were given here and here.
Thank you


[Question]using only QUIC without HTTP?

Hi, can this lib be used as a pure QUIC library? For example, implementing an echo server and client, communicating using QUIC protocol?
Looking forward for your reply. Thx!

Yes, it can.  As a matter of fact, we have a couple of non-HTTP client and server programs we can share.  They may need to be revived, though, as we haven't maintained them.

See echo and MD5 client/server pairs in release 2.4.7.
Closing.

wow, this is very helpful. thx!


Add permissive option to image registry initializer

This PR adds the --permissive flag to the image registry docker build to ignore errors when loading and initializing the OLM bundles restoring the behavior the image registry initializer had before this PR https://github.com/operator-framework/operator-registry/pull/74
This option should be removed once https://github.com/operator-framework/operator-registry/pull/76 is merged which fixes the initial issue where the initalizer attempts to parse any file as JSON or YAML.
This should be considered temporary to fix our Jenkins builds until the above is complete.

should we merge this as well https://github.com/openshift/aws-account-operator/pull/133

/lgtm


about the target label

Hi,Yinpeng. a question. why do you set the prediction of inception_resnet_v2 as the target label? and the loss is computed by the (y,pred_logits). so is this means that you want to genarate a adv with target label as inception_resnet_v2 predicted rather than a adv that can fool all the 4 models you gave in the model? if it is convient, hope to get you reply? thx！

Note that the code is for untargeted attack, so we want to fool all the 4 models to give a prediction different from that of inception_resnet_v2.  I used the prediction of inception_resnet_v2 as the ground-truth label for each image since this model gives the highest accuracy on the image set.


fix: runtime-web tarbar click to touchstart

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


ebiten: add 'Subtract' composite mode, or expose APIs to set the blend function

This is similar to #1347.
Requested by @wasedaigo
We might have to consider to expose the composite functors to be specified directly.

OpenGL: https://www.khronos.org/registry/OpenGL-Refpages/gl4/html/glBlendEquation.xhtml
Metal: https://developer.apple.com/documentation/metal/mtlblendoperation?language=objc

For example, if some part of the source is (0, 0, 0, 0), the result of the part will always be (0, 0, 0, 0). This seems unacceptable. Would subtraction really make sense?

@hajimehoshi I'm trying to draw a circle with a stroke and a fill. I can draw the circle with lines and it looks ok. However, for larger stroke widths it looks terrible. I'm trying a second solution which is to draw a filled circle with vector.Path and then a smaller circle with the fill color. This is working except for transparent fills. I think the Subtract composite mode might do it.
Is there another way to subtract a center from a filled path using the existing composites?

@hajimehoshi I'm trying to draw a circle with a stroke and a fill. I can draw the circle with lines and it looks ok. However, for larger stroke widths it looks terrible. I'm trying a second solution which is to draw a filled circle with vector.Path and then a smaller circle with the fill color. This is working except for transparent fills. I think the Subtract composite mode might do it.
Is there another way to subtract a center from a filled path using the existing composites?

Hi. So you want to render a torus, right? The subtract composition is not for this, and Ebiten should provide a way to fill a polygon with holes like a torus, but does not yet. If you want, you can do it with DrawTriangles by doing triangulation (splitting a polygon into triangles) by yourself so far.

Hi. So you want to render a torus, right? The subtract composition is not for this, and Ebiten should provide a way to fill a polygon with holes like a torus, but does not yet. If you want, you can do it with DrawTriangles by doing triangulation (splitting a polygon into triangles) by yourself so far.

Now I'm not sure we really need this 🤔

There have been various request about blending. Probably exposing everything about blending and letting uses use as they like seems the best so far.

Implemented Blend API. https://github.com/hajimehoshi/ebiten/issues/2382#issuecomment-1279994360


Authorization

Draft for now... any feedback welcome.
This is how I would do authorization... At least for the AzureAd side of things.
The bulk of the functionality is in AzureAdClaimsPrincipalFactory.CreateUserAsync which gets called by the framework when a claimsprincipal is constructed after a user signs in. There you have an opportunity to add claims to the claimsprincipal in question.
2 calls are done: First IdentityService.GetProfileDetailsAsync to get the current user's Id, which is used to call UserService.GetPermissionsAsync which returns the permissions for that user. Those permissions are added to the claims, so the rest of the authentication logic can take over.
I will also add the rest of the user info into claims, so it can be used in other places in the app (e.g. personcard).
We'll have to add an [Authorize(Policy = <permission>)] attribute to the individual pages with their respective permissions. I've done this now for the brands page to test, also for the dashboard I created a new permission to be able to see what happens when a logged in user doesn't have access to a particular page. I had to update app.razor to accommodate for that, otherwise we got into an login loop...
A couple of remarks:

We also will have to include logic into the NavigationMenu so menu items for which a user doens't have access won't be shown.
It might be interesting to create a new api endpoint which returns the info from both calls GetProfileDetails and GetPermissions, so only one call needs to be done in stead of 2.

I'll also try to figure out how to do a similar thing for regular Jwt auth...

For navigation permission checking, we can use
public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationService service, ClaimsPrincipal user, string policyName);. This will return a boolean based on the permission.
I have used this earlier this way bool _canViewBrands = (await _authorizationService.AuthorizeAsync(_authenticationStateProviderUser, Permissions.Brands.View)).Succeeded;

Jep, that's what I meant.

I am working on getting refresh tokens working as well...

Permissions and refresh tokens working...
I've got a strange thing wrt multiple calls at the same time when I set the token expiration time to 2 minutes on the server, Maybe that's to be expected as I'm asking for a refresh token when the expiration time is less than 2 minutes away... but still, it's something strange... happens when I go to the brands page as there, for some reason (another bug, but a good one to test this out) the call to search for brands is done twice on initialization. There's an experiment there with a semaphoreslim, but that's probably not needed (not a solution to the problem anyways).

When I put the token expiration time to 3 minutes (and I now also wait until expiration time is only 1 minute away) I can't seem to repeat that problem anymore, so I wouldn't worry too much about it.
The signalr connection actually keeps the access token refreshed, as I see the connection quickly gets disconnected and re-connected every 3 minutes... that means a valid access token should be available for calling the api at all times.
Still have to implement/test the scenario where the refreshtoken actually gets expired though...
Anyways, the more I read about it, the more it seems that this jwt authentication method is not done in this day and age (It's never good to "roll your own" when it comes to security). You shouldn't save access and refresh tokens on the client, ...
I would put it in the docs that this standard authentication method is made to get started quickly, but should be replaced by something more robust when you actually put your app online!

Btw, this should fix #12 as tokens now get refreshed automatically behind the scenes.

Is this PR ready to be merged?

Yes if review is ok ;-)


Standardize tool interface

Tools currently need to be registered in gptme/tools/__init__.py.

[ ] It'd be nice if we didn't have to enumerate them in init.py and instead auto-discover them, or something.

Perhaps even allow users to easily specify their own tools folder/files.

[x] enable/disable them in config or with a CLI param (done in https://github.com/ErikBjare/gptme/commit/48d559b487682a22d7614948270442c529596375)

Most of the work done in https://github.com/ErikBjare/gptme/pull/86

This is pretty clean now.

Should maybe make a @tool decorator that takes a function or ToolSpec and registers it/makes it loadable, then explicitly specify which tools to load during init.


Fix Decimal/Thousand markers for BRL.

Hi @brunoga,
Sorry for late response.
Thanks a lot for the PR, it looks great.


fix: use native array .flat() instead shim library

ES2019 is supported in Node now

Codecov Report

Merging #723 (324edf6) into master (f7dfff5) will decrease coverage by 0.18%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #723      +/-   ##
==========================================
- Coverage   50.52%   50.33%   -0.19%
==========================================
  Files          24       24
  Lines        1049     1045       -4
  Branches      199      199
==========================================
- Hits          530      526       -4
  Misses        472      472
  Partials       47       47

Impacted Files
Coverage Δ

src/commands/buildSoy.ts
28.57% <0.00%> (-4.77%)
:arrow_down:

src/commands/serve.ts
11.85% <0.00%> (-0.65%)
:arrow_down:

src/compiler.ts
51.41% <0.00%> (-0.23%)
:arrow_down:

src/gendeps.ts
93.15% <100.00%> (-0.10%)
:arrow_down:


Quick improvements

adds a .gitignore and a setup.py file.
I also slightly tweaked the CLI to work with the console_scripts entrypoint in setup.py, but there are some quirks to how the cli is currently written that made it a little difficult. I think the demo and other modes should be refactored to not be recursive which may simplify things

Thanks, Andrew, for these changes. I'm not familiar with creating the setup.py file, but it looks correct. I would set the version at 3.0.0, not to cause confusion with previous non-python versions. Would that make sense?
If I take the demo routine outside of cli.py would that solve the recursion problem?

hey Greg,
you can learn more about setup.py from the links I pasted below. In short it defines metadata required for a package to be properly installed and to be listed on the python package index. You may want to update the email address field to match your email address (I could not locate it).
As for the version number, I can update it to be 3.0.0.
For the demos, there are a few ways to implement it. Personally I would just turn the existing demo script into a bash script that the user can run directly removing the option from the CLI. Alternatively, the demo option in the cli could instead just print the commands directly to the terminal allowing users to copy/paste them to run them themselves, optionally with the paths to the craterstats files updated to allow the commands to work in any directory.
That said, what I have implemented does work but only if your are in the correct directory to find the crater stats files.
https://packaging.python.org/tutorials/packaging-projects/
https://the-hitchhikers-guide-to-packaging.readthedocs.io/en/latest/creation.html#basics-creating-and-distributing-distributions
https://python-packaging.readthedocs.io/en/latest/minimal.html

change version to 3

For the demos, there are a few ways to implement it. Personally I would just turn the existing demo script into a bash script that the user can run directly removing the option from the CLI. Alternatively, the demo option in the cli could instead just print the commands directly to the terminal allowing users to copy/paste them to run them themselves, optionally with the paths to the craterstats files updated to allow the commands to work in any directory.
That said, what I have implemented does work but only if your are in the correct directory to find the crater stats files.

Are you sure?  The importlib changes that you put in should abstract that away, such that craterstats -demo can be run from anywhere and should produce a demo/ directory full of output.  I haven't tested it, but what is the problem?
The demo does already print them to the terminal, but then also runs them.  The "user experience" of -demo seems fine to me.

yes I am, examine https://github.com/ggmichael/craterstats/blob/main/src/craterstats/config/demo_commands.txt, it only has relative paths to the crater stats files -p source=craterstats/sample/Pickering.scc and the importlib improvements do nothing to correct this. They only help find the demo_commands.txt file path, they do not update the contents. Running craterstats in the terminal with the above changes outside of the src directory fails:
FileNotFoundError: [Errno 2] No such file or directory: 'src/craterstats/sample/Pickering.scc'

Ah, that's what I get for trying to theory-craft half-remembered code and not running the actual test (my bad).
So maybe the thing to do here is to not just have those commands in a dead text file, but to have these commands perhaps as Python strings where the things that need file paths can be replaced with appropriate importlib_resources variables when run by demo().
Yes, that's more work.  However, I think there is a tremendous amount of demonstrative power for a new user to just run craterstats -demo without having to cut and paste or fiddle with inputs, and just get a wonderful, rich set of examples in the demo/ directory.
But maybe that's another PR.

yes, I think there is absolutely value in the demos but there are a few ways to go about it, a later PR could address this.
If there are no other issues then I think this pr can be merged, I may need to submit another pr to enable buffered crater counting in the CLI and I may want to quickly add a github action workflow to run the tests for the project

I think I put my last reply in the wrong place - in the merge comment - but anyway, thanks for those additions, I think I've also fixed the demo issue now.
Are you talking about the Poisson buffer calculation? Did you want to use it? I'll try to sort that out too.
I don't know about the github action workflow - but it sounds like a good idea!

@ggmichael, yes I believe so, I hadn't seen your newest paper (this is a new area of the field for me) so the thing I was thinking about specifically was Fassett's 2008 paper (https://doi.org/10.1016/j.icarus.2007.12.009), I am not sure yet if I need that exact methodology, or a modified version as of yet.  Thanks for enabling it so quickly though!

Ok - the buffered counting as Caleb did it was already included there. If you use Cratertools for your counts, it will account for the buffer by adding a 'fraction' column to the crater table, which normalises the crater weighting to its buffer area. You then estimate an isochron with a differential or cumulative fit. The new code lets you apply the Poisson estimation method, which has much better behaviour for a low number of craters - which is typically the case for a buffered count.  It requires adding a 'perimeter' value for the counting area into the .scc file - by hand, for now.


dirPagination: problem injecting the Module

Hi, I'm having toruble injecting the module into my angular application.
Is angularUtils.directives.dirPagination the name?
Thanks!

Hi,
Yes, that is the correct name. It would help if you provided any error message and also the code where you declare the module.

Hi Michael,
Thanks for reaching out. I'm doing this in a node.js application. And following your instructions with npm install angular-utils-pagination.
And then when I did: angular.module('myApp', ['angularUtils.directives.dirPagination']);
I am getting an injector error.
I also tried to link to the files directly but that doesn't seem to be helping either. Any suggestions here?

Are you including the dirPagination.js file in your app? And how do you use this in a node app? I've not heard of that before.

Yes, that's correct. I did include dirPagination.js with simple script tags. But your basic example isn't working for me.

I got the same problem too
 i only use one controller

@capc That is not enough for me to go off. It looks from that error like you might have an error finding a module named "academian".
@rhsu0268 I cannot do anything on this without seeing some more code and context of how this is used in a node app. Can you paste the exact error?

Hi Michael,
Thanks for getting back to me. I was able to get the sample code from plank to work with my project.
@capc: I think a good approach is to take Michael's sample code from plank as a starting point. If you can get his example to work, then adapt it to fit whatever data you need to display in your project.

@rhsu0268 Good to hear. I'll close this issue now.

@rhsu0268 Thanks man! It worked..

@michaelbromley With this plugin, is it possible to have 2 paginations on the same page?? I tried but it seems both paginations got synchronized somehow.


Broker publish acknowledgement?

I tried to test my broker as shown in a Lua client library, and I found that the Lua client hangs waiting on a acknowledge to a publish message.  I may have missed something when I built my broker off your test broker, but could you point me to the relevant handler function that is supposed to handle this?
https://github.com/xHasKx/luamqtt/blob/master/tests/spec/mqtt-client.lua#L180

I don't understand what are you asking. Could you elaborate your question?
Here is the document:
https://github.com/redboltz/mqtt_cpp/wiki
You can set publish response handler using https://redboltz.github.io/contents/mqtt/structMQTT__NS_1_1callable__overlay.html#a4910da7101ee9d3fa46ec576a1ebafd7

Thanks, your documentation at https://github.com/redboltz/mqtt_cpp/wiki/automatically-sending-messages was exactly what I was looking for.


[Notion Page Search] Failed to query: invalid input

Extension
https://www.raycast.com/reckoning-dev/search-notion
Description
The extension was working properly, but now I'm only getting 'Failed to query: invalid input'.
I can't figure out what caused this error.

Steps To Reproduce

install and configure the extension
search anything
Mac mini M1, macOS 13.4

Current Behaviour
No response
Expected Behaviour
No response

same issue

Same here

there was some change in underlying notion search API. I am looking into it. However, I am traveling so it might take a little time. Any contributions would be welcome.

There's broken change from notion. It has to do with a query type which now accepts an object as a value rather than a string. Patiently waiting for a fix 🤞🏾


Add new remote store options to Snapshot API

Fixes #4600
Fixes #4599
Fixes #4598

Checklist

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

@harishbhakuni: Adjusted.


[1186] Add babel-polyfill to fix JavaScript exception in IE11

Context
https://trello.com/c/lofoBlCW/1186-bug-conditional-fields-not-opening-in-ie11
Changes proposed in this pull request

Add babel-polyfill to fix JavaScript exception in IE11

When babel converts ES6 to ES6, it creates a helper function called _toConsumableArray(). This blows up because IE11 doesn't have the necessary polyfills.
Guidance to review
As you can see in the screen scapture shows JS behaviour is working now. Feel free to try it out in a real IE11 browser.

 Bug: Conditional fields not opening in ie11


Update netlink and manually bring up host side veth interface

This resolves the txqlen not being 0 on the host side interface problem.

LGTM

LGTM


gridss-1.1

Pull Request Checklists
Important: When opening a pull request, keep only the applicable checklist and delete all other sections.
Checklist for New Module
Checklist for Updated Module
Updated the changelog to reflect new module features.
Decided on a minor version update.

I'm preserving the old v1.0 so that the new reference files don't force a re-run of all existing GAMBL GRIDSS jobs.

IMPORTANT: I will migrate these files to modules/gridss/1.1 after this PR review is completed.

Won't all existing cases re-run anyway because there won't be an output in the version 1.1 directory?

Is this ready for re-review?

Let me try the bash script function exporting first!

This is ready for review. To address your question about versioning, I haven't run the hmftools module (which is part of this PR) on GAMBL yet. I have run the gridss module on most of GAMBL. The only difference between gridss 1.0 and 1.1 is the auto-downloading of reference files, so I plan to run gridss 1.0 on all of GAMBL and not run 1.1 on GAMBL at all.

Last approval, I promise!


fix Fix slf4j contains multiple SLF4J bindings

What problem does this PR solve?
Issue Number: close #45932
Related PR: #
Problem Summary:
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[x] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall

20:41,608 [ThreadedStreamConsumer] [INFO] Running org.apache.doris.load.loadv2.etl.SparkEtlJobTest java.lang.NoSuchMethodError: org.apache.logging.slf4j.Log4jLoggerFactory: method 'void <init>()' not found         at org.slf4j.impl.StaticLoggerBinder.<init>(StaticLoggerBinder.java:53)         at org.slf4j.impl.StaticLoggerBinder.<clinit>(StaticLoggerBinder.java:41)         at org.slf4j.LoggerFactory.bind(LoggerFactory.java:150)         at org.slf4j.LoggerFactory.performInitialization(LoggerFactory.java:124)         at org.slf4j.LoggerFactory.getILoggerFactory(LoggerFactory.java:417)         at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:362)         at org.apache.commons.logging.impl.SLF4JLogFactory.getInstance(SLF4JLogFactory.java:155)         at org.apache.commons.logging.impl.SLF4JLogFactory.getInstance(SLF4JLogFactory.java:132)         at org.apache.commons.logging.LogFactory.getLog(LogFactory.java:273)         at org.apache.hadoop.fs.FileSystem.<clinit>(FileSystem.java:186)         at java.base/java.lang.Class.forName0(Native Method)         at java.base/java.lang.Class.forName(Class.java:467)
after the fix, fe can print aws sdk debug log, but met new problem above


[BUG] Stream Side-Menu closes itself while typing in the search box(Mobile Web).

From @MadElf1337 on December 25, 2017 12:38

Whenever a person tries to search for a stream in the search box provided in the streams' side menu in
mobile version of chat.zulip.org, the menu closes itself.

Instructions For Reproduction:-

Go to https://chat.zulip.org from the mobile browser.
Login with valid credentials.
Open the side-menu of streams.
Tap on the search box to enable keyboard.

After following these steps, one sees that the menu closes itself to go back to the home-page.
(GIF attached below)

Copied from original issue: zulip/zulip-mobile#1631

From @tommyip on December 25, 2017 16:50
Hey @MadElf1337 this repo is for the react native app. What you are showing is the webapp in a mobile browser, can you move this to the zulip/zulip repo?

I can't reproduce. Can you provide a more detailed description or point out what I didn't do in order to reach where you encountered the bug?

Note: what's shown above is the intended behavior, if you were trying to search in the search input through a different method it's likely that it's unintended user behavior, which would call for an UI/UX enhancement and not bug resolvation.

@synicalsyntax Well, it is still happening to me and also happened to Tommy Ip, as he said that he can reproduce
What browser are you using?

@MadElf1337 Did you take the exact steps shown in the GIF? I tested through mobile view (developer tools) on Chrome & Firefox Mac OS X, and Safari iOS for actual mobile testing.

Yeah..I took them exactly like that

Well, it may not work like that on mobile view, try it on an actual android phone

@tommyip Are you on Android? If so this may just be an Android problem.

@synicalsyntax it's either an Android/Chrome issue, or a performance issue. I can reproduce it semi-reliably in Android 8.1. Sometimes the sidebar just collapse instantly, or the search box appear but then the sidebar closes after a second.

Yeah, likely an Android only issue then, since I can't reproduce on iOS/Safari. Pinging @brockwhittaker to see what's up.


node: wrong state root after 100 blocks unwind

While using the staged_pipeline utility to check the unwind procedure on mainnet, this sequence of commands:
staged_pipeline --exclusive --datadir ~/silkworm reset_to_download --keep_senders --force
staged_pipeline --exclusive --datadir ~/silkworm debug_unwind --height 1000000 --stop_before_stage HistoryIndex --step 100 --force

produces a state root mismatch at the end of InterHashes stage:
  INFO [12-13|23:35:57.238 UTC] Debug unwind datadir: ~/silkworm
  INFO [12-13|23:36:03.298 UTC] Reopen snapshot repository folder: ~/silkworm/snapshots
  INFO [12-13|23:36:05.043 UTC] Total reopened bundles: 72 max block available: 20'474'999
  INFO [12-13|23:36:05.043 UTC] Debug unwind: forward to=1'000'000 START
  INFO [12-13|23:36:05.043 UTC] ExecutionPipeline                        Forward start
  INFO [12-13|23:36:05.043 UTC] [5/13 Execution]                         op=kForward from=0 to=1000000 span=1000000
  INFO [12-13|23:36:35.045 UTC] [5/13 Execution]                         block=808791 blocks/s=26959 txns/s=37191 Mgas/s=1525
  INFO [12-13|23:36:48.233 UTC] Flushed state                            size=41.53 MB in=597.315ms
  INFO [12-13|23:36:48.489 UTC] [5/13 Execution] commit                  batch time=254.726ms
  INFO [12-13|23:36:48.490 UTC] [5/13 Execution]                         op=Forward done=43.446s
  INFO [12-13|23:36:48.490 UTC] [6/13 HashState]                         op=kForward from=0 to=1000000 span=1000000
  INFO [12-13|23:36:48.490 UTC] [6/13 HashState]                         clearing=HashedAccount
  INFO [12-13|23:36:48.490 UTC] [6/13 HashState]                         clearing=HashedStorage
  INFO [12-13|23:36:48.490 UTC] [6/13 HashState]                         clearing=HashedCodeHash
  INFO [12-13|23:36:49.177 UTC] [6/13 HashState]                         op=Forward done=686.119ms
  INFO [12-13|23:36:49.177 UTC] [7/13 IntermediateHashes] begin          op=kForward from=0 to=1000000 span=1000000
  INFO [12-13|23:36:49.177 UTC] [7/13 IntermediateHashes]                clearing=TrieAccount
  INFO [12-13|23:36:49.177 UTC] [7/13 IntermediateHashes]                clearing=TrieStorage
  INFO [12-13|23:36:49.696 UTC] [7/13 IntermediateHashes]                op=Forward done=519.858ms
  WARN [12-13|23:36:49.696 UTC] Stopping ...                             STOP_BEFORE_STAGE=HistoryIndex hit=true
  INFO [12-13|23:36:49.697 UTC] ExecutionPipeline                        Forward done
  INFO [12-13|23:36:49.697 UTC] Debug unwind: forward to=1'000'000 END
  INFO [12-13|23:36:49.697 UTC] Debug unwind: unwind down to block=999'900 START
  INFO [12-13|23:36:49.697 UTC] ExecutionPipeline                        Unwind start
  INFO [12-13|23:36:49.697 UTC] [7/13 HashState]                         op=kUnwind from=1000000 to=999900 span=100
  INFO [12-13|23:36:49.707 UTC] [8/13 IntermediateHashes] begin          op=kUnwind from=1000000 to=999900 span=100
 ERROR [12-13|23:36:49.720 UTC] Wrong trie root                          expected=0x33d8f415fd268da5169ac45e5f8e07fe710c23e4f4e2a318a9f781bd196f2066 got=0xd418c5ebada98e1a1bad2cec92153ca2baaa6cb6a543b23ae4968307b7192bc9
 ERROR [12-13|23:36:49.720 UTC] [8/13 IntermediateHashes]                function=unwind exception=kWrongStateRoot
 ERROR [12-13|23:36:49.720 UTC] [8/13 IntermediateHashes]                op=Unwind returned=kWrongStateRoot
 ERROR [12-13|23:36:49.720 UTC] ExecutionPipeline                        Unwind interrupted due to stage IntermediateHashes failure
Error: unwind failed: kWrongStateRoot

Keeping the bug open, as I understand the bug is not fixed, but only workarounded by using a different (slower?) code path.
See TODOs in stage_interhashes.cpp InterHashes::forward and InterHashes:: unwind

Keeping the bug open, as I understand the bug is not fully fixed, but only workarounded by using a different (slower?) code path. See TODOs in stage_interhashes.cpp InterHashes::forward and InterHashes:: unwind

Yep, see also #2596


.timestamp() generates NOT NULL column by default

Environment
Knex version: 0.15.2
Database + version: MySQL 5.7
OS: Ubuntu 16.04.5
Feature discussion / request
When using the .timestamp() schema builder method, the generated timestamp column is NOT NULL by default. This is contrary to other data types like .integer() and .string() that produce nullable columns by default.
This is a bit surprising and there's no mention of it on the documentation.
All column building methods should behave consistently whenever possible, and there's no reason to think that adding a simple timestamp column would make it NOT NULL by default.

That works for me.

I actually started to write out design guide + initial API spec for  my vision for new schema builder API... I've been thinking about it so long that maybe I'll just write it as a next feature I'll implement.
Have to lookup old rants where I have talked about it earlier :)


Allow gitlab.com handle in addition to github.com handle

User story
As an open source software contributor, I can find out where the source code is hosted so that I can contribute.
Best solution for this problem
Either allow a generic "source code" field or add the second most common one gitlab.com.
MVP
Button with link displayed on collective's page.
Metrics
Number of collectives making use of this feature.
Documentation
Self explanatory.

Already implemented in https://github.com/opencollective/opencollective/issues/5645, you can put any link in "Repository URL":

We're still surfacing a "Github handle" input in the Open Source apply flow, which is going to be updated with https://github.com/opencollective/opencollective/issues/5645.
Please let me know if I missed another case.

Thanks for the hint! I found the field repositoryUrl in the API now :smile:


Cleanup http: links

Move http: links to https: where possible and update old links. Also removed broken links.
I was triggered by the old werkzeug link in runserver_plus.

Awesome thanks !


Customers never reach mollie payments portal

Expected behavior
A customer should reach the mollie payment portal for ideal and other NL-foreign payment methods.
Actual behavior
In certain cases the customer never reaches the mollie payment portal.
Steps to reproduce the behavior
Add a product to the cart and try to check out. Doesn't always occur, though with specific products in our shop it always occurs. Checked the products, but unable to find flaws there. Checking out using wiretransfer (/direct bank) will work, so to me it should have something to do with the Mollie plugin specifically.
Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 3.3.5
PHP version: 7.2.19-0ubuntu<IP_ADDRESS>

Could you grant backoffice access so we could investigate why in your case mollie payment is not loading?
Please don't share it publicly and send it to my email<EMAIL_ADDRESS>


add a link for tenants who need more help

eg, legal aid? How to appeal to the rent board?
confer with rent board reps re: what this should include

PTLA might be a good link
https://www.ptla.org/self-help/2652


Windows installer

How to work on Windows without the symlinks?
Should we use cygwin or any other unix tools for win?
How about windows10 ubuntu shell? Can we support only this version?

Currently, we are able to produce dvc installer for windows by running .\build.cmd from windows terminal.
The script itself(build.cmd) has sufficient explanation about requirements and has info on where to get them.
When releasing new version of dvc, windows installer should be built and binary should be included into release as described here: https://help.github.com/articles/creating-releases/ .
Here is a test installer for dvc-0.8.4 to try it out: https://drive.google.com/file/d/0BxlwKSeztewYM0t6ckdIUm1ESFE/view?usp=sharing .
About symlinks: currently we are able to work with symlinks by adding CreateSymLink privilege to users(separate checkbox in the installer), but it would be nice if we could also work without symlinks(if at all possible).
Not closing this issue for now.

Added feature request for symlink-less dvc.
Verified that windows installer works for other people too, thus closing this issue now.


[Docs] broken link

Current State
the link https://docs.meshery.io/guides/creating-a-meshery-design is broken
Desired State
On Creating a Meshery design page, a redirect from https://docs.meshery.io/guides/creating-a-meshery-design could be added to point to https://docs.meshery.io/guides/configuration-management/creating-a-meshery-design

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

The correct link is this https://docs.meshery.io/guides/configuration-management/creating-a-meshery-design

Hey can I fix this issue it will be a great start for me to explore the repo as well

Not sure where the link is broken, from the guides page the link to Creating a Meshery Design points to the right location

Hi, @vishalvivekm can I work on this?

You can access "Creating a Meshery Design" at "https://docs.meshery.io/guides/configuration-management/creating-a-meshery-design". Hope the issue gets closed.

Thank you, folks.
Following FCFS,
@Souravpakhira, it will be helpful if you could raise a PR with the fix so it can be reviewed.
@MonalikaPatnaik Please find a bunch of good-first and help wanted issues that you might wanna work on:
https://github.com/issues?q=is%3Aopen+is%3Aissue+archived%3Afalse+org%3Alayer5io+org%3Ameshery+org%3Aservice-mesh-performance+org%3Aservice-mesh-patterns+label%3A"help+wanted"+

creating-a-meshery-design

Can you please help me identify where the broken link is? Knowing the exact page will make my work easier.

On Creating a Meshery design page, a redirect from https://docs.meshery.io/guides/creating-a-meshery-design could be added to point to https://docs.meshery.io/guides/configuration-management/creating-a-meshery-design

@Souravpakhira adding redirect_from: guides/creating-a-meshery-design
in guides/configuration-management/creating-a-meshery-design page's frontmatter will work.

when you google, creating-a-meshery-design, https://docs.meshery.io/guides/creating-a-meshery-design comes up as result and is broken.


[WIP] Email App on new API

[x] Upgrade to electron forge
[ ] Implement new Auth IPC
[x] faking authentication for test/development
[ ] fully implement IPC against authenticator
[x] Remove old crufty API and restructure for UI-Oriented Actions
[x] Load Configuration from our own Container (accounts)
[ ] Create new Email based on existing publicNames

[ ] Generate random inbox with random encrytion keys, filed under
{_publicNames}.email

[ ] Switch to creator when no emails found yet
[ ] Load emails from Inbox for account(s)
[ ] Send Emails by looking up publicNames email-Service

r? @krishnaIndia
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Create tasks.json

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Proposed Changes

...
...
...

Relevant issues
Closes #...
Closes #...
Closes #...
Related #...

I think this is another oopsie.


Email validation with pattern: email

Using version 1.6.6 and Email validation with pattern: 'email' is not validating the emails with certain formats
Ex<EMAIL_ADDRESS>is passed the validation. But .c is not a valid tld for now.
Can you please let me know if this is fixed in later releases.

Your specific case is a valid email. This kind of verification (is it a real email?) is done by sending a confirmation email to the user. The pattern is correct in se.

Any PR is welcome to improve the email regexp :)


Compatibility with Physics Mod Pro Questionable.

My legs get stuck in this running animation in first person after a couple steps. In third person it triggers for seemingly random amounts of time at random, which makes it hard to take a picture of. I tried several times.
I just kinda only now noticed this somehow while placing dirt. I've been looking more or less straight forward for a WEEK while digging a hole, so idk how long it's been an issue. Legs are generally kinda visible though so I don't think it's older than today at least.
Here's my latest log.
latest.log
I have quite a few mods, but I don't believe any are ones that edit player animations except your two. I do have a speed enchantment on my boots, so maybe the animation is playing too fast and getting stuck? I'll test in another world real quick, but here ya go.
Our last interaction wasn't awesome, I want to say that I completely understand your stance and accept any and all decisions you make - not that this has any real impact on you personally. At the very least, you know I'm not trying to give you a hard time, I just like things to do things <3

I can't get this to occur at all in a creative world that is a slightly older copy of the survival one in that screenshot. I've got the speed boots AND a potion on, and I tested with normal speed. Nada.
Maybe reloading fixed it.

No, reloading didn't fix it. This is unique to the survival world.

Possible mod compatibility issue

I can't recreate this in a side world in any realistic amount of time, so I'm limited in what I can do. I started by disabling 2 obvious character mods that I don't use. Neither affected the behavior.
If I continue disabling mods I will break my world, so this really isn't easy to pinpoint.

I've more or less disabled every mod that interacts with the player itself or the inventory. The problem seems to occur under and above Y = ~70 and Y=75. Weirdly, going uphill temporarily dissipates the issue, when it appears in short fraction of a second bursts, but never to the extent of seeing my leg all the time.
I also only now noticed that the entire walking and running animations system stops working. Neither my legs nor arms move and my body doesn't bob, only my camera.
Again, no idea what could possibly be doing this. I've disabled at least 24 mods by now and the only mods left after this batch are either very light content additions with few or no items or player interacting mechanics or full on performance mods. Things like Thin Air, Cold Sweat, Hearth and Home, and It Takes a Pillage are among these, as well as world gen overhauls with no new blocks, etc. The usual slew of performance mods like Embeddium, Ferrite Core, and many more. You can see my mod list in the latest log above.

Oh, and I figured out that a full current copy of the world will allow me to recreate the issue in some areas. Problem is, I can't remove too many or the world no longer works and I can't reproduce the issue.

It is 100% unrelated to chunk border alone. In the same chunk I am walking normally and abnormally, usually divided by a Y value, but not necessarily. It does seems to be more buggy in some chunks vs others, and I've noticed that it also relates to a weather bug I've had that makes some chunks not update their skybox color or even incorrectly adjust their skybox to rainy weather after none such events occurred.
So right now I'm testing each and every chunk loading mod I have on individually. Seems to me that this is a chunk update problem, not a FPM problem.

Physics Mod Pro is the culprit. Among 196 mods (not my whole list), loading Not Enough Animations and Physics Mod Pro together cause this issue.

Huh, I had physics mod and NEA running together a lot without issues. To be fair, haven't played much with the full/pro version. I know that there was some mod that caused with NEA to sync the crouch between all players for whatever reason, but the walking animation freezing is weird. That would I think require that the player itself doesn't get ticked/the tick not getting noticed by NEA.

Yeah, I have as well and am dismayed. I'm currently trying to load with the bare minimum requirements on that world, but it's looking like it's too baked to load without at least half the mods present. Instead, unfortunately, I think I will have to do an A-Z removal check to see if there's something else interacting with either mod.

And a new world just doesn't have the bug right off the bat like I need.

I'm now completely rid of all but the core mods that I couldn't remove without crashing. I'm 1 by 1 removing each, thoughwithout EVERY content addition mod besides Terralith, Tectonic, Continents, Nullscape, Physics Mod Pro, First Person Model, Player Animator Lib, and Not Enough Animations I still have the issue. This leads me to believe that this is a direct incompatibility between PMP and NEA.

This is the final mod set I require to load the world. None of the extras mess with anything but world gen.
The issue persists.

A 3rd person image of the issue for better clarity. The entire walking animation stops, including when sneaking.

Further removing Continents and Tectonic does not resolve it. Blueprint cannot be removed, the world doesn't load.

Found the issue.

This needs to be OFF

Congrats! bro

As I said, Animation Smoothing needs that the player knows when it gets ticked. So Physics mod pro breaks that somehow.


Added FAQ Info of VM types with no temporary disk.

Added FAQ Info of VM types with no temporary disk, because the statement Every VM contains a Temporary disk in no longer relevant due to Introduction of new VM Sizes like Dv4, Dsv4, Ev4, Esv4 series machines.

@EarthlyAlien : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@EarthlyAlien Thanks for making this change and helping improve our docs, we appreciate it! It looks like there's a merge conflict with this PR. Would you mind fixing it? I'll sign off after that.

I'm gonna create new PR for this. Mark this a closed for now.


jsonnet: better name for prometheus PrometheusRule object

The previous name resulted in k8s-rules by default and was confusing.

lgtm :+1:


Android example throws runtime error.

in  void OpTensorSyncDevice::init()
for (std::shared_ptr<Tensor> tensor : this->mTensors) {
        if (!tensor->isInit()) {
            throw std::runtime_error("Kompute OpTensorSyncDevice: Tensor param "
                                     "has not been initialized");
        }

coming from void KomputeModelML::train
sq->record<kp::OpTensorSyncDevice>({ wIn, bIn });

in the app's code.

Thank you for reporting this @tyler274 ! This is due to a change that was added this weekend to support asynchronous and parallel operations, namely  #75 and #73.
The fix for this is simple, and you have two options:

Running the example against release 0.3.2 (Which means just checking out that branch)
Add the same changes that were added in #75 to the logistic regression test (which uses the same code)

For option #2, you will need to run the tensor creation on a separate sequence.
Namely, you have to change this part:
        if (std::shared_ptr<kp::Sequence> sq =
                    mgr.getOrCreateManagedSequence("createTensors").lock()) {

            sq->begin();

            sq->record<kp::OpTensorCreate>(params);

            sq->end();
            sq->eval();

            // Record op algo base
            sq->begin();

            sq->record<kp::OpTensorSyncDevice>({ wIn, bIn });

To this:
        {
             std::shared_ptr<kp::Sequence> sqTensor =
              mgr.createManagedSequence().lock();

            sqTensor->begin();
            sqTensor->record<kp::OpTensorCreate>(params);
            sqTensor->end();
            sqTensor->eval();

            std::shared_ptr<kp::Sequence> sq = mgr.createManagedSequence().lock();

            sq->begin();

            sq->record<kp::OpTensorCreate>(params);

            sq->end();
            sq->eval();

            // Record op algo base
            sq->begin();

            sq->record<kp::OpTensorSyncDevice>({ wIn, bIn });

This is something you can find in the test/TestLogisticRegression.cpp file.
For now, I recommend you to go for option 1 - I will update the examples now so they work with the recent 0.4.0 release.

@tyler274 this now has been fixed in master, if you pull the latest you should be able to build - I just tested it with Android studio as well:

I'll re-open issue to confirm it has been fixed on your side.

Thanks I'll check out master in the morning.
On Tue, Oct 20, 2020, 2:08 AM Alejandro Saucedo<EMAIL_ADDRESS>wrote:

@tyler274 https://github.com/tyler274 this now has been fixed in
master, if you pull the latest you should be able to build - I just tested
it with Android studio as well:
[image: image]
https://user-images.githubusercontent.com/1447507/96546876-02bf6900-12a3-11eb-9722-0d48e7c64093.png
I'll re-open issue to confirm it has been fixed on your side.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/EthicalML/vulkan-kompute/issues/77#issuecomment-712616794,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7UPAY3IJWP7NLRCAX5EW3SLUSOZANCNFSM4SXS5K4A
.

Works on master, thanks!

Great - thanks for confirming @tyler274


🐛: switchboard/circuit appears multiple times

Description

Makes the circuits and switchboards appear just once please :)

@MagnusGjosund for now - can we simply do a unique in FrontEnd?

PR: https://github.com/equinor/lighthouse-client/pull/1797

Taking this out of Must Have as we have a front-end fix. Closing for now.


ValueError: need more than 1 value to unpack

EDIT: Found that I didn't assign a variable to scanner.scan(gray), assigned variable and error still there.
On Raspberry Pi3, Python 3.4, OpenCV3 I get the below error. Attached is code, error, and additional information. CODE:
import cv2
import zbar
import constants

camera = cv2.VideoCapture(0)

#initialize zbar reader
scanner = zbar.Scanner()

def capture(camera):
	while True:
		#end program if q is pressed
		if cv2.waitKey(1) & 0xFF == ord('q'):
			break

		#begin grabbing frames from the camera
		frame = camera.read()

		#convert frames to grayscale as zbar-py only accepts grayscale opencv
		gray = (frame, cv2.COLOR_BGR2GRAY)

		#scan frame for qrcode/barcode
		zbar_image = scanner.scan(gray)

		#print decoded data in frame
		for decoded in zbar_image:
			print(decoded.data)

capture(camera)

Error:
Traceback (most recent call last):
  File "Main.py", line 38, in <module>
    capture(camera)
  File "Main.py", line 32, in capture
    scanner.scan(gray)
  File "/home/pi/.virtualenvs/cv/lib/python3.4/site-packages/zbar/zbar.py", line 191, in scan
    height, width = image.shape
ValueError: need more than 1 value to unpack

When printing out the value of "gray" sometimes randomly I get
((False, None), 6)

And just as randomly I get
((True, array([[[159, 156, 166],
        [159, 156, 166],
        [161, 158, 168],
        ...,
        [ 75,  86,  77],
        [ 82,  85,  77],
        [ 79,  82,  74]],

       [[158, 155, 164],
        [160, 157, 166],
        [156, 159, 167],
        ...,
        [ 70,  85,  71],
        [ 79,  82,  73],
        [ 77,  80,  71]],

       [[156, 156, 166],
        [157, 157, 167],
        [156, 158, 170],
        ...,
        [ 70,  84,  73],
        [ 74,  79,  74],
        [ 73,  78,  73]],

       ...,
       [[231, 239, 233],
        [230, 238, 232],
        [229, 237, 231],
        ...,
        [103, 120, 106],
        [107, 122, 108],
        [112, 127, 113]],

       [[231, 239, 233],
        [230, 238, 232],
        [229, 237, 231],
        ...,
        [103, 122, 108],
        [106, 125, 111],
        [109, 128, 114]],

       [[232, 237, 233],
        [230, 235, 231],
        [233, 234, 231],
        ...,
        [109, 124, 110],
        [112, 127, 113],
        [116, 131, 117]]], dtype=uint8)), 6)

But it always no matter what the result is, it always gives me the ValueError exception.

		#convert frames to grayscale as zbar-py only accepts grayscale opencv
		gray = (frame, cv2.COLOR_BGR2GRAY)

This does not convert a frame to grayscale. It makes a 2-tuple of the frame and the cv2 constant. And sometimes frame is None, I guess. None of this is a zbar problem.


Feature request: export more ptvsd-related commands as api

Internal Team Ask (vscode-ros)

currently only getRemoteLauncherCommand() is exposed in vscode-python's api interface.
meantime, as described in ptvsd documentation, ptvsd could be used at entry (to launch a script/module), or while the (Python) process is running (to inject a ptvsd instance)
It would be nice if a new getInjectDebugServerCommand() api could be exposed.

TODO

If we go down this path, we'd need to document the API and create an npm package (https://github.com/microsoft/vscode-python/issues/3121)

Closing it since once the work for #1078 is done, this API won't be needed as VS Code will be able to automatically connect to a process you want to connect to.

Closing it since once the work for #1078 is done

thanks for the update, is there any pull requests related to #1078? not able to find links from that issue's page

@kejxu Not yet, it's marked as "needs spec" so we can finish some design decisions before we work on it, but we may start working on it soon :)


Create maven.yml - Github action

Create a GitHub action to automate the build and the test of the project on GitHub at every push and at every Pull request to the main branch.

It's fine for this PR but I suggest we start fixing the tests in subsequent PRs


Create transforms for NUnit 3

There should be four transforms, matching the existing four functionally but written for use with NUnit 3 XML input.

@CharliePoole / @rprouse - the v3 Default summary xslt I am almost done with differs with the console runner output from the xml v3 output
Taking the attached screenshot:

and comparing with this output:
Runtime Environment
   OS Version: Microsoft Windows NT 10.0.10586.0
  CLR Version: 4.0.30319.42000

NUnit Version: <IP_ADDRESS>

Test Files
    C:\Users\mark.harrison\Documents\visual studio 2015\Projects\ClassLibrary1\bin\Debug\ClassLibrary1.dll

Run Settings
    DefaultTimeout: 900000
    WorkDirectory: C:\Users\mark.harrison
    ImageRuntimeVersion: 4.0.30319
    ImageTargetFrameworkName: .NETFramework,Version=v4.5
    ImageRequiresX86: False
    ImageRequiresDefaultAppDomainAssemblyResolver: False
    NumberOfTestWorkers: 8

Test Run Summary
 Overall result: Skipped
 Test Count: 6, Passed: 2, Failed: 0, Inconclusive: 0, Skipped: 4
   Skipped Tests - Ignored: 2, Skipped: 2, Other: 0
 Start time: 2016-09-07 14:43:56Z
   End time: 2016-09-07 14:43:56Z
   Duration: 0.099475 seconds

Tests Skipped:
1) The_TEST : Some Explicit reason
2) IgnoreMe : Some Ignore reason

is it ok that there is a difference e.g. xml output says "Skipped", and console says "Warning" ?

Also, would you like the 4 new v4 transforms embedded in the nunit-summary project for now?
If so, how do we want to call them, or do we want to detect based on the root node of the inputs e.g. test-results vs test-run ?

For now, I'd say just use the same names as the existing ones with V3 appended.
When we are working issue #6, it may change.

@harrisonmeister BTW, note that our html transforms do not include the html or body elements. That's because we are sometimes creating output in a single file, sometimes in multiple files. The C# injects those elements at the start and end of each file as needed.
OTOH, if we publish these separately in the nunit-transform project we will probably want to include those elements.

No problems, I'll append v3 to them and exclude html and body. Adding them in later for #6 shouldn't be an issue.

@harrisonmeister Still one transform to go right?

@CharliePoole - correct, I have done 3 of 4. Hopefully get the 4th (HtmlTransform-v3) done today.

Created PR #10 to complete HtmlTransform-v3 and updated the summary one too to be consistent.

@harrisonmeister I believe this is done - feel free to re-open if not.


[Enhancement] - Upgrade local CSS from v6 to v7 and Inrupt packages that are Node 18 compatible

Describe the Current Behavior/Feature:
With the existing Node 16 no longer being supported, there are package dependencies that are no longer supported (as noted from a recent PR https://github.com/codeforpdx/PASS/pull/532#issue-2001156984). There are packages for Community Solid Server v6 that are also unsupported, namely "oidc-provider," which uses oidc-provider v7.
From the CSS documentation, it appears they've made the jump to Node 18 (https://communitysolidserver.github.io/CommunitySolidServer/7.x/usage/starting-server/) in CSS v7. For Inrupt's JS Client libraries, they've also mentioned they support Node 18 and higher (https://github.com/inrupt/solid-client-js/#nodejs-support).
Proposed Behavior/Feature:
For this enhancement, I propose we update our version of CSS from v6 to v7, if possible, and update any related packages/files/configurations so that they'll be Node 18 compliant.
We could do the same thing for any Inrupt libraries that's no longer supported.
Rationale:
Will keep existing libraries Node 18 compliant.
It's not too urgent at this moment in development, but it would be important once we're in production.

@timbot1789 Had some vulnerabilities pop up with some of the dependencies that are related to CSS.

I have attempted to resolve the issue with npm audit fix, but it was not enough, unfortunately. We'll have to update the existing version of CSS for the local server. Was wondering if we'll need to update it or it'll be fine since we're just using CSS as the local development server on our machine?


json dictionary_serializer

an attempt to address https://github.com/pola-rs/polars/issues/5723.  please lmk if there is anything that I'm missing here, as this is my first introduction/contribution to the lib.  thanks!

this will need to be rebased against the newer branch if it is to be merged.  lmk if you'd like I can do that.

closing in light of upcoming changes


The results obtained through group by are not correct

For English only, other languages we will close it directly.
Please answer these questions before submitting your issue. Thanks!
Which version of Sharding-Sphere do you using?
sharding-jdbc-core 2.0.3
Expected behavior
like https://github.com/sharding-sphere/sharding-sphere/issues/128
item|count
a|100
b|200
Actual behavior
but
a | 300
Reason analyze
SELECT
a.item AS item,
a.item1 AS item1,
COUNT(a.id) AS total
FROM
(
SELECT
id,
item,
item1
FROM
order
WHERE
area_id = '123'
) a
GROUP BY a.item,a.item1
When there are subqueries in the SQL statement, the result of group by is incorrect.
Steps to reproduce the behavior
For bug report, please MUST provide the reproduce example codes (such as a github link).

hello, please note, we don't support this SQL, just reference: http://shardingsphere.io/document/current/en/features/sharding/usage-standard/sql/


Enhance the performance of merge report

I need to test hundred of  machine with hundred of case. By use distrbuted cluster to run the case, it cost 1 minute to finish the case,but sadly 10 minute to merge the report from xml to html。
I sinerely need a way speed up

XML processing is already now performance optimized. There may be something special in your case, but that's something only you can investigate. This is what you can do:

Use something like the Fil profiler for inspecting where the memory is spent.
Change Robot's code where the memory is spent and see does it help by running the profiler again.
If you don't feel comfortable changing the code, just telling what the profiler finds here can help. The problem is that we cannot validate do changes help in your environment.

Are you interested to do the above or investigate the issue otherwise? If it sounds too complicated and too much work, you can just try buying more memory and see does it help.

Thank you for your advice, the most bottleneck in my case is not memory usage, but the time which cost in merge multi xml file to  html report.
I'd like to try to adjust the memory of server  and use profile tools.
by the way ,I had try to read the code , but it's real diffcult to understand because of  recursion.
anyway, I will to find more detail.

if you run out of memory, the machine starts swapping which is slow.


Inferred Typescript types don't seem to work well with variants

Description
Hi, I seem to have an issue with the types inferred by Unistyles.
Steps to reproduce
For example, I have this code which I got from the example in the Unistyles repo, and I modified it a bit match my situation.
It has 1 variant (type), which is used 2 times (for the button style and the buttonText style).
/* eslint-disable no-restricted-imports */
/* eslint-disable import/no-unused-modules */
import React from "react";
import { Pressable, Text, View } from "react-native";
import { createStyleSheet, useStyles } from "react-native-unistyles";

enum ButtonType {
  Primary = "primary",
  Secondary = "secondary",
}

const stylesheet = createStyleSheet((theme) => ({
  button: {
    borderWidth: 1,
    variants: {
      type: {
        primary: {
          backgroundColor: theme.colors.backgroundButtonPrimary,
        },
        secondary: {
          backgroundColor: theme.colors.backgroundButtonSecondary,
        },
      },
    },
  },
  buttonText: {
    fontWeight: "bold",
    variants: {
      type: {
        primary: {
          color: theme.colors.foregroundButtonPrimary,
        },
        secondary: {
          color: theme.colors.foregroundButtonSecondary,
        },
      },
    },
  },
}));

export const VariantsScreen: React.FC<{
  type: ButtonType;
}> = ({ type }) => {
  const { styles } = useStyles(stylesheet, {
    type,
  });

  return (
    <View>
      <Pressable
        accessibilityRole="button"
        style={styles.button}
        onPress={() => {}}
      >
        <Text style={styles.buttonText}>Change variant</Text>
      </Pressable>
    </View>
  );
};

I would expect TS to show me something like this as the type of the styles:
{
    button: {
        backgroundColor: string
    } & {
        borderWidth: number;
    };
    buttonText: {
        color: string;
    } & {
        fontWeight: "bold";
    };
}

However this is the type I'm getting (the style keys inside all the variants are missing):

Looks like it has something to do with the theme, because when putting the colors directly inline it seems to work:

I noticed that the types are correctly inferred when removing the secondary type from both variants (which leaves me with just the primary type):

Snack or a link to a repository (optional)
No response
Unistyles version
2.9.2
React Native version
0.73.6
Platforms
Android, iOS
Engine
Hermes
Architecture
Paper (old)

Thank you for the report. I will try to repro it and let you know

Tested it in Unistyles repo. I can't reproduce it.
My tests:

Create component with your styles (I use my own colors):

const stylesheet = createStyleSheet(theme => ({
    container: {
        borderWidth: 1,
        variants: {
            type: {
                primary: {
                    backgroundColor: theme.colors.accent
                },
                secondary: {
                    backgroundColor: theme.colors.aloes
                }
            }
        }
    },
    text: {
        fontWeight: 'bold',
        variants: {
            type: {
                primary: {
                    color: theme.colors.typography
                },
                secondary: {
                    color: theme.colors.fog
                }
            }
        }
    }
}))

Create component with variants:

enum ButtonType {
    Primary = 'primary',
    Secondary = 'secondary',
}

type ExampleScreenProps = {
    type: ButtonType
}

export const ExampleScreen: React.FunctionComponent<ExampleScreenProps> = ({ type }) => {
    const { styles, theme } = useStyles(stylesheet, {
        type
    })

    // ...
}

Result:

How themes are defined:
https://github.com/jpudysz/react-native-unistyles/blob/main/examples/expo/src/styles/theme.ts
Make sure that each theme has the same type.


Bug Report: game crashes after song ends

suddenly appeared out of nowhere when i was playing the x event mod then carried over to the other kade engine using mods (might be a keybind?? idk)
played on windows 10 home with version 10.0.19041 build 19041
version was <IP_ADDRESS> FNF - 1.2 Kade Engine
not tried anything to reproduce it
game just crashes after a song ends. might be something with the replays as the image shows

(error message varies from each song)

Do you have the replays folder in assets?

yep

yeah that only happens when you don't have that folder.

I also just realised you are on an outdated version of the game. So update.

ok so this problem is with outdated version ok


Upgrade go to 1.17 to fix build failure

Requires #10. Also use cimg/* image in CircleCI pipeline instead of the circleci/* image

Fixed in https://github.com/devatherock/slack-webhook-facade/pull/18, without upgrading to 1.14


Transactions table overflows off the screen due to long reference ID

Moved from https://github.com/wirecatllc/blesta-stripe-universal/pull/7
The Blesta admin client profile transactions table clips off the screen because of the long reference ids, and that hides the date and edit button.
My suggestion is to use the email address provided by Stripe and fallback to the ID when no email is provided. (not using real numbers, dont worry)
You can also see how the Blesta PayPal gateway does it too, I was using them previously - they use email address.

What it looks like with my suggested change

I understand email not being as unique but the transaction ID is already provided (That's what the "pi_" number is), which is what Stripe use to identify payments, you can see that by going to your Payments view in the Stripe dashboard.

At least, I don't think the checkout session ID should be used, Stripe doesn't normally provide that(You have to dig around in the events and logs section to find it), they're very long, and Stripe doesn't provide a filter for it which makes them hard to use.

I had a better idea:
Use the customer ID that Stripe provides for someone(they're a number that starts with gcus_ or cus_), they're short enough so they don't overflow and it's unique, plus Stripe provides a filter for customer ID in "More filters" where you can enter the ID to see all payments from that person.

Looking at this though, it seems the customer ID is rarely populated https://docs.stripe.com/api/checkout/sessions/object#checkout_session_object-customer only if the customer_creation is set to always.. but customer_email is populated so I'm not sure why. If thats due to the webhook being used?
It's why I used the customer_email in the first place because its populated else the customer ID can be used instead.

Yes Stripe seems weird about it. I'm not using a different theme or modified the CSS, so I'm not sure why there's a difference, maybe my font size is bigger or something, with how my browser/pc is setup.
I think the easiest option for now would be to trunk the reference ID or not have any reference ID at all, because it would be unnecessary and hard for us to modify the CSS of the table to accept longer fields. And emails are not unique so cannot use them.
The ideal way would be using the customer ID but Stripe is weird and it would require an extra database operation, which we should avoid if possible.
In the future, this could always be an option that people can change if it ever becomes a problem for some reason, that the reference ID is trunked.

What worries me now is we are fixing a problem that might not be a problem and we are breaking an existing fields. I think maybe just remove/comments out that field as it is not necessary useful (session id after transaction success/failed?)


Adding feature-flags to release configuration

This PR adds features and all_features as both flags to the CLI
as well as the release.toml configuration. This change allows you
to tell cargo-release to verify the package with certain features
enabled, in order to avoid needing to use no-verify for crates
that depend on certain features being enabled (or simply any
feature) before they can be uploaded.
This change is dependent on rust-lang/cargo#6453, which adds this
feature to the base cargo publish command.
The motivation behind this PR is the same as the above mentioned
cargo PR: reducing the amount of packaging errors that can happen
due to people not being able to verify their code because it has
some feature requirements for compilation, ultimately resulting in
a more healthy package ecosystem.

LGTM. Could you please also update README for new options?


Create docker image to be run by TravisCI

Docker image must be provided with multiple zsh versions so the same image is used in the travis matrix configuration.
This will reduce build and tests time greatly (currently ~3min)
See https://github.com/desyncr/antigen-docker

Done in https://github.com/zsh-users/antigen/pull/461


std.functional.memoize should be usable in @safe

greensunny12 reported this on 2018-01-03T03:36:14Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=18170
CC List

Ajieskola

Description

Ajieskola commented on 2021-11-01T17:35:39Z
Current status: the default `memoize` is `@safe` if it's function parameter is. But the size-capped `memoize` is not `@safe` yet.