Postgresql

@FinnStutzenstein This adds the postgres backend for long running polls.
My SQL is very bad. Could you look at the schema and the SQL-Queries?

It would be nice to add a concurrency test (maybe even independend of the backend): start a poll and let two "threads" vote. Stop the first thread right before commting the transaction/executing some statements. Let the second thread do it's job. resume the first thread and assert, that everything is fine.
I will do this. Can you give me a hint how to do this? How can I be sure that the one function is running at the same time?  Maybe we can talk via telephone.

I could add a test that calls the function many times (for example 1_000 times) at the same time and checks that the result is correct.  Then the chance is very high, that two of the functions are really running in parallel. What do you think?

@FinnStutzenstein I added concurrency tests. The sql-functions are now run many times until the transaction succeeds. Is this was you wanted?

I think not fully.You are trying to provoke race conditions with issuing many vote requests in parallel, right? I'm missing an assertion, that there was a race condition*. The combination of the assertion, that there was a race condition and everything is as expected at the end asserts, that the code is working as intended. Maybe this test goes well and there did not happen any errors, so no transaction has to be retried.
*As far as I understand the code there should be an transaction error, which is catched and the vote-routine is retried.

The Race conditions happend. This is why I added the code to retry the call.
I don't no how to write a test that 100% of the time will cause the race condition. I think it's in the nature of a race condition, that it only happens sometimes. With a lot of parallel request, it is very likely that it happens, but even then there in no guarantee.
If you like, I can add a test that calls voteOnce directly many times and checks, that it returns an transaction error. But I don't think, that this is necessary. It is not important. The tests that are already there make sure, that the backend works correctly even on very hight load. If this succeeds, then it does not matter how the backend solves this.
I think your problem is, that you don't trust my tests, that they really run the code in parallel. But it is not the responsibility of this code. I am sure that the go project has written tests that guaranties, that the go statement works.

I don't no how to write a test that 100% of the time will cause the race condition. I think it's in the nature of a race condition, that it only happens sometimes. With a lot of parallel request, it is very likely that it happens, but even then there in no guarantee.

You can provoke it. As I wrote before:

start a poll and let two "threads" vote. Stop the first thread right before commiting the transaction/executing some statements. Let the second thread do it's job. resume the first thread and assert, that everything is fine.

With the first go routine running, you somehow has to pause the execution of the first one. We used a lock that is acquired by the main routine: https://github.com/OpenSlides/openslides-backend/blob/3e449bc4a9987e35ebe366ec2790d88f5cc6dd05/tests/system/action/motion/test_create_sequential_number.py#L130 or https://github.com/OpenSlides/openslides-datastore-service/blob/49abad2adadcfd09c3467a0f0b9f581a127f1258/writer/tests/unit/core/test_writer_service.py#L108
Then the second go routine is started, does it's job and finishes. Release the lock, so the first one can continue. Assert, that this one fails and the retry-handling succeeds.
You have to force the "parallel"/"asynchronous" flow to take the steps in the order that causes the transaction to fail. I mean when opening two instances of psql and running all commands in a specific order, I can provoke it by hand - this has to be done by code and thus verify, that the code handles the situation right. This is the main purpose: Does the code handles a potential RC correctly. To test it, you need to provoke one since if not, you are not testing it.
I mean we do have a test where manual intervention is needed to test some extreme cases, but this should be the exception.

then it does not matter how the backend solves this.

But you do not make sure, that race conditions are handled correctly - this is what I miss here

I am sure that the go project has written tests that guaranties, that the go statement works.

I'm sure, too. It is not that I don't think the go statement does something in "parallel", but when accessing a single resource like the Postgres "parallel" doesn't work anymore. Even if there are multiple connections, the commands do have an order in which postgres executes them. The purpose of these tests is to fix the execution order, so something special happens.

You can provoke it. As I wrote before:

start a poll and let two "threads" vote. Stop the first thread right before commiting the transaction/executing some statements.
I don't know how to do this.

Please let as talk via telephone about this.
I merge this and add the test (if needed) later.


proper fallback for module check

We've had problems with Qunit having a global "module.require", so the route-recognizer thought we would use modules, instead of the window namespace.
This check fixes that assumption. If I'm on the wrong path here, I'm glad to be told why I'm wrong.

We're having a discussion about ES6 authoring -> AMD/CJS publishing later today. Hopefully we can just move away from UMD incantations entirely soon

cheers for the quick answer

QUnit 2.0 has remove all globals except for QUnit. Though there may still be outstanding issues with this UMD wrapper I'm closing this PR as "no longer relevant."


Use @next/third-parties

Change the way we setup Google Analytics per @RubenOlano 's advice to use the Next.js provided way.

How does this change google analytics

I shouldn't, this is just Next.js's suggested way to use google analytics


504 Gateway Timeout when using Get-IntuneManagedDevice | Get-MSGraphAllPages

