Cap on stableToken CP-DOTO tank size

Description
Adds a maximum size to the stableToken bucket of CP-DOTO.
The cap is a function of the current supply of stableToken, and it's calculated as follows:
maxStableBucketSize = stableTokenSupply* stableBucketFractionCap. If the calculated cap after a rebalancing is bigger than the cap, then the goldBucket shall be downsized accordingly.
For that to work, the stableTokenSupply needs to be big enough to generate a usable stableTokenBucket size, and it is unusable if a token has a zero supply (because the bucket will be always capped at zero). minSupplyForStableBucketCap is introduced for that, and the bucket will always be capped such that it's always bigger than minSupplyForStableBucketCap* stableBucketFractionCap
Added governable variables:

minSupplyForStableBucketCap
stableBucketFractionCap

TODO:

[x] Validate approach
[x] Verify it doesn't break proxy storage
[ ] Optimize gas usage
[x] More testing
[x] Replace hard-coded constants on test

Tested
Unit tested added.
Related issues

Closes #5849

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

Will lint checks, please make sure to review meanwhile!


[ISSUE #2370] clean up optimize [shenyu-plugin].

…d shenyu-plugin-uri .
About#2370
clean up optimize shenyu-plugin-tars and  shenyu-plugin-uri for the module [shenyu-plugin] .
Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

please  check your  title,  not  the whole  shenyu-plugin
module


[Build] skip renaming files for release wheels pipeline

Previously all the wheels are named to v1.0.0.dev but this is not correct for published wheels during my release process. This PR

preserve the previous behavior for nightly
fix the naming for release tag

it would be useful to review the shell script

Do you need to update the installation doc for the nightly wheel name as well?

what's the changing part?

This PR doesn't change the nightly behavior (renaming all wheels to 1.0.0.dev), it skips the nightly upload on main branch commit with the proper version tag so I can upload it to PyPI


Service.Resource command reference

service.Resource has list of command references, which currently is just a list of uint64. It should be a litst of lists, or better yet, a list of path.Command.

Done. 798ec738c8fd376cd26c81518c09f1de12d799a9


ASRock Z77 Extreme4 Will Sometimes Fail to Boot With Error 62

System

Motherboard: ASRock Z77 Extreme4 - Modifications Start from Latest 3.00 Beta BIOS Available From ASRock
GPU: ASRock Intel Arc A380 Challenger ITX 6GB OC
OS: Windows 10 22H2
[x] CSM is turned off
[x] 4G decoding is enabled
[x] UEFIPatch is applied (see Using UEFIPatch for more information). On some motherboards DSDT Patching is also needed
[x] I have read Common issues (and fixes)

Description
I was able to successfully modify the BIOS to allow ReBAR to work as per my post here: https://github.com/xCuri0/ReBarUEFI/issues/11#issuecomment-1605868162
However, every so often, most noticeably during a Windows Update "Install Updates and Restart" Cycle my board will fail to boot and display an Error 62 on the ASRock Motherboard Dr. Debug Readout. Looking online this error code relates to "Chipset initialization error." Clearing the CMOS and then restarting always has resolved the issue, but it just means I need to reapply my settings.
This doesn't appear to occur when a "normal" reboot happens or I shutdown and restart the machine. Again I have only noticed this on a Windows Update "Install Updates and Restart" Cycle or when automatic updates are executed. Any ideas on what might be different here?
I have the modified BIOS if that would help.

@aolszowka have you checked your CMOS battery voltage ? hwinfo should say it

@xCuri0 This is VBAT right? Seems to read at 3.248? I honestly don't know what expected range should be:

I'd be happy to swap out the battery its just a CR2032; I don't think I've ever replaced this (build occurred in ~2011).
Is there something that leads you to think this might be the issue?

@aolszowka voltage looks ok
Maybe you could redo the BIOS modification but use MMTool to replace modules (pad file workaround) because I've heard UEFITool sometimes causes corruption like this.

@xCuri0 Thank you for the reply, honestly its not that big a deal and really only happens on Windows Updates. It seems so bizarre that an Update and Restart is somehow different than a normal restart. I'll keep this open until Patch Tuesday (2023/09/12) and see if it reproduces again if that is alright?

@xCuri0 Closing the loop on this, Patch Tuesday (2023/09/12) after installing Windows Updates and selecting Update and Restart I experienced the same issue. This machine rebooted (Restart) and Shutdown just fine. Absolutely bizarre. I am going to close out the issue as I have not encountered any other issues with this mod, but perhaps at some point in the future I'll be able to figure out just what is different between an Update and Restart and Restart. Next Patch Tuesday I am going to attempt an Update and Shutdown to see if that yields any different results.

@aolszowka I hope I'm not bothering you, I have this same motherboard and the desire to have ReBar working but I have no experience with BIOS modification and I hope you can provide me with the already modified BIOS file if you still have it, accompanied by any tips that might be useful in the process. Thank you in advance. 😃

@Naddly I've got more details in my post here: https://github.com/xCuri0/ReBarUEFI/issues/11#issuecomment-1605868162
This part was critical I feel:
"During my entire process I actually had a CH341 Programmer available to me along with a bunch of spare W25Q64BVAIG's. The chip is socketed on this motherboard which makes swaps super simple and I always had a way to fall back when something went wrong. I had to use AsProgrammer_<IP_ADDRESS>_fix that I found off some YouTube guide, I picked up the programmer from Amazon and the spares from eBay from China (took 2 weeks to get here, but was worth it)."
Feel free to contact me @ my username from a mail service from Google.

@aolszowka I just sent you an email, though I'm not quite sure if I got it right. Is it aolszowka @ (google mail) com?


Some color emojis are scaled too small

Terminal v0.9.433 renders some color emoji too small:
Examples:
$ echo -e "\U2620"

$ echo -e "\U270f"

Hi @DHowett-MSFT. I see that you closed the issue and marked it as duplicate. Could you provide an issue ID we could track if we are interested in this problem?

@dgellow you're looking for #610


MacOS Pinta 2.2 beta, Can't build Pinta 2.2 beta

Can't build Pinta 2.2 beta.
I followed instructions at <>Code tab, ran first two commands. Worked perfectly. However, running the third command caused an error
MSBUILD : error MSB1009: project file doesn't exist.
Key: Pinta
MacOS Sonoma 14.4.1, Pinta 2.2 beta

This works for me on macOS - could you let me know the exact command you were running, which folder you were running it form, and your dotnet --version?

dotnet run --project Pinta
dotnet --version 8.0.204

Thanks. I was wondering if you were running these commands from the top-level directory of the repository, or some subfolder. Otherwise, I'm not sure why it isn't working for you if the other commands like dotnet build did work.
Alternatively, you should be able to just run dotnet build/bin/Pinta.dll to launch Pinta rather than dotnet run ...

Still didn't help... Maybe the thing is something to do about the "folder" thing?

mhlkhrpchnk@MacBook-Air-Mhl ~ % pwd
/Users/mhlkhrpchnk
mhlkhrpchnk@MacBook-Air-Mhl ~ % =ls Pinta.sln
ls: Pinta.sln: No such file or directory

Am I just a silly guy that doesn't know how to get that .sln file?

Pinta.sln is in the root folder of the Pinta repository, so you need to do cd path/to/Pinta depending on where you cloned the Pinta git repository to on your system. It looks like you are just executing those commands from your home folder
The tutorial steps assume that you are executing the commands from the root folder of the source code

Thank you! I'm too new at all this "command line" stuff


Update the linking between entities

As a wrangler I want to "Update the linking between entities" so that I can either edit (meta)data that is wrong or add additional (meta)data for a project that is already in the DCP/Archives.

this is supported with post submission updates, but we didn't split the story like this


[ADAP-584] [Bug] Non-conventional Version Check in Setup.py

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
The setup.py checks the python version using a manual condition statement. This is not conventional python packaging behavior. The package manager is responsible for checking the python version. The python version is documented in the package metadata using the python_requires property. Adding an extra check of the python version deviates from the behavior expected during python packaging.
Expected Behavior
Leave the python_requires setting but remove the condition block.
Steps To Reproduce
Inspect the setup.py.
Relevant log output
No response
Environment
- OS:any
- Python:>=3.7
- dbt-core:any
- dbt-snowflake:1.6.0b2

Additional Context
No response

Removing triage, I agree this should be considered as part of https://github.com/dbt-labs/dbt-core/issues/6855
When it's officially added in its scope, we should flag this issue as duplicate.

@Fleid : Any thoughts about merging https://github.com/dbt-labs/dbt-snowflake/pull/634 now? Do we need to wait for #6855?

I would wait for #6855 - if anything else to have everything done together.


Use PEP508 environment markers (r2.0)

PEP508 environment markers should always be used for conditional dependencies.
Using environment markers is especially important for package managers with static dependency resolution such as Poetry. The current master branch consistently uses environment markers, but TF 2.1 hasn't been released. I would appreciate a backport of this fix to TF 2.0.

Note that even if merging this, we won't do a patch release unless we find a security issue that we patch.


'RADIUS_FOR_CISCO_ASA' & 'AWS_FEDERATED_LOGIN' not recognized as valid ApplicationSignOnModes

When getting a list of all applications via: apps, resp, err = await client.list_applications() it fails with the message: 'RADIUS_FOR_CISCO_ASA' is not a valid ApplicationSignOnMode & 'AWS_FEDERATED_LOGIN' is not a valid ApplicationSignOnMode respectively. It seems that those two application sign on modes need to be added to the application_sign_on_mode.py file. Also, equivalent models need to be created for each. Once those models are created, they need to be added to the OKTA_APP_SIGN_ON_TO_MODEL variable in the constants.py file.

Hi @mvitt,
Thanks for reporting this issue. We'll review this internally and provide you an update when we have more info to share.

@mvitt, thanks again for detailed issue. Models are being generated according to our openapi spec. First, we need to update openapi spec. This requires some time. To unblock you, I can suggest some workarounds. For example, it is possible to get original response body (before it is being mapped to models):
apps, resp, err = await client.list_applications()
orig_resp = resp._body

And depending on your needs, you can filter failed apps or process manually or something else. Is it helpful?

@mvitt After some internal discussions, we've decided to avoid errors so far by using "custom" signOnMode when we encounter any "new" signOnMode. Thus, sdk will pick basic Application model for such apps and list_applications won't fail.
Here's PR: https://github.com/okta/okta-sdk-python/pull/168. It will be added within next sdk release.
Please, reach out to us if you have any additional questions or concerns.

@mvitt It turned out, there might be another issues related to signOnMode, thus a new PR should cover it: https://github.com/okta/okta-sdk-python/pull/174
It is merged and will be included into upcoming release. Will keep you posted.

@mvitt SDK v1.4.1 released. I'm closing this issue as resolved. Feel free to reopen this issue if your problem still exists.


chore: adopt Spinner from svelte ui library

Fixes #963
What does this PR do?
Use Spinner UI component from UI library
Screenshot / video of UI
N/A
What issues does this PR fix or reference?
#963
How to test this PR?
N/A

should ./button/Spinner.svelte be removed then ?

Fixed


[java] Setting Java module name to com.microsoft.onnxruntime.extensions

The Java module system (introduced in Java 9) requires that all modules on the module path have unique names, and the current name for ORT-extensions collides with ORT as both use com.microsoft.onnxruntime. This PR changes the ORT-extensions module name to com.microsoft.onnxruntime.extensions to remove the collision.
If they are colliding then work needs to be done in the user's build system to pull ORT-extensions off from the module path and put it on the classpath which can be irritating to do particularly if it's consumed by a third party dependency which might not be doing a module aware build.

Something is up with the connection to the CI. I manually ran onnxruntime-extensions.CI here and that passed.
Once it passed, the button to check in the PR became enabled, although it still claims to be waiting on the status for the CI.

Is it possible to get this in to the 0.11.0 release or has that already happened?

Is it possible to get this in to the 0.11.0 release or has that already happened?

sorry, the release team are already in publishing. Let me see if we can do a next release or a patch release shortly.

Ok, thanks for the update.


Can't open deletion and addition merge conflicts

VSCode Version:

1.1.1

OS Version:

Windows 7
Steps to Reproduce:

Initiate a pull rebase or sync (we use git config --global pull.rebase true)
If there's a file merge conflict, it's displayed in the merge conflicts, but single clicking won't open the file to show differences.

Workaround:
Right click the file in merge conflicts, then pick open file (show file?). Not at work right now so I'm not sure which menu option will open it, but you can pick one that will open it and show the merge conflict information, fortunately.

I cannot reproduce, seems to work for me.
Was your conflicted file also moved among those commits? Can you reproduce? If so, can you provide a sample repository where I could reproduce it too?

I may not be able to provide a sample repo because we are using GitHub enterprise and I'm not allowed. I will, however, try to find time to reproduce it today and hopefully offer more details.

We've been swamped, but I finally got a screenshot. Just did a pull rebase from our upstream and I can't open the merge conflict by single clicking the file or right clicking and choosing 'Open Change'. If I select 'Open File', it's fine

Ok, will rename the issue accordingly.

I just had a similar issue. I made some changes (no deletions) to a commit. I then tried to do a pull --rebase in VSCode terminal. Though i had deleted no files, every one of the files i had touched in my commit appeared as deleted. This is not the first time ive witnessed this and i was dumbfounded. But i aborted the rebase and ran the same command from Powershell and it worked without a hitch.
I just tried to take a screenshot of the terminal, but there was another bug in the terminal and it jumped and now it is gone. What happened was, i got the error "please stash these changes as they will be overwritten by merge" on the few files i had added in the commit. I had no working changes when i initiated the rebase :/
Its strange, and I could probably reproduce it if desired by checking out the reflog and trying it again. Any notes you want me to take?

Moving forward to #24919

I am getting some errors when trying to open files in the "conflict" state from supporting extensions. Opening the file in staged changes (with the conflict markers) also throws this error but the extension displays a message and the file is shown.

The issue still occurs for me in version 1.17.2 on fedora 25.


Updating mbed-os to mbed-os-5.7.4

Please test this PR
If successful then merge, otherwise provide a known issue.
Once you get notification of the release being made public then tag Master with mbed-os-5.7.4 .

Superseeded by #31


When extending a base config include and exclude get overwritten by base config.

Problem
I have a monorepo with the following tsconfig.lib.json for my library:
{
  "extends": [
    "./tsconfig.json",
    "../../../../../../configs/typescript/tsconfig.jsx.json",
  ],
  "include": ["src/**/*.ts", "src/**/*.tsx", "types/**/*.d.ts"],
  "exclude": ["src/**/*.spec.ts", "src/**/*.test.ts", "src/**/*.spec.tsx", "src/**/*.test.tsx", "t.ts"]
}

With the following tsconfig.json in the same directory:
{
  "extends": "../../../../../../tsconfig.base.json",
  "compilerOptions": {},
  "files": [],
  "include": [],
  "references": [
    {
      "path": "./tsconfig.lib.json"
    },
    {
      "path": "./tsconfig.spec.json"
    }
  ]
}

And with the following tsconfig.base.json in the root directory:
{
  "compileOnSave": false,
  "compilerOptions": {
    "allowImportingTsExtensions": true,
    ...
  },
  "exclude": ["node_modules", "tmp"]
}

When I parse tsconfig.lib.json I want to have the values from include and exclude from this file, but when I get the tsconfig with getTsconfig('path/to/tsconfig.lib.json') I get the exclude from the tsconfig.base.json file and no include at all.
Also the config from tsconfig.jsx.json extended in tsconfig.lib.json does not get merged.
Expected behavior
All config files should be merged by their priority.
Minimal reproduction URL
https://github.com/floratmin/get-tsconfig-error
Version
9.15.0
Node.js version
v22.8.0
Package manager
pnpm
Operating system
Linux
Bugs are expected to be fixed by those affected by it

[ ] I'm interested in working on this issue

Compensating engineering work financially will speed up resolution

[ ] I'm willing to offer financial support

Thanks for filing this! I confirmed it's a bug.
If anyone can work on it, please feel free to open a fix PR


Python3 version

cat /etc/lsb-release DISTRIB_ID=Ubuntu DISTRIB_RELEASE=16.04 DISTRIB_CODENAME=xenial DISTRIB_DESCRIPTION="Ubuntu 16.04.7 LTS"
python3 --version Python 3.5.2
pip3 install python-zimbra Traceback (most recent call last): File "/usr/bin/pip3", line 11, in <module> sys.exit(main()) File "/usr/local/lib/python3.5/dist-packages/pip/__init__.py", line 12, in main from pip._internal.utils.entrypoints import _wrapper File "/usr/local/lib/python3.5/dist-packages/pip/_internal/utils/entrypoints.py", line 4, in <module> from pip._internal.cli.main import main File "/usr/local/lib/python3.5/dist-packages/pip/_internal/cli/main.py", line 58 sys.stderr.write(f"ERROR: {exc}")
f-strings break compatibility with python version older than 3.6.
Home page says "Compatible with Python 2.7 and 3.x (cPython and PyPy)".

Sorry, this seem to be relate to pip itself.


Left panel unresponsive, files show empty, breakpoints don't stop app

Not sure if this is a 3.5.0 issue or a local issue I'm having.
The left panel in Ti-Inspector when debugging the app has somewhat unresponsive. I can't close folders, only open them. I have breakpoints set but they don't stop the app.
Also I've noticed there's an extra "world" folder below the main one. So my app is called "Matrix Insights", which has a world folder at the top level with that name with all my files in it. What's weird is that most of the files shown are empty when opened. The files inside controllers, models, moment/lang,styles, and sync are not empty, they show the proper code.
Below the main world folder is another world folder called "Matrix Insightspp". All the files inside of this are blank.

This could be an issue with my local but I remember I started having problems when I upgraded to 3.5.0. It will continue to do these things even when I select the 3.4.1 SDK before compiling.

I can confirm this is not working with Titanium SDK 3.5.0.GA

this should be fixed by 4f82abbc3efab5ca13a4e8d063ae42b3848864aa


Failure to extract read features in MF

Hi There
I installed MF and I am using cram input files. I make the input file for my variants of interest but during the first stage it gets terminated:

 python ReadLevel_Features_extraction.py inputs/P0003_T.input outputs/P0003_T.features  cram/  hg38.fa  ./hg38/k24.umap.wg.bw 2 cram

test_mf_local.sh: line 44:  2855 Terminated              python ReadLevel_Features_extraction.py inputs/$SAMPLENAME.input outputs/$SAMPLENAME.features $cramdir ${genomeref} ${k24bw} 2 cram
not enough alt reads:  chr1 2304920 2304921
Process ForkPoolWorker-2:
Traceback (most recent call last):
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/pool.py", line 125, in worker
    put((job, i, result))
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/queues.py", line 347, in put
    self._writer.send_bytes(obj)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 200, in send_bytes
    self._send_bytes(m[offset:offset + size])
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 404, in _send_bytes
    self._send(header + buf)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 368, in _send
    n = write(self._handle, buf)
BrokenPipeError: [Errno 32] Broken pipe

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/pool.py", line 130, in worker
    put((job, i, (False, wrapped)))
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/queues.py", line 347, in put
    self._writer.send_bytes(obj)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 200, in send_bytes
    self._send_bytes(m[offset:offset + size])
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 404, in _send_bytes
    self._send(header + buf)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 368, in _send
    n = write(self._handle, buf)
BrokenPipeError: [Errno 32] Broken pipe

The 'not enough alt reads' seems to come up quite frequently. What could I be missing with my dataset.
Note that I am not running mutect prior to running MF. I am starting with a list of variant positions for which I want to work with.
Many thanks

Hi There
I installed MF and I am using cram input files. I make the input file for my variants of interest but during the first stage it gets terminated:

 python ReadLevel_Features_extraction.py inputs/P0003_T.input outputs/P0003_T.features  cram/  hg38.fa  ./hg38/k24.umap.wg.bw 2 cram

test_mf_local.sh: line 44:  2855 Terminated              python ReadLevel_Features_extraction.py inputs/$SAMPLENAME.input outputs/$SAMPLENAME.features $cramdir ${genomeref} ${k24bw} 2 cram
not enough alt reads:  chr1 2304920 2304921
Process ForkPoolWorker-2:
Traceback (most recent call last):
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/pool.py", line 125, in worker
    put((job, i, result))
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/queues.py", line 347, in put
    self._writer.send_bytes(obj)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 200, in send_bytes
    self._send_bytes(m[offset:offset + size])
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 404, in _send_bytes
    self._send(header + buf)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 368, in _send
    n = write(self._handle, buf)
BrokenPipeError: [Errno 32] Broken pipe

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/pool.py", line 130, in worker
    put((job, i, (False, wrapped)))
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/queues.py", line 347, in put
    self._writer.send_bytes(obj)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 200, in send_bytes
    self._send_bytes(m[offset:offset + size])
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 404, in _send_bytes
    self._send(header + buf)
  File "miniconda3/envs/MF/lib/python3.6/multiprocessing/connection.py", line 368, in _send
    n = write(self._handle, buf)
BrokenPipeError: [Errno 32] Broken pipe

The 'not enough alt reads' seems to come up quite frequently. What could I be missing with my dataset.
Note that I am not running mutect prior to running MF. I am starting with a list of variant positions for which I want to work with.
Many thanks

Hi @bwzee ,
Have you tried running the demo? I'm not getting problems running the demo and others have succeeded running it:
python ReadLevel_Features_extraction.py demo/test.input demo/test.cram.features demo/ {fasta} {k24.umap.wg.bw} 1 cram
I ask because from your error, it seems your conda environment has some problems.
As for the "not enough alt reads:" issue, you could just ignore it, because whenever there are not enough reads for a site to calculate the p-values (for example, only 1 read for the alt allele or so) MF would report it.
Best wishes,
Yanmei

Thanks for the reply. Actually Im sorry I didnt mention this but the example data worked fine for me. Since I'm using hg38 I had to fix some of the reference downloads and bigwig creation, and then once that was all done I ran the test example and it worked fine.
However when I ran it on my own set of data it failed.
Exactly what sort of problem could there be with my conda environment ? For privacy concerns I sanitized file paths.
Thanks

Thanks for the reply. Actually Im sorry I didnt mention this but the example data worked fine for me. Since I'm using hg38 I had to fix some of the reference downloads and bigwig creation, and then once that was all done I ran the test example and it worked fine.
However when I ran it on my own set of data it failed.
Exactly what sort of problem could there be with my conda environment ? For privacy concerns I sanitized file paths.
Thanks

Hi @bwzee ,
Thanks for your reply! Have you tried running multi-thread mode using the demo and did it run well? it seems conda failed to use multiprocessing in your case.
Best wishes,
Yanmei

Hi
I actually ran this in multithreaded mode and it still failed. There doesn't seem to be an issue with the actual multitheading python library itself because other programs can use this package and run without any issues.


minor, clean deprecated configuration "kylin.job.controller.lock" in kylin-backward-compatibility.properties

Hi @shaofengshi , the previous commit don't clean this deprecated configuration completely. It's clean now after this commit.

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 3906

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.006%) to 25.935%

