Settings Plugin for Xamarin doesnt save key value in the first run

i developing a app that in the first run get a userid from the server. i want to store it locally, however i cant do it on android. i save the value and then try to get it and the value doesnt exists.
if i close the app and open it again the value is stored.
In IOS works fine. Any idea?

Do you have the code snippet where you are saving it?
Your get/set?

My Settings class:
private static readonly string SettingsDefault = string.Empty;
public static string GetSettings(string key)
{
return AppSettings.GetValueOrDefault(key, SettingsDefault);
}
public static void SetSettings(string key, string value)
{
AppSettings.AddOrUpdateValue(key, value);
}
My set:
Helpers.Settings.SetSettings("followerId", response["followerId"].Value());
(i confirm that my response as the followerid value)
my get:
Helpers.Settings.GetSettings("followerId")
I try to define the key in the settings class and problem still happens.
Could this be a problem with the Android Context?

When i do:

When i do:
Helpers.Settings.GeneralSettings = "12343241234321";
string b = Helpers.Settings.GeneralSettings;
in the same page works fine.
When i do the set in one page and the set in another, the string is empty

You can try 1.5.1 which I just pushed. Are you able to do this:
string test = response["followerId"].Value();
Helpers.Settings.SetSettings("followerId", test);

James, thanks for the quick response.
I think the issue is fixed.
I updated to 1.5.1 and tested the code snippet suggested, and works.
I'll do more tests to be sure.
I tested with:
Nexus 5 - (android 5.1)
Galaxy Nexus - (android 4.0.3)
IPad (4th Gen) - (IOS 8.3)
IPhone 4 - (IOS 7.1.1)
Iphonw 6 - (Emulator - IOS 8.3)

This is great to hear thanks!


Revert win2022 pool back to win2019

Contributing to the Azure SDK
Fixed the failure of pipeline: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2080863&view=results
Testing pipeline: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2081441&view=results
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

As discussed the plan is to update the storage matrix/pool/replace statements rather than reverting the global net matrix.


Bump up cluster nodes for pilot e2e tests on prow

The pilot e2e tests appear to be extremely flaky and take significantly
longer with a single node.  Bumping up to 4 seems to vastly greatly
things.

Prow itself should just be scheduling not running tests ? Cc @selmanj

Yes @ldemailly is right. Prow is only scheduling and reporting. The actual tests run on GKE cluster istio-e2e-rbac-rotation-1 in istio-testing project. It might be due to the fact that this cluster is shared for presubmit so that multiple pending PRs running tests at the same time might slow down the tests. I know @sebastienvas has been working on resource pool prepromisioning and we are dogfooding it on postsubmit right now. It provides hermetic environments on project level and should help to speed up this test. Perhaps Seb could add more details too.

@chxchx I don't think that's true.  In the log for a failed test, I saw:
W0221 16:41:39.977] + echo 'Default cluster version: 1.9.2-gke.1'
W0221 16:41:39.977] + gcloud container clusters create e2e-pilot-4bcd7d39 --zone us-central1-f --project istio-testing --cluster-version 1.9.2-gke.1 --machine-type n1-standard-4 --num-nodes 1 --no-enable-legacy-authorization --enable-kubernetes-alpha --quiet

This implies that it's using a cluster from create_cluster().  Notice that it only has a single node.

cc @sebastienvas @hklai

You're right @nmittler, your change will improve run time. I'll be working on updating this to take advantage of the pre-provisioning shortly after I am done with postsubmit.

Now I see what you mean. Sorry I was confused by the PR title and thought this were to change the Prow cluster node numbers. The pilot e2e test creates e2e-pilot-4bcd7d39 cluster and runs on it. By adding more nodes like you did in this PR it should mitigate this problem. In fact this PR is the only one that passes pilot e2e test in the last 20 runs or so.

Also the error seems unrelated:
Makefile:412: recipe for target 'mixer-test' failed

@chxchx

In fact this PR is the only one that passes pilot e2e test in the last 20 runs or so.

ha! We've all been feeling the pain of these tests ... it's now completing in ~ 20 min, which includes a complete cluster spin-up.  So, not too shabby .... and passing the tests is good too :P

Does anyone have any issues with forcing the merge? As I stated above, I don't think the error is related.

I was confused too by "prow cluster" but you mean indeed the target clusters
I didn't know those were dynamically created, I thought it was either a pool or shared
but
/lgtm

@ldemailly, @chxchx
I've updated the title to (hopefully) be more clear.  Also, I've changed the scripts around so that only the pilot e2e tests will use 4 nodes. PTAL

/lgtm

shouldn't it be multi nodes for everyone - which test will work well on 1 node ?

@nmittler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
8ed2743ce805fa9c48ecba545047222ae1332d99
link
/test e2e-simple

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

@ldemailly

shouldn't it be multi nodes for everyone - which test will work well on 1 node ?

I tried that, but the mixer tests were failing for some reason, so I changed it so that only pilot e2e is impacted. Didn't want to investigate - especially given that everything should be migrating to the pooled clusters eventually.

/test e2e-simple


Cross-test with Scala 3

Remaining issues

[ ] Fix compilation of http4s-server tests. There seems to be an issue in the usage of the Akka-http graph DSL in Scala 3. We use it to convert between Akka-streams and http4s streams.
[ ] Compilation of openapi module loops indefinitely

Relates to #566

Any help to move forward would be appreciated, I don’t have time to look at the remaining issues.

I do not have permissions to push to your branch, so here is commit to my fork that fixes http4s-server test compilation - https://github.com/ptrdom/endpoints4s/commit/d3e4f09a9d7785965d65233eff90b00d68c7438e.

And this fixes openapi tests - https://github.com/ptrdom/endpoints4s/commit/f3b2b4d9829fe4960e0698a756218186fe640905.

Remaining Scala 3 errors are all in example code, at a quick glance they do look like issues in build definition.

You will probably notice, but just a note that you might not want to take the following change from my commits -
https://github.com/ptrdom/endpoints4s/commit/d3e4f09a9d7785965d65233eff90b00d68c7438e#diff-993fc08217429dd4f90458a62bd3f0c461dd5c8c8031173e323d6fa94e95f21bR67-R70.

Thank you very much @ptrdom!

I do not have permissions to push to your branch, so here is commit to my fork that fixes http4s-server test compilation - ptrdom@d3e4f09.

IIUC we only need to bump akka and use createGraph instead of create? Nice!

And this fixes openapi tests - ptrdom@f3b2b4d.

Interesting! How did you figure out the cause of the issue? I wonder if there is a bug in Scala 3 or if this is expected.

IIUC we only need to bump akka and use createGraph instead of create? Nice!

Yeah, I noticed a comment about it in the code of the first Akka release for Scala 3 - figured it was very likely they ran into this issue too.

Interesting! How did you figure out the cause of the issue? I wonder if there is a bug in Scala 3 or if this is expected.

I just noticed that the class in error had private implicits in inner class, changed that - did not seem to help, then I moved the inner class definitions out to the companion object and that did it.
I see we have more errors now, will check them out.

Ah, about f3b2b4d, I think this is related to a limitation that has been fixed in lampepfl/dotty#15847, which has been shipped in Scala 3.2.0. We could also try to bump the Scala 3 version to 3.2.1, maybe?

Ah, about f3b2b4d, I think this is related to a limitation that has been fixed in lampepfl/dotty#15847, which has been shipped in Scala 3.2.0. We could also try to bump the Scala 3 version to 3.2.1, maybe?

Neat, will try it out!

~We could also try to bump the Scala 3 version to 3.2.1, maybe?~ (actually, I am fine with your change)
All good. That change does improve the code quality, so to speak - those definitions do not benefit from being inside the class, so might as well be in the companion object.

That’s amazing, thank you @ptrdom!


allow use with<EMAIL_ADDRESS>

address #16

:+1:

+1

@mathieumg @wiredmax you can use npm install --save-dev iam4x/react-a11y#packaged as workaround until this is merged ;)

:+1:


Link broken/missing

On this page, a link is given to:
https://github.com/uber/streetscape.gl/tree/master/examples/get-started/log-from-stream.js
I'm getting an error when I try to access it

fixing


[TACHYON-564] Improve for loop style in Common Utils

https://tachyon.atlassian.net/browse/TACHYON-564

Jenkins, test this please.

+1 LGTM

+1 LGTM. Thanks for the patch!


In react-router 6 Switch has been removed and using Route would give a Runtime error as "Route child should be

Issue

Please provide us with the following information:

This issue is for the sample

    - [ ] 1-1) Sign-in with Azure AD
    - [ ] 1-2) Sign-in with Azure AD B2C
    - [ ] 2-1) Acquire a Token and call Microsoft Graph
    - [ ] 3-1) Protect and call a web API on Azure AD
    - [ ] 3-2) Protect and call a web API on Azure AD B2C
    - [ ] 4-1) Deploy to Azure Storage and App Service
    - [ ] 4-2) Deploy to Azure Static App Service
    - [x] 5-1) Call a web API using App Roles
    - [ ] 5-2) Call a web API using Security Groups
    - [ ] 6-1) Call Microsoft Graph using on-behalf-of flow
    - [ ] 6-2) Call a web API using Proof of Possession tokens
    - [ ] 6-3) Call a web API using Conditional Access Auth Context

This issue is for a

    - [ ] bug report -> please search issues before submitting
    - [ ] question
    - [ ] feature request
    - [ ] documentation issue or request

Minimal steps to reproduce
Upgrade react-router-dom to ^6.0.2
Any log messages given by the failure
Attempted import error: 'Switch' is not exported from 'react-router-dom'.
Expected/desired behavior
Working with Routes
Library version

Browser and version

Chrome, Edge, Firefox, Safari?

Mention any other details that might be useful

Thanks! We'll be in touch soon.

If I try replacing Switch with Routes, at run-time it would give:
Error: [RouteGuard] is not a  component. All component children of  must be a  or <React.Fragment>

Hi @mkokabi , thanks for reaching out. The sample supports React Router version 5, and that's why you're getting the error in version 6, but If you want to migrate from React Router version 5  to version 6, please take a look at the React  Router documentation.

Hi @salman90 ,
I know how to upgrade from Router version 5 to 6 but the issue is the architecture of this project is based on creating a RouteGuard class and using it instead of Route and in version 6 this is not supported. The error in version 6 is clearly saying all children of Routes must be a Route or React.Fragment.

@mkokabi Thanks for the feedback and clarification. I added your request to support react-router version 6 in the samples that use the RouteGuard class to our backlog, and we will update the samples as soon as possible.


Suggestion: Create dynamic classes

