feat: support react-native

Summary
Hello guys! basically I'm trying to finalize a PR that was adding support for react-native.
Can find out a little more about it here: https://github.com/gregberge/xstyled/pull/273, thanks @diegotsi
My idea is basically the same as this PR except that I'll be adding the default use without utility props and the x function with utility props.
Test plan
const WithoutUtilityProps = styled.View`
  background-color: cool-gray-500;
  padding: 2;
`;

const WithConfig = styled.View.withConfig({})`
  background-color: rose-900;
  padding: 2;
`;

const StyledSomeComponent = styled(View)`
  background-color: teal-900;
  padding: 2;
`;

const WithUtilityProps = styled.ViewBox``;

const UsingAS = styled.View``;

const Test = () => {
  return (
    <ThemeProvider theme={defaultTheme}>
      <x.View p={2} bg="blue-gray-900">
        <x.Text color="blue-gray-500">Text X elements</x.Text>
      </x.View>
      <UsingAS as={x.Text} p={2} bg="emerald-500">
        Now is a x.Text with all props from Text (RN) + utility props from
        xstyled
      </UsingAS>
      <WithUtilityProps p={2}>
        <x.Text>With Utility Props</x.Text>
      </WithUtilityProps>
      <WithConfig>
        <x.Text color="white">With Config</x.Text>
      </WithConfig>
      {/*@ts-expect-error */}
      <WithoutUtilityProps bg="black">
        <x.Text>Without Utility Props</x.Text>
      </WithoutUtilityProps>
      <StyledSomeComponent>
        <x.Text color="white">Styled(SomeComponent)</x.Text>
      </StyledSomeComponent>
    </ThemeProvider>
  );
};

Working with the latest react-native template:

Thanks for this @igorwessel 🚀
This will help us a lot

Nice @igorwessel! 😄

I prefer @xstyled/styled-components/native over @xstyled/styled-components-native due to all the duplication.
Maybe there is a way to trim this down and get rid of the dependency for react-native itself?

Maybe there is a way to trim this down and get rid of the dependency for react-native itself?

We can get rid of the dependency of react native using the approach of creating the components using the styled-components itself, on line:
https://github.com/igorwessel/xstyled/blob/1eda12ab85de3c86ac3f4d4a0d35f91a9168f997/packages/styled-components-native/src/createX.ts#L31-L36
We can use
get() {
        // @ts-ignore
        return xstyled(scStyled[tag]``)``
      }

I realized that a lot of the code is duplication that is not worth it to move in new package. I made some changes in the functions that create the styled, xstyled, css to receive as parameters the dependencies of the styled, css function that will be used. We were able to have these functions with greater flexibility, which allows us to reuse them both for native/web.
I have separated the responsibilities of defining an interface into new functions so that we can use the functions that create styled,xstyled without an interface and then define from those returned values. e.g:
const { scStyled, styled, xstyled } = createStyled(scStyled, css, generator)

Object.keys(scStyled).forEach(key => styled[key] = styled(key))

Which allows us to reuse them both for native/web.
Finally I created a package inside styled-components so we can use @xstyled/styled-components/native

We can still have them as a separate package, and just put it as a styled-components dependency.
With separate package allow people to just install the native package.

🚀

@gregberge When you have time, can you take a look?

@igorwessel CI fails, could you try to fix it?

I needed to update the react-simple-editor dependency to @0.11.3 as it doesn't exactly include react@^16 peer-deps. So the NPM can resolve the dependencies without needing to set the flag --legacy-peer-deps.
I'll be sure to find some time this weekend to write something in the docs about react-native.

@igorwessel after the doc, I could merge it.

Thanks @igorwessel  can you rebase please?

@gregberge Sorry for the delay, I had some problems but the rebase is done. 👍

@gregberge Hey Greg, how you doing?
Have a any expectation to includes this support to the RN? It's will be help us a lot.

@igorwessel can you rebase it again? It's a nice feature and an awesome improvement for this library.

@probablyup @gregberge Is it ok for merge?

@gregberge 🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏

Will review this weekend :)

Any progress here? 👀


[Feature request] Add a link to GitHub API Token generator.

