Item detail with the UUID entity identifier

Hello,
is there any chance to set the $grid->setItemsDetail(); when we use Doctrine and the ramsey/uuid as the entity identifier?
The _datagrid_toggle_detail is always null in the response when the primary key value is an object. So we need to pass the $id->toString() to the detail link generator somehow.
Except of the $grid->setPrimaryKey('id.toString'); workaround :-(
Thanks!

The problem is even worse when there is no join clause in the query builder as the datagrid tries to count(id.toString) and ends with the 'id' is not defined error - so the workaround does not work.


OPexpect.spawn encountered a problem: Check the lpar activate command

[console-expect]#2020-07-13 18:01:33,686:op-test.common.OpTestHMC:connect:INFO:Opening the LPAR console
Traceback (most recent call last):
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 445, in connect
    raise OpTestError("Check the lpar activate command")
common.OpTestError.OpTestError: Check the lpar activate command

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/jenkins/jenkins_home/workspace/ioci/OpTestConfiguration.py", line 837, in objs
    hmc.set_system(self.op_system)
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 279, in set_system
    self.console.set_system(system)
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 352, in set_system
    self.pty = self.get_console()
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 465, in get_console
    self.connect(logger=logger)
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 449, in connect
    'OPexpect.spawn encountered a problem: ' + str(exp), -1)
