RE: Encrypting and Decrypting?

Hi Maira and Matt! Enjoyed your walkthrough! Built the app and it appears to be working. However, I tested the program in 4 different ways: 1. Just to check the program was working, inserted "Test001" as the Plain Text input (I presume this would be synonymous with Username?) and "pass001" as the Password. This produces a cipher code. Decoding produces the Plain Text(PT) result - "Test001";  2. Plain text = James and Password = pass002 which produces PT - James; 3. To verify the Password security of #2, I used Plain Text = James and Password = passFake which produces PT - James?; 4. To double check that result, I used a random full name, i.e. Plain Text = James Riley Watson and a true Password = pass003. Then, I used the same PT full name, James Riley Watson and a fake Password = passFake. This produces a different cipher code from the first time the full name was inserted, but when I decoded I was surprised that the PT result was James Riley Watson?!  Since the cipher code for the first insertion of the full name with a "true" password and the cipher code for the second insertion of the same full name is different, I should not be allowed access to the user's name/account, right? Besides, the Password "passFake" had already been used so I guess it should not be allowed either nor should it produce protected data? I am just posting this because curious and I'm trying to learn more about data security and don't understand the results I obtained? Seems like there's no option to post the BMP here? Also, I would really appreciate comments on where this data is being stored to understand more about how secure this method is? Please correct me if there's something I've missed here! Any help is welcome!

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

ID: 8fabe7ed-c89e-6984-af23-b2052814b71e
Version Independent ID: 5e4ff908-d8cf-7c1c-b1b5-ad942d895ac0
Content: Encrypting and Decrypting Strings in Visual Basic
Content Source: docs/visual-basic/programming-guide/language-features/strings/walkthrough-encrypting-and-decrypting-strings.md
Product: dotnet-visualbasic
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Strange that no one has commented on this issue?


can not load models[BUG]:

How are you running AnythingLLM?
Docker (local)
What happened?

Are there known steps to reproduce?
No response

Please refer to https://docs.useanything.com/ollama-models-not-loading and https://github.com/Mintplex-Labs/anything-llm/blob/master/server/utils/AiProviders/ollama/README.md
And lastly, ollama serve should be running on the host machine outside of docker so that it can be reached.


feat/starknet_syncing

Issue: Implement starknet_syncing RPC Call
Description:
The starknet_syncing RPC call has been implemented but requires further testing and verification against the entire state. This RPC call returns an object about the sync status, or false if the node is not syncing.
Parameters:
None.
Expected Result:
The state of the synchronization, or false if the node is not synchronizing. The status can either be FALSE or a Sync status object.
Testing:
Initial testings can be found here. However, these should not be considered complete and improvements are welcome.
Resources:

Alchemy RPC API Endpoint Documentation
Starknet API OpenRPC Spec

Comments:
Please comment below for coordination.

can i take this one?

diff with pathfinder

result.highest_block : pathfinder gives the highest block in the chain even if it is not totally synchronized, deoxys gives the highest synchronized block


Prevent the NaturalSpawner Mixin from crashing due to poor ASM & Mixin Timing, Jar now works for loading worlds

Passes checktyle.


board is not clearly visible

Image is not detailed, very small to use somehow.

Image in the repo is only a preview. After clicking image on the main page you will bee redirected to whimsical board.
I have added a small readme with a link in the image folder.
https://github.com/igorwojda/android-ecosystem-cheat-sheet/blob/master/misc/image/board_readme.md
thx for contribution.


