MAX32660 EVSYS Board Can't Flash

Hey,
I was running Hello_World example on it But I can't flash my board, it is giving the error:
Remote communication error. Target disconnected.:
I am using VS (Eclipse made me too much troubles) and unfortunately I had never succeed to flash it.
I did all the instructions mentioned on issue #883 step by step - updated the adapter firmware and erase but still no success.
this is my details on DAPALINK:
Unique ID: 04211702bfb8038600000000000000000000000097969906
HIC ID: 97969906
Auto Reset: 0
Automation allowed: 1
Overflow detection: 0
Incompatible image detection: 0
Page erasing: 1
Daplink Mode: Interface
Interface Version: 0256
Git SHA: 34fe95d79afeba62ef097774ecc609ef8c2b74ea
Local Mods: 1
USB Interfaces: MSD, CDC, HID, WebUSB
Interface CRC: 0xd7617efe
Remount count: 0
URL: http://www.maximintegrated.com/max32660
this is what written on terminal:

looking forward for any help :)

Hi @yuval5295, have you ever been able to program the board?  Or this is the error you're seeing on a first-time attempt?

Hey Jake!
I have never succeeded to program the board. I just started to try run the
first example and it’s keep fail flashing.
בתאריך יום ב׳, 29 באפר׳ 2024 ב-18:35 מאת Jake Carter <
@.***>:

Hi @yuval5295 https://github.com/yuval5295, have you ever been able to
program the board? Or this is the error you're seeing on a first-time
attempt?
—
Reply to this email directly, view it on GitHub
https://github.com/analogdevicesinc/msdk/issues/1003#issuecomment-2083055109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO467GTHTA27R74UEJIUVQTY7ZSEVAVCNFSM6AAAAABG5ARPISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBTGA2TKMJQHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Do you have a jumper installed on JP1 (labelled "Target VDD")?
You'll need to select either 3.3V or 1.8V.  Either setting should work, but you'll need to select at least one to supply power to the MAX32660

Yes I have and I put the jumper on 1.8V option.
בתאריך יום ב׳, 29 באפר׳ 2024 ב-19:42 מאת Jake Carter <
@.***>:

Do you have a jumper installed on JP1 (labelled "Target VDD")?
You'll need to select either 3.3V or 1.8V. Either setting should work, but
you'll need to select at least one to supply power to the MAX32660
—
Reply to this email directly, view it on GitHub
https://github.com/analogdevicesinc/msdk/issues/1003#issuecomment-2083195379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO467GVGX43SYPFVVB3LOMDY7Z2ADAVCNFSM6AAAAABG5ARPISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBTGE4TKMZXHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

We re-tested with a board on our side and that latest PICO firmware you're using should work.
Can you run the following command from your VS Code terminal, and paste the output?  It will print some additional debug info.
openocd -s $MAXIM_PATH/Tools/OpenOCD/scripts -f interface/cmsis-dap.cfg -f target/max32660.cfg -c "init;reset halt" -d3

The error you're getting is the generic error when there is a connection issue to the micro on the SWD port, the micro is powered off, the host PC can't find the debugger, etc.

@yuval5295 looks like the image didn't make it to the ticket from the email.  Can you paste in the terminal output, or attach the image to the ticket on Github?

Sure, here it is. [image: image.png]
On Sat, 4 May 2024 at 03:47, Jake Carter @.***> wrote:

@yuval5295 https://github.com/yuval5295 looks like the image didn't
make it to the ticket from the email. Can you paste in the terminal output,
or attach the image to the ticket on Github?
—
Reply to this email directly, view it on GitHub
https://github.com/analogdevicesinc/msdk/issues/1003#issuecomment-2093922281,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO467GVTULP6ERL67YLZ623ZAQVYXAVCNFSM6AAAAABG5ARPISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJTHEZDEMRYGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry, still not showing:

Can you paste the actual terminal output?  CTRL+SHIFT+C should let you copy from the VS Code terminal

Alternatively you can run
openocd -s $MAXIM_PATH/Tools/OpenOCD/scripts -f interface/cmsis-dap.cfg -f target/max32660.cfg -c "init;reset halt" -d3 > openocd_log.txt 2>&1

and attach "openocd_log.txt"

cfg -c "init;reset halt" -d3
Open On-Chip Debugger (Analog Devices 0.12.0-1.0.0-7)  OpenOCD 0.12.0
(2023-09-27-07:53)
Licensed under GNU GPL v2
Report bugs to @.***>
User : 3 3 options.c:52 configuration_output_handler(): debug_level: 3
User : 4 7 options.c:52 configuration_output_handler():
Debug: 5 8 options.c:233 add_default_dirs(): bindir=/usr/local/bin
Debug: 6 10 options.c:234 add_default_dirs():
pkgdatadir=/usr/local/share/openocd
Debug: 7 13 options.c:235 add_default_dirs():
exepath=C:/MaximSDK/Tools/OpenOCD
Debug: 8 16 options.c:236 add_default_dirs(): bin2data=../share/openocd
Debug: 9 18 configuration.c:35 add_script_search_dir(): adding
C:/Users/97254/AppData/Roaming/OpenOCD
Debug: 10 21 configuration.c:35 add_script_search_dir(): adding
C:/MaximSDK/Tools/OpenOCD/../share/openocd/site
Debug: 11 25 configuration.c:35 add_script_search_dir(): adding
C:/MaximSDK/Tools/OpenOCD/../share/openocd/scripts
Debug: 12 29 command.c:155 script_debug(): command - ocd_find
interface/cmsis-dap.cfg
User : 13 32 command.c:608 command_run_line(): embedded:startup.tcl:28:
Error: Can't find interface/cmsis-dap.cfg
in procedure 'script'
at file "embedded:startup.tcl", line 28
On Fri, 10 May 2024 at 01:34, Jake Carter @.***> wrote:

Alternatively you can run
openocd -s $MAXIM_PATH/Tools/OpenOCD/scripts -f interface/cmsis-dap.cfg -f target/max32660.cfg -c "init;reset halt" -d3 > openocd_log.txt 2>&1
and attach "openocd_log.txt"
—
Reply to this email directly, view it on GitHub
https://github.com/analogdevicesinc/msdk/issues/1003#issuecomment-2103553490,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO467GXGSNO6GUAWKMACZ73ZBP2XXAVCNFSM6AAAAABG5ARPISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBTGU2TGNBZGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hellp Jack!
it seems nothing printed on the command line after I sent this command but
it makes D1 blink green light fast (which not before I send it).
What should I do next?
Thanks!
Yuval.
On Tue, 14 May 2024 at 00:32, Jake Carter @.***> wrote:

@yuval5295 https://github.com/yuval5295 are you on Windows? Sorry - I
used the Linux syntax for environment variables. On Windows it should be:
openocd -s %MAXIM_PATH%/Tools/OpenOCD/scripts -f interface/cmsis-dap.cfg -f target/max32660.cfg -c "init;reset halt" -d3 > openocd_log.txt 2>&1
(%MAXIM_PATH% instead of $MAXIM_PATH)
—
Reply to this email directly, view it on GitHub
https://github.com/analogdevicesinc/msdk/issues/1003#issuecomment-2108837127,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO467GQLEDONHOJ2AKMOHFTZCEWQDAVCNFSM6AAAAABG5ARPISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBYHAZTOMJSG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @yuval5295,
That actually means that the command has successfully connected to the MAX32660.  The OpenOCD server has launched and is waiting for a GDB client connection.  Essentially it means that the debugger is working properly with the command.
This means that there could be some problem specific to the VS Code setup.
Have you set MAXIM_PATH correctly in your user settings.json file?  See here
Please copy/paste the entire contents of your settings.json file here as well.

Hello Jake!
Great news, you are totally right. the problem was the MAXIM_PATH I set, by
accident I put two // instead of one!

Also I wanted to ask, can you tell me what should I check if I can't see
the prints on VS terminal? I used PuTTy instead but it seems that VS
Terminal should work as well.
Thank you again,
Yuval.
On Wed, 5 Jun 2024 at 02:18, Jake Carter @.***> wrote:

Hi @yuval5295 https://github.com/yuval5295,
That actually means that the command has successfully connected to the
MAX32660. The OpenOCD server has launched and is waiting for a GDB client
connection. Essentially it means that the debugger is working properly with
the command.
This means that there could be some problem specific to the VS Code setup.
Have you set MAXIM_PATH correctly in your user settings.json file? See
here
https://analogdevicesinc.github.io/msdk//USERGUIDE/#getting-started-with-visual-studio-code
Please copy/paste the entire contents of your settings.json file here as
well.
—
Reply to this email directly, view it on GitHub
https://github.com/analogdevicesinc/msdk/issues/1003#issuecomment-2148565617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO467GX7YWL23KSZX6MYS3TZFZDMZAVCNFSM6AAAAABG5ARPISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBYGU3DKNRRG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey, no problem at all.
You can use the Serial Monitor extension.  This is what I use.
Just be aware it has a small quirk - it will send a special serial command to reset the micro when you connect to the serial port...  This can make debugging a bit strange if you're not aware of it.  It's why I have not officially recommended it in the User Guide.  But the extension otherwise works very well.


Does it Work for L5.4?

Environment

PHP Version: 7
Laravel Version: 5.4
LaravelCaffeine Version: 0.3.12

Question
I just installed for a 5.4 laravel instalation running on forge. It doenst trace any errors. i dont know if it works yet. I openen a form and have being waiting for over 2 hours and it looks it works. But have you test it out? so i can push it to production?

Yes, I have it installed on L5.4 systems and testing has shown it to work. Please let me know if you encounter any issues, I will try to respond as quickly as I can. Thanks for using Caffeine! :)


pythonPackages.*: disable failing python2 builds

Motivation for this change
all of these packages no longer build on python2 (determined by upstream). Most commits have related failure message
this in preparation of ZHF, but may as well clean up master anyway
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

https://github.com/NixOS/nixpkgs/pull/95540
2 packages built:
python27Packages.pymystem3 python27Packages.wsproto

$ nix-shell /home/jon/.cache/nixpkgs-review/pr-95540-1/shell.nix


Data

Generators for graphs. Huge graphs are compressed. I'm not sure should we provide GTgraph as sources or as precompiled binaries.