Hi,
I think in some cases it would be great to be able to show/hide only with css classes. This could be useful when we need to show hide lots of small elements as it will reduce the number of bindings in the page.
We could use a directive to enable the feature, which would great css classes based on breakpoints defined in the service and insert them in the dom.
Please tell me if you agree to include this before I start working on it.
You've done a great job with all new features and refactoring !!! Amazing.
Thanks.

Hello,
It would be great, working on it. And if you have any questions, I communicate it. When you have it you do a pull request. ; )
I am creating a directive in which you can select multiple options: responsiveShowIt = {device: 'mobile', os:'ios', browser: 'safari' ....}; and so we replace the specific directives and also working on the demo page. : )
Thank you!

Hi,
Ok I'll work on the css classes generation later today.

OK great!

I committed a first version which create show/hide classes for xs to xl but I don't like the way it's done.
Maybe you could help me figure out how to improve this. I have mainly 2 points I would like to change:

I would like to inject code into head instead of inside the component. But I don't know how to do this in an angular2 way. I mean I don't think I should use plain DOM javascript (document.getElement... ), but I don't know how to achieve this with ElementRef or any other angular2 tools. Do you have an idea ?

I would also want to be able to change the selector to "body[resonsiveCss]" but as it's outside of the bootstrapped application component I don't know how to do this. Do you know if it's possible to use component outside of the main component ? (I'm not sure if my question is clear).

An other way to do this might be to implement the code straight into the service, and in the ngOnInit process I could check if the body element has an "resonsiveCss" attribute, if it does then I could inject the style code. But I'm not sure this is very angular2 way.

Thanks for you feedback.

Hi,
I forgot to mention I have created a new branch for this feature.

We could try to make this directive in two options: What we've done is great and think that the styles are completely isolated in components and for certain things can be good.

Here is an idea in javascript but not very Angular 2 philosophy. This would be an option to inject CSS code directly in the head.

load-css-external-to-component
I seek solutions between the two have if we achieve something. :)
Do not worry about the new branch you can create all you need. ; )


FLAGS_compression_parallel_threads of IAA

I found a parameter called FLAGS_compression_parallel_threads in db_bench_tool. When using IAA as the compression type, setting FLAGS_compression_parallel_threads to a value greater than 1 significantly improves compression performance.
I would like to ask:

Does this parameter take advantage of IAA's multi-threading capabilities?
Is the performance improvement related to optimizations in IAA, or is it simply due to the benefit of multi-threading?

My CPU is: Intel(R) Xeon(R) Gold 5420+

Hi. That option enables multiple threads for compression. So, there will be multiple threads submitting compression jobs to IAA, allowing higher utilization of the devices.

Thank you for your reply.
In addition, I would like to ask two more questions:
Parallelism of IAA: How is the parallelism of IAA? How can I view its maximum parallelism? Is there any relevant documentation to support it?
Specifying IAA compression method: How can I specify IAA as the compression method in the compression_per_level setting? I have tried using kPluginCompression, but when running it, the error appears: open error: Invalid argument: Compression type is invalid.
Thank you for your assistance!

The IAA spec has more information on the architecture. There is a link on the IAA page.
To set IAA compression per level, you can use the compressor_per_level option added in PR 6717. It takes a vector of compressors (colon-separated). For each compressor, the syntax is the same as when setting the compressor option. For example:
compressor_per_level={{id=Compressor1;opt=value;}:{id=Compressor2;opt=value;}}
There is an example in the unit tests. We will add an example in the PR description and eventually docs.

Thanks for your answer!
I found that when db_bench is set with the following parameters: --compression_type=com.intel.iaa_compressor_rocksdb compressor_options="execution_path=hw;compression_mode=fixed", FLAGS_compressor will be constructed as an IAA Compressor, so we can directly set it using:
open_options_.compressor_per_level.push_back(FLAGS_compressor);

For other BuiltinCompressor, such as Snappy, you can use:
auto snappy_compressor = BuiltinCompressor::GetCompressor(kSnappyCompression);
open_options_.compressor_per_level.push_back(snappy_compressor);

This makes the code more intuitive and straightforward.

I find the flag QPL_FLAG_ZLIB_MODE suppots adding ZLIB header and trailer information to raw Deflate stream from IAA.
Does it means the IAA compressed SSTable can be decompressed by ZLIB on CPU?
How should I configure the compressor_per_level to support it?
Thank you!

I try to call ZLIB uncompressor for kPluginCompression SSTable, but it comes to a segment fault.

Can you give me some advice?
Thank you!

I find the flag QPL_FLAG_ZLIB_MODE suppots adding ZLIB header and trailer information to raw Deflate stream from IAA. Does it means the IAA compressed SSTable can be decompressed by ZLIB on CPU? How should I configure the compressor_per_level to support it?
Thank you!

Both the zlib library and QPL support 3 formats: deflate raw (no header/trailer), zlib (raw + zlib header/trailer), and gzip (raw + gzip header/trailer).
The IAA-compressed blocks can technically be decompressed by zlib, but that selection is not currently possible in RocksDB. The compressed blocks have compression type id set to the IAA plugin, and RocksDB will use the corresponding Compressor to decompress the blocks (the zlib Compressor has a different id). Compatibility across compressors is currently not supported, although we are looking into options to enable this (e.g., have an accelerated version of the zlib Compressor).
compressor_per_level won't achieve that. Data may be decompressed with zlib at one level and recompressed with IAA at the next level or vice versa. However, the data at a certain level will always be handled by the same Compressor for compression and decompression.

I try to call ZLIB uncompressor for kPluginCompression SSTable, but it comes to a segment fault.  Can you give me some advice?
Thank you!

Are you trying to force the decompressor to be ZlibCompressor instead of IAACompressor for blocks compressed by IAA? That may work as proof of concept, but I'm not sure. Let me think about it.


Correct typo in README.md

Change accomplish a specific object to accomplish a specific objective.

thank you @keithmccammon ! I hope we can catch up soon :)


Enrich notification with error information if edc communication runs into an error

As ... a test manager
I want ... to ensure that error information regarding edc communication is not lost
so that ... our end to end tests can retry and the users are informed.
Hints / Details

Currently our E2E Test are somethings failing because the edc can not handle that many requests at once.
If we would enrich the error into the notification we could adapt the e2e test to retry, which would cause the test to pass.

Acceptance Criteria

E2E Tests do not fail because of an edc exception has been thrown.
E2E Tests retry to send notification after a error has occured
Frontend displays the error

@ds-mwesener I dont get the description. Isn't the behaviour to be tested the one with an working EDC?

Hi @jzbmw that is correct. Unfortunately sometimes the edc fails while trying to reach the other edc while sending notifications or having contract negotiations.
If this happens our tests will fail. Even though it would be successful in a second try.
For our system to know that there was an error we need to persist that information into our system to be able to perform a retry.
Hope this clarifies it a bit better.

Successfully tested on E2E.
In the frontend the error message looks like this still:

In the backend the following happens:

And the error message field is filled with information about the error:

E2E tests work properly now.
Frontend display of the error message will be implemented in a different story.

@ds-crehm please refer to the story mentioned

During testing we noticed that the corner case of a user sending a notification to his own BPN is not covered in Trace-X yet.
I created the following ticket for it: https://github.com/eclipse-tractusx/traceability-foss/issues/783


PowerPC mac99 doesn't seem load -prom-env options

Describe the issue
I've managed to install Yellow Dog Linux 2.3 following these intructions: https://eaasi.gitlab.io/program_docs/qemu-qed/recipes/yellowdoglinux/#installation.
Now my problem is that the openbios doesn't seem to load -prom-env set variables as it defaults to looking for a bootloader on the cdrom.

Here are the 2 -prom-env options I've set:
-prom-env "boot-device=hd:,\yaboot"
-prom-env "boot-args=conf=hd:,\yaboot.conf"
Therefore I'm not able to boot the OS through the yaboot powerpc bootloader.
Configuration

UTM Version: 4.5.3 (99)
macOS Version: 14.5 (23F79)
Mac Chip (Intel, M1, ...): M1

Debug log
The Export Log... button stays greyed-out.
Upload VM
config.plist.zip

bootindex seem to override -prom-env "boot-device=hd:,\yaboot". We should provide a way to disable bootindex.


What is prefix??

what is prefix and where i can change it?

m! and src/strucutures/MiyakoClient.js


sparse_set_iterator::operator[] causes warnings when used with size_t

https://github.com/skypjack/entt/blob/df87206c940706ed6d06e9e4763628bed075a9cc/src/entt/entity/sparse_set.hpp#L75
Because this function is defined in terms of Container::difference_type, using Container::size_type causes a conversion warning. E.g. std::vector::size_type is std::size_t and std::vector::difference_type is std::ptrdiff_t
Shows up in clang-tidy warnings.
clang-tidy --use-color -p=build /home/runner/work/osp-magnum/osp-magnum/test/tasks/main.cpp
/home/runner/work/osp-magnum/osp-magnum/test/tasks/main.cpp:66:76: warning: narrowing conversion from 'unsigned long' to signed type 'entt::internal::sparse_set_iterator<std::vector<osp::TaskId>>::difference_type' (aka 'long') is implementation-defined [bugprone-narrowing-conversions]
Suppressed 47323 warnings (47310 in non-user code, 12 NOLINT, 1 with check filters).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
            TaskId const        randomTask  = rExec.tasksQueuedRun.begin()[rRand() % runTasksLeft];

Interesting. I've clang-tidy enabled and cannot reproduce it though.
Can I ask you what param triggers the error? Maybe it's not the only one.

It's this PR: https://github.com/TheOpenSpaceProgram/osp-magnum/pull/269/files , the file test/tasks/main.cpp which is the very bottom-most file in the review (at the time of writing).
The deprecation message says to switch from ::at(index) to ::begin()[index], but i was actually able to switch to simply ::operator[](index) directly.
However, originally (before i just switched to ::operator[](index))when i switched from ::at(index) to ::begin()[index] i got the warning from clang tidy i reported in my initial comment.
The types are:
'entt::internal::sparse_set_iterator<std::vector<osp::TaskId>>::difference_type', where osp::TaskId is an enum-class backed by a uint32_t.
https://github.com/TheOpenSpaceProgram/osp-magnum/blob/a747cab17ae3f5bea02a800c8b9cfbe8a1e345c3/src/osp/tasks/tasks.h#L67-L72
difference_type should be std::ptrdiff_t but apparently cpp reference thinks the only requirement is that it's a signed integer. Regardless, it's being calculated on my platform as long.
rRand is type std::mt19937 &rRand defined here: https://en.cppreference.com/w/cpp/numeric/random/mersenne_twister_engine
which apparently returns std::uint_fast32_t and i guess on the github actions runner is of type unsigned long.
runTasksLeft is auto const runTasksLeft = rExec.tasksQueuedRun.size(); which i imagine is std::size_t, though i'm not 100% sure what the type of tasksQueuedRun is off of the top of my head.
The full function is here:
https://github.com/TheOpenSpaceProgram/osp-magnum/blob/a747cab17ae3f5bea02a800c8b9cfbe8a1e345c3/test/tasks/main.cpp#L51-L73
since unsigned long can't convert to long without precision loss, we get the warning about narrowing conversions.
My recommendation is that you can either make the functions for sparse_set_iterator templated on the std::integral concept, or you can define two versions, one for signed integers and another for unsigned.
But don't fix this on my account, since ::operator[](index) works for me just fine. I just wanted to report the warning in case you wanted to fix it.

