Implement v1.2 local hash calculation, pending JSON LD normalization library

We need the ability to perform JSON LD normalization in order to perform the v1.2 local hash. This doesn't appear to be available for Flask yet.

This is mentioned in the #developers channel, but we've hit a problem the JSON-LD normalization. Our existing v1.2 certs aren't valid JSON-LD because of a design constraint where @type and type are treated as the same key.
https://github.com/json-ld/json-ld.org/issues/402
As such, the javascript implementation of the JSON-LD toolset complains that v1.2 certificates have colliding keywords.

Found the root issue. Our format currently adds an extra "@type" key to the receipt block. That's a Chainpoint v2 receipt, and their json-ld schema follows the emerging standard of aliasing "type" to "@type". Since we're trying to use that standard, the problem isn't with the JSON-LD parser or the schema, but instead with how we're creating the certificates. Indeed, v1.2 certificates we've created so far will (correctly) fail JSON-LD parsing.
On the plus side, the WKWebKit bridge I'm using to run jsonld.js totally works as a stop-gap measure until we invest time in an iOS native JSON-LD library. It's been incorporated into the validation mechanism and tested with a v1.2 certificate that has been modified to pass JSON-LD parsing. Unfortunately, that's not a fully conclusive test, since it (correctly) fails validation comparing local to signed hashes.
I'm going to leave this issue open until I can manually verify a published certificate.


CARDS-1708: The Patient chart shows only 10 out of 10+ child subjects

Quick fix: Increased the limit of the query to 1000, the same upper bound we have when showing forms on the patient chart.

Question: Why my tumor regions are sorted alphabetically before the change, and not sorted after the change?


⚠ TV Manager has degraded performance

In 417389e, TV Manager ($SITES_URL_TV_MANAGER) experienced degraded performance:

HTTP code: 200
Response time: 1904 ms

Resolved: TV Manager performance has improved in 47d7e9b after 12 minutes.


⚠ Bofrost.at has degraded performance

In ed5d354, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1931 ms

Resolved: Bofrost.at performance has improved in c354683 after 27 minutes.


revert to bare price for backward compatibility and add -timestamped_price file

A suggestion to support oracle feeds, without disrupting price widgets in the current Bitkit versions

@rbndg @dzdidi


Get-ACL doesn't support -Credential?

How come Get-ACL doesn't have -Credential support?

Your question will probably get more attention if you explain why Get-Acl should have that parameter.
Run Get-Acl from an elevated session to avoid problems stemming from lack of permissions.

Here is my thing. My regular account doesn't have credentials to get into
even the root folder holding our home network drives.
If I had -credential I could prompt or pass my file system admin or domain
admin credential object to it and get in their content
On May 13, 2017 12:55 PM, "Michael Klement"<EMAIL_ADDRESS>wrote:
Your question will probably get more attention if you explain why Get-Acl
should have that parameter.
Run Get-Acl from an elevated session to avoid problems stemming from lack
of permissions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/issues/3777#issuecomment-301260436,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADi_zDjkM74GWLH6tzdbQdtyNEPm1mW9ks5r5eCJgaJpZM4NZsWw
.

To generalize your requirement: you want an on-demand method for invoking a given command with elevation, similar to what sudo does in the Unix world.
That would indeed be helpful, and is already being discussed in #3232.
Unless this anticipated generic mechanism still wouldn't meet your needs, I suggest you close this issue.

I don't see why you wouldn't want to have -Credential on it so you can call
it as another user?
Like how you can say 'Get-Aduser username -Credential $DAcred'
I also have fields hidden from my regular, daily driver user that are
visible from my admin account. Like employee number and Administrator
password are visible to my server admin account and above, but not that
lower account.
Same type of thing, just filesystem right now instead of AD.
On May 13, 2017 7:04 PM, "Michael Klement"<EMAIL_ADDRESS>wrote:

To generalize your requirement: you want an on-demand method for
invoking a given command with elevation, similar to what sudo does in
the Unix world.
That would indeed be helpful, and is already being discussed in #3232
https://github.com/PowerShell/PowerShell/issues/3232.
Unless this anticipated generic mechanism still wouldn't meet your
needs, I suggest you close this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/issues/3777#issuecomment-301279991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADi_zFRbxE6PzCGL0UpWJ6qdVToIcCHSks5r5jbegaJpZM4NZsWw
.

I just realized that far more cmdlets that I was aware of in fact do have a -Credential parameter:
$VerbosePreference = 'Continue'
Get-Command -type cmdlet | ? { $c = $_; try { $_.Parameters.ContainsKey('Credential') } catch { write-verbose "Failed with: $c" } }

Still:

It's not readily obvious to me which cmdlets were given this parameter vs. which were not.

It seems to me that solving this problem at the level of individual cmdlets is the wrong approach.

That said, I'll leave it to someone more knowledgeable to take it from here.

Can't use the Invoke-Command to elevate ?
Example,  Invoke-Command -ComputerName . -ScriptBlock {get-acl} -Credential (Get-Credential)

I'll agree and bump this a little. Unlike #2761 and #2975 (similar -Credential requests), there doesn't seem to be any restriction that prevents a -Credential parameter from being implemented. I only took a quick look through the .NET docs on ACL classes, so I very well could be wrong on that.
While Invoke-Command would work, it would force a weird usage in my opinion. Everything I've used so far supports -Credential, then all of a sudden for working with ACL's I'd have to shove the ~10 lines of logic into a scriptblock and either choose to invoke it against localhost or the remote computer that holds the file shares (since Invoke-Command requires a -ComputerName when -Credential is used). That's not a fantastic option since it additionally requires remoting to be available when Get-ACL only requires Share\NTFS permissions regardless of local vs remote file locations.

I just ran into this with 'set-acl' and my script that runs otherwise unpriviliged/lower privileged needs to elevate to provision control over an OU that I create with some automation. how would I solve this?

The Get-Acl cmdlet uses reflection to invoke a method to get acl
PSMethodInfo methodInfo = _inputObject.Methods["GetSecurityDescriptor"];

So you should previously create the input object with elevated permissions.

makes things hugely difficult. i'm running a script that's delivered by aws systems manager agent. this runs as
'nt authority\system'
so my script passes around a credentials object. Except when it gets here. Now i'm in a rabbithole of trying to run the function using

start-job (so I can pass cred) - works when i use my own login on a test machine, fails with access denied when the system user calls this
Attempting 'invoke-command' seems to actually run the command as the user I want, however doing
if ([adsi]::Exists("LDAP://" + $OUPath))
fails with

Exception calling "Exists" with "1" argument(s): "An operations error occurred.
"
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : DirectoryServicesCOMException
    + PSComputerName        : localhost

so when you say 'you should previously create the input object with elevated permissions'. I agree with you. but HOW?
@iSazonov

You could use [adsi]::new(string path, string username, string password) to make elevated binding.

I solved my issue which was quite specific.
I found that to invoke a command while running as system on a windows system, you need to enable credssp and enable the server component of Credssp aswell as the client portion so the system can pass credentials to itself. Then you can invoke command with a service Account that has AD access.
# enable the box to accept credentials
Enable-WSManCredSSP -role server
# enable the box to trust itself
Enable-WSManCredSSP -Role "Client" -DelegateComputer "$env:computername.$env:userdnsdomain"

you can also force the command so it's possible in a script.
Credssp has some security implications and warnings about security, so for my scope since i'm running a system via SSM and I trust myself, I enable credssp at the start, do my work and disable again at the end. Hope this helps someone one day, it cost me 2 days of confusion.

Bump up this.
I ran also in this issue and also missing -server Argument when working with two different AD Domains and try to query the current ACLs for Objects ( for example Groups or OUs).

Hi @dogfish182 I am having the same issue trying to run SSM docuement on a windows system that suppose to give permission to an account on the computer OU
It is failing to run due to SYSTEM , I tried your solution and it is not working , might be I am missing something ?
Import-Module AWSPowerShell
Enable-WSManCredSSP -role server -force
Enable-WSManCredSSP -Role "Client" -DelegateComputer "$env:computername.$env:userdnsdomain" -force
$Domains = 'test.local'
$user = GET-SECSecretValue -SecretId arn:somearn -Select SecretString | ConvertFrom-Json | Select -ExpandProperty adminDomainUsername
$pass = GET-SECSecretValue -SecretId arn:somearn -Select SecretString | ConvertFrom-Json | Select -ExpandProperty adminDomainPassword | ConvertTo-SecureString -asPlainText -Force
$DomainUser = "$Domains$user"
$Cred = New-Object System.Management.Automation.PSCredential($DomainUser,$pass)
$Script= {
Import-Module ActiveDirectory
$AccountName = 'demo'
$Domain = Get-ADDomain -Identity test.local -ErrorAction Stop
$BaseDn = $Domain.DistinguishedName
$ComputersContainer = $Domain.ComputersContainer
$SchemaNamingContext = Get-ADRootDSE | Select-Object -ExpandProperty 'schemaNamingContext'
[System.GUID]$ServicePrincipalNameGuid = (Get-ADObject -SearchBase $SchemaNamingContext -Filter { lDAPDisplayName -eq 'Computer' } -Properties 'schemaIDGUID').schemaIDGUID
$AccountProperties = Get-ADUser -Identity $AccountName
$AccountSid = New-Object -TypeName 'System.Security.Principal.SecurityIdentifier' $AccountProperties.SID.Value
$ObjectAcl = Get-ACL -Path "AD:$ComputersContainer"
$AddAccessRule = New-Object -TypeName 'System.DirectoryServices.ActiveDirectoryAccessRule' $AccountSid, 'CreateChild', 'Allow', $ServicePrincipalNameGUID, 'All'
$ObjectAcl.AddAccessRule($AddAccessRule)
Set-ACL -AclObject $ObjectAcl -Path "AD:$ComputersContainer"
}
Invoke-Command -Computer localhost -ScriptBlock $Script -Credential $Cred
Write-Output "Content of variable Script"
Write-Host $Script