Files with Coverage Reduction
New Missed Lines
%

core-metadata/src/main/java/org/apache/kylin/source/datagen/ColumnGenerator.java
1
81.08%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
5
77.81%

Totals

Change from base Build 3901:
-0.006%

Covered Lines:
17686

Relevant Lines:
68193

💛  - Coveralls

Codecov Report

Merging #370 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #370      +/-   ##
============================================
- Coverage     23.64%   23.62%   -0.02%
+ Complexity     4838     4837       -1
============================================
  Files          1131     1131
  Lines         68193    68193
  Branches       9684     9684
============================================
- Hits          16121    16112       -9
- Misses        50506    50511       +5
- Partials       1566     1570       +4

Impacted Files
Coverage Δ
Complexity Δ

.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java
63.84% <0%> (-2.31%)
0% <0%> (ø)

...g/apache/kylin/source/datagen/ColumnGenerator.java
70.94% <0%> (-1.36%)
8% <0%> (ø)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
69.3% <0%> (-0.92%)
7% <0%> (ø)

...a/org/apache/kylin/dict/Number2BytesConverter.java
81.74% <0%> (-0.8%)
17% <0%> (-1%)

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 be5df4a...d2e431a. Read the comment docs.

Thanks, shaofeng!


Bump rubocop used for lining

