dynamic_annotations symbols in the global namespace

Abseil dynamic_annotations (https://github.com/abseil/abseil-cpp/blob/master/absl/base/dynamic_annotations.h, https://github.com/abseil/abseil-cpp/blob/master/absl/base/dynamic_annotations.cc) define macros and functions without ABSL_ or Absl prefix (this is C code, so we cannot use namespaces).
This behavior can cause ODR violations and macros clashing. A possible solution would be to prefix all the macros with ABSL_ and all the functions with Absl. This will make Abseil compatible also with projects that are using something like https://code.google.com/archive/p/data-race-test/wikis/DynamicAnnotations.wiki.

Any thoughts from the Abseil team on this?
tensorflow appears to pull this in. That means it clashes with gperftools which also provides the same symbols.

@MirkoBonadei is working on renaming the symbols in thread_annotations.h, I don't know if he is also planning on working on renaming the symbols in dynamic_annotations.h. I wouldn't count on an immediate fix in any case.
The symbols in gperftools don't appear to be part of the public API of gperftools (they are part of Abseil's public API). In that case renaming them in gperftools might be an easier short-term solution.

Thanks @derekmauro. I renamed the gperftools symbols and it appears to work fine now.

@philsc Could you share the patch you did for gperftools? I ran into this same problem. :)

Here's what I did:
diff --git a/third_party/gperftools/src/base/dynamic_annotations.c b/third_party/gperftools/src/base/dynamic_annotations.c
index 87bd2ecde9..1b3cbff8cd 100644
--- a/third_party/gperftools/src/base/dynamic_annotations.c
+++ b/third_party/gperftools/src/base/dynamic_annotations.c
@@ -149,7 +149,7 @@ static int GetRunningOnValgrind(void) {
 }

 /* See the comments in dynamic_annotations.h */
