SHARK tests are not deterministic

I found that the SHARK tests are not deterministic. The inputs used for tests are different between runs. We should have fixed input data for the integration tests.

@hanhanW please use the blank issue for filing generic issues. If you forget, please clean up all the boilerplate :-)

yes lets track this. @monorimet can you please take a look. And subscribe to the IREE github so I can assign it to you.

Inputs for PyTorch tests should be the same for each test case on one model. Do we want inputs to be the same across models as well (where applicable), or is this an issue with TensorFlow tests specifically?
If we're talking about TensorFlow, those tests will become deterministic once the tests are reworked to use .mlir and inputs downloaded from gs://shark_tank/<modelname>/ (like we have for pytorch models). I'm not sure how long that will take us (I would expect it done by the end of the week), but I can put up a fix for the meantime.

MLIR text isn't a stable format, so I'd be careful with that. We've had issues in the past where that sort of thing not stored in source is painful to update with syntax changes

We have a script to generate all of that data locally as well -- is it sufficient that we are able to easily generate and upload to gs://shark_tank a new set of compiler inputs for any SHARK-supported model if the need arises (i.e., upstream changes occur that would change the MLIR)?

Also, the TensorFlow inputs should be the same across runs (they're generated from a string that's hardcoded here)
@hanhanW I will take a look at tests for all frontends but please let me know if you're encountering this issue with a specific model/frontend.

I observed it when running LayoutLMModuleTest. I still observed the same issue. Here is how I reproduce the issue:
# update SHARK repo -- attach at 3c2f8ef24341b0476944cb24d7fd79154fd22a90
$ git pull
# create a new venv
$ VENV_DIR=0721_venv IMPORTER=1 USE_IREE=1 ./setup_venv.sh
$ source 0721_venv/bin/activate

# Run two times
$ pytest --save_temps tank/tf/hf_masked_lm/layoutlm-base-uncased_tf_test.py::LayoutLMModuleTest::test_module_static_cpu
$ pytest --save_temps tank/tf/hf_masked_lm/layoutlm-base-uncased_tf_test.py::LayoutLMModuleTest::test_module_static_cpu