Update the linter to a newer version. The old version seems to have issues with the update base TestFlight image.
Also addresses new warnings from the new linter.

merged with #319


bad file descriptor error

Describe the bug
Often notice me with this output:
_histdb_query_batch:7: 23: bad file descriptor

Screenshots

Desktop

OS + distribution: Arch Linux
Zsh version: 5.8.1
Plugin version: newest

This looks like it might be related to zsh-histdb Have you tried posting on that projects issue tracker?

Sorry for my fault. If u can, please delete this issue. I will post this issue on zsh-hisdb project.


How to draw box_map curve for fcos?

I use analyze_log.py to draw box_map curve for fcos,but encountered an error that no metric box_map

Please use the Error report issue template, otherwise we cannot provide any help.


[docs] Sets the instruction to extend the DefaultTheme interface in t…

Was following the v4 migration guide.
I noticed there was a problem on this line here:
declare module '@material-ui/styles' {
  interface DefaultTheme extends Theme {}
}

The problem is that it overrides every styles module, but what I really wanted was to extend the defaultTheme interface.
For this case it would be more recommendable to use this here:
declare module '@material-ui/styles/defaultTheme' {
  interface DefaultTheme extends Theme {}
}

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

@KsAkira10 We miss a reproduction for the issue.

Repro steps:
Using this example here:
https://github.com/mui-org/material-ui/tree/next/examples/nextjs-with-typescript
in src/Copyright.tsx change it for it:
import * as React from 'react';
import Typography from '@material-ui/core/Typography';
import MuiLink from '@material-ui/core/Link';
import { makeStyles, createStyles } from '@material-ui/styles';

const useClasses = makeStyles(({ palette }) => createStyles({
  root: {
    color: palette.text.secondary,
  }
}));

export default function Copyright() {
  const classes = useClasses();
  return (
    <Typography variant="body2" className={classes.root} align="center">
      {'Copyright © '}
      <MuiLink color="inherit" href="https://material-ui.com/">
        Your Website
      </MuiLink>{' '}
      {new Date().getFullYear()}.
    </Typography>
  );
}

in ./next-env.d.ts change it for it:
 /// <reference types="next" />
/// <reference types="next/types/global" />
/// <reference types="next/image-types/global" />

declare module '@material-ui/styles' {
  interface DefaultTheme extends Theme {}
}