Ohoo, I see, so the error is at the call site in your case and not in the body of the function, is it?
I got it the other way around initially. This would explain why it doesn't trigger on the CI at least.

Do you mind sharing some of the random access iterators that you were looking at? I'm curious.

You can check a few implementations here (working draft for the language). Then I checked the MSVC one from within VS (just look into the implementation of a container like the vector) and the ones offered by clang (easily available online). 👍


CEO-351 Reorganize SVG Component

Purpose
Gets rid of components/SvgIcon.js in place of a components/svg folder where each SVG is its own file. SVGs are still used through either the SvgIcon or ButtonSvgIcon wrapper function which lives in components/svg/SvgIcon.js.
Related Issues
Closes CEO-351
Submission Checklist

[x] Included Jira issue in the PR title (e.g. CEO-### <title>)
[x] Code passes linter rules (npm run eslint/clj-kondo --lint src)

src/js/components/svg/helpIcon.js
7/8, no need for id
9, move d to its own line like other files.
9, no need for id
src/js/components/svg/leftDoubleIcon.js
11, you dont need a group if there is no styling
src/js/components/svg/plus.js
5, move d to its own line like other files.
src/js/components/svg/rightDoubleIcon.js
8, you dont need a group if there is no styling


Fix small error in the Timer documentation

Not sure if this is the correct branch for this change. The contrib section says that you should never PR against master, but I'm not sure if that applies for documentation as well.

Hi - documentation is generated automatically from the source code jsdoc blocks, so corrections like this one need to be made in the source, not the output (otherwise they are lost when we next export).


Modal has page elements over the top

I'm using Modal, but in one instance, (some) page elements are showing through the modal. It's like the z-index is not correct:

You can either set the zIndex prop on the component itself or control the stacking context with Relative components


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3022

Questions ?
Please refer to the Backport tool documentation


Python refactoring

In integration branch, refactor existing code.

One main.py that loads modules from the backend directory
Command line should rely on calling python main.py
All Python files in front-end should go to backend
All function have to be refactored into classes

It's done.

Merged with integration.


reference projects based on conditional compilation symbols provided

This change, along with the updated docs in https://github.com/shiftkey/Perspex/commit/954fe6da11ba57bbcc085727801c16fb8efbdab3 - let's me build the project in VS2015 without touching csproj files.
And of course, to make it crystal clear:

xref #402

Hmm, this doesn't appear to remove the references from the projects for me. However, on my machine it doesn't matter if the cairo/gtk packages are built or not - as long as nothing tries to load them the sample applications run fine.
Are you not seeing the same?

@grokys oops, apologies for missing this.
If we're talking about the references list in VS, then yes, this change doesn't make them go away. That's just how the list (doesn't) work, visually.

Apologies I never merged this, we were just about to change the way that stuff works anyway... For now I think we need to insist on gtk# being installed to build. Hopefully we can remove that once we move to skia#.


response_type 'id_token' is not enabled for the application

Core Library
@azure/msal-browser
Core Library Version
2.12.0
Wrapper Library
@azure/msal-react
Wrapper Library Version
1.0.0-beta.0
Description
I have created a new app registration and added on Authentication SPA and in Redirect URLS I have typed:
https://localhost:3000/
and my azure url
https://dev-luloui.azurewebsites.net/
The app is enabled multiple organizations, so anyone can try it.
However I get the error above in the title
response_type 'id_token' is not enabled for the application
libraries used:
"@azure/msal-browser": "^2.12.0",
"@azure/msal-react": "^1.0.0-beta.0",
MSAL Configuration
import { Configuration, PopupRequest } from "@azure/msal-browser";

// Config object to be passed to Msal on creation
export const msalConfig: Configuration = {
    auth: {
        clientId: "c1074030-bf88-4aa0-ad1a-6fd14b8462db",
        authority: 'https://login.microsoftonline.com/organizations/',
        redirectUri: "http://localhost:3000"
    }
};

// Add here scopes for id token to be used at MS Identity Platform endpoints.
export const loginRequest: PopupRequest = {
    scopes: ["User.Read"]
};

// Add here the endpoints for MS Graph API services you would like to use.
export const graphConfig = {
    graphMeEndpoint: "https://graph.microsoft.com/v1.0/me"
};

Relevant Code Snippets
No response
Identity Provider
Azure AD / MSA
Source
External (Customer)

@levalencia on Azure portal > app registration > your app > Authentication blade, please make sure the implicit flow is not checked:

none of the options are checked

Strange :/ Can you share your app manifest as well? Are you trying one of the samples (if so, can you mention which one?)

Sure

{
	"id": "af7a61f7-da54-445e-b099-c66fd09ca554",
	"acceptMappedClaims": null,
	"accessTokenAcceptedVersion": null,
	"addIns": [],
	"allowPublicClient": null,
	"appId": "c1074030-bf88-4aa0-ad1a-6fd14b8462db",
	"appRoles": [
		{
			"allowedMemberTypes": [
				"User"
			],
			"description": "Customers",
			"displayName": "Customers",
			"id": "f800d482-a0da-4d7c-a947-93e79e44dd69",
			"isEnabled": true,
			"lang": null,
			"origin": "Application",
			"value": "Customers"
		},
		{
			"allowedMemberTypes": [
				"User"
			],
			"description": "Gold",
			"displayName": "Gold",
			"id": "caa9f825-4ff2-4de9-9bc4-67e3146dea6d",
			"isEnabled": true,
			"lang": null,
			"origin": "Application",
			"value": "Gold"
		},
		{
			"allowedMemberTypes": [
				"User"
			],
			"description": "Silver",
			"displayName": "Silver",
			"id": "d6f96fe0-7da3-4930-a63c-7eb8d7ccafa8",
			"isEnabled": true,
			"lang": null,
			"origin": "Application",
			"value": "Silver"
		},
		{
			"allowedMemberTypes": [
				"User"
			],
			"description": "Platinum",
			"displayName": "Platinum",
			"id": "48488944-b473-406b-8024-bcc8176677bd",
			"isEnabled": true,
			"lang": null,
			"origin": "Application",
			"value": "Platinum"
		},
		{
			"allowedMemberTypes": [
				"User"
			],
			"description": "Administrators",
			"displayName": "Administrators",
			"id": "21b25842-b85c-4fc1-aa22-a9a752b18a1e",
			"isEnabled": true,
			"lang": null,
			"origin": "Application",
			"value": "Administrators"
		}
	],
	"oauth2AllowUrlPathMatching": false,
	"createdDateTime": "2021-03-02T15:14:22Z",
	"disabledByMicrosoftStatus": null,
	"groupMembershipClaims": null,
	"identifierUris": [
		"api://c1074030-bf88-4aa0-ad1a-6fd14b8462db"
	],
	"informationalUrls": {
		"termsOfService": null,
		"support": null,
		"privacy": null,
		"marketing": null
	},
	"keyCredentials": [],
	"knownClientApplications": [],
	"logoUrl": null,
	"logoutUrl": "https://dev-luloui.azurewebsites.net/signout",
	"name": "dev-luloui",
	"oauth2AllowIdTokenImplicitFlow": false,
	"oauth2AllowImplicitFlow": false,
	"oauth2Permissions": [
		{
			"adminConsentDescription": "user_impersonation",
			"adminConsentDisplayName": "Acess client app",
			"id": "b5e1ed18-e3e2-4aca-b37d-e8cd00fb7ae1",
			"isEnabled": true,
			"lang": null,
			"origin": "Application",
			"type": "User",
			"userConsentDescription": "user_impersonation",
			"userConsentDisplayName": "user_impersonation",
			"value": "user_impersonation"
		}
	],
	"oauth2RequirePostResponse": false,
	"optionalClaims": null,
	"orgRestrictions": [],
	"parentalControlSettings": {
		"countriesBlockedForMinors": [],
		"legalAgeGroupRule": "Allow"
	},
	"passwordCredentials": [
		{
			"customKeyIdentifier": null,
			"endDate": "2299-12-30T23:00:00Z",
			"keyId": "a3eb8e3f-70c0-49ec-8016-440964afe3c9",
			"startDate": "2021-03-02T15:22:31.529Z",
			"value": null,
			"createdOn": "2021-03-02T15:22:31.8903846Z",
			"hint": "~7I",
			"displayName": "postman2021"
		}
	],
	"preAuthorizedApplications": [],
	"publisherDomain": "enovel.com.co",
	"replyUrlsWithType": [
		{
			"url": "https://localhost:3000/",
			"type": "Spa"
		},
		{
			"url": "https://dev-luloui.azurewebsites.net/",
			"type": "Spa"
		}
	],
	"requiredResourceAccess": [
		{
			"resourceAppId": "42721e08-3737-48a5-a6c0-7e3bc4f9a5c9",
			"resourceAccess": [
				{
					"id": "ac2766f2-a930-42f2-a6bf-17d345c46948",
					"type": "Scope"
				}
			]
		},
		{
			"resourceAppId": "00000003-0000-0000-c000-000000000000",
			"resourceAccess": [
				{
					"id": "e1fe6dd8-ba31-4d61-89e7-88639da4683d",
					"type": "Scope"
				}
			]
		}
	],
	"samlMetadataUrl": null,
	"signInUrl": null,
	"signInAudience": "AzureADMultipleOrgs",
	"tags": [],
	"tokenEncryptionKeyId": null
}

I created the app myself using the create react app documentation, then I used the React Router Sample as an starting point:
my msal config:
export const msalConfig: Configuration = {
    auth: {
        clientId: "c1074030-bf88-4aa0-ad1a-6fd14b8462db",
        authority: 'https://login.microsoftonline.com/organizations/',
        redirectUri: "http://localhost:3000"
    }
};

@levalencia couldn't replicate this on my end. Can you please update your msal-browser and msal-react packages to the latest, and then enable verbose level logging and share the logs here.
p.s. a minor thing, your redirect URI in configuration file is http://localhost:3000, but registered as https://localhost:3000 on Azure portal. Are you having this issue on localhost?

