support select_first function for non-None values that were declared as inputs

The select_first function selects the first non-None value from an array of values.
This fix uses refactors the get_ram_min_js function to accept a list of inputs and return the first non-null value of a valid javascript array of the inputs variables.
The expression contains a for loop that returns the first non-null element and also throws an error when only null values are found or the array is empty.
Undeclared inputs are not handled as the wdl-cwl translator does not support such inputs yet.

Fixes: #59

should send the javascript snippet to get_ram_min_js() and modify that function to incorporate it; test the result to confirm the fix

Can I get more clarification on this? I don't fully understand wish javascript snippet  am passing to get_ram_min_js().

should send the javascript snippet to get_ram_min_js() and modify that function to incorporate it; test the result to confirm the fix

Can I get more clarification on this? I don't fully understand wish javascript snippet am passing to get_ram_min_js().

from your code: https://github.com/common-workflow-lab/wdl-cwl-translator/pull/102/commits/ee4eb916dc4dbdc3251d6eb50bbe1902ae231b32#diff-b5a65c2befbbd0b6453805272729b622f41b2be683f8258432e08f4370332265R652
Except that Array.find is not part of ECMAScript 5.1, so you'll have to implement your own function.
Remember, according to https://github.com/openwdl/wdl/blob/main/versions/1.1/SPEC.md#x-select_firstarrayx your ECMAScript version should throw an exception if the array is empty or if the array only contains null values.

It might be easier to

make a copy of get_ram_min_js and edit it to be convenient and produce working CWL expressions
then compare that new function to the other uses of get_ram_min_js
then figure out how to refactor get_ram_min_js so that it supports this new use case (needing to evaluate some ECMAScript code in the same function that does dynamic unit conversion) with its existing usage.

HI @mr-c , The CI tests keeps reporting this
  Traceback:
  /opt/hostedtoolcache/Python/3.7.12/x64/lib/python3.7/importlib/__init__.py:127: in import_module
      return _bootstrap._gcd_import(name[level:], package, level)
  wdl2cwl/tests/test_cwl.py:6: in <module>
      from .. import main as wdl
  wdl2cwl/main.py:13: in <module>
      import cwl_utils.parser_v1_2 as cwl
  E   ModuleNotFoundError: No module named 'cwl_utils.parser_v1_2'

HI @mr-c , The CI tests keeps reporting this

Thanks. I just pushed a fix for that.

Forgive my confusion. I am not sure what part of the requested change am yet to implement. I already split the function and added the epilogue js string.

Closing as this needs to be redone from scratch given the refactoring. Hopefully it will be easier the 2nd time!


⚠ Bofrost.fr has degraded performance

In 8a2b579, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 941 ms

Resolved: Bofrost.fr performance has improved in 0341695.


idk

Fixed a bug that causes every player to be marked as cheater
Probably fixed #342
Added #339 (Visuals -> Players -> Health Text -> Bar)
Added #308 (Keyboard icon at the top right of the menu)
Removed display names from vars and renamed m_Var to Value
Refactored EntityCache: Added GetLocal(), GetWeapon() and GetObservedTarget()

pls merge before #345 because i don't want to resolve conflicts :D


Is there a way to use MediatR with pure JSON string based on a property name?

Having the below JSON.
{
   "eventName": "userCreated",
   "someKey": "someValue"
}

Use MediatR to send this JSON to a RequestHandler based on eventName?

No. Before you call MediatR convert that to an event object then call Publish.


android.os.Build missing on save

How do I set my class path so Recaf finds android studio install directory?

I'm not sure I understand the full issue. Can you elaborate?

I'm not sure I understand the full issue. Can you elaborate?