forge test is crashing with message: called Option::unwrap() on a None value

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 (1b08ae4 2024-05-21T00:18:03.928967000Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug
forge test running in github CI started crashing today for our project. After updating foundryup the problem is reproduced on local.
To reproduce:
git clone https://github.com/euler-xyz/euler-vault-kit.git && cd ./euler-vault-kit && forge install
forge test

The process crashes after executing a non-deterministic number of tests (1 suite, or more, or none) The message and location is always the same:
❯ forge test
[⠊] Compiling...
No files changed, compilation skipped

Ran 5 tests for test/unit/irm/IRMLimit.t.sol:IRMLimit
[PASS] test_IRMLimit_accum_50() (gas: 34546)
[PASS] test_IRMLimit_accum_500() (gas: 14143)
[PASS] test_IRMLimit_accum_max() (gas: 9334)
[PASS] test_IRMLimit_rpow() (gas: 15550)
[PASS] test_IRMLimit_sanity() (gas: 15492)
Suite result: ok. 5 passed; 0 failed; 0 skipped; finished in 1.06ms (760.08µs CPU time)
The application panicked (crashed).
Message:  called `Option::unwrap()` on a `None` value
Location: crates/evm/fuzz/src/strategies/state.rs:271

This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
                                ⋮ 8 frames hidden ⋮
   9: core::panicking::panic::hc59c8a709a9b37ae
      at <unknown source file>:<unknown line>
  10: core::option::unwrap_failed::h64934fbe08d33633
      at <unknown source file>:<unknown line>
  11: foundry_evm_fuzz::strategies::state::FuzzDictionary::insert_sample_values::h42f45967efcefd8c
      at <unknown source file>:<unknown line>
  12: proptest::test_runner::runner::call_test::h4f99e3104cde6929
      at <unknown source file>:<unknown line>
  13: proptest::test_runner::runner::TestRunner::gen_and_run_case::h87106920d1a7f96f
      at <unknown source file>:<unknown line>
  14: proptest::test_runner::runner::TestRunner::run_in_process::h3bc30562d4f2e349
      at <unknown source file>:<unknown line>
  15: forge::runner::ContractRunner::run_invariant_test::h0780e3e1779a90bc
      at <unknown source file>:<unknown line>
  16: core::ops::function::impls::<impl core::ops::function::FnMut<A> for &F>::call_mut::h0afc06bed2c5ba87
      at <unknown source file>:<unknown line>
  17: rayon::iter::plumbing::bridge_producer_consumer::helper::h97b33de326dcefc0
      at <unknown source file>:<unknown line>
  18: rayon_core::join::join_context::{{closure}}::h98f105d7c17b40d1
      at <unknown source file>:<unknown line>
  19: rayon::iter::plumbing::bridge_producer_consumer::helper::h97b33de326dcefc0
      at <unknown source file>:<unknown line>
  20: rayon_core::join::join_context::{{closure}}::h98f105d7c17b40d1
      at <unknown source file>:<unknown line>
  21: rayon::iter::plumbing::bridge_producer_consumer::helper::h97b33de326dcefc0
      at <unknown source file>:<unknown line>
  22: forge::runner::ContractRunner::run_tests::h0738715c1df1fa79
      at <unknown source file>:<unknown line>
  23: forge::multi_runner::MultiContractRunner::test::{{closure}}::h760504ea05713f37
      at <unknown source file>:<unknown line>
  24: rayon::iter::plumbing::bridge_producer_consumer::helper::hcfbd92bc36d4e856
      at <unknown source file>:<unknown line>
  25: rayon_core::join::join_context::{{closure}}::h004d9323ff3da6cb
      at <unknown source file>:<unknown line>
  26: rayon::iter::plumbing::bridge_producer_consumer::helper::hcfbd92bc36d4e856
      at <unknown source file>:<unknown line>
  27: <rayon_core::job::StackJob<L,F,R> as rayon_core::job::Job>::execute::hee9548015de2e41d
      at <unknown source file>:<unknown line>
  28: rayon_core::registry::WorkerThread::wait_until_cold::hc2f2f5c5d2ac906c
      at <unknown source file>:<unknown line>
  29: std::sys_common::backtrace::__rust_begin_short_backtrace::h26c5ce7ae435c375
      at <unknown source file>:<unknown line>
  30: core::ops::function::FnOnce::call_once{{vtable.shim}}::h7359e7db922f9b5d
      at <unknown source file>:<unknown line>
  31: std::sys::pal::unix::thread::Thread::new::thread_start::h9266fbbdd0c3d8be
      at <unknown source file>:<unknown line>
  32: __pthread_joiner_wake<unknown>
      at <unknown source file>:<unknown line>

Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
zsh: abort      forge test

release with fix https://github.com/foundry-rs/foundry/pull/7956 should be ready in couple

@grandizzy confirming the bug is fixed. Thank you!
I'm closing the ticket

bug was fixed


remove aqua queries

Closes #1033
Changes proposed in this PR:

remove interface QueryResult
update interface SearchQuery
remove ocean.metadataCache.transferOwnership  (no longer used)
remove ocean.metadataCache.getOwnerAssets (never used)
remove ocean.metadataCache.retire  (no longer used)
remove ocean.assets.ownerAssets
remove ocean.assets.resolveByDTAddress
remove ocean.assets.query
remove ocean.assets.search
move ocean.metadatacache tests from unit to integration

Labeled: Breaking because needs Aqua 3.1.0 or later.
PS: And since we are using Aqua 3.1.0, which has the events object in the ddo, a lot of improvements in the tests speed were possible

if we remove all the ocean.assets.* methods then even less clear what the difference between ocean.assets.* & ocean.metadataCache.* is, and we should either decide on one, or use one of it as a shortcut. E.g. over in #989 I use ocean.assets.* as a shortcut for methods https://github.com/oceanprotocol/ocean.js/pull/989/files#diff-193f6dfec7a03d2b356809b8e23c7ccd3790c854365aef4a58b96bdcb05b4c30R788-R808
So, what is the purpose of ocean.assets.* now?

Agree having MetadataCache and Assets is confusing and looking at the functions in them not sure exactly what handles each.

Agree having MetadataCache and Assets is confusing and looking at the functions in them not sure exactly what handles each.

I will refactor a bit

if we remove all the ocean.assets.* methods then even less clear what the difference between ocean.assets.* & ocean.metadataCache.* is, and we should either decide on one, or use one of it as a shortcut. E.g. over in #989 I use ocean.assets.* as a shortcut for methods https://github.com/oceanprotocol/ocean.js/pull/989/files#diff-193f6dfec7a03d2b356809b8e23c7ccd3790c854365aef4a58b96bdcb05b4c30R788-R808 but this is kinda only done because we have this weird split-up.
So, what is the purpose of ocean.assets.* now? Also ocean.assets.* sounds way more user friendly and we should rather work with that as main interface, e.g. ocean.assets.query() is way better than ocean.metadataCache.queryMetadata(), and it mirrors the actual API path, which always has assets in there

ocean.assets.query is back

@kremalicious @mihaisc - any more thoughts?

Hey @kremalicious, how do I get owner assets now that ownerAssets() is removed ?


ROS Bag?

When trying to install the ROS bag file to test the build of this repo on my machine the python file provided ends up creating a .db3 file is there a simple way around this or is there any other place to download a ROS bag.
Thanks

ROS2 bags are directories with a .db3 database and a meta data file.

Ok, thanks I have had more experience with ROS1 and those have an actual ros bag file I am relatively new to using ROS2 thanks.


.where() and .fillna() should preserve attributes

Fixes https://github.com/pydata/xarray/issues/1009
There is probably a more elegant way to do it, but the internals of xarray still confuse me. I'm happy to learn!

I implemented your comments with the exception of:

probably can put this in _fillna instead.

Do you mean this should/could be done in _binary_op instead?

probably can put this in _fillna instead.
Do you mean this should/could be done in _binary_op instead?

I forgot that we hack this into _binary_op right now. So no, please ignore this guidance then. This will cleaner after #964.

OK. Sorry for the mess with Travis, I had a lot of typos :( It's getting late here and the new review system is a bit confusing

Thanks!


Kindly provide support for x86_64 architecture

Currently, to the best of my knowledge, it is not possible to package both the arm64 simulator binary as well as as the x86_64 simulator binary within the same framework due to a limitation of the xcodebuild tool.
Thoughts?

I think since XCode 12.2, it is possible to ship for both architectures.  I am not so well versed in this, but i found some packages provide universal support and apple documentation also mentioned this.
https://developer.apple.com/documentation/apple-silicon/building-a-universal-macos-binary

A multi-architecture binary is not the same as an XCFramework.
A multi-architecture binary is created with the lipo tool, it appends the symbols of multiple single architecture binaries together into a single binary for the same library.
An XCFramework is created with the xcodebuild tool. The problem is that for the respective simulator binaries, the symbols appear the same to the tool.
I will however re-check the documentation just to be sure that there isn't currently any workaround for this.

Should be solved in this PR:
https://github.com/tkey/tkey-ios/pull/40


AWS Cognito fails to parse the metadata file/URL

Unable to unmarshall metadata

Thanks for reporting @digzapcg, are there any details on why the parsing fails in AWS Cognito?

Hey @deepakprabhakara , I couldn't find much info on the cognito console except for

when we provide the URL we get this error

when we provide the file we get this error

Aah looks like AWS Cognito is looking for the XML prefixes, thanks @digzapcg.

https://github.com/boxyhq/mock-saml/pull/104 should potentially fix this issue

@digzapcg The new version is out with a potential fix, could you please try and let me know?

@digzapcg Closing this, please reopen if it's not fixed yet.


Overflow issue with sap.m.MenuButton and sap.m.OverflowToolbar

OpenUI5 version: 1.40.8
Browser/version (+device/version): Google Chrome
Any other tested browsers/devices(OK/FAIL): IE 11 (FAIL)
URL (minimal example if possible): https://jsfiddle.net/2va1omz7/
Steps to reproduce the problem:

Resize the output pane

What is the expected result?
The same overflow behavior as sap.m.Button would be observed
What happens instead?
The sap.m.MenuButton controls do not overflow.

Hi @LucienCooper,
Thanks for your ticket!
Unfortunately this is not very well document it but not all controls can overflow inside m.OverflowToolbar.
All supported controls are in this file https://github.com/SAP/openui5/blob/master/src/sap.m/src/sap/m/OverflowToolbarAssociativePopoverControls.js this static property holds all supported controls OverflowToolbarAssociativePopoverControls._mSupportedControls. If control in this array has property "canOverflow: true," so it can overflow.
Unfortunately sap.m.MenuButton is not part of this list.
I will discuss with the appropriate people to add them to the control documentation.
Kind regards,
Petyo (openUI5 Dispatcher)


Wait type and Description table

Is there a table in sys already for the Wait_type and Description, so i can bring the description into a report?

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

ID: c981535f-9595-5acd-f1bc-c850e4073654
Version Independent ID: 8c73ce9a-0547-5d88-7b33-d09251ed7370
Content: sys.dm_os_wait_stats (Transact-SQL) - SQL Server
Content Source: docs/relational-databases/system-dynamic-management-views/sys-dm-os-wait-stats-transact-sql.md
Product: sql
Technology: system-objects
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@DeFlanko -- DeFlanko, thank you for your question.  You may find answers here:

DBA Stack Exchange
Stack Overflow

@MashaMSFT -- Masha, please look into this issue #4368.

Hi @DeFlanko , thanks for contacting Microsoft! My apologies,  but unfortunately there is no wait_type description within the product.
If this behavior is causing you a business issue and/or you feel that it should behave differently, please feel free to file a feedback item at https://aka.ms/sqlfeedback and our engineering team will be happy to triage the issue.
I hope that helps and please let us know if we can be of further assistance?
Thanks!
Masha from the SQL Docs team

Description is found in the documentation where it belongs. The product is geared towards minimizing footprint and therefore wait types have somewhat descriptive names, completed by docs, instead of very long strings resembling explanations.

@Masha thanks i only ask because our EMR vendor has description tables in
its SQL DB so i was wondering if Microsoft followed suit.  @Pedro the ask
was for reporting against this data for folks who were not technical and
dont know how to use microsoft documentation but still want to understand
why their business reports are being delayed.
solution: ill have to create a custom table, got it.
On Wed, Mar 18, 2020 at 8:36 PM Pedro Lopes<EMAIL_ADDRESS>wrote:

Description is found in the documentation where it belongs. The product is
geared towards minimizing footprint and therefore wait types have somewhat
descriptive names, completed by docs, instead of very long strings
resembling explanations.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/4368#issuecomment-600970427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABH52MPLDFNJYQ37OHFBX4LRIGHMPANCNFSM4LOTRPCA
.

--
Thanks, James

Hi @DeFlanko , thanks for providing the additional context! If this is important to you and the business, please be sure to submit your feedback request so the product group can track it.
I'll close out this Git Issue for the time being but please feel free to comment should you need anything further.
Thanks and hope you have a wonderful rest of your day!
Masha from the SQL Docs team


new tests for measure search

explores combinations of fixing fingerprint and similarity measure (as well as only_metric). Related to #166

@himaghna I think this should be passing tests -- could you take a look? Here is the test it is failing via GitHub action and here is the source code for the test.

test is failing because log directory is being passed to MeasureSearch() as path to log file. Therefore MeasureSearch creates a file with path log_dir which failes the isdir() assertion. Merging this PR and correcting the test in next commit.


File route URL escape error

What version of Remix are you using?
1.10.1
Steps to Reproduce
1.Create new router file name: $code[.]code.ts, It's OK
2.Create new router file name: code[.]$code.ts, But this doesn't match
Expected Behavior
--
Actual Behavior
--

What urls are you trying to match against?

React Router does not support partial dynamic parameters, such as path="prefix:param".  The parameter must be a full URL segment (between slashes).  This is why the second one doesn't match.  The first one matching is not quite valid since it generates a path of path=":code.code" and you can't currently use dots in param names.
For your case, you should use routes/$code.tsx and then you can split on the dot to get the first and second half:
// $code.tsx
export function loader({ params {) {
    let parts = params.code.split('.');
    // When URL = /code.123456
    //   parts[0] => 'code'
    //   parts[1] => '123456'
    // When URL = /123456.code
    //   parts[0] => '123456'
    //   parts[1] => 'code'
}


MH-13694 Sort roles alphabetically in UI

Instead of a roles listprovider, which returns the roles as a map, hand the roles to the admin ui as a list by a dedicated endpoint so the alphabetical order won't get lost.
This PR contains PR #1046, so shouldn't be merged before that one!
This work was sponsored by the ETH Zürich.

@KatrinIhler conflicts

@KatrinIhler conflicts again, sigh.

@KatrinIhler conflicts.  Also, should this go into 8.x, or develop?

On hold pending #1046, and conflict resolution here.

@KatrinIhler Please rebase :)

@gregorydlogan Done! :)


Be consistent on type of pictures in menu to showcase a common theme of food provided

All images are consistent with our designated theme


Password reset with SendGrid ( IEMailSender) not working

Followed the article, and New Registration successfully sends an e-mail. But when attempting Password Reset using a React SPA with Authentication (dotnet new react -au Individual) on 3.0.100-preview7-012821, the IEmailSender interface is never called.
Again, it works for new user registration, I get the e-mail, and break points on SendEmailAsync are reached. But on "Password Reset", the interface methods are never called.
Thoughts?

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

ID: fe1bea60-d173-f6d2-3559-b147699e295a
Version Independent ID: 98e7aacb-2a01-d581-9b03-bfef2893163e
Content: Account confirmation and password recovery in ASP.NET Core
Content Source: aspnetcore/security/authentication/accconfirm.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I you think it's a bug with the VS generated templates, Use the Visual Studio feedback menu item to leave feedback.  Help >  Send Feedback.
Have you compared your code to the Razor Pages version?

I can confirm this was a Preview7 bug. It works in Preview8. Also, in Preview7, after logging in the first time, it didn't take you to the RegisterConfirmation page. It took you back to the Home page, and you were fully logged in.
In Preview8, it follows the docs.

Thanks for the update.


Feature Request: Split package into multiple packages

Many users may only need one of svg, canvas and img renderers, in order to improve performance, is it possible to split this package into:

core
svg-renderer
img-renderer
canvas-renderer

Or:

core
server-renderer
client-renderer

So users can choose according to their needs.
Thanks for the great lib.

I was also thinking it'd be nice to be able to split it based in the barcode types as well. Tree Shaking doesn't seem to be removing the excess parts.

This is a planned feature of JsBarcode 4.0 🙂


Using offset storage in Kafka and not Zookeeper not working

Hey,
We are trying to move from Zookeeper to Kafka offset storage:
kafka.offsets.storage=kafka
kafka.dual.commit.enabled=false

We see though, that two consumers are created, one in Zookeeper, and one in Kafka, but the one that secor uses to commit and read the offset from is the Zookeeper.
Is there any support for having offset storage in Kafka in Secor?
Thanks,
Ron.

you need to set two params, take a look at: https://github.com/pinterest/secor/commit/c3c9b2dc745b6be858e070ff5d8d04f768d11ba9#diff-5bb7557074535d902a0519f1c5faeb23

I feel secor will always use zookeeper unless the code in ZookeeperConnector changes.  You can probably use dual commit to commit the offsets to both __consumer_offsets topic and ZK

We are facing the same issue; it seems that the kafka offset storage is not working and not used; It's only creating a zookeeper one. Which could also be seen here: https://github.com/pinterest/secor/blob/5d2fc90d6725f6abf8cb0acdbfc9f23187d3a131/src/main/java/com/pinterest/secor/uploader/Uploader.java#L114

I don't think secor is going to read from kafka offset storage unless these
code be changed.
But on the other hand, even if you moved offset storage to kafka, the ZK
dependency will still be there (kafka brokers will still need ZK to store
lots of other metadata information).
On Wed, Aug 15, 2018 at 6:48 AM, Sjoerd Mulder<EMAIL_ADDRESS>wrote:

We are facing the same issue; it seems that the kafka offset storage is
not working and not used; It's only creating a zookeeper one. Which could
also be seen here: https://github.com/pinterest/secor/blob/
5d2fc90d6725f6abf8cb0acdbfc9f23187d3a131/src/main/java/com/
pinterest/secor/uploader/Uploader.java#L114
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/447#issuecomment-413202683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCX_9u8YMLxGX6UoYAU5wHvg4ePGD_5ks5uRCaVgaJpZM4Vtqdb
.

I also tried migrating from zookeeper to kafka offsets storage, but got data loss. How can I migrate the offset storage without losing any data?
Secor seems to commit offsets to both kafka & zookeeper now (all partitions), but file uploading seems to be off sync. It could be related to this condition:
https://github.com/pinterest/secor/blob/c457ed4df255d5dcc9ce78ab6bd07f7faec3d08b/src/main/java/com/pinterest/secor/uploader/Uploader.java#L113-L116

Previous conf:
kafka.dual.commit.enabled=false
kafka.offsets.storage=zookeeper

Changed like this and restarted secor:
kafka.dual.commit.enabled=true
kafka.offsets.storage=kafka

I tested this on a group that has 2 secor instances. I edited the conf & restarted one instance at a time. Is that wrong? Another approach would be to stop all instances first, make conf changes on all, and only then start all again.

After the change what we see is:
Offsets on zookeeper:

Topic
Partition
Offset
logSize
Lag
Created
Last Seen

audit.ads

10237750057
10237823723
73666

0
2587551247
2587569392
18145
6 months ago
9 minutes ago

1
2589058112
2589065679
7567
6 months ago
4 minutes ago

2
2529974789
2529999425
24636
6 months ago
11 minutes ago

3
2531165909
2531189227
23318
6 months ago
10 minutes ago

Offsets on kafka:

Topic
Partition
Offset
logSize
Lag
Created
Last Seen

audit.ads

10237750057
10237823799
73742

0
2587551247
2587569416
18169
9 minutes ago
9 minutes ago

1
2589058112
2589065680
7568
4 minutes ago
4 minutes ago

2
2529974789
2529999454
24665
11 minutes ago
11 minutes ago

3
2531165909
2531189249
23340
10 minutes ago
10 minutes ago

They match, as expected.
But now output folder is missing the expected uploads:
s3smoke ls s3://ds-analytics-raw-smoke/secor/audit.ads/processdate=20181031/hour=14/
2018-10-31 16:40:38     685812 1_0_00000000002587527768.gz
2018-10-31 16:09:35     681064 1_1_00000000002588932703.gz
2018-10-31 16:09:23     675959 1_2_00000000002529868976.gz
2018-10-31 16:40:14     735433 1_3_00000000002531140914.gz

It's more than 30 minutes since anything was uploaded to partitions 1 & 2, yet their offsets have been committed recently! What's committing those offsets if nothing is uploaded?
I can see this effect even more clearly with the most recent folders, where the offsets have been recently committed for a partition, but there's no data file in the recent output folder for that partition yet. Possibly there is also data loss (offset gap) between different output folders.

This PR:
https://github.com/pinterest/secor/pull/450#pullrequestreview-170348487 is
going to change the offset storage, you can try that PR to see if that can
fix your problem
On Wed, Oct 31, 2018 at 8:09 AM Juho Autio<EMAIL_ADDRESS>wrote:

I also tried migrating from zookeeper to kafka offsets storage, but got
data loss. How can I migrate the offset storage without losing any data?
Secor seems to commit offsets to both kafka & zookeeper now (all
partitions), but file uploading seems to be off sync. It could be related
to this condition:
https://github.com/pinterest/secor/blob/c457ed4df255d5dcc9ce78ab6bd07f7faec3d08b/src/main/java/com/pinterest/secor/uploader/Uploader.java#L113-L116
Previous conf:
kafka.dual.commit.enabled=falsekafka.offsets.storage=zookeeper
Changed like this and restarted secor:
kafka.dual.commit.enabled=truekafka.offsets.storage=kafka
I tested this on a group that has 2 secor instances. I edited the conf &
restarted one instance at a time. Is that wrong? Another approach would be
to stop all instances first, make conf changes on all, and only then start
all again.
After the change what we see is:
Offsets on zookeeper:
Topic Partition Offset logSize Lag Created Last Seen
audit.ads<PHONE_NUMBER>7<PHONE_NUMBER>3 73666
0<PHONE_NUMBER><PHONE_NUMBER> 18145 6 months ago 9 minutes ago
1<PHONE_NUMBER><PHONE_NUMBER> 7567 6 months ago 4 minutes ago
2<PHONE_NUMBER><PHONE_NUMBER> 24636 6 months ago 11 minutes ago
3<PHONE_NUMBER><PHONE_NUMBER> 23318 6 months ago 10 minutes ago
Offsets on kafka:
Topic Partition Offset logSize Lag Created Last Seen
audit.ads<PHONE_NUMBER>7<PHONE_NUMBER>9 73742
0<PHONE_NUMBER><PHONE_NUMBER> 18169 9 minutes ago 9 minutes ago
1<PHONE_NUMBER><PHONE_NUMBER> 7568 4 minutes ago 4 minutes ago
2<PHONE_NUMBER><PHONE_NUMBER> 24665 11 minutes ago 11 minutes ago
3<PHONE_NUMBER><PHONE_NUMBER> 23340 10 minutes ago 10 minutes ago
They match, as expected.
But now output folder is missing the expected uploads:
s3smoke ls s3://ds-analytics-raw-smoke/secor/audit.ads/processdate=20181031/hour=14/
2018-10-31 16:40:38     685812 1_0_00000000002587527768.gz
2018-10-31 16:09:35     681064 1_1_00000000002588932703.gz
2018-10-31 16:09:23     675959 1_2_00000000002529868976.gz
2018-10-31 16:40:14     735433 1_3_00000000002531140914.gz
It's more than 30 minutes since anything was uploaded to partitions 1 & 2,
yet their offsets have been committed recently! What's committing those
offsets if nothing is uploaded?
I can see this effect even more clearly with the most recent folders,
where the offsets have been recently committed for a partition, but there's
no data file in the recent output folder for that partition yet. Possibly
there is also data loss (offset gap) between different output folders.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/447#issuecomment-434723129,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCX_yJLjiWbX9tVf6DX1YZHxGmv6bwTks5uqb1BgaJpZM4Vtqdb
.

@HenryCaiHaiying thanks, so does that PR support seamlessly migrating from zk to kafka as the offset storage without losing any data? Sorry, I read the comments in the PR but based on them I wouldn't be sure. Does it make secor initially read the offsets from zk but then switch to kafka? How secor know to do that? What is the migration process like – do I need to set some specific conf, should I first make sure that all instances are stopped, etc.? I'm willing to test it as long as I know how :)

is this issue finally resolved?

I haven't tracked this issue, there are several fixes on kafka 2.0.0 profile later.  I think you can give it a try.


fix: bump edge runtime to 1.58.10

What kind of change does this PR introduce?
bump edge runtime to 1.58.10

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 60.026%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.03%

Covered Lines:
6466

Relevant Lines:
10772

💛  - Coveralls


Add side menu links

Description
A side menu with links to other mymizu and sustainability related resources
The repo already includes a basic menu, so please check with mymizu members and add needed links to social media and other links
Design

Please check notion link for more details:
https://mymizu.notion.site/mymizu-Information-Menu-836971c3eeac40ed8bcb9b5b6c63e8c3


Context is None when using permission directives

Hi there! Thanks for putting so much effort into a great set of additional utilities for Django and Strawberry. I've encountered an issue whereby when using the IsAuthenticated permission directive results in the following exception:
  File "/usr/local/lib/python3.9/site-packages/graphql/execution/execute.py", line 625, in await_result
    return_type, field_nodes, info, path, await result
  File "/code/strawberry/strawberry/extensions/directives.py", line 19, in resolve
    result = await await_maybe(_next(root, info, *args, **kwargs))
  File "/code/strawberry-django-plus/strawberry_django_plus/optimizer.py", line 506, in resolve
    ret = _next(root, info, *args, **kwargs)
  File "/code/strawberry-django-plus/strawberry_django_plus/directives.py", line 175, in resolve
    return _next(root, info, *args, **kwargs)
  File "/code/strawberry-django-plus/strawberry_django_plus/permissions.py", line 234, in resolve
    user = cast(UserType, context.request.user)
AttributeError: 'NoneType' object has no attribute 'request'

Breakpointing in to the code shows that context is None, which seems like it should never happen.
I don't believe my setup is anything unusual; I'm running latest versions of strawberry, strawberry-graphql-django, and strawberry-django-plus; I've created a very basic schema; I'm testing using the included GraphiQL interface. Without the permission directive everything works as expected, including using the context elsewhere in my code. However, when I include the permission directive, it fails.
Thanks!

After some digging I think I've found where the issue is; it seems to be related to the use of asynchronous consumers with Django Channels and Strawberry. Thanks anyway!

Oh, nice to know in case I run into something like that in the future!
Other than that, feel free to open more issues if you find anything else!


Ask to support clip operation

Description
Clip operation is not supported. In fact, clip is widely used in some cases, for example, to clip the value to [-5, 5].
Actual behavior
I see many eltwise ops are already supported, but without clip.
Expected behavior
To support clip operation.

Hi @heagoo,
Alas, no plan to support clip primitive for now...
To some extend you can use bounded relu (as long as the lowest bound is 0).
On the other hand this eltwise doesn't really depend on the actual layout (as well as any other eltwise primitives just by design). So as a quick workaround you may just take the data a clipped with a single loop (preserving the initial layout).
// Sorry for this obvious suggestion

Yes, I get it implemented by myself. Thanks!


Cleans up birthday field

Fixes #4006

@aaronschachter

Looks good. please just make sure to both forms to make sure validates and saves ok then good to ship :+1:

Q/A checked per comment above ^
adding value


mutter: 3.36.3 -> 3.36.4

Motivation for this change
Version bump for mutter.
3.36.4
======
* Fix crash on area screenshots with fractional scaling [Sebastian; !1320]
* Do not paint textures of fully obscured windows [Robert; !1326]
* Turn off CRTCs as well when enabling DPMS [Michel; !1240]
* Improve selection support
  [Robert, Carlos, Sebastian; !1330, !1193, !1253, !1255, !1293, !1350]
* Use a more appropriate combine function on opaque areas [Daniel; !1331]
* Fix remote desktop being broken without screencast session [Olivier; #1307]
* Fix popovers disappearing on wayland and HiDPI [Robert; #1312]
* Fixed crashes [Jonas Å.; !1317]
* Plugged memory leaks [Jonas Å.; !1283]
* Misc. bug fixes and cleanups
  [Corentin, Sebastian, Jonas Å., Jonas D.; !1314, !1321, !1295, !1333]

Contributors:
  Jonas Dreßler, Michel Dänzer, Olivier Fourdan, Carlos Garnacho,
  Sebastian Keller, Robert Mader, Corentin Noël, Daniel van Vugt, Jonas Ådahl

Things done

[ ] 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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Thanks.


Fix typos in documentation

Fixes #2138
Contributor Checklist

[X] Did you specify the type of improvement?
[X] Did you add appropriate documentation in docs/src?
[X] Did you request a desired merge strategy?
[X] Did you add text to be included in the Release Notes for this change?

Type of Improvement

documentation

Desired Merge Strategy

Squash

Release Notes
Fix typos in documentation
Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.3.x, [small] API extension: 3.4.x, API modification or big change: 3.5.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you mark as Please Merge?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I've signed the CLA

@Mergifyio backport 3.4.x


if orbital is QEMU based, then it might be possible to make Orbital available for high end android phones (or maybe ios)

Because of that, developers made unofficial QEMU support/app for iOS and Android, and if orbital is QEMU based, then QEMU support/app for iOS and android could be modified for QEMU support/app to run OrbisOS, and it can be available for android and iOS, if it's possible, there will be links for QEMU support/app for Android and iOS.
For iOS: The QEMU support source for iOS is https://github.com/utmapp/QEMU , you can fork it and modify it a little bit for OrbisOS and then you can create an iOS app with the modified QEMU support that emulates OrbisOS, or if preferred to fork an emulator app  that can run QEMU VMs, fork it from https://github.com/utmapp/UTM and you could modify it a little bit for OrbisOS to run in the app, and you can compile the modified source into an ipa app, of course, it requires jailbreak, once the ipa is compiled, the ipa can be converted in to deb using ipa2deb in terminal in jailbroken devices, and you can make a cydia repo for orbital, and you can add the deb in (https;//www,examplerepo,com/debs/Orbital.deb).
For android: The creator hasn't provided QEMU support yet, but you can fork the QEMU emulator app, the app is in https://github.com/limboemu/limbo , you could modify it a bit for OrbisOS to run in Limbo Emulator, then compile the modified source to an apk app.
this might be impossible to make, but this is just an idea for making Orbital available for mobile phones, like android and iOS, it could be available for phones in the future, depending on how hard it is to make Orbital for iOS and android.

QEMU builds for any platform, but the HAXM accelerator that Orbital depends on is x86-only as it's basically a driver for hardware-assisted x86 virtualization present in Intel/AMD CPUs.
You could run QEMU/Orbital without accelerators in iOS/Android, but for it would be unbearably slow for PS4. So not practical at all. On top of that, the upcoming version of Orbital removes QEMU and offers no JIT based approach for non-x86 platforms, so this suggestion will become impossible in the future.

QEMU builds for any platform, but the HAXM accelerator that Orbital depends on is x86-only as it's basically a driver for hardware-assisted x86 virtualization present in Intel/AMD CPUs.
You could run QEMU/Orbital without accelerators in iOS/Android, but for it would be unbearably slow for PS4. So not practical at all. On top of that, the upcoming version of Orbital removes QEMU and offers no JIT based approach for non-x86 platforms, so this suggestion will become impossible in the future.

Yeah, maybe in 30 years who know

Yeah, maybe in 30 years who know

Note: Some phone use the x86_64 architecture and have compatible CPUs
But maintaining a Android build will be too troublesome because only a very few android phone use the x86_64 architecture and instruction set needed to run the PS4 system, and for the size of the Orbital team it's not worth the effort.
Doing android build might also further confuse the community as these build would be only compatible on phone using the x86_64 architecture, and most peoples have arm based phone.
So I agree with the idea of not supporting Android phones, even for the few having a compatible CPU.


Using os.LookupEnv(key) instead of os.GetEnv(key)

When checking if an environment variable is set, we should check if it's set instead of checking if it's empty.

Coverage remained the same at 100.0% when pulling 2977410bfedcaee3a280f43561374c2dd1038302 on bugfix/miss_unset_keys into d128de48baf12a8328975365011a2b3a961ed16a on master.

Coverage remained the same at 100.0% when pulling 2f4cb8b83d4f54917af99eba8d37ea3856dfb378 on bugfix/miss_unset_keys into d128de48baf12a8328975365011a2b3a961ed16a on master.


jspm node support

This PR includes the necessary jspm configuration for this package to work in Node with the coming jspm 0.17 release, allowing this single package to provide both client and server support.
I completely understand if you'd rather not support this here, but it will simplify user configuration locally having it in one place. There is no maintenance burden either as I would continue to maintain this personally.
Just let me know if you have any questions at all, and thanks for considering.

I'm generally supportive of this as it seems like a pretty minor thing to do with no obvious downside, of course that's what I thought about jam support.

Thanks! Yes second-order checking does get somewhat complicated when tools try to second-guess eachother. As mentioned, I'll sort out any (possible) discussions around this override in future though if you just ping me in.

I'm -1 on this. Not to say that my mind couldn't be changed.
Things that we need to figure out first:

Why jspm in the first place? I've read a few things about it, but the value proposition is not clear to me. A cursory glance at the web site suggests a few minor improvements over npm. The ones that stick out are AMD/ES6 module support. AMD support isn't a big deal as other tooling can take care of that. ES6 module support is already handled well with Babel. Some bundler tools like Rollup are doing a nice job of leveraging the benefits of ES6 modules, but I don't see that with jspm. Github dependencies, npm can already do that (albeit a bit flakey), but it still can.

This is a question more for our @crypto-browserify/owners team... if we accept this, where do we draw the line? Do we need to add bower support to then? What about all of the other package managers?

because @guybedford asked nicely and he's the one who who created it
I 100% agree that we should draw some lines and I'd say adding a static property to the package.json would be pretty feasible to do, if somebody wanted us to add a second config file that we had to update in tandem with the main one, or a a property that we had to update each time we updating things that would probably be too much work unless there were some really tangible benefits (this would rule out bower btw).

@calvinmetcalf thanks :)

To be clear with my questioning here, I hope that my tone didn't come off as unnecessary hostile / aggressive. I was trying to be sincere :)

This seems reasonable.

I think the "where do we draw the line" issue is hugely important.  What makes this package manager and this module special?  There are over 20 crypto-browserify modules.  Should we update all of them for jspm?  Should we be testing to make sure they keep working in jspm?  I personally am listed as an owner of over 400 packages, should I update all of them to work with jspm?
What information does jspm need that it couldn't fetch correctly by default from package.json.  I don't have a problem with cases where a module needs special handling, but it seems to me that this module will do the right thing out of the box, providing your package manager supports the "browser" field.

@ForbesLindesay so the we can probably avoid having to answer all of those questions right now becuase @guybedford is making a pretty clear ask  about what he wants.
As for your second question that I'll leave that to @guybedford to explain in detail but it looks like the data being added is similar to the browser field but jspm specific as it has a jspm style namespace

Thanks for the considered discussion here. As mentioned, I take full maintenance responsibility here, but don't expect any further updates will ever be needed unless this package goes through breaking changes in future. I have also waited to send any PR for configs until now since the jspm configuration system has finally stabilized with the upcoming 0.17 release.
@ForbesLindesay in terms of drawing the line - all of the other crypto npm packages work out of the box in jspm 0.17 respecting the browser config etc. The exception here is just that this is one of 22 browserify packages which need to conditionally be replaced with Node core modules in the Node environment when running in jspm to allow JS to run properly between both client and server, and that information is currently not available to jspm (while Node implements it in its resolution system).

Also... looking at your format, it looks like it's remapping this module in node? but this should only apply in the browser and will use core crypto library in node.
right, @guybedford  so browserify handles the mapping instead of node, and basically has a small set of polyfils that it uses when you request a core module, and those are hardcoded internally. what are the advantages of why and how you have done it? (would this just be better as a standard?)
I imagine you have made other similar pull requests and other people are having discussions such as this.
I don't think that you (or anyone) persionally maintaining this is a solution (there are now over 100k npm modules) it would be far better if it was to just work.

weak NACK.
I'm not sure why its justified.

To explain the configuration it just says that when loading this package in a NodeJS environment, instead of loading the main entry point here, load the Node core version of crypto. This allows jspm install crypto=npm:crypto-browserify to provide an import crypto from 'crypto' which will work and build in all environments.
There are 22 of these PRs just for the core Browserify libs, to give jspm full universal JS support. All other configurations do just work in jspm detected from the npm conventions but this is specifically the Node core branching case.
The alternative is to have a wrapper package, but I don't want to introduce an unnecessary dependency, so the effective best alternative does just end up being my running a fork, and hence this PR.

Two points:

All the modules that crypto-browserify require already just defer to the built in crypto module in node.js.  This means that there really is no meaningful performance hit to using this module directly in node instead of the built in crypto.
An infinitely better method than forking the ecosystem would be to just automatically provide the builtins like browserify does.  Why make people do jspm install crypto=npm:crypto-browserify for packages that work out of the box in most other js environments.  This pushes a maintenance burden from the package manager (where it can be maintained in one single place by someone who likely cares and uses the product) onto individual package maintainers (where each one has to maintain it separately, and most probably don't use your package manager).

A quick look at jspm returns the tag line "Registry and format agnostic JavaScript package manager".  There's no such thing as a "format agnostic" package manager; you're just requiring packages to use metadata in your own preferred format, rather than the standardised formats used by node & browserify.  You're also no more "registry agnostic" than npm, which already supports not only the npm registry and github, but also arbitrary tarball urls, git urls and file system paths.

@ForbesLindesay sure let me do my best to answer these questions succinctly. It's great to go though these things, but we should also be careful not to sideline the underlying PR discussion here. If you want to discuss underlying jspm questions in further detail, please feel free to post a question to https://github.com/jspm/jspm-cli/issues anytime and I'm more than happy to have discussions there.

All the modules that crypto-browserify require already just defer to the built in crypto module in node.js. This means that there really is no meaningful performance hit to using this module directly in node instead of the built in crypto.

Yes, that is fine in Node, but we are not building the assumption into jspm that require('crypto') must always resolve to Node core crypto. Instead this package is treated as the main crypto entry point, which can then defer to the Node core crypto itself with this config (which is available at require('@node/crypto')).

An infinitely better method than forking the ecosystem would be to just automatically provide the builtins like browserify does. Why make people do jspm install crypto=npm:crypto-browserify for packages that work out of the box in most other js environments. This pushes a maintenance burden from the package manager (where it can be maintained in one single place by someone who likely cares and uses the product) onto individual package maintainers (where each one has to maintain it separately, and most probably don't use your package manager).

We certainly do not make anyone type jspm install crypto=npm:crypto-browserify. The two ways users will get this is if installing an npm module that uses Node core crypto it will automatically add this package as a dependency, otherwise if users want crypto they would install it as - jspm install crypto, which is effectively a shorthand for the above. This way the browserify libs are installed as needed into projects like any other dependency, instead of being cached as part of jspm itself. This work is exactly about providing the Browserify builtins automatically in jspm.
The format agnostic in jspm refers to module format, it could definitely be worth clarifying that though.

What I'm not understanding, is why this logic should live in these packages, rather than as part of jspm.  I feel that it should be the responsibility of jspm to figure out that require('crypto') in a node environment should point to the core node module and that in a browser environment it should point to crypto-browserify.  Given that crypto is a core module, it doesn't make sense to mutate it to support lots of different package managers, rather the package managers should figure out how to handle core modules for themselves.

That's a good question - so jspm does detect that require('crypto') represents the Node core module, and sets it up as an alias of the npm:crypto-browserify package. The specific restriction here is that we only have the ability to provide conditional branching of code at the package level in SystemJS. That is, I need one package to represent crypto in jspm, which can then provide the necessary branching config within itself (as in this PR) to work in all environments. Having all configurations scoped to packages, with simple package aliasing on top of that forms the conceptual model for configuration in SystemJS and jspm. Global-level branching would require a global branching configuration outside of the package level, but these sorts of global configs are less preferable to package-scoped configuration as one loses the benefits of encapsulation. Just let me know if I'm no longer making sense though.

Is there any chance we might still get consensus for a merge here? I can run a fork no problem if you want to close this too.

I can work around this no problem.


Patch 22

I wasn't really happy with the implementation of FixedTermPolynomial / FixedDegreePolynomial, since it had a lot of indirection and could not use the results it had (since it could be the case that self.terms could not be set at all), so this fixes that by using two new functions, terms_are_valid and try_set_self to check if the terms and valid and set them, otherwise returning a less strict copy.

Hello @philippeitis! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file polynomial/core.py:

Line 530:80: E501 line too long (93 > 79 characters)
Line 670:80: E501 line too long (86 > 79 characters)
Line 673:80: E501 line too long (92 > 79 characters)
Line 694:80: E501 line too long (86 > 79 characters)
Line 697:80: E501 line too long (90 > 79 characters)


Kubeflow v1.2 Rebasing

This is to keep track of the steps for the rebasing, and the conflicts that arise.

Steps taken for stc-master-v1.2-merge branch:
Merge (keeps our commits and history, adds upstream changes as one big merge commit with one author aka me)

Clone whole repository (in this case, StatCan/kubeflow)
Go into repo locally
git remote add upstream https://github.com/kubeflow/kubeflow.git
git remote –v (list current configured remote)
git checkout stc-master
git checkout –b
git fetch upstream
git merge upstream/v1.2-branch
Resolve conflicts -> save -> stage changes
git commit
git push --set-upstream origin

If you want to rebase, brings in all commits from upstream, but loses our commit history (adds all commits on one date and adds me as author to everything), change merge in step 8 to rebase. Branch: https://github.com/StatCan/kubeflow/tree/stc-master-v1.2-merge

After discussing with the team, using git rebase is a better option since it will preserve both the kubflow upstream commit history as well as ours.

Branch https://github.com/StatCan/kubeflow/tree/v1.2-branch has been rebased to the v1.2-branch upstream.
Conflicts that came up while rebasing, which we tried to resolve but are going to take another look:
Central Dashboard now as configmap
Upstream commit: https://github.com/kubeflow/kubeflow/commit/ec7e01a28bf5339f784fdfddcb7bb0bcb46ce352#diff-a29b81659d6ca88b1df24f77d04886f8eb04631f08a34e41da53e6bb273e3be0
Will have to go back and integrate these changes with our links, documentation, and restructure.
i18n (en/fr)
Our commit: https://github.com/StatCan/kubeflow/commit/f85a597e5040ab518b44da81b7582d5c0252f492
Upstream has no translations at all, so we have to fit this in with their new configmap.
Security Update
Our commit: https://github.com/StatCan/kubeflow/commit/30116175161aace30ffc3b8007d194368ef9ae40#diff-a29b81659d6ca88b1df24f77d04886f8eb04631f08a34e41da53e6bb273e3be0
We changed for security vulnerabilities:
coreAPI: k8s.Core_v1Api; to k8s.CoreV1Api;
customObjectsAPI: k8s.Custom_objectsApi; to k8s.CustomObjectsApi;
Upstream has not updated this yet.

We need to find a way to test central dashboard. Since the links are now coming from configmap, and it's an API call '/dashboard-links' the local npm run dev now shows an almost empty page.
There are 4 errors like these:

Failed to load resource: the server responded with a status of 504 (Gateway Timeout)

We ran into an issue where there was an error retrieving the pipelines. Zach fixed the error in this commit: https://github.com/StatCan/kubeflow/commit/5a07fb421e54a110374c8df9aebc9a1d43e188b8 and Wendy fixed the pipeline tests here: https://github.com/StatCan/kubeflow/commit/77c965c7b1999756ae224909666529bd291a5da5. Not sure if this is a problem upstream but they didn't seem to have any visible fix for it.

We ran into the issue of the configmap. The menu links, quick links and documentation links now come from a centraldashboard-links config map which we were not able to customize to our docs directly from changing the file. It seems like a different resource when looking at the Network tab. The resource it uses is viewable with this command : kubectl -n kubeflow get configmap centraldashboard-links-config This will need to be addressed. For now, we removed the code that was using it and kept the arrays that were previously used.


New version: xpipe-io.xpipe version 9.0

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @crschnick,

This PR's version number 9.0 has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 22.3.3000.64, which has 4 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 877)


Quick question (DNS)

Hi, This looks amazing and I'm hoping I can use it.
I just had one question and that was, is it required to use Linode's DNS with this? My team uses Route53 for our DNS management currently and I can't switch that over to Linode.

Thanks for the kind words! Route53 works too, just point the A record (example.com) to your master IP, and create a wildcard CNAME domain (*.example.com) to your master IP as well :)

Perfect, thank you.
This should make my life a bit easier going forward ;)


Update vnet_service_endpoint_overview_armaha

@aravindmahadevan-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@aravindmahadevan-ms  Suggest rewriting like this "To define Vnet firewall rules on a logical server that is already configured with private endpoints, please set Deny public network access to No"

#assign:VanMSFT

Hey @aravindmahadevan-ms - Can you please change the edit according to my message above? You can copy and paste the text I provided above to replace your edits.
Go to the Files Changed tab, and then hit the ... button to edit file.

Hey @aravindmahadevan-ms - ping

Thanks @aravindmahadevan-ms!
#sign-off

@VanMSFT: I'm sorry - only the author of this article, @rohitnayakmsft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver - Hey Kristine, can you help merge this, please? Thanks!


Fix CVE 2021 4235: Update sigs.k8s.io/kustomize/kyaml to v0.13.10

This PR updates an indirect dependecy (kyaml) to address a CVE that can potentially cause panics due to an internal fork of go-yaml inside kyaml. The updated version of kyaml  fixes the CVE.
CVE: https://pkg.go.dev/vuln/GO-2021-0061

The latest release of kustomize/api (another indirect dependency) is pegged to an outdated release of kustomize/kyaml https://github.com/kubernetes-sigs/kustomize/blob/master/api/go.mod#L14 so forcing a version skew seems to be causing compile time failures as one of the function signatures between 0.13.9 and 0.13.10 has changed.


Drop Rails 4 support

@jeremyf I'm reducing the amount of variation we need to support.  Rails 5 has been out for 6 months now, so there's been plenty of time to shake out the bugs.  As Hyrax is a new product, I'd like to take the opportunity to start fresh.

@jcoyne That's good by me!

I'll add another reason: to strip out our janky polling solution for notifications and replace it with ActionCable.


Possible bug or I'm confused.

Edit: NVM, you can close this....
it turns out it was my fault; if you use syntax
for (i in arr){
collection.push({id: i, ...})
console.log(i, typeof i)
}
you'll see JS uses a string type for index, and that breaks the collections update/find Fn's
sorry.....
Hi I'm a little confused on Collection.set and Collection.push, can you tell me if this is a bug?
Anyway If you use collection.push({id: 9999, otherfield: 1234})
collection.set({id: anyid, ...}) no longer works... I assume this is because either you cannot set the ID from push, you cannot start above 0, or you cant use abritrary IDs from the backend ^_^.

What is the correct way to add an item to collection, .push()? Is it without specifying an id?

I am using push({ id: 9999, otherfield: 1234, otherfield2: 1234 }), and i'm using Ids > 0 as my first item, I think this is causing the problem

What is the correct way to update one field of one item to that collection?

If I do collection.set({ id: 1, field2: 'newdata'}), sometimes the entire object is replaced with this new smaller object (now only has these 2 fields) but usually nothing happens and .get(id) returns the old values

Maybe you can update the docs to be a little more clear? This is a great lib

one more question for ya...
how do i do render something like this...
render: func() { return {
    mylistdiv: { view: this.list, onclick: this.clickedlist }

view isnt the right attr name there im not sure what it is? of course "mylistdiv: this.list" renders fine...

just wrap the list in another element:

render: function() {
return {
wrapper: this.clickedlist
mylistdiv: this.ist
}

On 14 Jan 2015, at 05:07, falafflepotatoe<EMAIL_ADDRESS>wrote:
one more question for ya...
how do i do render something like this...
render: func() { return {
mylistdiv: { view: this.list, onclick: this.clickedlist }
view isnt the right attr name there im not sure what it is? of course "mylistdiv: this.list" renders fine...
—
Reply to this email directly or view it on GitHub https://github.com/techlayer/espresso.js/issues/8#issuecomment-69869865.


Ability to add local storage account to Quick Access

Storage Explorer Version: 1.2.0
Steps to Reproduce:

Navigate to (Local and Attached) > Storage Accounts > Development
Right-click on Development

Expected Experience:
It would be useful to be able to add the development storage account to Quick Access, to avoid having to navigate several levels deep.
Actual Experience:
No option to add to Quick Access. (Conversely, it is possible to add a local Cosmos DB database.)

Thanks for the feature request @davidgruar ! We'll take a look into making this possible with an upcoming refresh of our Local and Attached feature areas.


Fixed buffer concatentation

Fixing the Buffer concatenation to avoid character encoding issues, as per #6

It's not a case of a performance impact - the functionality before wasn't working, and now it is.

Coverage remained the same when pulling 3c274d0cceee8808a26d958a2b714ba06ee30dbf on abstractvector:master into af096e482542a4b9a26a4a64ee36cc6ecfbcb086 on derekcollison:master.

Did you check why the tests failed?

Looks like the tests got broken, so we need to fix that..

There is a utf8 test which works correctly as is (at least under 0.10 & 12).. Does not need Buffer.concat.


⚠ Samsung has degraded performance

In 887274a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2932 ms

Resolved: Samsung performance has improved in 71dd9b5.


Fix Select-AzureRmSubscription incorrect parameter

Fixed the incorrect parameter syntax for Select-AzureRmSubscription alias of Set-AzureRmContext reported on #682.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azurermps-6.6.0/manage-subscriptions-azureps.md
:white_check_mark:Succeeded
View (azurermps-6.6.0)

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

@kiazhi This change will not be picked up in future documentation unless it's also submitted to the engineering project (where docs are generated from). Please make a second pull request against https://github.com/azure/azure-powershell

Thank you for your contribution!


Integrating riscv-tests with the current test flow

This allows user to write directed assembly test with riscv-tests framework. It can be used a supplement approach to cover the corner cases that are not covered by random instruction generator.

We could further consider integrating RISC-V compliance test to the flow, so that user doesn't need to switch between 3 different environments in order to run all these tests.

I have changed the scope to support hand-coded assembly tests first. Will use other bug to track riscv-tests and compliance test integration if we decide to do it.


Linux Support

I'm a Linux guy mainly and would love to try this revolutionary piece of software. I would be glad to test if I can.

Hello!
Glad to hear you're interested in UltraEd. Linux support is sadly pretty far away right now. I would like to not have to rely on windows but a big change I'd have to do is replace the editor's renderer with OpenGL. I plan on doing that conversion someday and then the toolchain I'd have to recompile for a different target. Those I think are the only big changes that need to happen in order to have UltraEd start running on different operating systems. I guess you could try using Wine to run UltraEd but you'd still have to compile it first which right now it's tightly configured to use visual studio. Getting off the dependency to use windows would be great! Maybe I need to start thinking more about that since I have no interest currently to upgrade to win 11 lol.


How can I fetch all the unseen messages in a group?

I am trying to fetch all the unseen message from a Facebook group. Is there any way to do that?

You could probably use Client.fetchThreadMessages in a loop, and then look at the returning messages' Message.is_read, to figure out when to stop the loop. But no, there's no direct way

it would be really useful to have a "getUnread" method in the Thread class, I also need to get the unread messages, the Client.fetchThreadMessages has a before parameter that is a timestamp, this doesn't sounds a good way to loop to get the unread messages, currently I am setting limit=1 and iterating while the message is unread, because I don't want to get all messages and then discard the read messages, but this "before" parameter doesn't seems to be as good as an offset, maybe I will lost messages becasue they have the same timestamp and I am getting one message at the time and then setting "before" to the message's timestamp

actually what I tried of using the timestamp of the latest message I fetched with Client.fetchThreadMessages as the before parameter doesn't works, it  always return the same message

Hmm, if the before parameter doesn't work, then that's an issue in itself (which you may open if you feel like it)! But the reason it exist is not because we want it to, it's because that's what Facebook gives us to work with 😉

Closing, since this probably can't be fixed. Feel free to reopen if you find anything useful!


Add anonymised Pirmasens district heating stack instructions to TWA

Moved instructions to spin up district heating stack from private Pirmasens repo to TWA. Replaced references to confidential data with placeholders/examples and removed unnecessary agents

The stack name is an environment variable that should be visible in each container started up by the stack manager, I can modify my containers in a separate PR and then only merge this in once that is done unless this is urgent

I believe this entire discussion is unnecessary work creation. My point of view:

If we actually want to refine the stack name settings, we should do it in the Pirmasens repository and not here
The purpose of this PR is to make relevant code, required to inspect/reproduce the work of the paper, publically available.

Hence, I believe a hard coded stack name does not do any harm here

I believe this entire discussion is unnecessary work creation. My point of view:

Well, now that this can of worms is wide open, I would have to say that I strongly disagree:

If we actually want to refine the stack name settings, we should do it in the Pirmasens repository and not here
The purpose of this PR is to make relevant code, required to inspect/reproduce the work of the paper, publically available.

No, the purpose of this PR is not to dump some (bad) code for a paper, but to move the code where it should be. The pirmasens repository shouldn't even exist. Any future development should be done within TWA repository.

Hence, I believe a hard coded stack name does not do any harm here

This practice is appalling, sets a very bad example, and people will copy/imitate it. I would rather contain this cancer before it spreads.
Having said all that, I do recognise the need to get on with it, which is why I approved the PR with my initial comment. I have added issue #1093 for the record.


Fix job service test (resolves #185)

Please name your branches
issue-123-foo-bar
and mention the issue in the PR title. See the title of this one.

Oops, will do.
On Fri, Aug 14, 2015 at 5:42 PM, Hannes Schmidt<EMAIL_ADDRESS>wrote:

Please name your branches
issue-123-foo-bar
and mention the issue in the PR title. See the title of this one.
—
Reply to this email directly or view it on GitHub
https://github.com/BD2KGenomics/toil/pull/237#issuecomment-131273981.


False error reportings

This is an error that has been happening for a while whenever i write code then deleted it errors show up like the code is still there but it's deleted whenever close and reopen vscode it fixes it but that's really annoying to do often for ex:

this is a duplicate of https://github.com/OmniSharp/omnisharp-vscode/issues/4192#issuecomment-725598099
I will close this because it is not a problem of Omnisharp server, but of the VS Code extension


MNT: Re-rendered with conda-build 3.21.4, conda-smithy 3.10.0, and co…

…nda-forge-pinning 2<IP_ADDRESS>.54.41

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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.


Make ArRegion smarter at discovering the region

Story:
A bug was introduced (fixed: #6239) because the migrations depended upon the presence of a REGION file. Many developers do not have this file on their machine. I have also had issues with the dependence of a REGION file and the incorrect determination of the region when the file was not present.
I wanted the application to be smarter when determining the region. So my environment would not depend upon the REGION file and have me constantly updating it.
In the long term, it would be great if this file were not needed in development or production, but this PR does not have such lofty aspirations.
Before:

In production, users need to backup and restore a REGION file.
In development, users need to change the REGION file when ever switching their database.

After:

In production, the REGION file is optional. But it will be present since all the infrastructure currently creates one.
In development, the local REGION file is optional. When it is missing, it will properly determine the region rather than assume it is 0.

What this PR addresses:

This makes the REGION file optional.
It will mostly be leveraged in development, thought it could be leveraged in production if we want to go the route of removing that file.

How does this work:
The database has the region number embedded in the sequence of every table. This is necessary because every time a record is created, it needs to be assigned an id for the correct region.
If the REGION file is not present, it looks at the sequence of the very first table created (miq_databases) and uses that to determine the correct region. If that table does not exist, then it falls back to region 0 as it always has.
This also adds the provision of reading the region number from an environment variable REGION.
/cc @jrafanie @carbonin This is what I had in mind. think this may help with #6239
/cc @Fryguy Easier for me to just throw code together than explain
/cc @matthewd FYI

@fryguy

This will not work if there is no data in the table. Like I said in other conversations, it's better to look at the miq_databases id_seq and look at the minimum value.
The only time this value won't work is on initial migration, where something like an ENV var might be useful. We have those change in the ActiveRecord patches, so we may be able to be selective there about the ENV var, and not worry about it here at all.

I just heard you say put reading the sequence as a higher priority than the file

I'm really liking this idea.  Looking forward to seeing this play out.

@Fryguy I like it too ;)
But @jrafanie has a concern around detecting an incorrect region number and then running primordial seeds.
Personally, I feel more comfortable not having a REGION file that is possibly wrong, but that is my preference. His concern is very real as we do not want to do this wrong.
git checkout master
echo "2" > REGION
rails c
irb(main):001:0> MiqDatabase.first.id
=> 1
irb(main):002:0> MiqDatabase.my_region_number
=> 2
irb(main):004:0> MiqDatabase.seed # no op
irb(main):006:0> MiqRegion.seed
Exception: Region [2] does not match the database's region [0]

YAY.
Question from @jrafanie what happens if we don't have this check? What happens if we are always reading the region from the database? When will this break down?

This will not work if there is no data in the table. Like I said in other conversations, it's better to look at the miq_databases id_seq and look at the minimum value

@Fryguy looks like rails only sets the next value and leaves the minimum value at 1. So if you look at minimum, it always returns a region of 0. It is now looking at the sequence, but specifically at the next value.

Please test this with replication, as that is the main consumer of it.  cc @carbonin Please see if this affects any of the work you are trying to do.

@jrafanie think this hits your most pressing concerns:
bundle exec rake db:drop db:create
REGION=12 bundle exec rake db:migrate VERSION=20131021190044
bundle exec rake db:migrate
bundle exec rake db:seed
bundle exec rails r "puts MiqDatabase.my_region_number"
# => 12

bundle exec rails c
puts MiqDatabase.my_region_number
# => 12
puts MiqRegion.count
# => 1
puts MiqRegion.seed.region_id
# => 12
puts MiqRegion.count
# => 1

More scenarios to test:

developer adding a new migration that creates a new table or does a data migration (i.e. not rake from 0, but starting from an intermediate position)
appliance_console does the right thing when creating a new database
appliance_console joins a region.
switching from internal DB to external DB from the application
Global region properly sees its own records and not remote records (@gtanzillo, need your input here)

@jrafanie Can you think of other places where REGION is important?

@kbrock In your tests there, aside from the counts can you also verify the id numbers are in the right range?

@jrafanie Can you think of other places where REGION is important?

Considering replication is currently being investigated for rewrite using bdr/udr, I'm not sure what semantics REGION will have so I have no idea.  I'd rather have a better understanding what regions will become when replication's future is less :cloud:y.

Thanks @Fryguy

adding a new migration that creates a new table or does a data migration (i.e. not rake from 0, but starting from an intermediate position)

commented code above to make this more clear.
The database is created but I specified VERSION to put into an intermediate stage
When running the rest of the migrations, it creates the ImportFileUpload and verified that the id is correct.

appliance_console does the right thing when creating a new database

Added #6299 to remove the region discovery knowledge from appliance_console. so this should be all set. but yes, do need to test.

appliance_console joins a region.
switching from internal DB to external DB from the application

What ever database is referenced by database.yml will respond with the

Global region properly sees its own records and not remote records (@gtanzillo, need your input here)

YES! THIS needs to be tested
Ensuring that after replicating data, the sequences are still the same and the values come back correctly.

In your tests there, aside from the counts can you also verify the id numbers are in the right range?

I hope the changes to the comments above make this more clear.
I was not able to actually set a region in specs because this requires modifies the sequences. (All the current region tests don't really test this properly and probably can't)

This looks like it should be fine for working with what I have now. Currently I'm sticking as close to the current setup regarding regions as possible. So the user would select a region id when joining to a new replication group in the appliance console. I then manually reset the sequence values to the start of the given region number range before seeding the database like this:
    def self.reset_region_sequence_values(region_id)
      conn = PG.connect(:user => "root", :dbname => "vmdb_production")
      region_value = region_id.to_i * 1_000_000_000_000
      schema_seq = conn.exec("select sequence_name from information_schema.sequences where sequence_schema = 'public'").column_values(0)

      schema_seq.each do |s|
        conn.exec("select setval('#{s}', #{region_value})")
      end
    ensure
      conn.close
    end

As for creating the initial bdr group, I am currently reading from the region file, but only for the node name, so that can easily be changed.

Global region properly sees its own records and not remote records (@gtanzillo, need your input here)

YES! THIS needs to be tested
Ensuring that after replicating data, the sequences are still the same and the values come back correctly.
Then again, if the sequences are broken (I didn't touch them), then we have issues outside this code.

Yeah, that's another thing, but not what I was referring to.  @gtanzillo might be able to describe in more detail, but there are certain tables that scope on the region number so as to only show it's own region and not ALL of the regions combined.

@Fryguy Not any more https://github.com/ManageIQ/manageiq/pull/6206

@carbonin I think you have to take into account the current values in the table for setting the sequence numbers. I'm hoping either the database is empty or you posted pseudo code there.

Yeah, that's another thing, but not what I was referring to. @gtanzillo might be able to describe in more detail, but there are certain tables that scope on the region number so as to only show it's own region and not ALL of the regions combined.

The region number is discovered from the sequence in miq_databases. If that region number is used for scoping or not, it works just like it does today.

I'm moving a question from @jrafanie over from #4299

What is [removing the REGION file] buying us again besides developers having multiple regional databases that they want to swap in and out effortlessly?

I agree that this existing file is a pain but there are little intricacies that are hard to detect especially the detail above RE: missing miq_databases table. Also, many of the REGION changes are not easily unit tested so we have less protection. We have to weigh the pros/cons of each of these changes.

Regardless of what we put into the REGION file, records get created in the database at the ids determined from the sequence. Said another way, the region of the database is determined from the sequence. The desire is to get rid of the REGION file since it is not the source of this information.

@kbrock Too far off topic for explaining here, but the gist is that there are no rows for the new region when I set the sequence. The point is that if everything stays the same in terms of using region numbers in general, this won't be disruptive to my work with multi-master replication.

What is [removing the REGION file] buying us again besides developers having multiple regional databases that they want to swap in and out effortlessly?

Regardless of what we put into the REGION file, records get created in the database at the ids determined from the sequence. Said another way, the region of the database is determined from the sequence. The desire is to get rid of the REGION file since it is not the source of this information.

It was originally.  The database sequence was set by it.
I'm not opposed to all of this, I just don't see much gain for possibly destabilizing infrequently modified code.  It's hard to get my head into the several scenarios this area of code touches and most of these scenarios require complex setups.
I'm trying to see the benefit and I see mostly risk.

I'm sorry, my intent was not clear. I rewrite the description.
I'm not trying to eradicate the REGION file at this time. I just want it to be optional.
This change only takes effect when the REGION file is not present. So it should only effect developers.

@Fryguy @matthewd @gtanzillo @carbonin @jrafanie Please review


Translate network/validate-dual-stack.md in Korean

This is a Feature Request

What would you like to be added

Translate tasks/network/validate-dual-stack.md in Korean
Why is this needed
There is no document with tasks/network/validate-dual-stack.md in Korean.
Comments

https://kubernetes.io/docs/tasks/network/validate-dual-stack/

/language ko
/assign


[BUG] Markmap from cdn error

Looks like there is a new issue with markmap when loading the library from the cdn. Some kind of function error with t.buildJSItem and "Transformer is not a constructor".
Not sure what the issue is but you can see the error with this code using the codepen link:
Codepen of Markmap example
Using chrome to experiment with markmap.

Fixed in<EMAIL_ADDRESS>


The generated getter and setter checkboxes are very inconvenient to use on the left and hope to improve.

Issue Type: Bug
The generated getter and setter checkboxes are very inconvenient to use on the left and hope to improve.

VS Code version: Code 1.59.0 (379476f0e13988d90fab105c5c19e7abc8b1dea8, 2021-08-04T23:13:12.822Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz (4 x 2494)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
9.59GB (3.09GB free)

Process Argv

Screen Reader
no

VM
67%

Granted I come from a LTR world but I've always seen checkboxes on the left side.
At best, I think we could offer a configuration for having the checkboxes on the right side... but I'd only do that work if it's heavily requested because I personally don't think it provides a ton of value. I'll give this feature-request and we'll see how others feel about it.


How to use reactive custom validation

I've read the docs over and over and I just can't seem to get this working.  If anyone can point me in the right direction I'd appreciate it.  Once I understand what I'm missing I'd be happy to submit a PR for any docs improvement in this area.
http://stackoverflow.com/questions/32302359/meteor-autoform-custom-validation-not-reactive

I've figured it out - answer posted on the SO page.  Thanks for this fantastic package by the way.


Contact Us section is not responsive

Issue 1:
Submit button on the contact us form doesn't resize correctly on smaller size screen.
Issue 2:
The top margin of the contact section not appears intractive.

Could you please assign me this project.

i want to work with this issue.

The new UI looks completely different from the earlier ui


GET http://localhost:8081/images/<icon_name>.svg 404 (Not Found)

It tries to find in a local project's folder called images, instead of searching it in the folder of this component.
https://github.com/vpusher/game-card/blob/eae8568912ed85baa86e577fc5f329fbeeee43a5/game-card.html#L368
The icons are successfully loaded, but it first throw multiple exceptions like this.

@malkomich are you serving the files using polymer serve ? or a custom web server ?

I am using the component in a Vuejs + Webpack application.
I'm just using it like this:
<game-card :symbol="card.symbol" :rank="card.rank" :flippable="card.flippable" :unrevealed="!card.revealed"
              v-on:click="playRound(card)"></game-card>

in a loop with a preloaded array of cards:
export default class GameCard {

  static CLUBS = '♣';
  static DIAMONDS = '♦';
  static HEARTS = '♥';
  static SPADES = '♠';

  static RANKS = ['2', '3', '4', '5', '6', '7', '8', '9', '10', 'j', 'q', 'k', 'a'];

  constructor(symbol, rank, revealed = true, flippable = true) {
    this.symbol = symbol;
    this.rank = rank;
    this.revealed = revealed;
    this.flippable = flippable;
  }

  toggleFlippable(flippable) {
    this.flippable = flippable;
  }

  get key() {
    return ''.concat(this.rank).concat(this.symbol);
  }
}


Pass multiple directories to find logs

Is there a way that specify multiple directories that contain logs, i.e. I want to monitor all logs in docker containers as well as /var/log folders. Thank you

The harvester's env LOGS_DIRECTORIES accepts an arbitrary number of space separated " " directories.
Example:
$ docker run -d \
    -v /var/lib/docker/containers:/var/lib/docker/containers \
  -e "LOGS_DIRECTORIES=/var/lib/docker/containers /var/log" \
  --link logio:logio \
    -e "LOG_FILE_PATTERN=*-json.log" \
  --name harvester \
  --user root \
  quay.io/blacklabelops/logio harvester

For some reason when I pass space separated list of folders it only uses first in the list.
And is it possible to pass name of stream along with the list of files associated with that name, or when folders are searched for files - can we pull container name or host name as well or does it put everything into UnifiedStream only?

Looks like a bug, I will check the container.
This Image is only file based. It crawls for files, it does not take Docker Container Ids into account. If you need such an elaborate mechanism then please take a look at the native docker log mechanism.
Inside Docker you can set the Log setting for each Container separately, please see here: https://docs.docker.com/engine/admin/logging/overview/. The log technology must be available on the Docker Host, e.g. fluentd and you can specify logging for each container separately.

Okay fixed the bug. Multiple directories are working!

You can define one streamname per harvester. You could start one harvester with a different streamname for each directory or logfile pattern.
If you want to attach a specific container, you need to attach the specific logfile inside the container log directory. See the container id and you have the foldername of the logfile.
Also I could implement an environment variable so you can pass your own customized harvester config file. Not sure if that would help you.
The perfect solution would be a logio logging driver inside Docker, but such a thing is currently not available.

Awesome, thank you.
Would you take it one step further and give it an optional FILES format to provide stream name as an optional part of file pointer, ie:
Old format:
LOGIO_HARVESTER_LOGFILES=/var/log/cron /var/log/messages /var/log/nginx/site1-access.log /var/log/nginx/site2-access.log
Proposed change:
Add streamname followed by colon, preceding the path to file:
LOGIO_HARVESTER_LOGFILES=system:/var/log/cron system:/var/log/messages web:/var/log/nginx/site1-access.log web:/var/log/nginx/site2-access.log
Alternative failover support (if no streanmane specifies on a per-file basis, it could be specified on a per-directory basis) :
LOGS_DIRECTORIES=DockerAll:/var/lib/docker/containers SystemAll:/var/log
and further it would failover to the value specified in
LOGIO_HARVESTER_STREAMNAME
This would keep backwards compatibility but would allow to specify stream name to individual files or to directories

And just an FYI - here's another approach to pulling logs from all containers, but it provides a little more data about the container and the image itself, hence opens up the possibility to include that information as part of stream name as well.
Another alternative is to have a setting per-pattern and per-directory that indicates to grab a "level-up (..)" directory name as stream name (could be useful only to us, but seems like it would make this script mostly complete for all use-cases), ie:
LOG_FILE_PATTERN=streamname_from_parent_dir:*.log

Very nice! I like it, please give me some time to implement it.
I could offer a fast solution that I already used inside my other containers, enumerated env variables.
Example:
LOGIO_HARVESTER_LOGSTREAMS1=/var/log/cron /var/log/messages /var/log/dmesg
LOGIO_HARVESTER_STREAMNAME1=system
LOGIO_HARVESTER_LOGSTREAMS2=var/log/nginx/.log /var/log/php-fpm/.log
LOGIO_HARVESTER_STREAMNAME2= webaccess
Could you use this? I could implement this by simply adding a for-loop around my existing logic.

Sounds cool..  most certainly a cool work around over the docker-compose's yaml syntax and would work nicely for me, but a long-term and proper solutions seems like would be to support both ways - for docker syntax to use incremental enums and support for docer-compose too.
thank you so much for your time.

Would love to see a solution to this issue, did you every get any further on the implementation for enumerated env variables?

Implemented half-way.

Finished!


ListView's onItemTap doesn't work as expected

See kitchen sink https://studio.ensembleui.com/app/e24402cb-75e2-404c-866c-29e6c3dd7992/screen/w0Wmu9ZMP4csk7IELSx3#
onItemTap should work by clicking anywhere on the tile, but it only works if you select on the title/subtitle/photo

Fixed it in this PR - https://github.com/EnsembleUI/ensemble/pull/684


Missing Athena JDBC Drivers Cause Failed Spark Cluster Create

Hey guys, the linked S3 JDBC driver (AthenaJDBS41-1.0.0) is no longer available at the path in the SparkCluster CloudFormation template (specifically the one referenced in step #4 of the EMR Cluster Bootstrap). This causes the out of the box CloudFormation Template to fail on the EMR cluster creation step.
It looks like the following are available:
2017-03-24 17:21:27   10901192 AthenaJDBC41-1.0.1.jar
2017-05-16 18:34:47   11074334 AthenaJDBC41-1.1.0.jar

Hi Jacob, thanks for finding and reporting this!
I just committed the fix - the template now installs AthenaJDBC41-1.1.0.jar.
Bob


fix(): handle formatter with metatable

With this fix, a table with a __call metatable event can be used in
place of a formatter config function.

How would this work in practice? I originally went with a function so arguments could be determined when the formatter gets called, vs startup.

@mhartington this would work the same as before, but you could also add a metatable instead of a function which is also callable and would determine the arguments on call vs on startup. I personally think this is a good addition because metatables can hold state while functions cannot and they're both callable.
Now a question to @loqusion, do you have a good example of usage with metatables? It would be nice to add to the readme and vim help. I would suggest merging master into this branch first and updating the readme and vim help with a good example with metatables. If you don't have a good example just mention that formatters can be added as metatables in readme and vim help.


ENH: Support CustomBusinessHour

[x] closes #11514
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

lgtm. pls rebase

Rebased, and now green.


ECS Service desiredCount changes not triggering new deployment

What happened?
After creating an ECS Service, I am unable to change the initially set desiredCount in the provisioned resource.
It appears changes to desiredCount field do not trigger a new deployment or alter the existing deployment.
Initial Service:
apiVersion: ecs.aws.crossplane.io/v1alpha1
kind: Service
metadata:
  labels:
    testing.upbound.io/example-name: sample-app-service
  name: sample-app-service
spec:
  forProvider:
    desiredCount: 1
    deploymentController:
      type_: ECS
    deploymentConfiguration:
      maximumPercent: 200
      minimumHealthyPercent: 0
    launchType: FARGATE
    region: us-east-1
    schedulingStrategy: REPLICA
    platformVersion: 1.4.0
    clusterSelector:
      matchLabels:
        testing.upbound.io/example-name: sample-app-cluster
    loadBalancers:
    - containerName: sample-app-container
      containerPort: 80
      targetGroupARNSelector:
        matchLabels:
          testing.upbound.io/example-name: sample-app-target-group
    networkConfiguration:
      awsvpcConfiguration:
        assignPublicIP: ENABLED
        securityGroupSelector:
          matchLabels:
            testing.upbound.io/example-name: sg-us-east-1
        subnetSelector:
          matchLabels:
            all-subnets: included
    taskDefinitionSelector:
      matchLabels:
        testing.upbound.io/example-name: sample-app-task

Once this is up and running, I change desiredCount to 2 and apply.
I can see the change reflected in the object's Spec section on the cluster:

But the object's Status section shows that no new deployment has been rolled out and displays the old deployment's config:

The object's events section clearly shows only 1 deployment has ever been attempted:

At this point, to get another deployment, I have to delete the service and create a new one...
How can I redeploy a service with a new desiredCount value without deleting and re-creating it?
In the console, there is a "force new deployment" option that allows this when you hit "Update Service" :

Is this functionality available in Crossplane?
How can we reproduce it?
You can apply all yaml in this repo to lay down a network, ALB, and ECS Fargate cluster, task, service, and role.
https://github.com/martinflemingdev/crossplane-ecs-service-bug
When you change desiredCount to 2 and apply, you will see no new deployment occuring.
What environment did it happen in?
Crossplane version: v1.11.0
Kubectl client version: v1.25.0
Kubectl server version: v1.25.3
OS: Pop!_OS 22.04 LTS
KinD: 0.17.0

I think it comes down to the controller not providing the isUptoDate method. https://github.com/crossplane-contrib/provider-aws/blob/8f53f05ae2c6e355a8a2145f0a3c96452487bf32/pkg/controller/ecs/service/zz_controller.go#L92
So it needs to:

Implement the method. Likely will need some sort of way to convert one object to another and then diff.
Depends on changes, you need to re-deploy with --force-new-deployment.


error

this not work, can't copy full folder

this not work, can't copy full folder

Hi @anhvietnx,
Thank you for bringing this to my attention. I'm sorry to hear that you're having trouble with the folder copying script. Interestingly, I and many of my friends have been using it without any issues, so I'm keen to understand what might be going wrong in your case. To help me troubleshoot, could you please provide some additional information?

Permissions Check: Do you have the necessary read permissions for the source folder and write permissions for the destination folder?
Partial Copy or No Copy: Were some of the files copied over, or did the script not initiate the copying process at all?
Error Messages: Did you receive any error messages or notifications that might give us a clue about what went wrong?
Script Termination: At what point did the script stop working? Was it during the initial setup, while copying, or at a different stage?

Any details or screenshots you can provide will be incredibly helpful in resolving this issue. I appreciate your cooperation and look forward to getting this fixed for you!


Add median function for aggregates

We can choose whether to average out or ignore outlier host metrics.

@fujiwara What do you think about this?

@cohalz Sorry, I've missed this. Looks great! Thanks!

v0.3.6 released. Thanks!


keep_attrs=True does not work apply_ufunc with xr.Variable

Code Sample, a copy-pastable example if possible
keep_attrs=True works nice for xr.DataArray, but it is neglected for xr.Variable
In [2]: import numpy as np
In [3]: import xarray as xr
In [4]: da = xr.DataArray([0, 1, 2], dims='x', attrs={'foo': 'var'})
In [5]: func = lambda x: x*2
In [6]: xr.apply_ufunc(func, da, keep_attrs=True, input_core_dims=[['x']], outpu
   ...: t_core_dims=[['z']])
Out[6]:
<xarray.DataArray (z: 3)>  # attrs are tracked for xr.DataArray
array([0, 2, 4])
Dimensions without coordinates: z
Attributes:
    foo:      var

In [7]: xr.apply_ufunc(func, da.variable, keep_attrs=True, input_core_dims=[['x'
   ...: ]], output_core_dims=[['z']])
Out[7]:
<xarray.Variable (z: 3)>  # attrs are dropped
array([0, 2, 4])

Problem description
keep_attrs=True should work also with xr.Variable
Expected Output
<xarray.Variable (z: 3)>  # attrs are dropped
array([0, 2, 4])
Attributes:
    foo:      var

Is it an intended behavior? The implementation looks that it is designed so.
I thought that attrs is originally an attribute of xr.Variable not of xr.DataArray, keep_attrs should be valid also for this object.

I think I just forgot to implement this when I wrote apply_ufunc.


"Manage Roles By Username" icon isn't obviously the right place to add users to a space

Detailed Description
As a Space Manager, I need to add other users to my space. To do that, I need to find a very generic icon that shows two silhouettes and has the tooltip "Manage Roles By Username". I would like this action to be more discoverable.

Context
Twice in one week, new space/organization administrators have asked how they can add users to their space/organization. These users had to be pointed to documentation that described where on the "Users" page to click for adding new users.
Possible Implementation
Styling the "button" as a high-emphasis clickable action, similar to https://material.io/components/buttons#contained-button is one possible change. Another similar change would be to add a "+" to the icon to make it clearer that it can be used for adding users.

Hi @neilmb , thanks for raising this issue. This should be straightforward to fix, we'll look to address it in 4.2 due around the end of October.