in ./package.json change it for this:
{
  "name": "nextjs-with-typescript",
  "version": "5.0.0",
  "private": true,
  "dependencies": {
    "@emotion/cache": "latest",
    "@emotion/react": "latest",
    "@emotion/styled": "latest",
    "@emotion/server": "latest",
    "@material-ui/core": "next",
    "@material-ui/styles": "next",
    "clsx": "latest",
    "next": "10.2.2",
    "react": "17.0.2",
    "react-dom": "17.0.2"
  },
  "scripts": {
    "dev": "next",
    "build": "next build",
    "start": "next start",
    "post-update": "echo \"codesandbox preview only, need an update\" && yarn upgrade --latest"
  },
  "devDependencies": {
    "@types/node": "latest",
    "@types/react": "latest",
    "@types/react-dom": "latest",
    "typescript": "latest"
  }
}

@oliviertassinari I found this issue also when doing migration testing. The codemod will add what @KsAkira10 suggest.

Ok, I don't understand why the import path makes a difference, but if @siriwatknp is happy with it, then it sounds good.

@KsAkira10 It's not clear, I have tried your steps and it worked fine:
import * as React from 'react';
import Typography from '@material-ui/core/Typography';
import MuiLink from '@material-ui/core/Link';
import { makeStyles, createStyles } from '@material-ui/styles';
import { Theme } from '@material-ui/core/styles';

declare module '@material-ui/styles' {
  interface DefaultTheme extends Theme {}
}

const useClasses = makeStyles(({ palette }) => createStyles({
  root: {
    color: palette.text.secondary,
  }
}));

export default function Copyright() {
  const classes = useClasses();
  return (
    <Typography variant="body2" className={classes.root} align="center">
      {'Copyright © '}
      <MuiLink color="inherit" href="https://material-ui.com/">
        Your Website
      </MuiLink>{' '}
      {new Date().getFullYear()}.
    </Typography>
  );
}

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-next), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Please don't just paste snippets of code. Instead create a full example that reproduces a problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Automatic download of shapefiles not working on Windows 10

I couldn't get the automatic database download to work, when trying to generate the terrain it told me that some river tile was missing. I solved it by selecting a manual GSHGG Path in the plugin settings, downloading the dataset myself, extracting it and creating a touchFile. I'm not sure if I needed to do all those steps, for example the touchFile, but I looked in the source and tried to do the same things manually.
Error output (approximately): Could not run algorithm, couldn't find INPUT: WDBII_shp/f/WDBII_river_f_L01.shp
QGIS: 3.4.15-Madeira
OS: Windows 10 Pro N version 1909 build 18363.1379
Plugin installed via downloading the zip file and installing from QGIS.

@OlaRune I suggest using the plugin with Linux OS, works like a charm.

@OlaRune I suggest using the plugin with Linux OS, works like a charm.

My Windows desktop is more powerful than my Linux laptop, so I decided to use it on Windows. Maybe it isn't that much of a resource hog and the laptop would be enough.
The issue still stands though.

@erikquinn This is not really a "help wanted", since I've already found a workaround. It is more of a bug report / workaround, either for someone to fix or for others to find so they know how to proceed.

Unless I misunderstood, the issue is the fact that the automatic tile download process isn't working properly. The workaround is good information, but it would be good to just fix the auto-downloader thingy. And I don't know how, so stuck a "help us!" label for anyone who knows how to fix such things. 😄


Error processing liquid template causes site generation to silently fail

I had a misspelled tag in a file and my site stopped generating correctly.
After lots of printf-style debugging, I tracked it down to an unhandled exception causing the thread to exit.
5bc37d4ce2352935e45bcba8b6416d5e78d221d9 in master off of http://github.com/stevej/jekyll
catches the error and prints a message allowing the rest of your site generation to continue.

Solved. Latest jekyll shows errors Liquid Exception: bla bla


undergrads on people page

@sarahalamdari @nikitagrover can you guys add undergrad section to web page ?

@sarahalamdari @nikitagrover ^ can you do the above and also add a "group pets" section to the web page and add Boltzmann and Pizza? I kinda feel like we should havre a separate "Pets" section between Members and News.  But then we need to add Atticus too, which is totally fine.   :)

we could also just do pets @ the bottom of the members but waaaay cuter to have a pets page @anotherjoshsmith  how hard?

@pfaendtner very easy! Same style as the members page?

what does sarah think?
On Oct 15, 2018, at 10:00 PM, Josh Smith <notifications<EMAIL_ADDRESS>wrote:
@pfaendtnerhttps://github.com/pfaendtner very easy! Same style as the members page?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/UWPRG/UWPRG.github.io/issues/39#issuecomment-430074301, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD8szyv5W2s1zqVmNq5QxGzpdjfYWUttks5ulT2sgaJpZM4XHNo_.

hah!! that would be awesome. we can totally help out


Show Main hides the complete Chart

Describe the bug
Show Main disapears the complete chart
To Reproduce

Desktop (please complete the following information):

latest version of plugin
Core 2024.2.5
Supervisor 2024.02.1
Operating System Frontend 20240207.1
Safari and Firefox latest versions

Describe the bug Show Main disapears the complete chart
To Reproduce
Desktop (please complete the following information):

latest version of plugin
Core 2024.2.5
Supervisor 2024.02.1
Operating System Frontend 20240207.1
Safari and Firefox latest versions

SOLVED:
i deactivated entitie SUN and this is the problem. activated again and everything is ok.
TNX for your great Chart!


Provide Multi-AZ AutoScalingGroup for the scheduler

Instead of installing tools to handle scheduler failover, it's just simpler to launch the scheduler somewhere else.
Using an ASG for everythn also helps solving #95

Done in d7c2ec0c4a51fc414733574d8a87a41e0a4834e8


Make the performance dashboard private

Context
The performance dashboard being public has sparked discussions inside DfE about which data we should publish. Until these are resolved, make the dashboard private to support users.
Changes proposed in this pull request

the dashboard is no longer public
it now lives at support/performance/dashboard instead of integrations/performance-dashboard

😞

😔  At least we get a nicer URL…

😢  I was looking for this just now...


cmd/compile: memory usage above 1.5 GB

What version of Go are you using (go version)?
go 1.12.7 on NetBSD/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCACHE="/home/rillig/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="netbsd"
GOOS="netbsd"
GOPATH="/home/rillig/go"
GOPROXY=""
GORACE=""
GOROOT="/home/rillig/minipkg/go112"
GOTMPDIR=""
GOTOOLDIR="/home/rillig/minipkg/go112/pkg/tool/netbsd_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build435520277=/tmp/go-build -gno-record-gcc-switches"

What did you do?
bootstrap go1.12.7 using go14, via pkgsrc, the NetBSD package management framework
cd /usr/pkgsrc/lang/go112
bmake update
What did you expect to see?
Compiling go code takes less than 100 MB (ideally). 500 MB would already seem too high but still acceptable. But 1.7 GB is really too much.
What did you see instead?
  PID USERNAME PRI NICE   SIZE   RES STATE      TIME   WCPU    CPU COMMAND
 1160 rillig    78    0  1262M  675M RUN/0      0:31 85.50% 65.48% compile

The memory usage even went up to 1735 MB.

Can you find out what is being compiled when the compiler uses that much memory?

You wrote that what you're doing is:

bootstrapping go1.12.7 using go14

so I assume you're measuring the memory consumption of the old go1.4 toolchain while it builds the newest toolchain. That old 1.4 toolchain was written in C and it is kept around just for bootstrapping purposes, and it's basically frozen. It won't be optimized to make it faster or consume less memory.
Your screenshot shows the process using 675M of Resident Memory (RES), which is what you care about, so the title of your issue doesn't match your report, as it doesn't appear the toolchain is using 1.7GB of memory.
The newest compilers are written in Go and they also requires a fair amount of memory when bootstrapping master, anyway. You won't be able to compile the Go toolchain using 100 or 500 MB. AFAIK on 64-bit machines you'll need close to 800MB of memory per core.  In a previous build I didn't have swap memory enabled, and the build failed even earlier (the process 6g was killed then). Indeed, this is still less than 1 GB, so I must have measured something else when I initially wrote this issue.
Compared to go1.10.8, the largest process needed 27 MB more, which is a 4% increase. Since this is not much, it may have been pure coincidence that building go1.12 failed for me and building go1.10 had succeeded. Because of that difference I had opened this issue.
I still think 700 MB is quite a lot of memory for building a single go package, but that's probably already mentioned in several other issues, making this one redundant. Therefore I'm closing it.

Thanks for the info.  CC'ing @randall77 as FYI, although the issue is now closed.

Yes, cmd/compile/internal/ssa is a very large package. It contains lots of autogenerated code, 375K lines in total. The memory used to compile it is something to keep an eye on, but I don't think we need to do anything at the moment.  If we need to, it wouldn't be too hard to split out the autogenerated code into separate packages.


Double timestamps in logs

A recent update seems to have added duplicate timestamps to the container logs, I think https://github.com/lloesche/valheim-server-docker/pull/251:
$ docker logs -f valheim
Mar 24 08:20:42 /supervisord: valheim-server 03/24/2021 08:20:42: Zonesystem Start 278
Mar 24 08:20:42 /supervisord: valheim-server 03/24/2021 08:20:42: DungeonDB Start 278
Mar 24 08:20:43 /supervisord: valheim-server 03/24/2021 08:20:43: Game server connected