common.Exceptions.CommandFailed: Command 'OPexpect.spawn' exited with '-1'.
Output
OPexpect.spawn encountered a problem: Check the lpar activate command
2020-07-13 18:02:39,948:op-test.common.OpTestUtil:cleanup:INFO:OpTestSystem Starting to Gather ESEL's
2020-07-13 18:02:39,949:op-test.common.OpTestUtil:dump_versions:INFO:Log Location: /home/jenkins/jenkins_home/workspace/ioci/test-reports/test-run-20200713180115/*debug*
2020-07-13 18:02:39,949:op-test.common.OpTestUtil:dump_versions:INFO:
----------------------------------------------------------
OpTestSystem Firmware Versions Tested
(if flashed things like skiboot.lid, may not be accurate)
----------------------------------------------------------
Firmware Versions Unavailable
----------------------------------------------------------
----------------------------------------------------------

Traceback (most recent call last):
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 445, in connect
    raise OpTestError("Check the lpar activate command")
common.OpTestError.OpTestError: Check the lpar activate command

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./op-test", line 183, in <module>
    OpTestConfiguration.conf.objs()
  File "/home/jenkins/jenkins_home/workspace/ioci/OpTestConfiguration.py", line 923, in objs
    raise e
  File "/home/jenkins/jenkins_home/workspace/ioci/OpTestConfiguration.py", line 837, in objs
    hmc.set_system(self.op_system)
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 279, in set_system
    self.console.set_system(system)
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 352, in set_system
    self.pty = self.get_console()
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 465, in get_console
    self.connect(logger=logger)
  File "/home/jenkins/jenkins_home/workspace/ioci/common/OpTestHMC.py", line 449, in connect
    'OPexpect.spawn encountered a problem: ' + str(exp), -1)
common.Exceptions.CommandFailed: Command 'OPexpect.spawn' exited with '-1'.
Output
OPexpect.spawn encountered a problem: Check the lpar activate command
[Pipeline] }

We don't have a PowerVM system readily accessible to check this out.. Perhaps @harihare could take a look or @harish-24 could explain what he meant by 'lpar activate command' ?

The message is sightly misleading, It should be "Check lpar console activate command". Error seems to occur when trying to take a console of lpar. i.e.
# mkvterm -m <managed-sys> -p <lpar>

 Open in progress

 Open Completed.

Open Completed is expected to work always when the managed system is active. If this does not happen, then it means there is an issue with the command when connecting.
@abdhaleegit If you thing the expect string failed due to a timing issue, then that needs to be fixed.

@abdhaleegit any updates here. I this fixed or still issue exist?

@abdhaleegit please verify on the latest upstream

discussed with @abdhaleegit this is related to the console issue, which is not reproduced on the latest code. Hence closing the issue. Feel free to reopen the issue if the issue mentioned persist.


Gitrob Enhancement (False Positives handling, User Auth)

Enhancement

Added false positive handling to avoid detection on the next run. SHA256 is generated from commit hash, file path, owner name and repository name.
1.1 Clicking on the file fingerprint will autofill the false positive form to include that particular fingerprint into the false positive set.

Added authentication to only allow authenticated users to view Gitrob results.
2.1 Login credentials will be generated during Gitrob first run.

Added rate limiting by limiting Github client call to avoid hitting Github API rate limit if limited API keys available

Added an option to disable organization member scan to reduce noise and focus on organization repo only.

How can i create user and password i tried to create but couldn't can you please help on this

@johnp2686 Creds are created during gitrob's first initialization.


Upgrade to Typescript 3.5.1

Release notes: https://devblogs.microsoft.com/typescript/announcing-typescript-3-5/
Upgrades eslint typescript integration
Upgrades ancillary libraries like Rollup
Upgrade various typings to latest.
Enables skipLibCheck in tsconfig.json see https://github.com/microsoft/TypeScript/issues/31717 for details

shelving this for now..


fix: remove yarn engine requirement

It should work with all versions of Yarn, this prevents users of classic Yarn from consuming the package.

@tolgap Yeah agreed


Backend: Create docker image for backend and create CI/CD pipeline

Containerize 🐳 the backend, and create CI/CD pipeline in GitHub Actions

Not sure if we have to use docker, i will try to create a deploy pipieline for backend to render


[FR] Add custom commands to builtin

Proposal: allow users to add custom commands to the "builtin" view. In this sense, it is not "builtin" anymore, more like an extensible command palette. It seems doable, but not sure whether it makes to you.
Additionally, I would suggest having a dedicated place (a new repo) to host all kinds of new fzf commands (like the ones in your advanced wiki). It's like telescope extension, but I suggest putting them in a single place. This way, people can import that repo, pick the commands they like, then add them to the command palette.

Ty @liangxianzhe!
I love the idea but if I'm honest I would hate to manage that, the last thing I want is to have responsibility for code that other people wrote which is bound to happen if the repo is under my github profile, this plugin is huge as it is and I do not wish to have to deal with issues fixing / having the responsibility for other people's code.
Imagine I'd have to review each PR and make sure it doesn't break anything, or worse, that it doesn't contain malicous code.
You're more than welcome to open a repo and I'll be happy to mention it in the README as well as have a function that you can call to "register" your new functions into the "command palette".

Totally understand. I'm not sure I have the time for that right now.
But let's start with how I can register a few my functions into the "builtin" for my own convinence?

It's very simple, you can just add it to the fzf-lua module, the builtin methods are automatically enumerated from it, for example:
:lua require("fzf-lua").test = function() print "test" end

And test is now included in the builtin.

Perfect. I'm closing the issue for now. Thanks!


check why Ikarus detects the mod as a virus

https://www.virustotal.com/gui/file/29eeef3319fb44f181092de38d882067df991de7d04dea0598c2bdea9cc7dfc2/detection

seems it no longer detects it as a virus lol


Build problems

Description
It is neither a bug nor a feature.
I want to contribute to this project but first must build it. Unfortunately, the build fails with this error:
Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB8020	The build tools for v143 (Platform Toolset = 'v143') cannot be found. To build using the v143 build tools, please install v143 build tools.  Alternatively, you may upgrade to the current Visual Studio tools by selecting the Project menu or right-click the solution, and then selecting "Retarget solution".	CustomOpenDialog	C:\Program Files\Microsoft Visual Studio\2022\Preview\MSBuild\Microsoft\VC\v170\Microsoft.CppBuild.targets	440
The readme says I have to install "C++ ATL for UWP build tools" but the VS Installer doesn't have such an option. I have installed the "C++ ATL for latest v143 build tools (x86 & x64)". There is an option "C++ Universal Windows Platform support for v143 build tools (ARM64)", should I install it.
I am using VS Version 17.2.0 Preview 1.0
Steps To Reproduce

Download repo
Install VS 2022
Install Windows SDK
Change Solution Platform to x64
Build

Expected behavior
Solution should build
Files Version
latest master
Windows Version
Windows 11 Pro 21H2 22000.527
Relevant Assets/Logs
n/a

unload the CustomOpenDialog folder in solution explorer

Thx @Josh-65 , I entirely built this project after a few tries of installing and uninstalling components in VS. I have updated the Readme to help other contributors.


Method definition popover position

All extension are disabled except for Refined GitHub.
https://github.com/no2chem/bigint-buffer

Clicking on toBufferBE shows the method definition popover near the bottom of the window. Disabling Refined GitHub results in the popover appearing near the toBufferBE text.
Chrome 88.0.4324.192
Refined GitHub 21.2.23

Confirmed. No idea what's causing it. #2586 would have been useful again.

Out of curiosity I looked into this bug. And I think I found the problem.
It is a CSS position: relative that is added here:
https://github.com/sindresorhus/refined-github/blob/main/source/features/hide-readme-header.css#L2-L4
When I manually set position: initial on the #readme HTML element, the position of the popover is correct.


training iterations

Hi, @MC-E. I really thank for your awesome work.
I have some questions about training MM-RealSR.

In your paper, MM-RealSR was trained during 1000k iterations for phase 1 and 400k iters for phase 2, which are different with .yml files. How can i reproduce the results?

How long did it take with your env?

Thanks!

Thanks for reply, but I wondered the number of iterations for each phase, not loss function:)
The "total_iter" in .yml files are not same as the paper.

Sorry for the misunderstanding. We did some debugging before releasing the code. We found that it is unnecessary to train the net stage for too long, so we shortened the number of iterations for it and appropriately increased the number of training steps for the GAN stage. This has little impact on the quantitative metrics.


Add a method to get the Certificate type to the interface

A common use case for Certificate data will be to only operate on specific certificate types. It would greatly simplify downstream if we had a way to access the type from the Certificate interface.

The idea is to use Go's type switch/assertion for determining the cert type, since multiple types are returned using the ledger.Certificate interface. However, we have the information readily available to also return the integer value for the cert type, so adding a .Type() function wouldn't be difficult.
Type switch example:
switch v := myCert.(type) {
case ledger.StakeDelegationCertificate:
    // you can access v.StakeCredential and v.PoolKeyHash directly here
case ledger.PoolRegistrationCertificate:
    // you can access v.Operator, v.Pledge, v.Margin, etc. directly here
}


Introduce spaces field mask

We now use a field mask to select which properties to retrieve when looking up storage spaces. This allows the gateway to only ask for root when trying to forward id or path based requests. It allows the spaces storage registry to omit calling ListStorageSpaces on the storage provider.
Related: https://github.com/cs3org/reva/issues/2881

there are problems in the ocis test suite I need to check before continuing: https://drone.owncloud.com/owncloud/ocis/12029/37/7


Unable to Enable OSM on AKS with Kubenet

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks enable-addons Extension Name: aks-preview. Version: 0.5.28.
Errors:
The command failed with an unexpected error. Here is the traceback:
'NoneType' object has no attribute 'rpartition'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/raveendran/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 3430, in aks_enable_addons
    cmd, result, random_agent_pool.vnet_subnet_id)
  File "/home/raveendran/.azure/cliextensions/aks-preview/azext_aks_preview/custom.py", line 3007, in _add_virtual_node_role_assignment
    vnet_id = vnet_subnet_id.rpartition('/')[0]
AttributeError: 'NoneType' object has no attribute 'rpartition'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks enable-addons --addons {} --resource-group {} --name {}

Expected Behavior
Environment Summary
Linux-5.4.0-1056-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.27.0 *

Extensions:
aks-preview 0.5.28
ai-examples 0.2.5
ssh 0.1.5

Additional Context

Please provide additional details around your bug report, like what the cluster networking configuration was and exactly what command failed. The example in the report has redacted arguments.


Get NIOEmbedded clean under strict concurrency

Motivation:
NIOEmbedded is used all over NIO-land for testing various pieces of the infrastructure, and so requires a substantial audit for strict concurrency.
Modifications:

Mark a few things Sendable.
Fix the tests, which actually did have some nasty bugs

Result:
Sendable-clean NIOEmbedded

API breakage is expected.


Added Monad.filterNot(Predicate)

Towards #484

Current coverage is 91.44%

Merging #944 into master will decrease coverage by -0.30% as of f06658f

@@            master    #944   diff @@
======================================
  Files           80      80
  Stmts         8761    8817    +56
  Branches      1705    1706     +1
  Methods          0       0
======================================
+ Hit           8038    8063    +25
  Partial        175     175
- Missed         548     579    +31

Review entire Coverage Diff as of f06658f

Powered by Codecov. Updated on successful CI builds.


[bug] The file and folder names cannot be the same

I would like to report a bug.

The file and folder names cannot be the same, and if they are the same, the file will not be displayed.
like this

I found that the solution had to be uppercase and lowercase,but we can't keep using this method

The merging of the folder and note of the same name is the folder notes feature, which you can turn off in settings

On Oct 12, 2024, at 5:41 AM, Scuffle-man @.***> wrote:
I found that the solution had to be uppercase and lowercase,but we can't keep using this method
—
Reply to this email directly, view it on GitHub https://github.com/Make-md/makemd/issues/374#issuecomment-2408550864, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFHKNX2SXX667XKOF3YXR3Z3EKF5AVCNFSM6AAAAABPW5ETZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYGU2TAOBWGQ.
You are receiving this because you are subscribed to this thread.

It seems that this function is not bad, meaning that I can change this annotation to other names, is it like a picture, but after the modification, I still do n’t display my file. Is it my modification method wrong?

Thank you so much. It looks like this feature is pretty good,I can change the annotation to something else, isn't it like an image, but after changing it, I still don't show my file. Is my modification method wrong?

Try restarting obsidian to trigger a reindex
Get Outlook for iOShttps://aka.ms/o0ukef

From: Scuffle-man @.>
Sent: Saturday, October 12, 2024 10:19:17 PM
To: Make-md/makemd @.>
Cc: JPC @.>; Comment @.>
Subject: Re: [Make-md/makemd] [bug] The file and folder names cannot be the same (Issue #374)
image.png (view on web)https://github.com/user-attachments/assets/ded6be95-18e0-4a5e-b38e-e7887da089b4
It seems that this function is not bad, meaning that I can change this annotation to other names, is it like a picture, but after the modification, I still do n’t display my file. Is it my modification method wrong?
—
Reply to this email directly, view it on GitHubhttps://github.com/Make-md/makemd/issues/374#issuecomment-2408828079, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAFHKNX7WCLSBXEMKPTPDR3Z3H7FLAVCNFSM6AAAAABPW5ETZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYHAZDQMBXHE.
You are receiving this because you commented.Message ID: @.***>

Thank you very much, I followed your method, restarted the function, restarted the plugin, restarted the software, and it finally succeeded. It's a good feature, I won't use it, and I still need to learn the plugin feature


[Task]: FE changes for Homepage experience - Content side (Applications area)

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks

[ ] 1. Show only apps of the current workspace selected.
[ ] 2. UI changes on the right side - Font and styles
[ ] 3. Update the empty state on the right side
[ ] 4. Implement workspace and Application GET APIs - [TBD]
[ ] 5. Cypress

Reduced 0.5sp from this ticket as we are tracking cypress changes in this ticket https://github.com/appsmithorg/appsmith/issues/28980

fixed and merged in release. Hence closing the ticket


Compiler crashes when using a project with many project references

TypeScript Version:  3.5.3

Search Terms: out-of-memory, tsc, project refrences, garbage collection
Code
There is not easy way to show this.
The tsc fails (node dies) with out of memory in a project with may project references.
There is a point where adding one more project reference makes the problem occur.
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

<--- Last few GCs --->

[31:0x559282ba6b40]   187820 ms: Mark-sweep 1391.0 (1425.4) -> 1391.0 (1425.4) MB, 716.4 / 0.0 ms  (average mu = 0.171, current mu = 0.002) allocation failure GC in old space requested
[31:0x559282ba6b40]   188622 ms: Mark-sweep 1391.0 (1425.4) -> 1391.0 (1425.4) MB, 649.6 / 0.0 ms  (average mu = 0.179, current mu = 0.190) last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x204e63c5be1d]
Security context: 0x399cfbb9e6e9 <JSObject>
    1: set [0x399cfbb91ea9](this=0x04ae4a499231 <Map map = 0xe6135f04359>,0x3730a06da439 <String[16]: @BENIGN_POSITIVE>,0x3730a06da461 <Type map = 0xe6135f55fa9>)
    2: getLiteralType(aka getLiteralType) [0x4ae4a4a1bb9] [/builds/xxx/node_modules/typescript/lib/tsc.js:~34353] [pc=0x204e64e603d6](this=0x08616aa826f1 <undefined>,value=0x0f68702fbf49 <Str...

error Command failed with signal "SIGABRT".

Expected behavior:
Project references should not increase memory usage that much.
Related Issues:
Could be related to #31398  (https://github.com/microsoft/TypeScript/issues/31398#issuecomment-498270557)

@KnisterPeter We need some more info about this. Please share the project where this repros so we can actually see this happening and can determine the course of action that needs to be taken.

@sheetalkamat Sorry for my late answer. The project is private for a client of us, so unfortunately I could not give more insides.
I'm not sure how to create a minimal example for this since I guess it might require a critical mass.
Anything I could create (like memory dumps or stuff) which could help to debug this?

You might want to try typescript@next to see if this is fixed for you..  #33145 might be the fix that you need. Otherwise there isn't much for us to base investigation on unless you provide a repro

@sheetalkamat We switched to node 12 and its working now. That may be caused by the extended heap in node 12 and others.
Anyway, thanks for your support.


fix: settings tab crashes with third party cookies blocked, fixes #1097 #979

Hey! Thanks a lot for your PR! Howerver, It was superseded by https://github.com/vuejs/vue-devtools/commit/d971b4adb959f393fcdc95cc3c61dccdbc9a6ad7


Better Developper/ITC Teams display

Here is my situation, as many freelancers, I'm a member of a lot of Apple Developer teams (and many Itunes Connect). I often have fastlane outputs like this:
+-----------------------+-------------------------------------------------+
|                         Summary for match 0.8.1                         |
+-----------------------+-------------------------------------------------+
| type                  | development                                     |
	                                  [...]
| team_id               |<PHONE_NUMBER>                                      |
	                                  [...]
+-----------------------+-------------------------------------------------+

I don't know the ids of all my teams, so it's not easy to know directly which one is concerned.
I think it would be a lot more user-friendly to display the name of the team beside the team_id. It would give something like this:
+-----------------------+-------------------------------------------------+
|                         Summary for match 0.8.1                         |
+-----------------------+-------------------------------------------------+
| type                  | development                                     |
	                                  [...]
| team_id               |<PHONE_NUMBER> (Oceanic Airlines)                   |
	                                  [...]
+-----------------------+-------------------------------------------------+

The idea is to do this everywhere a team_id is displayed to the user. Does anyone see any drawback on this ? If it's OK for you, I'll make a PR with this.

Ah, nice idea, so you're talking about the summary of each tool that is being shown when you launch it, right? The team selection itself is okay?

So, I'll do it. But I have a huge schedule on the next 8 days (I'm doing a tour in China). I'll submit my PR just after :)
About your question, are you talking of this menu ?
Multiple teams found on the Developer Portal, please enter the number of the team you want to use:
1)<PHONE_NUMBER> "Oceanic Airlines" (Company/Organization)
2) KRO166486 "Dharma" (Company/Organization)

If yes, it seems perfect to me !

Alright cool, enjoy your trip @Onejjy 👍 Let us know if you need any help with this

So, after digging into the code, it seems to be a bad idea :/.
Indeed, each time we display summaries like above, it's just a dump of the passed parameters. If we want to be able to display the team name, we would have to make a web request for that. IMHO, it doesn't worth it/
Sorry for making you lose time.

@Onejjy No problem! Always great to work with you 👍

Thanks for looking into this @Onejjy and letting us know :rocket;


[BUG] Item 4 is picked by default, but not displayed on "Multi select" page

It is picked in a disabled select.


How to set overwrite == true?

Hi guys,
I saw that here:
https://github.com/bugsnag/bugsnag-android-gradle-plugin/blob/baeee1893e63eeed29cdd27639c3bbce39f8fb35/src/main/groovy/com/bugsnag/android/gradle/BugsnagUploadTask.groovy#L87
there's a bugsnag.overwrite property. How can I set it during build time?
Thanks!

Closed via 63e999c03aa2ebdf5319a132c87aec3bf3580fb0. Thanks again!


Support bzip2 archive format in ArchiveAnalyzer

The bzip2 archive format is enjoying increasing popularity. The core Apache Ant library includes the needed InputStream class (see https://git-wip-us.apache.org/repos/asf?p=ant.git;a=tree;f=src/main/org/apache/tools/bzip2). I believe this lives in the org.apache.ant:ant dependency already being used in dependency-check-ant, and so could very easily be made a dependency of dependency-check-core as well.

This can be added - but we should be using the code in commons-compress.

Even better. 😊

Dale - thanks for the PR!


Protections against 0-length string attributes

Files that have string metadata with length 0 can cause trouble
if we aren't careful. Put in a number of protections.

@grdanny

Hi @lgritz, sorry to dredge this one up, but could you elaborate on the problem with zero-length strings? I'm specifically interested in the case of reading an empty string attribute from an EXR file, which was disabled in this PR, although we can still write them, and exrheader seems to report them quite happily.

I'm not sure it was a problem, I think I just believed that empty metadata strings were being set accidentally and propagated around, cluttering "iinfo -v" output, etc. I think I assumed that setting it to "" was a shorthand for intending to clear it.
In many file formats, metadata (things like the description of an image) has fixed fields and having an empty string in the slot is truly the same thing as not having that metadata. I suppose in OpenEXR, where there is a list of metadata rather than slots for specific fields, there could be a semantic difference.
Should we preserve empty metadata strings in OpenEXR files? Is that a format in which people find it useful to denote metadata which is present, but empty?

Should we preserve empty metadata strings in OpenEXR files?

I would think so - I can't see a good reason to not round-trip data that is round-trippable. Should I make a PR?

Yes, please.


Sidebar items folding

The folding is very inconvenient for multiple panes/items . If a pane is pushed by other, it will keep the new size .
The panes is mainly resized in long term, based on their purpose . So this behavior results in requirement to never get them pushed . That doesn't allow to use more panes than fit in the pane that is closer to the other end of the list (i) . And means that when any pane is temporally resized, it has to be resized back to avoid the 'sub-panes' size growing .
Please make the panes: remember their size that've been set by resizing them with the up border/end, and try to restore it always when there is free space left in relation to the other unfolded panes' setting/preference sizes .

split view feature request

@sbatten i don't see how this is related to Split View


Proof-of-concept: e2e tests that work on Mac

Signed-off-by: Ed Snible<EMAIL_ADDRESS>This PR is an exercise to size the amount of work to get the e2e tests working on a Mac.
The work is incomplete.  To finish the work the hack to get the token in the tests must be put into the non-test code that gets the token.  (That seems like a bad idea.)
The basic problem is that on the Mac, for a container (such as Hydra) to talk to a process running on the Mac the container must use host.docker.internal as the host.  This host doesn't exist on non-OSX Docker.  Furthermore, because this is a security setup, Hydra must know the host the clients will use to talk to it -- and that value is different when the client is running in a sibling container.
This was somewhat handled in the Observatorium-API tests (which use Dex rather than Thanos.). It is handled there by custom code to create tenants.yaml.  For obsctl we may need something to deal with the config file URLs.

Thanks a lot for this!
For obsctl e2e tests, we don't run hydra in a container as we also don't run obsctl in a container. So supporting macOS would be as simple as changing the docker host URL to docker.for.mac.localhost:4444. Raised #42 for this! 🙂
Please let me know if that works for you, or if you would like to see some other improvements?


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2024-05-31)
Bug Fixes

changed the conditional check of approval_required input parameter to not check for null. The value has a default value defined in the input variable, and will never be null. (f324b60)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/fortytwoservices/terraform-azuread-entitlement-management/releases/tag/v1.0.2 :sunflower:


unable to download Symbols from online Sandbox

Hi,
Thanks in advance for your help,
Am trying to download symbols from online Sandbox but getting issue that path not found.
Please, see pictures below for more details
Do we need vsix file for online Sandox ?  (if is the case how can I get it )

Thank you.

Have you named your sandbox when creating it? If yes that name must be specified in the sandboxName parameter.

Did you already check bullet point 2 and 3 of the error text, about the application version numbers in the app.json? According to the compiler, you are depending on "Application" and "System", both version <IP_ADDRESS>. Are these really the app names and versions installed on your sandbox?

issue fixed by the same setting above, just change  app.json   "dependencies"   "version": "<IP_ADDRESS>"
in launch.json
serverInstance must be lowercase
set  port number   "port": 124X,


LXD 2.20 fails on startup trying to import a ZFS volume that is already available.

Required information

Distribution: Alpine
Distribution version: Edge
The output of "lxc info" or if that fails:

Kernel version:4.9.65
LXC version: 2.1.1
LXD version: 2.20
Storage backend in use: ZFS
ZFS version: 0.7.3

Issue description
After upgrading from 2.19, LXD fails trying to import a ZFS subvolume, even though it is already available.
Steps to reproduce

Set up a ZFS pool (on Alpine)
Try to start up LXD 2.20

Information
LXD server log:
# lxd --verbose
INFO[11-25|22:41:12] LXD 2.20 is starting in normal mode      path=/var/lib/lxd
INFO[11-25|22:41:12] Kernel uid/gid map:
INFO[11-25|22:41:12]  - u 0 0<PHONE_NUMBER>
INFO[11-25|22:41:12]  - g 0 0<PHONE_NUMBER>
INFO[11-25|22:41:12] Configured LXD uid/gid map:
INFO[11-25|22:41:12]  - u 0 1000000 65536
INFO[11-25|22:41:12]  - g 0 1000000 65536
WARN[11-25|22:41:12] AppArmor support has been disabled because of lack of kernel support
WARN[11-25|22:41:12] CGroup memory swap accounting is disabled, swap limits will be ignored.
INFO[11-25|22:41:12] Unmounting temporary filesystems
INFO[11-25|22:41:12] Done unmounting temporary filesystems
INFO[11-25|22:41:12] Closing the database
INFO[11-25|22:41:12] Saving simplestreams cache
INFO[11-25|22:41:12] Saved simplestreams cache
error: ZFS storage pool "storage/containers" could not be imported: cannot import 'storage': a pool with that name already exists
use the form 'zpool import <pool | id> <newpool>' to give it a new name

zpool list, shortened:
NAME      SIZE  ALLOC   FREE  EXPANDSZ   FRAG    CAP  DEDUP  HEALTH  ALTROOT
storage   127G  5.63G   121G         -     8%     4%  4.52x  ONLINE  -

zfs list (heavily shortened):
NAME                                                                                USED  AVAIL  REFER  MOUNTPOINT
<snip>
storage/containers                                                                          5.93G   117G    96K  none
<snip>

I've manually applied the patch on top of 2.20, still same issue.
INFO[12-01|17:48:24] LXD 2.20 is starting in normal mode      path=/var/lib/lxd
INFO[12-01|17:48:24] Kernel uid/gid map:
INFO[12-01|17:48:24]  - u 0 0<PHONE_NUMBER>
INFO[12-01|17:48:24]  - g 0 0<PHONE_NUMBER>
INFO[12-01|17:48:24] Configured LXD uid/gid map:
INFO[12-01|17:48:24]  - u 0 1000000 65536
INFO[12-01|17:48:24]  - g 0 1000000 65536
WARN[12-01|17:48:24] AppArmor support has been disabled because of lack of kernel support
WARN[12-01|17:48:24] CGroup memory swap accounting is disabled, swap limits will be ignored.
INFO[12-01|17:48:24] Unmounting temporary filesystems
INFO[12-01|17:48:24] Done unmounting temporary filesystems
INFO[12-01|17:48:24] Closing the database
INFO[12-01|17:48:24] Saving simplestreams cache
INFO[12-01|17:48:24] Saved simplestreams cache
error: ZFS storage pool "storage/containers" could not be imported: cannot import 'storage': a pool with that name already exists
use the form 'zpool import <pool | id> <newpool>' to give it a new name

The pool is already imported, LXD shouldn't attempt to import it again.

Actually, this might be my stupidity. I'll try to figure this out. Sorry for the noise.

FOUND THE ISSUE!
It wasn't in that part of the code at all, nor was it my fault. I'll provide a PR in a minute.


feat: do not check for context existence if a custom context file is set

As today, it a custom context file is set, then the code check for the existence of the context file and fails if it does not exist, however, this is not true for custom configuration location. This PR aims to make config and context behaving consistently.
Verification Steps

Set RHOASCONFIG to an not existing file
Run rhoas
No error should be reported and the RHOASCONFIG file should have been created

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation change
[ ] Other (please specify)

The change has been released in v0.51.2.


cmd/server: add initial transaction storage with QLedger

Issue: https://github.com/moov-io/gl/issues/23

Codecov Report

Merging #31 into master will decrease coverage by 6.97%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   43.35%   36.38%   -6.98%
==========================================
  Files           7       10       +3
  Lines         459      547      +88
==========================================
  Hits          199      199
- Misses        222      310      +88
  Partials       38       38

Impacted Files
Coverage Δ

cmd/server/account_storage.go
0% <0%> (ø)
:arrow_up:

cmd/server/transaction_storage.go
0% <0%> (ø)

cmd/server/main.go
6.45% <0%> (-0.61%)
:arrow_down:

cmd/server/transactions.go
0% <0%> (ø)

cmd/server/transaction_storage_qledger.go
0% <0%> (ø)

cmd/server/account_storage_qledger.go
7.22% <0%> (ø)
: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 28e8228...e05f39a. Read the comment docs.


Rm thiserror dep in favor of hand made error

Closes #146

Thanks for your contribution! I found it valuable to get that kind of perspective on the improvements that can be made.

Thanks for your contribution! I found it valuable to get that kind of perspective on the improvements that can be made.

Do worries! I'm optimizing an xtask project atm, so I'm hyper focused on dependencies and their cost / value.
Thanks for the quality crate :wink:


Allow sub directory matching

Original z allows jumping into sub directory. From z man:
z foo bar     cd to most frecent dir matching foo, then bar
Adding this feature would be useful.

@tokozedg sorry I took a while to reply; was busy organizing a hackathon 😄
Can I clarify:

You want this to match /foo/...../bar over /foo/.../baz, or:
/.../foo/.. is not found, so it matches .../bar/.. instead

Hi,
If I have following  directory structures: /foo/demo/bar/ and /something/bar/
And /something/bar/ is more frecent than /foo/demo/bar/
Than z foo bar should move to /foo/demo/bar and z bar to /something/bar/

Thanks for your input, I'll take a look at this during my very limited free time. I can't guarantee a fix before summer break.

@tokozedg
z foo bar should move to /foo/demo/bar and z bar to /something/bar/
I understand the desired behavior, but what's the current behavior in the above scenario?

@jethrokuan great thanks.
@jbucaran the second input is ignored, z will jump to the dir matching first input.

@tokozedg @jethrokuan Due to the current parsing of arguments, you need to quote the directories when providing multiple dir names.
Belows are the current behavior of z:
z -l 'foo bar' 'foo bar' is the argument of option -l
z -l foo bar foo is the argument of option -l, bar is the bare argument and is ignored
z foo bar foo, bar are both bare argument, but only the first bare argument is used
z 'foo bar' 'foo bar' is the first argument, this can move to /foo/demo/bar
z 'bar foo' the sequence of bar and foo is ignored, this will also move to /foo/demo/bar, if the rank of /foo/demo/bar if higher than /bar/demo/foo,

works on my machine, feel free to reopen if it's still not the case for you.


Decoder.marker returned (0, nil) instead of an error.

Running the marker method returned 0, nil if there was no marker found on the stack. Additionally, there'll be a lot more if len(d.stack) < X { return ErrInvalidStack } coming soon, as many invalid inputs will panic right now.

This is fixed by #28


Feature/combobox pageup and pagedown

Description
Combobox support for PageUp and PageDown
Also fixed issue where Home/End would scroll whole page and mouse scrolling would scroll past the dropdown container.

I think the PageUp, PageDown, Home and End features should be similar to how they are in select. So i think Home should set focus to the top element, not close it. That can be done with Escape instead, which it already is.
I noticed that native select also allows use of Cmd + up / down to do the same as Home / End. Could we add that too? I can look at it.

Potentially option + up / down to match PageUp / PageDown at the same time as cmd up / down

@it-vegard Found the current issue with focus:
virtualFocus.moveFocusToTop(); is called inside toggleIsListOpen in filteredOptionsContext line 143, thus setting focus to
setActiveElement(elementsAbleToReceiveFocus[0]);

toggleIsListOpen

Had to remove moveFocusToTop from onChangeHandler in Input as well. Since this also calls toggleIsListOpen, we moved focus twice, so this is a good optimalization too.


Implement using setVersion to detect stale primaries

Updates monitoring to use set_version as per the spec to fix #153. Once this and #155 and #156 are merged, all the SDAM tests should pass.

Oof, that if block. Welp, if that's what they provide in the spec, LGTM :)

Yeah, I was trying to decide if it was worth splitting it into another nested if statement, but I thought I should try to make it as close to the spec as possible


isArchivedRepo broken on Global navigation update

No ".Label" element anymore, it only appears on the repo home
https://github.com/fregante/iphone-inline-video
📕 0.0.0 → TypeError: Cannot read properties of null (reading 'textContent')
    at Module.isArchivedRepo (refined-github.js:26029:90)
    at isArchivedRepoAsync (refined-github.js:18073:60)
    at async init (refined-github.js:12546:7)
    at async runFeature (refined-github.js:3797:16)
    at async setupPageLoad (refined-github.js:3809:5)

isPublicRepo was already fixed in https://github.com/refined-github/github-url-detection/pull/159


Add more enemies

This pull request refactors the enemy movement and facing logic in the game. It includes changes to the Enemy class, EnemyFactory, SimpleMapData, AStar algorithm, MapData class, and the addition of a new Skeleton enemy, Ghost enemy and Zombie enemy  . The changes improve the enemy movement behavior and add a facing direction for the enemies.

@onon1101 PTAL

Monkey will implement it after completing the combat system

NAK


Proof verification is true for IDCommitment not in merkle tree?

Is it possible that verify (https://github.com/vacp2p/zerokit/blob/master/rln/src/public.rs#L211-L228) returns Ok(true) due to a difference in architecture (32bits vs 64bits) ?
I'm confused, because I see it returning true for an IDCommitment that does not belong to the merkle tree
In rln-wasm I commented this line: https://github.com/vacp2p/zerokit/blob/master/rln-wasm/src/lib.rs#L167
and then executed cargo make test

Thank you for this detailed information! It indeed clarifies why I was seeing the behavior I described with verify, which I mistakenly exposed in wasm instead of verify_rln_proof. In this PR https://github.com/vacp2p/zerokit/pull/58 I'm using the correct function.


Speaker Layout LUT corrections

0+5+0 should reference 5.1 pack
Correct usage of LFE1 in some speaker layouts to just "LFE" (they are different channel/track formats)
Fix track format for B-045

Codecov Report

Merging #46 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #46   +/-   ##
=======================================
  Coverage   87.95%   87.95%
=======================================
  Files         115      115
  Lines        5154     5155    +1
=======================================
+ Hits         4533     4534    +1
  Misses        621      621

Impacted Files
Coverage Δ

src/common_definitions.cpp
98.88% <100.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


fix: server side delta generation docs update.

Changelog: Title
Ticket: MEN-6288
Signed-off-by: Peter Grzybowski<EMAIL_ADDRESS>(cherry picked from commit 18d56358802a18e9cfe427cc41255eff891ebe7f)

Thanks, but I'll align 3.5.x with latest master just before publishing the release, so I'll get this one (together with other improvements) in one go :+1:


Changing WAS IPs

Modified the listed WAS IPs to reflect the change in the scanning addresses as WAS recently procured additional scanners and will be scanning form a slightly different address space as listed:
<IP_ADDRESS>/32
<IP_ADDRESS>/29
<IP_ADDRESS>/28
<IP_ADDRESS>/24
It is a simple change to the tuple that contained the CIDR-noted addresses for the WAS IPs.

@st0rmbl3ss3d You're failing checks currently.  Did you run pre-commit against the changes before you pushed?

You'll have to resolve the failing checks before this can be merged.


Submitting comma separated keywords will only render the first one

Subsequent ones only show up upon page reload. I think this was unintentionally introduced as part of the work done recently in https://github.com/hydroshare/hydroshare/pull/3515
This is specific to release 1.26 and not reproducible on www

(Upon reload)

I just noticed this bug is present in beta and not prod

Fix has been merged to develop


Add ACI 2017-10-01-preview API spec

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] 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 information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Take a look into the model validation. Looks like ContainerGroup model is incomplete.

BTW, the next time there is a swagger in a PR which is based on a previous API version, it is preferred to do it in 2 commits: the first one with the swagger exactly as it is in the API version it is based on, and the second one with the changes. It is easier to review that way. Thanks!

@mcardosos Do you mean the 1st commit against the previous API version, then revert it in the 2nd commit?

@yolo3301 No
I mean copy and paste the files into the new API version folder, commit. Then modify to reflect the new API, and commit again. That way I can look at the diff between these two commits just to look at the changes and not review the whole swaggers again.

@mcardosos I see. Thanks for the tip.

@ravbhatnagar could you take a look at this one? thanks!

Looks good. Signing off!!

 https://github.com/Azure/azure-sdk-for-python/pull/1554

 https://github.com/Azure/azure-sdk-for-ruby/pull/1079

No modification for AutorestCI/azure-sdk-for-node


Add E2E Tests for interconnection subcommand in metal cli

What problem are you facing?
We have added support for interconnections in metal-cli in https://github.com/equinix/metal-cli/pull/321
Need to add tests for the already existing support
How could the Equinix Metal CLI help solve this problem?
Tests will run in CI and make sure reliability of the code

Not sure exactly which PR did it, but tests covering these subcommands were added


Angular external component selectors are not converted

I am interested in helping provide a fix!
Yes
Which generators are impacted?

[ ] All
[X] Angular
[ ] HTML
[ ] Qwik
[ ] React
[ ] React-Native
[ ] Solid
[ ] Stencil
[ ] Svelte
[ ] Vue
[ ] Web components

Reproduction case
https://mitosis.builder.io/?outputTab=IYOw5grgNsBOQ%3D%3D%3D&code=JYWwDg9gTgLgBAbzgVwM4FMDKMCGN1wC%2BcAZlBCHAEQACARssADYAm6UAdMBAPQjAwIqYKioBuAFChIsRHACSAYwgA7IqXKUA5DRABPZeBwq9PYMpVbJE9AA8Z8NiRzIm8EshWKY3NQFk9AGEKSBV0FRgACjByMFQASkQJODgLVHgAbRUcEHQAGjgMGAA5HPQAXTgAXhQMbDx0SKpsdAA3dCp46xSodBhkKDVI5JS4AB4WYFaAPhHR8aVVHln5sZ5JmZGuiUIgA%3D
Expected Behaviour
Mitosis should convert the components names into the correct angular selectors when importing from external modules
Actual Behaviour
I import from an external mitosis module, but it doesn't convert the components into their correct angular selectors.
The mitosis code:
export default function Icon(props: IconProps) {
  return (

  );
}

gets turned into this angular code
@Component({
    selector: "icon",
    template: `

  `,
})
class Icon {
}

and the mitosis button code that imports Icon:
export default function Button(props: ButtonProps) {
  return (
    <button
      data-testid="button"
    >
        <Icon />
    </button>
  );
}

is transformed into this angular code:
@Component({
    selector: "button",
    template: `
    <button
      data-testid="button"
    >
          <Icon></Icon>
    </button>
  `,
})
class Button {
}

Notice that in the last angular template it's using Icon instead of icon (lowercase) which is the correct selector.
Whereas when I'm importing from a local mitosis file, it does use the selectors correctly (probably thanks to https://github.com/BuilderIO/mitosis/pull/357)
Additional Information
Angular generates the following error because even if I'm correctly importing the Icon module, it doesn't know any angular component with the selector Icon since the correct selector is icon:
index.js:1 NG0304: 'Icon' is not a known element:
1. If 'Icon' is an Angular component, then verify that it is part of this module.
2. To allow any element add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component.

The solution could be to generate an additional selector with the exact class name on top of the regular kebab-case selector here https://github.com/BuilderIO/mitosis/blob/main/packages/core/src/generators/angular.ts#L365 which would give something like that (I tested and it works fine for me with this change):
@Component({
    selector: "my-cmp, MyCmp",
    template: `

  `,
})
class MyCmp {
}

@ocombe I'm open to the selector solution you suggested! You already pointed to the right part of the code, feel free to put up a PR adding a json.name 👍🏽

Sounds good, @PatrickJS do you see any reason why we shouldn't do that ?

this might be a problem of detecting which imports are components in the template and correctly transforming the component name

This should've been closed by @alujs 's PR, so I'm doing so now. Feel free to repoen if there's still something missing?


Composition over Inheritance - What it means for your Activities

推荐理由：Composition over Inheritance - What it means for your Activities
原文链接: https://plus.google.com/+JoshBrown42/posts/FzNghPbKk2s

@chaossss


Please provide this Library with Stable Version

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This  Library is pretty good Just need a stable version instead of Beta Version

N/A

N/A

N/A

N/A

N/A


refactoring draw(Graphics g)

https://github.com/1chooo/uml-editor/issues/12

Please remove https://github.com/1chooo/uml-editor/pull/23/commits/34010279e4568b830ff7d86a2fa66a5a2555b2b4 and https://github.com/1chooo/uml-editor/pull/23/commits/cb70ce0e97a3bf9af2f7a5b20c713d3371f9d132 because they are not related to this PRs. Also please check the message I left in #22.


Assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
I have made changes to assignment.sh to complete my assignment.
What did you learn from the changes you have made?
I learned how to clone repo from my git to my pc and how to create directories and copy/move files.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
I think this is the easiest approach to make all these changes.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
I was unable to locate my rawdata folder, though it was in my local files and also had difficulty creating inventory file. The first issue I have resolved by trying multiple things and the second with the help of peers and chat gpt
How were these changes tested?
These changes were tested by executing commands in Git bash
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

You did a great job here!
Just a few things to note:

The branch name should be assignment not assignment-1
These two lines do not need to be in your file:

git pull https://github.com/UofT-DSI/shell coworker-changes --no-rebase
git status

Please go ahead and do the modifications, and then let me know. Thanks!

Hi Pedam,
Hope you are doing well.
I've made the changes as requested. Please have a look and let me know.
Thanks,
Saleha.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Pedram @.>
Sent: Thursday, November 28, 2024 12:38:04 AM
To: saleha-12/shell @.>
Cc: saleha-12 @.>; Author @.>
Subject: Re: [saleha-12/shell] Assignment 1 (PR #3)
You did a great job here!
Just a few things to note:

The branch name should be assignment not assignment-1
These two lines do not need to be in your file:

git pull https://github.com/UofT-DSI/shell coworker-changes --no-rebase
git status
Please go ahead and do the modifications, and then let me know. Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/saleha-12/shell/pull/3#issuecomment-2505299146, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BM5KNIH2UB7XDPWBPEQ6EYT2C2T3ZAVCNFSM6AAAAABSLYNJMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBVGI4TSMJUGY.
You are receiving this because you authored the thread.Message ID: @.***>


Convert to Python3

using : find . -name '*.py' | xargs 2to3 -w

Hi @ralic thank you for this pull request. It look like the tool you used made a number of changes that actually break ChatterBot in both Python 2 and 3.
Did you encounter any errors in Python 3 that motivated these changes?
ChatterBot designed to be compatible with both Python 2 and 3 and it has continuous integration that helps ensure this https://travis-ci.org/gunthercox/ChatterBot (however there is always a chance that something was missed in due to an oversight in the test cases)


WFLY-5699 Upgrade common collections to 3.2.2 to address serialization exploit

for more see: https://issues.apache.org/jira/browse/COLLECTIONS-580

We should remove this dep, but I will merge for now.


Explicitly support nameWithLanguage and textWithLanguage values

Currently the IPP data file format used by ipptool and ippserver does not support language overrides for nameWithLanguage or textWithLanguage values, preventing testing of that part of IPP.
Ideally we should allow strings of the form:
"my string"(language)

Moving this to future since it will require more extensive libcups changes that upstream is currently not able to accept.


When "add to #commits" isn't checked the channel is added to #commits anyway

Tested on n.tkte.ch in "Sync with GitHub".

It also looks like that when "update projects" isn't checked, webhooks aren't added at GitHub.

v2 kills this completely, there are no default channels.


Adding Facebook prophet

Facebook Prophet is a strong time series forecasting tool that may be used for derivative hedging. It is an open-source program created by Facebook that forecasts time series data using a decomposable additive model. This model considers trends, seasonality, and holidays, making it especially useful for data with numerous seasonalities.

Hi Akshat, could you elaborate a little, I would like to work on it.

I firstly suggest you to please refer to https://www.kaggle.com/code/paramarthasengupta/microsoft-stocks-price-prediction
I am also working on similar type of project wth hedge funds.Kindly ask if you need more clarification.

Hey have you done anything on this issue??

yeah, I am still open for it.Please feel free to ask anything related to it.

Okay, I will update you soon.
On Thu, 22 Feb, 2024, 08:44 Akshat Sharma, @.***> wrote:

yeah, I am still open for it.Please feel free to ask anything related to
it.
—
Reply to this email directly, view it on GitHub
https://github.com/Akshat111111/Hedging-of-Financial-Derivatives/issues/10#issuecomment-1958580552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWWLIHCRNNVXZIOHHX67JHLYU2ZYRAVCNFSM6AAAAABAJTPQRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJYGU4DANJVGI
.
You are receiving this because you commented.Message ID:
@.***
.com>

Could you please assign this to me, thanks

@Akshat111111  I have made a pull request . Kindly review it allow it to merge . Thank you

Kindly do the task so that I will merge it

Where can I find the dataset for the task

You can use any financial APIs or select dataset from kaggle.If you are using kaggle, dont forget to give references.

@Akshat111111  Kindly evaluate the changes made allow it to merge . Pls leave a feedback in case you feel any changes are required .

ive made a pull request, kindly check


Does Not Recognize Common Indexes

For some reason SwiftLCS does not recognize the two collections as containing common indexes. The two arrays being compared are exactly the same, but SwiftLCS removes all indexes and adds them back instead of simply recognizing that they are common indexes.
I essentially have a query that pulls all data, and I have SwiftLCS compare the cached data array to the new, in-coming data array.

Can you share a demo project to reproduce the issue?
I suspect that the issue might be with the implementation of Equatable of your data objects.

Sure, would a code snippet suffice or would you like me to upload a repo with the reproducible problem in a project?
The two arrays being compared are arrays containingPFObject's.

I believe that PFObject doesn't override isEqual: so it's using the NSObject default implementation that do pointer comparison.
When you download the newer data you have new objects with different pointers so SwiftLCS thinks that you're removing all the items and adding new items. To use SwiftLCS with Objective-C classes you should override isEqual: and hashCode (see Objective-C Equality).\
I'm closing the issue, feel free to reopen it if you find out this is not your issue.

Okay, thank you for the info - I will look into it and follow up if the issue stems from something else.

Fixed by added the following extension to PFObject for future reference..
extension PFObject {

    open override func isEqual(_ object: Any?) -> Bool {

        if (object as? PFObject)?.objectId == self.objectId {
            return true
        } else {
            return false
        }
    }

}


Add application layer documentation

closes #54

Codecov Report

Merging #64 (dd287ea) into main (771f3bd) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   98.72%   98.72%
=======================================
  Files          47       47
  Lines         705      708    +3
=======================================
+ Hits          696      699    +3
  Misses          9        9

Impacted Files
Coverage Δ

lib/application/client/bloc/client_state.dart
100.00% <ø> (ø)

...ion/client/register/client_register_validator.dart
100.00% <ø> (ø)

...ication/client/search/bloc/client_search_bloc.dart
100.00% <ø> (ø)

...cation/client/search/bloc/client_search_state.dart
100.00% <ø> (ø)

...pplication/client/search/client_search_status.dart
100.00% <ø> (ø)

lib/application/common/submission_status.dart
100.00% <ø> (ø)

lib/application/delete/bloc/delete_bloc.dart
100.00% <ø> (ø)

lib/application/delete/bloc/delete_state.dart
100.00% <ø> (ø)

lib/application/details/bloc/details_bloc.dart
100.00% <ø> (ø)

lib/application/edit/bloc/edit_bloc.dart
100.00% <ø> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Added the System trait

Closes #30
The new code somehow introduces a deadlock in the tests. AFAIK, all I did was changing a pulse/barrier into a channel.

Weird, if I move the same test code into the example, it works.
@csherratt do you see anything wrong here?

The test case that is failing is test_panics. I probably should have used something like timebomb to check for tests like this.
The test case is checking to see if the task panicking propagates to the dispatch thread. With pulse a pulse would be dropped forcing it into an error state. This would propagate to anything waiting on the pulse.
My guess is this is not the behavior of mpsc, it probably only enters an error state if all produces have been hung up. Which can't happen since you hold your own sender side.

@csherratt thanks for figuring this out! So what can be done to fix it nicely then?

Easiest way, add a drop guard to do the send in the case of a task panicking. This is the same technique the threadpool is doing internally. Well they spawn a thread, you would transmit the message.
https://github.com/frewsxcv/rust-threadpool/blob/master/src/lib.rs#L29-L60

@csherratt the more I look at threadpool the more I realize that we are essentially building on top of it without actually hooking up to the same internal mechanisms. We'd benefit from tighter integration. so perhaps we should just hand-craft something similar manually.

@kvark I don't disagree on the threadpool Their implementation is pretty much wait on a mutex for work to arrive. It's simple, but obviously sub-optimal.


Added missing security checks

Added the missing security checks for controller and navigation items in admin to Categories, Tags and Snippets.
Tasks:

[x] test coverage
[ ] gather feedback for my changes

Informations:

Q
A

Tests pass?
yes

Fixed tickets
fixes #467

BC Breaks
Security have to be set correctly, otherwise users won't be able to work on categories, tags and snippets

Doc
none

@wachterjohannes Fixed, also for the category bundle ;-)

@danrot perfekt (=


AuthZ instead of AuthN

Page improperly uses AuthN terminology instead of AuthZ

No, the article is correct. Look at the API
.RequireAuthenticatedUser()

Thanks for your contribution, @michaellyoungg. Be sure to check What's New early next month, where you'll see yourself listed as a valued contributor.


Purge command is not working

The documentation says that workspaces can be purged using the web api. However the documented command is not working ("'force' is not recognized as an internal or external command,
operable program or batch file") as it is not following the web api documentation (https://docs.microsoft.com/en-us/rest/api/loganalytics/workspaces/delete). Please tell us how to get rid of these "shadows".
 This should be an option that needs activation and not activated by default!

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

ID: 86984751-da7f-c77b-c78b-a9c9ffacb4a3
Version Independent ID: 57838082-5ee8-097c-9814-803a5077ba4e
Content: Delete and recover Azure Log Analytics workspace - Azure Monitor
Content Source: articles/azure-monitor/platform/delete-workspace.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

"You can recover your workspace by creating a workspace with the details of the deleted workspace, these include Subscription ID, Resource Group name, Workspace name and Region."
This is not working and giving "The workspace name xy is not unique".

Partially my bad: I was using the web api call with the "az rest" command and was escaping the ampersand wrong. In powershell it requires "...?api-version=2015-11-01-preview"&"force=true". Not very intuitive.
By fixing this the "not recognized" message disppears. However - nothing happens. There is no output and recovering or recreating the resource still fails with the "not unique" exception.

@srudin Thanks for your comment. We will review your issue and get back to you shortly.

@srudin Thanks for reaching out!!! To assist you in deleting the workspace, send mail to<EMAIL_ADDRESS>include your subscription ID , workspace name and a link to this GitHub thread (for context).

@srudin We will close the issue for now. Kindly reach out to us via email as mentioned above. If there are further questions , please revert and we will be glad to assist you.


Translate April 17 report into Italian

Hello @nextstrain/italian!
The situation report from 2020-04-17  is ready to be translated to Italian. Sorry it’s a little late this week. The report focuses on the USA. We do have a significant linguistic diversity here, but I’ll let you decide if you want to translate it. In general, I do think it’s worth understanding how the virus spreads in other regions of the world, but I’m not doing the hard work of translation.
Branch to translate on: sit-rep_it
File to translate: https://github.com/nextstrain/translations/blob/sit-rep_it/narratives/translations_ncov_sit-rep_it_2020-04-17.md You can translate directly in this file, but please remember to push often as other people may be working on the translation!
Link to preview: https://nextstrain.org/community/narratives/nextstrain/translations@sit-rep_it/ncov/sit-rep/it/2020-04-17
Twitter post to translate:
“This week’s #COVID19 situation report covers the outbreak in the United States. It is now available in Italian. Thank you @, @, and @ for the rapid translation.”
Logistics
Please update the translations board as you work on the translation: https://github.com/orgs/nextstrain/projects/7
Please assign yourself to the translation if you’re working on it this week.
When a translation is ready for publication -- at minimum meaning two people have worked on it and both agree that it’s ready -- please create a pull request to merge to master and tag me and the issue in it!
Discussion board for your team is here: https://github.com/orgs/nextstrain/teams/italian
Standard rules for translation:
Note that the file to translate is in Markdown. In general, it is important not to change any formatting or links. (Although it is okay to update Wikipedia or CDC links to the appropriate language version.) Comments to help with translation are noted in comment symbols like <! -- Comment -->.
Please add your name under translators: with a tab, dash, and new line, formatted just like authors are. Please also add a link to your professional page (can use Twitter) under translatorLinks:.
Note: You may know of other great resources available in Italian; please don’t add them to the situation report. We need to keep content the same across all translations, so we can be confident in the accuracy of what we publish. Thanks for understanding.
At minimum, I require two sets of eyes on every translation; one of those people should have a background in phylogenetics. Although if you have more, that’s fine too.
Feel free to use a live, online editor like hackmd (sign in with GitHub) to translate together.
As always a hearty thank you! I am really enjoying working with you all, even if it’s just digitally. Take care!

Hi all, I thihnk that it is useful to translate also into Italian, people can decide for themselves if they want to read or not. It is possible that this translation is going to take some more time, but I think this is not a real problem!
Thank you @cassiawag and the whole team for producing these reports!

I agree!

I agree too!
How do we do?
On Sun, Apr 19, 2020, 10:21 al-epo<EMAIL_ADDRESS>wrote:

I agree!
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nextstrain/translations/issues/63#issuecomment-616084032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACW4WUI23GFSOD4OLLXOBHLRNK7BVANCNFSM4MLOTJHQ
.

I wrote here https://github.com/orgs/nextstrain/teams/italian

HI @cassiawag we are ready for publication, but for some reason I can't move it to the "ready for publication" column.
Do you need help with the translation of the tweet?

@sarameiqi, yes please!
"This week’s #COVID19 situation report covers the outbreak in the United States. It is now available in Italian. Thank you @, @, and @ for the rapid translation."

Here it is!
Il rapporto sulla situazione del #COVID19 di questa settimana, che si occupa dell'epidemia negli Stati Uniti, è ora disponibile in italiano. Grazie @ Grazie a @MEKlab ,@ SaraMeiqi , e @alice_ledda_ per la rapida traduzione!

Published and tweeted! Thank you SO MUCH ladies!


Change hardcoded 1hr in knife status -h

This is a pre-screen for a feature that I am committing to implementing. I don't want to do work that is just going to get rejected, so I'd like to start the conversation here.
knife status has:
    -H, --hide-healthy               Hide nodes that have run chef in the last hour

It's poor.
Why?

For one, the definition of "healthy" is overloaded here to be "a host that has run chef". It makes no check in the code to determine if the chef run was successful, so it has the capability to provide false positives for "health". Just because a node object was saved with ohai_time set doesn't mean the Chef run was successful. There are exception handlers like lastrun that intentionally save the node object on exception.
The 1 hour hardcoded time is totally arbitrary. Perhaps "healthy" to others means "nodes that have run chef in the last 4 hours" (or 30 minutes, or 3 days...).

Option 1: Least disruptive
Add an optional argument of "MINS". Not specifying it will result in the previous behavior of checking for "healthy" hosts where "healthy" means ohai_time was in the last hour. This does not solve "Why: 1" above. It also is odd in that it is taking an optional argument. I don't believe I've seen any other Chef command allow for that.
    -H, --hide-healthy MINS         Hide nodes that have run chef in the last hour (or MINS)

Option 2: Fully corrected
This removes "health" and "healthy" from wording so does not set expectations. It keeps the short option as '-H'. It also requires MINS. Commence option name bikeshedding... in 3... 2... 1...
    -H, --hide-by-time MINS         Hide nodes that have run chef in the last MINS minutes.

Thoughts?

I like option 2. You're right -- "healthy" is misleading. After all, the node could be checking in and still have the flu :grin:

I would love to see option 2 for sure.  I actually had the same exact thoughts when I saw this command after starting to use chef a few weeks ago!

It also appears that the -H switch doesn't actually do anything, as does not exclude nodes that have updated recently; they're in there along with everything else.
(note that I've changed the server names)
[icberg7@vm1 ~]$ knife status -H | tail
103 hours ago, server10, redhat 6.6.
75 hours ago, server09, redhat 6.7.
8 hours ago, server08, redhat 6.7.
8 hours ago, server07, redhat 6.7.
5 hours ago, server06, redhat 6.6.
4 hours ago, server05, redhat 6.7.
28 minutes ago, server04, redhat 6.6.
23 minutes ago, server03, redhat 6.7.
20 minutes ago, server02, redhat 6.7.
2 minutes ago, server01, redhat 6.7.

I've seen this behavior with knife running under both chefdk-0.5.1 and chefdk-0.6.2

@juliandunn et al, is there anyone else who should review this idea before I start writing the fix?

I'd just say to see if @chef/client-core has any feedback, otherwise go for it.

Seems reasonable to me.

[@dev:~/src/chef/lib/chef/knife] 3679-knife-status 1 ± knife status
32 minutes ago, dev, centos 7.1.1503.
[@dev:~/src/chef/lib/chef/knife] 3679-knife-status ± knife status -H 33

[@dev:~/src/chef/lib/chef/knife] 3679-knife-status ± knife status --hide-by-mins 33

[@dev:~/src/chef/lib/chef/knife] 3679-knife-status ± knife status --hide-by-mins 30
32 minutes ago, dev, centos 7.1.1503.
[@dev:~/src/chef/lib/chef/knife] 3679-knife-status ±

The blank lines in the output courtesy of knife/core/ui.rb:msg() using puts without checking whether or not there is anything to put. It's rampant in Chef. Drives me nuts :)

The blank lines fixed in 3778 :)


Update auto-generated documentation

Repository: openmicroscopy/ome-documentation
Already up-to-date.
Generated by build OMERO-DEV-latest-docs-autogen#409.

--no-rebase

https://ci.openmicroscopy.org/view/Docs/job/OMERO-DEV-merge-docs/334/ is green with this included, merging ready for release.


feat: integrated tagsinput for project tags, fixes: dropdown for groups page

Fixes #2798
Fixes #2739
Describe the changes you have made in this PR -

Added Bootstrap Tagsinput using stimulus in Project.
Fixes Dropdown for groups page.

Screenshots of the changes (If any) -
https://user-images.githubusercontent.com/76901313/149915186-3ec26420-2cfb-42c7-80b7-2842101ee56a.mp4
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage decreased (-51.7%) to 30.587% when pulling 522f7cd840a04863a58bd70514b8d0a79f7a7fb1 on vedant-jain03:issue2798 into a00f85084a692decd4b192ef77cc9295c4a607ac on CircuitVerse:master.

@tachyons
I have imported bootstrap, jquery globally, and as a result dropdown is working fine for every page for using stimulus controller.
Kindly Review

@tachyons
I tried for group page dropdown the problem is when we try to import 'bootstrap' in the controller, navbar bootstrap becomes inactive and as a result, the dropdown is not showing. I tried some hacks by overriding files but somewhat how another problem is arising. So I think the best we can do is once write function for dropdown in connect() so that other controllers + initial+ dropdown works fine.
Although it was my solution, if someone have something in their mind, it would be really helpful.

@tachyons Kindly review.

we dropped bootstrap tags input


Ubuntu 20.04 - Links not opening in chromium browser

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I am new to atom and when I installed it, I wanted to click on the links in the welcome guide, but they are not opening.
After searching online, the only reference to this problem I found was this: https://discuss.atom.io/t/links-cannot-open-with-chrome/62435/2
And I tried it out and indeed, when I change the default browser to Firefox, the links are opening there.
But for Chromium the link opening seems to be broken.
Steps to Reproduce

Have Chromium set as your default browser
Click on a link in atom, e.g. any of here:

Expected behavior:
The links should open in Chromium Browser.
Actual behavior:
Nothing happens.
Reproduces how often:
100%
Versions
atom --version
Atom    : 1.49.0
Electron: 5.0.13
Chrome  : 73.0.3683.121
Node    : 12.0.0
apm --version
apm  2.5.0
npm  6.14.5
node 10.20.1 x64
atom 1.49.0
python
git 2.25.1
OS:
Ubuntu 20.04.1 LTS

Closing, as atom will be discontinued. Never used atom though as this issue didn't get fixed.


Added note regarding explorative programming to README.MD

:white_check_mark: Build PantherDI <IP_ADDRESS> completed (commit https://github.com/MarkusPalcer/PantherDI/commit/f0cca7027d by @MarkusPalcer)


⚙ Document how to specify multiple extensions

I looked through the README and could not find any example of how to do it.

Hi @spawnia,
This is mentioned here https://github.com/shivammathur/setup-php#extensions-optional
Also, In my opinion it would be better to update the existing examples in Extension Support to illustrate this.

Alright. It is really up to you, but I still think that my example does have value. The navigation led me to this section of the docs and I expected to find all relevant information about extensions there.


IdleDispatchedHandler requires a FnMut which is too strict

Delegate types require a closure with type impl FnMut to construct. The following demo fails to compile.
winrt::import!(
    dependencies
        os
    types
        windows::ui::core::{CoreDispatcher, IdleDispatchedHandler}
);

use windows::ui::core::*;

#[allow(unused)]
fn dispatch(dispatcher: &CoreDispatcher) {
    let data = vec![0];
    let _ = dispatcher.run_idle_async(IdleDispatchedHandler::new(move |_| {
        // error[E0507]: cannot move out of `data`, a captured variable in an
        // `FnMut` closure
        let _ = data.into_iter();
        //      ^^^^ move occurs because `data` has type `std::vec::Vec<i32>`,
        //           which does not implement the `Copy` trait
        Ok(())
    }));
}

Semantically, an IdleDispatchedHandler will be dispatched no more than once, and a FnMut is too strict than necessary. Given that it is not possible to tell from metadata how many times a delegate object will be called, can we have something like IdleDispatchedHandler::once to construct such delegate object with FnOnce, plus some runtime check to enforce that the closure cannot be called twice?

Sorry for the delay - I'm on vacation (mostly camping and offline). That sounds like a reasonable idea. Will look into it further when I'm back.

Just closing out some old issues I'm unlikely to get to. I think that in general this is not something we can guarantee is correct and the particular use case doesn't seem to be general enough to warrant further investment. Feel to chime in if you disagree.


View child fix for Angular 9

container element ref is undefined in the ngOnIt lifecycle when calling InitMapInstance function.
This is a fix so any projects using this repo can be upgraded to Angular 9.
#127

When will this be released ?


JSDoc fails parsing type imports containing */

