Overloaded base function isn't called

struct Test1 {
    void test() {
    }
};
struct Test2 : Test1 {
    void test(int value) {
    }
};

Test2.new():test()
Test2.new():test(2) # error stack index 2, expected number, received no value: not a numeric type that fits exactly an integer (number maybe has significant decimals) (bad argument into 'void(int)')

Is this normal, or there is a bug somewhere?

Perhaps you should show us how you define the Sol usertypes.
From a C++ (rather than Sol or Lua) point of view Test2::test does not actually overload Test1::test: it hides it.   To get an C++ overload you need to include a using Test1::test; in the definition of Test2.  This may or may not be relevant to your issue.


feat: amounts should be set in cts, not dollar

Summary
Implements enhancement proposed in #100.
~As far as I can tell it is not possible to avoid the flickering issue when adding values on the right hand side of the decimal point without overlaying a Text field that covers/masks the input field thus fixing the cursor to the end and sacrificing the ability to do selections and interact with the input field in ways that one may expect. Let me know if you think we should go with the masked approach, then I'll revise this PR. @nicolasburtey @jrbuhl93~ edit: New approach below
~More details about this issue here: https://github.com/CaioQuirinoMedeiros/react-native-currency-input#fakecurrencyinput~

Prior approach demo
https://user-images.githubusercontent.com/959142/126685362-42163139-fe99-4b14-bf3a-4f9af28d27a4.mp4

Changes

Utilizes a masked input field approach to give the user a flicker free experience: A Text component is overlayed on top of an invisible TextInput. The TextInput is technically still selectable. I couldn't find any trivial way of disabling this, but I think this is a minor issue.
Input field is no longer empty. Either is 0.00 for USD or 0 for sats. I have removed the placeholder text for the empty string/empty as this no longer is a valid state.
Formatting improvement for USD. As an example fifty thousand is no longer formatted as 50000.00 but rather 50,000.00. To enable this on Android a polyfill was necessary. See: https://stackoverflow.com/a/55861894. It is possible to have it formatted as $50,000.00 by changing the style option of toLocaleString to currency and specifying the additionally required properties.

New approach (Masked input):
https://user-images.githubusercontent.com/959142/126820798-182f9ad8-74d6-4220-8e7a-4e013cdc37de.mp4
Tested on:

[x] iOS
[x] Android

I think the masked input would go great with a UI where the input is more prominent and the center of attention, like this example: https://www.figma.com/file/aLAa3OuBW74EcSWxjjY82m/Bitcoin-Wallet-UI-Kit-Community?node-id=831%3A18002

Here is an example of how the masked input would look and behave like: (source)

without looking in the PR (yet) I can say that I spent some time on this (maybe 2 or 3 days) and struggled to come up with something that really worked well. glad this is being worked out.
I think the flickering is indeed not great. it give an impression of a "cheap" app.

without having a look into the PR (yet) I can say that I spent some time on this (maybe 2 or 3 days) and struggled to come up with something that really worked well. glad this is being worked out.

Was quite surprised with the amount of complexity here. May be able to simplify it a bit if we make the suggested changes below:

I think the flickering is indeed not great. it give an impression of a "cheap" app.

Agree. Will make this change. Do we want this to only apply to the USD view or also sats for consistency?
Furthermore, should we remove the decimal pad and only use number pad?

Agree. Will make this change. Do we want this to only apply to the USD view or also sats for consistency?

I think it's best to give a consistent feel between currencies.

Furthermore, should we remove the decimal pad and only use number pad?
I say use number pad for usd after this change.

@nicolasburtey I think this is ready for you to try out/review!

Without the cursor to act as a guide for the user, should we have another visual cue that the field is selected like changing the color of the underline of the input?

Sounds good to me. Maybe just a darker tint of the existing color or should we use something more vibrant?

When I changed between sats and usd the keyboard was dismissed. I think we should keep it up after switching.

Made this change by essentially removing some of the code. @nicolasburtey is it safe to remove this, I don't see a need for it?
https://github.com/GaloyMoney/galoy-mobile/blob/2e8b839e6e2222e713aa8415d96512e3a7f6d349/app/components/input-payment/input-payment.tsx#L130-L141

I had gave it a try. I was a bit confuse at first because there is no more cursor so it's unclear we are editing the field. but after a few try I got used to it.
I feel overall this is much better but I'm wondering if there are some insights to give to the user to suggest him the focus is on the price and not the memo for instance.
There is some misalignement on my iPhone (X). maybe this could be the address at the same time: https://github.com/GaloyMoney/galoy-mobile/issues/99

Sounds good to me. Maybe just a darker tint of the existing color or should we use something more vibrant?

I think we should do something a little more vibrant and use one of the the app's blue colors. We might need it on the other input fields that aren't masked in order to maintain consistency, but that could be out of scope for this PR.

Could also make amount text larger when input field is focused. Initially it would be large as input field is autofocused and when you go to the next field it would shrink (nicely animated of course). Maybe this would also resolve #99?
Should I give this one a shot or do we want to go with the colored underline on focus despite the inconsistency?
Could also add a faux blinking thing at the end of the masked input, but would prefer the larger input text to this.

Should I give this one a shot or do we want to go with the colored underline on focus despite the inconsistency?

I personally prefer the underline, but if you or @nicolasburtey prefer growing and shrinking the text you should do that.

Input from Nicolas:

good enough for merge since its already better than what we have
potentially highlight text that is being edited via "color bubble" but can be addressed in additional PR
need to increase font size as its hard to read


add decorators module

These are decorators that I have been adding in to scripts and tests in my PRs over the past couple days but are more generally useful, so I decided to put them into their own module.

What are the use cases for these in the main code base? Until a decorator is used in more than one module it should be defined where it's used --- I don't see any benefit from having a separate module right now.

@Naburimannu to answer your comment that strangely got deleted:
"@dcolligan Can you tell us how it [unittest.assertRaises] didn't work for you?"
Yes, I was using it wrong.  I pushed an update deleting that comment.

@Naburimannu @jeromekelleher is there anything preventing you guys from +1'ing this?  I'd like to get it in...

I'm -1 on this until I see a pressing reason for needing to share these across modules. Just define them where they are used.

Would you accept Timeout and Repeat in tests/utils.py, since they will be used in testing?
Also, I think both of those decorators should really be tested, so I want to include the tests somewhere.  I'm fine with not testing Timed and moving it to scripts/utils.py (where it already is).

Sure, that sounds sensible.

My feedback is that I prefer to see Repeat as a code fragment, and
decorators used for metadata (which Timeout is marginally; Timed is
really a border case for me), but could be convinced otherwise by strong
argumentation or examples of the use of this style leading to readable and
easily-understood code. When I can't predict the behavior of a function at
its callsite without reading the definition, I'm unhappy, and as I said
previously my skimming of the relevant PEP seems to support that.
So I'm -0 on any of these as decorators, but won't stand in the way.
Tom

Pushed a new commit that moves the newly defined decorators into tests/utils.py

Can we shelve this until we have some immediate use cases? The discussion is too abstract otherwise: we should evaluate these decorators on whether they make solving problems that we are facing easier. My feeling is that they probably will come in handy when we're doing the end-to-end tests, but I don't want to make a judgement until we really dig into that problem.

Alright.


General: Add option to export results with test cases and feedback

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] ~I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).~
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

~Changes affecting Programming Exercises~

[ ] ~I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).~
[ ] ~I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).~

Motivation and Context

Some instructors suggested the integration of this feature in order to increase the flexibility provided by the exported CSV results file.
Description

This PR replaces, in the scores view of programming exercises, the export results button with a dropdown one containing three possible export formats.

Without test cases: the standard results export format as used before
With test cases: additionally to the standard results, this includes a passed or failed information for each of the exercise's test cases. If information about the test case cannot be found (i.e. on failed builds), skipped is used.
With test cases and feedback: this includes the same additional data as With test cases but includes a copy of the provided feedback in case of failed test cases

An example of the three formats can be seen below:
Steps for Testing

Prerequisites:

1 Instructor
1 Programming Exercise with scores
1 Text Exercise (or any exercise type except for programming ones)

Log in to Artemis
Navigate to Course Administration
Navigate to the programming exercise's score view
Export the results using all three formats and check its resulting files
Navigate to the "non-programming" exercise
Export its scores and check the resulting file

Review Progress

Code Review

[x] Review 1
[ ] Review 2

Manual Tests

[x] Test 1
[ ] Test 2

~Exam Mode Test~

[ ] ~Test~

Test Coverage

Class/File
Branch
Line

exercise-scores-export-button.component.ts
77.65%
96.87%

Screenshots

Without test cases:

With test cases:

With test cases and feedback:

@Vinicius-Agreste thank you for testing 👍
The formatting seems just related to the viewer settings, as the separator character (,) is included in the text.. In this case the 'comma' symbol is used instead of the semi-colon..
But you made me realize that unwanted test cases are added from the manual feedbacks (i.e. File src/..... test cases), which I now fixed with https://github.com/ls1intum/Artemis/pull/5910/commits/28b1f437b0e0fee7b160b18a24e43a64d0e72240 🙂

Only "With test cases and feedback" working as expected, all other 3 exports have all information in the first cell of the line, as in the image below

Thanks @grochlitzzz for testing 🙂
This is due to the separator used in the exported CSV file.. This has remained unchanged for all export modes, expect for "With test cases and feedback" where I changed it to use the 'comma' character.
In Excel and other spreadsheet tools you can choose what separator symbol to use, so changing it to a semicolon ; should make the trick and visualize the data correctly 👍

"Without testcases" and "With testcases" both work as expected. "With testcases and feedback" is not working though. It's showing all information in one cell. Tested on TS1.

Hi @PhilipWerz and thanks for testing 🙂
Did you set the right separator when opening the file? For "With testcases and feedback" I had to use the more standard , character as separator because of newlines..


v3.8.0

/slp/burnTotal/{transactionId}
Conditional invalidReason

Pull Request Test Coverage Report for Build 2225

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

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/slp.js
140
49.42%

Totals

Change from base Build 2202:
-0.01%

Covered Lines:
2047

Relevant Lines:
2732

💛  - Coveralls


Support for IE11

Not sure if this was meant to be used in a browser or not, but I do, through one of the packages that I'm using.
The current version is only published with ES2015+ syntax, meaning it can't be used in IE11.
Is there a chance to also compile it down to ES5?

This module mainly targets Node.js, not the browser. It's up to you to transpile it with Babel if you want to use it in the browser. You can find a more detailed explanation here: https://github.com/sindresorhus/ama/issues/446
If you use Webpack, check out babel-engine-plugin, which transpiles only the dependencies that needs to be transpiled.