-int RunningOnValgrind(void) {
+int GperfRunningOnValgrind(void) {
   static volatile int running_on_valgrind = -1;
   int local_running_on_valgrind = running_on_valgrind;
   /* C doesn't have thread-safe initialization of statics, and we
@@ -163,12 +163,12 @@ int RunningOnValgrind(void) {
 #endif  /* DYNAMIC_ANNOTATIONS_EXTERNAL_IMPL == 0 */

 /* See the comments in dynamic_annotations.h */
-double ValgrindSlowdown(void) {
-  /* Same initialization hack as in RunningOnValgrind(). */
+double GperfValgrindSlowdown(void) {
+  /* Same initialization hack as in GperfRunningOnValgrind(). */
   static volatile double slowdown = 0.0;
   double local_slowdown = slowdown;
   ANNOTATE_BENIGN_RACE(&slowdown, "safe hack");
-  if (RunningOnValgrind() == 0) {
+  if (GperfRunningOnValgrind() == 0) {
     return 1.0;
   }
   if (local_slowdown == 0.0) {
diff --git a/third_party/gperftools/src/base/dynamic_annotations.h b/third_party/gperftools/src/base/dynamic_annotations.h
index 4669315ced..71e205395b 100644
--- a/third_party/gperftools/src/base/dynamic_annotations.h
+++ b/third_party/gperftools/src/base/dynamic_annotations.h
@@ -501,23 +501,23 @@ void AnnotateFlushState(const char *file, int line);
   If for some reason you can't use "valgrind.h" or want to fake valgrind,
   there are two ways to make this function return non-zero:
     - Use environment variable: export RUNNING_ON_VALGRIND=1
-    - Make your tool intercept the function RunningOnValgrind() and
+    - Make your tool intercept the function GperfRunningOnValgrind() and
       change its return value.
  */
-int RunningOnValgrind(void);
+int GperfRunningOnValgrind(void);

-/* ValgrindSlowdown returns:
-    * 1.0, if (RunningOnValgrind() == 0)
-    * 50.0, if (RunningOnValgrind() != 0 && getenv("VALGRIND_SLOWDOWN") == NULL)
+/* GperfValgrindSlowdown returns:
+    * 1.0, if (GperfRunningOnValgrind() == 0)
+    * 50.0, if (GperfRunningOnValgrind() != 0 && getenv("VALGRIND_SLOWDOWN") == NULL)
     * atof(getenv("VALGRIND_SLOWDOWN")) otherwise
    This function can be used to scale timeout values:
    EXAMPLE:
    for (;;) {
      DoExpensiveBackgroundTask();
-     SleepForSeconds(5 * ValgrindSlowdown());
+     SleepForSeconds(5 * GperfValgrindSlowdown());
    }
  */
-double ValgrindSlowdown(void);
+double GperfValgrindSlowdown(void);

 #ifdef __cplusplus
 }
diff --git a/third_party/gperftools/src/base/sysinfo.cc b/third_party/gperftools/src/base/sysinfo.cc
index 36f706791a..006162c62e 100644
--- a/third_party/gperftools/src/base/sysinfo.cc
+++ b/third_party/gperftools/src/base/sysinfo.cc
@@ -58,7 +58,7 @@
 #endif
 #include "base/sysinfo.h"
 #include "base/commandlineflags.h"
-#include "base/dynamic_annotations.h"   // for RunningOnValgrind
+#include "base/dynamic_annotations.h"   // for GperfRunningOnValgrind
 #include "base/logging.h"

 #ifdef PLATFORM_WINDOWS
diff --git a/third_party/gperftools/src/base/vdso_support.cc b/third_party/gperftools/src/base/vdso_support.cc
index f88aa30312..eb3506c0ee 100644
--- a/third_party/gperftools/src/base/vdso_support.cc
+++ b/third_party/gperftools/src/base/vdso_support.cc
@@ -77,7 +77,7 @@ const void *VDSOSupport::Init() {
     // on stack, and so glibc works as if VDSO was not present.
     // But going directly to kernel via /proc/self/auxv below bypasses
     // Valgrind zapping. So we check for Valgrind separately.
-    if (RunningOnValgrind()) {
+    if (GperfRunningOnValgrind()) {
       vdso_base_ = NULL;
       return NULL;
     }
diff --git a/third_party/gperftools/src/debugallocation.cc b/third_party/gperftools/src/debugallocation.cc
index 7c438f25bc..6a217fb1fe 100644
--- a/third_party/gperftools/src/debugallocation.cc
+++ b/third_party/gperftools/src/debugallocation.cc
@@ -1158,14 +1158,14 @@ REGISTER_MODULE_INITIALIZER(debugallocation, {
   // Either we or valgrind will control memory management.  We
   // register our extension if we're the winner. Otherwise let
   // Valgrind use its own malloc (so don't register our extension).
-  if (!RunningOnValgrind()) {
+  if (!GperfRunningOnValgrind()) {
     DebugMallocImplementation *impl = new (debug_malloc_implementation_space.chars) DebugMallocImplementation();
     MallocExtension::Register(impl);
   }
 });

 REGISTER_MODULE_DESTRUCTOR(debugallocation, {
-  if (!RunningOnValgrind()) {
+  if (!GperfRunningOnValgrind()) {
     // When the program exits, check all blocks still in the free
     // queue for corruption.
     DanglingWriteChecker();
diff --git a/third_party/gperftools/src/heap-checker.cc b/third_party/gperftools/src/heap-checker.cc
index 8e71f58232..d1dd9ae91d 100755
--- a/third_party/gperftools/src/heap-checker.cc
+++ b/third_party/gperftools/src/heap-checker.cc
@@ -1967,7 +1967,7 @@ void HeapLeakChecker_InternalInitStart() {
       // turns out we do not need checking in the end; can stop profiling
       HeapLeakChecker::TurnItselfOffLocked();
       return;
-    } else if (RunningOnValgrind()) {
+    } else if (GperfRunningOnValgrind()) {
       // There is no point in trying -- we'll just fail.
       RAW_LOG(WARNING, "Can't run under Valgrind; will turn itself off");
       HeapLeakChecker::TurnItselfOffLocked();
diff --git a/third_party/gperftools/src/malloc_extension.cc b/third_party/gperftools/src/malloc_extension.cc
index e8ca31d7e4..b4b5480562 100644
--- a/third_party/gperftools/src/malloc_extension.cc
+++ b/third_party/gperftools/src/malloc_extension.cc
@@ -236,7 +236,7 @@ void MallocExtension::Register(MallocExtension* implementation) {
   // callers should be responsible for checking that they are the
   // malloc that is really being run, before calling Register.  This
   // is just here as an extra sanity check.)
-  if (!RunningOnValgrind()) {
+  if (!GperfRunningOnValgrind()) {
     current_instance = implementation;
   }
 }
diff --git a/third_party/gperftools/src/tcmalloc.cc b/third_party/gperftools/src/tcmalloc.cc
index 37c14407d9..2e0a992db7 100644
--- a/third_party/gperftools/src/tcmalloc.cc
+++ b/third_party/gperftools/src/tcmalloc.cc
@@ -117,7 +117,7 @@
 #include <gperftools/nallocx.h>
 #include "base/basictypes.h"            // for int64
 #include "base/commandlineflags.h"      // for RegisterFlagValidator, etc
-#include "base/dynamic_annotations.h"   // for RunningOnValgrind
+#include "base/dynamic_annotations.h"   // for GperfRunningOnValgrind
 #include "base/spinlock.h"              // for SpinLockHolder
 #include "central_freelist.h"  // for CentralFreeListPadded
 #include "common.h"            // for StackTrace, kPageShift, etc
@@ -1112,7 +1112,7 @@ TCMallocGuard::TCMallocGuard() {
 #ifdef TCMALLOC_USING_DEBUGALLOCATION
     // Let debugallocation register its extension.
 #else
-    if (RunningOnValgrind()) {
+    if (GperfRunningOnValgrind()) {
       // Let Valgrind uses its own malloc (so don't register our extension).
     } else {
       MallocExtension::Register(new TCMallocImplementation);
@@ -1124,7 +1124,7 @@ TCMallocGuard::TCMallocGuard() {
 TCMallocGuard::~TCMallocGuard() {
   if (--tcmallocguard_refcount == 0) {
     const char* env = NULL;
-    if (!RunningOnValgrind()) {
+    if (!GperfRunningOnValgrind()) {
       // Valgrind uses it's own malloc so we cannot do MALLOCSTATS
       env = getenv("MALLOCSTATS");
     }


Adopt Gherkin as a standard way to describe test scenarios

See https://www.businessanalysisexperts.com/gherkin-user-stories-given-when-then-examples/
Example (unrelated to Eliza):
Feature: User Login
  As a registered user
  I want to log into the application
  So that I can access my account

  Scenario: Successful login with valid credentials
    Given the user is on the login page
    When the user enters a valid username and password
    And the user clicks the "Login" button
    Then the user should be redirected to the dashboard
    And a welcome message should be displayed

  Scenario: Unsuccessful login with invalid credentials
    Given the user is on the login page
    When the user enters an invalid username or password
    And the user clicks the "Login" button
    Then an error message saying "Invalid username or password" should be displayed
    And the user should remain on the login page

@TimKozak Could you put this on your radar? Thanks.


Issue 78

I can successfully populate a page for get-html if the hosted site is
on HTTP instead of HTTPS.
This is not for merge, but rather for comments.

@san983 The DOM morphs the information, so I have a lot of trouble trying to recreate the button. Aside from actually extracting the information. Shall I look around for an alternative method for this then ?

@san983 I was able to do the requested changes. Please take a look and give me an opinion.

Please @bnandaku, create a New Branch with a new PR including just the commits / file changes for this issue #78. BE aware of the indentation and if there is some other issues you're addressing open a new branch/PR for it. Thanks


Bridge Page

The bridge page enables a user to move funds between Nightfall and Mainnet (deposit / withdraw). See sections 7 to 10 of the HackMd for more visuals.
The completion of tasks will be based on corresponding PRs.
UI changes

[x] Update UI/UX based on figma
[x] Dropdown for token selection in deposit and withdraw
[ ] Max button clickable

Feature changes

[ ] Ensure contract address for operation is based on dropdown option selected.
[ ] Current balance calculation on-screen
[ ] Enable spend All (if Max is clicked)
[ ] Check hyperlinks in left half (information panel)
[ ] Exception handling (e.g. withdrawal does not have the correct commitments available, depositing more than balance.)

vcsdf

Updated the styles to follow the last ui/ux passed for us in the figma document: https://www.figma.com/file/wpH5sn90CARgFCSWiSGRzh/Web-wallet-2.1?node-id=1298%3A4453

Got a polygon tokens list json to use like example to start the dropdown.
https://github.com/maticnetwork/polygon-token-list

Dropdown for token selection in deposit and withdraw

commit 5b7e6e14305293987fe87a9603c275d9d7b25f9a

scss responsive for large and medium screens

Updating balances, get contract address from token selected and enabling the MAX buttons for spend all
0b561cf8b0379dcbfe1a1a034ad7b3613087f3a5

Some questions
Current balance calculation on-screen
@IlyasRidhuan I'm trying to understand better this requisite.
Balance rounding
What is the better way for round the balances?

Next task to finish Current balance calculation on-screen
[ ] - Filter the pkds from getWalletBalance() and after filter by token address.

Removed the links because we're not using now

Input validation for withdraw

Input validation for deposit

Exception handling (e.g. withdrawal does not have the correct commitments available, depositing more than balance.)
I think for withdraw is done alse, because we're getting all the commitments for the user in the Token selected and adding all values. So if the input is greater than this, the input is greater than all the commitments.
@IlyasRidhuan

Last commit
214f45e3801bf23a7ece0dfce0c67d2cc8139136

@IlyasRidhuan if you could test this branch and see if the transfers are working fine, please.
luizoamorim/558-bridge-page

Fixing the l2 balance show
a8187dbde07c146ccd12ad5318e540086be3e672


Potentially fixed Mongo-Observer ID Creation not Thread/Process Safe (issue 244)

The insert function was not looping when a DuplicateKeyError is raised and the key should autoincrement.

Coverage remained the same at 83.506% when pulling 0443d5178fdb0d70d7c71420fb2ce6fad6e9b65a on vnmabus:fix_duplicate_id_mongodb into 6ae104f54496c98e625afcbc58b5bd68deb01cac on IDSIA:master.

Thanks


Ordering columns problem

Hello ! i used the same lines of code but i didn't get the same result !
The column doesn't change its position

What version of Bootstrap are you using?

4.3.1
and that happened to me again in Dropdown buttons, when i click on the button it doesn't show the items inside that button.


json_decode in php 5.3 has only 3 parameters

proof: https://3v4l.org/Im2lI

Thank you for the pull request. I'm sorry that I haven't reviewed it yet — I've been a little busy, and rather out of the PHP world for a few years. I'll do my best to get to it ASAP.


PoCL for CI instead of Intel driver

I am not sure Intel ICD could be re-distributed, PoCL is packaged in PyPI:
https://pypi.org/project/pocl-binary-distribution/
Can we consider switching from Intel driver to PoCL ?

I have nothing against it. Let's hope POCL is mature enough to be used in the CI without causing more headaches.

It sounds a good idea to me.


Need better documentation

This really needs to be a bit clearer in terms of what these methods will output.
What is the expected output of the final method "root.WriteContentTo(writer)"  for example?

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

ID: 9a3b573b-6427-0b34-f970-347dc439b3cf
Version Independent ID: 3d8d49d7-6efb-86e5-4562-059785893bca
Content: XmlNode.WriteContentTo(XmlWriter) Method (System.Xml)
Content Source: xml/System.Xml/XmlNode.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for your feedback, @PrestonII. The documentation certainly could use improvement. We've added it to our backlog, as well as to the community contributor's project.


1.1.2_crab version of Add option to not override PSet numThreads for CRAB

Once this is merged please create a new tag at the head of the 1.1.2_crab branch. Thanks!

1.1.2.crab4 created


Initial ssh commands require interactively saying 'yes'

The first time one connects to a host that ssh has not previously seen the host-key for, it asks you to manually confirm.  There's an ssh option that can be used the first time to automatically add this thus making these commands more non-interactive.   Of course disabling host key checking like this should not typically be done lightly, however since these are transient/temporary hosts, it shouldn't be very big of a risk.
Will create a PR to enable this, and allow you to decide whether this is an acceptable trade-off for the sake of making the commands easier to run w/o having to interactive with them as much.

Thanks for sending the PRs. I have merged the others. I would like to leave this one. This adds a little more abstraction. Since this is a learning exercise, I would like to avoid another layer of abstraction and some of the people wondering and trying to figure out how the magic worked and spending time figuring it out. Let me know if you think otherwise.

Understood and i get where you're coming from.. it becomes quite easy to just copy/paste your way to a kubernetes cluster and takes the 'hard' part ouf the name of this project..  i'll close this issue out then and cancel my PR..


ci: remove py2 ci

Summary of Changes
remove py27 CI
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

Codecov Report

Merging #280 into master will decrease coverage by 0.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
- Coverage   74.39%   74.01%   -0.39%
==========================================
  Files         100      100
  Lines        4199     4199
  Branches      390      390
==========================================
- Hits         3124     3108      -16
- Misses        975      983       +8
- Partials      100      108       +8

Impacted Files
Coverage Δ

...ilder/transformer/regex_str_replace_transformer.py
89.28% <0.00%> (-7.15%)
:arrow_down:

databuilder/loader/file_system_neo4j_csv_loader.py
83.33% <0.00%> (-5.96%)
:arrow_down:

...abuilder/extractor/snowflake_metadata_extractor.py
91.80% <0.00%> (-3.28%)
:arrow_down:

...tabuilder/extractor/postgres_metadata_extractor.py
91.93% <0.00%> (-3.23%)
:arrow_down:

databuilder/extractor/mssql_metadata_extractor.py
92.53% <0.00%> (-2.99%)
:arrow_down:

databuilder/publisher/neo4j_csv_publisher.py
84.23% <0.00%> (-0.99%)
:arrow_down:

databuilder/task/neo4j_staleness_removal_task.py
83.62% <0.00%> (-0.87%)
:arrow_down:

Continue to review full report at Codecov.

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


[ENH] improvements to notebook 1

Some small improvements to notebook 1:

multiple alpha, coverage
output type explanation text

Codecov Report
Patch coverage has no change and project coverage change: -0.05% :warning:

Comparison is base (180491f) 71.67% compared to head (7402bf9) 71.63%.
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
- Coverage   71.67%   71.63%   -0.05%
==========================================
  Files          93       93
  Lines        4950     4949       -1
  Branches      900      806      -94
==========================================
- Hits         3548     3545       -3
- Misses       1173     1175       +2
  Partials      229      229

see 3 files with indirect coverage changes

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


Support cargo:rustc-env

Starting from Rust 1.19 (cargo 0.20.0) (next beta), one may use
println!("cargo:rustc-env=ENVVAR=value");

in the build.rs to define a variable for rustc, allowing code to use
env!("ENVVAR")

vergen could be extended to support this, so user could read the variables without including OUT_DIR.

Version 2.0.0 now supports passing of build information for use with the env! macro.  Closing.


Added file watch via text/event-stream content type.

Eventually this will lead to being able to listen for changes in Agregore using EventSource.
Based on some experiments I ran on Electron

Seems to be working!
Tested in hyper-gateway with hypergodots EventSource implementation (which is compatible with regular EventSource in the browser).


Explorer: folders should act like the ones in Finder/Xcode on Mac

As a Mac user, I expect all folders act the same behavior just like they do in Finder or Xcode:

click the folder: just select the folder (not disclose/enclose it).
click the disclosure triangle: disclose/enclose the folder.
option + arrow-right/arrow-left on a selected folder: disclose/enclose all sub-folders in the hierarchy.
hold option and click the disclosure triangle: just like what option + arrow-right/arrow-left do to disclose/enclose all sub-folders in the hierarchy.
option + arrow-down: open the selected file/folder in a window.
option + arrow-up: jump to parent folder which owns this currently selected file/folder.

Current behaviors which I did not expect:

Simply clicking the folder will open/close the folder (should just select it).
Can't find a familiar way to disclose/enclose all sub-folders in the hierarchy (using option).

I really do expect that VSCode team can fulfill this expectation in Explorer.

The option+click part is covered in #9456. Not sure whether there's an existing issue for the first part.

Yes, 1 is as-designed currently. Clicking and dragging or right clicking will not expand the folder though
Closing as duplicate of #9456 for 2


Commenting and style changes in .proto file

No API changes
Status
[ ] Under development
[ ] Waiting for code review
[x] Waiting for merge

Information
[ ] Contains breaking changes
[ ] Contains new API(s)
[x] Contains documentation
[ ] Contains test
[ ] Contains examples

After discussions, we saw that some information went missing when running the protoc-gen-doc tool on the new formatting. We decided, therefore, to keep the styling and commenting as is.


Rearrange SCSS

This PR is written as a step toward making changes called for in #3804. It moves the bulk of our styles to the client folder, and splits them into files approximately according to ITCSS.
The only changes to outputted CSS should be:

The order of the styles
@font-face is included in more CSS files because I grouped it with settings and tools. I'm not sure this is the right decision, but it does make the includes a lot simpler.

Replaces #4974

I couldn't reproduce the font issues showing in the console of @HvandenBerg's screenshot in IE11... I wonder if there was a build error?

Hi Naomi and Jonny,
I took the liberty of applying Jonny's PR to apply to this branch because ideally I would love to have this merged as soon as we can and I can probably can get Hugo to test it final this week.
Aside from that I did some investigation into the font-issue. Right now this is caused as it is also rendered into subfiles like css/layouts/home.css whereas previously we only rendered the font-face definitions in css/core.css. When opening the homepage of the admin interface it tries to loads the font-set twice. Once from css/core.css (which resolves) and once from css/layouts/home.css which doesn't resolve (hence it only goes up two folders).
But I'm not sure on what the best practice would be to resolve it within this naming convention. Perhaps remove them from the general _settings.scss and only them to core.scss? I'm happy to hear your thoughts about this.
A screenshot from the difference between this branch and the latest master in terms of home.css:

Thank you for doing this, @jjanssen.
How fonts are included is one of the few things I did actually change substantially. The way it was done before was a bit confusing to me, and maybe I need to understand a little better how we go about including css files in the templates. I was thinking that pages get  either main.css or another css file, but I see that I was wrong and they get main.css and other css files. I will take a look at this later today. I hope just to be able to put in a commit that fixes the font situation, but I might have further questions since I'm seeing some inconsistencies around how the css files are included in the templates.

It's pretty cool to see all of those changes happening! What's the status of this PR @nmorduch @jjanssen @jonnyscholes?
Any other changes to Wagtail's styles will be creating conflicts for this refactor, so we should try not to keep this open for longer than necessary (otherwise it'll just become harder and harder to ever merge this).

@thibaudcolas I actually did some work on this late last week to resolve the font issues raised by @jjanssen. I'll clean it up and commit it tomorrow - you're right it would be great to get this merged sooner rather than later!

Turned out I'd already cleaned it up... Hope you don't mind @nmorduch but I've pushed up a fix for the fonts!
I've moved them out of settings/_fonts.scss and to the top of elements/_typography.scss. One nice side effect of this is settings now renders no CSS output so can be safely imported in any other file with no side effects.
I did some testing and it looks like core.css is loaded on all pages and then some pages get an additional css file loaded. The commit just added should mean the fontface rules are only ever in core.css :)

I think we're nearly there. Could we close this PR and create a new one so @HvandenBerg can benefit from using our new Squash tooling for that? It will save him the hassle to re-checkout this branch, build it, etcetera.

This has been replaced by #5215


userAgent Test fails

When I run grunt, it executes and passes all the tests in /tests/Tests.purs successfully, except for the last one which tests the 'userAgent' of the browser.
Even if I open Chrome Dev Tools, and type self to return the Window object, and then expand it (i.e. Window > clientInformation > userAgent), and then copy its value, which in my case is: "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36", and then paste it in Test.purs (replacing the sample userAgent value), and then run grunt again, it still fails....
Do you know what could be causing this? (obviously I can get all the tests to pass if I just comment out the final userAgent test on Line 122, but i'm curious...)

Should be fixed in @e40a4fae333b84f8f61ab49592e29e7da4ae7142 thanks :)


EXPLAIN output contains insufficient information to understand types

For example:
> explain select a as c from (VALUES (NULL), (NULL), (NULL)) as T(a);
+---------------+---------------------------------------+
| plan_type     | plan                                  |
+---------------+---------------------------------------+
| logical_plan  | Projection: t.a AS c                  |
|               |   SubqueryAlias: t                    |
|               |     Projection: column1 AS a          |
|               |       Values: (NULL), (NULL), (NULL)  |
| physical_plan | ProjectionExec: expr=[column1@0 as c] |
|               |   ValuesExec                          |
|               |                                       |
+---------------+---------------------------------------+

from the output one could assume the type of a to be Null, but it used to be Utf8 eg in in DF v40, as can be seen using arrow_typeof
> select arrow_typeof(a) as c from (VALUES (NULL), (NULL), (NULL)) as T(a);
+------+
| c    |
+------+
| Utf8 |
| Utf8 |
| Utf8 |
+------+

Using arrow_typeof should not be needed to reason about EXPLAIN plans.

Using arrow_typeof should not be needed to reason about EXPLAIN plans.

It's not.
This is my mistake. i checked EXPLAIN plan from latest DF, but mistakenly arrow_typeof output was from a different version. No surprise they didn't quite match.

I think you can get this information with EXPLAIN VERBOSE already:
For example,
> explain verbose select * from values (1);
....
|                                                            |                                                                                |
| physical_plan_with_schema                                  | ValuesExec, schema=[column1:Int64;N]                                                |
|                                                            |                                                                                |

Maybe we should improve the documentation in this area


 The resulting script looks like this:

...
CREATE NONCLUSTERED INDEX [<Name of Missing Index, sysname,>]
ON [dbo].[tablename] ([source])
INCLUDE ([source],[division])
GO

This is incorrect, as it includes source instead of name. Here is what the (correct) XML shows (also I don't see the same symptom in SSMS):
    <MissingIndex Database="[tempdb]" Schema="[dbo]" Table="[tablename]">
      <ColumnGroup Usage="INEQUALITY">
	<Column Name="[source]" ColumnId="3">
	</Column>
      </ColumnGroup>
      <ColumnGroup Usage="INCLUDE">
        <Column Name="[name]" ColumnId="1">
        </Column>
        <Column Name="[division]" ColumnId="2">
        </Column>
      </ColumnGroup>
    </MissingIndex>

@AaronBertrand FYI that I'm closing this as a duplicate of https://github.com/microsoft/azuredatastudio/issues/20249.  Your issue was entered first :) but there's a bit more info on the other issue.  Will link to this issue from the other so your demo code is available if needed.


want to create cluster table by automigrate

If you have seen this issue, hope to make a version more than 0.2.1 to provide automigrate for distributed tables. Very grateful to you
^v^!!

v0.2.2 released.

thx!!


Make players bigger, Fix indentation

look at the overlap

The projectile from the enemy does hurt me even when overlapped.


ex2.31

r1 is a int&, while v2 is a const int.
Why it is legel?

r1 = v2; // legal, top-level const in v2 is ignored.

This is assignment rather than binding.

Same question, the example in the book says int &r = ci (const int ci = 42), is illegal, which is same as "r1 = v2", so I think "r1 = v2" should be illegal.

At first, I thought it's illegal.
Then I wrote a program to verify it:
const int v2 = 0;
int &r1 = v2;
This is illegal;
Then I find that r1 has been bound to v1, so r1 represents v1 which is an int type. The appropriate program is:
int v1 = 0;
const int v2 = 0;
int &r1 = v1;
r1 = v2;
That's right. It's legal as an assignment.


Editing the home page

According to me, the home page doesn't looks attractive at all.. So we need to change the CSS of the home page.
The site will look much better if every link can be arranged in the form of tile with a bit description and a pic of the same.

Here you can see each course has its own description and an image added to it... So we can do the same to the home page

I'd really love to work on this issue and make the home page interface better for the user.

@Aniket1026 we can work together in this issue if you want to pair up


ModuleInstance::getFileName() looks corrupted

To repro, modify the hellosureworld.cpp by adding the following print after the instance print (line 60 as of 3177342afb6004ec1a416372c4600c065c0a8669):
std::cout << "File: " << inst->getFileName() << std::endl;

The printout from parsing a design shows some of the file name but generally looks pretty corrupted.

Fixed by #1560


Why crash in the latest version?

When I started the server, the server loaded the plugin and started loading the bot, after which the server just crashed.
I upload the mdmp file.
Use:
Protocol version 48
Exe version <IP_ADDRESS>/Stdio (cstrike)
ReHLDS version: <IP_ADDRESS>8 (Extended)
Build date: 07:07:50 Jul 21 2023 (3387)
Build from: https://github.com/dreamstalker/rehlds/commit/f955b07
ReGameDLL version: <IP_ADDRESS>8-dev
hlds_<IP_ADDRESS>8-dev-f955b07_crash_20241020_215946_1.zip

hello, fixed now, sry for that

hello, fixed now, sry for that

it's okay now ^_^


Attempted to send a key down event when no keys are in keysPressed, when using barcode scanner

When I scan a barcode with my barcode scanner I get this error.
======== Exception caught by services library ======================================================
The following assertion was thrown during a platform message callback:
Attempted to send a key down event when no keys are in keysPressed. This state can occur if the key event being sent doesn't properly set its modifier flags. This was the event: RawKeyDownEvent#22ace(logicalKey: LogicalKeyboardKey#df100(keyId: "0x200000102", keyLabel: "Shift Left", debugName: "Shift Left"), physicalKey: PhysicalKeyboardKey#ed430(usbHidUsage: "0x000700e1", debugName: "Shift Left"), repeat: false) and its data: RawKeyEventDataWindows#b7606(keyCode: 160, scanCode: 42, characterCodePoint: 0, modifiers: 0)
'package:flutter/src/services/raw_keyboard.dart':
Failed assertion: line 695 pos 7: 'event is! RawKeyDownEvent || _keysPressed.isNotEmpty'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.yml

When the exception was thrown, this was the stack:
#2      RawKeyboard.handleRawKeyEvent (package:flutter/src/services/raw_keyboard.dart:695:7)
#3      KeyEventManager.handleRawKeyMessage (package:flutter/src/services/hardware_keyboard.dart:1050:30)
#4      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:223:49)
#5      _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:567:35)
#6      _invoke2 (dart:ui/hooks.dart:344:13)
#7      _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:45:5)
#8      _Channel.push (dart:ui/channel_buffers.dart:135:31)
#9      ChannelBuffers.push (dart:ui/channel_buffers.dart:343:17)
#10     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:722:22)
#11     _dispatchPlatformMessage (dart:ui/hooks.dart:257:31)
(elided 2 frames from class _AssertionError)
====================================================================================================

Platform: Windows
Keyboard: Azerty
Create new flutter project, and paste the "code" contents below in your main file.
While the app has focus, whenever you type on your keyboard, you should see the key events being printed in
the terminal.
To reproduce the error above, you'll need to use a barcode scanner, set it to Azerty keyboard layout, and scan a barcode with it.
The error happens on windows when using Azerty keyboard layout.
I don't know if it's something that happens with other keyboard layouts.
Main Code:
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return Focus(
      onKeyEvent: (node, event) {
        print(event);
        return KeyEventResult.handled;
      },
      child: MaterialApp(
        title: 'Flutter Demo',
        theme: ThemeData(
          // This is the theme of your application.
          //
          // TRY THIS: Try running your application with "flutter run". You'll see
          // the application has a purple toolbar. Then, without quitting the app,
          // try changing the seedColor in the colorScheme below to Colors.green
          // and then invoke "hot reload" (save your changes or press the "hot
          // reload" button in a Flutter-supported IDE, or press "r" if you used
          // the command line to start the app).
          //
          // Notice that the counter didn't reset back to zero; the application
          // state is not lost during the reload. To reset the state, use hot
          // restart instead.
          //
          // This works for code too, not just values: Most code changes can be
          // tested with just a hot reload.
          colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
          useMaterial3: true,
        ),
        home: const MyHomePage(title: 'Flutter Demo Home Page'),
      ),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // TRY THIS: Try changing the color here to a specific color (to
        // Colors.amber, perhaps?) and trigger a hot reload to see the AppBar
        // change color while the other colors stay the same.
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          //
          // TRY THIS: Invoke "debug painting" (choose the "Toggle Debug Paint"
          // action in the IDE, or press "p" in the console), to see the
          // wireframe for each widget.
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Doctor:
[√] Flutter (Channel stable, 3.16.9, on Microsoft Windows [Version 10.0.19045.3930], locale nl-BE)
    • Flutter version 3.16.9 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 41456452f2 (3 weeks ago), 2024-01-25 10:06:23 -0800
    • Engine revision f40e976bed
    • Dart version 3.2.6
    • DevTools version 2.28.5

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at C:\Users\PC-3030\AppData\Local\Android\sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.6)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.8.34525.116
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2022.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[√] VS Code (version 1.82.2)
    • VS Code at C:\Users\PC-3030\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.48.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.3930]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 121.0.6167.161
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 121.0.2277.112

[√] Network resources
    • All expected network resources are available.

• No issues found!

@nivle
When you open an issue, please try to use new issue template and fill out requested details so that we'll have enough info to work with.
For the issue you are reporting, try to provide minimal reproducible code sample, steps to replicate, platform on which you are seeing this (windows, mac, desktop, web) along with flutter doctor -v so that we can properly address the issue.

Channel stable, 3.3.10,

Thanks for the update. This is old version of stable. Can you upgrade to latest stable or master and re-run your scenario to confirm if the behavior still persists or not ?
flutter upgrade or flutter channel master and flutter upgrade.

it does still give the error on the barcode scanner.

What error does it throw ? on the barcode scanner means while trying to scan the barcode ? How are you scanning the barcode ?

Thanks for the update. Can you provide us complete runnable code sample along with replication steps that we can follow ?

Thanks for the update. We currently don't have barcode scanner to properly replicate it, but based on the report, I'll keep the issue open for team's attention. It may be similar / related to below open issues, although the error 'event is! RawKeyDownEvent || _keysPressed.isNotEmpty' thrown is different from the ones linked:
https://github.com/flutter/flutter/issues/80108
https://github.com/flutter/flutter/issues/107972
https://github.com/flutter/flutter/issues/107972

CC @dkwingsmt just in case you might have seen this kind of error before.

Yes I do. There are similar issues out there and unfortunately it's hard to investigate the cause due to various assumptions and how entangled the keyboard logic is.
I'm doing some experiments on a reworked keyboard logic that will hopefully resolve issues like this in my spare time, but don't count on it soon.

Got it, thank you @dkwingsmt!

Any news about this error? I'm experiencing the same error.


Choosing the Ciphertext Modulus for BFV

I am trying to choose the minimum ciphertext modulus for a given plaintext modulus in BFV. Could someone advise me on this? I would only like to perform a single plaintext-ciphertext multiplication, and then ciphertext-ciphertext addition.
My guess is that to decrypt an exact result, I require that the noise budget is slightly larger |e|x|p| for my plaintext space Zp? How much larger should it be for me to be ensured that I will not encounter any decryption errors? Is 1 bit enough, since from what I understand the error is ternary, thus my error  is bounded by 2x|e|x|p| <= 2|p|?
Am I right to say that this analysis changes for public key crypto to having |e| = 2, plus an additional 1 bit error during decryption, so 2x|e|x|p| <= 4|p|, so I need 2+1 = 3 bits?

Also, am I right to say that because during the encoding phase, we are using a geometrical embedding (canonical ring embedding), if the values represented are small, the coefficients of the plaintext polynomial will also be small?

choose the minimum ciphertext modulus for a given plaintext modulus in BFV
The computation described does not even consume any level. Try choose the smallest parameter given by BFV default parameters.

the error is ternary
The error is discrete Gaussian, not ternary.

slightly larger |e|x|p|
Not always. If batching is used, plaintext is a polynomial, then there should be a factor of $\sqrt{n}$.

if the values represented are small, the coefficients of the plaintext polynomial will also be small?
Encoding is performed mod p. If there is no overflow mod p during encoding, your statement can be correct.


Variable keep being typed as Promise event after async / await

Reproduction :
Try it
Context :
When calling a function called 'foo' returning a promise with await, the type of the variable is still Promise.
Flow correctly interprets the value if we just return the variable, but triggers an error if we type the return of the function called 'bar'.
Is there any issue / PR related, or a common workaround ? Casting with any didn't seem to solve the error.

The return of the function is a Promise
After await is only T
The behavior is correct


GD-303: Improve array assert to show better report

show only the invalid current values colored in red
show unexpected values in an indexed table

merged in v2.2.x and master


Added GFG Problem

Hey there!, Thanks for submitting a PR to this repo
Please complete the following checklist for your PR to be approved. Mark the items by adding an x.
Example:  - [ ] Mark me as completed
To mark: - [x] Mark me as completed

[x] My Pull Request adheres to the Contribution guidelines.
[x] I have added readme.md and solution files in the appropriate folder
[x] None of the folder and file names I have added contain any invalid characters, leading digits or trailing whitespaces
[x] The solution I have given is my own and not plagiarized in any form.
[x] My solution passes all the testcases.
[x] I have properly indented and documented my solution using comments.

If any of the above is unmarked, please make the necessary changes and then mark them so that you're PR can be approved and merged.

I didn't add any leetcode problems in my local repo, but it shows error in leetcode problem. It was in my stagging area too. But i don't know how i got this. Kindly  review this and help me.
Thank you

@logan0501
I am having same issue, in my PR

@R-Ayush777  How shall we approach them.

Hey @logan0501, How to resolve the issue I don't know how to resolve if u find any solution plz suggest me as well
Do I have to close the current PR and then issue a new PR?

@R-Ayush777  How to close and issue a new PR

In the blow area of your pull request, I believe you can see a close PR option. As a result, close it and create a new PR with
Delete the repo that was previously forked. and fork new one(this repo.) so that whatever problem there is is resolved. and create new PR.

Ok thanks


replace dict variables in defaults with plain

PR: https://github.com/lean-delivery/ansible-role-gitlab/pull/2
merged


fix(windows): multiple custom protocols, closes #323

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[ ] No

The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Also fixes https://github.com/tauri-apps/tauri/issues/2124

The issue is that all handlers were called on a specific protocol navigation, so one of them returned 404 and caused issues depending on the call order.


Incompatibility when installing

When I try to install via pip install memoripy I get incompatible versions of three requirements as seen below. I tried uninstalling the three and running the command again hoping it would install the versions it wanted but it just reinstalled the versions I had and then just gave me the exact same error. What is going on?!
C:\Users\pauln>pip install memoripy
Requirement already satisfied: memoripy in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (0.1.1)
Requirement already satisfied: aiohappyeyeballs==2.4.3 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2.4.3)
Requirement already satisfied: aiohttp==3.10.10 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.10.10)
Requirement already satisfied: aiosignal==1.3.1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.3.1)
Requirement already satisfied: annotated-types==0.7.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.7.0)
Requirement already satisfied: anyio==4.6.2.post1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (4.6.2.post1)
Requirement already satisfied: attrs==24.2.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (24.2.0)
Requirement already satisfied: certifi==2024.8.30 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2024.8.30)
Requirement already satisfied: charset-normalizer==3.4.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.4.0)
Requirement already satisfied: distro==1.9.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.9.0)
Requirement already satisfied: faiss-cpu==1.8.0.post1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.8.0.post1)
Requirement already satisfied: frozenlist==1.5.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.5.0)
Requirement already satisfied: h11==0.14.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.14.0)
Requirement already satisfied: httpcore==1.0.6 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.0.6)
Requirement already satisfied: httpx==0.27.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.27.2)
Requirement already satisfied: idna==3.10 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.10)
Requirement already satisfied: jiter==0.7.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.7.0)
Requirement already satisfied: joblib==1.4.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.4.2)
Requirement already satisfied: jsonpatch==1.33 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.33)
Requirement already satisfied: jsonpointer==3.0.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.0.0)
Collecting langchain==0.3.7 (from memoripy)
Using cached langchain-0.3.7-py3-none-any.whl.metadata (7.1 kB)
Requirement already satisfied: langchain-core==0.3.15 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.3.15)
Requirement already satisfied: langchain-ollama==0.2.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.2.0)
Requirement already satisfied: langchain-openai==0.2.5 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.2.5)
Requirement already satisfied: langchain-text-splitters==0.3.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.3.2)
Requirement already satisfied: langsmith==0.1.139 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.1.139)
Requirement already satisfied: multidict==6.1.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (6.1.0)
Requirement already satisfied: networkx==3.4.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.4.2)
Requirement already satisfied: numpy==1.26.4 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.26.4)
Requirement already satisfied: ollama==0.3.3 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.3.3)
Requirement already satisfied: openai==1.53.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.53.0)
Requirement already satisfied: orjson==3.10.11 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.10.11)
Requirement already satisfied: packaging==24.1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (24.1)
Requirement already satisfied: propcache==0.2.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.2.0)
Requirement already satisfied: pydantic==2.9.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2.9.2)
Requirement already satisfied: pydantic-core==2.23.4 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2.23.4)
Requirement already satisfied: PyYAML==6.0.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (6.0.2)
Requirement already satisfied: regex==2024.9.11 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2024.9.11)
Requirement already satisfied: requests==2.32.3 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2.32.3)
Requirement already satisfied: requests-toolbelt==1.0.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.0.0)
Requirement already satisfied: scikit-learn==1.5.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.5.2)
Requirement already satisfied: scipy==1.14.1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.14.1)
Requirement already satisfied: setuptools==75.3.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (75.3.0)
Requirement already satisfied: sniffio==1.3.1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.3.1)
Requirement already satisfied: SQLAlchemy==2.0.36 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2.0.36)
Requirement already satisfied: tenacity==9.0.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (9.0.0)
Requirement already satisfied: threadpoolctl==3.5.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (3.5.0)
Requirement already satisfied: tiktoken==0.8.0 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (0.8.0)
Requirement already satisfied: tqdm==4.66.6 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (4.66.6)
Requirement already satisfied: typing-extensions==4.12.2 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (4.12.2)
Requirement already satisfied: urllib3==2.2.3 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (2.2.3)
Requirement already satisfied: yarl==1.17.1 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from memoripy) (1.17.1)
Requirement already satisfied: greenlet!=0.4.17 in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from SQLAlchemy==2.0.36->memoripy) (3.1.1)
Requirement already satisfied: colorama in c:\users\pauln\appdata\local\programs\python\python312\lib\site-packages (from tqdm==4.66.6->memoripy) (0.4.6)
Using cached langchain-0.3.7-py3-none-any.whl (1.0 MB)
Installing collected packages: langchain
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
langchain-community 0.2.12 requires langchain<0.3.0,>=0.2.13, but you have langchain 0.3.7 which is incompatible.
langchain-community 0.2.12 requires langchain-core<0.3.0,>=0.2.30, but you have langchain-core 0.3.15 which is incompatible.
langchain-community 0.2.12 requires tenacity!=8.4.0,<9.0.0,>=8.1.0, but you have tenacity 9.0.0 which is incompatible.
Successfully installed langchain-0.3.7