As someone who's never used the GitHub API before, I'd like a link to https://github.com/settings/tokens in this popup message, so I can quickly find the well hidden page to make myself a token.
(I'll make a PR in the next few days, just writing this down).

Hi HamPoole,
I believe this feature is already covered. Try clicking the "Create" button, it should bring you there.
I appreciate your willingness to create a PR, though.


Animations is no longer with sync with refresh rate of the monitor

After https://android-review.googlesource.com/c/platform/frameworks/support/+/1449160 animations on desktop is no longer with sync with the current refresh rate of the monitor.
Snippet to check:
import androidx.compose.animation.animate
import androidx.compose.animation.core.TweenSpec
import androidx.compose.desktop.Window
import androidx.compose.runtime.mutableStateOf

fun main() {
    val target = mutableStateOf(100f)
    Window {
        val value = animate(
            target.value,
            animSpec = TweenSpec(durationMillis = 2000)
        )
        target.value = 0f
        println("$value ${System.currentTimeMillis()}")
    }
}

To sync animations we should not use Recomposer.current().
After https://android-review.googlesource.com/c/platform/frameworks/support/+/1478821 we can now use a local Recomposer and MonotonicFrameClock inside each Window.

Partial temporary fix:
https://android-review.googlesource.com/c/platform/frameworks/support/+/1507761

Should be fixed in 0.3.0-build146

Should be fixed in 0.3.0-build146

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


new commit run error

Traceback (most recent call last):
Traceback (most recent call last):
Traceback (most recent call last):
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 249, in _bootstrap
self.run()
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 249, in _bootstrap
self.run()
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/home/sdc/github/rl-teacher/rl_teacher/comparison_collectors.py", line 55, in _write_and_upload_video
upload_to_gcs(local_path, gcs_path)
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 249, in _bootstrap
self.run()
File "/home/sdc/github/rl-teacher/rl_teacher/comparison_collectors.py", line 55, in _write_and_upload_video
upload_to_gcs(local_path, gcs_path)
File "/home/sdc/github/rl-teacher/rl_teacher/video.py", line 138, in upload_to_gcs
subprocess.check_call(['gsutil', 'cp', local_path, gcs_path])
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/sdc/github/rl-teacher/rl_teacher/video.py", line 138, in upload_to_gcs
subprocess.check_call(['gsutil', 'cp', local_path, gcs_path])
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 286, in check_call
retcode = call(*popenargs, **kwargs)
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 249, in _bootstrap
self.run()
File "/home/sdc/github/rl-teacher/rl_teacher/comparison_collectors.py", line 55, in _write_and_upload_video
upload_to_gcs(local_path, gcs_path)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 286, in check_call
retcode = call(*popenargs, **kwargs)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 269, in call
return p.wait(timeout=timeout)
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/sdc/github/rl-teacher/rl_teacher/video.py", line 138, in upload_to_gcs
subprocess.check_call(['gsutil', 'cp', local_path, gcs_path])
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 269, in call
return p.wait(timeout=timeout)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1439, in wait
(pid, sts) = self._try_wait(0)
File "/home/sdc/anaconda3/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 286, in check_call
retcode = call(*popenargs, **kwargs)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1439, in wait
(pid, sts) = self._try_wait(0)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1386, in _try_wait
(pid, sts) = os.waitpid(self.pid, wait_flags)
File "/home/sdc/github/rl-teacher/rl_teacher/comparison_collectors.py", line 55, in _write_and_upload_video
upload_to_gcs(local_path, gcs_path)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 269, in call
return p.wait(timeout=timeout)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1386, in _try_wait
(pid, sts) = os.waitpid(self.pid, wait_flags)
File "/home/sdc/github/rl-teacher/rl_teacher/video.py", line 138, in upload_to_gcs
subprocess.check_call(['gsutil', 'cp', local_path, gcs_path])
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1439, in wait
(pid, sts) = self._try_wait(0)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 286, in check_call
retcode = call(*popenargs, **kwargs)
KeyboardInterrupt
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1386, in _try_wait
(pid, sts) = os.waitpid(self.pid, wait_flags)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 269, in call
return p.wait(timeout=timeout)
KeyboardInterrupt
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1439, in wait
(pid, sts) = self._try_wait(0)
File "/home/sdc/anaconda3/lib/python3.6/subprocess.py", line 1386, in _try_wait
(pid, sts) = os.waitpid(self.pid, wait_flags)

(2,) (11,)
Traceback (most recent call last):
File "rl_teacher/teach.py", line 330, in
main()
File "rl_teacher/teach.py", line 275, in main
label_schedule=label_schedule,
File "rl_teacher/teach.py", line 63, in init
self.graph = self._build_model()
File "rl_teacher/teach.py", line 114, in _build_model
self.q_value = self._predict_rewards(self.segment_obs_placeholder, self.segment_act_placeholder, mlp)
File "rl_teacher/teach.py", line 81, in _predict_rewards
rewards = network.run(obs, acts)
TypeError: run() takes 2 positional arguments but 3 were given

Closing because I can't replicate.


assemble script for nvidia/cuda

Hi,
How to write assemble script for nvidia/cuda to import tensorflow.

this isn't really something we can help with.  Someone who knows those components would be needed.


Issue 78 Markers Updates Programatically

fix(markers updates):issue #78 resolved
fix(coverage): coverage working

@jessertaylor @elesdoar @simison @zacronos @cgat @eczajk1 please review

looks good to me

LGTM


SceneRefreshPicker: Show the auto calculated value when auto is selected

Show the computed auto value when auto is selected.

📦 Published PR as canary version: 4.5.3--canary.680.8567692151.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

@dprokop - fixed!


Galactic2 nan doesn't recover

Another from @fractalgee to which I responded "I should add nancheck clears"
Take 2 airwin instances, set both to Reverb->Galactic2. Connect out1 to in 2 and out 2 to some destination (mixer/scope, doen’t matter. Now connect a SurgeXT FM2 VCO (guess any other wil do too) and observer a nanV on outs on either instance 1 or 2, seems to depend in which order they are wired. Switching to Galactic on the nanV one will make it work again (or any other reverb preset)
Was just seeing what a chain of Galactic2s would sound like, but alas, no dice
I

any chance to sort out this one, pretty please?

oh did i not add nancheck sweeps to the rack module? I'll do it in my next sweep.
I still couldn't make galactic 2 nan out in my test case though. Do you have a patch which does it reliably?

Just grabbed the latest nightly (should have done that before previous comment, as I had failed to do so since the official drop) and lo and behold, no more nanV! So go ahead and close this one as sorted.

Great!


Please clarify App Service Environment v3 zone redundancy minimum instance count

The documentation says there is a minimum of charge of nine instances when zone redundancy is enabled on an App Service Environment v3.

Zone redundant App Service Environment v3: There's a minimum charge of nine instances. There's no added charge for availability zone support if you have nine or more App Service plan instances. If you've fewer than nine instances (of any size) across App Service plans in the zone redundant App Service Environment, the difference between nine and the running instance count is charged as additional Windows I1v2 instances.

However, this does not make much sense as there should only be a requirement of three instances due to there being only three availability zones to deploy across. Furthermore, the Azure Portal states there is a minimum instance count of three which aligns with expectations.

Please confirm the minimum instance count and fix the documentation or portal to be accurate.
Thank you.

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

ID: 2dcd1a66-7ff7-c0da-b170-c302784221ae
Version Independent ID: 4766d1de-b033-029a-9860-12a306195c72
Content: App Service Environment overview - Azure App Service Environment
Content Source: articles/app-service/environment/overview.md
Service: app-service
GitHub Login: @madsd
Microsoft Alias: madsd

@krispenner Thank you for your feedback. We will investigate and update the thread further.

@krispenner the docs and portal are both correct. There is a difference between minimum App Service plan instance count and App Service Environment instance charge. When you have less than 9 instances across all you plans in the ASE, there is an added billing charge/padding with I1v2 Windows instances.

@madsd thanks for the reply. So to enable zone redundancy, there is a minimum plan instance count of 3, since there are three availablity zones. But for whatever reason it requires more or less 3 instances per zone as a minimum charge, hence 9 minimum instance charge. Therefore if I only deploy the minimum of 3, I'll be charged for 6 more? Is there any background logic as to why this is the case? I can deploy an ASEv3 without zone redundancy and only be charged a minimum of 1 instance, not 3.
So to confirm this in dollar amounts, in Canada Central using CAD per month, minimum charge for ASEv3 with zones is $4,675.35/month and without zones is $519.49/month.
Finally, regarding Linux, if we deploy 9 Linux plan instances to an ASEv3 with zones enabled, that would cost $3,331.19/month, but we would still need to pay the difference for 9 Windows instances and be charged the remaining $1,344.16/month as well?
Thanks!

Hi @krispenner. The logic is that the cost of the supporting infrastructure is significantly higher with AZ App Service Environment. All supporting roles are tripled, and the internal storage solutions, which is dedicated to each ASE instance, requires a Premium tier to support AZ. With ASEv3 instead of having the Stamp fee, we have set a minimal charge for an ASE environment. Most customers will deploy more than three instances in their ASEs, especially in production systems, which will then level that out.
And on the Windows vs Linux - you will not be charged the remaining 1,344.16, so if you know you will never go beyond 9 instances across plans in the ASE, and you still want AZ, you can "pad" it yourself with Linux instances and save some $.

@krispenner Just checking if you chance to see recent response of @madsd ? Let us know.


Add certmanager.k8s.io/acme-http01-ingress-class annotation

What this PR does / why we need it: to get letsencrypt's certificates for internal services.
Allow ingress-shim to use certificate ingress class other then original ingress class. Part of #942.
Release note:

Added certmanager.k8s.io/acme-http01-ingress-class annotation

/ok-to-test

This seems like a fairly non-intrusive and easy to understand idea.
Could you also update the ingress-shim page in the docs to contain information on this annotation and when it might be used (similar to the other annotations for ingress-shim!)
If possible, can you also add a few unit tests just to verify this logic? That should make it easier for us to keep the behaviour consistent in future. There should already be some examples of unit testing this area of the code in that package!

Tests is no problem.
English is not my native language but I will try to add documentation.

I have added test to pkg/controller/ingress-shim/sync_test.go and doc to docs/reference/ingress-shim.rst. Please check my english)

/retest

All green, do I need to do anything else?

/lgtm
/approve


memory option is missing in analyzer.storage.backend documentation

I think we can now close this. The new storage config format show that memory is a valid option. Please re-open if needed


Training prosocial agents

Dear Authors,
Prosocial agents perform well and I found there is not much text on training prosocial agents in the paper.  In your paper,

Since the prosocial agents were explicitly trained to maximize the reward of other players, it is unsurprising that they would most benefit the background population.

Here, how is the loss function defined?  I also found

We also trained prosocial variants of all three algorithms, which directly optimized the per-capita return (rather than individual return), by sharing reward between players during training.

Is it similar to the training paradigm of MADDPG and QMIX?
Thanks in advance.

The only difference between the vanilla and prosocial variants was that the prosocial variants used np.mean(timestep.reward) instead of timestep.reward[player_index] as the reward during training (e.g. to train the critic).
We did this by modifying the Substrate observations by adding a substrate_transform to add coplayer rewards (via all_observations_wrapper). This allowed the training step to calculation the mean from the observations in the generated trajectories.
Note that none of our agents' policies were conditioned on reward, so we didn't use this information at behavior time (nor at test time).
This is important because since the Scenario is a test of the trained policy, we consider it "cheating" to modify the Scenario observations. The timestep reward at test time can there only be timestep.reward[player_index], and you shouldn't train the policies under test to expect anything else.

The only difference between the vanilla and prosocial variants was that the prosocial variants used np.mean(timestep.reward) instead of timestep.reward[player_index] as the reward during training (e.g. to train the critic).
We did this by modifying the Substrate observations by adding a substrate_transform to add coplayer rewards (via all_observations_wrapper). This allowed the training step to calculate the mean from the observations in the generated trajectories.
Note that none of our agents' policies were conditioned on reward, so we didn't use this information at behavior time (nor at test time).
This is important because since the Scenario is a test of the trained policy, we consider it "cheating" to modify the Scenario observations (e.g. things like explicit coordination directives). The timestep reward at test time can therefore only be timestep.reward[player_index], and you shouldn't train the policies under test to expect anything else.

@jagapiou  Dear John, thanks for the clarification. In my code, I use timestep.reward from the substrate for MARL training. I think that is what you mentioned.
reward = np.sum(timestep.reward)/n_agents

Thank you!


centerface.onnx model

Good job！And the centerface.onnx contained face alignment??

Centerface contains detection and alignment, but the model centerface.onnx only contains detection.

对齐后续开放吗

对齐后续开放吗

涉及产品，这个还没想好


[SQL][Hiveconsole] Bring hive console code up to date and update README.md

Add import org.apache.spark.sql.Dsl._ to make DSL query works.
Since queryExecution is not avaliable in DataFrame, so remove it.

/cc @rxin

Can one of the admins verify this patch?

Jenkins, test this please.

Test build #26631 has started for   PR 4330 at commit d23ee9f.

This patch merges cleanly.

Test build #26631 has finished for   PR 4330 at commit d23ee9f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26631/
Test FAILed.

retest this please

ok to test

Test build #26669 has started for   PR 4330 at commit 46eb790.

This patch merges cleanly.

Test build #26669 has finished for   PR 4330 at commit 46eb790.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26669/
Test PASSed.

Thanks, merged to master and 1.3


Minor update of readme

in case you copy paste it as json, comma causes syntax error
and comments too

Coverage remained the same at 96.909% when pulling 388735905ea2ffa9d19df132e13cfde419099ed3 on pravdomil:patch-1 into c2cb726d6341989c0700dde5aad2bdb06837938c on jonaskello:master.

Nice catch! :-)


Possibility to unselect or delete studies/datasets from Projects.

Is your feature request related to a problem? Please describe.
There should be a possibility to unselect or delete studies/datasets from Projects before making it public.

[ ] #486

Update docs on how to delete studies and datasets
Change the update button text to "Manage studies" and "Manage datasets"