I think one timestamp ought to be enough, maybe even zero because docker logs has a -t option to add timestamps itself.

The update makes it so that all data is being logged via  a central syslog instance. All the components we have the source code for leave timestamp creation to syslog now. For Valheim server itself however we have no option of turning off their timestamps so you'll have them twice for now.


Chapter 14 Building a RNN

Hey Aurelien i want to first thank you for the great book i can't wait for your second edition.
i am having a lot of issues with building a rnn with anything else but the data specified in your book mnist and time-series data those work fine but when i try it with my data i get a multitude of errors a few which i solved however now i am well and truly stuck, this is the error :
InvalidArgumentError (see above for traceback): logits and labels must be same size: logits_size=[50,2] labels_size=[25,2]

here is my entire code, the data is a pandas df that was pickled so i don't have to do work on it every time i run the program
from sklearn.preprocessing import StandardScaler
import pandas as pd
import numpy as np
import tensorflow as tf

n_steps = 5
n_inputs = 11
n_neurons = 150
n_outputs = 2
batch_size = 50
n_epochs = 30
learning_rate = 0.001
mr_scaler = StandardScaler()

X_data_test = pd.read_pickle('X_data_test.pickle')
y_data_test = pd.read_pickle('y_data_test.pickle')
X_data_train = pd.read_pickle('X_data_train.pickle')
y_data_train = pd.read_pickle('y_data_train.pickle')
X_forward = pd.read_pickle('X_forward.pickle')
y_forward = pd.read_pickle('y_forward.pickle')
y_data_train = y_data_train.values.reshape((-1,1))

#X_data_train = mr_scaler.fit_transform(X_data_train)
#X_data_test = mr_scaler.fit_transform(X_data_test)
#X_forward = mr_scaler.fit_transform(X_forward)

X = tf.placeholder(tf.float32,[None,n_steps,n_inputs])
y = tf.placeholder(tf.int32,[None,n_steps,n_outputs])

basic_cell = tf.contrib.rnn.OutputProjectionWrapper(tf.contrib.rnn.BasicRNNCell(num_units=n_neurons),output_size = n_outputs)
outputs,states = tf.nn.dynamic_rnn(basic_cell,X,dtype=tf.float32)

logits = tf.layers.dense(outputs,n_outputs)
print(tf.shape(logits))

xenotropy = tf.nn.softmax_cross_entropy_with_logits(labels=y,logits=logits)
loss= tf.reduce_mean(xenotropy)
optimizer = tf.train.AdamOptimizer(learning_rate=learning_rate)
training_op = optimizer.minimize(loss)

logits_res = tf.reshape(logits, (-1, n_outputs))
logits_res = tf.squeeze(logits_res)
y_res = tf.reshape(y, (-1,n_outputs))
y_res = tf.squeeze(y_res)

correct_res = tf.nn.in_top_k(logits_res, y_res, 1)
correct = tf.reshape(correct_res, (-1, n_steps))

accuracy = tf.reduce_mean(tf.cast(correct,tf.float32))

init = tf.global_variables_initializer()

def fetch_batch(epoch, batch_index, batch_size):
    np.random.seed(epoch * n_batches + batch_index)
    indices = np.random.randint(len(X_data_train), size=batch_size)
    X_batch = X_data_train.values[indices]
    y_batch = y_data_train[indices]
    return X_batch, y_batch

n_batches = int(np.ceil(len(X_data_train/batch_size)))

with tf.Session() as sess:
	init.run()
	for epoch in range(n_epochs):
		for iteration in range(n_batches):
			X_batch,y_batch = fetch_batch(epoch,iteration,batch_size)
			X_batch = X_batch.reshape((-1,n_steps,n_inputs))
			y_batch = y_batch.reshape((-1,n_steps,n_outputs))
			sess.run([training_op],feed_dict={X:X_batch,y:y_batch})
		acc_train = accuracy.eval(feed_dict={X:X_batch,y:y_batch})
		acc_val = accuracy.eval(feed_dict={X:X_data_test,y:y_data_test})
		acc_forward = accuracy.eval(feed_dict={X:X_forward,y:y_forward})
		summary_str = accuracy_summary.eval(feed_dict={X:X_data_test,y:y_data_test})
		file_writer.add_summary(summary_str,epoch)
		print(epoch, ' Train accuracy: ',acc_train, ' Val accuracy:',acc_val,' Forward Accuracy',acc_forward)

I really hope you can help me  thanks :)

Hi @jetychill ,
Thanks for your kind words and your question. I will look into this issue this week.

I need some details on the data and what you are trying to do, please. As far as I understand, you want to perform binary classification on sequences of 5 time steps each, where each time step is 11 dimensional, is that correct?
If so, then the simplest way to go is to use the Keras API, something like this:
import tensorflow as tf
import numpy as np

n_steps = 5
n_inputs = 11
n_neurons = 150

# Fake data
X_train = np.random.randn(10000, n_steps, n_inputs)
y_train = np.random.randint(low=0, high=2, size=10000)
X_valid = np.random.randn(1000, n_steps, n_inputs)
y_valid = np.random.randint(low=0, high=2, size=1000)
X_test = np.random.randn(2000, n_steps, n_inputs)
y_test = np.random.randint(low=0, high=2, size=2000)

model = tf.keras.models.Sequential()
model.add(tf.keras.layers.LSTM(n_neurons, input_shape=[n_steps, n_inputs]))
model.add(tf.keras.layers.Dense(1, activation='sigmoid'))

model.compile(loss='binary_crossentropy',
              optimizer='rmsprop',
              metrics=['accuracy'])

model.fit(X_train, y_train, batch_size=16, epochs=10, validation_data=(X_valid, y_valid))
score = model.evaluate(X_test, y_test, batch_size=16)

Note that TensorFlow 2.0 will be out within the next few months, and it will strongly push users to migrate to Keras when using RNNs (read more about this here). For more details on TF 2.0, you can check out my YouTube video on this topic.
However, if you want to stick to using the tf.nn.rnn_cell API, or you just want to understand what you got wrong, then (still assuming you want to do binary classification of sequences) your neural network should just output a single probability for each sequence, not two probabilities per time step, so the final output shape should be [None, 1]. So this line looks suspicious to me:
y = tf.placeholder(tf.int32,[None,n_steps,n_outputs])

I would expect something like this:
n_outputs = 1
y = tf.placeholder(tf.int32, [None, n_outputs])

Next, you don't need the OutputProjectionWrapper in this case (again, assuming you want to do binary classification). Indeed, all it does is reduce the dimensionality of the output at each time step. But since you only care about the output at the last time step, it's a waste of resource to reduce the dimensionality at each time step. So I would define basic_cell like this (also note that tf.contrib.rnn was moved to the core API, so you should use tf.nn.rnn_cell instead):
basic_cell = tf.nn.rnn_cell.BasicRNNCell(num_units=n_neurons)

Next, you should probably use states instead of outputs, since you don't need the outputs at each and every time step, only at the last time step. In the case of the BasicRNNCell, the states is just equal to the outputs at the last time step). Alternatively, you could use outputs[:, -1] instead of outputs.
Hope this helps,
Aurélien

Hey Aurelien thank you for the in depth answer it helped me further understand the basics of how a RNN works, i did actually end up using the Keras api and a data generator to feed the sequences i honestly can't see myself going back to plain old tensorflow, but since we are at that topic do you think that using the Keras Api limits me in anyway, performance wise or limits the amount of customization of a network.
Thanks again

I'm glad I could help.  I used to think that Keras limited me for some exotic architectures (for example, when building Capsule Networks, see my video on this topic), but it has been more and more integrated into TensorFlow, and now I can't live without it.  The TensorFlow team probably felt the same way, because they clearly promote it in TF 2.0.  I don't think it will limit you in any way.  If you need something really exotic, you can always write your own custom layer.


REDIS.dbsize errors when there is no data in MockRedis

REDIS_APP_JOIN = Redis::Namespace.new(:appjoin, redis: MockRedis.new )
#
it 'invalid test' do
  # run the actual test
  expect(REDIS_APP_JOIN.dbsize).to eq 0
end
#
expected: 0
got: #<MockRedis::Future:0x00000001e8f660 @command=[:dbsize], @result_set=false>

The test is part of a gem I recently created https://github.com/dmitrypol/redis_app_join/blob/master/spec/redis_app_join_spec.rb#L37
Here is Gemfile.lock https://github.com/dmitrypol/redis_app_join/blob/master/Gemfile.lock
It works when there is data in Redis.  Has anyone experienced this?

We have a test that explicitly checks the case where db size is zero:
https://github.com/brigade/mock_redis/blob/68274651cfcfa7f5df9882160cdc1783a2688b44/spec/commands/dbsize_spec.rb#L7-L9
The fact that you're wrapping MockRedis with Redis::Namespace might be to blame here, though without an easily reproducible example (and the version of Redis server that you're using) it will be difficult to diagnose.
Would appreciate you digging into this further, as I personally won't have any time to look into this issue. Thanks!