We should not store .o files in git.
I did not understand where Data.tar.xz is. I see only Matrices.tar.xz (by the way, in several places).
Why do we need all these .c and .h files? Does this program generate test files or something like that?

@SokolovYaroslav 1) I think, that we can compile it from sources. Yes, .o files in git is a bad idea.
2) It is my mistake. Now I compress only big matrices for some cases. Readme should be updated. 3) It is a special graph generator. We use it for random sparse graphs generation.

@gsvgit
3) Are these random graphs automatically generated as part of benchmarking, or user should generate them manually?

@SokolovYaroslav

Docker image is changed. GTgraph is added.
Readme is updated (in PR)
Random graphs (and other graphs) are automatically generated by init script which should be run manually (https://github.com/SokolovYaroslav/CFPQ-on-GPGPU/pull/34/files#diff-200fddbfb083dd7a5f02df17f800800a). This script unpacks graphs for RDFs and static code analysis (it is too hard to generate these files) and generates other files (Random, Full, Worst).

I haven't computer with NVidia. so I can't test anything.
Such python call CMD ["python", "testing_system.py", "tests.csv", "init.py"] actually works? And what its behavior? It first runs testing_system.py with argument tests.csv and then runs init.py? Seems very strange.
In readme: 3. Run 'init.py' script to initialize environment.. You want to user runs init.py from Docker? I think, we should describe this as standalone step from the same docker image, but with different script.

@SokolovYaroslav Should we move existing implementations compilation to Docker image initialization step? For example, I can include the existing Python code into init.py. In my opinion, the testing system should not include a compilation step.

@SokolovYaroslav

Fixed. Now python interpreter is specified as the entry point. So, the user can execute arbitrary python script inside docker.
init.py should be executed by the user inside docker. Instructions are added into Readme.

We have two Matrices.tar.xz. Is it okay? I can't verify even that they are not same, because of the git lfs quota (at the moment no one can download the data of this repo from lfs).

@SokolovYaroslav Yes, we should have 2 different Matrices.tar.xz. They should be placed in different folders.


Problem with parents inheriting vh value

Hi @Faisal-Manzer - love the plugin and have been using it for a while, but have discovered an irritating bug.
In the below code, .form-field is oddly inheriting the min-height value from the nested child, which obviously it shouldn't do this. Is there a know bug or conflict with this?
Thanks
.form-field {
    width: 100%;

    // Textarea
    &.textarea {

      // Input
      textarea {
          min-height: 25vh;
      }
}

What is expected output and current output?

What is expected output and current output?

Sorry for the slow reply, this issue has just cropped up again.
The expected output would be (and this does happen):
.form-field.textarea textarea {
    min-height: 25vh;
    min-height: calc(var(--vh, 1vh) * 25);
 }

But additionally, the parent(s) inherit the value too:
.form-field {
    min-height: calc(var(--vh, 1vh) * 25);
}

.form-field.textarea {
    min-height: calc(var(--vh, 1vh) * 25);
}

My SASS markup is as follows (I'm using Tailwindcss, hence the @apply statement):
.form-field {

    // Textarea
    &.textarea {

      // Input
      textarea {
        @apply min-h-1/4;
      }
    }
}

Hope this helps

Was there ever a resolution to this @Faisal-Manzer ?


Add GitHub actions testing

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


How to free Memory after testing

After finished the YCSB test on redis cluster on kubernetes StatefulSet. The memory used by the container is still very high, how to free MEM after testing.
for example, after I run this command
./bin/ycsb.sh load redis -s -P workloads/workload_read -p redis.host=<IP_ADDRESS> -p redis.port=6379 -threads 100 -p redis.cluster=true
workload_read is
recordcount=10000000
operationcount=10000000
workload=com.yahoo.ycsb.workloads.CoreWorkload
readallfields=true
readproportion=1
updateproportion=0
scanproportion=0
insertproportion=0
requestdistribution=zipfian

After finishing test for a while, I got:
$ kubectl top pod
NAME                                  CPU(cores)   MEMORY(bytes)
centos7-deployment-7ddccf8767-c5tkg   0m           176Mi
redis-app-0                           2m           1059Mi
redis-app-1                           2m           1055Mi
redis-app-2                           2m           1581Mi
redis-app-3                           2m           1051Mi
redis-app-4                           2m           1059Mi
redis-app-5                           2m           1052Mi
redis-app-6                           2m           1059Mi
redis-app-7                           2m           1056Mi

that will be up to redis tooling. YCSB doesn't delete the dataset stored in a load or run. Since Redis is an in-memory store the dataset will still be present in memory.


Heads-Up: Copy edits

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

File
Status
Preview URL
Details

mixed-reality-docs/using-the-hololens-emulator.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add qualifiers to all imports in some form or another

It's incredibly confusing reading through the code and not being able to determine where a variable is being imported from (usually from the Tendermint side of things). It helps very much with clarifying where to find definitions of the code and I think there should be efforts in one form or another to add them in. Furthermore, it helps with open source contributors to know wtf they are doing as well. If there are equally named packages, by all means, throw a well named alias in there. But please get all . imports out of the code.
tl;dr No more . aliasing of imports.

On all new and refactored code this coding style has been and will be observed.

Okay...but there is still code that exists with this. Would you like me to help you clean it?


[Event Request] Codeunit 80 Sales-Post - OnBeforePostResJnlLine

Please add a new publisher event to the function PostResJnlLine as follows:
LOCAL PostResJnlLine(VAR SalesHeader : Record "Sales Header";VAR SalesLine : Record "Sales Line";VAR JobTaskSalesLine : Record "Sales Line")
//>> Start New Publisher Call
OnBeforePostResJnlLine(SalesHeader, SalesLine, JobTaskSalesLine, IsHandled);
IF IsHandled THEN
  EXIT;
//<< End New Publisher Call
IF SalesLine."Qty. to Invoice" = 0 THEN
  EXIT;
.
.

New Publisher Event:
LOCAL [IntegrationEvent] OnBeforePostResJnlLine(VAR SalesHeader : Record "Sales Header";VAR SalesLine : Record "Sales Line";VAR JobTaskSalesLine : Record "Sales Line";VAR IsHandled : Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


unix,benchmarks: use fd instead of FILE* after fork

The FILE* object is not guaranteed to be in the same state after
a fork. Instead store the file descriptor instead and use that in
the child process.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/296/

Landed in https://github.com/libuv/libuv/commit/d4f3a42ec74e8596fb88c1c56db57a86cc6470bb. Thanks!


[SPARK-35772][SQL] Check all year-month interval types in HiveInspectors tests

What changes were proposed in this pull request?
Check all year-month interval types in HiveInspectors tests
Why are the changes needed?
Check all year-month interval types in HiveInspectors tests
Does this PR introduce any user-facing change?
NO
How was this patch tested?
Added UT

Test build #140016 has started for PR 32970 at commit 7f755c6.

@MaxGekk  Since  Hive's type not support startField/endField, should we support such as in script transform, input as YearMonthIntervalType(YEAR, MONTH) and value is 13 then output defined as  YearMonthIntervalType(YEAR, YEAR) and result is 12?

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44542/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44542/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44542/

Test build #140016 has finished for PR 32970 at commit 7f755c6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140016/

Test build #140067 has started for PR 32970 at commit 0ea1636.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44594/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44594/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44594/

Test build #140067 has finished for PR 32970 at commit 0ea1636.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140067/

Test build #140175 has started for PR 32970 at commit 4c74dde.

Test build #140176 has started for PR 32970 at commit 79486b3.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44702/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44702/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44702/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44703/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44703/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44703/

Test build #140176 has finished for PR 32970 at commit 79486b3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #140175 has finished for PR 32970 at commit 4c74dde.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140176/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140175/


New erc20-balance-of strategy with a condition of having at least one…

Changes proposed in this pull request:

added new tested erc20-balance-of where votes are conditionally counted based of having at least oneindicated ERC721 token

HI @DesiredDesire I feel like you can achieve this with math strategy https://snapshot.org/#/strategy/math
let me know if it doesn't work :)

Hey @ChaituVR, I managed to do that with your strategy. I had to wrap all strategies in operands with multichain strategy to make it work.


Temporarily skipping Update-Help and Save-Help tests in CI builds since the fwlinks for powershell modules are pointing to the wrong location.

Will keep this issue open to track the task of reverting skipping the tests, once the URIs are fixed.
Steps to reproduce
Update-Help -Module microsoft.powershell.core
Expected behavior
The help content for microsoft.powershell.core is updated
Actual behavior
Update-Help : Failed to update Help for the module(s) 'Microsoft.PowerShell.Core' with UI culture(s) {en-US} : The
value of the HelpInfoUri key in the module manifest must resolve to a container or root URL on a website where the
help files are stored. The HelpInfoUri
'https://msdn.microsoft.com/powershell/reference/5.1/PSWorkflowUtility/Invoke-AsWorkflow' does not resolve to a
container.
At line:1 char:1

Update-Help -Module microsoft.powershell.core

  + CategoryInfo          : InvalidOperation: (:) [Update-Help], Exception
  + FullyQualifiedErrorId : InvalidHelpInfoUri,Microsoft.PowerShell.Commands.UpdateHelpCommand

Environment data

Name                           Value
----                           -----
PSEdition                      Core
WSManStackVersion              3.0
CLRVersion
PSVersion                      6.0.0-alpha
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
SerializationVersion           <IP_ADDRESS>
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.13-12-gea464fc169069126e00afce4cde263098bfc704d-dirty
PSRemotingProtocolVersion      2.3

@Francisco-Gamino URLs are supposed to be fixed in the back-end now

This issue has been fixed on the back end.


Refactored References to not require a selected dimension

Fixes #91 Refactored references to no longer require that the reference dimension is selected.

Coverage decreased (-0.007%) to 97.516% when pulling 3b2e5deac49201a610b3ac7ed5ad5a0ebd231b1c on references_no_dimension_in_query into 005ab8079f1023d9b3f201202d9eff688bba96a5 on master.


Regarding preferences

Just a question, why are you not using a UI window which uses NSUserDefaults to store the preferences instead of a .shuttle.json file ?

Hi @madhur
with the recent changes shuttle now has two sets of options.

Global Settings.
Command level Settings.

Eventually Global settings with most likely be placed in the user interface; as you suggest.
Command level settings will stay in the JSON file. Some users have several commands and scripts in nested menus and all this is much more elegant and easy to edit in the JSON file.

:+1:


Commants and typipos

Typo exisiting -> existing https://github.com/GoogleCloudPlatform/cloud-run-events/pull/35/files#diff-1d699b9cd7b3d4b5ad8802c07777a902R324 https://github.com/GoogleCloudPlatform/cloud-run-events/pull/35#discussion_r298765875
Add a comment saying "This method is generic for future compatibility with an SDK or some such" https://github.com/GoogleCloudPlatform/cloud-run-events/pull/35/files#diff-1d699b9cd7b3d4b5ad8802c07777a902R312 https://github.com/GoogleCloudPlatform/cloud-run-events/pull/35/files#r301248760

Remove add the finalizer from comment because that's already happened https://github.com/GoogleCloudPlatform/cloud-run-events/pull/35#discussion_r301238029 https://github.com/GoogleCloudPlatform/cloud-run-events/pull/35/files#diff-1d699b9cd7b3d4b5ad8802c07777a902R155


error when trying to build with cmake

Brief Issue Summary

help
CMake Tools Diagnostics
CMake: : The term 'CMake:' is not recognized as the name of a cmdlet,
function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ CMake: Log Diagnostics
+ ~~~~~~
    + CategoryInfo          : ObjectNotFound: (CMake::String) [], CommandNotFo
   undException
    + FullyQualifiedErrorId : CommandNotFoundException

Debug Log
No response
Additional Information
cmake is installed well but somehow vsc error

@dananothername Thanks for posting this issue, we'll look into it!
To help us, could you run the command CMake: Log Diagnostics within VSCode by using the command pallete? This should give us some information that may help us diagnose.
Additionally, are you able to invoke cmake on a fresh command-line? It seems that the extension can't find the cmake executable.
Thanks!

Another thing to try is to set the cmake.cmakePath to the location of your cmake executable on disk. Let us know if these suggestions and questions help!

about the vscode can't identify cmake executable yes i think so
while i already download cmake
also by setting cmake... do you mean
file/open folder /

@dananothername If I'm understanding you correctly, yes, you should navigate to the VSCode Settings UI, and edit the CMake Tools setting for cmake.cmakePath to the location of your cmake exe.

Open the Command Pallete by navigating View -> Command Pallete
Start typing and then select "Preferences: Open Settings (UI)"
Enter "cmake.cmakePath" into the box.
Edit the setting to point at your cmake installation.

Additionally, could you provide some information about your system? What OS are you using, what version of VSCode, what version of the extension?
Thanks!

i am on windos 6
vsc 2019
just downloaded this cmake 3.26.0
https://cmake.org/download/
after i set the cmake path

and trying to build it

this happens

In order to use CMake, you need to have a both a compiler (like MSVC, GCC, or Clang) and a build system (like Ninja or MSBuild, known as generators in CMake) installed. What were you planning to use? A simple way to get started is by installing Visual Studio, which includes MSVC and Ninja.

https://stackoverflow.com/questions/4822400/register-an-exe-so-you-can-run-it-from-any-command-line-in-windows#comment46075436_4822427
ok after installing bunch of things its still not working

Are you able to successfully use CMake to build your project on the command line outside of VS Code? If it's not working outside VS Code, then it means there's a configuration issue on your system that will need to be resolved before you can use CMake in VS Code.
Also, cmake.cmakePath should be the full path to cmake.exe, not just the directory containing the executable.


Fix CI and reorganize tests

By the way, @schungx, I don't think git will recognize the way I rearranged the files when you try to rebase your branch.
The easiest way to resolve the merge conflict is to put your test changes into the new files, and throw the old ones away.

Hhhmmm... rebase seems to work fine...


GEODE-9486: Fix getSrcPathFor in AnalyzeDataSerializablesJUnitTestBase

Rename DistributedSystemService to SanctionedSerializablesService and
remove unused init(DistributedSystem).
Move SanctionedSerializablesService to geode-serialization.
Implement SanctionedSerializablesService in both geode-core and
geode-management to remove special case for each in
InternalDataSerializer.
Fix sanctioned serializables support in geode-management and
geode-apis-compatible-with-redis.
Add sanctioned serializables support to geode-serialization and
geode-pulse.
Add sanctioned serializables support to geode-junit and geode-dunit
to simplify writing tests for validate-serializable-objects and prevent
having to list out DUnit Rules and other test framework classes when
validate-serializable-objects is enabled.
Use ExecutorServiceRule and reformat json strings in
RestRegionAPIIntegrationTest.
Reorganize QueryCommandDUnitTestBase.
Use InvalidClassException instead of Exception in
ObjectInputStreamFilterWrapper fatal log message.
Improve assertion messages in ResourceUtils.
Move loadSanctionedSerializablesServices and loadClassNames to
new SanctionedSerializables in geode-serialization.
Exclude Pulse GemFireAuthentication from sanctioned serializables.
Add SerializationTest Category to all AnalyzeSerializables integration
tests.
Tidy up SANCTIONED_SERIALIZABLES_DEPENDENCIES_PATTERN.
Convert to AssertJ and use BeforeClass in
InternalDataSerializerSerializationAcceptlistTest.
Note: If Git or GitHub is showing invalid file renames in the diffs, you
may need to pull the branch locally and configure diff.renameLimit to
something lower than the default value of 50.

@onichols-pivotal No, I think that must be an option that's enabled in IntelliJ. I committed from IntelliJ's git plugin. I'll fix it up.


Fixed errors and added examples with variety

Made precise that intercepts are (x,y)-points and not numbers. Added more examples to show different possibilities that can occur and removed the oversized image of a few short lines of equations.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


Java export command does not return an error code in case of a failure

We are using neptune-export.jar to run an export job from our AWS Neptune and in case of a failure during an execution command returns 0 code instead of an error code, which makes it quite difficult to track any failures during our pipeline execution.
Ways to reproduce:
Run
java -jar neptune-export.jar for export
In case of an error, example:
An error occurred while creating property graph config. Elapsed time: 32 seconds
java.lang.IllegalStateException: org.apache.tinkerpop.gremlin.process.remote.RemoteConnectionException: java.lang.RuntimeException: java.lang.RuntimeException: java.util.concurrent.TimeoutException: Timed out while waiting for an available host - check the client configuration and connectivity to the server if this message persist

The return code is 0, I would expect to receive something different since the command has failed.
We are using the latest version - https://s3.amazonaws.com/aws-neptune-customer-samples/neptune-export/bin/neptune-export.jar

Hi @dh-nickbilozerov
Thanks for reporting this. I merged a change this morning that addresses this:
https://github.com/awslabs/amazon-neptune-tools/pull/184
I've also published a fresh version of the jar to https://s3.amazonaws.com/aws-neptune-customer-samples/neptune-export/bin/neptune-export.jar
ian

Closing this issue


Hello World instructions unclear.

There are a few pieces in the Hello World chapter that are not as clear as they could be.

code snippets include optional dependency
Instructions on serving the wasm file are confusing.

1 - The file name referred to in the html/script is hello_world.gc.opt.wasm which is only the name of the file if you've chosen to 'make the binary even smaller'. Because this is an optional step, one may not take it, especially if they're just trying to experiment quickly or getting a working setup. A simple fix would be to not name the file differently during the optional step.
2 - There are two ways to serve the wasm file, one using the https crate (or a webserver one may have already installed on the machine) and another using the Python3 simple server. However the instructions in the Note say there is a stipulation on the webservers that may be used. The majority of users checking this out for the first time will likely be on a machine without a real webserver installed. The Python3 server is used fairly ubiquitously in example code throughout the web, so I think it might be a good idea to list this first.
It is confusing to include both example in a Hello World section. I think it might benefit users to only include one, either require the https crate to be used (which is a reasonable dependency) or only list the Python3 example with a note of the caveat and that there is a potentially better way to do it when the user gets to the real world.

This issue has been moved here


ImageMaxRewritesAtOnce of -1 should allow unlimited image rewrites

We document that setting ImageMaxRewritesAtOnce to -1 removes the limit on image rewrites, but in <IP_ADDRESS> it actually disables all image rewriting.

Fixed by https://github.com/pagespeed/mod_pagespeed/pull/1391


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-12-05)
Features

specify valid conventional commit types (#4) (950278e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/vincent-l-j/project-template/releases/tag/v1.0.0 :sunflower:


Raw UDP over IPv4 traffic with value lists for both source and destination ports

Config

Two back-to-back connected test ports

One for tx
Another for rx

One Flow with,

Fixed frame size of 64B
Rate of 25 PPS
Packet count 100
Packet headers ethernet, ipv4 and udp

UDP header with,

Source port value list
Destination port value list

Validation

Push configuration
Start capture on rx port
Start transmit on tx port
Wait until following is as expected,

Frames tx/rx, bytes tx/rx and transmit state in flow metrics

Fetch capture and ensure following is as expected,

ethernet.ether_type
ipv4.protocol, ipv4.total_length
udp.src_port, udp.dst_port, udp.length
size of each frame
(best effort) checksum of all headers


Help Implementing

Sorry for the noob question, new dev here!
I am using this version:
implementation ("com.github.alexzhirkevich:custom-qr-generator:2.0.0-alpha01")
jitpack is in my settings.gradle
maven("https://jitpack.io")
Yet theres a problem importing some functions:
import com.github.alexzhirkevich.customqrgenerator.QrData
import com.github.alexzhirkevich.customqrgenerator.createQrVectorOptions
import com.github.alexzhirkevich.customqrgenerator.dsl.QrVectorLogoShape
Unresolved reference: createQrVectorOptions
Unresolved reference: dsl
Am I missing something?

Still learning my way around the libraray;
changed import com.github.alexzhirkevich.customqrgenerator.createQrVectorOptions
to import com.github.alexzhirkevich.customqrgenerator.vector.createQrVectorOptions

Open 2.0 branch and check app example

here

Thank you!! Starting to figure things out, Literally my first year coding, still learning everything. Thanks!


Card payments not working after PR #54

Hey,
the changes to the WC_Adapter class in #54 have had the unfortunate consequence of breaking card payments for us (I haven't deployed it to production yet so no worries. :) ), we use the Swedbank Pay WooCommerce Payments plugin. The problematic change are these lines https://github.com/SwedbankPay/swedbank-pay-woocommerce-core/pull/54/files#diff-89a77008217563f17d3a6fe6a0152dda195c02c398465b7c5d879a94144ab877R148-R159.
The issue here is the boolean check $this->gateway->reject_credit_cards (and you can replace reject_credit_cards with any of the other properties) which will always be true since the value of reject_credit_cards is either (string) 'yes' or (string) 'no'. So no matter what one has choosen for these options they will always be set to true when posting to the swedbank pay api which means that the card payment will fail in swedbank pay with the error ACCOUNTHOLDERTYPEREJECTED or CARDTYPEREJECTED. Of course I don't know the reasoning behind this change, or if it's just for us it breaks, but reverting these lines (or changing them to use the same logic as the save_cc property) makes the card payments work again.

@lakrisgubben Thank you! We will fix it

@lakrisgubben It has been fixed in the Wordpress Plugins directory https://profiles.wordpress.org/swedbankpay/#content-plugins.
We will merge the fix after review there

Thanks for the quick response @aait! :)
It looks like ConfigurationInterface::USE_PAYER_INFO is declared twice now in https://github.com/SwedbankPay/swedbank-pay-woocommerce-core/commit/ded00f763e1a5d7d8300332854b1636734779661
I use the plugin from github so I'll test this out once #57 is merged and a 4.0.1 is released in this repo.
best


Citigo iV: "skoda-set" node not working

When trying to use set node - it does not log in and whole Node Red server restarts itself. Probably plugin not able to log into the servers. "skoda-get" plugin working and fetching data correctly after accepting Terms of Use on Skoda Connect portal (before accepting Terms of Use it baheved the same - "get" node was not able to log in and whole Node Red server also restarted). Not sure if there is something similar what I can do, so "skoda-set" plugin would work. Tested on changing climater temperature and also on switching climater on/off - inputs correct as advised in guide.

Now working - incorrect input node, now "skode-set" works as expected.


Fix policy_based_routing.sh script on simple-nva module

Fixed policy_based_routing.sh script on simple-nva module when dealing with network interfaces either not associated to an internal TCP Load Balancer (infinite loop in while cycle) or in case of multiple LB pointing to the NIC (probably ip route command failing with multiple line response received from ip r show table local).
As per documentation added in the script executing a curl request to  http://metadata.google.internal/computeMetadata/v1/instance/network-interfaces/$IF_NUMBER/forwarded-ips/ would return "" when no load balancer is associated to that network interface or a new line separated list of indexes (0 1 ..), one for each load balancer associated. A further request to http://metadata.google.internal/computeMetadata/v1/instance/network-interfaces/$IF_NUMBER/forwarded-ips/[0,1,..] will return the specific LB ip address.
We could evaluate an improvement using that curl instead of waiting for the route to be installed on the local routing table by the guest agent.

Review the script since previous version was not taking into account load balancer configured after instance creation. In the newer version policy_based_routing.sh script runs on background which allows routing table to be configure properly via the start_routing.sh script. The policy_based_routing.sh script will check every two seconds for new load balancer routes configured from the guest agent, for each LB available on that network interface it will check if the policy based route is configured, otherwise it will do that.
Overall this should solve the previous state issue with network interfaces not having LB associated (stuck in the while loop without configuring VM routes) as well as managing more than 1 LB associated to the network interface (i believe the previous version of the script would have failed in such circumstance.


fix arrow-json encoding with dictionary including nulls

Rationale for this change
While dictionary values are generally not null (it's more efficient to have a null in the keys), there's nothing to stop them from being nulls, and currently datafusion-functions-json generates dictionaries with null values.
Until now they caused arrow-json to panic, that is now fixed.
What changes are included in this PR?
Use make_encoder_impl, not make_encoder in DictionaryEncoder to avoid unnecessary and incorrect nulls.is_none() assertion.
Are there any user-facing changes?
no.

Related: we've copied arrow-json and customised it to support the unions generated by datafusion-functions-json, and also to inline JSON data within JSON based no field metadata, would you be inerested in accepting logic to allow customising of JSON encoding?

See https://github.com/apache/arrow-rs/pull/5318#discussion_r1465334902 for discussion of the nulls.is_none() assertion.


Do not rely on function names to populate Sharp's prototype so that the library can be minified. Fixes #1474.

That should solve the issue. All unit tests are passing on my machine. I wasn't able to control if there wasn't other issues in the code regarding minification compatibility since msbuild is being particularly difficult when I reference sharp as a file: dependency.

Coverage decreasing... that's what happens when you remove lines of code :/

Fantastique, merci beaucoup.


Release 3.0

This PR has the new authentication process complying with OAuth2, including refresh tokens and JWT access tokens.

SL 8.45

GH 8,45


no https url in firefox.sh

In https://the-raggedy-dev.teachable.com/courses/uncluttered-test-driven-web-development/lectures/50520258

the url should be http:// and not https://
the -no-remote option makes -new-instance redundant
adding -url option adds more clarity (completely subjectif)

# CLI options detailed here https://wiki.mozilla.org/Firefox/CommandLineOptions
DATA_DIR="$(mktemp -d -t 'firefox-unsafe_data_dir.XXXXXXXXXX')"
"${FIREFOX}" \
  -profile $DATA_DIR \
  -no-remote \
  -url http://localhost:8888/ \
  >/dev/null 2>&1 &!

Ah, you're absolutely correct.
Thanks for taking the time to submit an issue. I really appreciate it. It should be fixed now.


Add test for HandleHTTP

@h2non I'm not sure if these tests were properly handled. IMO it properly tests every part (writer, request, interception) of the method but there may be a cleaner/better way to do that.
I'll start working on the filters for the request once this is merged.

Changes Unknown when pulling 1e1f7b49ffca987b19a58cbdeb4f3a07843c2536 on groyoh:add_test_for_handle_http into ** on vinxi:master**.

I guess this is only for Request interceptor. It would be nice to have tests for the Response as well. But we can do it in separated PRs.

Yes, that's only for the request. I don't want to add too much at one cause I'm not always sure how to handle things.

Changes Unknown when pulling 58123eb45c9bf9e7cda3d3004eab77b210b7d644 on groyoh:add_test_for_handle_http into ** on vinxi:master**.


FYPO:0001214  sensitive to potassium chloride

FYPO:0001214  sensitive to potassium chloride is not currently a child of sensitive to salt stress

also, it seems sensible to also have "sensitive to NaCl", and then I can move a lot of the existing sensitive to salt stress + condition annotations to the more specific terms?

Hmm...
another 'popular' salt is CaCl2. For this I think I  pick "sensitive to calcium" or "sensitive to salt stress + CaCl2 condition" depending on the context....which doesn't really feel like a satisfactory way to do it.
How do you think we could handle these annotations best overall? (in reality it is impossible to know if the cells are sensitive to e.g. the calcium ion or the osmotic stress, unless they do several tests, which they often don't do.

Maybe we need to discuss this one.
I think I favour specific phenotypes, because as you say there is often know way to know what the sensitivity is to (calcium or osmotic stress), but not specific GO terms   for the chemical,  but for the response type
during response to heavy metal
during response to osmotic stress
where different pathways deal with the insult

I don't usually think of calcium chloride as a salt stress - chemically it's a salt, but I would annotate to "sensitive to calcium" because calcium ions have specific effects. (I also tend to think of salt stress sensitivity as being tested using NaCL or KCl, and osmotic with sorbitol.)
As for classifying sensitivity to NaCl or KCl, you know I try to avoid doing too much of that manually because it can be hard to maintain. But if NaCl/KCl always imposes salt stress, I can move those terms down. Do you know of any exceptions?
This issue has nothing to do with whether you would or wouldn't do a GO annotation so let's not drag that in any further.

Genetically there are differences between NaCl stress and KCl stress - sty pathway mutants are sensitive to both, whereas git mutants are sensitive to KCl (and CaCl).
I don't think it makes sense to specifically lump NaCl and KCl. This is a random image I grabbed when googling KCl NaCL cell import, it looks like they can go via quite different routes so it makes sense their effects are not same same but different

That's all interesting, and useful for annotations, but what I need to know for the ontology is kind of the other way round. I didn't think there was ever any suggestion that we would "lump KCl and NaCl" in the sense of merging the FYPO terms with each other or any less specific term.
Instead, I need to know whether there's ever any case of "sensitive to KCl" that would NOT count as "sensitive to salt stress", and same question for NaCl. That will tell me whether it's OK to move one or both terms to be is_a "sensitive to salt stress".

Well, I don't think "sensitive to potassium" or "sensitive to sodium" should be is_a "sensitive to salt stress" but KCl and NaCL are salts by definition..?

added
sensitive to sodium chloride FYPO:0005889
I haven't put it or 'sensitive to KCl' under salt stress yet.

closing because old; reopen if there is any specific thing to do


Rating

Hi, I was wondering whether you have a sample for the Rating. I'm not sure how to implement the thumbs up and down rating feature

@khaisayhai did you find the way ?if you did can you post the answer and close the issue ?

I don't think this is implemented. I would also like to know how to use this feature.

It is implemented, but currently it only shows up in external Android devices (e.g. smartwatches, cars, etc). It doesn't add anything to the notification, but there is already a feature suggestion for that in #67

So if you hit the thumbs up in the smart watch, how do you capture the event? Is there an event that fires?

Yes.
First of all, you need to set the the ratingType option using updateOptions. It must be a Rating Constant. This basically says which type of rating you want to use throughout the app.
The rating value changes depending on the rating type. For instance, a RATING_THUMBS_UP_DOWN is a boolean, being true (thumbs up), false (thumbs down) and null/undefined (unset rating)
You can set the defined rating value for each track through the rating property in the Track Object. If you don't define a value, it will treat as "unset rating".
When the user updates the rating, you'll receive the remote-set-rating event with the value (that depends on the rating type).
Heres is a quick example:
// Sets the rating type
TrackPlayer.setOptions({ratingType: TrackPlayer.RATING_5_STARS});

// Adds a track with the rating of 4 stars
TrackPlayer.add({
    // ...
    rating: 4
});

TrackPlayer.addEventListener('remote-set-rating', (data) => {
    console.log(data.rating); // A number from 0 to 5 stars
    // ...
});


Fix #59 - Use IPs instead of hostnames for datanode connections

Also fixed a memory leak where H_DATANODE_INFO hostname and port strings would not be freed
Unsure about the correctness/completeness of the updates to the Cython wrappers

Cleaned up and pushed.

Excellent!  Thank you.


First draft revision of triangle_facts.rb

Please fix Hound errors, refactor and resubmit.  Hound is pretty accurate in regards to Enova's style preferences.  Use rubocop locally with the included .hound.yml file to help detect these issues early.


TypeScript definitions

I've developed some (slightly partial, but easily completed) TypeScript definitions for my project (.d.ts file). It's typical for libraries to ship with them directly though. Would this be something interesting to include here?
If so, I'll spend some time cleaning up + making sure it works as part of the library.

it should not hurt, right?

It would hurt if I did the work and you didn't integrate it :) Since it'd
be an hour or so to get everything right, figured I'd ask first.

declare enum RoundingMode {
    halfUp = "half-up",
    halfDown = "half-down",
    halfEven = "half-even",
    up = "up",
    down = "down",
    floor = "floor",
    ceil = "ceil"
}
interface Rounding {
    roundingMode: RoundingMode;
    maximumSignificantDigits: number;
}
declare class BigDecimal {
    static BigDecimal(value: bigint | string | number | BigDecimal): BigDecimal;
    static toBigInt(a: BigDecimal): bigint;
    static toNumber(a: BigDecimal): number;
    static unaryMinus(a: BigDecimal): BigDecimal;
    static add(a: BigDecimal, b: BigDecimal, rounding?: Rounding): BigDecimal;
    static subtract(a: BigDecimal, b: BigDecimal, rounding?: Rounding): BigDecimal;
    static multiply(a: BigDecimal, b: BigDecimal, rounding?: Rounding): BigDecimal;
    static divide(a: BigDecimal, b: BigDecimal, rounding?: Rounding): BigDecimal;
    toString(): string;
    toFixed(fractionDigits: number, roundingMode?: RoundingMode): string;
    toPrecision(precision: number, roundingMode?: RoundingMode): string;
    toExponential(fractionDigits: number, roundingMode?: RoundingMode): string;
    static log(a: BigDecimal, rounding: Rounding): BigDecimal;
    static exp(a: BigDecimal, rounding: Rounding): BigDecimal;
    static sin(a: BigDecimal, rounding: Rounding): BigDecimal;
    static cos(a: BigDecimal, rounding: Rounding): BigDecimal;
    static atan(a: BigDecimal, rounding: Rounding): BigDecimal;
    static sqrt(a: BigDecimal, rounding: Rounding): BigDecimal;
    static abs(a: BigDecimal): BigDecimal;
    static sign(a: BigDecimal): number;
    static max(a: BigDecimal, b: BigDecimal): BigDecimal;
    static min(a: BigDecimal, b: BigDecimal): BigDecimal;
}
export default BigDecimal;

What I have locally is this:
    export type RoundingMode = "floor" | "down" | "ceil" | "up" | "half-even" | "half-up" | "half-down";

    export interface Rounding {
        maximumSignificantDigits?: number;
        maximumFractionDigits?: number;
        roundingMode: RoundingMode;
    }

    export class BigDecimal {
        toString(): string;
        toFixed(fractionDigits: number, roundingMode?: RoundingMode): string;
        toPrecision(precision: number, roundingMode?: RoundingMode): string;
        toExponential(fractionDigits: number, roundingMode?: RoundingMode): string;
    }

    export namespace BigDecimal {
        export function BigDecimal(value: bigint | number | string | BigDecimal): BigDecimal;
        export function toBigInt(value: BigDecimal): bigint;
        export function toNumber(value: BigDecimal): number;
        export function unaryMinus(value: BigDecimal): BigDecimal;
        export function add(a: BigDecimal, b: BigDecimal, rounding?: Rounding | null): BigDecimal;
        export function subtract(a: BigDecimal, b: BigDecimal, rounding?: Rounding | null): BigDecimal;
        export function multiply(a: BigDecimal, b: BigDecimal, rounding?: Rounding | null): BigDecimal;
        export function divide(a: BigDecimal, b: BigDecimal, rounding?: Rounding | null): BigDecimal;

        export function lessThan(a: BigDecimal, b: BigDecimal): boolean;
        export function greaterThan(a: BigDecimal, b: BigDecimal): boolean;
        export function equal(a: BigDecimal, b: BigDecimal): boolean;

        export function round(value: BigDecimal, rounding: Rounding): BigDecimal;

        export function abs(value: BigDecimal): BigDecimal;
        export function sign(value: BigDecimal): number;
        export function max(a: BigDecimal, b: BigDecimal): BigDecimal;
        export function min(a: BigDecimal, b: BigDecimal): BigDecimal;
    }

Which misses some of the advanced math functions (because I don't use them). It's also missing BigFloat.
I'm not sure if it's better to declare as static methods on a class, or whether to do the merged class + namespace as I have it.

Basically what needs to happen is primary a bunch of testing to ensure that the appropriate ways of importing work as expected with TS. And ensuring that things like "instanceof" behave as one might expect.

@imirkin I have added https://github.com/Yaffle/BigDecimal/blob/master/BigDecimsl.d.ts

Great, thanks! I'll test it out and let you know if there are any issues. I see you left out BigFloat - is that on purpose? I don't need it, but I guess it's in the lib, so good to have an export?
I was thinking maybe something clever can be done with typescript types to avoid the duplication.


Cannot access public available ports from inside non-rancher container

Edit: After some more digging (thanks to Brad over on the drone forum) I found out this is not only an issue with drone, but with every container started in a custom network outside of rancher.
Following Scenario: I have a Git server (Gitea) and a CI System (drone) on the same host running with Rancher. One loadbalancer in front of everything which routes based on domains and handles ssl termination. The CI System has access to the docker socket and spawns containers on the host. These spawned containers cannot acces the public facing services (made public by ranchers loadbalancer) when trying to access the public domain/ip of the server, however they have internet access and can reach the server via ping. Doing a scan with nmap from one of these contaiers gets me the following result:
Host is up (0.00013s latency).
Not shown: 995 closed ports
PORT      STATE    SERVICE    VERSION
80/tcp    filtered http
111/tcp   open     rpcbind    2-4 (RPC #100000)
443/tcp   filtered https
8080/tcp  open     tcpwrapped

The interesting part is in port 80/443, these are the two ports handeled by the rancher loadbalancer. Doing curl on them from inside a drone build container results in a timeout.
I also tried running manually a container on the host (outside of rancher), whithin this i can access everything perfectly normal.
After digging a bit, I think this is probably an issue with ranchers iptables configuration, which is why I posted it here. I also stumbled upon some other issues (#2929, #3618, #1898), but they seem to be resolved.
I've also created a topic in drone's forum: https://discourse.drone.io/t/cannot-access-public-facing-loadbalancer-from-spawned-container/1700
Rancher versions:
rancher/server: v1.6.14
rancher/agent: v1.2.9
Infrastructure Stack versions:
healthcheck: v0.3.3
ipsec: v0.13.7
network-services: v0.7.19 (manager)
scheduler: v0.8.3
Docker version: (docker version,docker info preferred)
Docker version:
Client:
 Version:	17.12.0-ce
 API version:	1.35
 Go version:	go1.9.2
 Git commit:	c97c6d6
 Built:	Wed Dec 27 20:11:19 2017
 OS/Arch:	linux/amd64

Server:
 Engine:
  Version:	17.12.0-ce
  API version:	1.35 (minimum version 1.12)
  Go version:	go1.9.2
  Git commit:	c97c6d6
  Built:	Wed Dec 27 20:09:54 2017
  OS/Arch:	linux/amd64
  Experimental:	false

Docker info:
Containers: 54
 Running: 40
 Paused: 0
 Stopped: 14
Images: 64
Server Version: 17.12.0-ce
Storage Driver: overlay2
 Backing Filesystem: xfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file logentries splunk syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 89623f28b87a6004d4b785663257362d1658a729
runc version: b2567b37d7b75eb4cf325b77297b140ea686ce8f
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.14.0-0.bpo.2-amd64
Operating System: Debian GNU/Linux 9 (stretch)
OSType: linux
Architecture: x86_64
CPUs: 6
Total Memory: 15.67GiB
Name: v45616
ID: PPND:U2AH:GR2O:3PER:XJFF:6NN3:USSQ:AB5B:GCMI:W2UH:3WDX:7A7U
Docker Root Dir: /export/vda3/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

WARNING: No swap limit support

Operating system and kernel: (cat /etc/os-release, uname -r preferred)
$ cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 9 (stretch)"
NAME="Debian GNU/Linux"
VERSION_ID="9"
VERSION="9 (stretch)"
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"
$ uname -r
4.14.0-0.bpo.2-amd64

Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
KVM VPS
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node rancher, internal DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Results:

Same issue here. I've created a Drone stack with the template provided in the catalog and then added a Gogs instance to it, plus its (public) port to the load balancer; everything works fine and I can interact with Gogs.
Both Drone's server and agent can successfully curl Gogs' repos urls if I start a shell inside them, but new containers spawned by Drone on a custom network can't - connection on Gogs' port just times out from inside them, even though the host can correctly be pinged and resolved.


Generate sockops bpf code after node config header is created

The sockops bpf map and code depends on the node config file.
So do this after node_config.h has been generated.
Also, the sockops bpf code is now generated only if dry mode is
not set.
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

This change is

Coverage increased (+0.006%) to 43.247% when pulling 9bc7f50a7fd22fb47047f25021d510df64f888cd on amshinde:generate-sockops-after-node-hdr into 7058789aa0053d0856a3a97079fe33ae6b07f145 on cilium:master.

test-me-please

test-me-please

test-me-please


Move Asciidoctor to the API

Remove deprecated render* methods
Remove rubyExtensionRegistry from the API (bounded to the JRuby implementation)
Only keep Asciidoctor.Factory.create() method to instantiate the processor

Other methods are tightly coupled with the context/implementation but it's also a limitation of the current implementation with ServiceLoader. In fact ServiceLoader will call the default constructor (that's why I've added a public no-arg constructor on JRubyAsciidoctor).

I've kept requireLibrary and requireLibraries methods because I think they can be useful in other implementations.

This was already merged as part of #760


ServiceBus stress tests

I think we can meaningfully say this is complete - we have several stress tests, executing regularly and adhoc. I can file future issues if we decide to enhance it further, and some stress work is also coming in naturally as part of other bug fixes.


Port to 1.18.2

Port for fabric 1.18.2

The mod works in 1.18.2. A list of supported versions is updated now.


[Behavior] auto-close stale issues and pull requests

Configuration:

delay - number of days to wait before closing
some kind of conditions for stale issues - e.g. a label, or a rejected code review, or comment content.

Examples:

Jekyll
Know of any projects that have this behavior?

We use this for all of Jekyll's projects. Here's an example of it in action

:boom: https://github.com/probot/stale


[scrips/install.sh] Ensure configuration backward compatibility in install script

Install script should generate valid configuration independently on sumologic binary version

This has been resolved by https://github.com/SumoLogic/sumologic-otel-collector/pull/779


Update events

We have a lot of them changed, and 1.5 changed them.

All the ones with prefix WTT_ have been already fixed in #478 , along with ss_recruitment_event.txt
stability_events.txt and Decision_events.txt give no problem, the latter one referencing tags but being "triggered_only" by decisions that are deleted.

Looking at it, stability_events.txt has at least one event that has ideology-dependent bits with democracy. Those either need to be removed, or the event being included needs to depend on the ideology existing.

Many of the political decisions have been removed, replaced by political decisions.


Path to /home/kyle is hard coded, somewhere

I'm trying to install the consul module on my OSX Laptop using librarian-puppet, the puppet module install command fails citing an non-existent path of /home/kyle.
Librarian puppet is generating the following module install command: https://gist.github.com/bhourigan/6fc71c3506fdd9cede81
Output of running it with --debug:
https://gist.github.com/bhourigan/8c3b9c1e72ad54e4fc66
I must be missing something important, as I can't find a telling location of the substrings 'home', or 'kyle'. Running puppet module installer under dtruss did give me more information about the full path, but alas, it has not given me more insight into where the problem lies.
Debug command: sudo dtruss -b 32m -f <command in gist 6fc71c3506fdd9cede81>
Interesting snippit (full output available upon request)
65195/0x3bfd22:  symlink("/home/kyle/Projects/puppet_modules/puppet-consul\0", "KyleAnderson-consul-0.4.2/spec/fixtures/modules/consul\0")               = 0 0
I'm running ruby 2.0.0p481, and according to https://docs.puppetlabs.com/guides/platforms.html#ruby-versions it is supported.

Looks like there is a dangling symlink in the released archives for that module, but me too.

Good catch! It's not present in the git checkout, but when I download the .tar.gz from puppet forge I see the following:
22886459        8 lrwxr-xr-x    1 bhourigan        staff                  48 Oct 28 23:26 ./spec/fixtures/modules/consul -> /home/kyle/Projects/puppet_modules/puppet-consul

Whoops. Ug :(
Um.
I've built 0.4.5? tar -tzf shows no symlink?

I'm going to close this as I think it is fixed in the latest release.

Thanks!


Feature Request: Generate Plain Text from Markdown String

Sometimes you need a plain text version of your Markdown syntax like when showing preview text like this:

See how the Markdown characters show up in the item on the left? It would be nice to just have plain text for that.
A workaround would be to create a style-less Markdown rendering to display in a preview, but that seems like overkill. 😅
Thank you!

Here is the solution:
const text = 'This is text, **bold**, and _italic_, ~~delete~~, and more.';

final nodes = Markdown().parse(text);
final plainText = nodes.map((e) => e.textContent).join();

print(plainText);

Thanks for your quick reply!
Interesting... where is Markdown() from? Is that a separate package I need to add to my project?

Ah, sorry, I did not realise that, here it is: https://pub.dev/packages/dart_markdown
It is a dependency of the markdown_reviewer.

Ah, perfect. Thank you very much!


Cardio Recovery must insert an excerside

Description
When creating a cardio / strength workout this extension expects me to fill in an exercise for the recovery phase. This is odd because this phase is used to rest for a few minutes/seconds.
Reproduction steps

Create a new strength workout
Add a recursion ( don't know the correct english translation )
Add a workout step
Open the recovery step and see that the exercise step field is required..

Which platform did you discover this bug?
Firefox Add-ons
Please provide the environment you discovered this bug
Firefox 133.0.3 (64-bit)
Ubuntu 24

Additional information
No response

Unless I am mistaken, this looks to be expected behaviour. In Garmin Connect, the recovery step always involved an exercise (I believe they expect it to be used for post-workout stretches, yoga, etc.).
Changing the step type to "Rest" seems to do what you want (resting for a short period with no exercise).

Hi @Wassup789. I think you are right. When I disable the extension it works the same. My bad.


Download the PackageDeployer zip -  URL is not valid anymore

Download the PackageDeployer zip  - URL is not valid anymore
https://cfsdemodata.blob.core.windows.net/download/PackageDeployer.zip

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

ID: e56ea0df-2e9a-989e-92a8-375ac6b00da4
Version Independent ID: de0efc1d-0813-029f-223b-9a7b4af55b6f
Content: Prerequisites for Connected Field Service for IoT Central
Content Source: ce/field-service/cfs-iot-central-setup.md
Technology: field-service
GitHub Login: @FieldServiceDave
Microsoft Alias: vihuang

Thank you so much for raising this issue - we have added a refresh / update for this doc to our backlog where we will correct this!


Allow users without account create issues and pull requests

Thanks for the hard work.
I'm running a Gogs instance on YunoHost thanks to a Gogs package integration.
Gogs is working well for mono user, but external users needs an accounts to contribute.
My main issue to really use it is that only YunoHost users based on LDAP users can contribute to Gogs repositories.
I can create an account for people who ask me, but it's not enough.
I think we should allow people without account to create issues and pull requests to permit external contributions.
That's the main issue with many Gogs instances (decentralized), you need many accounts not like with GitHub where all repositories and accounts are centralized.
Issues and pull requests forms could ask for their email address to identify contributors.
Issue of that feature is spam, like blogs are countering on comments with captcha to prevent bot spam.

Thanks your feedback!
Gogs needs something (user ID) to track user-issue relations which requires someone must register first.
It sounds great, but I don't think it's going to be implemented.

Ideally, for myself, I would not like users without accounts to be able to file PRs. I would prefer for them to be authenticated using OpenID, so that they can use their own GitHub account or the like to do that.
They do not need to be LDAP users, though.

So, ideally, LDAP users can login directly into Gogs using their LDAP credentials, while "guests", as I call them, can use OpenID to give feedback.

I don't think it is a good idea to allow everyone to add issues and PRs. This could be used to spam your Gogs installation, hence there is a reason why many project only allow editing e.g. their wikis for registered users.

Ping @M5oul ?

ping @M5oul

ping @M5oul

OpenID is a better proposition than my idea.

@M5oul thanks for following up!
So I'm going to do is close this one and let you follow on https://github.com/gogits/gogs/issues/1297


fix: new sequencer endpoints

Wait for signup-sequencer #493 to be completed

Unneeded in new release


Add support to attach filter onto socket on Linux

Hi
This patch add support to attach filter onto socket on Linux.
See also: Linux Socket Filtering aka Berkeley Packet Filter (BPF)

It seems aarch64-linux-android doesn't have libc::SO_ATTACH_FILTER. We should add that to the libc crate.

I have created a pr for libc.

I think this now depends on https://github.com/rust-lang/libc/pull/2506.

@hhyhhy if you can update to  libc v0.2.107 this can be merged 👍

It seems that the problem on the Android system has been solved.

Thanks @hhyhhy


PathPattern description conflicts with pattern examples on URL Path Based Routing overview

Paragraph under the PathPattern heading seems to conflict with the pattern tables under the v1 and v2 headings.  Paragraph states ...

the only place a "*" is allowed is at the end following a "/."

Each table has an example entry of /images* that's marked as supported.

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

ID: f2769aea-56d1-03aa-1949-9818d5ee3ba4
Version Independent ID: 25394ed7-b79d-1d85-03c2-0031b666635e
Content: Azure Application Gateway URL-based content routing overview
Content Source: articles/application-gateway/url-route-overview.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@cookbr Thank you for your feedback! We will review and update as appropriate.

@cookbr, Appreciate your patience. This issue has been assigned to the content author for updating the doc accordingly.

fixed
#please-close


Fix ordering issues between as: and default: validations

Fixes #2094 where it was found that defining as: before default: would fail to add the default whereas as: after default: would work fine.

Thanks for fixing this, merged. It could be an overkill, but maybe we should introduce some kind of Rack-style listing/mounting of validators so that we don't have to do special ordering gymnastics at runtime. Just food for thought.

@dblock
I think it'd make the behaviour more consistent too, perhaps there are other validators that don't play nicely with having :default before or after. I'd like to give it a go after I do more reading\thinking about it.


can't load lazy module with eject

Output from: ng --version.

@angular/cli: 1.0.0 (e)
node: 6.9.5
os: win32 x64
@angular/common: 4.0.0
@angular/compiler: 4.0.0
@angular/core: 4.0.0
@angular/forms: 4.0.0
@angular/http: 4.0.0
@angular/platform-browser: 4.0.0
@angular/platform-browser-dynamic: 4.0.0
@angular/router: 4.0.0
@angular/cli: 1.0.0
@angular/compiler-cli: 4.0.0

when using eject, we can't make lazy module work; access the route,just get something like
 Cannot GET /here_is_the_lazy_module_route
but "npm start" without error;
other routes binding on main module work right.
and "ng server" also works right ,if disable eject  (in .angular-cli.json file  “”ejected:false“”)

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


Parsing error

for query select * from ->contact->profile;
I got error: There was a problem with the database: Parse error on line 1 at character 0 when parsing 'select * from -Ϭontact->profile;'
somehow >c is encoded to Ϭ
I ran the same query from thunder client, and it returned fine without error, so it doesn't look like it's a problem with the server
here is Link to minimal reproduction repo

Seems like #62 and fixed in https://github.com/surrealdb/surrealdb/pull/1888
I think in the next release it will be fixed.


Persian Calendar Issue

[Info]

Tested in Persian Calendar with Asia/Tehran

Test time (Gregorian year 2018, month: 3, day:21) -->
It's converted to Persian Date:year:1397, month:1, day: 1)

DST time  peride offset value: 0.1875

Not DST time offset value: 0.1458333333

Issue:

If Persian date (year: 1397, month: 1: day,1)
It returns, 0.1875 in https://github.com/iLib-js/iLib/blob/development/js/lib/PersianDate.js#L229 line.
It should be 0.145833333
That day not in DST time. But iLib says it's in DST time.
Input value is Persian datetime,
https://github.com/iLib-js/iLib/blob/development/js/lib/TimeZone.js#L811 value is a bit weird.
Input parameter : PersianCal year:1397, , month:1,  day:1, hour 21: minute 13

var  d = new Date(date ? date.getTimeExtended() + offset: undefined);
new Date(date ? date.getTimeExtended() + offset : undefined);
Thu Mar 22 2018 01:43:17 GMT+0430 ()

*** More Test on Linux ***
Set time zone for Asia/Iran following steps:

Check current timezone info ls -al /etc/localtime
Changed timezone: sudo ln -sf /usr/share/zoneinfo/Asia/Tehran /etrdate -s timebora.netc/localtime
Update time :rdate -s timebora.net
Check a time with date command

(화) 05:39:02 +0430 <-- Confirmed It's in Iran Timezone.

The reposted issue is resolved with aP to  https://github.com/iLib-js/iLib/pull/119


Can this able to port in rancher

Hi All,
Can we able to port this in Rancher
https://github.com/ClusterHQ/mesos-module-flocker

Closing due to inactivity, please reopen if you think this is still relevant to current releases & important.   This is not something that Rancher will be able to do but would welcome any contributions in this area.


fix: try to split tests of apim

Description
Try to split the test jobs in three: one for the repository part, one for the gateway-container module, one for all the rest.

📚  View the storybook of this branch here

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

I also created a branch where I kept gateway container tests with other non-repository tests.
You can see here:
https://app.circleci.com/pipelines/github/gravitee-io/gravitee-api-management/6040/workflows/f8349715-9abc-4544-8732-3934a7828f95
The results are quite similar and with this we can have a jacoco report for gateway-container tests.


Fix #34057 - Allow default config exporter to access platform-specific configurations that don't apply on the build machine

Let me know what you think - this isn't enabled for extensions yet.
In core, we need a flexible mechanism because we check the OS version and may want to check something else arbitrarily. Extensions probably would only be allowed to set the OS version declaratively.

@roblourens I like the approach and support for this 👍 . Not sure about calling this "excluded" though, imho this negation makes it much harder to understand. Why not have it the other way around and call it platform or os and then it reads better?

I didn't want to tie it to something specific like platform or os, so we can use it for any arbitrary condition. Right now we check the os version, and there could be other checks in the future. I used 'excluded' so it's naturally not excluded when that property is absent but I agree that leads to lots of double negatives. Another option is 'included' where 'undefined' would be the same as true but it would be possible to mess that up by mistake, javascript...

I changed it to 'included' where if !hasOwnProperty('included') then the property is still included by default.


chore(other): update some dependencies to latest

update some dependencies to latest

I am getting the error,
[ ERROR ]  TypeScript: src/test/e2eTestUtils.ts:21:21
           Property 'violations' does not exist on type 'unknown'.

     L20:    // eslint-disable-next-line
     L21:    expect(axeResults.violations).toHaveNoViolations(axeScanDetails);
     L22:  }

when running the build locally. Is anyone else getting the same error ?

I did not get that when I was building or testing the branch but I could see it in vscode when I looked at the file. I added a fix for it.


Better results for merging config

The ServiceProvider::mergeConfigFrom() method uses array_merge which doesn't have desirable results, this PR attempts to improve this feature for a future release and encourage discussion.
please see this pure php example: https://gist.github.com/ryaan-anthony/3dacdf3a722217eb66f0

Recursive merging is actually not what we want here. This has been discussed many times before.

@ryaan-anthony It actually used to be pretty similar, but it was removed because the package config files are supposed to be defaults, not be merged together (which could obviously be discussed).
For more reference: #7141 and #7438.


Can't build the project

Hi! Unfortunately, when trying to build the project in PlatformIO, it fails with the same response:
FileNotFoundError: [Errno 2] No such file or directory: 'secrets/secrets.json'
It seems like the file is really missing - how can I get or generate one by myself?

Hi, you are right. This file is missing, since it holds the wifi credentials.
I added a file secrets/rename_to_secrets.json. You can rename it and provide your credentials.
Or leave them out. These are optional.


[WIP] Support Ember 3.13.0

This PR adds support for Ember 3.13.0

There is one issue which might need to be resolved before this is merged:

[ ] https://github.com/offirgolan/ember-cp-validations/issues/642 - I'm currently skipping this test. @snewcomer, perhaps you might have context on the need for this test

#642 - I'm currently skipping this test.

The test is related to this part of the code base:
https://github.com/offirgolan/ember-cp-validations/blob/2e901724390172d3b9584aec69696e2f8e28fc29/addon/validations/factory.js#L112-L117
And is pretty much testing that if there is no super class with a validations mixin, it shouldnt call this._super at all. This was related to a bug that was found some time ago that @rwjblue helped solve. Pretty much, if you instantiated the validations object within an ember action, it will call this._super on that action which would invoke that action's parent method.

We can skip the test for now and come back for this later if it means we can merge this and unblock everyone else.

@GavinJoyce is this PR still WIP?

This is our blocker for trying out the Octane preview. Anything I can do to help move this along?

Rebased in #668


Anchor hrefs hard coded to #

the following lines of code should you javascript:void(0); instead of pound. I'm using this in SharePoint Online 'Modern Pages' and the '#' is triggering navigation and it's breaking the calendar. I manually changed it to javascript:void(0); and it works.
EventEndingRow.js line 132
Month.js line 378
Timegrid.js line 332

This should either be a button, or the href in the anchor should be "javascript:void(0);" having an anchor element with the href set to "#" can cause postbacks in certain environments.


Async support

Fixes #10

@acautin and @shamis
I don't understand why library code like erlimem or imem_server must be aware of app level API like request_metric etc.

@c-bik for metrics we need to send the reply only when it is a metric result. btw imem_server is not involved in this...

@acautin
Try and alternate approach. From the calling point:
...
ErlimemSession = erlimem:open(...),
% synchronous call return value of imem_server apply call
ok = ErlimemSession:run_cmd(..., request_metric, [self(), ...]),
% can be a gen_server:handle_info too
receive
  {metric, Metric} -> ...
end
...

This eliminates the need for a separate async API in erlimem

@c-bik we didn't find the way to implement it like that, maybe you could help us by creating the pr ?

@c-bik also the caller should not be stopped because the connection to the other side is slow (this should be an async call after all), we would need to give a very short timeout that overrides the long one included by default.

Closing this as an alternate approach was agreed on


feat(rpc): add replay transaction

builds off of #2106
adds replay_transaction and unifies via new trace_transaction function

Codecov Report

Merging #2107 (36bbcfe) into main (b7395a8) will decrease coverage by 55.06%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2107       +/-   ##
===========================================
- Coverage   71.98%   16.92%   -55.06%
===========================================
  Files         436      436
  Lines       53359    53414       +55
===========================================
- Hits        38410     9040    -29370
- Misses      14949    44374    +29425

Flag
Coverage Δ

integration-tests
16.92% <0.00%> (-2.38%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/rpc/rpc/src/eth/api/transactions.rs
15.80% <0.00%> (-7.32%)
:arrow_down:

crates/rpc/rpc/src/trace.rs
0.00% <0.00%> (-39.74%)
:arrow_down:

... and 309 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


UCOB

is there gonna be ucob map? cause its really needed XD

Yes please.
UCOB module would be amazing.

As of v140, all ultimates have fully functional modules.


[$500] IOU - Inconsistency in Scan split bill IOU preview for creator&Participant

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/
Tap fab--Request Money
Tap Scan
Upload a picture by taking photo
Split with any 2 users
Tap add to Split
Tap split
Tap scan request created
Enter amount, description and merchant field
Tap split
Note description entered is shown in IOU preview
Open a participant account
Navigate to the specific IOU above created
Note merchant entered is shown in IOU preview

Expected Result:
There should not be inconsistency in IOU preview displayed for scan request creator and participant.
Actual Result:
For user who made scan request and split bill with 2 users, description details entered is shown in IOU preview. But for participant of split bill, merchant details entered is shown in IOU preview. There is inconsistency in IOU preview displayed for scan request creator and participant.
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/Expensify/App/assets/93399543/35ef0875-ce00-4825-b59f-1140d69257a0

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01890520fc54e620e1
Upwork Job ID:<PHONE_NUMBER>480401920
Last Price Increase: 2023-11-01

@cubuspl42 any thoughts on the above proposal?

@dukenv0307 I will not review any of your proposals as long as the "Please re-state the problem that we are trying to solve in this issue" section is a copy-pasted issue name.

Unable to reproduce issue.

@kbecciv
I re-watched the provided video multiple times and still cannot understand the issue's essence. Maybe it has something to do with the second attachment not loading correctly.
@dukenv0307 While your new updated issue re-statement is now acceptable in the formal sense, it didn't help me understand the actual problem here.

merchant

The merchant field does not appear when we follow the Action Performed steps

@kbecciv Would you re-record the videos?
Two reasons:

One of them is not available
It's suggested that there is an issue with reproducibility

@cubuspl42 I apologize for any inconvenience. I've reviewed the issue again, corrected the reproduction steps, and added a new video and screenshot. Please let me know if the reported issue is still difficult to understand, and I'll be happy to assist further.

@kbecciv Thank you! <3

The proposal by @dukenv0307 makes sense. I personally would prefer the "alternative solution", i.e. preferring the description over the merchant. In my opinion, the description is the most personal element of the form filled by the user and is the one that deserves to be presented with the highest priority.
C+ reviewed 🎀 👀 🎀

@nkuoch Would you share your opinion about which field should be consistently presented in the Split/IOU preview: should it be the "merchant" or "description"?

friendly bump @vitHoracek

heads up @nkuoch that Vit's handle is @mountiny not @vitHoracek - shared a link to this issue in e chat to him

I personally think this should be consistent so Merchant whenever available and fallback to description anytime Merchant is not defined. however with Scanned receipt, there will be merchant most of the time.

I personally think this should be consistent so Merchant whenever available and fallback to description anytime Merchant is not defined

Yes, that's the logic for the preview. Merchant. If no merchant, then description. If no merchant or description, then blank. 👍

@cubuspl42 - with the above in mind, do you have a preference amongst the proposals?

@cubuspl42 - with the above in mind, do you have a preference amongst the proposals?

There is one proposal and I approved it.
@nkuoch @mountiny
Thanks for the clarification of the requirements! Are we in agreement that this is the expected order...

Merchant. If no merchant, then description. If no merchant or description, then blank.

...and the code should be adjusted so it's consistent for all previews?

Yes!

@nkuoch In such a case, I think we're ready to assign @dukenv0307 and start working on a PR 🙂

Both contracts paid and ended. Thank you for your work!

@miljakljajic Did we reduce the pay? Since it caused a deploy blocker and PR had to be reverted.

Ah, I had no idea we needed to reduce the pay. I'll request a partial refund from @dukenv0307 and @cubuspl42

@miljakljajic Is it possible on Upwork?


updating to return the share_URL

adding the ability to return the URL from a query of the resource API
I'm wondering if I should be using this package or, better to hit the API directly

fixed my syntax.


Implement a basic (but real) NGS workflow: align & call SNPs

Starting with a WGS fastq pair in object storage:

split into chunks
align each fastq chunk with BWA
merge chunk BAMs
construct BED dividing genome into chunks
call snps on each genome chunk with GATK
concatenate VCFs
🍸

blocked on #48


resizimg IE 11 fix

An amendment for Unspecified error in ie11 console. For resizeimg plugin.

As I cannot reproduce the bug, I do not understand the need behind this fix, so I close this.


The buildkit service failed to start

Cluster information:
root@k3s-m1:~/gitlab/gitlab-runner# kubectl get node -o wide
NAME     STATUS   ROLES                  AGE     VERSION        INTERNAL-IP     EXTERNAL-IP   OS-IMAGE                       KERNEL-VERSION   CONTAINER-RUNTIME
k3s-m2   Ready    control-plane,master   3d2h    v1.22.5+k3s1   <IP_ADDRESS>   <none>        Debian GNU/Linux 9 (stretch)   4.9.0-9-amd64    containerd://1.5.8-k3s1
k3s-n2   Ready    worker                 4d12h   v1.22.5+k3s1   <IP_ADDRESS>   <none>        Debian GNU/Linux 9 (stretch)   4.9.0-9-amd64    containerd://1.5.8-k3s1
k3s-n1   Ready    worker                 4d12h   v1.22.5+k3s1   <IP_ADDRESS>   <none>        Debian GNU/Linux 9 (stretch)   4.9.0-9-amd64    containerd://1.5.8-k3s1
k3s-m1   Ready    control-plane,master   4d12h   v1.22.5+k3s1   <IP_ADDRESS>   <none>        Debian GNU/Linux 9 (stretch)   4.9.0-9-amd64    containerd://1.5.8-k3s1

Network mode：(cilium)
root@k3s-m1:~/gitlab/gitlab-runner# kubectl get pods -n kube-system
NAME                                     READY   STATUS    RESTARTS      AGE
cilium-operator-776958f5bb-mlsl6         1/1     Running   1 (10h ago)   11h
local-path-provisioner-64ffb68fd-n8vgc   1/1     Running   2 (10h ago)   4d12h
coredns-85cb69466-brj6p                  1/1     Running   2 (10h ago)   4d12h
cilium-2t59g                             1/1     Running   2 (10h ago)   11h
cilium-s6zws                             1/1     Running   1 (10h ago)   11h
cilium-vl5kz                             1/1     Running   1 (10h ago)   11h
metrics-server-9cf544f65-2bzp7           1/1     Running   2 (10h ago)   4d12h
cilium-gxccm                             1/1     Running   1 (10h ago)   11h

install buildkit service (instead of docker)
vi /etc/systemd/system/buildkit.service
[Unit]
Description=BuildKit
Documentation=https://github.com/moby/buildkit

[Service]
ExecStart=/usr/local/bin/buildkitd --oci-worker=false --containerd-worker=true --containerd-worker-addr=/run/k3s/containerd/containerd.sock

[Install]
WantedBy=multi-user.target

Start service error：
root@k3s-m2:~# systemctl status buildkit.service
● buildkit.service - BuildKit
   Loaded: loaded (/etc/systemd/system/buildkit.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Mon 2022-01-17 00:06:26 CST; 4min 45s ago
     Docs: https://github.com/moby/buildkit
  Process: 26655 ExecStart=/usr/local/bin/buildkitd --oci-worker=false --containerd-worker=true --containerd-worker-addr=/run/k3s/containerd/containerd.sock (code=exited, status=1/FAILURE)
 Main PID: 26655 (code=exited, status=1/FAILURE)
      CPU: 26ms

Jan 17 00:06:26 k3s-m2 systemd[1]: Started BuildKit.
Jan 17 00:06:26 k3s-m2 buildkitd[26655]: time="2022-01-17T00:06:26+08:00" level=warning msg="using host network as the default"
Jan 17 00:06:26 k3s-m2 buildkitd[26655]: time="2022-01-17T00:06:26+08:00" level=info msg="found worker \"uom9in3zqo7fbwjvmfb6nx7pe\", labels=map[org.mobyproject.buildkit.worker.executor:containerd o
Jan 17 00:06:26 k3s-m2 buildkitd[26655]: time="2022-01-17T00:06:26+08:00" level=info msg="found 1 workers, default=\"uom9in3zqo7fbwjvmfb6nx7pe\""
Jan 17 00:06:26 k3s-m2 buildkitd[26655]: time="2022-01-17T00:06:26+08:00" level=warning msg="currently, only the default worker can be used."
Jan 17 00:06:26 k3s-m2 buildkitd[26655]: buildkitd: is a directory
Jan 17 00:06:26 k3s-m2 systemd[1]: buildkit.service: Main process exited, code=exited, status=1/FAILURE
Jan 17 00:06:26 k3s-m2 systemd[1]: buildkit.service: Unit entered failed state.
Jan 17 00:06:26 k3s-m2 systemd[1]: buildkit.service: Failed with result 'exit-code'.

Hi @mpv945 , thanks for opening this issue!
You seem to be running plain K3s there, not k3d, right?
In that case you may rather want to open this issue on https://github.com/k3s-io/k3s , though it looks like an issue with the buildkitd setup/configuration.

I want to know what caused it to have to clean up the data before it can start properly,Or maybe I should use some methods to avoid this situation


[Spark-9337] [MLlib] Add an ut for Word2Vec to verify the empty vocabulary check

jira: https://issues.apache.org/jira/browse/SPARK-9337
Word2Vec should throw exception when vocabulary is empty

Merged build triggered.

Merged build started.

Test build #38422 has started for   PR 7660 at commit 17a18cb.

OK, ideally this would have been part of SPARK-7663, or could even have been retroactively attached. It logically goes with the change this is testing, and is trivial by itself.

Test build #38422 has finished for   PR 7660 at commit 17a18cb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

I was working on the ut for CountVectorizer and thought Word2Vec should have a similar one.


Improve Calibration Procedure - User Feedback

Right now, the calibration procedure totally opaque with essentially no user feedback. It would be great if the calibration procedure for the sensor incorporated flashing LEDs to alert the user to the current state of the calibration process. There should be some sort of way to signal the following states:

Calibration in progress, insert pin
Reinsert pin/retry
Calibration stage success, insert next pin
Error

Each of the "flash codes" for these states should be documented and the calibration procedure revised.

How about adding an I2C display?

Nothing wrong with this idea, but it could potentially raise the BOM cost significantly. At the very least, it will require changes to the PCB design and layout. Since calibration shouldn't be happening very often, it might be fine to stick with flash codes for now, since there are only 4 states.

It looks like in #8 there was an I2C interface implemented, though I'm not sure how it works and haven't tested it yet. Perhaps there is some potential to do this calibration by connecting to a remote serial console and doing the calibration interactively through that.

#8 does set up a serial user interface on the Arduino host, with a 2-way communication to the ATTiny85.


Implement BrakePositionSensor

Create a BrakePositionSensor class and use it in place of the existing brkpos code. This change got me thinking about some of the design decisions we've made, and it includes a few changes which I think are warranted:

move Potentiometer to uictrl.h. The pot is really mmore of a user-input than a sensor, and even though it worked as a child of Device it doesn't really need to be (it doesn't take a source, for example). It's cleaner to have it be separate, and the code for it is actually simpler.

make the existing linear conversion code the default for Trasnducer (as we had it originally). It's easy to override it if needed, and this way we don't have to copy it everywhere. There's probably a better way to do this but I think we'll need to convert some more components before we worry about it more.

move the controller source handling logic to Device, in a new function called update(). Different modes are handled with handle_MODE() calls which do nothing by default but can be overridden in child classes as needed. This change makes sense, since Device is already where we're handling source stuff.

move the ema_filt() algo (and some other util functions) to utils.h. The new Potentiometer implementation needs to call ema_filt() (since it's not using the one built into Sensor). I rewrote ema_filt() to have two versions, one that takes raw values and another that takes a pointer to a value to be updated. I temmplated the second one, so you can use it with any numeric type and it should work (the template is based off the args, so you don't even need to define any templates when you call it).

move Simulator code to bottom of file. Really, it should probably go in a different file if Devices.h gets much bigger, but for now it's enough just to be out of the way.

Addressed all Bobby's comments :)


Improve logging

The login should become more structured now that the game creator and the crawler have been merged.
We should also use SLF4J as the scribed here:
http://doc.akka.io/docs/akka/2.0/scala/logging.html

Some changes were made to the logging logic (some messages were switched from info to debug), soem improvement can still be done.