Bug in chatette.parsing.utils

First of all really nice library! I did however notice a small bug in the chatette.parsing.utils file
On line 49-56 it states the following:
escapable_chars = [
ESCAPEMENT_SYM,
COMMENT_SYM, OLD_COMMENT_SYM,
UNIT_START_SYM, UNIT_START_SYM,
RAND_GEN_SYM, RAND_GEN_PERCENT_SYM,
VARIATION_SYM,
ARG_SYM,
CASE_GEN_SYM,
]
However, line 52 repeats UNIT_START_SYM, UNIT_START_SYM twice. Instead it the second instance should be replaced with UNIT_END_SYM.
Thus it should be
escapable_chars = [
ESCAPEMENT_SYM,
COMMENT_SYM, OLD_COMMENT_SYM,
UNIT_START_SYM, UNIT_END_SYM,
RAND_GEN_SYM, RAND_GEN_PERCENT_SYM,
VARIATION_SYM,
ARG_SYM,
CASE_GEN_SYM,
]

Hello,
Thanks a lot for this very precise bug report :)
I'm fixing this right away. I'll also turn that list into a set, as it is never iterated over.

This is fixed on dev branch and will be included in the next release.
Thanks again for the heads up!


Factory reset the Sharepoint (database)?

If an admin mistakenly deleted or renamed SharePoint lists, is there a way to restore the TEOC to its original state or redeploy it with minimal effort? Is there any option to revert the changes?
When I click on create incident it gets stuck, it seems something is missing

Hi @tofighi,
Thanks for reaching out to us.
If a list is deleted, the admin can refresh the app and the list will be re created on load.
If a list is renamed you can try reverting the change and or deleting the list so it will be created again as per above logic.
Please let me know if you need more information.
Thanks.


cache result of toPrettyChars()

toPrettyChars() has some egregious faults:

it is called over and over again on the same symbol, consuming more memory each time
it allocates memory twice for its components (not very nice)

This fixes both problems by 1. caching the result and 2. caching the internal temporary results.

Is 1. an actual problem? I mean toPrettyChars is only used in diagnostic messages.
As it was only used temporarily until now, we should free the string after usage instead of caching it, using even less memory.

toPrettyChars is only used in diagnostic messages

I instrumented it, and compiling std.uni for unit tests, for example, called it 29,000 times, multiple times per function.

we should free the string after usage

That's a much more disruptive change. This is low hanging fruit, and does not affect anything outside of the function.

nice work

Auto-merge toggled on


fix memory example in usage

Issue #, if available:
N/A
Description of changes:

Fixed example in usage to use plain int for memory interpreted in GiB.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #39 into master will decrease coverage by 1.16%.
The diff coverage is 82.91%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
- Coverage   90.07%   88.90%   -1.17%
==========================================
  Files           8        9       +1
  Lines         877     1055     +178
==========================================
+ Hits          790      938     +148
- Misses         57       81      +24
- Partials       30       36       +6

Impacted Files
Coverage Δ

pkg/selector/types.go
100.00% <ø> (ø)

pkg/selector/selector.go
84.51% <56.25%> (-0.94%)
:arrow_down:

pkg/cli/types.go
95.23% <77.27%> (-4.77%)
:arrow_down:

pkg/cli/cli.go
86.00% <82.35%> (+0.51%)
:arrow_up:

pkg/cli/flags.go
87.70% <83.33%> (-2.82%)
:arrow_down:

pkg/bytequantity/bytequantity.go
87.30% <87.30%> (ø)

pkg/selector/aggregates.go
92.45% <100.00%> (ø)

pkg/selector/comparators.go
91.46% <100.00%> (+0.43%)
:arrow_up:

pkg/selector/outputs/outputs.go
91.81% <100.00%> (ø)

... and 1 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 ea271e8...af66b8c. Read the comment docs.


removing Data.Aeson.Types imports to be compatible with aeson v1

Data.Aeson.Types exposes an Options datatype that the compiler isn't able to disambiguate from Auth0.Management.Connections.Options. It was added in aeson v1.0.0, so any uses of this library need these changes to be compatible with it.

Hello @JBetz - thanks for taking an interest in improving this library.
If you could redo this commit so that it doesn't add a large number of unrelated formatting changes I'd be happy to merge it in.

Done.

Thanks. I squashed the commits down into one for you.


Unable to get profile image url

GoogleSignInAccount acct = result.getSignInAccount();
Log.d("Image",acct.getPhotoUrl()+"");
getPhotoUrl() return null value .

Is this a user that definitely has a public profile picture set on their account?

@swativishnoi closing this issue as there has been no activity in a few months and I believe this is is working as intended.  Profile photo can be null if the user does not have a public photo.

yes ..  working perfectly
thanks


ACNCD_DataConnectors_sub3

Fixes #
Proposed Changes

Okta Single-Sign On
Sophos XG Firewall

Closing as this was updated and resubmitted in #767


[UI] As a system user I want to change the forecast base date so I can perform analysis on multiple forecasts

Acceptance Criteria
This applies to all screens.

A date picker exists in the header to select forecast_base_time.
When selected, API should be re-queried to fetch data for this forecast_base_time
Dates can only be in the past and must be either midday or midnight
Add a label next to the date picker showing the date range
Remove the base date text from the top of the summary page and the forecast range
e.g. https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/date
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/time

Testing is through playwright with mocked API's
This will affect the summary and city pages
No automated e2e tests as part of this ticket
Test Checklist:

[ ] Test Analysis & discussion with developer
[ ] Write Test Cases / Charters
[ ] Test Cases & Charters Reviewed
[ ] Test Cases Executed
[ ] Test Charters Executed
[ ] Regression tests executed
[ ] Automation Tests added to project

Test Analysis

Verify date picker is in the header

city view
summary view
Automation tests in banner.spec.ts checking it exists

Verify date picker triggers the correct API calls

Automation test to check the forecast request on page load vs after selecting date
Assert the forecast base time changes in the request
Assert the label next to the date picker updates correctly

Verify dates can't be in the future
Verify dates can only be 12 or 00

what happens if system time is set to 11 and you query 12 for today?

Remove test that asserts on forecast base time text
Verify the UI is updated with new mocked data when changing the base date

mock the response from the API for page load
change the date
mock the response of the API with the different date
Summary page - check the table updates correctly
City page - snapshots should change

setBaseTime POM code written for citypage , can share and duplicate for summary.
Existing setBaseTime method is takes in a string, so we can actually force a future date and time to see what happens.

Otherwise, no further points. Looks good 💯

Regression Tests Passed ✅

Backend system tests

Frontend system tests

Playwright Regression 25.07.24.pdf

Finding - Can send a future date via keys
Relevant AC:

'Dates can only be in the past and must be either midday or midnight'

Steps to reproduce:

Click in the textbox next to the date picker
enter a future date
check the network requests

Expected:

Unable to provide a future data

Actual:

API requests the data
Displays empty graphs & tables

failing regression tests on merge

Regression -  31/07 PASSED ✅

Frontend:

Playwright Regression 31.07.24.pdf

Backend:


Implementing waveform extraction

Hello Neuronal Ensemble,
I've been working on a function that takes an AnalogSignal as input, finds spikes by threshold and returns a SpikeTrain object of the spike peaks. I also want to extract the waveforms of each spike in an interval around the peak but I am confused about the usage of SpikeTrain.waveforms.
By the neo doc: waveforms: (quantity array 3D (spike, channel_index, time)) The waveforms of each spike.
Therefore my .waveforms attribute has the shape (n, 1, i), where n is the number of spikes, 1 is just 1 because I have only 1 AnalogSignal and i is the extraction interval passed to the function. Is that the intended use of .waveforms?
Thanks,
Daniel

Shouldn't time be a range of time relative to the location of the spike in the train?

I think this is best answered by @samuelgarcia, but my understanding is that you are correct. The reason for it being 3D is that with multielectrode recordings it is possible to have a spike recorded on more than one channel, but in the case of intracellular recordings or simulations there will only ever be a single channel.

That is a good hint, @apdavison, I did not consider that multielectrode recordings can contain the same spike in different channels.
@rgerkin, time is a time range in my implementation. Specifically, each element of time is a sampling point of the waveform.


Markdown - Lengzhao Blog

https://lengzhao.github.io/2020/06/03/markdown/?
Every failure is leading towards success.

test


Added: Reflection fallback for Object Property Lookup

Added:     DateTime output type
Replaced: String return value with Stream for larger Templates and better write performance
Added:     MaxSize option for Truncated Templates (for better Web usage)
Architecture:
Added:     option object for better scalability of future features
Added:     option object to be be accessible in all parts of Template Generation.
Added:     Encoding of Template in Stream

overall performance improvement due to usage of streams instead of string concatenation.


Good but a little unclear

A really helpful guide, however it's not clear whether one needs to install the sdk and then the runtime. Are both required or just one?

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

ID: 38f33bad-b152-c53b-1b0a-234c9a075bd7
Version Independent ID: 565e7432-c4b3-a1c2-8ce3-c8c1a0d8ebe8
Content: Install .NET Core on Ubuntu 18.04 package manager - .NET Core
Content Source: docs/core/install/linux-package-manager-ubuntu-1804.md
Product: dotnet-core
GitHub Login: @Thraka
Microsoft Alias: adegeo

Thank you for the feedback. When we get to updating this soon, I'll put a sentence in each that describes that the sdk = developing+running apps and the runtime = running apps.


Enhancement: Transform Zero validator should allow Deltas

Is there an existing issue for this?

[X] I have searched the existing issues.