Closing due to lack of response. Feel free to reopen with a reproducible example. Thanks!


Refactor and common controls

WIP. Open this early for review. I'll revert denied commits.

DropDown refactor completed now and for opinions.
The Menu class can be used in more occations, just representing "a list of commands/selections".

MenuItem has some issues in its current state.

It should be abstract, as it can never be used in its current state.
It expects things to be created but in no way enforces (or documents) it. Consider making CreateForeground and CreateBackground protected abstract methods, or at very least Trace.Assert(Foreground != null) Trace.Assert(Background != null) with appropriate messages.

Forgot to move corresponding part when splitting from DropDownMenuItem. Only members moved, but constructor not.


Pip install dapr does not install all the correct packages  in non-development mode

Expected Behavior
pip install dapr should install all the correct modules. Currently it looks like there are some modules that you only get when you do pip install dapr-dev that are being used in samples. Why are there different packages herehttps://github.com/dapr/python-sdk#install-dapr-python-sdk which make it confusing?
Actual Behavior
Samples fail with dapr.ext.grpc module not found
Steps to Reproduce the Problem
On a new machine do
pip install dapr
Then run pub/sub sample https://github.com/dapr/python-sdk/tree/master/examples/pubsub-simple
Fails with dapr.ext.grpc module not found
Also what is the dapr.proto.grpc module do?
Release Note

RELEASE NOTE: Updated dapr python sdk pkg documents.

This is not the code bug. it is doc bug. The other SDKs, dotnet-sdk and java-sdk, splits to multiple pkgs.
I will update the doc.

@msfussell document is updated. please try out.


Failing tests

The tests are failing with an error -
File "/tmp/pip-build-env-uklwrsnk/overlay/lib/python3.9/site-packages/numpy/distutils/command/build_ext.py", line 380, in build_extension
      extra_cflags = ext.extra_c_compile_args or []
  AttributeError: 'Extension' object has no attribute 'extra_c_compile_args'
  ----------------------------------------
  ERROR: Failed building wheel for scikits.odes
Failed to build scikits.odes
ERROR: Could not build wheels for scikits.odes, which is required to install pyproject.toml-based projects

I tried looking it up and found an issue (https://github.com/bmcage/odes/issues/130) in the odes repository related to the distutils module of numpy. The latest numpy release has a "Compatibility note" section that says -
Distutils forces strict floating point model on clang
NumPy now sets the -ftrapping-math option on clang to enforce correct
floating point error handling for universal functions. Clang defaults to
non-IEEE and C99 conform behaviour otherwise. This change (using the
equivalent but newer -ffp-exception-behavior=strict) was attempted in
NumPy 1.21, but was effectively never used.
I am not sure if this is the root problem and I couldn't find a fix or any related issue on GitHub.

This might be relevant https://github.com/pybamm-team/PyBaMM/pull/1355

Thanks for investigating, I commented on the issue asking if we should just skip scikit-odes tests for now (python 3.7 is still working, so we could just do the coverage using that)


Move SyncData into its own file

Minor (pure) refactoring in preparation for upcoming changes (no change was done to the code).

LGTM


Set header to generated value or transformed value

Has there been interest in adding more features to the "Set" header handler?
Something beyond the capability to copy or join values?
I personally would like to have the ability to generate UUID4 or UUID1 values for headers, but apart from generating random header values, I imagine there might be interest in transforming values. Stuff like uppercase, lowercase, hash sum, regex substitution and so on.
There is a very small PR that was rejected from traefik, but might fit very well into the scope of htransformation's "Set" handler:
https://github.com/traefik/traefik/pull/6707
TLDR: It adds a value "{uuid}"
If there is interest in this, I would get in touch with the PR's author and ask him to submit it as a PR here, or I would adapt it myself.
Background:
There has been a lot of interest in adding the ability to add a "X-Request-ID" or Correlation ID header to traefik before.
But those pull requests have been rejected with a proposed workaround to use jaeger tracing with a 0 sampling frequency to avoid the huge overhead and changing the header name from uber-trace-id to something like X-Request-ID:
            - "--tracing.jaeger.samplingParam=0"
            - "--tracing.jaeger.traceContextHeaderName=X-Request-ID"

This "kind of" works, as is abuses the jaeger tracing to add a generated value to each request but you have no control over the format of the value which is not a UUID (Something like: 17f19df25f472def:65d24975fb779d8d:17f19df25f472def:0 instead of 27155b04-20a6-4b64-87f6-0a458ae33734 )

Hello @gumbo2k,
Thanks for your interest in this Traefik Plugin!
This would even be a nice feature to be able to specify go code that would be interpreted at runtime to modify the headers content. It could look like this:
- Rule: # Header -> X-UUID: X-UUID-27155b04-20a6-4b64-87f6-0a458ae33734
      Name: 'UUID Set'
      Header: 'X-UUID'
      Value: 'X-UUID-{{uuid.New().String()}}'
      Type: 'Setf'

WDYT ?

We could use Yaegi a Go interpreter to do the heavy lifting for this, but indeed this would open up some security implications.
Nonetheless, Traefik Plugins are already interpreted at runtime, idk what it will do to interpret the thing a second time.

It would indeed be great, to have the power to evaluate golang inside those value strings.
I wonder if/how one could make the current headers of the request (or response) accessible, in order to set derived values...
- Rule:
      Name: 'Set X-Client-IP from first X-Forwarded-For'
      Header: 'X-Client-IP'
      Value: '{{current.header[`x-forwarded-for`].Split(`,`)[0]}}'
      Type: 'Setf'

I tried to do a POC of using Yaegi to do so, but it cannot be imported as Traefik forbids the use of exec inside a plugin.

- Rule:
     Name: 'Set X-Client-IP from first X-Forwarded-For'
     Header: 'X-Client-IP'
     Value: '{{current.header[`x-forwarded-for`].Split(`,`)[0]}}'
     Type: 'Setf'

But for this use case, you would be better off using/creating a specific plugin for it, and then using this plugin to manipulate headers.


Update about-protected-branches.md

Why:

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Approved thanks 😊
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/2698#pullrequestreview-564770831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR56TKG5IUF7EWZIMDZOPATSZAOVXANCNFSM4V3DPLUA
.

Teşekkürederim aşkım
9 Oca 2021 Cmt 12:28 tarihinde DozerKatDC10<EMAIL_ADDRESS>şunu
yazdı:

@DozerKatDC10 commented on this pull request.
Approved thanks 😊
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/2698#pullrequestreview-564770831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR56TKG5IUF7EWZIMDZOPATSZAOVXANCNFSM4V3DPLUA
.

Teşekkürler canım
9 Oca 2021 Cmt 12:28 tarihinde DozerKatDC10<EMAIL_ADDRESS>şunu
yazdı:

@DozerKatDC10 commented on this pull request.
Approved thanks 😊
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/2698#pullrequestreview-564770831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR56TKG5IUF7EWZIMDZOPATSZAOVXANCNFSM4V3DPLUA
.

Teşekkürler canım
9 Oca 2021 Cmt 12:28 tarihinde DozerKatDC10<EMAIL_ADDRESS>şunu
yazdı:

@DozerKatDC10 commented on this pull request.
Approved thanks 😊
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/2698#pullrequestreview-564770831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR56TKG5IUF7EWZIMDZOPATSZAOVXANCNFSM4V3DPLUA
.

✋🏻
9 Oca 2021 Cmt 12:49 tarihinde DozerKatDC10<EMAIL_ADDRESS>şunu
yazdı:

@DozerKatDC10 commented on this pull request.
Uninstall ed firefox deleted other emails
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/2698#pullrequestreview-564771832,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR56TKCPZGGERTT2I54766LSZARDHANCNFSM4V3DPLUA
.

✋🏻
9 Oca 2021 Cmt 12:49 tarihinde DozerKatDC10<EMAIL_ADDRESS>şunu
yazdı:

@DozerKatDC10 commented on this pull request.
Uninstall ed firefox deleted other emails
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/2698#pullrequestreview-564771832,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR56TKCPZGGERTT2I54766LSZARDHANCNFSM4V3DPLUA
.


fix(v2): evaluate code in live editor on client only

Motivation
Attempt to fix #2986 without much large changes.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Page with demo example - https://deploy-preview-4318--docusaurus-2.netlify.app/classic/examples/markdownPageExample#test
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@slorber this PR does not solve the problem that we currently performing twice rendering (in order to define the proper Prism theme on the client-side during the initial rendering).
For a more detailed I recommend you checkout these links:

Description of bug with dark Prism theme (FOUC), for the solution of which it was necessary to create the isClient state in the root App whice causes re-renders: https://github.com/facebook/docusaurus/pull/1984#issuecomment-569015188 (PR with implementation)
https://github.com/facebook/docusaurus/pull/2057#issuecomment-559813720
https://github.com/facebook/docusaurus/pull/2150

Well, perhaps there is a better solution for this case?

Perhaps there is a better solution, but this one seems to solve the problem without side effect for now so let's use that

Yes, my previous comment was a challenge for the future to make you aware of why the isClient state was added, which causes a side effect as double rendering.

thanks :) didn't know about these issues