(https://docs.nmrxiv.org/docs/submission-guides/submission/folder-structure#delete-files)

it is possible to delete files during the initial submission or from "manage studies/datasets" afterwards.


Basic implementation of language server

This PR adds a language server can be started by python -m edb.language_server.main which will then speak LSP over stdin and stdout streams.
It will:

parse and report any syntactic errors in the opened edgedb files,
find dbschema/ directory in the workspace,
parse and compile the schema,
compile and edgedb files against that schema,
if any DDL is found, it will apply that DDL during each file compilation.

Example:
# dbschema/default.esdl

module default {
    type Player {
        property username: str;
    };
};

# a.edgeql

create TYPE Clan {
    create property name -> str;
};

alter type Player {
    create required LINK clan: Clan;
};

select default::Player {
    username,
    clan: {
        name
    }
}
filter .username ilike '%pickle%' and .age > 10;

TODO:

[ ] when a schema is changed, recompile it

To check it out, one must:

checkout this PR into your dev env,
checkout language-server branch of edgedb-editor-plugin,
edgedb-editor-plugin in vscode, using the same python venv that your dev env is installed in,
launch "Run Extension" task,
now a new vscode window should open and you can navigate to an edgedb project.

What should we do to test this? Some tests that talk to the language server directly?

Probably yes. Run python -m edb.language_server.main in a subprocess and feed it LSP requests.


goog.ui.DatePicker getActiveMonth setActiveMonth

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=260

UI Feature Request


[Bug] Remove vertical/horizontal conditionals sidebar file

Bug report
Inside the demo we are doing stuff depending on whether the layout is horizontal or vertical. That shouldn't be done by the developer (in sidebar_content), it should be done by the theme.
So let's:

[x] hide the separators using CSS, if the menu items are shown in a horizontal menu
[x] use JS to do data-bs-auto-close when the menu items are vertical

These should close it:
https://github.com/Laravel-Backpack/demo/pull/506
https://github.com/Laravel-Backpack/theme-tabler/pull/39


Wip/update docs

Pull Request Test Coverage Report for Build 187

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 54.375%

Totals

Change from base Build 184:
0.0%

Covered Lines:
4170

Relevant Lines:
7104

💛  - Coveralls


Evaluate ways to measure network traffic.

May be even out of Mint's scope, but as described in the issue https://github.com/minio/minio/issues/4236 , create internal documentation on measuring internode traffic, this will help as catch and reduce any unnecessary data transfer across nodes.

This is probably a test case.


Make all hooks free functions

Standardize so everything is use_ for hooks.

Might also be a good idea to rename to cx: Context?

@philpax Yup


White Balance

Please add white balance to the module. This is available over VISCA.
https://birddogtv.zohodesk.com/portal/api/kbArticles/574085000000642627/locale/en/attachments/6r81le95f4132358f4c51bd67954c5ff79410/content?portalId=edbsnfaffdf840d1eef8c81ee4ee9dfacd8a48be54e0a2e0cd1acde19c38c2612ea9e&inline=true

@rjurden What White Balance options are missing?

As far as I can tell, all documented options are covered by the module.
@rjurden Feel free to reopen if you can elaborate on what WB features are missing


Illegal argument: line must be positive: Error: Illegal argument: line must be positive

Environment

node -v: 8.6.0

npm -v: 5.4.1

npm ls react-scripts (if you haven’t ejected): [fill]

Operating system: ubuntu 17.04

Steps to Reproduce
opened up my project and saved a file
Expected Behavior
no error is thrown
Actual Behavior
error is thrown:

/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:29 [Orta.vscode-jest] Illegal argument: line must be positive: Error: Illegal argument: line must be positive
	at Object.t.illegalArgument (/usr/share/code/resources/app/out/vs/workbench/node/extensionHostProcess.js:28:56657)
	at new e (/usr/share/code/resources/app/out/vs/workbench/node/extensionHostProcess.js:28:237724)
	at new e (/usr/share/code/resources/app/out/vs/workbench/node/extensionHostProcess.js:28:240015)
	at diagnostics.set.asserts.map.assertion (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/out/src/diagnostics.js:19:48)
	at Array.map (native)
	at addTestsError (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/out/src/diagnostics.js:17:38)
	at testResults.forEach.result (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/out/src/diagnostics.js:32:21)
	at Array.forEach (native)
	at Object.updateDiagnostics (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/out/src/diagnostics.js:24:17)
	at JestExt.updateWithData (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/out/src/JestExt.js:307:23)
	at Runner.__dirname.startProcess.jestProcess.on.on (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/out/src/JestExt.js:61:18)
	at emitOne (events.js:96:13)
	at Runner.emit (events.js:191:7)
	at __dirname.default.start.debugprocess.stdout.on.data (/home/capaj/.vscode/extensions/Orta.vscode-jest-2.4.2/node_modules/jest-editor-support/build/Runner.js:67:18)
	at tryToString (fs.js:449:3)
	at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:436:12)

looks like either test result parsing has some problem or jest outputs something unexpected... however, regardless the cause, the extension probably better to report something (even though it might be "precise") than throwing an exception...
I will submit a PR to address the exception but will need more info to find out the actual root cause...
can you do a manual jest run (on shell) with the following options: --json --useStderr --outputFile xxx and send us the output file so maybe we can locate the root cause.


Port to Firefox

Implement the extension in Firefox to allow Firefox users to have the Netflix Tweaked goodness.

Extension is now available for Firefox: https://addons.mozilla.org/en-US/firefox/addon/netflix-tweaked/


What about zip files?

I installed the milter yesterday on my ubuntu server. It seems to work but only on files that are not inside a zip file. Is there a reason why you left this out?
Most of the "locky" like emails contain their crappy files inside zip container.

I tested it with different macro types and files - also .zip:
Mar 22 22:21:22 sendemail[14344]: Setting content-type: text/plain
Mar 22 22:21:22 sendemail[14344]: DEBUG => Sending the attachment [**ebill0200442.zip**]
Mar 22 22:21:39 incela sendemail[14344]: ERROR => Received:     550 5.7.1 **ERROR = Attachment contains unallowed office macros!**

The oletools api can handle only un-encrypted zip files in this implementation.
Please check the "run.log" file for errors or stop the service and start the macromilter.py manual (python /etc/macomilter/macromilter.py) and provide me the log or std-out output.
Please check also the oletools version --> https://bitbucket.org/decalage/oletools/wiki/olevba
My tested version is 0.42.
Thanks for your help!!

I am using oletools 0.44.
I tested it with the attached doc file (came in as spam email). The doc is detected.
24.03.16 23:13:56 [4273] - Find Attachment with extension - File content type: application/msword - File name: rechnung_-postmaster 789.doc
24.03.16 23:13:56 [4273] - Attachment b949b42c7dab6883551f38de52428a08 already parsed ! REJECT
I zipped the doc and tried again and the zip file is perfectly ignored by the milter with these log entries:
24.03.16 23:09:28 [4269] - Find Attachment with extension - File content type: application/zip - File name: rechnung_-postmaster 789.doc.zip
24.03.16 23:09:28 [4269] - VBA no Macros found in file
24.03.16 23:09:28 [4269] - VBA parsing exit
rechnung_-postmaster 789.doc.zip
 - due to upload restrictions, i had to add the png extension to the doc file.
I checked the documentation of the olevba and i am not quite sure if olevba can handle ordinary zip files. The documentation is mentioning zip file support always together with zip-containers for office files (docx, etc).