Please describe the feature you have in mind and explain what the current shortcomings are?
In Blender, the "Transform zero" validator wants us to apply a freeze transform, this is useful.
But there are 2 ways to achieve that goal, and this validator wants us to use the method that cannot be used in most productions.
The method we need to use (but is refused by the validator) :
(from inside Blender) Object-->Apply-->All Transforms to Deltas
It transfers the transforms to the Deltas of objects (it places the pivot at the center of the object, so you can use it for animation).
The method that Ayon's Blender validator wants us to use :
(from inside Blender) Object-->Apply-->All Transforms
It transfers the transforms to the datas of objects (it places the pivot at the center of the world, so you cannot use it for animation).
How would you imagine the implementation of the feature?
Here is the line :
https://github.com/ynput/OpenPype/blob/529ab6ca9bf0de6656fa11ba5f4e38af4b048049/openpype/hosts/blender/plugins/publish/validate_transform_zero.py#L35
It is currently using "matrix_basis", I hope there is a way to do the same thing while allowing deltas.
https://docs.blender.org/api/current/bpy.types.Object.html#bpy.types.Object.matrix_basis
If not possible using matrix, a brute force solution would be to test translation and rotation to be 0,0,0 and scale to be 1,1,1
Are there any labels you wish to add?

[X] I have added the relevant labels to the enhancement request.

Describe alternatives you've considered:
No response
Additional context:
No response

Sorry, I don't find a wauy to add the "Host: Blender" label on this issue.

In case some studios prefer the current behaviour, it would be even better to have a button that allows to select the behavior we prefer.

In case some studios prefer the current behaviour, it would be even better to have a button that allows to select the behavior we prefer (allow Deltas or not).

This would be a studio settings > project settings, yes?

This would be a studio settings > project settings, yes?

Yes, I mean to have a new button here, to choose between "Allow Deltas" or not :

Studio  Settings-->Blender-->Publish Plugins-->Validators-->Validate Transform zero
Project Settings-->Blender-->Publish Plugins-->Validators-->Validate Transform zero

Currently it only has those 2 options :      "Active", "Optional".

@simonebarbieri could be an easy fix, no?

I recall actually implementing this some time ago or testing it - but I can't find the code :D
I had this comment. Apparently my code is already quite different from develop so it might be a decent starting point.
Looking back now, my focus seemed mostly on adding a repair method at the time.

I would like to add that this feature is not as important for us as it was.
Because we changed our realtime tools to allow working with Ayon's current behaviour.


Testing Open-CE recipes

This is the matrix to ensure builds and test of all the packages in various environments are working fine after merging CUDA11 PRs.

Architecture
Containers/Baremetal
CUDA and CPU
Python
Build Status
Test Status
Owner

PPC (RH7)
Containers
10.2
3.6
Completed

Brad

PPC (RH8)
Containers
11.0
3.8
Completed
In progress
Nishidha

PPC (RH7)
Baremetal
10.2
3.6
Disk Quota Error

Deepali/Brad

PPC (RH8)
Baremetal
11.0
3.6
Blocked

Nishidha

X86 (RH7)
Containers
10.2
3.6,
In Progress

Deepali

X86 (RH8)
Containers
11.0
3.7
Completed
In progress
Nishidha

X86 (RH7)
Baremetal
10.2
3.8,
Completed
In progress
Nishidha

X86 (RH8)
Baremetal
11.0
3.6,
In Progress

Deepali

*Blocked due to space issue.

Python column can be updated as and when we test other versions too for given environment.

Closing this as the work items are released.

Closing this as the work items are released.


Password grant authorization for swagger-ui fails

I have an application setup with jersey controller, swagger-jaxrs2 for swagger-ui documentation and I'm trying to work with Resource Owner Password Flow using the @SecurityScheme annotations.  I am receiving the request for authorization, and sending back a response but for some reason I can't get the Swagger-UI to authorize.
Response
{
    "access_token": "token",
    "scope": "api",
    "token_type": "bearer",
    "expires_in": 3600
}

The issue was on my end, I was not processing the request properly and was sending back a token with a null subject claim.  Nothing to see here.


Let the height of the window be dynamic

For our current setup, the current height of 10 is too small. Setting it to zero lets dialog decide what to do.
This works even on a small screen (at least for my setup).

Please squash your commits and rebase.

Ah never mind LGTM


[top_earlgrey/dv] Fix timeout in chip_sw_keymgr_key_derivation_vseq

The default timeout was too short; see the added comment for details.
This resolves #23050, and it resolves #23084.

CI checks related to DV and lint have passed; the FPGA checks aren't relevant for this DV-only change, so skipping them


[SPARK-13992][WIP] Add support for off-heap caching

This patch adds support for caching blocks in the executor processes using direct / off-heap memory. I'll expand the description later; this is WIP and blocked behind a large series of smaller patches (opening now to test + be able to use GH PRs to track tasks).

Test build #53483 has started for PR 11805 at commit feffc20.

Test build #53483 has finished for PR 11805 at commit feffc20.

This patch fails Scala style tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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

Test build #53819 has started for PR 11805 at commit 222d80b.

Test build #53819 has finished for PR 11805 at commit 222d80b.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #53825 has started for PR 11805 at commit bf62983.

Test build #53825 has finished for PR 11805 at commit bf62983.

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

Merged build finished. Test FAILed.

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

Test build #54096 has started for PR 11805 at commit 5a15de2.

Test build #54096 has finished for PR 11805 at commit 5a15de2.

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

Merged build finished. Test FAILed.

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

Test build #54125 has started for PR 11805 at commit 3c122af.

Test build #54125 has finished for PR 11805 at commit 3c122af.

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

Merged build finished. Test FAILed.

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

Test build #54215 has started for PR 11805 at commit 334b727.

Test build #54215 has finished for PR 11805 at commit 334b727.

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

Merged build finished. Test FAILed.

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

/cc @andrewor14, @nongli, @rxin, can you glance over the PR description to check whether the user-facing semantics are okay for now? We might want to change some bits later (maybe to add separate nice shorthands for off-heap-memory-only vs off-heap-memory-and-disk), but I'd like to understand whether any semantics need to change here.

Test build #54239 has started for PR 11805 at commit 4d9489a.

Looks pretty good actually. There are some details about naming and on/off defaults we need to figure out, but that can go in later.

Test build #54239 has finished for PR 11805 at commit 4d9489a.

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

Merged build finished. Test PASSed.

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

Test build #54272 has started for PR 11805 at commit df8be62.

Test build #54272 has finished for PR 11805 at commit df8be62.

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

Merged build finished. Test PASSed.

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

Discovered a potentially major problem: if we bail out of a getOrElseUpdate call and don't end up caching a block and then do not fully consume or dispose of the resulting valuesIterator then we might leak off-heap memory. This problem didn't exist before because we'd never use off-heap memory for unroll space. I may have to add some task completion callbacks to dispose of PartiallySerializedBlocks.

Test build #54388 has started for PR 11805 at commit 33c9d9b.

Test build #54400 has started for PR 11805 at commit 74427e9.

Test build #54401 has started for PR 11805 at commit 8a0f944.

Test build #54404 has started for PR 11805 at commit 42d0356.

Alright, this should now be ready for an initial review pass. I just pushed a somewhat hacky patch to work around the direct buffer allocation limit (42d0356) and added a few simple tests. I also addressed the potential leak of direct memory in cases where partially serialized blocks' iterators aren't fully consumed.
I'm sure that this probably needs a bit more work and I'll do more self-review tomorrow once I've had time to set this aside for a bit.

/cc @andrewor14 @davies for review

Test build #54388 has finished for PR 11805 at commit 33c9d9b.

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

Merged build finished. Test PASSed.

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

Test build #54404 has finished for PR 11805 at commit 42d0356.

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

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

Merged build finished. Test FAILed.

Test build #54400 has finished for PR 11805 at commit 74427e9.

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

Test build #54401 has finished for PR 11805 at commit 8a0f944.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Have took a pass on this, looks good to me. Besides unit tests, have you done some stress tests on this (both on heap and off heap)?

Test build #54561 has started for PR 11805 at commit a5e9f0d.

Test build #54561 has finished for PR 11805 at commit a5e9f0d.

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

Merged build finished. Test PASSed.

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

@JoshRosen Overall looks good. I have not verified comprehensively whether all the things are disposed and released correctly, but it looks OK from what I can tell. If there are actually leaks I'm sure we'll find them quickly.

Forgot to mention, you'll need to update documentation for spark.memory.storageFraction. Right now its description is pretty specific to the on-heap stuff.

Test build #54678 has started for PR 11805 at commit 61920a9.

Test build #54678 has finished for PR 11805 at commit 61920a9.

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

Merged build finished. Test FAILed.

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

Test build #54681 has started for PR 11805 at commit fde020f.

Test build #54682 has started for PR 11805 at commit a604078.

Test build #54681 has finished for PR 11805 at commit fde020f.

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

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

Merged build finished. Test FAILed.

Test build #54682 has finished for PR 11805 at commit a604078.

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

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

Merged build finished. Test PASSed.

deep-review this please

@andrewor14 Review complete. In the future we should add tests to ensure buffers are released. @JoshRosen please file a JIRA for this issue.
LGTM.

Alright, I'm going to merge this into master and will file followup JIRAs to make sure that we update the user-facing documentation and add more tests before 2.0. There are a number of questions to answer about how best to expose the relevant off-heap configurations to users, but I'd rather address them separately.

Filed https://issues.apache.org/jira/browse/SPARK-14336 as followup


move the display of the bootspash to the menu

We don't want to run any code on europi import that we don't have to  because it can break things. In this case the docs build.
https://github.com/Allen-Synthesis/EuroPi/actions/runs/3243232272/jobs/5317644875#step:8:23
This change has the additional advantage of only showing the bootsplash when it is really needed (during the slow menu load) and not when it isn't (during a fast single script load).