Address compliance test failures against emulated nodes.

  * (M) cmd/ccli/ccli_test.go
    - add CLI argument for specifying network instance name.
  * (M) compliance/compliance.go
  * (M) compliance/election.go
  * (M) compliance/get.go
    - reference local configurable value of network instance name rather
      than fake server's const value.

--- FAIL: TestCompliance (18.53s)
    --- PASS: TestCompliance/Modify_RPC_connection (0.06s)
    --- PASS: TestCompliance/Modify_RPC_Connection_with_Election_ID (0.17s)
    --- PASS: TestCompliance/Modify_RPC_Connection_with_invalid_persist/redundancy_parameters (0.16s)
    --- FAIL: TestCompliance/Add_IPv4_entry_that_can_be_programmed_on_the_server_-_with_RIB_ACK (1.19s)
    --- FAIL: TestCompliance/Add_next-hop-group_entry_that_can_be_resolved_on_the_server,_no_referencing_IPv4_entries_-_with_RIB_ACK (0.34s)
    --- FAIL: TestCompliance/Add_IPv4_entries_that_are_resolved_to_a_next-hop-group_containing_multiple_next-hops_(single_ModifyRequest)_-_with_RIB_ACK (0.74s)
    --- PASS: TestCompliance/Add_IPv4_entries_that_are_resolved_to_a_next-hop-group_containing_multiple_next-hops_(multiple_ModifyRequests)_-_with_RIB_ACK (0.27s)
    --- FAIL: TestCompliance/Delete_IPv4_entry_within_default_network_instance_-_RIB_ACK (0.36s)
    --- FAIL: TestCompliance/Delete_NHG_entry_that_is_referenced_-_failure_-_RIB_ACK (0.56s)
    --- FAIL: TestCompliance/Delete_NH_entry_that_is_referenced_-_failure_-_RIB_ACK (0.35s)
    --- FAIL: TestCompliance/Delete_NHG_entry_successfully_-_RIB_ACK (0.47s)
    --- FAIL: TestCompliance/Delete_NH_entry_successfully_-_RIB_ACK (0.27s)
    --- FAIL: TestCompliance/Add_Metadata_for_IPv4_entry (0.35s)
    --- SKIP: TestCompliance/Add_IPv4_Entry_that_references_a_NHG_in_a_different_network_instance (1.07s)
    --- FAIL: TestCompliance/Add-Delete-Add_for_IPv4Entry_-_RIB_ACK (0.35s)
    --- FAIL: TestCompliance/Implicit_replace_NH_entry_-_RIB_ACK (0.34s)
    --- FAIL: TestCompliance/Implicit_replace_NHG_entry_-_RIB_ACK (0.34s)
    --- FAIL: TestCompliance/Implicit_replace_IPv4_entry_-_RIB_ACK (0.35s)
    --- FAIL: TestCompliance/Ensure_failure_for_a_NH_entry_that_does_not_exist (1.37s)
    --- FAIL: TestCompliance/Ensure_failure_for_a_NHG_entry_that_does_not_exist (0.26s)
    --- FAIL: TestCompliance/Ensure_failure_for_an_IPv4_entry_that_does_not_exist (0.94s)
    --- FAIL: TestCompliance/Get_for_installed_NH_-_RIB_ACK (0.41s)
    --- FAIL: TestCompliance/Get_for_installed_NHG_-_RIB_ACK (0.41s)
    --- FAIL: TestCompliance/Get_for_installed_IPv4_Entry_-_RIB_ACK (0.62s)
    --- FAIL: TestCompliance/Get_for_installed_chain_of_entries_-_RIB_ACK (0.41s)
    --- FAIL: TestCompliance/Benchmark_Get_for_next-hops (0.63s)
    --- PASS: TestCompliance/Election_-_Ensure_that_election_ID_is_not_accepted_in_ALL_PRIMARY_mode (0.17s)
    --- FAIL: TestCompliance/Election_-_Ensure_that_a_client_with_mismatched_parameters_is_rejected (0.26s)
    --- FAIL: TestCompliance/Election_-_Matching_parameters_for_two_clients_in_election (0.37s)
    --- PASS: TestCompliance/Election_-_Ensure_client_with_differing_parameters_is_rejected (0.26s)
    --- FAIL: TestCompliance/Election_-_Lower_election_ID_from_new_client (0.17s)
    --- PASS: TestCompliance/Election_-_Active_entries_after_new_master_connects (0.79s)
    --- PASS: TestCompliance/Election_-_Unannounced_master_operations_are_rejected (0.17s)
    --- PASS: TestCompliance/Election_-_Incrementing_election_ID_is_honoured,_and_older_IDs_are_rejected (0.98s)
    --- PASS: TestCompliance/Election_-_Decrementing_election_ID_is_ignored (0.26s)
    --- FAIL: TestCompliance/Election_-_Sending_same_election_ID_from_two_clients (0.36s)
    --- PASS: TestCompliance/Election_-_Sending_election_ID_as_zero (0.16s)
    --- FAIL: TestCompliance/Flush_of_all_entries_in_default_NI_by_elected_master (0.34s)
    --- FAIL: TestCompliance/Flush_from_non-elected_master_is_ignored (1.05s)
    --- FAIL: TestCompliance/Flush_from_client_overriding_election_is_honoured (0.35s)
    --- SKIP: TestCompliance/Flush_to_specific_network_instance_is_honoured (0.06s)
FAIL

A number of these failures are to do with Flush being unimplemented.

Thanks @liulk - PTAL at the updates and explanations :-)


"Income" should be replaced with IncShkDstn or something like that

As it stands currently, most of the components of HARK that are keeping track of the income shocks use the tag "Income".  That is confusing to users, who might not realize that the object being referenced is a distribution of income shocks, rather than the level of income.
At the next overhaul, we should do a global search-and-replace.  I'd propose using IncShk instead of Income

Can you point to an example of this?  Do you mean IncomeDstn?  I know that
I used IncShkDstn in some models, but that might have been for my own
papers.
On Thu, Feb 21, 2019 at 4:15 PM Christopher Llorracc Carroll <
<EMAIL_ADDRESS>wrote:

As it stands currently, most of the components of HARK that are keeping
track of the income shocks use the tag "Income". That is confusing to
users, who might not realize that the object being referenced is a
distribution of income shocks, rather than the level of income.
At the next overhaul, we should do a global search-and-replace. I'd
propose using IncShk instead of Income
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/227, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANUQFW3-6lk934BrpmTRJVNBCL1x0qPyks5vPwxggaJpZM4bIVyB
.

This terminology was used in a couple of DemARKs (Chinese-Growth, in particular) and I did a quick search within the ConsumptionSaving directory and thought I had confirmed that the DemARKs were just mirroring a usage in ConsumptionSavingModel.py.  Your message prompted me to look more closely, and I'm glad to say it seems I was wrong -- the problem is only in DemARK.  I'll fix it in Chinese-Growth and wherever else I saw it.

I don't understand the tag "Usages".  Can this be changed to something like "Terminology" or a related term that makes clear it's about variable names?

I don't understand the tag "Usages".  Can this be changed to something like "Terminology" or a related term that makes clear it's about variable names?

Actually, I like "usages" because the issue I was flagging was more about
whether spirit of the existing terminology/notation was being implemented,
than about defining yet more terminology.  And, though I withdrew my
erroneous claim that there was a misusage in the core tools, I subsequently
found some other cases where "Income" was used as a synonym for income
shocks.  I can't immediately identify those circumstances, but I think the
"Usages" flag is a good one to identify these kinds of mild issues that do
not require the definition of new terminology but just more thoughtful or
consistent application of existing terminoology.
On Mon, Feb 25, 2019 at 9:11 AM Matthew N. White<EMAIL_ADDRESS>wrote:

I don't understand the tag "Usages". Can this be changed to something like
"Terminology" or a related term that makes clear it's about variable names?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/227#issuecomment-467024969, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABQlf698lfMffFZDAlEOJ1eoq-OIcGx3ks5vQ-8PgaJpZM4bIVyB
.

--

Chris Carroll

I searched the HARK repo and it seems that it is only used for two things: IncomeDstn (which could arguable be called IncShkDstn instead, but... ), and in ConsIndShkModel.py where it's used in updateIncomeProcess and constructLognormalIncomeProcessUnemployment. If neither IncomeDstn nor the IncomeProcess names are candidates for a name change, this issue is resolved I think.

This issue is already resolved. See the previous discussion. Closing now.

This issue is already resolved. See the previous discussion. Closing now.

Nope. Still have to rename IncomeDstn to IncShkDstn this one.

Nope. Still have to rename IncomeDstn to IncShkDstn this one.


Q: is it possible to use update-alternatives with bob?

Normally, i change my default editor via update-alternatives to neovim.  I decided to try using bob to manage neovim for me, but when I tried setting vi to the nvim executable from bob, it acts like I just ran bob instead.  Is there a way to do this via update-alternatives, or should i just alias instead?

I have no idea what's update-alternatives

basically, a way to manage things like which editor I want to use when I type vi.
Man Page
Overview

I see, can you show me how you set it up?
Bob uses a proxy executable, if you rename bob to nvim it will execute neovim rather than bob.

I just ran sudo update-alternatives --install /usr/bin/vi vi /home/rsb177/.local/share/bob/nvim-bin/nvim 100.  It looks like since it's a proxy executable, if I set up a symbolic link to it (as any name other than nvim), it'll always run bob.  Was able to replicate this by just using simple symbolic links.  So it looks like I'll need to stick with just using aliases if I want to use nvim for commands like vi, view, etc.

@rsb177 what name does the update-alternatives gives the executable?

in the case of the command above, it shows up as vi

a worked example for vim / nvim https://thomasventurini.com/articles/how-to-use-update-alternatives/
I used update-alternatives, and somehow bob has replaced vim


Masked Language Modelling Imports Failing

When running all cells in this colab from here I get the following error:

ImportError                               Traceback (most recent call last)
 in ()
8 import pickle
9
---> 10 from perceiver import perceiver, position_encoding, io_processors, bytes_tokenizer
ImportError: cannot import name 'bytes_tokenizer' from 'perceiver' (/content/perceiver/init.py)

I've looked at #254 and added those lines, but they don't seem to help in this case.
How can I run the code in the colab?

!wget -O /content/perceiver/bytes_tokenizer.py https://raw.githubusercontent.com/deepmind/deepmind-research/master/perceiver/bytes_tokenizer.py

Thanks for reporting this issue! @lkhphuc is right -- that line was missing from the Colab. This fix has now been applied to the the notebook.


expected string or bytes-like object, got 'list'

