Base64 padding

The salmon spec defines the base64url encoding as follows:

The encoding used to convert between the MIME type and the "data" string. In this specification, encoding MUST be the string "base64url", indicating the url safe base64 encoding as described in RFC 4648 [RFC4648], sans any trailing padding with equals (=) characters, and with the additional whitespace normalization rules specified in Section 5.1. All implementations MUST support this encoding. Appears exactly once.

Mastodon is sending base64 data with padding = characters, and it shouldn't. You can set the padding parameter to false to solve this.
The decode_base64 hack is related to this: https://github.com/tootsuite/ostatus2/blob/master/lib/ostatus2/magic_key.rb#L12

There was a later rev of magic-sig which stripped all the '=' from base64url and we had an issue with StatusNet at the time (I filed this same bug against StatusNet in 2010 iirc) which forced everybody to (potentially) send two salmons for each message as you couldn't be sure which would verify against which revision of the spec. I note that this later rev (and one following that with different namespaces and which was even messier to implement - potentially requiring you to send the same message 3 times) both sort of vanished from the "official" salmon repo. I'm taking that to indicate the later revs were dropped but somebody should ask John Panzer. I think most everybody that's still using XML on the federated web is using rev 01 of magicsig (the one which allows '=').


Import not resolved in declaration of exported function

The new Bar() is not resolved / translated. If I remove the export it works.
user@dev:~/somepkg$ cat import.js
import {Foo as Bar} from 'something';

export const fn = ({a = new Bar()}) => a;
user@dev:~/somepkg$ npx swc --no-swcrc import.js
'use strict';
Object.defineProperty(exports, '__esModule', {
    value: true
});
exports.fn = void 0;
var _something = require('something');
var fn = ({ a =new Bar()  })=>a
;
exports.fn = fn;

user@dev:~/somepkg$ npm show @swc/core version
1.0.19

Fixed with @swc/core 1.0.20


Promise resolution type cannot be inferred inside a function with conditional types

Bug Report
🔎 Search Terms
conditional, generic
🕗 Version & Regression Information
03/12/2022

This is a crash
This is the behavior in every version I tried, and I reviewed the FAQ for entries about common bugs that aren't bugs

⏯ Playground Link
Playground link with relevant code
💻 Code

type Base64Data = FileReader['result'];

// fill-in for actual functions
function base64ToFile (x: any) { return new File([], ''); }
function fileModifier (y: File) { return new Promise<Blob>(res => res('' as unknown as Blob)); }

export function downsizeImageFile<T extends boolean = false>(
  imgFile: File,
  preserveBase64?: T,
  maxSize = 640
): Promise<T extends true ? Base64Data : File> {
  return new Promise(async resolve => {
      const resizedBlob = await fileModifier(imgFile);
      const reader = new FileReader();
      reader.readAsDataURL(resizedBlob);

      reader.onloadend = () => {
        resolve(preserveBase64 ? reader.result : base64ToFile(base64file)); // problem: the correct type cannot be inferred from the conditional statement
      };
  });
}

export async function downsizeImage<T extends boolean = false>(
  base64Img: Base64Data,
  preserveBase64?: T,
  maxSize = 640
): Promise<T extends true ? Base64Data : File> {
  return downsizeImageFile(base64ToFile(base64Img), preserveBase64, maxSize);
}

🙁 Actual behavior
The promise resolution type is not inferred correctly from the conditional statement.
🙂 Expected behavior
The promise resolution type must be Base64Data if the preserveBase64 argument is true, otherwise it must be File

Resolving of conditional types is deferred when they involve unbound generic types. This is a design limitation. You should use overloads instead in this case.

To expand on @MartinJohns reply, it's better to narrow the return type and widen the input type than the opposite.
async function downsizeImage(source: File|Base64Data, maxSize:number = 640): Promise<Base64Data>
{
	if (!(source instanceof File))
	{
		source = base64ToFile(source);
	}

	const resizedBlob = await fileModifier(source);
	const reader = new FileReader();
	reader.readAsDataURL(resizedBlob);

	const loadend_promise = new Promise<void>(
		(resolve) =>
		{
			reader.onloadend = () =>
			{
				resolve();
			};
		}
	);

	await loadend_promise;

	return reader.result;
}

async function downsizeImageFile(source: File|Base64Data, maxSize:number = 640): Promise<File>
{
	const data: Base64Data = await downsizeImage(source, maxSize);

	return base64ToFile(data);
}


ABI-safe tf_prefix cache

To comply ABI policy, adding new field to existing classes are not allowed.
Instead, I added a thread-safe global map to keep the parameter, with the key being is class instance pointer.

I have sent a PR to ros. It is another question that we GITAI have to keep ABI of this module.


2.0.0+1.20.1-fabric

This is the PR that gets us to 2.0.0+1.20.1 on fabric/quilt.
This will take quite the amount of time, we will draft this for now.

This is just a notice not to utilize this until a major issue is resolved. It's a draft anyway...

LGTM


Update DNS URL to covasim.idmod.org with HTTPS Support

Reproducing slack chat:

@celiot-IDM
HTTPS support has come up as an issue recently. For example, our DOD partners using locked-down browsers can't get to idmod.org. So let's plan to have https support from the start.
@cliffckerr   17:23
Would it be possible to have covasim.idmod.org? VOI stands for "value of information" and is for a completely different model and I just used that because it was the only DNS I had access to
@John-Sheppard   17:31
We could.  I'll check with ITOPS on the availability.  I thought that is what you wanted to use the whole time so I didn't pursue anything different.

Working: https://covasim.idmod.org/


Include message in exceptions raised for failed API calls

Hi I've aldready opened this ticket regarding timeout between Prefect and airbyte.
Since the 0.1.1 it works way better thanks for that
However I got a new timeout error with a 409 error code
Encountered exception during execution:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/prefect_airbyte/client.py", line 161, in trigger_manual_sync_connection
    response.raise_for_status()
  File "/usr/local/lib/python3.9/site-packages/httpx/_models.py", line 736, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Client error '409 Conflict' for url 'http://airbyte-prd-webapp.airbyte.svc.cluster.local/api/v1/connections/sync/'
For more information check: https://httpstatuses.com/409

Any idea ? This is the first time it happens

A 409 indicates that a request can't be completed due a conflict in state. 409 errors aren't documented in the Airbyte API docs, so it's hard to say what cause this error. I expect that the Airbyte server is returning a message that would give info on how to resolve this error, but we aren't bubbling that up. We can enhance our error handling so that in the future we'll be able to see the reason for errors like this.

Actually, the root cause is not the 409 errors
It's first a timeout issue (with no more details in the error) and then when retrying (specified in with_options) I got a 409 error (which seems to be a side effect).
So it's still a timeout issue like below
Encountered exception during execution:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/httpcore/backends/asyncio.py", line 33, in read
    return await self._stream.receive(max_bytes=max_bytes)
  File "/usr/local/lib/python3.9/site-packages/anyio/_backends/_asyncio.py", line 1265, in receive
    await self._protocol.read_event.wait()
  File "/usr/local/lib/python3.9/asyncio/locks.py", line 226, in wait
    await fut
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/httpcore/_exceptions.py", line 8, in map_exceptions
    yield
  File "/usr/local/lib/python3.9/site-packages/httpcore/backends/asyncio.py", line 35, in read
    return b""
  File "/usr/local/lib/python3.9/site-packages/anyio/_core/_tasks.py", line 118, in __exit__
    raise TimeoutError
TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/httpx/_transports/default.py", line 60, in map_httpcore_exceptions
    yield
  File "/usr/local/lib/python3.9/site-packages/httpx/_transports/default.py", line 353, in handle_async_request
    resp = await self._pool.handle_async_request(req)
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/connection_pool.py", line 253, in handle_async_request
    raise exc
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/connection_pool.py", line 237, in handle_async_request
    response = await connection.handle_async_request(request)
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/connection.py", line 90, in handle_async_request
    return await self._connection.handle_async_request(request)
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/http11.py", line 105, in handle_async_request
    raise exc
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/http11.py", line 84, in handle_async_request
    ) = await self._receive_response_headers(**kwargs)
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/http11.py", line 148, in _receive_response_headers
    event = await self._receive_event(timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/httpcore/_async/http11.py", line 177, in _receive_event
    data = await self._network_stream.read(
  File "/usr/local/lib/python3.9/site-packages/httpcore/backends/asyncio.py", line 35, in read
    return b""
  File "/usr/local/lib/python3.9/contextlib.py", line 137, in __exit__
    self.gen.throw(typ, value, traceback)
  File "/usr/local/lib/python3.9/site-packages/httpcore/_exceptions.py", line 12, in map_exceptions
    raise to_exc(exc)
httpcore.ReadTimeout

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/prefect/engine.py", line 1185, in orchestrate_task_run
    result = await task.fn(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/prefect_airbyte/connections.py", line 127, in trigger_sync
    job_status, job_created_at, job_updated_at = await airbyte.get_job_status(
  File "/usr/local/lib/python3.9/site-packages/prefect_airbyte/client.py", line 191, in get_job_status
    response = await client.post(get_connection_url, json={"id": job_id})
  File "/usr/local/lib/python3.9/site-packages/httpx/_client.py", line 1842, in post
    return await self.request(
  File "/usr/local/lib/python3.9/site-packages/httpx/_client.py", line 1527, in request
    return await self.send(request, auth=auth, follow_redirects=follow_redirects)
  File "/usr/local/lib/python3.9/site-packages/httpx/_client.py", line 1614, in send
    response = await self._send_handling_auth(
  File "/usr/local/lib/python3.9/site-packages/httpx/_client.py", line 1642, in _send_handling_auth
    response = await self._send_handling_redirects(
  File "/usr/local/lib/python3.9/site-packages/httpx/_client.py", line 1679, in _send_handling_redirects
    response = await self._send_single_request(request)
  File "/usr/local/lib/python3.9/site-packages/httpx/_client.py", line 1716, in _send_single_request
    response = await transport.handle_async_request(request)
  File "/usr/local/lib/python3.9/site-packages/httpx/_transports/default.py", line 353, in handle_async_request
    resp = await self._pool.handle_async_request(req)
  File "/usr/local/lib/python3.9/contextlib.py", line 137, in __exit__
    self.gen.throw(typ, value, traceback)
  File "/usr/local/lib/python3.9/site-packages/httpx/_transports/default.py", line 77, in map_httpcore_exceptions
    raise mapped_exc(message) from exc
httpx.ReadTimeout

And here are my trigger_sync definition
        return trigger_sync.with_options(retries=10, retry_delay_seconds=5)(
            connection_id=connection_id,
            poll_interval_s=5,
            status_updates=True,
            airbyte_server_host=urlparse(airbyte_api_base_url).hostname,
            airbyte_server_port=urlparse(airbyte_api_base_url).port,
            timeout=100,
        )

I will try to reduce my poll interval

At the end it seems related to a lack of CPU from my airbyte server pod


Publish a GCP 7.0 migration guide

Migration details will be in both the release notes in github and published as a page on https://www.pulumi.com/registry/packages/gcp/how-to-guides/.

Collaborative drafting doc here: https://docs.google.com/document/d/1GOovPt6gDKDiks9wo1ha6vMV8-0FOdhH4kmaTwmOito/edit#heading=h.5cb0dzlaz83h

https://github.com/pulumi/registry/pull/3474


[FEATURE] introduction of nullable argument in variables

Is your feature request related to a problem? Please describe
For many practitioners using external modules, they end up having to copy the module variables.tf and their default values.
This can be cumbersome, and if upstream changes the default values, hard for downstream to keep up.
Describe the solution you'd like
Terraform 1.1 introduced the concept of nullable
https://www.terraform.io/language/values/variables#disallowing-null-input-values
This means that for the variables in the module with already set default values, the downstream root module wont have to set a default, and can just have a null, and the apply run will work as expect
Additional context
If this is accepted, I am willing to provide a PR updating https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/variables.tf with the argument nullable=false in all places a default is set.
There is only one pesky problem, that this requires terraform v1.1 as min, which would mean this module would jump from 1.0 to 1.1
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/versions.tf#L2

is there any interest in this ?

hey @FernandoMiguel - could you share the value proposition of this addition?

@bryantbiggs This means that for the variables in the module with already set default values, the downstream root module wont have to set a default, and can just have a null, and the apply run will work as expect.
Do you feel this adds value, and reduce overhead for practitioners?

the root module is what the practitioners write on their own side.
I was going to show with the examples from this module, https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/examples/eks-cluster-with-new-vpc/variables.tf
but that one is pretty empty.
Most modules I see, folks copy the external module variables.tf to their own root module, so they can tweak all and any options they want.
and by doing that, they will also have to copy the default values, even if they dont want to change them.
with nullable = false added to this module variables.tf, practitioners would not need to do that.

why would users who are utilizing the module copy the variables.tf? If you don't copy the variables.tf then you achieve the same outcome that you are seeking - the defaults provided by the module will be used unless you opt in to providing a different value for the exposed variables - correct?

Thanks for surfacing this @FernandoMiguel but I think at this point we will not bring this into the project. We can always re-visit in the future if there are strong use cases for supporting this. Thank you again!

hi @bryantbiggs
here's an example
prior to https://github.com/aws-ia/terraform-aws-eks-blueprints/pull/634 being merged, those chart versions defaulted to very old versions.
our karpenter module using blueprints, uses a variable to allow to set the version
module "eks_blueprints_kubernetes_addons_karpenter" {
  source = "git::ssh://git@github.com/aws-ia/terraform-aws-eks-blueprints.git//modules/kubernetes-addons?ref=v4.0.10"

  eks_cluster_id   = var.eks_cluster_id
  enable_karpenter = true
  karpenter_helm_config = {
    cleanup_on_fail = true
    force_update    = true
    replace         = true
    version         = var.karpenter_helm_config_version
  }

}

variable "karpenter_helm_config_version" {
  type     = string
  default  = "0.11.1"
  nullable = false
}

but if blueprints track newer versions often, all we need to do is to track blueprints releases instead of the charts.
that means we can set our var default to null and if there is nullable enabled in blueprints and this value is used instead https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/modules/kubernetes-addons/karpenter/locals.tf#L20
if nullable is not available, we either have to not have any code since it would override (even with null) the blueprints default, or we would have to track it in two places (chart and blueprints version).
Hope this help understand the benefits of this little unknown but super useful feature.


Guidance on freezing PRs at release time

[{admiral}](https://pharmaverse.github.io/admiral/cran-release/) releases are targeted for the first Monday of the last month of each quarter.

I feel like we never follow that part of guidance and we should just drop it.
Originally posted by @bms63 in https://github.com/pharmaverse/admiral/pull/2129#discussion_r1339181653

Looks like this was done at some point


fix: alt text ui changes

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Question]: VSTest@2 and VSTest@3 are not working within Azure DevOps anymore

Task name
VSTest2
Task version
2.235.3
Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[X] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Windows 2022
Question
dear community,
since the update from 2.253.2 to 2.235.3 on Wednesday 13. March 2024 our test pipeline fails in azure devops:
'##[error]VsTest task failed.'

found this PR
https://github.com/microsoft/azure-pipelines-tasks/pull/19617
saying there is an update from 2.253.2 to 2.236.0, however when using '-task<EMAIL_ADDRESS>I can't run the pipeline 'A task is missing. The pipeline references a task called 'VSTest'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace:
https://marketplace.visualstudio.com.
(Task version 2.236.0, job 'testAndBuild4appService', step ''.)'

Also tried
'- task<EMAIL_ADDRESS>'- task<EMAIL_ADDRESS>'- task: VSTest@3'
always got
'##[error]VsTest task failed.'

Hi @thomastrinamix, could you try again with '- task<EMAIL_ADDRESS>

Hi @thomastrinamix, could you try again with '- task<EMAIL_ADDRESS>
Encountered error(s) while parsing pipeline YAML:
A task is missing. The pipeline references a task called 'VSTest'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version 2.236.1, job 'testAndBuild4appService', step ''.)

ok. it looks like this was more an Azure DevOps bug... you can close this one here.
just recreated the pipeline with exactly the same yml, branch and - task: VSTest@2
It uses 'Version : 2.235.3' and everything works as it did in the past
Old pipeline with reference to exactly same yml and branch still results in '##[error]VsTest task failed.'
---> Solutions (Azure DevOps): create a new pipeline with the same yml


Fix failed uploads

Title (required)
Fixes #1485
Will add the description if this actually fixes the issue. :)
@nicolas-raoul @misaochan Can you help in testing this.
For me:

I was able to reproduce the issue where image upload started failing with mustbeloggedin error.
I kept tweaking a few things while retrying the same upload and was running into authentication issues.
Finally the retrial failed with a different error. I am assuming this is an unrelated error.

<?xml version="1.0" encoding="UTF-8"?><api servedby="mw1284"><error code="verification-error" info="File extension ".John's_National_Academy_of_Health_Sciences" does not match the detected MIME type of the file (image/jpeg)."><details><detail>filetype-mime-mismatch</detail><detail>John's_National_Academy_of_Health_Sciences</detail><detail>image/jpeg</detail></details><docref xml:space="preserve">See https://commons.wikimedia.org/w/api.php for API usage. Subscribe to the mediawiki-api-announce mailing list at &lt;https://lists.wikimedia.org/mailman/listinfo/mediawiki-api-announce&gt; for notice of API deprecations and breaking changes.</docref></error></api>

I upload a new image and it uploads successfully.

This works for me!!!!! Well done @maskaravivek , you have solved a bug that has stumped us for over a month!! 🥇 👏 👍
I experienced a couple of minor issues while testing:

I think this needs to be rebased on 2.8-release and submitted to 2.8-release. I noticed some bugs with new changes on master (the desc language selector) while testing it
While the new uploads succeed, I don't get the usual upload progress bar in ContributionsActivity. It just says "Queued" until it succeeds, even if there is no other upload ongoing. However this might be due to a bug on master, so it would be worth doing the rebase first and then seeing if it persists

Codecov Report

Merging #1790 into 2.8-release will decrease coverage by 0.07%.
The diff coverage is 0%.

@@              Coverage Diff               @@
##           2.8-release   #1790      +/-   ##
==============================================
- Coverage         3.65%   3.57%   -0.08%
==============================================
  Files              186     190       +4
  Lines             9314    9501     +187
  Branches           823     839      +16
==============================================
  Hits               340     340
- Misses            8951    9138     +187
  Partials            23      23

Impacted Files
Coverage Δ

...r/free/nrw/commons/contributions/Contribution.java
0% <ø> (ø)
:arrow_up:

...va/fr/free/nrw/commons/explore/SearchActivity.java
0% <0%> (ø)
:arrow_up:

.../free/nrw/commons/upload/SingleUploadFragment.java
0% <0%> (ø)
:arrow_up:

...java/fr/free/nrw/commons/upload/ShareActivity.java
0% <0%> (ø)
:arrow_up:

...main/java/fr/free/nrw/commons/upload/Language.java
0% <0%> (ø)

...n/java/fr/free/nrw/commons/upload/Description.java
0% <0%> (ø)

...r/free/nrw/commons/upload/DescriptionsAdapter.java
0% <0%> (ø)

...nrw/commons/upload/MultipleUploadListFragment.java
0% <0%> (ø)
:arrow_up:

...c/main/java/fr/free/nrw/commons/AboutActivity.java
0% <0%> (ø)
:arrow_up:

...fr/free/nrw/commons/media/MediaDetailFragment.java
0% <0%> (ø)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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

Thanks for the fixes, @maskaravivek . I tested this again on a separate emulator, and the progress bar is fixed now. :) I am unable to test on the emulator with failed uploads due to the rebase - it requires an uninstall due to the recent contentprovider change, and I can't reproduce failed uploads if I uninstall.
So, I am in favour of merging this once the Travis build completes. :) Then we can release this to beta, and see if it fixes the issue for the users who are experiencing failed uploads.
If I'm not around when Travis completes, anyone please feel free to squash and merge.

@misaochan The build was failing because of a test failure.

Merging this as both checks are now passing. :)


Publicsuffix list

wget - increase pkgrel
Why you remove libnghttp2-devel from dependencies ?

It was a mistake that I fixed.


Add to show number of recent searches to input field

Create component which

show last recent (3-5-10?) searches -- could be set in preferences #942 #1160
would suggest people what they would like to search -- show recent top 3 BUYs - small preview name of the collection

Inpisration from gmail

I'd like to take this one

Please follow good code patterns from #1308

Blocked until #1308 is resolved

I'm now resuming work for this. Does it mean I have to also work on #942  and #1160 ? @yangwao

Blocked until #1308 is resolved

Should be good now, you can start adding stuff in store/preferences.ts

#349 is probably related!

@pandasamanvaya has something working on recently, if you can join forces with @rhaicode that would be amazing!

I added the images and gifs to the left of each nft. It works for most of the gifs("Appul" nfts)

But throws an error while reading animation_url of few gifs("Spaceship" nfts). So an empty image is rendered. Any ideas?

Btw @rhaicode, if you could comment your discord handle here then we could use discord for better communication.

Already done


Diversity dashboard

As an admin I need to be able to see diversity data for the close of an exercise and at every stage of shortlisting. This is a statutory requirement.

This needs to have the categories suggested below by the statistics team and shown in the mock up which Claire has made on Balsamiq (below).

The information needs to be accessed on the list of options on the left at this link https://admin.judicialappointments.digital/exercises/AaWKfpkKdSWOQryCgn1T/. This should have Overview with exercise options under. It then has Applications in the same style as Overview. And then it should have 'Exercise Reports'. This will fall under Exercise Reports as 'Diversity'. Eventually there will be further reports.

There will need to be the option to link back to Exercise Reports from Diversity and to Overview from Exercise Reports.

An example of what is required below provided by statistics:

The aim is for this to be provided in a dashboard - see Claire diagram below:
https://balsamiq.cloud/s156m8n/pe2ips8/r2278

For discussion.  We need to identify what is doable before close of applications on 4 February.

Best case scenario is a dashboard like the balsamiq that admin can easily copy from and distribute.
If a dashboard is not possible a CSV download which shows admin the answers from each candidate and calculates the numbers and percentages.
If the numbers and percentages are not possible at minimum admin needs all the answers in a CSV and may need assistance with formulas to calculate numbers and percentages.

Can this task be marked as done yet?
Any remaining work can be covered by other tickets

@ClaireTroughtonJAC


Open trade offer limit?

Is there a limit to the number of open trade offers you can have? I seem to be getting errors at about 5 or 6 open trades, but they are all to one person - Is it different if they are all to others?

The limit is 5 per person and 25 total if I'm not mistaken.


fix vault image

see deprecation notice in https://github.com/hashicorp/docker-vault:
Users of Docker images should pull from hashicorp/vault instead of vault

Thanks for providing the update!


PersistentVolumeClaims from the statefull set Template remain after CR removal

Describe the bug
The operator does not clean up PVCs
To Reproduce
Deploy Cr with persistence enabled, undeploy and peek at the PVCs, they will still be bound
Expected behaviour
Ideally, the cluster would be able to garbage collect the PVC
Additional context
If we set an owner reference to the PVC in the template, to the CR, then on undeploy, of the CR, the PVC can be removed

I see that there is a feature gate option that does what is needed here...
that needs to be explored a little more.
https://kubernetes.io/blog/2021/12/16/kubernetes-1-23-statefulset-pvc-auto-deletion/

the feature gate is more nuanced around the stateful set, when this goes mainstream we can adopt. Short term, setting an owner reference ensures that there is cleanup on undeploy.


New Pre-Loader

Reference
The color should be consistent with the theme of the website

Hey I can try working on this, please assign this issue to me!

Assigned for 3 days

https://user-images.githubusercontent.com/53107832/122638361-452a9580-d111-11eb-9542-3f6e97812d4b.mp4
Like these? (There are 2 samples in this)

No 😅
I specifically gave the atom pre loader reference

No 😅
I specifically gave the atom pre loader reference

So I'll use the same without any modifications..?
Colours?

No 😅
I specifically gave the atom pre loader reference

So I'll use the same without any modifications..?
Colours?

Modify the colors. Add a text like John Doe in the bottom, maybe play around with the electrons or the background or the axis. But keep the animation as an atom rotating about an axis

https://user-images.githubusercontent.com/53107832/122669503-5a6bf680-d1db-11eb-9c8d-31c087a061c2.mp4
The original colors weren't visible in this light background and for light mode, we need a light bg so that's why I changed the color to purple and also added a John Doe text with the same animation as "I am a ____"

the animation looks great . The only thing that I would want to change is increasing the width of the lines of these rather than changing the background.

Avoid the typewriter effect on the text. Just have it there without any animation


Fix typing extensions version issue for python 3.8 and 3.9

Have issues with poetry complaining about the version collision with other libraries. This seems to fix the typing version issue though haven't tested it locally as I'm using python 3.10 and relying on CI here and this changelog.

hey, there is already a pull request for it #203

#214 contains required updates, it has been already merged into master, release should be soon


⚠ Jenkins Plugins Artifactory has degraded performance

In d4ef3f7, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 2180 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 1e87729 after 24 minutes.


Improving the existing screenshot of How to Fragment

[x] Adding slides for Saved APKs, My Drafts.

[x] First screenshot is different form what actually is.

[x] Adjusting brightness of existing screenshot with background.

[x] Highlighting parts of the screenshot which are necessary. Like .. highlighting buttons with mentioned usages.

[x] Replacing screenshot with newly added four templates, this will let the user know what are the newly added features in our current releases.

@codingblazer, which site should(#262) would be better to decrease the sizes of these drawables ?

@opticod From what I know, pngcrush.com is better.

@codingblazer, Thanks. I experimented with both of it, and tinyPng comes out to be better. It reduced upto 75% image size and saved total 1MB. :open_mouth:


Process finished with exit code -1073741795 (0xC000001D)

Process finished with exit code -1073741795 (0xC000001D) appears after I try to import class 'Model' with
from pyllamacpp.model import Model.
I've tried to downgrade lib version but nothing changed.
I use Windows 10, python version 3.11.0

@AtomMaxTM, I honestly have no idea where this error should come from but it seems like there is something wrong with your MVCC++ compilers,
If you are on windows I suggest just to use WSL, it is better and hassle-free.


Adding perf test for workflows

Description

Issue reference

Please reference the issue this PR will close: #6749
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok--to-perf

/ok-to-perf

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-test workflows

/ok-to-test workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows

/ok-to-perf workflows


Prevent starting new event frame on element with existing open event frame of the same template

So, turns out this can actually occur :) Thanks @kkotaich for pointing this out. @jmbeisner and @bfaivre decided that we'd look to add a "allow multiple" checkbox to the event frame template to allow or deny this. For example, a brewhouse can have multiple brew event frames active.


js beta release minus the enforcement rule

null
Microsoft Reviewers: Open in CodeFlow

Could you write an excerpt of what does this PR do?


(logo) Adding animated Choco Logo

@RichiCoder1 would you have a minute to take a look at this?

When I run it initially, I get this:

However, when I maximize the window, it balloons to this:

And if I make the window smaller, it seems to go off the screen.
I am not exactly clear on what is causing the size/position to change.  Can you offer any advice here?

@punker76 while I have your attention...
Don't suppose you would have any input for this one would you?
Thanks in advance!

@gep13 Would that be okay? Or should it be more left?

@gep13 ...

@gep13 another one...

I prefer 2nd and this one.

Ideally, the upper left corner of the icon would always be off screen, i.e. as though the chocolate bar is off screen.  See the icon at the top left here:
https://chocolatey.org/
See how you never see the top edge of the bar?

So, if you could take the third one above, and bump it vertically about 15/20 pixels, I think it would be perfect 😄

@punker76 does the changes that you have made work if you resize the main application?  i.e. the image doesn't expand, or change position?

@gep13 it keeps on resizing and moving :-D
finally

PR is on the way

@punker76 Sweet!  Thank you so much for your help with this!

@punker76 very curious to see how you have went about doing it 😄

Closing this, in favour of this one: https://github.com/chocolatey/ChocolateyGUI/pull/476

Nope, that isn't going to work :-)
That PR is based on top of this one :-)

W/ @punker76's PR, lgtm.


OOPDEF-174: Form submission.

[x] After these changes, the app was run and still works as expected
[x] Tests for these changes were added (if applicable)
[x] All existing unit tests were run and still pass

Approved. Nice work, Harry.


TlsServerAuthorizationCheckConfig may cause SIGSEGV

this is the code which causes SIGSEGV
static std::shared_ptr<Channel> makeChannel() {
    auto tls_key_cfg_sp = std::make_shared<TlsKeyMaterialsConfig>();
    // set key and certs...
    auto tls_sac_sp = std::make_shared<TlsServerAuthorizationCheckConfig>(
        std::make_shared<MyServerAuthorizationCheck());
    auto opts = grpc::experimental::TlsCredentialsOptions{
        grpc_tls_server_verification_option::GRPC_TLS_SKIP_ALL_SERVER_VERIFICATION,
        tls_key_cfg_sp, nullptr, tls_sac_sp
    };
    auto credentials = grpc::experimental::TlsCredentials(opts);
    ChannelArguments arguments;
    arguments.SetSslTargetNameOverride("...");
    return grpc::CreateCustomChannel("...", credentials, arguments);
}

class MyClient {
    explicit TransMsgClient():stub_(MyMsgSrv::NewStub(makeChannel())){}
   // some methods with stub_
private:
    std::unique_ptr<MyMsgSrv::Stub> stub_;
};

auto cl = MyClient();
// do somthing with cl

this is how I fix it:
make  std::shared_ptr has longer lifetime than Channel
static std::shared_ptr<TlsServerAuthorizationCheckConfig> tls_sac_sp;
// init then use it in makeChannel

i do NOT think it's nesscessary to keep a shared_ptr of TlsServerAuthorizationCheckConfig.
it's a bug.
std::shared_ptr should NOT be moved.

@exiledkingcc Thanks for reporting this bug! I will be working on this in the next couple of days.
For what it's worth, we are in the middle of re-design and re-implement the whole TlsCredentials API. In the new API, TlsKeyMaterialsConfig is removed, and TlsServerAuthorizationCheckConfig might change slightly. You can find more information here: https://github.com/grpc/proposal/pull/205

This is still a problem with the new API in 1.38.x

The fundamental problem is that grpc_tls_server_authorization_check_config has a context_ pointer to its "owning" TlsServerAuthorizationCheckConfig which is stale when the TlsServerAuthorizationCheckConfig object is freed (eg., when it goes out of scope).
The access violation happens when that context_ is dereferenced in TlsServerAuthorizationCheckConfigCSchedule  [called from TlsChannelSecurityConnector::check_peer -> grpc_tls_server_authorization_check_config::Schedule()].
Once a channel is created, nothing preserves the c++ object chain-of-ownership from TlsCredentials -> TlsCredentialsOptions -> TlsServerAuthorizationCheckConfig.

This issue is actually the same issue that was reported here https://github.com/grpc/grpc/issues/22287 but never responded to.

Yeah, we are aware of this issue and wanted to get rid of it as soon as possible.
The whole custom verification API is under re-design and re-implementation in https://github.com/grpc/grpc/pull/25631. Once that is merged, all API related to TlsServerAuthorizationCheckConfig and TlsServerAuthorizationCheckConfigCSchedule will be replaced with a new verifier-based API.
@yihuazhang FYI.

@exiledkingcc @peterhurley-meraki I see that 25631 was merged a while ago, is the issue fixed?

Closing since the code in question has been deleted.


Investigate/add auto-merging for dependabot updates

We use 'dependabot' to propose dependency updates for egeria.
We do need to review each one for applicability  and identify any risks/testing required. However once this is done and we wish to merge it can take significant time - especially at the beginning of a release cycle - to iteratively rebase, merge - each iteration taking 30 mins or more.
Additionally these merges can impact other team members trying to merge other code fixes.
It would be helpful if, once reviewed/approved, the dependabot PRs could be identified as 'ready', and then queued, to be merged automatically in off-peak hours.
There are a variety of github actions which appear to support this approach - for example based on labels.

Depenabot has this feature built in - both automatic merging, or the ability to auto-merge once all approvals are complete. I've enabled this latter option. Note that I did need to reenable dependabot's ability to keep PRs up to date. The hope is we now can get away from having a large dependabot backlog and keep it small.
If we have no intention to merge a dependency update we should use the dependabot commands to defer it until the next release and/or raise an issue manually to track. Examples might include for lucene


reimplements the entire photo-registry use case

reimplements the entire photo-registry use case
WHAT DOES THIS PR DO?

[x] Addresses this issue-590.
[x] Also refactors the photo-registry use case by introducing new handy endpoints to be used by all other distribution channels.

WHAT JIRA STORY/TASK IS RELATED TO THIS PR?

README, dev or stage ENV.
WHICH ENDPOINTS SHOULD BE READY FOR TESTING?:

[x] view photos
[x] Soft create photo
[x] Soft update photo
[x] Soft delete Photo
[x] Delete on Cloudinary
[x] Delete photo

IS THERE ANY JENKINS CONSOLE LINK FOR CODE COVERAGE AND BUILD INFO?
N/A
ARE THERE ANY RELATED PRs?
N/A

Just realised that they are deleted from platform but not on Cloudinary. So changing the label to work-in-progress. @ja-odur

I get this error when I try the GET photos endpoint.
http://<IP_ADDRESS>:3000/api/v1/devices/photos?tenant=airqo
I think this happens when there are no photos

This has been resolved, could not reproduce it but knew about it. So I have just removed the utilisation of the jsonify util. @ja-odur

Adding some descriptions for the soft create and other endpoints in the docs will be helpful. details about some of the required body parameters such as metadata, image_code, image_url etc.
Why are we only creating the images on platform (usecase)? @Baalmart
Thanks for the review @ja-odur

The usecase for creating the image on platform only is where you first create the image on cloudinary from the frontend and then just send the cloudinary URL. Since the more efficient and powerful option is to allow our users to upload images and videos in our client-side code directly from the browser to Cloudinary instead of going through your servers. So this means that you can continue to create images from the frontend and just send the resulting url to the backend accordingly.
And now the meaning of the fields, I will update the documentation accordingly. In summary:

metadata: this is the one which stores the fields from the response of cloudinary. More details,
image_code: this is the public_id from cloudinary's response,
image_url:  this is the  secure_url or url from the cloudinary's response.


Force monotonic clock when starting async-profiler

We had a user report that the inferred-spans feature fails to generate spans even though it should.
After analyzing the diagnostic files, turns out that the profiling data was using a different clock than System.nanoTime, which is required for correlation.
Async-profiler 3.0 by default tries to use TSC only falls back to System.nanoTime if TSC is not available on the current CPU/System. The clock=m startup argument should force the monotonic clock (=System.nanoTime) to be always used.

User confirmed that this change fixes the problem.


Query::batch - memory limit

Hello, I need to iterate 200k record table, but when i use  foreach (Item::find()->each() as $item) it fails with memory error.  Loop doesn't execute at all.
I found only one topic about this with no solution.
It works only on development environment (mac, mamp 3.0.6, 128 mb memory limit), but on deployment servers  it throws an error.
Could you suggest where should i search the difference?

PHP version.
PDO version.
PDO settings about emulating prepared statements.

http://yiiframework.ru/forum/viewtopic.php?f=19&t=31157

https://github.com/yiisoft/yii2/issues/8420

I have the exact same problem and already tried the suggestion in #8420 (PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => true) but it didn't help. My code is basically this:
$query = (new \yii\db\Query())
    ->select('*')
    ->from('bigtable')
    ->where('active=1');

foreach ($query->batch(200) as $batch) {
    // this line is never excueted due to a memory exhaustion exception
}

'yii\base\ErrorException' with message 'Allowed memory size of 134217728 bytes exhausted (tried to allocate 32 bytes)' in /var/www/vendor/yiisoft/yii2/db/Command.php:837

@samdark Your above link goes to a russian site, so not sure, if I missed some essential information. Anything else we could try? It seems like batch() doesn't really behave as promised by default.

Is debug turned off?

No, should it be?

Yes beause Yii keeps debug stuff in memory and it grows.

I usually disable all log targets when processing large amounts of data in dev environment. Logs get spammed with query profiling info and I couldn't figure out how to force them to flush.

I've tried with YII_DEBUG==false and YII_ENV = 'prod'. Still got the same error. I also tried both, console and web application. It happens in both.
@samdark BTW your tipp should still be documented somewhere. It's not really obvious.
@nineinchnick Also tried that, didn't help.

I'm out of ideas.

@samdark Sorry, I've confused my DB connections (have 2 in my app). The above PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => false really helps. But now i got this:

ERROR: SQLSTATE[HY000]: General error: 2014 Cannot execute queries while other unbuffered queries are active.  Consider using PDOStatement::fetchAll().  Alternatively, if your code is only ever going to run against mysql, you may enable query buffering by setting the PDO::MYSQL_ATTR_USE_BUFFERED_QUERY attribute.

I know, this is probably a limitation of MySQL PDO. Any help is still welcome as this basically makes moving data from one DB to another impossible if you have large datasets.

Is the error ocurring in the first batch? If yes, try to lower them. If not, maybe it's about #9129?

Are you trying to run multiple queries?

I select a batch of rows through one DB connection then call Command::batchInsert() to insert them into another DB. I didn't expect that the above would happen even if you have 2 different database connections. And the second statement is actually not a query, but an insert statement.

OK, I've figured it out. Actually there really was another query: I have some code that creates my second DB connection for me. The configuration for that connection is also stored in the primary DB. I've now made sure, that the connection is created before the actual data migration.
Everything seems to work fine now.

Sorry for late answer, but for my case updating of php and pdo modules resolved this issue. I didn't change any params.

unable to reproduce the issue, see #11152

As a note: this will happen all the time on the default install of PHP7 now due to how mysqlnd now works by adding the result set's memory usage to the process's own:
http://php.net/manual/en/mysqlinfo.concepts.buffering.php

When using libmysqlclient as library PHP's memory limit won't count the memory used for result sets unless the data is fetched into PHP variables. With mysqlnd the memory accounted for will include the full result set.

And mysqlnd is the recommended lib for the PDO extension now, while libmysqlclient is now "not recommended"

https://github.com/yiisoft/yii2/pull/11393#issuecomment-253957000
@cebe  @aivanouski any update？will this be fixed at  milestone: 2.0.11, 2.0.10

In the mean time I made this https://github.com/Sammaye/yii2-pq

As an additional, unbuffered queries suck

as reference for me: https://github.com/yiisoft/yii2/issues/12637#issuecomment-250701260

Duplicates #8420

About Sammaye’s class, method each( $batchSize = 100, $db = null )
now becomes each($batchSize = 100, $page = true, $db = null)

This problem may still appear even on latest Yii2 version.
It's not the framework problem it's in your DB connection configuration.
I've solved this by adding PDO param to my connection config.
'attributes' => [ PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => false ]
So after it became
'db'=> [
    'class'=> 'yii\db\Connection',
    'dsn'=> 'mysql:host=<IP_ADDRESS>;port=3306;dbname=DBNAME',
    'username'=> 'root',
    'password'=> 'root',
    'charset'=> 'utf8',
    'attributes'=> [
        PDO::MYSQL_ATTR_USE_BUFFERED_QUERY=> false
    ]
]

Memory limit error has gone.

I don't use Yii anymore but as said in my repo:
"Added to that, my own observations that unbuffered queries suck meant that I created this."
Honestly unbuffered queries are not good, they cannot even be used like normal queries.

Just git the same.
And trick with:
'attributes' => [ PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => false ]
Did not helped.
What I found:
when I doing $query->batch(10);
In MySQL logs I see one signle request which fetching all data.
No mention of OFFSET or LIMIT.
Which leads on 3M table to a memory limit issue.

@andrey-denisenko that means buffered query was used.

@andrey-denisenko that means buffered query was used.
Well, maybe.
I did few attemtps to avoid buffering. But they ended with nothing.
More over, I'm not allowed to perform such changes over PDO configuration, cause base docker image are used in many other projects. So it just not possible to predict consequences from that.

So I just adopted @Sammaye code, which uses OFFSET and LIMIT to perform queries.
And frankly speaking, BatchQueryResult must do the same.
Querying of huge amount of data by small portions, is the main expectation from that class.
(Not like only when your PDO configuration is compatible)

Memory limits in batch can be also connected with transactions.
I had the same problem when using the safeUp method in migration. Because it uses transactions.
For me the up() method has sold the problem.


Update customize-the-app-launcher.md

Changed the metadata author

@kwekuako - PR is ready for final review. This is just to update the metadata author details.
Thank you.
cc: @kenwith


Add charts to Masters/Templates?

Hi,
Can I define charts in my masters? If yes, how (do you have a sample please)?
Also if I understood correctly it's not possible to use existing PPTX as template but do you think a tool/function could be created that extract the masters from an existing PPTX file?
Thanks a lot - your plugin is way easier than other I tried.

Hi @yipiha ,
Thanks for suggesting a new feature.  You can now add charts to master slides as well as images, etc.  Try it out and let me know what you think.
TITLE_SLIDE: {
  title:  'Corporate Presentation Title Slide',
  objects: [
    { 'line':  { x: 3.5, y:1.00, w:6.00, line:'0088CC', line_size:5 } },
    { 'chart': { type:'PIE', data:[{labels:['R','G','B'], values:[10,10,5]}], opts:{x:0.25, y:0.25, w:3, h:3} } },
    { 'rect':  { x: 0.0, y:5.30, w:'100%', h:0.75, fill:'F1F1F1' } },
    { 'image': { x:11.3, y:6.40, w:1.67, h:0.75, data:pptxgenjsLogoSml } }
  ]
},


Placeholder modification behavior

I am using Tensorflow 1.0.1, on Mac OS Sierra 10.12.4, with CPU in a jupyter notebook (but the behavior I am describing appeared in a regular python script too).
The problem I am having is with placeholders. I have been using tensorflow for only 1 month so I was not really comfortable with using them and I made a mistake that took me a very long time to debug.
Here is the type of code I wrote:
import numpy as np
import tensorflow as tf
x_ = tf.placeholder(tf.float64, [5])
x_ = tf.where(tf.is_nan(x_), tf.zeros_like(x_), x_)
a = tf.constant(1, shape=[5])
a = tf.cast(a, tf.float64)
c = a + x_
init_op = tf.group(
    tf.global_variables_initializer(),
    tf.local_variables_initializer())
sess = tf.Session()
sess.run(init_op)
x = np.arange(5, dtype=float)
x[3] = np.nan
sess.run(c, feed_dict={x_: x})

To this, the output was: array([  1.,   2.,   3.,  nan,   5.]), which is not what I expected since I had replaced all the nan with the tf.where. I understood only later that it was because tensorflow was feeding the placeholder in the latest position it appeared (maybe not exactly how it's implemented, but empirically, that is what happens).
Therefore a good (in the sense of what I intended to do) code is:
import numpy as np
import tensorflow as tf
x_ = tf.placeholder(tf.float64, [5])
r = x_
r = tf.where(tf.is_nan(r), tf.zeros_like(r), r)
a = tf.constant(1, shape=[5])
a = tf.cast(a, tf.float64)
c = a + r
init_op = tf.group(
    tf.global_variables_initializer(),
    tf.local_variables_initializer())
sess = tf.Session()
sess.run(init_op)
x = np.arange(5, dtype=float)
x[3] = np.nan
sess.run(c, feed_dict={x_: x})

This indeed outputs array([ 1.,  2.,  3.,  1.,  5.]). However, to me it's a bit weird that you would need to throw in an extra variable to achieve this purpose and even weirder that I didn't receive any error when modifying my placeholder.
My suggestions are:

Throw an error or a warning when reallocating a placeholder.
State clearly that placeholders are not meant to be modified (I looked for it in the docs and didn't find it, maybe I didn't look well enough).
Allow placeholders to be modified (might be offputting for someone who understands it well).

Hi @zaccharieramzi, I would argue this is just how Python works. In your code, you're not actually reallocating or modifying your original placeholder, but reassigning the x_ variable, and then feeding the new x_ value (which is the where op):
x_ = tf.placeholder(tf.float64, [5])
x_ = tf.where(tf.is_nan(x_), tf.zeros_like(x_), x_)
...
sess.run(c, feed_dict=x_: x)

This is analogous to doing something like:
x_ = 1
x_ = x_ * 2
print(x_) # prints 2

In general, I think it's not a bad idea to use a new variable for each TensorFlow op to avoid bugs like these. Note you can write it in the same number of lines:
x_ = tf.placeholder(tf.float64, [5])
r = tf.where(tf.is_nan(x_), tf.zeros_like(x_), x_)

I'm gonna close this for now as I don't think it's a TensorFlow bug, but feel free to comment further.

I agree with your explanation of what I am doing (wrong choice of words on my end).
I also agree with your analogy. However, it's because it's analogous that I made this mistake.
I am not saying that it's a bug, but I think it can be a bit puzzling the first time you look at it, and that is why I was suggesting to throw a warning when reassigning a placeholder because it is going to lead to an unwanted behavior.
I also agree with your general comment on naming ops.

True, I agree it's not really a bug per-se. Even so, I don't think we should change the default assignment operator to print a warning (in fact, I don't think it's even possible to do so in Python). Thank you for reporting though, hopefully if others run into the same problem they will find this thread.


Added IPP support

Thanks @harrypale for submitting this PR! The changes look good to me. Could you please add a README.md file to the tutorial and briefly explain the command to fuzz IPPSample? It would be good to add a sentence to introduce the protocol and the sample server under test. Since you have provided a Dockerfile, the set up instructions are not needed.

Hi, I added the README.md file to the tutorial

Thank you very much for your contribution. I followed the instructions and successfully run an experiment. The PR has been merged!


Problem with GOOD/THANK-YOU (sorry)

There is one case where we changed (2h)THANK-YOU to THANK-YOU in a compound, but where the old ones were not deleted.  So there is a duplication.  The (2h)THANK-YOU+...
need to be deleted.

Fixed

thank you!


Continuation of Rebuild

closes #7
Moved to a newer version.  This depends on https://github.com/conda-forge/nettle-feedstock/pull/4

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Waiting on https://github.com/conda-forge/automake-feedstock/pull/18 now

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender

toolchain linux is back to working,

Seems to be failing on tests involving pkg-config :(.  I thought i had patched those out

@mariusvniekerk - Ahh I see

@conda-forge/core, does anyone have any ideas here?

Why is it complaining about weak_imports when the test finds them:
https://travis-ci.org/conda-forge/gnutls-feedstock/jobs/469067183#L592

I can't push, but I could PR. but that seemed more complicated than you guys using the online editor.

Can you issue a PR to Marius's fork?

@scopatz https://github.com/mariusvniekerk/gnutls-feedstock/pull/1

Thanks! Let's try this!

i  may or may not have removed a space from the patch in my editor. lets see what happens....

Well now it failed with
=============================================
   GnuTLS 3.6.5: tests/slow/test-suite.log
=============================================

# TOTAL: 6
# PASS:  5
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: test-ciphers-api.sh
=========================

trying aes128-gcm
trying aes256-gcm
trying aes128-cbc
trying aes256-cbc
trying 3des-cbc
_check_wait_status:153: Child died with signal 11
default cipher tests failed
FAIL test-ciphers-api.sh (exit status: 1)

probably because it took too long to output something. On my own computer these tests took a LONG time. can we skip them?
mac is still failing..... so who knows.

This explains a likely cause for the early failure on OSX
https://apple.stackexchange.com/a/249741

Well I patched out no_weak_imports. https://github.com/mariusvniekerk/gnutls-feedstock/pull/2
Now somebody has to disable more tests becuase they are too slow.

It seems like the -no_weak_imports flag is still being passed.

@jakirkham i know. i patched it out here. https://github.com/conda-forge/gnutls-feedstock/pull/10
and made the pr to marius' branch https://github.com/mariusvniekerk/gnutls-feedstock/pull/2
i think the failure is just a hiccup in the build system


sass-binary-dir does not use node-sass version

For systems without free internet access, we need to provide alternative ways to find the "sass-binary".
When specifying a proxy site using sass-binary-site, it will append the url with a path segment "v + node-sass version" (eg v4.9.2) and then a file with a specific name, eg win32-ia32-64_binary.node.
When specifying a (local) directory using sass-binary-dir, it will look for a directory named win32-ia32-64, and then simply a file named binding.node. The version of node-sass is not used here, which means I need to update my servers manually every single time the node-sass version changes! And if I want to build separate projects that use slightly different node-sass versions, I can't use the same server.
Consistent behaviour!
I think sass-binary-site and sass-binary-dir should behave similarly, just with one providing a website and the other a directory.
Ideally, I'd like to be able to configure more about this process (for instance, not have the directory structure but use the version in the filename), but at the very least behaviour should be consistent.
I've tested this on two systems.
Info for Windows 10 system

NPM version (npm -v):
5.6.0
Node version (node -v):
v8.11.2
Node Process (node -p process.versions):

{ http_parser: '2.8.0',
 node: '8.11.2',
 v8: '6.2.414.54',
 uv: '1.19.1',
 zlib: '1.2.11',
 ares: '1.10.1-DEV',
 modules: '57',
 nghttp2: '1.29.0',
 napi: '3',
 openssl: '1.0.2o',
 icu: '60.1',
 unicode: '10.0',
 cldr: '32.0',
 tz: '2017c' }

Node Platform (node -p process.platform):
win32
Node architecture (node -p process.arch):
ia32
node-sass version (node -p "require('node-sass').info"):
4.9.2
npm node-sass versions (npm ls node-sass):

<EMAIL_ADDRESS>/project-path
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
Info for Mac OS Mojave system

NPM version (npm -v):
6.4.1
Node version (node -v):
v10.15.0
Node Process (node -p process.versions):

{ http_parser: '2.8.0',
  node: '10.15.0',
  v8: '6.8.275.32-node.45',
  uv: '1.23.2',
  zlib: '1.2.11',
  ares: '1.15.0',
  modules: '64',
  nghttp2: '1.34.0',
  napi: '3',
  openssl: '1.1.0j',
  icu: '62.1',
  unicode: '11.0',
  cldr: '33.1',
  tz: '2018e' }

Node Platform (node -p process.platform):
darwin
Node architecture (node -p process.arch):
x64
node-sass version (node -p "require('node-sass').info"):
4.9.2
npm node-sass versions (npm ls node-sass):

<EMAIL_ADDRESS>/project-path
└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>

I tried to make this work through sass-binary-site and file:/// URI to no avail, so it'd be nice if sass-binary-dir offered the version disambiguation.
[INFO] verbose 43.273 Downloading binary from file://localhost/opt/build-agent/.npm/node-sass/v4.11.0/linux-x64-57_binding.node
[INFO] Cannot download "file://localhost/opt/build-agent/.npm/node-sass/v4.11.0/linux-x64-57_binding.node":
[INFO]
[INFO] getaddrinfo ENOTFOUND http http:80


Profile

Implement option to add profile informations

[x] About
[x] Website
[x] Social Media
[x] E-Mail address

Assign @NathanNr

I add these features

Finished.


Can't find sign in button

Tried to sign in after latest update but I can't seem to find the sign in button

Should be fixed in 2.1.3


Telepresence 0.67 on Arch Linux

What were you trying to do?
I'm running Arch Linux, but there is "official" telepresence installer.
I installed telepresence 0.65-7 via AUR
exact version for referene
I already have minikube v0.22.0 running. I can kubectl get pods and I see my deployed pods.
Finally I ran telepresence which failed with following traceback.
So I forked the AUR PKGBUILD for telepresence and upgraded it to 0.67:
https://github.com/dhensen/telepresence-pkgbuild/commit/f2d6d4f6b72dc27404640e9a13fbd249d0de5cc3
(I'm a PKGBUILD noob)
With that PKGBUILD I built an installable package; installed it via arch's pkg-manager pacman and it installed successfully.
Then same thing happened again.
What did you expect to happen?
Have telepresence present me with a shell or a usage page.
What happened instead?
See traceback for what happened. I'm (limited) available to help make this work on Arch Linux if you need extra hands.
Automatically included information
Command line: ['/usr/bin/telepresence']
Version: 0.67
Python version: 3.6.2 (default, Jul 20 2017, 03:52:27)  [GCC 7.1.1 20170630]
kubectl version: Client Version: v1.6.6
oc version: (error: [Errno 2] No such file or directory: 'oc')
OS: Linux liekaliek 4.12.12-1-ARCH #1 SMP PREEMPT Sun Sep 10 09:41:14 CEST 2017 x86_64 GNU/Linux
Traceback:
Traceback (most recent call last):
  File "/usr/bin/telepresence", line 257, in call_f
    return f(*args, **kwargs)
  File "/usr/bin/telepresence", line 2363, in go
    runner, remote_info, args, env, subprocesses, socks_port, ssh
  File "/usr/bin/telepresence", line 2105, in run_local_command
    connect_sshuttle(runner, remote_info, args, subprocesses, env, ssh)
  File "/usr/bin/telepresence", line 1897, in connect_sshuttle
    get_hellotelepresence()
  File "/usr/bin/telepresence", line 1884, in get_hellotelepresence
    format(next(counter))
  File "/usr/bin/telepresence", line 395, in get_output
    result = unicode(check_output(*args, **kwargs).strip(), "utf-8")
  File "/usr/lib/python3.6/subprocess.py", line 336, in check_output
    **kwargs).stdout
  File "/usr/lib/python3.6/subprocess.py", line 418, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['python3', '-c', "import socket; socket.gethostbyname('hellotelepresence135')"]' returned non-zero exit status 1.

Logs:
me or service not known
  26.5 TL | [156] Capturing: (['python3', '-c', "import socket; socket.gethostbyname('hellotelepresence132')"],)...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
socket.gaierror: [Errno -2] Name or service not known
  26.7 TL | [157] Capturing: (['python3', '-c', "import socket; socket.gethostbyname('hellotelepresence133')"],)...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
socket.gaierror: [Errno -2] Name or service not known
  26.8 TL | [158] Capturing: (['python3', '-c', "import socket; socket.gethostbyname('hellotelepresence134')"],)...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
socket.gaierror: [Errno -2] Name or service not known
  27.0 TL | [159] Capturing: (['python3', '-c', "import socket; socket.gethostbyname('hellotelepresence135')"],)...
Traceback (most recent call last):
  File "<string>", line 1, in <module>
socket.gaierror: [Errno -2] Name or service not known

I just went through the telepresence.log. I think this is the underlying problem:
sudo iptables -t nat -nL
[sudo] password for dino:
iptables v1.6.1: can't initialize iptables table `nat': Table does not exist (do you need to insmod?)
Perhaps iptables or your kernel needs to be upgraded.

I'm going to figure out how to fix that and report back.

Thanks for the bug report and follow-up! Before you go very far down the rabbit hole with iptables, I have a quick request. Can you please try Telepresence with a remote cluster, i.e. not Minikube? If you see the same issue with a remote cluster, then we can eliminate one possible source of bad behavior (infinite looping of DNS lookups when using Minikube and method vpn-tcp under certain circumstances).
If you don't have easy access to a remote cluster, Kubernaut can get you time-limited access to one quickly.

Ok I figured out what the problem was. Earlier on today I did a system upgrade (arch = rolling release OS) without rebooting. That upgrade contained a kernel upgrade and that messed with some modules being available before rebooting. I tried rebooting, then starting minikube again and running telepresence I now have a shell and I can curl my http endpoint pods!
@ark3 Thanks for pointing out Kubernaut though, did not know that one yet. Also I have been down in many rabbit holes lately, so happy this did not become another trip. :+1:


IsRetweet / Retweeted propetries are not working

when I follow a user using IFiltertedStream and the action is called after the user re-tweets a tweet, the information that I get in ITweet interface does not indicate that the tweet was retweeted: IsRetweet and Retweeted properties are false and Retweets List is null.

I am aware of this. Unfortunately the public stream api does not provide the IsRetweet information. The data returned by Twitter are the data that Tweetinvi gives you.
If you want to know if the current user has effectively retweeted this tweet, you will have to authenticate with this user.


Feature Request: account mgmt of log in and sign up

hey thank you for the awesome website!
I kept losing my progress after I clean the cache for my laptop to run faster. Can we log in and store our progresses?

Hey, I facing this issue as well, would love to see this feature implemented!


Url -> URL

In the settings page, I think technically it probably should be spelled "URL".

Fixed in dev.


changing grid-column-count on breakpoint (Zurb-Foundation)

Hi everybody,
(Sorry for my bad english, i apologize in advance)
I would like to modify foundation to be able to generate a different number of columns on mobile (4 cols), tablet (6 cols) and desktop (10 cols) ... I'm sure it's possible by changing the mixin generation columns but I fails ... Someone can help me?
Thanks a lot !
Alexandre

@AlexGraphicDesign you can do this by using the breakpoint mixin. So if you were using the XY-Grid, you would do it like this:
.some-cell {
	@include breakpoint(small) {
		@include xy-cell(4);
	}
	@include breakpoint(medium) {
		@include xy-cell(6);
	}
	@include breakpoint(large) {
		@include xy-cell(10);
	}
}

Closing this out. If this doesn't answer your question, feel free to reopen. Thanks!


Fix the code license issues raised in 1.5.0 release

Description
Fix code license issues described in #15542

[x] ONNX export module
[x] FindJeMalloc
[x] /3rdparty/onnx-tensorrt/third_party/onnx/third_party/pybind11/tools/FindPythonLibsNew.cmake
[x] /src/operator/contrib/erfinv-inl.h
[x] /3rdparty/nvidia_cub/test/mersenne.h
[x] /3rdparty/onnx-tensorrt/third_party/onnx/third_party/pybind11/tools/FindEigen3.cmake
[x] /3rdparty/onnx-tensorrt/third_party/onnx/tools/protoc-gen-mypy.py
[x] /src/operator/special_functions-inl.h
[x] /3rdparty/tvm/3rdparty/rang/LICENSE

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@roywei @szha Please let me know if this looks good to you.


the command zzhack serve meet a problem

i am using in window11 pc.

Thx for ur issue, looks like a path compatibility issue, I'll fix it later

hi。
我运行zzhack serve也 遇到错误：
$ zzhack serve
⠁
use router::RootRoutes;

pub struct Page {
    pub route: RootRoutes,
    pub name: &'static str,
}

pub const PAGES: [Page; 1] = [
    Page {
        route: RootRoutes::Home,
        name: "wiki"
    }
];

✅Apply config
thread 'main' panicked at C:\Users\12799.cargo\registry\src\index.crates.io-6f17d22bba15001f\zzhack-0.1.5\src/main.rs:86:18:
called Result::unwrap() on an Err value: Error { kind: NotFound, message: "program not found" }
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
如何解决？


fix compilation with Coq trunk

This pull request represents the merge of the v8.6 branch into master branch; nothing more.
The reason why I think it make sense to do it is that the present state of the published master branch is not compilable (unless v8.6 branch is merged to it). The currently hangs here.

Hmm. The travis reports compilation failure which I do not see on my machine.
(when compiling this branch with Coq trunk).

Bas (@spitters ), what do you think about this pull request? Could it be merged?

@matejkosik, I was putting this off, as I was planning to solve the travis issue first.
Unfortunately, it is not completely clear what's up with travis. Perhaps @siddharthist knows?


Match xml regex doesn't work (0.9.4)

This doesn't work in 0.9.4
Feature: Test xml match
Scenario: Test xml match boolean regex
* def aXml =
"""
< root >
< name >Jhon< /name >
< active >false< /active >
< age >30< /age >
< /root >
"""

match aXml //active == '#regex (false|true)'

Error:
xmlMatch.feature:13 - syntax error, expected '==' for match
Was working in 0.8.0.  (I'm trying to upgrade my projects from 0.8.0 to 0.9.4
Regards,
Prashant

thanks for reporting this, fixed in the git dev branch. as a workaround, remove the space on the LHS:
* def aXml =
"""
<root>
<name>Jhon</name>
<active >false</active>
<age >30</age>
</root>
"""
* match aXml //active == '#regex (false|true)'

Thanks for the fix.

released 0.9.5


Swift 4 support?

Hi, are there any plans to add Swift 4 support for this library? Thanks.

Yes, thanks for asking. First https://github.com/rexmas/JSONValue and https://github.com/rexmas/Crust will need some updating too. I can work on that this week. I'll let you know if it takes longer than the end of the week.

Update:

JSONValueRX was easy to update to Swift 4. It was source compatible from the start, I made some nicer syntactic adjustments and bumped the major version and released an new CocoaPod.

Crust has been more challenging to deal with. Seems like existential class types don't behave exactly like subtypes to their base class types, at least with inout parameters. I'd expect NSNumber & RLMInt to be a subtype of NSNumber but it's failing where I attempt to pass the former into the later in the mapping operator (<-). You can see this below

Per the example, it does appear that doing a little bit of indirection through a local variable type checks. So I think I can maybe overload the <- operator with a NSNumber & RLM<Type> for all RLM types and get this to work appropriately.

Next issue I ran into is whether RealmSwift, which is used for testing (and by us at Remind), is compatible with Swift 4. Looks like if you build from binary it is, but our goal is to support CocoaPods without any extra steps. I asked if it's source compatible with Swift 4 here and waiting for a reply.

Finally, AutoGraph is currently reliant on Alamofire. I had assumed that Alamofire supports Swift 4 already, but I was wrong. Looks like they will for their 5.0 release, but this pr also makes it sound like version 4.5.1 is source compatible with Swift 4 anyway, so I'll give that a shot.

Things are looking mostly good it seems, I'll keep hacking away at this over the next few days. if updating via CocoaPods for Swift 4 isn't a smooth transition because of the above dependency issues I'll make a swift-4 branch separately with specific directions on how to get that working.
Once swift version support is released for CocoaPods this should be less of an issue.

Opened a pr for Swift 4 to Crust https://github.com/rexmas/Crust/pull/78

Waiting on review for https://github.com/remind101/AutoGraph/pull/71, will then update the pod to swift 4. Should happen tomorrow. There's been some source breaking changes fyi, but nothing too crazy.

releases 0.4.0 supports swift 4


Missing changes for "Add a comment" style and the Sort by button in Hitchhiker and possibly other layouts

YouTube Redux has these changes, example:

Yes, that still needs to be done. Maybe I'll get to it soon, but I can't guarantee it.

Latest 1.2.7 version fixed this, closing now


noSwiping option fails when loaded first with false as option

We use this library in our app, and it works really well, good work!
However, on one page we sometimes want to have the slider, sometimes not. When there is nothing to swipe between, we want the slider disabled. So we've used the noSwiping option, and it mostly works. But if the first data loaded has swipe and you then switch to having no swiping, you can still swipe.
I have set it up in a codepen, and the same issue occurs there. If
 state = {
    shouldNotSwipe: false,
  }

is set from the beginning, you can't toggle it to not swipe. But if you change the code and
do
  state = {
    shouldNotSwipe: true,
  }

there is no problem toggling between the states.
This is the code we run in the app:
    const swipeOptions = {
      ...this.swiperParams,
      shouldSwiperUpdate: true,
      noSwiping: (!qppsImages.length),
    };
    const swipeItems = [viewportElement, ...qppsImages].map(SwipeItem);

    return (
      <Swiper {...swipeOptions} >
        {swipeItems}
      </Swiper>);

So if the qppsImages has no length at first load, it works fine when switching between modes, but if it has a length from the beginning swiping will still be enabled in the swiper.
I have also checked the actual values passed down to Swiper, and those are correct. noSwiping is set to true, but it still swipes.
In the React inspector, it looks like this:

There are no pagination bullets, but it still swipes, though it of course doesn't switch slide, since there are no slides.

Grateful for help!

Hi @stinaq , which version are you using now ? There was a bug related with no swiping , but it should be fixed in newest version

I took the latest Swiper version in the Codepen, 4.0.6. And I copied the master branch of this repo to the Codepen, since I couldn't find a CDN of that :)
I also run the latest npm package in my app.

Make sure you did load stylesheet file. UMD build is now available, so u can try it on Codepen

Did you look at the codepen I linked? I looks like it should, so I don't think the stylesheet is missing, it's just the swiping that doesn't work sometimes, depending on what is first set on noSwiping. And I don't know what UMD build is :)

Hi @stinaq , sorry for missing your codepen u linked.  If u wanna update params when qppsImages  is changed, u should use rebuildOnUpdate = true. It will delete old Swiper instance and replace new instance with updated params. Let me know if it is still not working for u

We do that now, but it unfortunately has a huge performance impact. It just seems like a bug since it works differently depending on what the value on noSwiping is at load time.

any news ? performance is awful when using noSwiping with revuildOnUpdate

Just encountered exactly the same bug.
If swiper initialised with noSwiping: true I can switch between false and true without issues later. However if noSwiping: false is set initially, updating it to true has no effect.
rebuildOnUpdate does kinda work but performance, look and feel do become horrible so not really a viable option


[9.100.x-prod][SRVLOGIC-325] use KogitoVersion variable for org.apache.kie.sonataflow:sonataflow-quarkus-devui version

Jira: https://issues.redhat.com/browse/SRVLOGIC-325

It works for me. But I removed frontend-maven-plugin at sonataflow-quarkus-devui/pom.xml to make it compile. I think it's my local issue, not related with this PR

I have the same error locally. Is there any jira for this or should I create one?

@tomasdavidorg yeap, I think you can create an issue

Just some context related to the OSL product.
The frontend-maven-plugin is added in sonataflow-quarkus-devui/pom.xml just in midstream (product), because we need to build (productize) the Quarkus Dev UI.  The build is done via root pom.xml created on midstream repository only which in the end invokes the sonataflow-quarkus-devui modules.
For building the kn workflow binary in the product, we use Brew system and we are doing an RPM build. We have a spec file in dist-git which invokes the Kn workflow Makefile (packages/kn-plugin-workflow/Makefile). For better explanation please check Jira SRVLOGIC-325.
So if a Jira would be created, I believe it should be just for the product (SRVLOGIC) as we have those differences from midstream and upstream
(cc @mareknovotny)


(Maybe) merge regular dumb Vim completion with LSP completion with a setting

I have started using python-language-server at work for completion, and it's pretty good at suggesting things. Some of the time, however, the completion engine from Jedi has no idea what the type of my data is, but I do, and the dumb tag completion that comes with Vim will do a better job. It would be nice to have a setting for ALE which tweaks completion so that it can pull in completion suggestions both from LSP provides and also tag completion.

This is the same as #2465.


Wildcards for binary expressions

This issue is more to track and solicit feedback on a rather strange query syntax problem. We are asking for use cases where this feature might be useful to understand what the proper behavior should be.
Feature Request
Proposal: Support wildcards for binary expressions.
Current behavior: It throws an error.
Desired behavior: No idea.
Use case: No idea.
At the moment, it is possible to use a wildcard for both a function or just a raw query to obtain the values for a measurement. So if you do the following query while the measurement has the fields user and system:
SELECT count(*) FROM cpu

You will get this following after expansion:
SELECT count(system), count(user) FROM cpu

But, if you do this:
SELECT sum(*) / count(*) FROM cpu

What should you get if you get anything? Right now, we just took the easy solution of making this an error. But should the expansion form the cross product of the available variables or should the wildcard expand the expression in a way where one expression is created per variable? So should it produce this:
SELECT sum(system) / count(system), sum(system) / count(user), sum(user) / count(system), sum(user) / count(system) FROM cpu

Or this:
SELECT sum(system) / count(system), sum(user) / count(user) FROM cpu

I'm leaning towards the latter since I think it would be more useful for continuous queries, but I'm interested in what others think about this.

Also, whatever is decided, the same logic would have to apply to raw queries like SELECT * + * FROM cpu too.


[JetBrains Toolbox Recent Projects] Continuous loading when using "Search Recent Projects"

Extension
https://www.raycast.com/gdsmith/jetbrains
Description
When using the "Search Recent Projects" command, it just loads forever and doesn't show any result or errors. I've already tried changing tools and shell scripts director path but it still didn't help.
Steps To Reproduce

Install JetBrains Toolbox.
Install this extension
Try the command above

Current Behaviour
No response
Expected Behaviour
No response

@irozum can you try the debugging steps in #4483 and let me know the outputs?

Hi George,
Igor is no longer with SPC. What is the below regarding – is there an issue with the Google Chrome Extension?
Thanks,
Monish Bhambhani | Senior Product Manager
SPC Card | spccard.ca | 999 Edgeley Blvd Unit 1 | Vaughan, Ontario L4K 5Z4
From: George @.>
Date: Saturday, May 20, 2023 at 3:41 AM
To: raycast/extensions @.>
Cc: Igor Rozum @.>, Mention @.>
Subject: [External] Re: [raycast/extensions] [JetBrains Toolbox Recent Projects] Continuous loading when using "Search Recent Projects" (Issue #6680)
@irozumhttps://github.com/irozum can you try the debugging steps in #4483https://github.com/raycast/extensions/issues/4483 and let me know the outputs?
—
Reply to this email directly, view it on GitHubhttps://github.com/raycast/extensions/issues/6680#issuecomment-1555808362, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMF47BGKPM4O34253OVISG3XHBYSXANCNFSM6AAAAAAYIF5XNA.
You are receiving this because you were mentioned.Message ID: @.***>

@irozum this is a comment in a github issue. Igor raised an issue about a JetBrains plugin I wrote for Raycast. Is this still Igor's GitHub account and it's  just linked to the wrong email address or can we close this issue down?

Hi @gdsmith, yeah, the notifications were linked to a different email account.
The problem was that my PyCharm app was manually installed not through the Toolbox. But after I installed it through there, the extension started to work.
Thanks for the help.

@raycastbot close this issue


OSError (OS Error: Too many open files, errno = 24)

I'm running a new template Flutter application and using the example/port_scan.dart code but when I run it on my iOS simulator I'm getting the error OSError (OS Error: Too many open files, errno = 24)
I am able to run the host and mdns examples fine. So I'm not sure what I'm doing wrong here?

Try providing a shorter port range.

I tried limiting it from 1 to 100 and it did the same?

Internet says ~250 files can be opened at once in iOS byt depends from device to device. What are your views @guyluz11 ?

@helenburns I would love to have more information.
What iOS version is on the emulator.
Did it worked on a real iOS device?.
Does the log contain more information then what you copied?, for example on what file and line did the code crash?.

I'm also getting the same error while discovering the entire subnet (from .1 to .254) on a single port only. Running on a iPhone 13 Pro with iOS 15.5 (latest). I suppose that every device will be discovered in a separate process thus opening a new file, just a guess tho :)
I can't provide the entire error log right now I hope the picture is sufficient:

Hotfix (for @helenburns)
It can be fixed by manually iteration over the subnet and checking whether the port is active for every device. Makes it way slower tho (even withthe very short timeout) since it's run iteratively. Example:
for (int i = 1; i < 255; i++) {
  var port = PortScanner.isOpen(
    "$subnet.$i",
    PORT,
    timeout: const Duration(milliseconds: 100),
  );

  // Check the OpenPort-object and do your custom stuff ...
}

Sorry, bad wording from myself:
A process in this case would be a separate, parallel socket connection to each device in the entire subnet. And since Socket connections are handled as files, we get the explained error message

@helenburns Check dev branch code if the issue is fixed now then close this.


I saw it today. with ansible/ansible.

@schneems @nateberkopec seems like all three repos referenced here have languages set now. Do you think it's still an issue? If so, perhaps the next step would be to add logging of some sort?

We should periodically check and update this information with to the information on GH api since #1722
Can we close this issue?


result items are no longer highlighted on hover

On previous version hovering over a result item gave it a slight interaction animation setting a background color, this no longer seems to be the case in the new itemfilter version

You're right, this has been removed by mistake when changing the highlighted state.


Currently can't prevent bot spam.

Spam control should be implemented especially for commands like .help, who deal with multiple lines.

you should just have a timeout on .help since it takes no args and prints one thing, as long as .help is in the last like 40 lines noone needs it again

.help doesn't print to the channel; it sends a notice to the user issuing it. It's not so much an issue of spamming the channel in the case of .help, but more of flooding the bot with commands.

Despite not printing to the channel, what I'm worrying about is not bot spam. It is getting killed by the server for excess flood. I should look into how excess flood works to work out a spam prevention method.


Support HEAD method for /graphql

I have multiple instances of GraphQL servers running behind a AWS ELB (elastic load balancer).
ELB recurrently checks the status of each node performing a small ping to each node.  AWS default validation mode uses HEAD method (it will not use GET or POST).
Currently (as far I was able to check) koa-graphql will return a error if a HEAD request is performed. This causes ELB to assume that the node is dead and will stop to route requests.
Need HEAD to also be supported

If I run:
curl -X HEAD http://localhost:5000/graphql -I
I'll get the following response:
HTTP/1.1 405 Method Not Allowed
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: GET,HEAD,PUT,POST,DELETE
Allow: GET, POST
Content-Type: application/json; charset=utf-8
Content-Length: 1482
Date: Tue, 04 Oct 2016 16:55:17 GMT
Connection: keep-alive

this does not quite make sense, it should not be the responbility of the package to make routes for you. Why don't you just do
const koa = require('koa');
const Router = require('koa-router'); //<EMAIL_ADDRESS>const convert = require('koa-convert');
const graphqlHTTP = require('koa-graphql');

const app = new Koa();
const router = new Router();
const graphql = convert(graphqlHTTP((request, ctx) => {
  ...
}));

router
  .head('/graphql', (ctx, next) => ctx.status = 200)
  .get('/graphql', graphql)
  .post('/graphql', graphql)

app.use(router.routes()).use(router.allowedMethods());

Thanks for @thewillhuang 's tip!
There are a lot of ways to get this to be done by router and your middlewares, so just feel free to handle head request on your side.


Misc code updates

Updates to code

Besides some minor code style changes the majority of this PR has been merged as cherry picks in other PRs.
Thank you for all the fixes and improvements.


sdfmt -i isn't works on Windows

sdfmt -i (in place formatting) isn't works on Windows and reports:
The Process Cannot Access The File Because Another Process has Locked a Portion of the File
Version: v0.0.12

It does look like another process has locked a portion of the file. Is that process sdfmt itself? If not, this is likely a problem with another program on your machine.

Is that process sdfmt itself?

I think yes
But maybe this is because I tried to use sdfmt on file placed on network disk

You can use the method explained here: https://superuser.com/a/643312 to know which process is has a lock on the file. If it is sdfmt, then that's definitively a bug, but if not, I'm afraid there isn't much I can do.


Platinum analysis

Build dataset

[x] Copy all aflow platinum conf files from #narval to #h4h ✅ 2024-04-29
[x] Init new platinum dataset with new confs ✅ 2024-04-29

create_datasets(cfg.DATA_OPT.platinum,
                cfg.PRO_FEAT_OPT.nomsa, cfg.PRO_EDGE_OPT.aflow,
                ligand_features=cfg.LIG_FEAT_OPT.gvp,
                ligand_edges=cfg.LIG_EDGE_OPT.binary,
                k_folds=None, train_split=0, val_split=0)

Run inference

[x] Find trained weights for GVPL-aflow model on pdbbind (all 5 for each split to use as an ensemble) ✅ 2024-04-29

Model key is GVPLM_PDBbind1D_nomsaF_aflowE_128B_0.00022659LR_0.02414D_2000E_gvpLF_binaryLE
They are all on #h4h

[x] Save predictions of each model to a csv file ✅ 2024-04-29

loaders = Loader.load_DataLoaders(cfg.DATA_OPT.platinum,
                               cfg.PRO_FEAT_OPT.nomsa, cfg.PRO_EDGE_OPT.aflow,
                               ligand_feature=cfg.LIG_FEAT_OPT.gvp, ligand_edge=cfg.LIG_EDGE_OPT.binary,
                               datasets=['test'])

#%%
model = Loader.init_model(cfg.MODEL_OPT.GVPL, cfg.PRO_FEAT_OPT.nomsa, cfg.PRO_EDGE_OPT.aflow,
                          dropout=0.02414, output_dim=256)

#%%
cp_dir = "/cluster/home/t122995uhn/projects/MutDTA/results/model_checkpoints/ours"
MODEL_KEY = lambda fold: f"GVPLM_PDBbind{fold}D_nomsaF_aflowE_128B_0.00022659LR_0.02414D_2000E_gvpLF_binaryLE"
cp = lambda fold: f"{cp_dir}/{MODEL_KEY(fold)}.model"

out_dir = f'{cfg.MEDIA_SAVE_DIR}/test_set_pred/'
os.makedirs(out_dir, exist_ok=True)

for i in range(5):
    model.safe_load_state_dict(torch.load(cp(i), map_location=device))
    model.to(device)
    model.eval()

    loss, pred, actual = test(model, loaders['test'], device, verbose=True)

    # saving as csv with columns code, pred, actual
    # get codes from test loader
    codes, pid = [b['code'][0] for b in loaders['test']], [b['prot_id'][0] for b in loaders['test']]
    df = pd.DataFrame({'prot_id': pid, 'pred': pred, 'actual': actual}, index=codes)
    df.index.name = 'code'
    df.to_csv(f'{out_dir}/{MODEL_KEY(i)}_PLATINUM.csv')

Analysis  ^3fe12b

[x] Check for overlap in pdbIDs and remove them

provide a plot of with and without performance?

how can I analyze the performance of the model in its ability to detect dangerous mutations?

[x] Normal predictive performance analysis with cindex and MSE scores

Statistical t-test to determine how different the predicted and experimental distributions are from each other

[x] Mutation impact analysis:

calculate $\Delta pk_a$ (change in binding affinity for each protein-ligand pair). Then correlate the predicted $\Delta pk_a$ and the experimental $\Delta pk_a$

higher correlation indicates that the model effectively captures the impact of mutations

Split up scores depending on number of mutations for even more analysis (maybe it struggles with larger # of mutations)

[x] Identifying "significant" mutations

Based on the distribution of $\Delta pk_a$ scores classify all above [1 and 2] standard deviations to be "significant"?
Then Build a confusion matrix to analyze the true positive rate and true negative rate of the model in identifying these mutations.

There is huge overlap between the PDBbind training data and the platinum dataset unfortunately...
Removing all exact instances of pdbids leaves us with 975 rows:

However, if we consider both mutated and wildtype proteins as the same protein then we are left with ZERO rows.
The middle ground is to not remove wildtype proteins but this is still not optimal and leaves us with 1461 rows:

1. predictive performance

metric
With Overlap
Without Overlap
Significance

mae
1.380 $\pm$ 0.059
1.467 $\pm$ 0.031
*

pcorr
0.530 $\pm$ 0.052
0.415 $\pm$ 0.064
*

cindex
0.674 $\pm$ 0.023
0.641 $\pm$ 0.025

mse
3.106 $\pm$ 0.272
3.467 $\pm$ 0.249

scorr
0.503 $\pm$ 0.067
0.411 $\pm$ 0.086

rmse
1.761 $\pm$ 0.078
1.861 $\pm$ 0.068

CODE:
from src.analysis.figures import fig_dpkd_dist, fig_sig_mutations_conf_matrix, generate_roc_curve
from src.analysis.metrics import get_metrics
import numpy as np
import pandas as pd
from scipy.stats import ttest_ind

MODEL = lambda i: f"results/model_media/test_set_pred/GVPLM_PDBbind{i}D_nomsaF_aflowE_128B_0.00022659LR_0.02414D_2000E_gvpLF_binaryLE_PLATINUM.csv"

data_p = lambda set: f'/cluster/home/t122995uhn/projects/data/PDBbindDataset/nomsa_aflow_gvp_binary/{set}0/cleaned_XY.csv'
df_t = pd.Index.append(pd.read_csv(data_p('train'), index_col=0).index,
                       pd.read_csv(data_p('val'), index_col=0).index)
df_t = df_t.str.upper()

results_with_overlap = []
results_without_overlap = []

for i in range(5):
    df = pd.read_csv(MODEL(i), index_col=0).dropna()
    df['pdb'] = df['prot_id'].str.split('_').str[0]

    # with overlap
    metrics_with = list(get_metrics(np.array(df['actual']), np.array(df['pred']),
                               save_figs=False, save_data=False, show=False))
    # dont return p vals:
    metrics_with[1] = metrics_with[1][0]
    metrics_with[2] = metrics_with[2][0]
    results_with_overlap.append(metrics_with)

    # without overlap
    df_no_overlap = df[~(df['pdb'].isin(df_t))]
    metrics_without = list(get_metrics(np.array(df_no_overlap['actual']), np.array(df_no_overlap['pred']),
                                  save_figs=False, save_data=False, show=False))
    metrics_without[1] = metrics_without[1][0]
    metrics_without[2] = metrics_without[2][0]
    results_without_overlap.append(metrics_without)

# Convert results to DataFrame
results_with_df = pd.DataFrame(results_with_overlap, columns=['cindex', 'pcorr', 'scorr', 'mse', 'mae', 'rmse'])
results_without_df = pd.DataFrame(results_without_overlap, columns=['cindex', 'pcorr', 'scorr', 'mse', 'mae', 'rmse'])

# Statistics computation
mean_with = results_with_df.mean()
std_with = results_with_df.std()
mean_without = results_without_df.mean()
std_without = results_without_df.std()

# T-tests for significance
ttests = {col: ttest_ind(results_with_df[col], results_without_df[col]) for col in results_with_df.columns}

# Combine mean and std
combined_with = mean_with.map(lambda x: f"{x:.3f}") + " +- " + std_with.map(lambda x: f"{x:.3f}")
combined_without = mean_without.map(lambda x: f"{x:.3f}") + " +- " + std_without.map(lambda x: f"{x:.3f}")

# Add significance marks
significance = pd.Series({col: '*' if ttests[col].pvalue < 0.05 else '' for col in results_with_df.columns})

# Prepare markdown table
md_table = pd.concat([combined_with, combined_without, significance], axis=1)
md_table.columns = ['With Overlap', 'Without Overlap', 'Significance']
md_output = md_table.to_markdown()
print(md_output)

3. Significant Muation impact analysis

code
```python
from src.analysis.figures import fig_dpkd_dist, fig_sig_mutations_conf_matrix, generate_roc_curve
from src.analysis.metrics import get_metrics
import numpy as np
import pandas as pd
from scipy.stats import ttest_ind
MODEL = lambda i: f"results/model_media/test_set_pred/GVPLM_PDBbind{i}D_nomsaF_aflowE_128B_0.00022659LR_0.02414D_2000E_gvpLF_binaryLE_PLATINUM.csv"
data_p = lambda set: f'/cluster/home/t122995uhn/projects/data/PDBbindDataset/nomsa_aflow_gvp_binary/{set}0/cleaned_XY.csv'
df_t = pd.Index.append(pd.read_csv(data_p('train'), index_col=0).index,
pd.read_csv(data_p('val'), index_col=0).index)
df_t = df_t.str.upper()
results_with_overlap = []
results_without_overlap = []
i=0
df = pd.read_csv(MODEL(i), index_col=0).dropna()
df['pdb'] = df['prot_id'].str.split('_').str[0]
df_no_overlap = df[~(df['pdb'].isin(df_t))]
#%%
true_dpkd = fig_dpkd_dist(df, pkd_col='actual', verbose=True)
pred_dpkd = fig_dpkd_dist(df, pkd_col='pred', verbose=True)
#%%
conf1, tpr1, tnr1 = fig_sig_mutations_conf_matrix(true_dpkd, pred_dpkd, std=0.3)
conf2, tpr2, tnr2 = fig_sig_mutations_conf_matrix(true_dpkd, pred_dpkd, std=1)
%%
ROC
generate_roc_curve(true_dpkd, pred_dpkd, thres_range=(0,5), step=0.1)
</details>

Pretrained Davis results

[x] build dataset from aflow confirmations
[x] Run hyperparameter tuning
[ ] Train davis
[ ] Run platinum evaluation


FullOuterJoinMacro

Added macro FullOuterJoin with AdHocTest

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: nsulikowski sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[Bug]: Game crashed when launching StarRail

Affected Version
1.71.16
Bug Behavior
StarRail.exe crashed when using Collapse
Expected Behavior
game works fine
Steps to reproduce
1.start game by using Collapse
2.game crashed when loading
3.start game by using official launcher
4.game works fine
Related Issues
No response
Screenshot(s)

Additional Information
main_2023-12-24.log

Heyyo~ Thank you for submitting bug report for Collapse!
From the official launcher log file you have sent, official launcher flagged data.unity3d as corrupted. Before you switched to official launcher, have you tried doing game repair?
also for confirmation, is the HSR region you were trying to run is on Mainland China or Bilibili? @infinity8942
<+++++>
CC @neon-nyan @shatyuka
The file md5 on the log stated that the plugin was installed with its correct hash but the official launcher flagged it as corrupted and also the game wont start. This behavior is quite interesting to say the least and is consideration for our PR over at #342
Judging from the hashes, the game region is HSRCN
pkgversion

{"remoteName": "StarRail_Data/data.unity3d", "md5": "ee13471ab02aedbb59acfb26c48f7490", "fileSize": 21710086}

plugin api

"validate": "[{\"path\":\"StarRail_Data/data.unity3d\",\"md5\":\"92b75aff77304b851333dbce94730eab\"}]"

Try CI build https://github.com/CollapseLauncher/Collapse/actions/runs/7262750999

Thanks for your response. yes I'm at Mainland China and I did try to do the game repair this moring, but the result was same at that time.
Now It's running fine, and I don't know why... I just hope it can help you to impove Collapse.

I have put the change that knock on wood should fix this behavior on https://github.com/CollapseLauncher/Collapse/commit/3ef9f3781b5e2d6eac9b967169d35a866ce9f859
Since this issue seems to be quite random, we will close this for now and feel free to reopen or put another ticket shall this issue reappears.
Thank you for the bug report!