Nice fix!


No Module called geotransformer.ext

when I run steup.py build develop, the process runs but after trying to run the next step i get an error saying no module named geotransformer.ext found

Hello, I have also encountered this issue. Have you resolved it？

Hello, I managed to solve it!
if you are using an isolated container or an environment, are you running the commands in the same or not?
because this module adds to the system path.
Since I was using two different containers, was giving me that error.
Hope this helps!


Wait for ConfigMap controller to sync upon restart.

ConfigMapController wil be first controller to process any CR. ConfigMapController is said to be synced only when nephe-config CR is received and processed by the reconciler.
Add ConfigMapController as dependent controller for CPA controller.

/nephe-test-e2e-kind


blink cmd dialog all the time

Environment

Tesseract Version:tesseract-ocr-w32-setup-v5.0.0-alpha.20190708
Commit Number:
Platform:  Windows, 64-bit

Current Behavior:
when i cmd "tesseract.exe --help",it resize the cmd dialog and cmd front family;then recoverd after a blink.
Expected Behavior:
Suggested Fix:

What you mean with

when i cmd
Do you know, how to work with command line on windows???


chore: update commit related dependencies, update CONTRIBUTING.md

Description of changes

Bump @commitlint dependencies to latest
Remove dependency on commitizen
Remove section on using commitizen in contributing guide
Update stale sections of the contributing guide

Issue #, if available

Description of how you validated changes
Manually checked that an invalid commit was rejected, and this commit was accepted.
Checklist

[x] PR description included

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

Codecov Report

Merging #10192 (3f523b3) into master (f2ac2b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #10192   +/-   ##
=======================================
  Coverage   54.27%   54.27%
=======================================
  Files         849      849
  Lines       46933    46933
  Branches    10004    10004
=======================================
  Hits        25471    25471
  Misses      19445    19445
  Partials     2017     2017

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Update CAPO docs and scripts

Bumped CAPI version
Bumped kubernetes version
Bumped yq version (v4 is now supported in CAPO)
Updated docs and added cleanup instructions

Thanks for the review @smoshiur1237 ! I'm a bit hesitant to remove the "you" here because we already had it in multiple places before my changes.
These docs are very specific for developers in our team, not general user guides, so I think it is ok to refer to the developer as "you". It makes it clearer what the developer can choose (e.g. cluster name), and what is common guidelines that everyone has to follow.


Release v1.2.1

support for cmk multi-region keys
Warn that deleting HTPasswd IDP with cluster-admin user will delete the admin
Add username & password requirements to the flags' help messages
Fix login error
Upgrade cluster to 4.10.* - add delay after roles creation
Only prompt for HTPasswd IDP name when actually creating a new IDP
add metric for throttle
supporting different regions for create service command
Fix Throttle issue for Operator roles

@renan-campos @vkareh  PTAL

/lgtm


CB-12546 Added Flink GCP templates

@ferenc-csaky thanks. One small thing, can you please squash together the commits?

@doktoric I squashed the commits, although there is a "Squash and Merge" option on the UI, which does the same, is it disabled for this repo?

retest this please

retest


Link conflict when using a mentioned contact as an anchor text

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a chat and write “@” to mention a contact
Select one contact and hit enter to mention from the suggestion list
Put the mentioned contact inside of “[]” and put a link inside of “()” next to it (e.g., “[@nathan@gmail.com](google.com)”)
Click enter to post the comment
Hover over the mentioned contact as an anchor text and observe the link and the contact
Click on the mentioned contact as an anchor text

Expected Result:
Mentioned contacts should not be used as an anchor text for a link to avoid link conflict.
Actual Result:
Mentioned contacts can be used as an anchor text for a link and when hovered over, they show both the link and the contact. When clicked on, they open both the link on a new tab and the mentioned contact on RHN.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/1703a401-c946-4a03-9b98-17165d41030a
https://github.com/Expensify/App/assets/43996225/731ec41d-5cf8-4b30-ba40-0b6cba9acadb
Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1683805364578509
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
When sending a message with mention inside a link markdown, the message will be converted to both mention and link. When you press it, it will open both the link and the user/personal detail.
What is the root cause of that problem?
When we send a markdown message, it will be converted to html based on the rules defined in ExpensiMark. The rules will be executed from the top to bottom. Let see what happen when we send [@bernhardoj](google.com).

The first 3 rules are skipped. At the userMentions rule, it detects that we have a mention, that is @bernhardoj. The text now becomes [<mention-user>@bernhardoj</mention-user>](google.com)
https://github.com/Expensify/expensify-common/blob/5f4672c84e19122fc77b4bf1ac092d4919472112/lib/ExpensiMark.js#L74-L83

Then we will encounter a link rule which will convert the text to <mention-user>@bernhardoj</mention-user>
https://github.com/Expensify/expensify-common/blob/5f4672c84e19122fc77b4bf1ac092d4919472112/lib/ExpensiMark.js#L106-L126

We can see that we have 2 tag, link (anchor) and custom mention tag.
What changes do you think we should make in order to solve the problem?

Don't convert the text to mention if it's in a link markdown.

We should change the order of the rules such as the email and link rule run first before mention rule (hereMentions and userMentions).
Then, we will update the mention regex to not match if the text contains the closing tag of anchor ().
(?![^<]*<\/a>) <- append this to the end of the regex for both hereMentions and userMentions.

Also, I find it's weird that we accept (`||
) but later on in Str.isValidMention, we return false if the mention contains those characters. Instead of doing that, we can just prevent the regex to match in the first place like we do with anchor tag.
(?![^<]*(<\/pre>|<\/code>|<\/a>))
This is what we do on other rules too, for example:
https://github.com/Expensify/expensify-common/blob/5f4672c84e19122fc77b4bf1ac092d4919472112/lib/ExpensiMark.js#L134-L136

Reproduced

Yes, let's handle it in one PR

Assigned @bernhardoj since he's handling this with that PR

@eVoloshchak bump to complete the BZ checklist

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/expensify-common/pull/532

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/expensify-common/pull/532/files#r1213482401

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: I don't think additional discussion is needed, we already have an appropriate item in the reviewer checklist

Determine if we should create a regression test for this bug. We already have a couple of unit tests to catch this, no need for manual regression testing


Add other resource precheck functions

Changes introduced by this PR

Add Precheck helper functions that call the corresponding new precheck RPC introduced in https://github.com/microsoft/moc/pull/269 for the following services:
Identity (security)
Availability set (compute)
Gallery image (compute)
Virtual machine image (compute)
Load balancer (network)
Logical network (network)
MAC pool (network)
Network interface (network)
Network security group (network)
VIP pool (network)
Virtual network (network)
Container (storage)

/azp run


Add examples of use case in feature descriptions

Codecov Report

Merging #210 (14f8c54) into master (f15ac99) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #210   +/-   ##
=========================================
  Coverage     67.44%   67.44%
  Complexity      718      718
=========================================
  Files           114      114
  Lines          2436     2436
  Branches        247      247
=========================================
  Hits           1643     1643
  Misses          689      689
  Partials        104      104

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 f15ac99...14f8c54. Read the comment docs.


Can't load mediapipe task-vision inside a web worker

I need to load mediapipe task-vision in a web worker for an electron app. I tried all sorts of ways and nothing worked. Is there an example of a proper way to load the task-vision bundle in a web worker so I can run all the detection functions inside the web worker?
Thanks!

Hi @desi-dimitrova,
Currently, the web worker is not very familiar with our task API. However, we are marking this as a feature request and will share it with the team to ensure it is on our radar. We will consider making it more user-friendly if there is more demand in the near future.
Thank you!!


implement broadcast variables

Some jobs have large amounts of constant data that needs to be available to all tasks. One way to implement this would be to wrap such values with a Broadcast class that when pickled saves the wrapped value to bucket storage (perhaps named according to hash of value) and when depickled reads the value

Sketch of how this could be used
client = kubeface.Client(…)
big_training_data = pandas.read_csv(“path_to_training.csv”)
broadcasted_training_data = client.broadcast(big_training_data)

def train_a_model(data, model):
  unwrapped_data = data.value  # data is a kubeface.Broadcast
  return model.fit(unwrapped_data)

models = grid_of_models_to_try()
results = client.map(functools.partial(train_a_model, broadcasted_training_data), models)