Adds Portamento

A new parameter portamento=<time_ms> (m in wire protocol) sets a time-constant over which note changes (without intervening note-offs) are applied.

This is not yet integrated into the synth patches.  I suspect it will work directly with DX7, since I think the osc 0 pitch drives the whole note.  But for Juno, it will require some intermediation, since all three of the (pulse, sawtooth, subosc) oscs need to have their portamentos locked to the same value to make sure the whole note shifts as one.


Fix edge-case error attribution bug on WS downgrade

We determine whether to report Request vs RequestCreation based on whether destinationRequest is null or not.
If an exception was thrown by the request transform for the downgrade request, it would be reported as Request instead of RequestCreation.

When I was experimenting with request pooling I ended up with the request never being null, and the existing RequestCreation test failed.


Rename accounts_and_addresses

Description
Since accounts were removed (at the wallet level) change this directory to a more appropriate name.
Motivation
Cleanup.
Are you planning to do it yourself in a pull request?
Yes/No.

Closed by https://github.com/iotaledger/iota-sdk/pull/1939


Restricting the BLE peripheral device to connect to only one Master

Hi,
I have created a peripheral application in Xamarin forms using BluetoothLE.
I want to restrict my BLE Peripheral device to connect with only one master at a time always .IF i receive a connection request from some other master/central device, it should reject the connection request !

[x] iOS - Iphone8, Iphone8plus,IphoneX
[x] Android- All the versions above 8

Feature Library

[x] BluetoothLE

Version of OS(s) listed above with issue
Both Android and IOS have the same issue
Steps to Reproduce
1.Create Peripheral Application
2.Advertise packets (Connectable)
3.Central (Master) will get connect
4. Data exchange is happening between them contineously
5. Scan with another central
6.Connect from another central
7.It gets connect
8.Send some data from newly connectly Central
9. The data transfer between Firstly connected central got interuppted
Expected Behavior
When any request received from new central it should not accept, it should reject if it is already connected to the central and the connection is in active
Actual Behavior
Peripheral is connecting to the multiple centrals hence data exchange between the first connected central is getting disturbed. It should not happen.
Please anyone have any idea how to restrict multiple connections please help me to fix it.
Thanks in Advance.

This is the behaviour of the platform.  You are welcome to send a PR for this, but this isn't something I'll be doing.


Contributor doc tweak

I read the instructions and got it wrong, so another attempt at simplifying the instructions.

bot merge


MRES-32M Dataset Release

Hello.
Thank you for sharing the nice work!
I'm very interested in this work.
Are you going to release MRES-32M dataset?
Currently, RefCOCOm is only available.
I'm looking forward to hearing from you.
Thank you!

Looking forward to the open source of data engines and protective gear！

hello. i want to know how many references in MRES-32M


Importing solution Schema Bound Error

I have an environment with Accelerator V<IP_ADDRESS> installed.
I have a custom solution I have created on top of the Accelerator solution.
On attempting to install both Accelerator V<IP_ADDRESS> and V<IP_ADDRESS> I receive the following error:
Import failed: Cannot schema bind function 'dbo.fn_UDF_170b7e9618654a6f8e1a5ed85a5c8841'. 'dbo.msdyn_ProjectRisk' is not schema bound.
Could you please help me diagnose the issue?

I have actually found the issue as I have had an issue importing before.  The problem is that in my custom solution I am changing the calculation of field proj_costexposure to reference my custom fields in my custom solution instead.  By removing that solution layer (ie changing the calculation back to the original) the new accelerator will import.
The question then remains - why can you not change a calculation if you want Accelerator upgrades?


#583 and a few others

#583 and a few others

https://github.com/advantageous/qbit/issues/582

Queue (single thread handler, single thread sender) we have running at 100M calls per second. (In the past we have run many senders and one receiver thread at 200M requests per second).
ServiceQueue (using reflection) we have running at 18M calls per second for a call with no return.
Also cleaned up a memory leak when closing, and starting and closing and stopping a service queue.


Finding a needle in Haystack: Facebook’s photo storage

https://www.usenix.org/legacy/event/osdi10/tech/full_papers/Beaver.pdf?spm=a2c4e.11153940.blogcont37396.240.7c5c7b91q08hHy&file=Beaver.pdf

https://github.com/seaweedfs/seaweedfs-operator
https://github.com/chrislusf/seaweedfs

https://github.com/seaweedfs/seaweedfs-operator
https://github.com/chrislusf/seaweedfs

@gaocegege 链接 404 了 btw: 中秋快乐 :月饼:

Fixed


How to stream only a part of screen (or only a android view)?

I want to stream only a part of screen without capturing status bar or navigation bar.

If you want send an Android view you can see this (in this example you send only a button):
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/issues/125
If you want send only the Android view but no camera you can set background color to layout or use match parent in android view to set full screen. In this case you will hide camera behind layout.
Remember that your views must be inflated.

Thank you, but I want to stream a GLSurfaceView. Is it possible?

Use GlSurfaceView instead of my OpenGlView or stream both?
The first option is possible but need so many changes. The second option, I'm not sure I never try it.

GlSurface instead of your OpenGLView.
Can you please tell me what changes?