When he recompiles code, android.os.Build is missing which makes compiling fail. JPhantom probably failed.
Ensure phantom class generation is enabled and that it works (errors in console if it doesn't) or add the dependency to the workspace as detailed in https://www.coley.software/Recaf-documentation/use-edit-via-decompile.html

Also, if that's enabled and classes are still missing then chances are it is just failing. This happens often with target applications that are obfuscated. You can either edit the bytecode directly using the assembler or add the target dependencies into the workspace manually.

Closing with the assumption this is just another case of JPhantom (3rd party) failing for one reason or another.


Symbol Token Write Support

Dependent on https://github.com/amzn/ion-go/pull/165
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #68 (bc4c82b) into master (de5f96d) will increase coverage by 0.41%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   71.91%   72.32%   +0.41%
==========================================
  Files          15       15
  Lines        1036     1019      -17
==========================================
- Hits          745      737       -8
+ Misses        193      186       -7
+ Partials       98       96       -2

Impacted Files
Coverage Δ

struct_serializer.go
59.09% <0.00%> (+7.23%)
:arrow_up:

hash_writer.go
65.51% <60.00%> (+0.86%)
:arrow_up:

hasher.go
65.38% <100.00%> (-1.29%)
:arrow_down:

scalar_serializer.go
59.09% <100.00%> (+2.84%)
:arrow_up:

testing_utilities.go
90.37% <100.00%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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


Image connection broken (still)

Environment
Docker
Version
0.9.1
Describe the problem
Similar to #335, some of my icons are broken on just the main dashboard. When I click they show up just fine.
My situation is a bit different from #335, I'm using direct URLs to the png. https://cdn.overleaf.com/img/ol-brand/overleaf_og_logo.png for example. slightly
Additional info
Doesn't work:

Main Dashboard:

Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

Hey can you pull again ? I hot fixed the image 2 hours ago 😅
Tell me if you still have the issue in the latest latest

Yeah the issue is even with the hotfixed version.
I docker rmi'd all versions of it and still have the same issue.
I just tried to image with the dev tag and that's working perfectly fine. Tried it with the latest tag after the rmi's and the issue is still there.

Yeah the issue is even with the hotfixed version.
I docker rmi'd all versions of it and still have the same issue.
I just tried the image with the dev tag and that's working perfectly fine. Tried it with the latest tag after the rmi's and the issue is still there.

That's very strange. Can you compare the checksum of the image?

This is what I get for the 2 images.
latest: sha256:ba99d8c6a17505f40a593f7585b85398c17f3623b20b8231c4be53c7e7beac3b
dev: sha256:655a4bdc0b05f8fa6598c40c3ca497220a6430240d15bb8b6bc821866b918a89

That is strange because dev still has the caching feature enabled.
You probably can't have local images but you are encountering another issue but the display method to show images in the add service menu and the app shelf should be the same in the latest version.
I'll investigate further

I can confirm Its broken.
All #personal added icon urls stop to show in the latest version. The problem it's on dashboard. I currently use the latest version 0.9.1

@mariushosting can you send me the failing http requests or some logs please?

@mariushosting can you send me the failing http requests or some logs please?
warn  - using beta Middleware (not covered by semver) - https://nextjs.org/docs/messages/beta-middleware
Error: 'sharp' is required to be installed in standalone mode for the image optimization to function correctly
Error: 'sharp' is required to be installed in standalone mode for the image optimization to function correctly
Error: 'sharp' is required to be installed in standalone mode for the image optimization to function correctly

Same problem here, the log looks similar to @mariushosting

Same problem here, the log looks similar to @mariushosting

these are the server logs, can you show me your dev tool logs / network tab requests failing ?

This is what you want?

Same error here the latest update broke something.

Similar logs to me aswell

Can you confirm that this has been fixed with https://github.com/ajnart/homarr/releases/tag/v0.9.2?

Can you confirm that this has been fixed with https://github.com/ajnart/homarr/releases/tag/v0.9.2?

Fixed!

Fixed for me aswell! Cheers!

Can you confirm that this has been fixed with https://github.com/ajnart/homarr/releases/tag/v0.9.2?

Has been fixed. Thanks for the great work.


GP Panel: Show number of turns remaining until great person achieved

This would be in the bottom of the GP panel where the meters are, one number per player per row. See #147 for details.

The number of turns remaining for each player is the most important piece of information. To keep things legible, I'd replace the X/Y progress with the number of turns remaining and expand the popup:

Current / Needed (Turns left)
Points per turn
Points from each source: districts, buildings, wonders and policies
Order of players

Example
(icon) Great Scientist
309 / 360 (7 Turns)
+8 per (clock) Turn

+3 from Districts
+2 from Buildings
+1 from Wonders
+2 from Policies

Order

3 Turns - India
7 Turns - America (you)
29 Turns - Russia
Never - Spain

I've been messing around in GreatPeoplePopup.lua and .xml to add the the turns remaining, and this was the best that I could do without removing anything:

You'll notice that I had to make each Civ's entry bigger vertically, which means you might have to scroll down more. Any feedback? I can try making other versions, possibly moving the X/Y progress to the tooltip, to try to return it back to two lines.

I've been messing around in GreatPeoplePopup.lua and .xml to add the the turns remaining, and this was the best that I could do without removing anything:

You'll notice that I had to make each Civ's entry bigger vertically, which means you might have to scroll down more. Any feedback? I can try making other versions, possibly moving the X/Y progress to the tooltip, to try to return it back to two lines.

Probably should open a separate issue, but this reminds me that "All individuals of this type have already been earned" doesn't actually say WHAT type.  You can infer from position and which ones are still available, and it is especially easy to tell with Great Prophets if you are anything other than a complete beginner, but it would still be better to be explicit.  Maybe even shrink the column width when the GPP is unavailable; with my 4:3 monitor I don't fit the whole GP panel on the screen and have to scroll, with a bunch of that space being wasted on at least the Great Prophet slot.

Probably should open a separate issue, but this reminds me that "All individuals of this type have already been earned" doesn't actually say WHAT type.  You can infer from position and which ones are still available, and it is especially easy to tell with Great Prophets if you are anything other than a complete beginner, but it would still be better to be explicit.  Maybe even shrink the column width when the GPP is unavailable; with my 4:3 monitor I don't fit the whole GP panel on the screen and have to scroll, with a bunch of that space being wasted on at least the Great Prophet slot.

I've made some further adjustments. I've made the popup take up the whole width of the screen on a 16:9 monitor, just as the civic tree, tech tree, and government screen already did. This let me space out the recruitment standings so that they don't overlap a bit as they currently do. I also made the standings box a bit taller, and got rid of the X/Y progress line and moved those number to the tooltip as the progress bar already provides a visual representation of those. This made the standings return to two lines per civ, so more civs can be shown on the standings before having to scroll. My monitor is showing 5 now.
Also as vttale suggested I was able to get the GP class name back in even when there's none left. In pictures:


[API] Notifications, Fx 72 tasks

Trello
https://trello.com/c/ivi4m7SR/179-notifications-api-fx-72
Acceptance criteria

[x] Enable dom.webnotifications.requireuserinteraction on Release (probably Fx72; see also the intent to ship)

[ ] Also need to fix Notification API example doesn't work in Chromium in cross-origin iframe

[ ] Disallow notification permission requests from cross-origin iframes (legacy item, Fx 70)

See also https://hacks.mozilla.org/2019/11/upcoming-notification-permission-changes-in-firefox-72/

"notification/push user interaction required" docs done:

Pages updated to add in information about it:

https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notifications_API
https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notification
https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notification/requestPermission
https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notifications_API/Using_the_Notifications_API
https://wiki.developer.mozilla.org/en-US/docs/Web/API/Push_API
https://wiki.developer.mozilla.org/en-US/docs/Web/API/PushManager/subscribe

Example updated so that it gives an idea of how to use it in the new world: https://mdn.github.io/to-do-notifications/
Notes added to compat data: https://github.com/mdn/browser-compat-data/pull/5316
Rel note added: https://wiki.developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/72#DOM

"Cross-origin IFrame disallowed" bits documented:
Pages updated to mention this new behavior:

https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notification
https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notification/requestPermission
https://wiki.developer.mozilla.org/en-US/docs/Web/API/Notifications_API/Using_the_Notifications_API
Rel note added: https://wiki.developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/70#APIs
New note added to compat data to cover Firefox


Weird rendering of first page

Of this illustrious document:
http://former.vancouver.ca/ctyclerk/cclerk/20130409/documents/rr2.pdf
character positioning on first page is, I hope, not correct.
Using firefox nightly 23.0a1 (2013-04-05)

Closing since this no longer happens, most likely after recent font fixes. The character spacing is now much better given the fact that the fonts are not embedded.


more performant databases

I converted the pipeline to read from public copies of uniref and chocophlan stored in google cloud storage. The download speeds are 10x what we get from the humann2 website and will scale. The databases are no longer "configurable", but the docker image can be updated if needed.

@edoardopasolli @nsegata look!

Looks great, thanks so much Sean!
Nicola
On Wed, Oct 4, 2017 at 7:44 AM Levi Waldron<EMAIL_ADDRESS>wrote:

Merged #106
https://github.com/waldronlab/curatedMetagenomicData/pull/106.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/waldronlab/curatedMetagenomicData/pull/106#event-1277290615,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKGtgZCut43--Sr0ytrp4WwtSxMcj6RPks5soxskgaJpZM4PsDV3
.


Need to add Angelscript compilation patch for MSVC ARM architecture

你在什么场景下需要该功能？
angelscript update pr
描述可能的解决方案

Pocket PC with ARM CPU
The MSVC compiler doesn't support inline assembler for the ARM CPU, so a separate assembler file has been written with this code: as_callfunc_arm_msvc.asm.
In order to compile this file properly it is necessary to configure a custom build command with the following:
armasm -g $(InputPath)

描述你认为的候选方案
No response
其他信息
No response

https://stackoverflow.com/questions/59742466/building-arm64-asm-in-visual-studio-with-cmake


Create a generic v<MAJOR> release

Is your feature request related to a problem? Please describe.
Right now we are not releasing a vMAJOR tag/release pointing to the lastest version of the MAJOR stream.
The drawback of this approach is that, users MUST have to update the minor/patch version every time.
Describe the solution you'd like
For every release (either minor or patch), release an additional vMAJOR release pointing to the latest version.
Example
If we release v4.7.0, also release v4 pointing to the v4.7.0.
If v4 already exists, simply overwrite it with a new one such that it will always point to the latest.
Describe alternatives you've considered
n/a
Additional context
n/a

Temporarily creating manual tag and release (for v4) every time a new release is out, e.g., https://github.com/kiegroup/git-backporting/releases/tag/v4


package.json entry     "istanbul": "gotwarlost/istanbul#source-map",

the package.json entry
"istanbul": "gotwarlost/istanbul#source-map",
should likely be
"istanbul": "^0.3.13",
reference https://github.com/douglasduteil/isparta/blob/b86e8212df39bcf34cc2c9752940a58d71baa952/package.json

Interested in submitting a pull request to fix that?

Maybe next one, I don't know the process. A little heavy load this week.
--- Original Message ---
From: "Rob Eisenberg"<EMAIL_ADDRESS>Sent: January 5, 2016 10:46 AM
To: "aurelia/skeleton-navigation"<EMAIL_ADDRESS>Cc: "NormSoftwareEng"<EMAIL_ADDRESS>Subject: Re: [skeleton-navigation] package.json entry     "istanbul": "gotwarlost/istanbul#source-map", (#247)
Interested in submitting a pull request to fix that?

Reply to this email directly or view it on GitHub:
https://github.com/aurelia/skeleton-navigation/issues/247#issuecomment-169057591

I think this is an unused package that we can remove actually, looks like an artifact.

Don't know if it's important, but since this change I get the following warning from npm:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

@PWKad Looks like that broke things. Would you submit a new PR to add this back please? I'd like to get a new set of releases out in the next day or two.

Doh, on it now.


zephyr: frdm_k64f: Remove partition defines

Now that the partition table has been added to the device tree for the
frdm_k64f in upstream Zephyr, these symbols become redundant defines.
Remove them to fully use the partitions defined in Zephyr.
Signed-off-by: David Brown<EMAIL_ADDRESS>

It looks good. What did the other targets not change? What is the process of setting up the partitions for freedom k64f after this PR is merged?

In order for partitions to work, the partition table has to be added to the device tree in Zephyr first. If it is added and this isn't updated, the compiler generates warnings about duplicate #defines. If the change goes into mcuboot first, the result will not compile. So, we want to make sure they are merged into Zephyr first. I'll probably do the stm32l475 soon, since I have one, and will keep an eye on definitions that go into other SOCs into Zephyr.

As far as how to set up the partitions, they are defined in the device tree within Zephyr in dts/arm/nxp/nxp_k6x.dtsi. That way every app that builds with that Zephyr tree will have the same partitions defined.


Invalid request makes grpc server crash

Some stubs/handlers crash when receiving an invalid argument.
Over gRPC, invalid requests should be handled gracefully because caller cannot know how the state of the server has changed in the meantime (i.e., if a trajectory or submap still exists).

DeleteTrajectoryHandler throws out-of-range for non-existing trajectory_id but should ignore the request.
MapBuilderStub::GetTrajectoryBuilder throws out-of-range for non-existing trajectory_id but should return nullptr.
SubmapToProto should return an error string if any part of the submap_id is invalid. (This is also needed for cartographer_rviz.)

Fixed by #1222


Fix update expat

Summary
fixes https://github.com/codecov/internal-issues/issues/346
Note that expat needs to be upgraded at the end of the docker image, because reasons 🤷

@trent-codecov PTAL, you may need this change as well for api


Back-button causes app to crash

Navigating back to first page after reloading page, navigate and then press back.

Fixed in release 1.5.1.


Try debug osx/ilp64 crash

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender


CIV-9912 point to civil service pr under test

JIRA link
https://tools.hmcts.net/jira/browse/CIV-9912
Change description
just for testing, please do not merge
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

no longer needed as has been tested


feh: 3.6.3 -> 3.7

Motivation for this change
update to current version
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 122375 at d77430bf run on x86_64-linux 1

7 packages built successfully:

betterlockscreen
deco
feh
multilockscreen
pywal (python38Packages.pywal)
python39Packages.pywal
wpgtk


chore: update unchained client to v6.2.0

Description
Update @shapeshiftoss/unchained-client to v6.2.0
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Risk

Testing

Screenshots (if applicable)

Due to GitHub issues I manually added this change to develop.


Arc-ed assets

Objective
Fixes #15723. This is an alternative to #15346 (and the associated PR #15359) obsolete. Note if we decide asset locking is preferable, there are learnings from this PR we can apply to #15359.
Solution

Assets are now stored as Arc<A> instead of just A.

get remains the same, but I've added an alternative called get_arc which returns a clone of the Arc that stores an asset.

Replaced the get_mut with one of get_cloned_mut, get_inplace_mut, and get_reflect_cloned_mut (this last one is just because it's necessary for reflection, but not necessarily needed for users).

Added add_arc and insert_arc to allow adding Arc types directly.

add currently takes Into<A>. Trying to change this to Into<Arc<A>> breaks a lot of callsites like meshes.add(Rectangle::new(1.0, 2.0)) (since this requires two Into calls, which Rust can't figure out).
As a result, making add_arc take Into<Arc<A>> is a fine middle ground to allow both Arc-ed assets and owned assets.
For completeness, I also added insert_arc.

RenderAsset now takes an Arc<Self::SourceAsset>. This means in most cases, cloning the asset can be skipped when passing data to the render world. Data only needs to be cloned if you mutate an asset.

Testing

Added an example that shows off using an Arc-ed asset in an async context.

Showcase
fn my_task_launcher(configs: Res<Assets<ProceduralGenerationConfig>>, selected_config: Res<SelectedConfigHandle>) {
    let config = configs.get_arc(&selected_config.0).expect("The asset is loaded");

    AsyncComputeTaskPool::get().spawn(move async || {
        // Do something with the `config` in a task!
        std::thread::sleep(std::time::Duration::from_secs(config.time_it_takes_to_generate));
        println!("Done generating a level!");
    });
}

Migration Guide

Assets::get_mut has been removed. Use get_cloned_mut for Clone assets, or get_inplace_mut for non-Clone assets (and handle the case where the asset is aliased).
Assets::iter_mut now returns an iterator of &mut Arc<A>. Consider using Arc::make_mut for Clone assets, or Arc::get_mut for non-Clone assets (and handle the case where the asset is aliased).
RenderAssets now take an Arc<Self::SourceAsset>. Consider using Arc::try_unwrap() and falling back to a clone if the asset is aliased (try_unwrap returns an error). Otherwise, prefer borrowing the source asset's data instead of moving it.

Can you fix the inconsistency between "[Arc]-ed" and "[Arc]d"?

Can you fix the inconsistency between "[Arc]-ed" and "[Arc]d"? I prefer the latter.

Done!

Straight forward rebase to do with #16368, and updated error to using thiserror after #16684.


Avoid creating NaN in lpmn_values().

Solves https://github.com/google/jax/issues/19157

Looks reasonable – thanks. Could you add tests that cover the problematic cases?

Yes, done. It seems like this only is a problem when jit is disabled. In any case, this should be fixed now.


getting the error to fetch time service data for metric

Sep 12, 2019 6:50:51 PM com.google.auth.oauth2.ComputeEngineCredentials runningOnComputeEngine
INFO: Failed to detect whether we are running on Google Compute Engine.
Exception in thread "main" java.io.IOException: The Application Default Credentials are not available. They are available if running in Google Compute Engine. Otherwise, the environment variable GOOGLE_APPLICATION_CREDENTIALS must be defined pointing to a file defining the credentials. See https://developers.google.com/accounts/docs/application-default-credentials for more information.
at com.google.auth.oauth2.DefaultCredentialsProvider.getDefaultCredentials(DefaultCredentialsProvider.java:132)
at com.google.auth.oauth2.GoogleCredentials.getApplicationDefault(GoogleCredentials.java:115)
at com.google.auth.oauth2.GoogleCredentials.getApplicationDefault(GoogleCredentials.java:88)
at com.google.api.gax.core.GoogleCredentialsProvider.getCredentials(GoogleCredentialsProvider.java:67)
at com.google.api.gax.rpc.ClientContext.create(ClientContext.java:140)
at com.google.cloud.monitoring.v3.stub.GrpcMetricServiceStub.create(GrpcMetricServiceStub.java:176)
at com.google.cloud.monitoring.v3.stub.MetricServiceStubSettings.createStub(MetricServiceStubSettings.java:187)
at com.google.cloud.monitoring.v3.MetricServiceClient.(MetricServiceClient.java:157)
at com.google.cloud.monitoring.v3.MetricServiceClient.create(MetricServiceClient.java:138)
at com.google.cloud.monitoring.v3.MetricServiceClient.create(MetricServiceClient.java:129)
at com.example.monitoring.QuickstartSample.main(QuickstartSample.java:51)

Hi @madhubkk, can you provide us with more information on what you are trying to do and how you got this error? From the error it looks like you need to set your env var GOOGLE_APPLICATION_CREDENTIALS. Learn how here: Obtaining and providing service account credentials.

Please feel free to reopen this issue if your problem continues persist.


added a new "Rating" field, but the value which I entered does not show up

[Enter feedback here]

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

ID: cca95f5e-0fe4-bbd2-fd23-13562e02f888
Version Independent ID: c3fb9673-4da6-5eeb-08bb-6f8e7bc8e0c6
Content: Part 8, add a new field to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/new-field.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The tutorial works. I'm not sure what step you missed. You might try asking on StackOverflow,.


Add AES-SIV support

Checklist

[X] documentation is added or updated
[X] tests are added or updated

Is there specific reason for why is it "hard-wired" to AES? SIV is a mode that can operate with any block cipher. Why does it get wired unconditionally to AES? In other words, wouldn't it be better if it was cipher-agnostic crypto/modes/siv128.c?
[And btw, red cross from appveyor is totally relevant.]

Ah, Windows build failure, thanks @dot-asm.
As far as being wired to AES, I have only seen AES implementations and references, specifically RFC5297

RFC5297

Which opens with "This memo describes SIV, a block cipher mode of operation." Original paper reads "SIV can be based on an arbitrary blockcipher, such as AES or TDEA". The only additional and relevant reference to AES in the paper is "SIV never uses the inverse of the blockcipher, which is convenient for a blockcipher like AES." Well, admittedly RFC is more explicit asserting that E(K,X) indicates AES encryption of string X using key K, but this still doesn't preclude possibility of using it with alternative cipher. But there is even another reason. I'd argue that cipher-agnostic module would be more manageable and easier to maintain. Because it allows you to concentrate harder on one thing at a time, namely on mode specifics, and then on EVP wiring...

@dot-asm Fair enough; the title specifically mentions AES, and the implementation that this was based on was originally (and only AES). That being said, it'll take some time! :)

(Don't get me wrong, when I first started this, I had thought of adding a mode, but I lacked familiarity in that code.)

Well, there is certain ambiguity in suggestion. Modules in crypto/modes aim for EVP-neutrality in sense that they don't make calls to EVP or depend on EVP structures. They are rather providers to EVP. But this is not necessarily the only way. Well, one of original rationales behind making them EVP-neutral was that it would allow to make better decisions when it comes to performance and re-usability. But still it doesn't mean that every mode has to be that way. What I'm trying to say it that I'd [let's say] insist on taking mode specific code out of e_aes.c. For better maintainability. But whether or not it will be crypto/modes-style EVP-neutral thing or one that still relies heavily on EVP could be open for discussion. Personally I'd prefer to see crypto/modes-style, but maybe it would make it unnecessarily complicated. Maybe you can start by taking current code out of evp/e_aes.c, put it into say evp/e_mode_siv.c... EVP_aes_*_siv would go to the latter. Idea would be if we want to add another-cipher-siv, we'd add few lines to evp/e_mode_siv.c...
If I also may make another suggestion. Would it be sensible to arrange objects.txt update as separate commit?

We have a new mode! siv128

Travis error is:
The command "$make update; git diff --quiet" exited with 1.
My make update is clean, so this is unrelated?

Rebase. Should be fixed by #3688.

so is this all set and ready for final(!) review, @tmshort?

Yes @richsalz, I just rebased

travis says:
if [ -s doc-nits ] ; then cat doc-nits; rm doc-nits ; exit 1; fi
*** ERROR: unresolved internal link 'SIV Modes' at line 404 in file doc/man3/EVP_EncryptInit.pod
make: *** [doc-nits] Error 1

I'll look at doc-nits.

Fixed doc-nits, but now there's a libcrypto conflict.

ping, reviewers?

github says "This branch has conflicts that must be resolved"

Yes, there are conflicts with util/libcrypto.num (which is one of those always-conflicting files). I'm waiting to see if any other changes are needed before I rebase and fix util/libcrypto.num

Well, my question is what is the future for this. Thing is that when doing some background checks I've ran into https://datatracker.ietf.org/doc/draft-irtf-cfrg-gcmsiv/, I for one is not exactly happy with that proposal(*), but obviously performance doesn't play last role in production scenario. And if it would come to such point even here, then implementation should allow quite specific kind of optimizations. Which is one of the reasons for why I earlier commented what I'd rather see module that would be a provider to EVP, not user. Yes, I did say that it doesn't actually have to be that way, but if it actually comes to point when performance is an issue, then using "circular" EVP would be limitation.
(*) I'd rather see a generalization of original SIV with standard GHASH instead of CMAC and using standard CTR mode, but I don't really have a say.

This does both provide and use EVP services, and it's completely not optimized. At the very least, it provides a separate mode, which something that was requested.
@dot-asm if you have any suggestions for improvement, I'm all ears.

it provides a separate mode, which something that was requested.

What is rationale? Is performance important? If not, will it remain unimportant? Basically question is what are the chances for finding ourselves in situation when we have to re-implement it...

suggestions for improvement

As already said, I'd prefer to see "pure" "provider", instead of "circular" EVP calls...

As already said, I'd prefer to see "pure" "provider", instead of "circular" EVP calls...

In other words, don't use the generic EVP interface for the CTR and CBC interfaces and and instead call the AES primitives directly? That significantly reduces the flexibility of the siv128 mode and ties it back into AES, which was specifically something @dot-asm you had requested be broken out separately in your June 14th comment, but then asking for EVP-neutrality later tht day (which I apparently mis-read). The CRYPTO_siv128_init() would (and subsequent siv128 mode) code would have to be significantly modified, and then, I fear, would be fairly tightly coupled with AES internal implementation, and not a generic mode.
That being said, if there were to be an optimize version of this, yes, it would end up being a rewrite for that optimization (especially ASM, which is your area of expertise).

In other words, don't use the generic EVP interface for the CTR and CBC interfaces and and instead call the AES primitives directly?

No, make pointer to single-block cipher function a parameter. Then extend with pointer to multi-block/stream function as parameter. Well, this is not really "do this-n-that", but rather "let's consider this".
And as already said, what is the original rationale? If performance is not going to be an issue, then maybe "circular" EVP calls would do... Life is full of compromises...
[Signing off for today.]

Then extend with pointer to multi-block/stream function as parameter. Well, this is not really "do this-n-that", but rather "let's consider this".

And the purpose of "considering this" is actually to figure out what is meaningful to implement in assembly while minimizing the effort. Well, given previous cases and what's already available the answer is more or less obvious. There already is aesni_ctr32_encrypt and let's say there is dedicated aesni_cmac. So suggestion effectively is to implement function [or set of] with these two as pointers. [In addition set of functions that take aesni_encrypt. Or vpaes_encrypt, or AES_encrypt, or Camellia_EncryptBlock, ...]

NTP security is also going to use AES-SIV; that was the primary motivation for this work, by the way.

NTP security is also going to use AES-SIV; that was the primary motivation for this work, by the way.

This is kind of asks for another question. Since NTP was called primary motivation, "also" in first part calls for naming yet another party :-) Is there another party or was just figure of speech?
Anyway, so that basically priority is relatively short inputs. [Relatively to TLS's 16K that is...]

Ugh... CMAC code use the EVP interface in order to use the CBC cipher given to it, so I have to pass CMAC an EVP_CIPHER, whether I like it or not. There is no aesni_cmac (yet); and that wouldn't help in the generic, non-x86-64 or early-x86-64 processor case, anyways.
That leaves the CTR cipher, for which, to be very frank, has many different implementations based on architecture that the EVP interface kindly abstracts away. If I were to try to use CTR implementations directly, it would complicate the code significantly compared to using the EVP interface. So, given CMAC's dependency on EVP (CBC), I'm not sure it's worth the time to swap out the EVP (CTR) interface at this time.
And, as @richsalz points out, the motivator is NTP, which has fairly small inputs.

Looks as though 'openssl speed' is broken. It will be fixed as part of the rebase

Rebased. Given the use case (relative small encrypts), I think the code is sufficient. Again, removing the use of the EVP interface would require a non-EVP-using CMAC and introduce the complexity of AES implementation choice, all of which is handled by the EVP interface.

So, let's pick this up again.  @dot-asm and @tmshort ?

Rebasing...
-Todd Short
//<EMAIL_ADDRESS>// "One if by land, two if by sea, three if by the Internet."

On Feb 12, 2018, at 10:20 AM, Rich Salz<EMAIL_ADDRESS>wrote:
So, let's pick this up again. @dot-asm https://github.com/dot-asm and @tmshort https://github.com/tmshort ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openssl/openssl/pull/3540#issuecomment-364954209, or mute the thread https://github.com/notifications/unsubscribe-auth/AKeSGAFVOTX4x-X5nRzebmddfGGjjEu0ks5tUFbJgaJpZM4NlUZc.

@dot-asm where are we with this?  what needs to be done for your approval?

Rebased on top of master

This is really old...

Yes, it was delayed (along with a lot of other enhancements) due to the TLSv1.3 effort that was the primary focus of 1.1.1.

Rebased.

Still pending for an OMC review, @openssl/omc

Need a rebase too.

And I had just done that, too!

Merged to master, thanks to all involved.


'npm test' failed with an error

Hi,
I did a git clone and followed the README.md.
'npm install' is okay.
'gulp' is okay.
'gulp build --type production' is okay
'npm test' produced an error for me (see below).
Is there something that I did not do to make this work correctly?
Please let me know.
-- Regards: Jeff Franks
---- console session showing the error ----
jfranks@jlf-mint-17 ~/Desktop/ws1 - reflux-getting-started/react-starterkit $ npm test

<EMAIL_ADDRESS>test /home/jfranks/Desktop/ws1 - reflux-getting-started/react-starterkit
jest

Using Jest CLI v0.4.5
FAIL  app/scripts/tests/itemlist-test.js
Error: Worker process exited before responding! exit code: null, exit signal: SIGSEGV
stderr:
A worker process has quit unexpectedly! This is bad news, shutting down now!
npm ERR! Test failed.  See above for more details.

Thanks @jefffranks! I will check that. The steps you did are correct..

It's working on my machine, but I saw that there is an open issue regarding this error: https://github.com/facebook/jest/issues/243
Maybe it's because of the Node version you are using.

Hi moklick,
Yes, you are right,
I down-graded my node version from v0.12.2 --> v0.10.33.
npm is now at v2.11.1
Also, had to remove all npm cache scratch everything related to node v0.12.2 for the un-install.
Using these versions, I was able to obtain a good result.
The following console log shows the good results
======= console log ================
jfranks@jlf-mint-17 ~/Desktop/ws1 - reflux-getting-started/react-starterkit $ npm test

<EMAIL_ADDRESS>test /home/jfranks/Desktop/ws1 - reflux-getting-started/react-starterkit
jest

Using Jest CLI v0.4.5
PASS  app/scripts/tests/itemlist-test.js (1.922s)
PASS  app/scripts/tests/header-test.js (2.253s)
2 tests passed (2 total)
Run time: 3.861s

ok. nice to hear that.. So I think we have to keep an eye on the jest issue


Feature Request: Let's Encrypt DynDNS TSIG nsupdate script add

Not sure where this goes but here's what needs to be added to use Let's Encrypt DynDNS TSIG nsupdates using nsupdate. I just don't know enough about the inner workings to do the PR myself.
You can make the "nsupdate (RFC2136)" option work but a zone field that is your domain name managed by Dyndns needs to be added in the GUI or inserted in the script /usr/local/share/examples/acme.sh/dnsapi/dns_nsupdate.sh
My work around is to capture the domain name from the passed fqdn and add a zone statement in the nsupdate options
root@fw:/usr/local/share/examples/acme.sh/dnsapi # diff /usr/local/share/examples/acme.sh/dnsapi/dns_nsupdate.sh /usr/local/share/examples/acme.sh/dnsapi/dns_nsupdate.sh.orig
9d8
<   zone=`echo $1|egrep -o '[a-z]+\.[a-z]+$'`
16d14
<
19c17
< zone ${zone}
---
> zone
34d31
<   zone=`echo $1|egrep -o '[a-z]+\.[a-z]+$'`
40d36
< zone ${zone}

If you wanted to add the "DynDns TSIG nsupdates" option in the  validation method under DNS-01
the ${NSUPDATE_SERVER}" option needs to be set to  update.dyndns.com
the dns_nsupdate_add() function would look like
  nsupdate -k "${NSUPDATE_KEY}" <<EOF
server update.dyndns.com
zone ${zone}
update add ${fulldomain}. 60 in txt "${txtvalue}"
send
EOF

and the dns_nsupdate_rm function looks like
  nsupdate -k "${NSUPDATE_KEY}" <<EOF
server update.dyndns.com
zone ${zone}
update delete ${fulldomain}. txt
send
EOF

@nforceroh please open requests for plugins in our plugin repository here  https://github.com/opnsense/plugins/issues


Combos with "Each opponent loses the game" should also be included in is:winning

          I would add that combos with "Each opponent loses the game" should also be included in is:winning and the similar variants, since causing everyone to lose also causes you to win under normal circumstances.

Originally posted by @GordonsKitchen in https://github.com/SpaceCowMedia/commander-spellbook-backend/issues/372#issuecomment-2030493183

Solved in 00d3cbd64467c656c15f812ad16d553e88b20d1e


Empower frontend

Codecov Report

Merging #41 into master will decrease coverage by 11.31%.
The diff coverage is 50%.

@@             Coverage Diff             @@
##           master      #41       +/-   ##
===========================================
- Coverage   89.78%   78.46%   -11.32%
===========================================
  Files          11        5        -6
  Lines         137       65       -72
  Branches       14       11        -3
===========================================
- Hits          123       51       -72
+ Misses         12       11        -1
- Partials        2        3        +1

Impacted Files
Coverage Δ

packages/client/src/utils/locale.ts
62.5% <50%> (-2.72%)
:arrow_down:

packages/client/src/hooks/use-now.ts
75% <0%> (-12.5%)
:arrow_down:

packages/client/src/styles/index.ts

packages/client/src/test-utils.tsx

Continue to review full report at Codecov.

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


Relation overlays misaligned with basemaps

Bug description
When I search for a relation and see its outline, very often:

It is initially out of alignment with the basemap
It gets out of alignment with minimal zooming & panning

e.g.:
https://www.openhistoricalmap.org/relation/2696913#map=2/54.5/61.8&layers=O&date=1721&daterange=1721,1918

What should be happening?
Relation overlay should always be geo-aligned with the base map on initial render.
Relation overlay should always be geo-aligned on subsequent pans and zooms.
No gray tiles.
Repro Steps

Search for a relation using Nominatim search form: e.g. 'Russian Empire'
https://www.openhistoricalmap.org/search?query=russian empire#map=2/65.4/0.0&layers=O&date=1721&daterange=1721,1918

Click on one of the resulting links

See if it displayed properly (or if the base map has any gray tiles)

Screenshots & URL
Here's a search for Texas, followed by clicking on the first search result.

I'm pretty sure it's something about the MBGL layer at z2 and further out.

Home page

Start at www.openhistoricalmap.org and simply change the base map from one of the vectors (Historical, Woodblock) to Standard OSM.
At z2 and further out, see that the two don't align at all well. If you're at z3 or deeper, you wouldn't see a significant shift.

Russian Empire https://www.openhistoricalmap.org/relation/2696913

Depending on your screen size, zooming to the Empire's extent is z2 or z3. At z3 (large screen) it's fine. At z2 the map is badly aligned and at z3 it's fine. Changing to z2 makes the map misalign.
Changing the basemap to Standard OSM, you can see the base map jump and now matches to the GeoJSON overlay.

Texas https://www.openhistoricalmap.org/relation/2692911 I can't replicate it being misaligned at z7 as in the screenshot. Nor can Sanjay yet.

But, if you zoom out to z2 you can see that the vector basemap and Standard OSM tile basemap are in different places, so Texas' GeoJSON overlay looks wrong (rather, it fits to the tiles, not to the vector basemap).

So, I'm pretty sure the real issue is that the MBGL / OHM vector basemap. It may be the tiles, or the MBGL-Leaflet connector code.

We just deployed this fix to production:

Seems to be working, but @jeffreyameyer please confirm also working for you

@jeffreyameyer says:

it looks like the alignment issue persists if you first pan and then search...

And posts this screenshot (in Slack)

I have not been able to repro this.
I tried:

Search, then pan, then pick one boundary to load: https://recordit.co/kJ45Q82xVq
Pan, then search, then pick a boundary to load: https://recordit.co/HdtvHPAonk
Search, pick a boundary, and pan while the GeoJSON is loading -- it's slow and I thought this would be the one, but no. https://recordit.co/8JvTuyPuZy

In all three cases, when the GeoJSON finally loads, it snaps into the correct location on the map.
@jeffreyameyer can yoiu post a screencast of how you got to your screenshot above?

you betcha!
https://user-images.githubusercontent.com/1494546/162299632-2f72eaa6-1265-408d-b029-4c7cfc4ad9be.mp4

So it seems like eventually the vector tiles can be forced out of syn with the raster map/Leaflet. I wonder if we could "solve" this by adding a maxBounds to the whole leaflet
On reflection, I don't think the large relation loading has anything to do with this bug. Similar to what @geohacker mentioned in Slack, it's related to the gray bars, but not in an immediately obvious way.
Here's what I see: https://recordit.co/nmtej9K3rX
As one pans super low on the map, things go haywire.
Rather than digging into the internal of this bug, could we solve the problem AND get rid of the gray bars simply be setting a maxBounds on the whole map that prevents panning the map right off the page?

Dan has a point. With no search or anything, using vector tiles or a raster tile basemap, I can pan north north north forever. See how the scalebar keeps changing even though I've panned well past reason.
With a maxBounds in place, this does not happen and the world stays properly in the rectangle.

See new PR at https://github.com/OpenHistoricalMap/ohm-website/pull/83

OK, I just deployed that PR and I am unable to get the Russian Empire to fall out of alignment with the vector tiles. @jeffreyameyer let's have you give it another shot and see if the maxBounds did the trick. Side bonus is that we don't get mostly gray map panes by scrolling off the edge of the world (though a smaller gray band above and below is still possible at the top and bottom, but now it just matches what we see in raster tiles and seems acceptable for the benefit of allowing folks to see the whole world at various window sizes).

I think that last change fixed one issue but caused a much bigger issue.
Here's a screencap: https://recordit.co/7mHbka4kb3
Basically, if you try to zoom in on any location far east or west of the Prime Meridian, it snaps back.
I suspect some issue with the maxbounds we added. For now, I am reverting that change on production and we'll do some more thorough testing on staging before we deploy anything more on this ticket

Oy, how embarrassing for me. I swapped lat,lng. The line should read maxBounds: [[-90, -180], [90, 180]],
I'll have a new PR presently.

OK, that's now deployed at staging:
https://staging.openhistoricalmap.org/#map=6/3.689/139.395&layers=O&date=1900&daterange=1800,2022
I'm going to wait on pushing that to prod so we can do some more testing.
Here's the behavior on staging of being zoomed way out and then box-zooming on Japan:
https://recordit.co/FeVlJBefAC
The map jumps down to New Guinea for reasons unknown (to me). I wouldn't expect that since we're within the maxBounds.
But I just reproduced that on Production, where we currently don't have maxBounds set, so... this issue might be something more like "Leaflet with GL layers is buggy". Hmm. Not a satisfying answer!


WIP

@nikku So, it seems variables are initially taken from the builtins and then from the context during evaluation, which means that the context can never override the variables
I switched it around, (had to do null checks otherwise false isn't accepted as a context value), but I'm still not sure what rules we should be aligning against. The DMN spec? I see there's DMN TCK, how can I go about using that to check against?
Anyways, I think the issue of simple variable overriding is pretty easy, but doing stuff like "date and time" with composed variables is hell, so let's worry about this case later.

Getting back to your PR it looks like this is related to https://github.com/nikku/feelin/issues/62. Can you confirm or deny?

Closing in favor of https://github.com/nikku/feelin/pull/63.


feat(generators): allow java/ts/owl gen via makefile

This PR allows user to trigger java and typescript generators via config.env and make setup because user might want them but gen-project excludes.  Additional user can re-trigger owl generator if https://github.com/linkml/linkml/issues/1453 is an issue.
Changes are transparent to existing users.
Once owl bug is fixed and gen-project includes java/ts then most of this PR could be reverted.

Hi @noelmcloughlin - I'm confused at the difference between this PR and this one: https://github.com/linkml/linkml-project-cookiecutter/blob/main/{{cookiecutter.project_name}}/config.yaml :) can you help me figure out why we need both config.env and config.yaml?   (maybe its just ease of use for folks more comfortable with env file vs. yaml?)

Ideally config.yaml is enough and make setup and make gendoc are exhaustive.
However there are 3 issues where expected behavior does not match actual.

make setup does not generate typescript and java artifacts
make gendoc ignores config.yaml (I think)
there is a bug using with gen-owl + config.yaml (https://github.com/linkml/linkml/issues/1453)

For long time I  planned doing linkml demo to some colleagues, but working through few things. So this project is good starting point - however, I would prefer not to maintain my own fork to address issues, so I proposed config.env as short term workaround.
I'm also willing to raise issues for above and link to this PR

I corrupted my branch so closing - will have to redo


Build against Go 1.18

Fix lint issues that arose during the upgrade.
This does not yet verify whether gopatch works with Go 1.18 syntax.

Codecov Report

Merging #49 (c43c1e0) into main (849212e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #49   +/-   ##
=======================================
  Coverage   86.41%   86.41%
=======================================
  Files          38       38
  Lines        2032     2032
=======================================
  Hits         1756     1756
  Misses        221      221
  Partials       55       55

Continue to review full report at Codecov.

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


Using Sparrow for first time, creating new wallet has convoluted non-linear flow

I suggest NEW to a more prominent option, Like being first of these 4 options, few people have a hardware wallet.

Being switched to a page that doesn't have any visible NEW / Generate option.

Only at this point do we see "Generate New"

No idea what that step is about, "Create Keystore" Seems like a step only to teach the word "Keystore", still no idea what that's about.  How about "Generate Wallet"?
Although the suggested next button is highlighted in blue, it's very overwhelming to still have all other options and buttons visible.

One more step, What if the text field would be grayed out and clicking "Custom derivation" would make it editable with the option of one of 9 sub-accounts as a dropdown.

One more step, Now sure what the "Revert" button would do, Edit last step? Well, no It just deleted everything and bring us back at step one. Should be named "Clear", "Revert" sounds like Undo (last step) to me.

Nothing seems to happen after clicking "Apply" and not clear next that someone has to manually toggle that tiny gray button at the bottom.
Also seems like the first time I toggle connect it never end up connecting.
 Stuck on showing this, shouldn't the attempt eventually time-out, retry or something?
Might want to jump the user to Transaction, Receive, or Addresses panel to they can see something has happened and made available.

I understand where you're coming from - a UI can't please everyone, I've found. But these are my rationales:

few people have a hardware wallet

Actually, I believe Sparrow is usually used with a hardware wallet.

Being switched to a page that doesn't have any visible NEW / Generate option.

Adding this would direct people down a single route. Often for example, you want to restore a seed not create new one. BIP39 seed? Electrum seed? Xprv? Lots of choices.

How about "Generate Wallet"?

There is a problem in the Bitcoin wallet space that everything is called a wallet. What happens when you use multisig - do you call that a "wallet of wallets"? It gets very confusing. Sparrow (and Electrum) use the word "keystore" to help identify sources of keys.

it's very overwhelming to still have all other options and buttons visible.

It might be an idea to remove the other options at this point. But it could also be disturbing to have UI items vanish in unpredictable ways.

Also seems like the first time I toggle connect it never end up connecting.

Blockstream's server is often very slow. You can configure another.

One more step, What if the text field would be grayed out and clicking "Custom derivation" would make it editable with the option of one of 9 sub-accounts as a dropdown.

That disables the textfield at the expense of enabling the Custom Derivation button. I don't think it's an improvement.

"Revert" sounds like Undo (last step) to me.

That's exactly what it is. In this case, revert to the original state when you first created the wallet. Once the wallet has been saved, any changes you make to the Settings can be undone with the Revert button.

Nothing seems to happen after clicking "Apply" and not clear next that someone has to manually toggle that tiny gray button at the bottom.

Creating a wallet and connecting to a server are independent - there are good reasons to create a wallet and stay offline. The introduction wizard takes some time to explain connecting to a server.

Might want to jump the user to Transaction, Receive, or Addresses panel to they can see something has happened and made available.

The status bar does suggest this once connected, but I don't believe in changing the UI without the user requesting it.

AFAICT these suggestions (although well intentioned) are a personal preference which conflicts with the design and intention of Sparrow

Re, I'm creating a wallet to reproduce a bug,
I've been using sparrow for a little while and I got completely lost trying to create a wallet again.
I don't think it's normal to click Enter to find Generate

Improved in 1a9c6f8 and followups:


[TextField] letter descenders are cut off in Firefox

The descenders of letters are cut off in TextField when viewed in Firefox:

(screenshot taken on the Text Fields demo page)

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Letters should not be cut off in input fields.
Current Behavior

Letters with descenders are cut off in input fields (in Firefox, not in Chrome).
Steps to Reproduce (for bugs)

Type jjyyqqpp in a Material-UI text field in Firefox.
Observe that the bottoms of the letters are cut off.

Context

This StackOverflow question/answer suggest changing the height of the input to match its line height. I see that the computed line-height is 21.5px, so I changed my input's height to 21.5px and it fixed the problem. I doubt that this is any more than a hack, though.
Your Environment

Tech
Version

Material-UI
1.0.0-beta.20

React
16.1.0

browser
Firefox 57.0

OS
MacOS 10.13.1

how about change inputSingleline : {height: "1em"} to  inputSingleline : {height: "1.2em"}

@nuoyz Have a look at the related pull request for the full solution.

We are seeing this again on:
Material-UI: v1.0.0-beta.47
React: 16.3.2
browser: 66.0.3359.181
OS: Windows 10
This occurs at a resolution 2880x1800 scaling to 125%.  This does not happen with scaling turned off.  Also, if I set the input height to 1.2em the problem is resolved for us.


Custom icons index not working properly

When trying to add custom buttons to the toolbar, the index attribute should allow for setting the position of the custom element. But it does not work as intended. The index of existing default elements does not get shifted to accommodate for the new elements, making it impossible to properly position them.
For example, when adding a custom element with index 1, it gets placed at the second position from the left. When index is set to 0, it is placed all the way to the right. I had limited success with negative indexes, but I also noticed the the toolbar seems to have a limited predefined space available and adding new elements shifts the entire toolbar to the right to the point where it runs out of space and items start overlapping.
Also it doesn't seem possible to put an element on the left corner of the diagram window away from the default controls.

You did not follow the issue template and did not provide a reproduction app


Roll back case-insensitive sorting

Describe the bug
Case-insensitive sorting introduce for #253 breaks sorting for any non-String based properties (size, length, etc.)
See https://github.com/Adobe-Marketing-Cloud/asset-share-commons/issues/253#issuecomment-419430138 for more details.
Environment

AEM Version: 6.4.1
Asset Share Commons Version: 1.6.3-SNAPSHOT
Author, Publish or both? both

To Reproduce
Steps to reproduce the behavior:

Install latest from /develop
Go to 'the search page'
Select to sort by Width and Desc
See that assets with a widths of 9xx are shown before those with 5xxx

Expected behavior
Sorting of values would be sorted naturally based on the value type (String, Number, Date)

released in v1.6.4


Fix formatting

What this PR does / why we need it:
Fixes a small formatting issue in the README
Which issue this PR fixes
NA
Special notes for your reviewer:
NA
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/lgtm


gpu-memory  keep  increasing  after 2nd iteration

hi, i encountered an issue when i run train scipt.  Test result is ok and the same as shown in readme.
But the gpu-memory keeps increasing after 2nd iteration  when getting into  DeformableTransformerDecoderLayer(MutliHeadAttention  and MSDeformAtten).  The max_gpu_reserved can reach even more than 80G.
The environment is as follows(for some reasons, config and envs list are not allowed to paste out of inside network, sorry):
Ubuntu，pytorch1.10, cuda11.1, mmdet3d(same as readme)。
Nuscenes full1.0 and mini dataset are used to train as test.
config is the official  sparsefusion_nusc_voxel_LC_r50.py.
Look forward to your reply!

In our training process, the GPU memory may fluctuate, but it is always within the range of A6000 GPUs (48GB). In your training process, (1) does the memory overflow always happen after the 2nd iteration? (2) We use DeformableTransformerDecoderLayer multiple times in the pts_bbox_head. Can you find which time cases this problem?

@yichen928 Thanks for your reply.  in my opinion, after the 2nd iteration, the tensor cache of gpu memory should be stable(even if a little fluctuation). The memory increasing in my clone project occurs in MutliHeadAttention and MSDeformAtten forward function sometimes, and the deep condition of this phenomenon has not been found.
(1) this is debug process of gpu memory allocated and cached with torch.cuda.max_memory_allocated and cached.In this porcess, non-zereo value of allocated and cached are shown.

(2) the following img shows the exact position which generate non-zero value of increasing memory.

transformerdecoder.py

if key_padding_mask is not None:
   ...
    attn_output_weights = attn_output_weights.masked_fill(
        key_padding_mask.unsqueeze(1).unsqueeze(2),
        float('inf'),
    )
    attn_output_weights = attn_output_weights.view(bsz * num_heads, tgt_len, src_len)

i am not famialier with DeformableTransformer. May the bug be related to pytorch version(v1.10 vs. 1.7.1)? or dcn_v2 package(at the begining of train process, "import DCN failed" message is printed)?

I tried the training code again, and it can still run within 40G memory on my A6000 GPUs. From my experience, if you are using A100 GPUs, it would need more memory (but I do not know why), so it may not fit in the 40G memory of A100. You may try to half the batch size to see whether it still overflows.
I did not try with our pytorch version, but you can try to use 1.7.1 same as me. The DCN is not actually used, you can just ignore the "import DCN failed" message.

Another suggestion is to limit the max power of your GPU with "nvidia-smi -pl 200" (200 is the limitation). This would not slow down your training, but can indeed save a lot of memory.

The environment with torch-1.7.1 seems to work well, even if a little fluctuation exists.
However, the env with pytorch1.10 still overflows. But there is no evidence to suggest that torch.masked_fill can cause the overflow.

glad to know that! We will dig into this problem further.


sql: create sql stats compaction scheduled job

https://github.com/cockroachdb/cockroach/pull/68401 --- Release note (sql change): introduce crdb_internal.schedule_sql_stats_compaction() to manually create SQL Stats compaction schedule. Extend SHOW SCHEDULES command to support SHOW SCHEDULES FOR SQL STATISTICS.

Duplicate of https://github.com/cockroachdb/docs/issues/11453


function to parse sequence parameters from string

bmi3d saves the sequence parameters in a json-like string format, e.g. "centerout_2D:[nblocks=100, ntargets=8, distance=6, origin=0 0 0]"
we should write a function that parses this string into the parameters and sequence name. here's some code to start:
bounds = [np.nan, np.nan, np.nan, np.nan]
        radius = np.nan
        if 'sequence' in metadata:
            sequence = metadata['sequence']
            sequence_name, seq_params = sequence.split('[')
            seq_dict = collections.defaultdict(list)
            for p in seq_params[:-1].split(', '):
                k, v = p.split('=')
                try:
                    seq_dict[k].append(int(v))
                except:
                    seq_dict[k] = np.fromstring(v, sep=',')
                    if len(seq_dict[k]) < 3:
                        seq_dict[k] = np.fromstring(v, sep=' ')
            if 'boundaries' in seq_dict:
                bounds = np.array(seq_dict['boundaries'])
            else:
                radius = float(seq_dict['distance'][0])

            if 'rand' in sequence_name:
                continue
        else:
            # print(f"File {file_days[idx_day][idx_file]} has no task metadata. Please re-export from BMI3D")
            sequence_name = ""

probably we don't need this, it's much easier to get this information from the database.


Offer an un-shaded version of the Flight SQL JDBC driver

Describe the enhancement requested
The Arrow Flight SQL JDBC driver shades and relocates all of its Maven dependencies into the driver JAR. While this is probably good and desired for folks who need to download the JAR and install it into whatever tool they're using to connect to something, it can cause an issue for those who are embedding the driver within their own service.
As an example, we're using OpenTelemetry's Java auto instrumentation agent. This agent modifies bytecode as it's loaded to include OpenTelemetry instrumentation for distributed tracing. It has built in support for gRPC clients and servers. Unfortunately, because the Arrow Flight SQL JDBC driver shades and relocates the gRPC libraries, they no longer utilize the same package name as what the Java agent expects. Subsequently, OpenTelemetry does not instrument the client calls and can not perform context propagation from the client to the server.
Would it be possible to publish both a shaded+relocated version of the driver and an un-shaded version of the driver?
The Maven Shade plugin supports building and publishing both artifacts via the shadedArtifactAttached configuration option. The JDBC driver has this option explicitly disabled for some reason.
Component(s)
Java

As a test, I made the following change to the flight-sql-jdbc-driver POM:
diff --git a/java/flight/flight-sql-jdbc-driver/pom.xml b/java/flight/flight-sql-jdbc-driver/pom.xml
index 1fd9222be..08e47187f 100644
--- a/java/flight/flight-sql-jdbc-driver/pom.xml
+++ b/java/flight/flight-sql-jdbc-driver/pom.xml
@@ -172,7 +172,8 @@
                             <goal>shade</goal>
                         </goals>
                         <configuration>
-                            <shadedArtifactAttached>false</shadedArtifactAttached>
+                            <shadedArtifactAttached>true</shadedArtifactAttached>
+                            <shadedClassifierName>shaded</shadedClassifierName>
                             <createDependencyReducedPom>false</createDependencyReducedPom>
                             <minimizeJar>false</minimizeJar>
                             <artifactSet>

I then built the entire Java tree with the following command:
JAVA_HOME=/usr/lib/jvm/java-8-openjdk-arm64 mvn -Dmaven.repo.local=/tmp/arrow/.maven_repo clean install

Looking in the local Maven repository, I see the following:
root@0a9e22d01fcb:/tmp/arrow/java# ls -l ../.maven_repo/org/apache/arrow/flight-sql-jdbc-driver/14.0.0-SNAPSHOT/
total 35416
-rw-r--r-- 1 root root      444 Sep 27 17:57 _remote.repositories
-rw-r--r-- 1 root root   125160 Sep 27 17:56 flight-sql-jdbc-driver-14.0.0-SNAPSHOT-cyclonedx.json
-rw-r--r-- 1 root root   111299 Sep 27 17:56 flight-sql-jdbc-driver-14.0.0-SNAPSHOT-cyclonedx.xml
-rw-r--r-- 1 root root 35978036 Sep 27 17:57 flight-sql-jdbc-driver-14.0.0-SNAPSHOT-shaded.jar
-rw-r--r-- 1 root root    10394 Sep 27 17:56 flight-sql-jdbc-driver-14.0.0-SNAPSHOT-tests.jar
-rw-r--r-- 1 root root    11230 Sep 27 17:56 flight-sql-jdbc-driver-14.0.0-SNAPSHOT.jar
-rw-r--r-- 1 root root    10716 Sep 27 17:40 flight-sql-jdbc-driver-14.0.0-SNAPSHOT.pom
-rw-r--r-- 1 root root     1553 Sep 27 17:57 maven-metadata-local.xml

This seems pretty workable for most people. Folks using Maven/Gradle/whatever to manage their projects would switch to pulling the un-shaded artifact and its corresponding dependencies. Folks needing the shaded JAR would have to start specifying the shaded classifier to get that artifact, instead.

One issue though is that Maven does not allow to have different pom files attached to different artifacts produced from the same module, and the shaded and unshaded versions would have different dependencies

Looking back at the code, isn't flight-sql-jdbc-core  what you're looking for?


Explicit semantic data types for ChargePower and DischargePower

What's needed?
Add two more boxed types

ChargePower
DischargePower

Both reflecting non-negative power with the semantic meaning encoded into the type system.
There shall be convenience methods like

@staticmethod from_power(Power), and
as_power(self) -> Power

Proposed solution
s.a.
Use cases
as discussed in the meeting, to encode meaning into the power values to catch value errors as early as possible, also aiding code review.
Alternatives and workarounds
No response
Additional context
in the FCR Prequalification Actor we've prototyped the use of such abstractions and had quite positive impressions about it.

I guess the battery pool charge/discharge command should return this new type, right? So this would be a breaking change?


bug(forge build): Stack overflow caused by FOUNDRY_LIBS or DAPP_LIBS containing an absolute path

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (cc8e430 2024-10-16T00:22:08.572482504Z)
What command(s) is the bug in?
forge build
Operating System
Linux
Describe the bug
Steps to reproduce

Push a temporary directory onto the stack

pushd "$(mktemp -d)"

Create a forge project

forge init

Build the project with the environment variable FOUNDRY_LIBS set to an array that contains an absolute path to a directory. Also works with DAPP_LIBS. In this case, we use the absolute path to the lib folder just created by the forge init command.

FOUNDRY_LIBS=["$(pwd)/lib"] forge build

This last command will exit with status code 134 and print the following error message.

thread 'main' has overflowed its stack
fatal runtime error: stack overflow
Aborted (core dumped)

Backtrace:
...
    frame #1574: 0x00005555573c9b39 forge`foundry_config::Config::to_figment::h541c6e067745c370 + 12577
    frame #1575: 0x0000555557384ad6 forge`foundry_config::Config::figment_with_root::h375a3e4abd4042a9 + 37
    frame #1576: 0x00005555573db55b forge`foundry_config::providers::remappings::RemappingsProvider::get_remappings::h4582cffefceeb412 + 3529
    frame #1577: 0x00005555573c9b39 forge`foundry_config::Config::to_figment::h541c6e067745c370 + 12577
    frame #1578: 0x0000555557384ad6 forge`foundry_config::Config::figment_with_root::h375a3e4abd4042a9 + 37
    frame #1579: 0x00005555573db55b forge`foundry_config::providers::remappings::RemappingsProvider::get_remappings::h4582cffefceeb412 + 3529
    frame #1580: 0x00005555573c9b39 forge`foundry_config::Config::to_figment::h541c6e067745c370 + 12577
    frame #1581: 0x0000555557384ad6 forge`foundry_config::Config::figment_with_root::h375a3e4abd4042a9 + 37
    frame #1582: 0x00005555573db55b forge`foundry_config::providers::remappings::RemappingsProvider::get_remappings::h4582cffefceeb412 + 3529
    frame #1583: 0x00005555573c9b39 forge`foundry_config::Config::to_figment::h541c6e067745c370 + 12577
    frame #1584: 0x00005555570937d5 forge`foundry_cli::opts::build::core::_$LT$impl$u20$core..convert..From$LT$$RF$foundry_cli..opts..build..core..CoreBuildArgs$GT$$u20$for$u20$figment..figment..Figment$GT$::from::h321b45ab02f0c9ba + 549
    frame #1585: 0x0000555556106ef7 forge`forge::cmd::build::_$LT$impl$u20$core..convert..From$LT$$RF$forge..cmd..build..BuildArgs$GT$$u20$for$u20$figment..figment..Figment$GT$::from::heca88b3325567215 + 39
    frame #1586: 0x0000555556028a74 forge`forge::cmd::build::BuildArgs::run::hb7a2624b2cda8b4f + 84
    frame #1587: 0x00005555560e63b2 forge`forge::main::h580bf58d4be50b43 + 62754
    frame #1588: 0x0000555555bf0843 forge`std::sys::backtrace::__rust_begin_short_backtrace::h4216eb9ef7f856ad + 3
    frame #1589: 0x0000555556165330 forge`main + 1008
    frame #1590: 0x00007ffff7c8ce08 libc.so.6`___lldb_unnamed_symbol3261 + 120
    frame #1591: 0x00007ffff7c8cecc libc.so.6`__libc_start_main + 140
    frame #1592: 0x000055555590f58e forge`_start + 46

Happens because of this line which will end up trying to get remappings by recursing on itself
https://github.com/foundry-rs/foundry/blob/a970b36d451fe84b8fba456ec99bf822376e56b8/crates/config/src/providers/remappings.rs#L206

@DaniPopes Hi, could you please assign this issue to me? I will investigate and fix it.


Datastore fails to save model with empty but optional 1:n relation

Describe the bug
When having two models connected over an optional bidirectional 1:n relation, traditionally a Post with many Comments, and then trying to save a Comment with nil as "post" and "postID" attributes, the save fails saying it expected a String.
To Reproduce
Steps to reproduce the behavior:

Set up an Amplify App with two Models connected over a bidirectional 1:n relation. To reproduce I used the following graphql schema fully generated by the AdminUI:

type Comment @model @auth(rules: [{allow: public}]) @key(name: "byPost", fields: ["postID"]) {
  id: ID!
  content: String
  postID: ID
  post: Post @connection
}

type Post @model @auth(rules: [{allow: public}]) {
  id: ID!
  title: String
  content: String
  comments: [Comment] @connection(keyName: "byPost", fields: ["id"])
}

Try to save a Comment without any connection to a Post:

Amplify.DataStore.save(Comment(
    content: "Lorem Ipsum"
)).sink(receiveCompletion: { completion in
    print("Completion: \(completion)")
}, receiveValue: { result in
    print("Result: \(result)")
})

The Console Output should be sth like this:

Completion: failure(DataStoreError: The data couldn’t be read because it is missing.
Recovery suggestion: The data couldn’t be read because it is missing.
Caused by:
valueNotFound(
  Swift.String, Swift.DecodingError.Context(codingPath: [
    CodingKeys(stringValue: "elements", intValue: nil),
    _JSONKey(stringValue: "Index 0", intValue: 0),
    CodingKeys(stringValue: "post", intValue: nil),
    CodingKeys(stringValue: "id", intValue: nil)
  ],
  debugDescription: "Expected String value but found null instead.", underlyingError: nil)
))

Expected behavior
It should save the Comment to the database without any connection to a Post (even if this wouldn't make much sense in this case).
Screenshots
The Model within the AdminUI.

Environment

Amplify Framework Version: 1.7.2
Dependency Manager: Cocoapods
Swift Version: 5.0
AmplifyCLI Version: 4.46.1

Device Information

Device: iPad Pro Simulator
iOS Version: iOS 14.4

Additional context
The problem can be fixed by having a Post given to the Comment as "post" attribute.
In case this is expected behavior and I missed sth in the documentation or elsewhere I apologize for the inconvenience but I would still think that the error given by Swift is not well suited for explaining the underlying issue.

@Erik7354
Thanks for reporting this. I believe the behavior you're seeing is expected, but as you identify, we should do a better job helping you recover. Even better, we should try to prevent this from happening in the first place. Bottom line is, a 1:1 relationship requires both sides of the relationship to be present upon saving. In your case, as you noted, you'll have to provide a value for the post field when you save your `Comment.
@aws-amplify/ios-team: I think we've got a couple of things to look at here:
Conflicting requirements between schema and annotation

The GraphQL schema defines post as an optional field on Comment
The @connection annotation for a 1:1 relationship would enforce that post must NOT be null. An optional post on the Comment type would be modeled as a 0-1:1 relationship, which I don't think we support.

How can we resolve this ambiguity? Could codegen perform a validation and note that the @connection requires post to be a required type? Should we emit a warning about that? Could AdminUI generate the field as required rather than optional in the schema?
Unclear documentation
How can we clarify the documentation around relationship types? Is it extant, but not linked to the appropriate place? Should/can we link it in the AdminUI? Assuming codgen does validation, could we link to docs in the validation message?
Unclear error
In this case, the Swift error is technically accurate (our Swift GraphQL Document creation code inspected the 1:1 relationship and expected the post side of the relationship to be fulfilled with a String value containing the Post ID, but got a nil value instead), but not as helpful as we want. The "this should not happen" assertion is misleading in this case as well, since the current state of tooling allows for this to happen, and our code should be more resilient, or at least more helpful in determining the root cause.
We'll investigate this and make appropriate work items

@palpatim
Thanks for the reply. Let me know if you need any further information.

@Erik7354 Thanks for reporting this. We’re working on ways to make DataStore relationships simpler. Once we get that done, we may need to adjust code in the DataStore plugin to properly handle the situation above. We’ll update this issue when we have more information.

Hello,
Thank you for your patience and we apologize for the delay in response. This issue seems fixed in the latest CLI and Amplify releases. For context, I used Amplify version 2.9.0 and CLI version 11.0.3. With the new GraphQL v2 transformer, the schema would like below:
type Comment @model
@auth(rules: [{allow: public}]) {
  id: ID!
  content: String
  postID: ID @index(name: "byPost")
  post: Post @belongsTo(fields: ["postID"])
}

type Post @model
@auth(rules: [{allow: public}]) {
  id: ID!
  title: String
  content: String
  comments: [Comment] @hasMany(indexName: "byPost", fields: ["id"])
}

I used the below code snippet:
func createPostAndComment() async {
    let comment = Comment(content: "Lorem Ipsum")
    do {
        let savedComment = try await Amplify.DataStore.save(comment)
        print("[GH1140]Saved comment: \(savedComment)")
    } catch (let error) {
        print("[GH1140]Error saving comment: \(error)")
    }
}

Here's a screenshot from the console log:

Please let us know if you're still facing this problem.

Hello,
thanks for the continues work on the problem. Within the last year we already migrated away from AWS Amplify. Nevertheless I spun up the old project and can confirm that the issue is now fixed.


Grammar in "Additional context" Description

The description for the other field could be reworded slightly for clarity and to avoid an awkward sentence structure.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Security considerations for Viscerion

Considering this is a VPN app and users probably do things ranging from
whistleblowing to bringing a different flavour to porn recommendations
using it, it would be unwise to not give some thought to this aspect of
things.
What would you as a user like to see in Viscerion to ensure that the app
atleast feels secure to you?

An option to "rebuild" the app with an undetectable package name, so it can be hidden?

An option to "rebuild" the app with an undetectable package name, so it can be hidden?

Is there any actual purpose to it though? Are apps actually scanning for the Viscerion package name?

The ability to do https://github.com/MSF-Jarvis/viscerion/issues/66 to a WebDAV instance instead of Google's servers.

Ask @taviso to audit it :laughing:


initialLocation doesn't support what router.push() supports

Reproduction
The documentation says:

initialLocation accepts anything that can be passed to router.push().

However, this results in an error:
const router = createRouterMock({
    initialLocation: {
      name: 'Reports',
      params: {
        domain: 'example.com',
        code: 'code'
      }
    }
  })

...but this does not:
await router.push({
      name: 'Reports',
      params: {
        domain: 'example.com',
        code: 'code'
      }
    })

The error with the first one is:
Serialized Error: {
  "location": {
    "name": "Reports",
    "params": {
      "code": "code",
      "domain": "example.com",
    },
  },
}

(To be fair, the router.push() code also doesn't work when I try to use:
const route = useRoute()
const aliasDomain = String(route.params.domain)

...which I'm still trying to figure out?

I can't know what's wrong with just this info but maybe you are using artificial params? See https://github.com/vuejs/router/blob/main/packages/router/CHANGELOG.md#414-2022-08-22
Remember to use the Discord chat to ask questions!


check if the walker is on the crosswalk.

Hi,
How can I check if a walker cross the street on a crosswalk or not??
Thanks.

I need help, SOS.

@yousinf could you please elaborate a bit more?

Hi, currently we don't have that exposed. Probably wouldn't be too hard to expose that in property of the pedestrian that you can check in which kind of zone the pedestrian is walking (road, sidewalk, crosswalk or grass). Because that is done by the client there is no performance problems. I will add this as a feature request.


Fix typos

What current issue(s) does this address, or what feature is it adding?
Fixes two typos on print messages
How did you solve this problem?
Fixed the typos
How did you make sure your solution works?
N/A
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[ ] Did you add any tests?
[x] Did you run make lint?
[ ] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[ ] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage remained the same at 80.17% when pulling 93d97d23caf7c5a3270a4ebe5ed9158eddc4ee7f on tomascarvalho:fix-typo into f1a5db60d56df40f56f6baa97d725e8ebc0ba2a5 on CityOfZion:development.

thanks!


fix: Fixes inconsistent api usage of state registry

@AlexBHarley

What does this fix in practice?

The centralized state registry we used for deepstreamHub via redis. Will hopefully release it as a plugin sooner than later

Looks good.
But build is failing for what looks like proper reasons.


Support for vSphere

Hi okd working group,
After some initial problems pull-secret={"auths":{"fake":{"auth": "bar"}}}, I was able to install okd 4.3 on vsphere. I followed this (OpenShift 4.2 vSphere Install Quickstart)[https://blog.openshift.com/openshift-4-2-vsphere-install-quickstart/]
I encountered the following issues. The image registry was not able to start, because of a missing (ReadWriteMany) persistent storage (PV). After changing the storage to empty dir
oc patch configs.imageregistry.operator.openshift.io cluster --type merge --patch '{"spec":{"storage":{"emptyDir":{}}}}'
and
oc patch configs.imageregistry.operator.openshift.io cluster --type merge --patch '{"spec":{"managementState": "Managed"}}'
(Value before Removed) the image registry was able to launch.
Now I'll deep dive into okd4. Thanks to @LorbusChris for fast help via chat.
Best Regards
Reamer

thanks @Reamer for taking the time to test and also provide feedback. I guess there is no followup needed here in which case @LorbusChris  we need to think on a process to maybe add a label like note or s'thing which should provide folks the message that there is nothing to follow up/ action on it

Related to https://github.com/openshift/okd/issues/14 provider

Closing in favor of #14


Use new out of gopath codegen

Changes

Use new knative.dev/hack go path function to generate client code from anywhere.

@googlebot I fixed it.

/lgtm

/retest

/lgtm


Class User: the binary gender field

Backgrounds
Currently, the app has a model User with a Gender field like this:
GENDER_CHOICES = (
    ('male', 'Male'),
    ('female', 'Female'))

class User(AbstractUser):
    # ...
    gender = models.CharField(max_length=10, # ...

Screenshot

Issue
The Gender property value, on app, is constrained to a binary gender.
Potential approaches
Solution 1
Do nothing. It's right.
be free to vote with :-1:
Solution 2
Add a third option: "non binary" gender.
be free to vote with :+1:
Solution 3
Remove this field, it does not achieve anything.
be free to vote with :rocket:
Solution 4
Allow edit the gender to write a free text. Like Google does:
be free to vote with :tada:

Other alternative solutions.
please, be free to post your own as a new comment.

@ctrl-alt-d I'm agree with solution2 and solution3. For me: male, female, non binary and blank as first option. So really are three options and the fourth one (bank) says: thats not important.


Investigate and Adopt KEP 3157 Watch List in Istio

KEP is here: https://github.com/kubernetes/enhancements/issues/3157
Looks like we just need to set an env var based on what we detect the kube version to be during install?

I don't think it needs to be conditional enabled by version, client-go handles that already IIUC

Not need a env, just automatically detect kubernetes version?

Istio shouldn't auto-enable non-stable features so at best we should add
auto-detection once it goes GA
On Mon, Dec 16, 2024 at 10:24 PM hzxuzhonghu @.***>
wrote:

Not need a env, just automatically detect kubernetes version?
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/54344#issuecomment-2547600007, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXOM6WKQO3GZJN5JFBT2F67R3AVCNFSM6AAAAABTWWW3S2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBXGYYDAMBQG4
.
You are receiving this because you commented.Message ID:
@.***>


Add tests to lease multiple connections at once

Codecov Report

Merging #440 (cd70bc4) into main (d5d16e3) will increase coverage by 0.11%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #440      +/-   ##
==========================================
+ Coverage   59.37%   59.49%   +0.11%
==========================================
  Files         124      124
  Lines        9820     9820
==========================================
+ Hits         5831     5842      +11
+ Misses       3989     3978      -11

see 3 files with indirect coverage changes


Remove null content sourcefile before zipping

Preliminary fix for #2727, fishing for thoughts before I write an actual test for this.
Other options are:

figuring out whether content should ever be null and if it shouldn't stop it from ever being null
replace null content with empty string and then zip the empty file

This should be a draft and will not be merged.

Thinking an empty file might be better, can you post a count of how often this happens and with what files (in what workflows) on Slack?


queries sometimes return a different model

Sometimes when I iterate through a query like MyModel.where(...).each { ... }, or when I find by the hash key like MyModel.find, the object that is returned would be from a different model. For example, MyModel.find would return AnotherModel.
I can't reproduce this, but I see errors logged in the code daily that are like:
undefined method `some_method' for #<AnotherModel:0x000abcd>

I'm expecting MyModel but Dynamoid methods are returning AnotherModel for some reason. I've also put in logging to check returned instances, using .is_a? to validate. The code works as expected most of the time, but I'm seeing these type of errors come up more and more often, usually in the same place, and now with more and more models as the application is growing.
Wondering if you guys have any ideas as to what might be going on?

The first idea that comes into my mind is STI. It would be explain such cases.

gem 'dynamoid', '3.3'
Yes, some cases are STI:
class User; end;
class UserA < User; end;
class UserB < User; end;

UserB.where(...).each { ... } # returns `UserA` or the base `User` objects instead

I wasn't able to see any pattern to help me diagnose.
However, I also very recently seeing non-STI classes doing the same thing.

It would be great it you provided any example to reproduce the issue. If there is a bug in Dynamoid it should be easily reproducible.

I'm not exactly sure how to reproduce with code, but I can show you an example through my error logs and code if that can provide any help:

You can see here, that GeocoderCache.find(...) seems to return a Lawyer object, and both classes are totally unrelated. This is very strange.
Here are the snippets for both the GeocoderCache class and the Lawyer class:
class GeocoderCache
  include Dynamoid::Document
  table name: :geocoder_cache_tablename, key: some_key, capacity_demand: :on_demand
  ...
end

class Lawyer < User
...
end

class User
  include Dynamoid::Document
  table name: :some_users_table_name, key: :some_key, capacity_demand: :on_demand
  ...
end

This code path is run multiple times a day, but this error only happens sometimes, so I'm not sure how I can reproduce it.
This is in Rails 6.0, in Zeitwerk mode.

It would be helpful to have examples of GeocoderCache items, for which this error is reproduced (and find returns instance of other class).
The only valid case as I mentioned above is STI. So we need to check whether these items have the type attribute in the DynamoDB table (or another attribute defined with inheritance_field option). It actually doesn't matter whether there is a declared field type in the model  - it's enough to have such attribute (type) stored in the DynamoDB table. As far as DynamoDB is schemaless and allows items to have own attributes it's quite possible that some items have additional not expected attributes.
If you confirm that these items don't have type attribute or have but values are correct (equal "GeocoderCache") - then it's a Dynamoid bug. But without some particular example it would be difficult to figure out where the bug is.

Thanks. I will see if I can capture the item body when this happens so that I can investigate if it has unexpected type fields. Hopefully this will help me determine if it's my code's fault with setting an unexpected type or if I am able to reproduce a bug.

I did some logging of the type fields and I found an instance where it was my fault. I will close this for now, thanks!


Desktop Notifications Not Appearing

OS: Windows 10 (64 bit)
Issue Descriptions: I recently installed this music player and love it, but I don't appear to have desktop notifications. I have poked through the settings and haven't found an option to enable or disable them. Perhaps I'm just missing something?
Steps to Reproduce:

Are they turned on in the actual Google Play Music settings?  Not the desktop settings but the normal settings?

Aha! As I figured, I'm a dunce and missed that altogether. Issue closed, sorry for wasting your time and thanks for the insanely fast response!


fuzz: account for Unicode class size in compiler

This improves the precision of the "expression too big" regex
compilation error. Previously, it was not considering the heap usage
from Unicode character classes.
It's possible this will make some regexes fail to compile that
previously compiled. However, this is a bug fix. If you do wind up
seeing this though, feel free to file an issue, since it would be good
to get an idea of what kinds of regexes no longer compile but did.
This was found by OSS-fuzz:
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=33579

This PR is on crates.io in regex 1.4.6.


An unknown error

ProtoBuf ??
Hello, i am trying to build the code sample to understand the flow in the library.
My Dev Env:
OS: Win7 32 Bit OS
STUDIO: VS 2010
Am getting protobuf library reference issue.
How to solve this ?
Thanks
Ronald

which sample?

In the online phonebook example ..
I tried to build it, but I see lot of errors in VS.
Thanks,
Ronald
From: Halil İbrahim Kalkan<EMAIL_ADDRESS>Sent: 12 September 2016 23:04
To: hikalkan/scs<EMAIL_ADDRESS>Cc: ronea23<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [hikalkan/scs] An unknown error (#15)
which sample?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hikalkan/scs/issues/15#issuecomment-246426943, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKZD4qnMbFzra6sTpc1ByI8hj01vIgu_ks5qpY0CgaJpZM4J6dnR.


Disable dashboard if no metric & health ability

Relates to: #333
New topic: /topic/instanceAbility
Payload: InstanceAbilitiesRefreshedEventMessage$Payload


Rest api additions

This should be merged the same time https://github.com/RocketChat/Rocket.Chat/pull/5140 gets merged.

@sampaiodiego I agree, the permissions should be added. Would be a nice contribution for someone to add who wants to get familiar with the source of Rocket.Chat


Prevent upload of NCO reports flagged as "Hold"

Any NCO report with handling set to "Hold for Future Edit" should not be uploaded.  Perhaps provide a checkbox or other confirmation to force upload to the server, with the caveat that they can no longer be accessed or edited locally?

Implemented.  Will not implement force upload at this time.


func tabBarController not called on swipe

Expected Behavior
I expect the tabBarController didSelect method to be called when I change tabs with a swipe.
It it properly called when I tap on the tabs themselves
Current Behavior
The tabController didSelect method is not called on swipe
Possible Solution
No, sorry :/
Steps to Reproduce (for bugs)

Download this repository
in the Example folder open the workspace in xcode
Add print("changed") in TabBarController.swift below line 37
Run in simulator
Something is printed when tapping the tabs
Nothing is printed when swiping

Environment

SwipeableTabBarController version: 2.1.1
Device and OS running tests: Xcode simulator
Demo project link: https://github.com/marcosgriselli/SwipeableTabBarController/tree/master/Example

Hey @jfoucher thanks for reporting this. Selection works in a strange manner. I recommend you observe the selectedIndex property and react to changes. You can do it this way:

    private var observation: NSKeyValueObservation?

    observation = self.observe(\.selectedIndex, changeHandler: { [weak self] _,  _ in
        self?.selectedViewContollerWasUpdated()
    })

    private func selectedViewContollerWasUpdated() {
        /// code
    }

Ok, thanks for the workaround @marcosgriselli


In need of caching mechanism

When the object is moving draw call happens with 33ms periods. Only direction arrow changes. That causes flickering on map. Is there a possibility of caching base icon (without modifiers) and apply changes for every draw call?
If you can point out the related location on code base, I believe I can also apply that kind of caching mechanism as well.
Thank you,

Waht are you Doing?

There is a caching mechanism, but not for this kind of use. I would suggest that you request the symbol without the direction arrow, and then draw it on your map manually behind the symbol as another map symbol.


❓ - clusters

Description
Is there any way to display clusters in the template? Is there a component or an example implementation?
Library name

[ ] angular8-yandex-maps
[X] angular-yandex-maps-v3

Hi! It's not implemented right now. You can use a ready event on a map, then access ymaps and use a cluster without a component - https://yandex.ru/dev/jsapi30/doc/ru/ref/packages/clusterer/. I'll implement a component in a few weeks

Reopened, so I won't forget about the component

Available in 18.3.0.
Here is the example - https://github.com/ddubrava/angular-yandex-maps/blob/main/apps/angular-yandex-maps-v3-demo/src/app/app.component.html#L197


Error that follows execution. execvcp

Fresh install on hyprland-0.34.0-1
after running hyprland i get the following error
2024/01/25 15:54:16 socat[28025] E execvp("/home/someguy/.cache/yay/hyprevents-git/pkg/hyprevents-git/usr/share/hyprevents/event_loader", "/home/someguy/.cache/yay/hyprevents-git/pkg/hyprevents-git/usr/share/hyprevents/event_loader", "/home/someguy/.cache/yay/hyprprop-git/pkg/hyprprop-git/usr/share/hyprprop/event_handler"): No such file or directory

Additionally I am not able to pick from other monitors, only the monitor that has the hyprprop terminal window

This was PKGBUILD issue, but it's fixed and updated on aur.
I don't have additional monitors, so i have no ways of testing it, but feel free to open new issue as a feature request.

Multimonitor and special workspace support is now added.


New failures when using version flag

Recently the runner has been failing to execute this engine when specifying a version argument.
Workaround: remove any use of vs-version argument for the action and let it default to latest.

Discovered this was due to an incorrect version range being specified on user's workflow file.  Still recommend only defaulting to latest unless you are running a self-hosted runner.


Window Movment on Linux

It looks like I can't move the Window around on Linux. Running on an Arch VM with Wayland

So this appears to be an issue when I started using the i_slint_backend_winit private api crate so I could hide the window when "minimizing" to the tray, and to programmatically minimize the window from the minimize button.
On Windows and X11 the i_slint_backend_winit, seems to work fine. However on Wayland, I loose the ability to move the window for some reason with winit. If I switch to using i_slint_backend_qt instead of Winit, I loose the HiDPI scaling it seems, but I can now move the Window around on Wayland.
So something about using Winit on Wayland is causing an issue here.
I quickly tried to use the Qt backend on Windows and got a panic saying slint was compiled without Qt Support...strange since I'm pretty sure I've run other programs (not this one) on the same machine specifically with the Qt backend. I quickly did a cargo clean and recompiled in case some strange issue occurred with build artifacts, but got the same panic... Not sure what's going on there but I'll have to dig more into it later.
I wonder if a hacky temporary solution is some os specific config, on Linux I use Qt and on Windows I use winit....

https://github.com/slint-ui/slint/issues/3032
https://github.com/rust-windowing/winit/discussions/2920
It looks like I will either need to put in some cfg guards, basically use my existing logic, except on Winit Wayland do something else. Or force the winit background always and look into this drag_window method winit has. I'm not sure though if switching purely to winit and using this drag_window will have other consequences though...I'm currently leaning towards this instead of a whole bunch of cfg guards and trying to catch every possible edge case.

The movement logic has been updated to use winit::drag_window() and it seems to work across platform. This was fixed in commit 8c99939