Bug Report
🔎 Search Terms
JSDoc, type imports
🕗 Version & Regression Information

This is the behavior in every version I tried

⏯ Playground Link
Playground link with relevant code
💻 Code
/** @type {"*/"} */

/** @type {import('*/'.Foo)} */

/**
 * @type {import('*/').Foo}
 */

/**
 * @typedef {import('@*/foo').Foo} Foo
 */

JSDoc parsing fails on encountering the substring */ in type information.
This issue only occurs in JSDoc comments (i.e. type Foo = import('*/').Foo in a .ts file is fine).
In the context of npm, package names follow the pattern ^(?:@(?:[a-z0-9-*~][a-z0-9-*._~]*)?/)?[a-z0-9-~][a-z0-9-._~]*$, so modules containing */ such as @foo*/bar are valid and plausible module names. This bugs makes types from such modules nigh on inaccessible in JSDoc.
🙁 Actual behavior

Parsing the type fails, and the type isn't properly resolved.
🙂 Expected behavior

The characters are parsed correctly.

That's because */ closes multiline comment immediately. You have invalid JS on hand, so it's not JSDoc

ahh bit of an oversight there.
seems this is a general issue with jsdoc as a whole - https://github.com/jsdoc/jsdoc/issues/821
unfortunately the workarounds listed don't work in the context of type imports still, though.


