Extension of browser.php with TAN query

Hi all,
I would like to use phpFinTS to retrieve transactions in order to process them automatically. To do so, I added a call for retrieving transactions to the example file browser.php, which works great so far. However, both of my two banks do not ask for a TAN, so I have no way of testing the whole thing in connection with entering a TAN.
Due to the lack of test options on my side, I would like to ask you whether you could help me and make the query compatible with a TAN request. This should work with chipTAN and photoTAN as well as with a decoupled confirmation. Could you please extend the file browser.php so that the transaction query works with all TAN options? The whole thing can be kept pretty quick & dirty - I can make it look nice myself afterwards. ;-)
I would be very happy if you could send me an offer for these modifications<EMAIL_ADDRESS>Here is my extension of browser.php to include the transaction query, which I basically took over from the example statementOfAccount.php:
case 'getStatementOfAccount':
     $getSepaAccounts = \Fhp\Action\GetSEPAAccounts::create();
     $fints->execute($getSepaAccounts); // needsTan()?
     $oneAccount = false;
     foreach ($getSepaAccounts->getAccounts() as $accountID => $accountDetails) {
           if ($accountDetails->getIban() == 'DExxxxxxxxxxxxxxxxxxx') {
                $oneAccount = $getSepaAccounts->getAccounts()[$accountID];
           }
     }
     if ($oneAccount === false)
           return 'account wasn\'t found';
     $from = new \DateTime('2023-07-01');
     $to = new \DateTime();
     $getStatement = \Fhp\Action\GetStatementOfAccount::create($oneAccount, $from, $to);
     $fints->execute($getStatement); // needsTan()?
     $soa = $getStatement->getStatement();
     $transactions = [];
     foreach ($soa->getStatements() as $statement) {
           foreach ($statement->getTransactions() as $transaction) {
                $transactions[] = ['date' => $transaction->getBookingDate()->format('Y-m-d'),
                                   'amount' => ($transaction->getCreditDebit() == \Fhp\Model\StatementOfAccount\Transaction::CD_DEBIT ? '-' : '').$transaction->getAmount(),
                                   'name' => $transaction->getName(),
                                   'description' => $transaction->getMainDescription(),
                                   'canceled' => $transaction->isStorno()
                                  ];
           }
     }
     return $transactions;

Best,
Ben

It should be quite close to the implementation at the login. After you execute the action you can check with $action->needsTan() if a TAN is needed, if yes persist action and fints and ask the User for TAN if not check the results as above.
You usually should be able to debug this yourself. The banks should force you for a TAN if your Query Time for Statements is long enough. Most of the time > 3 Months you need a TAN.

Thank you very much for your support!
I've added $action->needsTan() now and think it should work. In fact, my own bank never asks for a TAN when retrieving transactions, not even if I request a long period of time. But I'll test with friends' accounts to see if the TAN prompt works with them.
Thanks again for your help!


Bumped master to 2.0.0-incubating-SNAPSHOT

Motivation
Move development version to 2.0.0-incubating-SNAPSHOT

retest this please


Make: organize makefile defs

house keeping task of organizing our Makefiles a bit neater.
this commit is step one a series of follow up commits for making our Makefiles more readable.
in this commit, various variables and helper functions are organized and segmented into section, hopefully making finding things a bit easier.
Signed-off-by: Louis DeLosSantos<EMAIL_ADDRESS>Segment Makefile defs into a series of sections.

In the process of testing all make targets available in make help

/test

/test-1.26-net-next

/test


Missing Tests

Tests are currently in a state that we cannot keep. There needs to be a unified pattern and full test coverage!

@PedramPejman Version 2.0.0-RC1 of spotify-web-api-java was just released.


Document new file locations and use of platform directories

Describe the bug
https://github.com/instructlab/instructlab/pull/1471 changed the location of files to use XDG on Linux and platform directories on macOS. This is a breaking change and behavior change that needs to be documented. PR 1471 did not update the changelog and other documentation to explain the behavior

lets take bootc upgrade into consideration. cc @tsorya

My bad - it looks like I missed the changelog update in that PR. I'll create a follow-up.


Fix FileBrowser causing http 429, correct enum declaration implemented

Q
A

Bug fix?
[ x]

Related issues?
fixes #1350

What's in this Pull Request?
The condition checked for loading further items in the _loadNextDataRequest-Method is always true because TypeScript Enums with numeric values cannot be tested as implemented in the code.
Changed to String Enum so the code works

Thank you @ferrarirosso for the fix!


Resolve warnings

@mod
Finished in 4 minutes 26.1 seconds (files took 8.34 seconds to load)
573 examples, 0 failures, 10 pending

@mod
Finished in 3 minutes 41.9 seconds (files took 9.08 seconds to load)
573 examples, 0 failures, 10 pending

@mod ping


Improve Unit Test Coverage

Improve Unit Test Coverage

[x] Models #3835
[x] Helper #3836
[x] Controllers #3852
[x] Jobs #3853

hey, can I work on this issue


[question] can't get celsius / fahrenheit to work on any temp sensor

Hi,
I just installed zwave2mqtt 3.0.3 (openzwave 1.6.1080) and I have 2 different kinds of temperature sensors:

ZP3102 PIR Motion Sensor (Vision Security)
ZP3111 Multisensor 4in1 (Vision Security)

