Not working in Windows

Under WSL with Node.js v17, I get the following in both the browser and shell. The file in question exists (it's a copy of a jscad example from https://www.openjscad.xyz/ ?->"Building a Dodecahedron") and can even be accessed from the opened server page.
Can't walk dependency graph: Cannot find module 'test_jscad.js' from '/home/david/test/jscad-now.js'
    required by /home/david/test/jscad-now.js

Under Windows 10 directly via Powershell in Node v16 I get no errors in the shell, a blank page in the browser, and an exception in the browser javascript console
parameterControls.js:19 Uncaught TypeError: Cannot read properties of null (reading 'map')
    at createParamControls (jscad-now.js:39537:41)
    at designParameters (jscad-now.js:39488:24)
    at module.exports (jscad-now.js:39341:45)
    at module.exports (jscad-now.js:39325:3)
    at Object.606../test1.jscad (jscad-now.js:39934:1)
    at o (jscad-now.js:1:265)
    at r (jscad-now.js:1:431)
    at jscad-now.js:1:460

Also, fyi, npm is reporting 5 'high severity vulnerabilities'.

My first instinct is that this must be happening somewhere in the Budo library layer / or its dependencies.
Probably also related to the vulnerabilities, as it does not seem to be maintained.

We can probably switch to webpack. I only chose Budo since it was already part of the regl-renderer 's dev script.
THis project is just a quick takeoff from that.
https://github.com/jscad/OpenJSCAD.org/blob/b4d808a34fccd3fc72a327921dfd9fa997de9d99/packages/utils/regl-renderer/package.json#L11

Is there any workaround for this, or is it just busted? I'm having the same problem on Debian.

Do your models provide parameter definitions? The previous code was expecting it:
const main = ()=> cube();

function getParameterDefinitions() {
    return [];
}
module.exports = { main, getParameterDefinitions }

I've just fixed this in 1.2.2, getParameterDefinitions is no longer required. Hopefully this fixes the issue.


R&D porting the existing pilot codes for alternate search results view(s)

This will include the following feature in dams5-cc-pilot:
https://github.com/ucsdlib/dams5-cc-pilot/issues/9

@hweng I think Sufia already includes this multiple view feature, which looks good. So feel free to close the ticket.


Integrate within PYPIT

i.e. no longer as a separate Repo.
What do you think @rcooke-ast ?
Is it time now?

Ahhh... I had no idea that this was a plan. I suppose it makes sense. It depends if you think it has some value outside of PYPIT. Do you have a preference?

Good point.  It should have value outside PYPIT.
At least the arc line lists..
Closing

Reopening this issue... If we make changes to arclines, it seems unreasonable to expect that people using PYPIT will know to update both packages. It seems sensible, therefore, to include the arclines code into PYPIT. What do you think, @profxj? I wouldn't object to including the arc line lists too, for the record.

I now agree.  While I am going to merge #30
we will push arclines into PYPIT before long.
Maybe as a sub-package, if that is easy enough to
do.


[Servicebus] az servicebus namespace authorization-rule az servicebus queue authorization-rule az servicebus topic authorization-rule:Migrating servicebus namespace, queue, topic authorization-rule cmdlets to aaz

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Servicebus

This PR is for Migrating servicebus namespace authorization-rule, servicebus queue authorization-rule, servicebus topic authorization-rule cmdlets to aaz.
Can someone please look into is & review this.
Thank you!


Indexing with ByteIndex

If I'm not mistaken, ByteIndex uses 1-based indexing, but that is never mentioned in the docs. This should probably be documented, since I would have expected it to be 0-based.
I'd happily submit a PR with some changes :)

This is because of how we don't have a separate 'file id' to differentiate files in the filemap. the '0' index is intended to denote positions that don't correspond to any file. I realize that this is confusing though! As mentioned in #52, the language-reporting crate simplifies this way of doing things greatly, by instead just going with a separate file id.

Definitely makes more sense in that case, I'm not sure how I even managed to miss ByteIndex::none to begin with.

Closed by #54.


broccoli-persistent-filter overwrites source files

I discovered that under certain conditions, when the input node contains two conflicting files like file.js and file.coffee, broccoli-persistent-filter will overwrite files in the source directory. I'm not sure if the culprit is broccoli-persistent-filter or fs-tree-diff, so instead of writing a test case I just created a sample repo for now. To reproduce this issue, run:
git clone https://github.com/joliss/broccoli-persistent-filter-issue
cd broccoli-persistent-filter-issue
npm install
./node_modules/.bin/broccoli serve

Then in a new terminal, run
echo 's = "CoffeeScript source"' > src/file.coffee
sleep 1
git diff src/file.js

src/file.js has been overwritten with the CoffeeScript output.

interesting...

I see the issue:

build one
2.src/file.js is sylinked to outputPath/src/file.js
build two
4.src/file.coffee's output is written to  outputPath/src/file.js, unfortunately outputPath/src/file.js is a symlink back to src/file.js

The solution I believe is:
_handleFile should keep track of symlinks it created, and if they end up being overridden later they are first unlinked.

thanks for reporting, sorry for lingering.


Sync eng/common directory with azure-sdk-tools for PR 2668

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/2668  See eng/common workflow

/azp run js - core - ci


Add hardware section to metadata

Related epic
https://app.shortcut.com/xanaduai/epic/45372/users-can-see-example-code-that-demonstrates-how-demos-run-on-real-quantum-hardware-from-aws
Changes

Updated the demo metadata schema to include an optional hardware section (only to include AWS for the time being, but can accommodate more vendors in future)
Added the hardware metadata to 4 demos

We don't have the real demo links from AWS yet, so I just put the PennyLane demo links there for now

Added AWS logo image asset

LGTM - let's just remember that Amazon will be creating new demos to host on our site to link to, so we'll have to change the demos we add these to for launch.

For testing, should I maybe remove the hardware metadata from all but one of the four demos I added it to on this PR?

LGTM - let's just remember that Amazon will be creating new demos to host on our site to link to, so we'll have to change the demos we add these to for launch.

For testing, should I maybe remove the hardware metadata from all but one of the four demos I added it to on this PR?

I think that's a good idea!


Inquiry About Open Source Plans

Thanks for your interesting work. When do you release the code?

Sorry, things have been crazy lately, we'll be sorting out the code in the next few days.


Add APKs project node and Actions

Add new Android project Node to show build APKs.
Actions:
Save As..
Install..
Copy
Properties


Add catboost

Note, this addressed the main idea behind #11 , but we'd still like catboost to be faster in gboost_module.


Add tests and documentation to the IRI shrinking feature

Discussed in #5.

Beautiful, thanks!

v0.1.1 is published now with that change.


[SPARK-39415][CORE] Local mode supports HadoopDelegationTokenManager

What changes were proposed in this pull request?
Start HadoopDelegationTokenManager when LocalSchedulerBackend starts.
The behavior is similar to CoarseGrainedSchedulerBackend startup, but it does not support local mode.
Why are the changes needed?
After SPARK-23639, we can use spark-sql --master=local --proxy-user xxx to proxy a user to access the hive meta store in the kerberos environment.
But spark-submit does not support accessing the hive meta store, we will get The following exception.
No valid credentials provided (Mechanism level: Failed to find any Kerberos tgt)
And spark-sql/spark-submit will not automatically refresh the token or automatically relogin when using --keytab
Does this PR introduce any user-facing change?
No
How was this patch tested?
Write a simple spark to access the Hive metastore application in the kerberos environment, submit it through spark-submit --proxy-user, in this PR, it can work normally.

Duplicate #32009


dask.dataframe.read_parquet fails unless written by fastparquet

The (current) pandas.Dataframe.to_parquet uses pyarrow as its parquet backend. dask.dataframe.read_parquet(_, engine='fastparquet') fails. It appears to be incorrectly handling "hive-style" partitioning.
I am using dask version ('2.8.1'), pyarrow ('0.15.1'), and fastparquet ('0.3.2').
$ conda create -c conda-forge -n dask.arrow.issue -y dask fastparquet pyarrow jupyterlab python-snappy

To reproduce
import dask, pyarrow, fastparquet
print((dask.__version__, pyarrow.__version__, fastparquet.__version__)) # ('2.9.1', '0.15.1', '0.3.2')

import pandas as pd
import dask.dataframe as dd
import pyarrow.parquet as pq

df = pd.DataFrame([{'str': str(i), 'int':i, 'group': 'ABC'[i%3]} for i in range(6)])

pq_dir = './parquet_partition_issue'
df.to_parquet(pq_dir, partition_cols=['group'])
# or the equivalent: df.to_parquet(pq_dir, partition_cols=['group'], engine='pyarrow')

ddf_fast = dd.read_parquet(pq_dir, engine='fastparquet')

# IsADirectoryError: [Errno 21] Is a directory: 'parquet_partition_issue/group=A'

Just a note that it is a known problem that fastparquet has problems reading directories written with pyarrow.  The problem is likley in fastparquet, rather than dask.  For example, I also get an error when using pandas for both the write and read:
import pandas as pd

df = pd.DataFrame([{'str': str(i), 'int':i, 'group': 'ABC'[i%3]} for i in range(6)])

pq_dir = 'parquet_partition_issue'
df.to_parquet(pq_dir, partition_cols=['group'], engine='pyarrow')

pd.read_parquet(pq_dir, engine='fastparquet')

# Note: Setting `engine='pyarrow'` works fine

Click for Output

---------------------------------------------------------------------------
FileNotFoundError                         Traceback (most recent call last)
~/miniconda3/envs/cudf++12/lib/python3.7/site-packages/fastparquet/api.py in __init__(self, fn, verify, open_with, root, sep)
    109                 self.fn = fn2
--> 110                 with open_with(fn2, 'rb') as f:
    111                     self._parse_header(f, verify)

~/miniconda3/envs/cudf++12/lib/python3.7/site-packages/fastparquet/util.py in default_open(f, mode)
     37 def default_open(f, mode='rb'):
---> 38     return open(f, mode)
     39

FileNotFoundError: [Errno 2] No such file or directory: 'parquet_partition_issue/_metadata'

During handling of the above exception, another exception occurred:

IsADirectoryError                         Traceback (most recent call last)
<ipython-input-2-38271898390a> in <module>
     10 os.system("rm -rf "+pq_dir)
     11 df.to_parquet(pq_dir, partition_cols=['group'], engine='pyarrow')
---> 12 pd.read_parquet(pq_dir, engine='fastparquet')

~/miniconda3/envs/cudf++12/lib/python3.7/site-packages/pandas/io/parquet.py in read_parquet(path, engine, columns, **kwargs)
    294
    295     impl = get_engine(engine)
--> 296     return impl.read(path, columns=columns, **kwargs)

~/miniconda3/envs/cudf++12/lib/python3.7/site-packages/pandas/io/parquet.py in read(self, path, columns, **kwargs)
    197         else:
    198             path, _, _, _ = get_filepath_or_buffer(path)
--> 199             parquet_file = self.api.ParquetFile(path)
    200
    201         return parquet_file.to_pandas(columns=columns, **kwargs)

~/miniconda3/envs/cudf++12/lib/python3.7/site-packages/fastparquet/api.py in __init__(self, fn, verify, open_with, root, sep)
    113             except (IOError, OSError):
    114                 self.fn = join_path(fn)
--> 115                 with open_with(fn, 'rb') as f:
    116                     self._parse_header(f, verify)
    117         self.open = open_with

~/miniconda3/envs/cudf++12/lib/python3.7/site-packages/fastparquet/util.py in default_open(f, mode)
     36
     37 def default_open(f, mode='rb'):
---> 38     return open(f, mode)
     39
     40

IsADirectoryError: [Errno 21] Is a directory: 'parquet_partition_issue'

Agreed that the issue most likely resides in the engine rather than dask (most likely in the fastparquet side of things). Closing here, though let us know if you make progress @caseyclements.


Enable model.properties updates when using structs

Okay, I think it is about time we port DrWatson's access function here... Let's just define a function set_value!(container, key, value) which dispatches to set index for arrays and dictionaries and to set property for anything else....... Otherwise you'll keep doing these bugfixes for the rest of your life xD

Never used DrWatson before. I'll have a look if I can make it work.

Sure, this is doable. I will implement three helper functions in utils.jl: has_value!, get_value!, and set_value!. Hopefully this will cover most (if not all) of our use cases.
Will push the changes to this PR here and rename it appropriately.

Anything else you'd like me to incorporate into this PR?


[qt5-webengine] potentially affected by CVE-2023-4863

Is your feature request related to a problem? Please describe.
The port qt5-webengine uses a bundled copy of chromium that in turn uses a bundled copy of libwebp. This copy of libwebp has version 1.1.0 that is affected by CVE-2023-4863.
The security advisory from Qt regarding CVE-2023-4863 contains a patch for imageformats, but not for webengine.
Proposed solution
(1) Make libwebp a direct dependency of qt5-webengine.
(2) Patch the bundled copy of chromium in  qt5-webengine to use libwebp from vcpkg rather than the bundled version.
Step 2 involves replacing the file src/3rdparty/chromium/third_party/libwebp/BUILD.gn with one that points to the system installation of libwebp which can be provided by vcpkg.
Describe alternatives you've considered

Alternative 1: Wait for an official patch, if it is deemed acceptable to expose users to CVE-2023-4863.
Alternative 2: Patch the local copy of libwebp with an updated version.
Alternative 3: Implement the proposed solution, but as a port feature rather than the default.

Additional context
The bundled copy of libwebp appears to have local changes as stated in  src/3rdparty/chromium/third_party/libwebp/README.chromium:
Local changes:
 * Removed documentation and build related files
 * Merged COPYING/PATENTS to LICENSE
 * Disabled the SSE2 version of VerticalUnfilter due to a crash on android
   (crbug.com/654974)
 * Pulled in fuzzers from OSS-Fuzz
   https://github.com/google/oss-fuzz/tree/master/projects/libwebp

This makes it a bit harder to argue for for using the version from vcpkg. However, there appears to be a build option for linux switches to the system installtion of libwebp.
Advice and suggestions for other alternatives are welcome. Thank you.

This would be a possible source for an "unbundled" BUILD.gn file for libwebp, as this is from the same version of Chromium used in Qt 5.15.10.


Fix segfault when no_cookies=true

Tried disabling cookies, got a SIGSEGV, wah wah. I would've assumed go initializes to null by default, but apparently not.

I'll have to devote a day or two to learning go. Cheers for your work - I've been using it for months, since before you had ietf support certainly, and am very happy with it.


Hashes provider UUIDs

Identifies and hashes identity provider unique identifiers.
Consolidates ability to look through multiple fields until values are found; including finding a valid email address.
Partially resolves #899
Second PR will add db-based remapping that solves mapping UUIDs between multiple identity providers.
Note: filter by commit, this includes dependent PRs; can be submitted as an independent PR if necessary.

pending rebase


store -> store all tabs in current window, does not show saved tab-list immediately

扩展版本 (extension version): 1.4.5
浏览器版本 (browser version): Chrome  74.0.3729.169 64 bit
操作系统 (operating system): Windows 10
问题描述 (problem description): store -> store all tabs in current window, malfunction: does not immediately show the new tabgroup in application page, it's just scary, after a while seems to appear, but I think I have lost a tab group
实际行为 (actual behavior): the new tab list DOES NOT immediately appear on top of better onetab page
期望行为 (expected behavior): the new tab list APPEARS IMMEDIATELY on top of better onetab page
重现步骤 (reproduce step): right-click on extension icon -> store -> store all tabs in current window
Let me write a few word of appreciation for your effort in creating better-onetab, I loved it but dropped it because it is unreliable, it would be a great feature if you would add to better-onetab an easy way to backup or save in the cloud the saved tabs.
Keep it up!

+1

I have a similar issue when taking the same steps. I'm using same browser and OS. I have had several times that the tabs never showed up though. Please help!

+1 on Ubuntu 16.04 with Chrome Version 76.0.3809.132 (Official Build) (64-bit) - It happened today, the list just did not display, so I came here to Issues, and found this post, and went back and looked at my Better Onetab, and there was the list.  Whew.  I must check if the same happened last week.  It only happens to a list of tabs, count over 10-12, but not all the time.  I have not had this problem when doing a single tab.  I do believe both times I selected the option to do all tabs in current window.
It might have to do with not having enough swap space?  I had 4gig, and 3 gig had to be removed, so running only on 1 gig at this time.  Point is, with swapping, things slow down, and might not this slow down account for the lack of update being displayed?  I have not looked at the code base to see if this could happen.  A flag method would prevent it?  Hope this helps.


Fatal error in PsrResponse

Hi,
on php version 8.2.20, I get following fatal:
PHP Fatal error:  Declaration of Comgate\SDK\Http\PsrResponse::withProtocolVersion(string $version): Psr\Http\Message\MessageInterface must be compatible with Psr\Http\Message\MessageInterface::withProtocolVersion($version) in /vendor/comgate/sdk/src/Http/PsrResponse.php on line 91

It should be resolved in v1.0.14. Can you check it? It was a problem with some version of PSR package. You have a different version propably and now SDK will require specific version.

thank you, PSR reinstalled to newest version, after reinstalling comgate/sdk , I get following error:
PHP Fatal error:  Uncaught Error: Class "GuzzleHttp\Psr7\Query" not found in /vendor/comgate/sdk/src/Entity/Response/PaymentCreateResponse.php:28
"composer require comgate/sdk" install only these packages (with no warning/error):

Locking comgate/sdk (v1.0.14)
Locking psr/log (3.0.0)
Installing psr/log (3.0.0): Extracting archive
Installing comgate/sdk (v1.0.14): Extracting archivee-dev)