Just checked the source of olevba.py and found this text
def detect_vba_macros(self):
        """
        Detect the potential presence of VBA macros in the file, by checking
        if it contains VBA projects. Both OLE and OpenXML files are supported

So maybe the way to handle document files inside a zip will be to check if the attachment is a zip file and if so, iterate through all files inside the zip and push each to the vba parser.

Yes you are right. The zip is not detected. I'll check this and maybe provide a fix or talk to Philippe.

Ok, now I know why the zip file is not recognized by the milter!
An OLE Office file (legacy 97-2003 formats) in a zip is technically like a docm or xlsm (with vbaProject.bin inside). But if it's a zip with a docm or xml or mht inside, it won't work. It's better to unzip each file in memory, then scan them.
Now, I add in the next days a build-in in-memory unzip function to handle these files correctly.
Thank you for the report and tests!

Great! Really looking for this.
Things may to consider might be a file size limit to secure system performance and zips inside zipfiles.

Am 29.03.2016 um 15:46 schrieb Stephan Traub<EMAIL_ADDRESS>Ok, now I know why the zip file is not recognized by the milter!
An OLE Office file (legacy 97-2003 formats) in a zip is technically like a docm or xlsm (with vbaProject.bin inside). But if it's a zip with a docm or xml or mht inside, it won't work. It's better to unzip each file in memory, then scan them.
Now, I add in the next days a build-in in-memory unzip function to handle these files correctly.
Thank you for the report and tests!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I added a "normal" zip extraction function to the milter.
The recursive and "rar" support is outstanding and I hope to add this in the next days.
I leave this issue open.

Thank you very much. I will give it a try tomorrow!
Von meinem iPhone gesendet

Am 29.03.2016 um 20:19 schrieb Stephan Traub<EMAIL_ADDRESS>I added a "normal" zip extraction function to the milter.
The recursive and "rar" support is outstanding and I hope to add this in the next days.
I leave this issue open.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Is working!! Thank you

Rar and recursive support is coming soon :smiley: (in V3).


Link missing for "Learn how to update your existing fragments in this section."

Issue in ./help/using/rn/release-notes.md (https://experienceleague.adobe.com/en/docs/journey-optimizer/using/whats-new/release-notes)
There is no link for " Learn how to update your existing fragments in this section." in June release notes.

hi @Lexie1919, thanks for your feedback. The section has been updated.
Internal reference: DOCAC-10673


[5.5] Mix refactoring

Third try...
I've fixed the #19764 which made things worst than before as nothing isn't cached anymore... 😓
(see https://github.com/laravel/framework/pull/19764#issuecomment-312885891. Imho we really shouldn't merge untested code!)
And I tried to do what you asked here : https://github.com/laravel/framework/pull/19666#issuecomment-310905919. Imo this was clearer and prettier before, but I can understand the need if it's for performances.
@taylorotwell I've merged 5.5 commits in it as you asked, but I really think it should go on 5.4 too, because it fix an issue, and add a feature that lot of people is waiting for.
Let me know if you see something else.

The manifest cache issue should be fixed in a separate PR to 5.4.

@taylorotwell yeah, of course we'll have to fix it also in 5.4 if you don't want to merge this one in it.
But for 5.5, it doesn't change anything, because in any way, this PR refactor the entire function. The previous bug, in the previous function doesn't exist anymore.

This won't go into 5.4.

What problem is this trying to solve?

@devcircus see  #19639 for the initial explanation.
Some features (that I personally don't need) was added after because demanded in comments.

Let's maybe try to make smaller, focused, incremental improvements to Mix instead of re-writing it into this class.

Guy really ?
I'm proposing a fully tested and complete proper refactoring after a discussion in internals.
I'm adding some demanded features, even by you apparently:

"A while ago Taylor said he was interested in resolving this issue, so I think a PR would definitely be welcome."

There is no breaking change here., and an user who know nothing about the new features will not see the difference.
I've considered each comment and changed my code with all your feedback, and now after three weeks of waiting, you're closing it without any real explanation?
What's the problem with that now?


Fix test_recurring_job tests failures

https://github.com/longhorn/longhorn/issues/2158
The client fixture should be used for these tests.
Fixed some flake8 checks also.
Signed-off-by: khushboo-rancher<EMAIL_ADDRESS>

LGTM

LGTM


Adding policy to allow fetching from secrets manager

This allows ECS tasks fetching secrets from secrets manager and not only from SSM.
Types of changes
What types of changes does your code introduce to <repo_name>?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING.md doc.
[x] I have added necessary documentation (if appropriate).
[x] Any dependent changes have been merged and published in downstream modules.

Would it require "secretsmanager:ListSecrets" permission as well?

no.. the task definition points to a specific secret ARN so it doesn't need to list all secrets.


Feet or Meter should be selectable based on user preference

Measuring in meters is popular in Asia. We need to provide settings to switch distance announcement.

This should be fixed in current master

This issue was moved to hulop/NavCogIOS#10


Fix State reducer to use state hydration from redux

Currently, when you call createStore() with an initialState argument, the initial object of each State overides the initialState passed down from redux. Not sure if this is intentional, but this means that you cannot hydrate the state tree at boot (i.e from localstorage, isomorphic rendering etc).
Loving your work though guys! This stuff rocks.

Also, thanks! We can only hope its making life easier :)

Yeah so the reducer would be a pure function then. In the event that the action matches the reducer, it will behave the same as before. However if the action is not matched it will return the state passed in or the initial state the user has specified in the event that the state passed in is undefined

We would need to formulate a way to get the last known state from the reducer then, else we deprecate that functionality for many users.

The reducer's job is to determine the next state given a previous state and an action. You can get the current state with getState() but you shouldn't be relying on the reducer to store state, that's the store's responsibility. You could maybe add a flag that preserves the current functionality, and default it to off in a future release but I would certainly deprecate that functionality as it will cause side effects (such as the issue I've ran into)

How about a reserved hanging method on the reducer:
state() // pure
state.getState() // will return the last-known state to pass through

sounds good, bumped PR with changes

thats correct, it will emit an event labelled "@@REDUX/init"

Thanks @e-r-w!  You rock!  I'll get a release out asap.


move configs structs to remove dependency on deamon

Moved the following config structs

ContainerRmConfig
ContainerCommitConfig

New top-level package for holding the config structs

The base builder code now no-longer has a dependency on daemon. The dockerfile builder still does have a dependency on daemon.
This is solely moving code around. It stands alone as a change. It can be the first of several other commits that consolidate config structs used to communicate between a server and a deamon backend, or I can continue to consolidate code movement in this commit.
I would like to have a short discussion to make a decision on where the config structs are to be finally placed. They are currently placed in a top-level package. If api/server is to be moved into pkg/ in the future, this config struct package will need to move into pkg/ as well.

I forget this part a lot @runcom @duglin @calavera yay.

let's move them to api/types, so we have them all together :smile_cat: :+1:

@calavera sounds good. Do you know any golang tools that make moving code around easier?

nope, sorry.

LGTM

LGTM


Parameter types of XCDYouTubeClient#getVideoWithIdentifie callback function aren't optional type in swift 2.0

Hi,
I have use this library and I want to say thank you.
Currently I use this library with swift project via Carthage.
Today , I tried to migrate my project from swift 1.2 to swift 2.0.
However the signature of completionHandler function of getVideoWithIdentifie is changed.
In swift 2.0, both video and  error are non-optional value,
I can't check if video exists.
So, I try to fix it by adding _Nullable annotation to callback function.
:(NSString *)videoIdentifier completionHandler:(void (^)(XCDYouTubeVideo * _Nullable video, NSError * _Nullable error))completionHandler
Please use it as a reference.
Thank you.

I just released version 2.3.1 which addresses this issue, thanks for reporting!


Update 040_backup.rst

added description of RESTIC_PASSWORD_COMMAND in list of environment variables (read it somewhere, tested it and it works)

What is the purpose of this change? What does it change?
enhance documentation

Was the change discussed in an issue or in the forum before?
no (it is a one-liner in documentation)

Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[x] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[ ] I have run gofmt on the code in all commits
[ ] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

please add this one line to the documentation describing RESTIC_PASSWORD_COMMAND
see my pull request  #2392 https://github.com/restic/restic/pull/2392

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@604b18a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2392   +/-   ##
=========================================
  Coverage          ?   46.58%
=========================================
  Files             ?      174
  Lines             ?    14480
  Branches          ?        0
=========================================
  Hits              ?     6746
  Misses            ?     6733
  Partials          ?     1001

Continue to review full report at Codecov.

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

I don't know if my pull request contains more than a one line change. the codecov output suggests that I changed much more by accident. so I re-try this in another pull request


Mentioning html artifacts in ros2 documentation

For small changes to the ROS 2 Docs you can view your changes as rendered HTML using artifacts generated in our Github Actions.
Relates #4548

@kscottz I can't add you as reviewer here, the gear icon beside the reviewer option doesn't show up for me,  so just commenting here

i think we need to backport this to all available distribution.
note: https://github.com/ros2/ros2_documentation/pull/4573 and https://github.com/ros2/ros2_documentation/pull/4572 tries to make this feature more clear for all developers, but for now it only works for the maintainers with writer permission for the repository. we can consider this is another enhancement, and do the backports when it is ready.

@Mergifyio backport jazzy iron humble


[docs] Fixing docstring within method

to: @ryandeivert @Ryxias
cc: @airbnb/streamalert-maintainers
related to:
resolves:
Background
Docstring was incomplete and formatting wasn't consistent
Changes

Corrected docstring

Testing

ran ./tests/scripts/pylint.sh
ran ./tests/scripts/unit_tests.sh

Coverage remained the same at 96.178% when pulling cf0d371516fe6ed24840cd64e730ef5bfde5b02e on jack1902:outputs/update_teams_docstring into 90adb651bdd9861c82c899e143d49b708e842ca3 on airbnb:release-3-1-0.

asd


Exercise 6 Task 4 gameOfLife

so i got this far where my return type is currently Stream<List<Integer>> and now i need to convert this into a Stream<String>

I attempted to do this by trying to map the List<Integer> into Stream<Integer> and then doing a map on that to convert the Integers into Strings (thats what my helper function is doing). However, when I try to compile I get this error. I don't really understand this error and I'm not very sure how to proceed from here. Any help is greatly appreciated! thanks!!

hi cheryl,
likewise, i did a stream, and then i did a .map(x -> x.toString()) where i had additional .replace(x, y) to replace the output with the desired string output
for eg.
.replace("1", "x") to replace the 1s with an x.

hi there, I had exactly the same approach as you. i think your code might be returning a Stream<Stream> instead of a Stream, assuming your helper function turns individual Integers into Strings. (does x -> x.stream().map(y -> helper(y))) actually turn a List into a Stream ? )
Hope this helps; can't say for sure since i don't know exactly what your helper function is doing / if it's working as you intended

I think aside from only using .map() and .replace(), we can also use .map() and .reduce() to convert each inner ImList to a String.

Im not sure, but is your map function supposed to return  or Stream? It seems that you have defined it to return , but you're currently returning Stream instead

Hey Cheryl -- hope you've resolved the issue.
For those of you still stuck and looking for hints:

Look at the code in Recitation 8, (19th-20th October, labelled incorrectly as Problem Set 7), question 2. This will give you an approximate syntax to start off with.
What are the steps you need? Stream of the Lists, then on each individual list, manipulate each element of the list from an integer to a string, then concatenate the strings to a single long string.
Think of what happens at each sector of the stream. The first stream contains elements of type List<Integer>. Change this to a stream to work on the inner integers. Then, think again!

Pro tip: At any stage, should you be lost, .getClass() is a helpful method where you can figure out what you're working with.
Still stuck? Ping for help.


Applied filters bar

Pt. 1. Only readOnly view
PR includes

[x] Refactor

Related issues
Resolve #2350
QA Instructions, Screenshots, Recordings
https://github.com/taskany-inc/issues/assets/95316053/7064e94f-7e17-4f9e-825c-a92f4768711b


Non-nullability unfork causing issues in the Fuchsia builder

This is currently happening on the dart CI engine builders.
Full logs at: https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8878866180302852880/+/steps/build_fuchsia_debug_x64_flutter_shell_platform_fuchsia:fuchsia_fuchsia_tests/0/stdout
String _rawScript;
       ^^^^^^^^^^
Unhandled exception:
Crash when compiling dart:async/async_error.dart,
at character offset null:
RangeError (offset): Invalid value: Not in range 0..352, inclusive: 576
#0      RangeError.checkValueInInterval (dart:core/errors.dart:302:7)
#1      Source.getLocation (package:kernel/ast.dart:9080:16)

cc: @a-siva , @aam
@yjbanov seems like we need to do some changes to the Fuchsia build targets to support NNBD?

The following files
flutter/shell/platform/fuchsia/flutter/kernel/libraries.yaml
flutter/shell/platform/fuchsia/flutter/kernel/libraries.json

needs to be changed similar to
flutter/lib/snapshot/libraries.yaml
flutter/lib/snapshot/libraries.json

to point to files in
third_party/dart/sdk_nnbd instead of third_party/dart/sdk


Reuse all connections

When testing, every time I make a request, the message appears:
`` `Starting a new HTTP connection (1): www. *****. With
I have seen one session per application. He tried to reuse the "session" for all requests and seems to work properly, even with connections that have been closed.
What implications would this change have?
In [9]: requests.get("https://github.com")
Starting new HTTPS connection (1): github.com
Out[9]: <Response [200]>

In [10]: requests.get("https://github.com")
Out[10]: <Response [200]>

In [11]: requests.get("https://stackoverflow.com")
Starting new HTTPS connection (1): stackoverflow.com
Out[11]: <Response [200]>

In [12]: requests.get("https://stackoverflow.com")
Out[12]: <Response [200]>

In [13]: requests.get("https://github.com")
Out[13]: <Response [200]>

In [14]: requests.get("https://github.com")
Out[14]: <Response [200]>

Hey @kianxineki, the api methods are intended to be single connection requests. If you'd like to utilize connection pooling in Requests, you'll need to use the Session object as shown below.
s = requests.Session()
req_git = requests.Request('GET', 'https://github.com')
req_stack = requests.Request('GET', 'https://stackoverflow.com')

resp_git = s.send(req_git.prepare())
resp_stack = s.send(req_stack.prepare())

You can find more information on how to use Session here.

Hey @kianxineki, I skimmed through your question and responded a bit too quickly. The implications of this change is we would now use a single Session object across ALL connections with requests.request which may not be what the user wants. The api methods are intended to be single connections that are cleaned up after completion.
As I think you've already noted above, you can accomplish the change by simply adding an extra line  in your code declaring your own Session instance. This will allow you to utilize connection pooling and avoid the new connections.
s = requests.Session()
resp_git =s.get('https://github.com')
req_stack = s.get('https://stackoverflow.com')

You can find more information on how to use Session here.

In my current code I do this to not have to use s.get() and to be able to use the methods that facilitates me requests.
import requests

session = requests.sessions.Session()

def request(method, url, **kwargs):
    return session.request(method=method, url=url, **kwargs)

requests.api.request = request

@kianxineki Thanks for sending us a pull request! It looks to be your first to the project. Welcome!
It seems as though you're trying to ask why we do not use a global session object in requests.api. To answer you, let me explain a little bit of history first.
We used to use a global session object and our Session objects used to be thoroughly thread-safe. At some point, something changed about our Session objects that no one has yet been able to identify. This means that they're no longer properly threadsafe. As such, people were having significant problems sharing a session between threads which also applied to using the functional API. Further, having a session that sits around until the process finishes was holding onto sockets that were never being reused.
The solution to the thread-safety issue and the socket leak was to stop using a global session.
Currently, if you want to eliminate those lines in your logs and utilize connection pooling, you must use a Session. There's no way to provide connection pooling to you via our functional API without actively harming other users.
In the future, please direct all of your questions to StackOverflow, which is the proper Question & Answer forum for Requests.
Again, thank you for your pull request, but we can't accept it.


bug: Attempt to fix iOS CI

For some reason it fails, I suspect a request to get some manifest file from google storage fails but the script continues with null values

For some reason it never failed for the "macos" build, so I adapted it to its order.

For some reason it fails, I suspect a request to get some manifest file from google storage fails but the script continues with null values

Could be a good reason, maybe the timeout is not enough, although 15 seems quite a lot: https://github.com/subosito/flutter-action/blob/0f2cd3391b2bfa32c731e6dc28acac302cfa8cbe/setup.sh#L59


python310Packages.pytelegrambotapi: 4.8.0 -> 4.9.0

Description of changes
Fix build (https://hydra.nixos.org/build/205591364)

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.pytelegrambotapi
python311Packages.pytelegrambotapi

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

2 packages built:

python310Packages.pytelegrambotapi
python311Packages.pytelegrambotapi

Thank you!


🎉 1.1.4

ℹ About this release

Version: 1.1.4
Type: patch
Last Release: 1.1.3 (3/26/2024, 5:37:16 PM) [?]
Commits to merge: 16 [?]

📦 Dependencies

Update @prisma/client from ^5.11.0 to ^5.12.1
Update @sentry/node from ^7.107.0 to ^7.109.0

📦 Development Dependencies

Update @types/node from ^20.11.19 to ^20.12.2
Update mocha from ^10.3.0 to ^10.4.0
Update prisma from ^5.11.0 to ^5.12.1
Update typescript from ^5.4.2 to ^5.4.3

:tada: This PR is included in version 1.1.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v1.1.4

Your semantic-release bot :package::rocket:


OCM-2447 | feat: add automatic upgrades for node pools

Features and UX should be aligned with control plane upgrades Related: OCM-2447

/hold pending backend fix

/unhold

/lgtm


Show etched runes on /block

Should be easy, since rune IDs start with the block ID

I dont know how to query by comparing IDs with the block ID.
What method is used to query in redb?

You'll need to query RUNE_ID_TO_RUNE_ENTRY for a range, like this: RuneId{ block: BLOCK_NUMBER, tx: 0}..RuneId{block: BLOCK_NUMBER + 1, tx: u32::MAX}.

Instead use RuneId{ block: BLOCK_NUMBER, tx: 0}..RuneId{block: BLOCK_NUMBER + 1, tx: u32::MAX}
I use:
let rune_min = RuneId {
      block: block_height,
      tx: 0,
    };
let rune_max = RuneId {
  block: block_height,
  tx: total_txs_in_block as u32,
};

but it's failed. I will retry.

Let's me resolve this issue!

released as part of v0.18.2


Updates to CloudFormation templates timeout.

Description
When running a pipeline for the second time with cfnUpdate it triggers an update on the CloudFormation stack which completes successfully with 'UPDATE_COMPLETE', however the completion is never picked up by Jenkins so the pipeline does not progress and the stage eventually times out and fails.
I think the error occurs here: https://github.com/jenkinsci/pipeline-aws-plugin/blob/master/src/main/java/de/taimos/pipeline/aws/cloudformation/CloudFormationStack.java#L146
The line should read:
new EventPrinter(this.client, this.listener).waitAndPrintStackEvents(this.stack, this.client.waiters().stackUpdateComplete(), pollConfiguration);

Steps to Reproduce

Create pipeline which deploys a CloudFormation stack.
Run the pipeline for a 2nd time, this will trigger an update on the stack rather than create it.

Expected behavior: Stage to complete successfully and pipeline to processed to the next step.
Actual behavior: Pipeline timeout without progressing.
Environment
Jenkins-Version: Jenkins ver. 2.121.3
Java-Version: "1.8.0_181"
Plugin-Version: 1.30
Master/Slave Setup: No

Indeed, you are right. I will fix it and release a new version today. Sorry for the inconvenience.

Release with 1.31


Add in Events

Seem that you already have the events because you select the events in the race results.
Let's list and provide a event search and maybe we will import the event files/PDFs later.

On Hold


Yet Another Fork

Description

[O] I have added a folder(Named it as my GitHub user id)
[X] I have added a Readme.md file inside the folder
[X] I understand the GitHub Forking Concept fully

Thanks for the Video it was great.

@anjanraj027 we are analyzing the auto closed repo… please do not close. I'll either merge or close this week. Thanks!


Improve core string type testing

I noticed that PCSTR and friends don't have good test coverage. I've added some basic testing, although I plan to add more once I've made some additional improvements to consistency. I also added some error convertibility to make using the various to_string methods more convenient.

@riverar does this help with your scenario?


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
18 seconds
sanity-inspect-image

✅ Succeeded
55 seconds
clamav-scan

✅ Succeeded
22 seconds
clair-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
20 seconds
sanity-label-check

✅ Succeeded
19 seconds
sanity-optional-label-check

✅ Succeeded
5 seconds
show-summary


Update builder.md

@coreydaley Do you happen to know how exactly should I use NewBuilder() in assembly := car.NewBuilder().Paint(car.RedColor) ?

@sluongng
There are really two ways to use it (given that this is an incomplete example):
NewBuilder could be a function that returns a Builder, meaning that what you return must implement the Builder interface.
You could change it to be:
NewBuilder could be a struct that contains the correct fields and functions to adhere to the Builder interface, in which case your code would look more like:
assembly := car.NewBuilder{}
assembly.Paint(car.RedColor)

could you please include code snippet of what you just explained in the PR?
I would love to read in detail and I think others might have same question as I did.

@sluongng
https://github.com/coreydaley/builderExample
go get github.com/coreydaley/builderExample

@sluongng @coreydaley
May I look at this code? https://github.com/coreydaley/builderExample
Where can I see this?

@crossgit that repo was deleted long ago


Every feature is set to Chrome 119

Seems like a bug - bit me while recording the podcast today.

For most mobile browsers, only the latest version is shown on Caniuse

@HM100 is correct, the versions you see are the versions that support is tracked for.

Wes, it's not that support was added in 119. It's that the current version is 119, and it has support.
That column simply doesn't show history all the way up into the past like the desktop columns do.

its very confusing


Doesn't read properly

I connected it to a 125khz ID wg26 reader but when it read i first converted the hex number to decimal and it didn't match the real number. I also tried to get some number of the binary data with no luck as well. I don't know, maybe my device is faulty. It restarts every time i read and i have to connect the wifi again. Or i don't know how to get the decimal number of the card. This is my log file. The last (underlined) reads are of a card with the number<PHONE_NUMBER>. I will be very thankful if someone could give me some much needed help.

Solved. Just had to look up the protocol (i'm not very good at programming, bits and protocols) and found that the 26 bit binary data has two bits at each end. I just copy the data without the very first and very last bit, convert it to decimal and get the correct card number.

Solved. Just had to look up the protocol (i'm not very good at programming, bits and protocols) and found that the 26 bit binary data has two bits at each end. I just copy the data without the very first and very last bit, convert it to decimal and get the correct card number.


Feature Request: Remove the EPINIO_APP_DATA env variable

Is your feature request related to a problem?
This variable is going to cause ~many~ some issues around, and we should try to remove it for the v1.8.1.
We should fetch all the needed information from the application manifest (ref: https://github.com/epinio/epinio/issues/2224)
Solution you'd like
Info

Application Source information will now come from the manifest (fetched in a separate API call, see https://github.com/epinio/ui/issues/227)
App source info is used

App Detail page
App Edit page

We need to

Remove places where the EPINIO_APP_DATA env var is managed
Add to existing API requests the additional app source info

at the moment i think this is just git type and git branch where we sent the git url and git revision

Add an async func to the app source

this fetches the manifest
parses manifest into the object we expect

Call new app source func on app detail and edit pages

Alternatives you've considered
No response
Anything else?
Blocked on https://github.com/epinio/ui/issues/227

@torchiaf is this ready to test?
I see it in our review column but I still see the app.
I tested against latest epinio-devwith commit 23d440198 already present (sso login labels)

@torchiaf  not sure if this is an item is not testable yet until https://github.com/rancher/dashboard/pull/8797 is merged.
I am moving this still to in progress on Epinio planning meanwhile is confirmed

@torchiaf not sure if this item is not testable yet until rancher/dashboard#8797 is merged. I am moving this still to in progress on Epinio planning meanwhile is confirmed

Hi Martin, yes please, this one is not merged yet, (UI status is To Review).

It seems working in latest epinio-dev:

Verified in 1.8.1-rc1:


chore(master): release 1.1.12

:robot: I have created a release beep boop
1.1.12 (2023-08-14)
Miscellaneous

deps: update dependency pestphp/pest to v2.14.0 (b692f8a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.12 :sunflower:


python3Packages.starlette: 0.13.8 -> 0.14.2

Motivation for this change
https://github.com/encode/starlette/releases/tag/0.14.0
https://github.com/encode/starlette/releases/tag/0.14.1
https://github.com/encode/starlette/releases/tag/0.14.2

ujson was dropped in https://github.com/encode/starlette/pull/1047

removal of UJSONResponse breaks fastapi https://github.com/tiangolo/fastapi/pull/2335

test_debug_html was fixed in https://github.com/encode/starlette/pull/1132

Things done

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

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

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

Result of nixpkgs-review pr 112701 at 8732d1b7 run on x86_64-linux 1

1 package marked as broken and skipped:

python39Packages.starlette

2 packages failed to build:

python37Packages.fastapi
python38Packages.fastapi

2 packages built:

python37Packages.starlette
python38Packages.starlette

Result of nixpkgs-review pr 112701 at 8732d1b7 run on aarch64-linux 1

1 package marked as broken and skipped:

python39Packages.starlette

2 packages failed to build:

python37Packages.fastapi
python38Packages.fastapi

2 packages built:

python37Packages.starlette
python38Packages.starlette

Rebased, because nixpkgs-review by @r-rmcgibbo didn't take the uvloop fixes on the target branch into account.
Result of nixpkgs-review pr 112701 run on x86_64-linux 1

2 packages failed to build:

python37Packages.fastapi
python38Packages.fastapi

3 packages built:

python37Packages.starlette
python38Packages.starlette
python39Packages.starlette

Sorry about that

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

2 packages failed to build:

python37Packages.fastapi
python38Packages.fastapi

3 packages built:

python37Packages.starlette
python38Packages.starlette
python39Packages.starlette

Sorry about that

Not actually your fault, the fixes arrived on master three minutes after this pull request was opened. I was merely stating the fact, that they didn't (couldn't) take that into account.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112701 run on x86_64-linux 1

2 packages failed to build and are new build failures:

python37Packages.fastapi: log https://termbin.com/7syp
python38Packages.fastapi: log https://termbin.com/o88x

3 packages built:

python37Packages.starlette
python38Packages.starlette
python39Packages.starlette

Because fastapi is the only consumer of starlette, I'd say we wait for feedback from fastapi maintainers.

The latest platformio release requires a starlette version of 0.14.x. Thus, I'd love to this one merged.

It seems sensible to disable FastAPI as it has been stuck at 0.13.x of Starlette for quite some time. FastAPI also fixes the exact Starlette bug fix version, which doesn't work well with Nixpkgs. There is an open FastAPI PR that hasn't been reviewed since December. So, I'm in favor of disabling FastAPI and updating Starlette especially if Starlette's dependencies are blocked.

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

2 packages failed to build:

python38Packages.fastapi
python39Packages.fastapi

2 packages built:

python38Packages.starlette
python39Packages.starlette


improve microG settings (fix issue #758 and more)

Follow up commit to #719 and #741 includes:

separates launcher icon from main activity (so it can be disabled without making microG inaccessible, think of Launchers that don't allow to hide Apps on their own)
fix issue #758 (launcher icon not shown on all devices)
#741 added 'Add account' to microG Settings, this here adds the 'Trust Google for app permissions' checkbox aswell (new 'Account' settings group)
#719 added 'microG settings' to System Settings > Device, move into System Settings directly (result may vary, depending on the ROM and Android Release)
improved version of the microG icon for System Settings
made several GCM related strings translatable
updated russian translation
updated german translation

updated ukrainian translation

added new pref to 'Account' settings category introduced by 30ed2720a0ecd3067fe760e801ecbdbe1b5ff5be

Can you change the class/xml name of the account settings to not be "add account"? It's a little confusing ;)
Thanks.

Also you are lacking the logic code required for the auth_manager_visible preferences to be applied directly (as implemented in the AccountSettingsActivity)

Yes, of course, completely missed it out. Will fix it later today.
Am 29. Mai 2019 17:19:47 MESZ schrieb Marvin W<EMAIL_ADDRESS>
Also you are lacking the logic code required for the
auth_manager_visible preferences to be applied directly (as
implemented in the AccountSettingsActivity)
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/microg/android_packages_apps_GmsCore/pull/765#issuecomment-496982776

--
Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.

Hi @mar-v-in !
What do you think about simplify string "Google Cloud Messaging" to "Cloud Messaging" or something else ?
Because GCM is not exist anymore

If there's anything left you'd like to see changed, just let me know.

How about using this Android 9 compliant settings menu? The current icon looks sort of wrong with Pie.
<?xml version="1.0" encoding="utf-8"?>

<layer-list xmlns:android="http://schemas.android.com/apk/res/android">

    <item>
        <shape android:shape="oval">
            <solid
                android:color="#C2185B" />
            <size
                android:width="36dp"
                android:height="36dp" />
        </shape>
    </item>

    <item
        android:width="24dp"
        android:height="24dp"
        android:gravity="center" >
        <vector xmlns:android="http://schemas.android.com/apk/res/android"
            android:width="24dp"
            android:height="24dp"
            android:viewportWidth="24"
            android:viewportHeight="24">
            <path
                android:fillColor="#ffffffff"
                android:pathData="M14.5335,0.0569535 L9.43985,0.0705542 C9.24288,0.102156,9.0985,0.18001,9.06273,0.352909 L8.55771,3.75569 C7.70454,4.01856,7.0852,4.47885,6.46902,4.94163 L3.48987,3.74936 C3.23326,3.67936,2.95937,3.47486,2.64004,3.95331 L0.0871706,8.21668 C0.0372893,8.41036,0.0580599,8.56989,0.145272,8.69686 L2.93012,10.8425 C2.83047,11.6279,2.86713,12.4355,2.90092,13.2426 L0.23227,15.219 C0.005066,15.4246,0.050044,15.7049,0.087127,15.8119 L2.78493,20.2165 C2.89848,20.3411,3.04458,20.3919,3.22668,20.3608 L6.49813,19.1434 C7.12487,19.6363,7.79275,20.0492,8.52873,20.3294 L8.99269,23.3505 C9.11365,23.56,9.09564,23.8073,9.4858,23.9434 L14.6784,23.9434 C14.8701,23.8649,15.0801,23.8217,15.1137,23.4353 L15.5776,20.2729 C16.4832,19.9372,17.01,19.5277,17.6663,19.1435 L20.7121,20.3576 C21.1885,20.4803,21.2695,20.2522,21.3504,20.16 L23.8741,15.8401 C23.9395,15.7104,23.9625,15.4495,23.787,15.2191 L21.1762,13.2708 C21.1953,12.4738,21.2347,11.6899,21.1762,10.8426 L23.874,8.72511 C23.996,8.49886,23.9302,8.24834,23.874,8.10387 L21.3214,3.92514 C21.1924,3.70382,20.9206,3.64557,20.7701,3.67097 L17.6082,4.94163 C16.8967,4.38465,16.2039,3.98905,15.5197,3.67097 L14.9974,0.395711 C14.9027,0.136848,14.7173,0.0991057,14.5334,0.0570135 Z M11.8718,5.62811 C13.7255,5.60431,15.517,6.32771,16.5932,7.54186 L14.7953,9.4995 C13.2639,8.11395,9.57199,7.53593,8.30443,11.1012 C7.82376,12.9227,8.54727,14.4138,10.0518,15.2852 C11.6506,16.1862,14.0977,15.4204,14.7343,14.7492 L16.6846,16.6771 C15.8496,17.3442,12.7132,19.81,8.67011,17.6858 C6.61824,16.5517,4.69843,14.6267,5.53146,10.4485 C6.00671,8.8177,7.06262,7.50059,8.60931,6.53345 C9.62512,5.92563,10.7596,5.64239,11.8718,5.62811 Z" />
        </vector>
    </item>
</layer-list>

@oldosfan what ROM are you using? For me it looks normal on Pie (both in Launcher and System Settings), maybe post a screenshot.
@mar-v-in is there anything left you'd like to see changed/improved?

@mar-v-in any interest in the parts which can be rebased on latest code?

Launcher activity alias: probably makes sense if some users have issues with that
Accounts settings: I planed to have a completely new screen with all accounts listed, ability to add accounts and all the account settings and a new option to disable the whole account management, so that everything is in one place. So this change is probably going to be replaced with something else soon anyway
Add to system settings: Has to be taken with care that it doesn't break any existing microG integrations (like /e/)
Translations: Probably doesn't hurt
Updated icon: Updating the icon to something that's more compliant with current android icon guidelines is on my todo anyway, but I'd have to check if this one fulfills my expectation. If it's the one that I remember, then I didn't really liked it.

see 1191 for replacement.


Add script to generate TPTP for any implication, extracted from Vampire scripts

TPTP is the standard format for theorem provers, used for instance by the Vampire script.
This is useful to run other automated theorem provers (or Vampire with custom settings) on the remaining implications.
Might be also be worth it to generate TPTP for all current open implications and provide it either on the repository or on the webpage, perhaps automatically with CI.

It'd be nice if this had a very simple description at the top, even just an example invocation so it was clear how to use it. Also a line in the README.md scripts section so others could find it.


[NetAppFiles]ANF-3998 Swagger RestAPI for 2020-02-01

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

      "201": {

A PUT should return 201 on creation and 200 if the resource was updated

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:1556 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

  "get": {

a "list" operation should be a POST action

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:1636 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

          "user1",

typically AAD interaction is done via object IDs or other truly unique identifiers.  Are you sure you want to deal in actual usernames here?  Isn't there a concern of someone using this as an attack vector (i.e. taking someone elses username when they leave the company)

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2025 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    },

"id" is a top level property (like "name" and "type") and should not reside inside the property bag

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2885 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

        "$ref": "#/definitions/hourlyScheduleObject"

including the "object" suffix will make the class names awkward in generated sdks.  I would remove it unless you have a specific need for it

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2891 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    }

For state-like properties booleans are very limiting in the future as your API evolves.  Something like "policyState": "Enabled" here (where "Enabled" is one value of a string enum) would give you much more flexibility

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2921 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    "location": {

its rare to have a tracked child resource.  Is it required that this be a tracked resource with its own location + tagS?

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2933 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    }

please include the nextLink property and add it to x-ms-pageable.  If you run into problems with the number of entities and need to start paging responses this will prevent you from breaking your clients AND your current backend implementation doesn't need to change

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2977 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

"snapshotPolicyDetails": {

where is this referenced?

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2980 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    },

you can't update any of these properties, they don't need to be in the request

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3040 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

      "$ref": "#/definitions/snapshotPolicyProperties",

the PATCH request should only include the properties that you support patch on.  If that is all of them then this is fine

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3047 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    }

please enhance the description here.  It is unclear what this property does

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3083 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

"hourlyScheduleObject": {

can a user specify multiple types of scheduled on one snapshot?  If not, what response do they get if they try?

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3067 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

      "description": "Indicates which hour snapshot should be taken",

in what timezone?  Should be explicit

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3096 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

      "example": "Sunday,Monday"

Do you support localized day names?  Why not have a string array instead of a csv?

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3121 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

    "daysOfMonth": {

why string instead of array of ints?

Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3147 in ad72d3c. [](commit_id = ad72d3cb0edc8f3001c7eccb9504f6827ea3822a, deletion_comment = False)

/azp run automation - sdk

/azp run automation - sdk

  "get": {

a "list" operation should be a POST action
Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:1636 in ad72d3c. [](commit_id = ad72d3c, deletion_comment = False)

@pilor This is an existing service and running version and I believe this needs a backend change internal ticket will be created for this on our end.

          "user1",

typically AAD interaction is done via object IDs or other truly unique identifiers. Are you sure you want to deal in actual usernames here? Isn't there a concern of someone using this as an attack vector (i.e. taking someone elses username when they leave the company)
Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2025 in ad72d3c. [](commit_id = ad72d3c, deletion_comment = False)

@pilor Good idea this is an existing service and running version and I believe this needs a backend change internal ticket will be created for this on our end

    }

For state-like properties booleans are very limiting in the future as your API evolves. Something like "policyState": "Enabled" here (where "Enabled" is one value of a string enum) would give you much more flexibility
Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2921 in ad72d3c. [](commit_id = ad72d3c, deletion_comment = False)

@pilor  I believe this needs a backend change internal ticket will be created for this on our end

/azp run automation - sdk

    }

please include the nextLink property and add it to x-ms-pageable. If you run into problems with the number of entities and need to start paging responses this will prevent you from breaking your clients AND your current backend implementation doesn't need to change
Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:2977 in ad72d3c. [](commit_id = ad72d3c, deletion_comment = False)

@pilor  I believe this needs a backend change internal ticket will be created for this on our end

"hourlyScheduleObject": {

can a user specify multiple types of scheduled on one snapshot? If not, what response do they get if they try?
Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3067 in ad72d3c. [](commit_id = ad72d3c, deletion_comment = False)

@pilor  yes they can.

      "$ref": "#/definitions/snapshotPolicyProperties",

the PATCH request should only include the properties that you support patch on. If that is all of them then this is fine
Refers to: specification/netapp/resource-manager/Microsoft.NetApp/stable/2020-02-01/netapp.json:3047 in ad72d3c. [](commit_id = ad72d3c, deletion_comment = False)

@pilor yes they do

/azp run automation - sdk

@pilor can you please review, we have addressed some points, see comments,  and noted where changes are required in backend that will be done in future version and internal tickets have been created.

/azp run automation - sdk

Draft status due to status of snapshotPolicy in backend

Draft status due to status of snapshotPolicy in backend

Added Do Not Merge label. Pls ping me when the service is ready, and the PR is not draft but ready to merge.

Some parts of this were deemed not ready and removed to be introduced in a later update. We have another PR for this version
https://github.com/Azure/azure-rest-api-specs/pull/9742
We can close this.


wast: Make V128Const, V128Pattern and NanPattern clonable

In some use cases it would be helpful if the enums V128Const, V128Pattern and NanPattern would derive (or otherwise implement) the Clone trait?
Or is there a particular reason against it?

No problem adding these, should be fine to do! Would you be interested in sending a PR for this?

Sure, I will open a PR.


LeetCode-CN premium account cannot access locked question.

Failed to access leetcode-cn premium problem with premium account.
LeetCode-CN 会员账户无法访问会员问题

Environment

os: Mac OS 10.14.6
nodejs version: 10.15.3
vscode version:1.38.0

Is the activated endpoint you are using is leetcode-cn? The default is leetcode

Is the activated endpoint you are using is leetcode-cn? The default is leetcode

Yes, it is.

Hmm.. I haven't met this issue before. Is the unlocked problem accessible?

Hmm.. I haven't met this issue before. Is the unlocked problem accessible?

Unlocked problems in both endpoints are fine. But when I switched to endpoint leetcode with a premium global account, I can now access to locked problem just fine...

Hmm, not sure what's happening there. I haven't a premium account so I can test it locally. 😂
Will the global endpoint works for you?

Uh.. that was my friend's account so I would prefer to access them in leetcode-cn endpoint. Do you think it will be helpful if I give you the account to test that lol?

@liuyaf If you don't mind, that would be great. 😄

+1
And the console shows:

Hey guys, sorry I just came back from vacation. Will start looking at this issue recently.

It's because user info get from Leetcode-cn DO NOT has the property 'paid'. You can print user info yourself.
{ login: '',
  pass: '',
  loginCSRF: null,
  sessionCSRF: '',
  sessionId: '',
  hash: '',
  name: ''
}

So to solve this problem locally, you can just add a property 'paid' and set it to true.

I cannot access locked question，either.
The same situation occurs to me.
I am using  LeetCode-CN endpoint after buying the leetcode-cn premium account.
ENIV

OS: WIndows 10 2004
vscode vision: 1.42.1
leetcode-extension vision: 0.16.1

Just add "paid": ture to ~/.LC/user.json and ~/.LC/leetcode.cn/user.json
Locked questions will show up.

Same issue. Can't access the locked problems.

Just add "paid": ture to ~/.LC/user.json and ~/.LC/leetcode.cn/user.json
Locked questions will show up.

true. spell check please.

Solution: Go to the ~/.lc/leetcode.cn/user.json add "paid":true.

Solution: Go to the ~/.lc/leetcode.cn/user.json add "paid":true.

can't find this file (┬＿┬)

Solution: Go to the ~/.lc/leetcode.cn/user.json add "paid":true.

can't find this file (┬＿┬)

You can make a dir and use.json and try, but the LC plugin seems break down these days. I can't log in.

Hi, my account is leetcode as endpoint, and I logged in using cookie as specified here - https://github.com/LeetCode-OpenSource/vscode-leetcode/issues/478#issuecomment-564757098.
~/.lc/leetcode/user.json file has "paid": true, but I don't see all the locked questions. Can someone please help. Thank you.

Solution: Go to the ~/.lc/leetcode.cn/user.json add "paid":true.

Hope for auto setting "paid": true for premium account.
Otherwise we need set it manually everytime sign in.

Hi, my account is leetcode as endpoint, and I logged in using cookie as specified here - https://github.com/LeetCode-OpenSource/vscode-leetcode/issues/478#issuecomment-564757098. ~/.lc/leetcode/user.json file has "paid": true, but I don't see all the locked questions. Can someone please help. Thank you.

I'm having the exact same issue.

add "paid":"true" works for me, not true but "true"

Just add "paid": ture to ~/.LC/user.json and ~/.LC/leetcode.cn/user.json
Locked questions will show up.

if still can't open, emove the cache folder and stats.json in the ~/.LC/leetcode.cn/ directory

Hi, my account is leetcode as endpoint, and I logged in using cookie as specified here - https://github.com/LeetCode-OpenSource/vscode-leetcode/issues/478#issuecomment-564757098. ~/.lc/leetcode/user.json file has "paid": true, but I don't see all the locked questions. Can someone please help. Thank you.

I'm having the exact same issue.

you should set up the "Leetcode: Show Locked" item in the leetcode settings as follows:


Restore gRPC limitation for protobuf data size to the default (4MB)

We have increased the gRPC data size for protobuf from 4MB to 100MB by #41, which introduced the event loop blocking problem described by #80, then we solved this problem by adding the stream support to our gRPC service by https://github.com/wechaty/grpc/issues/96.
So we will restore the default size to our gRPC service to keep our service robust.

For multi-language Wechaty, please upgrade your gRPC service by replacing the old messageFile to messageFileStream for receiving files larger than 4MB because the messageFile will throw an exception when the Protocol Buffers data size larger than 4MB.
@wechaty/python @wechaty/go @wechaty/java @wechaty/php @wechaty/scala @wechaty/dotnet


Create timer which sends a TCP message every 30 minutes

Use gobot.io to send a TCP message through ESP8266 WIFI every 30 minutes.
The code could start to live in a directory called device

Important to notice that this issue also includes configuring the Nodemcu environment to operate with gobot.io and create or point to documentation about how to do it (if it does not exist).

This has already been fixed in the last commit.The frequency can be set in an environment variable called FREQUENCY.


tree-wide: enable LTO on Darwin

Description of changes
This PR is the second follow-up to https://github.com/NixOS/nixpkgs/pull/307880. It enables LTO in packages flagged by https://github.com/NixOS/nixpkgs/issues/19098 as well as others I could identify as having it disabled on Darwin.
I tested building Python on both aarch64- and x86_64-darwin. I also ran a nixpkgs-review on aarch64-darwin with the Python LTO disabled (to keep the number of changed packages manageable). Most packages built. Those that failed did not appear to be LTO-related. I have a number of PRs I will be submitting for some of those broken packages.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

enable lto for nuitka
diff --git a/pkgs/development/python-modules/nuitka/darwin-lto.patch b/pkgs/development/python-modules/nuitka/darwin-lto.patch
deleted file mode 100644
index bd3a65905498..000000000000
--- a/pkgs/development/python-modules/nuitka/darwin-lto.patch
+++ /dev/null
@@ -1,15 +0,0 @@
-diff --git a/nuitka/build/SconsCompilerSettings.py b/nuitka/build/SconsCompilerSettings.py
-index 319b72c4e..89d40f2a2 100644
---- a/nuitka/build/SconsCompilerSettings.py
-+++ b/nuitka/build/SconsCompilerSettings.py
-@@ -173,8 +173,8 @@ def _enableLtoSettings(
-         lto_mode = False
-         reason = "known to be not supported (CondaCC)"
-     elif isMacOS() and env.gcc_mode and env.clang_mode:
--        lto_mode = True
--        reason = "known to be supported (macOS clang)"
-+        lto_mode = False
-+        reason = "known to not be supported (macOS nix clang)"
-     elif env.mingw_mode and env.clang_mode:
-         lto_mode = False
-         reason = "known to not be supported (new MinGW64 Clang)"
diff --git a/pkgs/development/python-modules/nuitka/default.nix b/pkgs/development/python-modules/nuitka/default.nix
index fa0842267031..cce4467e73e5 100644
--- a/pkgs/development/python-modules/nuitka/default.nix
+++ b/pkgs/development/python-modules/nuitka/default.nix
@@ -23,9 +23,6 @@ buildPythonPackage rec {
     hash = "sha256-nKdCMgA92v9VsSgfktXDbSh3DyKsGlcTjpn0Y7u4rxU=";
   };

-  # default lto off for darwin
-  patches = [ ./darwin-lto.patch ];
-
   build-system = [
     setuptools
     wheel

enable lto for nuitka

Thanks. I added nuitka to the tree-wide commit.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/darwin-updates-news/42249/16


SSID with space

Hi
My wifi SSID has a space in it.  When I try to join it, it only picks up the first word of the SSID before the space. How can I escape the space?
See below:
COMMAND
adb -s AQH7N17B18019814 shell am start -n com.steinwurf.adbjoinwifi/.MainActivity -e ssid "MY 5G_OFFICE" -e password_type WEP -e password mypassword
RESPONSE
Starting: Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] pkg=5G_OFFICE cmp=com.steinwurf.adbjoinwifi/.MainActivity (has extras) }
On the app itself, it is connecting to the wifi called "MY", the first word of the SSID.

 Thanks for the quick reply.


Re-enable AppVerifier and crash dump collection in Helix

It's hard to debug crashes without the crash dump.

Useful for IIS crashes and for hanging tests.

Also, it would be good to be able to extract our test logs as well as xunit diagnostic logs

Let's try to prioritize this work. There is a service with Helix called Dumpling that we can potentially use to capture crash dumps. @anurse and I agree the goal E2E experience would be to have:

Test failures link to all crash dumps that occurred during a test run.
All pdbs are also linked to debug said crash dumps

There are many scenarios we are missing because of this. There was an IIS crash I had to debug on aspnetci because it was the only place that had dumps.

All pdbs are also linked to debug said crash dumps

You can get pdbs from VSTS artifacts.

You can get pdbs from VSTS artifacts.

Yep, just a direct link to them in the RAAS issue.

Also https://github.com/dotnet/arcade/pull/2103

@anurse this looks like a feature-team item, but I'm not 100% sure what work is involved here so lmk if you need build team to help. Moving to area-servers.

I think this issue should be retitled: Get Crash Dumps in Helix. Today there is no way to collect dumps at all. The appverifier stuff isn't relevant to the underlying issue.

@jkotalik does the download task added in https://github.com/dotnet/arcade/pull/2103 cover our needs here? IIRC we just needed to find a way to get the files back from Helix, then we can add the crash dump collection logic.

this looks like a feature-team item

I mostly agree, though I would definitely want the build team involved since it will likely affect every Helix run. We'll probably be putting some commands before and after the test run to set up dump collection, grab any dumps generated, and use DownloadFilesFromResults to pull them back to AzDO and publish them as artifacts.

Closing in lieu of https://github.com/aspnet/AspNetCore-Internal/issues/2079


Setup New Cloud/New Docker Template via API

Is there a way to add a new cloud and Docker template via the REST API instead of clicking in the UI?

no, but people do it via script console https://github.com/KostyaSha/yet-another-docker-plugin/pull/139


plant gametogenesis 2

This is to follow on from [ 800444 ] Plant gametogenesis
https://sourceforge.net/tracker/index.php?
func=detail&aid=800444&group_id=36855&atid=440764
which is getting quite long.
Reported by: jenclark
Original Ticket: "geneontology/ontology-requests/1537":https://sourceforge.net/p/geneontology/ontology-requests/1537

thanks www.movied.org


[ARCTIC-1363][hive] Fix Optimizer retry report result costing too much time when task is deleted

Why are the changes needed?

fix #1363
Brief change log

AMS should not throw exception to optimizer when handing reported result

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduces a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

The codes look good to me.
However, I would like to sync up that in the new refactored implementation for #1372, when AMS cannot handle the complete request correctly, it will throw a specific exception. In this case, the optimizer will not choose to retry, only when it receives an unknown exception will it choose to retry. I think this is a better solution.

The codes look good to me. However, I would like to sync up that in the new refactored implementation for #1372, when AMS cannot handle the complete request correctly, it will throw a specific exception. In this case, the optimizer will not choose to retry, only when it receives an unknown exception will it choose to retry. I think this is a better solution.

I agree with that. This PR is for 0.4.x before the refactored code. And also, it's easy for the user of 0.4.x to upgrade with this patch, because they only need to upgrade the AMS.


Logarithmic Y scale

Some data can get get hidden when there are spikes in the data. Using a logarithmic scale could help here.
Complication is that the 2nd Y axis values are actually converted and added on the 1st Y axis. Nivo does not support 2nd y axis so had to find a way round it. would be more complicated to convert logarithmically.

get error as some values are zero, also need to programatically figure out when to use logarithmic scale and what scale to use.

get error as some values are zero, also need to programatically figure out when to use logarithmic scale and what scale to use.

wont do for now