Ok this is what I did
npm update
added https locally in AdalConfig.
It works locally!
I also added some logging config according to the documentation:

import { Configuration, PopupRequest, LogLevel} from "@azure/msal-browser";

// Config object to be passed to Msal on creation
export const msalConfig: Configuration = {
    auth: {
        clientId: "c1074030-bf88-4aa0-ad1a-6fd14b8462db",
        authority: 'https://login.microsoftonline.com/organizations/',
        redirectUri: "https://localhost:3000"
    },
    cache: {
        cacheLocation: "sessionStorage",
        storeAuthStateInCookie: false,
        secureCookies: false
    },
    system: {
        loggerOptions: {
            loggerCallback: (level: LogLevel, message: string, containsPii: boolean): void => {
                if (containsPii) {
                    return;
                }
                switch (level) {
                    case LogLevel.Error:
                        console.error(message);
                        return;
                    case LogLevel.Info:
                        console.info(message);
                        return;
                    case LogLevel.Verbose:
                        console.debug(message);
                        return;
                    case LogLevel.Warning:
                        console.warn(message);
                        return;
                }
            },
            piiLoggingEnabled: false
        },
        windowHashTimeout: 60000,
        iframeHashTimeout: 6000,
        loadFrameTimeout: 0,
        asyncPopups: false
    }
};

// Add here scopes for id token to be used at MS Identity Platform endpoints.
export const loginRequest: PopupRequest = {
    scopes: ["User.Read"]
};

// Add here the endpoints for MS Graph API services you would like to use.
export const graphConfig = {
    graphMeEndpoint: "https://graph.microsoft.com/v1.0/me"
};

Locally works perfect
Pushed the code to Azure and I still see same error even in a new private window.
The console logs are empty. I think I misunderstood the log config options

@levalencia for logging, you can use it like this
export const msalConfig: Configuration = {
    auth: {
        clientId: "c1074030-bf88-4aa0-ad1a-6fd14b8462db",
        authority: 'https://login.microsoftonline.com/organizations/',
        redirectUri: "https://localhost:3000"
    },
    cache: {
        cacheLocation: "sessionStorage",
        storeAuthStateInCookie: false,
    },
    system: {
        loggerOptions: {
            loggerCallback(logLevel: LogLevel, message: string) {
                console.log(message);
            },
            logLevel: LogLevel.Verbose,
            piiLoggingEnabled: false
        }
    }
}

When you deploy this app to Azure App Service, by any chance did you enable the Authentication/Authorization support there? If so, can you make sure to disable it (as it might be interfering):

thanks a lot, that last setting was the one causing the behavior.
I had authentication enabbled on the azure app service, I assumed I had to put it because for a .net core webapi I had it enabled too.
Maybe a good remark for the documentation

@levalencia you're welcome! Check out one of the deployment guide we have here. Closing this as resolved.


Vanilla Pico Support

Hi,
Are there any plans to make the Picosystem Micropython library available as a Python script?

There are not. It's written in C and would be unusably slow in Python.

Thanks for clarifying. It's just that I can't code in C.


WIP: Simplify dom-chef usage

@nickytonline I'm getting a weird error:
./refined-github/node_modules/dom-chef/index.d.ts' is not a module.

It appears you added an index.d.ts file to dom-chef recently and there's something it's not liking about it. I can take a peek tonight.
From what I can tell looking at the React types on DT, try adding these lines at the top of your index.d.ts
export = React;
export as namespace React;

@nicksnyder That is legacy TS syntax.
I think the correct solution is putting this at the bottom of the file:
declare const react: React;
export default react;

It's annoying that you can't "default export" namespaces.

Correct. My bad. I was just grabbing what was in the React DT types. ;) Agreed that it's annoying that you can't default export a namespace.

Wrong @ :)

I think he just wants you to contribute @nicksnyder. We need two Nicks. ;)

@nicksnyder Sorry, the GitHub autocomplete sucks...

Like this?
https://github.com/vadimdemedes/dom-chef/pull/32

Seems to look good. Try npm linking it to RGH and see how that goes.

That worked<EMAIL_ADDRESS>published


support mocha@3

Currently leche requires mocha at version >=1.18 <3 as a peerDependency. I already used it with mocha@3 and I didn't notice any problems.
Can you please allow leche to be used with a recent version of mocha. Maybe it's just a matter of updating the peerDependencies property of the package.json file.

Hey @chrisguttandin, it does look like we might just be able to update the package.json.  I'm going to do some more testing on some of our internal codebases to make sure that's the case.

Thank you very much, @mattwiller. Can you maybe also publish the changes to npm?

Sorry for the delay, the script for cutting a new version wasn't working correctly<EMAIL_ADDRESS>is published now and should work with mocha 3.x

Sorry for being impatient and many thanks again.


Import Multiple GPX Tracks

Previous version of OpenStreetMap allowed me to import and display multiple GPX tracks on the map so I could use them all to determine the location of paths and trails.

I don't think it is covered by either of those tickets. Those refer to large and or vague features which don't include the specific request from the first comment:

import and display multiple GPX tracks on the map

@jfirebaugh I also looked at the two tickets you mentioned.
Maybe they are prerequisites, but they don't seem to cover the functionality requested here.

It's been almost 10 years LOL, I'm still using Potlatch instead of iD.


 Not working at all

 Always ineligible for all the users, I have Added  button in xml. but still can not able to load it as requirement.
As per documentation and the Classes you provided , this must be helpful , but not working at all.
Please have look.

Hi @rajat-android,
Thank you for reporting the issue and I will be more than happy to help you!
Do you happen to know if the PayLater button was eligible in all the use cases where the PayPalCredit button was ineligible? You should only see either PayLater button or PayPalCredit. There will never be a use case where the user is eligible for both PayLater and PayPalCredit at the same time.

Hi @rajat-android,
I want to followup with you to see if you have any update for us. If it's no longer the issue I will close the ticket and feel free to reopen again. Thank you :)

We were a little busy at the time, And thought of doing another payment
gateway.
We still have that issue and the Simple Paypal button is working as
expected, but Paypal Later Button and Credit button eligibility always
comes up false and we are not able to see those buttons in the UI of App.
We will start work on that soon and I will let you know the status and if
we face any issues.
On Thu, Sep 1, 2022 at 12:48 AM Tim Chow @.***> wrote:

Hey @rajat-android https://github.com/rajat-android - I'm going to
close out this issue. But if you're still having issues please respond here
and we can reopen. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/paypal/android-checkout-sdk/issues/118#issuecomment-1233327856,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATALHLP25AMHNZI2JDH2X7TV36VY7ANCNFSM5YIXI5AA
.
You are receiving this because you were mentioned.Message ID:
@.***>


"Folder XXX not found in personal space"

hi @Caleb9
thanks again for the great update.
i am trying to utilize the new export function of your script.
i tried pretty much all possible folder formats, sadly i always get an error:
"Folder XX not found in personal space"
for example i create a directory "test" under the users home directory (home/test/)
using the export function, i define the directory as "/test".
i am running the script from a remote device, not the SynologyNAS itself.
connection ist fine, list command of the same album shows expected result.
am i doing it wrong?
one additional feature request:
any chance i would be able to specify a directory in another accesible home directory
(homes/userB/test) instead of home directory of user A? or just any other accesible path on the NAS itself?

Just to clarify my usecase:
I have a shared album between 2 users. both users can add photos from their personal space aswell as shared space.
With your script listing the album by logging in User A echos all pictures of User A personal space and shared space, and echos errors for User B added pictures.
Therefore i will execute the script twice, one time logging in user A, on time logging in user B. the whole album should then be exported to a directory which is accesible by both users (as i understand by your readme, duplicates will be skipped - in this case all pictures from shared space).
Do you think this is the right approach for my usecase?
Thank you so much in advance.

@grandmastermarco Currently, what you're trying to do is not really possible. The reason is that your /home/test folder is not a folder accessible by Synology Photos. To make it work the folder would have to be in /home/Photos/test. Best way to create one is through the Photos interface itself, not via File Station:

And then use /test as the parameter for the app.
Because the folder has to be in personal space of the user you execute the app as, your scenario of collecting photos from personal spaces of two different users (available via the shared album) into a single folder in personal space of one of those users is not going to work currently. You can only export photos from shared space and personal space of the user you're executing the app as.
A workaround for now would be to:

User A exports into /test1 in user A's personal space - physically this folder will be in /homes/user_a/Photos/test1. After this /test1 will contain photos from the album that are in User A's personal space and in shared space.
User B exports into /test2 in user B's personal space - /homes/user_b/Photos/test2. /test2 now contains User B's personal space photos and shared space photos (which will be the same as shared space photos of User A).
User A moves photos "manually" (via Photos web client or File Station) from /test1 to a target folder in shared space, e.g. /dest - physically this folder will be in /photo/dest.
User B moves photos from /test2 to the same target folder in shared space.

The code is currently communicating with Synology Photos API only, to make it copy to an arbitrary folder on the NAS would require it to use File Station API most likely, I'm not saying it's impossible but it would be much more difficult to implement because Photos uses its own internal IDs for files it manages, which is probably different from File Station IDs or paths - not sure as I don't have experience with File Station API.

I might think of making it possible to use shared space folder as export destination in the future though :)

Thank you so much once again for the very detailed explanation. I tought that i would be able to export to my normal Synology NAS folder with the --export option. But it makes sence that it is not possibleaccording to your explanation.
I was finally able to implement my usecase with your script. I used the --list command and then wrote a custom bash script to export those listed folders into a seperate directory.


Issue #SH-649 fix: Lib integration for Add resource filter

Issue #SH-649 fix: Lib integration for Add resource filter

This PR is not required now, hence closing this PR


Add support for OpenAI custom base url and default headers + RITS Inference engine

Added support for passing open base url, default headers, and setting credentials in OpenAIInferenceEnginer
Added RITSInferenceEngine (also to CrossProviderInferenceEngine)

@yoavkatz Is RITS available from outside the network? Maybe that is why the test fails

@OfirArviv @yoavkatz IMO we should remove the VLLMRemoteInferenceEngine as it adds no functionality to OpenAiInferenceEngine. In addition, other servers/frameworks can potentially expose LLMs using the OpenAI API, so this is not something exclusive to a remote vLLM instance. Users can directly create an OpenAiInferenceEngine and set the lable accordingly.

@OfirArviv @yoavkatz IMO we should remove the VLLMRemoteInferenceEngine as it adds no functionality to OpenAiInferenceEngine. In addition, other servers/frameworks can potentially expose LLMs using the OpenAI API, so this is not something exclusive to a remote vLLM instance. Users can directly create an OpenAiInferenceEngine and set the lable accordingly.