installed guzzle separately and its ok


[BUG] apply_noise referenced before assignment

Describe the bug
In timm/scheduler/scheduler.py local variable apply_noise may be returned not initialized if no noise is applied.
To Reproduce
Steps to reproduce the behavior:

run training with the following command:
python train.py --batch-size 14 --model swin_v2_cr_tiny_224 --pin-mem ./data
3.Wait for Tests ending after First epoch

Test: [   0/82]  Time: 0.205 (0.205)  Loss:  5.6608 (5.6608)  Acc@1:  0.0000 ( 0.0000)  Acc@5:  0.0000 ( 0.0000)
Test: [  50/82]  Time: 0.077 (0.079)  Loss:  1.1223 (1.8036)  Acc@1: 92.8571 (35.4342)  Acc@5: 100.0000 (89.9160)
Test: [  82/82]  Time: 0.076 (0.079)  Loss:  3.4149 (1.6274)  Acc@1: 14.2857 (32.4441)  Acc@5: 50.0000 (93.2014)
Traceback (most recent call last):
File "/home/guest/IA/IA/CLASSIFIER/pytorch-image-models/train.py", line 834, in
main()
File "/home/guest/IA/IA/CLASSIFIER/pytorch-image-models/train.py", line 643, in main
lr_scheduler.step(epoch + 1, eval_metrics[eval_metric])
File "/home/guest/IA/IA/CLASSIFIER/pytorch-image-models/timm/scheduler/scheduler.py", line 71, in step
values = self._add_noise(values, epoch)
File "/home/guest/IA/IA/CLASSIFIER/pytorch-image-models/timm/scheduler/scheduler.py", line 88, in _add_noise
if self._is_apply_noise(t):
File "/home/guest/IA/IA/CLASSIFIER/pytorch-image-models/timm/scheduler/scheduler.py", line 100, in _is_apply_noise
return apply_noise
UnboundLocalError: local variable 'apply_noise' referenced before assignment
function   _is_apply_noise(self, t) should probably initialize apply_noise variable to False as first instruction ?
Desktop (please complete the following information):