Hi paulsybrandy,
Thank you for reporting this! The issue is due to dependency conflicts between memoripy and langchain-community. I recommend using a clean virtual environment to avoid these conflicts. If that’s not possible, you may need to adjust the installed package versions manually. Let me know if you run into issues with a clean env.

@caspianmoon that worked nicely. Not sure why I did not consider just creating a new venv. Thanks for your help. Closing.

Thanks.

You're welcome


Login failure with Chrom/Firefox/headless

Hey everyone,
I got issues while login.
[2023-03-28 09:19:15,800]:[main]:[:144]:[INFO]:Script started
[2023-03-28 09:19:15,800]:[main]:[:157]:[INFO]:Downloading webdriver for your version of Chrome
[2023-03-28 09:19:15,801]:[main]:[:159]:[INFO]:Initializing instagram user
[2023-03-28 09:19:18,123]:[main]:[:168]:[INFO]:Setting target to: 'target'
[2023-03-28 09:19:18,123]:[main]:[:182]:[INFO]:Attempting to log in with 'username'
[2023-03-28 09:19:20,097]:[main]:[:294]:[ERROR]:Script ended with error : Failed to login. Incorrect username/password, or 2 factor verification is active.
[2023-03-28 09:19:22,168]:[main]:[:300]:[INFO]:Total time taken: 6.3697 seconds