Fix culture issue error that occurs when using turkish culture and Invoke-Formatter

PR Summary
Fixes #1095 as reported by @alatas (I could reproduce locally, see tests). Thanks for the detailed report.
This is an alternative to #1096 that seems to cause problems in CI, this approach is using ToLowerInvariant instead.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x Summarized changes
[x] Change is not breaking
[x] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[x] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

I will close this in favour of PR ##1099 which has a better, more generic solution to culture in general and addresses other culture issues as well


Enum Request - Sales Line (Table 37) - Field 107 "IC Partner Ref. Type"

Please implement the field "IC Partner Ref. Type" in Table 37 "Sales Line" as Enum.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[SPARK-7782] fixed sort arrow issue

Current behaviour::
In spark UI

In YARN

In jira

Can one of the admins verify this patch?

LGTM

ok to test

Merged build triggered.

Merged build started.

Test build #33611 has started for   PR 6437 at commit cd068b9.

Test build #33611 has finished for   PR 6437 at commit cd068b9.

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

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

Merged build finished. Test PASSed.

Thanks - I've merged this.


Allow ncov-ingest to download from non-AWS remote file paths

Context
To support running ingest on Terra, we need to support downloading existing Nextclade alignments and metadata from remote storage other than AWS's S3.
Description
We need to support the definition of a "source" bucket in the workflow configuration YAML associated with at least S3 or GS URIs. This means changing the name of the configuration variable from s3_src to a more generic name and modifying all of the logic in the workflow that refers specifically to downloading from S3 (e.g., "download_from_s3" script, etc.).
As part of this work, we will also need to modify the Pipfile used to populate the ncov-ingest Docker image by adding the Python bindings for Google Cloud Storage. See the Dockerfile for the docker-base image.
Examples
The modified ingest should continue to work with our production S3 buckets, but it should also work from GS buckets accessed through Terra.
See code in ncov for handling remote files.