OS: Ubuntu 21.10
commit 9440a50c954dd4fa2a0fffab05ea7a69d73cdd32

@SixK sorry, working through comining multiple outstanding merges right now and trying to pull them together before doing final test and fix... will address this shortly

@SixK FYI swin v2 is still under development and consider unstable (despite merge), I needed to get it off the 3rd party merge branch and integrated with other changes, there are some changes outstanding that would break weight compatibility.


FIX: Unlink destination files before saving derivative

If you have write permissions in a directory but not on a file (main context: locked git-annex files), then you can get a situation where trying to write without first deleting will fail while deleting and rewriting will succeed.

Hello @effigies, thank you for submitting the Pull Request!

In the file niworkflows/interfaces/bids.py:

Line 667:13: F821 undefined name 'os'

To test for issues locally, pip install flake8 and then run flake8 niworkflows.


UI unreadable if font is Inter

Lapce Version
0.2.1
System information
Alpine Linux edge
Describe the bug
UI font is so small that it's hard to see anything, has lots of space between the characters
Additional information
window manager: sway (Wayland), haven't tried any other
output is using scale 1.2 setting

after setting font family to JetBrains Mono it looks fine:

I had a similar problem when lapce could not find the font name I set. Here is how I fixed it #1540, maybe it will help you.

it does find the font- if you enter gibberish it instead correctly displays the system fallback. it's inter specifically that ends up looking like that micro jumbled mess.

Same issue here. Version 1.102.
OS: Pop!_OS 22.04 LTS x86_64
DE: GNOME 42.5
WM: Mutter

Should be fixed now since we overhauled whole UI


Fix regex as sole body of a do-while loop

Fixes #320.

Thanks! Merged

Oh, that was extremely fast! I just opened Sublime :D Thanks @TimothyGu!

 Thanks!


Upgrade c-blosc to 1.16

https://github.com/Blosc/c-blosc/blob/master/RELEASE_NOTES.rst#changes-from-1151-to-1160

Went straight to 1.17.0 in fact. 🙂


It's there any idea to support socket based client fuzzing?

Why winafl do not use NtCreateThread or CreateProcess method to recover program state, just like Linux AFL's forkserver? I want to have a try, but I don't understand the problem.

if(!CreateProcess(NULL, cmd, NULL, NULL, inherit_handles, CREATE_SUSPENDED, NULL, NULL, &si, &pi)) {
FATAL("CreateProcess failed, GLE=%d.\n", GetLastError());
}

 if(!CreateProcess(NULL, cmd, NULL, NULL, inherit_handles, CREATE_SUSPENDED, NULL, NULL, &si, &pi)) { FATAL("CreateProcess failed, GLE=%d.\n", GetLastError()); }
Thank you!


Country conflict USA & CA

when we enter 'ca' as default country  and try to enter canadian number it convert it automatically in usa number