Command I used:
python instalikecombot.py --loadenv --delay 5 --numofposts 10 --nocomments
Also tried it with -u user -p password, headless, forcing firefox etc.
My Credentials are working, I logged in manual with both browsers, no 2FA.
Are you still working on this project and could help me?
Thanks!

Hey everyone,
I got issues while login.
[2023-03-28 09:19:15,800]:[main]:[:144]:[INFO]:Script started [2023-03-28 09:19:15,800]:[main]:[:157]:[INFO]:Downloading webdriver for your version of Chrome [2023-03-28 09:19:15,801]:[main]:[:159]:[INFO]:Initializing instagram user [2023-03-28 09:19:18,123]:[main]:[:168]:[INFO]:Setting target to: 'target' [2023-03-28 09:19:18,123]:[main]:[:182]:[INFO]:Attempting to log in with 'username' [2023-03-28 09:19:20,097]:[main]:[:294]:[ERROR]:Script ended with error : Failed to login. Incorrect username/password, or 2 factor verification is active. [2023-03-28 09:19:22,168]:[main]:[:300]:[INFO]:Total time taken: 6.3697 seconds
Command I used: python instalikecombot.py --loadenv --delay 5 --numofposts 10 --nocomments
Also tried it with -u user -p password, headless, forcing firefox etc.
My Credentials are working, I logged in manual with both browsers, no 2FA. Are you still working on this project and could help me?
Thanks!