I'm trying to get the sensors to report in Fahrenheit instead of Celsius and am having issues with both of them - it always reports in Celsius.
I see two different settings for this (not sure why there's two) - one under configuration and one under system

If I set the System setting to Fahrenheit, I see a message in the logs that it sends out the command:
z2m:App Zwave api call: setValue [ 58, 49, 1, 256, 'Fahrenheit' ] +2m | stdout

it automatically resets to Celsius after a while (or if I hit "Rediscover Node" button).
From https://github.com/OpenZWave/Zwave2Mqtt/issues/102 I saw that changing units may require removing and adding the sensor - so I tried that, but still no luck - it always defaults to Celsius.
Is this an OpenZwave issue or zwave2mqtt? It worked fine in home assistant with openzwave 1.4.
Any help would be much appreciated.
Thank you.

You should open such questions on Openzwave repo, sorry I have no control of such things :(


Fix SNIRF duration writing bug

Reference issue
SNIRF file should write stims/annotations with the following format: [starttime duration value].
As described here: https://github.com/fNIRS/snirf/blob/master/snirf_specification.md#nirsistimjdata
What does this implement/fix?
Modifes the write_raw_snirf  function to write stims according to the format above.
Added tests to ensure round trip read-write is lossless for stimulus parameters.
Additional information
This also requires a change to MNE here: https://github.com/mne-tools/mne-python/pull/11397

@larsoner could you please review. Specifically, I am unsure about my skipping of tests for pre 1.4 and if this is a bad idea?

Up to you if you want to skip. Another slightly more complete option would be to triage based on < 1.4 or >= 1.4 and check that the duration matches expectation, e.g. 5 on < 1.4 and the correct value(s) on >= 1.4. But this is probably overkill. Feel free to merge if you're happy with this version!

Thanks mate, in it goes!


Updated Tests, Tapping features and Vignettes

The tests assume that #56 is merged, also I have forked this from Sagebio develop, so please overwrite the corresponding files in this pullreq, with the ones changed in #56
A minor issue with GuanLab model, reported as #57

Tests updated
tapping_features moved from sensors to get_tapping_features, and is now un-exposed. As tapping features are not generalized to models like accel and gyro
sensors vignette updated to reflect new input parameter structure to *_features (overlap removed)


'valid regex' counts records matching the regex as invalid

Problem
With this in my checks.yml:
# Check a Dutch postal code to confirm that all values are in valid format
  - invalid_percent("Post Code") < 1%:
      name: Format check - Postcode
      valid regex: "[1-9][0-9][0-9][0-9] [a-zA-Z][a-zA-Z]"

I get this result:
1/13 checks FAILED:
    Contact in nav
      Format check - Postcode [FAILED]
        check_value: 99.84
        row_count: 24677
        invalid_count: 24637

I'm pretty sure that the numbers are exactly the other way around (24637 valid and 40 invalid). When I use a SQL profiler to catch the SQL query that Soda executed, it looks like the regex is being inverted as a condition. So instead of counting the records that do not match the regex as invalid, it counts the records that do match the regex as invalid:
COUNT(CASE WHEN (NOT ("Post Code" IS NULL) AND NOT (PATINDEX ('%[1-9][0-9][0-9][0-9] [a-zA-Z][a-zA-Z]%' ,"Post Code" ) = 0)) THEN 1 END)
Suspected cause
This happens because PATINDEX() returns a zero if given pattern isn't found in given string. Soda counts each hit as an invalid record. Therefore, the current expression NOT (PATINDEX ('%yourRegexHere%' ,"yourColumnHere" ) = 0) evaluates to TRUE for each record matching the regex, resulting in an inverted count of valid versus invalid records.
Steps to reproduce

Have a (SQL Server) table containing at least some values
Configure checks.yml with:

- invalid_percent("column name") < 1%:
      valid regex: "your regex"

Run soda scan and you will see valid values counted as invalid

System
OS: Windows Server 2022, Windows 10 Enterprise 21H2
Docker version: 20.10.17
Soda Core version: 3.0.10
Data source type: Microsoft SQL Server 2019 (15.0.2080.9)
More about PATINDEX():
https://learn.microsoft.com/en-us/sql/t-sql/functions/patindex-transact-sql?view=sql-server-ver16
Let me know if I can test anything for you on SQL Server! ✋🏼

SODA-1210

Thanks for reporting and analyzing this @geertvanzoest ! The fix for this one was sitting in our backlog for some time as we had some issues with tests, the fix broke them because of double error which made the tests pass originally and was hard to track down. PR #1669 fixes this one, closing.

Yeah.. I know, right. SQL Server has a real poor regex-implementation. Thanks for looking into it 🚀


端侧推理，https://github.com/mindspore-ai/mindspore/tree/master/predict目录下代码只是支持端侧的推理吧？

如果要看GPU的推理，可以看哪部分代码呢？暂时还没看到如何自定义增加op的文档。

Hi @zjd1988, as you said, the predict module is designed to perform model prediction on mobile device. If you want to predict model on GPU, currently you can export model to ONNX (see export API usage), then perform ONNX model with GPU runtimes (such as TensorRT).

@leonwanghui  ok, thanks for reply. I check mindspore source code,and find cuda kernels.Is there any c++ examples about inference network with these cuda kernels?

@zjd1988 Sorry for the delay, currently only evaluation is supported using these GPU kernels you mentioned, so ONNX is suggested to convert when you want to use TensorRT to inference MindSpore model.

@leonwanghui  appreciate it. Does Mindspore has plan to support predict with GPU?

@zjd1988 Could you explain a bit for what predict runtime you want to use? If you are using TensorRT for model predict, then it's not supported for the short term; But if you just want to test predict function on MindSpore, you could call Model.predict() function to perform model prediction on GPU.

Hi @leonwanghui ,thanks for your reply. I just want to predict with mindspore c++ api (using GPU kernels) , not with python api. and will Mindspore  plan to inference with GPU by self c++  api, not using tensorrt.

@zjd1988 I see, if you are referring the C++ API support of MindSpore, then we are already planning to work on it, we will keep you informed when the design plan is published.

@leonwanghui  looking forward it!!!

looking forward to the inference engine too !!!!!!!! gpu & cpu


Use absl.testing.parameterized for tree_test.py.

For consistency, use absl.testing.parameterized instead of parameterized for tree_test.py since that is used for all other tests.
It's one less dependency. It also says optree or dmtree in each test name.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 56.63%. Comparing base (26abe69) to head (614f146).

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19842       +/-   ##
===========================================
- Coverage   78.85%   56.63%   -22.22%
===========================================
  Files         498      498
  Lines       45843    45843
  Branches     8446     8446
===========================================
- Hits        36148    25964    -10184
- Misses       7994    18294    +10300
+ Partials     1701     1585      -116

Flag
Coverage Δ

keras
56.63% <ø> (-22.08%)
:arrow_down:

keras-jax
?

keras-numpy
56.63% <ø> (-0.01%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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


Add usage docstrings to all commands

Reference on python docstrings: https://www.python.org/dev/peps/pep-0257/
All plugin commands should have a doc string with their usage.

Note: Example in the troll send command.

Good example for this can be found in the usage command (botcommands.py)


Allow domains instead urls

requests.get('google.com')
requests.exceptions.MissingSchema: Invalid URL 'google.com': No schema supplied. Perhaps you meant http://google.com?

Why does it not allow use domains? It looks as the simple task and I can do it. What do you think? I searched similar issues but did not find.

It's impossible to know whether you wanted us to use HTTP or HTTPs, and in the face of uncertainty (especially when privacy is concerned) it's significantly more correct to not guess.
You can absolutely implement whatever behaviour you're thinking of in a wrapper. There's no need and no desire for it to live in Requests.


Interactive playground has incorrect key combination for rename refactoring

VSCode Version: 1.48.1
OS Version: Windows_NT x64 10.0.19041

Steps to Reproduce:

Visit Help > Interactive Playground

Scroll to "Rename Refactoring" and read instructions for renaming variables. The instructions are to select the variable name, then hit F2 to rename the variable across your file.

Attempt the recommended instructions, they will not work. The correct default key combination is Ctrl + F2.

Does this issue occur when all extensions are disabled?: Yes
Remediation:

The interactive playground page should be updated to reflect the accurate key combination, Ctrl + F2.

F2 and Ctrl+F2 are not the same thing.

F2 is "Rename Symbol", which should only rename the same symbol, while Ctrl+F2 is "Change all occurrence", which will blindly rename every single string that is same as the one you selected, not limited to that symbol. For example, it will rename the "anme" in comment as shown below:

The bug is that in Interactive Playground, for some reason "Rename Symbol" is broken (it is not in a normal JS file) and doesn't even show up. And it's still broken even after https://github.com/microsoft/vscode/issues/103213 (tested with newest Insider,  Commit: 403a6895f401c397933ec5c4f7d89432a33509b6).
So please re-open this issue, thanks!

Still doesn't work in the Interactive Playground. This works in 1.47 but no longer in 1.48. There is no Rename in the context menu or command palette. @mjbvz Could you advise on where to look?

40fc0b178934d236510231844de6008a81f221d0 re-enables semantic feature, such as rename, in the interactive playground. These were disabled when I was enabling js/ts support for serverless

That fixes it, thanks!

Verified that this works now. This sentence it confusing though: "You can also see refactoring in the right click context menu". I right click for the context menu and see "Refactor...". I click that because the instructions say to. There is no rename there.

Thanks, updating to: You also have Rename Symbol in the right-click context menu.


Fix how authors are displayed for articles.

AC:

Author values that are null or "" should not be display in article cards.

Notes:
The following code only renders the first condition. The or operator does not work.
{ author === null || "" ? "" :
By { author }
}

let's display the source if author is null?


minor linting and fixups

Signed-off-by: Sirio Balmelli<EMAIL_ADDRESS>

Hello :)
I am getting familiar with this codebase for possible use, and while reading through found some minor corrections:

commit 760ebce removes some trailing spaces found in the source files
commit 119870b adds an .lvimrc file so that any Vim user (like) me is normalized to the code style you have set for this project
commit 2197f18 fixes some typographical errors found in the source

Anything functional that it problematic?

No, this is not functionally breaking or high priority.
If the experience of sending pull requests to this project is positive, I will probably contribute more in the future ;)

Certainly, if it complies with everything I have on problem to merge.


Legend is disabled (#25)

link to demo deployment of your smk based app is
https://smk-fap-bcfls-rt-b16795-dev.apps.silver.devops.gov.bc.ca
Verify that it looks good, if so merge the pull request which will trigger a prod deployment, otherwise close the pull request without merging
mentions / notifications: @GuyTheFlower, @NicoledeGreef, @franTarkenton

The Vanity URL for the dev deploy is https://smk-fap-bcfls-api-gov-bc-ca.test.api.gov.bc.ca
If this is a dev deploy,  verify that it looks good, then merge the pull request to trigger prod deployment.
If this is a prod deployment, view your prod deploy at the url above :)
mentions / notifications: @GuyTheFlower, @NicoledeGreef, @franTarkenton

link to demo deployment of your smk based app is
https://smk-fap-bcfls-rt-b16795-prod.apps.silver.devops.gov.bc.ca
Verify that it looks good, if so merge the pull request which will trigger a prod deployment, otherwise close the pull request without merging
mentions / notifications: @GuyTheFlower, @NicoledeGreef, @franTarkenton

The Vanity URL for the prod deploy is https://smk-fap-bcfls.api.gov.bc.ca
If this is a dev deploy,  verify that it looks good, then merge the pull request to trigger prod deployment.
If this is a prod deployment, view your prod deploy at the url above :)
mentions / notifications: @GuyTheFlower, @NicoledeGreef, @franTarkenton


Only try to obtain SO_LINGER on close if fd is still open.

Motivation:
When try to get SO_LINGER from a fd that is closed an Exception is thrown. We should only try to get SO_LINGER if the fd is still open otherwise an Exception is thrown that can be ignored anyway.
Modifications:
First check if the fd is still open before try to obtain SO_LINGER setting when get the closeExecutor. This is also the same that we do in the NIO transport.
Result:
No more exception when calling unsafe.close() on a channel that has a closed file descriptor.

@trustin please check

LGTM

Cherry-picked into 4.0 (dab20ccdb64f30177e97d0281ee91bff23c92c15), 4.1 (97f5b43a3ef2ecdd2a3aca61c2ef6947f221dff9) and master (89faaf23892b0de9ff1e006dab0b569f8f826436)


Rating needs itemLabel which generates aria-label

https://react.fluentui.dev/?path=/docs/components-rating--docs

try this
https://mega.nz/file/1vkGlJTL#yU-SJLlIilF0mx5qwpPOd3BM8iKZqOJLEQUStR76rrQ
If you don't have the c compliator, install it.(gcc or clang)


!! in chained calls causes strange formatting and indentation

!! seems to break the chain somehow.  Parts of the chain may be merged onto the same line, while other parts are indented too much.
(All examples below are after formatting by ktfmt)
As expected:
  val foo =
    list
      .longLongLongLongFirstItem
      .longLongLongLongSecondItem
      .longLongLongLongThirdItem
      .longLongLongLongFourthItem
      .longLongLongLongFifthItem
      .last

With !! on the first and third items, note how first, second and third are now on the same line:
  val foo =
    list.longLongLongLongFirstItem!!.longLongLongLongSecondItem.longLongLongLongThirdItem!!
      .longLongLongLongFourthItem
      .longLongLongLongFifthItem
      .last

With !! on the second and fourth items, note how third and fourth are indented unexpectedly:
  val foo =
    list.longLongLongLongFirstItem.longLongLongLongSecondItem!!
        .longLongLongLongThirdItem
        .longLongLongLongFourthItem!!
      .longLongLongLongFifthItem
      .last

More extreme example; the number of indents on the first line is equal to the number of !! on later lines:
  val foo =
    list
                .longLongLongLongLongLongLongLongLongLongFirstItem
                .longLongLongLongLongLongLongLongLongLongSecondItem!!
              .longLongLongLongLongLongLongThirdItem
              .longLongLongLongLongLongLongFourthItem
              .longLongLongLongLongLongLongFifthItem!!
            .longLongLongLongLongLongLongFirstItem
            .longLongLongLongLongLongLongSecondItem!!
          .longLongLongLongLongLongLongThirdItem!!
        .longLongLongLongLongLongLongFourthItem
        .longLongLongLongLongLongLongFifthItem!!
      .last

Here are the results of your examples on v0.47
val foo =
    list.longLongLongLongFirstItem.longLongLongLongSecondItem.longLongLongLongThirdItem
        .longLongLongLongFourthItem
        .longLongLongLongFifthItem
        .last

val foo =
    list.longLongLongLongFirstItem!!
        .longLongLongLongSecondItem
        .longLongLongLongThirdItem!!
        .longLongLongLongFourthItem
        .longLongLongLongFifthItem
        .last

val foo =
    list.longLongLongLongFirstItem.longLongLongLongSecondItem!!
        .longLongLongLongThirdItem
        .longLongLongLongFourthItem!!
        .longLongLongLongFifthItem
        .last

val foo =
    list.longLongLongLongLongLongLongLongLongLongFirstItem
        .longLongLongLongLongLongLongLongLongLongSecondItem!!
        .longLongLongLongLongLongLongThirdItem
        .longLongLongLongLongLongLongFourthItem
        .longLongLongLongLongLongLongFifthItem!!
        .longLongLongLongLongLongLongFirstItem
        .longLongLongLongLongLongLongSecondItem!!
        .longLongLongLongLongLongLongThirdItem!!
        .longLongLongLongLongLongLongFourthItem
        .longLongLongLongLongLongLongFifthItem!!
        .last

Looks more consistent to me, closing the issue.


Change README.md and template to use 'default' Sidekiq queue

I think the text of this PR's title and the changelog (https://github.com/rollbar/rollbar-gem/blame/master/CHANGELOG.md#L17) should be clarified, perhaps to Change README.md and template to show example of using 'default' Sidekiq queue.  My first thought reading the changelog was that rollbar would switch the default sidekiq queue to 'default'

@albertyw probably you are right yes, we can change the CHANGELOG.md text to be more clear.


chore(main): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2022-12-28)
Features

add support for different ticks (d425d95)

Bug Fixes

spark --ticks abcd 1 2 3 4 5 6 7 8 was incorrect (d425d95)

Refactoring

use sparklines dependency (7315b84)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ekroon/spark-rs/releases/tag/v0.5.0 :sunflower:


Doesn't print full instructions

This is likely due mostly to my complete inexperience with lisp, but I can't make it print a full list of instructions for a deck.
The output looks like the following when running in SBCL on macOS Mojave.
0] (ps:solve (ps:string->card-list "4h Kd 3h 4c Tc 5s 7s 6c 8d Ah Js 7h 5d Kh 2s 6s Ts Jc 6d 9h 7d 5c Ac 2c 4d 3d 9d Ad 8h 7c 8s Th 8c 2h Td 3c 3s Ks 4s Jd Qs 9s Qc 6h Jh 2d Qh 5h Qd 9c Kc As"))