We cannot right now because some users are using it. But I defined it as minimal as it can be, basically just setting the label


Скласти таймлайн

Скласти список всіх (важливих) життєвих подій, визначних дат, досягнень, періодів, що дали приємний і неприємний досвід, тощо. В цьому конкретному випадку довгих списків не буває, натомість бувають погано згорнуті, тому пакуємо сюди все підряд, а про "лишні" події можна буде промовчати пізніше. Список повинен бути систематизованим і посортованим в такий спосіб, щоб читач швидко зрозумів "паттерн" і міг легко орієнтуватись в просторі і часі цього резюме.

The debut:

"When We Were Very Young" (6 November 1924) by A. A. Milne
Theatre:
Winnie-the-Pooh at the Guild Theater, Sue Hastings Marionettes, 1931[34]
Winnie-the-Pooh, a play in three acts, dramatized by Kristin Sergel, Dramatic Publishing Company, 1957
Winnie-the-Pooh, a musical comedy in two acts, lyrics by A. A. Milne and Kristin Sergel, music by Allan Jay Friedman, book by Kristin Sergel, Dramatic Publishing Company, 1964
A Winnie-the-Pooh Christmas Tail, In Which Winnie-the-Pooh and His Friends Help Eeyore Have a Very Merry Christmas (Or a Very Happy Birthday), book, music and lyrics by James W. Rogers, Dramatic Publishing Company, 1977
"Bother! The Brain of Pooh", Peter Dennis, 1986
Winnie-the-Pooh, small cast musical version, dramatized by le Clanché du Rand, music by Allan Jay Friedman, lyrics by A. A. Milne and Kristin Sergel, additional lyrics by le Clanché du Rand, Dramatic Publishing Company, 1992
Audio:
Winnie-the-Pooh (consisting of three tracks: "Introducing Winnie-the-Pooh and Christopher Robin"; "Pooh Goes Visiting and Gets into a Tight Place"; "Pooh and Piglet Go Hunting and Nearly Catch a Woozle") 1956
In 1951, RCA Records released four stories of Winnie-the-Pooh, narrated by Jimmy Stewart
In 1960, HMV recorded a dramatised version with songs (music by Harold Fraser-Simson) of two episodes from The House at Pooh Corner (Chapters 2 and 8)
In 1979, a double audio cassette set of Winnie the Pooh was produced featuring British actor Lionel Jeffries reading all of the characters in the stories.
-In the 1990s, the stories were dramatised for audio by David Benedictus, with music composed, directed and played by John Gould
Radio:
The BBC has included readings of Winnie-the-Pooh stories in its programmes for children since very soon after their first publication. One of the earliest of such readings, by "Uncle Peter" (C. E. Hodges), was an item in the programme For the Children, broadcast by stations 2LO and 5XX on 23 March 1926.
Pooh made his U.S. radio debut on 10 November 1932, when he was broadcast to 40,000 schools by The American School of the Air, the educational division of the Columbia Broadcasting System.
Film:
2017: Goodbye Christopher Robin, a British drama film exploring the creation of Winnie-the-Pooh by A.A. Milne, with Domhnall Gleeson playing Christopher Robin
Television:
1966: Winnie the Pooh and the Honey Tree
1968: Winnie the Pooh and the Blustery Day
Винни-Пух (Winnie-the-Pooh, 1969) – based on chapter 1
Винни-Пух идёт в гости (Winnie-the-Pooh Pays a Visit, 1971) – based on chapter 2
1974: Winnie the Pooh and Tigger Too
1981: Winnie the Pooh Discovers the Seasons
1983: Winnie the Pooh and a Day for Eeyore
Video games:
Winnie the Pooh in the Hundred Acre Wood(1984)
Ready to Read with Pooh(1997)
Disney's Winnie the Pooh's Rumbly Tumbly Adventure(2002)
Pooh's Hunny Blocks(2003)
Disney's Winnie the Pooh and the Honey Tree Animated Storybook(2014)


"Available" Field for Paper in IEEE Style.

I am currently exporting my bibliography from Zotero in bibtex form to use in Typst - so admittedly, I may well have some problems in Zotero/with that pipeline.
However, when citing journal articles in the IEEE style, Hayagriva is including the URLs I used to access those - which is something I have been told should not be included in my bibliography for the document I am writing, when the source is already part of a journal or other source that can be found canonically using the other information present.
If this is on me and the format of my bibtex file, then more than happy for this issue to be closed - but in case others are experiencing this and the output style of Hayagriva possibly needs to be looked at, thought I would raise.
Thanks!

Can you provide the exported .bib entry?
The ieee.csl implements Reference Guide of IEEE Editorial Style Manual where the subsection "Periodicals Online" describes the cases with URLs.

A possible solution is using the Better BibTeX plugin and setting the option "Add URLs to BibTeX export" to "no".