This package (https://github.com/viclafouch/mui-tel-input) handles this problem


Evaluator Review: FACINGS NFT Collection Manager Stage 2 - Milestone 1 delivery

Review, approve, or deny https://github.com/eosnetworkfoundation/grant-milestones/pull/85

Closed as unplanned https://github.com/eosnetworkfoundation/grant-framework/issues/199 due to issue in framework repo. OPENED AND CLOSED https://github.com/eosnetworkfoundation/grant-milestones/issues/105 as it's in milestone repo.


WidgetLibraryService can not be imported correctly

Hi!
I am trying to port form angular2-json-schema-form to angular6-json-schema-form but I am having some issues with it.
I am using WidgetLibraryService like this:
... importing the service at the top of the file:
import { WidgetLibraryService } from 'angular6-json-schema-form/lib/widget-library/widget-library.service';
(also tried with:)
import { WidgetLibraryService } from 'angular6-json-schema-form';
... injecting the provider in constructor:
private widgetLibrary: WidgetLibraryService
... registering widgets:
widgetLibrary.registerWidget('widgetone', WidgetOneComponent);
widgetLibrary.registerWidget('text', InputComponent);
widgetLibrary.registerWidget('textarea', TextareaComponent);
But when I try to serve the app (non-prod), I get the following error:
ERROR in src/app/layout/content/content-create/content-create.component.ts(43,10): error TS2305: Module '"xxx/xxx/xxx/xxx/myproject/node_modules/angular6-json-schema-form/angular6-json-schema-form"' has no exported member 'WidgetLibraryService'.
What could be the problem here?
which template:

[ ] MaterialDesignFrameworkModule — Material Design
[ ] Bootstrap3FrameworkModule — Bootstrap 3
[x] Bootstrap4FrameworkModule — Bootstrap 4
[ ] NoFrameworkModule — plain HTML
[ ] Other (please specify below)

A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Use the code snippet from above.
Run "npm run start" for serving the app (ng serve)

Expected behavior
To have no errors at serving the app.
Desktop (please complete the following information):

OS: [Windows 10]
Browser [chrome]
Version [latest]

I use this service regularly to create different widgets. If you know a potential fix for it I would be willing to submit a merge request. I just need this ASAP. Thanks!

Closing as it was already fixed in https://github.com/hamzahamidi/Angular6-json-schema-form/pull/10/files#diff-55


[REVIEW] Create a notebook to demonstrate vertex renumbering

New notebook to demonstrate the cugraph renumbering feature.

@ChuckHastings , is this notebook ready to be reviewed?

Yes, this should be ready for review.

@BradReesWork  - I would argue for option 2.  At least for cugraph I would expect that we will ultimately have many notebooks that reference networkx functions.  The goal in the cugraph python integration is to mimic networkx - as cudf mimic's pandas.


"Only owner and team can mount" not defining individually

Earlier today I set Minicopter's "Only Owner and team can mount" to "true" whilst simulatenously having the Attack heli, and scrap heli set to "false" and the "false" flag was overriding the minicopter's true.
Not sure if it's something you'll be able to replicate, or if it was due to some sort of interference from MinicopterOptions somehow (I doubt it, though.)
Also, is there any way you can make it so that there's a config option for "Only owner and team can CONTROL/loot fuel containers" instead of just mount?  I'd be more than happy to throw you some coffee or beer money for this customized edit if you can.  I'd like to be able to give people rides without allowing them control over my fuel-less mini.

Thank you for reporting this issue. Sorry for missing it, I wasn't subscribed to issues on this repository since it isn't mine, but I have subscribed now. Since this is a uMod plugin, the recommended way to get support going forward is via the uMod forums.
I confirmed in the code that the CanMountEntity method is missing a check for the specific type of helicopter, so the symptom you described makes sense. This happened because originally the feature was enabled/disabled by subscribing/unsubscribing a hook, and so I forgot to add a check in the hook function itself when transitioning to multiple types of helicopters. A similar miss was originally done for the Destroy on disconnect option, but that was identified and fixed during the v3.0.0 beta.
I've made a fix on a branch, which you can get at the following link.
https://github.com/WheteThunger/SpawnHeli
As for your suggestion regarding giving players rides, that might be a good fit for the Vehicle Deployed Locks plugin, where players that authenticate with the code lock's guest code could potentially only mount the vehicle (possibly only passenger seats), with no access to other vehicle features. Either way, please open a feature request on the respective uMod plugin forum.


Fix CI Flakes with paperprovision flag

Looks like it works: https://ci.keyba.se/job/kbfs-server/job/PR-249/

had a few questions


generalize DarkSwitch to ThemeSwitch

Currently, DarkSwitch is used with Tailwind's class dark mode.  All it really does is apply a class to <body>, which would play well with CSS variables:
Each theme would have a css class name to apply to <body> and what is essentially a RenderFragment:  Currently SVGs and light / dark / system.  Generalizing this to Themes rather than just dark/light, the C# and JS would be largely the same, but the UI would differ - perhaps a dropdown if in a menu, or radio buttons if in a settings page.  The RenderFragments would be text instead of the current light/system/dark SVGs.
tl;dr: arbitrary UI; click on a RenderFragment and its associated class name is applied to <body>

On second thought, while it's a good idea in the context of useful Components, the amount of complexity it would add to this project is too much to demonstrate a simple aspect of Tailwind: ability to use var(--some-variable).  Keep it simple.


full word search should ignore punctuation

searching for "mY min cwau Gxw swic ivgwsI rwm ]"  finds nothing whereas "mY min cwau Gxw swic ivgwsI rwm" works

fixed in commit cde09db4f2f2198e86522b600d5ea6a8525c4dcc


Added colours and Transition Effects to the Language Box || ISSUE #1033

I have added the colours and Transition effects to the language box to look more dynamic to the users. Also, I have ensured that the scrolling goes smoothly and rounded the box's shape from the corners.
Also, I have made changes in the HTML file and added lines of CSS and javascript lines in the code so that the webpage fully loads and then set up the widget with specific settings (like the default language).
https://github.com/user-attachments/assets/d326621a-65d9-45e4-b4b1-3d8b10c4e80e
@sanjay-kv @HimangshuSharma01 Sir, Please merge my PR under gssoc-ext and hacktoberfest labels
This PR closes the issue #1033

can u fix the conflicts.


AppLovin Adapter | 1.1.0 Release

Description
Now compatible with AppLovin Android 12.4.3 and AppLovin iOS 12.4.2.
Improved code documentation
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Hi @LTPhantom. I'm facing this issue when build for iOS.
[!] The 'Pods-Runner' target has transitive dependencies that include statically linked binaries: (/Volumes/Data/Working/SourceCode/LearnEnglish/FlutterApp/learn_english/ios/Pods/GoogleMobileAdsMediationAppLovin/AppLovinAdapter-<IP_ADDRESS>/AppLovinAdapter.xcframework)

Please help add this line s.static_framework = true #<-- add this into gma_mediation_applovin.podspec when you release new version.
This is issue I created https://github.com/googleads/googleads-mobile-flutter/issues/1066
Thank you!


Add links to CONTRIBUTING.MD

Fixes
Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

Per issue #584, this PR adds links to CONTRIBUTING.MD for easier navigation.

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

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

Hello @jstevans,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Quill Deprecation Error

How to solve this problem?
[Deprecation] Listener added for a 'DOMNodeInserted' mutation event. This event type is deprecated, and will be removed from this browser VERY soon. Usage of this event listener will cause performance issues today, and represents a large risk of imminent site breakage. Consider using MutationObserver instead. See https://chromestatus.com/feature/5083947249172480 for more information.

Yet to be updated this issue.
Today after the Chrome update on July 30th we are getting this new message:
[Deprecation] Listener added for a 'DOMNodeInserted' mutation event. Support for this event type has been removed, and this event will no longer be fired. See https://chromestatus.com/feature/5083947249172480 for more information.
The editor seems to work but every time I loaded in my app, throws an error that we have to manage.
Any further action to patch this issue will be appreciated.

Updating to Quill v2 resolved the issue for us.

We are using react-quill 2.0.0, and we are encountering the same message. I have seen in some issues that version 2 is supposed to resolve this problem; however, we are using version 2 and still seeing this message.
[Deprecation] Listener added for a 'DOMNodeInserted' mutation event. Support for this event type has been removed, and this event will no longer be fired. See https://chromestatus.com/feature/5083947249172480 for more information.
What’s going on?

Hi @Anna-AP - when I look at the package.json of React Quill v2, it looks like they're still using v1 of Quill.
https://github.com/zenoamaro/react-quill/blob/91aa93f4f909c2b8023657d8ee3669a0990be891/package.json#L48

jxjj Thanks, I see the problem. Is there a solution for this?

You may have better luck checking directly with the react-quill project. It looks like someone's made a fork which might be useful to you: https://github.com/zenoamaro/react-quill/pull/973
We're using quill with vue apps, so I can't speak to the react updates. But, we opted to just use quill directly to create a vue component rather than use a separate vue wrapper project. Then we forked a abandoned quill modules projects to update them as well.


Document zoomed if still scrolling and meta keys pressed.

STR:
On OSX scroll quickly using the touchpad, then press command.
Expected:
Document is not zoomed
What Happens:
Document is zoomed
We should only zoom if them command key was pressed before the scroll.

Fixed by #7097.


Implement index parameter on api/create issue

Fixes #7790.

Note: I should write an integration test for this. Will do tomorrow. Changed to WIP.

Codecov Report

Merging #7876 into master will increase coverage by 0.01%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master    #7876      +/-   ##
==========================================
+ Coverage   41.37%   41.39%   +0.01%
==========================================
  Files         475      475
  Lines       63826    63827       +1
==========================================
+ Hits        26411    26424      +13
+ Misses      33973    33962      -11
+ Partials     3442     3441       -1

Impacted Files
Coverage Δ

routers/api/v1/repo/issue.go
23.38% <100%> (+0.16%)
:arrow_up:

models/issue.go
50.34% <50%> (+0.38%)
:arrow_up:

models/error.go
33.04% <0%> (+0.65%)
:arrow_up:

routers/repo/view.go
43.25% <0%> (+1.01%)
:arrow_up:

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 a077c9d...66504c5. Read the comment docs.

I feel it's dangerous to let user give an index but I haven't read the codes carefully.

I feel it's dangerous to let user give an index but I haven't read the codes carefully.

@lunny For what I've seen, there should be no side-effects. Just as @argv-minus-one, the index column is calculated from a query (select max(index)+1) right before inserting the new record in the database. It's not an identity column, as I was confused before. The rest of the code couldn't care less what indexes the issues have. In fact, there is a potential problem with the current implementation: there's a slight chance of a racing condition where two people attempt to add issues the traditional way:

User A submits the form creating an issue for repo abc.
User B submits the form creating another issue for repo abc.
At issue.go, in the context of user A's request, getMaxIndexOfIssue() returns 99. It will be used at step 5.
At issue.go, in the context of user B's request, getMaxIndexOfIssue() also returns 99. because at the moment that's the latest issue for the repo. It will be used at step 6.
At issue.go, in the context of user A's request, newIssue() inserts A's issue with Index: 100.
At issue.go, in the context of user B's request, newIssue() attempts to insert B's issue with Index: 100 and fails, because 100 was used at step 5.

This will of course depend on the database locking model.
And this is for current code. Inserting issues by explicit index will not improve or worsen this situation.
Using the API to insert issues is the user's responsibility. The field is uint64, so there will be no negative values, and a 0 means default. If the user sends a duplicate index, he will get a 500 error response with:
{
    "message": "newIssue: pq: duplicate key value violates unique constraint \"UQE_issue_repo_index\"",
    "url":"http://srep.sif.com.ar:3080/api/swagger"
}

Which they should be expecting, anyway (the actual message could be different for each DB type and version). No intermediate states are kept, so the whole operation fails successfully. 😂
But!
The only "danger" I can think of is if the user creates an issue with index: 2^64-1 (i.e. max value), so Gitea will be unable to create any more indexes. Could it be considered a security problem (potential DoS)? Perhaps we should have the feature disabled by default and the user would set it to enabled in app.ini at their discretion.

@guillep2k  step 6 will failed because we have a unique index with repo_id and index on database design. A slight enhancement should be retry it 3 times to retrieve the max id.
Yes, an index number with 2^64-1 will broken the gitea. It's a security problem if we allow customize index number.

@guillep2k step 6 will failed because we have a unique index with repo_id and index on database design. A slight enhancement should be retry it 3 times to retrieve the max id.
Yes, an index number with 2^64-1 will broken the gitea. It's a security problem if we allow customize index number.

@lunny The "step 6" problem is independent of this PR. I'll create an issue for it (chances are slim to reproduce this problem in a real-life scenario; even to test it).
The "index == 2^64 - 1" problem is contained by allowing only admins to create issues like this; it would be their responsiblity to avoid that. With my last commit, it won't break anything; it will just prevent creating anymore issues with error "Index numbers depleted".

Created separate issue #7887 to handle the racing condition.

Thanks for doing this!
Some problems I found/thought of:

The issue number field is named id in the newly-defined JSON API. This is inconsistent; other parts of the JSON API for issues (such as GET /api/v1/repos/<user>/<repo>/issues) put it in the number field instead, and have a separate id field that corresponds to the database id column (that is, the globally-unique issue identifier, not the per-repo issue number).

The return type of getMaxIndexOfIssue is int64, not uint64. When it overflows, the result is -9223372036854775808, not 0. (Tested on Gitea 1.9.0.)

Though Gitea and the database can deal with 64-bit issue numbers, JavaScript can't. Integers greater than 2^53-1 (that is,<PHONE_NUMBER>740991) are not representable as a JavaScript Number. They are representable as a JavaScript BigInt, but most browsers don't support that, and even in those that do, the JSON parser doesn't generate BigInts.

The API probably shouldn't allow a negative issue number.

Suggestions:

Rename the newly defined JSON id field to number, for consistency with the rest of the API.

When creating an issue, check that the issue number (whether given explicitly or automatically chosen) is >= 0 and <= 0x20000000000000.

Another thing: Should non-admin users be allowed to set explicit issue numbers on repositories they own? If I'm reading the code right, only admin users are allowed to do this right now. This limits the usefulness of the feature, since non-admin users can't use it to import their issues without an admin's assistance.
In Gitea 1.9.0, setting an issue number to 2^63-1 in the database blocks creation of new issues on the affected repository, but does not seem to affect the other repository I created in my test instance. Repository owners creating issue #2^63-1 would be blocking only themselves.
Of course, there's the possibility of a repository owner's credentials being stolen, and an attacker creating issue #2^63-1 in order to make issue tracking permanently unusable for that repository. Since deleting issues is not allowed, only someone with direct access to the database can fix that.
This could be solved by allowing repository owners to delete issues on their repositories. Is there a reason why deleting issues isn't currently allowed?

* The issue number field is named `id` in the newly-defined JSON API. This is inconsistent; other parts of the JSON API for issues (such as `GET /api/v1/repos/<user>/<repo>/issues`) put it in the `number` field instead, and have a separate `id` field that corresponds to the database `id` column (that is, the globally-unique issue identifier, not the per-repo issue number).

I'll fix that.

* Though Gitea and the database can deal with 64-bit issue numbers, JavaScript can't. Integers greater than 2^53-1 (that is,<PHONE_NUMBER>740991) are not representable as a JavaScript `Number`. >
* The API probably shouldn't allow a negative issue number.

Good catch. I'll fix that too.

Another thing: Should non-admin users be allowed to set explicit issue numbers on repositories they own? If I'm reading the code right, only admin users are allowed to do this right now. This limits the usefulness of the feature, since non-admin users can't use it to import their issues without an admin's assistance.

It's a repository admin (e.g. owner, or member of a team with admin rights for that repo), not necessarily the site admin.
As I understand it, this modification is only useful for a very small group of Gitea users so, the simpler, the better the chance to see the light.

@guillep2k Maybe even repositories' admin or owner should not use this API. Only site admin.
@argv-minus-one This is a http/https request, unrelated with Javascript. You can use any language which support int64 to invoke that.

This is a http/https request, unrelated with Javascript. You can use any language which support int64 to invoke that.

@lunny perhaps some front-end will fail if Gitea provides a number larger than supported? I'm not familiar with Gitea's JS code. For instance, the feature that triggers when the user types in #9999 to reference another issue....
In my latest commit I allow index == 2^63-1 (max int64), but in the one before it I've set it to 2^53-1 if you want to take a look. We could implement it either way.

In fact router/api/v1/repo/pull.go is calculating issue.Index by it's own, but that value should not be used. It originally ignored; it was dead code that I attempted to "honor" because the tests were failing. Instead of "accepting" the index number when it comes from a PR, I'll remove the PR's dead code, as I should have.

@lunny I made one change after your approval. Please review again.

Having become a bit of an expert on Issue.Index in the past few days, I now think that this PR should not be merged. My reasoning is as follows:

If it's for some kind of one-time migration, it should be implemented separately in the migration package, as suggested at some point.
If it's for continuous usage, then it will create a problem. Say the admin adds 232 issues from an external source: 1001 through 1232. Then someone uses Gitea's UI to file a new issue, which is assigned with index 1233. The next time the admin tries to migrate issues from the external source, 1233 will have been taken, and it could be clashing with the index number they're trying to create.

So, I think this is a no-win in all cases.

Let's close then.


Get Node.js and npm paths from VS package

Attempt to find bundled node.exe and npm.cmd from Visual Studio
@paulvanbrenk Please note I didn't test this.

thanks

@paulvanbrenk argh. The second argument to RegistryRoot should have been false (for writable)

Had to roll this back since it doesn't work for the npm tooling (no reference to VS Shell). Looks like more work, I'll do this next week.


k3s-selinux testing rpm does not have a compatible version with Rhel 9+

Environmental Info:
K3s Version:

N/A
Node(s) CPU architecture, OS, and Version:

$ uname -a && cat /etc/os-release
Linux ip-172-31-4-177.us-east-2.compute.internal 5.14.0-162.6.1.el9_1.x86_64 #1 SMP PREEMPT_DYNAMIC Fri Sep 30 07:36:03 EDT 2022 x86_64 x86_64 x86_64 GNU/Linux
NAME="Red Hat Enterprise Linux"
VERSION="9.1 (Plow)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="9.1"
PLATFORM_ID="platform:el9"
PRETTY_NAME="Red Hat Enterprise Linux 9.1 (Plow)"
ANSI_COLOR="0;31"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:redhat:enterprise_linux:9::baseos"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://access.redhat.com/documentation/red_hat_enterprise_linux/9/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"

REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 9"
REDHAT_BUGZILLA_PRODUCT_VERSION=9.1
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="9.1"

Cluster Configuration:

N/A
Describe the bug:

There is no RPM built yet in the new testing RPMs for EL9. Using latest: https://github.com/k3s-io/k3s-selinux/releases/tag/v1.3.testing.4. Therefore, the rpm fails to install on el9 machines due to incompatible container-selinux version.
Steps To Reproduce:

Install container-selinux rpm: sudo yum install -y container-selinux
Attempt to install k3s-selinux rpm: sudo rpm -i https://github.com/k3s-io/k3s-selinux/releases/download/v1.3.testing.4/k3s-selinux-1.3-4.el8.noarch.rpm

Expected behavior:

This should install the RPM successfully. OR there should be an el9 rpm that is available for el9 machines.
Actual behavior:

This fails to install with the following error:
warning: /var/tmp/rpm-tmp.fYXWX2: Header V3 RSA/SHA256 Signature, key ID d161f542: NOKEY
error: Failed dependencies:
	container-selinux < 2:2.191.0-1 is needed by k3s-selinux-1.3-4.el8.noarch

Additional context / logs:

$ rpm -qa | grep -i container-selinux
container-selinux-2.189.0-1.el9.noarch

$ sudo yum --showduplicates list container-selinux | expand
...
Installed Packages
container-selinux.noarch      3:2.189.0-1.el9        @rhel-9-appstream-rhui-rpms

It appears the epoch has increased from 2 to 3 and fails this check: Requires(post): container-selinux >= %{container_policy_epoch}:%{container_policyver}, container-selinux < %{container_policy_epoch}:%{container_policyver_max}

@rancher-max @ShylajaDevadiga I think we should test this issue while the PR is open in master since its directly affecting all installations once its merged.

@galal-hussein I am seeing the below with v1.3.testing.7/k3s-selinux-1.3-7.el9.noarch.rpm
$ sudo rpm -i https://github.com/k3s-io/k3s-selinux/releases/download/v1.3.testing.7/k3s-selinux-1.3-7.el9.noarch.rpm
warning: /var/tmp/rpm-tmp.GFYilo: Header V3 RSA/SHA256 Signature, key ID d161f542: NOKEY
Conflicting name type transition rules
Binary policy creation failed at /var/lib/selinux/targeted/tmp/modules/200/k3s/cil:135
Failed to generate binary
/usr/sbin/semodule:  Failed!
uavc:  op=load_policy lsm=selinrpm -qa | grep -i container-selinux-45-232 ~]