Describe the bug
Toyota EU integration in HA return error "Failed setup, will retry". As additional error message I found this
expected string or bytes-like object, got 'list'
To Reproduce
Steps to reproduce the behavior:

Go to (homeassistantIP:8123/config/integrations/integration/toyota)
See error

Expected behavior
Expected is to work
Screenshots

Additional context
Latest integration version. I erased it from HACS and checked in custom_components folder for any remained files.
Restarted HA, installed again integration. Same behaviour.

Which Version of mytoyota is installed on your Home Assistant?
With mytoyota v2.0.0 there should be already a fix for this: https://github.com/DurgNomis-drol/mytoyota/pull/319

I just released a new version which pins the used mytoyota version to 2.0.0:
https://github.com/DurgNomis-drol/ha_toyota/releases/tag/v2.0.1
Can you please test and report back, if it works for you?

Updated to 2.0.1 and now it works ok. Thanks.

Thx for checking!


Can you setup rollup summary to calculate mode to realtime and then also set up a schedule calculation at the same time.

I have several rollup summary that are setup to run realtime. But I would like to setup a schedule calculation also. Is this possible?
My rollup are used to count records, but I would like the count to start over at the beginning of the year. So that they are zero out.

Hmmm one approach would be to have a zeroing rollup that you schedule once a year. Maybe the same as your current rollup, but with a criteria that will never be met, thus the tool will put a zero in the count. This would not be the same rollup definition though. One for your realtime, one for your scheudle zeroing rollup. Keep in mind though, once your realtime one fires will it not count all existing records again? Or have you sorted that with your criteria logic for the realtime rollup?

The realtime will re-count existing records again.
Chris Brunet
Salesforce Consultant/Engineer
StarrData
(510) 556-1997
www.starrforce.com
<EMAIL_ADDRESS>Salesforce.com Registered Partner
Facebook: facebook.com/StarrForce
Twitter: @STARRFORCE https://twitter.com/STARRFORCE
Blog: http://starrforce.com/cloudthink
On Fri, Apr 14, 2017 at 9:25 PM, Andrew Fawcett<EMAIL_ADDRESS>wrote:

Hmmm one approach would be to have a zeroing rollup that you schedule once
a year. Maybe the same as your current rollup, but with a criteria that
will never be met, thus the tool will put a zero in the count. This would
not be the same rollup definition though. One for your realtime, one for
your scheudle zeroing rollup. Keep in mind though, once your realtime one
fires will it not count all existing records again? Or have you sorted that
with your criteria logic for the realtime rollup?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/483#issuecomment-294271341,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALrUESsUM0ok099vLUNQEJjDxuzxVloXks5rwEbMgaJpZM4MsIf9
.

Ok is that what you want? If not add some criteria to exclude prior rows by date perhaps?

@BrunetChris let me know if you still want to discuss this, i'll close for now, we can for sure re-open. 👍


refactor: adapt account-address component with formly

## PR Type[ ] Bugfix[ ] Feature[ ] Code style update (formatting, local variables)[x] Refactoring (no functional changes, no API changes)[ ] Build-related changes[ ] CI-related changes[ ] Documentation content changes[ ] Application / infrastructure changes[ ] Other: ## What Is the Current Behavior?Currently, the account-address component uses the old select-address component, which is a specification of ish-select.## What Is the New Behavior?Now, the page uses formly and is refactored to be more functional.## Does this PR Introduce a Breaking Change?[ ] Yes[x] No## Other Information
AB#65504

Pls ensure both select boxes have always the same (max) width


How can i build static framework for tf-lite under 300kb for iOS?

Please make sure that this is a documentation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:doc_template
System information

TensorFlow version: master
Doc Link:https://www.tensorflow.org/lite/overview

Describe the documentation issue
Dear supporter,
I found "Smaller in size: TensorFlow Lite is smaller than 300KB when all supported operators are linked and less than 200KB when using only the operators needed for supporting InceptionV3 and Mobilenet." under TensorFlow Lite highlights.
I got libtensorflow-lite.a for each archtecture afte using build_ios_universal_lib.sh script. Binary size is far bigger, around 7.5MB. How can I get one smaller than 300KB?  Do I need to do any optimization on compiling and how?

I have the same problem，has been looking for a solution.

@jvishnuvardhan  Do we got any progress on this issue?  I still had no clue on this problem.

@aselle  ?

Have you looked at the tensorflow lite micro code?  tensorflow/lite/experimental/micro  Targeted at very small footprint implementations, but very new (and incomplete) at this point.  I'm not sure how to build a library of the size indicated in the docs.  Maybe @jdduke can comment?

When you say "binary size" is bigger, are you measuring the size impact on the actual iOS app? Or just the static library? Looking at just the static library size is misleading, as it ignores all the size reduction that will occur from op stripping (selective registration) as well as symbol stripping (which is controlled by your build options).
See this guide for information on how to trim some of the builtin operators. If you know exactly which ops are used by your model, you can create a custom OpResolver and register only the ops you need. When your app is actually built/linked, it should trim away unused code.

@jdduke  hi, I mean static library.  there is no size impact on actual iOS app if I only drag the libtensorflow-lite.a into my Xcode project without using it. How can I quick test the size impact as all iOS examples using downloaded .framework ?   when the "selective registration" happens , who trigger it and how does it work?  which option you mean when you say "controlled by your build options", tf-lite build option? Xcode build option?

The guide is pretty vague on how to trim the built-in operators; is there more specific documentation particularly for iOS/android? The size impact on the actual iOS app is about 1.2 MB just for the framework, so trimming it down would be very useful.

Agreed that the documentation could be more explicit. You can start by creating a MutableOpResolver, and then adding only the kernels used by your model. There's an example of doing this here, which takes advantage of a bazel build rule for generating only the ops used by a specific model. If you can't use the bazel build rule to generate that code, you'll have to manually add each op yourself (this code demonstrates how this is done for the BuiltinOpResolver).

Thanks @jdduke I'll try that out

Worked great! Thanks so much for your help @jdduke

Automatically closing this out since I understand it to be resolved, but please let me know if I'm mistaken.Thanks!

@jdduke fellowed by your example，got “l17_ops_registration.cc” which contains only 12 operators .  but it seems not reduce library size when i include it . How to use it properly? thanks!

So, using the MutableOpResolver approach won't change the size of the static library, but it should shrink the size of the actual binary that gets produced in an app, as the unneeded code/symbols can be stripped.

So, using the MutableOpResolver approach won't change the size of the static library, but it should shrink the size of the actual binary that gets produced in an app, as the unneeded code/symbols can be stripped.

Oh，is there another way to reduce static library?
here is my BUILD file.  Only a "cc_binary + static" .so can be loaded ok in unity(c#).

cc_binary(
name = "Prophet.so",
srcs = ["minimal.h", "Prophet.cc",   ],
linkopts = tflite_linkopts() + select({
"//tensorflow:android": [
"-pie",  # Android 5.0 and later supports only PIE
"-lm",  # some builtin ops, e.g., tanh, need -lm
],
"//conditions:default": [],
}),
deps = [
"//tensorflow/lite:framework",
"//tensorflow/lite/c:c_api_internal",
"//tensorflow/lite/kernels:builtin_ops",
"//tensorflow/lite:builtin_op_data",
"//tensorflow/lite/schema:schema_fbs",
],
linkshared = True,
)

Agreed that the documentation could be more explicit. You can start by creating a MutableOpResolver, and then adding only the kernels used by your model. There's an example of doing this here, which takes advantage of a bazel build rule for generating only the ops used by a specific model. If you can't use the bazel build rule to generate that code, you'll have to manually add each op yourself (this code demonstrates how this is done for the BuiltinOpResolver).

a little  confusion：
i got "_ops_registration.cc".
it is ok when extract codes to "lib.cc",
but wrong when "lib.cc" include "_ops_registration.cc" and BUILD file is  srcs = ["llib.h" ,"_ops_registration.cc", "lib.cc"].

Agreed that the documentation could be more explicit. You can start by creating a MutableOpResolver, and then adding only the kernels used by your model. There's an example of doing this here, which takes advantage of a bazel build rule for generating only the ops used by a specific model. If you can't use the bazel build rule to generate that code, you'll have to manually add each op yourself (this code demonstrates how this is done for the BuiltinOpResolver).

why cann't recently on fresh source?

bazel build tensorflow/python/tools:print_selective_registration_header && \
bazel-bin/tensorflow/python/tools/print_selective_registration_header \
--graphs=models_arm64/1.pb > ops_to_register.h

<long int, 1>; int packet_size = 4; bool inner_dim_contiguous = false; bool inner_dim_reordered = true; int Alignment = 0]':
./tensorflow/core/kernels/eigen_spatial_convolutions.h:308:11: warning: 'orig_r' may be used uninitialized in this function [-Wmaybe-uninitialized]
./tensorflow/core/kernels/eigen_spatial_convolutions.h:308:11: warning: 'orig_r' may be used uninitialized in this function [-Wmaybe-uninitialized]
gcc: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
Target //tensorflow/python/tools:print_selective_registration_header failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 724.181s, Critical Path: 123.60s
INFO: 2191 processes: 2191 local.
FAILED: Build did NOT complete successfully


Query Failure

I deployed marbles in localhost, everything run fine until some of time.
When I wake up this morning, and tried to run marbles again, I got this error:
read  - success: { jsonrpc: '2.0',
  error:
   { code: -32003,
     message: 'Query failure',
     data: 'Error when querying chaincode: Error:Failed to launch chaincode spec(Timeout expired while starting chaincode 14b711be6f0d00
b190ea26ca48c22234d93996b6e625a4b108a7bbbde064edf0179527f30df238d61b66246fe1908005caa5204dd73488269c8999276719ca8b(networkid:dev,peerid:
c9527fe1-2332-48aa-8e6f-797d0689c52e_vp1,tx:f77a672d-b239-498b-9e54-803ebeccfca5))' },
  id:<PHONE_NUMBER>791 }
marbles index msg error: [SyntaxError: Unexpected token u]

All marbles did I create also dissapear and I can't create new marble.
Any idea what happened?

@kimdotnet   - are you running everything on localhost - including the peers?

no, I use blockchain service for the peers.
Now I also got this error when trying to get marbles data, any idea what happen:
[ibc-js] read  - success: { jsonrpc: '2.0',   result: { status: 'OK', message: 'null' },   id:<PHONE_NUMBER>732 }

Unfortunately, I think you may just need to delete your instance of the blockchain service on BMX and create a new one.  We had a few issues which we have since corrected and unfortunately a few networks are not in an easily recoverable state.

