SystemTimer microseconds

Is there a way to access this DAL function from within MakeCode, it surfaces the micro-seconds value of the system timer.
https://github.com/lancaster-university/microbit-dal/blob/master/source/core/MicroBitSystemTimer.cpp#L145
We see a lot of projects where people use the 'wait microseconds' block to implement software bit-bashed protocols, and they tend to be quite brittle.  I saw a great IR driver on hackster today, but they use wait_us and hand calibrated/tweaked delay values, and that will be quite brittle.
If this function was accessible as a MakeCode block 'running time microseconds', it would be possible to do non-brittle time horizon style software modulation loops that would be much more resilient to both changes in the underlying code, and the effects of interrupts coming in and out along the way. I'd even be happy for it to be surfaced with a truncation so that it fits into a smaller data type, as typically you only need it to last a second or two before wrapping to be used for time-horizon based software bit-bashing (one would use the running time ms for anything longer anyway).

We could only surface it as a 32bit integer so it would only work for short horizons.

That's fine - if my maths is correct, a U32 running in 1uS increments would last just over 1 hour. That's more than enough (and if you wanted longer you would use 'running time ms' anyway.
The use case here is for timing short periods of time relatively accurately, so that it is less brittle to timing changes in the underlying code, and also less brittle to interrupt inserted delays. e.g. bit-bashing simple binary interfaces in a way that there is less jitter on the timing (and therefore making them more reliable in practice).
So, for all the scenarios I can think of this would be perfect.

Fixed in beta.

Thanks @abchatra @pelikhan


Bugsnag

 Issue by evansiroky
Friday Jul 06, 2018 at 19:05 GMT
Originally opened as https://github.com/catalogueglobal/datatools-server/pull/101

Add bugsnag reporting

evansiroky included the following code: https://github.com/catalogueglobal/datatools-server/pull/101/commits

:tada: This issue has been resolved in version 3.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add DigitalOcean CSI driver

What this PR does / why we need it:
This PR:

Adds the DigitalOcean CSI driver. The CSI driver is deployed automatically if .cloudProvider.external is enabled
Adds the default StorageClass and VolumeSnapshotClass for the DigitalOcean CSI driver. The StorageClass and VolumeSnapshotClass can be deployed by enabling the default-storage-class embedded addon

The following manifest has been used for testing:
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: csi-pvc-do
spec:
  accessModes:
  - ReadWriteOnce
  resources:
    requests:
      storage: 1Gi
  storageClassName: do-block-storage
---
apiVersion: v1
kind: Pod
metadata:
  name: nginx
spec:
  containers:
  - image: nginx
    imagePullPolicy: IfNotPresent
    name: nginx
    ports:
    - containerPort: 80
      protocol: TCP
    volumeMounts:
      - mountPath: /var/lib/www/html
        name: csi-data-do
  volumes:
  - name: csi-data-do
    persistentVolumeClaim:
      claimName: csi-pvc-do
      readOnly: false

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
xref #1710
Does this PR introduce a user-facing change?:
Add the DigitalOcean CSI driver. The CSI driver is deployed automatically if `.cloudProvider.external` is enabled
Add the default StorageClass and VolumeSnapshotClass for the DigitalOcean CSI driver. The StorageClass and VolumeSnapshotClass can be deployed by enabling the default-storage-class embedded addon

/assign @kron4eg

@xmudrii how do you think, should we update again? there is new release v4.0.0

@kron4eg I don't think it matters much, but I'll create a PR anyways.

Created #1820 to update the CSI driver.


Update hashFiles documentation reference

Update hashFiles documentation reference Link to the valid doc https://docs.github.com/en/actions/learn-github-actions/expressions#hashfiles
Description
The current hashFiles link is 'broken', it does not forward to the proper documentation

M
El mié., 9 de noviembre de 2022 1:39 p. m., Asaf Levy <
@.***> escribió:

Update hashFiles documentation reference Link to the valid doc
https://docs.github.com/en/actions/learn-github-actions/expressions#hashfiles
Description
The current hashFiles link is 'broken', it does not forward to the proper
documentation
You can view, comment on, or merge this pull request online at:
https://github.com/actions/cache/pull/979
Commit Summary

0e93819
https://github.com/actions/cache/pull/979/commits/0e93819da5a9071acff85480f406cd1e4d58631b
Update hashFiles documentation reference

File Changes
(1 file https://github.com/actions/cache/pull/979/files)

M README.md
https://github.com/actions/cache/pull/979/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(2)

Patch Links:

https://github.com/actions/cache/pull/979.patch
https://github.com/actions/cache/pull/979.diff

—
Reply to this email directly, view it on GitHub
https://github.com/actions/cache/pull/979, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVP7HD5D6KN644TAWAPFHILWHP4XLANCNFSM6AAAAAAR3ZFMZU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

M


shift+enter creates new terminal that does not activate base (or other environments)

Type: Bug
Open a python file. Type "import numpy", press shift+enter. This would create a new terminal and will try to activate base environment. This used to work but now hangs and does not finish even after a couple of minutes. However, it stopped working after today (didn't change anything other than restarting VS Code).
If I create a new terminal from terminal menu and it successfully loads base env. I can import modules by typing but when when I shift+enter a line, it creates a new terminal and hangs.
Anaconda command line shell works as expected.
This is making VS code unusable.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5600U CPU @ 2.60GHz (4 x 2594)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (9.98GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

python
ms-
2020.1.57204

Please open an issue on the Python extension repo


how to suppress these error reported by Terrascan

terrascan version: v14
Operating System: Ubuntu (Github Action)

Description
trying to run terrascan on my modules
Showing false scanning errors :
IaC Type            :	terraform
Directory           :	/path/to/module/modules
Error Message       :	directory '/path/to/module/modules' has no terraform config files

-----------------------------------------------------------------------

IaC Type            :	terraform
Directory           :	/home/runner/work/path/to/module/modules/compute
Error Message       :	directory '/home/runner/path/to/module/modules/compute' has no terraform config files

What I Did
I ran simple terrsacan
Paste the command(s) you ran and the output.
terrascan scan  -d . -i terraform --iac-version v14 -t all --log-output-dir $(pwd)

-----------------------------------------------------------------------

	IaC Type            :	terraform
	Directory           :	/path/to/module/modules
	Error Message       :	directory '/path/to/module/modules' has no terraform config files

	-----------------------------------------------------------------------

	IaC Type            :	terraform
	Directory           :	/home/runner/work/path/to/module/modules/compute
	Error Message       :	directory '/home/runner/path/to/module/modules/compute' has no terraform config files

@Rchanger please help how i can supress these warning

@ravitomardevops we added these logs since we were receiving multiple bugs stating we don't see any error but the exit code is non-zero. We will start a discussion about these to make it controlled via flag. I'll update you on this once we decide something. Thank you

@Rchanger It would be really nice to have an option to not fail the scan just because it found an empty directory.
I want to use terrascan to scan a terraform module but it fails because examples/ and modules/ don't have any terraform in them even though they have subdirectories that do.
Maybe the title of this issue could be updated?

flagging --non-recursive resolves this problem for me.


Update README.md

Add information about installation, usage and available generator commands to README.md

Makes sense, thank you!

Is this in the new docs anywhere? I couldn't find it

@marshallswain Only in quick start section. Maybe when we eventually roll a Feathers CLI we'll give it it's own.

Cool! You guys have done a lot of work.


Possibility to decrypt without mel-cp?

Would it be possible to decrypt the text without obsidian-encrypt plugin. My worry is if you lose interest in maintaining the plugin what will happen with my encrypted data :)

somehow a standalone rescue boat like a standalone decryptor will help if possible, thanks

I agree, I started working on A standalone decrypt tool
It's in it's early stages and there's no mention of it on the main pages yet.
The idea is to simply save the file locally and open it in your browser for offline decryption if needed.
Still some work to do here though.


<추가> wordle 클래스 랜덤으로 정답 단어 csv로 생성

PR 설명
연결된 issue (없다면 issue부터 만들기)

Codecov Report

Merging #23 (33f6b29) into master (b05201f) will increase coverage by 1.60%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   96.52%   98.13%   +1.60%
==========================================
  Files           6        6
  Lines         144      161      +17
==========================================
+ Hits          139      158      +19
+ Misses          5        3       -2

Impacted Files
Coverage Δ

wordle_py/models.py
95.65% <83.33%> (-1.85%)
:arrow_down:

tests/TestCase_AnswerString.py
100.00% <100.00%> (ø)

wordle_py/wordle.py
97.14% <100.00%> (+11.42%)
:arrow_up:

Continue to review full report at Codecov.

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


[Feature]: Want to add priority search algorithm in algorithm folder

Feature Name
priority search algorithm
Feature Description
Priority search is an algorithmic approach commonly used in scenarios where elements have different priorities and need to be accessed or processed based on those priorities. The goal is to retrieve the highest (or lowest) priority item efficiently, which is particularly useful in scheduling, pathfinding, and resource allocation.
Motivation
A priority search algorithm is designed to quickly retrieve or process high-priority items, making it ideal for tasks like scheduling and pathfinding. It helps users save time, boosts responsiveness, and improves the user experience by focusing on the most relevant or urgent information first.
Implementation Suggestions (Optional)

Choose a Data Structure: Use a priority queue (often a Min/Max Heap) to manage elements based on priority.
Define Priority Rules: Establish how priorities are assigned to elements (e.g., smallest number, shortest path).
Insert Elements: Add items to the priority queue with their associated priorities.
Retrieve Top Priority: Use the queue to get the highest-priority item efficiently.
Repeat or Terminate: Continue retrieval until all desired elements are processed or the condition is met.

Feature Type
New Algorithm
Does this feature require additional resources?

[ ] Additional Documentation
[ ] Third-Party Libraries
[ ] UI/UX Design
[X] Other

References (Optional)
https://en.wikipedia.org/wiki/Priority_search_tree

@ajay-dhangar assign this issue to me


Cost model empty state shows add a source modal

Describe the bug
When we try to add a new cost modal from empty state, add source modal opens.
To Reproduce
Steps to reproduce the behavior:

Go to cost model details with no cost models in the db
Click on "Create a cost model"
See error

Expected behavior
Cost model wizard
Screenshots

Additional context
Tested on CI beta

Thank you @dchorvat1 - I think I know what's the problem.


[llvm-arc-opts] When RAUWing of bridgeRetain->bridgeRetainN, cast the result of the bridgeRetainN to the type of bridgeRetain.

Since we introduce the declaration for bridgeRetainN, its result type may be out
of sync with bridgeRetain's. This means that when we perform a RAUW of one for
the other, the types do not match and we get an LLVM error. Instead, just cast
the bridgeRetainN's type to bridgeRetain's result type.
rdar://40507281

@swift-ci smoke test


fix: forwardRef render functions not using ref

Closes #
📝 Description

⛳ Current behavior (updates)

🚀 New behavior

💣 Is this a breaking change (Yes/No):

📝 Additional Information

Summary by CodeRabbit

New Features

Enhanced functionality for dropdown, listbox, menu, modal, and popover components.

Bug Fixes

Resolved issues with ref forwarding in multiple UI components, improving integration with React components.

Refactor

Simplified component definitions by removing forwardRef wrappers, streamlining prop handling and internal logic across various components.

#4099 updated in here @wingkwong


Result init by async throws closure

Add async support to Result, using with await
let result = await Result {
 try await asyncFunction()
}

Welcome to the Swift project! This contribution looks great. A public API change like this needs to go through the Swift Evolution process, as specified in this process document.
That said, I personally think this is a fairly non-controversial new convenience and fits in with Swift nicely.

Thank you. I want to add test before discussing this. But I can not find where the old test is, please help.

They're in test/stdlib/Result.swift.


fix: Update content.mdx typo

Update content.mdx typo
Update a typo at the content.mdx file

Lunaria Status Overview
🌑 This pull request will not trigger status changes.

Learn more
Lunaria automatically ignores changes on specific PRs by adding a ignored keyword in its title. Found: typo.
You can change this by either removing the keyword above from the PR's title, or modifying the ignoreKeywords property in your Lunaria configuration file.

Tracked Files

File
Note
Locale

src/content/docs/en/community-resources/content.mdx
Source changed, localizations will be marked as outdated.
en

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.


fix missing tokenCodeFn

I was missing the tokenCodeFn and allow to pass mfa serial through environment variable.

Hi, instead of readline, let's make this another command line argument?

What about I update to have both, so that it ll fallback to readline if not present in arguments?

that sounds ok to me.

addressed your suggestion @gojko ! Let me know if you need anything else.
Cheers.

hi - thanks for this, looks good. I will test and merge over the weekend.

Hi - I merged your code and tried to add some specs around it so we can maintain it easier. can you please try if this branch still works for you: https://github.com/claudiajs/claudia/tree/gregory-fix_bug_with_mfa
if it does what you need, we can merge everything into master

sorry for the delay @gojko  - that was a good idea.
Code is cleaner & still works for me.
Thanks, let me know if i can help with anything else.

@gregory thanks, will release this later today then

I can't publish this yet because the current aws-sdk release breaks claudia, I've reported a bug there and waiting on them to release it. as soon as that's done, I'll re-run the tests again and push a new release out

wow - crazy - can you link the bug you reported so that we can track status here?
Thanks for the head's up!

@gregory the bug was https://github.com/aws/aws-sdk-js/issues/2588. I released claudia 5.4.2 just now, with your patch, so upgrading to the latest version should have this PR merged in


Superfluous GraphQL middleware

The Graphene middleware (KeycloakGrapheneMiddleware) does not have any additional functionality over the standard middleware (KeycloakMiddleware). Both of them only add the user object to the request through the request's token. The only difference is that the Graphene middleware checks this for each field in the GraphQL query even though performing this once when the query is received is adequate.
The main reason for this issue is to remove it to avoid confusing future users of the library and simplifying the remaining middleware.

An idea would be to mark the functions of the class as deprecated and only remove it after releasing v2


Proposal: change x to × in title bar resize message

This is really minor... I changed the 'x' (lowercase X) to '×' (multiplication sign) in the title bar resize message. It just looks a little bit nicer, that's all.

Looks good. Thanks for the PR and sorry took me a while to get to this!


Oracle: Combine DateTime member translator.

Client Eval DateTimeOffset.Hour/Minute (See issue#10515)

This fixes Oracle build break.


set default serviceAccountName to k3s upgrader plan on updates

Issue:
Starting Rancher v2.6.0, serviceAccountName for k3s upgrader genericPlan was updated. But genericPlan is used only for creates (generateMasterPlan and generateWorkerPlan). On updates (configureMasterPlan and configureWorkerPlan), it picks the name from existing plan so it gets the old svc account name if the plans were created before Rancher 2.6.
https://github.com/rancher/rancher/issues/35797

Testing another use case....

Tested on downstream k3s clusters and local k3s cluster.


Padding Length Validation

Title: Validate Padding Length during Decryption
Description:

Package: encryption
Function: DecryptFile(encryptedFilename string, key []byte, toDisk bool)
Line: 132
Before trimming the padding from plaintext, validate if the paddingLength is within expected bounds to prevent potential issues or panics.

Re-writing ticket


0.0.2

Changes:

Fixed a bug in expression decomposition.
Added an option for maximal numbers of post-processing.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

qhdopt/_version.py
0.00%
1 Missing :warning:

qhdopt/qhd.py
80.00%
1 Missing :warning:

qhdopt/utils/function_preprocessing_utils.py
90.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

qhdopt/_version.py
0.00% <0.00%> (ø)

qhdopt/qhd.py
82.75% <80.00%> (-0.07%)
:arrow_down:

qhdopt/utils/function_preprocessing_utils.py
81.53% <90.00%> (+1.18%)
:arrow_up:


chore: Filter entries to sentry

Send only ERRORs as events and log only log messages above WARNING level.

Warnings aren’t very critical, but might be worth checking what they are used for.
Currently, warnings include conditions like OAuth2 misconfiguration at OAuth2 callback stage:
https://github.com/ietf-tools/bibxml-service/blob/1c669e97185fce7749da1ae6ad47b1fffaa48d84/datatracker/oauth.py#L307-L320

@strogonoff thanks for spotting that.
There seems to be other use of warning but that OAuth2 warning is going to happen for each request if OAuth is disabled, correct?
$ grep -A1 -r log.warning
common/query_profiler.py:            log.warning(
common/query_profiler.py-                "Query took %s seconds (too long): %s (params %s)",
--
sources/indexable.py:            log.warning(
sources/indexable.py-                "Attempt to register source with same ID: %s",
--
sources/indexable.py:            log.warning(
sources/indexable.py-                "Error indexing item %s in %s (and no error handler): %s",
--
datatracker/internet_drafts.py:        log.warning(
datatracker/internet_drafts.py-            "Unable to retrieve complete Internet Draft metadata: "
--
datatracker/oauth.py:        log.warning(
datatracker/oauth.py-            "Datatracker OAuth2: client ID/secret not configured, "
--
datatracker/oauth.py:        log.warning(
datatracker/oauth.py-            "Datatracker OAuth2 callback: Datatracker OAuth2 misconfigured, "
--
datatracker/oauth.py:        log.warning(
datatracker/oauth.py-            "Datatracker OAuth2 callback: no state in session, "
--
main/views.py:        log.warning("Could not locate item by docid: %s, %s", docid, doctype)
main/views.py-        metrics.gui_bibitem_hits.labels(docid, 'fallback_to_search').inc()

No, only for OAuth2 callbacks. They shouldn’t happen if OAuth2 was not enabled in the first place, of course, so they would likely be bots.

On 2 Sep 2022, at 6:49 PM, Kesara Rathnayake @.***> wrote:

@strogonoff thanks for spotting that.
There seems to be other use of warning but that OAuth2 warning is going to happen for each request if OAuth is disabled, correct?
$ grep -A1 -r log.warning
common/query_profiler.py:            log.warning(
common/query_profiler.py-                "Query took %s seconds (too long): %s (params %s)",
sources/indexable.py:            log.warning(
sources/indexable.py-                "Attempt to register source with same ID: %s",
sources/indexable.py:            log.warning(
sources/indexable.py-                "Error indexing item %s in %s (and no error handler): %s",
datatracker/internet_drafts.py:        log.warning(
datatracker/internet_drafts.py-            "Unable to retrieve complete Internet Draft metadata: "
datatracker/oauth.py:        log.warning(
datatracker/oauth.py-            "Datatracker OAuth2: client ID/secret not configured, "
datatracker/oauth.py:        log.warning(
datatracker/oauth.py-            "Datatracker OAuth2 callback: Datatracker OAuth2 misconfigured, "
datatracker/oauth.py:        log.warning(
datatracker/oauth.py-            "Datatracker OAuth2 callback: no state in session, "
main/views.py:        log.warning("Could not locate item by docid: %s, %s", docid, doctype)
main/views.py-        metrics.gui_bibitem_hits.labels(docid, 'fallback_to_search').inc()
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


(P)react component support

Is your feature request related to a problem? Please describe.
I found issue https://github.com/tsl0922/ttyd/issues/492 after I finished the following words.
The issue said that being a react library is not intended, but preact version is also an option to me.
Therefore I wonder if that could happen or not.

I want to use App from ttyd/html/src/components/app.tsx in my react project, and put it inside a container instead of using up a page.
Having little knowledge to Node.js/React, I tried to do this by

webpack'ing ttyd/html/src/components/app.tsx into ttyd/html/dist/ttyd.js, and
setting main in package.json to ./dist/ttyd.js.
It does produce ./dist/ttyd.js, but it still doesn't work.
I tried using node to import it, but it failed in loading.

Welcome to Node.js v15.13.0.
Type ".help" for more information.
> try { require('./dist/ttyd'); } catch(e) { console.log(e.stack); }
ReferenceError: window is not defined
    at Object.<anonymous> (/ttyd/html/node_modules/xterm-addon-web-links/lib/xterm-addon-web-links.js:1:208)
    at Module._compile (node:internal/modules/cjs/loader:1092:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1121:10)
    at Module.load (node:internal/modules/cjs/loader:972:32)
    at Function.Module._load (node:internal/modules/cjs/loader:813:14)
    at Module.require (node:internal/modules/cjs/loader:996:19)
    at require (node:internal/modules/cjs/helpers:92:18)
    at eval (webpack:///external_%22xterm-addon-web-links%22?:1:18)
    at Object.xterm-addon-web-links (/ttyd/html/dist/ttyd.js:267:1)
    at __webpack_require__ (/ttyd/html/dist/ttyd.js:20:30)

Describe the solution you'd like
Something like this.
import { App } from 'ttyd';
function MyComp {
    return <App prop1="xxx" />;
}

Describe alternatives you've considered
None, but not limited to the one mentioned above.
Additional context
None.

Nope. You can still copy the code to your project, or load it with iframe.

ReferenceError: window is not defined

you should load it on a web browser.


error when no permission to find services on remote server

regenerate error to collect data and console output.


Suspect that leave-redirect does not migrate children

See: https://github.com/symfony-cmf/RoutingAuto/blob/master/DefunctRouteHandler/LeaveRedirectDefunctRouteHandler.php#L42
It would seem that the redirect status is set on the old auto route, but the children from the previous route stay where they were.

its a question of semantic whether you want to move the children as well or keep them where they are. if you move the whole subtree, maybe using the variable pattern part could help, to match anything below a moved route and redirect it.

I think moving the children is a requirement really. If you don't then your route tree is quickly going to become a mess and, in anycase, the routes would progressively migrate themselves to the new location!

@dantleech do you have time to do this within the next coming weeks, so it can be included in 1.1?

I will try and have a look this week.

This is done.


Update UpdatedDate fields where we've forgotten to

Mostly in GraphQL mutations involving project membership.
Fixes #579.

Nice job, I'm not sure how we should handle updated date with membership changes. Let's discuss in the next team meeting.

I'm guessing that User.LastActive is also a bit neglected.
We're currently updating it when the user logs in. Maybe it should be called 'LastLogin', but it's a bit late for that.
But it probably makes sense to update it in LoginController.RefreshJwt, so that it gets updates when users login via forgot password and verify-email.

We didn't come to a great consensus on when we should update things. Let's get this merged in after resolving the pending comments @rmunn .

@myieye - Done in commit https://github.com/sillsdev/languageforge-lexbox/pull/601/commits/82db7115e1055e82fc34bebfeaaebf9e7c77f3f8. I put it in LexAuthService.RefreshUser() rather than LoginController.RefreshJwt() because otherwise LexAuthService.RefreshUser() would have had to make another DB load in order to get the user entity, whereas it's already available in RefreshUser(). (Or I could have changed RefreshUser() to pass both the AuthUser and the User back, but that's more change than it's worth).


Alphabetize lists

Hi Brett! Just stumbled upon this astonishing collection of yours! I'm seriously blown away.
Would you consider sorting the lists in alphabetic order?
At least for me it's makes digesting the content easier.
E.g. to something like this:
## Miscellaneous

- [A Visual Survey of Text Visualization Techniques](https://textvis.lnu.se/)
- [Amazon Web Services open-sources biological knowledge graph to fight COVID-19](https://www.amazon.science/blog/amazon-web-services-open-sources-biological-knowledge-graph-to-fight-covid-19)
- [ArchiveBox](https://github.com/pirate/ArchiveBox) - An open source self-hosted internet archiving solution.
- [Artificial Brain Networked notebook](https://www.notion.so/Artificial-Brain-Networked-notebook-a131b468fc6f43218fb8105430304709)
- [Automated Knowledge Base Construction conference](https://www.akbc.ws/2020/)
- [Contextualise thread on Hacker News](https://news.ycombinator.com/item?id=22282583)
- [Debate: Roam Research vs. Notion](https://capiche.com/q/debate-roam-research-vs-notion)
- [Digital Gardening](https://github.com/MaggieAppleton/digital-gardeners) - A collective of gardeners publicly tending their digital notes on the interwebs.
- [Ontology is Overrated: Categories, Links, and Tags thread on Hacker News](https://news.ycombinator.com/item?id=18972861)
- [Second-Brain](https://github.com/KasperZutterman/Second-Brain) - A curated list of awesome public Zettelkastens / Second Brains / Digital Gardens.
- [treevis.net - A Visual Bibliography of Tree Visualization](https://treevis.net/)
- [Visualizing Group Structures in Graphs](http://groups-in-graphs.corinna-vehlow.com/)

Thanks for the feedback! Regarding sorting the list in alphabetic order... it's something to, at least, consider. The current approach is that entries are added to the bottom of their respective section. There's something to say for that approach, as well :)


Multiple/parallel download

Feature request: make it possible to download multiple files without having to wait for the previous video conversion process to finish

This got requested like 10 times 😄
Part of #46 and a potential rewrite of TL.


SBT dependency for 2.12 artifact

I've added the dependency
val cron4s = "com.github.alonsodomin.cron4s" %% "cron4s" % "0.3.0"

and when SBT tries to resolve it, it fails on
[warn] 	Note: Unresolved dependencies path:
[warn] 		com.github.alonsodomin.cron4s:cron4s-tests_2.12:0.3.0 (/Users/larsa/Code/portfolio/build.sbt#L74-75)
[warn] 		  +- com.github.alonsodomin.cron4s:cron4s_2.12:0.3.0 (/Users/larsa/Code/portfolio/build.sbt#L74-75)

I see cron4s-testkit_2.12 on mvnrepository.com, but not cron4s-tests_2.12.
I suspect I may only need to pull down cron4s-core though, which resolves fine.

This is, unfortunately, a sloppy SBT configuration problem in cron4s side. The fix for it has already been merged in #45.
I still have not published a new release but in the meantime the solution is simple, use the following in SBT when declaring the dependency:
val cron4s = "com.github.alonsodomin.cron4s" %% "cron4s-core" % "0.3.0"

The tests won't get published into Maven Central as there is no value in for end users but next release won't be referencing it.
Sorry for this.

Not a big problem. I'm happy there's a 2.12-artifact to use now :)


Fix collision detection when objects are intersecting.

Fixes  #1980

I updated the little playground I made for testing colide_aabb a while back and hooked it up to this PR.
https://github.com/rparrett/bevy-test/tree/collision/src
This seems to work well.
But this PR also turns the "ball can escape in breakout" situation into a "breakout can occasionally panic" situation, right?

bors try

Coud you run cargo fmt to fix formatting?
And cargo run -p ci to also check clippy if possible 😃

done, thanks

bors try

Hit this earlier, is anything blocking this PR? Licensing agreements?

@geckoblu could you comment on https://github.com/bevyengine/bevy/issues/2373 if you agree with the license change since you opened this PR?
There's also the question opened in https://github.com/bevyengine/bevy/pull/2489#issuecomment-881497439: is Intersecting the right name for this case?

Hi,
I commented on #2373 approving the dual license.
#2489 is probably a cleaner solution for the same problem, so if #2489 is merged this could be closed.

Closing in favor of #2489


Expose ImageDeprecationStatus Property in Virtual Machine Image list cmdlet

Preconditions

[X] No need to upgrade Python SDK or the Python SDK is ready.

Related command
az vm image list
Resource Provider
Microsoft.Compute/ComputeRP
Description of Feature or Work Requested
We want to expose imageDeprecationStatus property also in the above cmdlet. Currently, this cmdlet is calling REST API with this request url having expand param equals to properties:
https://management.azure.com/subscriptions/823ca539-d44d-43ee-8dc8-023fd4f27396/providers/Microsoft.Compute/locations/eastus/publishers/center-for-internet-security-inc/artifacttypes/vmimage/offers/cis-rocky/skus/cis-rockylinux-9-l1-gen2-azure-observability/versions?$expand=properties&api-version=2023-09-01
We want it to call with this request url having expand param equals to properties/imageDeprecationStatus:
https://management.azure.com/subscriptions/823ca539-d44d-43ee-8dc8-023fd4f27396/providers/Microsoft.Compute/locations/eastus/publishers/center-for-internet-security-inc/artifacttypes/vmimage/offers/cis-rocky/skus/cis-rockylinux-9-l1-gen2-azure-observability/versions?$expand=properties/imageDeprecationStatus&api-version=2023-09-01
The new request url will expose the imageDeprecationStatus property as well with this cmdlet.
Minimum API Version Required
2022-11-01
Swagger PR link / SDK link

Link to the OpenAPI (swagger) spec

https://github.com/RajalakshmiSwaminathan/azure-rest-api-specs/blob/e5f4f0f98ed6417790f8ea0db56b20a3f71dbeff/specification/compute/resource-manager/Microsoft.Compute/ComputeRP/stable/2022-11-01/virtualMachineImage.json

Link to any other Github work related to this request

Swagger PR: https://github.com/Azure/azure-rest-api-specs/pull/21615
SDK PR: https://github.com/Azure/azure-sdk-for-net/pull/32660

Request Example
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/compute/resource-manager/Microsoft.Compute/ComputeRP/stable/2022-11-01/virtualMachineImage.json
Target Date
2024-09-30
PM Contact
<EMAIL_ADDRESS>Engineer Contact
<EMAIL_ADDRESS>Additional context
This request is to just include one more property in list cmd. This change is of 0 risk because we are not removing any existing property but want to show one more property i.e 'imageDeprecationStatus'

@nidhjain-ms1 May I ask if this feature request is related to security? Please note that currently, due to the CLI team being busy with the security wave and prioritizing security in this semester, so all the feature requests unrelated to security will be postponed. cc PM @Jacekey23

Hi @zhoxing-ms , this is not directly related to any particular security wave, but yes, it is related to security in a way that deprecated images may be end of life and insecure so allowing customers to easily find and exclude images that are deprecated/scheduled for deprecation would prevent them from using such images.

@nidhjain-ms1 Sorry, due to our recent busy schedule, I am concerned that we may not have enough effort to do work outside of safety waves. But I understand that this feature is indirectly related to security. May I ask if this feature is urgent? If you can provide some business evidence, we will discuss with the PM @Jacekey23 whether to prioritize it

@zhoxing-ms @Jacekey23 I am the PM for this feature. The business justification is that we want to ensure customers move away from images that are deprecated/scheduled for deprecation as these images might be End of life, lack security patches and pose security risks to Azure customers. A lot of customers have requested for a way to list and exclude images that not scheduled for deprecation or already deprecated via the CLI. This is why need the Image deprecation status parameter added to the az vm list command. I hope this explains it clearly. Let me know if you have any further questions

Hi @zhoxing-ms , can you please provide any updates/timelines for this issue. Please let me know, if any other information is required.

@nidhjain-ms1 OK, we will plan this feature in this sprint, it will be released on 10-08

Hi @zhoxing-ms , would it be released on 10-08? I see in the PR, milestone to be mentioned as 11-05..

@nidhjain-ms1 It should be released today


[Quick] Gate round_start_delay sleep behind #[cfg(test)]

This PR:
Gates usage of round_start_delay behind #[cfg(test)].
This PR does not:
There wasn't a clear path to removing it entirely without breaking tests or adding a feature flag. This should just prevent us from inadvertently having consensus sleep in production.
Key places to review:

It looks like #[cfg(test)] doesn't work the way I thought it did. Closing this for now.


HDDS-3782. Remove podAntiAffinity from datanode-statefulset

What changes were proposed in this pull request?
Now, i think the yaml files under examples folder are trial purpose, so we should not add the strict limitation to prevent people easy to taste the example, if who want reference this example and use it in production env, then they will do a lot of modification.
For this ticket, it suppose to remove the podAntiAffinity check to let on node kubernetes run the ozone cluster on kubernetes successfully.
What is the link to the Apache JIRA
https://jira.apache.org/jira/browse/HDDS-3782
How was this patch tested?
No need, just loosing restrictions.

Codecov Report

Merging #1057 into master will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1057      +/-   ##
============================================
- Coverage     69.48%   69.46%   -0.03%
  Complexity     9110     9110
============================================
  Files           961      961
  Lines         48132    48148      +16
  Branches       4672     4679       +7
============================================
- Hits          33446    33445       -1
- Misses        12468    12481      +13
- Partials       2218     2222       +4

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/hadoop/hdds/scm/pipeline/Pipeline.java
85.23% <100.00%> (-0.27%)
44.00 <0.00> (+1.00)
:arrow_down:

.../org/apache/hadoop/ozone/om/helpers/OmKeyInfo.java
86.25% <100.00%> (+0.33%)
42.00 <0.00> (+2.00)

...adoop/ozone/om/helpers/OmKeyLocationInfoGroup.java
75.60% <100.00%> (+1.25%)
12.00 <2.00> (+1.00)

...g/apache/hadoop/ozone/om/helpers/OzoneAclUtil.java
81.91% <100.00%> (+1.23%)
39.00 <4.00> (+2.00)

...adoop/ozone/om/request/key/OMKeyCommitRequest.java
97.00% <100.00%> (ø)
18.00 <0.00> (+1.00)

...otocol/commands/RetriableDatanodeEventWatcher.java
55.55% <0.00%> (-44.45%)
3.00% <0.00%> (-1.00%)

...ache/hadoop/ozone/om/codec/S3SecretValueCodec.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (-1.00%)

...apache/hadoop/hdds/scm/block/BlockManagerImpl.java
66.66% <0.00%> (-5.41%)
18.00% <0.00%> (ø%)

...e/commandhandler/CreatePipelineCommandHandler.java
87.23% <0.00%> (-4.26%)
8.00% <0.00%> (ø%)

...apache/hadoop/hdds/server/events/EventWatcher.java
77.77% <0.00%> (-4.17%)
14.00% <0.00%> (ø%)

... and 16 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 53395a0...77253c3. Read the comment docs.

Thanks to open this issue @maobaolong
This approach is already considered, and you can find the arguments in https://issues.apache.org/jira/browse/HDDS-1646 why do we have these persistence lines.

Now, i think the yaml files under examples folder are trial purpose

Depends on how do you define trial purpose. I think the provided examples should work in a real, production-like environment. For example if somebody tries Ozone out in a small Kubernetes cluster, we should provide an example how can it be configured.
Removing the podAntiAffinity rules would result fake results of the initial measurements.
I understand that we have an other use case: To quickly try it out locally on Minikube, K3s or one node cluster. And I agree that we should support it. Today we have the minikube definitions for that specific use case.

I am sure that the real ozone example should have real podAntiAffinity as it provides an example to use real Ozone
minikube shouldn't have the podAntiAffinity
We can discuss getting-started and ozone-dev, I am open the adjust them.
But it seems to unclear what is the goal of different subdirectories. Would be great to do it with additional clarification: either we need to add more README to clarify the goals, or create different structure (or dir names?)

@elek I agree with your suggestion,

I am sure that the real ozone example should have real podAntiAffinity as it provides an example to use real Ozone

Thank you to let me know well about the "ozone" and "ozone-dev" mean, now i agree to keep the podAntiAffinity for ozone dir

minikube shouldn't have the podAntiAffinity
If fact, i don't know minikube at all, keep it.

We can discuss getting-started and ozone-dev, I am open the adjust them.
I strongly recommend to remove  podAntiAffinity and let it easy to start successfully.

But it seems to unclear what is the goal of different subdirectories. Would be great to do it with additional clarification: either we need to add more README to clarify the goals, or create different structure (or dir names?)

Yeah, I have created a ticket to track this work, https://issues.apache.org/jira/browse/HDDS-3790
maybe someone who are very familiar with these directories can write a better README
So, for this PR, i suppose to let ozone CSI run successfully and easily, so i will revert the unrelated and un unnecessary file.

Thanks the update @maobaolong
My problem is that the examples are generated (see #1123). If you modify only the generated files, the changes will disappear.
If you propose to modify ozone-dev to run it on one single node by default, you should do the following:

copy this file to hadoop-ozone/dist/src/main/k8s/definitions/ozone/definitions

Add ozone/onenode transformations to the Flekszible file under ozone-dev and getting-started.

For example:
diff --git hadoop-ozone/dist/src/main/k8s/examples/ozone-dev/Flekszible hadoop-ozone/dist/src/main/k8s/examples/ozone-dev/Flekszible
index 96e8c6254..350ea73c5 100644
--- hadoop-ozone/dist/src/main/k8s/examples/ozone-dev/Flekszible
+++ hadoop-ozone/dist/src/main/k8s/examples/ozone-dev/Flekszible
@@ -26,6 +26,7 @@ import:
     - type: ozone/profiler
     - type: ozone/emptydir
     - type: ozone/csi
+    - type: ozone/onenode
   - path: prometheus
   - path: jaeger
     transformations:

Run regenerate-all.sh from #1123 (You might need to install flekszible which can be done with brew or putting the single binary to the path)

Update: when you import the file, the name of the statefulset should be updated in the trigger lines (datanode instead of ozone-datanode AFAIR)

@elek Thank you for telling me how to change the example by Flekszible,  but, in fact, i still want to start 3 datanode by default, and remove the affinity limitation, so that I can start 3 datanodes on an only one node kubernetes cluster.

I got it, and I convinced by you to remove the affinity rules from ozone-dev and getting-started and we agreed to keep it in ozone (as it supposed to be a production-like example).
My problem is that the files under example are generated to simplify the management. For example, we can add any new labels to the master template and regenerate all the examples.(with #1123).
I am convinced to remove the affinity rules by default, but if you do it only in the generated files, they will be overridden with the next generation. (To make all the files independent of the generation, I prefer to commit the end results, what you can see in the examples which makes the existence of the generation tool optional).
I tried to propose to modify BOTH the generated files AND the rules for the generation, to persist the change. It was not about following a different default.
(Sorry if it was not clear or misunderstood your comment)

With other words:
Try this: keep this commit, apply #1123, and run the new script. You will see that your changes are disappearing. I described the method to avoid this and save your changes ;-)

@elek Thank you for your help, i catch you point is let me modify the source file of the Flekszible. I think I've done following your instruments. PTAL.


Add progress bar for upload progress

This replaces the "Uploading XX.X%" info string with a progress bar similar to the cloud builder, along these lines:
[Info]    [========>                                          ] 17.4%

Change-Type: patch

One more thing, sorry, you also need a changelog for this. The CLI doesn't have versionist yet, it's manual changelog updates for the moment.


Allow a client to configure/enable its preferred method of SD-JWT-VC signing key resolution via a builder.

Makes the constructor of SdJwtVcVerifier private
Introduces a new fluent builder instantiated via SdJwtVcVerifier.builder() to allow clients to enable the desired resolution strategies. As a bare minimum the builder requires either SD-JWT-VC Issuer metadata or X509 Certificate trust to be configured. DID resolution is not required and can be enabled/configured alongside any other resolution strategy.

Closes #248

@babisRoutis What do you think?


Make the Feature Flags Root Section Card Scrollable

Is your feature request related to a problem? Please describe.
Currently, the feature flags root section card expands infinitely as the list grows, making the top UI elements (heading, search field, buttons) invisible when scrolled to the bottom.
Describe the solution you'd like
The feature flags root section card should have a maximum height so that its content section can be scrollable. This makes the top UI elements always visible, which is better for the user experience.
Additional context
We might consider also applying this to other section cards on the details page later if needed.

I will QA this

QA passed


Improve feedback for correctness tests

Work-in-progress. Needs tests. Looking for initial feedback.

Updated to address most of the comments. Main thing left is to perhaps merge OutputSnippetDirective.js with SyntaxErrorSnippetDirective.js.

Addressed most comments. Outstanding: (a) see about refactoring the output logic to make it more readable, (b) investigate why feedback goes back and forth between suite level feedback and correctness feedback, (c) add tests

(Also, please note that e2e tests are failing.)

@seanlip PTAL.

Hi @rabidbit -- I'll take a look at this within the next hour, but FYI you might want to take a look at the codecov reports in the meantime. It looks like test coverage has dropped quite a bit with this change, so it might be worth taking a look at the red bits and ensuring those are covered by tests. Thanks!

Also, @rabidbit -- please could you address my most recent round of comments? https://github.com/google/tie/pull/438/#pullrequestreview-73621377

(Adding @eyurko for review of text in questions.js)

@seanlip Yeah, I see errors when I indent as you requested (and as I originally had them). E.g.
/home/travis/build/google/tie/client/question/services/FeedbackGeneratorService.js
  400:9   error  Expected indentation of 6 spaces but found 8   indent
  403:11  error  Expected indentation of 8 spaces but found 10  indent
  405:11  error  Expected indentation of 8 spaces but found 10  indent
  407:11  error  Expected indentation of 8 spaces but found 10  indent
  409:11  error  Expected indentation of 8 spaces but found 10  indent
  410:9   error  Expected indentation of 6 spaces but found 8   indent
  412:11  error  Expected indentation of 8 spaces but found 10  indent
  414:11  error  Expected indentation of 8 spaces but found 10  indent
  418:11  error  Expected indentation of 8 spaces but found 10  indent
  420:11  error  Expected indentation of 8 spaces but found 10  indent
  421:9   error  Expected indentation of 6 spaces but found 8   indent
  423:11  error  Expected indentation of 8 spaces but found 10  indent
  425:11  error  Expected indentation of 8 spaces but found 10  indent
  429:11  error  Expected indentation of 8 spaces but found 10  indent
  431:11  error  Expected indentation of 8 spaces but found 10  indent

Ah ok, thanks. That's weird, I don't know why they haven't triggered historically. But yeah, in that case, let's do what's needed to make them go away.
Thanks for checking!

PTAL

Think I addressed comments. PTAL.


Error when installing on a Teltonika rutx08

Hi,
When I install this package, I get this error:
`root@Router:~# opkg install netbird_10.4-1_arm_cortex-a7_neon-vfpv4.ipk
Upgrading netbird on root from 0.10.1-1 to 0.10.4-1...
Configuring netbird.
Collected errors:

opkg_conf_parse_file: Duplicate src declaration (openwrt_luci https://downloads.openwrt.org/releases/21.02.0/packages/arm_cortex-a7_neon-vfpv4/vuci). Skipping.`

CPU: Quad-core ARM Cortex A7, 717 MHz
Package: netbird_0.10.4-1_arm_cortex-a7_neon-vfpv4.ipk
If I remove the last line of /etc/opkg/distfeeds.conf it works, but I get the following error when running it:
Error relocating /usr/bin/netbird: __nanosleep_time64: symbol not found

Duplicate of #6


Add Enable PNI Option for AKS Clusters

What kind of request is this (question/bug/enhancement/feature request):
enhancement
In order to support Project Network Isolation for AKS clusters, there needs to be a UI option for Enable Project Network Isolation on the AKS provisioning page AND on the AKS cluster import page.
For RKE clusters, this option appears in the Kubernetes Options section of the provisioning page. Something similar can be done for the AKS import cluster page.
Checking this option, i.e. Enabling PNI, should set enableNetworkPolicy on the cluster object to true

Added here for AKS


Alignment: Corner case when fiducial points aren't detected

From @Strateus in #13:

Brandon, why do you have this test in python alignment warped image?:
if 0 <= l <= w and 0 <= r <= w and 0 <= b <= h and 0 <= t <= h:
There is quite big probability one of the points will be out of this range, dont you think its too strict?

Hi @Strateus - why is there a big probability the points will be outside the range? Do you have examples where this is failing for reasonable images? I haven't extensively profiled or tested this portion of code, but the alignment code works for 13081 out of 13233 images from the LFW dataset.
This code portion checks if the boundary points of the detected landmarks are outside of the image, which should only happen in extreme cases like if somebody's face is outside of the image. There may be heuristics that can improve this, like filling in the missing portions with black pixels or using image completion.

Well, i used my web camera, and added 30% margins to face, and still after warping there were points out of warped image all the time i tried, maybe 5% it was ok, but rest failed.
Anyways i would consider this check to be more thorough since you transform only on eyes and nose. Just thoughts, i elminated this check locally and will be testing it.

Code reference: https://github.com/cmusatyalab/openface/blob/3dba58e0ef1dd15bd9b10c933214d859c51b4e6c/openface/alignment/naive_dlib.py#L140

Be careful when expanding the margins since the neural network nn4.v1.t7 is trained with no margin added. I wouldn't be surprised if this severely hurts the accuracy.
You need to train a new neural network if you want to add margins.
Note that in a few experiments I ran, adding a margin didn't improve the accuracy,
which is why there is no margin in nn4.v1.t7.
Also, won't eliminating the check cause array out of bounds errors in:
cwImg = cv2.resize(warpedImg[t:b, l:r], (size, size))

Yea i got the point about network. Accuracy is not my primary concern now.
As for eliminating, i did it like this:
    l = 0 if l < 0 else l
    r = w if r > w else r
    t = 0 if t < 0 else t
    b = h if b > h else b

Fixed with the new alignment technique.


Task 19, Task 20

@SunkireddyRagini
Task -19
Code verified.
Task -20
Code verified.
Task -22
Code verified.
Task -23
Code verified.
Task -24
Code verified.
Task -25
Code verified.
Task -26
Code verified.


feat: Optimize docker image building

In PR #130, the Docker image size became significantly large. To address this issue, I refactored the Docker build logic into a two-stage build process and modified the Docker Compose file accordingly. |
As a result, the Docker image size has been significantly reduced, making it much lighter.

Thanks!


Support for heroku-18 stack and update FreeTDS version

We are using a build from this branch (roshan92:master) in production, because the current rails-sqlserver:master branch is not working with our heroku-18 stack.

Any updates on merging this PR?

I am so sorry I never saw these

Closed by https://github.com/rails-sqlserver/heroku-buildpack-freetds/pull/8


Add support for client filters

https://docs.aws.amazon.com/kendra/latest/dg/data-source-web-crawler.html

Closing this request.
Addressed in QnABot release 5.1.0.
More details: https://github.com/aws-solutions/aws-qnabot/blob/main/CHANGELOG.md#510---2021-12-09


Make sure the URLClassLoader instances are properly closed

@phillip-kruger I would appreciate if this could land in Quarkus relatively fast.
Thanks!

Actually, maybe wait a bit as I have other fixes coming.

I added a few additional commits, this PR is now ready.
Also I opened https://github.com/quarkusio/quarkus/issues/41868 in the Quarkus repository, and it's slightly related.


Update url.ts

change how we import url

Hmm I feel this might break as well for the client bundle. Can we maybe do const {parse} = require('url'); only in the branch where we know that we are on the server? That should work 100% of the time I think.

The linter complain about it
src/url.ts[1, 14]: require() style import is forbidden

just disabled it for this case?

Yeah, just disable it.


fix(tm): Handling the problem of slow access speed to the testing env…

…ironment
Creating indexes during tm initialization to solve this problem
Closes TAP-2468

for mongodb, gridfs will use files_id to do query, and use n to do sort, so { files_id: 1, n: 1 } is a better choice, and this is in mongodb document: https://www.mongodb.com/zh-cn/docs/v6.0/core/gridfs/
and also, please create index in fs.files: db.fs.files.createIndex( { filename: 1, uploadDate: 1 } );

MongoDB Driver will create indexs on fs.chunks, but on dev environment, this index is missing. Suspect it was manual deleted. When attempting to rebuild, a duplicate index error occurs. Close this pr is the final decision.


Translation updates

Hooray! Lot's of changes!
Translation Settings Page Update

page layout was simplified
there is a lot of helping text again.

New translation id/methods

translation id's are now counted up hex numbers (New method DialogicUtil.get_next_translation_id())
events can now register multiple properties for translation
in the csv files keys are now a combination of event name, translation id and property name (e.g. Text/3a/text, Choice/2f1/disabled_text)
translation settings have been moved into the dialogic/translation subcategory in the project settings (e.g. dialogic/translation/enabled)

Translation methods
The event class now has the following methods for translation:
To be overwritten in (translatable) event scripts:
-  _get_translatable_properties() -> Array
- _get_property_orginial_translation(property_name) -> String
To be used in (translatable) event scripts:
- get_property_transated(property_name) -> String
Other translation methods (only used internally):
- add_translation_id() -> String
- remove_translation_id() -> void
- can_be_translated() -> bool
- get_property_translation_key(property) -> String
Misc:
- update_text_version() -> void ## I've added this method as I had trouble with saving because saving uses the event_node_as_text property that doesn't auto-update when changing properties of events. I hope @thebrd or I can make another saving simplification now that translation has been moved out of the TimelineSaver, so maybe this might be changed later.
Text event and Choice event/Subsystem have been updated to work with the new translation methods.
AutoTranslation updates removed
Translation files are not auto-updated whenever timelines are saved now. This makes the whole translation and saving process a lot simpler. I'm pretty sure just using the buttons is enough.
Misc changes
These are not really related to translations, so I hope it's okay that they are in here
- timeline._to_string() changed to show the path instead of the instance id as it is much more insightful
- complex picker now also searches the tooltip of each item
- file picker now allows to set the mode to FILE_MODE_OPEN_DIR. In that case it will not only show the file name oc
- Added DisabledText option to the Choice event. This text will be shown when the condition is wrong and else mode is "disable". This needed some change in the syntax of the Choices system. It is now:

- Choice Text [if Condition] [else="disable" alt_text="DisabledText"]
The last part is basically a regular shortcode. I've done this more to test multiple translatable properties on one event (so disabled text is translatable as well).
New workflow:
The text below is also present in the translation settings tab. I've pasted it here for anyone wanting to understand how to work with translations/ the menu now.
Setting up translations:
1. Enable translations
2. Set default locale (the language you used) and file mode. For per-project-mode also select a folder. In per-timeline mode files will be put next to timelines (and need to stay there!).
3. Hit "Update CSV files". This will create new csv files containing all translatable strings from your timelnes as well as one for the character names.
4. Wait until reimport is finnished. Then click "Collect translations"

How to work with translations:
Editing:
You can edit the csv files in most table editors (like google sheets or libre office calc). You can add new languages by creating a new column with a valid locale in the first cell. After doing this you will have to hit "Collect translations" again.

Updating:
If you make changes to your timelines after having created translations, just hit "Update CSV files" again, it will not remove changes already made to the file.

Testing:
You can test in different languages by editing the test_locale setting in project settings (Advanced Settings>Internationalization>Locale).

Should fix #1264
Should fix #1171
Should fix #1063

[ ] Settings don't save. After changing something like Per Timeline to Per Project, or  selecting a translation folder, after closing and opening it again things revert to the default state


Add dart executable gn rule for engine tests

Currently many tests in the engine are run via Python scripts. Instead, tests written in Dart should be generated using a gn target just like we do with tests written in C++ or Objective-C.
Output could either be a dart native binary or a shell script (Mac, linux) or ps1/bat (Windows) that invokes the standalone binary on the input Dart files.

I am going to take over this issue after talking with @cbracken.
He has something working here: https://github.com/flutter/engine/pull/52241.
Something we'd want to consider is deps tracking (as-per @zanderso's comment: https://github.com/flutter/engine/pull/52241#discussion_r1575461664).


Fix Traffic Ops GET statuses to allow null descriptions

What does this PR do?
Fix Traffic Ops GET statuses to allow null descriptions
Includes API Tests.
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[x] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Run API tests.
Check all that apply

[x] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2975/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3286/
Test PASSed.


Quick docs fix

Uses cargo_dist instead of cargo-dist
closes https://github.com/axodotdev/oranda/issues/139

We do give an error saying that is not a known name


mgmt, update test on chaos

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - chaos - ci

/azp run java - chaos - mgmt - tests


fix(input): horizontal overflow in IE and Edge

Currently input labels are set to 133% width and are scaled down to fit into their container. This works fine in most browsers, however it causes horizontal overflows on IE and Edge, because they don't consider the transform when determining whether an element overflows. This change fixes it by hiding all of the overflowing content.
@mmalerba I didn't notice any issues when going through the input container demo page, but do you think that this may have some unexpected side-effects?

In most cases this will be fine, but users are allowed to put custom content inside the md-input-container and if they try to put some absolutely positioned thing that intentionally overflows, this would clip it. Could we instead put a wrapper around the label and just clip that?

That should work as well. I'll give it a try.

If it's just a div with overflow: hidden, it doesn't help since the label's dimensions are based on the closest element that's relative. The other approach that I tried was having a wrapper that is position: absolute instead, but that ends up clipping the label when the input is focused.

@jelbourn how do you feel about this?

@crisbeto Just to make sure I'm looking at the same bug, are you referring to the fact that when it's animating the placeholder overflows? because it looks like after it finishes animating its not a problem. how do you feel about this as an alternate solution? https://github.com/angular/material2/compare/master...mmalerba:input-overflow

It's not only when animating it, it happens when the input is floating since it gets expanded to 133% and Edge/IE don't respect the 0.75 scale. Here's what it looks like for me on load in Edge (note the scrollbar at the bottom):

That works pretty well @mmalerba, although I'm not a huge fan of the  . I tried working around it with a pseudo element, but it didn't work. I can play around with it tomorrow.

@mmalerba I re-did it, based on your changes. I worked around the   with a pseudo element so we can keep everything inside the CSS. Could you re-review?

@crisbeto Can you rebase?

@crisbeto Rebase again? Sorry


MINOR: onCompletion: metadata valid only if no exception

Modifies example in doc change

LGTM

LGTM and merged to trunk.


Remote Play Desync after inactive on ethernet

After maybe 60 seconds of inactivty, the client's video desyncs from the stream..if you move the stick you can hear the cursor moving around, but the video shows no change in the image.  (seems to happen even sooner/more frequently on wifi)

What platform are you using xbox-xcloud-client on, and what version xbox console?

Tried on both Linux and Windows.  Xbox Series S.

Please try v2.0.0-beta1 and check if that fixes the problems. The video and audio streams has been greatly improved.

Please try v2.0.0-beta1 and check if that fixes the problems. The video and audio streams has been greatly improved.

So it is more stable, but I'm getting more artifacts in the stream and sometimes the decidubg entire screen turns green, etc.

I've noticed the lag on wifi (both devices connected with same network and 5 Hhz) almost from the very beginning (about 10 seconds after connection to console)
tried v2.0.0-beta1 - it does not allow to login (as I'm not in a supported country)
v1.2.0 - works fine, except the video lag
tried options:
changed server, connected controller with cable to client device, connected controller with radio to xbox - same result. The sound of buttons is instant, but not picture.
The strange behavior is that the stream itself works fine, no artifacts or lost frames (launched Doom 1993 and waited until gameplay demo)
OS: Linux
Installed with Appimage

logs
Bootstrapping Xbox-xCloud-Client using args: [ '/tmp/.mount_Xbox-xcAxCfV/xbox-xcloud', '--help' ]
Updater.ts - Checking for updates...
mesa: for the --simplifycfg-sink-common option: may only occur zero or one times!
mesa: for the --global-isel-abort option: may only occur zero or one times!
mesa: for the --amdgpu-atomic-optimizations option: may only occur zero or one times!
tokenStore: e {
  _web: { uhs: '', userToken: '' },
  _streamingToken: '',
  _xCloudStreamingToken: '',
  _xCloudRegionHost: '',
  _xCloudRegions: [],
  _msalToken: '',
  _eventOnWebToken: [ [Function (anonymous)] ],
  _eventOnStreamingToken: [ [Function (anonymous)] ],
  _eventOnxCloudStreamingToken: [ [Function (anonymous)] ],
  _eventOnMSALToken: [ [Function (anonymous)] ],
  _eventOnMSALData: [ [Function (anonymous)] ]
}
Updater.ts - Got response. Current version: 1.2.0
Updater.ts - We got the latest version already.
web tokens set
Requesting xHome and xCloud tokens..
- Error while retrieving xCloud token
  statuscode: 403
  body: {"code":"InvalidCountry","statusCode":403,"message":""}
xhome tokens set

Is this issue still relevant on the newest beta release? (v2.0.0-beta6)


Provider unable to create AWX client

Currently working on trying to get the awx terraform provider to work with awx version 21.3.0. But when I do I get this error:

My config is like this:
terraform {
  required_version = ">= 1.0.0"
  required_providers {
    awx = {
      source  = "denouche/awx"
      version = "0.15.6"
    }
  }
}

provider "awx" {
  hostname = "https://my.awx.domain:443"
  username = "my_user"
  password = "my_passw"
}

Again, using AWX version 21.3.0 being deployed to kubernetes via their helm chart. I've looked at this 100 different ways but Im not sure how it wouldn't work lol. Let me know if there's anything I'm missing or if there's more ways to test this

Any help on this? @denouche

Hello @SomniVertix , sorry for the late answer.
We use the exact same thing with awx version 21.5.0 and it works great.
Could you retry after updating this terraform provider to the last version (0.19.0) ?

Hi @SomniVertix on our case it got solved by adding insecure=true.


Type error with global app.derive followed by onError

What version of Elysia.JS is running?
1.1.0
What platform is your computer?
WSL Ubuntu
What steps can reproduce the bug?
import { Elysia } from "elysia";

new Elysia()
  .derive({ as: "global" }, () => ({
    logger: {
      log(msg: string) {},
    },
  }))
  .onError((ctx) => {
    ctx.logger?.log("yay");
//        ^ Property 'logger' does not exist on type ...
  });

What is the expected behavior?
no type error
What do you see instead?
type error (see comment)
Additional information
Everything besides global works fine.
reported on discord

Should have been fixed with 69eb72a, published under 1.1.2

@SaltyAom, yes, it fixed it but there's a follow up error:
import { Elysia } from "elysia";

export const logger = new Elysia({ name: "logger" }).derive(
  { as: "global" },
  () => ({
    logger: {
      log(msg: string) {
        console.log(msg);
      },
    },
  })
);

export const error = new Elysia({ name: "error" })
  .use(logger)
  .error({
    Error,
  })
  .onError({ as: "global" }, (ctx) => {
    ctx.logger.log(ctx.code);
//        ^ Property 'logger' does not exist on type ...
  });

new Elysia()
  .use(error)
  .get("/", () => {
    throw new Error("whelp");
  })
  .listen(8080);

The logger exists at runtime but type error as shown in comment.
Open http://localhost:8080/ in browser

Should have been fixed with https://github.com/elysiajs/elysia/commit/20431a290e1a1339a380253e3c10a068f4a131b1, published under 1.1.3

Thanks, works for me


Search in directory: search in file with Windows-1251 (cyrillic) encoding

Description
When you try to search for a latin substring in the list of files containing latin and cyrillic characters and saved in windows-1251 encoding, the search engine does not return any result.
Steps to Reproduce

Create file with containing text:
some text with кирилическими символами
Save file with windows-1251 encoding in some directory
Try to search in this directory substring some text with no opening file on new page.

Expected behavior: returns a list of files containing the searched substring
Actual behavior: the search engine does not return any result
Reproduces how often: always
Versions
OS: Windows 10
Atom: 1.14.2

I think I was able to reproduce but to clarify:

Save file with windows-1251 encoding in some directory

Can you confirm how you perform this step?

Yes, the bug is present. It is caused by bug in node module 'scandal'. It simply opens files for search in wrong codepage or even worse - as a binary...

to @rsese

Can you confirm how you perform this step?

Sure, you can do this in Atom, just open new file, select encoding windows-1251 (on the bottom of Atom panel), insert into file content some text with кирилическими символами, save them in some folder. Now you can try Search in direcory phrase 'some text'.

@rsese I found three issues related to find-and-replace not working with various encodings. https://github.com/atom/find-and-replace/issues/741 and https://github.com/atom/find-and-replace/issues/751 that can be duplicates for this and https://github.com/atom/find-and-replace/issues/819 that is possibly related.

Thanks for the pointers @Ben3eeE :zap:   Not sure why I didn't search the find-and-replace package....those do seem like basically the same issue.
We'll go ahead and close this as a duplicate @st1gus but you can subscribe to any of those issues above if you want to follow along.


Issue with .jfif file formats

Images in .jfif format

Upload occurs normally, it is sent and the loading icon shows.
After processing the image return is crashed

In the image url i get /_next/image?url=Failed%20to%20restore%20image&w=1080&q=75
When I sent the same image converted to png it worked fine.

it looks like the app isn't able to handle an error. i tried to fix it if the code gets merged it should be fixed on your end as well


Does the multipart Form need 'static strings for everything?

I would like to dynamically set the multipart::Form.text() using a String object, but this is quite hard since .text() requires a 'static str. Is this really necessary and am I missing a step to properly pass a non-static string?
Here is an example:
// These will be dynamically filled
let url = Url::parse("https://test.com/image.png");
let mut buf: Vec<u8> = vec![];

let form = reqwest::multipart::Form::new()
                .text("desired_filename", url.path())
                .part("upload", reqwest::multipart::Part::bytes(buf));

Oh sorry for misunderstanding.
But then it consumes the String, wouldn't a &str reference be better?

In cases where a String will need to be created, it's usually considered better to allow the user to create it, than to ask for &str and copy it. This is because it's quite likely a user already created a String, probably from user input, even just simply let text = format!("something-{}", counter);. It'd be wasteful to take &str and make another copy.
If the user needs to keep the String, they can keep a clone.


Add django utility functions

Add poetry config and functions for managing django settings.DATABASES and settings.ALLOWED_HOSTS

Is the idea that we'll package this up like we've done with copilot-tools and have individual projects add it to their requirements as needed?

Yep, exactly.  It'll replace dj-database-url.

Is there a ticket to go with this pull requests so I can get a bit of context for what, why etc?


mupdf: Refactor desktop item

Description of changes
I refactor the expression to use makeDesktopItem and the copyDesktopItems hook so we don't have to manually craft and copy the desktop item.
I also improved the desktop item by adding the available icon and appropriate categories and keywords gleaned from the Debian package.
Generation of the desktop item is now conditional on either of the viewer binaries being built, rather than tied to mupdf-x11. mupdf-gl supercedes the former on Linux and on macOS the desktop item can be used to synthesize an application bundle.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

1 package marked as broken and skipped:

beamerpresenter

9 packages failed to build:

img2pdf (python39Packages.img2pdf)
mcomix3
mupdf
ocrmypdf (python39Packages.ocrmypdf)
paperless-ng
python310Packages.img2pdf
python310Packages.ocrmypdf
python310Packages.pikepdf
python39Packages.pikepdf

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

1 package marked as broken and skipped:

beamerpresenter

3 packages failed to build:

ocrmypdf (python39Packages.ocrmypdf)
paperless-ng
python310Packages.ocrmypdf

6 packages built:

img2pdf (python39Packages.img2pdf)
mcomix3
mupdf
python310Packages.img2pdf
python310Packages.pikepdf
python39Packages.pikepdf

Rebased to resolve the merge conflict due to one of my other MuPDF PRs.

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

1 package marked as broken and skipped:

beamerpresenter

9 packages failed to build:

img2pdf (python39Packages.img2pdf)
mcomix3
mupdf
ocrmypdf (python39Packages.ocrmypdf)
paperless-ng
python310Packages.img2pdf
python310Packages.ocrmypdf
python310Packages.pikepdf
python39Packages.pikepdf

$ nix log /nix/store/q16llcnl01lr6injlp4959ihqwhd6c27-mupdf-1.19.0.drv | ix
http://ix.io/3ToK

$ nix log /nix/store/q16llcnl01lr6injlp4959ihqwhd6c27-mupdf-1.19.0.drv | ix
http://ix.io/3ToK

Can't visit the site because I get ERR_SSL_PROTOCOL_ERROR.


remove warning test around ownership_mixin.rb

DEPRECATION WARNING: This dynamic method is deprecated. Please use e.g. Post.where(...).all instead. (called from set_ownership at /home/travis/build/ManageIQ/manageiq/app/models/mixins/ownership_mixin.rb:19)

/cc @matthewd confused why my first pass did not get rid of warnings. thought the ar extension would have hidden the warning

only failure is root_disk_size_by_flavor - can we merge, or do we need to rebase and make green?


Window stops working.

VSCode Version: 1.47.3 (user setup)
OS Version: Windows 10 2004

Steps to Reproduce:

I am facing this issue where the window stops working when an error is encountered while running the code.

Please help me out with this.

Does this issue occur when all extensions are disabled?: No

Does this issue occur when all extensions are disabled?: No

I suggest you enable your extensions one at a time until you find the one that is causing the problem. Then report it to the extension author.

Does this happen after you close a terminal? https://github.com/microsoft/vscode/issues/76548


Plan: add typing

Summary by CodeRabbit

New Features

Enhanced type hints for parameters in the Plan class constructor, improving clarity on expected input types.
Introduced hooks_array attribute to store hook specifications.
Added a new entry to .gitignore for ignoring the .local_dev/ directory.

Bug Fixes

Updated logic for adding hooks to virtual machines to ensure both hooks_array and virtual_machines_list are present before proceeding.

Changes

Modified return types of the __enter__ and deploy methods in the Resource class to Any, broadening expected return values.

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).

You can add extra args to the Podman build command

Example: /build-and-push-container --build-arg OPENSHIFT_PYTHON_WRAPPER_COMMIT=<commit_hash>

To add a label by comment use /<label name>, to remove, use /<label name> cancel
To assign reviewers based on OWNERS file use /assign-reviewers
To check if PR can be merged use /check-can-merge

Supported /retest check runs

/retest tox: Retest tox
/retest python-module-install: Retest python-module-install
/retest all: Retest all

Supported labels

hold
verified
wip
lgtm

/verified

/verified


Python check will always fail on macOS with homebrew

The check for python in the script fails on macOS, if python is installed via homebrew (https://docs.brew.sh/Homebrew-and-Python), which is very common.
The script checks for py, but if python is installed via homebrew, it is available with command python3. Of course I can workaround this by creating a symlink.
The part of the script is this one:
# Check if Python is installed
python_installed=0
if command_exists py; then
    echo -e "${GREEN}Python is installed.${RESET}"
    py --version
    python_installed=1
else
    echo -e "${RED}Python is not installed.${RESET}"
fi
echo ""

It should also check for python3 command

Thanks for testing out on macOS! I've made some changes based on this and the color formatting issue.


"IS NOT NULL" query doesn't return same result as n1ql

IS NOT NULL returns all the document except when "callsign" is "null". Whereas n1ql return all docs where callsign is there and is not null. I'm using travel-sample default bucket data (31591 docs in both CBS and CBL app).

System Info -
-- CBS 5.5.0-1877 on centos6
-- CBL app on android 2.0.0-299
-- SG 2.0.0-773 on Centos6

Query Method

public List<Object> isNot(Args args) throws CouchbaseLiteException {
        //SELECT * FROM `travel-sample` where callsign is not null limit 5
        Database db = args.get("database");
        String prop = args.get("prop");

        List<Object> resultArray = new ArrayList<>();

        Query query = QueryBuilder
                .select(SelectResult.expression(Meta.id),
                        SelectResult.expression(Expression.property("callsign")))
                .from(DataSource.database(db))
                .where(Expression.property(prop).isNot(Expression.value(null)))
                .orderBy(Ordering.expression(Meta.id).ascending());
        for (Result row : query.execute()) {
            resultArray.add(row.toMap());
        }
        return resultArray;
    }

N1Ql query
select count(*) FROM travel-sample WHERE callsign IS NOT NULL

Query Result

[
  {
    "$1": 166
  }
]

CBL Query

I/LiteCore [DB]: Opening DataFile /data/user/0/com.couchbase.TestServerApp/files/test_db.cblite2/db.sqlite3
V/Query: Query encoded as {"WHAT":[["._id"],[".callsign"]],"ORDER_BY":[["._id"]],"WHERE":["IS NOT",[".callsign"],null]}
I/LiteCore [SQL]: Compiling JSON query: {"WHAT":[["._id"],[".callsign"]],"ORDER_BY":[["._id"]],"WHERE":["IS NOT",[".callsign"],null]}
I/LiteCore [SQL]: Compiled Query: SELECT fl_result(key), fl_result(fl_value(body, 'callsign')) FROM kv_default WHERE (fl_value(body, 'callsign') IS NOT x'') AND (flags & 1) = 0 ORDER BY key
I/LiteCore [SQL]: Created prerecorded query enum with 31570 rows (842318 bytes) in 113.068ms

The SQL translation of the IS NOT NULL test is fl_value(body, 'callsign') IS NOT x'' (where x'' is an empty blob, which is how we represent a JSON null.) The discrepancy here is when the result of fl_value is MISSING, which is a SQL NULL.
The SQLite docs say "If one operand is NULL [i.e. MISSING] and the other is not, then the IS operator evaluates to 0 (false) and the IS NOT operator is 1 (true)." Which explains the discrepancy.

I'm wondering if IS NOT NULL is a special operator in N1QL, or whether IS NOT just behaves differently when its right hand side is NULL?

Oh, and I just noticed this was filed under Android when it's a LiteCore issue...

This issue was moved to couchbase/couchbase-lite-core#410


Update to zend-servicemanager v3

This component will need to be updated to zend-servicemanager v3 in order to work with that version, due to providing abstract plugin manager implementations.

It is solved already.


Mocking useNavigate with jest does not work

Version
"react-router-dom": "6.0.0-beta.0",
Test Case
https://codesandbox.io/s/react-router-forked-t90r7
The sandbox doesn't properly work because jest.mock is still not a function (aka sandbox doesn't support it yet boo).
Steps to reproduce

Create a component that uses useNavigate hook and make a navigate call inside of it.
Create a react-testing-library test for it and mock the hook like

jest.mock('react-router-dom', () => ({
    ...jest.requireActual('react-router-dom'),
    useNavigate: () => ({
        navigate: jest.fn(),
    }),
}))

Expected Behavior

Test works and the navigate mock function is called.

Actual Behavior

Test fails with

Error: Uncaught [TypeError: navigate is not a function]

That's an issue with Jest, not this library.

@timdorr This used to work with the useHistory hook and mocking the push method. We already had such tests which are currently working. This is why I assumed something in the router is not properly set.

It is such a pain in butt to mock useNavigate() hook.
@andradaaag
I asked a question on stack: https://stackoverflow.com/questions/66284286/react-jest-mock-usenavigate

Update: I was wrongly mocking the hook since it's returning a function that returns a function and not an object with the navigate property on it like I did in my example above. Here is the working code:
jest.mock('react-router-dom', () => ({
    ...(jest.requireActual('react-router-dom') as {}),
    useNavigate: () => jest.fn(),
}))


fs: Support incremental file updates

We still don't advertise we support this since I have not tested this
thoroughly. However, I am working on a PR now which refactors this logic to make
it more testable.

CI is acting very funny with this. Both runs should be testing identical commits, but they are not. The PR CI build is testing an older commit I pushed, and it won't update to the new one. CI is passing on this if travis tested the proper commit. It also passes locally.

@keegancsmith Indeed, Travis CI does seem to be testing the wrong commit. Strange.
LGTM.

I am rebasing on latest master and repushing just to try and trick travis into giving me the green I deserve :)


flaky Error: Can't set headers after they are sent

I'm running the doiuse.com code locally to work on https://github.com/anandthakker/doiuse.com/issues/31.
I haven't found reliable steps to reproduce, but every so often, it exits with status 1 and this output:
_http_outgoing.js:335
    throw new Error('Can\'t set headers after they are sent.');
          ^
Error: Can't set headers after they are sent.
    at ServerResponse.OutgoingMessage.setHeader (_http_outgoing.js:335:11)
    at cssFeatures (/Users/lcrouch/code/doiuse.com/lib/css-features.js:18:7)
    at /Users/lcrouch/code/doiuse.com/index.js:46:9
    at ConcatStream.<anonymous> (/Users/lcrouch/code/doiuse.com/node_modules/mississippi/node_modules/concat-stream/index.js:36:43)
    at ConcatStream.emit (events.js:129:20)
    at finishMaybe (/Users/lcrouch/code/doiuse.com/node_modules/mississippi/node_modules/concat-stream/node_modules/readable-stream/lib/_stream_writable.js:511:14)
    at endWritable (/Users/lcrouch/code/doiuse.com/node_modules/mississippi/node_modules/concat-stream/node_modules/readable-stream/lib/_stream_writable.js:521:3)
    at ConcatStream.Writable.end (/Users/lcrouch/code/doiuse.com/node_modules/mississippi/node_modules/concat-stream/node_modules/readable-stream/lib/_stream_writable.js:486:5)
    at DestroyableTransform.onend (/Users/lcrouch/code/doiuse.com/node_modules/mississippi/node_modules/through2/node_modules/readable-stream/lib/_stream_readable.js:545:10)
    at DestroyableTransform.g (events.js:199:16)

npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
npm ERR! node v0.12.2
npm ERR! npm  v2.7.4
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node index.js'.
npm ERR! This is most likely a problem with the doiuse.com package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node index.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls doiuse.com
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/lcrouch/code/doiuse.com/npm-debug.log

The corresponding npm-debug.log output:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info prestart<EMAIL_ADDRESS>6 info start<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec start script
9 verbose stack Error<EMAIL_ADDRESS>start: `node index.js`
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/lifecycle.js:213:16)
9 verbose stack     at EventEmitter.emit (events.js:110:17)
9 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/spawn.js:14:12)
9 verbose stack     at ChildProcess.emit (events.js:110:17)
9 verbose stack     at maybeClose (child_process.js:1015:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd /Users/lcrouch/code/doiuse.com
12 error Darwin 14.5.0
13 error argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
14 error node v0.12.2
15 error npm  v2.7.4
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>start: `node index.js`
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>start script 'node index.js'.
18 error This is most likely a problem with the doiuse.com package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     node index.js
18 error You can get their info via:
18 error     npm owner ls doiuse.com
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]

Damn. Sorry I was testing this with my local vs. prod site and I accidentally took down the prod site. :cry:


kinetic fixes

Dear Roberto,
finally I found some time myself to build your framework. I had some issues on Kinetic, which I finally resolved. Please find my fixes enclosed. Maybe they are useful for you as well.
I'm not sure, why you included .svn of vcglib. I think this bloats the repo and should be removed via repo rewriting at some point...
Thanks for your great work!

Dear Robert,
thanks a lot for your suggestions! I will merge your request. Including the .svn files seems to be a mistake, they are not necessary at all.
I hope the system works fine on kinetic since I didn't test it at all, just fix quickly some things to make it compilable.
Very best regards,
Roberto


Feedback for "Developing Java in the Cloud Foundry Environment"

Hi colleagues,
This guide
https://help.sap.com/products/BTP/65de2977205c403bbc107264b8eccf4b/a3f90069d6cd41da82f34a6123d82ce6.html --> components mentioned calculator V1 is by default
Memory Calculator V1 (SAP JVM Memory Calculator) - by default
But in guide Memory Calculator
https://help.sap.com/products/BTP/65de2977205c403bbc107264b8eccf4b/505a71ae53e84c7a972bb6c34b4316fb.html
V2 is marked as default:
Memory Calculator V2 - this memory calculator is activated by default.
May I know which version exactly is used by default in sap java buildpack?
Thanks,
Donny

Hi Donny,
The Memory Calculator V2 is the default version. Documentation page Developing Java in the Cloud Foundry Environment has been corrected.
Thank you for the valuable feedback!
Best regards,
Gergana


consider reducing the mutex contention in the worker thread pool

I find that our worker thread pool uses the traditional mutex + condition var mechanism for the thread interaction.
The task queue is also handled in the mutex, this may cause the frequent lock contention even the task queue has enough tasks and the worker doesn't need to wait.
I abstract the queue from the mutex and use a spinlock to lock separately, do some benchmarks and find the performance increased, see:
./origin_thread_pool
Using 8 threads
204678 QPS
185131 QPS
186840 QPS
165325 QPS

./spin_thread_pool 8
Using 8 threads
232116 QPS
244754 QPS
216727 QPS
235721 QPS

So I think it is worth to re-improve the worker thread pool.
The benchmark cases:
Origin: https://gist.github.com/siddontang/fa81a59e7234e9960f8514785af69e1e
Spinlock: https://gist.github.com/siddontang/5b4b023475a55e0d4898117790a224c9
/cc @BusyJay @zhangjinpeng1987 @wentaoxu

I think so, it is worth to do more.

Update:
I change the spinlock code using for + yield like:
for _ in 0..3 {
    if let Some(t) = self.queue.lock().pop() {
        self.task_counter += 1;
        return Some(t);
    }

    thread::yield_now();
}

And the output is:
Using 8 threads
263701 QPS
315363 QPS
308941 QPS
337130 QPS
303962 QPS

Of course, using the yield way can cause a high CPU cost, but it may a way for our later optimization if possible.

Parkinglot sync types usually perform better than their stdlib counterparts. Worth trying.


Verify SciPy canonical input

Hi @hameerabbasi,
When consuming SciPy sparse object, Finch expects it to be in a canonical format. When it's not true then array isn't loaded properly.
Here I verify that input is in canonical format and raise a proper warning.

does "canonical" mean "sorted"? just curious. thanks!

Yes! finch-tensor requires it. I realized that creating SparseCOO with coords arrays that aren't sorted results in a broken Tensor (e.g. when densifying (copying to a dense Tensor) the resulting tensor is incorrect).

does "canonical" mean "sorted"? just curious. thanks!

Almost, sorted + no duplicate entries. 😄


changes for username

@juhiagl8
No changes required from util.java file...
All these changes should be merged into the PR #41


Bad request: invalid escaped module path

I'm not sure if the problem is on my side, but I am unable to refresh my project and get the following error code:
There was an error processing your request: Could not analyze the repository: could not download repo: could not get latest module version from https://proxy.golang.org/gokapi/@latest: bad request: invalid escaped module path "gokapi": malformed module path "gokapi": missing dot in first path element
The project is Gokapi: https://github.com/Forceu/gokapi

Hi @Forceu, could you try changing the first line of your go.mod to github.com/forceu/gokapi?

Still the same error? Or is the request cached?

@Forceu the Golang module proxy sees the latest version as v1.3.0:
https://proxy.golang.org/github.com/forceu/gokapi/@latest
<EMAIL_ADDRESS>If you tag v1.3.1 I think it should work (you do have a 1.3.1 tag but it needs the v prefix)

Ah I did not realise that it needs to be tagged. Has that been changed? Because half a year ago goreportcard would use the latest master

@Forceu yes, we used to use git but we're now downloading modules from the proxy.


refactor: cleaner public api

Closes #53
Summary of Changes
Define a small public interface consisting of

safeds_runner.file_mtime,
safeds_runner.memoized_call,
safeds_runner.save_placeholder.

These are used by the code generator of the DSL.

@WinPlay02 Would you mind reviewing this?

It's also quite interesting that when I also add start_server to src/safeds_runner/__init__.py, the runtime of the tests increases from roughly 15s to 33s, apparently due to the imports of Image and Table in src/safeds_runner/server/_json_encoder.py.

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

v0.7.0
GitHub release

Your semantic-release bot :package::rocket:


RFC: Create SECURITY.md

this is related to our community profile

[x] direct security reports to group dev email address
[x] the release process wiki should be edited to mention that  this file would need to get updated every time a release is made
[ ] decide whether SECURITY.md is ignored or added to the manifest

Sorry forgot to submit my comments from yesterday, just did it and one was outdated.

Thanks @drammock

Whoops, missed the manifest. I would say add it, why not. Feel free to add to #8365


Creating Setting Gear on top right

Add a Setting Gear with a small modal, in which user can select option, coming next.
For now, it will contain just the "about" information, same as the one available in the footer.
Those information can be hardcoded.
The modal should appear in the middle of the page, like a small window. Both clicking outside or a "close"button in the bottom will close it, either an X on top right or a close button.
Please take care to respect the "feel" and colors of the website

On it Sir

Cool thank you!


Fixed tags action path

Hey @matt-major!
Great wrapper package you have there.
I've encountered an issue when trying to use the tags module: the actionPath has a trailing / symbol, which caused create and getAll methods to fail with 404. For example, for the getAll method, the wrapper would call https://api.digitalocean.com/v2/tags/ instead of https://api.digitalocean.com/v2/tags.
Removing the / makes sure that all of the methods work correctly (except for the issue fixed in #54).
Reference docs: https://developers.digitalocean.com/documentation/v2/#tags.
I would really appreciate it if I could get this merged! 🙌

Good spot @Albinutte - shall get this merged in now


Error: segmentation fault (when running on MacBook M1)

After quick install with quick_install.sh
still can not work on M1 chips using:
python seal_link_pred.py --dataset USAir --epochs 50 --num_hops 2 --seed 1 --profile

Thanks for reply. I did as you said:

update llvm and redirect CC=/opt/homebrew/Cellar/llvm/15.0.5/bin/clang
del profile argument, with python seal_link_pred.py --dataset USAir --epochs 50 --num_hops 2 --seed 1
reinstall dependencies as you listed

but it fails again. PS: origin SEAL works on my Mac.
Well, I will try other methods to fix this prob, and share that if it works.

Thanks for the update. Please let us know.

Closing stale issue. However, please feel free to continue the discussion here.


Hide Sidebar Panel From Being Focusable If Hidden

Storybook: https://vimeo.github.io/iris/sb/fix-hidden-panel-focus/?path=/story/components-sidebar-props--attach
What
Hides sidebar panel from being focusable when not active. Currently when the sidebar is attached to the right of the window a user is able to tab through the sidebar items and once they pass the last item in the sidebar, the sidebar panel is focused because it is a sibling element of the sidebar.
Current Issue

How it does that?
Adds a hidden attribute to PanelStyled. If the panel is not active then hidden is true.

Hi Kenneth. This breaks the current behavior of the Sidebar. The hidden attribute also has a controversial history (and can easily be overridden). I agree that the children of the Sidebar's panel should not be focusable once it is closed, but another method will be needed to accomplish this.

Good call, I'm thinking of updating the tabIndex of the button since that is what's getting the focus.
tabIndex={!active ? -1 : 0}

We could conditionally render the dismiss button 🤔
{panel && (
  <Dismiss
    aria-label="Dismiss"
    format="basic"
    icon={<DismissX />}
    onClick={close}
    variant="minimalTransparent"
  />
}

Also thought about using framer-motion and having it handle the panel transition animation
<AnimatePresence exitBeforeEnter>
  {panel && (
    <MotionPanelStyled
      variants={variants}
      initial="hidden"
      animate="visible"
      exit="hidden"
     >
      ...
    </MotionPanelStyled>
  )}
</AnimatePresence>

@kenneth-duverge
Since there will almost always be focusable content in the panel itself, I think we need a fuller solution than just the conditional rendering of the dismiss.
Like you suggested I think framer motion could be a good solution. I've also put together this idea as well https://github.com/vimeo/iris/pull/132

Ah okay, sounds good @tylerthegrove ! I'll go ahead and close this PR