You need get InputSurface from VideoEncoder and copy GlSurfaceView frame to this InputSurface every times (I haven't an example for you now, you will need wait to monday or maybe tuesday and I will develop the example if you really need it).
Anyway OpenglView is more flexible than GlSurfaceView, also you can see LightOpenglView that is more easy to understand and modify. You can modify SimpleCameraRender and emulate your GLSurfaceView (of course you can ignore SurfaceTexture id on this class and no send camera frame)

Thank you very much.
I really really need it. So please help me if you have time.
Thank you.

I tried replace button of example to GlsurfaceView but not succeed.
Can you please help me find a solution?

CustomGLSurfaceView with stream was added: https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/a3ff2ca07cf0952b1d68ab655f1d04b08804a90d
AndroidViewFilter doesn't work with GLSurfaceView because this view don't draw anything on draw method in the canvas. The only way to do this view work is override that method and draw in the canvas with the result of GLES20.glReadPixels();

Thank you very much.
I tested but the streaming video was not smooth like this

Can you check it please?

Yes I know it. I didn't enabled AA in opengl. This is only an example. I will check it anyway this is only an orientation for developer to know, how to use your own custom GLSurfaceView to stream.

Thank you. I want to know that enable AA in opengl can make the output as same as the input?

AA is antialiasing (reason of jagged edges normally). If you are talking about this method:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/base/Camera1Base.java#L726
This is for FXAA antialiasing, It is low cost but no really good result (only available in OpenGlView, shader used: fxaa). The best way is use MSAA antialiasing.

I did a commit to enable AA in CustomGlSurfaceView. To enable AA you need uncomment this lines:
EGL14.EGL_SAMPLE_BUFFERS, 1 /* true */,
EGL14.EGL_SAMPLES, 4, /* increase to more smooth limit of your GPU */

You can find this lines here:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/rtplibrary/src/main/java/com/pedro/rtplibrary/view/CustomGlSurfaceView.java#L60
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/java/com/pedro/encoder/input/gl/SurfaceManager.java#L99
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/java/com/pedro/encoder/input/gl/SurfaceManager.java#L111

Thank you. But I am talking about the playback side.
https://user-images.githubusercontent.com/4996420/36705397-8395513a-1b97-11e8-8d0c-30cc7db037ac.gif
I sent captured glsurfaceview data to youtube server and the video is very lagged like this.

Low frame rate(fps)? I tested with youtube and work fine. Try with other devices and with other stream mode like using OpenGlView.
If you are talking about delay, nothing to do, youtube produce delay.

Thank you very much.


Generate and print backtraces on swift test crash to help debug the root cause

Generate and print backtraces on swift test crash to help debug the root cause

Hello @na-gupta -

Can you move this new logic to a separate file that we can then source from build_package.sh? Maybe you name it run_tests.sh or execute_tests.sh (something along those lines)?

Can you move line 142 to the line right after line 121? Or are you expecting anything in that new block to fail?

This seems to assume a given suffix for the module getting tested: *PackageTests.xctest? Would this following suffix work instead: *Tests.xctest? We have modules that do not end in PackageTests.xctest.

Can you rename the core variable to coreFile (or something similar)?

@rolivieri  Addressed 1,3 and 4.
For 2, I did it this way because I did not want any failures/errors in the core handling block to cause the script to exit, but can change it if you still want to.

@na-gupta Thanks. One more question, with these changes, what happens to the status of the build if any of the test cases fail? We still need to build to be marked as failure under such scenario after you collect/display the core dump. Since we are executing set -e after evaluating the value of TEST_EXIT_CODE, would that mean that the build will be labeled as successful even if a test failed? Maybe I missed something.

Line 26 of run_tests.sh - exit $TEST_EXIT_CODE - causes the script to exit with the exit code for swift test whenever the exit code is non-zero.

@na-gupta Got it, I overlooked that. Sorry one more thing, can you do this PR against develop? We now have a develop branch where we merge all PRs.

@rolivieri Done. Thanks!


Keyboard will flash in a "medium damage/in danger state" (orange/yellow) in ROTTR after first bear attack even after healing/out of danger

What are you experiencing an issue with?

[ ] Aurora
[ X] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[ ] Aurora Profile
[X ] Aurora supported device, (Logitec G910 +G502)
[ ] Aurora Scripts
[ ] Aurora Updater
[X ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[ ] Performance problems
[X ] Visual/lighting bug
[ ] Localization issue
[ ] Enhancement suggestion
[] Device issue
[ ] Other, (Fill in another type of issue here)

Expected behavior:
Keyboard should no longer flash as if the player damaged/ in danger after healing/ player is out of danger or major battle has ended.
Actual behavior:
Keyboard continued to flash orange if the player is no longer in "in danger" state
How often does this problem occur?
After first encounter with the bear very early game
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Using Aurora with logitec g910 + 502 mouse
Playing rise of the tomb raider until first bear encounter, the game will put you into a damaged state until you heal (scripted event in game) even after healing, the lighting will reflect as if the character is damaged.
3.Will reset to normal after fast travel or /game/aurora/computer restart

What is the version of Aurora you are experiencing this issue with?
Current release as of 5/7/18
Does the previous version of Aurora have this issue?
did not test, first time using Aurora However after restarting my computer and re-entering the game after the fight, the keyboard lighting went back to normal. (UPDATE: also resets after fast travel after such event)
Please include any logs, screenshots, or files that are related to this issue:

After continued game play you end up fighting the bear again, however the orange flashing started before the fight, indicating that the orange flashing means you are in potential danger maybe? Even after the fight is over the keyboard continues to flash, I will keep playing without a restart this time and see if it goes away as this may be intentional, but I am unsure.

UPDATE: so after being able to fast travel for the first time back to the first camp you are able to make, the "in danger orange" flashing stops, which may indicate a bug with the game profile after events like this occur.

Seems to have come back randomly while re-exploring the first area. Haven't been damaged yet as far as I can tell and there are no enemies around.

After fast traveling a second time it did go away again. I did record what happened via cell phone video.

The lighting effects shown for RoTTR is just whatever the game normally sends to the Razer devices, we have no control over what is shown and when. It's possible there is an issue with the wrapper causing it to light incorrectly, however, from your description it sounds like the lighting effects aren't being generated correctly by the game.


feat(LightProfinite): light profinite sets as sequential limits with surjective transition maps

Adds the file LightProfinite/AsLimit which mirrors Profinite/AsLimit.

[x] depends on: #13319

bors merge


Enhancement Request: Grid Numbers

A handy option would be a way to automatically place grid numbers.  Basically having an extra row and column to each contain one half of the grid's coordinate pairs showing on the map without needing to check the coordinates shown in the bottom right of the window.
Something like this:
   00 01 02 03 04 05 06 07 08 09
00
01
02
03
04
05
06
07
08
09

Nevermind.  Just found it.

@dericpage Glad you've found it 😄 Reading the manual helps finding these little "hidden" features; there are many of them.


Revision of approximations

The issue at hand pertains to the ReactiveMP.jl folder named approximations/, which is growing and being poorly maintained.
In essence, approximation methods should not be included in message-passing since they are semantically distinct entities.
The proposed solution is to enhance and relocate the approximation/ folder within ExpectationApproximations.jl

Is this a duplicate of #27?


cargo lambda build --bin not behaving as expected

I am in a mono-repo with a bunch of different bins and when I run the command to build with a single bin specified, and with the --verbose command I see that the majority of the time is spend on "checking function binary" of lots of binaries that I am not looking to build.
Was wondering if this was expected behavior and there is a flag I'm missing or this is a bug and the code should be modified to not check the function binaries if a specific bin is specified? In the meanwhile during development I just comment out all of the other bins for faster development iterations.

Logs of what it looks like:
DEBUG run: checking function binary binary="/local/home/drydench/Workspace/ReplicatorWorkflow/src/ReplicatorRustLambdas/build/private/cargo-target/x86_64-unknown-linux-gnu/debug/features" exists=true
DEBUG run: checking function binary binary="/local/home/drydench/Workspace/ReplicatorWorkflow/src/ReplicatorRustLambdas/build/private/cargo-target/x86_64-unknown-linux-gnu/debug/validateInputs" exists=true


⚠ Bofrost.at has degraded performance

In 0a637d6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1762 ms

Resolved: Bofrost.at performance has improved in 9cbfe69 after 9 minutes.


tag v0.2.1

@JuliaRegistrator register branch=release/v0.2

Registration pull request created: JuliaRegistries/General/89725
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.1 -m "<description of version>" f695b8491a5871fe0523f7e657a5fa3a05b68e06
git push origin v0.2.1


fix #29

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

23 of 23 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 89.165%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1111

Relevant Lines:
1246

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

23 of 23 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 89.165%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1111

Relevant Lines:
1246

💛  - Coveralls


Enable nf_conntrack module by default

What this PR does / why we need it:
Enable nf_conntrack (nf_conntrack_ipv4) module by default on all operating systems. This fixes an issue with pods unable to reach services running on a host on operating systems that are using the NFT backend
What type of PR is this?
/kind bug
Does this PR introduce a user-facing change? Then add your Release Note here:
Enable `nf_conntrack` (`nf_conntrack_ipv4`) module by default on all operating systems. This fixes an issue with pods unable to reach services running on a host on operating systems that are using the NFT backend.

Documentation:
NONE

/test pull-kubeone-e2e-aws-default-install-containerd-v1.24.3
/test pull-kubeone-e2e-aws-centos-install-containerd-v1.24.3
/test pull-kubeone-e2e-aws-rhel-install-containerd-v1.24.3
/test pull-kubeone-e2e-aws-rockylinux-install-containerd-v1.24.3
/test pull-kubeone-e2e-aws-flatcar-install-containerd-v1.24.3
/test pull-kubeone-e2e-aws-amzn-install-containerd-v1.24.3


Return value for route-action helper

This is not an ideal solution, but I wanted to open this PR to get the ball rolling and to add a test.
Closes #3

I'll try to review in more detail when I'm back at a computer.
A few general concerns (that we can add tests for):

this looks the action up in the current route structure at the time the helper is rendered, and that it may change between then and when it is invoked.
_actions is a deprecated property in Ember 2.x (in favor of just using actions directly), so we should use actions and fallback to _actions to avoid the deprecation

I addressed your comments, I think

What about action bubbling by return true in an action?  Do we intend to support that also?

Hmm, I'm not sure if we should support that. Wouldn't you just use the ordinary string action in that case?

Ya, I don't actually care if we support it or not, but we need to be deliberate about it.  We should also update the README with a decent explanation (of what we do and do not support).

Ya, I'll update and ping you again when it's ready for review!

@rwjblue Updated! :eyes: please

Ok, I'm game to try this. We can always rollback if we hit significant issues...

@homu r+

:pushpin: Commit e9e5179 has been approved by rwjblue

:zap: Test exempted - status


Beautify recursive directory tree of files

This is a question, not an issue.
Is it possible to give a path to a folder containing JS files to js-beautify?

Not yet. Similar to #787.

Someone seems to have forked this project just to add directory recursion.
https://github.com/delta-db/beautify-proj
@delta-db @redgeoff
Would you be willing to submit a PR to add this functionality to this repo?

Is it possible to give a path to a folder containing JS files to js-beautify?

Can this can be handled by file globs (#1580)? If so, this issue can be closed.

@spl
It isn't quite the same thing.  Globs are supported, but that is different than pointing a directory.

@bitwiseman I thought you could recursively reference all of the contents of a directory dir with something like dir/**/*. But I'm no file glob expert, so perhaps there's something missed by that pattern.


[WIP] Add team switch shortcut

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[ ] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
This is a rough work in progress aiming to add a shortcut for team switch
Issue link
https://github.com/mattermost/desktop/issues/850
Test Cases

Connect to a mattermost instance where you belong to two teams
Press Cmd/Ctrl+1 to switch to team 1
Press Cmd/Ctrl+2 to switch to team 2
Press Cmd/Ctrl+3 to switch to team 3

Additional Notes
As I said, this is very early in the development but pushing this PR will help me discussed with advanced electron developer (I'm just a newbie in this area).
Currently the teams are added statically (three team numbered from 1 to 3) but I would like it to be dynamic and correctly named.

Hi @MartinDelille, thanks for your interests!
The desktop app already has shortcuts Cmd/Ctrl+{1~9} to switch servers. Just curious, how will you arrange them? And if possible, I suggest to implement this kind of shortcut in webapp. Users can switch teams also on web browsers.

@yuya-oc you're right. I could replace the Cmd/Ctrl+{1~9} by Cmd/Ctrl + Shift +{1~9}.
I will look at the server code.