I think you were right to close this ( @julia326 implemented broadcast vars in #28 )


[Candidate 3v3 Subjugation Mk 14 [WBC Winner]] map.str contains script messages. Are there custom event scripts on map?

Issue Description
map.str contains script messages. Are there custom event scripts on map?
SCRIPT:Sell your Demo Trap to vote against the Firefly FX
"Sell your Demo Trap to vote against the Firefly FX"
End

SCRIPT:Vote Number
"Number of votes not counted"
End

SCRIPT:Vote Count
"Vote tally x2"
End

At least half did not sell their Demos, the Fireflies will remain
"At least half did not sell their Demos, the Fireflies will remain"
End

More than half sold their Demos, the Fireflies will be removed
"More than half sold their Demos, the Fireflies will be removed"
End

Expected behavior
No event scripts in ranked maps.

This was in a previous version of this map, I'm quite sure the mapper removed this for it to become WBC winner, he is already planning to edit this map himself but I will mention to him to remove these text files.

He has confirmed that these scripts no longer work.


Add option to select no-dir-clues before Across/Down.

Hi!
A Feature request which perhaps sounds a bit specialized... But for my music quizes which  try to follow swedish national radio "musikkrysset"  standard, it would be very convenient.
You can see "Sveriges Radio" solution here:
https://sverigesradio.se/sida/artikel.aspx?programid=2078&artikel=723047
And they don't even have any support for turning words entries. I have a mission to make them fix that, and one way should be to recommend them using your program. I think they have 1.7 million listeners every saturday :-)
And I think many users should miss your wonderful feature of following the no-dir-clues direction when they enter words, if they have to click three times to get to "no-dir-clues-mode"
Test:
https://fripp.changeip.org/melodikryss/melodikryss1/index_clues.html
And click on no-dir-clue 21. You need to click three times to get to the one "you want"
/Erik S

Hi Erik,
In your puzzle, the "down" lights at 1 and 21 are actually extended and supplanted by the nodir clues, right?
So, I'll create a way for you to specify that certain clues (1d and 21d in this puzzle) should be ignored: they will never get highlighted in the grid.
Additionally, I can add an option to prioritize nodir clues when having to change clue direction.
With these two, you should be able to get the behaviour that you are looking for, I think.

I agree! These two fixes should be both logical and solve my "problem". Great!

I have checked in v0.89. You should be able to get these features from it.
To mark clues such as 1d and 21d as "deleted:, use something like this (https://github.com/viresh-ratnakar/exolve#deleted-clues):
exolve-down:
1 *
21 *
I was going to add an option to set direction priority. But I realized that this is better done by using the order of the sections as specified in the specs (it had actually been a pending thing for me to fix: if someone lists the nodir section before the down section, and both of these before the across section, for eg., then that should be the order shown when displaying the clues).
So, in your puzzle, just make the nodir section appear before exolve-across and exolve-down (you can leave them blank if needed). See https://github.com/viresh-ratnakar/exolve#order-of-rendered-clue-lists

Just perfect!
I use two new options together: The "~" no remove the numbering together with the "*" to skip the clue.


Email security audit service

https://www.emailprivacytester.com

~github/email-privacy-tester~
https://gitlab.com/mikecardwell/ept3

0 test pass with Thunderbird, Rainloop and K-9 Mail :muscle:


Styling Task - Navigation bar for recipe pagination partial

lightly style the navigation link elements. Simple things like a colored small box or simply good spacing. Try to add a indication for the current 'active' link. General UI goodness

Completed


i18n(fr): Add components/asides.mdx in French language

Description
Add components/asides.mdx in French language

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
components/asides.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Which backend for imx7dsabresd ( NXP MCIMX7 Sabre Board) in fullscreen mode

Hi,
we want to evaluate performance of the webkit running on imx7dsabresd on the NXP Sabre board.

We want to launch an website in fullscreen mode using the hdmi port.
This board does not support any hw accelleration.

Which backend and configuration should we choose. I read the wiki but I still don't know.
Thanks a lot

Probably easiest approach towards this setup would be to use Mesa with swrast, and WPEBackend-fdo on top of it. This would enable CPU-based rasterization.

I have a similar setup to this but I've been trying to run wpewebkit on top of a weston compositor(for display management), I've built mesa with just gallium swrast support only as my hardware(gma500 based) does not support hardware acceleration either.

From there on, what's missing is integration of displaying software-rasterized buffers via the display system. Cog must gain support for handling such exports, and then also find the proper way to handle their display.
In your case, per your description, it might be possible to use the DRM backend, copying the exported CPU memory containing rendered Web content into writable gbm_bo objects, and then display those via DRM/KMS.

I've been attempting to run cog on top of weston using the DRM backend and pixman software rendering, is that a setup that should currently work or is cog missing some features needed still(I'm basically running master branches for all the wpewebkit project related software)? A similar setup to this seems to work fine for different hardware I support that does have hardware rendering(I'm running weston without pixman and building mesa with the i915 driver there but otherwise it's basically the same as what I'm trying to do with the gma500 hardware).

Has this made any progress?


iOS 12: Use the Network.framework for socket connections

Hey there!
I want to start a general discussion on implementing the Network.framework for iOS 12+ targets. As presented during the WWDC 2018, the new framework offers great benefits over the current native socket handling by having clean states and handling the waiting state more performant.
While exposing the Network.framework, Apple is also planning to deprecate the SFSocket and all related API's. They did not so far (but mentioned they may do over the next view betas) during the session.
Read more about the new API in the official API docs.

I just watched  the WWDC 2018 Presentation on Network.framework and immediately came here to see what kind of plans are in store for Starscream. Not only are there perf benefits (mentioned above by @hansemannn), but there's also a big focus on connection durability across different network environments and switching between them (wifi->cellular).

I had same idea, that utilising Network.framework Starscream could be more stable. Because behind the scenes now there are really old C type APIs.
A while back I got survey from Apple about Networking. I guess this is the outcome of that survey.
I think soon I will start digging more into this Network.framework and can it be used with Websockets.
I have seen SwiftNIO Websocket implementation by Vapor guys https://github.com/vapor/websocket but it's not for iOS.

hey all. I agree the network framework is something we should probably look at implementing in Starscream. This of course is going to bring up backward compatibility issues, so I want to be careful on how to handle the transition. All that being said, I think it will be fairly straight forward to implement the Network framework since Starscream was setup back in version 3.0.0 to take in different network adapters. I plan to start working on and releasing this in a few weeks.

Wow, very exciting! Thanks @daltoniam

One thing we need to take in consideration is that Network.framework doesn't support watchOS.

Is it possible some functionality is broken in Starscream 3.0.5 on iOS 12? Just tried some SocketIO stuff (uses Starscream) on iOS 12 and it does not work. Could be my fault, but just wondering.

@gimlids can you tell what kind of errors are you having?

Sorry, not having the issue anymore I think this was my fault.
On Mon, Sep 24, 2018 at 3:34 PM Kristaps Grinbergs<EMAIL_ADDRESS>wrote:

@gimlids https://github.com/gimlids can you tell what kind of errors
are you having?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/daltoniam/Starscream/issues/521#issuecomment-424097706,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALXpUJgbnk2x_falGW_Mo1OlEx4GM75ks5ueTPagaJpZM4Up3Ye
.

@tamastimar Current version works well enough atm. And Network.framework is available only from iOS, tvOS 12.0+ and macos 10.14+. That makes life harder a bit.
Also I'm considering working with https://github.com/apple/swift-nio-transport-services

Hey y'all. #653 should add support for the Network.framework. I'm going to close this issue out. If you have any questions, feel free to ask on the PR, so we can track everything over there.


Only remove runs from GUI after delete which actually got deleted

GUI automatically deleted run rows in the list even though it can be the case that the server did not remove a particular run. After page reload or filter change the "deleted" run appears again.

Implement only after #1307 has been merged.

This has been fixed.


Hide parameters that are not supported in the generic step extension config panel

Describe the Bug
If you open up the config panel for a step you will often find bold labels which (from a user pov) look like headlines or section headers but instead these are labels of parameters which cannot be changed in the generic step extension. This is a confusing experience for users.
I suggest to not show these parameters at all or at least not making them appear in bold.
Steps to Reproduce the Bug or Issue

drag a component on the canvas (for instance git)
click on the component
look at the config panel

Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 0.30.0]

For 1.0.0, just remove it or add some warning as to why they can't be configured.

So we'd want to hide the non-primitive parameters as we have no way to let user configure it, correct? For example Type parameter on camel-git has GitType as a type. We might exclude by field types in uniforms. We might able to add enum support, but would be for post 1.0.

So we want to hide the non-primitive parameters as we have no way to let user configure it, correct?

hide or make them smaller with an information mentioing that user needs to go to source code editor to edit it (/!\ in thi case, check that the data is not removed the code)

Removing object properties was easy
diff --git a/src/components/JsonSchemaConfigurator.tsx b/src/components/JsonSchemaConfigurator.tsx
index 1f87a659..a0fdb689 100644
--- a/src/components/JsonSchemaConfigurator.tsx
+++ b/src/components/JsonSchemaConfigurator.tsx
@@ -1,8 +1,8 @@
+import { CustomJsonSchemaBridge } from './CustomJsonSchemaBridge';
 import Ajv from 'ajv';
 import { useRef } from 'react';
 import { AutoForm } from 'uniforms';
 import { AutoFields, ErrorsField } from 'uniforms-patternfly';