So far, I have tried using Better Bibtex, but this has been causing more problems in other areas (e.g. backslashes and latex instead of unicode in the text) than the built-in Zotero plugin, and sometimes Hayagriva panics. (As an aside, I opened an issue asking if Better Bibtex would consider Hayagriva support, but it sounds like the maintainer would prefer Hayagriva to support CSL input. I'm not sure where the boundaries lie exactly internally, but seems fair comment.)
I have brought up the bibliography; using the latest release on the Typst web app, it doesn't seem to be reproducing immediately in a minimal example - so possibly has changed since CSL support was added for styles. I will keep an eye out if it comes up again, but if CSL is now in the master branch and this is definitely the way forward, then I'm happy for this to be closed.
Thanks!

So far, I have tried using Better Bibtex, but this has been causing more problems in other areas (e.g. backslashes and latex instead of unicode in the text)

You can disable this in the BBT preferences, but this just means Hayagriva doesn't really parse TeX. There are some constructs that must use latex commands if you want them in the output (like a literal backslash, or a percent sign), and these will be exported as valid TeX commands even if you turn unicode mode on for BibTeX. Better BibLaTeX has unicode mode as the default.

So far, I have tried using Better Bibtex, but this has been causing more problems in other areas (e.g. backslashes and latex instead of unicode in the text) than the built-in Zotero plugin, and sometimes Hayagriva panics.

Please open some issues for the panics!
In terms of the URLs, Hayagriva behaves in a CSL-spec-compliant way here. However, we could introduce an option to always suppress URLs for "formally published" items


Added correct values of fillMissingData and ConditionalData

CSS Classes should find the node data with processorType fillMissingData and ConditionalData to add the processor images to those nodes.

Coverage remained the same at 98.492% when pulling 530f6d63ce2b4a34c6b2f0d0025ed41b6872e4dc on mwernimont:master into 4d1459eb858dc80f27614f3a84a5352e457bc0dd on USGS-R:master.


Do not clear screen for question/list prompts

This one is from Inquirer.js too:

Basically Inquirer.js can ask questions without clearing the screen (and printing the question to x=0 y=0) ishell currently clears the screen when called like ishell.New().MultiChoice(...)
(P.S. also in the long-list.js example above, if the list is longer than N items, it allows limiting the scroll area to N items, while still allowing user to scroll. but that's a separate topic.)

I've actually gotten this to work, at least to a certain degree.
See here:

There are still some kinks to work out, but I can open up a pull request if desired, @abiosoft @ahmetb


twindow scrolling

hi. good morning. klamonte.
TWindow's scroll support?

What is the question here?

sorry,
I did not find a place to ask questions.

Ok jeakun, Kevin is just telling you to elaborate your question better
because we don't understand what you are asking for
El 9:52PM Lun Mar 27, 2017, Jeakun<EMAIL_ADDRESS>escribió:

sorry,
I did not find a place to ask questions.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/klamonte/jexer/issues/4#issuecomment-289647375, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIAiQ7uoTS-HJFmZmXZdcsSmaHX17GNlks5rqHXigaJpZM4MntyE
.

I'm currently using jexer.
As you configure the UI, TWindow does not support scrolling.
I tried to fix it myself, but it failed.
So, I have written a question here to see if there is a way to be supported.
Can I get help?

TWindow is just a container widget for other things.  Currently, to get scrolling behavior you have to add TVScroller or THScroller to TWindow and change their X/Y positions as TWindow is resized to keep them on the edge.
TText shows an example of a window with both horizontal and vertical scrollbars.  TTerminalWindow has just a single vertical scrollbar.  The reflow() function in each one shows the logic of creating the scrollbar and changing its location to stay on the outside edge of the window.
The overall API is still a work in progress.  This would be a good time to either add a scrollbar flag to TWindow, and/or create a TScrollingWindow class that does reflow() automatically.

Let me think about TScrollingWindow.
But maybe not. I am.
We'll wait for work in progress.
Thank you for developing these APIs. Kevin.
bye.

FYI a Scrollable interface with TScrollableWidget and TScrollableWindow classes is in the works, should be uploaded later tonight.  These classes handle the resize and expose the scrollbar functions to callers.

Commit https://github.com/klamonte/jexer/commit/56661844475522242093c8858ceb20fb15589da5 has a common interface to scrollbar support.


Windows-only compilation failure using #sourceLocation

Description
Using #sourceLocation(file:line:) fails on Windows only with the latest nightly toolchain (as of 2023-01-02.)
Steps to reproduce
In a Swift file, write:
#sourceLocation(file: "example.swift", line: 123)

Then build.
Expected behavior
Build succeeds.

Environment

Swift compiler version info compnerd.org Swift version 5.8-dev (LLVM ab856b0fa3177ee, Swift 71c62c04ace2b5b)
Deployment target: x86_64-unknown-windows-msvc

The following test target fails to build:
import XCTest

final class Test: XCTestCase {
  func testHuh() {
    #sourceLocation(file: "a.swift", line: 1)
  }
}```

Dup of https://github.com/apple/swift/issues/57553


Missing containers

I have two containers running, but Kitematic shows only one.
These are my running containers.
 It is running, but still does not show up in the Containers tab.

Hi Ankush,
Thanks for checking out Kitematic!
Sorry about the confusion. Kitematic won't show containers without exposed ports. In this case running docker run -d nginx doesn't expose any ports. Adding -p 80:8080 (or whatever ports you're using) should work!
The problem is that the Docker build process creates a bunch of temporary weirdly-named containers, so we used this as a heuristic to hide them.
We're fixing this in 0.5 - instead of hiding containers with no exposed ports, we'll most likely hide containers whose images aren't tagged or something similar.
Hope this helps! Thanks for the issue!

Most of the times, I just want to look at the running containers. The exited containers and temporary containers - not so much. If there could be a view within Kitematic which showed me docker ps, that'd be great (or a toggle switch in the containers tab which shows all running containers).
Thanks.
Love the UI of Kitematic.

Fixed in 0.5! Kitematic now syncs all container state with the CLI
The only containers that won't be shown are ones created from images without tags. These are probably intermediate containers that exist due to the build process.

Thanks!


Why not scanning subpackages

Currently (v0.11) the code of RuleBookRunner has this code:
  protected List<Class<?>> getPojoRules() {
    Reflections reflections = new Reflections(_package);

    List<Class<?>> rules = reflections
        .getTypesAnnotatedWith(com.deliveredtechnologies.rulebook.annotation.Rule.class).stream()
        .filter(rule -> _package.equals(rule.getPackage().getName())) // Search only within package, not subpackages
        .filter(rule -> rule.getAnnotatedSuperclass() != null) // Include classes only, exclude interfaces, etc.
        .collect(Collectors.toList());

    rules.sort(comparingInt(aClass ->
        getAnnotation(com.deliveredtechnologies.rulebook.annotation.Rule.class, aClass).order()));

    return rules;
  }
}

Why are the scanning limited to the exact same package?
That's not an expected behaviour, and it takes a lot of time (debugging) to understand why your rule rule is not being loaded.
I know is probably for performance, not to load all classes at startup time, but wouldn't it be better just to recommend people to have rules in a separate package?
Thanks

I think the thought was that subpackage would be considered a separate package, and, therefore would not be loaded. No reason why subpackages couldn't be scanned, tho. Perhaps default behavior could be to scan only the package specified with an option to scan subpackages?

I would expect the default behaviour to scan sub packages, just like a spring @ComponentScan would do, with the option to disable subpackage scanning.
I believe is harder to find the issue of NOT SCANNING then scanning too much :-)

Fair point.
Agreed. Subpackages should be scanned by default with an option to disable.

Updated in 0.12-SNAPSHOT.


[Stripe] Implement Refund event listener in Stripe webhook

Which area(s) of Podkrepi.bg are affected? (leave empty if unsure)
Donations
Describe the feature you'd like to request
We need to handle Stripe refund event, so that in case of Refund request, the refunded amounts to be properly substracted.
The refund itself is triggered by a billing admin in Stripe Admin UI and then Stripe will send us a redund.created event: https://stripe.com/docs/api/refunds/object
Describe the solution you'd like
When handling refund.created please check:

if it is the only event that confirms that the refund was successful.
if refunded amount will return the whole amount or the the amount without taxes - we may need to send email notification to the billing_email of the donation

The amount of the donation need to be substracted from the vault in similar way that it is added to it when successful donation is received.
Describe alternatives you've considered
No response

closed with https://github.com/podkrepi-bg/api/pull/550

oops, wrong reference to PR - this issue is still not implemented


'django_postgrespool' isn't an available database backend

I am getting:
Traceback (most recent call last):
...
raise ImproperlyConfigured(error_msg)
django.core.exceptions.ImproperlyConfigured: 'django_postgrespool' isn't an available database backend.
Try using django.db.backends.XXX, where XXX is one of:
'dummy', 'mysql', 'oracle', 'postgresql_psycopg2', 'sqlite3'
Error was: cannot import name Diagnostics
What solved my problem was to install psycopg2-2.4.6 instead of psycopg2-2.5.1 (both from http://www.stickpeople.com/projects/python/win-psycopg/).
I tried the advice in issue https://github.com/kennethreitz/django-postgrespool/issues/6 and it didn't help.
I am using Windows Python 2.7.3 AMD64.

I'm having the same problem with the following packages:
database-url 0.3.0,
Django 1.8.1,
django-postgrespool 0.3.0,
gunicorn 19.3.0,
psycopg2 2.6,
SQLAlchemy 1.0.4,
whitenoise 1.0.6.
This is the error:
django.core.exceptions.ImproperlyConfigured: 'django_postgrespool' isn't an available database backend.
Try using 'django.db.backends.XXX', where XXX is one of:
u'base', u'mysql', u'oracle', u'postgresql_psycopg2', u'sqlite3'
Error was: DLL load failed: The specified module could not be found.

Looks like both of these issues were caused by psycopg2 installations.


Value iteration

Wrt #357
Havent added a test yet, added the value iteration file as genrl/agents/classical/valueiteration/valueiteration.py. Can be trained with the current Classical trainer

Few comments:

It might be worth coming up with a general design for Model Based Agents.
Value Iteration can happen with DNNs as well.

Since this is a classical method, I didn't include the base model based agent here, its in my CEM PR
This is on hold

So I guess this is all for the classical VI

Does the agent train?

It did before, I'll run it again and check

Does the agent train?

Yes it does train on FrozenLake

Ok great. Any comparison with Q Learning?
Will merge it post CI re-run


ui: refine handle errors

What did:

Use key for the message box to avoid multiple same message boxes

Only show the first error if has multiple errors

Show "Network Error" in the Alert component as well

TODO:

Move Alert out of CardTableV2 after label branch in PR #591 is merged. Currently moving out loses the margin.

close this PR since it is replaced by PR #662, and this branch can be deleted.


AttributeError: module 'torch.nn' has no attribute 'Hardswish' on Raspberry Pi

AttributeError: module 'torch.nn' has no attribute 'Hardswish' on Raspberry Pi

from d2go.model_zoo import model_zoo gives AttributeError: module 'torch.nn' has no attribute 'Hardswish'
from d2go.model_zoo import model_zoo
Full logs or other relevant observations:

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-1-ffdbfe040df2> in <module>
----> 1 from d2go.model_zoo import model_zoo
      2 model = model_zoo.get('faster_rcnn_fbnetv3a_C4.yaml', trained=True)

~/.local/lib/python3.7/site-packages/d2go/__init__.py in <module>
      5 # Set up custom environment before nearly anything else is imported
      6 # NOTE: this should be the first import (no not reorder)
----> 7 from d2go import initializer, optimizer  # NOQA

~/.local/lib/python3.7/site-packages/d2go/initializer.py in <module>
     64 _INITIALIZED = False
     65 if not _INITIALIZED:
---> 66     initialize_all()
     67     _INITIALIZED = True

~/.local/lib/python3.7/site-packages/d2go/initializer.py in initialize_all()
     59     _setup_env()
     60     _register_d2_datasets()
---> 61     _register()
     62
     63

~/.local/lib/python3.7/site-packages/d2go/initializer.py in timed_f(*args, **kwargs)
     15         def timed_f(*args, **kwargs):
     16             start = time.perf_counter()
---> 17             ret = f(*args, **kwargs)
     18             time_list.append(time.perf_counter() - start)
     19             return ret

~/.local/lib/python3.7/site-packages/d2go/initializer.py in _register()
     41 @_record_times(REGISTER_TIME)
     42 def _register():
---> 43     from d2go.modeling.backbone import (  # NOQA
     44         fbnet_v2,
     45     )

~/.local/lib/python3.7/site-packages/d2go/modeling/backbone/fbnet_v2.py in <module>
     20 from detectron2.modeling.roi_heads import box_head, keypoint_head, mask_head
     21 from detectron2.utils.logger import log_first_n
---> 22 from mobile_cv.arch.fbnet_v2 import fbnet_builder as mbuilder
     23 from d2go.modeling.modeldef.fbnet_modeldef_registry import FBNetV2ModelArch
     24 from mobile_cv.arch.utils.helper import format_dict_expanding_list_values

~/.local/lib/python3.7/site-packages/mobile_cv/arch/fbnet_v2/fbnet_builder.py in <module>
    140 import torch.nn as nn
    141
--> 142 from .blocks_factory import PRIMITIVES
    143
    144

~/.local/lib/python3.7/site-packages/mobile_cv/arch/fbnet_v2/blocks_factory.py in <module>
     10 from torch import nn
     11
---> 12 from . import basic_blocks as bb, irf_block
     13
     14

~/.local/lib/python3.7/site-packages/mobile_cv/arch/fbnet_v2/basic_blocks.py in <module>
    227
    228
--> 229 HSwish = torch.nn.Hardswish
    230
    231

AttributeError: module 'torch.nn' has no attribute 'Hardswish'

I think the error is because the pytorch/libtorch is too old. Could you try using the latest version?


ConcavePolygonShape3D

Description
This issue tracks the progress of implementing the functionality of Godot's ConcavePolygonShape3D node type. Below you'll find all user-facing properties, methods and signals, including ones inherited from relevant node types. If you notice any missing from this list, please leave a comment down below.
Status

Not supported
Not started
Partial (not functional)
Partial (functional)
Done (as much as it can be) 👈
Done (completely)

Links
Documentation: ConcavePolygonShape3D
Inherits: Shape3D
Legend

🚧 indicates a partial, incomplete or broken implementation.
🛑 indicates that it's blocked or incompatible with Jolt in some way.

Properties
Uncategorized

[x] Backface Collision (backface_collision)

Properties (Shape3D)
Uncategorized

[ ] 🛑 Custom Solver Bias (custom_solver_bias)

Not supported by Jolt

[x] Margin (margin)

Methods

[x] get_faces
[x] set_faces

This tracking issue has served its purpose and will now be closed. 🫡
Any remaining work has had new issues created for it.


metrics: fix collecting hugetlb statistics from cgroup v1

Hugetlb statistics collector matched both usage and (later added) reservation accounting files (.rsvd.). When both were present, the collector tried to create duplicate metrics entries: usage and reservation values got the same name and labels.
This patch ignores .rsvd. files when collecting statistics in order to keep reporting exactly what was reported before them.
Fixes error "cgroup_hugetlb_usage... collected before with the same name and label values" error in cri-resmgr output.
Fixes issue #859.

I guess it was cherry picked but can we update the PR description a bit, because some of the comments don't really apply here. Example, Fixes issue #859.

I removed the "fixes" line as it does not make sense with this repo

I removed the "fixes" line as it does not make sense with this repo

thanks, but I was also talking about PR description without noticing the same in the commit message. But I fixed it now in the PR description too.
LGTM


MethodCall info for infix operators

When a binary operator application is present in the project source module, the backend should associate a methodPointer and method info with the OprApp node. Currently it seems to do so occasionally (code making use of this information was introduced in 73487adce9040a4e4e0fcfe00704a0c9a6caeb16), but I'm not sure what circumstances reach it--usually the methodPointer is missing.
In particular, we need to receive the information reliably for a graph like this:

With method information for the + operator above, we will be able to display an appropriate placeholder argument similarly to how we create placeholders for non-operator method calls, fixing the secondary issue mentioned here

@farmaazon Just checked the latest state and while backend added the support, somehow we are not seeing the expected results in IDE still. @4e6 will investigate if the problem is in backend or GUI.


geospatial data

is it possible to use this for geospatial data by precomputing a distance matrix for lat-long values based on haversine distance?

Yes, definitely it is possible. If you have precomputed distance matrix, you can make clustering with this code:
DISTANCE_THRESHOLD = 1  # set your distance threshold
model = MaxDiameterClustering(max_distance=DISTANCE_THRESHOLD, sparse_dist=False, precomputed_dist=True)
labels = model.fit_predict(dist_matrix)

Don't forget to set sparse_dist=False if your distance matrix is ordinary numpy array (not sparse matrix).
I haven't tried this clustering with geospatial data, but it looks like very suitable use case. Having maximum distance between points in cluster makes a lot of sense for geo data.
Feel free to ask if you have any questions.

thanks. I'll try this out with the above suggestions and compare with the R library. Will let you know.

ok so by and large we're getting results for LeaderClustering with the library that are similar to the R implementation. I did this by defining a custom function "my_hav" to calculate the haversine distance and passing it to the model parameter - see sample code below. Turns out didn't need to precompute a distance matrix.
The only request would be if it is possible to add functionality to return cluster centroids for each cluster as well. This helps especially if one needs to plot the points and draw a circle around them
`
import numpy as np
import diameter_clustering as dc
import haversine as hs
coords = [lat, long]. to_numpy()
def my_hav(c1, c2):
return hs.haversine(c1, c2)
max_rad = 1
model = dc.LeaderClustering(max_radius = max_rad, metric = my_hav, deterministic = True)
labels = model.fit_predict(coords)
`

It is interesting that you don't need to precompute distance matrix, I missed this opportunity :)
May be I will be able to add cluster centroids for each cluster next week.
As for now, after model was fitted it has leaders_ attribute. It contains array with 1 for points which are cluster centroids and with 0 for all other points. Unfortunately there is no information which cluster each centroid belongs to. But I think it can help with drawing centroids and circles around them.

hi! @gurdeep101  how many points did you use while clustering and max_rad=1 is in miles or kms or radians?


Update project-property-inheritance.md

This doesn't work without ''

@TylerMSFT
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


CameraView issue - EventToCommandBehavior: Couldn't resolve the event

Hello there,
I'm using CameraView and EventToCommandBehavior to create some sort of a simple app that makes photos.
But I've faced with the issue of using EventToCommandBehavior. When running app in Release mode on both android emulator and real device I do see the following error in Application Output:

[MonoDroid] UNHANDLED EXCEPTION:
[MonoDroid] Android.Runtime.JavaProxyThrowable: Exception of type 'Android.Runtime.JavaProxyThrowable' was thrown.
[MonoDroid]   --- End of managed Android.Runtime.JavaProxyThrowable stack trace ---
[MonoDroid] android.runtime.JavaProxyThrowable: System.ArgumentException: EventToCommandBehavior: Couldn't resolve the event.
[MonoDroid] Parameter name: EventName

The code itself isn't that complicated:
<xct:CameraView
               x:Name="CameraView"
               BindingContext="{Binding CameraModel}"
               CameraOptions="{Binding CameraOption}"
               HorizontalOptions="FillAndExpand"
               VerticalOptions="FillAndExpand"
               CaptureMode="Photo">
               <xct:CameraView.Behaviors>
                   <xct:EventToCommandBehavior EventName="MediaCaptured" Command="{Binding CaptureCommand}" />
               </xct:CameraView.Behaviors>
</xct:CameraView>

The Environment is the latest one at this moment:

Visual Studio Community 2019 for Mac
Version 8.10.3 (build 15)
Installation UUID: c4debf69-38dc-430a-b7ff-71b5f73fb9f3
GTK+ 2.24.23 (Raleigh theme)
Xamarin.Mac <IP_ADDRESS> (d16-6 / 088c73638)
Package version: 612000140
Mono Framework MDK
Runtime:
Mono <IP_ADDRESS> (2020-02/51d876a041e) (64-bit)
Package version: 612000140
Roslyn (Language Service)
3.10.0-4.21269.26+029847714208ebe49668667c60ea5b0a294e0fcb
NuGet
Version: <IP_ADDRESS>34
.NET Core SDK
SDK: /usr/local/share/dotnet/sdk/5.0.301/Sdks
SDK Versions:
5.0.301

Thanks in advance for helping!!

Hi @slprog1, could you please share the Command that you are Binding to? It works for me without any error on Android, but I had to specify the event args as MediaCapturedEventArgs :
public DelegateCommand<MediaCapturedEventArgs> MediaCapturedCommand { get; }

private void ExecuteMediaCapturedCommand(MediaCapturedEventArgs e)
{
...
}

Hi @GUOLDEV , thanks for replying!!
Regarding the command I'm binding to, well it's nothing special here, just a command, which was accepting the object as a parameter type. I did try to specify the exact type MediaCapturedEventArgs as you mentioned, but still the behaviour is the same - in Release mode it does no work(
When adding event handler and doing all needed stuff for me there - it works fine. But that is the thing - I need to "bind" the event to command from my ViewModel :)

All good 👍 by the way, the code that I posted is in the ViewModel and it works. When I have a chance I will try in release mode as you reported to make sure it works.

Hi @slprog1 I can reproduce the issue that you reported, CameraView.Behaviors works only in Debug Mode.

@GUOLDEV can you change the Linker configuration in DEBUG mode to be Link SDK and assemblies? After that let me know if the issue happens in Debug as well. Thanks in advance.

@pictos thanks mate for the prompt reply. I have tested and it stops working setting the Linking to "Sdk Assemblies Only" or "Sdk and User Assemblies" in DEBUG mode. I have tested on Galaxy S9+ Android 10. Please let me know if I can help with any other info.

Yeah, this looks like a linker issue. I'll try to investigate this tomorrow or next week. Thanks for your help ❤

@GUOLDEV if you can attach your repro here will be amazing!

@pictos do you mean the sample code?

@pictos do you think it is affecting just CameraView or other controls in general? Sorry, I didn't have time to try other controls to confirm that. If it is just CameraView it would be good to add the CameraView label.

@GUOLDEV I didn't test either, for now, I'll assume that issue is general

@pictos I have created a PR for the CameraView Linker issue. Could you please have a look when you have a chance? Thanks in advance.

I tried with release 1.3.0-pre1, but the issue is still present.

Hi @workgroupengineering, what is the event that you are using? It would be great if you could add more info 👍

I using MediaCaptured
XCT 1.3.0-pre1
Tested on Android 8 and 10 Emulator in Release mode.
If add MediaCaptured="CameraView_MediaCaptured in xaml with empty eventhandler it work.

Hi @workgroupengineering as I thought, 1.3.0-pre1 doesn't have the fix, it's merged into dev branch only.

Hi @workgroupengineering 1.3.0-pre2 was just released, please have a look 👍


Add parens to application QuickFix for NotAFunction error

https://github.com/JetBrains/fsharp-support/pull/142
Depends on:
https://github.com/auduchinok/visualfsharp/pull/3
https://github.com/auduchinok/visualfsharp/pull/6

I've made a small adjustment: when there's a single candidate and there's a way to present it, we'll show a bit better text for the quick fix:

@DedSec256 Thanks for this quick fix, it's a really cool thing to use! 🙂


ASHLAR registers tiles as cycles/fileseries not working

Hi guys,
I am using ashlar to read two .ome.tiff images with 18 channels each that are two tiles of a bigger image. Unfortunately, ASHLAR initially added the images on top of each other, creating one image with 36 channels (18 channels x 2 cycles).
When I used the fileseries command: ashlar 'fileseries|/folderpath|pattern=tile{series:2}.ome.tiff|width=2|height=1|pixel_size=0.182|overlap=0.02' --output-channels 15 I got the following error:
aligning edge 1/1Traceback (most recent call last):
File "/home/t141g/.local/lib/python3.9/site-packages/ashlar/reg.py", line 674, in register_pair
shift, error = self._cache[key]
KeyError: (0, 1)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/xxxxx/.local/bin/ashlar", line 8, in
sys.exit(main())
File "/home/xxxxx/.local/lib/python3.9/site-packages/ashlar/scripts/ashlar.py", line 212, in main
return process_single(
File "/home/xxxxx/.local/lib/python3.9/site-packages/ashlar/scripts/ashlar.py", line 243, in process_single
edge_aligner.run()
File "/home/xxxxx/.local/lib/python3.9/site-packages/ashlar/reg.py", line 485, in run
self.register_all()
File "/home/xxxxx/.local/lib/python3.9/site-packages/ashlar/reg.py", line 596, in register_all
self.register_pair(t1, t2)
File "/home/xxxxx/.local/lib/python3.9/site-packages/ashlar/reg.py", line 697, in register_pair
error = utils.nccw(o1, o2, self.filter_sigma)
File "/home/xxxxx/.local/lib/python3.9/site-packages/ashlar/utils.py", line 55, in nccw
correlation = np.abs(np.sum(img1w * img2w))
ValueError: operands could not be broadcast together with shapes (2976,109) (3695,109)
I don't really get the problem her, is there somebody who could help?
Thank you!

Sorry for the late response, but did you figure this out? Your command line looks OK if your files are named tile00.ome.tif and tile01.ome.tif. Very small tilesets like this do sometimes fail with unhelpful error messages, but I think I've fixed this in the master branch. I plan to make an official release soon including this fix, but you could try installing directly from github and see how it works. Also, in the current master branch, the included preview_slide tool now supports fileseries so you could use that to make sure ashlar is interpreting your inputs as you expect.

Hi,
I figured it out, by setting maximum shift to ≤4 µm it worked.
However, when using the same command on another fileseries with 19 channels, ashlar fails at the process of saving the pyramid when I have it save all channels. When I limit it to ≤18 channels, it works (regardless of the channel I omit). It shows the following error message:
...
Channel 18:
    merging tile 35/35
Generating pyramid
  Level 1 (12788 x 15437)
<tifffile.TiffPages @16> invalid page offset<PHONE_NUMBER>
    processing channel 3/19Traceback (most recent call last):
 File “/home/xx/.local/bin/ashlar”, line 8, in <module>
  sys.exit(main())
 File “/home/xx/.local/lib/python3.9/site-packages/ashlar/scripts/ashlar.py”, line 212, in main
  return process_single(
 File “/home/xx/.local/lib/python3.9/site-packages/ashlar/scripts/ashlar.py”, line 277, in process_single
  writer.run()
 File “/home/xx/.local/lib/python3.9/site-packages/ashlar/reg.py”, line 1247, in run
  tiff.write(
 File “/home/xx/.local/lib/python3.9/site-packages/tifffile/tifffile.py”, line 3337, in write
  iteritem = next(dataiter)
 File “/home/xx/.local/lib/python3.9/site-packages/tifffile/tifffile.py”, line 22910, in newiter
  yield from iterator
 File “/home/xx/.local/lib/python3.9/site-packages/ashlar/reg.py”, line 1206, in subres_tiles
  a = zimg[c, y:y+th, x:x+tw, 0]
 File “/home/xx/.local/lib/python3.9/site-packages/zarr/core.py”, line 842, in __getitem__
  result = self.get_orthogonal_selection(pure_selection, fields=fields)
 File “/home/xx/.local/lib/python3.9/site-packages/zarr/core.py”, line 1122, in get_orthogonal_selection
  indexer = OrthogonalIndexer(selection, self)
 File “/home/xx/.local/lib/python3.9/site-packages/zarr/indexing.py”, line 617, in __init__
  dim_indexer = IntDimIndexer(dim_sel, dim_len, dim_chunk_len)
 File “/home/xx/.local/lib/python3.9/site-packages/zarr/indexing.py”, line 150, in __init__
  dim_sel = normalize_integer_selection(dim_sel, dim_len)
 File “/home/xx/.local/lib/python3.9/site-packages/zarr/indexing.py”, line 124, in normalize_integer_selection
  raise BoundsCheckError(dim_len)
zarr.errors.BoundsCheckError: index out of bounds for dimension with length 2


Access bans in O(1) time and don't panic on file not found.

This commit makes it so that we can access bans in O(1) constant time.
It also allows the program to run if the twitter or slack files are not found.

Map access is only better than slice search at high numbers. I'll run a modified version of https://gist.github.com/JelteF/1251f180966eb974d7732ea6c3d3b7ff to see if using a map[ban]bool‘ really is better than a []banat our scale (< ~20 bans at any time) Another thing is thatmap[ban]boolimplies that bans can be stored with the bool set to false, which we do not want.[]ban` makes it absolutely clear that we will only store the bans, so it's simpler to understand. I like the Twitter and Slack changes, but I would modify them to return a no-op result and a message indicating that the files are not present instead of carrying on like the API keys are empty (which will cause confusing errors).

You can already set DEVZAT_OFFLINE=1 to run offline

Yes, what about it?

Shouldn't that A) be split into another commit, and B) just say 'set DEVZAT_OFFLINE to run without connecting to external services'?

@TommyPujol06 if the file isn't found, you can set the global variable offline to true and it should do everything needed (except print a message that the file doesn't exist)

Map access is only better than slice search at high numbers. I'll run a modified version of https://gist.github.com/JelteF/1251f180966eb974d7732ea6c3d3b7ff to see if using a map[ban]bool really is better than a []ban at our scale (< ~20 bans at any time) Another thing is that map[ban]bool implies that bans can be stored with the bool set to false, which we do not want. []ban makes it absolutely clear that we will only store the bans, so it's simpler to understand. I like the Twitter and Slack changes, but I would modify them to return a no-op result and print a message indicating that the files are not present instead of carrying on like the API keys are empty (which will cause confusing errors).

Do you think that documenting this behaviour would be enough, or shall I revert the change and keep the O(n) code since we know that for our use case it will be fast enough?

@TommyPujol06 if the file isn't found, you can set the global variable offline to true and it should do everything needed (except print a message that the file doesn't exist)