There will be two directories under shark_tmp. If we look into the expected_out.txt, we'll find that they are different. E.g.,
❯ head -n1 iree_tfsmbr9krai/expected_out.txt
[[[ 1.95667243e+00  1.97509766e+00  1.49286246e+00  7.99876928e-01

❯ head -n1 iree_tfst9ufel26/expected_out.txt
[[[-5.47550440e-01 -1.52990317e+00 -1.70632601e-01 -6.93688989e-01

This definitely suggests that there's something wrong with how we're getting the golden values (expected output), which adds up with the issues we were having with numerics earlier -- can you check the function inputs to see if they are the same? diff -s iree_tfsmbr9krai/input1.txt iree_tfst9ufel26/input1.txt should do the trick.
I'll bring this up in our sync today to make sure we have a plan on fixing those -- ideally by the end of the week we'll have our TensorFlow tests operating as intended.

I was able to reproduce this issue for LayoutLM. However, the expected outputs are consistent on some other models -- for example, the expected outputs between runs on bert-base-uncased from Tensorflow are identical (they still fail to validate against SHARK results, so there's probably another problem with LayoutLM).
I'm working on this as a first priority, and will update as soon as progress is made. Thanks for the help with repro.

We have a script to generate all of that data locally as well -- is it sufficient that we are able to easily generate and upload to gs://shark_tank a new set of compiler inputs for any SHARK-supported model if the need arises (i.e., upstream changes occur that would change the MLIR)?

If we're running something that reports commits as failed, especially if it happens on presubmit, we need to have something that can be updated via source code changes. It is otherwise impossible to test a commit with the new MLIR file without breaking tests that relied on the old MLIR file (from previous commits). It also makes the test a function of more than just the source code. Occasional network flakes and such are manageable (though I'd really like to isolate them into an initialization step that clearly doesn't report as a test failure), but something that can be updated out from under is isn't. In particular, I really don't want to put more burden on the LLVM integration process, which is already painful. That's the case where we're most likely to run into the need for updates. That said, we obviously can't be storing big input files in git. So I think order of preference would be:

A model expressible as a reasonable amount of source code (e.g. Python source)
A stable serialization format that we fetch from GCS (e.g. TFLite flatbuffers)
An unstable serialization format (e.g. MLIR files) stored in GCS and indexed by version in some way. The simplest indexing scheme I think would be a hash digest of the file itself, but something like the commit it was built from, or even just a monotonically increasing version number would be fine too. It is then possible to add a new version of the file to the storage without breaking anything and to update the source to point at this new version as a change to the source code. I still don't want that to be something that weighs heavily on the shoulders of integrators though, so updating needs to be straightforward. That means there's a quick skip they can run to point at "latest" versions or there's only one place to update and the thing to update it to is somewhat meaningful (like an LLVM commit hash iff the files are dependent solely on LLVM commit). The scripting could also create the latest version, if that's something can be done quickly. Otherwise, there would need to be some separate process to create the latest versions at least as often as needed so that they were already there when the person integrating LLVM needed them.

@GMNGeoffrey Thank you for the careful explanation. For IREE's use cases and especially testing purposes I see how this would make the LLVM integration harder to maintain, and would like to address this with our team to decide how to navigate this issue.
@hanhanW I have a pretty important update that might help diagnose the numerics issues for all our TF hf_masked_lm model tests. I noticed that even during CPU cases, TensorFlow was utilizing the A100 GPU to generate the model, inputs, and expected output for our tests.
If we establish os.environ["CUDA_VISIBLE_DEVICES"] = "-1" before generating the huggingface masked_lm model (to hide the A100 from TensorFlow), all the static_cpu tests -- including LayoutLM -- passed, with our results within tolerance of the TF golden results.
Thus, there's probably some interaction between TensorFlow and CUDA (or the A100 specifically, since we don't have this issue without it) that introduces the numeric discrepancies in the results.
Is it also a possibility that certain optimizations happen in TensorFlow with the A100 that are not accurately reproduced in IREE?
I will keep investigating, but please let me know in the meantime if you have any insights based off of this update.

I noticed that even during CPU cases, TensorFlow was utilizing the A100 GPU to generate the model, inputs, and expected output for our tests.
If we establish os.environ["CUDA_VISIBLE_DEVICES"] = "-1" before generating the huggingface masked_lm model (to hide the A100 from TensorFlow), all the static_cpu tests -- including LayoutLM -- passed, with our results within tolerance of the TF golden results.

Ah the classic problem that TensorFlow is always trying to do something with Cuda, no matter what you actually asked for

SG, thanks for the investigations and detailed context! They might have some reasons for doing it, but I'd suggest to disable those fancy optimization at this moment. It should be an option to us, not always turn it on by default.

There are a few remaining problems with the masked_lm model tests that we are tracking in the SHARK issues, but this issue should be resolved by https://github.com/nod-ai/SHARK/pull/199.

From today's meeting: @powderluv checking to see if this is resolved.


Immediate reject execution

Trying to understand why sometimes calling reject() suddenly causes immediate execution of the reject chain.
Example:
function Test() {
    return new Promise(function (resolve, reject) {
        console.log("step-1");
        reject();
        console.log("step-2");
    });
}

Test()
    .then(function () {
    }, function () {
        console.log("step-3");
    });

This is always expected to produce the following output:
step-1
step-2
step-3

However, when under heavy load and with lots of things running in parallel I suddenly get the following logged:
step-1
step-3
step-2

which indicates that under some special circumstances that are hard to reproduce, calling reject() results in immediate execution of the reject chain.
How is this even possible?

I don't know what could cause this.  If you can provide a failing test case (e.g. as a pull request) we can try and investigate this.  It shouldn't be possible.

As I stated above, this is very difficult to reproduce, but over time the anomaly appears to be consistent.
I'm not sure I will be able to create such a failing test. The only thing seems feasible right now is to analyse implementation of the reject call to see if there is any scenario when a direct execution may occur. Also, I will try to switch to a different promise library to see if that makes a difference.

It always goes through asap, so the only thing I can see that would cause this would be a bug in asap.  There were some bugs in some older versions of asap, so check your up to date.

I've been using the very latest version of Promise, which presumes an up-to-date version of the asap also. This very well may be that asap faults under a heavy load and executes a reject out of order.

Closing it, as I cannot prove that this is the case. Maybe my code is broken elsewhere.


Support Service Principal Certificate Login

So far it only allows Service Principal login with password.
However, one of our biggest customer wants to use certificate for security reason.
Can you add this feature on?

Hi @MS-HerbChung
We currently recommend to use ADAL for Python to authenticate and to use our ADAL wrapper in the SDK. ADAL is maintained by the Active Directory team and will be always on top of the current authentication mechanism in Azure.
I think your feature is covered by ADAL. If you have an issue to authenticate with your scenario, please open an issue on their repository with your need.
I close this issue, since either the feature is covered by ADAL, either it's not and as to be added in the ADAL lib. Feel free to still comment here if I miss a point and I will re-open the issue in that case.
Thank you,


[TF2XLA] Support ResizeNearestNeighborGrad

This PR adds support for ResizeNearestNeighborGrad similar to how ResizeBilinearGrad is implemented.
Fixes #57575

Thanks a lot! @changm could you take a look?

Hi @Princejain1101, Can you please take a look on this PR? Thank you!

Hi @Princejain1101, Can you please take a look on this PR? Thank you!

Sorry for the long delay here, this mostly looks OK. Please fix the other reviewers comments and ensure the tests pass. Thanks!

Hi @lgeiger Can you please check @changm's comments and keep us posted ? Thank you!

Sorry for the delay, I'm trying to run this internally to ensure tests pass however a lot of tests fail. I'm not exactly sure how external PRs work versus internal CI/CD runs, but can you get external tests to pass?
I'm also getting errors such as :
10135 xla_op_registry.cc:105] Registrations of ResizeNearestNeighborGrad have incompatible compile time constant inputs.
xla_op_registry.cc:606] XLA op registration ResizeNearestNeighborGrad is incompatible with existing registration of the same name.
*** Check failure stack trace: ***

absl::log_internal::LogMessageFatal::~LogMessageFatal()
tensorflow::XlaOpRegistrar::XlaOpRegistrar()
 _GLOBAL__sub_I_image_resize_ops.cc
 __libc_csu_init
__libc_start_main
 _start

Which sounds like you need to mark the op as having compile time constant inputs.

Hi @lgeiger Can you please check @changm's comments and keep us posted? Thank you!

Any chance this PR will be finished and merged?

From March 7th, still waiting on reply from the CL author.


[INFRA] Simplify the publishing process to CRAN

Prerequisites: #10
Currently (v0.2.0), the release process explains that the source archive that is submitted to CRAN has to be built locally. This is a pain and error prone. So this should be automated instead
We could

create a GH worfkow triggered on tag creation (see also #47) and workflow_dispatch (for manual rerun on issue)
or reuse the existing "build" workflow and allow to run it with workflow_dispatch (for manual run)

Decision: add workflow_dispatch to the existing build workflow


"Page not found" when clicking on my name

I apologize in advance if this isn't the correct venue for this but when I click on my name in the menu I receive an error that the page isn't found.  The URL shows up as https://joind.in/user/1.0155671600264E+16

Where is this linked from ?

I can duplicate it from https://joind.in/event/midwest-php-2018 and https://joind.in/event/midwest-php-2018/community

Did you sign in with Facebook ?

Think I know the fix. I will be able to look at next week. Moving house ATM

Did you sign in with Facebook ?

Yes

Think I know the fix. I will be able to look at next week. Moving house ATM

No rush I'm not going to be using the site for a while.  :-)

Thanks for creating this BTW.  It was a great resource at the conference.

Found and fixed the issue. I don't have internet hooked up so will submit a pr when I can. It's in a few places.


Live editing

Storing scene information in a text editor so that the runtime can be used to modify properties. This is a core aspect of my philosophy that "the game is the editor".
By enabling some hidden flag [ ~ ] or a command line argument, for instance, the game opens in an editable mode. Using the first person controller the game designer can see how the game looks from the player's perspective while editing the game. They are free to toggle Noclip to get to the hard to reach places. (This is inspired by the Portal 1 mod: map editor, and it's similar to Minecraft/Terraria)
This is partially implemented on main, where the player can use the 'E' button to pick up and move world boundaries while in Noclip mode.
model, textures, geometry should all be live resources which can be selected in the game runtime.

I'm probably going to rollback existing progress on this, because I think the inclusion of ODE is essential to make this engine worthwhile for actually making games in. When Physics is done, then I may come back with Live Editing features, such as moving statics around, and picking mat/tex.


Travis: add task to check the PHP files for cross-version compatibility issues

Installs the PHPCompatibility package to check code for cross-version compatibility using static analysis based on PHP_CodeSniffer.
Includes installing the DealerDirect Composer PHPCS plugin which will automatically handle registering the external PHPCompatibility standard with PHP_CodeSniffer.
Includes adding a Composer script to run the command easily with the correct command-line arguments.
Alternatively, the configuration could be setup in a phpcs.xml.dist file which could at a later time be expanded to also include code style rules.
Let me know if you'd prefer that.

Note: using @php will make sure that the script is triggered with the same PHP version as Composer is using (if different from the system default PHP version).
Note: calling the actual phpcs script directly instead of via the vendor/bin will allow for the script to run cross-platform as otherwise devs on Windows may run into errors.

Includes adding the task in a separate stage to the Travis script.
The PHPCompatibility scanner will give the same results on all supported PHP version, so there is no need to run it on all builds, only once will do.
Using stages allows for running the task in isolation using a different script.
Ref: https://docs.travis-ci.com/user/build-stages/
Includes adding one whitelist comment to ignore a specific issue for one particular file.
The issue being ignored is that the double underscore prefix for function names is reserved by PHP itself and should not be used in user-land code.
As this is a compatibility layer for WP and WP still uses those function names, this is not an issue for BrainMonkey to solve until WP changes those function names.
Also, as - at this time - PHP does not use those specific function names yet, in practice this currently will not give any cross-version compatibility issues.

Related to #63

Awesome. Thanks a ton.


The script does not show in task manager when I run the .exe file for 1.03 release, still works for 1.02 release though

It does on my side. can you recheck ?


Not possible to enter fractions of a unit in 'Request Payment'

It's not possible to enter fraction of a unit, e.g. 0.5 GB or 0.5 GBB, if you use the 'Request Payment' button in chat.
I've attached a screenshot.

fixed


Fragment directive parsing bugs

This fragment directive parsing code looks flimsy and doesn't correctly handle multiple directives. The selector validation logic is fine but the code which extracts the fragment directives seems problematic.
Feel free to copy this getFragmentDirectives() function. I am dedicating that entire code snippet block to the public domain.

Do you want to PR this in? Asking since you were interested in contributing on Twitter.

Yeah I can do that later today 👍

I ended up quite busy today. I'll definitely try to contribute this weekend though

Oh I hadn't realized that this syntax was not compatible with the URLSearchParams parser. Has there been any discussion about aiming to make fragment directives follow a similar syntax to search params?
To re-use the getFragmentDirectives() utility I wrote, this spec would have to use a URL-encoded key-value pairing. I can look into improving the existing code where possible later, as I'm definitely interested in helping get this project running.

In the case of media, the aim was to reuse the existing syntax of WebAnnotations. See https://github.com/WICG/scroll-to-text-fragment/blob/main/EXTENSIONS.md#proposed-solution and https://www.w3.org/TR/2017/NOTE-selectors-states-20170223/#FragmentSelector_frag.


PS2 Controller with Adapter for PC - Dolphin Gamecube Controller setting

Input device?
Twin USB Gamepad (PS2 Controller with adapter)
Input device ID?
DInput/0/Twin USB Joystick
Operating System?
Windows 10
What is the key assignment.
"Button 3" = Buttons/A
"Button 2" = Buttons/B
"Button 1" = Buttons/X
"Button 0" = Buttons/Y
"Button 4" = Buttons/Z
"Button 9" = Buttons/Start
"Axis Y-" = Main Stick/Up
"Axis Y+" = Main Stick/Down
"Axis X-" = Main Stick/Left
"Axis X+" = Main Stick/Right
"Axis Z-" = C-Stick/Up
"Axis Z+" = C-Stick/Down
"Axis Zr-" = C-Stick/Left
"Axis Zr+" = C-Stick/Right
"Button 5" = Triggers/L
"Button 7" = Triggers/R
"Button 5" = Triggers/L-Analog
"Button 7" = Triggers/R-Analog
"Hat 0 N" = D-Pad/Up
"Hat 0 S" = D-Pad/Down
"Hat 0 W" = D-Pad/Left
"Hat 0 E" = D-Pad/Right

do you mean like this? please fill in missing buttons.
"Button 3" = X
"Button 2" = O
"Button 1" = ∆
"Button 0" = □
"Button 4" = R1
"?" = L1
"?" =selects
"Button 9" = Start
"Button 5" = L2
"Button 7" = R2

"Button 3" = Buttons/A -> □
"Button 2" = Buttons/B -> X
"Button 1" = Buttons/X -> O
"Button 0" = Buttons/Y -> ∆
"Button 4" = Buttons/Z -> L2
"Button 9" = Buttons/Start -> Start
"Button 5" = Triggers/L ->  R2
"Button 7" = Triggers/R -> R1
"Button 5" = Triggers/L-Analog ->  R2
"Button 7" = Triggers/R-Analog -> R1
"Button 6" -> L1
"Button 8" -> Select
"Button 10 -> L3
"Button 11 -> R3


Add --private argument

This allows for uploads of packages and index.html files  to your pypi repository to be marked
as private, and therefore ONLY accessible via the Cloudfront
distribution.  This also negates the need for the --secret argument
since these assets will not be available directly from the S3 bucket,
but I decided to leave that argument alone for backwards compatibility.

Thank you!


Roadmap, sonarr?

Is this still being developed? Particularly interested in sonarr integration.

I didn't had much time lately but I will try to add sonarr support (tv shows) until Christmas.

Thank you. I’ll get on with installing and testing. Just wanted to check the project was alive. Looks like the tool I’ve been searching for.


Does it load the data when selecting a word or when clicking the addon button?

If it only sends data when clicking the extension icon but nothing earlier, then it would be good to add this to the addon description. If it currently sends data to fetch wikidata items as soon as one selects things it would be good to at least add a setting so that one configure it to only send data when clicking the addon.

Hi @prototyperspective thanks for reaching out. It only sends data when the extension is clicked. We will consider your point.


Practices where setting = 4 appears to be incorrect

This stems from issue #321, where it was discussed that some non-standard practices are incorrectly assigned to setting 4, and these could perhaps be reported to NHS Digital(?), to be corrected in the practices dataset.
In particular, some practices with setting 4 are care homes (hence should have code 13 hospice or 14 Nursing/care home). These can usually be identified via their highly elderly populations. (Homes for younger people are less straightforward to identify!).
The first 7 in this list have disproportionate numbers over 65 (>90%), averaged over all months. The next practice is clearly odd (5 patients), but after that the numbers are more comparable with CCG averages and probably just represent more elderly neighbourhoods.

[ ] Report these potential errors for investigation
[ ] Check for other types of practice given incorrect setting codes such as walk-in-centres.

Here is (potentially more widely useful) code used to calculate percent of registered patients over 65/75 for each practice, compared to their parent CCG average.
WITH a AS ( -- pct/ccg-level data
SELECT pct_id, month,
(sum(male_65_74)+sum(male_75_plus)+ sum(female_75_plus)+sum(female_65_74))/sum(total_list_size) AS Percent_over_65,
(sum(male_75_plus)+ sum(female_75_plus))/sum(total_list_size) AS Percent_over_75
FROM ebmdatalab.hscic.practice_statistics p --prescribing
INNER JOIN  ebmdatalab.hscic.practices prac ON prac.code =p.practice AND prac.setting = 4
WHERE total_list_size >0
group by pct_id,  month
order by pct_id, month),

b AS ( -- average CCGs over all months
SELECT pct_id, avg(Percent_over_65) AS ccg_65, avg(Percent_over_75) AS ccg_75
from a
group by pct_id),

c AS ( --practice-level data
SELECT pct_id, practice, name, month,
(sum(male_65_74)+sum(male_75_plus)+ sum(female_75_plus)+sum(female_65_74))/sum(total_list_size) AS Percent_over_65,
(sum(male_75_plus)+ sum(female_75_plus))/sum(total_list_size) AS Percent_over_75,
sum(total_list_size) AS total_list_size
FROM ebmdatalab.hscic.practice_statistics p --prescribing
INNER JOIN  ebmdatalab.hscic.practices prac ON prac.code =p.practice AND prac.setting = 4
WHERE total_list_size >0
group by pct_id, practice, name, month
order by pct_id, month)

SELECT c.practice, name, c.pct_id, max(b.ccg_65) AS ccg_avg_over_65, max(b.ccg_75) AS ccg_avg_over_75,
avg(Percent_over_65) as Avg_Percent_over_65,
avg(Percent_over_75) AS Avg_Percent_over_75,
avg(total_list_size) AS Avg_total_list_size
FROM c --prescribing
LEFT JOIN b ON c.pct_id = b.pct_id
WHERE Percent_over_65 > ccg_65*2 or Percent_over_75 > ccg_75*2
GROUP BY practice, name, pct_id
order by avg_percent_over_75 desc,avg_percent_over_65 desc

@lmfrench I've assigned this to you for following up with the authorities!

BSA have said this can only be changed by someone at a CCG, they won't give us a full list of contacts but will give us contact details for CCGs where we think a code change is needed.
I'm not sure if all of those listed above need reporting e.g. Eldercare isn't based in a care home, it visits elderly people in the local area so although not a standard GP practice it doesn't really have anything else it could be classified as

Seems like we should perhaps close this issue and not address it? Fixing seems time consuming and potentially error-prone.

yep agree


Who is the community administrator and how should they be contacted?

@jimbrig

@fumitoh and @MatthewCaseres
The repo/README are now public facing (we can switch back to internal/private only is you want and have two separate repos', one for the public and one for members as the profile homepage).
Additionally, I added various useful organizational files and documentation to the repo (i.e. see CONTRIBUTING.md and the rest of the updated repository's contents for details).


Profiles and versioning; individual types and versioning

Could this issue more elaborated?

I added the following test case that illustrates the problem:
https://github.com/oasis-open/tosca-community-contributions/tree/master/tests/tosca_2_0/profile_versions


AppUp plugin, vagrant and docker-compose commands

This plugins adds start, stop, up and down commands when it detects a docker-compose or Vagrant file in the current directory (e.g. your application). Just run up and get coding!
Docker
Aside from simply running up, you can also extend your configuration by running up <name>, which will run docker-compose with both docker-compose.yml and extend it with docker-compose.<name>.yml. For more on extending please see the official docker documentation: https://docs.docker.com/compose/extends. Additional arguments will be directly supplied to the docker-compose.
Compatibility
It does not interfere with any other docker/vagrant plugins as far as I know.

An up-to-date and maintained version of this plugin is available here: https://github.com/Cloudstek/zsh-plugin-appup


Ability to hide buttons for the JAMF tools not setup on device/in organisation

Hello,
Thanks for the hard work! Loving this tool so far!
Would it be possible to pick and choose/remove the JAMF tools that we do not use in our organisation?
(i.e.: if we do not use Jamf protect/Jamf connect, have the ability to hide the buttons)

Hey!
Thats already possible today, although i have not documented it.
As example deploy this config using Jamf
com.txhaflaire.JamfCheck
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>jamfConnectEnabled</key>
	<true/>
	<key>jamfProEnabled</key>
	<true/>
	<key>jamfProtectEnabled</key>
	<false/>
</dict>
</plist>

Awesome! Thanks


Generate spec templates for the data migration

It is extremely useful to have specs alongside the data migration for testing purposes. I know this doesn't exist with standard schema migrations but one can definitely test data changes better than the presence of a new column.
We do so here:
https://github.com/dimroc/datafix

This is interesting, but I typically don't run any specs against Production data, and I almost never run data migrations against test data -- so I'm not sure how this would work.
Looks like https://github.com/dimroc/datafix does something very similar to this gem?

Yes they're almost identical, but dimroc/datafix built off of a data fix
gem that didn't reuse much of ActiveRecord Migrations, which is what yours
does. That's a great thing.
I'm a big fan of writing specs for data migrations. You would write a spec
that would generate test data similar to what you would have in production,
and then get your test green to give you more confidence in the migration.
Sometimes the data migrations can get hairy and it's a great way to have
confidence in your data migration before running against production
critical data.
Here is a gist that shows a simple example:
https://gist.github.com/dimroc/3a8bec16993f384c6398
I'm sure you can imagine a migration against production data that could get
much more complicated. A test makes you feel that much safer.
On Mon, Mar 2, 2015 at 8:34 AM, Jason Fleetwood-Boldt <
<EMAIL_ADDRESS>wrote:

This is interesting, but I typically don't run any specs against
Production data, and I almost never run data migrations against test data
-- so I'm not sure how this would work.
Looks like https://github.com/dimroc/datafix does something very similar
to this gem?
—
Reply to this email directly or view it on GitHub
https://github.com/jasonfb/nondestructive_migrations/issues/5#issuecomment-76711879
.

+1

So basically, this is a feature request so that whenever you run rails generate data_migration XYZ it also generates a match spec?


Multiple values (arrays) in one data field in IoText protocol v0

General idea overview
For example, you want to send/encode array of integers like this with IoTextdata protocol:
[-611,611,-583,592,-587,-605,-590,-649,620]

You can optimize array syntax to below:
-611+611-583+592-587-605-590-649+620

Idea for data type is:

upper case for data type sign in array:
I for array of integers
T for array of strings
D for array of decimals
B for array of booleans

Optimization for highest integers values
Additional optimization idea for integers -> convert values to HEX, for values > 999999, we can decrease -1 character per integer value.
Integer example values:
-999999+123123123-555+1234567890

Converted to HEX values:
-F423F+756B5B3-22B+499602D2

PR: https://github.com/bieli/IoText-data-protocol/pull/25


Detect state change in composer function

Is it possible to detect state change of a component in composer function?

Could you give us more information on this?

As of now we can already detect props change inside a composer function, right? I was wondering if there could be same thing for state, but only read only access to components state.

React-komposer does not detect props-change, maybe there is a misunderstanding.
Additionally, you never have access from "outside" to a component's state, so that also makes little sense to me. Maybe you can explain what you want to achieve with this 👍

I wanted to shorten this chain "component -> action -> ReactiveDict -> container" into "component -> container".
For example if I wanted to set the ordering

component calls action
action does the ReactiveDict.set('ordering', 'asc')
container reads ReactiveDict.get('ordering')

this could be lot shorter if

component does this.setState({ordering : 'asc'})
container reads value of that state

I'm really not following. Why do you need react-komposer if you're simply calling setState? You can just not use react-komposer at all. Maybe you'd be better off with a library like recompose

Need it to set parameter to subscription.

I guess people don't get it because I'm asking for what doesn't exist in React. Can higher order component read wrapped component's state?

No. You can't read a component's state from outside. And that's a good thing.
With stateless components you "know" everything about your component anyway, because you pass everything as property

I thought if it is just a read access it would not be a bad practice


Gate Individual Interfaces Behind Feature Flags

Many other libraries have a way to only compile exactly what you need. Although I can't think of many cases myself, I suppose there could be cases where you only want access to a few interfaces.
General guidance is that accessible and accessible_ext should be gated together.
So for Odilia at this time, we actually only use acessible*, text, and hyperlink interfaces. We should be able to only compile what is needed for our functionality.

This will not reduce the dependency tree, and only artificially restricts features.


[6.1] don't use obsolete/deprecated/unavailable symbols as the main symbol

Explanation: Updates the "alternate symbol" logic to handle rare occurrences where Objective-C symbols imported into Swift have an obsolete alternate that is currently selected as the primary symbol.
Scope: Bug fix to handle an uncommon issue
Issues: rdar://138648876
Original PRs: https://github.com/swiftlang/swift-docc-symbolkit/pull/91
Risk: Low.
Testing: New tests have been added to ensure the new functionality. Existing tests pass.
Reviewers: @d-ronnqvist

@swift-ci Please test


Update alpine to 3.19

Alpine Release Notes

Reran the tests since golang uploaded the new images, but now there's a build failure with boringcrypto:
#14 [build 5/5] RUN GOOS=linux GOARCH=amd64 make rollout-operator-boringcrypto
#14 0.183 GOEXPERIMENT=boringcrypto GOOS=linux GOARCH=amd64 CGO_ENABLED=1 go build -tags netgo ./cmd/rollout-operator
#14 50.25 # github.com/grafana/rollout-operator/cmd/rollout-operator
#14 50.25 /usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
#14 50.25 /usr/lib/gcc/x86_64-alpine-linux-musl/13.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: /tmp/go-link-4052097540/000023.o: in function `BIO_new_file':
#14 50.25 (.text+0xcb1dc): undefined reference to `fopen64'
#14 50.25 /usr/lib/gcc/x86_64-alpine-linux-musl/13.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: /tmp/go-link-4052097540/000023.o: in function `BIO_rw_filename':
#14 50.25 (.text+0xcb60d): undefined reference to `fopen64'
#14 50.25 collect2: error: ld returned 1 exit status

The build issue has already been reported upstream. There's no rush in going to 3.19 for us so I'll keep this PR in draft for the time being.

In the go issue fixing this was declined. Seems that we'll have to go back to compiling in a glibc environment in order to move forward. As a note we've been compiling in alpine since boringcrypto was introduced here in https://github.com/grafana/rollout-operator/pull/71.

Double checked for BoringCrypto in the runtime boringcrypto container (modified to include gofor the check):
$ go tool nm /bin/rollout-operator | grep Boring
  16ea60 T crypto/internal/boring/sig.BoringCrypto.abi0
   666a0 t kBoringSSLRSASqrtTwo
   667a0 t kBoringSSLRSASqrtTwoLen
$ go tool nm /bin/rollout-operator | grep StandardCrypto


⭐ 36 - Health Monitor (Non-wearable) (Hacktoberfest Demo)

🤩 Thank you for participating in the CopilotKit Hacktoberfest event.
Follow This Link to Get Started!🎉
NOTE: CopilotKit will allow multi-assign issues
💡 Please show your support and give a ⭐ to the CopilotKit repository.
Documentation to:

CopilotKit
CopilotKit CoAgents (Advanced)

Demo:
Health Monitor (Non-wearable)
Description of app functionality:
uild a health tracking app where users input their daily data (sleep, diet, etc.), and AI agents offer health tips and advice.
Technology Requirements
Using CopilotKit

Required: Your project must utilize CopilotKit as a core component.
Installation: Include CopilotKit by following the installation guide at CopilotKit Quickstart.
Documentation: Familiarize yourself with CopilotKit's features through the Official Documentation.

Styling with Shadcn-UI

Required: All UI components in your project should be styled using Shadcn-UI.
Installation: Install Shadcn-UI documentation

Hi @NathanTarbert ,
I find this as interesting to work and contribute. Please assign this issue to me.
Thank you.

@quest-bot embark

@NathanTarbert..I'm interested

@quest-bot embark


feat(storage): filter data by watermark in fast compact algorithm

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

For ascending watermark, if the smallest key of a block is larger than watermark key, all keys in this block whose vnode equals the first key shall not be deleted by watermark. But if there are some keys which belong to another vnode, they may need to skip by watermark.
For descending watermark, if the smallest key of a block is smaller than watermark key, there may be some other keys are larger than watermark key, which means that they should be deleted by watermark but we keep them in output file.
Since watermark will be always monotonically. The data of every vnode of one table will only keep at most one block which should be deleted. It means that for a table with 256 vnode, it will take up to 16MB(256 * 64KB) of additional space because of fast compact algorithm


Looks like the script is grabbing the Azure CLI AppId instead of the one provided

Looks like the app Id that is being used to update the permissions is erroring out since the appid is for the Azure CLI, not the appID for the SPN I've created

I'll try to take a look at this to see where it's failing a little later but wanted to let you know.

Which bootstrap script is this? There is one for each configuration.

Vnet shared
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Roger Doherty (MSFT) @.>
Sent: Wednesday, August 24, 2022 2:20:08 PM
To: doherty100/azuresandbox @.>
Cc: Joey Brakefield @.>; Author @.>
Subject: Re: [doherty100/azuresandbox] Looks like the script is grabbing the Azure CLI AppId instead of the one provided (Issue #4)
Which bootstrap script is this? There is one for each configuration.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdoherty100%2Fazuresandbox%2Fissues%2F4%23issuecomment-1226075701&data=05|01||b2da27bc531242869bc108da85fd485a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637969620139244368|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=klkcVb7kUcdxsU2vgZiIT9Aw6C7CL93GK%2Bp7mhtskR8%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE6FA7NRDYIYS7Z4LXHB4ALV2ZRVRANCNFSM57PPAXCA&data=05|01||b2da27bc531242869bc108da85fd485a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637969620139244368|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=X875SJ5y8d42FlamUXYxAAA18a%2Bp0oGQ%2BJrlrlFXZuo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

Yes. Chmod +x to it and then dot source execute in Ubuntu wsl
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Roger Doherty (MSFT) @.>
Sent: Wednesday, August 24, 2022 2:37:27 PM
To: doherty100/azuresandbox @.>
Cc: Joey Brakefield @.>; Author @.>
Subject: Re: [doherty100/azuresandbox] Looks like the script is grabbing the Azure CLI AppId instead of the one provided (Issue #4)
The bootstrap scripts need to be run in bash, not powershell. Can you confirm you are executing bootstrap.sh in bash?
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdoherty100%2Fazuresandbox%2Fissues%2F4%23issuecomment-1226092770&data=05|01||5356e75de9d7410117e008da85ffb395|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637969630537298382|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=cpvUyRf7wHKTvdal0xdeejhfiNiTVYOu6vsEwNfX1Io%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE6FA7PKLGD7LD3XMTJW2UTV2ZTWPANCNFSM57PPAXCA&data=05|01||5356e75de9d7410117e008da85ffb395|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637969630537298382|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=sAifFqPOgSuhu72WipdWOm1W%2FK6hpGQaQOwVNs4P0Vo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

There was a recent change in azure cli for createing SPNs, new format is:
az ad sp create-for-rbac -n AzureSandboxSPN --role Contributor --scopes /subscriptions/00000000-0000-0000-0000-000000000000
Can you confirm this is the format you used?

Unknown.
But I did confirm via the GUI that the son is a contributor on the sub
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Roger Doherty (MSFT) @.>
Sent: Wednesday, August 24, 2022 2:46:30 PM
To: doherty100/azuresandbox @.>
Cc: Joey Brakefield @.>; Author @.>
Subject: Re: [doherty100/azuresandbox] Looks like the script is grabbing the Azure CLI AppId instead of the one provided (Issue #4)
There was a recent change in azure cli for createing SPNs, new format is:
az ad sp create-for-rbac -n AzureSandboxSPN --role Contributor --scopes /subscriptions/00000000-0000-0000-0000-000000000000
Can you confirm this is the format you used?
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdoherty100%2Fazuresandbox%2Fissues%2F4%23issuecomment-1226103205&data=05|01||95a7e19d237942fe25ad08da8600f72a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637969635946182788|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=E4bJPVLwhdJr1iZ9C0H1ECZXFJQVBVgK2Q66%2FK945f8%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE6FA7KZMN6NJHAHICEUAETV2ZUYNANCNFSM57PPAXCA&data=05|01||95a7e19d237942fe25ad08da8600f72a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637969635946182788|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=hVCKSwWjjf5zbzLorKXJoSrRrxchizhHCtoV8eEBKFw%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

I see the issue. Lines 171 - 176 of bootstrap.sh. $owner_object_id is not set correctly. That is set on line 16. Can you run this command substituting the correct UPN:
# Get upn for signed in user
az ad signed-in-user show --query userPrincipalName --output tsv

# Get object id for upn
az ad user show --id<EMAIL_ADDRESS>--query id --output tsv

Scan the json output for property named "id". If you don't find it then that is the bug.

If you could send me the json privately for the second command that would be helpful.

Ah. Something is up with the AZ CLI python bits:

I did an az upgrade. The newest bin doesn't resolve the issue either.

Ah. Something is up with the AZ CLI python bits:

I did an az upgrade. The newest bin doesn't resolve the issue either.

funny enough, the command works by itself. Just not when you do an $() first interpolation. I get an MSI error when running in Cloud Shell:

Per the workaround, I tried az login inside the shell and got some weird conditional access error.  So, something is up with the cli. Not sure what exactly between Python interpreter or our conditional access.

I'll try on my MSDN in the morning. Just afraid of the costs, etc. Will let you know :-)

I am running the same version, if you want you can just hard code owner_object_id in bootstrap.sh, you can look it up in portal.azure.com > azure active directory > Manage > Users. Search for your UPN and click on the user. You want "Object ID" on the "Overview" tab.

Found the issue:
https://github.com/Azure/azure-cli/issues/22776
I am going to remove dependencies on az ad user show, it will operate differently on various tenants because of conditional access policies.

Fixed in commit https://github.com/doherty100/azuresandbox/commit/edd546f3fe099e768bc3c23d247208f18133c40c


astro-webmanifest compatability with Astro 3

Currently astro-webmanifest is incompatable with Astro 3 because of dependency from<EMAIL_ADDRESS>On build it gives error:
MissingSharp: Could not find Sharp. Please install Sharp (`sharp`) manually into your project.

Updating dependency to<EMAIL_ADDRESS>will fix the problem

Thanks for your issue. Just updated  to v1.0.


Par_iter_mut does not seem to be utilizing available (idle) cores?

Full disclaimer, this is likely my fault in some way :)
I'm using Rayon master as the dependency, since I wanted some of the 0.3 features.
My code uses par_iter_mut to operate on a vector of "Pages" in parallel.  There are 3969 Pages in total, and each page has variable amount of work.  Total runtime is about 60 seconds on my Macbook air, and it appears to be using at least some of my cores (two cores are fairly heavily utilized, the two virtual hyperthreaded cores less so).
However, I tried to run this on my linux "heavy testing" server (hexacore Xeon E5-1650V2, 12 cores if you count hyperthreading) and find that only a single core is used.    After the processing is done, I'm printing out some values from the computation to make sure the result is used.  And processing still takes ~40s on my server, which makes me think it isn't just optimized away.
I thought the work chunks might be too small, so I tried variously sized work units and it seems to display the same behavior.
I'm not sure what would be helpful to debug.  The code itself is here (and is mostly an unmitigated disaster atm, I haven't had a chance to clean it up). I'm using the bench_grow example as a simple test: cargo run --release --example bench_grow
The salient code looks like this:
fn main() {
    rayon::initialize(rayon::Configuration::new().set_num_threads(12));

    let num_pages = 63u32;
    let dimension = num_pages * PAGE_WIDTH;
    let mut cajal = Cajal::new(num_pages, 0.01, &[1,2,3,7]);

    let start = SteadyTime::now();
    cajal.grow();
    let elapsed =  SteadyTime::now() - start;
    let signal = (*cajal.get_cell(10, 10)).get_signal();
    println!("Elapsed time: {:?} (signal: {})", elapsed, signal);
}

fn grow() {
  self.pages.par_iter_mut()
    .for_each(|page| page.grow());;

  // some sequential processing

  self.pages.par_iter_mut()
    .for_each(|page| page.update());
}

Try adding a call to weight_max in there?  The default -- which I suspect I should change -- is to assume that each task is very cheap. weight_max tells is to treat tasks as very expensive, and hence to avoid lumping them together.

Ahhh, I see.  That did indeed fix it, full usage across all cores.  Dropped from 40s to 5s :)
I assumed that Rayon assumed the opposite: each task was expensive, and so maybe my tasks weren't expensive enough and were finishing quicker than Rayon could schedule them.  I'll read up on the weighting code, perhaps a middle-ground will work better.
Tangentially, would it be possible for each task to set it's own weight?  E.g. each Page knows how much work it needs to compute before it starts computing, and is relatively variable.  It might be nice if each individual task can set it's weight dynamically.  Not sure if that's feasible with Rayon's architecture though.
Thanks for the help (and for Rayon in general)!


Beet queries very slow on custom fields

I specified some custom album and item fields in my config:
http://pastebin.com/Ti3wHbi1
If I try beet list -a added:2015-12-11 the albums added today are returned within a second. Try with beet list -a added:2015-12-11 herkomst:discogs it takes more than a minute. Same for queries on custom item level fields.

Yes, they're slow. There's no obvious fix—it's fundamental to the flexibility of flexible attributes that fixed attributes will be faster in SQLite—so we'll need to consider specific ideas for overhauling the schema.


Via HTML should strip non-user sourced headers, like Cloudflare and Amazon

Currently Via strips a number of headers it recieves when passing on requests related to:

Cloudflare
Amazon
Cookies and auth

We should definitely strip the Cloudflare and Amazon ones, as there's no point in sending them on.
pywb also adds a power of nonsense the user almost certainly doesn't care about on the way back. We should remove these on the way out

Some PRs for this:

https://github.com/hypothesis/viahtml/pull/5
https://github.com/hypothesis/viahtml/pull/4


Upgrade to @toast-ui/calendar v.2.1.3

Thanks! Should be simple to do :).. unless...Are there any new features or API changes on the toast calendar side?

Not that is would know.
It would be important to me that this also works well for WASM without server-side.

Of course a 1.0.0 version would also be cool :)

I use MudBlazor and I noticed that when I press Edit in the popup for an appointment, it reloads the page.

It would be great if we could get Syncfusion, Devexpress and Telerik to drop their jaws.

I use MudBlazor and I noticed that when I press Edit in the popup for an appointment, it reloads the page.

Can you upload a self-contained project that reproduces this error? I can take a look. I also use MudBlazor.  Also, please open a new issue for this.

Not that is would know.
It would be important to me that this also works well for WASM without server-side.

Should work just fine with WASM too.

Okay, i created a new Empty site and use your test code and it works now XD

I have already gone through it with the debugger. Nothing conspicuous.

Ref V2 Migration guide:
https://github.com/nhn/tui.calendar/blob/main/docs/en/guide/migration-guide-v2.md
I'll make branch for V2. Looks like there are some nice changes/improvements, but will take a little work to update everything. It should be minimal changes to the C# Side.

I am available for extensive testing at any time.

Of course! It's the least I can do for the open source community.

I am available for extensive testing at any time.
This will be helpful!  Thanks

I've begun the conversion. NPM Packages are updated, updated options classes, and the project builds and the test project starts. No runtime errors, but there are display/rendering issues. Not sure where they are coming from. I'll share the branch tomorrow. Could be a CSS issue. More to come.

Am I mistaken or have they now added a toolbar to the calendar or is it just something they developed themselves?

I have comitted my changes (Pull-Request "Update & Create Schedule") which somehow disturbed me in the development.
#53

@KevinKrueger I pushed up DevV2 branch. Have a look. I think a few of the changes in your commit are covered in this branch. Maybe you can re-apply your changes on this branch.
Month view looks like this and I don't know why.

@gismofx Cooooool :) . Looks like a css problem.
I think it would be easier to accept my pull request and make a merge  in V2. // TODO: V2 is marked

#55 You need to merge it (has 1 conflict) @gismofx

Merged. Have a look. Maybe you can dig into the display issue?

The first problem I had was that it was suddenly called TUIEvent instead of TUISchedule.
The second problem is that pulling the length of the appointments no longer works.
The third problem is then logically the wrong display of the month view

And my changes are not there.

I sent you a discord link to your email in your profile. I believe a lot of your changes were absorbed within the changes I made. Overall, it's about the same.


about hyperparameter setting and detail

Hi Jed. I am interested in your work MTAG, and I also think it's excellent. I just download the code, prepare the dataset and directly run 'bash run.sh', but I could not get a satisfactory result.
Do you use the same hyperparameters in run.sh to get the numbers reported in the paper? If not, could you please tell me what are the correct hyperparameters to get the best performance? Are there any other details that need to be paid attention to during training?
Thanks.

Hi @ztqwdk, thanks for your interest in our work! Note that you might be looking at the validation results when you directly run run.sh. To evaluate on the test set, please take a look at #1. Let me know if you need additional help!

Hi @ztqwdk, thanks for your interest in our work! Note that you might be looking at the validation results when you directly run run.sh. To evaluate on the test set, please take a look at #1. Also please take a look at the appendix of the paper for the hyperparameters used for each dataset. Let me know if you need additional help!

Thanks for your reply @jedyang97 . Yes, I have noticed the #1 and I have evaluated on the test set.
And the hyperparameter in run.sh, the default parameters in main.py, and the hyperparameters in the appendix, these three kinds of settings are all different. That made me confused.
If it is convenient, could you please tell me specifically every args of main.py ?

@ztqwdk I believe the ones in the appendix should be the ones to use.

@jedyang97 The appendix tells me batch size, initial learning rate, optimizer, MTAG layers, attention heads, node embedding dimension, edge pruning keep percentage, and epochs. But there are still some other config settings in the args of main.py.
I hope to get the whole ‘run.sh' file or specifical every args to make sure that I would not miss any detail.
Thanks for your reply.

@ztqwdk I compiled a more comprehensive hyperparameter list in this Google Sheet. For any parameters that are not specified here, we used the default values in main.py.


[AppDev] Project Map - Buildpacks

Review and finalize "Buildpacks" section of Project Map Document: https://github.com/cncf/tag-app-delivery/blob/wg-app-dev-map-review/app-development-map/v1alpha1/index.md#buildpacks

Since I added the answers for this, I will take it.

Thanks @cerebro1! Did you get any answer from the Buildpacks team? In case, let me know, and I can try reach out to them to see if they have any feedback on this section, thanks!

Closing this for now as we wrapped up iteration 1. We are starting a second iteration on the deliverable, and we'll have a second look at each section for further improvements.


Allow e2e tests customization based on the flavor of k8s being used (vanilla k8s/OCP etc)

Feature Request
Is your feature request related to a problem? Please describe.
When an e2e test is written that works in vanilla k8s, but not in OCP (eg ), OCP specific information has to be included in the test to make it run in both upstream and downstream.
Describe the solution you'd like
A flexible framework that allows customization of e2e tests based on the flavor of k8s being used.
One possible solution is using an env var to distinguish upstream/downstream (this will require overriding of the e2e test package downstream)

I haven't given this too much thought, but I think one way to approach this that would generally solve this problem for just about all upstream/downstream differences could be consistent use of build tags:
Go files with // +build upstream would only be included in upstream builds
Go files with // +build downstream would only be included downstream builds
Go files with no build tags would be included both upstream and downstream
Our Makefile could have stuff like:
TAGS ?= "upstream"
go test -tags $TAGS ./...
go build -tags $TAGS ./...

Upstream we'd invoke the makefile without any special var setting ("upstream" would be the default)
Downstream we'd always invoke make with TAGS=downstream
This would work for both test code and binary builds.

@anik120 could you point to the specific test you were writing when you encountered the issue that encouraged you to open this ticket? It may be nice to use it as a reference while developing a POC of @joelanford's suggestion.


Swift.app

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves #NNNNN, fix apple/llvm-project#MMMMM.

na


feat: Add graph description as tooltip content over y-label

Pls, add a video or screenshot of the expected.


Make GFX mode default for windows 10 and up

As mentioned in #4326 newer windows 10 versions do no longer support legacy RDP modes.
Change FreeRDP to announce GFX by default.
Unitl #4266 is resolved it may be a good idea to only announce 10.2 capabilities by default.

FYI: As mentioned in https://github.com/FreeRDP/FreeRDP/issues/4326, newer Insider builds (I tested 17093 and 17101) don't require egfx anymore, so it might make sense to postpone switching GFX on by default.

Fixed with https://github.com/FreeRDP/FreeRDP/pull/6786


how to access scope cpc_advertising:camapign_management

i am not able to generate authorization code and refresh token because scope is unknown givw error like that how to solve tghis error

@JBLathiya are you following this URL? to generate authorization code.
https://www.amazon.com/ap/oa?client_id=YOUR_LWA_CLIENT_ID&scope=cpc_advertising:campaign_management&response_type=code&redirect_uri=YOUR_RETURN_URL

@dbrent-amazon
Hi could you please help me to generate authorization code by CURL followed by this url.
https://www.amazon.com/ap/oa?client_id=YOUR_LWA_CLIENT_ID&scope=cpc_advertising:campaign_management&response_type=code&redirect_uri=YOUR_RETURN_URL
or is there any other method to generate authorization code. I dont want to hit this url to generate code.
Thanks.