@mastersingh24 I already delete my instance, now that problem is solved. But now I got new problem.
My marbles is dissapear after some time (most likely 8 hours).
In my console I got this error, after deploy success message.
        deploy success [waiting another 50 seconds]
         14b711be6f0d00b190ea26ca48c22234d93996b6e625a4b108a7bbbde064edf0179527f30df238d61b66246fe1908005caa5204dd73488269c8999276719ca8b

[ibc-js] ibc.save() error { [Error: ENOENT: no such file or directory, open 'C:\Users\kim\Documents\Learn node.js\marbles\cc_summaries\14b711be6f0d00b190ea26ca48c22234d93996b6e625a4b108a7bbbde064edf0179527f30df238d61b66246fe1908005caa5204dd73488269c8999276719ca8b.json']
  errno: -4058,
  code: 'ENOENT',
  syscall: 'open',
  path: 'C:\\Users\\kim\\Documents\\Learn node.js\\marbles\\cc_summaries\\14b711be6f0d00b190ea26ca48c22234d93996b6e625a4b108a7bbbde064edf0179527f30df238d61b66246fe1908005caa5204dd73488269c8999276719ca8b.json' }

What the meaning of that error? is that error related to the loss of my marble?

That error is harmless and would have nothing to do with your symptoms.  It simply cannot save an optional summary file that can be used to debug/transfer ibc settings.
As for losing your marbles, do you notice if the blockchain ledger is resetting? Examine the block height of the chain from either the UI at the bottom, or by using the GET /chain API directly.  Is it possible someone else is using your network?

@dshuffma-ibm okay, after trial and error. Now I got this pattern.

Run gulp command
insert marble
exit program (by pressing ctrl+c)
run gulp again
marbles are still there

This is the case:
My marbles only lost when I run "gulp" command while the CC status is "Exited (0) X hours ago". Any idea why this is happen?
My marble is safe whenever I run gulp comman, but CC status is "UP".

interesting, so you are saying:

everything is okay if you run gulp when the status for the chaincode container is "up"
previous marbles are lost if you run gulp when the status is exited

I will try this out myself in a few minutes and let you know.

Well there is something going on. I was able to create 1 network to mimic exactly what you saw. Unfortunately I can't seem to do it again!
For some unknown reason its possible for the chaincode container to be stuck in a created state.  Its not running, but its existence means the peer will be unable to bring up another.
A temporary fix is we can delete the stuck container if you provide your network ID.  The long term fix is we have noted this bug and will attempt to make it repeatable so we can fix it.

@dshuffma-ibm

interesting, so you are saying:

everything is okay if you run gulp when the status for the chaincode container is "up"
previous marbles are lost if you run gulp when the status is exited

yes, exactly. Btw, thank you for your answer.
===============================================================================
One more question david, Is it possible to send JSON error message from chaincode? if it is possible, is it a good practice or not?
For example:
In case I adding marble, which the marble already exist in the chaincode state. Is it possible to send this json:
{
  "jsonrpc": "2.0",
  "result": {
    "status": "FAIL",
    "message": "Marble already exists"
  },
  "id": 0
}

rather than this:
{
  "jsonrpc": "2.0",
  "result": {
    "status": "OK",
    "message": "c124a17a-7a6a-406c-bf9f-125a8cf8326b"
  },
  "id": 0
}

Because in marble app, if I tried to add 2 or more marble with the same name. There is no error message for client to know, like "Hey, this marble already exists".
In current state, marble app only will redirect to home page. Regardless is marble success being added to the chain or not.

So we need to separate the question into two parts.
Is it possible to return an error from a query cc function?

Yes!. The chaincode should return an error such as:

return nil, errors.New("Something is wrong. Error details and such here as a string")

this should be visible in the response from a query API call, I believe its buried in the response.error.message JSON (might not be exact field name, but its close)
I would consider this excellent practice.  Getting an error msg back directly is a lot faster than burying your self in log messages to see what happened.

Is it possible to return an error from an invoke cc function?

No. The outcome of an invoke is not decided at the time of the API call. In the real world there is a consensus algorithm to go through. Therefore the success/failure of the chaincode is not known until its executed and there is consensus on the transaction.
tldr: You will always see invoke successful
there has been talks about adding callbacks via event subscriptions to determine the status of a transaction, but as far as I know it is not yet implemented

@dshuffma-ibm okay, I get it. So at this time, it's not possible to return error message from invocation.
Hope it will be implemented later, because it will be confusing for the user if he can't know whether his action is successfully saved or not. example: when adding marble which is already exists.
Thank you for your support david.

@dshuffma-ibm  okay I get it. So at this time, it's not possible to return an error message from invocation.
Hope it will be implemented later, because it will be confusing for the user if he can't know whether his action is successfully saved or not. Example: when adding marble which is already exists.
Thank you for your support david.


AzureML Onboarding

This item should cover all issues created to cover the AML onboarding.
This includes

Transitioning pshell scripts to add new release functionality
Transitioning Builds to YML
Adjusting existing pipelines to accommodate new build requirements
Rationalize SemVer

Closing old epic.


Update to new NetQASM interface

Closes #6

Blocked until https://github.com/QuTech-Delft/netqasm/pull/14 is merged.

Unblocked now that https://github.com/QuTech-Delft/netqasm/pull/14 has been merged.


Institution Login Workflow Diagram

https://www.figma.com/proto/mLw9vmpZj7BM0T8Y59Qf8Q/FreshWorks-Studios-StudentAid-BC-User-Flows?node-id=122%3A498&viewport=5842%2C-6476%2C1.2912873029708862&scaling=min-zoom


Dynamically added eqcss styles to the DOM don't apply to target elements

Hi, elements queries are great! I am developing WordPress themes and in WordPress Live Customizer we basically append CSS styles (with style tag) dynamically to the dom and of course elements get affected immediately without needing a page refresh. However, this doesn't seem to work with eqcss styles added dynamically to the dom;

adding this to the dom dynamically doesn't change any styles on the target element (.box)
can this be solved naturally or do I have to use eval() for dynamic properties and make things more complicated?
Thanks.

Hi @ahmetsali, thanks for the question - I can see two ways you can load EQCSS styles any time after the page has loaded, the method you use depends on whether you want (or need) to put the EQCSS syntax into the document directly, or if you can use the plugin to parse and load the styles directly without needing to even add the EQCSS custom syntax to the page.
Loading styles with EQCSS.load()
The first technique would involve adding EQCSS custom syntax to either a <style> tag, or <script type=text/eqcss> tag in the document somewhere. If you have added new EQCSS custom syntax and want the EQCSS plugin to become aware of it, you need to run EQCSS.load(). Here's a demo:
<body>

<script src=http://elementqueries.com/EQCSS.js></script>

<script>
  setTimeout(() => {

    let styles = `

      @element body and (min-width: 800px) {

        :self {
          background: lime;
        }

      }

    `

    let tag = document.createElement('style')

    tag.innerHTML = styles

    document.body.appendChild(tag)

    EQCSS.load()
  }, 1000)
</script>

This method only requires you to run EQCSS.load() some time after the document loads, after the new EQCSS styles have been added.
You wouldn't have a setTimeout() I just added that so EQCSS had a chance to run once before we started to add the styles (you said you're adding them to a page that's already loaded).
Loading Styles with EQCSS.parse() and EQCSS.register()
The second way you can load EQCSS custom syntax is a little cleaner, rather than calling EQCSS.load() which looks for new syntax to read and then passes it to EQCSS.parse() for processing, if you have the EQCSS you want to load into the plugin you can skip adding it to the document and run it through EQCSS.parse() instead and it gives you back the same JS object that you see loaded in in EQCSS.data on a page where EQCSS is running. If you have a parsed EQCSS object like that you can give it to EQCSS.register() which will register the stylesheet with the plugin.
<script src=http://elementqueries.com/EQCSS.js></script>

<script>

  let styles = `

    @element body and (min-width: 800px) {

      :self {
        background: lime;
      }

    }

  `

  let parsedStyles = EQCSS.parse(styles)

  EQCSS.register(parsedStyles)
</script>

In this demo we have the EQCSS stylesheet as a string and we run that through EQCSS.parse() to parse it into an object, and EQCSS.register() to load it into the plugin. The same demo could also be written as:
EQCSS.register(EQCSS.parse(`

  @element body and (min-width: 800px) {

    :self {
      background: lime;
    }

  }

`))

Hopefully that clears up a couple of ways you can load in EQCSS queries any time after the page has loaded :D I'm really glad I added EQCSS.register() for this purpose, it's a relatively newer feature and made exactly for what you're trying to do!
Hope you have a wonderful day, and please let me know if you need any more explanation or demo code

Hi again @tomhodgins , both method works great, I think I will go with #2. Now I can code element-first CSS, thanks for this great plugin!
Have a wonderful day, too!


fix make RuntimeFilterTimerQueue graceful exit

Proposed changes
Issue Number: close #xxx

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

run buildall


Update mill-main to 0.10.7-14-d5a14a

Updates com.lihaoyi:mill-main from 0.10.7-5-08ae68 to 0.10.7-14-d5a14a.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "mill-main" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.lihaoyi", artifactId = "mill-main" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #21.


allow case sensetive urls

Cannot do redirect to case sensetive urls. Since they get tolowered.
i.e. trying to do a redirect from: /downloads
to: /foobar?currentTab=FreeTrials
gets redirected to /foobar?currenttab=freetrials

Sorry for taking so long, but this one is a bit more complex than I initially thought. If we do not lowercase the old url, any existing redirects will have a case sensitive lookup, which is bound to make a lot of them fail. At least it looks like that when looking at the code.
I can see that the new url could be case sensitive, there is no good reason for making it lowercase (especially including the query string), well - except that urls really should be case insensitive by default :-)
If you could prepare a new PR, keeping the lowercasing of the old url as is, but not lowercasing the new one, make sure you test this with existing urls (with varying casing), I'll merge it in.

Agreed. Only changing the new url is fine for us and the old urls are still lowercased wich prevents old lookups to fail


Page outdated - needs to be updated with new instructions and screenshots

The section "Sort using the Sort by Column button" on this page is outdated and confusing - the options described aren't found in those menus any more.

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

ID: bc6eda16-f142-5163-b0af-38fa634d7e42
Version Independent ID: 58599a34-91ac-4947-c2e5-a0508a554080
Content: Sort by column in Power BI Desktop - Power BI
Content Source: powerbi-docs/create-reports/desktop-sort-by-column.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi @Ascyron ,
Thank you for your feedback.  We've made the update and set the article to republish.
All the best,
The DOCS team


Text pile-up in LaTeX-generated PDF