As part of this work, we will also need to modify the Pipfile used to populate the ncov-ingest Docker image by adding the Python bindings for Google Cloud Storage. See the Dockerfile for the docker-base image.

This should not be necessary, as the nextstrain/ncov-ingest image is based on the nextstrain/base image. I believe the only reason the GCS Python bindings aren't available in the latest nextstrain/ncov-ingest image is that it predates (~3 Feb) the addition of the bindings to the nextstrain/base image (~11 Feb). I triggered an image update which is running now, and that should take care of bringing in the GCS bindings.

Works in the latest image now:
$ docker image ls --digests nextstrain/ncov-ingest
REPOSITORY               TAG       DIGEST                                                                    IMAGE ID       CREATED          SIZE
nextstrain/ncov-ingest   latest    sha256:2bb78caa7dfc38703724a5a36fa280744256d53f18c18c6e371a6c4048c77b65   d150b94d8afd   20 minutes ago   2.36GB
nextstrain/ncov-ingest   <none>    sha256:63bd513524c1e71eb1ce60c1fd4ac90d21ba443a43865c60062b96bf433e7536   d2f7e3fef415   3 months ago     2.4GB

$ docker run --rm -it nextstrain/ncov-ingest@sha256:2bb78caa7dfc38703724a5a36fa280744256d53f18c18c6e371a6c4048c77b65 python3 -c 'from google.cloud import storage'
# 👍 no error

