Fix min max on rowversion/timestamp mssql

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[x] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@ayush-shah  from what I see we'll need to create a custom type (similar to what we did above) that will 1) handle the hex string for the mssql timestamp (from what I see it returns and hex string) -> convert hex string to int (e.g. int(, 0) -> this implies the hex string is received as with 0x otherwise we need to make it base 16 int(hex, 16)) or 2) return the value as is the dialect is not mssql
More info here


Att-tput-driver: small style/naming/typos enhancements

Small style changes, removal of unused variables/parameters renaming (from underscore to camel case as preferred by Go's guidelines), fix typos and comments.

We avoid three addresses in each subnet: base+0 because it is traditional, base+1 for router, and last for multicast.

G

We avoid three addresses in each subnet: base+0 because it is traditional, base+1 for router, and last for multicast.

Right, thanks!


Oracle exception ORA-3113 during random row validation

Sample command reported to be raising ORA-3113:
data-validation validate custom-query row --source-conn ora --target-conn pg \
-sq "select * from dvt_test.t1 where ID between 100 and 500" \
-tq "select * from dvt_test.t1 where id between 100 and 500"  \
--primary-keys id  --hash id  \
--use-random-row  --random-row-batch-size 5

Exception:
    cursor.execute(statement, parameters)
cx_Oracle.DatabaseError: DPI-1080: connection was closed by ORA-3113

@nj1973 could you please check if this exception still happens? Regardless, I'll change its label and added low priority for now

I was never able to reproduce it and was waiting on more information from the customer. I'll close it because we have nothing useful recorded here.


Add hardware state for OpenStack Hosts

Purpose or Intent
Adds an introspected and a provision_state attribute to the
Hardware model. Introspected means Ironic introspection has already
been run and has gathered the hardware details. Provisioning state
indicates whether the node is available for use or is undergoing
some sort of maintenance.
The attributes are needed to indicate where each node is at in the
Ironic workflow. Typically nodes are added an placed in manageable
state. Nodes are then introspected. And then finally placed in the
available state when ready for use.
The patch also adds the new attributes as additional fields to the
Host summary page.

@Fryguy please review SQL migration

SQL migrations are just adding columns, no tests needed


added tests with nop and try objects

#66

@Graur, we need gh-pages brunch for paste tests in paper

@MikhailLipanin @yegor256 I faced some problems with github pages checks. Can somebody check do we have such option turning on right now: https://github.com/JamesIves/github-pages-deploy-action/issues/1110#issuecomment-1117481234 ?

@Graur we have read+write permissions

@rultor merge

@rultor merge

@mikhaillipanin OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@mikhaillipanin Done! FYI, the full log is here (took me 3min)


dead code with winrt/base.h

I just include the header and it gives me so much dead code in the assembly. This means some header files should not be included.
"??$_Getvals@_W@?$time_get@DV?$istreambuf_iterator@DU?$char_traits@D@std@@@std@@@std@@IEAAX_WAEBV_Locinfo@1@@Z": # @"??$_Getvals@_W@?$time_get@DV?$istreambuf_iterator@DU?$char_traits@D@std@@@std@@@std@@IEAAX_WAEBV_Locinfo@1@@Z"
.seh_proc "??$_Getvals@_W@?$time_get@DV?$istreambuf_iterator@DU?$char_traits@D@std@@@std@@@std@@IEAAX_WAEBV_Locinfo@1@@Z"
# %bb.0:
	pushq	%r15
	.seh_pushreg %r15
	pushq	%r14
	.seh_pushreg %r14
	pushq	%rsi
	.seh_pushreg %rsi
	pushq	%rdi
	.seh_pushreg %rdi
	pushq	%rbx
	.seh_pushreg %rbx
	subq	$80, %rsp
	movq	%rcx, %r14
	leaq	32(%rsp), %rdx
	movq	%r8, %rcx
	.seh_stackalloc 80
	.seh_endprologue
	movq	%r8, %r15
	callq	*"__imp_?_Getcvt@_Locinfo@std@@QEBA?AU_Cvtvec@@XZ"(%rip)
	vmovups	32(%rsp), %ymm0
	vmovups	44(%rsp), %ymm1
	vmovups	%ymm1, 56(%r14)
	vmovups	%ymm0, 44(%r14)
	vzeroupper
	callq	*__imp__W_Getdays(%rip)
	testq	%rax, %rax
	je	.LBB1_2
# %bb.1:
	leaq	40(%r15), %rcx
	movq	%rax, %rdx
	movq	%rax, %rsi
	callq	*"__imp_??4?$_Yarn@_W@std@@QEAAAEAV01@PEB_W@Z"(%rip)
	movq	%rsi, %rcx
	callq	*__imp_free(%rip)
...
...

Sounds like an issue for the C++ compiler/optimizer. I'd check with Visual Studio or https://stackoverflow.com/.

no. it has nothing to do with compiler and optimizer. looks like more of problems of msvc stl?
A lot of headers you just should not use tbh.

Then maybe you wanted https://github.com/Microsoft/STL?

Then maybe you wanted https://github.com/Microsoft/STL?

I do not know.
That is exactly what happens when you define things in headers without using inline to ALL functions or defining some global objects.
All things in headers MUST be inline.


Count notes with links in properties (frontmatter) as "not orphaned"

As far as I can tell, if I don't have a link in the body of a note, it will be counted as "orphaned". Now, Obsidian has introduced the concept of "properties" (which is just rebranded yaml frontmatter), in which you can also have links like so:
---
company: "[[Company A]]"
---

This is a meeting with some people from this company.

The above note will still be counted as "orphaned", even though it has a link to Company A.

I would also like to request this feature - almost all of my linked notes contain some form of frontmatter links.

This is now resolved with bb1b21b7725b7bc3f8ad1cc2817fcfde9002f15e and released :tada:


The key / environment variable MaxPublishRequests does not work as expected

Describe the bug
The key / environment variable MaxPublishRequests seems to have no impact on the publisher behaviour.
To Reproduce

Start publisher via docker compose without this variable (default value) and with diffent values of this variable
Configure publisher with appropriate configuration
Check the value of CurrentPublishRequestsInQueue for the publisher session in the server with any client (e.g. UaExpert)
The value of CurrentPublishRequestsInQueue is allways equal to the number of created subscriptions independent of the value of MaxPublishRequests

Expected behavior
CurrentPublishRequestsInQueue == MaxPublishRequests
Screenshots
1

2

3

Desktop (please complete the following information):

OS: Windows 10
Browser chrome
Version 2.9.9

Unfortunately this also requires a change in the UA stack, which overrides the max setting and always at least submits subscription count publish requests.  Hence this will take a bit of time.

#2352


Update README.rst

Codecov Report

Merging #239 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #239   +/-   ##
=======================================
  Coverage   89.46%   89.46%
=======================================
  Files          24       24
  Lines        1168     1168
  Branches      134      134
=======================================
  Hits         1045     1045
  Misses         93       93
  Partials       30       30


[#591] Update dotnet projects to target NET 6

Addresses # 591
Description
This PR updates all Tests, Bots and Libraries projects from netcoreapp3.1 to net6.0, as well the CI, Deploy, Functional and Integration test pipelines, and documentation.
Detailed Changes

Update all Test, Bots and Libraries projects' TargetFramework from netcoreapp3.1 to net6.0.
Applied Analyzer's fix for WaterfallHostBot regarding StringComparison.Ordinal warnings.
Fixed Analyzers error SYSLIB0014 in the WaterfallSkillBot.FileUploadDialog, where the WebClient class is obsolete, and instead use HttpClient.
Updated CI, Deploy and Test Scenarios pipelines vm image from windows-2019 to windows-2022, the step to use net6.0 instead of 3.1 and the MSBuild version from 16 to 17.
Updated related documentation in markdowns, code and json files.

Testing
The following image shows the CI and Test Scenarios pipelines working after the changes.

/promoted 593


Release for \n fix

Hello! I opened a bug report for a utility I was using, which uses this library under the hood. Turns out there's a fix for the bug in question (#24), but it hasn't been released yet. Any chance the lovely maintainers could cut a new release with that fix? ❤

+1 for a new release.
I'd love to get rid of the regex dependency

Hi @ZoeyR! Sorry to bug you, but would it be possible to cut a new release? (I'm also happy to do it, if you're comfortable giving me maintainer bits. I was a maintainer of the old git repo, although I don't think I ever had an owner bit on crates.io.)

Published, sorry it took so long

Thanks for the new release 👍 Please tag, push that tag, and update the changelog


Allow for an app to be pre-installed

Update cosmos to allow for an app to be pre-installed from the initialized repository set and priority.
Cosmos SHOULD NOT do anything with artifacts or the package itself, just install it like it would a normal service.

No activity in a while. Closing. I think we have an Epic for this.


Typo correction in CantorSet.lean

Correcting a typo in a doc-string of CantorSet.lean where the function description did not match the formal specification.

Co-authored-by: Ruben Van de Velde<EMAIL_ADDRESS>

Remark: I put Ruben-VandeVelde as a co-author because he spotted the typo in https://github.com/leanprover-community/mathlib4/pull/11761#discussion_r1557106996

Thanks! Sorry for introducing that error.
bors merge


Add probable starter attributes, fixes #32

The overview API hits linescore.xml which has all of the useful probable starting pitcher info (ERA, names, player IDs, etc.)
Example: http://gd2.mlb.com/components/game/mlb/year_2018/month_02/day_25/gid_2018_02_25_chnmlb_sfnmlb_1/linescore.xml
This adds all of those attributes. Would appreciate a quick review and tag 😄 my LED board wants to scroll starting pitcher text across the screen and this file has the info readily available on at least the day before the game it seems.

@panzarino so it looks like the unit tests are hitting the MLB servers directly. Since the probable starter attributes are time-dependent (they go away once the game starts) any test that grabs them would start failing.
Also don't really see much I can improve upon for the code that's triggering the climate issue but I'd appreciate any feedback.

@ajbowler looks good to me, for the tests that constantly change I just set it to test that the attributes are the correct type (basically just making sure they aren't null and the method actually works for the current data, ex. test roster). Can you please add tests for these new attributes and then I will merge.

@panzarino code updated. The Travis build appears to be failing on some unrelated test

@ajbowler not sure why that happened, restarted the build and it passed. Great work!


GPU memory not cleaned properly when using multiple workers in DataLoader

Hi all!
When I try to use multiple workers in the DataLoader by specifying num_workers some of the processes stay alive after the run and occupy GPU memory.
For my tests I am using this script and the following setup
python3.8.10
pytorch=1.9.0=py3.8_cuda11.1_cudnn8.0.5_0
pytorch-lightning==1.4.2
ray==1.6.0
ray_lightning==0.1.1

When setting e.g. num_workers=4 here I also increase num_cpus_per_worker in the RayPlugin to 4 ( I also tried a larger number then num _workers specified in the DataLoader). I was not able to make this issue fully reproducible but in 3 out of 4 runs the GPU memory is not cleaned and several ray::RayExecutor.execute() processes are sleeping.
# output of ps aux | grep ray
markus.+ 21130  1.2  0.1 210983644 2969908 pts/8 S  10:36   0:00 ray::RayExecutor.execute()
markus.+ 21131  1.4  0.1 211112136 2963912 pts/8 S  10:36   0:01 ray::RayExecutor.execute()
markus.+ 21275  1.4  0.1 210982456 2967680 pts/8 S  10:36   0:01 ray::RayExecutor.execute()

I have managed to get rid of the problem by setting persistent_workers=True in the DataLoader for now, but these processes should be cleaned in any case.

@MarkusSpanring by any chance you got this issue resolved?

@scv119 not yet.
FYI, I was able to boil it down to the PyTorch DataLoader. I have opened an issue already but there is no comment/fix yet.

Thanks, I also encounter this issue when triggering validation_end. Hope it will be fixed soon

I get this issue as well. Currently I solve it with ray stop and ray start --head --port=6379 --object-manager-port=8076 --autoscaling-config=~/ray_bootstrap_config.yaml --metrics-export-port=8080 but this is harder when you have multiple nodes


feat: add billing related customer fields

Overview
This change adds the required fields and the related schema. The strategy is that if an override should be set, the billing_profile_id must be set.

This is back to draft, as the billing overrides must be handled by the billing service or we will end up in customer being a kitchen-sink service.

Closing in favor of:
https://github.com/openmeterio/openmeter/pull/1566
https://github.com/openmeterio/openmeter/pull/1572


negate current query-token command

To quickly change query word to !word.

Change mind, instead of supporting query string transformation command,
I added following configuration parameter

negateNarrowQueryByEndingExclamation, true for select-files provider, and false for other provider.
When true, !word to exclude word matching item can be written as word!.
So !word is euqal to word!.
When excluding markdown file in selec-files provider

First type query m, then d, now query is md and list md matching files
Then add ! all filex except md matching files are listed.
Ending ! is better than typing !md keystroke, since user can see what file to is excluded by adding !. So ending ! allows "confirm then exclude" flow.


gadgets/snapshot_socket: Missing tests

The snapshot_socket gadget is missing the tests. We can port them from the built-in gadget (https://github.com/inspektor-gadget/inspektor-gadget/blob/main/integration/k8s/snapshot_socket_test.go)

hey, I would like to work on this. Please assign me.

hii @mauriciovasquezbernal  I would like to work for this issue as it aligns with my skills and interests , could you please assign it to me so that I can work over it

@kairveeehh Hey, many similar issues are open, so go ahead. I'm currently working on this

@kairveeehh Hey, many similar issues are open, so go ahead. I'm currently working on this

But nobody is assigned , so ig issue is open

Hi folks, I assigned it to @btwshivam as they're first. @kairveeehh feel free to start working on other issues on the repo (always commenting before to avoid multiple folks working on the same).

@mauriciovasquezbernal I'll work on it as assignee didn't seem to be working on it.

@mauriciovasquezbernal I'll work on it as the assignee doesn't seem to be active.

@mauriciovasquezbernal The tests are failing due to timeout. Can environment setup cause this problems?
`IG_PATH=/home/anuj_agrawal/inspektor-gadget-lfx/inspektor-gadget/inspektor-gadget/kubectl-gadget IG_RUNTIME=kubernetes go test -v
=== RUN   TestSnapshotSocket
utils.go:178: Seed used:<PHONE_NUMBER>569292488
command.go:92: Run command(Create test namespace):
command.go:94: Command returned(Create test namespace):
    namespace/test-snapshot-socket-1636736912831736689 created
    NAME      SECRETS   AGE
    default   0         0s

command.go:111: Start command(Run test-snapshot-socket)
command.go:92: Run command(SleepForSeconds):
command.go:94: Command returned(SleepForSeconds):

command.go:92: Run command(WaitForTestPod):
command.go:94: Command returned(WaitForTestPod):

    pod/test-snapshot-socket condition met

command.go:128: Stop command(Run test-snapshot-socket)
command.go:130: Command returned(Run test-snapshot-socket):

    pod/test-snapshot-socket created

command.go:92: Run command(Run_snapshot_socket):

panic: test timed out after 10m0s
running tests:
TestSnapshotSocket (10m0s)
goroutine 30 [running]:
testing.(*M).startAlarm.func1()
/home/anuj_agrawal/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.22.0.linux-amd64/src/testing/testing.go:2366 +0x385
created by time.goFunc
/home/anuj_agrawal/go/pkg/mod/golang.org/toolchain@v0.0.1-go1.22.0.linux-amd64/src/time/sleep.go:177 +0x2d
`

@btwshivam are you still working on this one?


[Feature request] Ability to link Gitlab and Bitbucket profiles

I have a feature request to be able to link Gitlab and Bitbucket accounts. I use Gitlab for 95% of my projects since it has free private repos and Github doesn't. This would make Sourcerer 100x more valuable to me since it would then have most of my repos.

@PlanetVaster note that you can process any repos as private repos on your local machine. There will be no links in the repo table, but the profile will have everything else. Gitlab/bitbucket support is something we are looking into, although there is no ETA yet.

+1 for GitLab Public and Private Repos

The website says: "Include repos from GitLab, GitHub, Bitbucket, and more." but there is no option to link bitbucket accouts, I think you guys should remove that until the integrations is ready or explain that is a feature comming out soon.


Zuul is not sending logs to Zipkin

Hi,
I have setup like below
Zuul --> micro service
It was working fine when i am not using sleuth with zuul.
Micro service is able to send logs to zipkin.
micro service to zipkin is fine
Zuul --> Zipkin is not working
When i m adding sleuth with zipkin , its not working. Means i am not able to see the result and getting below error.
2016-12-09 17:11:34.929  WARN [gateway-service,fe145b7b89120e25,e28d9e0575f254f,true] 48086 ---
[nio-1113-exec-9] o.s.cloud.sleuth.util.ExceptionUtils     : Tried to close span but it is not the current span: [Trace: fe145b7b89120e25, Span: fe145b7b89120e25, Parent: null, exportable:true].  You may have forgotten to close or detach [Trace: fe145b7b89120e25, Span: e28d9e0575f254f, Parent: fe145b7b89120e25, exportable:true]
I have below settings.
spring:
zipkin:
baseUrl: http://<IP_ADDRESS>:9411/
sleuth:
sampler:
percentage: 1.0
Please suggest.

This should work out of the box. Can you post your sample somewhere? Do you have some custom filters in your code? Can you also run this with debug level of sleuth logs and check the output?

Yes, i have defined one SimplePreFilter extending  ZuulFilter and below is the very basic implementation
@Override
public Object run() {
RequestContext ctx = RequestContext.getCurrentContext();
HttpServletRequest request = ctx.getRequest();
    log.info(String.format("#####Hello recevied %s request for %s #####", request.getMethod(), request.getRequestURL().toString()));
    log.info(String.format("#####For Zipkin::  Hello recevied %s request for %s #####", request.getMethod(), request.getRequestURL().toString()));

    return null;
  }

We already have a filter registered in Zuul. Can you try to play with registration order so that our filter gets executed before yours (pre) and after yours (post)

I have upload the code https://github.com/ajayaks/gateway-service.

Even on commenting custom filter i am getting the same.
2016-12-10 01:29:03.442  WARN [gateway-service,c3343e9ec85466b5,e98ddbec636826e3,true] 48890 --- [nio-1113-exec-1] o.s.cloud.sleuth.util.ExceptionUtils     : Tried to close span but it is not the current span: [Trace: c3343e9ec85466b5, Span: c3343e9ec85466b5, Parent: null, exportable:true].  You may have forgotten to close or detach [Trace: c3343e9ec85466b5, Span: e98ddbec636826e3, Parent: c3343e9ec85466b5, exportable:true]

Its working now.
Issues got fixed,  when i added added Camden.SR3 bom.
Its working with custom filters also.
Thanks for your support.

No problem :)


Bug 1449300 - Show Reps with Alumni Mentors in 'No mentor' dashboard

@akatsoulas this is waiting for your final review :) Thanks!

@akatsoulas r? (see my comment and update)

Fyi, travis is not very happy (flake is failing due to long lines).


Opcode Byte Display

I need to be able to show opcode and operand bytes in graph view as is possible in IDA Pro.
Only having access to the instruction mnemonic is not good enough...

Ahh, I like the easy bugs.
This is already fixed in the next release.

Would it be possible to ask for spaces between the bytes for readability?  Thanks!

Hmm, maybe. We're debating it internally. The problem is we already crop off x86 opcodes after 7 by default to keep the graph sizes manageable, but adding spaces would mean even less of the opcodes would be shown or the graphs would be even wider.
Is the goal to just better understand which was the opcode versus operands? Or just to help reading individual bytes in general?

Identifying opcode vs operands would be the purpose of this.
Intel ISR manual shows spaces between opcode bytes so it would be nice to follow their convention.  Maybe include a feature to suppress opcode printing?  IDA offers a max number to print (0+) in its config.


Artist unable to mint properly on RMRK2 (NFT not loading)

What happened?
The artist tried to mint his NFT on rmrk2. After minting it did not load. and he cannot find it in his profile: The webpage shows that he doesn't have that NFT. His account has been debited (indicating that the transaction was made), but there is nothing.
His NFT:

His Telegram: @skullz_xxx
Link to the NFT: https://kodadot.xyz/ksm/gallery/19449356-9692FA834AFCCE2D3E-BNG-GRL_1-00000001
Please reproduce in steps
https://github.com/kodadot/nft-gallery/assets/104380776/fb5dc396-7854-4a33-9e07-cf0e7a10db14
Expected Behavior
show minted NFt
What browsers are you seeing the problem on?
No response
At which address did you encounter bug?
No response
Are you logged in?
None
Which wallet you are using?
No response
Screenshots
added
Relevant log output
No response
Payment link for reward
https://kodadot.xyz/ksm/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

[X] I agree to follow this project's Code of Conduct

not sure why the video is not loading properly - I tried to reupload it but idk, I can send it to you personally

indexer returns null for this nft id

just minted my own collection + nft on RMRK2 => runs perfect:
nft on kodadot
mint collection tx
mint nft tx
list nft for sell tx

@JustLuuuu can you provide the address the artist minted with, so I can find him on subscan

@daiagi I'm waiting for the answer from the artist. To share his address.
Meanwhile, I have another artist who had a problem minting on rmrk2: her address GEn74YSY6scZRozMi1prDyMP6toAz6stgiFK1LWzGSZB4TC. She was able to create a collection and she was adding NFTs:
This is the link she sent me: https://kodadot.xyz/ksm/gallery/19462247-A20986FF921B82A06C-WHC-THE_ANGRY_DAD-00000001?congratsNft=The Angry Dad

Minting through Nova wallet. There was no indication that minting was not successful - only the fact that NFts did not load after

@JustLuuuu can you provide the address the artist minted with, so I can find him on subscan

14QRqymDrSbmxcBQnoPeWTfrwcoCkKCFYQfFZfyqNLmsfS4G

I'm waiting for the answer from the artist.

Yup looks like the whole signature was passed to system.remark instead of just RMRK::string

i closed in on it
it originates from this:
https://github.com/kodadot/nft-gallery/commit/42585bbb1efe287f35d75405cbeff5c7efaf3d3d
which is part of this pr
https://github.com/kodadot/nft-gallery/pull/6570
which was an attempt to fix https://github.com/kodadot/nft-gallery/issues/6508

Related #6916

@roiLeo after fix, will artists need to mint the NFTs again? 👀

after fix, will artists need to mint the NFTs again? 👀

From what I can see they are not present on our indexer, there's a good chance to mint it again, may @daiagi check if the NFT transaction was valid?

It wasn't valid. They willy need to mint again


Check for Valid Names in Controller

Check if name already exists (for duplicates) or if name contains special characters (escape if so) for names of Toolboxes (and toolbox categories), Workspace elements (contents and configurations), and BlockDefinitions. Checking should be done before adding a new resource element to the model.

Handled in New object pop-ups


Add grid showing sample ids on pooling

Can someone provide a more in-depth explanation of this issue?

@ElDeveloper, when the user has computed the values for pooling, this is what is shown to the user:

The goal is to have another grid above or below that shows the sample plate layout (i.e. the layout when the user goes to 'Plate samples'). This way they can match the pooling values with the samples. Does that make sense?

Thanks @josenavas, this helps a lot!

👍


The best approach for pseudo-elements?

What is the best approach for work with pseudo-elements like :before, :after, etc?
We need to create custom utilities/components?
Maybe should be a good idea to have a feature for this like: text-red:before?

Anything (almost?) could be applied to a pseudo-element. I don't think default config should include it though.
I could imagine it being a variant for the modules, which could be made with a plugin too if not in the core.

You could probably use a plugin to register a new variant to do that. This was introduced in v0.6.2 and it's probably the easiest way to deal with your issue since you'll be able to carry that plugin between projects and also publish it as an npm package.


Update GraphQL types and operations for box-transfer

To be reviewed in combination with the database ERD and the user action list linked in the Trello ticket below.
This is a first version, feel free to comment/review if anything is of major concern. I likely change things a little bit during implementation.

the types represent the hierarchy of the data layer: agreement -> shipment -> shipment detail
the agreement details are available as TransferAgreement.sourceBases and .targetBases
updateShipment can be used for three different actions
sendShipment and completeShipment serve as trigger for Shipment state transitions
accept/rejectTransferAgreement serve as trigger for TransferAgreement state transitions

 Draft user actions and GraphQL API for Box-Transfer

@aerinsol @vahidbazzaz I'd like to start the first implementation of mutations/queries related to box-transfer. Hence I'd like to merge this soon. Do you have any major objections?

@pylipp I think it was marked as draft for a while, which is why @vahidbazzaz and I probably ignored the reminders. Will see if I can get to it today.

@pylipp You are right, and I believe Roanna mentioned this during the box design meeting. This is also necessary for MVP, so it's up to @aerinsol to decide.

Yep.
On Wed, Dec 15, 2021, 11:17 PM Vahid Bazzaz @.***>
wrote:

@pylipp https://github.com/pylipp You are right, and I believe Roanna mentioned
this
https://docs.google.com/spreadsheets/d/1WfdnhRIV2OXbCnUYtqCe0DigdSJbReZPunZotlHvW9c/edit#gid=0&range=10:10
during the box design meeting. This is also necessary for MVP, so it's up
to @aerinsol https://github.com/aerinsol to decide.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/boxwise/boxtribute/pull/112#issuecomment-994891831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABP2BFCESAZ2PS6CXBFTWP3URCWRHANCNFSM5H2YBR2A
.


Apply basic optimizations

Hi Andrew, I found your package on godoc.org.
I've applied some optimizations that improve the efficiency of the Align function.

fmt is no longer imported.
bytes are used to build the result instead of strings.
slice-of-slice is not used; instead, a single slice with an index function is used.
building the results is done in reverse order, and then a reversal is done in-place afterwards.

With the benchmark I wrote, this results in a nice 3.5x speedup, and a very large reduction in allocations, from 268 allocs/op to 8 allocs/op. The amount of memory used is also reduced, mostly owing to using []byte instead of []int for the 'pointer' variable.
I've formatted the files I touched with gofmt, so the diff isn't all that great to look at. I left the commits un-squashed, so you can get a better idea of what is actually changing in the PR.
Cheers...

Hi Eric, awesome work. All the changes look good to me.
I tested this out on a few 100K sequences and gives a nice speedup.  Thanks so much!


Dragon Quest - Incorrect .exe

When making the .bat with the tool an incorrect name is populated.
cd %UserProfile%\ReshadeInjectUWP
cmd /c powershell -windowstyle hidden -Command Start-Process -filepath inject.exe Game.exe; Start-Process -filepath explorer.exe shell:appsFolder\39EA002F.DQXISWW_n746a19ndrrjg!Game

Correct .exe is DRAGON QUEST XI S.exe
As the name is composed with blank spaces it needs to be between double quote. Looks like it doesn't works.
Managed to be able to inject ReShade manually starting from the %UserProfile%\ReshadeInjectUWP by typing Inject.exe "DRAGON QUEST XI S.exe" and starting the game.

Hope it helps

Fixed! It would be awesome if you would tell me if it works.

Fixed! It would be awesome if you would tell me if it works.


Scintilla.Net <IP_ADDRESS> don't work when Devexpress libraries added to project

Hi!
Github issue from original branch
https://github.com/jacobslusser/ScintillaNET/issues/526
Step-by-step reproduction

Create blank WinForms app (Net 7)
Install Scintilla.Net <IP_ADDRESS> nuget package
Put Scintilla.Net control on form from toolbox
All works fine
Add Devexpress libraries to project (via open DevExpress Project Settings and click Add Libraries)
Try to open Form In Design mode
Scintilla control disappear.
Try to put Scintilla control on form again - we get the following error

Request failures: DesignerHosts/CreateComponent.
                       Microsoft.DotNet.DesignTools.Client.DesignToolsServerException: Method not found: 'Void System.Security.AccessControl.MutexAccessRule..ctor(System.Security.Principal.IdentityReference, System.Security.AccessControl.MutexRights, System.Security.AccessControl.AccessControlType)'.

                       For information on how to troubleshoot the designer refer to the guide at https://aka.ms/winforms/designer/troubleshooting.

Remove Devexpress package from project - all works fine now

Detailed description of problem from Devexpress
https://supportcenter.devexpress.com/ticket/details/t1133409/scintilla-net-5-3-1-3-don-t-work-when-devexpress-libraries-added-to-project
I understand that you do not use commercial libraries, but please see what you can do
Thanks!

Hi,
I can use the DevExpress evaluation to debug the issue - when I have the time.  I'm not sure if the Mutex creation is even required in the constructor of the control. It seems to relate to file use synchronization among processes, StackOverflow when extracting the Scintilla native DLL if one doesn't exist already.

Hi. the control should now work with the DevExpress library in design mode. The package was already targeting .NET 6

If you have time, add support for net 6 and net 7, please

.NET 7 TFM was added.


st2actionrunner graceful shutdown

This ticket will hold research into graceful shutdown of st2actionrunner.  This is in anticipation of adding a way through OS or otherwise to allow us to scale st2actionrunners based on some factor.
My initial research led me to this section of code where the st2actionrunner takes ownership of a scheduled action:
st2actionrunner takes ownership
The st2actionrunner abandon code is here:
st2actionrunner abandon code
The teardown for the parent process is here:
st2actionrunner teardown
We are probably going to create a custom heartbeat script that monitors the number of st2actionrunner processes on a vm to tell the autoscaler to wait until the work is done.
import boto3

response = client.record_lifecycle_action_heartbeat(
    LifecycleHookName='string',
    AutoScalingGroupName='string',
    LifecycleActionToken='string',
    InstanceId='string'
)

Another possiblity is for the autoscaler system to query if the st2actionrunner being shutdown has taken ownership of any jobs.  If so wait until it no longer has ownership.

What is an autoscaler in this context?

aws dynamic autoscaling policy

Do we need some kind of way to mark the specific st2actionrunner as "unschedulable"?
Otherwise, in a heavily used st2 dynamic environments it'll pick up the next task from the queue once the previous one is finished.
Talking about the mechanisms.
Maybe the SIGTERM signal to st2actionrunner process so it'll stop picking up new jobs?
Or do we need something more advanced, like a new API endpoint to drain the st2actionrunner?

It looks like a SIGTERM is all that is needed.    The only problem is AWS Dynamic Scaling will immediatly kill the VM unless you use the boto3.record_livecycle_action_heartbeat to tell AWS to wait while it is still shutting down the process.  I see this as a python script that would be supplemental and specific to AWS autoscaling.  I don't even think it should be part of core st2 codebase imo.

Yeah, right.
Higher level orchestrator/logic should give some time for st2actionrunner to finish its work after sending the signal.
In the context of K8s, when the pod is terminated it goes through the following lifecycle:

Pod is set to the “Terminating” State and removed from the endpoints list of all Services
A SIGTERM signal is sent to the main process in each container, and a “grace period” countdown starts.
Upon the receival of the SIGTERM, each container should start a graceful shutdown of the running application and exit.
Graceful shutdown period could be adjusted and configurable (up to a really long periods) to let the process (in our case st2actionrunner) finish its work.
If a container doesn’t terminate within the grace period, a SIGKILL signal will be sent and the container violently terminated.

More:

https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#pod-termination
https://cloud.google.com/blog/products/containers-kubernetes/kubernetes-best-practices-terminating-with-grace


Problem in installing

Hi there!
I just cloned the repository and tried pip install as suggested in the docs but it said You must give at least one requirement to install (see "pip help install"). I am on Linux.
Then I tried sudo python setup.py install. IMHO while manual installing, the second method is more favorable. Should we change the docs then?
Further I got some errors.
1.
byte-compiling build/bdist.linux-x86_64/egg/dit/utils/tests/test_bindargs3.py to test_bindargs3.pyc
  File "build/bdist.linux-x86_64/egg/dit/utils/tests/test_bindargs3.py", line 5
    def F1(a, b, c=2, *, d, e, f=5, **kwargs):
                       ^
SyntaxError: invalid syntax

File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 1425, in _get
    return self.loader.get_data(path)
zipimport.ZipImportError: bad local file header in /usr/local/lib/python2.7/dist-packages/dit-1.0.0.dev0-py2.7-linux-x86_64.egg

I checked out the dit/utils/tests/test_bindargs3.py code which had different arguments of the function F1 as reported here in build/bdist.linux-x86_64/egg/dit/utils/tests/test_bindargs3.py.
Even though I can successfully import dit in my python interpreter.

Thanks for the report. Will look into this.

Is it correct that you are using Python 2.7.x?

Regarding the SyntaxError. Unfortunately, this not easily fixed...but the error is harmless and you can ignore it.
The issue is that we have 2 versions of a file, one for 2.x and one for 3.x. When you install, Python tries to byte-compile both, even though the package will only use the correct one. It might be possible to fix this by modifying package_data in setup.py so that it only includes one of those Python files, but I think this would only fix setup.py bdist.  The source distribution (setup.py sdist) uses MANIFEST.in, and as far as I can tell, there is no way to conditionally include a file depending on the Python version (unless we manually generate our own MANIFEST file and forgo using MANIFEST.in).

So short of the missing ".", I think everything is good. I'll update the documentation to make it more clear to include the dot.

Yea. I didn't notice the dot there. Everything's good then.


[5.6] Sandbox blocks output to default plugin output directory when it's under <pkgdir>/.build

This is the 5.6 nomination of #4009.
Explanation:  The sandbox rules introduced in #3996 made the entire package directory read-only, but that isn't appropriate when .build is inside the package directory.  Swapping the order so that the sandbox applies the allow-write rules after the deny-write rules allows .build (or whatever the output directory is) to be written to while keeping the rest of the package directory readonly to the plugin.
Scope of Issue:  This affects SwiftPM users who run command plugins that write to the plugin working directory and who are not customizing the intermediate-files directory using the --build-path option.
Reason for Nominating to 5.6:  This is a common use case in new functionality (command plugins).
Risk:  Low — although the sandbox is used for manifest loading as well as running plugins, the ability to add custom writable locations is only used in the new functionality of command plugins.
Reviewed By:  @tomerd
Automated Testing:  A new unit test covers this case.
Dependencies:  None
Impact on CI:  None
How to Verify:  Use a command plugin that writes intermediate files to the plugin working directory (as opposed to /tmp) and check that it works even when the SwiftPM intermediates directory is inside the package directory.
rdar://87417780

@swift-ci please test


Does not work with nestjs

Testing hooks in a nestjs project. The hooks seems to be ignored by nestjs build even using the --tsc flag and specifying correct file path.
It works correctly using tsc -p tsconfig.build.json

Hi @samuelerodi,
Working on an update now to allow tsc-hooks to be used with frameworks. This took awhile to figure out what the best approach would be, since this in particular could have had some ugly over head, but I believe I have figured out a way to best handle adding the tsc-hooks injection script to any CLI.


Issue #1834 - Fix for High Severity CVE's in karate-core

Description

Relevant Issues : #1834
Relevant PRs : (optional)
Type of change :

[ ] New feature
[ ] Bug fix for existing feature
[x] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

Another high-severity CVE was found in armeria so I upgraded from 1.13.2 to 1.13.4.
I added dependency-check-maven to the root POM so that dependencies all modules will be checked for CVE's.  When doing a build, the check will run during the verify phase, and the aggregate report will be written to target/dependency-check-report.html.
The dependency check plugin has an option to fail the build if any high-severity CVE's are found.  I decided to turn that option off as I believe it will make the build too brittle.  This means that a manual review of target/dependency-check-report.html is required to see if any vulnerabilities are present.
The report currently shows high-severity CVS's in karate-gatling, karate-demo, and karate-robot.  While it would be nice to address those in the future, I believe we can leave them for now since those modules are not shaded.

Hm, it builds fine locally. I can see that the Dependency Checker is failing, but I'm not sure why...


Extensions

Superceeds #430.

This looks good right now.  I still would like to see it used in practice, to check that it's really the right API for this.

Just noticed/was told that this is not aimed at master.  +1, then

I'm assuming you don't want to merge this until we get #437 merged?

@gsilvis probably makes sense, yeah.

subsumed by #455; closing


Fixes error when sending django admin test gcm message.

Fixes display issue when sending test bulk message > max_recipients
Adds original_registration_id to results dict on GCMError
Addresses:
https://github.com/jleclanche/django-push-notifications/issues/354
&
https://github.com/jleclanche/django-push-notifications/issues/391

@jleclanche I think this is ready.


Error: app.canvas.requestFullscreen is not a function, when call fullscreen()

Description
It occurs when I use fullscreen() in some browsers like Safari, both on MacOS and iOS.
Seems the Fullscreen API in some browser engines should have prefix, such as webkit, moz, ms.
If you need, I could create a PR for it.
References:

Fullscreen_API
sindresorhus/screenfull.js

@KusStar please check if #234 soles your problem.
It solved for me under a Safari experimental build.
I would appreciate if you could also help me test in MS Edge and Firefox.

I run the code on stable Safari, it solved.
Also, I test it on stable FireFox and MS Edge, it works perfectly.
Thank you for your work.

@KusStar Did you get it working on stable Safari? webkitRequestFullscreen doesn't do anything to me on Safari 14.1 🤔

Sorry, I'm still using Catalina so my stable Safari is 13.1, it actually works.
But I test on Safari 14.1, it does't do anything.

I think the new Safari update broke the Fullscreen API compatibility. I will investigate a bit more.

Okay, I just confirmed.

It works on all Safari 13 variants.
Compatibility was broken on Safari 14.1+
Compatibility is also broken on Technology Preview 15+.

With that said, it works perfectly on Google Chrome.
Considering the poor support on iOS Safari, I don't think there's much we can do other than waiting for Apple to fix it.
I filed a bug report to the Safari Team.


[BUG]

Describe the bug
For some reason (probably unrealted to tensordict), functorch dims fails with big tensors.
import torch
from tensordict import TensorDict

# build splits
total = 0
l = []
while total < 6400:
    l.append(torch.randint(2, 10, (1,)).item())
    total += l[-1]

# build td
tensordict = TensorDict({"x": torch.randn(6401, 1), "y": torch.randn(6401, 1)}, [])

# split
for k, x in tensordict.items():
    x.split(l, 0)

This should result in an error like this:
libc++abi: terminating with uncaught exception of type c10::Error: allocated_ <= ARENA_MAX_SIZE INTERNAL ASSERT FAILED at "/stuff/pytorch/pytorch/pytorch/functorch/csrc/dim/arena.h":227, please report a bug to PyTorch.

I don't understand this yet, but can confirm that this appears to be unrelated to tensordict.
import torch

t = torch.rand(6401, 1)

# build splits
total = 0
l = []
while total < 6400:
    l.append(torch.randint(2, 10, (1,)).item())
    total += l[-1]

# could get RuntimeError if `sum(l) != 6401`
l[-1] = 6401 - sum(l[:-1])

# this is fine
_ = t.split(l, 0)

But now, if we import functorch.dim.Tensor it will fail
from functorch.dim import Tensor

_ = t.split(l, 0)

So something is happening at import with functorch that results in this bad behaviour. I'll try to pin it down a bit more but seems clear it's an issue upstream.

see this bug for ref


Fuzzy SaaS

Description

quick implementation of fuzzy saas from https://github.com/icssc/peterportal-api-next/pull/141

TODO

[ ] test

Screenshots

Steps to verify/test this change:

[ ] Verify changes work as expected on staging instance

Final Checks:

[ ] Verify successful deployment

(optional)

[ ] Write tests
[ ] Write documentation

Issues

Closes #

searching ics instead of i&c sci can lead to unexpected results
e.g.

top result for "art" is not an art class


How to stop logging and view error report before sending

Avoid duplicates

[X] Question is not already answered in the FAQ
https://www.coronawarn.app/en/faq/#error_log and
https://www.coronawarn.app/en/blog/2021-05-12-corona-warn-app-version-2-2/
do not answer the question.
[X] Affects Android and iOS
[X] Question has not already been asked in another issue

Your Question
The question is how to view exactly the data which the Error Report sends to technical support. There seems to be no way to stop error logging, view the data, then send it.
Also I could find no way, after the data has been sent, to view what data was sent. The only remnant on the mobile device seems to be the timestamp and the ID.
App version: 2.2.1
In App Information > Error Reports, the text (from string debug_log_privacy_card_container_section_two) says:
"Initially, only you will have access to the error report. You can then decide whether to send the error report to the technical support team, whether to save the error report on your smartphone first, or whether to stop and erase the recorded information. If you first save the error report locally, this will give you the opportunity to view the error report yourself before submitting it to support."

After tapping "START" there is no option to STOP and VIEW the data.

The "STOP AND DELETE" button does what it says, so there is no data to view.
The "SHARE AND CONTINUE" button opens up the Android Share option (by default with apps Drive, Bluetooth and Gmail available) however "Recording Active" is still shown, so it is still collecting data as can also be seen by the fact that the Current size continues to increase.
The "SEND ERROR REPORT" only provides the option to "AGREE AND SEND".

Internal Tracking ID: EXPOSUREAPP-7292

@dsarkar
I agree with your assessment. Let me know your recommendation about how to proceed now.

@MikeMcC399

There is no way to:  STOP -> (optionally) VIEW -> (optionally) SEND the error log
Consequently, there here is no way to see the FULL report before sending
In that case the describing txt is a bit mis-leading
it is not obvious/intuitive than one can view the ongoing error report via  SHARE AND CONTINUE button

Suggestion to suggest internally these options to fix this issue:

Adapt the text to the "reality" and maybe explain the SHARE button better and/or rename the SHARE button.
Or to change the work flow:

Introduce separate functions/buttons/menu: START, STOP, CONTINUE, DELETE error logging
Introduce separate functions/buttons/menu: SEND to technical support, SHARE/VIEW error report

@MikeMcC399 anything you would like to add/change? Thanks.

@dsarkar
Thanks for the summary! I will submit a separate issue with repro steps to keep it simpler, and I will link to this issue.

@dsarkar
I have created the issue "Cannot view exact error report before submitting" #614. I think the function STOP (without DELETE) is missing. Let's wait for dev feedback now. Your suggestions about modifying the workflow of course are also valid.
I also created the issue "Timestamp of shared error report uses UTC" https://github.com/corona-warn-app/cwa-app-android/issues/3183 which I noticed whilst I was capturing the information about the first issue.


Enable building on powerpc

These changes enable building on the powerpc family of targets - powerpc (32bit BE), powerpc64 (64bit BE) and powerpc64le (64bit LE).  Only tested on powerpc64le since that's the only target properly supported in Swift overall.

@swift-ci please test and merge


create export utility as a standalone command

the export command will export database content to data files

This is needed at end of next week 4/10


Fixed a crash in mgf scan number search

When searching for a scan number in an mgf file the search crashed at this location. I saw that it always panics when the index is not found while the function is allowed to return None. So I made it return None when the name could not be found. Additionally, I changed the later match statement into a map.

Thank you for fixing this. It points out a pattern I replicated all across those early item-retrieving methods.


AFFL_AccRecordType_TDTM queries unnecessary contacts

The method of AFFL_AccRecordType_TDTM queries any contacts whose primary affiliation field is not null for any of the accounts whose old record type had a primary affiliation field mapping. It should only query for contacts related to one of the accounts that have changed. This is a problem when, for example, a business organization account's record type is changed and there's more than 50,000 contacts that have a related business organization in the primary business organization field. Here's the section of code: https://github.com/SalesforceFoundation/HEDAP/blob/master/src/classes/AFFL_AccRecordType_TDTM.cls#L108-L112. I will submit a pull request.

Included in beta release 1.28 (Beta 8)

Included in production release rel/1.28


override authenticate_user! error response method

Hi. How can i override authenticate_user! error response method ?

Hi! What I did is to override the render_authenticate_error within the controller, I did it in my application controller.


Fix status bar height calculation for all cutout sizes

Summary:
Google has discouraged attempting to read the status_bar_height resource since 2017. With the introduction of display cutouts there can be a mismatch between the resource value and the true status bar size (and issues like this one popped up). The recommended approach is to instead call getInsets with the proper status bar and navigation flags provided by WindowInsets. On older APIs where getInsets is not supported, we have access to systemWindowInsetTop.
Changelog:
[Android][Fixed] - Fixed StatusBar.currentHeight calculations to honor all cutout sizes
Differential Revision: D57878119

See #44805


Does the software support multiple languages?

Does the software support multiple languages?
When the game language is set to Simplified Chinese and Traditional Chinese, the chat bar cannot display the correct content, and all Chinese characters are incorrectly displayed as? number

Error like this

I currently do not plan to support any languages besides English in the near future.


Improve info command

Improve info command
Description

This PR is to improve daytona info command in cli
Added support to display WorkspaceId, Target, RepositoryUrl

Fixes #114
/claim #114
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings

Related Issue(s)
This PR addresses issue #114
Screenshots

Thank you for the PR @ezhil56x
This looks good - just a couple of changes:
As you can see in the screenshot of a multi project workspace below, we should move the “repository url” to the projects section between each project name and state line. While you/re at it, also rename the wokrspaceURLStyle to projectURLStyle since the project is the one “connected” to a repository.
Last thing we could do is change the label WorkspaceId to just “ID” for less clutter.
Additionally, if you didn't already, please read through our contributing guidelines. Specifically, this section which requires you to add your username to the CONTRIBUTORS.md  file.

@ezhil56x By the way, here is an example of adding your entry to CONTRIBUTORS.md - a separate PR and specific commit message: https://github.com/daytonaio/daytona/commit/2f3e8b9ac3ce1bdf3f379ab3ad9f2951cd02fb41

@idagelic
Added to CONTRIBUTORS.md and made required changes. Can you review once?
Screenshot after changes

@idagelic Done 👍🏻

@idagelic @Tpuljak
Required changes are done


Delete properties properties list should not require user to press the Enter key

Describe the bug
Can enter a property to delete, but it doesn't take unless you also press Enter.
Version
Version:  2.0.0-nightly.245345.48d2167
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Add a Delete properties action to a dialog.
Edit the action's properties.
Error: notice that the property doesn't take unless you press Enter.

In most other actions, pressing Enter is not required. You only have to add text and then click outside the input box.
Even with this action. Once an input box is recognized as having a value, you can change that value and have it stick without having to press Enter first.

Expected behavior
The semantics of user input boxes is as consistent as possible throughout the product.
Screenshots
Additional context

The vertical ellipsis has hover text for the empty input box, but not for an input box that already has a value.

PR #https://github.com/microsoft/BotFramework-Composer/pull/6633 resolves this issue and will be included in the R14 release.


Finer grained access control for ControlAPI, Process task type

I've got a bit of a use case for which swarmkit is very close to being a very good fit. My primary goal is to schedule processes to run on members of a cluster and to have them in turn be able to cause other similar processes to run in the cluster (but with different arguments). swarmkit is already pretty close but lacked a couple of things I needed to make this work:

A task type able to describe the arguments to the processes I wanted to run and corresponding control APIs to manipulate them;
The ability for workers to be able to make certain control API calls (to allow them to act on requests from locally scheduled tasks to spawn other tasks by calling the new interfaces);

This PR adds both of those capabilities.
It's a bit unfortunate that I need to patch swarmkit here to add the ProcessSpec and related additions to the control API, is there some way we could make this more dynamic? For the latter I wondered about maybe passing in a list of gRPC server instances to be attached to the underlying server? For the former I'm not sure how much support PB has for opaque types or what other options we might have.
Access by workers to the control API was relatively easy to expose, but the access control/authorization took a little more thought. I ended up taking the approach of allowing the application to specify which control APIs a worker is allowed to access via subjectAltName entries on the worker's certificates, mapping onto unique roles given to each API RPC call, via the docker.protobuf.plugin.tls_authorization roles. I only actually marked up the two I was experimenting with to have the additional role, if this approach is deemed acceptable I should do the rest for consistency.
Perhaps I should split this into multiple PRs, but thought I would start with one for initial discussion.

ping @diogomonica

@ijc25 I really like the addition of the ProcessSpec. We discussed this in detail and I think this is a great general addition. There are some details we need to work out in how flexible it is (you have a note about argv[0], which we need to think about).
I am really unsure about providing control api access through the agent. I like the approach proposed here, as the manager is sort of "delegating" access to the agent to allow it to launch tasks. This needs more thought, as I think we can use it in a variety of use case, such as plugins to the manager that launch tasks of their own.
Do you mind if we examine this in further detail after the code freeze?

Do you mind if we examine this in further detail after the code freeze?

:+1: no problem. Thanks for the initial feedback.

The security model was designed exactly so workers have no privilege of anything other than their own resources.
I genuinely don't believe we should have the ability of a worker being able to schedule tasks that will run on other nodes in the cluster. This is a slippery slope, and I'm of the opinion that we should keep workers unprivileged.
Arguably, if you want a service to deploy other services, you can run it on the manager and give it swarm API access.

@diogomonica thanks for the feedback. I think one critical point here is that workers are not given any inherent privilege, any privileges an individual work is given are handed out explicitly by the managers (via their CA hat).
Given that then running another service on the managers and giving workers access to that service is not different in terms of level of security from what is allowed with this PR. But it is at the expense of having to setup parallel cluster-wide communication channels and RPC mechanisms instead of using an existing mechanism which is already present in the cluster (and debugged, with good and well understood security semantics etc).
FWIW I wouldn't expect the managers in docker swarm mode's use of swarmkit to ever want to give workers any additional privileges (my use case is outside of swarm mode), I'd be unsurprised to see you guys taking a hard line on using the facilities added here in that context.

@ijc25: Noticed this PR has been idle since November. Is it still something you want to pursue?

I've largely moved on to other things. Lets park it for now.


Fix the done button issue in WhatsNew

Description
LEARNER-2491
Fix the done button issue in WhatsNew

Codecov Report

Merging #1009 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1009      +/-   ##
==========================================
- Coverage   45.75%   45.74%   -0.01%
==========================================
  Files         407      407
  Lines       27071    27075       +4
==========================================
  Hits        12385    12385
- Misses      14686    14690       +4

Impacted Files
Coverage Δ

Source/WhatsNewViewController.swift
65.33% <0%> (-1.79%)
:arrow_down:

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 f15c27e...dad208a. Read the comment docs.

@saeedbashir I have done with improvements.


Sorting Morton indices does not scale for small problem sizes

The default bvh_driver parameters, OpenMP run.
OMP_NUM_THREADS=1
2.99e-01 sec 10.0% 98.6% 0.0% 96 ArborX:BVH:sort_morton_codes_and_init_leaves [region]
|-> 3.82e-02 sec 1.3% 100.0% 0.0% 96 Kokkos::Sort::BinCount [for]
|-> 1.20e-01 sec 4.0% 100.0% 0.0% 96 Kokkos::Sort::BinBinning [for]
|-> 5.39e-02 sec 1.8% 100.0% 0.0% 96 Kokkos::Sort::BinSort [for]

OMP_NUM_THREADS=2
4.26e-01 sec 14.2% 98.9% 0.0% 93 ArborX:BVH:sort_morton_codes_and_init_leaves [region]
|-> 1.13e-01 sec 3.8% 100.0% 0.0% 93 Kokkos::Sort::BinCount [for]
|-> 2.18e-01 sec 7.3% 100.0% 0.0% 93 Kokkos::Sort::BinBinning [for]
|-> 3.84e-02 sec 1.3% 100.0% 0.0% 93 Kokkos::Sort::BinSort [for]

Note: the number of calls slightly different (96 vs 93).
BinCount is an order of magnitude slower, BinBinning twice as slow.

How many primitives?

The default, so 50000.

It seems that Kokkos uses std::sort by default for OpenMP (see issue #1208 and continued discussion on kokkos/kokkos). There was a PR there to use __gnu_parallel::sort but it was reverted.
One thing we could try is to set the second parameter of sort() call to true, which would make it to always use Kokkos sort, but it's unclear how good it's going to be. Quick try did not compile due to some instantiations with unsigned, and something else. Will look into it.

I quickly tried the following patch:
+  Kokkos::View<size_t *, DeviceType> permute(
+      Kokkos::ViewAllocateWithoutInitializing("permute"), n);
+  iota(permute);
+
+  __gnu_parallel::sort(
+      permute.data(), permute.data() + n,
+      [&view](size_t const &a, size_t const &b) { return view(a) < view(b); });
+
+  // DO first touch here
+  Kokkos::View<int *, DeviceType> view_copy("view_copy", n);
+  Kokkos::deep_copy(view_copy, view);
+  Kokkos::parallel_for(
+      ARBORX_MARK_REGION("apply_permutation"),
+      Kokkos::RangePolicy<ExecutionSpace>(0, n),
+      KOKKOS_LAMBDA(int i) { view(i) = view_copy(permute(i)); });
+
+  return permute;
+

This results in slightly better scaling, and outperforms original for more than one thread. It also speeds up spatial and knn as they also use search. I think it is worth looking into it more.
m = n = 10000

== Original version ==

=== Total construction time ===
1: |-> 9.02e-01 sec 29.2% 94.8% 0.0% 0.8% 544 ArborX:BVH:construction [region]
2: |-> 8.33e-01 sec 30.5% 94.1% 0.0% 0.8% 489 ArborX:BVH:construction [region]
4: |-> 7.65e-01 sec 30.2% 90.8% 0.0% 1.3% 694 ArborX:BVH:construction [region]

=== Spatial queries times ===

=== Morton sorting time ===
1: |   |-> 2.71e-01 sec 8.8% 92.4% 0.0% 7.6% 544 ArborX:BVH:sort_morton_codes_and_init_leaves [region]
2: |   |-> 4.64e-01 sec 17.0% 95.1% 0.0% 4.9% 489 ArborX:BVH:sort_morton_codes_and_init_leaves [region]
4: |   |-> 4.57e-01 sec 18.0% 93.0% 0.0% 7.0% 694 ArborX:BVH:sort_morton_codes_and_init_leaves [region]

== __gnu_parallel ==

=== Total construction time ===
1: |-> 9.43e-01 sec 30.3% 65.3% 0.0% 0.7% 476 ArborX:BVH:construction [region]
2: |-> 7.54e-01 sec 28.5% 64.0% 0.0% 1.1% 580 ArborX:BVH:construction [region]
4: |-> 7.03e-01 sec 29.9% 61.0% 0.0% 1.8% 883 ArborX:BVH:construction [region]

=== Morton sorting time ===
1: |   |-> 3.83e-01 sec 12.3% 20.6% 0.0% 79.4% 476 ArborX:BVH:sort_morton_codes_and_init_leaves [region]
2: |   |-> 3.19e-01 sec 12.1% 24.3% 0.0% 75.7% 580 ArborX:BVH:sort_morton_codes_and_init_leaves [region]
4: |   |-> 3.06e-01 sec 13.0% 26.2% 0.0% 73.8% 883 ArborX:BVH:sort_morton_codes_and_init_leaves [region]

It still did not produce great results. I would like to try to run the patch on CADES to see if it improves our scaling.

This is part of the wider sorting challenge that have multiple duplicate issues. Closing this one in favor of #145.


[RFC] Refactor PowerVM code aligning to existing framework

This patch refactors existing PowerVM specific code implementation to be aligned with the other BMC types i.e. irrespective of the BMC type.
An LPAR is considered a SUT equivalent to a PowerNV machine.
This patch helps us achieve a common interface of accessing and using SUT (LPAR or PowerNV). Methods which are not applicable for LPARs are overridden wherever possible.
Fixes: #543
Signed-off-by: Harish<EMAIL_ADDRESS>

Sample RunHostTest run log: pvm_debug.txt

ping?

@oohal Sorry for the delayed response. I was out of office on multiple instances. Sending in the v2 sooner. Thanks.


Enable adding metadata for how to replicate each component

Currently we have at compile time a ComponentSyncMode. It's useful for it to be at compile time because the SyncMode is only used on client, but we don't want to replicate a SyncMode component through the network.
Still, it might be useful to have some metadata be computed at runtime, because it might be different depending on the entities?
In any case, some metadata that would be useful to have:

in which direction the component is replicated (InputMap should only be replicated from server to client, not client to server)
frequency of replication: Once (don't replicate updates, just insert/removes), Every time, and maybe can specify the frequency? (e.g. ActionMap should only be replicated once)
component sync mode: how do we copy the component from the client Confirmed entity to the Interpolated/Predicted entities

This is now supported with the addition of PerComponentReplicationMetadata


asdf

Type: Incomplete
Summary:
asdf
Details:
asdf
Resolution:
asdf
Affected definitions:

lsif-npm 0.5.3

This was meant to be opened on staging, apologies!


Update Title of App

changed 'Address App' to 'HR Manager'
removed 'view' command in UserGuide and Developer Guide
edited phrasing in UserGuide

Codecov Report

Merging #45 (223110f) into master (695cb4e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #45   +/-   ##
=========================================
  Coverage     72.07%   72.07%
  Complexity      416      416
=========================================
  Files            73       73
  Lines          1289     1289
  Branches        132      132
=========================================
  Hits            929      929
  Misses          324      324
  Partials         36       36

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 695cb4e...223110f. Read the comment docs.


Add support for source git-repos where the template is located in a subdirectory

Actual Situation
We use a monorepo branching strategy where the root directory is shared and kept in common among multiple branches.
In this strategy, there is only one solution per branch and it's added into a distinct subdirectory.
Desired Situation
I'd like to maintain Copier templates, one per branch, but can't seem to identify a way to indicate the subdirectory that each template should be found in. If this isn't already possible, could it be added?
Many thanks for the project!
Proposed solution
It brings to mind the ?dir=<subdir-path> git-url suffix used by Kustomize to specify a file or directory path within a git repo.
Something like that would be a nice addition to the project and elegantly remove this seemingly arbitrary template-placement restriction.

This is answered in the docs: https://copier.readthedocs.io/en/latest/configuring/#subdirectory

Can I have multiple templates in a single repo using this option?
The Copier recommendation is: 1 template = 1 Git repository.
[...]

Looks like a duplicate of #315 #1078 #1279. See especially https://github.com/copier-org/copier/issues/315#issuecomment-746007580.
I'm closing this issue, but feel free to continue the discussion.

Hiya @sisp I want to clarify my use-case is 1 Template = 1 Git repo
Our branching strategy discourages making changes to the root directory, so I'd like to add the template to a sub-directory.

@nickgarber And having only copier.yml in the root directory is no option? The template content can be located in any subdirectory using the _subdirectory setting.

Right, I think I'll eat the frog and pursue that approach.  Thanks for your guidance!

Hi there! Thanks for this proposal. The feature has been asked many times as you can see. However, none of the solutions had in mind important features such as updating and migrations. The main problem was that git tags are repo-global, so by using branches copier would lose those features.
The monorepo approach you suggest is the 1st proposal that could actually work. By changing the src url directly, we get a canonical path where to search for updates. Since all can be done still in the same branch, we can still rely on tags.
Problems still present with that approach:

it's still not so simple
you could get updates where nothing changes.

FWIW this could still happen depending on your answers on any normal project.

Benefits:

You could have different copier.yml files, each one with different settings
We could support an alternative repo layout, giving our users more freedom of choice without compromising other features.

Due to my limited time availability lately, I will be honest: I will not develop this. However, if @copier-org/maintainers  agree, we can reopen and leave it as proposed for community contribution.
Although it probably requires more gymnastics depending on your case, you can already get it with the workaround suggested. So, do you still think it's worth it @nickgarber? Would you be willing to push a PR?

Appreciate it! I've taken some time to consider and would be interested to create a draft implementation.
I think monorepo users the world over would thank us.
Is there still a warm feeling towards reviewing a proposal in support of per source branch encoded templates?
If so, is something in line with how nix flakes handles this seen as a suitable approach?

ref: https://nix.dev/manual/nix/2.18/command-ref/new-cli/nix3-flake#url-like-syntax

No, copier considers a repo a template.
This is because copier uses tags to compute migrations, and tags are repo-wide, not branch-specific. Thus, we can't rely on branches for the auto-updating.
Yes, you can use branches, as long as you always specify -r branch_name when running copier.


Python Beginner

I am very new to python but have some programming experience...
I was experimenting with you bank_managment_system python app.  When I looked at the code, I found some functions were defined twice and I could not understand why you would need to define a function twice, with the same name and all the same code.
Thank you!

Maybe number of arguments are different for method overloading.

Maybe one was function definition and one was function call statement. Can you please send the link to the code?


typeName & targetField can't read dots

sourceField can read dots which is convenient but placing the downloaded URL in a tidy location inside the graphQL tree could be better.
Scenario 1: Read typeName with dots
{
      use: '@noxify/gridsome-plugin-remote-image',
      options: {
        'typeName': 'portfolio.photo.thumbmails.large', // does not work
        'sourceField': 'url',
        'targetField': 'downloadedUrl',
        'targetPath': './src/assets/remoteImages'
      }
},

Scenario 2: Write targetField with dots
{
      use: '@noxify/gridsome-plugin-remote-image',
      options: {
        'typeName': 'portfolio',
        'sourceField': 'photo.thumbnails.large.url', // works!
        'targetField': 'photo.thumbnails.large.downloadedUrl', // does not work
        'targetPath': './src/assets/remoteImages'
      }
},

Hi,
Thanks for reporting.
Currently only the sourceField supports dot-walking.


Turbo does not cancel pending requests that were prefetched on consecutive link clicks

In a situation when page loads or internet connection are slow, clicking multiple links within a short period of time should only load the last link clicked.
When prefetch is disabled, Turbo works correctly by canceling pending requests issued by previous clicks.
When prefetch is enabled however, all requests issued by clicks will run to completion. The page will re-render on each completed request resulting in flickering. And the last link clicked is not guaranteed to be the one visible when all requests complete.
In the following example, there are 3 links that take 2s,6s, and 4s to load. Clicking them consecutively in that order will result in the slowest page being opened last, even though it was a second click, not the last one.

Screen recording of example
https://github.com/hotwired/turbo/assets/12209038/285444cb-d201-4d60-bf50-f64bb6f28e8d

Example with Rails
require "bundler/inline"

gemfile(true) do
  source "https://rubygems.org"

  gem "rails"
  gem "puma"
  gem "propshaft"
  gem "turbo-rails"
end

require "action_controller/railtie"

class App < Rails::Application
  config.load_defaults "7.1"

  config.consider_all_requests_local = true
  config.turbo.draw_routes = false

  routes.append do
    root to: "application#index"
  end
end

$template = DATA.read

class ApplicationController < ActionController::Base
  include Rails.application.routes.url_helpers

  def index
    sleep(params[:sleep].to_i) if params[:sleep].present?
    render inline: $template, formats: :html
  end
end

App.initialize!

Rack::Server.new(app: App, Port: 3000).start

__END__

page <%= (params[:page].present? ? params[:page] : 'root').upcase %>

    <%= link_to "root", root_path %>

without prefetch

<%= link_to "slow", root_path(page: 'slow', sleep: 2) %>

<%= link_to "slowest", root_path(page: 'slowest', sleep: 6) %>

<%= link_to "slower", root_path(page: 'slower', sleep: 4) %>

with prefetch

<%= link_to "slow", root_path(page: 'slow', sleep: 2) %>

<%= link_to "slowest", root_path(page: 'slowest', sleep: 6) %>

<%= link_to "slower", root_path(page: 'slower', sleep: 4) %>

This is even more noticeable when adding view transitions.

Has anyone found a solution to this issue? It's still happening with latest version of turbo.


Climo script

@durack1 @gleckler1 can we merge this now? I teak the test suite and except for graphics it passes, @jservonnat will like this!

@doutriaux1 looks like travis is now failing again, what have you done?

@doutriaux1 looks like there is a CMake issue on linux: https://travis-ci.org/PCMDI/pcmdi_metrics/jobs/108624719#L4160-L4165

@doutriaux1 I'm reluctant to merge until the travis builds are passing fine.. Do you know what is going on here?

@doutriaux1 are we holding you up here?


Add a License

👋, I was wondering if you planned on adding a license to your project?

Hey, i have added a License

Awesome! Thanks!


Add theme groups for Dashboard

Add named groups for dashboard.nvim
https://github.com/nvimdev/dashboard-nvim
Useful for a lot of folks using nvim these days.

OK. I'll remove this PR and send a new one with the changes done that way.


support some kind of caching w/ the "Merkle backing" for Merkleized types

relevant data to consider:

https://github.com/ralexstokes/mev-rs/pull/27

partial implementation deleted in the one commit of #74
could be a good starting place to pick this back up when perf becomes an issue

see #6 for an initial direction here

some current thoughts:

consider putting the cache behind an immutable wrapper, in which case HashTreeRoot::hash_tree_root could take an immutable reference as well -- this has implications for all the consumer crates of this one so it would imply quite a substantial code refactor down the line...

a "lightweight" version of caching here just caches the root of the tree and marks any leaf elements that are "dirtied". a more substantial version would cache the entire Merkle tree alongside the data structure to make the hash tree computation even more efficient in terms of compute. an even more sophisticated option would support "journaling" writes so it becomes easy to undo mutations.


Picture folder location

I use the docker versio of pigallery
I want to change the picture folder to a external location, to accomplish this i created a bind volume which directs to a external loaction as a path (/data/pictures).
In the container the path exists and i can get to it in the container console.
I can not edit the folder location in the UI, is keeps turning back to the default location. editiing in de config.json file does not do te trick either.
What am I doing wrong?
Jhod

Hi,
With docker tmp, images and these settings are fixed:
https://github.com/bpatrik/pigallery2/blob/b68d5d174d8f485e766865892816710a4794b836/docker/debian-stretch/Dockerfile.build#L21-L24
They should not be changed within the container, rather you should point your volumes to the right folder on your host.
Edit docker-compose.yml here (change ./pigallery2/images to your images folder):
https://github.com/bpatrik/pigallery2/blob/b68d5d174d8f485e766865892816710a4794b836/docker/docker-compose/default/docker-compose.yml#L22-L25
Or set up the path correctly if you are running directly:
  -v <path to your config file folder>/config.json:/app/data/config/config.json \
   -v <path to your db file folder>:/app/data/db \
   -v <path to your images folder>:/app/data/images \
   -v <path to your temp folder>:/app/data/tmp \

If you really want to override these settings within docker, you can try setting them as environmental variables :
 environment:
 - Server-Media-folder=<your path>

thanks for the quick reply, I tried the ENV variant but in capitals ;-(
Using them in normal letters did the trick!
Now experimenting with Pigallery, so far is looks very nice en exactly what I searched for!
Jhod

I'm happy to hear!


added tests for healthcheck

Notes for Reviewers
This PR fixes #3988
PS: more tests for others files are coming in different PR
Signed commits

[x] Yes, I signed my commits.

@Freedisch
Let's discuss this on Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

@afzal442 , It should be alright now, there are some non-tested functions left I'll open another PR for them.

@Revolyssup

@afzal442 thanks for this review! 👏

@afzal442 The remaining functions are using c *meshkitkube.Client  as arguments which makes them hardly testable right now


Add cool animations wherever possible

Currently, the front-end of this web app lacks good animations. Let's see what can be improved here!

I can try but it'll be a bit later on.

I can give this a go. Mind assigning this to me?

Sure go ahead.
On Wed, Oct 23, 2019 at 7:49 PM, Conner Luzier<EMAIL_ADDRESS>wrote:

I can give this a go. Mind assigning this to me?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/schedutron/home-server/issues/27?email_source=notifications&email_token=AFOA42HG4WHVDOYYSEY3GRLQQBMQBA5CNFSM4EBJMOTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECBSWCQ#issuecomment-545467146,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOA42BC4Z7HC7ZPOK5U443QQBMQBANCNFSM4EBJMOTA
.

--
Saurabh Chaturvedi
pre-final year,communication and computer engineering
The LNMIIT, Jaipur
[image: Twitter] https://htmlsig.com/t/000001D9J0JH  [image: Facebook]
https://htmlsig.com/t/000001D8QZD0  [image: LinkedIn]
https://htmlsig.com/t/000001D70VYD  [image: Github]
https://htmlsig.com/t/000001DA29NG  [image: Stack Overflow]
https://htmlsig.com/t/000001D3NZ8Q

heyy, can I take this issue? I'm pretty sure I can add some cool animations to your website

@Capt-Titanium Sounds cool. Go ahead.


Check consistency with VBS Server

To check if we handle submissions in the same way as the VBS Server does, we should have a helper utility in place which sends the same input to both systems.

Superseded by #208


Push/pull errors improvement and cleanup

Several improvements to error handling:

Introduce ImageConfigPullError type, wrapping errors related to
downloading the image configuration blob in schema2. This allows for a
more descriptive error message to be seen by the end user.

Change some logrus.Debugf calls that display errors to logrus.Errorf.
Add log lines in the push/pull fallback cases to make sure the errors
leading to the fallback are shown.

Move error-related types and functions which are only used by the
distribution package out of the registry package.

cc @docker/distribution-maintainers

Job: Docker-PRs-WoW-TP4 FAILED:

---
many errors
+ ec=1
+ set +x

----------------------------------
ERROR: Failed to build test binary
----------------------------------

-----------------------------------------------
ERROR: Failed with exitcode 1 at Thu Feb 11 22:45:51 CUT 2016.
-----------------------------------------------

INFO: Tidying up at end of run
INFO: Nuking /d/CI
INFO: Zapped successfully
INFO: End of cleanup
INFO: Ended at Thu Feb 11 22:45:52 CUT 2016 (1m 29s)
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe C:\Users\jenkins\AppData\Local\Temp\hudson4792218102522574197.sh
+ set +e
+ set +x
INFO: End of cleanup
Notifying endpoint 'HTTP:https://leeroy.dockerproject.org/notification/jenkins'
---

Job: Docker-PRs-userns FAILED:

---
ined: fallbackError
distribution/pull_v1.go:58: undefined: fallbackError
distribution/pull_v1.go:64: undefined: fallbackError
distribution/pull_v2.go:63: undefined: fallbackError
distribution/pull_v2.go:67: undefined: fallbackError
distribution/pull_v2.go:70: undefined: continueOnError
distribution/pull_v2.go:72: undefined: fallbackError
distribution/pull_v2.go:108: undefined: fallbackError
distribution/pull_v2.go:108: too many errors
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Docker-PRs-userns] $ /bin/sh -xe /tmp/hudson6931043139778391987.sh
+ set +x

---

Job: Docker-PRs-Win2Lin FAILED:

---
ution\pull_v2.go:67: undefined: fallbackError
distribution\pull_v2.go:70: undefined: continueOnError
distribution\pull_v2.go:72: undefined: fallbackError
distribution\pull_v2.go:108: undefined: fallbackError
distribution\pull_v2.go:108: too many errors
Build step 'Execute shell' marked build as failure
Performing Post build task...
Could not match :Build timed out  : False
Logical operation result is FALSE
Skipping script  : cd c:\scripts
sh timeout-post-build.sh

END OF POST BUILD TASK 	: 0
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe C:\Users\Jenkins\AppData\Local\Temp\hudson7794810406949715488.sh
+ set +x

---

Job: Docker-PRs FAILED:

---
ined: fallbackError
distribution/pull_v1.go:58: undefined: fallbackError
distribution/pull_v1.go:64: undefined: fallbackError
distribution/pull_v2.go:63: undefined: fallbackError
distribution/pull_v2.go:67: undefined: fallbackError
distribution/pull_v2.go:70: undefined: continueOnError
distribution/pull_v2.go:72: undefined: fallbackError
distribution/pull_v2.go:108: undefined: fallbackError
distribution/pull_v2.go:108: too many errors
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Docker-PRs] $ /bin/sh -xe /tmp/hudson2666168850042828200.sh
+ set +x

---

Job: Docker-PRs-experimental FAILED:

---
ined: fallbackError
distribution/pull_v1.go:58: undefined: fallbackError
distribution/pull_v1.go:64: undefined: fallbackError
distribution/pull_v2.go:63: undefined: fallbackError
distribution/pull_v2.go:67: undefined: fallbackError
distribution/pull_v2.go:70: undefined: continueOnError
distribution/pull_v2.go:72: undefined: fallbackError
distribution/pull_v2.go:108: undefined: fallbackError
distribution/pull_v2.go:108: too many errors
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Docker-PRs-experimental] $ /bin/sh -xe /tmp/hudson4950611112597394122.sh
+ set +x

---

Job: Docker-PRs-gccgo FAILED:

---
defined name 'retryOnError'
    return retryOnError(err)
           ^
distribution/push_v2.go:339:10: error: reference to undefined name 'retryOnError'
   return retryOnError(err)
          ^
distribution/push_v2.go:364:10: error: reference to undefined name 'retryOnError'
   return retryOnError(err)
          ^
distribution/push_v2.go:369:10: error: reference to undefined name 'retryOnError'
   return retryOnError(err)
          ^
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Docker-PRs-gccgo] $ /bin/sh -xe /tmp/hudson5092131334012648757.sh
+ set +x

---

LGTM

Job: Docker-PRs-userns FAILED:

---
test.go:772: DockerSuite.TestRunEnvironment	0.460s
PASS: docker_cli_run_test.go:811: DockerSuite.TestRunEnvironmentErase	0.434s
PASS: docker_cli_run_test.go:846: DockerSuite.TestRunEnvironmentOverride	0.496s
PASS: docker_cli_run_test.go:642: DockerSuite.TestRunExitCode	0.449s
PASS: docker_cli_run_test.go:76: DockerSuite.TestRunExitCodeOne	0.498s
PASS: docker_cli_run_test.go:71: DockerSuite.TestRunExitCodeZero	0.657s

----------------------------------------------------------------------
FAIL: docker_cli_run_test.go:1735: DockerSuite.TestRunExitOnStdinClose

docker_cli_run_test.go:1783:
    c.Fatal("docker run failed to exit on stdin close")
... Error: docker run failed to exit on stdin close

----------------------------------------------------------------------
PASS: docker_cli_run_test.go:2316: DockerSuite.TestRunExposePort	0.038s
PASS: docker_cli_run_test.go:907: DockerSuite.TestRunFullHostnameSet	0.466s
PASS: docker_cli_run_test.go:1034: DockerSuite.TestRunGroupAdd	0.477s
PASS: docker_cli_run_test.go:3971: DockerSuite.TestRunInitLa
---

LGTM

Job: Docker-PRs-Win2Lin FAILED:

---
tar -cC 'bundles/1.11.0-dev/test-integration-cli/frozen-images' . | docker load
++++ tar -cC bundles/1.11.0-dev/test-integration-cli/frozen-images .
++++ docker load
Error response from daemon: Untar re-exec error: exit status 1: output: unexpected EOF
Build step 'Execute shell' marked build as failure
Performing Post build task...
Could not match :Build timed out  : False
Logical operation result is FALSE
Skipping script  : cd c:\scripts
sh timeout-post-build.sh

END OF POST BUILD TASK 	: 0
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe C:\Users\Jenkins\AppData\Local\Temp\hudson5649795326224312922.sh
+ set +x

---

Job: Docker-PRs-WoW-TP4 FAILED:

---
	      56	docker-1.11.0-dev.exe.md5
  0%
100%
	    New File  		      88	docker-1.11.0-dev.exe.sha256
  0%
100%
	    New File  		  27.1 m	docker.exe
  0.0%
  3.6%
  7.3%
 11.0%
 14.7%
 18.3%
 22.0%
 25.7%
 29.4%
 33.0%
 36.7%
 40.4%
 44.1%
 47.7%
 51.4%
 55.1%
 58.8%
 62.5%
 66.1%
 69.8%
 73.5%
 77.2%
 80.8%
 84.5%
 88.2%
 91.9%
 95.5%
 99.2%
100%

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

               Total    Copied   Skipped  Mismatch    FAILED    Extras
    Dirs :         1         0         1         0         0         0
   Files :         4         4         0         0         0         0
   Bytes :   54.39 m   54.39 m         0         0         0         0
   Times :   0:00:00   0:00:00                       0:00:00   0:00:00

   Speed :<PHONE_NUMBER> Bytes/sec.
   Speed :            60440.618 MegaBytes/min.
   Ended : Friday, February 12, 2016 12:32:58 AM

+ ec=0
+ set +x
INFO: Linking the built binary to /d/CI/CI-8f26
---

LGTM 🐲


WIP: Adaptive Sampling Integration

Signed-off-by: Ashmita Bohara<EMAIL_ADDRESS>Which problem is this PR solving?
Wire up the adaptive sampling component from collector/main.
Short description of the changes
This PR is to know if I am working in the right direction. There is still lot of work to do. Please let me know.
Question:

Whether we need to have defaultStrategies for adaptive sampling ? If not, we don't need to have adaptive/constants.go.

Thank you for the feedbacks Yuri. I will work on them.


adjust docs for feature maturity

as per the docs this is no longer an alpha feature, but stable.
Either that or I don't understand it well as a k8s noobie and it should be clarified 😄

Deploy preview is actually good.
/approve


xrdp: init at 0.9.2

Motivation for this change
RDP server (remote desktop which is good for high latency connections).
nix package + nixos module + test script
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

When closures were deploying by nixops, rsakeys.ini was generated once, before the deployment and then was deployed to many machines.
This has a bit of sense of being insecure, so generating rsakeys.ini postponed until systemd service start.

The test is not added into the NixOS list of tests to run by default/for channel, but I assume it is intentional

I am just not familiar with running tests on Hydra.
Perhaps it could make sense to add the test to the list, I do not know.


[iOS] double copying root app

On iOS right after call #sync method plugin perform copying root app. And right after downloading of zip file it's again perform copying root app.
Here my log:  http://pastebin.com/bcJJKETg

@abonec thanks for reporting this


Node doesn't start

It gets stuck here:
bin/start
[info] Chain is loading.
[info] Checkpoints are enabled.
[info] Coin cache is enabled.
[info] Content server listening on port 9301
[info] Opening ChainDB...
[info] Chain is loading.
[info] Checkpoints are enabled.
[info] Coin cache is enabled.

It would be helpful to include some info on how to recreate this bug. What OS are you using? Are you running natively or in Docker?

I am using a virtualbox VM with Ubuntu 16.04. Native inside of the VM.
I haven't used this in a while. After a "git pull" it is working.

Don't forget to mark as closed then!


How to reduce the margin of pages

I want to narrow the sidebar and reduce the margin of pages. What change can I make?

Hi @yichx,
Can you describe how you achieved this?
Thanks

I also want to know how to reduce the right margin. Thx.

@CaucherWang Please see #1996, if all goes well I'll have something written up in the next couple days.


How to visualize the results?

Thanks for your wonderful work. I would like to know how to visualize the results after validation or test? @LiyaoTang

Hi,
Thanks for your interest. But could you be more specific? For example, what would you want to visualize, and what type of visualization do you want to see?
Best,
Liyao

Hi,
As indicated by the error messages, you are trying to use uninitialized values, which indicates that the checkpoint that you specify to load is different from the model that you specify to use. Please double-check if you are loading the correct checkpoint.
Best,
Liyao

Thanks for the advice, I've fixed the problem. Back on topic, I would like to ask for advice on how to visualize the results of a validation or test. Because currently I can only get quantitative results like the below figure but not qualitative results. @LiyaoTang

Hi,
Good to know that. Yes, we do not include the visualization code in the release, because it is coupled with several other projects and is hard to clean. However, we mostly adapt from the official RandLA-Net repo for visualization tools, so you could check out there if you want to do visualization.
Best,
Liyao

I found some functions related to visualization in the utils/tester.py you provided. However I didn't find any place to call them. So can I just call them directly in main.py to visualize? @LiyaoTang

I found some functions related to visualization in the utils/tester.py you provided. However I didn't find any place to call them. So can I just call them directly in main.py to visualize? @LiyaoTang

I look forward to you answering my simple question at your convenience. Thank you very much! @LiyaoTang

Hi,
I do not think there is any visualization I wrote in the utils/tester.py, though there might be some leftover codes from KPConv / Closerlook3D.
But yes, you can certainly leverage those codes for your visualization purposes.
Best.


Feature: Adds new CLI parameter entrypoint

Implements suggestion from this issue: https://github.com/olzhasar/pytest-watcher/issues/11
The commits are rather dirty, please let me know if I should squash them.
I only tested locally with Python 3.10 with the following parameters (using pip install -e .)
ptw .
ptw . --now
ptw . --now --entrypoint tox
ptw . --entrypoint tox

@olzhasar I've made the requested changes and dropped support for py36 in tox.ini, please double check
I've tested with these
 ptw .
 ptw . --now
 ptw . --runner tox
 ptw . --now --runner tox -v

@paolorechia
I've made some minor changes for clarity
Overall, looks good to me!
Feel free to merge it
Thanks for the contribution!

@olzhasar Thanks again for reviewing! Happy to contribute.
I don't think I have permission to merge this PR, would you mind doing it?
I also forgot to ask before, but I guess we should bump the version and release a new package.


Provide prebuilt Docker images for ARM

The Docker images that are currently build in GitHub Actions are built only for x86_64. When someone runs one of these on a different architecture, such as ARM, some kind of binary translation or runtime emulation is performed to get the image running.
It would be better if instead of relying on emulation, Nosey Parker provided multi-architecture Docker images.

It's possible to build multi-platform Docker images even on one system (Docker will fall back to QEMU). But this introduces its own complications, and is not a trivial switch.
Some reference materials:

https://docs.docker.com/build/ci/github-actions/test-before-push/
https://www.docker.com/blog/multi-platform-docker-builds/
https://docs.docker.com/build/building/multi-platform/

I have experimented with building a linux/arm64 image on an x86_64 machine, and the final image seems to work, but it takes a couple hours to build under QEMU.

You could also add a cron workflow and build multi-platform at night if it takes too long for your taste.

Thanks for the report, @vogler. Indeed, the Docker image is not being tagged as intended: the latest tag is supposed to point to the latest stable release, and edge points to the latest build from the main branch.
The multiplatform images are there for releases, but the tags are not right. I will fix this.

@vogler regarding the multi-hour Docker builds, yes, that's what I have observed. I think most of the time goes to building a few big native library dependencies under qemu. I will take a look at your reference! I'd love for it to be fast enough to build the multiarch Docker image on every commit.

I think most of the time goes to building a few big native library dependencies under qemu.

Could try to split building these in separate layers to have them cached if they don't change often.

@vogler There are 2 different GitHub Actions workflows that generate Docker images for Nosey Parker:

https://github.com/praetorian-inc/noseyparker/blob/main/.github/workflows/docker-image.yml
https://github.com/praetorian-inc/noseyparker/blob/main/.github/workflows/release-artifacts.yml

The second one builds the multi-platform image. The first one was supposed to build just the image with the edge tag for x86_64, but it happened to also be tagging the images it built with latest and with release tags, hence clobbering the multiplatform images.
I have fixed this issue of bungled tags, so it should not be an issue going forward. I will also see if I can manually fix the release tag for the older releases (v0.14.0, v0.13.0), so they refer to the intended multi-platform images.

I think most of the time goes to building a few big native library dependencies under qemu.

Could try to split building these in separate layers to have them cached if they don't change often.

Yes, this is a perennial problem, keeping build speeds up!
Many months ago there was additional caching for rust dependencies when building Docker images in Nosey Parker's GH actions. But that was challenging to maintain (obnoxious technical reasons involving Cargo, GH Actions caching, and Docker), and after restructuring the Nosey Parker code to be split into several internal crates, it became too much to keep up with. There is certainly room for build-time improvements in Nosey Parker.

@vogler I have updated the v0.14.0 and latest tags for the Docker image. They should now correctly point at the intended multi-platform image that supports both ARM64 and AMD64:
https://github.com/praetorian-inc/noseyparker/pkgs/container/noseyparker/119700654?tag=v0.14.0
https://github.com/praetorian-inc/noseyparker/pkgs/container/noseyparker/119700654?tag=latest
Please let me know if you have further trouble with this. Thanks again for the report 😎


Drives don't appear anymore no matter what, rendering the program useless

Description
Drives have stopped appearing at the sidebar and root.
No idea why or how they stopped showing, they just don't appear anymore.
Steps To Reproduce

Open Files
Check sidebar

Toggling the "show drives" option doesn't do anything.
Expected behavior
Drives appear
Files Version
<IP_ADDRESS>
Windows Version
10.0.19043.1288
Relevant Assets

I'm having the same issue on the same Windows & app version.
I'm uploading the two files you've requested, but they don't look like they contain anything useful.
debug.log
debug_fulltrust.log


Django Component Governance vulnerability

Django 1.11 before 1.11.28, 2.2 before 2.2.10, and 3.0 before 3.0.3 allows SQL Injection if untrusted data is used as a StringAgg delimiter (e.g., in Django applications that offer downloads of data as a series of rows with a user-specified column delimiter). By passing a suitably crafted delimiter to a contrib.postgres.aggregates.StringAgg instance, it was possible to break escaping and inject malicious SQL.
https://dev.azure.com/FuseLabs/SDK_v4/_componentGovernance/112465/alert/2370216?typeId=4354877

Thanks @BruceHaley for creating an issue to track. <3


Remove deprecation warning in node-sass 4.0

There are a lot of such warnings:
DEPRECATION WARNING: Passing a string to call() is deprecated and will be illegal
in Sass 4.0. Use call(get-function("output-float")) instead.

This PR is going to fix them 💯

To use just a get-function we need to have at least sass 3.5.0. So I've added backward capability which support sass 3.3.x and up.

We are experiences the same problem in our project. Is there any change to get the fix in soon?

It seems project is not maintained.
What I do:

install patch-package: npm i patch-package
create a folder patches in root of your project and put https://gist.github.com/ApacheEx/3a0296391c14b5103e3bad461e55b316 as patches/singularitygs+1.8.0.patch.
run npm i

or just create a fork and apply this patch/PR.

Thnx @ApacheEx works like a charm!


feat: add validation params to generated freezed classes as static consts

Only the last commit is the change, other commits shown here are from the previous already merged PR.

@Carapacik Possible to get this merged? Test failed after #269. Everything is good with this branch.