Using the latest XPI build of the pdf.js viewer, rendering of a basic PDF produced with emacs org-mode and LaTeX fails, resulting in a pile-up of all the words from each line, superimposed one upon the other. This is an improvement-- the XPI build from last week just showed the red error bar and rendered nothing.
File here: http://gimranov.com/avram/writing/final_exam_euro.pdf
I'm viewing it so: chrome://pdf.js/content/web/viewer.html?file=http%3A%2F%2Fgimranov.com%2Favram%2Fwriting%2Ffinal_exam_euro.pdf
The same occurs with the online viewer.

Type3 font character print function is not "advancing" position properly

Thanks for pinning that down. I've just confirmed that everything works fine when I install a Type 1 font for pdflatex to use instead; in my case cm-super.

@brendandahl Is this an easy Type3 fix?

I started looking into this. It looked fine way back when I added the original type3 font support. db47fa2a9f8523afc8a2ae84bce407e4b9e7f3a6
So we've regressed.
@ajlyon Thanks for reporting the issue. Do you own the copyright to this file? And if so, would you be okay with us using it in our test suite?

I'm the sole author of that PDF, hold copyright, and am OK with that
file being in the test suite, although we could probably lop off a few
pages and make it a one-page PDF; no real reason to have the whole
paper.
Thanks for looking into this.

The regression was in 0414c0711f4b0d20a6680cbf3e5dc02a6e1d7bb0 142023bbc944699aa599b0efed3866ab08ffbc99 is fine.
@notmasteryet Any ideas how we should handle that for type3 fonts?

This issue has been resolved by #929.  The addon on addons.mozilla.org has not been updated yet but the plugin on the github site should be up to date.  I'm closing the issue, but if we missed anything leave a comment and we'll re-open it.
Also, thanks for the test pdf.


Unable to add watermark to video

I can add watermark to image successfully. But I'm not able to add it to a video and I have followed the readme just as I did with the image. Not sure what I'm doing wrong?

Did you solve the problem?

Did you solve the problem?


GIProbe does not function properly when used on Gridmaps.

Godot version
v3.5.beta3.official
System information
GLES3 - All supported operating systems/GPU's.
Issue description
This is a compilation of my findings from some other issue's discussing problems with GridMaps and GIProbe lighting.
I thought it'd be a good idea to make a separate issue detailing this information, for clarity, as its scattered in the 2 different issues.
Sources: #18056, #18836
As explained here,  there is a very clear difference to how GIProbe looks on regular MeshInstances and on GridMaps.
My hunch is that for whatever reason, perhaps because of a bug or an oversight, GIProbe is not applying the proper Bias value on GridMaps, which I explain further here.
The GIProbe lighting in the scene with just MeshInstances starts to look exactly the same as the one with GridMaps, when you turn the Bias in GIProbe down to 0.
It's as if the Bias on GridMaps is always 0, regardless of what is actually set in the GIProbe.
I hope this can be fixed because its quite a bummer GIProbe currently does not work with GridMaps.
Steps to reproduce

Download the MRP and compare the 2 different scenes provided.
In the scene that demonstrates only MeshInstances, select the GIProbe node and turn down the Bias value to 0.

Minimal reproduction project
Minimal Reproduction Project

As I understand it, the GIProbe bias and normal bias properties are only used in the shader, not in C++ code during baking.
The uniform is set here: https://github.com/godotengine/godot/blob/b6968ab0602bfe72c71d4efcafe608f9cac36252/drivers/gles3/rasterizer_scene_gles3.cpp#L1861-L1862
PS: Does the same issue occur with the Normal Bias property? Try setting Bias to 0.0 and Normal Bias to 1.0, then compare a MeshInstance and a GridMap.

Its the same story for Normal Bias.
In the scene with the GridMap, they both appear to do nothing, while in the scene with the MeshInstances it functions like you would expect it to.


Getting official update after rooting?

First of all, thanks for the great work!
This is just a question (not an issue)
Is there a way to get future update of sony’s official firmware even after rooting?
If possible, brief guide would be helpful.
Below is just my guess!

Flash the official PKGs described in wiki PKGs-I-Made,
Follow the update instruction from Sony (doing update from Digital Paper App in PC or macOS)

The mod considers that and you will still be able to do the official
update. But with that, you may loose the root.
So yes, if there's future updates but you want to keep the root, you need
to flash a customized PKG.
On Fri, Mar 29, 2019, 03:11 Athan Yi<EMAIL_ADDRESS>wrote:

First of all, thanks for the great work!
This is just a question (not an issue)
Is there a way to get future update of sony’s official firmware even after
rooting?
If possible, brief guide would be helpful.
Below is just my guess!

Flash the official PKGs described in wiki PKGs-I-Made,
Follow the update instruction from Sony (doing update from Digital
Paper App in PC or macOS)

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/HappyZ/dpt-tools/issues/83, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEdSKiycrKacaUfhhhRML_g-JpmGoBKqks5vbbyagaJpZM4cRxXk
.

Using the same way you update dpt before.
On Fri, Mar 29, 2019, 09:53 Yanzi Zhu<EMAIL_ADDRESS>wrote:

The mod considers that and you will still be able to do the official
update. But with that, you may loose the root.
So yes, if there's future updates but you want to keep the root, you need
to flash a customized PKG.
On Fri, Mar 29, 2019, 03:11 Athan Yi<EMAIL_ADDRESS>wrote:

First of all, thanks for the great work!
This is just a question (not an issue)
Is there a way to get future update of sony’s official firmware even
after rooting?
If possible, brief guide would be helpful.
Below is just my guess!

Flash the official PKGs described in wiki PKGs-I-Made,
Follow the update instruction from Sony (doing update from Digital
Paper App in PC or macOS)

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/HappyZ/dpt-tools/issues/83, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEdSKiycrKacaUfhhhRML_g-JpmGoBKqks5vbbyagaJpZM4cRxXk
.

Thanks!

A new firmware just comes out with a  version number 1.6.  It has some new features such as the TOC support and the zooming by pinching.
I have a rooted device with modified Applauncher as suggested in the wiki page.  I am wondering whether or not it is safe to update the firmware.  If updated, will it keep the extra functionalities I had added?  In my case, I have installed KOReader and made it launchable from the Applauncher.


Refactor: fix typing for fetchPreviousReport.ts

What does this PR do?
A small fix to correctly guess the octokit response type and removed any.

@all-contributors please add MH4GF for code

@MH4GF thank you for the contribution!

@ArtiomTr Thanks for the quick response, too! 😸


Is there the data preprocess code to band in a box?

I have download Band-in-a-box and refer to https://github.com/cifkao/pybiab. But there are still some problem that I can't fix. For example, I find the generation result from Real-Band is a little different every time. There are different notes every time I save the same song with the same style. And how to fix the wrong program number in MIDI files?
Thank you very much!

I find the generation result from Real-Band is a little different every time. There are different notes every time I save the same song with the same style.

Yes, this is a feature of the RealBand algorithm. It's randomized, so you will get a different result every time. I'm not aware of a way to fix the random seed. Why do you want to get the same result every time? Because of reproducibility?

And how to fix the wrong program number in MIDI files?

Try this script: https://github.com/cifkao/groove2groove/blob/master/code/groove2groove/scripts/fix_rb_midi.py

I find the generation result from Real-Band is a little different every time. There are different notes every time I save the same song with the same style.

Yes, this is a feature of the RealBand algorithm. It's randomized, so you will get a different result every time. I'm not aware of a way to fix the random seed. Why do you want to get the same result every time? Because of reproducibility?

And how to fix the wrong program number in MIDI files?

Try this script: https://github.com/cifkao/groove2groove/blob/master/code/groove2groove/scripts/fix_rb_midi.py

Thank you!


Release azure-cli-keyvault 2.0.1

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-keyvault 2.0.1...

Release of 'azure-cli-keyvault' with version '2.0.1' successful. View at https://pypi.python.org/pypi/azure-cli-keyvault/2.0.1.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

Merging #2884 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2884   +/-   ##
=======================================
  Coverage   63.17%   63.17%
=======================================
  Files         466      466
  Lines       26139    26139
  Branches     3997     3997
=======================================
  Hits        16514    16514
  Misses       8544     8544
  Partials     1081     1081

Continue to review full report at Codecov.

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


Testing, please ignore. whvzpp

This is a bug bounty test. Please do not approve this! whvzpp

Can we close this PR? and report a malicious PR?
Do not click "approve workflow"


Invalid operation (segfault, sigbus, etc) whenionc.ionc_write is given more arguments than expected.