Alto, I was trying to do the same thing.  Slightly different script.
Import-Module ActiveDirectory
New-ADOrganizationalUnit -Name $newOU -Path "OU=example,DC=example,DC=com" -Description $ServiceUserName -PassThru -Credential $credential
New-ADUser -Name $ServiceUserName -Accountpassword (ConvertTo-SecureString "Password1!" -AsPlainText -force) -Enabled $true -Credential $credential
$OrganizationalUnit = "OU=" + $newOU + ",OU=example,DC=example,DC=com"
Set-Location AD:
$Group = Get-ADuser -Identity $ServiceUserName
$GroupSID = [System.Security.Principal.SecurityIdentifier] $Group.SID
$ACL = Get-Acl -Path $OrganizationalUnit
$Identity = [System.Security.Principal.IdentityReference] $GroupSID
$Computers = [GUID]"bf967a86-0de6-11d0-a285-00aa003049e2"
$ResetPassword = [GUID]"00299570-246d-11d0-a768-00aa006e0529"
$ValidatedDNSHostName = [GUID]"72e39547-7b18-11d1-adef-00c04fd8d5cd"
$ValidatedSPN = [GUID]"f3a64788-5306-11d1-a9c5-0000f80367c1"
$AccountRestrictions = [GUID]"4c164200-20c0-11d0-a768-00aa006e0529"
$RuleCreateAndDeleteComputer = New-Object System.DirectoryServices.ActiveDirectoryAccessRule($Identity, "CreateChild, DeleteChild", "Allow", $Computers, "All")
$RuleResetPassword = New-Object System.DirectoryServices.ActiveDirectoryAccessRule ($Identity, "ExtendedRight", "Allow", $ResetPassword, "Descendents", $Computers)
$RuleValidatedDNSHostName = New-Object System.DirectoryServices.ActiveDirectoryAccessRule ($GroupSID, "Self", "Allow", $ValidatedDNSHostName, "Descendents", $Computers)
$RuleValidatedSPN = New-Object System.DirectoryServices.ActiveDirectoryAccessRule ($GroupSID, "Self", "Allow", $ValidatedSPN, "Descendents", $Computers)
$RuleAccountRestrictions = New-Object System.DirectoryServices.ActiveDirectoryAccessRule ($Identity, "ReadProperty, WriteProperty", "Allow", $AccountRestrictions, "Descendents", $Computers)
$ACL.AddAccessRule($RuleCreateAndDeleteComputer)
$ACL.AddAccessRule($RuleResetPassword)
$ACL.AddAccessRule($RuleValidatedDNSHostName)
$ACL.AddAccessRule($RuleValidatedSPN)
$ACL.AddAccessRule($RuleAccountRestrictions)
Set-Acl -Path $OrganizationalUnit -AclObject $ACL
Did you ever get yours to work?

Yes.
add the following before the script to enable credssp:
Enable-WSManCredSSP -role server -force
Enable-WSManCredSSP -Role "Client" -DelegateComputer "$env:computername.domain.com" -force
  New-Item -Path HKLM:\SOFTWARE\Policies\Microsoft\Windows\CredentialsDelegation -Name AllowFreshCredentialsWhenNTLMOnly -Force
  New-ItemProperty -Path HKLM:\SOFTWARE\Policies\Microsoft\Windows\CredentialsDelegation\AllowFreshCredentialsWhenNTLMOnly -Name 1 -Value "wsman/$env:computername.domain.com" -PropertyType String

and add this at the end to the invoke command:  -Authentication Credssp
Like:
Invoke-Command -Computer localhost -ScriptBlock $Script -Credential $Cred -Authentication Credssp


ConnectionResetError()

What might be the possible reasons for this error? I'm attempting to catch the exception at the end of the 'send_notification' method.

I realized that it was because I'd set use_alternate_port to true.


Fix domain persistence in docker_compose_domains

Related to #4326
Update domain persistence in docker_compose_domains field after redeployment.

app/Livewire/Project/Application/General.php

Add validation for docker_compose_domains in the rules and validationAttributes arrays.

app/Models/ApplicationPreview.php

Add validateDockerComposeDomainsFormat method to validate the format of docker_compose_domains.

app/Http/Controllers/Webhook/Bitbucket.php

Add validateDockerComposeDomainsFormat method to validate the format of docker_compose_domains.

app/Http/Controllers/Webhook/Gitea.php

Add validateDockerComposeDomainsFormat method to validate the format of docker_compose_domains.

app/Http/Controllers/Webhook/Github.php

Add docker_compose_domains to the data array when creating a new ApplicationDeployment.

/claim #4326

Is this AI generated slop?

Not needed by the author anymore, so I will close it.

@Bilge I hope not. I just put up #4326 and this guy managed to pick it up (bounty was attached) and create this pull request. What made you think this was AI generated? Is the solution not right?

Even a cursory glance would tell you that creating private methods that are never called makes no sense, notwithstanding that same code was copy/pasted three times and this guy only showed up to collect the bounty (has no prior contributions to this project).


Updated artwork?

So, it turns out the old xpm artwork is rather low quality. Have you considered newer images? I notice that transparent PNG files work just fine for the TARDIS and the headshot of the Doctor. For example, here are the images I ended up using on my machine for -tardis, -head, and -marquee:

That looks great!
I should mention that I am not Sean Brennan, the writer of timetunnel, so I'm not the owner of zettix.com. Therefore, I'm not sure what exactly you are asking for.

Oh, I know you're not Sean Brennan. I was imagining that your script would improve the original.
1a) The 'timetunnel-logos' package could include the new artwork directly.
or,
1b) if there are legalistic issues, it can detect if those files are missing and download them from the web, the same as it does now for Sean's xpm images.

'timetunnel-logos' could add multiple lines to the .xscreensaverrc so it'd have multiple different versions available.

[But, you're probably right, I should also talk to Sean. I'd like to see him update his timetunnel so each time through it gives you the head of a different Doctor. (Ideally, with the appropriate TARDIS that matches).]


Support

Hi Kiran I have an issue with this library

I  have two arrays in a dictionary one array with different dynamic kind of data for color selection size selection and all. And another array is a tree structured one.
Currently what I'm doing is one cell using for the dynamic items and there we load a collection view and expansion of the header loading the content of the Collection cells with dynamic data.
And in the second where I'm using the KJTree and implemented the tree structure and for both case using delegate methods to handle the parent view controllers height. Now somewhat it's working with some known bugs. Can You please come up with a nice suggestion to achieve this scenario?
How can I add check box for finally selected child in KJ Tree

HI, Kiran is there any way to identify that I have clicked exactly on the final child of the table view ?

@sarathdev you need to assign the key value for final child means last child so on didselect method you will get that value


Add e-Manifest

Insert URLs to the project here: https://github.com/USEPA/e-manifest

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as Good First Issue of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.
All new projects listed will be posted on our Mastodon channel.

@scottdchristian @dpgraham4401 @wiljnichepa


Query always provides answers regardless of whether document context is found.

Using Ollama LL (phi3) and chat mode set to "Query". Query will provide answers only if document context is found.
Using ChromaDB with Ollama embeddings (nomic-embed-text)
Asking a question that is obviously outside the document context (i.e. What is a prime number) rather than getting the specified query mode refusal response, I get an answer to my question with citations that point to document context that are clearly incorrect.
So, this is two issues. 1. I should be getting a query mode refusal and 2. the citations given have no relationship to the answer provided.

FWIW, get exactly the same behavior using System default LLM with Ollama embeddings.

You should likely modify the Document Similarity Threshold since that is likely why you are getting results for a search even though the prompt is irrelevant. What is the score reporting for those chunks when you view the citations? The score is below each chunk

I've deleted all of that and going back to basics. Just maiting for llama2 to download and I will give it a try again using "System Default" LLM with Anything LLM embeddings and Lance DB. I have Max Context set to 10 and Document Similarity Threshold set to High for this run. Will post the update when I have run it with this setup.

I did get it to work properly with all the defaults (LLM, Embeddings and VectorDB) but I had to reload my data (from the git repo) because deleting the vectors wasn't enough.
If I may make a suggestion, it would be nice to be able to enter a similarity threshold in numeric form as well as have the low, medium and high options.

@Freffles I agree, there should be some custom option. The reason that we have those "pre-defined" stops is because it prevents people from playing with it too much since it's a very black-box kind of toggle. It can be useful but I was initially worried about people accidentally foot-gunning themselves due to bad configs

Should have added this but before. Regarding score, I found that things that should have been a direct hit were returning scores less than 60. I could not get anything returned when I set to high. Maybe I need to play around with the chunk size.


re-enabled the watcher in Bakery.

re-enabled the watcher in Bakery.

Codecov Report