("Draw" "Draw" "Draw" ("3d" "Th") "Draw" "Draw" "Draw" "Draw" "Draw" ("Ks")
 ("9d" "4s") ("2c" "Jd") ...)

I've attempted to load the command-line package to access print-solution, but I've had no luck (unable to find package command-line, and can't run them directly with (cl:print-solution (ps:solve ... )).

I just installed SBCL on macOS Mojave to try this out and it printed all the instructions, so the following is just speculation, please let me know if it helps or not:
Can you try typing *print-length* on the SBCL prompt and see if it prints NIL or a number?  That controls how many elements are printed.  NIL means it will print everything, if it's a number then that's the limit to how much it will print before printing the periods.  For me it's NIL so it prints the whole list of things in the solution.  If it's a number, this should work:
(let ((*print-length* nil))
  (ps:solve (ps:string->card-list "4h Kd 3h 4c Tc 5s 7s 6c 8d Ah Js 7h 5d Kh 2s 6s Ts Jc 6d 9h 7d 5c Ac 2c 4d 3d 9d Ad 8h 7c 8s Th 8c 2h Td 3c 3s Ks 4s Jd Qs 9s Qc 6h Jh 2d Qh 5h Qd 9c Kc As")))

Also, does this line below work at all?  It's calling command-line::print-solution with two colons because the print-solution function isn't exported by the package.
(command-line::print-solution (ps:solve (ps:string->card-list "4h Kd 3h 4c Tc 5s 7s 6c 8d Ah Js 7h 5d Kh 2s 6s Ts Jc 6d 9h 7d 5c Ac 2c 4d 3d 9d Ad 8h 7c 8s Th 8c 2h Td 3c 3s Ks 4s Jd Qs 9s Qc 6h Jh 2d Qh 5h Qd 9c Kc As")))

Another thing that may help is:
(loop for step in (ps:solve (ps:string->card-list "4h Kd 3h 4c Tc 5s 7s 6c 8d Ah Js 7h 5d Kh 2s 6s Ts Jc 6d 9h 7d 5c Ac 2c 4d 3d 9d Ad 8h 7c 8s Th 8c 2h Td 3c 3s Ks 4s Jd Qs 9s Qc 6h Jh 2d Qh 5h Qd 9c Kc As"))
    do (format t "~A~%" step))

This is just iterating through each step in the solution and printing them one by one.

*print-length* was set to 12, but the command you provided didn't seem to change the output.
The other two commands never seem to finish running.
After your comment that it worked when you downloaded it, I downloaded the binary distribution from the SBCL website and it works in that distribution (my system install is from Homebrew, so probably just built with different options).
Thanks :)