-import { CustomJsonSchemaBridge } from './CustomJsonSchemaBridge';

 const ajv = new Ajv({
   allErrors: true,
@@ -26,6 +26,11 @@ type JsonSchemaConfiguratorProps = {
   onChangeModel: (configuration: unknown, isValid: boolean) => void;
 };

+function getOmitFields(schema: any) {
+  return Object.keys(schema?.properties).map((key) =>
+    schema.properties[key].type === 'object' ? key : undefined
+  );
+}
 export const JsonSchemaConfigurator = ({
   schema,
   configuration,
@@ -46,7 +51,7 @@ export const JsonSchemaConfigurator = ({
       }}
       data-testid={'json-schema-configurator'}
     >
-      <AutoFields />
+      <AutoFields omitFields={getOmitFields(schema)} />
       <ErrorsField />

     </AutoForm>

But maybe better to show it in gray to indicate the prop exists but cannot configure. Will look a bit more.

Succeeded to bind a custom field markup, but struggling to apply "disabled" style

The style specified here is somehow ignored...
https://github.com/igarashitm/kaoto-ui/commit/aad212da5d6c382998eeeeef015fb3425076844f#diff-de1b89306e97afbdd2e03dd06330677116153a86aadbe2a2c22bf99134ee4b06R11
while I do see it in final HTML

  Task executor
  <button aria-describedby="form-group-label-info" aria-disabled="false"
    aria-label="More info for field" class="pf-c-button pf-m-plain pf-c-form__group-label-help"
    type="button" data-ouia-component-type="PF4/Button" data-ouia-safe="true"
    data-ouia-component-id="OUIA-Generated-Button-plain-8">
...
  </button>

@kahboom @lordrip any idea?
P.S. I'm a CSS dummy

@igarashitm Maybe it's because of the styles priorities, that being said, what do we want to achieve? a different color for the text?

@igarashitm So, just to be clear you don't want the popover button to be disabled, you are just wanting to show the text in grey so that it looks like it's not an active property, right? Maybe try to add that class to a  around the text instead of at the
 level.

@kahboom exactly, and also gray out the background. I just tried with  without luck... in theory, the class specified on the deepest element should win, right? while I can see the CSS classes are on
 or  in the output HTML, I can't see those classes in the chrome devtool CSS tab. :upside_down_face:

    Task executor

  <button aria-describedby="form-group-label-info" aria-disabled="false" aria-label="More info for field" class="pf-c-button pf-m-plain pf-c-form__group-label-help" type="button" data-ouia-component-type="PF4/Button" data-ouia-safe="true" data-ouia-component-id="OUIA-Generated-Button-plain-8">
  ....
  </button>

Nailed it - we have to add BackgroundColor and Text here - https://github.com/KaotoIO/kaoto-ui/blob/main/src/App.tsx#L4-L10


Decide on project topic & technologies

Examples of the project topics.

Effective interpersonal communication and its influence on Business Management.
How to be a good speaker by using and combining verbal and nonverbal communication?
Interpersonal communication in virtual reality.
Communication technology trends that will shape 2025.
Trends and challenges for interpersonal communication in the next decade.
The importance of interpersonal communication in developing personal life and professional career.
The importance of non-verbal communication in mass-media and journalism.
The importance of digital communication for businesses.
Traditional communication channels versus modern communication channels.
The use of virtual reality in the future interpersonal communication.

Work env.

PowerPoint
Overleaf
Typst (?)
Prezi
Some other LaTeX based editor

Trends and challenges for interpersonal communication in the next decade.
PowerPoint
10.04


Logging and checkpointing is better done on a time interval basis instead of per iteration basis

Checkpointing is done to not repeat time consuming training, it should happen at regular time intervals. Logs can blow up in size if each iteration does not take a lot of time.
I recommend logging every 0.5 secs and checkpointing every 5 mins

thanks for your feedback!  we'll take it into consideration as we improve dopamine.


Support for Block Storage

This PR fixes #131 by adding support for all the available features of Block Storage exposed via the API.

[x] List all volumes
[x] Create a new volume
[x] Retrieve an existing volume with either by its ID or by its name and region
[x] Delete a volume with either by its ID or by its name and region

wow, thanks for this update!

 Don't forget to report issues in case you find any.


Scala 3 not found under name scala_3

In PR https://github.com/NixOS/nixpkgs/pull/171641, dotty was renamed to scala_3 with the name dotty remaining available for backwards compatibility. However, on search.nixos.org a search for scala_3 finds no packages and the package is still called dotty.

The deprecated alias should have been added to pkgs/top-level/aliases.nix so that it's not indexed by nixos-search.
There is a known bug where nix-env discards duplicate attributes based on pointer equality, which is the case here between pkgs.scala_3 and pkgs.dotty, so Nix just picks one of them (in this case dotty). Anyway, this should be fixed in nixpkgs.


[composer][7/7]Implement unary callable rpc method's samplecode

Unary rpc callable method sample code:
   * try (EchoClient echoClient = EchoClient.create()) {
   *   EchoRequest request = EchoRequest.newBuilder().build();
   *   ApiFuture<EchoResponse> future = echoClient.echoCallable().futureCall(request);
   *   // Do something
   *   EchoResponse response = future.get();
   * }
   *

refactor #636

refactor #636


linear SVC, NuSVC, SVR and NuSVR support.

Fixes #218
Only binary SVC and NuSVC are supported (see https://github.com/scikit-learn/scikit-learn/issues/9196).

Codecov Report

Merging #221 into master will decrease coverage by 0.21%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
- Coverage   97.37%   97.15%   -0.22%
==========================================
  Files          41       41
  Lines        2550     2565      +15
  Branches      488      492       +4
==========================================
+ Hits         2483     2492       +9
- Misses         35       38       +3
- Partials       32       35       +3

Impacted Files
Coverage Δ

eli5/sklearn/utils.py
89.85% <100%> (+0.3%)
:arrow_up:

eli5/sklearn/explain_weights.py
95.68% <50%> (-4.32%)
:arrow_down:


Genetic data processing

Hello author, I am working with genetic data, filter_vcfs.py,
start = vcf_file.find("ADNI_ID.") + len("ADNI_ID.")
end = vcf_file.find("output.vcf")
For the genetic data in ANDI dataset, do you use.plink files for genetic data processing or.vCF files? I can't find the string ADNI_ID. Did you name these by yourself? Thank you for your answer

你好，作者，我正在处理遗传数据，filter_vcfs.py，
start = vcf_file.find("ADNI_ID.") + len("ADNI_ID.")
end = vcf_file.find("output.vcf")
对于ANDI数据集中的遗传数据，您是否使用.plink文件进行遗传数据处理或.vCF文件？我找不到字符串ADNI_ID。这些是你自己命名的吗？谢谢你的回答

hello, May I ask if your problem has been resolved? I would like to discuss some issues with you

Hi there! Sorry for the delayed response. We use vCF files as the variable vcf_file suggests. It seems that ADNI might have updated some of their files throughout the years (this was done in early 2021), so ADNI_ID might be named something different. Either way, it's the identifier of the patient, so you just need to look for that in whatever file you have.

Hi there! Sorry for the delayed response. We use vCF files as the variable vcf_file suggests. It seems that ADNI might have updated some of their files throughout the years (this was done in early 2021), so ADNI_ID might be named something different. Either way, it's the identifier of the patient, so you just need to look for that in whatever file you have.


Development test

Add information to dependencies regarding ESMF.

It also has updates from upstream.

This is going to the wrong branch!


price-feeder: tx submission

Summary of Bug
Currently, we broadcast pre-vote and vote txs using sync mode. This means we get a happy response if the tx is valid, but we don't know if/when the tx actually makes it into a block.
We need to evaluate if we need to ensure that the current tx is committed in a block prior to submitting the next tx. Assuming we do, and I think we do, we need to either:

Use block mode, OR
Query for the current tx prior to submitting the next tx

Or maybe submitting via sync is OK?
ref: https://github.com/umee-network/umee/blob/main/price-feeder/oracle/client/client.go#L212

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Note, I gave block mode a quick try and got the following errors:
5:21PM INF starting price-feeder oracle...
5:21PM INF starting price-feeder server... listen_addr=<IP_ADDRESS>:7171
5:21PM ERR failed to broadcast tx; retrying... error="post failed: Post \"http://localhost:26657\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)" last_check_height=1405 max_height=1415 module=oracleClient tx_code=0 tx_hash=
5:21PM ERR failed to broadcast tx; retrying... error="rpc error: code = InvalidArgument desc = account sequence mismatch, expected 38, got 37: incorrect account sequence: invalid request" last_check_height=1406 max_height=1415 module=oracleClient tx_code=0 tx_hash=
5:21PM ERR failed to broadcast tx; retrying... error="post failed: Post \"http://localhost:26657\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)" last_check_height=1407 max_height=1415 module=oracleClient tx_code=0 tx_hash=

Look's like Terra's version does sync mode which I guess makes sense. We will keep that for now too.
ref: https://github.com/terra-money/oracle-feeder/blob/main/feeder/src/vote.ts#L203


[$500] Expense - Workspace under "To" field is highlighted after opening user avatar

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.42-0
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
Go to FAB > Request money > Manual.
Enter amount > Next.
Select a workspace > Next.
On confirmation page, click on the workspace under "To".
Click on the user avatar (not workspace avatar).
Return to confirmation page via back button on the top left.

Expected Result:
The workspace name under "To" will not be highlighted.
Actual Result:
The workspace name under "To" is highlighted.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/ba52b88c-f125-47b3-b1b3-80caeba73266

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01bd38066315528994
Upwork Job ID:<PHONE_NUMBER>406108160
Last Price Increase: 2024-02-15

We think that this bug might be related to #wave5-free-submitters
CC @dylanexpensify

I agree this should be changed, but I don't think it should be a blocker

Proposal
Please re-state the problem that we are trying to solve in this issue
The workspace name option under "To" is highlighted after opening user avatar.
What is the root cause of that problem?
I was not able to find an offending PR. This being said, here's what causes the option to be highlighted once user views avatar (modal opens):

when the avatar modal is opened this causes the componentDidUpdate of the BaseOptionsSelector to run due to re-render which changes the focusedIndex from 1 to 0 (selectedOptions.length) here:

https://github.com/Expensify/App/blob/32648bbd01b2f1e98ff7273a92deb4c4bb10d7cb/src/components/OptionsSelector/BaseOptionsSelector.js#L163-L171
but this shouldn't happen in our case since both previous and current selectedOptions.length  are 0 (empty array) and focusedIndex was initialized with 1 on component mount.
What changes do you think we should make in order to solve the problem?
Above the code block mentioned in RCA, add the following check:
    if (_.isEqual(this.props.selectedOptions.length, prevProps.selectedOptions.length)) {
        return;
    }

just like we did above in the same componentDidUpdate by comparing previous / current sections and returning if equal.
This will return if both previous and current selectedOptions.length are 0 (not changed), preventing the setState to set focusedIndex to 0 and highlighting the option.
Videos

MacOS: Chrome / Safari

Before
After

Updated proposal

added offending PR and explanation of what lead to this issue

Not reproducible on main branch, we don't allow users to open workspace details.

That's correct as currently on production we don't allow users to open workspace details and preview the avatars but since the PR https://github.com/Expensify/App/pull/35594 was merged, we don't disable the option anymore - and this issue can be observed on staging.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Workspace option kept highlighted once we come back after opening the profile avatar
What is the root cause of that problem?
It seems the problem persists after this PR gets merged.
Reason: When we open the profile avatar it calls the api openPublicProfilePage and its response will update the personalDetailsList  onyx data which eventually regenerates the participants array here. Due to that sections prop gets changed and the execution will reach to set a new focused index part unnecessarily in BaseOptionsSelector.
https://github.com/Expensify/App/blob/50e07e8604d40b278c1ede2fc5c6fb78c97f1c1f/src/pages/iou/request/step/IOURequestStepConfirmation.js#L99-L106
https://github.com/Expensify/App/blob/421bbc5e319ffe906471a93917d789ed606e1a7e/src/components/OptionsSelector/BaseOptionsSelector.js#L149-L172
What changes do you think we should make in order to solve the problem?
I believe it should be || here instead of ?? bcoz currently it won't check the second condition as the first will always return the boolean value which isn't null. With that change, it returns early and won't make the unnecessary api call when the avatarURL is available.
https://github.com/Expensify/App/blob/38c6c164c91ccc4c1a31e513301ec94436eac9b6/src/pages/settings/Profile/ProfileAvatar.tsx#L30-L34
if (!ValidationUtils.isValidAccountRoute(Number(accountID)) || !!avatarURL) {

it returns early and won't make the unnecessary api call when the avatarURL is available.

I must mention the call is not unnecessary and that the 2nd proposed solution would break the avatar functionality in case the avatar is changed from another device because if we apply the solution that would mean the OpenPublicProfilePage won't be called anymore when avatarURL changes.

Proposal 2 - Solution Result Video (breaking avatar functionality)
https://github.com/Expensify/App/assets/56457735/e2a24bc6-fdb4-4cd4-981f-44a1fb3b578b

Proposal 1 - Solution Result Video
https://github.com/Expensify/App/assets/56457735/48850668-d35b-4e65-9bac-aca92ba90223

cc @Pujan92 @ntdiary

Updated proposal

added alternative solution which fixes the issue from ProfileAvatar.tsx without breaking ProfileAvatar's functionality

Updated proposal

updated RCA with detailed information about the specific field which causes the re-render

The re-render is caused by the fact that within BaseOptionsSelector, the previous and current props of sections[0].data[0].participantsList[0].isLoading change their boolean value when OpenPublicProfilePage  is called by opening the avatar preview 🙂

Video
https://github.com/Expensify/App/assets/56457735/e07f3127-d794-4bfe-987e-4e3f0a16e67f

cc @ntdiary

@ntdiary How about early return when previous and current isLoading mentioned changes, would that satisfy ?

Updated proposal

changed solution according to reviewer's #36577 (comment) + detailed breakdown of how the solution works

@ntdiary Hope this one satisfies 🙂

Updated proposal

changed solution according to reviewer's #36577 (comment) - fixing the issue while ensuring that the code after the condition can be executed normally in any other scenarios

@ntdiary Hope this satisfies 🙂

Updated proposal

updated solution to satisfy latest reviewer's #36577 (comment) by disabling the focus behaviour all together leveraging existing disableFocusOptions prop based on:

Actually, focusedIndex doesn't seem to be used on this confirm page, so some possible directions are: how to keep it unchanged when re-rendering, or whether it can be disabled, etc.

cc @ntdiary Maybe this one will satisfy 🙂

Hmm Kevin's updated proposal is essentially

Nope it's not 😅  I'm not disabling any Pressable nor changing any existing styles.
I'm using an already existing prop to disable the focus behaviour since as mentioned by latest reviewer's comment:

Actually, focusedIndex doesn't seem to be used on this confirm page

which also hinted at disabling the focus behaviour if possible.
My solution is simply adding a prop (1-liner), exactly what was suggested for as possible solution since all my previous solutions were too complicated. Also your solution is pretty vague:

We should extend our BaseGenericPressable class to account for pressable that we don't want to have a hover state.

don't you think ? Besides suggesting changing the common BaseGenericPressable's styles which has nothing to do with the RCA of our issue 🙂

@ntdiary is there a proposal here that will do the trick or are we still waiting on a better one?

There might be potentially better ideas.
BTW, this issue is somewhat similar to issue https://github.com/Expensify/App/issues/37238, so I will also discuss it there. :)

This issue's root cause is related to OpenPublicProfilePage being called which updates focusedIndex because isLoading changes its boolean value, while the other issue's accepted RCA was pretty vague, mentioning that the sections prop changes and suggested to compare previous / current sections as a fix.
Turns out the fix PR from https://github.com/Expensify/App/issues/37238 selected proposal didn't work as expected and a regression was reported.
I think regardless of the root cause of the other issue, my latest proposed solution would fix both issues since as you mentioned above:

Actually, focusedIndex doesn't seem to be used on this confirm page

To me, this looks like a straight-forward slam dunk 🏀
I'm confused as to what's the hold up w/ this one really 🤔
cc @ntdiary @dangrous

I just went on parental leave so I haven't been paying attention to this one haha - @slafortune do you think you could fish for an internal volunteer in Slack? I can help if needed!

Updated proposal

improved solution by passing selectedOptions.length <= 1 to disableFocusOptions, therefore accounting for possible regressions when it comes to a similar issue's solution (multiple options)

To +1 on my previous  https://github.com/Expensify/App/issues/36577#issuecomment-1967517205, this is a direct quote of @ neil-marcellini from https://github.com/Expensify/App/issues/37238#issuecomment-1968029563:

I think we have a pretty clear problem and solution for this specific issue. There are other issues I'm sure, but let's solve those elsewhere.

My take from that is that we should handle this issue here and treat it as separate from the other similar issue - which just merged the PR that will soon arrive on staging.
I'm pretty sure our issue will still occur even after that issue's PR arrives on staging, since my proposal's RCA will still occur even though the other proposal's solution added an early return if previous - current sections are equal.
cc @ntdiary

@ntdiary are you able to recreate this still?

@ntdiary do you have an update?

AH - Thanks for the details and linking that @ntdiary !  I think since this is a pretty edge case and OptionsSelector is deprecated we don't really need to hold we can simply close this.


setup base express server

You have to completed your Setup Documentation to ensure that each developer first install all packages and then create a copy of your .env.example file as .env to ensure be able to run the project.

@VictorNico  I'm working on it as issue #2, that's why I closed this one


ENH When using composer_install cat composer.json

Issue https://github.com/silverstripe/gha-ci/issues/60

Oops - should have checked the branch first! Needed to be 1.3. will merge up.

Tagged as 1.3.2


New changes seem to slow model creation down

To reproduce: you need a full Moose image with FamixReplication repository up to date with master.
Run MooseScriptsTest>>#testCreateLightModels

Tried to understand a bit. This test takes too much time because of
MooseScripts>>#createLightModelForNetwork.
:warning:
[ MooseScripts createLightModelForNetwork ] timeProfile

normally takes less than a second. With the new changes loaded, it seems to never end.


add support for multiple gpgkeys, excludepkgs, and includepkgs

Support multiple gpg keys
Support includepkgs and excludepkgs
store version number in metadata.json

Closes #17


TitanPropertiesStep not applied by TitanLocalQueryOptimizerStrategy

I was profiling Titan 0.9 M2 with query.batch=true and noticed that property retrievals were not being batched.  My simple test query is as follows.
g.V(4344).out('contains').values('thingType')
I took a look in TitanLocalQueryOptimizerStrategy and found the TitanPropertiesStep.  The strategy is applying the VertexStep -> TitanVertexStep transformation, but not doing so for the PropertiesStep.  I went ahead and updated it to do this but could not figure out a good non-runtime way to determine if the properties retrieval was going to be against a Vertex or an Edge.  My updates worked as expected and batched the properties retrieval but I figured you guys may have had a reason for not including this initially.  I was hesitant to submit a PR until I figured out a better way to determine if I was operating on a Vertex or Edge and also wanted to see if you guys had another reason for not including this transformation in the optimizer strategy initially.
Thanks,
Ted

Excellent point. We had excluded TitanPropertiesStep (i.e. it wasn't applied in any strategy) because we needed additional method access from TinkerPop. That happened but we forgot to add a ticket to re-add TitanPropertiesStep optimizations and hence it never happened.
I have updated TitanPropertiesStep and re-added the optimization. Take a look at 53bb28ad855af750b22eb5f1c3cb0e540852d182 and please make sure this does what you need. We will deploy another 1.0.0-SNAPSHOT soon.
Thanks for pointing this out. That's a good catch for the 1.0 release.
BTW, we won't be optimizing PropertyMapStep until https://issues.apache.org/jira/browse/TINKERPOP3-844 is closed.

I took a look at the commit and reran my tests with the new code and everything looks good.  Property retrievals were batched as expected.  I appreciate you taking a look at this.  It'll be a big help.  As far as the PropertyMapStep goes, that makes sense.  I had been puzzled by the duplication between the PropertiesStep and PropertyMapStep.
Thanks again,
Ted

This problem looks like it is now resolved.  Can we close this now ?

Yeah, this has been resolved.  Not sure if it would be a good idea to create a new issue to make sure that when TINKERPOP-844 is finished, any necessary updates will be made.  Although, that'll probably trickle down through the release notes/upgrade instructions with whatever TP release it makes it into.

I would guess that we'd pick that up through the upgrade process. closing - but if you want to create another one referencing that issue then i guess that's cool. up to you....thanks.


IPython display function not recognized?

Language Server version: 2021.7.5
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Conda 4.10.3

I'm not sure why the following below is showing up as an error, get the same error in a python file

display is not a symbol that is in the builtins namespace, and you are not importing it into your module's global namespace.
Does ipywidgets inject new symbols into the builtins or module namespace even if you don't import them? If so, pylance would have no idea that it's doing under the covers. This is a really terrible practice that some older Python libraries did. If you explicitly import the symbol into your module's namespace, then it will work as expected.
Alternatively, you can disable the reportUndefinedVariable diagnostic check for your project.

I've reported same issue in vscode-jupyter project: https://github.com/microsoft/vscode-jupyter/issues/8539

Our concat document can likely handle this. Not sure yet if we should or not. Essentially we'd put this at the top of the concat doc:
from IPython.display import display

and then pylance would understand display.

@rchiodo is it only display or "from IPython.display import *" ?

@rchiodo is it only display or "from IPython.display import *" ?

Second one I think? That would enable stuff like this:
from IPython.display import *

display_javascript()

I've tried to use from IPython.display import display and from IPython.display import *, but it's not resolving the issue for me. I now get an Import "IPython.display" could not be resolved error from Pylance as well as the pre-existing "display" is not defined errors.

My Jupyter notebook uses an Anaconda environment, which might be relevant.

Sorry @DavidHopkinsFbr the comment I made was in reference to the internals of pylance and what it should do. Not what a user should add to resolve the problem. The root cause of the issue is that pylance doesn't have the implicit includes that jupyter includes.

Moving this issue to discussion as an enhancement request for comments and upvotes.


Different instances

Hello
My Use Case
I have an API that allows users from different tenants to connect.
To do this, I first check if my user exists with the email address provided. If it exists I retrieve an access token with oAuth.

const config: HostConfig<'oauth2'> = {
  authType: 'oauth2',
  host: tenant.url,
  clientId: tenant.oAuthClientId,
  clientSecret,
};

qlikApi.auth.setDefaultHostConfig(config);

const response = await qlikApi.users.getUsers({email});

if (response && response.data.data.length > 0) {
  const user = response.data.data[0];

  const accessToken = await qlikApi.auth.getAccessToken({
        hostConfig: {
          ...config,
          userId: user.id,
          noCache: true,
        },
      });
}

The problem is that if 2 users from 2 different tenants try to connect at the same time, this will create an authentication conflict when retrieving a user.
Question
Is there any possibility to create different instances of connection ?

Hello, thank you for your answer.
Is there a roadmap for the feature ?

Yes, it's on its way. I can't say exactly when, but within the next 2 months I hope.

This has been solved in version v1.20.0


kulhanek/ranlux is no longer available

README.md:3430 references github.com/kulhanek/ranlux which is no longer available.
[ranlux](https://github.com/kulhanek/ranlux): random number generator described in the paper [RANLUX: A Fortran implementation of the high-quality pseudorandom number generator of Lüscher](https://www.sciencedirect.com/science/article/abs/pii/001046559490233X), by F. James, Computer Physics Communications, Volume 79, Issue 1 (1994). A Fortran 90 version by Alan Miller is [here](https://jblevins.org/mirror/amiller/luxury.f90).

removed


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/jhedtmann/iobroker.vbb

[ ] [E003] Repository must have name ioBroker.adaptername, but now io"b"roker is in lowercase
[ ] [E004] No adapter name found in URL: https://raw.githubusercontent.com/jhedtmann/iobroker.vbb/main
[ ] [E020] Name of adapter in package.json must be lowercase and be equal to "iobroker.". Now is "undefined"
[ ] [E019] Invalid repository URL: https://github.com/jhedtmann/ioBroker.iobroker-vbbadapter. Expected: git+https://github.com/jhedtmann/iobroker.vbb.git
[ ] [E103] common.name in io-package.json must be equal to "". Now is VBB
[ ] [E125] External icon not found under URL: https://raw.githubusercontent.com/jhedtmann/ioBroker.iobroker-vbbadapter/main/admin/iobroker-vbbadapter.png
[ ] [E200] Not found on npm. Please publish
[ ] [E300] Not found on travis. Please setup travis
[ ] [E802] No topics found in the repository. Please go to "https://github.com/jhedtmann/iobroker.vbb", press the settings button beside the about title and add some topics.
[ ] [E603] NO "## Changelog" found in README.md
[ ] [E604] No "## License" found in README.md

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "" in latest repository
[ ] [W801] .npmignore not found

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


word-break: break-word exists in W3C text module

There is word-break: break-word so this is no longer unofficial. -> https://www.w3.org/TR/css-text-3/#word-break-property

Thanks!


Documentation for FirewallRules/FirewallRuleName/Direction appears incorrect

The documentation for FirewallRules/FirewallRuleName/Direction states "Comma separated list."  However, testing CSP on recent OS versions (OS Build 17134.523) shows that a comma separated list is not supported by the CSP.  Values accepted by the CSP are "IN" or "OUT" but not "IN,OUT"

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

ID: bfea4e59-ec8c-cbdf-a039-23612c2266a5
Version Independent ID: 7058544b-7ef2-a398-366a-cf71520aa9d8
Content: Firewall CSP
Content Source: windows/client-management/mdm/firewall-csp.md
Product: w10
GitHub Login: @MariciaAlforque
Microsoft Alias: maricia

@officedocsbot assign @mypil

@JeffStong - thank you for submitting feedback.
In reference to the other issues #2602 & #2735 you opened for this article, Could you please help us by suggesting how this content could be edited to incorporate your feedback?  What do you suggest is the best way to fix this?


TNT-38932 Implement getOffers() and sendNotifications()

Description
Implement getOffers() and sendNotifications()
Related Issue
https://jira.corp.adobe.com/browse/TNT-38932
https://jira.corp.adobe.com/browse/TNT-38934

@artur-ciocanu I've also added some unit tests and sendNotifications() implementation, please take another look when you have a moment.. Thanks!

@XDex I was wondering if there are any code coverage tools that we could use to keep us honest.

@XDex I was wondering if there are any code coverage tools that we could use to keep us honest.

We're already using Codecov which is free for OSS projects, I just need to add coverage stats display in README


[REQUEST] Use only distinct characters for URLs

Sometimes link URLs need to be typed (e.g. when one sees a link on slides or in a video).
This can be frustrating if the link contains codepoints which are easily confused such as "oO0" "1lI".
It would be friendly to disallow easily confused codepoints from links.
BTW the motivation for this came from the deck:
https://software.intel.com/en-us/videos/advanced-single-instruction-multiple-data-simd-programming-with-intel-implicit-spmd-program

I like the idea! Thanks. We'll have to ensure whatever we do is of course backwards compatible with older URLs but this should "just work"™ if we change the algorithm.

Whoah that was quick @RabsRincon ! A new record...!


fix: :bug: fix workorder powerbi bug

Release as preview
Test in workorder
If works merge
If not works, cry


DUS fails to deploy due to nodejs12 is not longer supported by Lambda

Describe the bug
DUS deployment fails because Lambda functions created by CDK for custom resources still uses nodejs12 environment, which is no longer supported by Lambda.
To Reproduce
Deploy DUS solution, v1.0.11
Expected behavior
Successful deployment
Please complete the following information about the solution:

[x] Version: v1.0.11
[x] Region: us-east-1
[x] Was the solution modified from the version published on this repository? No
[n/a] If the answer to the previous question was yes, are the changes available on GitHub?
[x] Have you checked your service quotas for the services this solution uses?
[n/a] Were there any errors in the CloudWatch Logs?

Screenshots
Error message from failed DUSStack:
Resource handler returned message: "The runtime parameter of nodejs12.x is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (nodejs18.x) while creating or updating functions.

For resources: dusstackkendraindexprovideruanfh4dbcex3irvekny2amframeworkonTimeout88FF4ADB
Additional context
Looks like currently DUS uses CDK v1.58.0, the fix for deprecated nodejs12 environment was introduced in v1.60.0

Thank you @qing-at-work for reporting the bug.
It has been fixed in v1.0.12.


Functional: Weird behavior of ast.parse breaks FixMissingLocations

Test case from: https://github.com/BenWeber42/gt4py/commit/b85013a26eec495ac3592b4ad9f8a9c52aeeb91c
def get_fixed_ast(code: str) -> ast.AST:
    python_ast = ast.parse(textwrap.dedent(code))
    python_ast.lineno = 1
    python_ast.end_lineno = 999
    python_ast.col_offset = 0
    python_ast.end_col_offset = 999
    return FixMissingLocations.apply(python_ast)

def test_reused_op_nodes():
    """
    Python's `ast.parse` will reuse the same nodes multiple times in the AST sometimes
    (e.g., for operators like `ast.Add`). In this case, all those shared nodes
    will share the same location information which can be plain wrong in some cases.
    """

    astree = get_fixed_ast(
        """
        a = (
            1
            /
            2
            /
            3
            /
            4
        )
        """
    )

    match astree:
        case ast.Module(
            body=[
                ast.Assign(
                    value=ast.BinOp(
                        left=ast.BinOp(
                            left=ast.BinOp(op=ast.Div() as first), op=ast.Div() as second
                        ),
                        op=ast.Div() as third,
                    )
                )
            ]
        ):
            pass
        case _:
            assert False

    assert first is not second
    assert second is not third
    assert third is not first

For some reason, Python's ast.parse will reuse the same ast.Div instances throughout the whole tree. This will break the FixMissingLocations pass (not really break it, but it will produce wrong results). It's not necessarily an issue in FixMissingLocations, but maybe a pre-condition should be that no nodes are reused within the same tree and a new pass should be added that makes sure this pre-condition is established before.

Fixed in #1020 by doing a copy (fine for leaf nodes).


alacritty: fix darwin build

Description of changes
fix darwin build by using a newer version of the sdk, the solution was contributed by @reckenrode, thank you!
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 279907 run on x86_64-darwin 1

2 packages built:

alacritty
alacritty.terminfo

Result of nixpkgs-review pr 279907 run on aarch64-darwin 1


Breaking into modules [WIP]

Fixes #38

Hi! Thanks for PR.
Make sure you have checked points in PR Template correctly.
(Powered by GitMate.io)


test(dm): enable multi-rocksdb in integration tests

What problem does this PR solve?

Issue Number: ref #9446
What is changed and how it works?
Check List
Tests

Integration test
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/test ?

/test all


[2.x] Semantic Markdown heading permalinks

Part of https://github.com/hydephp/develop/pull/2047
Todo: Fix offset to add spacing above the link
Also refactors to use Tailwind instead of CSS

Advantages:

Cleaner semantics: The id directly ties to the heading itself, which aligns well with HTML5 standards.
Improved accessibility: Assistive technologies can better associate the id with the heading content.
CSS styling: Easier to target headings via their id for custom styles.

See https://github.com/hydephp/develop/pull/2047#issuecomment-2511210214

Edge case we need to fix: If there are multiple headings of the same name, we need offset.


Travis-ci running a docker image

We talked about this briefly in our last meeting, before I forget here is an example of a working docker image on travis-ci https://github.com/lukecyca/travis-docker-example

The travis script executes without error now, so we are ready to work on the docker image problem

We will see if I can get it working. I spent a few hours on it already with no luck. I have looked at some examples of other repos that have implemented the functionality, but none have passed there builds recently. The build stalls and times out at the moment.


New Command: Get-DbaTraceFlagIndex

Feature Request
Provide the ability to have a central list of all known trace flags for SQL Server. Build it out similar to the Find-DbaCommand where we have an index.json file that contains pertinent information...just as a starting point:

TraceFlag number
Description/Synopsis of use
SQL Server version applied to
Link (KB, or default docs page)

Problem to solve

Having to look up a TF for a particular need or fix.

What is the latest thinking on this?
I normally go to https://docs.microsoft.com/en-us/sql/t-sql/database-console-commands/dbcc-traceon-trace-flags-transact-sql if I need information on trace flags.
One option would be to parse that page when building the traceflags.json file, the other would be to simply create the file and a method that reads it. The trace flags are not added that frequently, and the existing ones don't change a lot.

I had played with this and was able to scrape the web page to get the trace flag and descriptions info. However, I got stuck on how to load to json file and search the file with the function. Never had a chance to revisit.

It is the same consideration as the build reference and the data we keep on that, or the wait type information we include in commands (links to SQLSkills site). However, it is more or less to also include undocumented trace flags that are not included in the docs but other resources speak/write on them and the usage.
Other references:

https://www.brentozar.com/blitz/trace-flags-enabled-globally/
http://www.sqlservercentral.com/articles/trace-flag/152989/
http://www.sqlservice.se/updated-microsoft-sql-server-trace-flag-list/

Excellent info in this one: https://dba.stackexchange.com/q/141092/507
Check out the links that Paul White shares, that PDF download is stinking awesome.

This is updated on an ongoing basis, as recently as March 24th. But it looks hard to parse.
https://github.com/ktaranov/sqlserver-kit/blob/master/SQL Server Trace Flag.md


Can't start with ubuntu's latest QEMU

I'm on the latest ubuntu 18.04.3 with the QEMU installed from apt. When I start arm_now, the error message is shown below:
qemu-system-arm: -nic: invalid option

My QEMU version:
(arm_now) ~/tmp ᐅ qemu-system-arm -version
QEMU emulator version 2.11.1(Debian 1:2.11+dfsg-1ubuntu7.15)
Copyright (c) 2003-2017 Fabrice Bellard and the QEMU Project developers

But the -nic option is introduced in QEMU 2.12. See the post here: https://www.qemu.org/2018/05/31/nic-parameter/

Try this:
https://askubuntu.com/questions/1067722/how-do-i-install-qemu-3-0-on-ubuntu-18-04

:+1: to the proposal of @MorphBC , it works perfectly. At least, tested with qemu <IP_ADDRESS> sources.
This issue could be closed with that solution, IMO.

Is that fine to everybody if we close at this point ? :)

I just changed a line to following and seems to start ok:
$ cat ./.local/lib/python3.6/site-packages/arm_now/arm_now.py | grep nic
args = ["-net nic -net user"]

Hello,
I test with docker on the lastest version of Ubuntu, It was just fine.
I'm closing this issue, feel free to reopen.
Have a good day.


Implemented: functionality to store the job as favourite for a user (#28u2mjf).

Related Issues

Closes #
Short Description and Why It's Useful

:- Implemented functionality to fetch and store search preferences related to the current user and update the status based on the click event on star icon.
:- Improved: code to show job's enumName instead of enumId as pinned jobs on the footer of the pipeline page.
Screenshots of Visual Changes before/after (If There Are Any)

https://user-images.githubusercontent.com/52008359/168375926-754cc9d3-6f51-4e75-a6cc-4aa695929fcb.mp4
IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

https://user-images.githubusercontent.com/82817616/168823037-e5c497a4-3992-4f2b-96b1-a6f4b7018dbd.mp4


C++ client

Thanks

cpp-httplib
SHA-1

Complie and Run
sh compile_and_run.sh

Don't put the binary on, and run some script for compiling

Should be good now

I think you might need to include LICENSE from cpp-httplib, or maybe you could use something like libcurl, boost.asio if you don't want to be windows-friendly.

Since this repo is also licensed in MIT, I'm not sure if a redeclaration is required...

I think using libcurl is a better choice, it can reduce core code size and people can just download libcurl from their system PM

Please use HTTPS


STORM-1769 Added a test to check local nimbus with notifier plugin

This test is added to ensure plugin for Config.STORM_TOPOLOGY_SUBMISSION_NOTIFIER_PLUGIN works fine. Atlas uses this and we want to make sure it always works fine.

+1

@harshach @ptgoetz Can you please review this?

+1


Review use of demos and demo_temp dir cf install vs editable install

Possible solution - enforce only use from dev editable install otherwise through warning and exit


Has front_matter moved?

https://github.com/denoland/fresh-blog-example/blob/8854933d4beefb7a1ec7d2bbff0ddd0b81c73b21/utils/posts.ts#L1
Am I correct in saying that front_matter has now moved in more recent versions of Deno's standard library? I believe the article and code may need updating if that is so.

Hi @mickgardner, I'm not sure when this repo and article will be updated, but I managed to solve it this way:
I manually installed the packages:
deno add jsr:@std/front-matter deno add jsr:@std/path
I wrote the imports inside 'utils/posts.ts' like this:
import { extract } from "@std/front-matter/any"; import { join } from "@std/path";
Hope this helps you!


Pull-Request [我的生活-3-8-2024] from Obsidian

👷 Deploy Preview for lixiaodong processing.

Name
Link

🔨 Latest commit
5dfcef1813e72f35258796e19a07896028fac840

🔍 Latest deploy log
https://app.netlify.com/sites/lixiaodong/deploys/65eadd85a271540008a70929


Support translation for all Scratch blocks in numbers projects

One of the blocks contributed by numbers projects (train new model) supports translation
That block uses a text NLS item identified here:
https://github.com/IBM/taxinomitis/blob/14844892c99c146e22fbfc2dd8351cadda95c22d/resources/scratch3-numbers-classify.js#L128-L131
and provided here:
https://github.com/IBM/taxinomitis/blob/14844892c99c146e22fbfc2dd8351cadda95c22d/public/languages/de.json#L1591
.
However the other blocks just have hard-coded English labels
https://github.com/IBM/taxinomitis/blob/14844892c99c146e22fbfc2dd8351cadda95c22d/resources/scratch3-numbers-classify.js#L101
https://github.com/IBM/taxinomitis/blob/14844892c99c146e22fbfc2dd8351cadda95c22d/resources/scratch3-numbers-classify.js#L60
https://github.com/IBM/taxinomitis/blob/14844892c99c146e22fbfc2dd8351cadda95c22d/resources/scratch3-numbers-classify.js#L38

I remember now why I didn't make these translatable when I first did them - it's because of the variable number of arguments in them, which makes it incompatible with the way Scratch does NLS. I don't think this is easily do-able. :-(


Update verifyCredentials

Refactored code to make it cleaner.
If the password is incorrect, we should return the same messages of Incorrect username or email! to prevent bruteforce attacks.

Thanks!


Forecast API optional parameters don't work

Source: Forecasts (Azure Consumption)
The Azure Forecast API does not seem to process optional parameters and only returns the current month's forecast with daily grain regardless of grain or date filter specified.
Filter by date
https://management.azure.com/subscriptions/$subId/providers/Microsoft.Consumption/forecasts?$filter=usagedate ge '2018-04-01' AND usagedate le '2018-04-02'&api-version=2019-05-01
Expected: The API returns the cost or forecast information for the date range supplied (this was tested for with date ranges in the past, future, and both)
Result: The API defaults to current month, daily, regardless of dates supplied
Filter by grain
https://management.azure.com/subscriptions/$subId/providers/Microsoft.Consumption/forecasts?$filter=Grain eq 'Monthly'&api-version=2019-05-01
Expected: the API returns monthly forecasts when grain filter is set to monthly
Result: the API defaults to current month, daily, regardless of grain

@F5veRings Thank you for the valuable feedback,we are investigating the issue.

@Erikre , can you help with this please?

@rloutlaw - Updated you to be the assignee for this issue because you are listed as the current document owner in the article metadata. Could you please work to address the feedback and close the issue? See these instructions.