Merging #1146 (8320612) into master (44a135d) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1146      +/-   ##
==========================================
- Coverage   81.48%   81.44%   -0.05%
==========================================
  Files         121      121
  Lines        3953     3955       +2
  Branches      119      122       +3
==========================================
  Hits         3221     3221
- Misses        732      734       +2

Impacted Files
Coverage Δ

...e/src/main/scala/com/ing/bakery/baker/Bakery.scala
0.00% <0.00%> (ø)

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 44a135d...8320612. Read the comment docs.


⚠ Bofrost.at has degraded performance

In cf78f60, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1229 ms

Resolved: Bofrost.at performance has improved in 3cfc58f after 7 minutes.


[FIX] Do not apply XML new style checks to 14.0

This will about most of the failing updates to 14.0 dotfiles, since the current pre-commit is running new checks not verified on the previous pre-commit version.
Example where this was verified and got a green build:
https://github.com/OCA/product-attribute/pull/1440
cc @etobella @sbidoul @moylop260

Looks good to me.


Strict spacing for maps?

Hey, I really like what you are cooking here! Currently looking into koto to use as a live-coding language for a new framework.
Any particular reason the syntax is so strict about no-space before colons in maps, like error on {a : 10} in favor of {a: 10}? I feel like this is not aligned well with the otherwise forgiving and simple nature of the syntax.

Hey @unlessgames 👋 Glad you like the look of it! Live-coding is one of the main reasons I made Koto, please let me know how you get on.
I think there isn't really a good reason for the syntax to be strict here, especially with inline maps where parsing is generally flexible.
It should be a relatively simple change. peek_token / consume_token can become peek_next_token_on_same_line / consume_next_token_on_same_line in parser.rs), go ahead and make a PR if you like, or I can get to it later on.

Neat, thanks! I submitted the PR.

Thanks!


Convert DateTime to seconds

When using a DateTime or Carbon object in the later method:
ErrorException: Object of class Carbon\Carbon could not be converted to int in...
Sorry for the noise. I messed up my PR ;)

Would you like me to remove the return null;, that's making StyleCI grumpy?

@mortenhauberg StyleCI will make his job himself. ;)


Running ray with pytorch lightning in slurm job causes falure with error "ValueError: signal only works in main thread"

🐛 Bug

I followed the instructions at https://docs.ray.io/en/master/tune/tutorials/tune-pytorch-lightning.html  to integrate ray with pytorch lightning. However, when I submitted a slurm job to run the tuning I get the following error:
ValueError: signal only works in main thread
I submitted the same to ray project at ray/issues/10995 and I was suggested a hack to fix the issue.
Could we look for a way to disable the SLURM detection in pytorch lightning itself so that external parties do not have to hack its way around it?
To Reproduce
Steps to reproduce the behavior:

Set up ray by following instructions at here.
Submit a slurm job to run the tuning
See error

ray.tune.error.TuneError: Trial raised an exception. Traceback:
ray::ImplicitFunc.train() (pid=4432, ip=<IP_ADDRESS>)
  File "/home/user/.local/lib/python3.7/site-packages/ray/tune/function_runner.py", line 227, in run
    self._entrypoint()
  File "/home/user/.local/lib/python3.7/site-packages/ray/tune/function_runner.py", line 290, in entrypoint
    self._status_reporter.get_checkpoint())
  File "/home/user/.local/lib/python3.7/site-packages/ray/tune/function_runner.py", line 497, in _trainable_func
    output = train_func(config)
  File "tune.py", line 261, in train_run
    trainer.fit(model)
  File "/home/user/.local/lib/python3.7/site-packages/pytorch_lightning/trainer/states.py", line 48, in wrapped_fn
    result = fn(self, *args, **kwargs)
  File "/home/user/.local/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 1073, in fit
    results = self.accelerator_backend.train(model)
  File "/home/user/.local/lib/python3.7/site-packages/pytorch_lightning/accelerators/gpu_backend.py", line 51, in train
    results = self.trainer.run_pretrain_routine(model)
  File "/home/user/.local/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 1184, in run_pretrain_routine
    self.register_slurm_signal_handlers()
  File "/home/user/.local/lib/python3.7/site-packages/pytorch_lightning/trainer/training_io.py", line 240, in register_slurm_signal_handlers
    signal.signal(signal.SIGUSR1, self.sig_handler)
  File "/usr/local/easybuild-2019/easybuild/software/mpi/gcc/8.3.0/openmpi/3.1.4/python/3.7.4/lib/python3.7/signal.py", line 47, in signal
    handler = _signal.signal(_enum_to_int(signalnum), _enum_to_int(handler))
ValueError: signal only works in main thread

Code sample
slurm script
#!/bin/bash

#SBATCH --gres=gpu:4
#SBATCH --nodes=1
#SBATCH --ntasks=1

# load necessary modules #
module purge
module load scikit-learn/0.21.3-python-3.7.4
module load python/3.7.4

python -u tune.py &> "tune_output.txt"

tune.py
class CustomDataSet(Dataset):
    def __init__(self, csv_file, img_dir, transform):
        self.data = pd.read_csv(csv_file)
        self.img_dir = img_dir
        self.transform = transform

    def __len__(self):
        return len(self.data)

    def __getitem__(self, idx):
        img_loc = join(self.img_dir, self.data.name[idx])
        image = Image.open(img_loc).convert("RGB")
        tensor_image = self.transform(image)
        label = self.data.label[idx]

        return tensor_image, label

class ExperimentAE(pl.LightningModule):
    def __init__(self,
                 params: dict,
                 **kwargs) -> None:
        super(ExperimentAE, self).__init__()

        self.params = params
        self.model = SegNet()

    def forward(self, z):
        return self.decoder(z)

    def _run_step(self, x):
        x_hat, z = self.model(x)
        return x_hat, z

    def generate(self, x):
        return self._run_step(x)[0]

    def step(self, batch, batch_idx):
        x, y = batch
        self.curr_device = x.device
        x_hat, z = self._run_step(x)
        loss = F.mse_loss(x_hat, x, reduction='mean')
        return {"loss": loss}

    def training_step(self, batch, batch_idx):
        train_loss = self.step(batch, batch_idx)
        logs = {"ptl/train_loss": train_loss}
        return {"loss": train_loss, "log": logs}

    def validation_step(self, batch, batch_idx):
        val_loss = self.step(batch, batch_idx)
        return {"val_loss": val_loss["loss"]}

    def validation_end(self, outputs):
        avg_loss = torch.stack([x['loss'] for x in outputs]).mean()
        tensorboard_logs = {'val_loss': avg_loss}
        return {'val_loss': avg_loss, 'log': tensorboard_logs}

    def configure_optimizers(self):
        optims = []

        optimizer = optim.Adam(self.model.parameters(),
                               lr=self.params['lr'],
                               # weight_decay=self.params['weight_decay']
                               )
       optims.append(optimizer)
       return optims

    def train_dataloader(self):
        transform = self.data_transforms(train=True)
        img_dir = "/data/brca/"
        train_csv = "/original/train.csv"

        dataset = CustomDataSet(train_csv, img_dir, transform=transform)
        loader = DataLoader(dataset, shuffle=True, batch_size=self.params['batch_size'], num_workers=4)

        self.num_train_imgs = dataset.__len__()

        return loader

    def val_dataloader(self):
        transform = self.data_transforms(train=False)

        img_dir = "/data/brca/"
        val_csv = "/original/validation.csv"

        val_dataset = CustomDataSet(val_csv, img_dir, transform=transform)
        self.valid_dataloader = DataLoader(val_dataset, shuffle=False, batch_size=self.params['batch_size'],
                                           num_workers=4)

        self.num_val_imgs = self.valid_dataloader.__len__()

        return self.valid_dataloader

    def test_dataloader(self):
        pass

    def data_transforms(self, train=True):
        if train:
            transform_train = transforms.Compose([
                transforms.ToTensor(),
            ])
            return transform_train

        else:
            transform_val = transforms.Compose([
                transforms.ToTensor(),
            ])
            return transform_val

def train_run(config_params, num_epochs=10, num_gpus=1):
    model = ExperimentAE(params=config_params)

    trainer = Trainer(
        max_epochs=num_epochs,
        gpus=num_gpus,
        logger=TensorBoardLogger(
            save_dir=tune.get_trial_dir(), name="", version="."),
        progress_bar_refresh_rate=0,
        callbacks=[
            TuneReportCallback(
                {
                    "loss": "val_loss",
                },
                on="validation_end")
        ])

    trainer.fit(model)

def tune_run(num_samples=20, num_epochs=10, gpus_per_trial=1):
    tune_config = {
        "lr": tune.loguniform(1e-4, 1e-5, 1e-3),
        "batch_size": tune.choice([2, 4, 8]),
        # 'weight_decay': 0.0,
        'scheduler_gamma': tune.choice([1, 0.95, 0.9, 0.85, 0.6]),
    }

    scheduler = ASHAScheduler(
        metric="loss",
        mode="min",
        max_t=10,
        grace_period=1,
        reduction_factor=2)

    reporter = CLIReporter(
        parameter_columns=["lr", "batch_size"],
        metric_columns=["loss", "training_iteration"]
    )
    tune.run(
        partial(
            train_run,
            num_epochs=num_epochs,
            num_gpus=gpus_per_trial
        ),
        resources_per_trial={
            "cpu": 1,
            "gpu": gpus_per_trial
        },
        config=tune_config,
        num_samples=num_samples,
        scheduler=scheduler,
        progress_reporter=reporter,
        name="tune_segnet_v1"
    )