Ran it to check. It works fine on Chrome. There are some known issues in Firefox which are to be fixed, but I see that it works fine in Chrome.

[*] => Script started
[*] => Downloading webdriver for your version of Chrome
[*] => Initializing instagram user
[*] => Setting target to: #target
[*] => Attempting to log in with myusername
[*] => Validating login with xpath: //img[contains(@alt, " profile picture")]
[*] => Login successful
[*] => Opening target #target
[*] => [open_target]: Attempt - 1
[*] => [target: #target] No. of posts found: 102
[*] => [target: #target] Checking if #target is a private account
[*] => Failed to find text: This account is private
[*] => Failed to find text: This Account is private
[*] => Failed to find text: This Account is Private
[*] => [target: #target] Account not private
[*] => [target: #target] Opening first post
[*] => [target: #target] Number of posts to like: 10
[*] => [target: #target] Liking post: 1
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 2
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 3
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 4
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 5
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 6
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 7
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => [target: #target] Liking post: 8
[*] => [like]: Attempt - 1
[*] => [target: #target] Comment disabled? No
[*] => [target: #target] Moving on to the next post
[*] => [target: #target] Waiting for 5 seconds
[*] => Total time taken: 71.7119 seconds


[TBD] Django templates styling

Due to the fact, that currenty we are handling pages generated by Angular and Django and moving everything to Angular will take some time, some hacks will be required. Styling of Django-based templates will need to match styles from Angular, so user won't get confused, when he will be redirected from /login to /o/login.
Match angular style

footer
header

I guess that's already done.


fix istioctl analyze to panic when the server port in gateway is nil.

Please provide a description of this PR:
fix bug: https://github.com/istio/istio/issues/44318

I think we are missing check for server.port nil in validation.
@hzxuzhonghu

gateway.servers[*].port nil check already exists, ref:
https://github.com/istio/istio/blob/704afa70f64c6efc886793453ea70b07e9a89d27/pkg/config/validation/validation.go#L467-L522
https://github.com/istio/istio/blob/704afa70f64c6efc886793453ea70b07e9a89d27/pkg/config/validation/validation.go#L549-L564

@nicole-lihui: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
3dd273165f604b7640fe39e36c057f47138bb0f0
link
unknown
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@nicole-lihui It seems like tests are missing, can you also add test cases to cover this?


Prefix "sidecar-" always gets added with Sidecar-container name

Expected Behavior
Sidecar-container name shouldn't get prefixed with "sidecar-" instead it should use the specified name only.
Actual Behavior
Prefix "sidecar-" always gets added with Sidecar-container name(eg:sidecar-)
Steps to Reproduce the Problem
1.Create a task with a sidecar and give the sidecar a specific name (eg: name: authenticator)
2. When running the task/pipeline, the sidecar-container gets renamed with prefix "sidecar-" (eg: sidecar-authenticator)
Additional Info

Kubernetes version:
Output of kubectl version:

(paste your output here)

Tekton Pipeline version:
tkn version
Client version: 0.31.1
Chains version: v0.16.0
Pipeline version: v0.47.4
Triggers version: v0.24.1
Operator version: v0.67.0
Output of tkn version or kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'

(paste your output here)

So I think today, we prefix those to be able to detect they are sidecar containers and not something else. One could say we may want to change this behaviour by prefixing only step containers (we already do step-*) and assume any non-prefixed container is a sidecar.

I think this is the expected behavior. It makes sense to me, I believe it's more future-proof if they decide to add containers for other concepts.

So you mean that is feasible to eliminate the prefix sidecar- ?
My customer is constrained to name sidecar container in a way that not contain “sidecar-“ prefix, so they really would like if this would be possible.
To give more context, authentication system that the sidecar container is directed towards is based on a pre-established naming convention, that not allow use of prefix "siddecar-".
Currently many other customer's systems already join this naming convention, so a change like this would mean to avoid to changing a large number of configurations in all the environments.

So you mean that is feasible to eliminate the prefix sidecar- ?
My customer is constrained to name sidecar container in a way that not contain “sidecar-“ prefix, so they really would like if this would be possible.
To give more context, authentication system that the sidecar container is directed towards is based on a pre-established naming convention, that not allow use of prefix "siddecar-".
Currently many other customer's systems already join this naming convention, so a change like this would mean to avoid to changing a large number of configurations in all the environments.


Broken Docs Link for hasChanged()

On https://lit-element.polymer-project.org/guide/lifecycle#updated the link to " configuring hasChanged to customize what constitutes a property change." ie. https://lit-element.polymer-project.org/properties#configure-property-changes is broken and displays "Page not found"

fixed


Removed superfluous code bits

Removed all the qualified names in api.Spek, since they share the same
package and make it a lot harder to read.
Removed all the "public" modifiers, since they just cluttered the code
and were inconsistent.

Thanks for this. This was back from early Kotlin days :)

I thought it might be. And I'm guessing that the qualified names were from when you moved the api to a different package?

Yeah that was actually more like a bug that I didn't notice :)


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): remove ./ (exports../)

Codecov Report

Merging #724 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #724   +/-   ##
=======================================
  Coverage   81.48%   81.48%
=======================================
  Files           7        7
  Lines         378      378
  Branches       16       16
=======================================
  Hits          308      308
  Misses         70       70

Continue to review full report at Codecov.

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


bugfix non utf8 characters

OP-2044 && OP-1915
would be nice to have it for 1.14

Looks good to me. I guess what's still pending - in other prs - is to:
* find the root cause

* verify all strings, not just `André`?

I verified all ?o with type xsd:string  (added csv file under OP-1915 the git sparql.csv)
seems that's all we have. iconv gives 0 as an output.

Do you also have a curl call, to update QA's index without having to reindex it ? No problem to ship it with 1.14, you can merge directly to master then, I have a frontend fix to do as well, i'll deploy both at the same time on QA, then prod when it's all ok :)

I've added the curl calls


More detailed tutorial for newbies?

Sorry for the elementary question... but is it possible to have a more detailed tutorial for newbies?
Certain things I don't understand are:

SSH Keys added to your DigitalOcean account (which keys? how?)
manager_ssh_keys = [1234, 1235, ...] (what are these numbers? where can I get them?)
you can modify this using the variable provision_ssh_key (how? usually I have a custom key I use from Gitlab-CI)
"Note that for this to work, you need to open the Docker remote API port in both iptables (not necessary with default images) and the DigitalOcean cloud firewall." - how do I open Docker remote API port in both ip tables? What does that mean and what is DO cloud firewall?

Fyi, I am currently creating images with packer (following a book I read), like so:
{
  "variables" : {
    "SNAPSHOT_NAME" : "{{env `SNAPSHOT_NAME`}}"
  },
  "builders": [{
    "type": "digitalocean",
    "region": "nyc1",
    "image": "ubuntu-16-04-x64",
    "size": "1gb",
    "private_networking": true,
    "snapshot_name": "{{user `SNAPSHOT_NAME`}}",
    "ssh_username": "root"
  }],
  "provisioners": [{
    "type": "shell",
    "inline": [
      "sudo apt-get update",
      "sudo apt-get install -y apt-transport-https ca-certificates curl software-properties-common",
      "curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add -",
      "sudo add-apt-repository \"deb [arch=amd64] https://download.docker.com/linux/ubuntu $(lsb_release -cs) stable\"",
      "sudo apt-get update",
      "sudo apt-get install -y docker-ce"
    ]
  }]
}

Hi @drozzy ,
The ssh keys are the public key from your machine which will be executing the Terraform commands, any any additional public key from machines that you will be using to connect to the provisioned machines through SSH. You have to add those public keys to your account.
You can follow this tutorial for a how-to: https://www.digitalocean.com/docs/droplets/how-to/add-ssh-keys/to-account/
The numbers can be either numbers of the fingerprint of the public ssh key. The fingerprint will be shown in the UI, after you have added the public key to your account. The number is an id that refers to the public key. To find those, you can query the DigitalOcean API. I usually use doctl to find it:
$ doctl compute ssh-key ls
ID          Name                      FingerPrint
1234        my-key                    .......

Regarding the Docker remote API firewalling; it may not be necessary to open the port - it depends on your system and iptables set up.
An example to open a port with iptables:
sudo iptables -A INPUT -p tcp --dport 2376 -m conntrack --ctstate NEW,ESTABLISHED -j ACCEPT.

For the DigitalOcean cloud firewall, you can use the following snippet for that:
https://github.com/thojkooi/terraform-digitalocean-docker-swarm-mode/blob/9c7b0d92a15b762d0a3a434af9f059596cd47dd8/examples/usage/main.tf#L80-L86
It uses the following Terraform package: https://github.com/thojkooi/terraform-digitalocean-firewall-docker-api
Note: this is only necessary if you apply other firewall rules to your cluster droplets (you definitely should). If you don't, applying this rule will means you have to open up other ports in your DO cloud firewall as needed.
You can see this file for more examples:
https://github.com/thojkooi/terraform-digitalocean-docker-swarm-mode/blob/master/examples/usage/main.tf

Ok, thank you. This gives me something to try!


Better explanation for how to use to forward pass needed

There are examples for saving a model, but none easily visible for forward passing through the model.
The README suggests looking into the non-existent directory "jtorch_test"
Man this repository is way too good to not have some decent code samples for different use-cases in the Readme. Please add some!

This is definitely a reasonable request. I'm a little busy with the CVPR paper deadline this week, but I'll try getting around to it soon.

Updates: still haven't found time :-( Sorry @falconscript.  I'll write a demo app soon, but unfortunately now I'm working on another paper deadline.


Show validation errors added by onSubmit

The onSubmit action may set additional validation errors
(typically errors received from a backend server).
This change makes these errors visible.

Would be great if you could add tests.

Yes, please!

Similar I'm wondering if it should be considered a breaking change

I agree with your concerns. When not adding validation errors in onSubmit, setting showAllValidations will introduce a side effect that likely is unwanted, as all fields will be marked as successful (green), even when hideValidationsOnSubmit has been set, or some (not required) field hasn't been filled at all.
So an option like showValidationsOnRejection seems preferable probably!? 🤔


Running o365 flow export throws error 'administrator has not consented to use the application'

Steps to reproduce:

Run o365 login
Authenticate using device flow
Run o365 flow export -e Default-a1214787-77d5-4b72-a96d-1c278f72bbb0 --id 5083f6a3-04bb-4a88-8ff5-095169c289f2 --path "C:/MyFlow.zip"

Expected:
Zip file.
Actual:
Error: AADSTS65001: The user or administrator has not consented to use the application with ID '31359c7f-bd7e-475c-86db-fdb8c937548e' named 'PnP Office 365 Management Shell'. Send an interactive authorization request for this user and resource.
Trace ID: 3ed6a7eb-f580-469a-9d7c-4a993d3e1d00
Correlation ID: e7b5b545-bee7-4ebd-95e4-296c1bd1c3bc
Timestamp: 2019-08-04 15:31:42Z

Additional:
The 'PnP Office 365 Management Shell' consent is actually given. What am I missing?

Works for me but I used this flow instead...

Run o365 azmgmt login
Authenticate using device flow
Run o365 azmgmt flow export -e Default-eff8592e-e14a-4ae8-8771-d96d5c549e1c -i d9160435-2822-4baf-a412-7ed676e15c00 -p flow.zip -f zip

Thanks @garrytrinder ! I am currently using the dev branch (BETA) that is v2.0 and it has different authentication implementation. I will try to see if what you did with v1 will work for me at least then I will try to test few more things with v2.0, perhaps I am doing something wrong.
I have changed my notes on top to specify the version I am using.

This is actually a bug specific to v2.0 of the CLI. When the command is about to export zip, it is trying to do an OAuth against PowerApps blob storage instance where the zip is stored. Passing bearer token to the blob results in failure where the correct move is to just GET the zip from the blob storage without any access tokens in the headers because the blog key is already know and included in the url with the request.
I will remove the bearer token from the headers of the request and this should solve the issue.

Bug fix applied as part of the latest CLI now.


Update readme.md

These are small language models, not large ones. Correct terminology by removing the word 'large' when describing them.

I'm going to leave this as-is. There are certainly models much larger than those used here, but they all belong to the same family of models generally considered to be LLMs. BERT is considered to be an LLM, and these models are roughly the same scale as BERT.


Change the regex for castStringToInt function

Change the Regex so that it won't take any string that starts with "0"
It's for the behat test contains zipcode, so that we don't need to hack with single quote inside double quote  like "'07093'" anymore.

Testing file changes were not found.
Paging @awesomeousPrime to review the back-end work in this pull request!
Paging @joshmedeski to review the testing work in this pull request!

@amberwjq @joshmedeski @awesomeousPrime Please do not submit, approve or merge any changes for this project to the master branch. All PRs should be submitted to both the 1.0 and the 2.0 branches  (assuming they make sense for both). Also, 1.0 is deprecated, so all projects using it should plan to move to 2.0.


Ignore Toolchain auto-provisioning test when precondition is not met

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed. Click here to see all build failures.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


Work out weaklaplacian(::DirichletPolynomial) for non-evenly spaced points

I think the only thing that's not trivial is the A block. The D blocks should just be changing step(r) to diff(r)

Note the A block is just standard FEM with hat functions. @ioannisPApapadopoulos do you know this off the top of your head?

Yeah should be quite simply, something like -1/r for the off-diagonal and 1/r0 + 1/r1 for the main diagonal, where r0 and r1 are the interval lengths on the left and right hand side of the hats.
E.g. if a hat was scaled onto -1/2..0, 0..5, then the off diagonals are
(2 * -2)/2 = -2. = -1/r0 (left)
(-1/5 * 1/5)* 5 = -1/5 = =1/r1 (right).
the main diagonal is
(2*2)/2 + (-1/5 * -1/5)*5 = 2 + 1/5 = 1/r0 + 1/r1 = 11/5.

Ah ok so it’ll be something like 1 ./ diff(r)[1:end-1] + 1 ./ diff(r)[2:end] ? I think the dimensions match at least


Use checked_mod in ir operations to avoid panics

We should use checked_mod for the Mod operations to avoid panics. This also makes sense in the context of fuel since the flag may allow division by 0 to not abort execution.

Already fixed by #6112. Closing.


Fixed #470 -- Support database defaults for fields

https://code.djangoproject.com/ticket/470
Rebased from #13709.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

This SQL is generated by Django for Oracle, but leads to the ORA-00936: missing expression error:
INSERT INTO "FIELD_DEFAULTS_DBARTICLE" ("HEADLINE", "PUB_DATE")
    SELECT * FROM (SELECT DEFAULT col_0, TO_TIMESTAMP(DEFAULT) col_1 FROM DUAL
        UNION ALL SELECT DEFAULT, TO_TIMESTAMP(DEFAULT) FROM DUAL)

The relevant python code:
class DBArticle(models.Model):
    headline = models.CharField(max_length=100, db_default="Default headline")
    pub_date = models.DateTimeField(db_default=Now())

    def __str__(self):
        return self.headline

        articles = [DBArticle(), DBArticle()]
        DBArticle.objects.bulk_create(articles)

What I'm not yet clear on is:
1: Which bit of the generated SQL is invalid
2: Why we use a subquery selecting from DUAL instead of a VALUES clause

I can work around the above error somewhat by setting supports_default_keyword_in_insert to False for Oracle. This isn't ideal for a few reasons:

Oracle supports DEFAULT just fine in a single insert, rather than in a bulk insert.
I instead run into a new error: ORA-01830: date format picture ends before converting entire input string. This is because the Now() expression becomes CURRENT_TIMESTAMP which is then wrapped in TO_TIMESTAMP() by BulkInsertMapper in bulk_insert_sql.

I currently suspect that we can't use the DEFAULT keyword in the SELECT from DUAL subquery.
I'm wondering if the best fix here is to change how we handle Oracle bulk inserts to use the INSERT ALL INTO syntax:
INSERT ALL
  INTO mytable (column1, column2, column_n) VALUES (expr1, expr2, expr_n)
  INTO mytable (column1, column2, column_n) VALUES (expr1, expr2, expr_n)
  INTO mytable (column1, column2, column_n) VALUES (expr1, expr2, expr_n)
SELECT * FROM dual;

which I hope parallels single inserts sufficiently to support DEFAULT properly and won't cause other issues.

This looks like a nice feature, but someone reading the documentation (me included) might be wondering what is the recommended argument to use (default or db_default) when defining model and attributes. Do you think adding a small section in the doc could be beneficial?

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

@felixxm I've fixed the Oracle failures, but unfortunately this has left field_defaults.tests.DefaultTests.test_field_database_defaults_returning as a flaky test. Unfortunately this doesn't appear to be as simple as a millisecond/microsecond precision difference, as it was for sqlite on Windows.
Traceback (most recent call last):
  File "/home/jenkins/workspace/pull-requests-oracle/database/oracle19/label/oracle/python/python3.11/django/test/testcases.py", line 1603, in skip_wrapper
    return test_func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jenkins/workspace/pull-requests-oracle/database/oracle19/label/oracle/python/python3.11/tests/field_defaults/tests.py", line 51, in test_field_database_defaults_returning
    self.assertLess((a.pub_date - now).seconds, 5)
AssertionError: 86399 not less than 5

For some reason, occasionally the now value is later than the a.pub_date value, despite being calculated earlier. I'm wondering if Oracle and Python are using different clocks and therefore there is always a random chance of this failure on Oracle.
I saw this failure occasionally when debugging the other tests, but I can't reliably trigger it.

buildbot, test on oracle.

buildbot, test on oracle.

I think the postgres failure is completely unrelated to my changes.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

nice work here @LilyFoote , highly anticipated PR! is there any timelines of release?

@HaddadJoe In 5.0 hopefully.

@felixxm I think I did everything now. Let me know if I missed something.

I will try to push it to the finish line in the next few days :crossed_fingers: . Starting from checking all comments and pushing small edits. Please don't force-push.

buildbot, test on oracle.

buildbot, test on oracle.

Pushed set of small changes, 64% reviewed :sweat_smile: Tomorrow I want to start working on tests and docs.

In the light of ticket-34553 and whole saga of % escaping in literal values usage in schema alterations I think it would be worth adding tests for db_default="%", db_default="'", and `db_default='"'.

In the light of ticket-34553 and whole saga of % escaping in literal values usage in schema alterations I think it would be worth adding tests for db_default="%", db_default="'", and db_default='"'.

Added tests. It's funny because they work in almost all cases :smile: The only exception is the single quotation mark ' on Oracle, it's properly quoted and is set for new rows, but .... :drum: is not set on existing rows :exploding_head: I believe it's a bug in Oracle itself.

buildbot, test on oracle.

I noticed that inserting primary keys with db_default crashes, and fixed it.

I pushed final edits :rocket:
@LilyFoote Thanks for your humongous efforts :1st_place_medal: :star2:
We implemented database defaults 18 years :smiling_face_with_tear:  after the ticket was created :boom:

buildbot, test on oracle.

buildbot, test on selenium.

“Festina lente”, or “more haste, less speed” 🐢
🏅 Thank you @LilyFoote and @felixxm ! 🤘


ImportError: cannot import name 'StringIO' from 'pandas.compat'

python : 3.7.1
pandas : 1.0.0
tushare: 1.2.48
import tushare 报错：
Traceback (most recent call last):
File "", line 1, in
File "c:\dev\Anaconda3\envs\tushare\lib\site-packages\tushare_init_.py", line 11, in
from tushare.stock.trading import (get_hist_data, get_tick_data,
File "c:\dev\Anaconda3\envs\tushare\lib\site-packages\tushare\stock\trading.py", line 24, in
from tushare.stock.fundamental import get_stock_basics
File "c:\dev\Anaconda3\envs\tushare\lib\site-packages\tushare\stock\fundamental.py", line 19, in
from pandas.compat import StringIO
ImportError: cannot import name 'StringIO' from 'pandas.compat' (c:\dev\Anaconda3\envs\tushare\lib\site-packages\pandas\compat_init_.py)

I have the same problem, and my python version is py3.6

me too！

Same issues here.
@newmann @wwdlook @libsyc Do you all already have solution on this issue?

Fixed this issue by upgrading to latest tushare latest version, with following commands:
pip install tushare --upgrade


SIGSEGV fatal error when re-assigning a tensor that has been previously split

Hello!
I stumbled upon a fatal error while using torch.split + reassignment of a tensor - not sure how to even start debugging this, but I am documenting it here in case someone knows how to investigate this further.
Here is a way to replicate the error.
@ val data = torch.arange(0L, 1_000_000L)
data: Tensor[Int64] = tensor dtype=int64, shape=[1000000], device=CPU
[0, 1, 2, ..., 999997, 999998, 999999]

@ val Seq(a,b) = torch.split(data, 600_000)

@ a
res2: Tensor[Int64] = tensor dtype=int64, shape=[600000], device=CPU
[0, 1, 2, ..., 599997, 599998, 599999]

@ b
res3: Tensor[Int64] = tensor dtype=int64, shape=[400000], device=CPU
[600000, 600001, 600002, ..., 999997, 999998, 999999]

@ val x = a
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00000001b984535a, pid=19939, tid=9731
#
# JRE version: OpenJDK Runtime Environment Zulu19.30+11-CA (19.0.1+10) (build 19.0.1+10)
# Java VM: OpenJDK 64-Bit Server VM Zulu19.30+11-CA (19.0.1+10, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, bsd-amd64)
# Problematic frame:
# C  [libjnitorch.dylib+0x4ff35a]  Java_org_bytedeco_pytorch_TensorBase_sizes+0x5a
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /experiments/hs_err_pid19939.log
#
# If you would like to submit a bug report, please visit:
#   http://www.azul.com/support/
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#

Trying with other variations.. any operation done after any of the portions of tensor.split causes this panic, even a + 1

It might have to do something with the fact that split returns a view.
https://pytorch.org/docs/stable/generated/torch.split.html:

Splits the tensor into chunks. Each chunk is a view of the original tensor.

It's just a guess for now, but it would explain why clone() makes a difference.
I ran into this while implementing tensor printing, which needs to convert tensor values to buffers, and crashed on non-contiguous values, as the memory layout of views can sometimes be non-contiguous.
https://github.com/sbrunk/storch/blob/19abf3eedb4d101df2c545850fded40f346c7e21/core/src/main/scala/torch/Tensor.scala#L558
In this case the view should be contiguous, so it's not exactly the same issue, but it could still be related to being a view.
Interestingly, your example works on my machine (I tried in ammonite too):
object Split extends App {
  val data = torch.arange(0L, 1_000_000L)
  val Seq(a, b) = torch.split(data, 600_000)
  println(a)
  println(b)
  val x = a
  println(x)
}

But I was also able to trigger a segfault, I.e. with this
object Split extends App {
  val data = torch.arange(0L, 1_000_000)
  val Seq(a,b) = torch.split(data, 10)
  val x = a
  println(a)
  println(b)
  println(x)
}

Exception in thread "main" java.lang.ExceptionInInitializerError
        at Split.main(Split.scala)
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGBUS (0xa) at pc=0x00000001afdcf493, pid=27593, tid=8707
#
# JRE version: OpenJDK Runtime Environment Temurin-19.0.2+7 (19.0.2+7) (build 19.0.2+7)
# Java VM: OpenJDK 64-Bit Server VM Temurin-19.0.2+7 (19.0.2+7, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, bsd-amd64)
# Problematic frame:
# C  [libtorch_cpu.dylib+0x87c493]  at::native::as_strided_tensorimpl(at::Tensor const&, c10::ArrayRef<long long>, c10::ArrayRef<long long>, c10::optional<long long>)+0xa3

Perhaps we also need to understand why the Python impl calls split_with_sizes in certain cases. We might need to do something similar.
https://github.com/pytorch/pytorch/blob/9adfaf880784ec0cf5f085fc3f282cf53650050f/torch/_tensor.py#L770
    def split(self, split_size, dim=0):
        r"""See :func:`torch.split`"""
        if has_torch_function_unary(self):
            return handle_torch_function(
                Tensor.split, (self,), self, split_size, dim=dim
            )
        if isinstance(split_size, Tensor):
            try:
                split_size = int(split_size)
            except ValueError:
                pass

        if isinstance(split_size, (int, torch.SymInt)):
            return torch._VF.split(self, split_size, dim)  # type: ignore[attr-defined]
        else:
            return torch._VF.split_with_sizes(self, split_size, dim)


Azure Blob Storage

Hi there,
Is your feature request related to a problem? Please describe.
I saw the library does not have support for the Azure Blob Storage as a cloud store. I had implemented tusd in a freelancing project recently and was forced by my employer to use Azure which I in the end coded on top of the tusd library as a custom store which worked out quite well.
Describe the solution you'd like
Have Azure Blob Storage support out of the box.
Can you provide help with implementing this feature?
I thought since this isn't currently in the tusd library that I'd integrate the code I wrote and do a pull request. The code might not be up to the library's standard and there are some things I would test before adding it - I just thought I'd get some feedback first.
What are your thoughts?
Kind Regards
Benehiko

I would love to see  Azure Blob Storage supported by tus.io out of the box.

Yes, we would be very much interested in such a PR! Thanks for the offer :heart:

I will see if I can have a PR by this weekend :)

Can we extend the feature request by adding:

Support Azure File Storage along with Blob Storage.
Compute the MD5 hash of the file and put it base64 representation in the Content-MD5 property of the blob.

It seems that the linked PR doesn't have it if I'm not mistaken (I'm terrible in Go).

@AKlaus may I ask what benefit computing the MD5 hash of the file has? (I've never used it, so I am curious)
Would you compute the MD5 hash of the block of the blob, or the entire file?

The hash is used to verify the integrity of the blob/file during transport.
Form experience, the likelihood of broken transmission is 1/1,000,000 and may appear negligible, but my users transfer many million files.

I am going to close this issue since the main feature request (general support for Azure has been implemented in #458). @AKlaus If you have some additional ideas or feature requests, please feel free to open a new issue. Having separate issues helps with avoiding confusion :)


Question: Can one use this to read an existing sqlite database?

Hi
Wonderful job, as per subject title.

Hi. Depends on what you mean by “reading an existing database”
Вс, 17 сент. 2023 г. в 15:05, Mashy @.***>:

Hi
Wonderful job, as per subject title.
—
Reply to this email directly, view it on GitHub
https://github.com/catamphetamine/sql.js-as-sqlite3/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADUP36YUPIX3S6L6T3XIBDX23RR5ANCNFSM6AAAAAA43SFQF4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Update logs to not use info with custom depth

This PR ensures all info logs are on the same depth. I realized I was not seeing info logs for 5 depth without using the debug flag. I'd say better using default depth values for everything unless we have a reason not to do so.

Well, on controllers we just have the Info and the verbosity level (no more Debug, Warning, etc... just Info and Error).
So, the verbosity level 5 is meant to be the debug verbosity level.
I think would be fine to remove the additive verbosity for some messages that we may want always to be reported... but moving all the logs at info level honestly seems to much (e.g., messages like "Object was not found, not an error" should stay in the debug domain).

Well, on controllers we just have the Info and the verbosity level (no more Debug, Warning, etc... just Info and Error). So, the verbosity level 5 is meant to be the debug verbosity level. I think would be fine to remove the additive verbosity for some messages that we may want always to be reported... but moving all the logs at info level honestly seems to much (e.g., messages like "Object was not found, not an error" should stay in the debug domain).

Yes agree, just adapted the "Object was not found" messages. Note that in most controllers it was already at info level, I moved to Debug for all of them. I believe we need a second iteration on logging to not only use the same klog implementation everywhere, but also use the same wrapper or interface everywhere. klog import should only be needed within the pkg/log, anywhere else we should use our own abstraction layer. That way we could eventually switch the logging implementation for tests or to something else in the future if we see fit.


RESLICER crash with some parameters

Changing the parameters spacing create a crash.
But i got an idea on this.

@mathildemerle : I think that in applyResamplingPix ,  resamplePr->setParameter(outputSpacing[0], 0) should be  resamplePr->setParameter(outputSpacing[0], 3)
and ...
Do you agree ?

    resamplePr->setParameter(outputSpacing[0], 3);
    resamplePr->setParameter(outputSpacing[1], 4);
    resamplePr->setParameter(outputSpacing[2], 5);

This? It could be indeed, as far as i see, but i did not test :)

ok thanks I will give it a try.


performance for query with/without index.

As we know index will make the query faster, but i feel strange about the following issue.
A. with composite index as follows:
place.distance NOTUNIQUE  ["lat","lng"] SBTREE
explain select from place order by lat limit 1

order will using 'lat' index for query

explain select from place order by lat, lng limit 1

order will not work for composite index.

B. how to apply alias projection field for order clause to care about performance issue.
explain select  DISTANCE(lat, lng, 3.1333, 101.3833)  as distance from place order by distance limit 1

this query will have slow performance because new projection field 'distance' will not be indexed beforehand, but it's created on the fly.

C. spatial index is really provide poor performace.
explain select   from place where DISTANCE(lat, lng, 3.1333, 101.3833) < 100000 limit 1

perfomance better than spatial index query.

explain select   from place where  [lat,lng,$spatial] NEAR [3.1333, 101.3833,{"maxDistance": 100000 }]  limit 1
-- poor perfomance.

@martingg88  can you paste here the explain?

@maggiolo00  i think any order clause that is created/aliased on fly will have huge performance degradation.
example as here.
select sum(total)  as total from class order by total.
can you experience this issue at your end?

hope there is a huge performance improvement for order clause. otherwise we found meaningless for this db.


Add a document: All programs show up in Program menu including inactive ones

Reported by: Nepal team, via Marie.
Summary
When adding a new document, the program menu should only include programs with the "funded" status. Currently, the menu includes programs that do not have Funded status. Note: The term "Funded" comes from GAIT; for us, it means "This program is active and should show up in TolaActivity."
Details
Add a Document window
Program Drop Down Menu when adding a New Document in TolaActivity. Many of these programs do not have "Funded" status.

Reference for Nepal's programs in TolaAdmin. Note the “Archived” status.

On QA, I set my country to Nepal and then pulled up the list of programs on the Documents page. Here's what I see:

I am no longer seeing archived programs such as all the ones starting with "2015".


Add SteelSoulHUD

Well,I have already make a mod about this named ChangeHud...,and it can toggle in game.
https://github.com/ygsbzr/HollowKnight.ChangeHud

it's not like there are also 2-3 mods to do with fury of the fallen and 1 hp shenenigans