Multiplex cleanup

Resolves #3951
This PR includes both backend and frontend changes related to graphql. Would appreciate an extra careful look to check that there are no backwards compatibility issues, specifically changing DiseaseResult to MultiplexResultInput on the frontend. I can look into separating out the backend and frontend changes if necessary.
BACKEND PULL REQUEST
Related Issue
The backend changes address this concern in the ticket:

Define a naming pattern for data types used in requests and responses of graphql mutation/query. Currently a disease result is sent in the requests of the multiplex result mutation and also part of the response. However even though we are representing the same data it has a different structure and different type definitions. We should standardize the structure and come up with a consistent naming schema (MultiplexResultInput and MultiplexResult, perhaps)

The changes proposed are intended to make the code more readable and follow our domain language for Multiplex.
Changes Proposed

Refactoring the graphql layer to more closely align type and input objects for test results. MultiplexResultInput replaces DiseaseResult, and I structured it for reasons discussed here.
Note that I did not create a new input object for the diseaseName field in MultiplexResultInput to mirror the type object for disease in MultiplexResult because I felt this would be overengineering since we do not need the disease id and loinc fields in mutations at this point, and this can easily be modified if behavior is extended in the future.
To support the graphql changes, I also modified the java model

Testing
To test, visit sr-test, go to the Conduct Tests page and conduct a test. When you submit the test with a result, observe the payload for graphql mutation SubmitTestResultMultiplex contains MultiplexResultInput, and observe that your test result saves correctly. You can also observe this when editing a test result.

FRONTEND PULL REQUEST
Related Issue
The frontend changes address these concerns in the ticket:

Clean up redundant types used to represent multiplex test results and disease results in the frontend. This needs to be clean up after the changes to bring uniformity in the backend are settled.

Defining constants in the frontend to represent repetitive string values like the test results (POSITIVE, NEGATIVE, UNDETERMINED). We have types defined for these but not constants.

Changes Proposed

Following the changes from this ticket, the frontend types have now been cleaned up so we no longer have PxpMultiplexResult and SrMultiplexResult. This is replaced by a single type MultiplexResult.
Cleaned up and consolidated tests - after cleaning up the types, there were duplicate tests since we were previously testing the same things for both PxpMultiplexResult and SrMultiplexResult
I've added an enum type for multiplex disease and test result strings so that we don't have to repeat string literals all over the test results code, and implemented this throughout the test results code. This seemed like the cleanest implementation to me, but feel free to suggest a better alternative.

Testing
To test, visit sr-test, go to the Conduct Tests page and conduct a test. When you submit the test with a result, observe the payload for graphql mutation SubmitTestResultMultiplex contains MultiplexResultInput.
Screenshots / Demos

Checklist for Author / Primary Reviewer

[x] Any large-scale changes have been deployed to test, dev, or pentest and smoke tested

[x] Deployed to sr-test

[ ] GraphQL schema changes are backward compatible with older version of the front-end
[ ] Changes comply with the SimpleReport Style Guide
[X] N/A Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[x] N/A Any dependencies introduced have been vetted and discussed
[x] N/A Any changes to the startup configuration have been documented in the README

if anyone want to test in an environment, lmk and I will spin one up with this branch!

@emmastephenson So one issue I saw when deploying only backend changes locally (frontend from current main) is that the frontend sends a data with DiseaseResult and when it hits graphql, qraphql doesn't know what DiseaseResult is b/c it's now MultiplexResultInput everywhere on the backend. This results in this error:

What's the preferred way to handle this? Keep both DiseaseResult and MultiplexResultInput on the backend? Seems ugly but not sure there's a better way.

I'm doing some backwards compatibility testing to see what happens when a user has an old frontend and the new backend code. I created a new branch george/3951-validate-bw-compatibility-2 that is a copy of main with only backend changes pulled in. Still investigating, but once I pulled in the backend changes and pushed the branch, I was not able to build the frontend and deploy in dev, see frontend build error.
I also recreated it locally following the same build steps as the github frontend build job, reproduced the same error:
Failed to compile.

src/generated/graphql.tsx
  Line 6087:16:  Unknown type "DiseaseResult". Did you mean "TestResult" or "UploadResult"?  graphql/template-strings
  Line 6172:16:  Unknown type "DiseaseResult". Did you mean "TestResult" or "UploadResult"?  graphql/template-strings

This is a bit confusing b/c backend changes shouldn't affect frontend build. Any ideas why this might be occuring with graphql?
@emmastephenson @zdeveloper

I'm doing some backwards compatibility testing to see what happens when a user has an old frontend and the new backend code. I created a new branch george/3951-validate-bw-compatibility-2 that is a copy of main with only backend changes pulled in. Once I pulled in the backend changes and pushed the branch to deploy to dev, the frontend did not build in dev, see frontend build error.
Still investigating, but I also recreated it locally following the same build steps as the github frontend build job, reproduced the same error:
Failed to compile.

src/generated/graphql.tsx
  Line 6087:16:  Unknown type "DiseaseResult". Did you mean "TestResult" or "UploadResult"?  graphql/template-strings
  Line 6172:16:  Unknown type "DiseaseResult". Did you mean "TestResult" or "UploadResult"?  graphql/template-strings

This is a bit confusing b/c backend changes shouldn't affect frontend build. Any ideas why this might be occuring with graphql?
@emmastephenson @zdeveloper

Emma and I found out why this is happening! yarn codegen runs on frontend build, and is based on main.graphqls which is technically backend...
So current solution is to double check that the backwards compatibility issue happens in dev (we think it might be fine because the frontend VersionService reloads the app when there is a version mismatch detected). If it's still an issue, I'll separate out backend changes in a new PR and refactor to keep both the old and new type (DiseaseResult and MultiplexResultInput), probably in a union.

@emmastephenson looks like we will need to separate this into two PRs 🥲
I got the same error after deploying in dev, and the error only went away after manually refreshing the page:
https://user-images.githubusercontent.com/89797785/185489886-99afc9d0-f72d-4f0d-9ef5-d7086d78d793.mov
https://user-images.githubusercontent.com/89797785/185489914-157c51b1-acd0-4c77-917e-92ac0b0fc28e.mov


Pressing Enter doesn't work in the cells in the same column.

Hello, this library has saved my time a lot. I would like to appreciate your contribution before I report it.
I am using this library in a special case, which uses the autocomplete function only for the first row.
In addition, I use ag-grid as row-based.
When I select a value in the first row, Enter and Tab don't work after the second row.
I handled it by adding the below code to the table and now it works properly.
onCellKeyPress = ({ data, event }) => {
  if (data.field !== "exampleRowFirst" && event.keyCode === 13) {
    for (let cell of this.gridApi.getEditingCells()) {
      this.gridApi.stopEditing();
      this.gridApi.setFocusedCell(cell.rowIndex, cell.column.colId);
    }
  }
};

Thank you.

The same thing happened at Tab, couldn't find a solution yet.

In addition, I use ag-grid as row-based.

Could you elabore on this part ?
If you mean you use the Full Row Edit feature, it is intended that this editor (which inherit from PopupComponent) cannot work properly with this option, like any other Popup editor.


[hotfix] Removing the obsolete comment of JobAutoScaler#scale

What is the purpose of the change
After FLINK-33081, the JobAutoScaler#scale doesn't return value anymore, so the comment can be updated.
Brief change log
[hotfix] Removing the obsolete comment of JobAutoScaler#scale

Could we include this in some other commit instead of having it separately?

Could we include this in some other commit instead of having it separately?

Sure, I will do it in the subsequent commit. I will close this PR.


EOF Error in exp_classification.py

I wonder why I get EOFError: Ran out of input error at for i, (batch_x, label, padding_mask) in enumerate(train_loader) line in the train() function of exp_classificaton.py file. I am running EthanolConcentration experiment.

I also encountered the same problem

resolution method ： import dill as pickle

resolution method ： import dill as pickle

It didn't work. Still getting the same "EOFError: Ran out of input" error.

import dill as pickle does solve this problem.
However, you should change  import pickle to import dill as pickle at line 15 of the Lib\multiprocessing\reduction.py file.
I guess there is a strange library conflict.