Identified in #372.
Issue
(venv) glitch@147dda5e5395 ~/C/s/i/ion-python> python
Python 3.12.5 (main, Aug  6 2024, 19:08:49) [Clang 15.0.0 (clang-15<IP_ADDRESS>.5)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from amazon.ion import ionc
>>> ionc.ionc_write(None, True, False, False, None)
fish: Job 1, 'python' terminated by signal SIGBUS (Misaligned address error)

Looking at the code, it looks like the issue is the uninitialized PyObject pointers values at the top of ionc_write. In the event that the arguments are wrong, the function jumps down to the fail label, and pointers that are uninitialized get Py_DECREF'd.. which is probably what is generating the random segfault, and bus errors.

Fixed with #376.


[WIP] Helm chart to deploy GCS

I'm working on a Helm chart to deploy GCS. This PR is just to make the progress public for those interested.
Fixes #12
Current status:

The components come up (etcd operator, etcd, gd2, gluster-csi)
Devices are not added to gd2 yet, so no storage is available.

I tried to test this out. Centos-atomic does not have socat available which is required for Helm to work, and centos-atomic doesn't provide an easy way to get it installed using Ansible either.
So I'm planning to switch the base to regular centos which doesn't have the issue.

I tried to test this out. Centos-atomic does not have socat available which is required for Helm to work, and centos-atomic doesn't provide an easy way to get it installed using Ansible either.
So I'm planning to switch the base to regular centos which doesn't have the issue.

Looks like there's a related issue open against kubespray: https://github.com/kubernetes-incubator/kubespray/issues/2879

With #69 merged, Helm should now work, so this approach is now viable. However, this PR is significantly out of date wrt how we are currently deploying GCS components.


Trying to install AjaxControlToolkit Using an Old VS 2008 with .Net Framework 3.5

Hi,
I am trying to download AJAXControlToolkit from DevExpress web site. However, I am running Visual Studio 2008 Professional and I cannot upgrade this software.
When I try to run the setup wizard from DevExpress, tells me that Visual Studio could not be found. How can I install AjaxControlToolkit running my old Visual Studio 2008?
PS: I am running .NET Framework 3.5, as well. Thanks!

Hello
Since version 15.1, AJAX Control Toolkit minimum requirements are .NET 4.0 and VS 2010. For compatibility with .NET 3.5, use version 7.x from NuGet.


installer should have experimental option to force install

delete all the dlls that are conflicting type of thing. warn the user though

You could do that with a radar function to search for dlls and delete them and warn user. If a system warning comes up, it will abort.

You could do that with a radar function to search for dlls and delete them and warn user. If a system warning comes up, it will abort.

We only need to remove mega hack DLLs and a few others not every dll

But you also have stuff like gdmo and other free modding menus. It could list the folders and dlls that will be deleted and it can be modified


new subplugin - MarkerCluster.List

Hi again,
I made a new subplugin that allows displaying children layers in a table instead of "spiderfying" them.
it looks something like this:

here is the github repository
... and interactive demo
We use it within few projects where we try to focus on mobile devices and realised that the screen is not large enough to spiderfy all the children layers.
I created a pull request that adds this library to the list of subplugins in readme.md, nothing more.
Adam

Cool, thanks


Ubuntu common feature breaks on rootless podman on WSL

(reporting here because this is where features are hosted)
This was working until last week, I believe.

VSCode Version: 1.68.1 (user setup)
Local OS Version: Windows_NT x64 10.0.22000
Local chip architecture: x86_64
Reproduces in: Remote Containers
Name of Dev Container Definition with Issue: Ubuntu common feature

Steps to Reproduce:

Install WSL
Set podman as the executable path instead of docker.
Install Ubuntu (installed 22.04 from the Windows store GUI) and update it (sudo apt-get update && sudo apt-get -y upgrade)
Mount root as shared (ignore the error/warning from tee):

$ sudo tee -a /etc/wsl.conf >&- <<EOF
#Necessary to quiet a podman warning
[boot]
command="mount --make-rshared /"
EOF

Add necessary entries for XDG runtime directories:

$ cat << 'EOF' >> ~/.bashrc

if [[ -z "$XDG_RUNTIME_DIR" ]]; then
  export XDG_RUNTIME_DIR=/run/user/$UID
  if [[ ! -d "$XDG_RUNTIME_DIR" ]]; then
    export XDG_RUNTIME_DIR=/tmp/$USER-runtime
    if [[ ! -d "$XDG_RUNTIME_DIR" ]]; then
      mkdir -m 0700 "$XDG_RUNTIME_DIR"
    fi
  fi
fi
EOF

Restart WSL; From a Windows command prompt: wsl --shutdown

Install podman: sudo apt-get -y install podman

Create user-specific config to allow podman to run as rootless;

mkdir -p ~/.config/containers
cp /usr/share/containers/containers.conf ~/.config/containers/
sed -i 's|#cgroup_manager = "systemd"|cgroup_manager = "cgroupfs"|g' ~/.config/containers/containers.conf
sed -i 's|#events_logger = "journald"|events_logger = "file"|g' ~/.config/containers/containers.conf
sed -i 's|#\(log_driver = "k8s-file"\)|\1|g' ~/.config/containers/containers.conf

Create a temp repro and open in VSCode: mkdir -p ~/repositories/repro && code ~/repositories/repro/

Add dev container config files as appropriate (see attached - there are some podman-specific settings for rootless/inside-container users required)
repro.zip

Attempt to open in dev container, see logs.
remoteContainers-2022-06-16T23-12-06.157Z.log

Note that it's just the use of it as a feature - downloading the script manually during dockerfile build works just fine.
working.zip

cc @joshspicer for any thoughts as this seems to be feature related.


Either -> Maybe in Basics

We define Either but pattern-match on Maybe.
Changed the definition.

You are correct, the definition of Mayve is more suited.


jaeger collector sampling not working with Ambassador

I am using jaeger 1.18, with sampling strategy:
kind: Jaeger
metadata:
  name: jaeger
spec:
  strategy: production
  sampling:
    options:
      default_strategy:
        type: probabilistic
        param: 0.1

  storage:
    type: elasticsearch
    options:
      es:
        server-urls: http://elasticsearch-master.openfaas:9200

And using zipkin tracer in ambassador:
apiVersion: v1
kind: Service
metadata:
  name: zipkin
  annotations:
    getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind: TracingService
      name: tracing
      service: zipkin:9411
      driver: zipkin
  labels:
    app: jaeger
    app.kubernetes.io/name: jaeger
    app.kubernetes.io/component: zipkin
spec:
  ports:
    - name: jaeger-collector-zipkin
      port: 9411
      protocol: TCP
      targetPort: 9411
  selector:
    app.kubernetes.io/instance: jaeger
    app.kubernetes.io/component: collector
  type: ClusterIP

But sampling strategy just doesn't work, jaeger still keeps all the traces.
Any idea on how to fix that?  Thanks a lot!

@Space-Banana-42 The sampling configuration within the Jaeger operator's CR is only understood by the Jaeger tracers, not Zipkin tracers.


Kaldi version

Hello,
can you pls specify Kaldi version (e.g. by commit) which you have used?
I would like to try your solution but I do not want to run into problems.
Thank you
Ondra

Hi Ondra,
The Kaldi version I used is 489a1f5. I updated the README.txt as well.
Best,
MIlos


Seperate database/messagebroker modules to separate projects

Thought I'd try giving this one a crack #493, atm its just the baseline module migrations into their separate projects.
I had considered separating the test project into separate Databases/MessageBrokers/more generic Docker test projects, I wonder if thats inline with how the general structure of the test projects is envisioned going forward.
Todo:

[x] Separate modules into individual projects
[ ] Split tests into multiple test projects and add common test structure
[ ] Add/Update Cake task to deploy modules as Nuget dependencies

Thanks for your PR. Like mentioned in the main contributing guidelines it is better to discuss changes with the authors first, especially large changes.
#493 is not just about moving the current implementation into dedicated packages. The builder pattern has major flaws in regard to creating module specific container instances. It is necessary to fix this design flaw first. Afterwards we can start to migrate module after module incl. their tests. The branch feature-disabled/493-add-example-module contains the first proposal. If I remember correct merging the configuration is left (incl. adding the removed classes etc. again).
The first increment should update the builder pattern without breaking existing modules. Legacy modules are kept as long as the new modules are deployed due to breaking changes / compatibility.

Might've jumped the gun a bit here :(. Yeah as you say looking through that example module branch it seems to be quite a big deviation away from the existing builder pattern and test container inheritance/implementation - Im happy to close this PR

I'll add more information to #493 about the necessary changes next week. I appreciate your efforts, but as you said the deviation is quite big. Nevertheless, feedback about the current proposal would be great. If you would like to contribute, it makes probably sense to continue to work on the proposal (with pleasure together).


upgrade to angular 7 with ng update @angular/cli@7 @angular/core@7

I've upgrade to angular 7 using angular cli upgrade command. Can you release it on version 3.0.0?
I plan to upgrade to angular 8 with version 4.0.0.
then I can do upgrade to angular 9 with version 5.0.0.


Micro800 Controller.scan() Error

Current Behavior
I am now having some issues now with the subscription/scan methods. The following works and I can verify it is setting the Tag value in Connected Components Workbench, but when I uncomment the PLC.scan() method, with or without the other writeTag code I get an unhandled rejection error
Code:
const { Controller, Tag, EthernetIP } = require("st-ethernet-ip");
const { BOOL } = EthernetIP.CIP.DataTypes.Types;

const PLC = new Controller();

let tag1 = new Tag("TEST_TAG_1", null, BOOL);
PLC.subscribe(tag1);

PLC.connect("<IP_ADDRESS>", Buffer.from([]))
  .then(async () => {
    tag1.value = true;
    PLC.writeTag(tag1);

    // PLC.scan_rate = 50;
    // PLC.scan();
  })
  .catch((err) => {
    console.log(err);
  });

Error:
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "#<Object>".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Any thoughts what might be causing that?

I tried your code on a non Micro800 without a problem.
For the PLC.scan()
Try:
PLC.scan().catch(e => console.log(e))

To catch the scan error

{ generalStatusCode: 8, extendedStatus: [] }

That is a service not supported error.  The scan function uses MULTIPLE_SERVICE_PACKET service to send multiple read or write requests at the same time. It appears that the Micro800 doesn't support this service.

@SerafinTech What CompactLogix model(s) are you using to develop this library? We've also got a Micro820 that we'd be willing to send over to see if we can get some sort of scan() equivalent supported on that platform.

@SerafinTech if the library can read/write to the micro800 no problem, do you see any issues to just put a read inside a polling loop and roll your own "scanner" via hitting the PLC say every second?  Is that essentially what the library is doing at its core anyway?  Polling is usually less than ideal.  Also, I'd be curious to know which PLC you are using and have tested against.  Perhaps a compatibility chart and feature matrix would be a nice addition.  Let me know if youd like to get one going I am happy to help in any way I can.

@TeeLusk I currently develop and test against a CompactLogix 1769-L32E Firmware 20.14 . I wish I had a bunch of controllers to test against.  ControlLogix and CompactLogix "should" all be compatible with the library. I would be happy to create a scan function for the Micro820 if you sent me the controller, but I couldn't promise a timeline for completing.  It shouldn't be long, but you never know what problems come up.

@anthonywebb You can do polling which is what the scan function basically does, but you have to wait for the read or write to complete before you initiate another read or write and catching errors can be tricky.

In general I am finding that maybe the micro800 just isn’t cut out for scanning, which is a shame because it is a great entry point into PLC.


commands/operator-sdk/cmd/local: add namespace flag

ref: #273
This adds the --namespace flag to the up local command and the default value is the "default" namespace.
Example:
operator-sdk up local --namespace operator

Help output
$ operator-sdk up local --help
The operator-sdk up local command launches the operator on the local machine
by building the operator binary with the ability to access a
kubernetes cluster using a kubeconfig file.

Usage:
  operator-sdk up local [flags]

Flags:
  -h, --help                    help for local
      --kubeconfig string       The file path to kubernetes configuration file; defaults to $HOME/.kube/config
      --namespace string        The namespace where the operator watches for changes. (default "default")
      --operator-flags string   The flags that the operator needs. Example: "--flag1 value1 --flag2=value2"

Could you also update the https://github.com/operator-framework/operator-sdk/blob/master/CHANGELOG.md to include this change?
Just modify Added operator-sdk up command ... to incorporate this change.

I've applied the changes please take a look.

lgtm after nits


Recursive structs cause a stack overflow

hi! trying to generate bindings for a recursive struct with tauri_specta causes a stack overflow:
#[derive(serde::Serialize, specta::Type)]
struct B { a: Option<Box<A>> }

#[derive(serde::Serialize, specta::Type)]
struct A { b: B }

thread 'main' has overflowed its stack

here's an MRE. just run:
git clone https://github.com/cAttte/tauri-specta-overflow && cd tauri-specta-overflow && npm i && npm run tauri dev

given that recursive types are fine in TypeScript, this should be no problem. but even if this was unsupported behavior, tauri_specta should detect the recursion and fail intentionally!

Yeah stack overflows are not okay and this should be supported. Will give this a look when I have a moment.

Also gonna move this to to the Specta repo as it’s gonna be a problem with it.


Validation regex not available

Hi,
I tried to set a regex validation on a field as per documentation. But I think either code or documentation is erronous on that matter.
There is a regex validation available on fields as per this ref.
https://github.com/sethsandaru/vue-form-builder/blob/b16f3e2d4436603c7f3089f3e8d45e3d24c50f4e/src/configs/validation.js#L40-L47
But because there is no corresponding branch here it throws an error.
https://github.com/sethsandaru/vue-form-builder/blob/b16f3e2d4436603c7f3089f3e8d45e3d24c50f4e/src/libraries/validation.js#L94-L115
like Error in event handler for "renderer.run_validation": "TypeError: This validation type regex is not supported

Copy of #78

Copy of #78

Thanks, v2.1.0 is released and added the regex validation. Documentation coming soon!


Doesn't start then breaks Zellij

I get this when I first start the plugin:

Then nothing happens. If I try to relaunch the plugin it doesn't output anything and zellij goes completely unresponsive including the cli so I have to kill the processes.
I am on Linux and zellij v0.37.2. If you could point me to logs I can provide more information.

Could be... do you get a stacktrace or some other error?

No other errors or outputs, just freezes

So other panes still work? How did you load it?

Please make it open on the CWD of the current pane. That would make the entire thing so much more useful!

That's the plan!

I'm facing the same behavior. In my case took around 10 seconds then it good to go. Only if I spawn a new terminal and zellij again it has to repeat the process.

@imsnif I'm wondering if it makes sense to create plugin disk caches beneath the top-level of the zellij plugin directory. This way immediate disk entries could be retrieved and then correct/update after an initial thread renders the last recorded positions.


New Joe Pools for week of Feb 06 2022

Added AVAX-GMX and AVAX-DOMI strats
Moved _takeFeeRewardToSnob() and _swapBaseToToken() to joe-rush-farm-base.sol

Last comment was Looks Good to Me from @AbigailTCameron . Is there anything additional happening with this @zeroecksYeti ?

Last comment was Looks Good to Me from @AbigailTCameron . Is there anything additional happening with this @zeroecksYeti ?

Was good to go until new Joe pools dropped today, I've added them to this PR


[DUOS-1557][risk=low] multi dataset voting grouped dataset vote component

Link to Jira ticket
Summary of Changes:
Component for DAC Members to cast data access votes and Chairs to view vote information about their DAC.
Rules for expected behavior:
Data Use Pills:

list of data uses supplied by the bucket itself

Vote Box:

Given the votes within this bucket by the current user
If the vote value (true/false) all match, appropriate vote button is preselected; otherwise, none selected
If the rationale value among the votes all match, the rationale text area is prefilled; otherwise, filler text visible
If some of the elections corresponding to the votes are closed, vote box is disabled
Chairs can only vote once

Pie Chart:

Only visible to chairs
Displays results from votes within the current user's DAC only
Uses the raw counts of the votes (no collapsing); for example, if there are two members voting on 3 elections, there will be 6 votes fed to the pie chart

Vote Summary Table:

Only visible to chairs
Displays results from votes within the current user's DAC only
Collapses votes if possible: if the same user has the same vote value (true/false) for multiple votes, the vote will only be represented in the table once. If only the date or rationale differs, append date and rationale values together in the same row of the table. If the vote value differs, represent with separate rows in the table.

Datasets Requested:

Displays only the datasets voted on by the DAC of the current user
Determined by providing a list of all datasetIds the DAC has access to, a list of the datasetIds in this bucket, and a list of the datasets in the collection

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

TESING:
I tested this component by adding it to DarCollectionReview.js, whose page can be accessed via http://localhost:3000/dar_collection/{#}. This component is intended to take in the title of the slab, a bucket with DataAccess votes and elections, a list of datasets  within this collection, a list of datasetIds that the DAC of the current user has access to, and a boolean isChair.
In DarCollectionReview.js, buckets are already produced by
 const processedBuckets = await flow([
          generatePreProcessedBucketData,
          processDataUseBuckets,
  ])({ dars, datasets });

To help with testing different scenarios, while coding I created my own examples of the props above:
The bucket:
const [bucket, setBucket] = useState({
    elections: [
      [
        {dataSetId: 385, electionId: 101, status: 'Open', electionType: 'DataAccess'},
        {electionType: 'RP Vote'}
      ],
      [
        {dataSetId: 415, electionId: 102, status: 'Closed', electionType: 'DataAccess'},
        {electionType: 'RP Vote'}
      ],
      [
        {dataSetId: 103, electionId: 103, status: 'Closed', electionType: 'DataAccess'},
        {electionType: 'RP Vote'}
      ]
    ],
   votes: [
      {
        dataAccess: {
          memberVotes: [
            {dacUserId: 5060, displayName: 'Shae', rationale: 'test1', electionId: 101, voteId: 1, createDate: 1},
            {dacUserId: 5007, displayName: 'Grace',  vote: false, rationale: 'test1', electionId: 101, voteId: 2, createDate: 1},
            {dacUserId: 3365, displayName: 'Julian', vote: true, electionId: 101, voteId: 3, createDate: 1}
          ]
        }
      },
      {
        dataAccess: {
          memberVotes: [
            {dacUserId: 5060, displayName: 'Shae', rationale: 'test2', electionId: 102, voteId: 5, createDate: 2},
            {dacUserId: 5007, displayName: 'Grace',  vote: false, rationale: 'test1', electionId: 102, voteId: 4, createDate: 1},
            {dacUserId: 3365, displayName: 'Julian', vote: false, electionId: 102, voteId: 6}
          ]
        }
      },
      {
        dataAccess: {
          memberVotes: [
            {dacUserId: 5007, displayName: 'Grace', vote: false, electionId: 103, voteId: 7},
            {dacUserId: 3365, displayName: 'Julian', vote: true, rationale: 'test3', electionId: 103, voteId: 8}
          ]
        }
      },
    ],
    dataUses: [
      {code: 'GRU', description: 'Use is permitted for any research purpose'},
      {code: 'HMB', description: 'Use is permitted for a health, medical, or biomedical research purpose'}
    ]
  });

In the return statement of the component:
 h(MultiDatasetVoteSlab, {
        title: 'GROUP 1',
        bucket,
        dacDatasetIds: [
          385, 415
        ],
        collectionDatasets: [
          {dataSetId: 385, name: 'Dataset 1', datasetIdentifier: 'DUOS-1'},
          {dataSetId: 415, name: 'Dataset 2', datasetIdentifier: 'DUOS-2'},
        ],
        isChair: true
 })

And then I navigated to any http://localhost:3000/dar_collection/ page. If you use this example, replace my name and dacUserId with your own.

Small bugs I have noticed:

When multiple rationales are appended together in the vote summary table, although I add \n between different rationales, it is just displayed as a space
Currently createDate for votes is a number that is converted into the proper date format within VoteSummaryTable.js, so appending multiple dates together doesn't work properly. VoteSummaryTable.js currently isn't used anywhere else, so one possible solution is to do the formatting in MultiDatasetVoteSlab.js then append the formatted dates together and pass them off as a string to VoteSummaryTable.js. However, this will contradict the typical structure of votes

Note: II would also be interested if there is any refactoring that can be done on this component to make it easier to understand.

I think HTML ignores newline characters by default unless changed via CSS. You may need to use something like white-space: pre-line to handle it.
Here's a link to the CSS documentation: https://developer.mozilla.org/en-US/docs/Web/CSS/white-space
As for the second point, refactoring is always okay so as long as you ensure that existing functionality doesn't break. Your approach makes sense, especially since the slab is the source of information for the sub-components.
As for vote structure on the table itself, it's more flexible than you would think. Things like id on VoteSummaryTable doesn't mean it has to be equal to a specific voteId. It's used to assign unique identifiers to DOM elements to allow React to re-render elements selectively and correctly. So you can make that value whatever you want so as long as it isn't shared by any other element. The same can be said of data and label, they're just strings.
 Focus on making it a JS object that has the data you need to render it correctly on a table.

Recent change:  I made the Chairperson view of this component use Final votes instead of Member votes for the voting buttons


detect messenger browser

Is there any chance to detect messenger browser?
userAgent:
"userAgent":"Mozilla/5.0 (iPad; CPU OS 9_1 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Mobile/13B143 [FBAN/MessengerForiOS;FBAV/<IP_ADDRESS>.47;FBBV/17291106;FBDV/iPad2,5;FBMD/iPad;FBSN/iPhone OS;FBSV/9.1;FBSS/1; FBCR/;FBID/tablet;FBLC/cs_CZ;FBOP/1]"

Looks like you can add an entry for it and use 'FBAN/Messenger' in the
regex. Hopefully this will work to devices without IOS too.
If you do that - please remember to add a test for any such user agent use
cases.
10x.
On Mon, Nov 16, 2015 at 10:57 AM Jakub Křižka<EMAIL_ADDRESS>wrote:

Is there any chance to detect messenger browser?
userAgent:
"userAgent":"Mozilla/5.0 (iPad; CPU OS 9_1 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Mobile/13B143 [FBAN/MessengerForiOS;FBAV/<IP_ADDRESS>.47;FBBV/17291106;FBDV/iPad2,5;FBMD/iPad;FBSN/iPhone OS;FBSV/9.1;FBSS/1; FBCR/;FBID/tablet;FBLC/cs_CZ;FBOP/1]"
—
Reply to this email directly or view it on GitHub
https://github.com/srfrnk/ng-device-detector/issues/39.


Feat: 26 - Add About page

Is your feature request related to a problem? Please describe
A landing page to make the user feel welcomed to site
Describe the Solution You'd Like
An attractive landing page using animation like colours changing automatically to temp user
Describe Alternatives You've Considered
N/A
Additional Context
N/A

Hi @RNAdvani
Thanks for this suggestion.
Unfortunately, I'm not convinced about this yet. When the user opens color-conjure, they should directly land on the thing to do, without navigating through multiple pages.
I guess we could have an About page though, which can have some details about the project, probably a copy and paste of README.
Would you be willing to do that?

Hi @RNAdvani Thanks for this suggestion.
Unfortunately, I'm not convinced about this yet. When the user opens color-conjure, they should directly land on the thing to do, without navigating through multiple pages.
I guess we could have an About page though, which can have some details about the project, probably a copy and paste of README.
Would you be willing to do that?

i would like to work on the About page

@utk09-NCL  i would be willing to work on the About page, lets have a discussion about this

yes i could do that now my work on the header is done as of now but the reason i was saying a landing page would be good it improves user experience better if you wanna you can check out this it's just it improves ux nothing more

This is good @RNAdvani But I don't think we have so many things right now 😅. Coolors has Apps, Plugins, etc. Maybe something in future!

okay so for now lets add about page, we could use our readme for now

@utk09-NCL

is this looking good?

and just have to add about link to the header and update test for it,
anything more @utk09-NCL ?

@RNAdvani if you need any help implementing the design, I would be interested in assisting

Looks good @RNAdvani
Go for it!

@RNAdvani if you need any help implementing this feature, I would be interested in assisting

thanks @obi-sammy, I've completed designing


How to submit my model to the leaderboard?

Hey Waneila,
If you have tested your models on the datasets, please share the model id on huggingface then we can probably help do the inference. If the model is too large or private, then you may create a PR to submit your results.

Hi yuchen,
Are there any restrictions for models on this ranking list? Are there any models that are not allowed to appear on the ranking?

Hello yuchenlin. Can you evalute my model? Id: OnlyCheeini/greesychat-turbo

Hi yuchen,
Are there any restrictions for models on this ranking list? Are there any models that are not allowed to appear on the ranking?

we have no particular restrictions

Hello yuchenlin. Can you evalute my model? Id: OnlyCheeini/greesychat-turbo

sure will do

Hello yuchenlin. Can you evalute my model? Id: OnlyCheeini/greesychat-turbo

sure will do

Can u notify me after you fully evaluate the model? And thanks 👍

Please test this model:
EpistemeAI/Fireball-Meta-Llama-3.1-8B-Instruct-Agent-0.003-128K-code

Hey @yuchenlin. Can you evaluated my model?

Could you please send evaluate my model? https://huggingface.co/EpistemeAI/Fireball-Meta-Llama-3.1-8B-Instruct-Agent-0.003-128K-code-ds


Plugin broken?

It seems like sometime along the past few days the plugin broke for me on Linux (maybe with 1.0.5 and the new release of Obsidian?) and I can't quite figure out why;
Thing tried:

Open the help vault, no other plugins installed or enabled
Install and enable Budget WYSIWYG
After a few seconds, plugin switches to preview, but then never switches back to edit after any type of keystrokes

Experiencing this with v 1.0.5 and Obsidian v0.12.15 on multiple Linux Mint machines - tried on empty/no other plugins help vault and my personal vault, both same thing.
I do get a possible error message on the Console, but I think it might be a red herring and not related:

GET https://github.com/OwnJoke/obsidian-budget-wysiwyg/releases/download/1.0.5/styles.css 404
Budget WYSIWYG: styles.css not found

Anyone else experiencing this?

I just downloaded on linux and am having the same issue. Was wondering if you were using vim mode when it was working, as my first assumption was that may be the cause, but if it was working on vim mode before I guess it may just be a linux issue.
Arch btw

Was indeed using vim mode + some related vim plugins. I changed my workflow to use CodeMirror Options so never went back to try to debug this.

Sorry for the late reply, the plugin is broken and I'm currently pretty busy, I might eventually try and fix it but for now my only option is going back to code from an older version with some other bugs (which I might do). Sorry I haven't updated the plugin or written about this anywhere until now.


Localize Kubernetes website into Farsi

This is a Feature Request
Initial minimum-viable Farsi localization.
Hope to add the first live version of Farsi localization for k8s.io
This issue is to (re)initiate Farsi localization efforts based on following actionable items.
Actionable items identified from
https://kubernetes.io/docs/contribute/localization/#start-a-new-localization

[x]  1. Select the language code for Farsi

ref:https://kubernetes.io/docs/contribute/localization/#start-a-new-localization

the language code for Farsi: fa

[x]  2. Add a slack channel for Farsi

https://kubernetes.io/docs/contribute/localization/#find-community

Slack channel for Farsi : #kubernetes-docs-fa

[x] 3. Add a label for Farsi

https://kubernetes.io/docs/contribute/localization/#configure-the-workflow

label for Farsi: language/fa

[x]  4. Localize the community code of conduct

https://kubernetes.io/docs/contribute/localization/#localize-the-community-code-of-conduct

code of conduct: https://github.com/cncf/foundation/blob/main/code-of-conduct-languages/fa.md

[x]  5. Invite candidates for @kubernetes/sig-docs-fa-owners and @kubernetes/sig-docs-fa-reviews

https://kubernetes.io/docs/contribute/localization/#add-your-localization-team-in-github

Note: Seeking for Interested Contributors In Farsi localization

current reviewers: @SamanKhalife

current approvers: @SamanKhalife

current team members: @SamanKhalife

[ ]  6. Check the all candidates are the member of the Kubernetes GitHub organization

https://kubernetes.io/docs/contribute/localization/#join-the-kubernetes-github-organization

[ ]  7. Add a development branch for Farsi localization

https://kubernetes.io/docs/contribute/localization/#source-files

https://kubernetes.io/docs/contribute/localization/#branch-strategy

Note: members of @kubernetes/website-maintainers can create new localization branches to coordinate translation efforts.

Note: Netlify preview needs to be configured for the dev branch

[ ]  8. Open a PR to configure a localization environment

Note: base and target branches of the PR should be the dev branch of FA.

[ ] Add a configuration block for the new language to hugo.toml
https://kubernetes.io/docs/contribute/localization/#modify-the-site-configuration
[ ] Add a new localization directory
https://kubernetes.io/docs/contribute/localization/#add-a-new-localization-directory
[ ] Add and localize site strings in i18n
https://kubernetes.io/docs/contribute/localization/#site-strings-in-i18n
[ ] Set up the OWNERS files based on the candidate list
https://kubernetes.io/docs/contribute/localization/#set-up-the-owners-files
[ ] Check build test and site-preview of the PR

[ ]  9. Open PRs to localize all minimum-required-contents and a localized README file

https://kubernetes.io/docs/contribute/localization/#minimum-required-content

https://kubernetes.io/docs/contribute/localization/#add-a-localized-readme-file

Note: base and target branches of the PR should be the dev branch of FA.

[ ]  10. Open a PR to merge the dev branch into

https://kubernetes.io/docs/contribute/localization/#launch-your-new-localization

[ ] Check build test and site-preview of the PR

/language fa

/area localization

Invite candidates for @kubernetes/sig-docs-ar-owners and @kubernetes/sig-docs-ar-reviews

This doesn't sound right. Did you mean @kubernetes/sig-docs-fa-owners and @kubernetes/sig-docs-fa-reviews?

Hi @sftim,
Thank you very much for your feedback.
I've edited the issue to correctly refer to the appropriate groups. Now it specifies @kubernetes/sig-docs-fa-owners and @kubernetes/sig-docs-fa-reviews for the localization of the Kubernetes website into Farsi.

Hi @SamanKhalife,
I believe https://github.com/kubernetes/website/pull/22565 is the first attempt at Farsi localization (https://github.com/kubernetes/website/tree/dev-1.18-fa.1).
It seems the initial attempt was not successfully completed at that time, and the Slack channel #kubernetes-docs-fa has been inactive.
We need to recruit enough contributing members to make it active again! :)
Could you help bring more contributors who are interested and willing to devote their efforts to this project?