$ docker run --rm -it nextstrain/ncov-ingest@sha256:63bd513524c1e71eb1ce60c1fd4ac90d21ba443a43865c60062b96bf433e7536 python3 -c 'from google.cloud import storage'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'google'


[Task]   com/usthe/alert/controller/AlertDefinesController.java

Description
Description
Help us impl Unit Test For com/usthe/alert/controller/AlertDefinesController.java
You can learn and refer to the previous test cases impl.

controller example unit case:  https://github.com/dromara/hertzbeat/blob/master/manager/src/test/java/com/usthe/manager/controller/AccountControllerTest.java
service example unit case:  https://github.com/dromara/hertzbeat/blob/master/manager/src/test/java/com/usthe/manager/service/TagServiceTest.java
jpa sql dao example unit case: https://github.com/dromara/hertzbeat/blob/master/manager/src/test/java/com/usthe/manager/dao/MonitorDaoTest.java

Task List

[ ]  Impl Unit Test For com/usthe/alert/controller/AlertDefinesController.java

please assign this task to me

please assign this task to me if others don't have time


Snyk fix 7e1aac1c1d5087cc21e11b4fe7d219f0

Changes
Please provide a brief description of the changes here.
Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[x] CHANGELOG.md updated to document new feature additions
[x] Appropriate documentation updates in the docs
[x] Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@JoKacar, you must sign the CLA before your PR can be reviewed.
Also, the PR's title and description need to follow our standard templates and describe the change; otherwise, we will close the PR. We have closed your other PRs previously because this was not done.