把 源代码库中的这一行import给注释掉
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2023年8月22日(星期二) 晚上6:03
收件人: @.>;
抄送: "Orange @.>; @.>;
主题: Re: [thuml/Time-Series-Library] EOF Error in exp_classification.py (Issue #155)
import dill as pickle does solve this problem.
However, you should change  import pickle to import dill as pickle at line 15 of the Lib\multiprocessing\reduction.py file.
I guess there is a strange library conflict.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

It still doesn't seem to solve the problem, the program is stuck here, does not continue to run and does not report errors, I wonder if there is a way to disable multiprocessing.

I set the "--num_workers" to 0 to stop multiprocessing. The code runs fine after that.
Sent from Yahoo Mail on Android
On Thu, Aug 24, 2023 at 9:19 PM, @.***> wrote:
It still doesn't seem to solve the problem, the program is stuck here, does not continue to run and does not report errors, I wonder if there is a way to disable multiprocessing.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Sorry for your trouble but we meet that weird problem here. Could you pls provider some further information?

Hi! I have the same problem with it. What information can I provide to solve this problem?

Hi,
As I mentioned previously, I set num_workers=0 and it solved my problem. Mind that I was using windows operating system.
Saqib.
On Wednesday, October 18, 2023 at 10:57:44 PM GMT+8, rkdlight @.***> wrote:
Hi! I have the same problem with it. What information can I provide to solve this problem?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hi, thanks for the answer from @saqibejaz . Setting num_workers as 0 is useful for windows users.


Add TryFrom<Quote3<T>> impl for TcbInfo

Previously the TcbInfo could only be obtained from a Certificate.
Now one can try to get the TcbInfo directly from a Quote3<T>.

Motivation

Future Work

Current dependencies on/for this PR:

main

PR #401   👈

This comment was auto-generated by Graphite.


[YUNIKORN-512] Remove redundant log from placeholder manager

Remove "cleanup orphan pods" log that gets printed every few second
Update the dependency to the latest core

@kingamarton could you please help to review? Thanks

@kingamarton could you please help to review? Thanks


Typo error

Please, fix typo error, replace Single Spectrum Analysis(SSA) with Singular Spectrum Analysis(SSA)

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

ID: 32d03435-cc6c-1123-a86c-d5c3eacc24bd
Version Independent ID: 460afd2b-383d-7b90-6804-4b900c3bc9b2
Content: Tutorial: Forecast bike rental demand - time series - ML.NET
Content Source: docs/machine-learning/tutorials/time-series-demand-forecasting.md
Product: dotnet-ml
GitHub Login: @luisquintanilla
Microsoft Alias: luquinta

Thanks for raising this issue @AntonNalyvaiko . Changes will be published shortly.


Cannot transfer to subaccount in GUI

The 'Send' button is disabled and avatar is not shown.

Duplicate: https://github.com/BitShares/web_wallet/issues/639


Analytics: Add event triggers for the Carbon Modal

Jeff-Chew created the following on Sep 06:

User Story

As a [user role below]:
IBM Data Analyst

I need to:
have dynamic event triggers from the Carbon modal

so that I can:
track dynamic events for the modal in my analytics tool

Additional information

http://ibmdev.webmaster.ibm.com/v18/metrics/
Will need more details on what events need to be triggered from Juan Leon and team
Carbon's modal apparently has generic event listeners we can subscribe to
https://www.carbondesignsystem.com/components/tabs/code/#events
Added to the global.js, we will listen to the events emitted from Carbon and add these to those.

Acceptance criteria

[ ] Event triggers added to the Masthead per specs provided by Juan Leon's team
(https://ibm.ent.box.com/s/jw44u3vh2cxg0exgjwthvk03do79iki4)

Original issue: https://github.ibm.com/webstandards/digital-design/issues/1603

Similar to Tabs, the React Modal is not emitting the global events, while Vanilla Modal is. Created an issue on the Carbon issue board to request for this enhancement:
https://github.com/carbon-design-system/carbon/issues/4455


Incorrect return for getAuth response then the expected type

Describe the bug
Incorrect types and enum
To Reproduce

Eslint error for (the = in HealthStatusCode = { is not needed)

export enum HealthStatusCode = {
    NotDetermined = 0,
    SharingDenied = 1,
    SharingAuthorized = 2,
  }

Steps to reproduce the behavior:
const permissions = {
permissions: {
read: [
AppleHealthKit.Constants.Permissions.Workout,
AppleHealthKit.Constants.Permissions.ActiveEnergyBurned,
AppleHealthKit.Constants.Permissions.BasalEnergyBurned,
AppleHealthKit.Constants.Permissions.HeartRate,
],
write: [
AppleHealthKit.Constants.Permissions.Workout,
AppleHealthKit.Constants.Permissions.ActiveEnergyBurned,
AppleHealthKit.Constants.Permissions.BasalEnergyBurned,
AppleHealthKit.Constants.Permissions.HeartRate,
],
} as HealthKitPermissions

AppleHealthKit.getAuthStatus(permissions, (err: string, results: HealthStatusResult) => {
console.log(results)
})
following is the console log
{permissions: {…}}
permissions:
read: (4) [0, 0, 0, 0]
write: (4) [0, 0, 0, 0]
__proto__: Object
__proto__: Object

Results contain permissions property that is not defined in the
export interface HealthStatusResult {
    read: HealthStatusCode[],
    write: HealthStatusCode[],
  }

And as a result when using results.write, it is returned as undefined and if I use results.permissions.write then eslint shows error as permissions is not defined on results.
Expected behavior
The eslint should not throw error. And the return should match to the defined type
The return type matches the expected type
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

I recently fixed this in #39. I'll fix the enum part right away, but I wonder if there is more that needs to change with the implementation of the getAuthStatus native function for iOS

I recently fixed this in #39. I'll fix the enum part right away, but I wonder if there is more that needs to change with the implementation of the getAuthStatus native function for iOS

I recently fixed this in #39. I'll fix the enum part right away, but I wonder if there is more that needs to change with the implementation of the getAuthStatus native function for iOS

I actually am using the new code you added in #39, in the return type the results contains permissions property that is not expected in the HealthStatusResult type which is causing the eslint to fail

I recently fixed this in #39. I'll fix the enum part right away, but I wonder if there is more that needs to change with the implementation of the getAuthStatus native function for iOS

I actually am using the new code you added in #39, in the return type the results contains permissions property that is not expected in the HealthStatusResult type which is causing the eslint to fail

Gotcha. I'll work on a fix right now open a PR.

Gotcha. I'll work on a fix right now open a PR.

Sorry about that! Just to clarify - it's just the typing not working with the linter? I've opened a PR (#43)
  export enum HealthStatusCode {
    NotDetermined = 0,
    SharingDenied = 1,
    SharingAuthorized = 2,
  }

  export interface HealthStatusResult {
    permissions: {
      read: HealthStatusCode[],
      write: HealthStatusCode[],
    }
  }

Sorry about that! Just to clarify - it's just the typing not working with the linter? I've opened a PR (#43)
  export enum HealthStatusCode {
    NotDetermined = 0,
    SharingDenied = 1,
    SharingAuthorized = 2,
  }

  export interface HealthStatusResult {
    permissions: {
      read: HealthStatusCode[],
      write: HealthStatusCode[],
    }
  }

I think that should work nicely now. Thanks.

I think that should work nicely now. Thanks.

Hey @vishalj!
Thanks a lot for submitting that request. I just merged PR #43 from @NS-BOBBY-C and will be releasing a new minor version with the fix!


HOV - 10 | Flex Component | Passing ref and style in Flex props

Expected Behavior

ref should be passed in props as the component is made ForwardRefRenderFunction.
style prop is not coming in autocomplete, eg:

Current Behavior

You can't pass ref as props
You can pass user-defined style it will be overwritten if default is present

Possible Solution

Steps to Reproduce (for bugs)

import Flex
Use Flex and pass ref as well as style props.

Context

Your Environment

Version used: nodejs 16.x
Browser name and version: Chrome (103.0.5060.134)
Operating System and version (desktop or mobile): MacOS (12.4)
Link to your project: https://github.com/antstackio/hover-design/pull/25#pullrequestreview-1061682355_

Fixed this issue in Commit 145157ea2a42ff52988aaea96a6c865d41b7ba00
[https://github.com/antstackio/hover-design/commit/145157ea2a42ff52988aaea96a6c865d41b7ba00](Link to the above commit)


GH-787 Fixing the issue with cart restoration on page reload

Fixed cart restorarion, uncomented e2e test.

That was accomplished in some other PR with state synchronization feature.


CORE-9790 Vault support in crypto plugin

Added support for Vault in the crypto initial config plugin. This involves removing the wrapping key and wrapping salt from the Config entirely. I believe these are used to create a root key for EncryptionSecretsServiceImpl only and are not required for Vault.
@dickon please let me know if I got this wrong at all.

Ok, this is not the correct thing to be doing, it results in the following error in the db worker
net.corda.libs.configuration.validation.ConfigurationValidationException: Configuration failed to validate for key corda.cryptoLibrary at schema version null: $.hsmMap.SOFT.hsm.cfg.wrappingKeyMap.salt: is missing but it is required, $.hsmMap.SOFT.hsm.cfg.wrappingKeyMap.passphrase: is missing but it is required


feat: Implement codegen for source verification providers

PR is ready to go!

Previous exploration message
This is a proof of concept PR as I'm exploring a small refactor around source verification. Please do NOT merge!
Previously, the steps to add new provider is (example platform: Acme)
add new provider code sourceverification/verification_acme.go
update Terraform model types at sourceverification/model.go
update Terraform schema logic at sourceverification/schema.go
update SDK logic at sourceverification/sdk.go
Of all these steps, the SDK logic is the most tedious as it usually takes me some time to locate the right place before adding 2 lines of code. It's a bit scary since if there's a bug in this step, we won't realize it until it's properly tested end-to-end.
I want to reduce the process to just one single step
add new provider code sourceverification/verification_acme.go
However, I have not been able to achieve this goal yet. The main challenge is the Terraform static types. I haven't been able to figure out a way to do so dynamically just yet. Therefore, this is where we're at with this PR
add new provider code sourceverification/verification_acme.go
update Terraform model types at sourceverification/model.go
I do think this is a significant step up from before as it's quite straightforward to update these 2 steps.
To be honest, I'm not quite100% happy with where things are at yet. However, given that this is not a huge value add (more internal DX vs actual value for users), I'm not sure if we should continue exploring this effort further. Just want to share this idea and my progress here if you want to explore this idea further!

That already feels like a meaningful improvement. Looking forward, we'll be adding hundreds of providers and we should be looking into how that can be fully automated across all the product surface, terraform included. cc @leggetter

Hi @leggetter, @alexbouchardd, this PR for source verification codegen is ready to go. In the future, whenever we want to support new providers, we can run go generate and things should work as expected.
I did a review to call out a few concerns that require your input, please let me know what you think!
I'm also around if you'd like to elaborate on the PR further or want to go over the flow or the logic. Let me know!

Hi @leggetter, I added the 2 things you mentioned, please let me know if there's anything else you need for the release!

moved to #100


Updated logic on GPU usage if use_gpu is set to None.

Changed test to automatically decide if it should use CPU or GPU
Updated logic by only deciding what to use when use_gpu is set to None.

@Lukas113 please review and approve.


How to change default Arch path

I previously installed Arch on WSL2 using scoop. For disk-space issues, I decided to duplicate the arch installation to another partition with the wsl export and wsl import commands. In the near future, the idea is to delete the old wsl copy and use the new one. The procedure seems to have worked fine. The only problem is that Arch.exe still run the old installation.
Is there any way I can modify the path in which the binary takes the default system? One of the main reason for that is that all the commands, including Arch.exe config --default-user can't be executed on the new system.

The ArchWSL launcher (wsldl) uses the name of the exe as the instance name.
For example, if you import the name "Arch2" like this:
wsl --import Arch2 /path/to/inst exported.tar

You can run it by renaming the exe to "Arch2.exe".

Hey @yuk7, thanks for the quick reply. By analysing the files used by scoop, I made some additional changes. I'll leave them here as a reference. Assuming your scoop directory is SCOOPDIR (default %USERPROFILE%\scoop):

copied all SCOOPDIR\shims\arch.{exe,ps1,shim} into SCOOPDIR\shims\arch2.{exe,ps1,shim}.
copied SCOOPDIR\apps\archwsl\current\arch.exe into SCOOPDIR\apps\archwsl\current\arch2.exe
modified SCOOPDIR\shims\arch2.{ps1,shim} to point to the new location of point 2.

If you think there are no drawbacks with this, you can close the issues.
Thanks again for your help.

Manipulating the Scoop directory directly is not recommended as it may cause unintended problems.
Scoop automatically generates shims and ps1.
I recommend copying only the exe to another directory and using it.

Noted. For now I'll just go with it and see what happens. In case of problems I'll try to put just the exe file in my bin dir.

Ok.


play doesn't clear playlist first

The difference between play and queue should be that play starts the selected item directly (erasing the current list of tracks) and queue should just add it for later playing. It seems that currently the play also only enqueues, which always require a clean before. So the play functionality should be fixed or changed.

@baschny yes, play with reference clears the tracklist. That is already implemented. But the plain play command simply starts the current track from new, it is the good twin of the stop command. I think the bot's output Adding xy to tracklist is a little bit misleading. The tracklist clearing does not trigger any output.

That's not what happend today:

Ernesto Baschny [12:31]
search phat rock afternoon
dopebot BOT [12:31]
Albums:
Nothing found
Artists:
Nothing found
Tracks:
Nothing found
Playlists:
0: phat rock afternoon/evening by 22i7xoermnczqupaajrjg4vmy
Ernesto Baschny [12:31]
play playlist 0
dopebot BOT [12:31]
Adding spotify:user:22i7xoermnczqupaajrjg4vmy:playlist:5TssY1eILr6UR15NWvOX39 to tracklist
[12:31]
Track: Dark Ages by Thomas Bergersen from SkyWorld
State: playing | Volume: 10 | Consume: false | Random: true | Repeat: false | Single: false
Initiator: ingomueller 30 minutes ago

So my play playlist 0 did not really cleaned the tracklist, but instead continued to play the tracks added by ingo before (state was "stopped").
Only after a clear it worked as expected:

Ernesto Baschny [12:32]
clear
Ernesto Baschny [12:32]
play playlist 0
dopebot BOT [12:32]
Adding spotify:user:22i7xoermnczqupaajrjg4vmy:playlist:5TssY1eILr6UR15NWvOX39 to tracklist
[12:32]
Track: Beverly Hills by Weezer from Make Believe
State: playing | Volume: 10 | Consume: false | Random: true | Repeat: false | Single: false
Initiator: ernesto a few seconds ago

@baschny hmm, it is definitely calling the API for clearing the tracklist: https://github.com/100hz/dopebot/blob/master/lib/commands.js#L46
But yes, also saw some strange behaviour with the clear tracklist command. In the above case i assume, that mopidy was just changing the track, before you command was handled.


AdoptOpen JRE 11-07 for Win 64 has doubled in size

Platform and architecture (e.g. Linux/x64:
MS WIndows, x64

Java version (from "java --version") e.g. 1.8.0_181, 11.0.4+11 etc.:
11.0.7+10

Error messsage and/or description of what is not working as expected:
The JRE.zip is now 87MB, which is about double what it was in the .6 release.
It appears that it contains full debug information.

Duplicate of https://github.com/AdoptOpenJDK/openjdk-build/issues/1670. Rebuild is underway.

fixed in https://github.com/AdoptOpenJDK/openjdk11-binaries/releases/tag/jdk-11.0.7%2B10.1


Add GitHub Workflow

We are currently running tests only on our internal repository (gitlab) while we do not have the workflow for the github public repo.

Fixed in #2


A consultation over the results by your released code

Dear @comzyh :
It seems that you have not prepared to release the inference code, at least in this REPO recently. Emmm, here is the thing, and your reply is very desired.
My  trouble is that the performance reported in your paper cannot be determined reproduced, successfully or not, by deploying this project.  I get the following results. Could you please tell me whether these are approximate to yours on the validation set of the TuSimple?
########################################
2021-06-21 10:20:51,067 - resa - INFO - [{"name": "Accuracy", "value": 0.9450447819314601, "order": "desc"}, {"name": "FP", "value": 0.07953510663791019, "order": "asc"}, {"name": "FN", "value": 0.08024802300503252, "order": "asc"}]
2021-06-21 10:20:51,399 - resa - INFO - Best metric: 0.9450447819314601
########################################
B.T.W., anyone with the same puzzle is also desired.

Why you @"comzyh" ?
I don't know how you train. Please show more information.

In https://github.com/ZJULearning/resa/issues/16, they can reproduce the result. I believe there is something wrong in your training setting. I close this issue first. If you have other questions, please raise again.


Improving the wrapper script for Windows

Windows users struggle to get the bat file to work.  Problems first noted in #14 include:

[x] Need more guidance for running the software on Windows when Anaconda is not on the PATH.
[ ] Determine why Windows does not launch the GUI the first time the batch script is run.
[x] Add a note about the warning Windows shows about running a batch script from an unknown publisher.

Users have difficult with the Windows launch script.  We may want to update setup.md to recommend launching from the Anaconda Prompt as the primary option.

This was again a problem during the April 2019 workshop.  We should only recommend running the install scripts from the command line.


[Request] Dyson Spheres, Black Hole Generator Buff

What do you think is missing or what do you think can be done better

I would like to see Dyson Spheres as a late game way of getting RF, possibly after getting loads of iridium and defeating the wither. If around the sun it would produce 7,720,000,000,000,000,000,000,000 RF per tick, so it's power output would have to be cut significantly to fit into the mod and not break java, therefore, I propose that it generate 1000000 RF per tick for a dimmer star, 10000000 RF per tick for a star like our sun, and<PHONE_NUMBER> RF per tick for a brighter star. Small Dyson Sphere would be crafted with 4 iridium blocks, 4 iridium coils, and 1 nether star, a Medium Dyson Sphere would require a Small Dyson Sphere surrounded by 8 titanium blocks, and a Large Dyson Sphere would require a Medium Dyson Sphere surrounded by 4 iridium coils, and 4 nether stars.

That would be impossible. The only mod that can do it is draconic evolution.
On Mon, Oct 7, 2019, 11:54 AM NutMaster6942<EMAIL_ADDRESS>wrote:

What do you think is missing or what do you think can be done better
I would like to see Dyson Spheres as a late game way of getting RF,
possibly after getting loads of iridium and defeating the wither. If around
the sun it would produce 7,720,000,000,000,000,000,000,000 RF per tick, so
it's power output would have to be cut significantly to fit into the mod
and not break java, therefore, I propose that it generate 1000000 RF per
tick for a dimmer star, 10000000 RF per tick for a star like our sun, and
1000000000 RF per tick for a brighter star. Small Dyson Sphere would be
crafted with 4 iridium blocks, 4 iridium coils, and 1 nether star, a Medium
Dyson Sphere would require a Small Dyson Sphere surrounded by 8 titanium
blocks, and a Large Dyson Sphere would require a Medium Dyson Sphere
surrounded by 4 iridium coils, and 4 nether stars.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1714?email_source=notifications&email_token=AGVDBJBXBKZHE2FHDPOX2A3QNNSTDA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HQD473A,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVDBJEWC7NPDRII2AH74KDQNNSTDANCNFSM4I6HFC6Q
.

That would be impossible. The only mod that can do it is draconic evolution.

The Draconic Reactor only generates 1 million RF though.

It would also be nice if Black Hole Generator produces 2000 RF per tick instead of 500.

Well this mod should be more of middle tier progression than the late game.
On Mon, Oct 7, 2019, 12:19 PM NutMaster6942<EMAIL_ADDRESS>wrote:

That would be impossible. The only mod that can do it is draconic
evolution.
The Draconic Reactor only generates 1 million RF though.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1714?email_source=notifications&email_token=AGVDBJGUJZKCQANAA32EKITQNNVTNA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAREEPA#issuecomment-539116092,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVDBJBKENC6NXPYBHS5SO3QNNVTNANCNFSM4I6HFC6Q
.

Well this mod should be more of middle tier progression than the late game.

Perhaps in an addon?

It isn't "impossible", its just difficult to code. z would have to code some way of saying whether or not a star has a dyson sphere, a way of launching them, as well as a way to receive energy. He would also have to make it possible to orbit stars (assuming you need to orbit them to collect energy).
Also textures and models for all of the above

And, as well, the material cost would be astronomical. This is unlikely to be at all feasible. There's so little use for a dyson sphere, as nothing needs the power nor has the resources. It's likely that they'll never be implemented, due to the sheer amount of work. Your RF conversion looks remarkably accurate to what Reika calculated the ratio to be, so that's likely fine.
It'd be a swarm anyway, you're better off using multiple microwave receivers and many satellites.

 That's definately a astronomical cost.

bro this is minecraft not stellaris
youre trying to sustain life for 1 person (in most cases) and not 50 trillion
you would be better off with a creative capacitor

This. I rarely (outside of specially constructed packs) use more than a couple of millions of RF/t (Reika's mods are power hungry if you convert to RF), but even his fusion reactor is ~220M RF/t. There's no need for much more than that. Dyson swarms, however, are many levels of magnitude greater power output. There's simply no need for it, outside of powering a server for eternity many, many times over.

although it would actually a cool flex lol

The ReC fusion reactor (or most fusion reactors) are enough of a flex, being more than enough for servers. They're already insane resource sinks and automation sinks, and while the flex would be huge, it'd be almost worthless once you got it. It amount of computational power your base would be consuming to produce the needed components at any reasonable clip (enough to get done in a couple years is a reasonable clip) would outpace most hardware, and, not to mention the in-game RF cost would be so high you'd need something like a fusion reactor to power it. At that point, you've already got the power needed for whatever you were trying to do previously.

Arguably an always-sunlight dimension (if any there are in AR) covered in solar panels would be equivalent to a Dyson swarm/sphere. You don't need some stupid singular overpriced thing for a Dyson, it would be built up over time from pieces.
I dunno how AR satellites work, but if they're abstracted, just allow sending them to a star for increased power output but EXTREME rocket costs. Dyson swarm will form naturally if the player desires so.

You'd have to use a station dimension with a high station build limit, but yes that would work. And yes, as far as I know, satellites are abstracted, so that would be the best option. Dyson swarms/spheres would never be in one piece. The sending to a star would be the only new thing needed.

Power output in a 100 chunk area with a 16x16 area of solar panels per chunk would be: 16x16 = 256x100 = 25600 RF per tick. Now, if you used Nuclearcraft solar panels, and the chunks all had the same type, you could get: 128000 RF per tick, 512000 RF per tick, 2048000 RF per tick, or 8192000 RF per tick. So, a max of 25600 in 100 chunks with just AR solar panels.

New crafting idea: Large Iridium Plates, Dyson Plates, Dyson Controllers, Dyson Support Pillar Segments,  Precision Assembly Table
Large Iridium Plates would be used to make Dyson Plates, one Large Iridium Plate would be crafted with 9 blocks of iridium,
Dyson Plates would be crafted using 9 Large Iridium Plates.
7 Dyson Plates around a nether star with a UI in the top centre would craft a Dyson Controller.
9 Dyson Plates would craft a Dyson Support Pillar Segment.
9 Dyson Support Pillar Segments would craft a Dyson Support Pillar.
A Precision Assembly Table would be crafted in a Precision Assembler with one Advanced Machine Structure, one Iridium block, one UI, and two 2x small battery, it would have a 5x5 grid and craft certain advanced machinery, it would require 10000 RF to craft one item.
A Dyson Sphere would be crafted in a Precision Assembly Table, it would have one Dyson Controller in the centre, the corners would be Dyson Plates, Dyson Plates would surround the Dyson Controller, and the remaining spots would be Dyson Support Pillars
Total Crafting Cost: 81 Iridium per Large Plate, 729 Iridium per Dyson Plate, 5103 Iridium, 1 nether star and 1 UI per Dyson Controller, 6561 Iridium per Dyson Support Pillar Segment, 59049 Iridium per  Dyson Support Pillar, 12 Dyson Plates in a Sphere, 12 Dyson Support Pillars in a Sphere, and 1 Controller in a Sphere, 12x729 plus 12x59049 plus 7x729 = 8748 plus 12x59049 plus 7x729 = 8748 plus 708588 plus 7x729 = 8748 plus 708588 plus 5103 = 722439
So, total resource cost is 722439 Iridium, 1 nether star, and 1 UI!
New power output idea: Because of the astronomical resource cost it should produce 1M RF per tick, and need a Dyson Sphere Collector to gather energy from the Dyson Sphere, craftable with 5 Iridium on the bottom and in the top corners, 1 Advanced Machine Structure in the middle, a UI in the top middle, 1 Control Circuit on the Left, and 1 2x Small Battery on the left.

Dragonic evolution uses a lot more nether stars than just one.
On Tue, Oct 8, 2019, 1:47 PM NutMaster6942<EMAIL_ADDRESS>wrote:

New crafting idea: Large Iridium Plates, Dyson Plates, Dyson Controllers,
Dyson Support Pillar Segments, Precision Assembly Table
Large Iridium Plates would be used to make Dyson Plates, one Large Iridium
Plate would be crafted with 9 blocks of iridium,
Dyson Plates would be crafted using 9 Large Iridium Plates.
7 Dyson Plates around a nether star with a UI in the top centre would
craft a Dyson Controller.
9 Dyson Plates would craft a Dyson Support Pillar Segment.
9 Dyson Support Pillar Segments would craft a Dyson Support Pillar.
A Precision Assembly Table would be crafted in a Precision Assembler with
one Advanced Machine Structure, one Iridium block, one UI, and two 2x small
battery, it would have a 5x5 grid and craft certain advanced machinery, it
would require 10000 RF to craft one item.
A Dyson Sphere would be crafted in a Precision Assembly Table, it would
have one Dyson Controller in the centre, the corners would be Dyson Plates,
Dyson Plates would surround the Dyson Controller, and the remaining spots
would be Dyson Support Pillars
Total Crafting Cost: 81 Iridium per Large Plate, 729 Iridium per Dyson
Plate, 5103 Iridium, 1 nether star and 1 UI per Dyson Controller, 6561
Iridium per Dyson Support Pillar Segment, 59049 Iridium per Dyson Support
Pillar, 12 Dyson Plates in a Sphere, 12 Dyson Support Pillars in a Sphere,
and 1 Controller in a Sphere, 12x729 plus 12x59049 plus 7x729 = 8748 plus
12x59049 plus 7x729 = 8748 plus 708588 plus 7x729 = 8748 plus 708588 plus
5103 = 722439
So, total resource cost is 722439 Iridium, 1 nether star, and 1 UI!
New power output idea: Because of the astronomical resource cost it should
produce 1M RF per tick, and need a Dyson Sphere Collector to gather energy
from the Dyson Sphere, craftable with 5 Iridium on the bottom and in the
top corners, 1 Advanced Machine Structure in the middle, a UI in the top
middle, 1 Control Circuit on the Left, and 1 2x Small Battery on the left.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1714?email_source=notifications&email_token=AGVDBJF7RLA3UIA2TTZRXNTQNTITRA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAVGQII#issuecomment-539650081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVDBJBWUSHYF3TPV2KJMBLQNTITRANCNFSM4I6HFC6Q
.

I feel like another idea for crafting could be to have the dyson sphere be built in sections and not all at once. It would allow the cost to be truly astronomical while still having high rf output. Just an idea though

^ This is what we've been suggesting the entire time. And you can already basically do this. Launch as many solar satellites as possible, if you need to you can add a crafting recipe to them if they're not already automatable. Then link up many microwave receivers, as many as you need for your satellites.
Dyson spheres, if they weren't actually swarms of satellites, would need to be assembled in space anyway.
IMPORTANT PART
This is already possible on any scale attainable by a Survival player, as any Survival player is not working on an astronomical scale. Use many, many solar satellites and microwave receivers. You'll get as close to the result as you're going to get in a MC mod, so the discussion if moot other than making them able to orbit the sun.

Dragonic evolution uses a lot more nether stars than just one.
…
On Tue, Oct 8, 2019, 1:47 PM NutMaster6942 @.***> wrote: New crafting idea: Large Iridium Plates, Dyson Plates, Dyson Controllers, Dyson Support Pillar Segments, Precision Assembly Table Large Iridium Plates would be used to make Dyson Plates, one Large Iridium Plate would be crafted with 9 blocks of iridium, Dyson Plates would be crafted using 9 Large Iridium Plates. 7 Dyson Plates around a nether star with a UI in the top centre would craft a Dyson Controller. 9 Dyson Plates would craft a Dyson Support Pillar Segment. 9 Dyson Support Pillar Segments would craft a Dyson Support Pillar. A Precision Assembly Table would be crafted in a Precision Assembler with one Advanced Machine Structure, one Iridium block, one UI, and two 2x small battery, it would have a 5x5 grid and craft certain advanced machinery, it would require 10000 RF to craft one item. A Dyson Sphere would be crafted in a Precision Assembly Table, it would have one Dyson Controller in the centre, the corners would be Dyson Plates, Dyson Plates would surround the Dyson Controller, and the remaining spots would be Dyson Support Pillars Total Crafting Cost: 81 Iridium per Large Plate, 729 Iridium per Dyson Plate, 5103 Iridium, 1 nether star and 1 UI per Dyson Controller, 6561 Iridium per Dyson Support Pillar Segment, 59049 Iridium per Dyson Support Pillar, 12 Dyson Plates in a Sphere, 12 Dyson Support Pillars in a Sphere, and 1 Controller in a Sphere, 12x729 plus 12x59049 plus 7x729 = 8748 plus 12x59049 plus 7x729 = 8748 plus 708588 plus 7x729 = 8748 plus 708588 plus 5103 = 722439 So, total resource cost is 722439 Iridium, 1 nether star, and 1 UI! New power output idea: Because of the astronomical resource cost it should produce 1M RF per tick, and need a Dyson Sphere Collector to gather energy from the Dyson Sphere, craftable with 5 Iridium on the bottom and in the top corners, 1 Advanced Machine Structure in the middle, a UI in the top middle, 1 Control Circuit on the Left, and 1 2x Small Battery on the left. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1714?email_source=notifications&email_token=AGVDBJF7RLA3UIA2TTZRXNTQNTITRA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAVGQII#issuecomment-539650081>, or mute the thread https://github.com/notifications/unsubscribe-auth/AGVDBJBWUSHYF3TPV2KJMBLQNTITRANCNFSM4I6HFC6Q .

Yeah, but it needs 722439 iridium.

 Doing that many asteroid missions would just be grindy, and not fun at all.

Wither eliminating is more of a grind because the skeletons does not drop
skulls all the time, the chance is so low even with looting 3 it still be
hard. Therefore nether stars are more expensive than iridium.
 Doing that many asteroid missions would
just be grindy, and not fun at all.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1714?email_source=notifications&email_token=AGVDBJB6FEM5F6YH35TAFJTQNYUYFA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAZAGMQ#issuecomment-540148530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVDBJEZ5ILTWCMBF64ASQLQNYUYFANCNFSM4I6HFC6Q
.

Only 1 skull seems too small tho. And 722439 irridium is pretty unreallalistic without other mods to automate iridium production and storage

You did the math wrong. You need 3 to summon the wither.
On Wed, Oct 9, 2019, 4:04 PM TripTripy<EMAIL_ADDRESS>wrote:

Only 1 skull seems too small tho. And 722439 irridium is pretty
unreallalistic without other mods to automate iridium production and storage
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1714?email_source=notifications&email_token=AGVDBJC2EMBGHWL5ZVNW5ALQNZBORA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAZL74Y#issuecomment-540196851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVDBJDQPXS3HZAJLI3IL5DQNZBORANCNFSM4I6HFC6Q
.

I meant stars. 1 star for an entire dyson sphere is not much

The more I'm reading into these comments, the less I like this idea.
I suppose the idea was doomed to this in the first place, but I don't like the idea of a massive grind being required to build this thing, whether it be asteroid missions or mass-wither killing. I far prefer gated-progression over grind-based-progression.
I think it'd be better to make a Dyson sphere as a form of star you can go to rather than a ridiculously large number of satellites built by one player. You'd be able to land on the surface, which would be covered in some kind of cheap metal plating block.
Random structures would spawn that would either act as turrets defending the station or power stations for taking advantage of the sphere's infinite power supply, for the sake of recharging batteries or something else. (Maybe a station where the infinite power of the dyson sphere can turn quartz into dilithium or something? IDK)

I would actually really like to see a Dyson Sphere as a star type.

That sounds quite cool

Wouldn't the terrain be on the inside of the sphere?

Maybe? The inside would be sunlight collection. The defenses would be outside.

The inner and outer sides of a sphere would have to be basically two different dimensions if they were implemented, seeing as there's no real way to have upside-down gravity in the game (yet?) but also because realistically, a Dyson sphere would likely have to be miles thick in order to remain structurally intact, enough that the Dyson sphere itself should generate some surface gravity I reckon.
I'm not sure what the point would be of having a giant inner Dyson sphere biome, though. It would effectively look like a normal planet besides power collector structures and maybe a different skybox.

The inner and outer sides of a sphere would have to be basically two different dimensions if they were implemented, seeing as there's no real way to have upside-down gravity in the game (yet?) but also because realistically, a Dyson sphere would likely have to be miles thick in order to remain structurally intact, enough that the Dyson sphere itself should generate some surface gravity I reckon.
I'm not sure what the point would be of having a giant inner Dyson sphere biome, though. It would effectively look like a normal planet besides power collector structures and maybe a different skybox.

And loads of cities, possibly abandoned with loads of loot.

Galaxies might be a cool addition for something like this to work.

Again, the inside of a Dyson sphere would be power collection. What is the point of building the entire sphere if you're then immediately not going to use all of it for power, and use some for cities/etc?. Those would be on the outside. The best way to represent it would be juts the outer side.

Again, the inside of a Dyson sphere would be power collection. What is the point of building the entire sphere if you're then immediately not going to use all of it for power, and use some for cities/etc?. Those would be on the outside. The best way to represent it would be juts the outer side.

Wouldn't the cities be cold husks without the star giving them heat and light?

I just realized there is a config for the black hole generator.

the cities would have sealed buildings (or city ruins)

Wouldn't the cities be cold husks without the star giving them heat and light?
You're producing practically infinite power. You can spare some of it to heat and light your cities.

This by far the longest issue. This should be on another thread instead of
github.
On Tue, Oct 22, 2019, 8:44 PM Silfryi Kalsandryn<EMAIL_ADDRESS>wrote:

Wouldn't the cities be cold husks without the star giving them heat and
light?
You're producing practically infinite power. You can spare some of it to
heat and light your cities.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1714?email_source=notifications&email_token=AGVDBJFKAYOS6XUNSOEUAE3QP6UAHA5CNFSM4I6HFC62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB7X4SA#issuecomment-545226312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVDBJGVO4GLHXH3T37EPYLQP6UAHANCNFSM4I6HFC6Q
.

This by far the longest issue. This should be on another thread instead of github.

It's still got 20 comments to go until it's the largest issue.

This by far the longest issue. This should be on another thread instead of github.

It's still got 20 comments to go until it's the largest issue.

Roughly.

Damn, I didn't see this pretty interesting request getting so much attention. I'd love to see a thread that would discuss this more.

Having this be in another galaxy would make this more difficult to get to, as dilithium costs would spike.

aaaaaaaaaaaaaaaaaaa

Having this be in another galaxy would make this more difficult to get to, as dilithium costs would spike.

That is, if the max was increased from 500 to something an awful lot higher.

Yes. It's free infinite power, so it would need an exceedingly high setup cost to be balanced. That makes total sense.

Yes. It's free infinite power, so it would need an exceedingly high setup cost to be balanced. That makes total sense.
And any more conversation should be done on the Discord.

There's a Discord?

Yes, linked to on the readme of this repository

Dyson spheres are not happening.


Erase after typing (right to left after left to right)

Not sure if it is just an error by me or not but when I try to change the typing prop from 1 ➡ -1 once the phrase has completed typing the whole Text element disappears before it starts the deleting phase of the animation
check out a demo:
https://snack.expo.io/@peterw-ens/react-native-typewriter-change-directions

The issue is don't have a <Text/> element inside of <TypeWriter/>


Add git current directory path

@n0ts Not sure I understand what this does, those commands should already be ran in the current release folder by default no?


customizing when it comes to zooming a specific pdf file in linux

<What went wrong? (add screenshot)>

When browsing the pdf file using pdf.js viewer, I found out that when it comes to zooming,
there are the selected options (e.g 50%, 75%, 100%, etc).
how can I zoom in the file while using customized zooming version? (e,g 53% 20%)
<What is the expected behavior?>
There are no personally customized zooming options
Using the customized option , it would be much helpful for the ones who want to see the pdf file based on what they want to see the slide with the size they need
pdf-options

If you need a specific zoom level, the easiest solution is to append e.g. #zoom=20 (or whatever value you need) to the URL; please see https://github.com/mozilla/pdf.js/wiki/Viewer-options for additional details.

Thanks for the reply, and after entering (e.g 17%) in URL, it works.

and after giving several tries, found out the file is zoomed with the %(up to second decimal place)
(e.g #zoom = 80.50923 -> 80.51%; as shown above)


Roll engine 75387dbc147d..49445ce4949b (32 commits)

https://github.com/flutter/engine/compare/75387dbc147d...49445ce4949b
git log 75387dbc147d1cee8164f5d61582eab8ba1e5058..49445ce4949bf0d28506a6b22e28b4264f624e5f --no-merges --oneline
49445ce49 FLEViewController/Engine API changes (flutter/engine#9750)
2a79462fd Add Fuchsia build CI presubmit steps (flutter/engine#9736)
67cebdb92 Roll fuchsia/sdk/core/linux-amd64 from KGmm_RIJoXS19zTm2crjM3RYpmp5Y03-fLUeVdylbTYC to ehWVT9QJbC-vFMM6SkkQM9HJ9oITFCws7FC9JnrFq2gC (flutter/engine#9765)
089c74084 Roll fuchsia/sdk/core/mac-amd64 from lCQWEeR_Ert7t_qAbMRycwrRyZC-dIprYPyPJzwPmg4C to EYnRdXFT9l-d8Qkz4zeTRXnqfV3KQzpQhoPs1r0-740C (flutter/engine#9759)
b22410ef6 Include SkParagraph headers only when the enable-skshaper flag is on (flutter/engine#9758)
2cd650d26 Minimal integration with the Skia text shaper module (flutter/engine#9556)
f775f5e4d Re-enable the Wuffs GIF decoder (flutter/engine#9466)
aca048236 Make all shell unit tests use the OpenGL rasterizer. (flutter/engine#9746)
bc57291ff Make FLEViewController's view an internal detail (flutter/engine#9741)
9776043ea Synchronize main thread and gpu thread for first render frame (flutter/engine#9506)
f600ae830 Use libc++ variant of string view and remove the FML variant. (flutter/engine#9737)
564f53f0a Revert "Improve caching limits for Skia (#9503)" (flutter/engine#9740)
b453d3c3d libtxt: fix reference counting of SkFontStyleSets held by font asset providers (flutter/engine#9561)
fa7627d17 Fix backspace crash on Chinese devices (flutter/engine#9734)
56885f79b Let pushColorFilter accept all types of ColorFilters (flutter/engine#9641)
6dccb21e6 Roll src/third_party/skia 96fdfe0fe88e..af4e7b6cf616 (1 commits) (flutter/engine#9735)
8511d9b47 Roll fuchsia/sdk/core/mac-amd64 from byM-kyxL4bemlTYNqhKUfJfZoIUrCSzS6XzsFr4n9-MC to lCQWEeR_Ert7t_qAbMRycwrRyZC-dIprYPyPJzwPmg4C (flutter/engine#9742)
b3bf0a182 Roll fuchsia/sdk/core/linux-amd64 from I2Qe1zxgckzIzMBTztvzeWYsDgcb9Fw-idSI16oIlx8C to KGmm_RIJoXS19zTm2crjM3RYpmp5Y03-fLUeVdylbTYC (flutter/engine#9743)
7e568232a Fix windows test by not attempting to open a directory as a file. (flutter/engine#9745)
6cf0d1350 Roll src/third_party/skia a3ffaabcc4f2..96fdfe0fe88e (5 commits) (flutter/engine#9731)
49a00aed8 Fix Fuchsia build. (flutter/engine#9730)
b3bb39b0b Roll src/third_party/dart 06c3d7ad3a...09fc76bc51 (flutter/engine#9728)
2284210f4 Make the license script compatible with recently changed Dart I/O stream APIs (flutter/engine#9725)
ad582b508 Rework image & texture management to use concurrent message queues. (flutter/engine#9486)
1dcd5f52b Roll src/third_party/skia 6b82cf638682..a3ffaabcc4f2 (24 commits) (flutter/engine#9726)
129979cab Revert "Roll src/third_party/dart 06c3d7ad3a..7acecda2cc (12 commits)" (flutter/engine#9724)
8020d7e43 Roll src/third_party/skia 56065d9b875f..6b82cf638682 (3 commits) (flutter/engine#9718)
e24bd7846 Roll src/third_party/dart 06c3d7ad3a..7acecda2cc (12 commits)
3d2668c35 Reland isolate group changes
802bd1518 iOS platform view opacity (flutter/engine#9667)
3b6265b74 Roll src/third_party/dart b5aeaa6796..06c3d7ad3a (44 commits)
887e05233 Refactor ColorFilter to have a native wrapper (flutter/engine#9668)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (garyq@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


[Docs] Fix typo in create page module guide

Description
Fixes a small typo in the "Creating the page module" Page builder guide.

Related Issues

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


[CARBONDATA-2421] Remove unused code of CarbonGetTableDetailCommand

CarbonGetTableDetailCommand is unused by any other class or method, better to remove it as well as the related tests.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed?
NO
[x] Any backward compatibility impacted?
NO
[x] Document update required?
NO
[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
Removed tests
- How it is tested? Please attach test report.
Tested in local
- Is it a performance related change? Please attach the performance test report.
NA
- Any additional information to help reviewers in testing this change.
NA
[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5549/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4386/


libneko.* is not needed and may lead to crash in neko target

See the discussion over https://github.com/andyli/hxSerial/issues/7.
Probably OpenFL should not add libneko.* by default, and then provide a way to opt-in it.

Does this affect using Neko builds standalone, with no Neko installed on the system? I had assumed this was a core Neko file that was needed, but if not (and it causes problems) then we could remove it

libneko.* is required if the user doesn't have neko installed, without it the app won't start.

The interesting thing is, if libneko.* is present together with a "NEKO_COMPATIBLE" ndll, the app will crash. I'm not sure so I will let @hughsando to explain the technical detail.

Perhaps it is the opposite? Wasn't it crashing without being NEKO_COMPATIBLE?
OpenFL and Lime use NEKO_COMPATIBLE, and haven't experienced crashes

The hxSerial.ndll is NEKO_COMPATIBLE, compiled with hxcpp 3.1.39.


Speeds up Airlocks

I thought the airlocks were super sluggish without a real need for them to be. I halfed their open/close times and I think they look much snappier, plus they mesh with the airlocks "when it lets you through" state much better.
Only modified the basic airlock in-case this pr in case it gets shot down immediately.
Comparison of different timings. (Blue is original timing, Green is quicker timing)

I like it.

>hasnt died because the airlock didnt open in time

Ok i'll go through and edit all the airlocks. This should be ready for merge next commit.

Added everything except for the external airlocks, since I think they're weightiness is warranted. Walking around the station during testing was so nice.

Updated the C#. Put it at 0.8f from 1.2f it feels ok to me but could need some tweaking.


[Select] How to hide the popup on scroll?

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
When the Select  component is open scrolling is disabled. This works on desktop but not on mobile (ios). So when scrolling the page the selectbox stays open and does not move along with the scrolled content.
Expected Behavior 🤔
It would be really nice if the Select used more of the native select features. Instead of disabling the scroll of the body the open selectbox should be closed when scrolling. On mobile, if not closing the selectbox when scrolling the body, it should at least move along with the content.
Steps to Reproduce 🕹
Steps:

Visit https://material-ui.com/components/selects/ on computer.
Open one of the selectboxes and scroll the page. Scrolling is NOT allowed and the selectbox stays open.
Visit https://material-ui.com/components/selects/ on iOS device.
Open one of the selectboxes and scroll the page. Scrolling IS allowed and the selectbox stays open but does not move along with the content.

Context 🔦
I feel like this is an unfinished component in that sense that as a user I'm not expecting this behavior to occur, either on desktop or mobile.
Your Environment 🌎

`npx @material-ui/envinfo`
  System:
    OS: Windows 10 10.0.19041
  Binaries:
    Node: 14.15.4 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 7.5.4 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 88.0.4324.150
    iOS Safari: 12.4.8
  npmPackages:
    @material-ui/core: ^4.11.3 => 4.11.3
    @material-ui/icons: ^4.11.2 => 4.11.2
    @material-ui/styles:  4.11.3
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react:  17.0.1
    react: ^16.0.0 => 16.12.0
    react-dom: ^16.0.0 => 16.12.0
    typescript: ^3.7.5 => 3.7.5

I'm also a bit confused why I'm not able to select aSelect while another is open. In essence I have to double click to open another Select if one is already open. One click to close the open one, and one click to select the other.

@DavidVonAmri We recommend using the native select anytime possible

Okey, but I'm guessing all of the components using the "popup box" has the same issue as described. Like Menu, Date/Time and so on?

Perhaps it would be possible to take a look at the Popover element and not just the Select.
Ideally it would work something like this:
Desktop

Do not disable scroll of body, instead close the Popover when doing so.
Accept interaction with any other element even if a Popover is open, doing so would close the already open Popover.
When scroll of the Popover reaches top or bottom no scrolling is bubbled up to the body making it scroll and close the Popover.

Mobile or below certain breakpoint?

Reveal opened Popover from bottom of viewport. Make Popover 100% of viewport. Take inspiration from iOS?
Show overlay when Popover  is open, when overlay is clicked or scrolled close the Popover.
Add "Close" button at bottom on all open Popovers.
Not sure if nested Popovers are available, if they are theses will be opened with a higher index.

Just my thoughts.

I note this issue has been open for some time and labelled an enhancement. In the meantime, is there a work-around to close the drop down on scroll?

Found quick solution for this issue
import { useScrollTrigger } from '@mui/material';
in component:
 const trigger = useScrollTrigger();

   useEffect(() => {
    trigger && isOpenMenu && handleClose();
  }, [trigger, isOpenMenu]);

it closes not so beautful as onClick but at laest it works 😁

useScrollTrigger only triggers when scroll direction changes, so it will not trigger if you've scrolled already in a direction, then opened the dropdown then continued to scroll in the same direction.


Question on table created by sql-cache command

A community member has asked on the aspnet/Docs repo about the table created with the sql-cache create command, which is described in the docs at ...
https://docs.microsoft.com/aspnet/core/performance/caching/distributed#using-a-sql-server-distributed-cache
Here is the table created by the tooling ...

@wanton7 asks ...

Isn't there 900 byte limit for indexes? nvarchar will take 2 bytes for every character so 900 will be 1800 bytes. That Id column should probably be nvarchar(450) or varchar(900).

https://github.com/aspnet/Docs/issues/8383

The code was updated at some point (looks like quite a while ago), so I'll update our doc to reflect the change.


Make "next question" button superfluous

Wouldn't it be better, if the next question would be shown when you've chosen an answer? That would reduce the required clicks per question by 1 click.

I prefer that the user has the possibiltyto do it himself. Currently the user has the possibility to review the answer before he/ she continues the test.


Object.entries correct typings

Search Terms
Object.entries
Potentially related issues:
#32771
Suggestion
Typings of Object.entries() returned [key, value] tupple array is very weak and infer poorly from argument, I think it would be better to implement more relevant ones in lib.es2017.object.d.ts
Use Cases
When iterating on an object entries array returned by Object.entries, we need to manually type cast the array to avoid false positive and false negative typings shortcomings.
const a = Object.entries({foo: "bar", baz: 0}); // [string, string | number][]
const b = Object.entries({foo: "bar", baz: 0} as const); // [string, 0 | "bar"][]

It is not possible to infer typings in .mapor .forEach
a.forEach(entry => {
  if(entry[0] === "foo") {
    let value = entry[1]; // string | number
    entry[1].toUpperCase(); // error (false positive)
  } else if (entry[0] === "baz") {
    let value = entry[1]; // string | number
  } else if (entry[0] === "invalid") { // no error (false negative)
    // ...
  }
})

b.forEach(entry => {
  if(entry[0] === "foo") {
    let value = entry[1]; // 0 | "bar"
  }
})

Examples
We can easily write our own entries function:
type Entries<T> = {
  [K in keyof T]: [K, T[K]]
}[keyof T][]

function entries<T>(obj: T): Entries<T> {
  return Object.entries(obj) as any;
}

Wich gives me an array of tupples with inferables key/value:
const c = entries({foo: "bar", baz: 0}); // (["foo": string] | ["baz": number])[]
const d = entries({foo: "bar", baz: 0} as const); // (["foo": "bar"] | ["baz": 0])[]

c.forEach(entry => {
  if (entry[0] === "foo") {
    let value = entry[1]; // string
    entry[1].toUpperCase(); // no error
  } else if (entry[0] === "baz") {
    let value = entry[1]; // number
  }
  if (entry[0] === "invalid") { // error: "baz" and "invalid" have no overlap
    // ...
  }
});

d.forEach(entry => {
  if (entry[0] === "foo") {
    let value = entry[1]; // "bar"
  } else if (entry[0] === "baz") {
    let value = entry[1]; // 0
  }
  if(entry[1] === "bar") {
    let key = entry[0]; // "foo"
  }
});

We may implement a better default behaviour for Object.entries in the lib:
  /**
   * Returns an array of key/values of the enumerable properties of an object
   * @param o Object that contains the properties and methods. This can be an object that you created or an existing Document Object Model (DOM) object.
   */
  entries<T>( o: T): T extends ArrayLike<infer U> ? [string, U][] : { [K in keyof T]: [K, T[K]] }[keyof T][];

The same concepts can be applied for better typings of Object.values
/**
   * Returns an array of values of the enumerable properties of an object
   * @param o Object that contains the properties and methods. This can be an object that you created or an existing Document Object Model (DOM) object.
   */
  values<T>(o: T): T extends ArrayLike<infer U> ? U[] : (T[keyof T])[];

Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code

I need some feedback to identify potential breaking changes.
I think it 'may' induce breaking change since it's reducing typecast necessity. If manual casts are incompatible with the current feature request, there may be a need to remove cast or force through an intermediary any.
I tested it in a project of mine and had no issue since the existing cast was correctly made with the same source of truth as the Object.values argument type.

[ ] This feature would agree with the rest of TypeScript's Design Goals.

I can't say I'm sure it does. Feedback would be appreciated.

Duplicate of #20322. The issue boils down to: https://github.com/microsoft/TypeScript/issues/20322

Also see https://stackoverflow.com/questions/55012174/why-doesnt-object-keys-return-a-keyof-type-in-typescript for rationale.

Hello @MartinJohns @fatcerberus and @RyanCavanaugh
I red previous issues, the 2016 Anders comment and the stackoverflow answer.
I am not convinced though, I thinks some assumptions can be challenged in regards of the current state of typescript.
In particular, contravariance issues are widespread and don't always lead to any fallback.
Would you mind refreshing this topic and allowing me to advocate another opinion ?

If you think there's some solution available, please do describe

So any further work on this? Okay, if object can hold more properties than keyof T then typesystem should state that it returns at least these properties and maybe some more. While it would require to handle defualt cases this code would always typecheck without additional casts:
const source: SomeType = ....;
const result: SomeType  = Object.assign({}, ...Object.entries(source).map(([k, v]) => ({[k]: v})));

While it may process some extra properties that aren't part of SomeType it still says that entries contain at least keyof SomeType values, which is enough to construct a valid SomeType instance

I've also found a great answer about how to restrict object from having extra properties. This may help as well: https://stackoverflow.com/questions/49580725/is-it-possible-to-restrict-typescript-object-to-contain-only-properties-defined

i had to do smth like this:
 {Object.entries(navigation).map(([key, value]) => (
              <DropdownItem
                key={key}
                item={{ pathname, lang, key, value }}
              />
            ))}

and as you might guess key, value was not inferred properly
so I did something like this:
type Entry<T> = [keyof T, T[keyof T]]
  function objectEntries<T extends object>(obj: T): Entry<T>[] {
    return (Object.keys(obj) as Array<keyof T>).map((key) => [
      key,
      obj[key],
    ]) as Entry<T>[];
  }

and transform the first part to:
    {objectEntries(navigation).map(([key, value]) => (
              <DropdownItem key={key} item={{ pathname, lang, key, value }} />
            ))}

i had to do smth like this:
 {Object.entries(navigation).map(([key, value]) => (
              <DropdownItem
                key={key}
                item={{ pathname, lang, key, value }}
              />
            ))}

and as you might guess key, value was not inferred properly
so I did something like this:
type Entry<T> = [keyof T, T[keyof T]]
  function objectEntries<T extends object>(obj: T): Entry<T>[] {
    return (Object.keys(obj) as Array<keyof T>).map((key) => [
      key,
      obj[key],
    ]) as Entry<T>[];
  }

and transform the first part to:
    {objectEntries(navigation).map(([key, value]) => (
              <DropdownItem key={key} item={{ pathname, lang, key, value }} />
            ))}

everything is typed OK now

worked for me on TS 4.8.4!!


chg: Bump max python version, deps

Just bumping all the dependencies and add support for python 3.12 so I can also enable python 3.12 on pandora.

Thank you Raphaël!

Damn I merged into main. 🏎

Released!

Thank you!


add readerModeDelay param

Hello!
I was noticed that after detection one tag should pass about 10 seconds to detect another (because of NfcAdapter.EXTRA_READER_PRESENCE_CHECK_DELAY). It could be not very convenient when you read one-to-one tags. I decided to add new parameter readerModeDelay which set delay between tag detection. Default value of parameter is 10 seconds (to save backward compatibility).

@AlexandraOlegovna thanks for the contribution!


Minor Typo

Minor Typo FYI.
At https://github.com/TileDB-Inc/TileDB-ML/blob/master/tests/test_pytorch_dataloader_api.py#L1, probably forgot to change TensorFlow to PyTorch.

Minor Typo FYI.
At https://github.com/TileDB-Inc/TileDB-ML/blob/master/tests/test_pytorch_dataloader_api.py#L1, probably forgot to change TensorFlow to PyTorch.

Thanks @RAbraham ! Will take care of it.


change ssl option type in Microserver

SSL option type needs to be boolean.

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/478/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/487/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/475/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2170/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/2066/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/601/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/730/


lib: custom_widgets: Animate simulation card when removed/added to favourite

Currently the card just pops in and out when changing favorite status, an animation on the card would look better.

I would like to work on this.

Sure! Let me know if you have any problem. If contributing under GSSoC, make sure you discuss before opening a PR on slack, it make the review process easier for us.

Sure! Will you assign me to the issue ?

I would love to work on this under Gssoc21 @yashlamba

I would love to work on this under Gssoc21 @yashlamba

@pawan2806 please go ahead!


Migrate to JUnit 5

I've updated a lot of the remaining tests to JUnit 5
I think this will leave the tests extending the following to do:

AbstractDateConverterTest
AbstractNumberConverterTest
BaseLocaleConverterTest

I've added the changes for static imports. I've also reverted the visibility changes. I think I've caught all of them, but will have another look later. My fault for trusting my IDE's refactoring too much :)

@stevebosman
Thank you for your update. Yeah, all IDEs I've tried mess up the conversion or leaves you with code that does not even compile (I'm looking at you IntelliJ...)
Another item that would be nice to have (but don't feel obligated):
assertTrue(e.toString().contains("Unknown property"), "Wrong exception was thrown: " + e);

->
assertTrue(e.toString().contains("Unknown property"), () -> "Wrong exception was thrown: " + e);

which will avoid creating garbage Strings in the majority of build runs.

I have fixed that one and the few others in BeanComparatorTestCase. There are similar in some other test files, but my instinct is they should be addressed in another much smaller PR.


button_count is corrupted when no pointer release is encountered

I am writing a program and I had it use right click as a means of exiting the event loop. I was trying to validate my serial to initiate a drag and drop and noticed that the serial in sway debug logs weren't being validated, but the button_count also seemed incorrect. It also kept incrementing.
I realized that by using right click to exit the program, button_count wasn't being decremented.
Tracking increments and decrements seems like it is a difficult balance act and could easily get corrupted if we encounter a state which is unaccounted for, so it seems like there needs to be an appropriate point at which we should reset button_count so as to at least have a catch-all in case we screw up.
In this case, if the surface originating a pointer event is destroyed, then it makes sense to reset the state.

potentially related to  swaywm/sway#4168

Related: https://github.com/swaywm/wlroots/issues/1593


Installing from requirements fails

I'm using pip 9.0.1 and having issues installing prophet from a requirements file.
Example requirements.txt:
Cython==0.24
fbprophet==0.1.post1
numpy==1.12.0
pandas==0.19.2
pystan==<IP_ADDRESS>
python-dateutil==2.6.0
pytz==2016.10
six==1.10.0

(These should be the bare requirements to install prophet)
Here is the output of a pip install:
# pip install -r requirements.txt --no-cache-dir
Collecting Cython==0.24 (from -r requirements.txt (line 1))
  Downloading Cython-0.24-cp27-cp27mu-manylinux1_x86_64.whl (5.9MB)
    100% |████████████████████████████████| 5.9MB 33.8MB/s
Collecting fbprophet==0.1.post1 (from -r requirements.txt (line 2))
  Downloading fbprophet-0.1.post1.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-7F2dAF/fbprophet/setup.py", line 96, in <module>
        """
      File "/usr/lib64/python2.7/distutils/core.py", line 112, in setup
        _setup_distribution = dist = klass(attrs)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/dist.py", line 317, in __init__
        self.fetch_build_eggs(attrs['setup_requires'])
      File "/tmp/env/lib/python2.7/site-packages/setuptools/dist.py", line 372, in fetch_build_eggs
        replace_conflicting=True,
      File "/tmp/env/lib/python2.7/site-packages/pkg_resources/__init__.py", line 851, in resolve
        dist = best[req.key] = env.best_match(req, ws, installer)
      File "/tmp/env/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1123, in best_match
        return self.obtain(req, installer)
      File "/tmp/env/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1135, in obtain
        return installer(requirement)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/dist.py", line 440, in fetch_build_egg
        return cmd.easy_install(req)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 674, in easy_install
        return self.install_item(spec, dist.location, tmpdir, deps)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 700, in install_item
        dists = self.install_eggs(spec, download, tmpdir)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 881, in install_eggs
        return self.build_and_install(setup_script, setup_base)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1120, in build_and_install
        self.run_setup(setup_script, setup_base, args)
      File "/tmp/env/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1108, in run_setup
        raise DistutilsError("Setup script exited with %s" % (v.args[0],))
    distutils.errors.DistutilsError: Setup script exited with Cython>=0.22 and NumPy are required.

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-7F2dAF/fbprophet/

If fbprophet is removed from the requirements file, everything installs as expected - also important - fbprophet will install properly if a pip install fbprophet is executed after.
I don't know the internals of pip (and this could very well be an issue with pip) but it looks like easy_install is attempting to run before cython and numpy are installed. I have never seen this behaviour, so I decided to post the issue on this package.
This makes it very complicated to integrate fbprophet in our deployment methodology (packages MUST be installed via a requirements file).
Tested in both macos sierra and Centos 7.

It looks like the error is being raised by PyStan here:
https://github.com/stan-dev/pystan/blob/e9fc7336c338417232497217f4028a0f2654b1e6/setup.py#L221
I'm really fuzzy on why this would be caused by including fbprophet in the requirements but not just by including pystan. What do you have for setup_requires and install_requires in your setup.py maybe?

If there are any further issues here please reopen, but I'm going to close for now since I don't yet see any changes to be made to Prophet.

I have the same problem when including Prophet in my setup.py requirements 😢  I have tried adding Prophet to install_requires then setup_requires without success.

This was fixed by 231 I believe. The reason I was getting this was because I used v1.0.

Oh, wonderful, thanks for figuring that out!

Any plans to release this fix? The issue is still present in the most recent version on pypi (0.1.1).

Coming out later this week!

Thanks @bletham!


Deprecated zksolc compiler version in hardhat config file

I get an error during zk init command execution at the commit 1283a410c918568808de7897f3b4bb6b57da98a0:
> Compile l2 contracts
yarn run v1.22.19
$ /home/zk-module-experimental/node_modules/.bin/hardhat compile
Nothing to compile
Done in 0.52s.
yarn install v1.22.19
[1/4] Resolving packages...
success Already up-to-date.
Done in 0.28s.
yarn run v1.22.19
$ /home/zk-module-experimental/etc/system-contracts/node_modules/.bin/hardhat compile
Error in plugin @matterlabs/hardhat-zksync-solc: Deprecated zksolc compiler version in hardhat config file (1.3.1). Please use versions 1.3.13 to 1.3.13.

when I tried to change the version from 1.3.1 to 1.3.13 I got an error
$ /home/zk-module-experimental/etc/system-contracts/node_modules/.bin/hardhat run ./scripts/compile-yul.ts
Creating Typechain artifacts in directory typechain for target ethers-v5
Successfully generated Typechain artifacts!
Error: (0 , hardhat_zksync_solc_1.getZksolcPath) is not a function

my command history
 2083  zk
 2084  zk init
 2085  cargo update --package regex
 2086  zk init
 2087  cd etc/system-contracts/
 2088  yarn add -D @matterlabs/hardhat-zksync-solc
 2089  cd ../../
 2090  zk clean --all && zk init

could you please help me with that?

sudo snap install solc
this might help you

`

Compile l2 contracts
yarn run v1.22.19
$ /usr/src/zksync/node_modules/.bin/hardhat compile
Nothing to compile
Done in 2.77s.
yarn install v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
Done in 68.53s.
yarn run v1.22.19
$ /usr/src/zksync/etc/system-contracts/node_modules/.bin/hardhat compile
Error in plugin @matterlabs/hardhat-zksync-solc: Deprecated zksolc compiler version in hardhat config file (1.3.7). Please use versions 1.3.13 to 1.3.13.

ZkSyncSolcPluginError: Deprecated zksolc compiler version in hardhat config file (1.3.7). Please use versions 1.3.13 to 1.3.13.
at Function.getDownloaderWithVersionValidated (/usr/src/zksync/etc/system-contracts/node_modules/@matterlabs/hardhat-zksync-solc/src/compile/downloader.ts:55:23)
at async /usr/src/zksync/etc/system-contracts/node_modules/@matterlabs/hardhat-zksync-solc/src/index.ts:77:34
at async OverriddenTaskDefinition._action (/usr/src/zksync/etc/system-contracts/node_modules/@matterlabs/hardhat-zksync-solc/src/index.ts:76:5)
at async Environment._runTaskDefinition (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:308:14)
at async Environment.run (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:156:14)
at async SimpleTaskDefinition.action (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/builtin-tasks/compile.ts:1308:9)
at async Environment._runTaskDefinition (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:308:14)
at async Environment.run (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:156:14)
at async SimpleTaskDefinition.action (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/builtin-tasks/compile.ts:1411:7)
at async Environment._runTaskDefinition (/usr/src/zksync/etc/system-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:308:14)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Child process exited with code 1
`
I have the same issue.

sudo snap install solc this might help you

after that I got
> Building contracts
yarn run v1.22.19
$ yarn workspace l1-zksync-contracts build
$ hardhat compile
Nothing to compile
Creating Typechain artifacts in directory typechain for target ethers-v5
Successfully generated Typechain artifacts!
Done in 6.28s.
yarn run v1.22.19
$ yarn workspace l2-zksync-contracts build
$ hardhat compile
Solidity 0.8.19 is not fully supported yet. You can still use Hardhat, but some features, like stack traces, might not work correctly.

Learn more at https://hardhat.org/hardhat-runner/docs/reference/solidity-support

Error HH409: Trying to import ./interfaces/IL2Weth.sol from cache-zk/solpp-generated-contracts/bridge/L2WETH.sol, but it has an incorrect casing.

For more info go to https://hardhat.org/HH409 or run Hardhat with --show-stack-traces
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed.
Exit code: 1
Command: /home/.nvm/versions/node/v16.19.1/bin/node
Arguments: /home/.yarn/lib/cli.js build
Directory: /home/zk-module-experimental/contracts/zksync
Output:

info Visit https://yarnpkg.com/en/docs/cli/workspace for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Child process exited with code 1

solc version is Version: 0.5.16+commit.9c3226ce.Linux.g++

https://github.com/matter-labs/zksync-era/issues/51#issuecomment-1610893857

thanks, it worked!

Nice!

should be working with no kludges in current head of main branch. please feel free to create another issue of it does not

sudo snap install solc this might help you

Thanks a lot


Fix: Select Overflow Restore

[x] I have followed (at least) the PR section of the contributing guide.

This solves #27486

Sorry for the commit names inconsistency by the way

No idea why the test failed...

@eps1lon may you help me out with this one? Any idea why the test took so long?

Great, I had a feeling the tests simply needed a restart. May we merge now?

Great, I had a feeling the tests simply needed a restart. May we merge now?

Sure, just needed to make sure we don't merge unnecessary test changes.

@PCOffline  It's a great first pull request on Material-UI. Thank you for working on it!

Thank you for helping me out with it! <3


Update clean part of the example doc

Pull Request Checklist

[ ] implement the feature
[ ] write the documentation
[ ] extend the test coverage
[ ] update the specification
[ ] adjust plugin docstring
[ ] modify the json schema
[ ] mention the version
[ ] include a release note

This is really a tiny  merge request, which is pretty safe to be merged directly:)

Updated^^
On Mon, Jul 8, 2024 at 7:25 PM Miloš Prchlík @.***>
wrote:

@.**** commented on this pull request.
In docs/examples.rst
https://github.com/teemtee/tmt/pull/2883#discussion_r1668455527:

@@ -1539,11 +1539,11 @@ Clean workdirs

The goal of clean runs is to remove workdirs of past runs.
Similarly to above, /var/tmp/tmt is used by default as run
-location and this can be changed using an argument:
+location and this can be changed using --workdir-root:
How about "... using --workdir-root option:`"
—
Reply to this email directly, view it on GitHub
https://github.com/teemtee/tmt/pull/2883#pullrequestreview-2163017182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKFR23DCMFATZUW46AWC4S3ZLJZKVAVCNFSM6AAAAABGV7WVUSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCNRTGAYTOMJYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Documentation-only change, not waiting for full functional tests.


Fix furi crash

Fixes #9
Also cleans up some double function calls.
While @EstebanFuentealba and I briefly chatted about moving pins due to conflicts with Flipper firmware; this can still work (just need to reboot the Flipper Zero before using NFC again), and once the API settles in OFW 1.0 that can be a choice made then. For now, it doesn't make sense to re-do all of the photos and graphics in this repo.

Tested and working against 0.84.2


Update scache to 4.0.2

Updates com.evolutiongaming:scache from 4.0.0 to 4.0.2.
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!

Files still referring to the old version number
The following files still refer to the old version number (4.0.0).
You might want to review and update them manually.
project/Dependencies.scala

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

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

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #378.


Use consistent spacing for app workflow nodes

Fixes
Fixes RHTAPBUGS-157
Description
Update the computations for widths of the app workflow nodes to be more accurate and handle the group nodes when there are no children.
Type of change

[x] Bugfix

Screen shots / Gifs for design review

/approve
/lgtm


Failover group Update not detecting existing Integration types

Local Environment:
Terraform v1.3.4
on linux_amd64

provider registry.terraform.io/snowflake-labs/snowflake v0.56.3

Describe the bug
When failover group is updated, the existing integration types are not detected.
2023-02-13T12:07:25.975Z [INFO]  provider.terraform-provider-snowflake_v0.56.3: 2023/02/13 12:07:25 [DEBUG] sql-conn-exec [query ALTER FAILOVER GROUP FOG_ALL_EXCEPT_DB SET OBJECT_TYPES = INTEGRATIONS,USERS,WAREHOUSES,ROLES,NETWORK POLICIES,ACCOUNT PARAMETERS,RESOURCE MONITORS err 500043 (42601): ALLOWED_INTEGRATION_TYPES should be specified when INTEGRATIONS is part of OBJECT_TYPES. duration 54.572167ms args {}]: timestamp=2023-02-13T12:07:25.975Z
│ Error: error updating object types for failover group FOG_ALL_EXCEPT_DB err = 500043 (42601): ALLOWED_INTEGRATION_TYPES should be specified when INTEGRATIONS is part of OBJECT_TYPES.
│
│   with snowflake_failover_group.source[0],
│   on main.tf line 563, in resource "snowflake_failover_group" "source":
│  563: resource "snowflake_failover_group" "source" {
Expected behavior
Alter statement should include the allowed integrations.
Code samples and commands
Version 1:
resource "snowflake_failover_group" "source_failover_group" {
  name                      = "FG1"
  object_types              = ["WAREHOUSES", "DATABASES", "INTEGRATIONS", "ROLES"]
  allowed_accounts          = ["<org_name>.<target_account_name1>", "<org_name>.<target_account_name2>"]
  allowed_databases         = [snowflake_database.db.name]
  allowed_integration_types = ["SECURITY INTEGRATIONS"]
  replication_schedule {
    cron {
      expression = "0 0 10-20 * TUE,THU"
      time_zone  = "UTC"
    }

    // replication_schedule could also be specified with interval instead of cron
    // interval = 10
  }
}

Version 2: (Removing warehouse now)
resource "snowflake_failover_group" "source_failover_group" {
  name                      = "FG1"
  object_types              = ["DATABASES", "INTEGRATIONS", "ROLES"]
  allowed_accounts          = ["<org_name>.<target_account_name1>", "<org_name>.<target_account_name2>"]
  allowed_databases         = [snowflake_database.db.name]
  allowed_integration_types = ["SECURITY INTEGRATIONS"]
  replication_schedule {
    cron {
      expression = "0 0 10-20 * TUE,THU"
      time_zone  = "UTC"
    }

    // replication_schedule could also be specified with interval instead of cron
    // interval = 10
  }
}

@venkad-intc the failover group resource was completely refactored in the 0.65 release of the provider and a specific test was added to verify that removing a warehouse (like the example you have included) works in the update step: https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/pkg/resources/failover_group_acceptance_test.go#L70. So this should no longer be an issue. Please let me know if there is any other trouble.


eigenpy: 3.1.1 -> 3.1.2

Description of changes
ref. https://github.com/stack-of-tasks/eigenpy/releases/tag/v3.1.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 266438 run on x86_64-linux 1

8 packages failed to build:

python310Packages.crocoddyl
python310Packages.example-robot-data
python310Packages.hpp-fcl
python310Packages.pinocchio
python311Packages.crocoddyl
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.pinocchio

2 packages built:

python310Packages.eigenpy
python311Packages.eigenpy

Result of nixpkgs-review pr 266438 run on x86_64-linux 1

2 packages failed to build:

python310Packages.crocoddyl
python311Packages.crocoddyl

8 packages built:

python310Packages.eigenpy
python310Packages.example-robot-data
python310Packages.hpp-fcl
python310Packages.pinocchio
python311Packages.eigenpy
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.pinocchio

Result of nixpkgs-review pr 266438 run on x86_64-linux 1

10 packages built:

python310Packages.crocoddyl
python310Packages.eigenpy
python310Packages.example-robot-data
python310Packages.hpp-fcl
python310Packages.pinocchio
python311Packages.crocoddyl
python311Packages.eigenpy
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.pinocchio

@ofborg build python311Packages.eigenpy


Enable Direct Video Download Within the App

Is your feature request related to a problem? Please describe.
Currently, there is no direct option to download videos within the app, making it complicated to save content for offline viewing or sharing. This can be frustrating for users who want to access videos without an internet connection or share content with friends who don't use the app.
Describe the solution you'd like
I would like a feature that allows direct downloading of videos within the app. This functionality would make accessing content more convenient and enable users to save their favorite videos for later viewing. Additionally, this option is already available on platforms like Instagram Reels and TikTok.
Describe alternatives you've considered
Alternatives include screen recording the video during playback or external tools available that allow users to download videos from the app, but having a native feature would be more practical.
Additional context
Implementing this feature would not only improve the user experience but also align the app with market practices, making it more competitive. Facilitating access to and sharing of content can increase user engagement and satisfaction.

I'd also like to see this.  Hopefully my show of interest might help prioritize the feature.


Entitlement enforcement

This change will enforce entitlements if the "entitlements" key is present in the x-rh-identity header.
If the "entitlements" key is present, the user must be entitled for either "insights" or "hybrid_cloud". I neither is entitled a 403 response is returned

@gtanzillo Is this ready to go?