Let me mark this issue as triage accepted to keep it open. We're still looking forward to more participation. :)
/triage accepted

/remove-lifecycle stale


Min distance b/w two given nodes of a Binary Tree

Find the distance between two keys in a binary tree, no parent pointers are given. The distance between two nodes is the minimum number of edges to be traversed to reach one node from another.

congratulations on your hactoberfest contribution. if you like this repo then give it a star. Happy hacking..


Frontend Performance.

Issue description:
Hi aristath,
I have a problem with performance in Front-end. I think many people faced with this problem, too.
Could you tell me properly hook to register Kirki fields?
If i use hook "init" to register fields, then front-end is very slow when you have many fields that used query from database.
For eg:
add_action( 'init', 'load_customizer_fields' );
Then i have some select fields with categories (post or anything query from database):
$categories_list   = get_terms( array( 'taxonomy'   => 'category', ) );
Kirki::add_field( 'theme', array(
    'type'        => 'select',
    'settings'    => 'post_categories',
    'label'       => esc_html__( 'Filter By Cats', 'text-domain' ),
    'section'     => 'blog_settings',
    'priority'    => 1,
    'multiple'    => 1000,
    'choices'     => $categories_list,
) );

This is okay in Customize preview but it will run these queries in all front-end pages & admin pages, too.
If i use hook 'customize_register'. Then all fields with 'output' is not working.
For eg:
add_action( 'customize_register', 'load_customizer_fields' );
...
Kirki::add_field( 'theme', array(
	'type'        => 'color',
	'settings'    => 'body_color',
	'label'       => esc_html__( 'Body Color', 'text-domain' ),
	'section'     => 'color_settings',
	'priority'    => 2,
	'transport'   => 'auto',
	'default'     => '#ff0000',
	'output'      => array(
		array(
			'element'  => 'body',
			'property' => 'color',
		),
	)
) );