Now I made it so that a message is printed and offline mode is set.

Do you think that documenting this behaviour would be enough, or shall I revert the change and keep the O(n) code since we know that for our use case it will be fast enough?

Keep the O(n) code.

Now I made it so that a message is printed and offline mode is set.

I just realized that setting offline would disable both slack and twitter if either file is not found, so I would instead replace offline with two new variables: offlineSlack, offlineTwitter. Also, the check for offline happens before you set it, so move the new code before the check. For example, over here: https://github.com/quackduck/devzat/blob/6c82f969ffe9321c0982ce4151246e0a2cf710ff/slack.go#L57

After you revert the map changes, if you'd like, I can help implement the changes to offline.

All right, I added a slack and twitter offline as well as a global offline that adds backwards compatibility.

This looks great! I would just replace errors.Is(err, os.ErrNotExist) with os.IsNotExist() and invert the ifs like so for readability:
-	if !errors.Is(err, os.ErrNotExist) {
-		panic(err)
-	} else {
-		offlineSlack = true
-		l.Println("Did not find slackAPI.txt. Enabling offline mode.")
-	}
+	if errors.Is(err, os.ErrNotExist) {
+		offlineSlack = true
+		l.Println("Did not find slackAPI.txt. Enabling offline mode.")
+	} else {
+		panic(err)
+	}