$ rpm -qa | grep -i container-selinux
container-selinux-2.205.0-1.el9_2.noarch

$ sudo yum --showduplicates list container-selinux | expand
...
Installed Packages
container-selinux.noarch      3:2.205.0-1.el9_2      @rhel-9-appstream-rhui-rpms

CC @rancher-max

Verified using the commands below on Rhel 9.1

Config.yaml

write-kubeconfig-mode: 644
cluster-init: true
node-name: server

$ sudo yum install -y container-selinux

$ curl -sfL https://raw.githubusercontent.com/galal-hussein/k3s/add_el9_selinux_rpm/install.sh | INSTALL_K3S_CHANNEL=testing sh -

$ kubectl get nodes
NAME     STATUS   ROLES                       AGE   VERSION
server   Ready    control-plane,etcd,master   5m   v1.27.1-rc3+k3s1

$ kubectl get pods -A
NAMESPACE     NAME                                     READY   STATUS      RESTARTS   AGE
kube-system   coredns-77ccd57875-8bbb2                 1/1     Running     0          8m
kube-system   helm-install-traefik-crd-npsqg           0/1     Completed   0          8m
kube-system   helm-install-traefik-w8zj4               0/1     Completed   1          8m
kube-system   local-path-provisioner-957fdf8bc-z68hn   1/1     Running     0          8m
kube-system   metrics-server-54dc485875-std7t          1/1     Running     0          8m
kube-system   svclb-traefik-7c672fc4-mrcqc             2/2     Running     0          8m
kube-system   traefik-84745cf649-ndh2p                 1/1     Running     0          8m

Validated in testing channel with latest install script from https://github.com/k3s-io/k3s/pull/7443
Multiple OSes were covered as part of this, including: RHEL 9.1, RHEL 8.7, Centos 7.8, Centos Stream 9, SLE Micro 5.4, SLES 15 SP4, Rocky Linux 8.6, Rocky Linux 9.1, and Fedora CoreOS 38. The solution involved creating RPMs for k3s-selinux where we didn't have them before (like for sle micro and coreos) and ensuring that some OSes us the latest container-selinux (>=191) and others pin a container-selinux that it was previously using (<191). Currently, these fixes are available only in the testing channel, but will be pushed out to latest and stable at the same time as the May patch releases (very soon).
Some of the el8 distros have updated container-selinux, but not all, so across the board that container-selinux version is being required to be <191. Same with "*sle*" distros. EL9 and coreos will use >=191.
The scenarios covered were:

Fresh install using new install script and pointing to testing channel
Upgrade by:
a. Install with older install script and stable channel
b. Upgrade using new install script and testing channel
For all cases, validate sudo ls -laZ /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/ results in either container_file_t for container-selinux >= 191, and container_ro_file_t for container-selinux < 191.

In all scenarios, the following config file was used:
# /etc/rancher/k3s/config.yaml
write-kubeconfig-mode: 644
selinux: true

Please be advised: for Fedora CoreOS and SLE Micro, a reboot is required after install. This is a limitation of the OSes themselves. The install script gives a warning indicating this is required, but it's easy to miss when using existing scripting to install k3s.
I'd also like to add that as and when the OSes currently using container-selinux < 191 update to the latest version, they might break. We will be keeping a pulse on the supported operating systems versions and update accordingly when it's appropriate.


DesktopConfig should be exported

The DesktopConfig structure is publicly usable via the dioxus_desktop::launch_cfg function, but is not exported, so no documentation is generated for it.

Yes, this doesn't seem to be documented anywhere.
https://docs.rs/dioxus-desktop/latest/dioxus_desktop/ says

To configure the webview, menubar, and other important desktop-specific features, checkout out some of the launch configuration in the API reference.

But there is no such "launch configuration" documentation anywhere.


[IN] Ricardo's Test

You do not need to put in anything here :)
Just open the issue by clicing the green "Submit new issue" and within 10-15s a message instructing you do upload your files should appear!

28output_poscar.zip


Error when importing package: ModuleNotFoundError: No module named 'packaging'

When I perform a fresh install of the package and then attempt to import it, I get this error
Traceback (most recent call last):
  File "/Users/nathandunn/Projects/replicate-test/test.py", line 1, in <module>
    import replicate
  File "/Users/nathandunn/Projects/replicate-test/venv/lib/python3.10/site-packages/replicate/__init__.py", line 2, in <module>
    from .client import Client
  File "/Users/nathandunn/Projects/replicate-test/venv/lib/python3.10/site-packages/replicate/client.py", line 9, in <module>
    from replicate.model import ModelCollection
  File "/Users/nathandunn/Projects/replicate-test/venv/lib/python3.10/site-packages/replicate/model.py", line 6, in <module>
    from replicate.version import VersionCollection
  File "/Users/nathandunn/Projects/replicate-test/venv/lib/python3.10/site-packages/replicate/version.py", line 7, in <module>
    from replicate.schema import make_schema_backwards_compatible
  File "/Users/nathandunn/Projects/replicate-test/venv/lib/python3.10/site-packages/replicate/schema.py", line 1, in <module>
    from packaging import version