Is there any way to make Field Output working properly with 'customize_register' hook
Using theme_mods or options?
theme_mod
Code to reproduce the issue (config + field(s))

Hmmmm if you have some heavy queries then you should be able to do something like this:
global $wp_customize;
Kirki::add_field( 'theme', array(
    'type'        => 'select',
    'settings'    => 'post_categories',
    'label'       => esc_html__( 'Filter By Cats', 'text-domain' ),
    'section'     => 'blog_settings',
    'priority'    => 1,
    'multiple'    => 1000,
    'choices'     => $wp_customize ? $categories_list : array()
) );

That way if you're on the customizer the queries run, and in the frontend they don't.


Add commit hash text, hidden on the bottom of the tabs

Description
This fixes #984
Adds hidden text at the bottom of the tabs, showing the commit hash of the code deployed.
How to test the feature:

navigate to the bottom eg. layers tab
highlight the hidden text to see the commit hash.
confirm it is the same as the hash of the code of this deployment

Checklist - did you ...
Test your changes with

[ ] REACT_APP_COUNTRY=rbd yarn start
[ ] REACT_APP_COUNTRY=cambodia yarn start
[ ] REACT_APP_COUNTRY=mozambique yarn start
[ ] Add / update necessary tests?
[ ] Add / update outdated documentation?

Screenshot/video of feature:

Great start

I think the text can be smaller and we can just show the short hash (8 first characters)
Let's call it "version hash", @wadhwamatic does that work for you?
Let's make it clickable to open up GitHub as discussed