(easier to parse "if the err is a not exist error, then handle it, otherwise panic" than "if the err is not a not exist error then panic, otherwise handle it")

This looks great! I would just replace errors.Is(err, os.ErrNotExist) with os.IsNotExist() and invert the ifs like so for readability:
-	if !errors.Is(err, os.ErrNotExist) {
-		panic(err)
-	} else {
-		offlineSlack = true
-		l.Println("Did not find slackAPI.txt. Enabling offline mode.")
-	}
+	if errors.Is(err, os.ErrNotExist) {
+		offlineSlack = true
+		l.Println("Did not find slackAPI.txt. Enabling offline mode.")
+	} else {
+		panic(err)
+	}

(easier to parse "if the err is a not exist error, then handle it, otherwise panic" than "if the err is not a not exist error then panic, otherwise handle it")

Done.

Thank you so much! Merging ✅


Re2parser and re2 library

This pull requests:

Adds RE2 library to the 3rd party libraries
Adds parser for transforming RE2 regexps to our NFA. It was implemented by Michal Horky, a student of Lukas.
Adds tests for the mentioned parser.

The pull request is based on integration of RE2 parser and library which Michal Horky did to the automata library devoloped by Lukas and Juraj.

I cleaned up interface and removed re2 dependencies.

seems good to me from a bird's-eye view

maybe also check the pointer is not null...

null pointer check added.


Example format error while passing environment variable

Hello,
I attended a workshop and I had a understanding example on provisioning container instances via Azure CLI by passing environment variable, it seems that the following command is not working ( should use " instead of ' ). I asked another DevOps engineer who is used to working with containers. He never saw this way. Here is the command :
az container create
--resource-group myResourceGroup
--name mycontainer2
--image microsoft/aci-wordcount:latest
--restart-policy OnFailure
--environment-variables 'NumWords'='5' 'MinLength'='8'
The cmdlet works well as follows  :
az container create
--resource-group myResourceGroup
--name mycontainer2
--image microsoft/aci-wordcount:latest
--restart-policy OnFailure
--environment-variables "NumWords"="5" "MinLength"="8"
Related Url : https://docs.microsoft.com/en-us/azure/container-instances/container-instances-environment-variables
Looking forward to hearing from you :)

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

ID: 8cd0d0a1-65c0-ec3f-ab56-ef68f9f819a0
Version Independent ID: 9a0d1414-0cf7-e17d-ca40-5a9cc0ea369e
Content: Set environment variables in Azure Container Instances
Content Source: articles/container-instances/container-instances-environment-variables.md
Service: container-instances
GitHub Login: @dlepow
Microsoft Alias: danlep

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

@cloudmelon - Thanks for raising this issue, and I will investigate. May I ask what shell environment you used to run the command? For example, Command Prompt on Windows, bash on Linux/Mac, Azure Cloud Shell? The example in the article was tested in Azure Cloud Shell.

Hi, sorry, I should provide this information previously, I tested it in Windows 10 bash ( installed Azure CLI ).
Melony QIN

On 15 Nov 2018, at 17:44, Dan Lepow<EMAIL_ADDRESS>wrote:
@cloudmelon - Thanks for raising this issue, and I will investigate. May I ask what shell environment you used to run the command? For example, Command Prompt on Windows, bash on Linux/Mac, Azure Cloud Shell? The example in the article was tested in Azure Cloud Shell.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@cloudmelon - Thanks for the additional information. I reproduce your issue using the Command shell in Windowl and  I will add a note about this in the article.  In other shells, we currently recommend the single quotes for environment variable names and values passed through the CLI. This allows for values that include special characters such as '=' and '&'.

Thank you Dan :)
Melony

On 17 Nov 2018, at 01:32, Dan Lepow<EMAIL_ADDRESS>wrote:
@cloudmelon - Thanks for the additional information. I reproduce your issue using the Command shell in Windowl and I will add a note about this in the article. In other shells, we currently recommend the single quotes for environment variable names and values passed through the CLI. This allows for values that include special characters such as '=' and '&'.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I added a sentence about this to the article, and this update should publish within the next day. Thanks again for submitting the issue and your interest in container instances.
#please-close


Updates ftp transmission task to use the gather_files dict.

Fixes #986

I'm working on some refactoring to address #985


ServiceName cannot have spaces

Hi, the ServiceName doesn't like spaces so instead of a name like "My Demo Service", I have to type it in as "MyDemoService". It will be good for the extension to be able to handle service names with spaces.

Agreed with what its said.
A good thing would be to use quoted arguments by default, eg:
MyService.exe install -username “DOMAIN\Service Account” -password “Its A Secret” -servicename “Awesome Service” –autostart

Agreed! That would make a great feature enhancement. I would happily merge in the pull request and publish the extension.

I agree, this would make a great future enhancement

Fixed with PR #35


Upload python package to pip

The Python variant of the package needs to be uploaded to PyPI.

We are running a few projects with valid and mature pyproject.toml configurations to ease uploading to PyPI. Let us know if we should contribute a corresponding piece here, to save you a few keystrokes.

Hi. I just wanted to have a look at packaging, but then, I discovered in this documentation section, that the parser would actually not be part of the package by default, so it would not be usable right away for people having different expectations?
Let's look into this on behalf of an out-of-band sync, and report back later here about the outcome?

Not part of the repo, we have to create the parser at packaging time so the packages are usable

Hi again. That patch brings a few things into place, in order to prepare packaging and publishing.

GH-10

Because the JavaScript package was already at version 0.0.2, the repository has now been tagged with v0.0.2.
Based on that, and the previous improvements, the Python package has been published to PyPI now.
-- https://pypi.org/project/cratedb-sqlparse/

Amazing, thanks for the help.
We should also document somewhere (maybe in the README) the relation between the cratedb-sqlparse version  and the CrateDB version itself, also I would add a small note on the release notes.


Some yaml files indentations were malformed

Some yaml files indentations were malformed - this caused Prometheus and Grafana to not start

@realmgic Thanks for fixing the indentation. Normally, I prefer .yaml extension as per the standard - https://yaml.org/faq.html.
But I think it's okay if it's creating confusion with the default used by prometheus.io.