if __name__ == "__main__":
    tune_run(num_samples=20, num_epochs=10, gpus_per_trial=1)

Expected behavior
The ray tune program to run properly in a slurm environment.
Environment

CUDA:

GPU:

Tesla V100-SXM2-16GB
Tesla V100-SXM2-16GB

available:         True
version:           10.2

Packages:

numpy:             1.17.3
pyTorch_debug:     False
pyTorch_version:   1.6.0
pytorch-lightning: 0.9.0
tqdm:              4.46.0

ray:              0.8.7
tensorflow:              2.1.0

System:

OS:                Linux
architecture:

64bit
ELF

processor:         x86_64
python:            3.7.4
version:           #1 SMP Tue May 26 15:05:43 EDT 2020

Additional context

hey @Rashindrie! Would you mind upgrading to 1.0.2 to see if the issue persists?

Hi @edenlightning
Sure, will try that.
Give me some time to get back to you. Currently overloaded by some other tasks.

Closing this for now, feel free to reopen!

@edenlightning I have the same behaviour. The 'hack' fixes it.
I am running a Python script on a SLURM cluster.
Environment
CUDA:
    GPU:
        GeForce RTX 2080 TI
        GeForce RTX 2080 TI
    available: True
    version: 10.2
Packages:
    numpy: 1.19.2
    pyTorch_debug: False
    pyTorch_version: 1.7.1
    pytorch-lightning: 1.1.3
    tqdm: 4.55.1
        ray: 1.1.0
System:
    OS: Linux
    architecture:
        64bit
        ELF
    processor: x86_64
    python: 3.7.9

@edenlightning I have the same behaviour. The 'hack' fixes it.
I am running a Python script on a SLURM cluster.
Environment
CUDA:
    GPU:
        GeForce RTX 2080 TI
        GeForce RTX 2080 TI
    available: True
    version: 10.2
Packages:
    numpy: 1.19.2
    pyTorch_debug: False
    pyTorch_version: 1.7.1
    pytorch-lightning: 1.1.3
    tqdm: 4.55.1
        ray: 1.1.0
System:
    OS: Linux
    architecture:
        64bit
        ELF
    processor: x86_64
    python: 3.7.9

Hi, I'm having this issue as well. I don't like that you have to basically circumvent the normal functionality of the code in order to get it to work...

This is still an issue in 1.2.1

Update: calling ray.init() causes this error even with the "hack", effectively making distributed computing impossible. You can only use ray tune on a single node

Can confirm setting os.environ['SLURM_JOB_NAME'] = 'bash' works well as suggested in the issue on the Ray repo (and can be seen here). Could be worth noting in the Slurm Cluster part of the documentation.

Does it work correctly when you’re running a distributed job across multiple nodes?

I've never been able to get Ray properly working on multiple nodes on my SLURM cluster, nothing to do with lightning. The init script they provide fails 9/10 times when trying to start workers unfortunately, not sure if it's to do with Ray or the cluster itself.

I've never been able to get Ray properly working on multiple nodes on my SLURM cluster, nothing to do with lightning. The init script they provide fails 9/10 times when trying to start workers unfortunately, not sure if it's to do with Ray or the cluster itself.

I mean maybe it's a problem with your code 🤣
The hack works for me as well running on a single node but not on multiple nodes. Also I'll say again that I don't think that the officially supported solution to this problem should be to change the job name to circumvent PTL's slurm detection

I mean maybe it's a problem with your code 🤣

Not to get off topic but it's not actually my code, it's just the sbatch script they provide. A raylet exits unexpectedly, but that's before anything Lightning-related is invoked so probably unrelated.
I agree it would make sense to have a way to interface with the connectors a little more directly.

Actually I think I’ve run into this before testing scripts on my local machine. It’s a SIGABRT that crashes silently, right? I’m not sure what causes it but restarting my computer fixed it. I don’t know what causes it.

On Mar 10, 2021, at 7:44 PM, Jacob Danovitch<EMAIL_ADDRESS>wrote:

I mean maybe it's a problem with your code 🤣
Not to get off topic but it's not actually my code, it's just the sbatch script they provide. A raylet exits unexpectedly, but that's before anything Lightning-related is invoked so probably unrelated.
I agree it would make sense to have a way to interface with the connectors a little more directly.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I ran into this problem on dask + SLURM. The hack described above works if it is run on each worker process. I also needed to set workers to 0 for data loaders. I hope this helps the next person.

Is still happens on master on SLURM with ray, and probably any processing spawning library.
perhaps it would be nice to add an additional check like: threading.current_thread() is threading.main_thread() before setting the signal?


Safari dropdown fields not working anymore

It's strange, I believe I created an issue today, but can’t find it. On the current MacOs/Safari the dropdown-fields with possibility to enter text cannot be accessed.
Reason seems to be the plugin: autocomplete_light.min.js that cannot be found, see screen-shot below

You may be refering to the closed #292 :)

No, that was because PMB was moved to cluster. But afterwards it still didn't work. ;-)

The bug exists in Safari 15.1 as well.

That is a DAL issue according to this issue https://github.com/yourlabs/django-autocomplete-light/issues/1226 upgrading to latest version should fix it. I'll check and update.

made a mistake in the commit message. This c06d06433b6a119b1f88635edfb20c0604389e5f should resolve the autocomplete bug

This is still (again) a bug that includes IOS and iPadOS. While on the Mac switching to Firefox or Chrome solves it, on the mobile devices all browser have the bug.


How can I filter if Kafka sink sends messages repeatedly?

Replication order
For all DDL or DML statements, TiCDC outputs them at least once.

When the TiKV or TiCDC cluster encounters failure, TiCDC might send the same DDL/DML statement repeatedly. For duplicated DDL/DML statements:

MySQL sink can execute DDL statements repeatedly. For DDL statements that can be executed repeatedly in the downstream, such as truncate table, the statement is executed successfully. For those that cannot be executed repeatedly, such as create table, the execution fails, and TiCDC ignores the error and continues the replication.
Kafka sink sends messages repeatedly, but the duplicate messages do not affect the constraints of Resolved Ts. Users can filter the duplicated messages from Kafka consumers.

How can I filter if Kafka sink sends messages repeatedly? I use the canal-json format, which is consumed by flink.

Since the current version of TiCDC does not support exactly once delivery, maybe you can try deduplicating using Flink's stateful computation? If you need any help implementing this, we can get in touch.

Since the current version of TiCDC does not support exactly once delivery, maybe you can try deduplicating using Flink's stateful computation? If you need any help implementing this, we can get in touch.

Since the current version of TiCDC does not support exactly once delivery, maybe you can try deduplicating using Flink's stateful computation? If you need any help implementing this, we can get in touch.

Ok i try this.

Since the current version of TiCDC does not support exactly once delivery, maybe you can try deduplicating using Flink's stateful computation? If you need any help implementing this, we can get in touch.

Ok i try this.

If you have any other questions please feel free to comment and I will close the issue.


Enhancement Request: Move indexing to localized sqlite db file

I would be curious about interest in this tool moving from a text based storage and tracking system via a massive quantity (for some of us) of per-blog index tracking text files, to a more robust sqlite database file.
The general enhancements over things like recovering from an issue, migration, aggregation of multiple scattered points of data and so on into a singular collated set of tables would offer considerable advantages over the current hierarchical parse and scan of parent and sub folder system where every blog needs it's data parsed from a text file.
It would also expedite the process of moving/migrating an instance, as the current env pathing is saved hard coded per blog index meaning that in a recovery situation we're looking at manually editing what could be a vast about of file paths. I also suspect it would add several considerable speed boosts when it comes to reloading, loading and importing large quantities of blogs.
In short, index.db as a collated sqlite file, nix the per blog index text files, and move to a record-per-blog instead of a config-file-per-blog methodology.

I personally have only had bad experiences with these type of files so i am definitely bias against it. Here are some examples:

I have seen them become corrupt more than one time, losing all data.
I have seen them eat unrealistic large amount of disc space on similar software.
Many times have they been part of crashing issues in the main software.

These may be preventable, but still i feel that when it comes to archiving data, then you don't want to have all files in the same place. The more centralized, the more vulnerable the data becomes. But on the other hand it becomes easier to work with. So it all depends on your philosophy when it comes to archiving over time and what your purpose is.


Clean up Preprocessor Directives After NET6 Removal

Building on #2927. That PR should merge first.
This PR includes a few different types of preprocesser directive cleanups:

in netcore, removing #if NET6_0_OR_GREATER and  #if NET8_0_OR_GREATER

these always evaluated to true

in netcore,  removing #if NETFRAMEWORK

these always evaluated to false

in netfx, removing #if NETX_0_OR_GREATER

these always evaluated to false

in common and test, replacing #if NETX_0_OR_GREATER with #if NET

for readability, these didn't actually care what version of .NET was being used.

LGTM


psgeodb.js: Only finnish (d.fi) and english (d.en) descriptions are processed

A small change in the nested if-then-else is warranted: access d.language or, if it is undefined, d.en, or return undefined by default.

The tests now work, some code cleanup has been done, language support for finding categories has been improved, test inputs and results have been modified appropriately.
The root cause for the errors was that

lack of test updates and analysis of what changes a code change had
hasty fixes (by me) to the test inputs with a typo which led to merging/classification not working
lack of recognising non-en/fi languages.

