Clarification on what "precision" means in roundingPriority: morePrecision/lessPrecision

Hi, I landed here through reviewing the test262 tests for this proposal, specifically this change: https://github.com/tc39/test262/pull/3506
In short it's about the following code:
const formatter = new Intl.NumberFormat('en', {
  useGrouping: false,
  roundingPriority: "morePrecision",
  minimumSignificantDigits: 2,
  minimumFractionDigits: 2,
});
formatter.format("1");

There's a conflict here that roundingPriority is supposed to resolve. If we take minimumSignificantDigits the result is "1.0". If we take minimumFractionDigits the result is "1.00". The test262 test asserts that "1.0" is the correct result.
@romulocintra and I have stepped through the algorithm in the spec, on paper, and have verified that that test262 test conforms. The correct result according to the spec is "1.0". However, if I read the proposal's documentation I'm not really understanding why "1.0" should be correct. Here's what the documentation says:

roundingPriority: "morePrecision" means that the result with more precision wins a conflict.

With my user hat on, who reads the documentation but not the spec, I'd expect that "1.00" has more precision (3 significant digits) than "1.0" (2 significant digits), so the outcome should be that "1.00" wins the conflict.
Where is my understanding faulty? My guess is that the documentation is using a different meaning of "precision" than what I expected. Could the meaning of "precision" be clarified in the documentation?
(Note the documentation actually has this example in it already, but it doesn't mention what the result is :smile:)

Thank you for the issue! I agree that the result is a bit counter-intuitive. Here is my mental model. The choice between "more precision" and "less precision" is only made with respect to the maximum fraction or significant digits. The minimum digits, which only control trailing zeros, play no part in the "more precision" or "less precision" game.
Since maximum significant digits default to 21 and maximum fraction digits normally default to only 3, significant digits will almost always "win" on morePrecision mode unless they are overridden with the maximumSignificantDigits setting.
In other words, the surprising behavior only happens when minimum digits are set in the absence of maximum digits being set.

This algorithm was chosen because it solves our known use cases and reads cleanly in the spec. That said, I am open to exploring alternative algorithms, although doing so would need to meet a high bar since this is a Stage 3 proposal.

I'm not sure anything would need to change, if only the meaning of "more precision" were explained more clearly in the document. Maybe "roundingPriority: "morePrecision" means that the result with more digits of maximum potential precision wins the conflict, regardless of the minimum number of digits"?
Alternatively if you did explore alternative algorithms, for reference here is roughly what I expected when I read the current documentation:

determine the result using the min/max precision digits
determine the result using the min/max fraction digits
pick one result out of the above two, that has the largest number of significant digits (i.e. "1.00" wins over "1.0".)

TC39-TG2 discussion: https://github.com/tc39/ecma402/blob/master/meetings/notes-2022-06-16.md#clarification-on-what-precision-means-in-roundingpriority-moreprecisionlessprecision-96
I plan to draft up the change both here and in ICU to make sure that it behaves the way we expect and satisfies all of the original use cases.

OK, I drafted https://github.com/unicode-org/icu/pull/2260 in ICU to see the impact of the proposed change.
What I found: it appears that all ICU tests pass (have the current behavior) both before and after this change; no expectations needed to be updated. I added a new test where I was able to demonstrate the change in behavior.
This tells me two things:

If we make the change, it will continue to satisfy the original intent and use cases
If we make the change, it is unlikely to make a big impact, since it's a bit hard to reproduce

We already know from experience that there really isn't a strong use case behind setting minimum digits without setting maximum digits. If you set both, this is a non-issue. The Test262 test is just fairly esoteric.
I am therefore inclined to update the docs without touching the spec or implementation.


bbbconfgen: generate files from template with embedded Redis values

Application reads template text file that contains placeholders which specify Redis keys, replace the placeholders with corresponding Redis values and write the final file to disk.
Helpful in configuration management to generate config files. Multiple fallback options are availabe to ensure resilient creation of valid output.
$ bbbconfgen --help

bbbconfgen version 0.1
generates text files from a template, substituting placeholders with Redis values

Command-line arguments:
  --template      input template text file
  --output        output text file
  --redis-addr    redis connection address (default "localhost:6379")
  --redis-db      redis database number
  --redis-pass    redis password
  --verbose
  --version
  --help

Optionally, the output file can be specified on the first line in the template text file.
This line will be dropped and only used if no --output argument is supplied.

  {{ #output: /tmp/output.txt }}

Placeholders in the template text file are defined as follows.
Make sure to respect spaces between arguments.

  {{ key }}                     is replaced by Redis 'key', only if key is present
  {{ key #rm }}                 ...deletes the placeholder if key not found
  {{ key #rmLine }}             ...deletes the whole line if key not found
  {{ key #default: some val }}  ...uses default value if key not found

Dependency github.com/gomodule/redigo/redis is not yet included. I'll ask @Tomasvrba tomorrow how to do that best.

Looks great to me on first look, but will need to take a deeper look next week when I dive into base stuff

Once the script redis-pipe.sh is merged, Redis mass insertion can be executing by piping a regular text file with Redis cli commands (e.g. SET key value) through the script to redis-cli:
cat redis-commands.txt | sh redis-pipe.sh | redis-cli --pipe

For the example, I replaced bitcoin-redisimport.txt with regular bash commands, for CI testing I think the file rest-redisimport.txt is ok to use with some documentation.


buildah-unwrapped: 1.33.2 -> 1.34.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/buildah/versions.
meta.description for buildah-unwrapped is: A tool which facilitates building OCI images
meta.homepage for buildah-unwrapped is: https://buildah.io/
meta.changelog for buildah-unwrapped is: https://github.com/containers/buildah/releases/tag/v1.34.0
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.34.0 with grep in /nix/store/q9lpxkdky1ngarhjqmzl0h18xk09gzyd-buildah-1.34.0
found 1.34.0 in filename of file in /nix/store/q9lpxkdky1ngarhjqmzl0h18xk09gzyd-buildah-1.34.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
buildah
buildah-unwrapped
krunvm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q9lpxkdky1ngarhjqmzl0h18xk09gzyd-buildah-1.34.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A buildah-unwrapped https://github.com/r-ryantm/nixpkgs/archive/c9c1b4aa6120cb4f7d6fe184aea4ec997374fb98.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c9c1b4aa6120cb4f7d6fe184aea4ec997374fb98#buildah-unwrapped

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q9lpxkdky1ngarhjqmzl0h18xk09gzyd-buildah-1.34.0
ls -la /nix/store/q9lpxkdky1ngarhjqmzl0h18xk09gzyd-buildah-1.34.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

buildah
buildah-unwrapped
krunvm

Maintainer pings
cc @adisbladis @saschagrunert @vdemeester for testing.

@ofborg build buildah-unwrapped


Nicer API for using credentials

ArangoDB 3 enables auth by default. Passing credentials to arangojs is a bit awkward at the moment (need to construct a URL). This could be improved.

This has been addressed with useBasicAuth.


'Microsoft.Isam.Esent.Interop.Api' threw an exception

Get the following error-
Get-ADDBAccount : The type initializer for 'Microsoft.Isam.Esent.Interop.Api' threw an exception.                                 At /root/pwcheck/pwcheck.ps1:1 char:1                                                                                + Get-ADDBAccount -All -DBPath ntds.dit -BootKey 4e439b5d264f1f475c52d5 ...

CategoryInfo          : OpenError: (:) [Get-ADDBAccount], TypeInitializationException
FullyQualifiedErrorId : DBContextError,DSInternals.PowerShell.Commands.GetADDBAccountCommand

Using this command-
Get-ADDBAccount -All -DBPath ntds.dit -BootKey 4e439b5d264f1f475c52d57100cc83ab | Test-PasswordQuality -WeakPasswordHashesFile passwords.txt -IncludeDisabledAccounts | Out-File results.txt
Any ideas?
I've installed Powershell using the 'snap install powershell --classic' command on Ubuntu 18.04

Hi, DSInternals uses esent.dll, a native Windows library, for AD database operations. As a consequence, Linux is not a supported platform for running DSInternals.
Michael Grafnetter

From: Filk47473831<EMAIL_ADDRESS>Sent: Thursday, June 6, 2019 12:00:12 PM
To: MichaelGrafnetter/DSInternals
Cc: Subscribed
Subject: [MichaelGrafnetter/DSInternals] 'Microsoft.Isam.Esent.Interop.Api' threw an exception (#86)
Get the following error-
Get-ADDBAccount : The type initializer for 'Microsoft.Isam.Esent.Interop.Api' threw an exception. At /root/pwcheck/pwcheck.ps1:1 char:1 + Get-ADDBAccount -All -DBPath ntds.dit -BootKey 4e439b5d264f1f475c52d5 ...

CategoryInfo : OpenError: (:) [Get-ADDBAccount], TypeInitializationException

FullyQualifiedErrorId : DBContextError,DSInternals.PowerShell.Commands.GetADDBAccountCommand

Using this command-
Get-ADDBAccount -All -DBPath ntds.dit -BootKey 4e439b5d264f1f475c52d57100cc83ab | Test-PasswordQuality -WeakPasswordHashesFile passwords.txt -IncludeDisabledAccounts | Out-File results.txt
Any ideas?
I've installed Powershell using the 'snap install powershell --classic' command on Ubuntu 18.04
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/MichaelGrafnetter/DSInternals/issues/86?email_source=notifications&email_token=ACER6CGZ3DQLDHF55MCPFRTPZFNDZA5CNFSM4HVHNI62YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GYDD4BA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACER6CH36PWZOK5JTX43SQ3PZFNDZANCNFSM4HVHNI6Q.


Audio Not Working

Hey guys,
When I upload MP3s the Alexa service doesn't seem to recognise it at all. Any ideas? This is for flash news briefing.

@goos3die can you provide a sample file or the URL of the endpoint?

Sure thing.
So right now the skill is working for text-to-voice briefings at https://goos3d.ie/wp-json/voicewp/v1/skill/briefing - but previously, using the plugin I'd just upload audio and that would work too. Now it's not.

I’m seeing this same behavior. I upload an mp3, and it only reads the title but doesn’t play the enclosure.

I'm going to run some tests today. @goos3die since the feed doesn't contain an mp3, can you send me a link to one of the mp3 files that weren't working? @deskpops ditto.

This was mine: https://alexaguy.com/wp-content/uploads/2018/02/daily-briefing-1-122617-4.29-PM.mp3
Thanks for your help!

Cheers Tom. Here's mine - https://goos3d.ie/wp-content/uploads/2018/02/flash-news-briefing-010218.mp3
Thanks for your help man.


gradle error when i add your plugin to dependencies

hello
when i just add your plugin to dependencies i got this error with gradle without even import or use this plugin in any page or dart code
`
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:transformClassesWithMultidexlistForDebug'.

com.android.build.api.transform.TransformException: Error while generating the main dex list.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 4s
Finished with error: Gradle task assembleDebug failed with exit code 1
`
thank you

You might have other plugins as well due to which total class limit in android might have exceed. You'll need to enable multidex for it.

I Have a lot of  plugins and enabled multidex in gradle file
shared_preferences: ^0.5.3+4   cupertino_icons: ^0.1.2   retrofit_generator: ^1.0.1   build_runner: ^1.7.1   json_serializable: ^3.2.2   http: ^0.12.0+2   fluttertoast: ^3.1.3   url_launcher: ^5.1.0   chewie: ^0.9.8+1   video_player: ^0.10.2+5   cached_network_image: ^2.0.0   intl: ^0.16.0   firebase_messaging: ^5.1.6   firebase_dynamic_links: ^0.5.0+6   flutter_html: ^0.10.4   cloud_firestore: ^0.12.10+2   firebase_analytics: ^5.0.6   flutter_launcher_icons: ^0.7.3   flutter_share: ^1.0.2+1   webview_flutter: ^0.3.19+5   flutter_spinkit: ^4.1.1   youtube_player_flutter: ^6.0.3+2   package_info: ^0.4.0+13   firebase_crashlytics: ^0.1.3   flutter_widgets: ^0.1.11   icons_helper:     git: git://github.com/sjmcdowall/icons_helper   audioplayers: 0.13.2   permission_handler:     git:       url: https://github.com/Baseflow/flutter-permission-handler       ref: develop   screen: ^0.0.5
but importing youtube_player_flutter still causes error: Execution failed for task ':app:transformClassesWithMultidexlistForDebug'.


System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Actor\Pack\

Whenever I try to use the tool it gives me this error. I don't know what I am doing wrong

Please help

Make sure you have set your game path in the settings.

Now it's giving me this error

Sorry for the trouble

Ah, now that may be an error on my end, I'll look into it.

Okay thank you

Ah, I see the issue, change the BaseActorName to nothing. That field is for a vanilla actor name to use when building the custom one.

Hey I'm really sorry to keep bothering you but that didn't seem to work. I'm still getting the same error but now it's just saying tis:

I'm really sorry for the trouble

I must have made a mistake when checking the field. Please try restarting the app and avoid filling out that field.

Tried that and it just keeps defaulting to "TwnObj_HunterHouseBed_A_01" whenever I run it for some reason
I am very sorry for the trouble

That is expected behaviour, however, if you update to 2.0.1 you can clear it without an issue.

It's still defaulting but now it's saying this
"System.IO.FileNotFoundException: The actor 'TwnObj_HunterHouseBed_A_01' is not a vanilla actor. Please change the 'Base Actor' field."
Even when I clear it and run it again it just defaults back again

It should default to that because it automatically looks for the best match. However, that actor does exist, so the issue is with your game dump. Possibly the path is wrong, or else the dump is incomplete.

I have the actor in my game directory. What should the path look like so I can verify if I have it right?
Sorry again for the trouble

I have the actor in my game directory. What should the path look like so I can verify if I have it right?
Sorry again for the trouble

It depends on where it's stored on your computer, but it should end in content (or the folder containing Actor, but not the Actor folder itself).

There we go! Finally! I had the game directory folder wrong I think.
Thank you very much for the help. Sorry it ended up being so complicated

No worries, I'm glad you got it sorted.


Suggestion: try to add an "archiving" section

Suggestion: try to add an "archiving" section
This is important to avoid having or being forced to delete old issues/problems/decisions when they're no longer relevant.
This would be nice to see what you did previously in hindsight, instead of having to delete it.
Also, this would keep the main window/view/section clean, instead of it being cluttered with non-relevant items/issues/decisions (that is, if you don't delete the items, of course).
Obviously, eventually you can export and import again, but that might not be as convenient as being able to access old "items" inside an "archive" section/field.
That's it.

Update: export/import seem to not be working properly.
Tried to create one extra item, export it, delete that item and then import the previous save/export and the item didn't reappear (and eventually the app began malfunctioning/crashing for a while, until it got back to normal).

Good idea! 👍  And yes, I can totally understand that exporting and importing again is quite cumbersome and not a good solution to this. So I will add an Archive as soon as I find the time.


(glib) Can we support 2.40?

Currently, only >=2.42 is supported, but Travis CI uses Ubuntu 14.04 by default, which has 2.40. Is 2.42 mandatory or can we downgrade it a bit?

Travis has Xenial machines too: https://github.com/gtk-rs/sys/blob/master/.travis.yml#L1

Yes, but

uses Ubuntu 14.04 by default

But support 14.04 ends in this year: https://www.ubuntu.com/about/release-cycle

If 2.40 can't be supported by just lowering the minimum versions than it's fine.
Is there a list of minimal required versions for all gtk-rs crates?

Default versions is released with gtk3.14 (I don't have link)
others listed in https://github.com/gtk-rs/examples/blob/master/Cargo.toml#L25

I meant for all crates, like cairo, pango, etc. Yes, you can look in *-sys, but it's a bit troublesome.

Agree that we need table somewhere on https://gtk-rs.org/

Can someone edit this issue? It should be about adding a table to the website about the minimum supported versions of the C libraries, and the latest version for which we have bindings right now.
@GuillaumeGomez ?


Update Beast package to version 54

Changes from the last update:
Version 54:

static_buffer coverage
flat_buffer coverage
multi_buffer coverage
consuming_buffers members and coverage
basic_fields members and coverage
Add string_param
Retain ownership when reading using a message
Fix incorrect use of [[fallthrough]]

API Changes:

basic_fields refactor

Version 53:

Fix basic_parser::maybe_flatten
Fix read_size_helper usage

Version 52:

flat_buffer is an AllocatorAwareContainer
Add drain_buffer class

API Changes:

auto_fragment is a member of stream
binary, text are members of stream
read_buffer_size is a member of stream
read_message_max is a member of stream
write_buffer_size is a member of stream
ping_callback is a member of stream
Remove opcode from read, async_read
read_frame returns bool fin
opcode is private
finish(error_code&) is a BodyReader requirement

Actions Required:

Change call sites which use auto_fragment with set_option
to call stream::auto_fragment instead.

Change call sites which use message_type with set_option
to call stream::binary or stream::text instead.

Change call sites which use read_buffer_size with set_option to
call stream::read_buffer_size instead.

Change call sites which use read_message_max with set_option to
call stream::read_message_max instead.

Change call sites which use write_buffer_size with set_option to
call stream::write_buffer_size instead.

Change call sites which use ping_callback1 with set_optionto callstream::ping_callback` instead.

Remove the opcode reference parameter from calls to synchronous
and asynchronous read functions, replace the logic with calls to
stream::got_binary and stream::got_text instead.

Remove the frame_info parameter from all read frame call sites

Check the return value 'fin' for calls to read_frame

Change ReadHandlers passed to async_read_frame to have
the signature void(error_code, bool fin), use the bool
to indicate if the frame is the last frame.

Remove all occurrences of the opcode enum at call sites

Version 51

Fix operator<< for header
Tidy up file_body
Fix file_body::get() not setting the more flag correctly
Use BOOST_FALLTHROUGH
Use BOOST_STRINGIZE
DynamicBuffer benchmarks
Add construct, destroy to handler_alloc
Fix infinite loop in basic_parser

API Changes:

Tune up static_buffer
multi_buffer implementation change

Actions Required:

Call sites passing a number to multi_buffer's constructor
will need to be adjusted, see the corresponding commit message.

Version 50

parser is constructible from other body types
Add field enumeration
Use allocator more in basic_fields
Fix basic_fields allocator awareness
Use field in basic_fields and call sites
Use field in basic_parser
Tidy up basic_fields, header, and field concepts
Fields concept work
Body documentation work
Add missing handler_alloc nested types
Fix chunk delimiter parsing
Fix test::pipe read_size
Fix chunk header parsing

API Changes:

Remove header_parser
Add verb to on_request for parsers
Refactor prepare
Protect basic_fields special members
Remove message connection settings
Remove message free functions
Remove obsolete serializer allocator
http read_some, async_read_some don't return bytes

Version 49

Use  instead of

HTTP:

Add HEAD request example

API Changes:

Refactor method and verb
Canonicalize string_view parameter types
Tidy up empty_body writer error
Refactor header status, reason, and target

Version 48

Make buffer_prefix_view public
Remove detail::sync_ostream
Tidy up core type traits

API Changes:

Tidy up chunk decorator
Rename to buffer_cat_view
Consolidate parsers to parser.hpp
Rename to parser

Version 47

Disable operator<< for buffer_body
buffer_size overload for basic_multi_buffer::const_buffers_type
Fix undefined behavior in pausation
Fix leak in basic_flat_buffer

API Changes:

Refactor treatment of request-method
Refactor treatment of status code and obsolete reason
Refactor HTTP serialization and parsing

Version 46

Add test::pipe
Documentation work

API Changes:

Remove HTTP header aliases
Refactor HTTP serialization
Refactor type traits

Version 45

Workaround for boost::asio::basic_streambuf type check
Fix message doc image
Better test::enable_yield_to
Fix header::reason
Documentation work
buffer_view skips empty buffer sequences
Disable reverse_iterator buffer_view test

Version 44

Use BOOST_THROW_EXCEPTION
Tidy up read_size_helper and dynamic buffers
Require Boost 1.58.0 or later
Tidy up and make get_lowest_layer public
Use BOOST_STATIC_ASSERT
Fix async return values in docs
Fix README websocket example
Add buffers_adapter regression test
Tidy up is_dynamic_buffer traits test
Make buffers_adapter meet requirements

Version 43

Require Boost 1.64.0
Fix strict aliasing warnings in buffers_view
Tidy up buffer_prefix overloads and test
Add write limit to test::string_ostream
Additional constructors for consuming_buffers

Version 42

Fix javadoc typo
Add formal review notes
Make buffers_view a public interface

Version 41

Trim Appveyor matrix rows
Concept revision and documentation
Remove coveralls integration
Tidy up formal parameter names

WebSocket

Tidy up websocket::close_code enum and constructors

API Changes

Return http::error::end_of_stream on HTTP read eof
Remove placeholders
Rename prepare_buffer(s) to buffer_prefix
Remove handler helpers, tidy up hook invocations

Thanks for taking the time to make a PR over here!


Azure WebApp, MSI Token Failure, missing 'expires_on' field in response

Driver version

=10.2.3 + 11.x

Client Operating System
Linux / Docker
JAVA/JVM version
Java 17
Problem description
We are using Azure WebApp to run our Java/SpringBoot applications. For the authentication we are using the system assigned identity. That works well so far with JDBC version 9.4.1. Now we wanted to upgrade the JDBC driver version to 11.2.3.
With all JDBC versions from 10.2.3 on, we are getting the following error: com.microsoft.sqlserver.jdbc.SQLServerException: MSI Token failure: Failed to acquire token from MSI Endpoint.
After investigation, I found the following code change to be responsible for the issue (version 10.2.2 --> 10.2.3):

Before 10.2.3 the library was using the expires-on field of the response. But with version 10.2.3 it now uses the expires-in field. But this field is not existing in the response of the provided token service. Here is a (shorted) response example:
{"access_token":"ey...w","expires_on":"1680169483","resource":"https://database.windows.net/","token_type":"Bearer","client_id":"51481984-5176-4f0f-b02e-7812ea679522"}

As you can see, the expires-in field is completely missing there. Any idea what is wrong here?

Hi @Jeffery-Wasty
Thanks a lot for your answer. We also tried to directly go to version 12.2.0, but there we get the following exception (again it is working in a VM but not on a Azure WebApp:
java.io.IOException: Server returned HTTP response code: 401 for URL: http://<IP_ADDRESS>:8081/msi/token?resource=https%3A%2F%2Fdatabase.windows.net%2F&api-version=2017-09-01

Do you have any idea about this issue?
Disabling TokenCache would be a valid option, I would say. Can this property be set in the JDBC connection string? Or does it need to be configured programmatically?

Hi @Jeffery-Wasty again
Sorry for coming back again. Your suggestion to use a msiTokenCacheTtl of 0 doesn't solve the issue at all:

Still getting the famous
com.microsoft.sqlserver.jdbc.SQLServerException: MSI Token failure: Failed to acquire token from MSI Endpoint

Hi @Nyffeneggerr,
It looks like there might be some confusion on our end. The changes we made for 12.1 should work for Azure Functions, but may not work for Azure WebApps. We're now examining whether we'll need to make additional changes to ensure compatibility for your use case. Thank you for bringing this to our attention.

Hi @Nyffeneggerr, for 10.2.3 it looks like a regression on webapps with MSI (I was able to reproduce your issue) we are in progress of doing more investigating. For 12.2.0, we were able to successfully auth with msi. Your Server returned HTTP response code: 401 error makes me think it may be a permissions issue. Does the managed identity used on the azure webapp with 12.2.0 work on a VM as well?

Hi @tkyc
I'm not 100% sure what you mean. We are using system assigned identities and they cannot be used across different services. But I always used the same webapp for testing different versions of JDBC drivers. With 10.2.2. everything worked for the given identity and I could fetch example data from database. With 12.1 + 12.2, the 401 appears.
I tested the same drivers with the same (test-) application on a VM (but with a different system assigned identity, because it's another service). There, everything with each version (9.x + 10.x + 11.x + 12.x) is working flawless.
Does this answer your question?

@Nyffeneggerr Yes, that answers my question thanks. This very strange, can you share a repro application that reproduces your 401 error? I want to test it on my end as I'm able to use system managed identities with webapps.

In case it helps, and for visibility, I've attached my test app to this comment that I used to perform a successful auth with 12.2.0.
test-app.zip

Found the same issue after upgrading my app to spring boot 3.0.5. Updated the JDBC driver to 12.2 and added the msal4j and azure-identity dependencies according to the above test-app. The connection worked fine on spring boot 2.7.x where the JDBC driver used the 10.2.1.jre8 version without explicitly specifying the msal4j and azure-identity dependencies.

Hi @tkyc
Yes, I can give you the demo application. Its not much more than the demo code from the Microsoft website here: https://learn.microsoft.com/en-us/sql/connect/jdbc/connecting-using-azure-active-directory-authentication?view=sql-server-ver16#connect-using-activedirectorymanagedidentity-authentication-mode
database-jdbc-test-app.zip

@Nyffeneggerr Thanks. I'm able to reproduce the error and I was able to fix it. So, you'll need to specify msal4j 1.13.3 and azure-identity 1.7.0 in your gradle file to fix the issue. These are the exact dependency versions the 12.2.0 driver are built with.
@vitorcsbrito Can you confirm the msal4j and azure-identity versions you used?
Let me know if that resolves the issue for you guys.

Our doc work-with-azure-active-directory-authentication should specify the dependency versions appropriate with each driver version. It looks like for 12.2.0 msal4j 1.13.3 wasn't specified, we'll make an update.

Just to add to the above, we should explicitly add to the doc @tkyc mentioned for reference however all the versions of dependent libaries are in explicit in the pom file in each release. One should always make sure to use the exact versions as specified in the pom file otherwise unexpected results may occur.
Please let us know if this resolves your issue or if you have more questions otherwise we will be closing this issue soon.

I had initially tried with the same version for msal4j but azure-identity 1.8.1.  Just gave a go at azure-identity 1.7.0 and it worked perfectly! Thanks a lot for the help

I can also confirm that azure-identity 1.7.0 works as expected with JDBC 12.2.0. So far you can close the ticket from my side. It would still be interesting (maybe not for us, but for others) if there is a chance that versions >10.2.2<12.0.0 can work with system assigned identity in Azure WebApps.
Thanks a lot for your help!

Awesome, glad it worked. As for drivers >10.2.2<12.0.0 we'll be releasing a hotfix and when that comes around I'll let you guys know in this thread.

Closing since it's addressed in hotfix releases.

Is it possible to publish these hotfix releases to Maven Central?
They're still unavailable there and, as a result, they aren't included in the Spring-Boot BOM
Thank you in advance.


Update Spec for ClassExpression

Currently the compiler support parsing and emitting classExpression, we should update our spec to reflect this update

Class expression support is not scheduled until 1.6

The spec is now an "archived" artifact and we won't be tracking defects against it anymore.


Uses open instead of executing from the editor

The original implementation launched the executable directly from vscode integrated terminal, this caused issues with macOS apps that use specific entitlements, while it's not clear why it appears to be some kind of system protection issue. This PR fixes the issue by using the systems open command which launches the application indirectly. This in turns resolves the issue as per #47
Fixes https://github.com/sweetpad-dev/sweetpad/issues/47

Sorry for the previous PR originally I thought the issue was a code-signing one and went down a rabbit hole. This fix is much simpler and directly fixes my issues.

Only, problem I'm seeing with this is that we lose the stderr and stdout.  I could open the executable directly instead of app bundle and this would open a external terminal session which would would show stdout and stderr but this isn't a nice solution as it takes you out of vscode.

Ok, one solution that works is to use lldb instead of open or executing the application directly. This mirrors Xcode's "Run" (Cmd+R) action which builds, run and attaches a debugger. While this is a change in behaviour it's actually with iOS/macOS developers would expect Run to do anyway.
I would also suggest the same is done for apps running in the iOS simulator. Having to run and then later attach the debugger is problematic for debugging events that happen at launch time. Though this PR doesn't make that change.

For some reason, the version with the LLDB debugger doesn't work for me. Maybe I should give it a chance tomorrow.
Also, I’ve found a way to redirect stdout, but I haven't tried how it would work in vscode:
open -W  --stdout $(tty) /Users/hyzyla/Library/Developer/Xcode/DerivedData/ControlRoom-gdvrildvemgjaiameavxoegdskby/Build/Products/Debug/Control\ Room.app/

That's interesting, I personally like the launching with debug session option as this not only fixes the issues I was having it's also representative to how Xcode works when running the app. It always starts a debug session, this was already a pain point for me using this library having to manually attach the debugger.
@hyzyla what error did you see when trying to launch the app using lldb?

For some reason, the version with the LLDB debugger doesn't work for me. Maybe I should give it a chance tomorrow.
Also, I’ve found a way to redirect stdout, but I haven't tried how it would work in vscode:
open -W  --stdout $(tty) /Users/hyzyla/Library/Developer/Xcode/DerivedData/ControlRoom-gdvrildvemgjaiameavxoegdskby/Build/Products/Debug/Control\ Room.app/

This looks like a great quick fix, maybe we could add new build options (run with debugger) and run (without debugging) that way we can choose.

Technically if you honour the selected scheme sweetpad should attach debugger if the scheme has that option enabled:

Currently the run command ignores this. Personally I think it should honour this and use lldb as per my PR if this option is enabled in the scheme otherwise use the open command. This should be the same for running apps in simulator too.

I am going to close this, for now, will have another PR up shortly that doesn't break existing functionality


Refactor: edit users as members of projects and groups

instead of removing and re-adding them for a cleaner audit log.
Also stop allowing to pass user_id separately from username

this is unnecessary complexity with little benefit (1 less API call in some rare cases).

Related to #466.

Codecov Report

Merging #469 (80c8928) into main (f2e30a6) will decrease coverage by 44.51%.
The diff coverage is 15.15%.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #469       +/-   ##
===========================================
- Coverage   85.35%   40.83%   -44.52%
===========================================
  Files          70       70
  Lines        2628     2635        +7
===========================================
- Hits         2243     1076     -1167
- Misses        385     1559     +1174

Impacted Files
Coverage Δ

...abform/processors/group/group_members_processor.py
15.11% <0.00%> (-83.74%)
:arrow_down:

gitlabform/processors/project/members_processor.py
16.92% <0.00%> (-73.85%)
:arrow_down:

gitlabform/gitlab/members.py
24.00% <20.00%> (-73.62%)
:arrow_down:

...tlabform/processors/multiple_entities_processor.py
20.25% <0.00%> (-75.95%)
:arrow_down:

gitlabform/lists/projects.py
20.58% <0.00%> (-73.53%)
:arrow_down:

gitlabform/lists/groups.py
26.19% <0.00%> (-71.43%)
:arrow_down:

...tlabform/processors/project/schedules_processor.py
21.15% <0.00%> (-71.16%)
:arrow_down:

...labform/gitlab/project_merge_requests_approvals.py
20.83% <0.00%> (-66.67%)
:arrow_down:

... and 53 more


Bump latest-version.

🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Description
A deep transitive dependency is got, which has a security alert. Bumping latest-version gets us to the latest version.
Related # (issue)

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Will probably need some help testing.
Checklist:

[ ] I have followed the CONTRIBUTING doc and the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

This breaks node 12 support, and also builds will fail. I made an issue #7990 to track this. I'll make a separate PR.

Thanks @DarryQueen ! Closing in favor of @kevcube's PR :)


data/store to normal object

Hello i work on a project with vue.js.
But sometimes i need the raw data.
Actually same problem with this issue; How to convert the object of Vue to normal object?

JSON.parse(JSON.stringify(vm.$data))

this solution is great but if you need it more than few times it's uncomfortable. so i wrote normalize;
/** vue */
vm.$data.normalize(); // return to normal object

/** vuex */
this.$store.state.normalize(); // return to normal object

Sorry, but I think it's a bad idea to extend data objects like that. This could easily lead to naming conflicts. Just creat util functions of your own.


[Enhancement] Allow new_key_type! to declare generic key type

Such as
new_key_type! {
    pub struct NodeKey<T>;
}

Any updates on this? I was hoping to do this exact thing as well. Basically so you don't have to manually create "NodeKey" for all types.


pretrained model

would you mind sharing your pretrained model or demo for more exploring

Hi, I do not offer the pretrained model since both of the data and model are very small, it only takes a few minutes to train them. And, I suppose the seed setting may affect the results sometimes, but more or less you can obtain the performance as I claimed. The demo link is given in https://colab.research.google.com/drive/19gq3bUigdxIfyMCoWW93YhLEi1KQlBit, just download the data from my /data folder and upload it for running.

thx, I try, good work

DD_Net.load_weights('weights/fine_heavy.h5')
where can I find or get the fine_heavy.h5

I got your point. Sorry for this confusion setting. I use it for testing purpose. It can be ignored and train the model from scratch.

@fandulu
I got the keypoint from the data folder and they are, as following:
[[ 0.286 -0.377]
[ 0.286 -0.066]
[ 0.261 -0.451]
[ 0.189 -0.332]
[ 0.392 -0.362]
[ 0.256  0.002]
[ 0.295  0.032]
[ 0.106 -0.269]
[ 0.512 -0.387]
[ 0.257  0.208]
[ 0.244  0.262]
[-0.05  -0.218]
[ 0.638 -0.42 ]
[ 0.419  0.189]
[ 0.111  0.476]]
I want to kwon how to deal with the keypoint to get the format like above. If I have the new keypoint from the openpose? It is each keypoint's x and y coordinate subtract the keypoint center(x_kp_new = x_kp_old - all_x_kp_center)/all_x_kp_center?
I try to get the information from the JHMDB'paper, but I did not find clearly information about it

pose_path = './joint_positions.mat'
pose = read_pose(pose_path)
print(len(pose))
print(pose[0])
[[ 0.286 -0.377]
[ 0.286 -0.066]
[ 0.261 -0.451]
[ 0.189 -0.332]
[ 0.392 -0.362]
[ 0.256 0.002]
[ 0.295 0.032]
[ 0.106 -0.269]
[ 0.512 -0.387]
[ 0.257 0.208]
[ 0.244 0.262]
[-0.05 -0.218]
[ 0.638 -0.42 ]
[ 0.419 0.189]
[ 0.111 0.476]]

In my project, it is the center of the image other than the center of poses since the global coordinates are used for calculating the motion features. However, it is really depended on your target task, like the paper explained, when your target does not correspond to global trajectories, use pose center is also fine.

@fandulu yeah, my target dose try to deal with the global trajectories, I just would like to make sure it. I would be better if you support one demo or example about the new video as input, which have been processed to the pose format.
Thanks

@fandulu and by the way, the print(pose[0])
[[ 0.286 -0.377]
[ 0.286 -0.066]
[ 0.261 -0.451]
[ 0.189 -0.332]
[ 0.392 -0.362]
[ 0.256 0.002]
[ 0.295 0.032]
[ 0.106 -0.269]
[ 0.512 -0.387]
[ 0.257 0.208]
[ 0.244 0.262]
[-0.05 -0.218]
[ 0.638 -0.42 ]
[ 0.419 0.189]
[ 0.111 0.476]]
, it is computed by (x-center_x)/width, the (x,y) is the pose's position and the (center_x, center_y)is the image's center and the (width, height)is the image's width and height?


Unit Testing Coverage

This PR introduces unit test coverage for OV-TORT

This PR introduces unit test coverage for OV-TORT

Hi @ashishbdatta, can you please raise PR request to GA branch: https://github.com/intel-staging/ort/tree/ov_torchort_integration_ga

This PR introduces unit test coverage for OV-TORT

Hi @ashishbdatta, can you please raise PR request to GA branch: https://github.com/intel-staging/ort/tree/ov_torchort_integration_ga

Done


Heapster can't access nodes and pods

jx/jenkins-x-heapster-65fd697bb-v4blk[heapster]: E1111 12:36:16.766318       1 reflector.go:190] k8s.io/heapster/metrics/util/util.go:51: Failed to list *v1.Node: nodes is forbidden: User "system:serviceaccount:jx:default" cannot list nodes at the cluster scope
jx/jenkins-x-heapster-65fd697bb-v4blk[heapster]: E1111 12:36:16.766369       1 reflector.go:190] k8s.io/heapster/metrics/heapster.go:322: Failed to list *v1.Pod: pods is forbidden: User "system:serviceaccount:jx:default" cannot list pods at the cluster scope

Using helmBinary helm with feature flag: none
NAME               VERSION
jx                 1.3.516
jenkins x platform 0.0.2871
Kubernetes cluster v1.11.2-gke.15
kubectl            v1.12.0
helm client        v2.11.0+g2e55dbe
helm server        v2.11.0+g2e55dbe
git                git version 2.19.1

I didn't notice it was all warnings accessing cluster scope, I'm guessing that's the intended behavior and heapster is just noisy about it


Facter variable for PHP Extension Version, addresses issue #102, replacing pull request #103

Hi,
This is a new pull request replacing pull request #103.
It adds a php_extension_version fact with only one pipe and "20121212" as default value.
Also, I added .project to .gitignore, because I worked with Gepetto IDE.

:+1:

thank you @danygielow!


Post-Identity-Linking-Redirect-Url header is not present in initial request

Hi,
'Post-Identity-Linking-Redirect-Url' header is not present in the initial request to my action endpoint. I am using Office365 ProPlus to trigger the action.

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

ID: 3718830a-252f-fbb0-a915-7bbb7a1583af
Version Independent ID: c97c6ce0-a184-d58e-0742-0289e3eaeeb8
Content: Link Azure Active Directory identity with your own identity provider (Preview) - Outlook Developer
Content Source: docs/actionable-messages/identity-linking.md
Product: outlook
Technology: o365-connectors
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

What type of card are you doing this with?

Hi, sorry for not mentioning it earlier. I am trying this with https://amdesigner.azurewebsites.net/. When I trigger HTTP action from outlook and observe headers in my API, this particular header is not present.
I am not sure if this is intended or not. If it is, it should be documented.

Since that tool creates Adaptive Card messages, it should work. I've tested it yesterday to confirm. Can you post your card JSON? Also, are you sending this to an Office 365 user?

I verified again and in fact, redirect url is present but with name 'identity-linking-redirect-url'.

Ah! Yep, I see that too. Didn't notice they dropped the "Post-". Let me check with the team on that.

Ok..I've confirmed that the header you're seeing is the correct expected header. The docs have the wrong value. I'll get that updated.


Purpose of sub package check

This is not a bug report, I just want to understand something.
I've used abapgit to export everything in a Z-Package that has many subpackages, some of which are associated with abapgit repos.
When trying to do an export to a zip file, the following check (251 to 260) prohibits it:
https://github.com/abapGit/abapGit/blob/fa66b626de7c578428bd07519be2d0d1bed356e3/src/repo/zcl_abapgit_repo_srv.clas.abap#L251
I've commented these lines out, to continue and later put them back in.
I think it would make more sense to issue a warning or something, because if you just want to export stuff, subpackages which are associated with repos are not a problem.
Is there something I'm missing here?

You are right when it comes to "just exporting something" but a) the sub-package logic is used in other places, and b) abapGit matches exports to what's in a repo.
Sounds like you're trying to backup a part of your system. That is not the purpose of abapGit.
I suggest looking into the APIs and exporting the repos which you already have in the sub-packages automatically. The it becomes a repeatable process as well.

The "export to zip" was added after the "offline repository" feature, so I guess it uses some of the same code.
I'd see it as a bug, the "export to zip" should just export whatever asked for. While creating repositories keeps the check

@danielwegner can you please test #6815?

@danielwegner can you please test #6815?

Well, it works, but it's not exactly what I wanted.
The "Package to zip" feature does exactly what I want and I wasn't aware of it. Thanks!
Sorry for taking so long to respond.


VideoHttpExecutor on hub not returning videos correctly?

We are running this version for hub and nodes: SeleniumGridExtras-1.7.1-SNAPSHOT-jar-with-dependencies.jar
Could have sworn this was working yesterday...now when we try to load any video from the hub's Selenium-Grid-Extras video list, we get a video page that doesn't load and this:
31 Mar 2015 15:32:36 [Thread-2] INFO  seleniumgridextras.VideoHttpExecutor  - New video download request for /srv/SeleniumGridExtras/video_output/cd17e72a-0009-4cf3-8ca0-151265f2ef52.mp4
31 Mar 2015 15:32:36 [Thread-2] INFO  seleniumgridextras.VideoHttpExecutor  - Requested video /srv/SeleniumGridExtras/video_output/cd17e72a-0009-4cf3-8ca0-151265f2ef52.mp4 does exist, responding with 404

I rebooted the server just in case...no change to behaviour.  The only difference is how the devs are sending tests (but even this I'm not sure of...).
Also - the text 'does exist' - should that be 'does not exist' ?

Did more digging, found this:
./log/grid_extras.log.2015-03-30:31 Mar 2015 13:56:50 [pool-1-thread-88] INFO  video.VideoDownloaderCallable  - Successfully downloaded video for session cd17e72a-0009-4cf3-8ca0-151265f2ef52, from host <IP_ADDRESS>, to directory on hub /srv/SeleniumGridExtras/video_output/cd17e72a-0009-4cf3-8ca0-151265f2ef52.mp4
./log/grid_extras.log.2015-03-30:31 Mar 2015 13:56:50 [pool-1-thread-88] ERROR video.VideoDownloaderCallable  - Something unexpected happened response {exit_code=1.0, error=[Cannot call this endpoint without required parameters: session and action],
[list of available videos follows]
[this happens for 2-5 of 5 attempts]
./log/grid_extras.log.2015-03-30:31 Mar 2015 13:56:50 [pool-1-thread-88] WARN  video.VideoDownloaderCallable  - Failed to download video for session cd17e72a-0009-4cf3-8ca0-151265f2ef52 from host <IP_ADDRESS> after 5 attempts

So the reason why the video is not playing is because it's an invalid file!
Just have to figure out why that node is acting up.

Yup, then I see this on the node:

Will reboot the node, see if this continues.

After rebooting the node, things are back to normal.  This node gets run infrequently, so I'll just set it to reboot after every test. (was set to every 10 tests)
Aside from that little typo, this can be closed.


Search is blocked by this

I love the cleaner UI but I went to search (or "explore", ugh) and the search bar is gone. Is it possible to sneak that through without allowing the crap to return?
With extension:

Incognito (no extension):

Both are https://twitter.com/explore.

@noididntreadit  It's actually anchored to the top right, can you see if it's there on your browser?

@noididntreadit On second thought you're right, it would probably better if it was in the right place, centered on the explore page. I'll make that the default in the next update!

@noididntreadit this commit addresses this issue! https://github.com/thomaswangio/minimal-twitter/commit/d3123ddad24fee3918a73c8a612d3e15d057b32b
Check back for the next update and it should be fixed :) thanks for pointing this out


MVP: Module Manager

Description
The Module Manager is part of the new reconciler architecture. It is following the K8s operator pattern reacting on data changes (defined in CRs).
A Module Manager is responsible for the lifecycle management of a Kyma module. It is behaving idempotent and determines whether a Kyma module has to be installed, updated or deleted on Kubernetes cluster. If necessary, it applies migration steps if a component update requires enhanced adjustments on Kubernetes side (e.g. data migrations, cleanup of resources etc.).
A Module Manager can be responsible for the reconciliation of one or multiple Kyma modules, but a Kyma module itself is always processed by only one Module Manager at the same time.
The MVP includes a reference implementation of a simple Module Manager which is capable to reconcile a common Helm chart.
Acceptance criteria
General ACs:

[x] #4
[x] #40
[x] #41
[x] Caching of rendered manifests
[x] https://github.com/kyma-project/manifest-operator/issues/38
[x] #45
[ ] Documentation

[ ] Architectural design documentation available
[x] #143

[x] A strategy is defined, how a Module Manager can be scaled (e.g. via sharding or following an active-passive cluster-mode pattern etc.)
[ ] Operational requirements are covered

[ ] For debugging purposes are component-IDs injected into each log message (to correlate messages per component CR)
[ ] Exposing analysis and statistic relevant data via Prometheus endpoint
[ ] Logging of critical events
[ ] Operational test was executed and trouble-shooting guides were created

[x] Support local and remote reconciliation strategy
[x] Threat modelling findings implemented (see https://github.com/kyma-project/lifecycle-manager/issues/144)

[x] https://github.com/kyma-project/lifecycle-manager/issues/151

[ ] Microdelivery process established
[x] Cleanup for manifest and CRs

Issues
https://github.com/kyma-project/kyma/issues/13759
Appendix

CRD = Customt Resource Definition
CR = Custom Resource
BYOC = Bring your own cluster
SKR = SAP Kyma Runtime

/reopen
/remove-lifecycle stale


rename isCircle to isTileRound

isCircle isn't a clear name for how the roundness of the tile should be. I think isTileRound is a better prop name.

Hey @maracuja-juice,
Your changes look good to me!


Do we use Angular?

@TigerApps IMO Angular looks like an overkill atm. Lets begin with vanilla JS. Thoughts?

Considering templating and modularity provided by AngularJS, working on boiler plate Angular looks to be easier than trying to convert messy VanillaJS to AngularJS at later run.

But we are going to introduce complexity unnecessarily. I was going through it, and realised that we won't need Angular at all.
Our use case is to simple, angular will only bloat it.
Maybe we will use it later when we feel the need to, right now it makes sense not to.

ok! lets roll with vanilla JS, jquery.

I will avoid jquery too for the beginning :dancer: lets use things only when we need them!


What is a CLI token?

@jamil should we just add a paragraph explaining what the token is about?


⚠ ClamAV has degraded performance

In 4dc28d6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in 39adc43.


CaptionIngo Project Submission

A Photo Caption Directory.

Thank you Mayuko for approving my project. Hope you like the project.
HA, it's like you read my mind - I always have a lot of trouble coming up
with Instagram captions! Great work on this!
Thanks for being a part of CWF, and congrats on your project!
Fill out this form by Monday to receive your sticker in the mail!
https://forms.gle/zy6nr8czF4uoRA4j9
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ScottKwang/CodeWithFriends-Spring2020/pull/91#pullrequestreview-421381082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL6SSNZSB653VR5E24D5HCLRUCNEDANCNFSM4NORRUPA
.


Correct wrong OSNAME-ARCH in problemlist*.txt #2908

When merged this pull request fixes #2908 by correcting OSNAME-ARCH in the following problemlist*.txt files:

aqa-tests/openjdk/excludes/ProblemList_openjdk8.txt
aqa-tests/openjdk/excludes/ProblemList_openjdk11-openj9.txt

Changes made:

Updates arm-linux to linux-arm on Line 51
Updated x86-64_windows to windows-x64 on Line 30

@mittaljethwa my bad, could you update x86-64_windows to windows-x64?

@mittaljethwa my bad, could you update x86-64_windows to windows-x64?

No issues, @sophia-guo I have updated it.


Not recognizing admin and SPOsite:

Not recognizing admin and SPOsite:
I used below code:

Please help

Hi @soundd2,
You are using Microsoft365DSC, this repository is for SharePointDsc which is a different module.
Please create issues for Microsoft365DSC at: https://github.com/microsoft/Microsoft365DSC
But I do know the cause of this issue: You are using the GlobalAdminAccount parameter. This parameter was recently renamed to Credential. You can find the documentation for this resource here: https://microsoft365dsc.com/resources/sharepoint/SPOSite/

Hi Yorick Kujjis,
I am glad that you responded, thank you. But after changing to credentials
also it is not recognizing. Please help
[image: image.png]
Regards
Shyam
On Mon, Feb 21, 2022 at 5:28 PM Yorick Kuijs @.***>
wrote:

Hi @soundd2 https://github.com/soundd2,
You are using Microsoft365DSC, this repository is for SharePointDsc which
is a different module.
Please create issues for Microsoft365DSC at:
https://github.com/microsoft/Microsoft365DSC
But I do know the cause of this issue: You are using the
GlobalAdminAccount parameter. This parameter was recently renamed to
Credential. You can find the documentation for this resource here:
https://microsoft365dsc.com/resources/sharepoint/SPOSite/
—
Reply to this email directly, view it on GitHub
https://github.com/dsccommunity/SharePointDsc/issues/1387#issuecomment-1046800050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTOEL6545FJBEWYSJFTYXLU4ISGVANCNFSM5O5S26IQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

The screenshot is not showing.
SharePointDsc (this repository) is targeting on-premise SharePoint: 2013, 2016, 2019 or Subscription Edition.
However you are trying to configure SharePoint Online, which is part of Microsoft 365 and which can be configured using Microsoft365DSC. This is also mentioned in your configuration:

Import-DscResource -ModuleName Microsoft365DSC

Hi Yorick Kuijs,
I have tried to add a SharePointDSC module, but it is not recognizing it
seems. Please help. PFA
[image: image.png]
Regards
Shyam
On Tue, Feb 22, 2022 at 2:50 PM Yorick Kuijs @.***>
wrote:

Closing this here since the issue is being handled in Microsoft365DSC
—
Reply to this email directly, view it on GitHub
https://github.com/dsccommunity/SharePointDsc/issues/1387#issuecomment-1047585734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTOELYUYYE4HUMZ2NAWS73U4NIOBANCNFSM5O5S26IQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @soundd2,
The screenshot is missing in your message. When you add a screenshot via mail, it is not added to the message in GitHub. Could you please add it directly in GitHub, not via mail?
Also, can you please explain what you are trying to accomplish? And the steps you are taking to do that? Share your configs, etc?

@ykuijs this issue is related to SharePoint Online and Microsoft 365 DSC - or at least I think so.
@soundd2 could you share your use case, so that we can elaborate the right topic. Thanks

HI Andi,
here is the screenshot, How to get rid of below error.

Use case , I want to use Microsoft DSC and perform a testing on SharePoint Online site collection and SP Online admin centre
and eventually  I want to perform CI/CD
I have been struggling but failing to get rid of the below error and other error for other ticket:

If u could help me with steps to be performed to get rid of these errors and also  I have gone through DSC tutorials and MS links to understand and implementing but in process getting these errors.

SharePointDSC is for SharePoint Server (On-Premises). We should proceed with the issue in Microsoft365DSC repository as this is related to cloud services.

Can u please transfer to Microsoft 365 dSC thank you
On Fri, 25 Feb 2022 at 1:39 PM, Andi Krüger @.***>
wrote:

SharePointDSC is for SharePoint Server (On-Premises). We should proceed
with the issue in Microsoft365DSC repository as this is related to cloud
services.
—
Reply to this email directly, view it on GitHub
https://github.com/dsccommunity/SharePointDsc/issues/1387#issuecomment-1050627134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWTOELZ6QGD2K7342UKGJNDU442M5ANCNFSM5O5S26IQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @soundd2,
Unfortunately GitHub does not allow to transfer issues between repositories. Could you please recreate the issue in Microsoft365DSC.
Looking at the above screenshots, a comment: You can't run the Import-DscResource outside a DSC configuration.
PowerShell Desired State Configuration is a framework which you can use to implement Infrastructure as Code / Configuration as Code practices. Out of the box it is only able to configure a few basic components like registry, Windows Features, etc. To extend this functionality, you can install modules. Each module targets a specific technology.

SharePointDsc: SharePoint on-premise environments, like SharePoint 2019 or Subscription Edition. The name of these resources all start with SP*
Microsoft365DSC: Microsoft 365 workloads, like SharePoint Online. The name of SharePoint Online resources all start with SPO*

I recommend viewing this very good training series by Jason Helmick and Jeffrey Snover to get started with DSC: https://docs.microsoft.com/en-us/shows/getting-started-with-powershell-dsc/
Or these:

https://docs.microsoft.com/en-us/shows/sharepoint-automation-with-dsc/video-introduction-to-powershell-dsc
https://docs.microsoft.com/en-us/shows/sharepoint-automation-with-dsc/demo-creating-sample-files
https://docs.microsoft.com/en-us/shows/sharepoint-automation-with-dsc/demo-securing-a-.mof-file

Closing since this is for Microsoft365DSC, which is a different module.


mender-client: remove patch for building with openssl3

The fix has been mainlined and does not need to be carried in consuming layers anymore.
Signed-off-by: Josef Holzmayr<EMAIL_ADDRESS>

Thanks!


fix: paths for wasm binaries per version

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.849%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
2516

Relevant Lines:
4068

💛  - Coveralls

looks good in vhd notes:
-rwxr-xr-x 1 root root  36014944 Apr 19 17:59 /usr/local/bin/containerd-shim-spin-v0-3-0-v1
-rwxr-xr-x 1 root root  47622592 Apr 19 17:59 /usr/local/bin/containerd-shim-slight-v0-3-0-v1
-rwxr-xr-x 1 root root  45334640 Apr 19 17:59 /usr/local/bin/containerd-shim-spin-v0-5-1-v1
-rwxr-xr-x 1 root root  52232184 Apr 19 17:59 /usr/local/bin/containerd-shim-slight-v0-5-1-v1

also passed e2e but failed gpu due to quota


ArcGIS Solution Map Symbols - July 2015

@ScottOppmann @NikkiGolding @WalterPotts @jessnoo
Added the following symbols to the repo:
Point Symbols

[x] Bridges
[x] Road Block
[x] Disaster Recovery Center
[x] Emergency Clothing Distribution Center
[x] First Aid station
[x] Food Distribution Center
[x] Incident Command Post
[x] Medivac Site
[x] Pet Collection Area
[x] Staging Area
[x] Water Distribution Center
[x] Animal Health Emergency
[x] Emergency Medical Services
[x] Fire and Hazardous Materials
[x] Incident Management
[x] Law Enforcement and Security
[x] Medical and Public Health
[x] Search and Rescue
[x] Emergency Assistance

Line Symbols

[x] Road Closures
[x] Road Detours

Polygon Symbols

[x] Agriculture Animal Health Incident
[x] Air Incident
[x] Civil Disturbance Incident
[x] Criminal Activity Incident
[x] Fire Incident
[x] Geologic Incident
[x] Hazardous Material Incident
[x] Hydro-Meteorological Incident
[x] Infestation Incident
[x] Marine Incident
[x] Public Health Incident
[x] Rail Incident
[x] Vehicle Incident

@Dbarnes1


Connecting from outside the local network

Hello, I was wondering if it's possible to use this plugin to listen to connections over the internet?
I have used this plugin successfully to establish connections within the same network, but when I try to connect to the host's IP address from outside the local network, no connection can be established. Should I look into the settings of my router or some other place, or is there some restriction or setting in the plugin that I'm unaware of?

It is not possible. But maybe a VPN or other similar technique could help you?


http_proxy support

<
What would you like to be added:
Please add support for http_proxy / https_proxy so we can ship logs from our systems to splunkcloud using this plugin
Why is this needed:
In our corporate network we cannot connect to Internet without using http_proxy

Hi @hzproe, you can configure proxy by adding them to SCK's env variable.
For example,
splunk-kubernetes-logging:
  environmentVar:
    - name: HTTP_PROXY
      value: http://my_proxy:myport
    - name: HTTPS_PROXY
      value: http://my_proxy:myport


CyberCAT version 0.10a doesn't support Cyberpunk version 1.3

When trying to load a Cyberpunk save game from version 1.3, I get the following error message:
Unknown class: "gameCurveStatModifierData_Deprecated"
at CyberCAT.Core.Classes.Parsers.GenericUnknownStructParser.GetInstanceFromName(String name)
at CyberCAT.Core.Classes.Parsers.GenericUnknownStructParser.InternalRead(NodeEntry node, BinaryReader reader)
at CyberCAT.Core.Classes.Parsers.GenericUnknownStructParser.ReadWithMapping(NodeEntry node, BinaryReader reader, List1 parsers)    at CyberCAT.Core.Classes.Parsers.StatsSystemParser.Read(NodeEntry node, BinaryReader reader, List1 parsers)
at CyberCAT.Core.Classes.Parsers.ParserUtils.ParseChildren(IEnumerable1 children, BinaryReader reader, List1 parsers, Boolean primary)
at CyberCAT.Core.Classes.SaveFile.LoadFromStream(Stream stream)
at CyberCAT.Core.Classes.SaveFile.Load(Stream inputStream)
at CP2077SaveEditor.SaveFileHelper.Load(Stream inputStream)
at CP2077SaveEditor.Form1.<>c__DisplayClass31_0.<openSaveButton_Click>b__0(Object sender, DoWorkEventArgs e)
at System.ComponentModel.BackgroundWorker.OnDoWork(DoWorkEventArgs e)
at System.ComponentModel.BackgroundWorker.WorkerThreadStart(Object argument)

Release v0.10a_r1 temporarily fixes this issue, with limitations. Body Gender, Unlocked Vehicles, Item Stat Data, and Player Stat Data cannot be edited.

Release v0.10a_r1 temporarily fixes this issue, with limitations. Body Gender, Unlocked Vehicles, Item Stat Data, and Player Stat Data cannot be edited.

Idk why but once I downloaded this I was unable to get either this updated version, or the previous version, to run.

Release v0.10a_r1 temporarily fixes this issue, with limitations. Body Gender, Unlocked Vehicles, Item Stat Data, and Player Stat Data cannot be edited.

Idk why but once I downloaded this I was unable to get either this updated version, or the previous version, to run.

same here

0.10a_r1 allows me to save games from game version 1.3.  I don't have the issues that the others on this thread have (I downloaded the zip file, and it runs stand-alone - it doesn't interact with previous versions).


files-18

[file-service] replace disk identifier from "files" to default
[test] replace disk identifier from "files" to default
Closes #18

Please resolve conflicts and merge


Replace import_role with include_role

include_role should be used in this case for conditional execution
of workloads[1]. If using import_role, the when condition only applies
to each task in the role but not to the import statement[2]. Also, if you
are using import_role, all the variables within the role are available
in this playbook and hence the when condition uses vars from the role
defaults. In the case of include_role, the role is conditionally imported
based on the when statement which is what we want in this case.
Also adding check to see if the workload was defined before continuing more
specific checks into workload parameters. This will prevent undefined var errors
by shortciruting with the and operator
References:

https://docs.ansible.com/ansible/latest/modules/include_role_module.html
https://docs.ansible.com/ansible/latest/modules/import_role_module.html

This needs updating of the operator image as well. So a new image has to be built after merging this.

@jtaleric PTAL


Make EditStack correctly report canUndo

Currently, canUndo() will incorrectly return false in the case that this.past.length is 0 and this.currentOpenStackElement is populated.
You can call undo() and it will do the right thing but there's no indication that you should be able to do it as canUndo() returns false.

:+1: Nice catch!


Please distribute unminified source code for debugging

Because Epoch seems to lack a lot of documentation right now, I'm not entirely sure how to use it, and tracebacks from minified JS are utterly unhelpful in figuring that out. Please consider making an unminified version of epoch.js available for easy debugging!

@toolness - Actually there is full documentation for the project. Run cake documentation and it will generate full html documentation in the doc directory. As far as non-minified source is concerned you can easily build this yourself by running cake build and then using the resulting js/epoch.js.

@rsandor Hello, I tried the approach you mentioned for creating the epoch.js, it created many js files for me (adapters.js, basics.js,  data, model, epoch, time) as well as folders of the same name with more js files. Are all files needed when creating the epoch.min.js?

@jrmartin - The js/epoch/ directory will contain each of the individual files, but the file js/epoch.js contains the fully concatenated javascript for the entire project. The individual files are created using build task and then they are packaged into a single javascript file via the package task. Finally, the compile task takes the single js file and runs it through the Google Closure Compiler.
Does this clarify things?

I am also starting to lean towards having a non-minified version in the repository. Thinking about redoing the build system to use grunt or gulp. Anyone have opinions either way?

@rsandor  Thanks. I was able to compile using "cake build" from src, but I had to remove the "SASS" target from the "build" command in the CakeFile. It was getting stuck for me, and not executing the "package" target, which was why I wasn't able to see the final result of epoch.js.
It's an awesome library, haven't seen any other graphs that do it so smoothly.A non minified version would be useful, would make debugging easier when using the console. It would also make it easier for those us without much coffeescript knowledge to get it from github, although coffeescript ain't that hard.

Hmm, must be an issue with the current node-sass package. I'll look into it
soon. Thanks for the heads up!
On Mar 4, 2015 6:37 PM, "Jesus M"<EMAIL_ADDRESS>wrote:

@rsandor https://github.com/rsandor Thanks. I was able to compile using
"cake build" from src, but I had to remove the "SASS" target from the
"build" command in the CakeFile. It was getting stuck for me, and not
executing the "package" target, which was why I wasn't able to see the
final result of epoch.js.
It's an awesome library, haven't seen any other graphs that do it so
smoothly.A non minified version would be useful, would make debugging
easier when using the console. It would also make it easier for those us
without much coffeescript knowledge to get it from github, although
coffeescript ain't that hard.
—
Reply to this email directly or view it on GitHub
https://github.com/fastly/epoch/issues/154#issuecomment-77295611.

Ah, thanks!
I guess I just meant how libraries like jQuery distribute both minified and unminified source, you know? It's very useful for developers who are just experimenting with a library to be able to poke into how it works without having to fork the project's github repository, install its build system prerequisites and generate its documentation and unminified source. That would have made my initial experience with your library a lot more pleasant.
That said, thanks for the help, I will definitely generate the docs and unminified version next time!

This is now a pretty high priority. Also it seems having everything in coffeescript without a modern build system is causing a slew of maintainability issues. Going to file a new ticket to cleanup the build scripts.

This is now done, you can find the fully un-minified compiled javascript in the repository at dist/js/epoch.js.


Constify OSSL_PROVIDER getter input parameters

Some OSSL_PROVIDER getters took a non-const OSSL_PROVIDER parameter.
There's no reason to do so.

Merged.
24626a47fb5fbb8b120e0ba696c8f173b9ad5d3c Constify OSSL_PROVIDER getter input parameters


Eliminate Fee on Subasset Registration (?) #1840

Alternate discussion space to issue: https://github.com/CounterpartyXCP/counterparty-core/issues/1840.

100% agree @adamkrellenstein. Have been saying this for a while (https://github.com/CounterpartyXCP/Forum/discussions/129#discussioncomment-8125058 and maybe another previous post is now hard to find for me) and is a super simple and non-disruptive change.


Add memory game 2 project

Please add screenshots, or a live link of project.
Thank You.

Ok, will do in a bit.

Ok, will do in a bit.

Please raise another one with the changes, thank you,


Add BOND: BarnBridge

[x] I understand that filing an issue or adding liquidity does not guarantee an addition to the Rigoblock token whitelist.
[x] I understand that the addition of the new token to the protocol is subject to governance approval.

Please provide the following information for your token.
Token Address: 0x0391D2021f89DC339F60Fff84546EA23E337750f
Token Name (from contract): BarnBridge Governance Token
Token Decimals (from contract): 18
Token Symbol (from contract): BOND
Network: Ethereum
Link to the official homepage of the token: https://www.barnbridge.com/
Link to CoinMarketCap or CoinGecko page of token: https://coinmarketcap.com/currencies/barnbridge/

The token has been added. Closing.


License types filter order and presentation

The categories could be more specific, e.g. "commercial" should be "commercial use allowed". "Modification" should be "modification" allowed. Also, what if people want both commercial and modification allowed? A dropdown doesn't allow that.
"all-cc" could be written "All Creative Commons" to be clearer. See Flickr's search interface filters at https://www.flickr.com/search/?text=cat.
In general, like Flickr's search interface, filter categories should be capitalized and written so it is human readable.

While it is a music site rather than a photo search engine, I might look to Free Music Archive's search for another good example. It uses checkboxes rather than a binary dropdown, so one can select multiple specific licenses or license categories. (I think allowing for both is useful -- I generally know the particular license I need, but not everyone does, and FMA's strategy makes it easy for both groups to find what they're looking for with minimal clicks.)

@gargargarrick , thanks for the feedback. I've updated the filters with new text, please let me know if that works.

this issue is now resolved.


Refactored code to update the database instance on each API call

With these changes, whenever a modification is made to the user's cart or wishlist, the respective database entry gets updated.
This also fixes an existing issue -- all the user's carts & wishlists were getting updated even though an individual user's ID was being modified. This was due to a shallow copy of initialData

Combining all the changes in #66


Albums in TopLevelKeepsakes folder are not properly handled

Hi,
yesterday I was starting to use osxphotos. I am still plying with the options, which are the best for my export.
But during execution I see that it is tried for some pictures to not export to the given export directory path, but to a path starting with /TopLevelKeepsakes
As this is not existing, the script is complaining.
I am executing as a dry-run:
osxphotos export /Volumes/Temp/Bilder_Photo_Export/ --from-date "2011-10-02" --to-date "2011-10-04"  --jpeg-ext JPG --convert-to-jpeg --directory "{folder_album}" --exiftool --dry-run --verbose
and no-dry-run:
osxphotos export /Volumes/Temp/Bilder_Photo_Export/ --from-date "2011-10-02" --to-date "2011-10-04"  --jpeg-ext JPG --convert-to-jpeg --directory "{folder_album}" --exiftool
in the output with the dry-run there are good cases and bad cases:
good case:
Exporting IMG_2304.JPG (IMG_2304.JPG) as IMG_2304.JPG
Writing metadata with exiftool for /Volumes/Temp/Bilder_Photo_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2304.JPG
Exported /Volumes/Temp/Bilder_Photo_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2304.JPG
bad case:
Exporting IMG_2330.JPG (IMG_2330.JPG) as IMG_2330.JPG
Writing metadata with exiftool for /Volumes/Temp/Bilder_Photo_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2330.JPG
Exported /Volumes/Temp/Bilder_Photo_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2330.JPG
Exporting IMG_2330.JPG (IMG_2330.JPG) as IMG_2330.JPG
Error exporting photo (o6FalzTsQiC6JSqs7Z+dLw: IMG_2330.JPG) as IMG_2330.JPG: '/TopLevelKeepsakes/Neuer Kalender/IMG_2330.JPG' does not start with '/Volumes/Temp/Bilder_Photo_Export'
I was checking these pictures in the Photos app. But I could not find anything specific for what could be the reason why they are treated differently.
I attach output for a dry-run case and a without dry-run.
Could you please have a look or do you have an idea what might be the reason for this behavior ?
Thanks a lot.
output_error_dryrun.txt
output_error.txt

I forgot to mention I am executing the script on photo library on a macOS 10.13.6 "High Sierra".

This definitely should not be happening.  Please try the following command and post the output here.  (This command will produce additional debug data regarding the photo in question):
osxphotos debug-dump --dump photos --uuid 'o6FalzTsQiC6JSqs7Z+dLw' > debug.txt

Output file debug.txt attached.
debug.txt

Thanks.  I suspect there's an album with a name that's causing issues.  Could you try the following command and post the output here? Note: this will include the name and creation date of all albums in your library but no other private information.  If there's any information such as an album name you'd prefer not to publicly post, you can search through the text document and delete it.
osxphotos debug-dump --dump albums > album.txt

OK, here is the complete album.txt content
album.txt

I was now trying see how many pictures are affected and wether the special characters ä,ö,ü,ß are related to this. I put the stdout and stderr output to a file test_äöüß.txt.
osxphotos --db /Volumes/Seagate\ Portable\ Drive/Fotos\ Library.photoslibrary export /tmp/ --from-date "1970-01-01" --to-date "2021-12-27" --jpeg-ext JPG --convert-to-jpeg --directory "{folder_album}" --exiftool --dry-run --verbose  > test_äöüß.txt 2>&1
On this I am using a "grep -B 3" to check for TopLevelKeepsakes and output 3 lines before the match to see the affected album names.
cat test_äöüß.txt |grep -B3 TopLevelKeepsakes > test_äöüß_TopLevelKeepsakes.txt
At least I can see that using of  ä, ö, ü, ß is not a problem. Pictures in albums without using these characters are also affected.
In total 63 photos do have this behaviour.
grep TopLevelKeepsakes test_äöüß.txt|wc -l
63
With the attached files, you have now also the information which of the photos are affected in the database.
test_äöüß_TopLevelKeepsakes.txt
test_äöüß.txt

OK, here is the complete album.txt content

Thank-you for the additional data.  I believe I've found the problem using the data you provided.  There is a "Keepsakes" folder (I believe left over from iPhoto) which is used to store calendars, cards, etc. that were printed from photos in the library.  The code doesn't properly account for the presence of albums inside this folder.  (It may be called something different in your library due to language localization).  There are "Neuer Kalendar" and "Neue Karte" albums (and possibly others) which are part of this top-level Keepsakes folder and they're causing the problem as the logic which constructs the folder/album path doesn't properly account for the presence of the Keepsakes folder.  I think I know how to fix this but won't be able to work on this until later in the week.
For now, you could temporarily work around this by finding the albums in the Keepsakes folder and copying all the photos contained in these albums to another "regular" album in the library.  osxphotos would still generate the errors for the Keepsakes albums but the photos would get exported correctly as part of the new album.

At least I can see that using of ä, ö, ü, ß is not a problem

That's a logical guess about where the problem could have been and indeed, in the early days, osxphotos did have problems with unicode characters, but I've since tried to be very thorough to properly handle all cases of accented and special characters.  (There may still be some bugs but no one has reported issues with unicode in a long time).

Note: This error likely exists with 'TopLevelSlideshows' as well.  Check that when implementing fix.

Yes, you are right. In 2012 I was ordering one Calendar. At that time it was still iPhoto. This Calendar is still visible. It is in a folder structure at the bottom ("Meine Projekte"). See picture.
And this is using many of the pictures which are throwing the TopLevelKeepsakes error.
I made now a copy of the Photo Library to play around.
When I delete the "Neuer Kalender" and "Neue Karte" entries, then I do not see the TopLevelKeepsakes entries during export dry-run.
Removing "Neuer Kalendar" was not enough, also "Neue Karte" had to be removed, as it had links to some pictures.

I still have a machine running 10.12.6  with iPhoto so I was able to create a test library and recreate this. I've confirmed the problem also exists for slideshows created in iPhoto.  The test library will allow me to start working on a fix.
Error exporting photo (lmFQY5k+RYup5Xpox1oEtg: Tulips.jpg) as Tulips.jpg: '/TopLevelKeepsakes/Test Card/Tulips.jpg' is not in the subpath of '/Users/rhet/Desktop/export' OR one path is relative and the other is absolute.
Exporting Tulips.jpg (Tulips.jpg) as Tulips.jpg
Error exporting photo (lmFQY5k+RYup5Xpox1oEtg: Tulips.jpg) as Tulips.jpg: '/TopLevelKeepsakes/Test Calendar/Tulips.jpg' is not in the subpath of '/Users/rhet/Desktop/export' OR one path is relative and the other is absolute.
Exporting Pumkins2.jpg (Pumkins2.jpg) as Pumkins2.jpg
Exported /Users/rhet/Desktop/export/iPhoto Events/Sep 28, 2018/Pumkins2.jpg
Exporting Pumkins2.jpg (Pumkins2.jpg) as Pumkins2.jpg
Exported /Users/rhet/Desktop/export/Pumpkin Farm/Pumkins2.jpg
Exporting Pumkins2.jpg (Pumkins2.jpg) as Pumkins2.jpg
Error exporting photo (FO2q5khAS9y4O9CkjpuYaw: Pumkins2.jpg) as Pumkins2.jpg: '/TopLevelSlideshows/Slideshow1/Pumkins2.jpg' is not in the subpath of '/Users/rhet/Desktop/export' OR one path is relative and the other is absolute.
Exporting Pumpkins3.jpg (Pumpkins3.jpg) as Pumpkins3.jpg
Exported /Users/rhet/Desktop/export/iPhoto Events/Sep 28, 2018/Pumpkins3.jpg
Exporting Pumpkins3.jpg (Pumpkins3.jpg) as Pumpkins3.jpg
Exported /Users/rhet/Desktop/export/Pumpkin Farm/Pumpkins3.jpg
Exporting Pumpkins3.jpg (Pumpkins3.jpg) as Pumpkins3.jpg
Error exporting photo (47wXm7h6RfGRACCdcbLiCA: Pumpkins3.jpg) as Pumpkins3.jpg: '/TopLevelSlideshows/Slideshow1/Pumpkins3.jpg' is not in the subpath of '/Users/rhet/Desktop/export' OR one path is relative and the other is absolute.

@thomashelders I'm curious for any input you have as an osxphotos user on how the fix should be implemented.

The simplest solution is to treat any cards/calendars/slideshows as top-level albums and export them as such.  E.g. in your example, it would be /Volumes/Temp/Bilder_Photo_Export/Neuer Kalender etc.  The downside to this approach is users might be surprised to find these albums in the top-level folder as they don't appear as regular albums in Photos.

Another possibility would be to treat "My Projects" as a top-level folder (slideshows, calendars, and cards all appear in My Projects in Photos) and thus in your example, the export path would be /Volumes/Temp/Bilder_Photo_Export/My Projects/Neuer Kalender.  The downside to this approach is that osxphotos has no way to access the localized names (for example, Meine Projekte in your case) as this information is not stored directly in the Photos database.

A third possibility is to just exclude these "project albums" from the export altogether.  Alternatively, there could be a flag to either include or exclude the project albums and use approach 1 or 2 above.

I have #1 above implemented -- was very easy.  As a bonus, this now exposes the iPhoto Events albums which had previously been inaccessible.  Not sure if people will also want photos exported to iPhoto Events albums.

If these iPhoto Event albums are these Calendars and Cards which can be printed, I would say that the use case is not so important, because the content from these iPhoto Event albums are pictures which are stored in or derived from other albums of the database.
So it is somehow redundant.
Only if somebody wants to know which pictures were used in a specific Calendar or card it might be of use. But this is an edge case, I think.

iPhoto grouper photos automatically into "events" based on date/time. These are similar to Photos "Moments". When converting from iPhoto, Photos creates a folder called "iPhoto Events" and puts all the events as separate albums there. Every photo in an event would also be in an album or in no album and in either case would get exported twice so probably best to hide the events so users don't get duplicates unknowingly

I was wrong about the Events -- they're normal albums created by Photos when migrating from iPhoto.  My personal library and test libraries were all created in Photos so I hadn't seen these Events albums before.

For now, I've decided to implement a separate type for projects so they don't get treated by osxphotos as albums.  This solves a number of problems and makes the implementation consistent across all versions of Photos (the schema for Photos 5+ is very different in how it handles albums).  I'll also expose a {project} template type to provide access to the project name(s) if desired and maybe a --projects-as-albums flags that would instruct osxphotos to include projects in things like {folder_album} by treating them as ordinary albums. This adds some complexity but allows users to include projects if they want or to exclude them easily.  As they're not ordinary albums and don't appear in "My albums" in Photos, I think excluding projects by default is the "best" behavior but also provide a way for users to access the projects if desired.

@thomashelders this should be fixed in v0.44.0.  By default, the albums in "My Projects" will not be exported; they'll be ignored as if the photos contained in them aren't actually in those albums thus the photos will still be exported but not as part of the project albums.  If you want to include them in the export, you can use the new {folder_album_project} template which will treat them like regular albums.
Assuming you installed with pipx, you can do a pipx upgrade osxphotos to upgrade

I have tried the new version and is not exporting the pictures without failure. Thanks a lot !!!
One note I have - regarding the created folder structure. Not sure whether you have seen it in your environment, as well.
I tried now both options
{folder_album}
and
{folder_album_project}
option{folder_album} is creating:
"iPhoto-Event"
->album
The new option is now putting
"iPhoto-Event/album"
If you see the attached pictures, you know what I mean.
The dry-run verbose output excerpt is showing a difference in the output - a colon
{folder_album}:
Exporting IMG_2330.JPG (IMG_2330.JPG) as IMG_2330.JPG
Writing metadata with exiftool for /Volumes/WD Book/Photos_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2330.JPG
Exported /Volumes/WD Book/Photos_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2330.JPG
=> here no colon between the folder but a /
Exported /Volumes/WD Book/Photos_Export/iPhoto-Ereignisse/2011-10-03 Erntemarkt...
^
{folder_album_project}:
Exporting IMG_2330.JPG (IMG_2330.JPG) as IMG_2330.JPG
Writing metadata with exiftool for /Volumes/WD Book/Photos_Export/iPhoto-Ereignisse:2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2330.JPG
Exported /Volumes/WD Book/Photos_Export/iPhoto-Ereignisse:2011-10-03 Erntemarkt in Hörde und Phönixsee/IMG_2330.JPG
Exporting IMG_2330.JPG (IMG_2330.JPG) as IMG_2330.JPG
Writing metadata with exiftool for /Volumes/WD Book/Photos_Export/Neuer Kalender/IMG_2330.JPG
Exported /Volumes/WD Book/Photos_Export/Neuer Kalender/IMG_2330.JPG
=> here is a colon between the folders
Exported /Volumes/WD Book/Photos_Export/iPhoto-Ereignisse:2011-10-03 Erntemarkt...
^
Can this be the reason for the different folder structure ?

I hadn't seen this but thanks for reporting. This is a bug as the output of both {folder_album} and {folder_album_project} should be the same with exception of adding project albums to the latter. I'll take a look at the code.

Found the problem and fixed in in v0.44.2

Works now perfectly. Thanks :-)
Happy new year !!!


Login Failed

Login Failed.
please check the credentials you entered.
Will not let add my ring doorbell.

This one is odd as I initially got the same error while testing but it went away subsequently after I had logged in using a different system to debug the authentication process. I have yet to encounter the problem since then.


Verbose text formatting

see #3 first because this builds on top of it
https://github.com/Raboro/obsidian-timer-plugin/assets/4106884/4cc7eedc-0489-4252-92cb-46f5a8e0ff4a

thx :)

I also added a setting to not display 00. For e.g when you add 1h, the result is 1h 00m 00s. With the setting enabled it´s 1h.


[FEA] Test HugeCTR notebooks as part of the CI process

We need to test the hugectr movielens notebooks as part of the CI process, testing both the criteo and movielens example notebooks.
For instance with movielens we test the PT/TF notebooks in this function: https://github.com/NVIDIA/NVTabular/blob/a0b14b5a2cbc7989ad00adcd357817367a18c00d/tests/unit/test_notebooks.py#L156
Likewise we have similar tests for criteo here:
https://github.com/NVIDIA/NVTabular/blob/a0b14b5a2cbc7989ad00adcd357817367a18c00d/tests/unit/test_notebooks.py#L36
https://github.com/NVIDIA/NVTabular/blob/a0b14b5a2cbc7989ad00adcd357817367a18c00d/tests/unit/test_notebooks.py#L89

@zehuanw per our earlier discussion we'll be prioritizing this.  We should figure out how to integrate our CI process so it can support HugeCTR.

@benfred @jperez999 Our CI container does not have HugeCTR installed. So we need to solve that


"Measurements in systems with multiple qubits" kata content feedback

This issue aggregates feedback related to the content of the "Measurements in systems with multiple qubits" kata.
Feedback includes, but is not limited to, any of the following:

Correctness (e.g. solutions are checked correctly, math is right, examples compile and run).
Clarity of content (e.g. descriptions for exercises, explanations of solutions).
Organization of content (e.g. ordering of sections, length of sections, titles).
Formatting (e.g. consistency of formatting, unaligned content).
Typos.

Please add a comment with your feedback, preferably as TODO items or bullet points.

[ ] Rename kata to "Measurements on multi-qubit systems".
[ ] Use "kata" instead of "tutorial" consistently.

This Kata covers the following topics:

How to implement such measurements in Q# (should start with capital H)

Lesson: Measurements and Entanglement
Entanglement has an effect on the measurement statistics of the system. (remove "Qubits ")
(there’s an error, “Misplaced &” above the post-measurement state equation. Also, I suggest putting the “=” symbol first in the line.)

Lesson: Joint Measurements
Single-Qubit Pauli Measurement
(Pauli Z operator is wrong, it should be a matrix)
Z = \begin{pmatrix}
1 & 0 \\
0 & -1
\end{pmatrix}
A $Z$-measurement is then a measurement in the $\lbrace \ket{0}, \ket{1} \rbrace$ basis,... (basis is represented always between braces)
Parity Measurements
(All matrices and vectors in tensor product of Z matrices are wrongly placed. I suggest to press enter after \\, so each row is in different line)
Z \otimes Z = \begin{pmatrix}
1 & 0 & 0 & 0 \\
0 & -1 &  0 &  0 \\
0 & 0 & -1 & 0 \\
0 & 0 & 0 & -1
\end{pmatrix}

(Conclusion is missing)
Lesson: Conclusion
Congratulations! In this Katas you learned how to apply measurements on multi-qubit systems. Here are a few key concepts to keep in mind:

Full measurements: you measure all the qubits simultaneously in an orthogonal basis ( $2^n$ possible outcomes)
Partial measurements: you measure qubits $m$ out of $n$, for $m< n$ ( $2^m$ possible outcomes).
Joint measurement: Pauli measurement of all $n$ qubits ( 2 possible outcomes).

Next, you will implement a quantum algorithm to generate random numbers in "Quantum Random Number Generation" Kata.


Add functionality to add link by clicking on nodes

Most part is done by https://github.com/rashchedrin/claimarg-prototype/pull/23
One more step: add ability to link existing nodes. I see it this way: similarly to how now a reply can be added, but if textarea contains nothing but the link to another node, then instead of posting such a message, a link should be added.


Free Agents not showing up during draft?

Dez Bryant was only listed in the watch list, not in player search.
Complication is that free agents in this scenario show up with active = 0 and get grouped with the 5,000+ former players, and we don't want all of those showing up.
One solution might be to track when a player was last "seen" in the player update process, then including players with active = 0 as long as they've been seen a certain time frame in the past.

I do think that tracking when a player was last "seen" the best option for the issue.

A last_seen column was added to the player table to solve another issue. Next step for this one is to modify the queries to include this column. First I'll watch and verify that the players found in the player update are what I'm expecting. It's been a while since I've gone over this logic, so it's easier to wait and see than to dig into it now.


Generalize build key description re: parallelized builds

This PR generalizes our language around unique build key because the exact env var will be different depending on CI (Build ID was a little Buildkite-specific, and perhaps hard to match up on the CI page).
For example:

For Buildkite, it is BUILDKITE_BUILD_ID
For Circle, it is CIRCLE_WORKFLOW_ID-$CIRCLE_BUILD_NUM
For GH Actions, it is GITHUB_ACTION-$GITHUB_RUN_NUMBER-$GITHUB_RUN_ATTEMPT
For generic CI, it is BUILDKITE_ANALYTICS_KEY

Happy for wordsmith humans to suggest better words in place of the ones I chose, should inspiration strike ⚡💡. Like, would unique build key be better than unique build identifier? 🤷🏻♂

@plaindocs @karensawrey I took another crack at that sentence, taking the lead from what @plaindocs wrote. What do you think?

@meiqimichelle I do like the expanded version 👍

@meiqimichelle let's merge ;)