[Night Light] Clear search when triggering command

Description

clear search for no-view commands; the same behavior was implemented for the one view command by the author
shorten UI strings for error messages so that they don't get truncated (see screenshots)
refactor condition to check if user input is an integer, not just any number
upgrade project dependencies

Note
This PR makes further interaction with Raycast upon triggering the command more convenient. That said, I would personally much prefer for the main Raycast window to be closed, which I consider by far the most likely subsequent action. I don't want to do that manually every single time. I was not sure what the guideline is for changing the behavior in such a way for an extension that someone else created. Is that acceptable?

Screenshots

UI errors before vs. after

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for your contribution @kgrhartlage 🔥
@devmoath do you mind checking this?

I'll add the preference setting

@pernielsentikaer  I added the preference setting to close the main Raycast window when the command is triggered and updated the PR description accordingly. It is ready for review.
clearSearchBar() does not seem to work anymore 😕 Am I missing something or did this potentially break with a recent release? popToRoot({ clearSearchBar: true }) works perfectly fine

I just tested with the below code and it seems to work for me
import { List, clearSearchBar } from "@raycast/api";
import { useEffect } from "react";
import { setTimeout } from "timers";

export default function SystemMonitor() {
  useEffect(() => {
    setTimeout(() => {
      clearSearchBar();
    }, 3000);
  }, []);

  return <List />;
}