But as said, it all works now 👍
The new code goes through the languages supported by the language module. I chose that approach over going over any language given in the input. You could also have done the latter. But this is a special feature that we use to recognise text, and going more forward, I expect the inputs to use the classification in the "sa", so at some point we can remove this code.


feat(tracer): add span processor and exporter

This is very big PR (sorry for that), but I felt this was important to connect most of the pieces together like SpanProcessor, SpanExporter, ReadableSpan, BatchSampledSpanProcessor etc.
*Tests are still pending, but I would like to get initial feedback before moving forward (either I will split into smaller PRs or update this PR).

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 2.63%.
The diff coverage is 77.29%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   97.66%   95.02%   -2.64%
==========================================
  Files          38       35       -3
  Lines        2652     2434     -218
  Branches      304      280      -24
==========================================
- Hits         2590     2313     -277
- Misses         62      121      +59

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
59.61% <59.61%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
95.58% <82.35%> (-0.71%)
:arrow_down:

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (-12.98%)
:arrow_down:

...opentelemetry-core/src/trace/globaltracer-utils.ts

...ckages/opentelemetry-core/src/common/NoopLogger.ts

...metry-core/src/trace/instrumentation/BasePlugin.ts

...entelemetry-core/test/trace/TracerDelegate.test.ts

... and 6 more

Codecov Report

Merging #149 into master will decrease coverage by 1.19%.
The diff coverage is 85.42%.

@@            Coverage Diff            @@
##           master     #149     +/-   ##
=========================================
- Coverage   93.84%   92.65%   -1.2%
=========================================
  Files          41       47      +6
  Lines        3282     3606    +324
  Branches      385      412     +27
=========================================
+ Hits         3080     3341    +261
- Misses        202      265     +63

Impacted Files
Coverage Δ

...ic-tracer/src/export/SimpleSampledSpanProcessor.ts
100% <100%> (ø)

...metry-basic-tracer/src/platform/node/timer-util.ts
100% <100%> (ø)

...ages/opentelemetry-basic-tracer/src/BasicTracer.ts
89.58% <100%> (-2.61%)
:arrow_down:

...ntelemetry-basic-tracer/src/platform/node/index.ts
100% <100%> (ø)

...s/opentelemetry-basic-tracer/src/platform/index.ts
100% <100%> (ø)

...emetry-basic-tracer/src/export/NoopSpanExporter.ts
100% <100%> (ø)

...sic-tracer/src/export/BatchSampledSpanProcessor.ts
58.09% <58.09%> (ø)

packages/opentelemetry-basic-tracer/src/Span.ts
72.47% <90.9%> (+12.34%)
:arrow_up:

...kages/opentelemetry-basic-tracer/test/Span.test.ts
82.5% <94.44%> (-17.5%)
:arrow_down:

... and 8 more

Will revise this PR based on specs decision: https://github.com/open-telemetry/opentelemetry-specification/pull/205. Feel free to add comments there.

Per SIG discussion, please add comments on open-telemetry/opentelemetry-specification#205 to shape SpanProcessor in the right direction.

Closing this now, handled this in the multiple separate PRs.


Add forum link to discord in navbar

When visiting f15a I want to chat with people (quickly) so that I can get help / meet the community
Maybe also add to footer with number of people online - see e.g. footer of https://apostrophecms.com
Acceptance

[ ] Link in navbar that has good UX (people clear there is chat)

Tasks

[ ] Get discord unlimited invite link
[ ] Add icon to navbar with link (or maybe just text "Chat") (want hover
[ ] (?) add to footer (with info on number online)

Great! I will replace where we have gitter linked to discord instead and probably make it a bit more easy to find on the navbar. @rufuspollock


⚠ Bofrost.com has degraded performance

In a4dc5da, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 890 ms

Resolved: Bofrost.com performance has improved in 7a2d7f9.


Feature: Wands & Spells casting

Should calculate the appropriate target value. See castSpell().
After reviewing this #72, and instead of adding a new type of items, basic items with the linked spell in description and with the spell DC indicated for now. If we get requests for this later, we could implement it, but I think the current solution is good enough for initial release at least.

We could jack this into the Scrolls mechanics as they use the same game rules. Scrolls have been implemented as treasure right now, should we do the same for Wands?

This is probably solved by #329

Yeah, #329 will implement this.


Move RIPE instance-related logic to mixin

-
-

Issue
https://github.com/ripe-tech/ripe-sdk-components-vue/issues/39

@BeeMargarida please check how this behaves with respect to what was being addressed in https://github.com/ripe-tech/ripe-sdk-components-vue/pull/33?

@BeeMargarida can you try and run the scenarios we discussed? Assign me back once you got the chance to do it :)
Another scenario which should work, and maybe it deserves it's own story, is:
<configurator structure="..." />
<price />
<pickers />

@gcandal I noticed some problems and made some solutions, but before committing I would like to discuss it. I will insert here the problems so that it doesn't get lost.

Weird loading behaviour: when using 2 configurators, one with all the props and another using structure, there were some mismatches in the way the loading was happening.

A possible fix would be the insertion in ripe-configurator of the code below and disabling loading on the "loaded" event of the configurator (which was already happening).

this.onPreConfig = this.ripeData.bind("pre_config", () => {
    this.loading = true;
});

However, there is even some more weird behaviour:

Solution: What fixed this was removing the loading flag from configRipe and having the configurator maintain the state and use the configuring flag as well:

2. Ripe Image refactor: convert the state prop into a computed (it works)
state() {
            return {
                initialsExtra: this.initialsExtra || {},
                initials: this.initials,
                engraving: this.engraving
            }
}