Yes, I had downloaded it from the SBCL website.
Ah I should have been more careful when posting the first one, it would need to be (setf *print-length* nil) followed by running the (ps:solve... to make it work.   With the let, it's just temporarily setting it to NIL while running the solver and resets back to 12 just before printing the result :)
I'm trying to figure out what's going on with it changing to 12...

In light of the discovery of it happening after errors in the version from their website, I tried again in the Homebrew version, being careful not to mistype anything and trigger an error, and it does work in that version as well.


Add support for e2e test filenames

Fixes #2022
Changes proposed:
Trying to add support for the .e2e-spec.ts, .e2e-test.ts files
Exploring the code. I found the extension file. This applies to all ts files, I do not know if this completely solves the problem. I await your review.

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Yeap. It was that simple.


Group creation API

Is there an API available to create a group using the bot token and API to create a channel?

No. Use td-lib.

No. Use td-lib.


Use the default transform definition

Merged build finished. Test PASSed.

@calvinjia I can't do an end-to-end test easily, but I added a simple unittest which exercises some of it.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6334/
Test PASSed.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.1 please.


[TypeSpecValidation] throw error on incorrect tspconfig file extension

closes #25122
I think it's better to throw any non .yaml, having it as a more generalized error

pipeline failed successfully https://dev.azure.com/azure-sdk/public/_build/results?buildId=3022127&view=logs&j=52599800-cfcd-5615-dd98-137d26e8cb9a&t=c0acbdc7-6ccc-5699-6c3a-c41428f73fe2


Allow multiple peptide lengths

esp. to cover the cases where we want to allow for better manufacturability

Right now we generate only 25mers; here we'd want to generate e.g. 23, 24, and 25mers (possibly allow to specify ranges) and use manufacturability ranking to choose.


Sm/coft-deploy-fix

What does this PR do?
makes it possible to deploy just a CustomObjectFieldTranslation (decomposed child of customObjectTranslation)
Background:
COFT is not in the metadata api--it has to be rolled up into a COT to be deployed.
The following would not work:

force:source:deploy -p [path to COFT]
changing a COFT file locally and calling force:source:push

This now:

will resolve a CFT to its parent COFT when building componentSets
when a COT is deployed, will create FileReponses for each of the child CFT (we don't know which ones were deployed, so we mark them all.  These fileResponses also make sure that source tracking updates correctly for CFT)

What issues does this PR fix or reference?
@W-11811429@

QA Notes:

setup
with a project with COFT

✅ deploy -p force/app
✅  deploy -p force-app/main/default/objectTranslations
✅ deploy -p force-app/main/default/objectTranslations/customObject__c-es/customField__c.fieldTranslation-meta.xm
✅ deploy -p force-app/main/default/objectTranslations/customObject__c-es/customObject__c-es.objectTranslation-meta.xml
✅ : retrieve -p force-app/main/default/objectTranslations
❓ :
retrieve -p force-app/main/default/objectTranslations/customObject__c-es/customField__c.fieldTranslation-meta.xml
Preparing retrieve request... done
=== Retrieved Source

No results found

✅ retrieve -p force-app/main/default/objectTranslations/customObject__c-es/customObject__c-es.objectTranslation-meta.xml


File name change not taken into accout in export

In Chalk'it hosted version, file rename is not taken into account when creating xprjson file name

In this example file is saved as Untitled.xprjson instead of new.xprjson

Thanks Ghiles !


Faker Image not working

http://lorempixel.com/ website is down. so no images is being downloaded or grab.

See #1712 and #1713.

I use "picsum.photos" in my local env like following lines. But I think this is evil approach.
    public static function imageUrl($width = 640, $height = 480, $category = null, $randomize = true, $word = null, $gray = false)
    {
-         $baseUrl = "https://lorempixel.com/";
+        // $baseUrl = "https://lorempixel.com/";
+        $baseUrl = "https://picsum.photos/";
        $url = "{$width}/{$height}/";

and
            $fp = fopen($filepath, 'w');
            $ch = curl_init($url);
            curl_setopt($ch, CURLOPT_FILE, $fp);
+            curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
            $success = curl_exec($ch) && curl_getinfo($ch, CURLINFO_HTTP_CODE) === 200;

https://lorempixel.com/ up again

The follow location however is a good addition. Just replacing it is a bit too much where the service runs fine right?

Duplicate of #1712.


New theme: Hallo

Hi! I've created a new theme called Hallo, which can be used as a single page personal introduction page with links to your social media accounts. As such the theme (for now) cannot contain posts. I have therefore included an empty example site with only a config.toml included. This works like a charm on my pc; I hope it works with the theme site as well! If not, let me know what I can do to help fix it. Thanks a lot!
Theme submission
Before submitting a theme please make sure you've read the README.md. It describes in detail what information are needed and which requirements have to be fulfilled for the submission. The check list below can help you.
Link to my theme repository: https://github.com/EmielH/hallo-hugo
I made sure that...

[X] the repository contains a good README.md describing my theme
[X] an open source license has been added to LICENSE.md
[X] all metadata have been added to theme.toml
[X] screenshots have been added in the images/ folder with the required dimensions
[X] in case I'm using a customized demo via the exampleSite folder that

[X] https://example.com is set as base url in exampleSite/config.{toml, yaml, json} to avoid the abuse of unused domains
[X] the site's content is as neutral as possible (contains no ads etc.) and reflects the features of my theme

[X] I tested my theme against the gohugoio/HugoBasicExample

[X] I've checked the developer tools' console in my browser for error messages

[X] in case my theme is using Hugo Pipes features like toCSS and PostCSS that I have committed the /resources directory with all generated assets, for my theme to work in the basic version of Hugo

Feel free to ask questions. We're glad to help.

I stumbled upon https://github.com/gohugoio/hugoThemes/issues/387 so I added a content directory to my example site.

Hi @EmielH and thanks for sharing your theme with the community!
When I execute the Themes Site Build Script everything looks good with your demo. 👍
@digitalcraftsman This theme is ready to be added to the website.

Hello @EmielH,
thank you for sharing this theme with the Hugo community. Your theme should be listed soon on the themes site.
Furthermore, I'll promote your theme on Hugo's official Twitter account.


pkg/test: create new frameworkClient

This commit create a new FrameworkClient type accesible as
Global.Client. It is a simple wrapper of the controller-runtime's
client with the exception of Create, which now adds cleanup
functions. This also changes "Finalizer" to "Cleanup" and trims
the Framework struct to only include variables that are useful
to the user. All other variables required in the framework are
now separate, unexported variables.
/cc @hasbro17 @shawn-hurley

LGTM


Introduce more secure way of generating random values (replace os.urandom)

Goal is to improve generating of random values. I assume that package secrets will handle.
Steps:

introduce new package secrets
replace all occurances of os.urandom(xxx)

It's good first issue cause it needs small modification in code and pyproject.toml file.

Hello my name is Bavya and I am a software engineering student looking for a good first issue. I would like to try resolving this issue. Can I work on this if no one else is assigned yet?

Hi Bavya, sure, thanks for interest in improving this package.

Hi, @BavyaMittal  👋
When can I expect a solution?
If you don't want to perform it let me know.

Hi @Veinar!
Sorry about the delay I am working on it and should have it ready by end of day today.

Hi, @BavyaMittal 👋
Any update?

Hi @Veinar please see the pull request: https://github.com/Veinar/envcloak/pull/63 .


Microsoft.Azure.Documents.InvalidPartitionException: Partition range id 6 does not exist, please retry shortly after re-initializing BulkExecutor instance,

Hi, I am getting this error. The exception is an instance of Microsoft.Azure.Documents.InvalidPartitionException but I cannot catch because is in a protection level. Could you let me know if I cannot do the retry manual or the bulkinsert do automatically by me?
Thanks in advance.
PS: There is more people with the same issue, https://github.com/Azure-Samples/azure-cosmosdb-graph-bulkexecutor-dotnet-getting-started/issues/4

This should be fixed by 1.2.0 version of the BulkExecutor library.


Remove Advancement on Race deletion doesnt remove tool proficiency box [2.4.0]

When you try to remove a race that grants a tool proficiency it doesn't remove the box from the tool proficiency list.
Tested it in a newly made world with no modules. made a player actor and added the hill dwarf race. deleted it and had it remove all advancements
Before the deletion (just for show)

After deletion, brewers supplies is set to not be proficient, but in the large proficiency list it is proficient?

This is intended behavior so it doesn't delete the associated ability or bonuses included with the tools. It also remains if you use the circle to unset proficiency.

Arbron didn't point this out explicitly, so I will.
When you delete the race/background/etc. it unchecks the proficiency for the tool. It does leave the tool on the character sheet, but the character is not proficient, even though the tool is still listed.
These light gray widgets are very subtle, and it's easy to miss them.


Implemented quote.

My knowledge of scheme is primitive. So I only created a simple test.

Thanks @ss I missed the PR somehow. I'll review and try get this merged in.


Add pthread mutex type constants

These are needed to create a pthread mutex that doesn't use lock elision on glibc.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+ b2b9f29eecaf6f8d5650a676d9669f3f871595df
Thanks!

:hourglass: Testing commit b2b9f29 with merge 1f870b8...

:sunny: Test successful - status-appveyor, travis


How increment column version on each modify model via hook?

Hi, I have model and BeforeSave hook
func (m *User) BeforeUpdate(tx *gorm.DB) (err error) {
	tx = tx.UpdateColumn("_ver", gorm.Expr("_ver + ?", 1))
	return tx.Error
}

And save method in repository
func (repo *Repo) Save(model *User) (*User, int64, error) {
	gormDb = repo.Db // *gorm.DB
        tx := gormDb.Debug().Save(model)
	return model, tx.RowsAffected, tx.Error
}

I want save/update/persist model and apply increment for column _ver (version) on each modify.
Hook not overlap value from model now.
Expected SQL is  UPDATE "users" SET "up_at"='2023-02-19 13:54:01.912',"_ver"=ver + 1,"name"='alex4' WHERE "id" = 2
Actual SQL is  UPDATE "users" SET "up_at"='2023-02-19 13:54:01.912',"_ver"=23,"name"='alex4' WHERE "id" = 2

https://github.com/go-gorm/optimisticlock

@a631807682 I don`t undestand how optimisticklock help me in my case. I need generate sql expression via gorm hooks.
optimisticlock has not example with hook. Expression sql on Update work correct in native gorm.

Please, repopen issue

tx.Statement.SetColumn("_ver", clause.Expr{SQL: tx.Statement.Quote("_ver") + "+1"}, true)

https://github.com/go-gorm/optimisticlock/blob/master/version.go#L160

@a631807682 this example not work for my case.
Debug sql log:
UPDATE "users" SET "up_at"='2023-02-23 23:40:38.722', "_ver"=0, "name"='bob' WHERE "id" = 2
See full code from my first post.
I assume that their hook changes are ignored and data for SET is taken from the .Save(model).

In fact, the setting of BeforeUpdate will be overwritten when Update later, because Save will save all values, including zero values. We can only build the complete clause through hacks.
Why do you need to refer to optimisticlock.Version, because it contains custom data types and custom Clauses implementations.
https://gorm.io/docs/serializer.html#content-inner
https://gorm.io/docs/sql_builder.html#StatementModifier
func (m * User) BeforeUpdate(tx *gorm.DB) (err error) {
	dbname := "_ver"

	stmt := tx.Statement
	dv := reflect.ValueOf(stmt.Dest)
	if reflect.Indirect(dv).Kind() == reflect.Struct {
		selectColumns, restricted := stmt.SelectAndOmitColumns(false, true)

		sd, _ := schema.Parse(stmt.Dest, &sync.Map{}, stmt.DB.NamingStrategy)
		d := make(map[string]interface{})
		for _, field := range sd.Fields {

			if field.DBName == dbname || field.DBName == stmt.Schema.PrioritizedPrimaryField.DBName {
				continue
			}

			if v, ok := selectColumns[field.DBName]; (ok && v) || (!ok && (!restricted || !stmt.SkipHooks)) {
				if field.AutoUpdateTime > 0 {
					continue
				}

				val, isZero := field.ValueOf(stmt.Context, dv)
				if (ok || !isZero) && field.Updatable {
					d[field.DBName] = val
				}
			}
		}

		stmt.Dest = d
	}

	stmt.SetColumn(dbname, clause.Expr{SQL: stmt.Quote(dbname) + "+1"}, true)
	return tx.Error
}

Thanks!


Propagate the changes from DPGAlliance/DPG-Standard: #37 + #41

See:

DPGAlliance/DPG-Standard#41
DPGAlliance/DPG-Standard#37

@nathanbaleeta @lacabra I would like to work on this issue if it is still open. Can this be assigned to me?

Thanks @rachita19082, but we prefer @nathanbaleeta handles this, thanks again 🙏

Closing this issue as all the requested changes were implemented and successfully merged on DPG-Standard#37 and DPG-Standard#41


Adds redis cache for distributed environments

Current coverage is 92.29% (diff: 100%)

Merging #2691 into master will increase coverage by 0.05%

@@             master      #2691   diff @@
==========================================
  Files           102        103     +1
  Lines         12446      12536    +90
  Methods        1559       1583    +24
  Messages          0          0
  Branches       2040       2044     +4
==========================================
+ Hits          11480      11570    +90
  Misses          966        966
  Partials          0          0

Powered by Codecov. Last update cab69b4...c79aa60

@flovilmart updated the pull request - view changes

@flovilmart updated the pull request - view changes

Also sets the PG tests to allow failure until we figure out good repro steps, at least the history will be kept in travis, and build won't fail for bad reasons.

Can you add some docs for this one as well?

On the wiki yes!

@flovilmart adding support for caching on the server is great.
Is there a place I can see an example how to use Redis in Cloud Code?
Thanks

This isn't for cloud code, this is for the internal cache.

@flovilmart updated the pull request - view changes

Is there any way you suggest to use some type of caching in cloud code.
For example I want to build a cache on the server to save result of previous queries with time stamps, and when I get the same query from another user I can check first the local cache to see if I have a recent result, if not issue the query to MongoDB via Parse server.

You would have to use cloud functions for that.

Right, but do you have any suggestion for how to cache the data in cloud function over future calls. I can create something from scratch, but I am sure their is already good other solution that other used, as this should be a common need by many to improve performance.

Again, that is not the purpose of that adapter. it's only purpose is the same as the original InMemoryCacheAdapter. For other caching mechanisms you should not use that, as we internally flush the cache entirely.

I understand that I should not use it. My question was do you have advise about another cache mechanism I can use in cloud code that will be persistent as long the server is up? If server is restarted I will rebuild the cache from scratch.

You should probably as those questions over stackoverflow or stackexchange. I have no suggestion for building such cache as it depends on many parameters.

Cool. I see the docs now. @mmarshak since Cloud Code in parse-server is run in node, you can use any of the available caching libraries on npm within your Cloud Code. That could mean a pure-js, in memory cache adapter, or you could install a redis or memcached javascript client and use it directly within your Cloud Code.

@drew-gross thank you for the pointers.


Provide way to suppress grid empty text entirely

Passing overlayNoRowsTemplate: '' does not override the default template - presumably the empty string is considered falsey in terms of how the defaulting works (which caught me a bit by surprise).
It seems reasonable to have a way to not show any empty text.

https://github.com/exhi/hoist-react/compare/hideEmptyTextExample?expand=1


update otel go dependency to v1.0.0-RC1

Bumping the depency of OpenTelemetry go to v1.0.0 release candidate 1 in preparation for the 1.0.0 release.

Yep, ran the command manually and it all worked as it did before.

There are unit tests in cmd/ and some functional tests in the demos directory, if all those play out, we're probably good. I'll make an issue to start shaping those demos into actual tests we can automate.

There are unit tests in cmd/

Whoops 🤦


AutoGluon Tabular excluded_model_types doesn't work.

Hello.
I had a test on Ubuntu 18.04 / Python 3.6.9 w/ AutoGluon ver. 0.1.0b20210225.
When I call TabularPredictor(label=label_column).fit(train_data, excluded_model_types='["KNN", "NN"]') ,
It works well w/o any error message.
But, Even though I ordered `Do not train KNN, NN kinds of models',
AutoGluon Tabular still trains those kinds of models.
Indeed, I got the log message `Excluded Model Types: ["KNN", "NN"].
But, NOT ACTUALLY WORKS..
This is why I got the *** Summary of fit() *** which contains KNeighborsUnif, NeuralNetFastAI, etc.

Are you passing the string value '["KNN", "NN"]' instead of a list ["KNN", "NN"]? You have to pass a list. A string will not be properly handled.


namespace not being logged when using fetch_multi

namespace isn't logged when using fetch_multi.
Rails.cache.fetch_multi(1, 2, namespace: "foo")

logs the following:
Cache fetch_multi: [1, 2]

I would expect namespace to be logged as well.

I believe this should be addressed by #511

Closing as #511 is merged.  This fix will be included in 2.7.5


boa params updated -- working well though dropping off a bit at end. …

… rl, pcore defaults updated and dummy funs for prjn funs not used

Codecov Report
Base: 37.76% // Head: 37.76% // No change to project coverage :thumbsup:

Coverage data is based on head (ec1eaed) compared to base (1689021).
  Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Searching for a working central server scrypt

Hi,
I am very new to the OCPP system and I'm looking for a python scrypt of a central server that can actually handle a connection with a charging point via OCPP 1.6. From what I have understood is the example code in this repository incomplete and can't actually handle transaction notifications or most other messages. Would any of you be willing to share a more complete version of a central system that I can use to test around with?
Additionally, how did you start coding the different handlers for each type of message? I feel like there is a lack of documentation for this library so I have no idea how to even start on making handlers.
Thanks!

Hello, this library only contains the building blocks create requests and responses and route them to handlers. The library does not provide business logic to tie handlers together.
I recently created https://github.com/OrangeTux/ocpp-for-humans, a project to explain certain OCPP concepts using this library. It's empty at the moment, but you can keep an eye on that.

I plan to write an open source controller that will use this library for ocpp communications. It won't be ready for testing until November or so.
What brand of charger do you have? Did it come with any ocpp documentation? Regards, Larry

The documentation on how to use this library has been raised as needing improvement in https://github.com/mobilityhouse/ocpp/issues/55,
regarding OCPP documentation, this library has all updated OCA OCPP documentation of all available versions under docs
I will close this issue for now


.NET Framework containers are no longer supported for 1803 servers

https://github.com/dotnet/announcements/issues/141
How is the support for containers when running the win1803 image?

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

ID: 99af681d-41ad-da15-0b72-08b5cd5a11ac
Version Independent ID: 2fa0df36-a437-8cbb-b019-b49b31b1e116
Content: Microsoft-hosted agents for Azure Pipelines - Azure Pipelines
Content Source: docs/pipelines/agents/hosted.md
Product: devops
Technology: devops-cicd
GitHub Login: @steved0x
Microsoft Alias: sdanie

@rastrup  Thank you for the feedback, support for the 1803 image ended on November 12th, 2019.
Please refer to the  Microsoft Docker repo for more information and a current list supported images.
To see the version used by an agent, refer to the azure-pipelines-image-generation repo, for example, the vs2017-win2016 uses the mcr.microsoft.com/windows/servercore:ltsc2016 image.
Assigning this to the author for review.


face_recognition fails to detect some obvious faces.

face_recognition version:1.2.3
Python version: Python 3.5.6 :: Anaconda, Inc.
Operating System: Ubuntu 16.04

Description
Thank you for a great tool! I am trying to detect faces for COCO dataset(Images which have faces). For 50% of times, it is able to detect the faces correctly, but other 50% of images it fails. I presume this has to do something with the threshold.
What I Did
I got a chance to go through the code flow. But I could not quite understand how can I actually tweak the threshold to detect the faces which are most probably missed due to low confidence.
The Command which I am using is:
face_locations = face_recognition.face_locations(image) , which  eventually calls following function(according to my understanding):
_raw_face_locations(img, number_of_times_to_upsample=1, model="hog") ---> face_detector(img, number_of_times_to_upsample) ---> dlib.get_frontal_face_detector()
Now in dlib repo, I see that this is connected to file dlib/dlib/image_processing/object_detector_abstract.h, where we have following definitions :
template <
            typename image_type
            >
        std::vector<rectangle> operator() (
            const image_type& img,
            double adjust_threshold = 0
        );
        /*!
            requires
                - img == an object which can be accepted by image_scanner_type::load()
            ensures
                - This function is identical to the above operator() routine, except that
                  it returns a std::vector<rectangle> which contains just the bounding
                  boxes of all the detections.
        !*/

What i am not able to understand how can I change adjust_threshold ? using the Function: face_recognition.face_locations(image).So, that it can detect some of the less obvious faces in the images .
Example Image for which it misses the faces:

None of the faces were detected. I understand this could be one of the difficult cases, But If I can detect the faces, with some tweaking (by lowering the confidence) It will serve my purpose.
Thank you.
Regards,
Nitin

Hi @nbansal90,
If I'm not wrong with my understanding, you're asking how to tweak the threshold in the face_recognition.face_locations(image), such that you can find the faces in the above image?
In this library, the threshold is used to find out the best match, while comparing the input encodings with the saved encodings, the lower the threshold, the stricter the comparison. You can  read the info more clearly in the documentation
Now, to detect the faces more clearly in the given image, use model='cnn' instead of 'hog', by adding the  parameters next to the image.
face_recognition.face_locations(image, number_of_times_to_upsample=1, model='cnn').
You can find two faces in the above image.
Hope this might clear your doubt.
Regards,
Chaitu

Or You can just increase the number_of_times_to_upsample to improve the detector's performance.


cartridge: pg_dump and SMILES

Am I right to understand that the pg_dump of a table with a mol field is done via its SMILES ?
This produces other mol information such as mol properties or conformations to be lost after a  dump and restore.
It's how it is intended to work, or am I missing something ?
RDKit version: 2019.03.1
RDKit cartridge version: 0.74.0
Thanks !

Hi @xjalencas ,
Unfortunately you are correct: an RDKit mol column appears in the output of pg_dump just as SMILES.
And, yes, this does result in conformations being lost.
A partial workaround here is to create a column containing the molecule coordinates:
select id,mol_to_ctab(m)::text as ctab into ctabs from mols1;

Then at least you'll have the coordinates.
It occurs to me that we can get even closer to solving this problem by using CXSmiles within the cartridge. I'll look into that.

Thanks!  This helps a lot.
I also tried with a byte column with mol_to_pkl(mol), so multiple conformations can be stored.

Hello, are there any updates or changes to this? It would definitely be helpful to be able to pg_dump to MOL/SDF format and not lose 3D information without keeping extra columns in sync.
Currently, am using pg_dump with the plain output format, i.e. to a textual SQL dump. Any idea if there is any benefit to using the custom format with RDKit, i.e. whether that retains the mol as binary data, or whether it also does the SMILES conversion?
I can experiment, but any existing knowledge much appreciated.

@harryjubb : the cartridge now will write cxsmiles (instead of smiles) for molecules that have conformations saved, so you still get the coordinates:
hembl_26=# select molregno,m from demo_mols limit 2;
 molregno |                                                                                m
----------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
        1 | Cc1cc(-n2ncc(=O)[nH]c2=O)ccc1C(=O)c1ccccc1Cl |(4.7417,-4.1417,;5.2667,-3.8417,;5.2667,-3.2417,;5.7917,-2.95,;5.7917,-2.35,;6.3125,-2.05,;6.3125,-1.45,;5.7917,-1.15,;5.7917,-0.55,;5.2792,-1.45,;5.2792,-2.05,;4.7542,-2.3417,;6.3042,-3.2417,;6.3042,-3.8417,;5.7875,-4.1417,;5.7875,-4.7417,;5.2667,-5.0417,;6.3042,-5.0417,;6.8167,-4.7417,;7.3417,-5.0417,;7.3417,-5.6417,;6.825,-5.9417,;6.3042,-5.6417,;5.7792,-5.9417,)|
        2 | Cc1cc(-n2ncc(=O)[nH]c2=O)ccc1C(=O)c1ccc(C#N)cc1 |(4.7417,-4.1417,;5.2667,-3.8417,;5.2667,-3.2417,;5.7917,-2.95,;5.7917,-2.35,;6.3125,-2.05,;6.3125,-1.45,;5.7917,-1.15,;5.7917,-0.55,;5.2792,-1.45,;5.2792,-2.05,;4.7542,-2.3417,;6.3042,-3.2417,;6.3042,-3.8417,;5.7875,-4.1417,;5.7875,-4.7417,;5.2667,-5.0417,;6.3042,-5.0417,;6.8167,-4.7417,;7.3417,-5.0417,;7.3417,-5.6417,;7.8625,-5.9417,;8.3792,-6.2417,;6.825,-5.9417,;6.3042,-5.6417,)|

I believe that this is what should also end up in the output of pg_dump.

Thank you @greglandrum, am currently precluded from upgrading RDKit for other reasons but hoping to do so soon.
How lossy is the CXSMILES output compared to SDF? Do tags get retained? Another aspect thta has an impact on some programs is atom ordering, if explicit non-polar hydrogens and their coordinates are lost, presumably that is also affected too?
Ended up in the meantime going with a solution inspired by @xjalencas' byte column solution, using a Python script to write out any mol column data in pickle format, converting on-the-fly, which had the advantage of maintaining all of the above information while not needing to maintain / sync an extra database column.
Thanks for your help and time.

I'm closing this because it looks like the question is answered. Please reopen if necessary.


notice when TAILSCALE_AUTHKEY is missing, and error early

When a pull request is initiated from a fork, none of the secrets of the original repo are available. This notably includes the TAILSCALE_AUTHKEY, which means tailscaled in the CI runner is trying to start with "tailscale up --authkey= --hostname=name"
We should notice if the authkey is empty and exit early with a comprehensible error message.

It would be nice if you could also detect an expired authkey. :) Not sure if worth making a separate issue for that?

perhaps related, but we recently had an issue where an auth key was accidentally revoked (https://github.com/tailscale/corp/issues/7280).  The behavior there was that the tailscale action hung and was eventually killed by GitHub after ~6 hours.  Like the above cases, this seems like something we should be able to detect and fail fast.


Complete voltLib

It would be nice to complete @moyogo’s work on voltLib in fontTools at some point. Also, SIL has a Perl script that compiles VOLT to TTF, which might be helpful to look at — thanks to @mhosken for pointing this out.
https://github.com/silnrsi/font-ttf-scripts/blob/master/scripts/volt2ttf

@moyogo has this been done, or is it planned?

@khaledhosny finished it and it’s used in TiroTypeworks/TiroTools.


As per our understanding Windows does not support System Clock and PHC clock synchronization. But Time Based Packet Transmission has clock synchronization as prerequisite. Any plans on synchronizing the clocks in Windows?

https://github.com/microsoft/quic-offloads/blame/fc6f00badd3567394a59ed8d1267fd0a8764776c/time-based-packet-transmission-offload.md#L26

Closing #64 and #65 because we have updated TPTO to use relative (to Now()) time, which eliminates the need for any clock synchronization.


Feature Request: Allow Override of Container Image Name for SurrealDB In Memory Container

I’m using surql-gen (v2.6.0) in an environment where pulling images directly from Docker Hub is blocked due to a VPN/proxy. I have a SurrealDB image available in our Artifactory, but the tool tries to pull the latest image from Docker Hub.
Feature Request:
It would be really useful to have an option to override the default SurrealDB container image name, like how Helm charts allow for container overrides. This would let users specify a custom image from an internal registry (e.g., Artifactory), making the tool more flexible in different network setups where Docker Hub isn’t accessible, and images need to come from an internal registry.
Proposed Solution:
Add a config option in surql-gen to let users pull surrealdb:latest from an internal registry.
Example usage:
npx surql-gen -f schema.surql -o ./output --no-generate-client --container-registry myregistry.com/surrealdb/surrealdb:latest

That would update pass in the registry URL here.
Thanks for considering this! The tool has been super helpful, and this feature would make it even better for different environments.

Seems reasonable to me, should be a simple change. Any concerns @sebastianwessel?

no makes sense.
maybe, a short look into testcontainer config might be good - don´t know out of my mind if there are other helpfully options

Released version 2.6.1, which includes the config option. Thanks @hammo92

Note for Private Registry Users:
If you're using a private registry, you’ll need to configure Testcontainers to pull the Ryuk image from your internal registry. You can do this by creating a .testcontainers.properties file in your user directory (windows: e.g., C:/Users/YourUsername/.testcontainers.properties) with the following configuration:
ryuk.container.image = yourregistry.com/ryuk:0.3.3

This ensures Testcontainers pulls the Ryuk image from your registry, which is necessary for container cleanup.
See here for more details


Adds command planner roster member remove, Closes #4394

This PR adds command planner roster member remove.
Closes #4394

Thanks @nicodecleyre for your awesome work. We will review it ASAP

Great start so far @nicodecleyre! Let's change some things first before we continue.

Thank you for your good review @milanholemans, as always 🤩

Great work refactoring so far @nicodecleyre! Let's do some more changes before we proceed. We're almost there! 💪

Your new feedback is processed!

Thank you for your detailed review @milanholemans, updated the PR!

Merged manually. Thank you for this great addition! 👏


adding totals row to the top, summary on top would be helpful for most cases

Description of PR
Summary row on the top would be helpful as user may be looking for one or two states and then country wide summary at any given point.
Checklist

[x] Compiles and passes lint tests
[x] Properly formatted
[x] Tested on desktop
[x] Tested on phone

Screenshots

@jayanthkaturi
Don’t really see a merit in this addition. But if you’re adding the total row to the top, might as well keep it fixed while scrolling

@aseef17 Good idea, fixed total row on scroll.

@jayanthkaturi
Maybe you could fix the label and the count header without the space in between. Just a suggestion to make it easier on the eye

Fixed it. Let me know if it doesn't work.

@jayanthkaturi
Doesn’t seem to work still

Checked it on my mobile/chrome dev tool, could please tell me the resolution or phone model?

Phone Model - iPhone XR
Resolution - 1792 x 828 px

Unable to reproduce the issue, any chance it's a cached version?

Unable to reproduce the issue, any chance it's a cached version?

Seems like it was cached. Works fine now 👍🏽

Thanks @aseef17

Please read the Contributing Guidelines before creating a PR


Feature/help

Added HelpView
Inserted Help to navbar for logged in users

Fixed merge conflict in index.js


Add v1.10.0 to portability tests

Thanks for adding this!

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Looks like Interop Cloud-to-Cloud Tests (internal CI) has successfully finished, but the status is not updated for some reason (there's an "invalid ioctl" error at the end after the tests finish successfully).  Merging.


"This type of CollectionView does not support changes to its SourceCollection from a thread different from the Dispatcher thread"

Just got this stack trace when trying to update the resolvers list (with the just released version 0.3.8)

Me too..
Updated over previous installation of 0.3.7....DNS cache service is enabled...Using Windows 8.1 Pro 64bit...
run at admin previledge....
Full message in text
Message: This type of CollectionView does not support changes to its SourceCollection from a thread different from the Dispatcher thread.
StackTrace:    at System.Windows.Data.CollectionView.OnCollectionChanged(Object sender, NotifyCollectionChangedEventArgs args)
   at System.Collections.ObjectModel.ObservableCollection`1.OnCollectionChanged(NotifyCollectionChangedEventArgs e)
   at System.Collections.ObjectModel.ObservableCollection`1.ClearItems()
   at System.Collections.ObjectModel.Collection`1.Clear()
   at SimpleDnsCrypt.ViewModels.MainViewModel.<RefreshResolverListAsync>d__104.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.<ThrowAsync>b__6_0(Object state)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)


feat(dashmate): extend enabled / total count

Expected Behavior

dashmate status mastrenode should show enabled / total count for each node groups, like total, hpmn and regular. Just like we have in the dashcore
Example:

Current Behavior

It only shows total enabled count
Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop, server, or mobile):
Link to your project:

https://github.com/dashpay/platform/pull/1505


Actively set hostname only on Android 5

Fix #7524
https://cs.android.com/android/platform/superproject/+/android-6.0.0_r1:external/conscrypt/src/main/java/org/conscrypt/SSLParametersImpl.java;l=539
jsse.enableSNIExtension property is backported, so users can configure SNI themselves if need.

We haven't tended to adopt the configuration idioms of different Android and JVM versions. For example we try to enable newer TLS versions on the JVM before they were the default.  See https://github.com/square/okhttp/blob/3ad1912f783e108b3d0ad2c4a5b1b89b827e4db9/okhttp/src/jvmMain/kotlin/okhttp3/internal/platform/Jdk9Platform.kt#L76
Your change affects isSniEnabledByDefault, but in getUseSni, the useSni flag is also checked. setHostname causes this to be set, so it seems consistent.
https://cs.android.com/android/platform/superproject/+/android-6.0.0_r1:external/conscrypt/src/main/java/org/conscrypt/SSLParametersImpl.java;l=343
    /**
     * This method enables Server Name Indication
     *
     * @param hostname the desired SNI hostname, or null to disable
     */
    public void setHostname(String hostname) {
        sslParameters.setUseSni(hostname != null);
        peerHostname = hostname;
    }

Do you have servers where this isn't working on legacy android where we are sending SNI? But would work without? I assume sni.cloudflaressl.com requires SNI, so I'd like to understand for a real world scenario what is broken and what will be fixed by this PR.
Thanks

#7524 see test app

The problem is not setUseSni , but okhttp doesn't respect SSLParameters.setServerNames parameter.  sni being forced to enable is just a side effect.

Thanks, I'll check

https://github.com/square/okhttp/pull/5702
I believe this patch will not cause a regression. okhttp doesn't force set hostname can save a lot of trouble.

If it results in use not using SNI in more cases, then it's likely to cause a lot of requests to fail. SNI is pretty critical to modern web infra.
But let me run the sample you provided.

OK, I think I'm up to speed.  However my comment

We haven't tended to adopt the configuration idioms of different Android and JVM versions.

Still applies.
Your change has two changes

don't call sslSocket.setHostname on 24+, since we assume android does this for us, and it may override a custom ssl socket factory.
don't call on <=23 if jsse.enableSNIExtension is false as user may be configuring it off or setting manually.

My two objections are

using jsse.enableSNIExtension, we generally want the best TLS connection the platform can support
Disabling SNI in 2022

So the change seems too broad.
Similar requirements have come up before for disabling SNI, and also domain fronting.  We supported domain fronting explicitly as a feature, but stopped at allowing disabling SNI.
I'm still looking at your reproduction, but wanted to give initial thoughts. apologies if I've misunderstood, this timeline of different android versions is hard to keep in my head.

I'm not trying to disable SNI, I'm trying to okhttp respect the parameters I set.

I used SSLParameters.setServerNames() to set SNI to another host, but okhttp reset it before the handshake and I didn't have a chance to intervene.

When this came up previously the workaround was to set the SNI host in the request url, but then use a custom DNS, to redirect to the actual host you need.
Would this work for your use case?

You've stopped actively enabling SNI in Android 10+, I don't understand why you're so resistant to stop actively setting incorrect hostname in Android 6+. That doesn't disable SNI, does it?
Also, please stop the discussion around SNI enable or disable, the issue is okhttp not respecting the user's hostname settings, I think it's a behavioral error, not a feature request.

I added jsse.enableSNIExtension not to disable SNI, but to make okhttp stop setting hostname. In Android 5, okhttp can't know what hostname the user wants, and as you said before, there won't be a new API, so simply disable this behavior and give the user a chance to set it themselves.

So the change seems too broad.

Android 10+: no changes
Android 6-9: Because the platform has enabled SNI, SNI is still enabled after okhttp stops setting hostname. And users can configure the hostname by reflection or SSLParameters.setServerNames()
Android5: okhttp will still set the hostname and enable SNI by default, if the user wants another hostname, they can set jsse.enableSNIExtension property to stop the behavior.
In summary, nothing has changed from before, and the problem of not respecting user configuration has been fixed.

I added jsse.enableSNIExtension not to disable SNI, but to make okhttp stop setting hostname. In Android 5, okhttp can't know what hostname the user wants, and as you said before, there won't be a new API, so simply disable this behavior and give the user a chance to set it themselves.

I hope this hostname is correctly coming from the request url host.
  private fun connectTls(sslSocket: SSLSocket, connectionSpec: ConnectionSpec) {
    val address = route.address
    var success = false
    try {
      if (connectionSpec.supportsTlsExtensions) {
        Platform.get().configureTlsExtensions(sslSocket, address.url.host, address.protocols)
      }

Which also points out you can probably disable this via the deprecated flag in ConnectionSpec. It may be we should un-deprecate this and make it a feature to allow clients complete control of SSL session.
But I'm still not convinced about OkHttp supporting legacy setups via a platform flag jsse.enableSNIExtension, we usually done the opposite, ignored platform flags and enabled all supported modern features.

You've stopped actively enabling SNI in Android 10+, I don't understand why you're so resistant to stop actively setting incorrect hostname in Android 6+. That doesn't disable SNI, does it?
Also, please stop the discussion around SNI enable or disable, the issue is okhttp not respecting the user's hostname settings, I think it's a behavioral error, not a feature request. Or, add a note in documentation that okhttp ignores SSLParameters.setServerNames() until Android 10

Good point. Can we change the PR to respect this on high enough android versions? And not use the jsse.enableSNIExtension flag?

And apologies if this is frustrating for you, not my intention, I just want to make the right change, given I'll need to support this for lots of apps and devices.  Thanks for the PR and your patience walking me through it.

Can we change the PR to respect this on high enough android versions?

No need to set anything, when the user used SSLParameters.setServerNames(), the hostname has changed. As long as okhttp does not reset it, everything is as expected.

I hope this hostname is correctly coming from the request url host.

SSLParameters.setServerNames() is a public API and set hostname is a function that works properly in other Java platforms and Android 10+.

And not use the jsse.enableSNIExtension flag?

I agree that the jsse.enableSNIExtension property is dirty, it comes from Android 6+ and allows the user to disable hostname that platform auto added for TLS handshake, which is exactly the same as okhttp isn't it?
When the user actively set hostname, jsse.enableSNIExtension is ignored. But in okhttp, since there is no new API, using this property to avoid breaking the user's configuration I think is the logical thing to do.
Maybe it is possible to use reflection to know if the user has a custom hostname? But this would complicate the code and it would be better to use an existing property.

I'm doing some local testing with https://github.com/square/okhttp/pull/7526/files, which is ported from your repo, thanks.

@vvb2060 can you take a look at https://github.com/square/okhttp/pull/7526, this approach will only call setHostname on API 21+22.  All other versions should rely on the platform.
But it also doesn't add special meaning to jsse.enableSNIExtension, we ignore this and set the hostname forcing it to be used even when jsse.enableSNIExtension==false.
Also take a look at the SniOverrideTest.getWithDns test, I think it shows the simpler cleaner way to implement what you are trying to do without a custom socket factory.

https://github.com/vvb2060/okhttpsni/commit/4ff0ed7afed0e0c9eb2d9f0871c859e1a746796d
I made some changes to configure the hostname without using reflection in API 17-29. Also modified the test url to avoid disable hostnameVerifier.

SniOverrideTest.getWithDns cannot customize hostname, for example uuid.one.one.one

Since API 21-22 has public API custom hostname, I think okhttp should not reset the hostname already configured by the user.
Android 5.0 api 21：
https://cs.android.com/android/platform/superproject/+/android-5.0.0_r1.0.1:external/conscrypt/src/main/java/org/conscrypt/SSLParametersImpl.java;l=100
boolean useSni;

Android 5.0 api 22：
https://cs.android.com/android/platform/superproject/+/android-5.1.0_r1:external/conscrypt/src/main/java/org/conscrypt/SSLParametersImpl.java;l=100
private Boolean useSni;

If user use SSLCertificateSocketFactory.setHostname(), useSni will be set, just check its value and skip rerset hostname.

SniOverrideTest.getWithDns cannot customize hostname, for example uuid.one.one.one

In this example there is 2 configured hosts

the sni host taken from the url
the target address taken from Dns

And if you want a separate host header to use for domain fronting.
What else do you need to customise?

Chinese GFW will censor SNI, one.one.one.one is in the blacklist, to avoid TCP reset, I need right IP, right Host, and custom sni.

The dns example should work if you add a host header. In fact it should already work for this with no changes.

?

Sorry, I didn't understand.
When sni is sni.cloudflaressl.com, the TLS handshake fails, which is the current behavior in Android 5-9, because okhttp resets the hostname.
When sni is one.one.one.one or cloudflare-dns.com, it will receive TCP reset sent by ISP, which is the behavior of SniOverrideTest.getWithDns.

Also, you seem reluctant to disable SNI. SNI has caused serious privacy breaches and internet censorship. I don't think feature requests to disable SNI should be ignored until okhttp supports ECH.

I think we were working directly away from that. As I mentioned above, we have a flag (connectionSpec.supportsTlsExtensions) that is now deprecated for that.
https://github.com/square/okhttp/pull/7525#issuecomment-1328256367
But you should raise a separate issue for that.

My mistake, I have no problem anymore, your PR is great.

Nice, so the request doesn't need special headers set in advance.
Actually the Interceptor could rewrite the host in the url also, at the same time it copies original host to the header


Small suggestion (+kc dks)

🚀 Feature Proposal
Just a small suggestion that should be easy to implement, can the command "+kc DKS" output the killcount for all 3 kings? For example:
+kc dks
Dagannoth supreme: 151
Dagannoth prime: 5234
Dagannoth rex: 512
It doesn't seem very important but it would be nice to be able to check the three all at once without having to do the "+m kc" command which outputs a massive list of all creatures killed on the bot, and so I wouldn't have to do the kc command for each king separately.

Implemented in /cl


fix: ensure from_address is correctly set from blockifier

Usage related changes
It does not affect the user directly, it corrects some situations of messaging when the message is sent by a contract call, and not a library call.
Development related changes
When using the script run_e2e.sh, it's important to use scarb 2.3.1 for now as starkli is not release with 2.4.0 compiler version.
I'm sorry for missing that earlier, it was found debugging and comparing messaging with Devnet, Katana (were we also had this issue) and Testnet.
Checklist:

[X] Applied formatting - ./scripts/format.sh
[X] No linter errors - ./scripts/clippy_check.sh
[X] No unused dependencies - ./scripts/check_unused_deps.sh
[X] Performed code self-review
[X] Rebased to the last commit of the target branch (or merged it into my branch)
[X] Documented the changes
[X] Linked the issues which this PR resolves
[X] Checked the TODO section in README.md if this PR resolves it
[X] Updated the tests
[X] All tests are passing - cargo test

Related: https://github.com/0xSpaceShard/starknet-devnet-rs/pull/298


operator joget-dx-operator-rhmp (0.0.31)

Re-triggered to fix a generalized index problem in OpenShift v4.11 and v4.12 marketplace bundle indexes - nothing specific to this operator, and the re-publish will have no effect on the operator, partner, or customers.

Re-triggering again to fix another index problem, similar to what John did 2 days ago. Again nothing specific to this operator.


Error: Cannot find module 'angular2-react-native/tools/copy-dependencies'

Followed this instruction:
git clone https://github.com/mlaval/angular2-react-native-seed.git
cd angular2-react-native-seed
npm install -g gulp react-native-cli typings
npm install
gulp init
This error occurs:
[14:13:02] Using gulpfile ~/Projects/experiments/react_tuts/angular2-react-native-seed/gulpfile.js
[14:13:02] Starting 'clean'...
[14:13:08] Finished 'clean' after 6.38 s
[14:13:08] Starting '!create'...
This will walk you through creating a new React Native project in /Users/devuser/Projects/experiments/react_tuts/angular2-react-native-seed/dist/ngRnSeed
Installing react-native package from npm...
Setting up new React Native app in /Users/devuser/Projects/experiments/react_tuts/angular2-react-native-seed/dist/ngRnSeed
<EMAIL_ADDRESS>/Users/devuser/Projects/experiments/react_tuts/angular2-react-native-seed/dist/ngRnSeed
└─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>[14:17:34] Finished '!create' after 4.42 min
[14:17:34] Starting 'init'...
[14:17:34] 'init' errored after 1.7 ms
[14:17:34] Error: Cannot find module 'angular2-react-native/tools/copy-dependencies'
Any thoughts?

You should have this file ./node_modules/angular2-react-native/tools/copy-dependencies.js. Otherwise, it seems that something went wrong during the npm install.
I just tried the instructions again it and it was successful.

Ok, started from scratch again. Followed your instruction again.
$ git clone https://github.com/mlaval/angular2-react-native-seed.git
$ cd angular2-react-native-seed
$ npm install -g gulp react-native-cli typings
...
/usr/local/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>$ npm install
npm ERR! cb() never called!

Reinstalled node, now it works. Sorry I got you bothered with my node problems.

It seems that you were facing this issue: https://github.com/angular/angular/issues/8053
Moving to the latest node was the right thing to do.


Log issues to the Vite overlay

Hello,
Any plans or desire to support logging ESLint issues to the Vite overlay in-browser?
Cheers!

Hello!
I don't plan to support this as I think a linter should not bother you when iterating and trying hacky things to see if it works. It should only be something that helps you produce good and standard production code.
For the same reason the default of the plugin is to use one line console logs, and all rules are set to warning in our config (but we use --max-warnings 0 in CI)


CMakeLists: add rules for installation

This commit adds some CMake rules that allow to do a "make install" to
install the zxing binary, its library and corresponding header files.
Signed-off-by: Thomas Petazzoni<EMAIL_ADDRESS>

Thanks mate nice one


Reading binary UInt is broken for values requiring exactly 32 bits

Discovered while working on #209, part of #205.
Javascript's bitwise operators convert their operands into 32-bit signed integers. This means that we can only store unsigned values using up to 31 bits (inclusive) in each number value.
ParserBinaryRaw::load_value will call readUnsignedLongInt() if the _len value is > 4, which covers all numbers needing more than 32 bits. We need a special case for reading unsigned integer values where _len is 4, but the first byte (which we can peek() at before officially reading it) is >127, requiring the highest bit to be set. This will allow us to call readUnsignedLongInt() on values which require exactly 32 bits.

We believe this is fixed, closing

Closed by #355.


Refactor Save and Open functions

Save and Open a closely tied together and I therefore think they need to be refactored together.
There are some fundamental flaw in the way I have made the structure of the two methods, which them more convoluted than they need to be (or I think that is the case). I think they need to be rethought from the ground up.
We need to have backwards compatibility in mind, so users saved files doesn't break. I think this can be done by the next release being able to convert from the old structure to the new and then the next release after that only using the new save file structure.

closed by #84 #83


IndexOutOfRangeException for importing mujoco_menagerie models into Unity

IndexOutOfRangeException: The mesh exceeds the number of vertices per mesh allowed by Unity.<PHONE_NUMBER> > 65535)
Hi,
I'm using MuJoCo with unity plugin. And trying to import the mujoco_menagerie models.
However Unity won't let me do it,  and throws out following error message:
IndexOutOfRangeException: The mesh exceeds the number of vertices per mesh allowed by Unity.<PHONE_NUMBER> > 65535) Mujoco.StlMeshParser.ParseBinary (System.Byte[] stlFileContents, UnityEngine.Vector3 scale) (at /Users/shreyas/mujoco/unity/Editor/Importer/StlMeshParser.cs:64) Mujoco.MjImporterWithAssets.CopyMeshAndRescale (System.String sourceFilePath, System.String targetFilePath, UnityEngine.Vector3 scale) (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterWithAssets.cs:181) Mujoco.MjImporterWithAssets.ParseMesh (System.Xml.XmlElement parentNode) (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterWithAssets.cs:160) Mujoco.MjImporterWithAssets.ParseAssets (System.Xml.XmlElement parentNode) (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterWithAssets.cs:125) Mujoco.MjImporterWithAssets.ParseRoot (UnityEngine.GameObject parentObject, System.Xml.XmlElement parentNode) (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterWithAssets.cs:88) Mujoco.MjcfImporter.ImportXml (System.Xml.XmlDocument mjcfXml, System.String name) (at /Users/shreyas/mujoco/unity/Runtime/Importer/MjcfImporter.cs:100) UnityEngine.Debug:LogException(Exception) Mujoco.MjcfImporter:ImportXml(XmlDocument, String) (at /Users/shreyas/mujoco/unity/Runtime/Importer/MjcfImporter.cs:104) Mujoco.MjImporterWithAssets:ImportString(String, String, String) (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterWithAssets.cs:74) Mujoco.MjImporterWithAssets:ImportFile(String) (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterWithAssets.cs:59) Mujoco.MjImporterEditorWindow:Apply() (at /Users/shreyas/mujoco/unity/Editor/Importer/MjImporterEditorWindow.cs:28)
Is there a workaround for this? Thanks a lot!

No Menagerie model has 2.7B vertices, this looks like a Unity-specific bug.

I'm suspecting some sort of underflow.
@shreyasgite, can you specify which model you are trying to import?
Also, Unity version, MuJoCo version, operating system, generally stuff required to reproduce.

Sorry about not giving this info before. Thanks Kevin.
I am trying to import this into Unity to create simple robot learning sims from computer vision perspective.
The above error happens with all the mujoco_menagerie  (Cassie, anymal, panda, unitree) models. But the import works for standard models form Mujoco library (humanoid, humanoid 100, tendon arm etc)
OS: macOS Monterey 12.6
Chip: Apple M1 Max
Unity: 2021.3.10f1
Mujoco: 2.2.2

@pkrack Thank you very much! I'm able to load panda.xml by following your steps.

@pkrack thank you, I've done everything you've said, but still something goes wrong.
OS: MacOS Ventura 13.3.1
CPU: Intel Core i5
Unity: 2021.3.23f1
MuJoCo: 2.3.5
When I press "Play" there is an error:
Error:
NullReferenceException: Failed to create Mujoco runtime.
Mujoco.MjScene.StepScene () (at /Users/akvelon/Desktop/mujoco-main/unity/Runtime/Components/MjScene.cs:345)
Mujoco.MjScene.FixedUpdate () (at /Users/akvelon/Desktop/mujoco-main/unity/Runtime/Components/MjScene.cs:98)
https://images2.imgbox.com/a8/cf/ZUR6m4fO_o.png
could you please suggest something?
Thanks


Create an entity type creation dialog flow that embeds a regex and jsonpath playground

Mockup
Divided areas are expandable with arrow buttons. Regex and JSONPath text area's auto-grow. Option to load signal data from prior signals or just put any arbitrary json blob. Highlight matching for values that match the supplied entity type pair (jsonpath +/- regex).
Retrieve list of all signal instances over last 30 days and run javascript version of find_entities function that checks the entity type pair supplied in realtime as the user updates the regex or jsonpath text area. Matches are returned and update the playground JSON object with any matches via CSS highlighting.

Create Dialog

MVP
The MVP here is everything minus the right hand side JSON playground. We should still check for matches with the supplied values and display a count + table of matches.

Abandoning for new PR


Configuring Table of content title translation (i18n)

What version of Hugo are you using (hugo version)?

$ hugo version
 v0.101.0

Does this issue reproduce with the latest release?
it does

It seems that there is no way to translate the TOC title. It stays at "Table of contents", when I'd like it to be "Sommaire" in another language. The documentation is unclear on this. I did not find any information either in markup sections or multilingual configuration.
I did find how startLevel, endLevel, and ordered options are used here:
https://github.com/gohugoio/hugo/blob/a7e3da242f98d4799dad013d7ba2f285717640d6/markup/tableofcontents/tableofcontents.go
but the string rendering starts at "<nav id" and the title appears just before that in the HTML.
Can I configure it, and how?

Given this markdown:
## Heading Level 2

### Heading Level 3

Hugo renders .TableOfContents to:

HL2

HL3

Hugo does not render a title for the TOC. The issue you are experiencing is related to your templates or theme.
Please use the forum (https://discourse.gohugo.io/) for questions and troubleshooting. We prefer to use GitHub for verified bugs and vetted enhancements. Thanks.


chore(codegen): print typescript code for typescript files

#3515  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite


Provide module CSS support

Can use zx script like this:
https://github.com/designervoid/ton-design-system/issues/7#issuecomment-1218063802

Modular css can be integrated externally


Implementing organisations list as a solr multivalued field

I had been trying to replicate our existing implementing organisation filter API call using solr but, I realised that is not possible because, when I am checking against participating organisation field, it does not really say if an organisation has an implementing role or some other role.
To handle this situation, it will be best if you can introduce a new multi valued field (can be called implementing_org_ref) for an activity that will list down only those participating organsiation refs (participating_org_ref) that are playing the implementing role.

Hey Rumy, thanks for the issue.
Objective sounds good!
Tip/request for this and future issues: If you can, leave a url to the old call on any issues so they are easier to track down!

Hi Sylvan,
Here's the API call that provides us with the implementing organisations list in the current live OIPA.
https://devtracker-entry.oipa.nl/api/activities/aggregations/?format=json&group_by=participating_organisation&aggregations=count&reporting_organisation_identifier=GB-GOV-1&recipient_country=BD&hierarchy=1

Although, the API call does say that the group by is used on participating organisations, but, can you please confirm if this call actually returns a proper implementing organisations list or just the participating organisations list? Because, we have been using it from the very beginning to show our implementing organisations list on our search and projects page.

ping @sylvanr


Not validating the first time we load a model

I have put all my validations on model. The very first time that I load the page some of the fields are empty but I dont want that page to load with the error message on that (currently as soon as I load the page it apears with error msg). Has anyone have any idea what I should do?

You can take a look on how it's done in the demo app via the validated-input component

I have those already but still not sure how I can use it on template to hide it as the page loads and shows when some activity has been done on the page? (clicking the save button?)

You should use a form component which shows validation errors only if there has been user interaction. There are several out in the wild. Have a look at https://emberobserver.com/categories/forms.

@nbehboudi what you are looking for is not in the scope of this addon. There are many projects out there (as @jelhan noted) that can help you with what you are looking for. Im gonna go ahead and close this issue. Feel free to reopen if anything else comes up.


Guidelines implementation: Props

[ ] Implement and apply rule awslint:props-no-property: Constructs should not have a "props" property that exposes their init props
[ ] Implement and apply rule awslint:props-coverage: Props should expose the entire surface area of the AWS resource

[ ] Step functions
[ ] ECS
[ ] AutoScaling

[ ] Implement and apply rule awslint:props-no-arn-refs: Props should use strong types instead of attributres (forbid props with "arn" suffix)
[ ] Implement and apply rule awslint:props-no-cfn-types: Props should not expose L1 types (types which start with "Cfn")
[ ] Implement and apply rule awslint:props-no-tokens: Props should not use the "Token" type
[ ] Implement and apply rule awslint:props-decdk: Props should be deCDK compatible (see doc)
[ ] Implement and apply rule awslint:props-default-doc: All optional props should have a @default documentation and specify "-" as the default value
[ ] Implement and apply rule awslint:props-flat: Prop types should not be structs to encourage flat set of props
[ ] Implement and apply rule awslint:props-no-unions: Props should not use TypeScript unions

Split into multiple issues


CITZ ADM Procument Options Decision Note

Describe the issue
Develop a note for the ADMs around the procurement options for the MCS pilot.
Which Sprint Priority is this issue related to?
2
Additional context
Add any other context, attachments or screenshots
Definition of done

Completed decision note reviewed by Gen and Paul

Emily: Still unsure of the priority on this one.
@scayzer please confirm if otherwise
We will close this for now and reopen if it's needed later.


Adding initial unit tests for jaeger agent using udp latest

@saktib there are 2 TODOs around commented out code (one was there before, one I caused) that probably need looking at (I'll mark them with comments too)
Also do you know what the CodecUtility file is for? I couldn't seem to find any usages of it (I'll add a comment on the OG PR in the upstream repo too so I don't forget about this later)

These are the remaining areas possibly needing test coverage. Mostly input validation and error handling from what I could tell
AgentExporter

Input validation in the constructor (the 2 exception throwing lines)

JaegerTransport

Line 70
Line 93 - 96 - error handling

ThriftUdpTransport

Lines 28 - 33 - error handling
Unused methods that need to be defined for the interface (isOpen, open, read)
Line 63 - error handling
Lines 82-84 - error handling


Can't connect to localhost:1025

Hi.
I have tried to use MailHog which seems promising.
However, after running the binary (on Windows 7), I can access the Web UI but my php application returns that it can't connect to localhost:1025 (timed out).
I am using swiftmailer with the following config:
swiftmailer:
    transport:      smtp
    username:       testuser
    password:       testpwd
    host:           <IP_ADDRESS>
    port:           1025
    encryption:     null
    auth_mode:      plain

Am i missing something?
Thanks in advance

Hi @vialcollet
Are any errors output when you start MailHog? If it can't bind to the specified port it should output something similar to these:
[SMTP] Error binding to address :25: listen tcp :25: bind: permission denied
[SMTP] Error binding to address :1025: listen tcp :1025: bind: address already in use

I'll try to reproduce the problem on Windows 7 when I get a chance, but it should just work :smile:

Same here. It hangs connecting to localhost:1025 but Web UI seems to be fine

I have the same problem, I can connect to the ui (but from outside the host) but I can't use mailhog from the same host

I'm using the following setting with swiftmailer and it works:
swiftmailer:
    transport: "%mailer_transport%"
    host:      "%mailer_host%"
    username:  "%mailer_user%"
    password:  "%mailer_password%"
    auth_mode: "%mailer_auth_mode%"
    port:      "%mailer_port%"
    spool:     { type: memory }

parameters:
    mailer_transport: smtp
    mailer_host: <IP_ADDRESS>
    mailer_user: null
    mailer_password: null
    mailer_auth_mode: null
    mailer_port: 1025

I had the same issue and got it to work using @n3o77 's example.
The problem was that the config.yml file is missing the port: '%mailer_port%' directive so setting mailer_port in parameters.yml has no effect.
Personally I found that the auth_mode bit isn't needed.

Closing based on y2keepers comment


Add support for "_is" identity comparison operator in Majestik language

Summary by CodeRabbit

New Features

Introduced the IdentityExpressionNode class for handling identity expressions.
Added support for equality expressions in the MajestikCodeVisitor.
New utility method is in MagikObjectUtils for comparing objects.

Bug Fixes

Improved handling of null values in visitIfExpression.

Tests

Added unit tests for IdentityExpressionNode and MagikObjectUtils.
New test cases for identity_comparison.magik in the test suite, including identity comparison using the _is operator.

Chores

Minor formatting adjustments for consistency in method type descriptors and existing test cases.

@coderabbitai resolve


Fixed some errors in prostate zone masks

Updated all the masks in which I was able to find errors. The issue was discussed here.
There is also a .csv file attached that logs all the changes.

Thank you for noticing the problem and providing a fix. Greatly appreciated!


UI shows timestamps different than CLI

CLI uses UTC. We should be consistent with CLI but in UI allow timestamps to be displayed in local time by clicking some button.
reported by: @skarmark on IRC

Agreed. Should default to UTC everywhere, optionally allowing user to select a local time zone (not a big priority right now).

I'm sorry, but I'm unable to reproduce the original problem. As far as I can tell, timestamp representation is consistent between UI and CLI.
enykeev@st2stage201:~$ st2 run core.local "date -u"
.
id: 5582ba96ce36d21702e3a4e4
status: succeeded
result:
{
    "failed": false,
    "stderr": "",
    "return_code": 0,
    "succeeded": true,
    "stdout": "Thu Jun 18 12:33:26 UTC 2015
"
}

The second execution, 5582bae2ce36d21702e3a4e6, was created a minute later in UI. The history records of both executions are:
enykeev@st2stage201:~$ st2 execution list -n 2
+--------------------------+------------+--------------+-----------+---------------------------+---------------------------+
| id                       | action.ref | context.user | status    | start_timestamp           | end_timestamp             |
+--------------------------+------------+--------------+-----------+---------------------------+---------------------------+
| 5582ba96ce36d21702e3a4e4 | core.local | testu        | succeeded | Thu, 18 Jun 2015 12:33:26 | Thu, 18 Jun 2015 12:33:26 |
|                          |            |              |           | UTC                       | UTC                       |
| 5582bae2ce36d21702e3a4e6 | core.local | testu        | succeeded | Thu, 18 Jun 2015 12:34:42 | Thu, 18 Jun 2015 12:34:42 |
|                          |            |              |           | UTC                       | UTC                       |
+--------------------------+------------+--------------+-----------+---------------------------+---------------------------+

As for default TZ to show, I would suggest instead make both UI and CLI to show timestamps in local TZ with an ability to switch to UTC on request. It just seem more natural to me.

@enykeev This might be more natural, but I think it's actually better if it defaults to UTC so by default, everyone sees the same outputs.
That's important if you for debugging / troubleshooting and if you share output with someone. It results in less surprises.

@Kami can you please validate that it's no longer issue?

I checked and verified that UI now correctly displays timestamps in UTC.

https://stackstorm.slack.com/archives/community/p1474054719001775

there should be second option to use the server time too.
People who have servers in different zones, need to follow three timezones.
Pretty troublesome...

I'm going to close the issue for now. The problem stated in the title does not exist. Selecting a custom TZ is better handled as a part of https://github.com/StackStorm/st2web/issues/292.


Delete jobs tables after deleting resource

How do I delete the jobs database/schema once I created it? I deleted the resource in the Azure portal, but the tables are still in the database?

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

ID: 386f5b1f-e835-0a3b-8e3f-302a6bf5670e
Version Independent ID: 2470305f-0366-36ba-5365-adba61dd02b2
Content: Create and manage Elastic Database Jobs (preview) with Transact-SQL (T-SQL) - Azure SQL Database
Content Source: articles/azure-sql/database/elastic-jobs-tsql-create-manage.md
Service: sql-database
Sub-service: scale-out
GitHub Login: @jaredmoo
Microsoft Alias: jaredmoo

@Thank you for the feedback. We are actively investigating and will get back to you soon.

@bmeverett There are 2 databases that are in use when creating the agents. First is the target database where in we need to execute the scripts, and second one is the job database which we configure the elastic jobs on.
Running below command will only clean up the job history and not the job objects:
EXEC jobs.sp_delete_job @job_name='ResultsPoolsJob'
For complete cleanup, we need to delete the resource (job database) as well from Azure which in turn will delete all the objects on the database. We as well need to delete the Elastic Job that was created. This will take care of deleting all the resources
Again target database can also be cleaned up if needed.

@bmeverett We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


How to detect wrong values provided by Orakl Network data source(s)?

will be reopened if occurred again