ModuleNotFoundError: No module named 'packaging'

I am running macOS 11.6.6, Python 3.10.4
Steps to reproduce
$ mkdir new-folder
$ cd new-folder
$ python -m venv venv
$ source venv/bin/activate
$ pip install replicate
$ echo 'import replicate' > test.py
$ python test.py

Running pip install packaging fixed this for me, so I think your package has an undeclared dependency

Ah, this is actually fixed in main, but we didn't ship a new release -- sorry! Doing that now. You should be able to pip install -U replicate in 5 mins. :)

Thanks for the report!


Fix crash when calling reloadPagerTabStripView multiple times

Calls updateContent() before the new cachedCellWidths are calculated and reload if uniqueIndexPaths.count > 1

Thans #388 and #544

Hi guys... Is there any ETA for the merge of this PR ?

Hi , is this fix available in new release, I have updated to 9.0 but still getting this crash randomly of index out of bound.
please look into this .
Thanks

Thanks #388 and #544

i am still getting that crash, please help


Update Contributors.md

Signed-off-by: Mayank-Sriva<EMAIL_ADDRESS>Describe your changes in brief
Changes

[ ] Why is this change required? What problem does it solve?
[ ] If proposing a new change then please raise an issue first

How Has This Been Tested? (Optional)

[ ] Please describe in detail how you tested your changes.
[ ] Include details of your testing environment, and the tests you ran to
[ ] How your change affects other areas of the code, etc.

Screenshots (if appropriate)
Checklist

[ ] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

View rendered CONTRIBUTORS.md

@Mayank-Sriva, can you update your branch


icc 13.0.1 gives gateway timeout

e.g. https://godbolt.org/z/5UIn90  (gateway timeout)
On the backend:
May 19 14:49:50 ip-172-30-0-189 amazon:  warn: Timeout for /usr/local/bin/firejail --quiet,--profile=etc/firejail/execute.profile,--private=/tmp/compiler-explorer-compiler119419-1177-pm9oi0.uqem,/opt/compiler-explorer/intel/bin/icc,-g,-o,./output.s,-masm=intel,-S,-gxx-name=/opt/compiler-explorer/gcc-4.7.1/bin/g++,./example.cpp after 20000ms

Confirmed locally it's firejail causing issues. I get "error 5" locally though.

Seems it's private-tmp that breaks it. Intriguing: I wonder what's going on here!

Annoyingly the process actually seems to work. I suspect some kind of intel daemon is fork()ed - there's a intel/ism/ hierarchy of files created in the home directory. I wonder if some daemon process expects to be communicated with via /tmp.

running with strace outside of the firejail:
21412 stat("/opt/intel/ism/bin/intel64/intelremotemond",  <unfinished ...>
21401 unlink("/tmp/iccgasGRvXtr" <unfinished ...>
21412 <... stat resumed> 0x7ffddaf2b320) = -1 ENOENT (No such file or directory)
21401 <... unlink resumed> )            = 0
21412 stat("/home/matthew/intel/ism/bin/intel64/intelremotemond",  <unfinished ...>
21401 unlink("/tmp/iccldashv4vGqza" <unfinished ...>
21412 <... stat resumed> 0x7ffddaf2b320) = -1 ENOENT (No such file or directory)
21401 <... unlink resumed> )            = 0
21412 semop(294914, [{0, -1, SEM_UNDO|IPC_NOWAIT}], 1 <unfinished ...>
21401 unlink("/tmp/icctempfilejczF71" <unfinished ...>
21412 <... semop resumed> )             = 0
21412 semop(294914, [{1, -1, SEM_UNDO|IPC_NOWAIT}], 1 <unfinished ...>
21401 <... unlink resumed> )            = 0
21412 <... semop resumed> )             = 0
21401 unlink("/tmp/iccargyXnVFT" <unfinished ...>
21412 semop(294914, [{2, -1, SEM_UNDO|IPC_NOWAIT}], 1) = 0
21401 <... unlink resumed> )            = 0
21412 semop(294914, [{3, -1, SEM_UNDO|IPC_NOWAIT}], 1) = 0
21412 semop(294914, [{4, -1, SEM_UNDO|IPC_NOWAIT}], 1) = 0
21412 semctl(294914, 0, IPC_RMID, NULL) = 0
21412 exit_group(5)                     = ?
21401 exit_group(0)                     = ?
21402 <... futex resumed>)              = ?
21412 +++ exited with 5 +++
21402 +++ exited with 0 +++
21411 <... wait4 resumed> [{WIFEXITED(s) && WEXITSTATUS(s) == 5}], 0, NULL) = 21412
21411 --- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=21412, si_uid=1000, si_status=5, si_utime=0, si_stime=0} ---
21411 exit_group(5)                     = ?
21401 +++ exited with 0 +++
21411 +++ exited with 5 +++

It seems the main process exits with 0, but one of the child processes (both the "intelremotemond" and its parent) exit with 5

If I run my own intelremotemond (from another install) It Just Works™. Ugh. Trying to work out how to disable it.

With thanks to twitter, some maybe helpful env vars:
ltrace -e getenv /opt/compiler-explorer/intel/bin/icc -g -o ./output.s -masm=intel -S -gxx-name=/opt/compiler-explorer/gcc-4.7.1/bin/g++ ./example.cpp 2>&1 | sort -u | cut -d\" -f 2 | sort -u | egrep '^[A-Z]+'
FLEXLM_BATCH
FLEXLM_BORROWFILE
FLEXLM_DEBUG_BADDATE
FLEXLM_DIAGNOSTICS
FLEXLM_NO_CKOUT_INSTALL_LIC
FLEXLM_RC
FLEXLM_USE_FINDER
GCCROOT
HOME
ICCCFG
IC_USE_PATH
INTEL_LICENSE_FILE
INTEL_LMD_DEBUG
LANG
L_BORROW_ERR
LC_ALL
LC_CTYPE
LM_APP_DISABLE_CACHE_READ
LM_BORROW
LM_LICENSE_FILE
NLSPATH
TEMP
TMP
TMPDIR
Vxq2Rfw2442332332332XWF25345

wonder if it may be worth it to have a small wrapper process inside firejail that catches all forks and kills them when the main child exits?

Enabling shell has seemingly "fixed" it locally. This seems...unlikely. But I'm going with it for now :/

Ah no -- it's actually random! ARG. Seems whichever process exits last out of the jail is what firejail returns as the exit code. That means if the fork()ed, failing intelremotemond happens to exit last, its failing return code is the one returned. The shell thing seems to just make it more likely that the compiler exits last, but it isn't infallible.

https://github.com/netblue30/firejail/blob/master/src/firejail/sandbox.c#L325
there is even a special case in firejail for an application that forks a child that must be killed with a todo note about making this generic.
having firejail not deterministically return the original child processes exit status is also a result of this mechanism.
Firejail handles child terminations until it sees the "monitored_pid" terminate, which initially is the forked child firejail creates.
It then scans /proc for any remaining pids and chooses one to be the new "monitored_pid".
Seems like having two options added to firejail could fix these kinds of issues across the board. (although mitigating them on a case by case basis may still be worth it over time, to cut down on wasted cpu cycles).

always exit firejail with the originally forked childs exit code
sigterm all remaining processes once the originally forked child exits

See if this firejail patch helps at all.
If it proves useful I will try to come up with a less obnoxious flag than --deterministic-exit-code, look into adding these options to the profile file parser so they don't clutter up the command line, and submit a PR upstream.
https://github.com/apmorton/firejail/commit/7de5c1ef558c2d3eacd012729bbe622ca5362519
Patch tested as follows:
file test.sh creates some double forked children that delay and each return a unique exit code
#!/bin/bash

echo "starting orphans"
{ nohup bash -c "sleep 2; exit 1" &> /dev/null & } &
{ nohup bash -c "sleep 2; exit 2" &> /dev/null & } &
{ nohup bash -c "sleep 2; exit 3" &> /dev/null & } &

echo "sleeping 0.5 seconds"
sleep 0.5

echo "main application exiting"
exit 4

existing behavior
firejail --noprofile --quiet ./test.sh; echo status $?

will block until all orphaned children naturally exit
will exit with code 1, 2, or 3 randomly

although it heavily favors exit code 3 in this example, since it is the last sleep to be started

with deterministic exit code
firejail --noprofile --deterministic-exit-code --quiet ./test.sh; echo status $?

will block until all orphaned children naturally exit
will exit with code 4

exit codes of other children are swallowed

with terminate orphans
firejail --noprofile --terminate-orphans --quiet ./test.sh; echo status $?

sigterm all remaining children and wait for them to naturally exit

does not send sigkill, so misbehaving children will cause a hang

will exit with code 255 (from firejail itself)

with both
firejail --noprofile --terminate-orphans --deterministic-exit-code --quiet ./test.sh; echo status $?

sigterm all remaining children and wait for them to naturally exit

does not send sigkill, so misbehaving children will cause a hang

will exit with code 4

Wow, thanks @apmorton - will give this a go today. (Have other possible workarounds if needed too, but this looks great!)

Fixed, thanks to @apmorton :)


windows terminal freezing while changing the background image

Windows Terminal version (or Windows build number)
Terminal version: 1.8.1521.0
Other Software
No response
Steps to reproduce

Terminal setting
choose profile in the profile section
Go to appearance
Choose background image than click save

Expected Behavior
No response
Actual Behavior
Terminal freeze for a moment after click the save button

Did it freeze for a moment then everything just works? Or did it freeze for a moment then crashed?

@skyline75489  sometime it freeze for a moment then works and sometime it freeze for a moment then crashed

What CPU & GPU are you using?

@skyline75489  CPU - Intel i5 8400
GPU - gtx 1660 (driver version 466.47)

Wonder if this is a driver issue. We've seen driver issue causing crashes & instability before.
What is the Windows version you are using?