3. Interaction story: using structure, more specifically
![interaction-story](https://user-images.githubusercontent.com/25725586/100361665-56377c00-2ff2-11eb-882d-e5adf8bda0d9.gif)
![interaction-story-model](https://user-images.githubusercontent.com/25725586/100361669-56d01280-2ff2-11eb-8f5a-61420bbcf871.gif)

@BeeMargarida:

Feel free to move the loading toggling to pre/post_config
Is state even needed? Can't we get rid of it and rely on the existing mixin props and state management?
Looking good
You're right, we should probably add options to ripe.Ripe.prototype.setStructure (same as ripe.Ripe.fromStructure)

@gcandal Found a problem with the structure approach, will assign back when solved

@gcandla Problems fixed (there were 2):

The structure sync problem, which was fixed by only triggering a new config when the structure is different from both the previous structure prop and from structureData. This stops the loop that was happening.
The initials not being updated when changing the initials prop. This was fixed by cleaning out initials_extra if they were not changed but the initials and/or engraving changed. The setInitials and setInitialsExtra calls were added for the structure, and the setInitialsExtra was overriding the personalization choices of initials with the outdated initialsExtra.

Problems: in the gifs below it can be noticed that when switching to sergio_rossi, the personalization still exists for a tiny moment and is later removed. Even by resetting, the initials are only overridden when running the setInitials, which is after the config. Not sure how to fix this problem with the current approach.
It is now working:

unchangedStructure -> equal...
Check if we can get rid of  this.equalStructure(structure, this.structureData) || this.equalStructure(structure, previousStructure) by adding a previous + value compare before emitting the event. If it doesn't solve the problem, guard the event emission anyway.

@gcandal It is still necessary to check the equality of  the structure prop and structureData in configOptionsStructure watcher, due to the first setup, where the event will be sent, the structure will correctly update itself on the client side. However, this will change the structure prop, which it will then trigger the configOptionsStructure watcher again and make a new config.

@gcandal The test of the vue components are failing npm run test-all due to the use of optional chaining, which is not supported in webpack < 5. Since webpack > 5 is not supported by storybook and mochapack, a workaround would be to use the @babel/plugin-proposal-optional-chaining plugin in the webpack, which works. The tests are also failing, so will assign back to me to solve them.

@gcandal The test of the vue components are failing npm run test-all due to the use of optional chaining, which is not supported in webpack < 5. Since webpack > 5 is not supported by storybook and mochapack, a workaround would be to use the @babel/plugin-proposal-optional-chaining plugin in the webpack (webpack/webpack#10227), which works. The tests are also failing, so will assign back to me to solve them.

@BeeMargarida please remove the optional chaining (stuff?.other becomes stuff && stuff.other)

@gcandal this is a bit of a leap of faith... too many changes in a single PR, let's please avoid this for the future

@gcandal this is a bit of a leap of faith... too many changes in a single PR, let's please avoid this for the future

@gcandal we need to back-port this to ripe-components-react... Let's be as "literal" as possible to avoid missmatch

@gcandal we need to back-port this to ripe-components-react... Let's be as "literal" as possible to avoid missmatch


Copy files over to another instance.

Will this overwrite the destination's storage or append to it?

It is just like loading from a gist, so files with the same name but different contents will not be overwritten.


Quick successive clicks on tray do not fire clicked event

I have only tested this on OS X. But I consistently see the same result. OS X 10.10.3, Electron 0.27.1.
Here's a quick video demonstrating the issue (~300KB).
menubar.tray = new Tray(iconPath);
menubar.tray.on('clicked', function (e, bounds) {
  console.log('clicked'); // doesn't fire after quick successive clicks
});

You should turn off highlight mode, otherwise OS X would try to toggle between different states during your clicks.

@zcbenz: Thanks, I will do that. Is this not a bug though? Native apps are able to highlight the icon and still respond to clicks.

It is an expected behavior of OS X.

Sorry for labouring the issue and thank you for replying but could you explain why it is expected behaviour?

You can write a simple Cocoa app to reproduce this behavior, no one knows the reason of it except for the Apple devs.

@zcbenz Thanks!
Would it be possible to implement this by introducing a stateChanged event that listens for changes in the tray's state? This could be Mac OS X only (or it could probably be easily polyfilled on other platforms by listening for clicks and toggling the pseudo-state internally in Electron).
HighlightMode is necessary for a native feel on Mac, it would be very nice to have it.

I have tried tray.setHighlightMode(false) and the issue is still there. The icon no longer highlights but it still doesn't respond to clicks in quick succession.

Your clicks might have been trapped as double-clicked:
https://github.com/atom/electron/blob/master/docs/api/tray.md#event-double-clicked

That was the issue, huge thanks! I'm now performing the show/hide action on double-click also and it works perfectly.
Should the tray icon be trapping double-clicks if the app doesn't use them? Could Electron listen for new double-clicked listeners and add a double click action dynamically to the tray button, instead of always adding a double click action? Perhaps this could be done by listening for newListener events on the EventEmitter?

Should the tray icon be trapping double-clicks if the app doesn't use them? Could Electron listen for new double-clicked listeners and add a double click action dynamically to the tray button, instead of always adding a double click action? Perhaps this could be done by listening for newListener events on the EventEmitter?

This behavior has been there for quite a long time, changing it would break existing apps. So no matter whether it is a good behavior or not, I think we should stick to it.

I think the behaviour I have suggested should be backwards compatible with existing Electron apps though, I don't understand how it would break them if they're not listening for the double-clicked event. It's also possible to have a more explicit Tray.setIgnoreDoubleClicks() method but this introduces API bloat.


Can not access over network

I have installed the project on a Ubuntu 16 machine. I go to the folder directory and I npm start opens a new browser tab 'http://localhost:5555/dist/dev/#/plans' and I can access the software. I want it to be accessed from other machines of my lan. So I made changes in nginx;
location /pev {
proxy_pass http://localhost:5555/;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection 'upgrade';
proxy_set_header Host $host;
proxy_cache_bypass $http_upgrade;
}
But when I try to access <IP_ADDRESS>:/pev, I can't access. But I can access another node app for which
nginx configuration is :
location /demo {
proxy_pass http://localhost:3000/;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection 'upgrade';
proxy_set_header Host $host;
proxy_cache_bypass $http_upgrade;
}
How to fix this?

I changed nginx configuration to
proxy_pass http://localhost:5555/dist/dev/#/plans;
Now when I point my browser to
http://<IP_ADDRESS>/pev
it says 'Loading....' in browser.

I changed
server.listen(PORT, () =>
//openResource('http://localhost:' + PORT + APP_BASE + APP_DEST)
openResource('http://<IP_ADDRESS>:' + PORT + APP_BASE + APP_DEST)
);
in /tools/utils/server.ts and now it can be accessed over network. Closing the issue.


example: add workflow for SC18 student cluster competition

Create a workflow for the SC18 student cluster competition. The workflow can have the following stages:

download data using the Zenodo action.
install software dependencies using the spack action.
run parameter sweep using the SLURM action (see #558).
analysis by writing a script in python and running it using a jupyter docker image.

The CK workflow can be used as a reference.

done in https://github.com/popperized/seissol-workflows


Should the default StatusCodePageOptions body result be cached?

👋 G'Day!
I noticed that the StatusCodePageOptions  class has a default way of generating the the appropriate statuscode response to the client. Right now, it's just some standard text (and no, this issue is not about the CONTENT of the StatusCodePageOptions response). Great!
So right now, it basically does this:
 var body = BuildResponseBody(statusCode);
...
return context.HttpContext.Response.WriteAsync(body);

Generate body (based on status code).
Write to response.

The body is always the same per StatusCode, right? If so, why not just cache this in say a ConcurrentDictionary ? Am I missing something?

Yes, that would be possible for the default content. It's a trade off. E.g how often are you sending blank status codes? Would there be dictionary contention. Would you be holding onto unneeded memory? In a normal mvc app I don't expect the default implementation to be invoked often, it's there more as a diagnostic fallback.

how often are you sending blank status codes?

?? Blank status codes? I thought every response has to have an HttpStatusCode?

Would there be dictionary contention.

This is the biggest worry out of all of this. I was thinking of a ConcurrentDictionary only because I was hoping to 'generate' the response once (think -> 2+ requests that arrive at the same time / racing each other). I'm not an expert in concurrency and high-throughput code, so this was my naive thoughts on this.

Would you be holding onto unneeded memory?

We talking about a small number of status codes here, though, right?
The other issue here is also copying the string output from the Dictionary to the Response.  Right now, I think this is still the case with the current code => generate output, send to response, wait for GC to clean up later.
For me, this is mainly about people creating static but rich response pages (think, nice html pages or for me, json results conforming to a standard api error schema/model).

In a normal mvc app I don't expect the default implementation to be invoked often, it's there more as a diagnostic fallback.

Sorry @Tratcher I don't understand what you mean there 😊  Are you saying "under a normal MVC app with just a standard .UseStatusCodes() middleware, routes that end with a 404 or 400 or some common status code that's not 500 and not 200, don't really get hit, much?

By blank status code I meant a response with no body.

We talking about a small number of status codes here, though, right?

In theory it could be all the status codes between between 200 and 999, but in practice it would only be about 20.

routes that end with a 404 or 400 or some common status code that's not 500 and not 200, don't really get hit, much?

404 and 400 are error cases that the user shouldn't see in the average browsing session. In my subjective experience as a client I hit 404s maybe once per day. Your app logs would give you a better frequency analysis.
As for caching custom content, no way, the content generator should take care of that or utilize the response caching middleware. The caching of content on the server has a lot of hazards and rules around cross user contamination.

By blank status code I meant a response with no body.

We talking about a small number of status codes here, though, right?

In theory it could be all the status codes between between 200 and 999, but in practice it would only be about 20.

routes that end with a 404 or 400 or some common status code that's not 500 and not 200, don't really get hit, much?

404 and 400 are error cases that the user shouldn't see in the average browsing session. In my subjective experience as a client I hit 404s maybe once per day. Your app logs would give you a better frequency analysis.
As for caching custom content, no way, the content generator should take care of that or utilize the response caching middleware. The caching of content on the server has a lot of hazards and rules around cross user contamination.

If the concern here is performance, we would need to see data that demonstrates that it's a problem. In the several years that we've had this logic, it's never come up as a concern from anyone. As @Tratcher said, these just aren't very common, so optimizing their performance isn't a big need. And, there's risk of breaking things when caching is introduced. I think in this risk vs. reward equation, it doesn't add up.

Totally no probs at all. Thanks for your input folks :)


"Mindera" Company - No logo

URL: https://peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 11
STEPS TO REPRODUCE:

Open the URL in the browser
Click on the "Cauta" button
Click on the "Companie" button, writhe "Mindera" and seletct it in the search bar.

OR:

Open the URL in the browser
Type a job title from "Mindera" companie and press  "Cauta" button

Text data: ".NET Backend Developer - Mid or Senior"
EXPECTED RESULTS:
"Mindera" companie should appear with a logo.
ACTUAL RESULTS:
"Mindera" company has no logo.

@Florin201092 cand descoperi buguri in search-engine punele in repozitoriul search-engine


fix: rename match a python 3.10  reserved word

Issue #, if available:
Description of changes:

Rename match to match_results
fix Docstrings for optionals.

Checklist

[x] Meet tenets criteria
[x] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

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

Codecov Report

Merging #616 (6825fbc) into develop (25f067a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #616   +/-   ##
========================================
  Coverage    99.95%   99.95%
========================================
  Files          114      114
  Lines         4581     4581
  Branches       249      249
========================================
  Hits          4579     4579
  Partials         2        2

Impacted Files
Coverage Δ

...utilities/data_classes/appsync_authorizer_event.py
100.00% <ø> (ø)

aws_lambda_powertools/event_handler/api_gateway.py
100.00% <100.00%> (ø)

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 25f067a...6825fbc. Read the comment docs.


Is it possible to communicate via cloudflare‘websocket?

any configuration suggestions or samples?

Done
https://github.com/erebe/wstunnel/discussions/351


Basic support for nv17 with v9 actors (calibnet only)

Summary of changes
Changes introduced in this pull request:

Minimal support for running nv17 on calibnet.

Reference issue to close (if applicable)

Closes
Other information and links

@lemmih is "basic" enough to re-enable calibnet sync check in the GH Actions CI?

@lemmih is "basic" enough to re-enable calibnet sync check in the GH Actions CI?

Yeah, should be. At least once https://github.com/ChainSafe/forest/pull/2170 has been merged.

Woo! Calibnet sync check!


Move fixtures directory to the project root

This has been replaced with https://github.com/GoogleCloudPlatform/cloud-debug-nodejs/pull/314.


日志名称 建议以定时任务的名称命名

Clear and concise description of the problem
at
Suggested solution
AT
Alternative
No response
Additional context
No response
Validations

[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

可否有名称时，以名称命名，无名称时再以命令 命令


Create MANIFEST to resolve missing files

Created MANIFEST.in in order to include additional files that are missing after building docker image. This may be too inclusive though since I just added everything in the src/dispatch folder, so adjust as necessary. Not familiar enough with the code to know what can or cannot be excluded.
This should resolve #59 and #60

Great catch, thank you!


always start wiki, chat, webinterface packages

Verified
For wiki and chat:
wikis and chatbot are loaded once server is started.

webinterface:
bcdbfs, gedis http and gedis websocket tests are passing


Trouble with Safari playing sound

Hello,
I have some trouble on Safari 11 for playing the recorded sound.
It seems like the blob created doesn't load.
It used to work before and now the safari player is broken.
I first thought it has to do with my code but then I went back to your demo and there is the same bug.
Are you aware of this problem and do you have any clues to solve it ?
Thanks you in advance

@AllanSV sorry for delay. I had too much work.
We have some fixes in master for Safari. Can you try to replace "audio-recorder-polyfill": "^0.1.3" to "audio-recorder-polyfill": "ai/audio-recorder-polyfill", then call npm install (or yarn install), and test Safari again?

The fix released in 0.1.4

Hi,
I tried again, but on IOS Safari and it seems to have the same issue =/

I have same problem.
I added EventListener on dataavailable , but event.data was empty Blob.

@progfay does online demo work for you?

Yes, online demo does not work on my iPhone.
My problem is maybe same as #4 .
The reason why it does not work was that onaudioprocess did not fire.

@progfay what Safari version do you use?

@ai This is my iPhone's User Agent.
Mozilla/5.0 (iPhone; CPU iPhone OS 12_1_2 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/12.0 Mobile/15E148 Safari/604.1


override default node selector in suggested namespace

ECOPROJECT-1783 , OCPBUGS-22305
override the default node selector, in order to make sure that SNR node selector which is defined in the Daemonset will take precedence.

/test 4.12-openshift-e2e

Can you test please what happens when the namespace already exists without that annotation? If it isn't added, we should add it to the other operators as well, and need to document it.

I've done a quick test:

created a catsrc with the new version (after I've already created the ns)
installed snr from that catsrc

As far as I can see the annotation wasn't added to the ns.
However I've also did a simpler test, installing the operator when the ns didn't exist and I saw the same result :sweat_smile:
since the namesapce was created and the pods where created in the correct ns I'm thinking that only operatorframework.io/suggested-namespace: annotation was proccessed from the csv and not operatorframework.io/suggested-namespace-template: but it needs furtther investigation.

Can you test please what happens when the namespace already exists without that annotation? If it isn't added, we should add it to the other operators as well, and need to document it.

I've done a quick test:

created a catsrc with the new version (after I've already created the ns)
installed snr from that catsrc

As far as I can see the annotation wasn't added to the ns.
However I've also did a simpler test, installing the operator when the ns didn't exist and I saw the same result 😅
since the namesapce was created and the pods where created in the correct ns I'm thinking that only operatorframework.io/suggested-namespace: annotation was proccessed from the csv and not operatorframework.io/suggested-namespace-template: but it needs furtther investigation.

just to be sure, you installed using the console in both cases, right? 🙂

just to be sure, you installed using the console in both cases, right? 🙂

Indeed I have

/retest

After modifying the format of operatorframework.io/suggested-namespace-template I've managed to properly test it.

In case the namespace does not exit the annotation is added as expected.
Otherwise it isn't.

I've updated the related epic: https://issues.redhat.com/browse/ECOPROJECT-1761
@clobrano I see that so far it was only implemented in MDR so make sure to update as well.

In case the namespace does not exit the annotation is added as expected.
Otherwise it isn't.

too bad, thanks for checking!
There seems to be an unrelated code change, otherwise lgtm

/retest

/retest

/lgtm


Updating records on a filtered result.

Open 2 browser tabs. Then on tab 1, filter the query to show only 1 result.
On the second tab update a record that does not exists in the first tab.
Sometimes the record updated on the second tab is shown on the first tab. But it shouldn't become visible because of the declared filter.
This problem should pose problems to pagination.
Maybe this problem is because the server treats both browser tabs as the same client?

interesting, thanks for your bug. I will try for myself, hopefully I can write a test to reproduce this over the weekend.

Hi,
Any new developments on this issue?
Thanks


Add serve support for summarization and translation

What does this PR do?

Fixes # (issue)
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests? [not needed for typos/docs]
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @ethanwharris! Thanks for updating this PR.

In the file tests/text/seq2seq/summarization/test_model.py:

Line 93:1: W391 blank line at end of file


Prefer system libraries over intree copies

I want to package OpenTimelineIO for openSUSE. With the 0.12.1 tarball from github, I end up with:
CMake Error at src/deps/CMakeLists.txt:1 (add_subdirectory):
  The source directory
    /home/abuild/rpmbuild/BUILD/OpenTimelineIO-0.12.1/src/deps/pybind11
  does not contain a CMakeLists.txt file.
-- Configuring incomplete, errors occurred!

Though we have version 2.4.3 available in the distribution and it could use that. Could the project prefer system copies and only fallback to intree copies if not found?

Some feedback/thoughts:

An immediate workaround that should work would be: before configuring the project, remove the pybind11 directory and symlink yours.
The significantly complicating factor about the OTIO build process is the integration with pip, any solution needs to consider it's requirements.
The CMakeLists.txt file in deps could be modified to check the environment for the necessary packages, and only sync the submodules when they are not available in the environment.
Whether or not the environment is checked might need an external control such as an options flag at the top level cmake to accommodate pip.


Fix Linux compatibility issues

The Security framework is not available on Linux so we need a different way to handle APNS.

See also: NSURLCredential.swift:126


Company go error running spacemacs

Getting the following error running spacemacs<EMAIL_ADDRESS>go: 1.9 :
"company-go error "Wrong type argument: stringp, nil" with args (candidates)"

https://github.com/nsf/gocode/issues/466


C implementation of Graph Algorithms

Ground rules:

Whoever claims the Issue first, should send a Pull Request within 24 hours else it'll be assigned to another claimant and further Pull Requests regarding that issue will not be entertained.
Pull Requests sent without claiming and referencing the Issue will not be entertained.
Make sure your Pull Requests have just one commit in them. Squash multiple commits into one if you have to.

Check out this link to understand better.
The following Algorithms need to be added:

BFS
DFS
Kruskal's
Prim's
Dijkstra's
Floyd-Warshall Algorithm

u need all the C graph algorithms or only the standard ones(bfs,dfs etc)?

Hey, check the description.

I would like to do the DFS algorithm. Thank you!

Go ahead!

Issue has been created separately in the Issues tab. Kindly take a look!


🆕 Software Suggestion |  Tails Operating System

Basic Information
Name: Tails
Category: Operating System (Live CD)
URL: https://tails.boum.org/index.en.html
Description
Official Description:
Tails is a portable operating system that protects your privacy and helps you avoid censorship.
My Description:
Tails is a Live CD Operating System which route all your Internet traffic through Tor by default, preinstalled with multiple security and privacy enhancement tool. An OS similar to Whonix.
Why I am making the suggestion
Tails is recommended by EFF and Roger Dingledine, co-founder of the Tor Project.

Tails also allows creation of a Encrypted LVM (tails called it encrypted persistence volume), which allows persist of data, browser bookmarks, gpg keys, ssh keys and known-hosts, and configurations of multiple software, even packages (downloaded from apt CLI or Synaptic only) that you want to reinstall after reboot.
I'd suggest Tails' inclusion in the "Worth Mentioning" part of "Operating System".
My connection with the software
I am not familiar with the developer of Tails.

[√] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

Sorry for the bothering, didn't scroll down enough.


Open Multiple ignores the selected order of the files

Describe the bug
When multiple files are opened, it seems to sort them alphabetically by filename and that determines the order they are assigned to be Log0, Log1, Log2 ....
The order the files are selected is not used.
The saved layout is expecting a certain logfile as Log0, Log1, Log2, etc...So if the order is not correct the layout fails to get the data.
To Reproduce
Make a layout that uses, say 2 files, select files using ctrl left mouse-click such that the file alphabetically last if the first you select.  It will not be Log0, it will be Log1.
Expected behavior
Want the selected order to be used.
Screenshots
Version (required):
using windows11, AdvantageScope v3.2.1
Additional context
Maybe fix might become feature request if you can reorder the files after opening.

AdvantageScope uses the native file selector for each operating system, which means that this behavior is inconsistent across operating systems. The behavior on macOS matches what you're requesting, but Windows sorts the files alphabetically before returning them to AdvantageScope. Changing file paths after the logs are decoded is a bit tricky, but I could potentially add a pop-up to reorder files when multiple logs are selected (on Windows at least, and Linux if that's also an issue).


feat: Remove dequeued bundles and messages after a day

Description

References

#0000

Codecov Report

Merging #350 (b13d03c) into main (21978a1) will increase coverage by 0.54%.
The diff coverage is 85.50%.

@@            Coverage Diff             @@
##             main     #350      +/-   ##
==========================================
+ Coverage   83.07%   83.62%   +0.54%
==========================================
  Files         456      519      +63
  Lines       10418    12213    +1795
==========================================
+ Hits         8655    10213    +1558
- Misses       1438     1631     +193
- Partials      325      369      +44

Impacted Files
Coverage Δ

...ivedMessages/ArchivedMessageGetDocumentListener.cs
0.00% <ø> (ø)

.../ArchivedMessages/ArchivedMessageSearchListener.cs
0.00% <0.00%> (ø)

...ation/Middleware/Correlation/CorrelationContext.cs
66.66% <ø> (ø)

...uration/Middleware/UnHandledExceptionMiddleware.cs
0.00% <0.00%> (ø)

...essages/RequestAggregatedMeasureMessageReceiver.cs
0.00% <0.00%> (ø)

...equestAggregatedMeasureTransactionQueueListener.cs
0.00% <0.00%> (ø)

...rchivedMessages/GetArchivedMessageDocumentQuery.cs
0.00% <0.00%> (ø)

...ChangeAccountingPointCharacteristicsTransaction.cs
0.00% <0.00%> (ø)

...r/RequestChangeOfSupplierIncomingMarketDocument.cs
100.00% <ø> (ø)

...rce/Application/SearchMessages/GetMessagesQuery.cs
12.50% <0.00%> (ø)

... and 147 more

... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


panicked at distribution key [2] must be a subset of primary key []

Describe the bug
dev=> describe business_user_event;
       Name       |            Type
------------------+-----------------------------
 XXX_user_id    | varchar
 event_type_id    | integer
 event_created_at | timestamp without time zone
 event_date       | varchar
 event_id         | varchar
 dw_creation_date | timestamp with time zone
 primary key      | _row_id
(7 rows)

dev=> select definition from pg_views where viewname = 'business_user_event_view';
definition
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
SELECT
  DISTINCT ON (event_id) event_id,
  XXX_user_id,
  CAST((split_part(event_date, '/', 1) || '-' || split_part(event_date, '/', 2) || '-' || split_part(event_date, '/', 3)) AS TIMESTAMP) AS event_date,
  event_type_id,
  event_created_at,
  dw_creation_date
FROM business_user_event
ORDER BY
event_id,
dw_creation_date ASC

dev=> describe completed_user_info;
dev->
         Name         |                  Type
----------------------+-----------------------------------------
 email                | varchar
 user_id              | varchar
 first_name           | varchar
 last_name            | varchar
 country              | varchar
 telegram             | varchar
 role                 | varchar
 company              | varchar
 company_size         | varchar
 company_homepage     | varchar
 category             | varchar
 title                | varchar
 subscribe_newsletter | boolean
 status               | varchar
 primary key          | email, business_user_info.XXX_user_id
(15 rows)

CREATE MATERIALIZED VIEW completed_user_info  AS
WITH user_table as (
    SELECT
          XXX_user_id as user_id,
          email as user_email
    FROM business_user_info
    WHERE lower(SPLIT_PART(SPLIT_PART(email, '@', 2), '.', 1)) != 'XXX'
),
applicant_table as (
    SELECT
          XXX_user_id as applicant_id,
          email,
          first_name,
          last_name,
          country,
          telegram,
          role,
          company,
          company_size,
          company_homepage,
          category,
          title,
          subscribe_newsletter,
          status
    FROM business_applicant_info
    WHERE lower(SPLIT_PART(SPLIT_PART(email, '@', 2), '.', 1)) != 'XXX'
)
SELECT email,
      COALESCE(user_id, applicant_id) as user_id,
      first_name,
      last_name,
      country,
      telegram,
      role,
      company,
      company_size,
      company_homepage,
      category,
      title,
      subscribe_newsletter,
      status
FROM applicant_table
LEFT JOIN user_table ON email = user_email

Panic when:
CREATE MATERIALIZED VIEW YYY as
SELECT event_date,
      event_type_id,
      XXX_user_id,
      email,
      first_name,
      last_name,
      country,
      telegram,
      role,
      company,
      company_size,
      company_homepage,
      category,
      title,
      subscribe_newsletter,
      status
FROM business_user_event_view
LEFT JOIN completed_user_info ON XXX_user_id = user_id

thread 'risingwave-main' panicked at 'distribution key [2] must be a subset of primary key []', src/common/src/catalog/internal_table.rs:51:21
stack backtrace:
   0:     0x55737bc2156f - std::backtrace_rs::backtrace::libunwind::trace::h2ae08ac77e721453
                               at /rustc/28a29282f6dde2e4aba6e1e4cfea5c9430a00217/library/std/src/../../backtrace/src/backtrace/libunwind.rs:93:5
   1:     0x55737bc2156f - std::backtrace_rs::backtrace::trace_unsynchronized::hb7d3015d68fe20ce
                               at /rustc/28a29282f6dde2e4aba6e1e4cfea5c9430a00217/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:     0x55737bc2156f - std::sys_common::backtrace::_print_fmt::h60e26e0e8c18d6ce
                               at /rustc/28a29282f6dde2e4aba6e1e4cfea5c9430a00217/library/std/src/sys_common/backtrace.rs:65:5
   3:     0x55737bc2156f - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h3b0d15c01f4c6ce1
                               at /rustc/28a29282f6dde2e4aba6e1e4cfea5c9430a00217/library/std/src/sys_common/backtrace.rs:44:22
   4:     0x5573768a420f - core::fmt::write::h53efc8feba3ef479
                               at /rustc/28a29282f6dde2e4aba6e1e4cfea5c9430a00217/library/core/src/fmt/mod.rs:1254:17
   5:     0x55737bc1bd93 - std::io::Write::write_fmt::h36688c4bf2bf4993

To Reproduce
No response
Expected behavior
No response
Additional context
No response

What does the SQL want to express with the ORDER BY clause? Does the writer want the view always maintain the Record with the topest (event_id, dw_creation_date ASC) value for each distinct event_id?
dev=> select definition from pg_views where viewname = 'business_user_event_view';
definition
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
SELECT
  DISTINCT ON (event_id) event_id,
  XXX_user_id,
  CAST((split_part(event_date, '/', 1) || '-' || split_part(event_date, '/', 2) || '-' || split_part(event_date, '/', 3)) AS TIMESTAMP) AS event_date,
  event_type_id,
  event_created_at,
  dw_creation_date
FROM business_user_event
ORDER BY
event_id,
dw_creation_date ASC

i guess he used this SQL as a batch query to view the results on superset first, and then directly copy paste it as a view/MV.


[WIP][HUDI-3579] Add timeline commands in hudi-cli

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@yihua : ping me once the patch is ready to be reviewed again


Orphaning a key under "Remap a key" also orphans all modifier+key possibilities under "Remap a shortcut"

Microsoft PowerToys version
0.64.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
In Remap a Key, set Caps Lock (for example) to "Disable".  (The problem occurs with any key I tested, not just Caps Lock.)
In Remap a Shortcut, set Ctrl-Caps Lock (or Win-Caps Lock, or Alt-Caps Lock, etc.) as the Physical Shortcut and the desired operation as the Mapped To key combination.
✔ Expected Behavior
That the Mapped To action would work using the new Physical Shortcut, Ctrl-Caps Lock.
❌ Actual Behavior
The Physical Shortcut Ctrl-Caps Lock (etc.) does not implement the desired Mapped To action; it does nothing.
Essentially, orphaning a single key under Remap a Key orphans the key not only when the key is used alone, but also when the key is used with a modifier.
This may be an unavoidable result of how Windows works, but it reduces keyboard versatility.
Other Software
No response

This is by design. See, when you remap one key, that remapping will be maintained even when "inside bigger shortcuts". It's the key that changes. You might want to look at "Remap a shortcut" if you have specific ones, or try (something like) AutoHotkey or MS Keyboard and Mouse Center.

This is by design.  Key remappings take priority.
We do try and directly call this out in the UX as well for shortcuts.


Latest Prometheus image has high vulnerability (i.e. asking update busybox to <IP_ADDRESS>)

What did you do?
Pull the image and did scan using scan tool, found High Vulnerability
What did you expect to see?
Zero High/Critical Vulnerability.
Also I do see apt-get is been removed. so I'm not able to even run the update command.
Vulnerabilities
+------------------+----------+------+---------+---------+-----------------+-----------+------------+----------------------------------------------------+
|       CVE        | SEVERITY | CVSS | PACKAGE | VERSION |     STATUS      | PUBLISHED | DISCOVERED |                    DESCRIPTION                     |
+------------------+----------+------+---------+---------+-----------------+-----------+------------+----------------------------------------------------+
| CVE-2018-1000500 | high     | 8.10 | busybox | 1.31.1  | fixed in 1.32.0 | > 2 years | < 1 hour   | Busybox contains a Missing SSL certificate         |
|                  |          |      |         |         |                 |           |            | validation vulnerability in The "busybox          |
|                  |          |      |         |         |                 |           |            | wget" applet that can result in arbitrary code    |
|                  |          |      |         |         |                 |           |            | execution. This a...

Prometheus version:
Latest

Hello,
First, the process to report security issues is described here: https://prometheus.io/docs/operating/security/
Preferably, it should not happen in public.
We are aware of this, it is a limitation of the upstream busybox base image you are using. You can followup the issue here: https://github.com/docker-library/busybox/issues/80
This is not affecting Prometheus users directly as prometheus can't run commands. You need to have local shell access to use this, in which case you are having other issues.

I have published an updated prom/busybox with 1.32.0.
Please note, Prometheus does not use wget within the image, and uses Go certificate validation. So Prometheus is unaffected by the busybox issue.

I have published an updated prom/busybox with 1.32.0.
Please note, Prometheus does not use wget within the image, and uses Go certificate validation. So Prometheus is unaffected by the busybox issue.

While 1.32 has the fix, it is compiled in the default docker library image.