Import-Module microsoft.graph.intune
Connect-MSGraph -Credential $cred
Get-IntuneManagedDevice | Get-MSGraphAllPages
Get-MSGraphNextPage : 504 Gateway Timeout
{"error":{"code":"UnknownError","message":"{"Message":"{\r\n  \"_version\": 3,\r\n  \"Message\": \"An error has occurred - Operation ID (for customer support): 00000000-0000-0000-0000-000000000000 - Activity ID:
772721f8-7c3f-4ca5-a7b5-21d7beca61ae - Url: https://fef.amsua0402.manage.microsoft.com/TrafficGateway/TrafficRoutingService/DeviceFE/StatelessDeviceFEService/deviceManagement/managedDevices?api-version=2023-07-04&$skiptoken=LastDeviceName%3D%2
7PSD200115-HS-22%27%2cLastDeviceId%3d%275515d4ef-2551-456f-a975-e772ffefe6a0%27\",\r\n  \"CustomApiErrorPhrase\": \"\",\r\n  \"RetryAfter\": null,\r\n  \"ErrorSourceService\": \"\",\r\n  \"HttpHeaders\":
\"{}\"\r\n}"}","innerError":{"date":"2024-01-29T13:43:35","request-id":"772721f8-7c3f-4ca5-a7b5-21d7beca61ae","client-request-id":"772721f8-7c3f-4ca5-a7b5-21d7beca61ae"}}}
At C:\Program Files\WindowsPowerShell\Modules\microsoft.graph.intune\CustomModules\UtilCmdlets.psm1:60 char:25

...                $page = Get-MSGraphNextPage -NextLink $currentNextLink

                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : ConnectionError: (@{Request=; Response=}:PSObject) [Get-MSGraphNextPage], HttpRequestException
FullyQualifiedErrorId : PowerShellGraphSDK_HttpRequestError,Microsoft.Intune.PowerShellGraphSDK.PowerShellCmdlets.GetNextPage

Get-IntuneManagedDevice does work fine, it generates errors when it is piped to | Get-MSGraphAllPages
Any suggestions?

Resolved Created AAD App and set application permissions
Connect-MgGraph -ClientId $clientId -TenantId $tenantId -CertificateName $certSubject
#use the following cmdlet instead of Get-IntuneManagedDevice
Get-MgDeviceManagementManagedDevice -All

Resolved Created AAD App and set application permissions
Connect-MgGraph -ClientId $clientId -TenantId $tenantId -CertificateName $certSubject
#use the following cmdlet instead of Get-IntuneManagedDevice
Get-MgDeviceManagementManagedDevice -All


Two remaining PRs in CodePlex

When we did the whole move to GitHub we left two PRs unreview in CodePlex:

Comparison Null Semantics
TruncateTime performance enhancement

For each of them we should decide whether these should be ported so that we can take them in 6.3.
cc @maumar @ajcvickers @brandondahler

@maumar to move these to GitHub and provide some feedback if needed.


Image links now broken in doc pages

The recent fix to the broken links ( PR #32 ) has indeed fixed the broken links on the landing page. However, now the image links are broken in those pages. Example would be the Network Topology page which has lots of embedded images.
Hope we can fix this before my INTEGRATE presentation in London on 7th June - as I was hoping to show this page! :-)

Actually, it is slightly more complicated as there are two documents that reference the images, but one has not moved.  I've cloned the repo and fixed it myself, but I apparently don't have the permission to publish a feature branch and submit a PR?  Can someone provide that?


Changed to use Wal colors properly

I have changed a few colors so that it detects colors more nicely. and especially themes Fortran language.
tried different wal color schemes and wal extracted wallpaper colors. it looks okayish now.
give it a try.

I took a look at the commit and it seems fine. Tomorrow I'll test it on  my Linux pc and if everything looks good I'll publish the update.

Sorry, I waited way more than a day to review this pr. I tested it now and everything is in check so I'll merge the pr. Before updating the extension I will also take a look at the opened issue, but I think that the update will be live very soon.


Nested panels do not play well together

Current Behavior
If you try to render a Select inside a Dialog inside a Popover it seems that all the "panels" fight for the focus or something...
Expected Behavior
I expect to close the panels in the revers order that they were opened. panels should respect the order in which they where opened, regardless of the nesting (I believe!)
Tamagui Version
latest

Reproduction
here's a code repro: https://github.com/horacioh/tamagui-nested-panels
Here's a video of the code reproduction
https://github.com/tamagui/tamagui/assets/725120/b309ccbe-2795-4075-8743-3a886b7b79ae
System Info
System:
    OS: macOS 13.4.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 16.43 GB / 96.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.4.0 - /opt/homebrew/bin/node
    Yarn: 3.5.0 - /opt/homebrew/bin/yarn
    npm: 9.7.2 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 116.0.5845.96
    Safari: 16.5.2
  npmPackages:
    @babel/runtime: ^7.18.9 => 7.21.0
    @manypkg/cli: ^0.19.1 => 0.19.2
    check-dependency-version-consistency: ^3.0.3 => 3.0.3
    eslint: ^8.21.0 => 8.39.0
    node-gyp: ^9.3.1 => 9.3.1
    prettier: ^2.7.1 => 2.8.8
    turbo: ^1.10.3 => 1.10.3
    typescript: ^5.1.3 => 5.1.3

closing this. Found a solution:

when dealing with multiple nested panels, make sure you add keepChildrenMounted to the parent panels. this will make sure the nested panels will be rendered.
I believe is better to have "controlled" panels in this context. this are complex UX scenarios, is better if you control all the state.
make sure you don't forget components like Select.Viewport!! (this was one of the issues in my app...)

the current state of the reproduction is fixed! you can also see how the behavior change if you toggle the keepChildrenMounted prop (there's a checkbox for that in the demo).
cheers!

cc @natew


Netgear R6300v2 login invalid response

Home Assistant release with the issue:
0.91
Last working Home Assistant release (if known):
None. My first install was 0.8x
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io 2.8
Component/platform:
pynetgear
Netgear Router R6300v2 Firmware Version V<IP_ADDRESS>_1.0.63CH
Description of problem:
First login attempt fails on reboot. Never tries again.
 2019-03-09 09:22:33 INFO (SyncWorker_9) [homeassistant.components.device_tracker.netgear] Logging in
 2019-03-09 09:22:33 INFO (SyncWorker_9) [pynetgear] Get attached devices
 2019-03-09 09:22:33 DEBUG (SyncWorker_9) [pynetgear] Login v1
 2019-03-09 09:22:34 ERROR (SyncWorker_9) [pynetgear] Invalid response
 2019-03-09 09:22:34 DEBUG (SyncWorker_9) [pynetgear] 200
 {'CONTENT-LENGTH': '435', 'CONTENT-TYPE': 'text/xml; charset="UTF-8"', 'SERVER': '"OS/version" UPnP/1.0 "product/version"'}
 <?xml version="1.0" encoding="UTF-8"?>
 <soap-env:Envelope
         xmlns:soap-env="http://schemas.xmlsoap.org/soap/envelope/"
         soap-env:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/"
         >
 <soap-env:Body>
     <m:AuthenticateResponse
         xmlns:m="urn:NETGEAT-ROUTER:service:ParentalControl:1">
     </m:AuthenticateResponse>
     <ResponseCode>401</ResponseCode>
 </soap-env:Body>
 </soap-env:Envelope>

 2019-03-09 09:22:34 DEBUG (SyncWorker_9) [pynetgear] Login v2
 2019-03-09 09:22:34 ERROR (SyncWorker_9) [pynetgear] Invalid response
 2019-03-09 09:22:34 DEBUG (SyncWorker_9) [pynetgear] 200
 {'CONTENT-LENGTH': '310', 'CONTENT-TYPE': 'text/xml; charset="UTF-8"', 'SERVER': '"OS/version" UPnP/1.0 "product/version"'}
 <?xml version="1.0" encoding="UTF-8"?>
 <soap-env:Envelope
         xmlns:soap-env="http://schemas.xmlsoap.org/soap/envelope/"
         soap-env:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/"
         >
 <soap-env:Body>
     <ResponseCode>501</ResponseCode>
 </soap-env:Body>
 </soap-env:Envelope>

 2019-03-09 09:22:34 ERROR (SyncWorker_9) [pynetgear] Get attached devices failed
 2019-03-09 09:22:34 ERROR (SyncWorker_9) [homeassistant.components.device_tracker.netgear] Failed to Login

 device_tracker:
   - platform: netgear
     host: 192.168.x.x
     username: "login"
     password: "password"

same issue here.


dense video object captioningn

When will your group release the code and dataset of dense video object captioning?

Hi,
Thank you for your interest in our work! We are still preparing the code release. I would say 1-2 months is a reasonable timeline.
Best,
Xingyi

Hi @xingyizhou , any update on the inference code?  Thanks so much!

Hello @xingyizhou! I too enjoyed your group's paper Dense Video Object Captioning from
Disjoint Supervision.
The task your paper proposes is quite interesting and I'm looking forward to having a look at how your model architecture is like. Is there any timeline on releasing the code for the paper?


Update deps

[x] CHANGELOG.md not updated, internal change

Is the eventual plan to include this in polymer/tools?

That would make sense to me I think


Enable to work DATABASE_URL with Heroku postgres

Problem
Heroku postgres sets DATABASE_URL with postgres schema, but it does not match with the postgresql adapter. It should be "postgresql".
Solution
Replace "postgres" with "postgresql".
ActiveRecord applies the same approach.
https://github.com/rails/rails/blob/2a28b7c292b3d6c0a01722f50beedb8ec1bf074c/activerecord/lib/active_record/connection_adapters/connection_specification.rb#L42

v0.8.2 has been released.


Look into some abstraction and refactoring of classes

Based on a previous PR (#17 ) and comment:
class Cmd:
    #Redshift stuff

class S3:
    #S3-only stuff

class Copy(Cmd, S3):
    #Stuff that needs both (like COPY/UNLOAD/etc.)

Closing: Should be captured in PR #23 and the latest release: https://github.com/capitalone/Data-Load-and-Copy-using-Python/releases/tag/v0.2.0


How to read Mifare cards sectors and blocks

Hi, Im trying to read from a Mifare Card with you plugin, in Android the Mifare Class is used to connect and read the data, but if I get the tag as a Json from your plugin, How could I read the data as this:
mifareClassic.connect();
mifareClassic.authenticateSectorWithKeyA();
mifareClassic.getBlockCountInSector();
mifareClassic.readBlock();
Is it possible to do that with javaScript?
Im using a Android mobile that can read Mifare.

@valemunoz, @don, i need to read sectors and blocks too. where line on NfcPlugin.java that need to change? btw, i dont know java programming.

@sutriadi there are 2 pull requests #155 and #156 that have code for reading mifare. I haven't had time to review and clean up before merging. Give them a try and see if it works for you.

Hi, I try the code for reading mifare but it's not working for me. Someone have a sample code ?

@JohnMcLear I did it! ... I change the nfc plugin and add 2 functions one who reads and returns the data from all sectors and the other reads a sector specific block. code so no more optimal for the moment but it works :) if anyone needs it let me know

@valemunoz, where do i find this plugin code? We are trying to do a similar read from Ionic stack, to read bytes / values from all blocks / sectors and they dont follow NDEF formats. Please help to get a walk through on this solution you did..


jstree is not populated in IE 11.0.29

Hi,
I have used jstree 3.2.1 plugins but facing some browser specific issues as tree is not populated on "IE 11.0.29" but its working fine on "IE 11.0.27"
FYI.. i have used json data to populate tree.
FYI...:
http://stackoverflow.com/q/35942493/1166364?sem=2
Need your help.
Thanks,
Laxman

Can you open http://www.jstree.com in the same browser and report if the demos work. If they do - it is not a browser or jstree issue, but something related to your data. If it does not - could you please provide a jsFiddle demo and I will reopen the issue.
Best regards,
Ivan

Ivan,
Its working fine with IE 11.0.27 and other version, except IE 11.0.29.
We checked  jstree demo directly "https://www.jstree.com/demo/" its also not populated tree on IE 11.0.29 version,
could you please check the same.
Thanks,
Laxman

I just did - 11.0.29 is a version from 3 years ago (2013) - maybe there was a bug with the browser itself. I doubt I will be able to do anything about this, but still - do you get any errors in the console?

We don't get any error on the console.

Please suggest if you are able to reproduce the same.

Its working fine on IE 11.0.29 also. Thanks


Waiting for SSH to become available...

(Mac):
when executing the Start the computing environment command:
==> mahout: Warning! The AWS provider doesn't support any of the Vagrant
==> mahout: high-level network configurations (config.vm.network). They
==> mahout: will be silently ignored.
gets stuck at: Waiting for SSH to become available...

As far as I understand the Vagrant file, ssh uses the hard-coded key-pair named idomaar. In order to login, I edited the vagrant file defining that my personal key-pair should be used.

Ah! If the participants need to do this, then it should be clearly explained in the directions. Thanks.

I've added configurable options (via environment variables) for:

instance type
keypair name
region

updated also wiki to describe the new parameters


Load maps for vina

Hi,
I'm trying to load existing maps for vina and only score the pose with the following code, but it terminated with "Reading Vina maps ... Segmentation fault".
~/vina_1.2.2_linux_x86_64 --ligand 3ozt_105.pdbqt --maps 3ozt_protein --scoring vina --score_only
The maps were previously generated by command:
/opt/software/MGLTools/1.5.6/MGLTools-1.5.6/MGLToolsPckgs/AutoDockTools/Utilities24/prepare_gpf4.py -l 3ozt_105.pdbqt -r 3ozt_protein.pdbqt -o 3ozt_protein.gpf -y
autogrid4 -p 3ozt_protein.gpf -l grid.log
Could you help to debug?
Thanks!
Best,
Lin

Segfault fixed in #38, closing.


Add symbols (constant, variable etc…) icons

This PR aims to cover relevant (we already have https://lucide.dev/icons?search=function-square, for example) programming symbol-* icons from codicons, used in VS Code breadcrumbs, Outline panel and other places…
variable related to #1063.

Alternatively:

Alternatively:


Add "active" to getkeypool's output to match importdescriptors

importdescriptors uses "active" instead of "keypool" to indicate that new addresses can be fetched from that import object.

Merging this as it's a simple change, the importdescriptors api probably won't change, and having extra key-value pairs in a JSON object doesn't hurt anything.

post-hom tACK, just noticed this :)

great :+1: needed this today when testing importdescriptors. Although now I kind of wish there was a way to dump all relevant descriptor requests with a single command, something like getdescriptors but in importdescriptors/getkeypool format?

dump all relevant descriptor requests with a single command

The --all option? #316

The --all option? #316

excellent


Stage 1 Pre-training Loss Objective?

Hi @yxbian23 @ailingzengzzz et al.,
thx for ur great contribution! may i ask it seems not clear in the paper, what the loss objective for stage 1 pre-training is?
best,

Hi @yxbian23 @ailingzengzzz et al.,
thx for ur great contribution! may i ask it seems not clear in the paper, what the loss objective for stage 1 pre-training is?
best,

Thanks for your interest in our work! Currently, our loss is just the MSE loss between the SMPL-X result predicted by the model and the Ground Truth, but we are exploring adding physical loss functions such as foot contact, local positions, local velocities, etc. Stay tuned!


Bump ledger depth

That's a huge difference, allowing 34 billion accounts. Is it needed?

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


⚠ Fortnite has degraded performance

In cb3eec0, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1656 ms

Resolved: Fortnite performance has improved in a2f35d6.


DOMAIN in .env file has no effect

The DOMAIN parameter in the .env file which contains the default address to bind the huggin server doesn't seem to work and always binds to <IP_ADDRESS>:3000

Are other params from the .env being picked up? Are you running Rails in
development or production mode?
On Tuesday, June 16, 2015, Puneeth Nanjundaswamy<EMAIL_ADDRESS>wrote:

The DOMAIN parameter in the .env file which contains the default address
to bind the huggin server doesn't seem to work and always binds to
<IP_ADDRESS>:3000
—
Reply to this email directly or view it on GitHub
https://github.com/cantino/huginn/issues/861.

I am trying to bring up the vagrant box using the vagrant file provided. I will update here soon.

Works with docker. May be some problem with chef version/ cookbook

Is the address you have set in the DOMAIN option a hostname or an IP?  If it's a hostname, is it resolvable by DNS or is it in /etc/hosts?  If it's an IP, make sure it's on one of the network interfaces on the system.

I provide the ip address in the env file and definetely it is on my ethernet network interface. I moved to docker as it worked seamlessly.

Closing since we haven't heard back.  Hope it's working for you!


Setting up seed properly in Custom env

Question
`# Create the training environment
train_env = gym.make("BabyAI-GoToLocal-v0",max_episode_steps= 512)#, render_mode = "human")
train_env = RGBImgPartialObsWrapper(train_env)
train_env = CustomEnv(train_env, mission)
train_env = ImgObsWrapper(train_env)
train_env = Monitor(train_env)
train_env = DummyVecEnv([lambda: train_env])
train_env = VecTransposeImage(train_env)
Create the evaluation environment
eval_env = gym.make("BabyAI-GoToLocal-v0", max_episode_steps= 512)#, render_mode = "human")
eval_env = RGBImgPartialObsWrapper(eval_env)
eval_env = CustomEnv(eval_env,mission)
eval_env = ImgObsWrapper(eval_env)
eval_env = Monitor(eval_env)
eval_env = DummyVecEnv([lambda: eval_env])
eval_env = VecTransposeImage(eval_env)
save_path = f"main_code/New_model/PPO/{mission.replace(' ', '_')}_model"
eval_callback = EvalCallback(eval_env, callback_on_new_best=stop_callback, eval_freq=8192,
best_model_save_path=save_path, verbose=1, n_eval_episodes= 30)
model = PPO("CnnPolicy", train_env, policy_kwargs=policy_kwargs, verbose=1,
learning_rate=0.0005, tensorboard_log="./logs/PPO2/",
batch_size= 2048,
n_epochs= 100, seed = 42)
model.learn(2.5e6, callback=eval_callback)
Close the environments
train_env.close()
eval_env.close()`
I'm trying to train babyai env on specific instructions, but while setting up the seed it gets stuck on first frame and doesn't do anything. The issue seems to be with Custom env because when I run the code without it, the seed works fine. Help from the community would be much appreciated :)

This appears like a sb3 issue rather than Gymnasium, could you clarify what the Gymnasium issue is?


Errors do not disappear

This extension is awesome! But the latest version seems not working. Errors do not disappear after I fix them.
And I reinstalled the 3.4.2 version, everything is OK.
vscode-error-lens version: 3.5.0
Platform:
Version: 1.67.1
Date: 2022-05-06T12:39:22.915Z
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.13.0-41-generic snap

What settings do you use (from Error Lens extension)?

Only one,  "errorLens.delay": 3000,

Same problem here. Errors are left littered all over the code, even after I fix something. The only settings I have are:
  "errorLens.gutterIconsEnabled": true,
  "errorLens.delay": 500,

I seem to be getting the same issue. I noticed that hitting up/down arrows clears the errors, but after cmd+s they are back. Reloading VSCode fixes it.

same after setting "errorLens.delay": 3000,

Same issue here, only settings I have set is "errorLens.delay": 750,

Not ideal that the errorLens.delay doesn twork though - the onSave option does remove them, but it also means errors do not show up inline until you save the document and commit the change to your environment (which may be running with HMR in react so those errors are critical to have been fixed already in many cases.

Note the issue is with the delay - if no delay is present the errors disappear but they never go away if not

I have the same issue. Here's what it looks like with a syntax error:

And here's what it looks like after it's corrected:

The error text never goes away even after leaving it untouched overnight. I'm not saving anything here, just typing and undoing in the editor. This wasn't a problem a week or 10 days ago, just started recently for me.

In addition, closing and re-opening the file doesn't make it go away either. Only shutting VSCode down and restarting it clears it for me.

Same issue here - installing a previous version fixed it 🤷

Rolling back to the previous version fixed it for me too. Thanks for the suggestion. Forgot you could even do that!

yeah I'm also getting this problem, thanks to whoever suggested reverting the version!

Hi! Same problem here.
The extension was working fine last week, and now it bugs. Tested under Win 7 & Win 10.
Here are my settings,:

  "errorLens.delay": 10000,
  "errorLens.exclude": [
    "Expected linebreaks to be"
  ],
  "errorLens.gutterIconSet": "defaultOutline",
  "errorLens.gutterIconsEnabled": true,
  "errorLens.messageTemplate": "$source $code: $message",
  "errorLens.statusBarIconsEnabled": true,

Hi. I have same problem. Previous version 3.4.2 works fine.
My settings:
"errorLens.gutterIconsEnabled": true,
"errorLens.gutterIconSet": "circle",
"errorLens.delay": 1000,
"errorLens.enabledDiagnosticLevels": [
   "error",
   "warning"
],

At first setting delay = 0 seemed to resolve the issue immediately. But then it started breaking again.
Out of Sync ?
If you have ScriptAnalyzer open, (view problems) you can see the lens is not updating with the new values. It's as if there's caching that goes stale

I've tested files using different languages, and never-saved vs saved files. It seemed to break on most setups.
Environment
> (code.cmd --version) -join ', '
    1.67.1, da15b6fd3ef856477bf6f4fb29ba1b7af717770d, x64

> code.cmd --list-extensions --show-versions  | grep 'lens|power|shell|query' | sort

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Full Config, excluding colors.
{
    "[powershell]": {
        "errorLens.exclude": [
            "cmdlet.*uses an unapproved verb" // "is an alias of"
        ],
    },
    "errorLens.delay": 1300,
    "errorLens.followCursor": "closestProblem", // "allLines",
    "errorLens.exclude": [], //# regex
    "errorLens.excludeBySource": [], // ["eslint"]
    "errorLens.followCursorMore": 2,
    "errorLens.statusBarIconsEnabled": true,
    "errorLens.enabled": true,
    "errorLens.gutterIconsEnabled": true,
    "errorLens.errorGutterIconColor": "#e45454",
    "errorLens.infoGutterIconPath": "",
    "errorLens.errorGutterIconPath": "",
    "errorLens.warningGutterIconColor": "#ff942f",
    "errorLens.warningGutterIconPath": "",
    "errorLens.statusBarColorsEnabled": true, // not sure what this is
    "errorLens.gutterIconSet": "circle",
    "errorLens.gutterIconsFollowCursorOverride": true, // show just selected errors or whole file?
    "errorLens.enabledDiagnosticLevels": [
        "error",
        "hint",
        "warning",
        "info"
    ],
}

Hey everyone: do you think the issue is with the errorLens.delay setting? 😉
(At this point, instead of spamming everyone's inboxes, just click the subscribe button please).

Should be fixed in 3.5.1.

Should be fixed in 3.5.1.

Still happens to me in 3.5.1.

Fixed for me. Update required a reload.

Get this in 3.5.1 (even with delay set to 0)

but only with the problemMatcher; clangd errors clean up fine

Cleanup time. Closing as completed. If it reproduces on the latest version of extension (with steps to reproduce 100% of the time, not randomly), please open a new issue.


Feature/spell check 2

For real this time, spell-check with Hunspell on all platforms, using no native dependencies. Using the apparently excellent and relatively new electron-hunspell library. Appears to be extremely fast and adds virtually no size to the binary.
This was a lot of fun to implement and seems to be, by far, the best option. The other libraries were cumbersome (due to native dependencies) and don't seem as maintained.
Dictionary files are provided by the also-excellent dictionaries which appear to mirror the same .dic and .aff files used by Google Chrome. This means, at least in English, more recent words (such as "selfie") will not be considered typos.
The dictionary files for your language are downloaded the first time the application launches after the update. If you change your system language and restart the app, it should download the dictionaries for the new language and check spelling accordingly, as long as the spell-checker supports that language. If it were not implemented this way and I just included all dictionary files in the app, it would double the size of the app .gzipped. The total unzipped size of the dictionary files for all languages is over 200 megabytes.
Each language will maintain its own distinct list of words you add to the dictionary.
See #121 for more details on this feature and implementation notes.

On a first pass through it looks good, it's a large changeset so I'll look over everything again.
I'll pull down the branch and mess with it to see if I notice anything weird going on. I'll follow up here after I do that.
Good work with the spellchecking!

About to release. This also includes an update to Electron 3.x because of dependency requirements. Thanks for your help again @ajivoin
Geronimo!


Implement the convolutions

The filter bank can be used as is and loaded to vectors on demand, should be possible to convert it to an array of vectors to avoid a round trip later.
Some functions of the family can have specific implementations adding some constraints on blocksize and filter offsets. Some can be efficiently implemented only for some blocksizes.

All the functions being used by the current codebase are implemented. The code can be made faster leveraging the fact 8x8 and larger blocks could be unrolled to use all the registers.

Code accepted.


rkt/status.go: add 'app-' prefix to exit codes.

This helps to parse the exit code without ambiguity. (e.g. an app can be named "exited" or "pid"...)

@alban @jonboulle @iaguis OK to merge this?

There is no ambiguity; after the first two lines you expect appname=status.

Here are some status outputs:
# Running:
state=running
networks=
pid=7337
exited=false

# Running with networks:
state=running
networks=containers:ip4=<IP_ADDRESS>, default:ip4=<IP_ADDRESS>
pid=6635
exited=false

# Exited cleanly:
state=exited
pid=-1
exited=true
etcd=0

# Exited cleanly (2 apps):
state=exited
pid=-1
exited=true
etcd=0
busybox=0

# Killed:
state=exited
pid=-1
exited=true

# Error starting (no metadata server):
state=exited
Unable to print status: no such file or directory

So if you have exit codes they're after the first 3 lines, I don't see the need for this either.

Even though it is possible to parse the output without adding a prefix, I would prefer if it was easier to parse. Is it for Kubernetes?
And it would be more robust if we ever add other fields to this output.

And it would be more robust if we ever add other fields to this output.
+1
Is it for Kubernetes?
Yes.

@vcaputo : do you have more objections to this patch?

Nope, didn't have a strong objection before, it just wasn't ambiguous.  My only objection would be based on breaking compatibility with the current output, but I doubt there are enough consumers of this output today for it to matter.

Ok, thanks for the feedback.
LGTM


Add a describe command to SLO

Is your feature request related to a problem? Please describe.
I would like to be able to run something like reliably slo describe <slo name> to get information about it:

its labels
its indicators selectors
its original author

Not just my own slos by the way, anyone I can access through my orgs.

I'm wondering, since we have more than just objectives as entities, shouldn't we have a top level describe command for any sort of entities?
Something similar to kubectl describe
Why is this needed?
Because when you have a long list of objectives (or otherwise) in the terminal, you sometime forgot where they came from and their labels. It's handy to be able to get a fuller description for them.
cc @daidokoro @dmartin35 wdyt?


{AKS} Refactor 5th part of the update sub-command in acs module

Description

Refactor update loadbalancer profile
Add unit test for openservice-mesh addon (PR #19574)

Unit Test Results:

"covered_lines": 1332,
"num_statements": 1356,
"percent_covered": 98.23008849557522,
"missing_lines": 24,
"excluded_lines": 0

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.

AKS


Fixed Distributors page width issue

Relocated style from integrators.css to distyle.sass
Removed map on devices with small screen resolution

We should also make sure that if no region is selected - we show all distributors by default.


Convert project to a library with examples

Currently the project is a binary project with a main method containing some sample code. We should change it to a library and add examples instead.

https://github.com/andygrove/query-planner-rs/pull/5


chore: fix example code in README

 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.


feat(DataStore): observe query API

Issue #, if available:
Description of changes:
Check points:

[ ] Added new tests to cover change, if needed
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1389 (0630077) into main (fd11eaf) will increase coverage by 0.43%.
The diff coverage is 76.25%.

:exclamation: Current head 0630077 differs from pull request most recent head d090663. Consider uploading reports for the commit d090663 to get more accurate results

@@            Coverage Diff             @@
##             main    #1389      +/-   ##
==========================================
+ Coverage   61.21%   61.64%   +0.43%
==========================================
  Files         667      657      -10
  Lines       20099    19713     -386
==========================================
- Hits        12303    12153     -150
+ Misses       7796     7560     -236

Flag
Coverage Δ

API_plugin_unit_test
65.78% <ø> (-0.19%)
:arrow_down:

AWSPluginsCore
76.28% <ø> (ø)

Amplify
48.64% <0.00%> (-0.09%)
:arrow_down:

Analytics_plugin_unit_test
73.41% <ø> (ø)

Auth_plugin_unit_test
82.35% <ø> (ø)

DataStore_plugin_unit_test
76.61% <79.22%> (+0.08%)
:arrow_up:

Predictions_plugin_unit_test
30.42% <ø> (-3.10%)
:arrow_down:

Storage_plugin_unit_test
57.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...aStore/Subscribe/DataStoreCategory+Subscribe.swift
0.00% <0.00%> (ø)

...s/DataStore/Subscribe/DataStoreQuerySnapshot.swift
0.00% <0.00%> (ø)

...gin/Subscribe/DataStoreObserveQueryOperation.swift
74.19% <74.19%> (ø)

...WSDataStorePlugin+DataStoreSubscribeBehavior.swift
100.00% <100.00%> (ø)

...WSDataStoreCategoryPlugin/AWSDataStorePlugin.swift
79.62% <100.00%> (+3.03%)
:arrow_up:

...n/Operation/AWSGraphQLOperation+APIOperation.swift
47.27% <0.00%> (-7.28%)
:arrow_down:

.../DataStore/Model/Internal/Schema/ModelSchema.swift
62.50% <0.00%> (-2.09%)
:arrow_down:

...lugin/CoreMLPredictionsPlugin+ClientBehavior.swift

...ugin/Dependency/CoreMLNaturalLanguageAdapter.swift

...ionsPlugin/CoreMLPredictionsPlugin+Configure.swift

... and 11 more

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 fd11eaf...d090663. Read the comment docs.


fix crash on targeted blame

Now picks a random user on error

Coverage decreased (-0.05%) to 65.029% when pulling bbe2418878f35d913be11913e648c5154a837c92 on micsthepick:master into e1690748703c4dec7428d3587d9cbae2426135c8 on Charcoal-SE:master.


Remove recents and favorites when entities are deleted

Fixes #3873
Fixes #3697

Codecov Report

Merging #4431 into master will decrease coverage by 0.03%.
The diff coverage is 24.00%.

@@            Coverage Diff             @@
##           master    #4431      +/-   ##
==========================================
- Coverage   58.91%   58.88%   -0.04%
==========================================
  Files         891      892       +1
  Lines       29085    29109      +24
  Branches     4201     4204       +3
==========================================
+ Hits        17136    17141       +5
- Misses      11949    11968      +19


nil event date

The event_date was having problems if set to nil so I updated the code to only set the event_date if the date was not nil.

I think this actually has an issue so closing the request


Add file association icons for easy viewing, Thank you! Something like that

We already have icons in the product, just pick a file icon theme.
If you mean icons in the native OS finder, we can not control that nicely from VS Code as far as I know. Though @joaomoreno can correct me here
Closing


[PT Run] Windows Settings search doesn't return 'Device Discovery' option.

Microsoft PowerToys version
0.5.1
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
With the settings plugin being enabled,  search for 'device discovery' in PowerToys Run.
✔ Expected Behavior
When searching by 'Device Discovery' an option referent to 'ms-settings-connectabledevices:devicediscovery' should be displayed, since it's part of the ms-settings commands specification.
❌ Actual Behavior
At the moment, no results are displayed.
Maybe it is intended behavior because this option opens the control center instead of the settings panel, but since I couldn't find anything talking about it here, perhaps opening a report is the right way to go.
Other Software
No response

When searching by 'Device Discovery' an option referent to 'ms-settings-connectabledevices:devicediscovery' should be displayed, since it's part of the ms-settings commands specification

This should be the setting plugin, For ms-settings-connectabledevices:devicediscovery, this will be handled by the URI plugin.
cc: @htcfreek

@dougwl
The destination of this url isn't a setting in the settings app. That's why it's not listed official as settings url and not part of the settings plugin atm.
Can you plz share your source where you found the list with this uri? This would help us to implement it.

@htcfreek here: https://4sysops.com/wiki/list-of-ms-settings-uri-commands-to-open-specific-settings-in-windows-10/history/?revision=1555539
Wondering why this one isn't using the ms-settings: scheme.

The necessary information can be found here (IANA registration template):
https://www.iana.org/assignments/uri-schemes/prov/ms-settings-connectabledevices
and all the other uri-schemes that follow this pattern (middle of the page, just search by 'ms-settings-'):
https://www.iana.org/assignments/uri-schemes/uri-schemes.xhtml
This URI scheme seems reminiscent to others used in the previous Windows Phone 8
(https://docs.microsoft.com/en-us/previous-versions/windows/apps/jj207065(v=vs.105))

I'm guessing it's registered like that because it access a different compartment from the main settings app (connect ux), although still being part of it.

@crutkas , @jaimecbernardo
Do we like to show results on settings search that aren't any real setting like the Monitor/Projector connection ux?
I think it doesn't fit into the concept of the settings plugin and I asking me which application (property for subtitle) we should assign to this results? Or should we make things more complicated by having a custom subtitle on this results? (Imo a bad idea.)
If we want such results I can implement it. Then please assign me.

Is the ask to go here, correct?

@crutkas This is the one, what you've mentioned is a Developer option that share the same name with the uri
'ms-settings-connectabledevices: devicediscovery'.

for me, ms-settings-connectabledevices:devicediscovery just does Win+K
I'm ok with this but it would be low priority

@Aaron-Junker
can you please assign me. thank you.

Can someone please update tags and project. Thank you.

This is fixed with 0.57.0. Please head over to https://aka.ms/installpowertoys


Custom field

Hello Mohsen
It's possible to add a custom field?
for example, add to field 63 the following:
BIT     Data element    length      type
63.1    Service level      1              ASCII
63.3     Product code    3             Numeric
Thanks,
Carmelo.

Hi Carmelo,
Yes it's possible and I prefer to use some standard like TLV to create data for private fields (61, 62, 63, ...)
Best


Code and data release

Hello author, thanks for your great job and I want to know when the code and data will go to open source？

We just released the code and checkpoints: https://github.com/PrieureDeSion/visualnav-transformer/releases/tag/code_release


Dashboard dataset name different from Dataset registered dataset

In the Dashboard JSON we have three datasets
"datasets": [
    {
        "id": "0FbB00000008cShKAI",
        "name": "LoginWithUsers"
    },
    {
        "id": "0FbB00000008gziKAA",
        "name": "PermissionSet"
    },
    {
        "id": "0FbB00000008gwtKAA",
        "name": "PermissionSetAssignment2"
    }
]

But in the Dataset Json we dont have LoginWithUsers Dataset. Am I missing something ?

Check to see if LoginWithUsers is anywhere else in the dashboard and if
not, delete it.
On Thu, Jan 14, 2016 at 3:28 AM Praneel PIDIKITI<EMAIL_ADDRESS>wrote:

In the Dashboard we have three datasets
"datasets": [
{
"id": "0FbB00000008cShKAI",
"name": "LoginWithUsers"
},
{
"id": "0FbB00000008gziKAA",
"name": "PermissionSet"
},
{
"id": "0FbB00000008gwtKAA",
"name": "PermissionSetAssignment2"
}
]
But in the Dataset Json we dont have LoginWithUsers Dataset Am I missing
something ?
—
Reply to this email directly or view it on GitHub
https://github.com/atorman/permissionsWaveApp/issues/1.


Fix leaves destruction problem

Backport of https://github.com/ferreusveritas/DynamicTrees/commit/a51653afe27b151475f88f7f6f7e8a0aeabdaad9 and https://github.com/ferreusveritas/DynamicTrees/commit/84102a77aab5908b0b91681679f6a7860368934d.
This fixes the issue where breaking a small branch destroys a lot of leaves that are unrelated.
Since the 1.7.10 version doesn't receive fixes anymore, I backported the fix myself, since I find that bug pretty annoying.
Would you mind reviewing and merging this?

After reviewing with ferreus, this will be merged. However, there will not be a JAR released on CurseForge, so any player wishing to use this patch will need to compile it themselves.


MSID: 98031 Version: 1 DOI: 10.1101/2<IP_ADDRESS>6546

MSID: 98031
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>6546
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=98031
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6546

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/98031v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>6546
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/98031/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!


GroupManagerX repo

Hello, if you want you can use the GroupManagerX artifact from xephi's repo as dependency ;) (i's just a mavenized version of the latest GroupManager build)

Thanks ;)

now I just need to find one for BanManager, bPermissions and PermissionsEx :-p

https://pex-repo.aoeu.xyz
bperm is on xephi's repo

banmanager http://ci.frostcast.net/plugin/repository/everything/

[ERROR] Failed to execute goal on project FactionChat: Could not resolve dependencies for project nz.co.lolnet.james137137:FactionChat:jar:1.9.12-SNAPSHOT: Failed to collect dependencies at ru.tehkode:PermissionsEx:jar:1.23.1-SNAPSHOT: Failed to read artifact descriptor for ru.tehkode:PermissionsEx:jar:1.23.1-SNAPSHOT: Could not transfer artifact ru.tehkode:PermissionsEx:pom:1.23.1-SNAPSHOT from/to pex-repo (http://pex-repo.aoeu.xyz/): sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target -> [Help 1]
something was wrong with the https. It doesn't like let's encrypt cert yet :-(


Tests fail intermittantly because of "missing NestedWishlistContext"

Several times now, tests have failed either locally or on Travis because of an unknown constant NestedWishlistContext. Re-running the build/tests usually fixes it, but this is a recurring issue.
This class is defined in lib/nested_wishlist_context.rb and is required explicitly in AmazonSearchController (it was moved to its own file from app/policies/amazon_search_policy.rb due to the same error).
#76 was supposed to resolve this issue, but it still occurs intermittently.

Is this only impacting tests, or do we know if it affect functionality as well?

Just tests, from what I can tell.

Here's a screenshot of the failures:

I bet I forgot to include NestedWishlistContext in the policy tests, and it's failing depending on whether a feature test ran first...

It just failed on Travis with a seed of 32078, but it didn't fail locally. Boo.


Add support for Custom Kubernetes Resources

Updated KubernetesResourceTypeProvider.TryGenerateFallbackType to always return a generic fallback type.
Closes https://github.com/Azure/bicep-extensibility/issues/128.
Microsoft Reviewers: Open in CodeFlow

So let me get this right, it still won't generate the custom type but it will generate a fallback type. That will enable bicep to still build successfully you just won't get full intellisense and whatnot?

You can do this by customizing the token claims: https://learn.microsoft.com/en-us/azure/active-directory/develop/jwt-claims-customization


Revamp the generator system

A single script updates the vssettings for every VS version, as
described in issue #39.

I don't know if this is what you have in mind, or if there's PowerShell styling you'd prefer for me to follow, but I hoped this would address what you were thinking of.


How to specify the gcc version used to build and install jaxlib without root authority?

System information

Alpa version:

alpa-1.0.0.dev0

Are you willing to contribute it (Yes/No):

Yes
Describe the new feature and the current behavior/state
As the documentation of Alpa says, gcc=9.4 will cause a compilation error, which I indeed encountered. I want to switch my gcc version to 7.5 as recommended, but unfortunately I don't have the root authority. It seems that Bazel can build with specified gcc version, so what can I do to? Thx!
Will this change the current API? How?
Describe alternatives you've considered
Additional context

I have tried commands like:
cc=/usr/bin/gcc-7 python3 build/build.py --enable_cuda --dev_install --bazel_options=--override_repository=org_tensorflow=$(pwd)/../third_party/tensorflow-alpa --bazel_options=--action_env=cc
which aims to use '--action_env' to pass the env variable cc=/usr/bin/gcc-7 to Bazel, but still don't work...
Here is my error report:
[0 / 897 [Prepa] BazelWorkspaceStatusAction stable-status.txt
(8 actions, 0 running)
[2,381 / 13,025] Compiling src/cpu/cpu_convolution_list.cpp;34s local ... (40 actions, 38 running)
[3,551 / 13,025] Compiling tensorflow/compiler/xla/service/cpu/runtime_matmul.cc; 74s local
(40 actions, 39 running)
[5,556 / 13,111] Compiling tensorflow/compiler/xla/service/cpu/runtime_matmul.cc;159slocal
(40 actions, 39 running)
[6,235 / 13,111] Compiling mlir/lib/Dialect/SPIRV/IR/SPIRVOps.cpp;83slocal
(40 actions,
39 running)
[8,705 / 13,181] Compiling 11vm/lib/Transforms/Vectorize/LoopVectorize. cpp; 33s local
... (40 actions, 39 running)
[9,320 / 13,181] Compiling mlir/lib/Dialect/LLVMIR/IR/LLVMDialect.cpp;79slocal
.. (40 actions, 39 running)
[11,729 / 13,181] Compiling 1lvm/lib/Target/X86/X86ISelLowering.cpp; 785 local
(40 actions, 39 running)
ERROR: /home/cyxue/.cache/bazel/_bazel_cyxue/7f0d3638ac6ebab783173044f28f4f4d/external/org_tensorflow/tensorflow/compiler/xla/service/BUILD:
6736:11: Compiling tensorflow/compiler/xla/service/pass_context.cc failed: (Exit 1): crosstool_wrapper_driver_is_not_gcc failed: error executing command (cd/home/cyxue/.cache/bazel/_bazel_cyxue/7f0d3638ac6ebab783173044f28f4f4d/execroot/main &&
exec env -
LD_LIBRARY_PATH=/usr/local/cuda/lib64: PATH=/home/cyxue/miniconda3/envs/jax_framework/bin:/home/cyxue/miniconda3/condabin:/usr/local/cuda/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
PWD=/proc/self/cwd
TF_CUDA_COMPUTE_CAPABILITIES=sm_35,sm_52,sm_60,sm_70,compute_80
TF_NCCL_VERSION=2.14.3
external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc-MD-MFbazel-out/k8opt/bin/external/org_tensorflow/te

@DicardoX  try these two lines?
https://github.com/alpa-projects/alpa/blob/main/docker/scripts/build_jaxlib_docker_entrypoint.sh#L10-L11

@DicardoX try these two lines?
https://github.com/alpa-projects/alpa/blob/main/docker/scripts/build_jaxlib_docker_entrypoint.sh#L10-L11

You mean commands like these?
export CC=/usr/bin/gcc-7
export GCC_HOST_COMPILER_PATH=/usr/bin/gcc-8
python3 build/build.py --enable_cuda --dev_install --bazel_options=--override_repository=org_tensorflow=$(pwd)/../third_party/tensorflow-alpa

@zhisbug Thx, it works!!


Address #78

If an error occurs in a call to Calc.update(), the exception is now re-raised with a printout of the instance repr.
See #78

@hriebl I just requested you as the reviewer mostly because Paul is busy and Gianmarco on vacation - I hope that's fine for you.

Should this error get it's own type instead of the most general RuntimeError?
I guess the question is rather if we should start to define error types for liesel in general.

Another option that would not require us to define new error types could be to raise the same error as the original one and just append something to its message.


Demo suggestion

Just a suggestion if you ever re-write the demo you might want to have a look at MaterialDesignToolKit demo. Their new demo makes it very easy to copy the code necessary for a new user.

Something similar can be done, though for this library we would have to display the class or xml that creates that form instead of XAML.

Crazy idea: walk the built form definition and generate code from the fields/resources. This would allow generating code from the form.

@EdonGashi you mean using this?

Writing xaml would not be possible unless we embed a small runtime for features like validation/bindings.
What I was thinking was something like this:
You have a form with 2 rows, first row contains a StringField with a Name that is an IValueProvider of PropertyBinding(Name="MyName")
The other row may contain an ActionElement with a label IValueProvider of LiteralValue("LOGIN")
From these facts I can derive that we have a form with a string and an action, more exactly:
[Action("something", "LOGIN")]
class Form
{
  [Field(Name = "{Binding MyName}")]
  public string Something  { get; set; }
}

I have another idea which is much easier, any idea what to use for syntax highlighting?

Plain old xcopy did the trick 🔥


fixing non existing block issues

closes #744

You are changing reliquary and masterchef service, but the error is on avax??

i don't see a specific issue, but might be the RPC logs lagging behind


Edit survey questions

It would be a really awesome feature if the questions the Bot ask during the survey could be changed, this would allow for any types of servers to use the bot and change the questions to their needs.

Hello @ChrisCartagena
Thank you for the feedback.
This idea has already been appearing in my mind, plus it was raised as a ticket once (#36).
For now, adding optional/server-specific questions was not created due to performance considerations.
You are welcome to suggest a different set of questions that you think should be in the survey, I'd gladly consider feedback from the bot's user.
Also, just for more clarity - could you please add more specifics what is your case to be in the need of different/additional questions? I've tried to make questions quite generic exactly to be able to be used not for just games, but for other activities as well. If you provide a bit more info about your specific vision of the survey I might as well reconsider optional questions implementation.

Closed due to inactivity.
Consider reopening this ticket in case you want to resume the conversation


add the text "change by infitake" in article

java SE add some new feature in java which can increase the mechanism of exception handling in java for better use by the programmer;
these are as follow:
1: catching multiple exception in same catch block;
2: the try-with-resource statement;
1: //first we use that type of coding :
/*
try
{
//code that may throw exception.
}
catch(ExceptionType1 e)
{
// ex_handle(e);
}
catch(ExceptionType2 e)
{
// ex_handle(e);
}
*/
by these enhancements in exception handling:
this code should be written as:
/*    try
{
//code that may throw exception.
}
catch(ExceptionType1 |ExceptionType2 e)
{
// ex_handle(e);
}
*/
2:using try-with-resources statement
it automatically close all the opened files and resources at the end of the try block, thus there is no need for explicit closure statement;

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Hi,
As there has been no response from your end for a while, I am closing this PR.
Feel free to create a new PR if you still wish to make the changes you originally added in this PR.  Also, remember to re-sync your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding!


Page infinite loading if contains apollo query

Version
v4.0.0-rc.4
Reproduction link
https://graphnewreader-uqpacjfhny.now.sh
Steps to reproduce
Click one of the link.
The page load with data.
Reload the page.
link for git: https://gitlab.com/yaki4/grapheader.git
What is expected ?
Page load and display data.
What is actually happening?
The second and third page with apollo query load forever.
Additional comments?
You can fin in package.json the requirement for vue, vue-template-compiler and vue-server-renderer.
I try remove and rebuilt yarn.lock without success

This bug report is available on Nuxt community (#c181)

I think there is something with the skip function. Remove it will solve the problem
Same as https://github.com/nuxt-community/apollo-module/issues/200
Also note that vue-apollo v3.0.0-beta.28 with the prefetch option default is true so set it to false if you don't want to prefetch data.

Oh you right if i remove the skip function the page will load correctly.
However the other issue show another problem. The fact that if i use a store value in my client, it'll not update properly.

I've had similar issues with the upgrade to the new RC.
SSR seems to hang infinitely even when skip option is removed.
Tricky to look at just where it seems to be failing.

I'd highly recommend to delete package-lock/yarn.lock remove the whole node_modules folder and run the install again. Seems to me that some version conflict exists

@dohomi I'm afraid this had no effect and the development still stalls

I am also experiencing this issue when setting the skip field in a query in SSR. It doesn't matter if skip is a literal boolean, or a method that returns a reactive value.
I was able to get around it in some cases by setting prefetch: false on the query, but that's not an ideal workaround.

This looks to be a bug in vue-apollo. Here's a PR that purportedly fixes it: https://github.com/Akryum/vue-apollo/pull/511

Merged https://github.com/nuxt-community/apollo-module/pull/211


Add new WithSliceDeepMerge config option

This PR aims to add a new feature to make it possible to deeply merge slices within maps by combining their contents.
Can be considered as a continuation of #180

Codeclimate issues haven't been fixed, also no new features will be added to v1.


Remove untracked files doesn't properly quote names

When the plugin prompts removing files from Git because they are untracked (i.e. have been committed then added to .gitignore) the plugin nicely prompts that it can remove them. However, the commands it prompts don't quote the files correctly, so the commands fail silently if there are any spaces in the filenames.
Example commands:
git rm --cached --force .ipynb_checkpoints/Client Rejections-checkpoint.ipynb
git rm --cached --force .ipynb_checkpoints/CUSUM wrong...-checkpoint.ipynb
git rm --cached --force .ipynb_checkpoints/Rejection Count-checkpoint.ipynb

If you actually run these commands, they fail because of the spaces in filenames.

@bspeice Thanks for the report, Fix will be available in the latest release.


PropertyAttributeConvention does not apply to non-public properties

Both PropertyAttributeConvention <T> and NavigationAttributeConvention<T> does not apply to protected and private properties. They use method Type.GetProperty(name) to get CLR property which always returns null for non-public ones.

@smitpatel we don't map privates by default, but we should respect annotations on them once they are included


Merging Duplicate medicines in the  database

Why do you want to merge the medicines. As it can be seen from the screenshot, the medicines are of different variants both are having different Price and also different ExpiryDate.
Also maybe while inputing only you can check in the database if such medicine exists.


Error on Rest chapter

I got an error following your code at the Consuming Rest service chapter.
I also download the code form your repo but on 222 on Authenticate Task you write:
client.Authenticator = new OAuth2AuthorizationRequestHeaderAuthenticator(accessToken, "Bearer");
but my VS trap an error and tell me :
CS0618: 'RestClient.Authenticator' is obsolete: 'Use RestClientOptions.Authenticator instead'
CS0200: Property or indexer 'RestClient.Authenticator' cannot be assigned to -- it is read only

Hi there, i also run into this issue. Easiest way to fix it is to downgrade a RestSharp to 108.0.3 version or lower. Kinda worked for me... =)


servicequota: add quota name to the entity

I somehow forgot to add the name which is the only human-readable piece of information. :(

👍

:+1:


Default to escape true on all column types

Some column types do NOT have "escape" to "true" by default. That means the output is not filtered for XSS, so if the admin stored some malicious JS code in the database, that code will be run for whoever previews it (perhaps another admin).
The same is true if the content did NOT come from the admin, but from something a user would input, outside Backpack. For example a contact form or something. A malicious user plugs in something like afafa"> then when the admin previews it, that code will run if it's in any of the columns addressed in this PR.
Not sure how/whether we should be doing this... What do you guys think? It'd definitely be a breaking change, but I'm wondering if it's even worth doing. If it's a Backpack problem indeed. If we should add a warning, or default all columns to escaping. Or if any of the columns here actually need to not be escaped. I don't really remember why we chose to leave these ones unescaped by default...

A new inspection was created.

The inspection completed: No new issues

I think we should do this. I think it's reasonable to default to secure, and then let the dev make it unsecure if they specifically want to. But it's also a breaking change. So I propose that we do this in 4.2, so that we can add a note in the upgrade guide about it.
But before we merge this into 4.2, let's take a little time to carefully consider how it would impact each column type, and if it's ok to default to "true" for each one of them...

Update: just pushed https://github.com/Laravel-Backpack/demo/commit/5ed0799e9497170532b79c9d4b5edbd886b83d74 in the demo, these lines have to be commented out for the issue to be reproducible.

The inspection completed: No new issues

I've just double-checked, edited and added docs for this. I still think we need to do this. We can't assume the dev knows what he's doing in terms of security... And personally anything a software can do to force security on me... Even if I sometimes grumble... I appreciate it, I'm sloppy sometimes too.
However, @pxpm - I think you were right in this being too strict for some columns, thanks for the reality check. You said:

it would create some BC like in custom_html, closure or markdown for example where user would need to setup escaped => false to display the html, and be open to some XSS anyway.

Which of course I agree with 😅 So I've decided to increase security for some columns, but not all columns. More exactly, I've increased security for everything except the columns that don't really make sense escaped (markdown and custom_html). I think this is a sweet spot:

columns that can be used to output something other than HTML are now "secure by default" (eg. closure, model_function, model_function_attribute, textarea etc.); it will be inconvenient during the upgrade process to add 'escaped' => true in all places you output HTML, but it's also an opportunity to question if what that HTML is safe;
columns that cannot be used escaped (custom_html and markdown) have big fat notices in the docs, that they should sanitize the input or output; I've included both a package recommendation and examples; I've even submitted a PR to that package to make it even easier to use, by just casting that attribute in your model to CleanHtml::class, CleanHtmlInput::class or CleanHtmlOutput::class; it's dead-simple now!

I've edited the PR so that it's:

as easy as possible to upgrade (add 'escaped' => false in a few places where you output HTML);
not inconvenient at all for people using 4.2 directly;

You also said:

I think the best defence would be to prevent those things to be stored inside the database, but I get that it's dificult to manage for example in a WYSIWYG editor.

I agree with that too. So I added notices that the user should sanitize the input/output when using those columns. I didn't add the notices in fields too because... you can probably trust the admin not to add malicious JS. But if you do create a form to edit user-introduced HTML, if you'd like the admin to see it... you have to use the textarea column. And it'll escape it. Then you'll run into the notice that you should sanitize the input/output. So we don't protect the user when building his app, but when he builds the admin panel for it, we remind him and recommend a security improvement.
@promatik what do you think about this? Good/bad? Sweet spot or not? Eager to merge this.

I agree with it — 100%. Security first, and if while upgrading backpack to v4.2 a column that used to be;

Value

becomes;

Value

The first thing that comes to a developer mind is "escaping", I think developers will easily figure out what's the problem. Also, since they are now forced to use the "escaped" => false in those cases, it will enforce devs to use it only when it's needed and that is a good security behavior.


 contains regression

Description
The shadow component has a regression in the C++ renderer
Version
0.1.165
Steps to reproduce
Take the following two test cases:
https://github.com/Shopify/react-native-skia/blob/main/package/src/renderer/__tests__/e2e/ImageFilters.spec.tsx#L74
https://github.com/Shopify/react-native-skia/blob/main/package/src/renderer/__tests__/e2e/ImageFilters.spec.tsx#L97
Reduce the image comparaison threshold:
checkImage(img, docPath("image-filters/dropshadow.png"), { threshold: 0.05 });

It will reveal that no shadow is applied to the component
Snack, code example, screenshot, or link to a repository
https://github.com/Shopify/react-native-skia/blob/main/package/src/renderer/__tests__/e2e/ImageFilters.spec.tsx#L74

Hey, we're experiencing this when drawing multiple shadows on text, I'm not familiar with what the "new decorate paradigm" is referring to - is there any known fix for this issue? Thanks!

indeed!


Suggest to create a new command

Instead of complaining about an unknown command, the CLI should described how to create this unknown command.

Rejected.


Pattern: Lead Space QA testing - Prod level - Round 1

User Story

As a [user role below]:
developer using the ibm.com Library Lead Space - centered pattern

I need to:
have a version of the pattern or component that has been tested for accessibility compliance as well as on multiple browsers and platforms

so that I can:
be confident that my ibm.com web site users will have a good experience

Additional information

Browser Stack link
Browser Standard
Browser versions to be tested:  Tier 1 browsers will be tested with defects created as Sev 1 or Sev 2.  Tier 2 browser defects will be created as Sev 3 defects.
Platforms to be tested, by priority:  1)  Desktop  2)  Mobile  3)  Tablet
Mobile & Tablet iOS versions: 13.1 and 13.3
Mobile & Tablet Android versions: 9.0 Pie and 8.1 Oreo
Browsers to be tested:  Desktop: Chrome, Firefox, Safari, Edge, IE, Mobile: Chrome, Safari, Samsung Internet, UC Browser, Tablet: Safari, Chrome, Android
Accessibility Checklist
Creating a QA bug
Lead Space - centered designs
Lead Space - centered functional specifications
Lead Space - centered design issue (#379)
Lead Space - centered alpha dev issue (#437)
Design QA testing issue (#1218)

Acceptance criteria

[x] Accessibility testing is complete.  Pattern is compliant.
[x] All browser versions are tested
[x] All operating systems are tested
[x] All devices are tested
[x] Defects are recorded and retested when fixed

@Praveen-sr @chetankan I am closing this issue as the work is finished for sprint 20-04.  I have opened a new issue (#1554) for the work to be done in sprint 20-05.


feat: allow users to use version suffix pipeline runs

What changes were proposed in this pull request?
Currently, when pipelines are created using elyra from the Notebooks, each additional iteration of the pipeline run creates a new pipeline version identified by timestamp.
The PR will allow user to switch from timestamp identifiers to versions numbers (v1, etc) using an environment variable KFP_SUFFIX_USE_VERSION. This will make the pipeline runs easily distinguishable.
Fixes #3182
How was this pull request tested?

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@rkpattnaik780 overall LGTM, but could you mention the new env var in the docs

@rkpattnaik780 could you also rebase the pr to make sure all the tests are passing

In an off-thread discussion, we noticed the current implementation will fail if:

once we have few pipeline. ex: 10 , latest version will be -v10
a user delete the 6th pipeline, the total_size=9
the next call of pipeline, would try to create the version '-v10`
again

We are trying to find a workaround for this issue. Cc @harshad16

Thanks for the pull request @rkpattnaik780
and for the review @lresende
due this reason: https://github.com/elyra-ai/elyra/issues/3182#issuecomment-1756077849
we don't require the feature, we have closed the issue.
Please feel free to close the PR as well.


Default to "latest" version (and more)

this pr actually kinda tackles many things at once:

if version is latest, replace it with latest version from https://github.com/tinygo-org/tinygo/releases
use more "official-y" funcs (check_packages and rm -rf /var/lib/apt/lists/*)
use dpkg --print-architecture which would appear(?) to print "amd64" on my x64 machine so i think just arch=$(dpkg --print-architecture) is ok? idk tho. I've been wrong before 😅
rm -rf /var/lib/apt/lists/* at the end

😬 idk if you want me to split this into multiple prs to selectively merge? 👆 - they just kinda happened as i replaced code 🤣
this pr would:

if the VERSION var is "latest" ("" defaults to "latest" via ${var:-defaulttext})...

curl https://github.com/tinygo-org/tinygo/releases/latest
that redirects too https://github.com/tinygo-org/tinygo/releases/v0.29.0
grep (sed actually) \/v(.*)$ but print ONLY the (.*) capture group https://stackoverflow.com/a/58379307/19522682
use grep -oP '\d+\.\d+\.\d+$' to get that juicy version number on the end
set VERSION to that 1.2.3 thing

change the arch detection to just use arch=$(dpkg --print-architecture)
rm -rf /var/lib/apt/lists/* at the end like https://github.com/devcontainers/features do a lot
change devcontainer-feature.json to use latest as the default instead of 0.28.1

@lee-orr this is your feature so idk if you want to change any of this at all. it works fine as-is. this is just some ideas that i had 😊.

good news: the url properly redirects and gets printed!
jcbhmr@PIG-2016:~/Documents/23$ curl -fsSL https://github.com/tinygo-org/tinygo/releases/latest -w '%{url_effective}' -o /dev/null
https://github.com/tinygo-org/tinygo/releases/tag/v0.29.0

bad news: the sed magic to extract that last v0.29.0 part fails 😭😭
time to fix! 🔧

back to green!
jcbhmr@PIG-2016:~/Documents/23$ curl -fsSL https://github.com/tinygo-org/tinygo/releases/latest -w '%{url_effective}' -o /dev/null | grep -oP '\d+\.\d+\.\d+$'
0.29.0

This all looks great to me!


Reproduction

Spawn a ghost role using reinforcement teleporter
Try to leave via Arrivals

this would likely apply to cerberus and holos as well


Multiline comments

The comments widget currently only allows commenting on a single line. For some use-cases of comments, such as code review comments, commenting on a range is not only beneficial, it's already supported by the underlying tooling (see GitHub pull requests for example). We should support a way to comment on a range. This support will require:

API that allows comments to be made on a range
UX that indicates commenting on a range is possible
UX that shows that an existing comment is on a range

In the editor for the comments widget
In the Comments view

@rebornix FYI since I think you also want this in notebooks and @laurentlb since I know you were interested in this.

@rebornix yes, the API does currently support this! I was just outlining all the parts.

First pass at UX for leaving a comment on a range and indicating that an existing comment is on a range:

Looks great!
Is it possible to add explicit lines range to the header of comment as it is in GH with "Commenting on lines 63 to 66" as well?
For example: "Start discussion on lines 15 to 20" or "Start discussion (lines 15-20)" in your example

Is it possible to add explicit lines range to the header of comment...

I do want to add this information somewhere, but the content of the header/title of the comment thread is owned by the extension that provides the comment. I will try adding the range info somewhere else in the comments widget.

I've tried out adding line numbers in the comment thread title area, but I haven't found a place that I like it.
Example right aligned:

Example left aligned:

Since the extension controls the contents of the title, all we can do is append it in there, and it just ends up looking out of place. Individual extension can of course put the line numbers in their in an integrated way themselves. The rest of the widget is comment node focused, not comment thread focused, so it doesn't belong there either.
Happy to have suggestions for where else to put it, but for now I will leave it out since the editor decoration already provides this info.

@alexr00 is it possible to extend comment block at the top for multiline comments as GH does? So it will be part of thread and displayed above extension controlled title

@brianjmurrell this does indeed open the door for the suggestion feature: https://github.com/microsoft/vscode-pull-request-github/issues/603

@alexr00 Ha!  That's the one, and yes, I am subscribed to it already.  Too bad GitHub's tool to find all of your subscriptions is not listing anything from this repo.
Thanks for the pointer!

Hrm.  This does not seem to be working for me.  I try to click and drag but I just get a plus-sign that moves with my mouse cursor, no multi-line selecting for comment.

Clicking and dragging on the + is not currently supported. Instead you make a text selection then click the +: https://github.com/microsoft/vscode/issues/148214

Ahhh.  I see.  Thanks @alexr00!

hey @alexr00 thanks for implementing this, works really well!
just noticed one UX issue: when the comment is collapsed, it still shows the background and border on the code... fairly unpleasant to look at 😬

ideally, that styling should only display if the comment is expanded, right? :)

I intentionally left the background and border when the comment is collapsed so that there was a strong indication that the comment is there and it's how other code review tools I've used in the past have worked.
However, this is your code editor too and not just a code review tool. If there's lots of feedback that this is undesirable then we can change it. I could also have a separate theme color for when the comment is collapsed so that you could essentially hide the background and border in this case.

I suspect it can be annoying to have the region always highlighted, in particular when the comment is resolved.
I don't have data or user feedback yet, I can share it in the future.
cc @albertelo

I find the highlight distracting even with the comment card open. I would prefer a setting for the highlight to only show up when the comment card is moused over or focused.

Just FYI, https://github.com/microsoft/vscode/issues/152067 will hide the range highlight when the thread is collapsed.


Add DCO GitHub Workflow

Signed-off-by: Erik Körner<EMAIL_ADDRESS>

Codecov Report

Merging #1 (7ee733d) into master (2b164f9) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #1   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines            4         4
=========================================
  Hits             4         4

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 2b164f9...7ee733d. Read the comment docs.


Potentially fixing crash with not parseable certificate

Closes #247

Fix confirmed here - https://github.com/projectdiscovery/nuclei/issues/3619#issuecomment-1531037614


Observed Errors in Solutions in d06 & d07

While going through excercises d06 and d07 I had noticed a few issues in reference to the solutions:

d06 Data:  Deriving Quantities - The answer showed up on my copy of the downloaded course files for q4 and q5.
d07 Vis: Histograms - q4 doesn't have a solution.

Thanks for the feedback! Addressed in commit 4c566da


[HOW-TO] Camera Flash

Describe what it is that you want to accomplish
Hey, I want to build a mobile camera.
After allowing per tuning file cranking up the range for AnalogueGain to 16 for the imx708 the shutter speed got on the faster side. It helps to achieve sharp images if hand held.
But if it is getting darker or indoors, a flash could be useful to get decent lit sharp captures.
I think about a LED or a speedlite to add to the camera.

For speedlite, it might be hard to get the right timing and it's unclear to me how to deal with the AGC algorithms when there is just a speedlite exposing one frame instead of a more permanent light.
For LED the light usually cannot be as bright as with the speedlite, but I think it would be OK to tinker around with it in the beginning.

My questions:

Which is the current status in libcamera/picamera2 to use a flash? Are there any examples/best practice since it is not part of libcamera itself (yet)?  Best I could find so far is maybe use this implementation to control the actual flash? https://forums.raspberrypi.com/viewtopic.php?t=368437. Adding dtparam=cam1_sync=24 to config.txt seems to have no effect on RPI5.
Specifically how to implement something like light metering for example based on 1 test frame illuminated vs 1 frame not illuminated to adjust the camera algorithms?

Additional context
For reference, I found these related issues.

https://github.com/raspberrypi/picamera2/issues/635
https://github.com/raspberrypi/picamera2/issues/893#issuecomment-1855459186
https://github.com/raspberrypi/picamera2/issues/891#issuecomment-1852330916
https://forums.raspberrypi.com/viewtopic.php?t=190314
https://forums.raspberrypi.com/viewtopic.php?p=2208717&hilit=flash#p2209016
https://picamera.readthedocs.io/en/release-1.13/recipes2.html#using-a-flash-with-the-camera

There is no explicit support in libcamera for flash metering at the moment, so I think you would have to implement something for yourself. I think quite a bit of experimentation might be necessary. Here's what I might try and do.

Monitor the image exposure and gain, and the brightness of the image. You could get the brightness either from the statistics that the ISP produces (this would differ between Pi 5 and earlier Pis), or from a scaled down low resolution stream which you could analyse with numpy.

Fix the exposure and gain to the current values. Once that has taken effect, turn on the flash. Once you're sure you've got a frame illuminated by the flash (you could use capture_request with the flush parameter?), turn off the flash and analyse the brightness of the image.

Use these values to calculate the shutter/gain you want for your final capture. Be careful if you're using the low resolution stream because this image will have gamma applied to it, making it non-linear in terms of exposure vs. brightness.

For the capture, create a full resolution configuration and in it specify the exposure/gain controls that you want, all as part of that configuration. These will then be applied to the capture. Once the capture is done, set up a preview mode with the original exposure/gain in the configuration, and then, once that mode has re-started, set the AEC/AGC back to auto.

I'm not familiar with speedlite, but I'm guessing these are xenon flashes? This would pose quite a few challenges. Timing is very critical, in fact I've known cameras that expect you to use a xenon flash to have a hardware trigger line that gets pulsed at the right moment. Even worse, most of our cameras are rolling shutter cameras, so you can't even guarantee to illuminate the entire frame with a single pulse.
As regards the mechanics of getting an LED flash to come on, that's not really a Picamera2 question and I don't know the answer. Maybe try asking that on the Camera Board forum and you'll find someone who can help you. Good luck!

Hi, thank you for the elaborated answer. I was kind of expecting that I might need to find my own algorithm to apply.
Any chance to rely on the internal libcamera AEC/AGC algorithms without need to reimplement them?
If you have any idea about a suitable algorithm I could implement in python/numpy that would be great also.
I like to avoid switching between modes, because at the same time I want to synchronize two cameras as we found a way here: #1050

I don't think the existing AEC/AGC will help you as that's all very bound up with the runtime C++, the camera metadata and ISP statistics. I think re-inventing some of that in Python is going to be necessary.
First of all, using the low resolution image is complicated somewhat by the fact that gamma is applied to it, whereas ISP statistics are complicated by the fact that you have to unpack them, and they're different on a Pi 5 (don't know if that's an issue for you).
If you're using the low resolution image, it'll probably be YUV420 (it can be RGB on a Pi 5, but not earlier Pis). We probably only need the Y channel. I would also build yourself an inverse-gamma lookup table (you can find the gamma transform in the .json camera tuning file). Reducing the size of the low resolution image will leave you with less computation, though there will be a limit as to how low you can go. It might be 64x64, not sure.
So I'd do this:

Run your camera normally and push your low res Y image through your inverse-gamma table, then get the average. (I know gamma operates on RGB, not Y, but if you only have Y then this will have to do. Again, the alternative would be to use the ISP stats.)

Do the same for an image that has identical exposure/gain but where the flash is on. Again, push this through the inverse-gamma table and compute the average (or record the brightness from the ISP stats).

Now you can calculate what total exposure (exposure*gain) you want, based on a "target exposure value". This is likely to be quite low, say 40, for un-gamma-ed 8-bit numbers.

To calculate this value, bear in mind the following. The flash (if it has different levels) will scale the brightness delta between your non-flash and flash images. Both exposure and gain scale everything. (Xenon is a bit more complex in this respect - the effect of the flash scales with gain but not exposure.)

This will give you the total exposure that you want, but you'll still have to divide them up into shutter time and gain (what exposure profiles do in the built-in algorithm). If your flash has different levels then there's also a trade-off there. Mostly you probably want to go with flash at maximum brightness, unless that leads to an exposure so low you can't manage it (in which case, select the minimum achievable exposure and calculate the amount of flash that this leaves you with).

There's probably quite a lot to digest there - hope it helps!


Fix for shutdown crash

This is a clumsy fix that I'm not happy with, but it highlights the problem.
During shutdown it's possible for module shutdown and the CommunicationsHost thread to fight against each other. Perhaps there should be a mutex at the ClientInfo level?

I think that the problem is that the listening socket is supposed to be blocking (it is on the other platform). This lead to the thread sync issue and high cpu usage. I will have to investigate why, despite the call to SetNonBlocking(false), is is not waiting. It would still be theorically possible to have a socket listen and destroy at the same time, but highly unlikely. Might still want to have  a mutex still.

Ok, looking more closely at Unreal socket code, looks like we can't rely on being able to set the socket to 'blocking' on all platform UE4 ships to. This means, that we'll need the sleep and mutex.

Fixed with 2f4737522b93733f063e8374f3262f8927e60b32


test_hybrid_module_exposure failing

Hi @alopezrivera , after merging the MPC and Horizons code, it looks like the new unit test for the hybrid modules is failing. I'm pretty sure it's not something critical, but is a result of a new directory being added, and this test comparing things incorrectly. In the process of moving this code into the correct place, could you also check whether this works again? Thanks!
https://dev.azure.com/tudat-team/feedstock-builds/_build/results?buildId=1319&view=logs&j=2928da41-3b64-5ab6-d69b-0a36120eed79
(relevant part copy-pasted below)

2023-10-31T07:44:35.2212041Z tests/test_hybrid_module_exposure.py::test_tudatpy_kernel_module_exposure FAILED [100%]
2023-10-31T07:44:35.2213025Z
2023-10-31T07:44:35.2213832Z =================================== FAILURES ===================================
2023-10-31T07:44:35.2214471Z _____________________ test_tudatpy_kernel_module_exposure ______________________
2023-10-31T07:44:35.2214854Z
2023-10-31T07:44:35.2215147Z     def test_tudatpy_kernel_module_exposure():
2023-10-31T07:44:35.2215474Z
2023-10-31T07:44:35.2215646Z         import tudatpy.kernel as kernel
2023-10-31T07:44:35.2216188Z
2023-10-31T07:44:35.2216353Z         exposed_kernel_modules = [
2023-10-31T07:44:35.2217008Z             # 'utils',
2023-10-31T07:44:35.2217257Z             'astro',
2023-10-31T07:44:35.2217520Z             'trajectory_design',
2023-10-31T07:44:35.2217829Z             'constants',
2023-10-31T07:44:35.2218091Z             'interface',
2023-10-31T07:44:35.2218330Z             # 'io',
2023-10-31T07:44:35.2218544Z             'math',
2023-10-31T07:44:35.2218799Z             'numerical_simulation'
2023-10-31T07:44:35.2219066Z         ]
2023-10-31T07:44:35.2219220Z
2023-10-31T07:44:35.2219571Z         module_methods = lambda module: inspect.getmembers(module, inspect.isroutine)
2023-10-31T07:44:35.2220084Z
2023-10-31T07:44:35.2220262Z         module_variables = lambda module: [
2023-10-31T07:44:35.2220669Z             (name, value) for name, value in inspect.getmembers(module)
2023-10-31T07:44:35.2221508Z             if (not inspect.isroutine(value) and not inspect.ismodule(value) and not name.startswith('__'))
2023-10-31T07:44:35.2222140Z         ]
2023-10-31T07:44:35.2222296Z
2023-10-31T07:44:35.2222621Z         for (submodule_name, submodule) in _recursively_find_children(kernel):
2023-10-31T07:44:35.2223085Z
2023-10-31T07:44:35.2223465Z             if submodule.__name__.split('.')[2] in exposed_kernel_modules:
2023-10-31T07:44:35.2223995Z                 kernel_module  = import_module(submodule.__name__)
2023-10-31T07:44:35.2224667Z                 exposed_module = import_module(submodule.__name__.replace('kernel.', ''))
2023-10-31T07:44:35.2225141Z
2023-10-31T07:44:35.2225357Z                 # Assert both modules contain the same methods
2023-10-31T07:44:35.2226037Z >               assert all([method in module_methods(kernel_module) for method in module_methods(exposed_module)])
2023-10-31T07:44:35.2226684Z E               assert False
2023-10-31T07:44:35.2227061Z E                +  where False = all([True, False, True, True, True, True, ...])
2023-10-31T07:44:35.2227412Z
2023-10-31T07:44:35.2227678Z tests/test_hybrid_module_exposure.py:51: AssertionError
2023-10-31T07:44:35.2228183Z =========================== short test summary info ============================
2023-10-31T07:44:35.2229034Z FAILED tests/test_hybrid_module_exposure.py::test_tudatpy_kernel_module_exposure - assert False
2023-10-31T07:44:35.2229781Z  +  where False = all([True, False, True, True, True, True, ...])
2023-10-31T07:44:35.2230318Z ============================== 1 failed in 4.46s ===============================
2023-10-31T07:44:35.8377782Z Tests failed for tudatpy-0.7.3.dev29-py39hd7dfd30_0.conda - moving package to /home/conda/feedstock_root/build_artifacts/broken

Addressed in https://github.com/tudat-team/tudatpy/pull/123

https://github.com/tudat-team/tudatpy/pull/123 has been merged.


stage1-fly: don't share /dev,/proc,sys by default

If the user wishes these bindmounts to be shared, manually specifying
them should make it so again.
This is technically backwards incompatible, but in general I don't think additional mounts are pretty much ever created under these directories and I think that sharing them in the first place might have been an oversight.

After looking at this more, it seems this doesn't do anything on any modern systems since systemd just makes all those things shared on the host anyways (thanks for pointing that out @lucab!)

I agree with the whole idea behind this PR (we shouldn't be doing that by default), but given that this may break backward compatibility and should be a no-op on most of hosts (with systemd) I'm inclined not touch this until we have some way of overriding propagation mode per-mount.


chore(Topology/Algebra/InfiniteSum): split up large file

Split up Mathlib.Topology.Algebra.InfiniteSum.Basic (1600 lines) into smaller files.

bors r+


Deploy to Pypi

Lets see if this works

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@fda33a6). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage          ?   96.42%
=========================================
  Files             ?        7
  Lines             ?      140
  Branches          ?        0
=========================================
  Hits              ?      135
  Misses            ?        5
  Partials          ?        0

Impacted Files
Coverage Δ

ouroboros/ouroboros
100% <100%> (ø)

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 fda33a6...d7a360d. Read the comment docs.


Spyder error on opening object in Object Explorer while running debugger

Description
I got this error report on my work machine, but could not reproduce it after several attempts. Feel free to close of course if you can't either, but I figured I'd submit it in case it was helpful.
Was in ipdb, on a line of a function, and doubleclicked on an object to open it, a list with two strings (the first being the empty string ""), and immediately received the error. However, the dialog opened successfully, Spyder continued to function normally, and nothing looked amiss. In an immediately previous debugging session on I believe the same code, I'd opened another object and didn't receive the message. Also, I had two variable explorer windows open in the background from a previous debugging session open (both of the same object). I attempted to open other objects afterwards, as well as the same object that cause, and they all worked with no errors reported.
What steps will reproduce the problem?
Attempted to reproduce by setting the same breakpoint, rerunning the debugger and opening the same object, but everything worked fine. I even tried opening some other objects in another debugging session, and in the main scope as before in case that was the issue, but to no avail. Therefore, unlikely this will be replicable/fixable unless the below automatic call stack trace provides any insight.
What is the expected output? What do you see instead?
Expected: To not see the error message.
Actual: Crash report-type error message issued (see below). Otherwise, everything appeared to function normally before, during and after.
Please provide any additional information below
Automatic error trace:
  File "C:\Users\christopher.gerlach\Anaconda3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 573, in setEditorData
    value = self.get_value(index)
  File "C:\Users\christopher.gerlach\Anaconda3\lib\site-packages\spyder\widgets\variableexplorer\collectionseditor.py", line 1362, in get_value
    return self.get_value_func(name)
  File "C:\Users\christopher.gerlach\Anaconda3\lib\site-packages\spyder\widgets\variableexplorer\namespacebrowser.py", line 295, in get_value
    value = self.shellwidget.get_value(name)
  File "C:\Users\christopher.gerlach\Anaconda3\lib\site-packages\spyder\widgets\ipythonconsole\namespacebrowser.py", line 94, in get_value
    raise ValueError(msg)
ValueError:     return dir_names

Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Windows 10 x64

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)

Thanks for reporting. I think we could safely remove the block code that generates this error.
But I'll have to carefully look at it :-)

No rush if its a nontrivial task; try as I might I was unable to reproduce the error, have only ever seen it once, and it had no apparent adverse effect.

@impact27 is this fixed with any of the work you did on the debugger?

Well as the code changed significantly since this issue I would say we can close it.


Nick: Watch samyakinfotech.com

Nick requests the watch of the watch_keyword samyakinfotech\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cocomac in Charcoal HQ

samyakinfotech\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Debugger hangs when inspecting contents - possible too large collection

I put a halt in a method, and then attempted to examine the collection.  When I click on the little arrows of any of the far left objects, nothing happens.  I believe it's trying to do something.  There are 244 objects in the collection.

I wait for a few minutes, then log out, and I get a Broken Promise exception which (sounds very sad).

Martin and I ran into an issue that sounds like what you hit. There is a situation where RSR can deadlock due to an interaction w/ Ephemeron finalization. The deadlock occurs in a critical section which is growing a Dictionary. The Dictionary growth is enough to trigger the GC causing the issue. There is a ticket over in the RSR project to work around this problem. See: https://github.com/GemTalk/RemoteServiceReplication/issues/108

Note, the only limit on size of collection should be memory. RSR doesn't have an explicit upper-bound on size of an object.

I think this is fixed, at least I can inspect a 10000-element Array in the debugger without problem. Please re-open if the GroverCollection example still fails.
I tested with this code:
| array |
array := (1 to: 1000) collect: [:index | 'Element ' , index printString ].
^ array at: 0.