@skyline75489  version 21h1( build  19043.1023)

Can you try this option here?

This will force software rendering. Sometimes it helps mitigating GPU driver issue. Details can be found here.

@skyline75489  ok thanks 🙂. I will try it


⚠ TorrentLeech has degraded performance

In d94d79f, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 1779 ms

Resolved: TorrentLeech performance has improved in 0321028.


Bad data in 14,2519,5484

https://s3.amazonaws.com/elevation-tiles-prod/geotiff/14/2519/5484.tif
There's two rows of bad data in this tile, shown in black from QGIS:

Here's the output from gdalinfo:
Band 1 Block=256x256 Type=Float32, ColorInterp=Gray
Minimum=-8566.333, Maximum=2539.531, Mean=2173.470, StdDev=497.209
NoData Value=-3.4028235e+38
Metadata:
STATISTICS_MAXIMUM=2539.5305175781
STATISTICS_MEAN=2173.4702906613
STATISTICS_MINIMUM=-8566.3330078125
STATISTICS_STDDEV=497.20867381496
STATISTICS_VALID_PERCENT=100
This also exists in neighboring tiles. This shouldn't have -8566. It should have nodata or the correct values.

There was a GDAL config error related to NODATA that manifests where two or more sources touch in this old build. To work around it you'd need to do light pre-processing on the tile data to scrub non-sensical data like this.
Ideally there would be another global build to fix this, but it needs resourcing of engineering time and compute.

Bad data also exists in
13 1258 2742, related to the row of data.
In 13, 2358, 2742 the elevation data is -6872m.
In 14, 2519, 5484 there's two rows of data that are invalid with data -8566.33m and  82.75m.
What would you suggest on doing reliable  "light pre-processing"? Currently, I'm using z-score to through out outliers. Do you any a better method?


Slack

[ ] bug report
[ ] feature request
[x] other

Asslamualaikum, Ramadan Mubarak and I am creating an issue to be added to the slack group. My email is<EMAIL_ADDRESS>

wa3laikum alsalam, done.


How do I get a pie chart?

How do I get a pie chart for a query type?
select name, sum (field) as val group by name
where the pie chart shows the groups and their sum.

I don't think this is related to the SQLite plugin. I would either recommend asking on stackoverflow or the pie-chart plugin's site.

Hmmmm, pie charts for MySQL or MSSQL work fine.
For pie charts to work (MySQL) must use the SQL Format as Time series type, but the SQLite plugin does not have this option.
I think it only uses the Format as Table option.

For Time series:

For the Format as Table option - no pie chart

The SQLite plugin does not use the old timeseries or table formats, but the new "DataFrame" format.
Maybe the pie chart plugin does not support it?

I don't know what data format the pie chart uses.
For me, pie charts work fine with data sources: mysql, mssql, elasticsearch.
I do not use other data sources.
I only have a problem with SQLite.
Thanks again for your help and a good job with SQLite.

I checked the issue and I also cannot get the pie chart working correctly with the SQLite plugin (same query works with PostgreSQL)
SELECT 1 as time, 5 as value, 'sth' as label
UNION ALL
SELECT 1 as time, 10 as value, 'else' as label

I don't have anything to go on right now. I am guessing it has to do with SQLite using the new DataFrame format.
It might be worth asking in the pie-chart repo :shrug:

I think the pie chart works with dataframes.
See screen shot.

I think that the grafana-sqlite-datasource plugin should create N (for each name) group dataframes for a grouping sql query:
select name, value from table group name
The same problem with grafana-sqlite-datasource occurs with other charts.
Can you add this functionality?

The same is true for line charts.

I think the pie chart works with dataframes.