@pernielsentikaer this needs your attention.


Update KMS policy

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

The resource types for kms has changed to keys. Need to update the policy statement so that OKE can use KMS.
New or Affected Resource(s)

oci_identity_policy.oke-kms
Potential Terraform Configuration

# Copy-paste any Terraform configurations for how the requested feature may be used.

References

Closed in #372


[Android][Build 2.1.5-3] Query test "select" failed with float number

Functional test case "test_getDoc_withValueTypeDouble[doc_with_double_1]" failed while selecting a float number from the db.
Repro Steps:

unzip and use the attached cblite2 database file,
db_name.cblite2.zip
use cblite tool to select item_price == 2.754

Expected result:
return 1 row of data which has item_price is 2.754
Actual result:
return 0 row of data
Additional info (see attached logs):
cblite_tool_results.txt
./cblite query ~/temp/db_name.cblite2/  '{WHAT: [["."]]}'
[ * ]
[{"item":3,"item_price":3.754}]
[{"item":2,"item_price":2.754}]
[{"item":1,"item_price":1.754}]
[{"item":0,"item_price":0.754}]
[{"item":4,"item_price":4.754}]
./cblite query ~/temp/db_name.cblite2/  '{WHAT: [["."]],WHERE:["=",[".","item_price"],2.754]}'
[ * ]
./cblite query  ~/temp/db_name.cblite2/  '{WHAT: [["."]],WHERE:["=",[".","item"],2]}'
[ * ]
[{"item":2,"item_price":2.754}]
The following is from Android TestServer while search_query.execute();
SELECT fl_result(key), fl_result(fl_value(body, 'item')), fl_result(fl_value(body, 'item_price')) FROM kv_default WHERE (fl_value(body, 'item_price') = 2.754) AND (flags & 1) = 0

The number 2.754 can't be represented exactly in binary floating point, so there will be roundoff error. This can result in equality tests failing, depending on how the number got rounded by various pieces of code during ASCII–binary conversions. In particular, converting "2.754" to a float isn't going to give the same result as converting it to a double, so comparing the two types will fail.
tl;dr: This test is probably invalid and shouldn't be used. If you want to test FP for equality, you have to add a fudge factor, like abs(x - y) < 0.0001.

I think I'm gonna make like a lawyer and counter-sue.  I'm gonna open an issue against the test!

As described, this isn't a bug.  If there is still a problem after accounting for roundoff error, please open a new issue.

Tested with a test framework fix from Sridevi's working-in-progress branch, which handles this issue.


sphinx-proof on conda-forge?

Context
Currently, sphinx-proof is only available on pypi and can be installed via pip, but not via conda/mamba.
Proposal
I propose to also make sphinx-proof available on conda-forge, such that it can be installed using
conda/mamba and the conda-forge channel.
Creating a recipe is "easy" (can be automated) and there is overhead for maintenance is minimal
(the conda-forge bot will auto-create a PR when a new version is release on pypi).
Tasks and updates

[ ] disucss and decide
[ ] auto-create recipe using grayskull
[ ] stage recipe

See https://github.com/conda-forge/staged-recipes/pull/27118 :)


Add command line run options, including show output in terminal

Already there - 'run logged' - now greatly improved in develop branch.


I can't use the DAO in vertx-jooq-rx-reactive

I am not be able to instanciate the SqlClient,  that the Dao is expecting, I received the following error:
java.lang.ClassCastException: class com.julienviet.pgclient.impl.PostgresClientImpl cannot be cast to class io.vertx.reactivex.sqlclient.SqlClient (com.julienviet.pgclient.impl.PostgresClientImpl and io.vertx.reactivex.sqlclient.SqlClient are in unnamed module of loader 'app')
at com.example.starter.MainVerticle.handlerrequest(MainVerticle.java:58)
at com.example.starter.MainVerticle.lambda$start$0(MainVerticle.java:35)
Could you help me to use the daos properly?
Thanks

It looks like you still have a dependency to com.julienviet.pgclient.impl.PostgresClientImpl somewhere. Remove that and it should work.


360-Degree Viewer Example in documentation is broken

Unfortunately the used background-image link does not load the image due to an invalid certificate. Same applies to the "Photo credits: Microsoft Surface Pro 6" link below the example.

@christianMurschall What is your network and browser?

Company network.... I just checked. From my phone and from home it works.


ARROW-9464: [Rust] [DataFusion] Physical plan optimization rule to insert MergeExec when needed

This PR adds the first physical optimization rule, to insert explicit MergeExec nodes into the physical plan when operators require a single partition of input (such as GlobalLimitExec, SortExec, HashAggregateExec in final mode).
This removes the merging logic from the operators, making them more easily re-usable in different contexts (such as in a distributed query engine, which could provide its own planner and/or optimization rules).

@alamb @jorgecarleitao I'm pretty excited about this PR. This is a good example of how we can write optimizer rules against a trait-based plan.

It looks fantastic! Super excited to see this!
Gave it a quick look, and my understanding so far:

Physical nodes have requirements (required_child_distribution)
Physical nodes can fulfill requirements (output_partitioning)
The physical optimizer knows which physical nodes fulfill certain requirements (Merge fulfills Distribution::SinglePartition so far), and introduces them to fulfill a nodes' requirements, by using children to get the node's children and with_new_children to re-write the node with new children (that fulfill requirements).

Is this a good overview of this PR (before going into the specifics)?

It looks fantastic! Super excited to see this!
Gave it a quick look, and my understanding so far:
* Physical nodes have requirements (`required_child_distribution`)

* Physical nodes can fulfill requirements (`output_partitioning`)

* The physical optimizer knows which physical nodes fulfill certain requirements (Merge fulfills `Distribution::SinglePartition` so far), and introduces them to fulfill a nodes' requirements, by using `children` to get the node's children and `with_new_children` to re-write the node with new children (that fulfill requirements).

Is this a good overview of this PR (before going into the specifics)?

Yes, I think that's a good overview.

Thanks for the reviews @alamb and @jorgecarleitao .. I have implemented the suggested and also improved the error handling so we return Err results rather than panic if the partitioning is incorrect at execution time.


Issue with getting pass percentage

TestNG Version

Version: 6.9.10

Hi Guys,
I am trying to get pass percentage of all the passed test methods using below code:
iTestResult.getMethod().getSuccessPercentage()
or
iTestContext.getAllTestMethods()[0].getSuccessPercentage()
But in both the cases even if the test fails it returns 100%.
Can I please know if there is anyway I can get the pass percentage of only passed tests.

@arunaousula9421 - The latest released version of TestNG is 7.1.0. Please try again using that. If the problem persists, please attach a sample standalone that can be used to reproduce the problem.
If there are no updates on this issue, we will have to close the issue.

Sure @krmahadevan . Will try the latest version and see if that works.

@krmahadevan As you said I have updated testng version to 7.1.0. But I still see the issue in the getting the pass percentage.
1582026737617000000 100  com.rxp.hpp.selenium.tests.hpp.qa.dcc.V2DccRedSysST         FAIL
Though the test is marked as failed we still see the percentage as 100 instead of 0.
Here's the sample code, I have used
.addField("Passpercentage", iTestResult.getMethod().getSuccessPercentage())
or
.addField("Passpercentage", (iTestContext.getAllTestMethods()[0].getSuccessPercentage()))

@arunaousula9421 - I took a closer look at this issue.
This is NOT a problem with TestNG but it looks like you are trying to read up the value via the wrong means.
org.testng.ITestNGMethod#getSuccessPercentage returns the pass percentage that was set via the @Test annotation's successPercentage attribute.
Its not meant to getting the pass percentage as what you are expecting.
Now coming back to pass percentage evaluation that you are trying to do. Its not clear what you mean by pass percentage of "passed tests".
Either ways I would suggest that you please create a full fledged example and add more details in terms of what you are trying to do and post it as a question on the testng-users google forums.
Closing this issue with resolution as Working as designed


How to add GBIF node logo in the footer?

Hi,
As mentioned in the node logo guidelines, we would like to add the GBIF node logo in the footer of the antarctic hosted portal. However, I couldn't find a way to do it.
@rukayaj and I have explored the html here but it doesn't seem like it is possible to add a logo as depicted in page 8 of the node logo guidelines (see screenshot below)? Can you please advise how can we do that? Thank you so much!!
Related issue: https://github.com/gbif-norway/helpdesk/issues/115

You could add something like this as a new column in your footer.
columns:
  - description: |
      <img src="/assets/images/your-image.png" width="150" />

Ahhh~~ You are the best Morten!! It works!! Thank you so much!!


refactor: removed unnecessary line

The ternary operator can be used.
This is smarter than before.
The author should do the following, if applicable

[ ] Add tests
[x] Run tests
[x] bun run format:fix && bun run lint:fix to format the code

hmm... why JSR --dry-run is rejected now...

hmm... why JSR --dry-run is rejected now...

resolved
https://github.com/honojs/hono/pull/2870


fix on launch.json

Error: launch: program 'SOME_PATH\aspnetcore-angular2-universal\bin\Debug\netcoreapp1.0\aspnetcore-angular2-starter.dll' does not exist:

right path for this build is: 'SOME_PATH\aspnetcore-angular2-universal\bin\Debug\netcoreapp1.0\aspnetcore-angular2-universal.dll'

Thank you Paolo, just noticed that as well!

you're welcome


Update README.md

azure guide added

Hi @OhhHellooow thank you very much for your PR. I think your info is more helpful on the home-gallery-doc repo, were I added your changes there https://github.com/xemle/home-gallery-doc/commit/785eaafc9305b64ebcdebc26c39c6e0ddd193305. See https://docs.home-gallery.org/install/index.html#quickstart-using-azure-container-instance-aci
Is that OK for you?

Sounds good!


Prepare the release

And I'd also like to merge #883 before the release

And we are also waiting for a new release of ocaml-git by @dinosaure