At least you got it working with separate queries. That is good :ok_hand:
Of course, it is not as convenient as a group by (maybe you can hack it together with "repeating queries" over a query variable :shrug:

I think that the grafana-sqlite-datasource plugin should create N (for each name) group dataframes for a grouping sql query

I am not so sure about that. There is not much documentation about dataframes to be found online, but in general if you have multiple return columns for your SQLite query (doesn't matter if it is grouped or not) it would make sense to put it just to another column in the dataframe.
I don't know of any other plugin using dataframes yet, where I could take a peek, but returning multiple responses for just one query does not seems the right way here.
I would really question the pie-chart plugin here. Dataframes are not around that long and maybe they just have not been implemented yet :shrug:
Also, the example of a graph is good because the below query works just fine (as I would expect)
WITH sth(time, value, other) as (VALUES
(1,2,3), (2,3,4), (3,4,5)
) select * from sth

No, no, that's not good.
This is just an example of how dataframes graphs work.

A separate plot is drawn for each series.
I do not know the number of series and their names.
This all makes me a SQL query ".... GROUP BY name ..."
Separate graph lines can only create separate data series.
Data should not be placed in one dataframes without additional information about which data is related to each series.
Currently, the SQLite plugin is very good for tables, but not very useful for charts.
I will wait for the next versions - thanks for the work.

I don't know of any other plugin using dataframes yet, where I could take a peek, but returning multiple responses for just one query does not seems the right way here.

Hi,
I opened a separate thread ("Support for time series output format") related to time series in general.
Inside there is a sample working* implementation of multiple time series using dataframes. Please bear with my code (I'm a golang novice) - but this should give you an idea of how to put multiple time series in dataframes.
*working - yeah, I did test it locally with a couple of visualization panels requiring multiple time series data. Also - looking at "Query Inspector" -> "Data" it appears that Grafana is handling the generated data frames properly: indeed it took some experimenting to get there - as you say there is not much documentation available for dataframes

Data should not be placed in one dataframes without additional information about which data is related to each series.

I would love to see some examples or documentation to back this up. Only then it makes sense to take action.
@JohnnyBee2 Maybe you could provide a more reproducible case to take a further look (something like a small test database and query and an exported dashboard).

I think with the workaround from above and the further discussion in this issue, this issue can be closed
https://github.com/fr-ser/grafana-sqlite-datasource/issues/16#issue-752140710


Daprd caching

In what area(s)?

/area runtime

/area operator
/area placement
/area docs
/area test-and-release

Ask your question here
Hello,
I noticed that the Daprd side car makes use of caching to cache secrets. When the container starts, it resolves all the components and already tries to pull secrets from the secret store (in my case azure key vault). It makes sense to avoid hammering the secret store with requests. However, say that you scale your deployment with 5 replicas and encounter a transient error with keyvault for one of the pod instances. In that case, you'll  end up with one unhealthy instance and 4 healthy ones which will give sporadic errors later on according to which instance is handling the request.
On the other hand, when no problem occurs, is the secret cached or kept in the daprd memory during the whole life of the container or is it recycled from time to time (should the secret be disabled in the meantime in keyvault, it keeps working right now).
Thanks
Best Regards

Hello @yaron2,
I tested this with a busybox injected with dapr and I do not observe this behavior anymore. I then thought it might be something specific to the workflows but even with the workflows it does not behave like this. I could swear I observed it as described above. Is there any caching? Looking at the Go source code (I'm not a specialist) I can indeed see that there seem to be some sort of caching but the behavior doesn't seem consistent.
Could you please shed some light on this?
Thanks

Dapr does not have any sort of caching in the sidecar, the cache in question is in the scope of a method that's called per component and only once.
This is in regards to using secrets to establish things like connection strings to different pubsub/state components.
In regards to the Get Secrets API, there is also no caching being used.

Thanks but now I'm a bit confused :). I'm sure that I observed some sort of caching behavior, hence the reason why I created the issue but I couldn't reproduce it the tomorrow (do I turn senile? :))...That said, if there is no caching, it might also be a problem with some secret stores (throttling) so you'd advise to let the application cache secrets itself then?

Thanks but now I'm a bit confused :). I'm sure that I observed some sort of caching behavior, hence the reason why I created the issue but I couldn't reproduce it the tomorrow (do I turn senile? :))...That said, if there is no caching, it might also be a problem with some secret stores (throttling) so you'd advise to let the application cache secrets itself then?

If the app uses the Get Secrets API, it's advised it caches secrets if it needs to, although in general the best practice is to get the secrets, use them to establish the connection to whatever they're needed and then discard them in memory.
In the future we're looking at doing automatic retries for things like getting secrets inside the Dapr sidecar (just like we do today for service invocation), so that if the target secret store is throttling, Dapr will handle that on behalf of the user.

Great, thanks for your answer!


PHOENIX-5960 : Creating view on non-existent table should throw TNFE

4.x backport of PR #987

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  4s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ 4.x Compile Tests _

+1 :green_heart:
mvninstall
11m 52s
4.x passed

+1 :green_heart:
compile
0m 59s
4.x passed

+1 :green_heart:
checkstyle
0m 34s
4.x passed

+1 :green_heart:
javadoc
0m 47s
4.x passed

+0 :ok:
spotbugs
3m  9s
phoenix-core in 4.x has 950 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
6m 27s
the patch passed

+1 :green_heart:
compile
1m  0s
the patch passed

+1 :green_heart:
javac
1m  0s
the patch passed

+1 :green_heart:
checkstyle
0m 33s
phoenix-core: The patch generated 0 new + 156 unchanged - 4 fixed = 156 total (was 160)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 44s
the patch passed

+1 :green_heart:
spotbugs
3m 21s
the patch passed

_ Other Tests _

-1 :x:
unit
184m 58s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

218m 31s

Reason
Tests

Failed junit tests
phoenix.end2end.NotQueryWithGlobalImmutableIndexesIT

phoenix.end2end.AppendOnlySchemaIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/989

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 2a028b6eab51 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x / 7ac4dff

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/2/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/2/testReport/

Max. process+thread count
5934 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/2/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  7s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ 4.x Compile Tests _

+1 :green_heart:
mvninstall
12m  9s
4.x passed

+1 :green_heart:
compile
1m  1s
4.x passed

+1 :green_heart:
checkstyle
0m 35s
4.x passed

+1 :green_heart:
javadoc
0m 47s
4.x passed

+0 :ok:
spotbugs
3m 10s
phoenix-core in 4.x has 950 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
6m 36s
the patch passed

+1 :green_heart:
compile
1m  0s
the patch passed

+1 :green_heart:
javac
1m  0s
the patch passed

-1 :x:
checkstyle
0m 33s
phoenix-core: The patch generated 1 new + 155 unchanged - 5 fixed = 156 total (was 160)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 45s
the patch passed

+1 :green_heart:
spotbugs
3m 19s
the patch passed

_ Other Tests _

-1 :x:
unit
198m 35s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

232m 40s

Reason
Tests

Failed junit tests
phoenix.end2end.PointInTimeQueryIT

phoenix.end2end.SequencePointInTimeIT

phoenix.tx.TransactionIT

phoenix.end2end.ViewMetadataIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/989

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux f14fc4ad02bb 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x / 7ac4dff

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/3/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/3/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/3/testReport/

Max. process+thread count
6174 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/3/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  6s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ 4.x Compile Tests _

+1 :green_heart:
mvninstall
11m 47s
4.x passed

+1 :green_heart:
compile
1m  1s
4.x passed

+1 :green_heart:
checkstyle
0m 33s
4.x passed

+1 :green_heart:
javadoc
0m 47s
4.x passed

+0 :ok:
spotbugs
3m  6s
phoenix-core in 4.x has 950 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
6m 30s
the patch passed

+1 :green_heart:
compile
1m  1s
the patch passed

+1 :green_heart:
javac
1m  1s
the patch passed

-1 :x:
checkstyle
0m 34s
phoenix-core: The patch generated 1 new + 155 unchanged - 5 fixed = 156 total (was 160)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 45s
the patch passed

+1 :green_heart:
spotbugs
3m 20s
the patch passed

_ Other Tests _

-1 :x:
unit
184m 17s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

217m 45s

Reason
Tests

Failed junit tests
phoenix.end2end.DerivedTableIT

phoenix.end2end.DropTableWithViewsIT

phoenix.end2end.ConcurrentMutationsExtendedIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/4/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/989

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 20b5257e5f94 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x / 7ac4dff

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/4/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/4/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/4/testReport/

Max. process+thread count
6204 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-989/4/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Protocol not found

Describe the bug
I get the message in log:
Door: FFmpeg prebuffer process exited with error! (null) - https://<IP_ADDRESS>:554/h264_stream: Protocol not found - Did you mean file:https://<IP_ADDRESS>:554/h264_stream?
To Reproduce
"source": "-i https://<IP_ADDRESS>:554/h264_stream"
Expected behavior
In VLC Player the h264 stream with this url is just working
Logs
Door: FFmpeg prebuffer process exited with error! (null) - https://<IP_ADDRESS>:554/h264_stream: Protocol not found - Did you mean file:https://<IP_ADDRESS>:554/h264_stream?
Environment

Node.js Version: v16.16.0
Homebridge Version: v1.6.1
Camera UI Version: v5.0.27
Operating System: MacOS
Browser: Chrome

Ah I found it. Error in the domain. Should start with rtsp://


thuang-next-js-138313

Description
This PR seems like a huge change, but it can be boiled down to the following main points:

Replace Create React App related dependencies and scripts, and use Next.js to manage all that for us
Create pages/ directory required by Next.js to host all pages, since they use file system based routing for both static and dynamic routes: https://nextjs.org/docs/routing/introduction
Update docker-compose to pass HAPPY_COMMIT to use as consistent build id for all Next.js servers in a cluster
Next.js has its own next/router, so we no longer need react-router
Add pages/_document.tsx and pages/_app.tsx for hosting global Next.js page configuration
Add src/frontend/next.config.js to configure Next.js to process SVG files as React components, add CSP/HSTS, specify prod build directory (default is .next/, but Infra expects build/), add HAPPY_COMMIT as consistent build id, and make envVariable.js available in the browser (see next point)
In Next.js, by default process.env variables are only available in Node.js and not in the browser, and they allow exposing env variables to browser through prefixing a variable with NEXT_PUBLIC_, which I find noisy and affecting across too many stacks and files (Docker files, entrypoint.js, and FE files). So I created envVariable.js as the single source of truth for all env variables used in the FE app and export them as process.env variables to the browser, so we don't need NEXT_PUBLIC_ prefix!
Update package.json scripts to use Next.js' commands
Add react-query for handling API calls, which handles caching, garbage collection, refetch for us automatically
Use import Head from 'next/head' to add page titles

That's it! PTAL thank you 😄
Issue
https://app.clubhouse.io/genepi/story/138313/migrate-fe-to-next-js
Test plan

Visit: https://thuang-next-1-frontend.dev.genepi.czi.technology

The site should be fully functional

Some things I explicitly checked are:

Aspen logo works in Auth0 - inferred through https://thuang-next-1-frontend.dev.genepi.czi.technology/AspenLogoBlack.svg works

All routes work - /samples, /trees, /agreeTerms, /terms, /privacy/, /faq, / (homepage)

Styling looks fine with both scss and emotion.js

Pages now have page title (e.g., Aspen | Samples, Aspen | Phylogenetic Trees

Tests pass

HSTS and CSP work

Remove dependencies no longer needed

Use HAPPY_COMMIT for consistent build id in next.config.js. This is needed because FE servers run in a cluster, while each Next.js build by default has a random id, so if Server A sends the browser some static files fingerprinted with its unique build id, and then the browser makes a request for another static file, and such request gets routed to Server B, Server B won't  recognize Server A's build id and thus not being able to find the static asset the browser needs. Source

Replace <ProtectedRoute /> component with useProtectedRoute hook for AgreeTerms page and Sample page

Move font import to root html page with preconnect, so the browser is able to download the font faster!

Thanks so much for the quick review, @lvreynoso 🙏 !! This is the second time I do Next.js migration now, so all the learnings from Single Cell have been applied here 😆 I don't expect any surprises, but will be ready to resolve them if they happen 💪 Merging!!


OS-Interaction output of os env has many Escape Sequence, Not suitable for human reading

Every output of os env occurs，I am not sure whether it is normal or not, and if it affect model generation.
{
    "history": [
        {
            "role": "user",
            "content": "Now, I will start a new problem in a new OS. My problem is:\n\nFind out count of users."
        },
        {
            "role": "agent",
            "content": "Think: To find the count of users, I need to run the `cat` command on the `/etc/passwd` file.\n\nAct: bash\n\n```bash\ncat /etc/passwd | wc -l\n```"
        },
        {
            "role": "user",
            "content": "The output of the OS:\n\n21\r\n\u001b[?2004h\u001b]0;root@59d75ad36c35: /\u0007root@59d75ad36c35:/# "
        },
        {
            "role": "agent",
            "content": "Think: The output has been truncated because it is too long, so I need to count the lines in the output.\n\nAct: bash\n\n```bash\nwc -l /etc/passwd | awk '{s+=$1} END {print s}'\n```"
        },
        {
            "role": "user",
            "content": "The output of the OS:\n\n21\r\n\u001b[?2004h\u001b]0;root@59d75ad36c35: /\u0007root@59d75ad36c35:/# \u001b[K"
        },
        {
            "role": "agent",
            "content": "Think: Now I get the answer, it is 21.\n\nAct: answer(21)"
        }
    ],
    "exception_raised": false
}

That's normal. We found that GPT4 can solve the problems normally. It may be worthy considering the method of tokenization for these escape characters.

Feel free to reopen if any problem still exists.

I have opened a pull request which fixes this issue https://github.com/THUDM/AgentBench/pull/162


PYIC-6242: Add ProcessMobileAppCallbackHandler

Proposed changes
What changed

Add ProcessMobileAppCallbackHandler
Add event mapping and state for successful progression from the download page
Refactor CriResponse fetching logic
Update ProcessStepResponse to allow for context
Use mam/ dad context to condition clientCallbackUrl inclusion

Why did it change

Allow users to callback from dcmaw async

Issue tracking

PYIC-6242

Tested in dev, except the clientCallbackUrl sent to CRI stub. Currently looking at


Start in user home

Hi, wondering if you can tell the wslbridge to start in the user's home.
The built-in bash.exe in system32 allows you to start it with
bash.exe ~ -c "linux program to start"
Is there a way to start the wslbridge in a specific lxss directory?

I was about to ask the same, caused by the discussion in https://github.com/mintty/wsltty/issues/3

@mintty I know. I realized that the correct place to ask was upstream here.

I think I agree that there should be some option for controlling the start dir.

@rprichard By the way, this project is amazing.Was wondering if there is a way to make it work using ttys rather than ptys in WSL.

@fpqc What do you mean by tty support?
Regarding the feature request, there are three kinds of paths wslbridge could accept for the startdir: a Windows path, a Cygwin path, or a WSL path.
My first instinct is that only WSL paths are worth handling -- the others can be accommodated by setting the starting directory before invoking wslbridge.  Maybe wslbridge could have a -C<WSLPATH> option.  If the first path component of WSLPATH is ~ or ~user, wslbridge could replace it with the appropriate directory.  (Someone invoking wslbridge from Cygwin bash would have to be a bit careful.  Running wslbridge -C ~ from bash would do the wrong thing, because Cygwin would interpret ~ before wslbridge.)

wslbridge could also do things like substitute environment variables in WSLPATH, but that strikes me as unnecessary.

@rprichard Only one worth doing is a WSL path.  The other two can be legitimately achieved with the "start in" function of Windows.

I added a -C option that changes the working directory before exec'ing the child process.  It has special code to interpret a ~ at the start, so passing -C~ to wslbridge will start the child in the WSL home directory.

Cool!