Skip RPC selection step

For the DappNode package, we want to be able to either hardcode or skip the RPC selection step, since dappnode offers it's own RPC endpoint and being asked to provide an Infura endpoint goes against the mission of DappNode and decentralizing Ethereum in general.
On request by @eduadiez the 3 arguments he'd like to be set are:
      - KEYSTORE_PASSWORD=
      - NETWORK_ID=mainnet
      - "RPC_ENDPOINT=http://fullnode.dappnode:8545"

@eduadiez please chime in if you have any additions.


Package Control marks LSP-pyright as "incompatible" when starting Sublime

I'm on Manjaro Linux and this occurs after I upgraded to Sublime 4148 (don't know the previous version), if that's of any help

Does it only popup one time, or this happen everytime you start ST?

We do have some changes for LSP-pyright in https://github.com/sublimelsp/repository/blob/dc7ef04096b9d2758ce15a8ef9960727e40ae138/repository.json#L642-L651. But from my test, I can install it with clean ST 4148 without any error. Can you try to reinstall it?

Or try "Package Control: Upgrade Package"

It should use "version": "1.2.29", "sublime_text": ">=4148".


Support returning arrays from functions in GLSL/MSL, refactor array copies in MSL.

Not possible in HLSL apparently, need workaround ...
... And neither can MSL.
The workaround for returned arrays is to inject a magic "out" argument to functions which have this return type.
For HLSL, we can directly assign the array, while in MSL, we need to inject a spvArrayCopy, because arrays are not value types (sigh).
While testing this I ran into a different issue with MSL where spvArrayCopy would be ignored if the right hand side of an assignment is constant. This however was buggy except in the cases where a variable is declared.
float4 array[4] = { ... };

is fine, but
float4 array[4];
array = { ... };

is not.
The workaround here is similar to HLSL, declare arrays as constant globals, then assign using that variable.
MSL arrays cannot be assigned so, we end up with:
constant float4 array[4] = { ... };
float4 t[4];
spvArrayCopy(t, array);

This will likely break performance of LUTs, so I added a rather ugly special case which is such that if a variable is declared and we assign a constant expression, we can declare the array inline. This leads to some redundancies (same constant is declared both as a constant and declared inline on assignment), but I don't see a better solution which is also robust ... :
If the MSL compiler is able to optimize the spvArrayCopy away, we can remove this hack.
Fix #428.

@billhollings This modifies MSL significantly, so please have a look if this is acceptable.

@HansKristian-ARM  Looks good to me. Thanks for all the effort!

Thanks very much @HansKristian-ARM!


Feature/const json

Creating pull request for basic checks

I am not ab le to run any design on this branch even with designs having no constraints, e.g., inverter_v1. Do I need to make any local hacks?

I am not ab le to run any design on this branch even with designs having no constraints, e.g., inverter_v1. Do I need to make any local hacks?

The current flow requires a .const.json file for each sub circuit. As it is not available now, I reset it back to read the text constraint reader and it runs now. After the json constraint file is ready, I will set it to json constraint reader and test it.
A json constraint file should be named like ".const.json".

Can we keep it to json constraint but make the const.json file requirement not compulsory. So that I can verify if the format is running.

Can we keep it to json constraint but make the const.json file requirement not compulsory. So that I can verify if the format is running.

It is now not compulsory. Even the json constraint file does not exist, the flow still runs.

@854768750 Can you check the error in PlaceRouteHierFlow?

@854768750 Can you check the error in PlaceRouteHierFlow CI check?
I have updated the symmetry constraints so that it dumps both formats. I am working on cap constraints then I will fix other constraints.

Thanks. Let me have a check.
Also, the const.json file is in 1_topology directory, could you please make a copy to 2_primitive and 3_pnr, just like the const file?

@854768750 Can you check the error in PlaceRouteHierFlow CI check?
I have updated the symmetry constraints so that it dumps both formats. I am working on cap constraints then I will fix other constraints.

I have checked symmnet and symmblock constraints. They are correct. There is no error when reading the file. Thanks.

Done copying, In cap json constraints what does "cap_r" and "cap_s" mean?

Done copying, In cap json constraints what does "cap_r" and "cap_s" mean?

I found these two are not used now. Just keep them to be -1. Thanks.

@kkunal1408  CC constraint is missing the const json. Circuits with capacitor cannot run.

@kkunal1408  CC constraint is missing the const json. Circuits with capacitor cannot run.


problem with P

I made a build with a wemos d1 mini, the RX/TX pins align mirrored with those of the CJMCU  atmega32u4, so i could use two header pins and solder the boards together and then only use two wires for 5V and Ground (the Wemos has a power regulator).
It kind of works, but i found two problems so far and can't imagine that those are because of my hardware setup?
One is that the capital P doesn't get output. And when i send:
STRING ABCDEFGHIJKLMNOPQRSTUVWXYZ
i get:
ABCDEFGHIJKLMNO
lower case there is no problem, and i did not find any other characters yet which "misfire".
The other thing is that if i send
SPACE
it just seems to send capital S
and then it seems to hit the "P" problem
any ideas?
(should probably try with a lower baudrate, but now i have to desolder every time i want to program the CJMCU atmega32u4 :(  and if it was a baudrate problem i would expect more problems)

This is why I've soldered a switch on my build

nice setup for testing.
So do SPACE and P just work in your case?

Yup both work fine (BAUD_RATE=115200).
Did you observe this SPACE/P truncation on all browsers?

Found it, i used the BAUD_RATE in spacehuhn's source which is 57200, this is 400bps off from the standard baudrate of 57600. When i switched to 57600 on both devices, my problem with P was gone ;) Will now try 115200 and if it works consistently stick to that.
It might have to do with production, firmware or tolerances. I had a different device from aliexpress which worked fine (as far as i could tell) on 57200, but 57600 should work for anything.

hmm, cheered to soon, it's back even with 57600 baud  😢 still investigating

I solved the problem by reducing the baud rate on both the Teensy as well as the ESP8266 to 19200.  See #65


Mute Unmute causing error and multiple tumbnails

Switching between un mute and mute is causing weird behavior
Please check attach screen shots

Thanks for the issue @asyfsaeed! Sorry to hear that you are having issues.
I am unable to reproduce the issue in the screenshot. It is difficult to know what the problem may be without additional information.
Specifically, it would be very helpful to us if you could provide us with the following information:

What steps need to be taken to reproduce this issue?
How quickly are you switching between mute and unmute?
What operating system and OS version are you using?
What browser and browser version are you using?
What version of the App are you using? (master branch, specific commit hash)
What version of Node.js version are you using?

Hi,

Joined room and start switching mute and unmute
Two seconds between clicking mute and unmute
AWS Ubuntu t2.medium
Chrome Version 83.0.4103.97 (Official Build) (64-bit)
latest master code
10.13.0

And sometime camera  video remains on even toggling camera button

seems like trackUnpublished causing issue when pressed frequently. any help

Thanks for the additional information @asyfsaeed!
Unfortunately, I'm still not able to reproduce this issue. However, I will soon have a fix available for a related issue that arises when the 'Mute Video' button is pressed frequrently (see https://github.com/twilio/twilio-video-app-react/issues/218). I'll keep you posted.
Thanks!

@asyfsaeed I've merged a fix that may help your issue. Please try it out and let me know how it works for you. Thanks!


Noob routing issue

Forgive me because this is my first attempt at doing anything practical with ES6, React, and Webpack.
It seems views/Routes.js includes views/Main.js, which is fine. As soon as you want to have multiple views, you would have Routes.js include these views. No problem.
Since these views are imported initially (forgive me if this is not how ES6 importing works), the Routes export is not available until after import.
How would you be able to reference Links to different Routes (as per react-router) if you need to parse the imports before the Routes are defined?
I guess an example with more than one route would be pretty practical...

@RickWong I still actually can't figure out what would be the proper way to have a "container" like React component and multiple routes. Any insight you can offer would be welcomed.


can't change domain name

I'm running master as of 2015-11-04 (603c7389b8e56740033e51b36b79e61136b46002) using docker-compose as provided. Now I need to change the "domain" from the temporary name to the permanent.
unfortunately changes done to /data/gogs/conf/app.ini are totally ignored and the containers don't provide a command line mysql client.
I couldn't even find the schema to attempt to write a go app for the purpose...

Hi, I suppose you have read the docker dictionary Readme and restarted Gogs?

Typo: directory

I used docker-compose instead of docker manually as I assumed that was the preferred way. after stopping rm-ing and restarrting the problem is solved. thank you.

Cool! I should remove that compose file 😂😂


[MM-12737] Migrate preference-related actions from user_actions.jsx to redux

Summary
Moved preference related actions [savePreferences, savePreference, deletePreferences] from mattermost-webapp/actions/user_actions.js into mattermost-redux/src/actions/preferences.js.
Additionally updated channels.js, preferences.js to pass flow check.
Webapp PR
Ticket Link
MM-12737
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit tests passed
[x] Ran make flow to ensure type checking passed
[x] Added or updated unit tests (required for all new features)

Monile already uses redux actions, desktop is the webapp running on a webview so we are good there.
Webapp was using flux before redux that is why we need the migrations

Monile already uses redux actions, desktop is the webapp running on a webview so we are good there.
Webapp was using flux before redux that is why we need the migrations

got it, so we close this PR and I should update webapp PR?

Yes we can close this PR


Unable to open PR from the web environment

All the drop downs in "merge changes from" and "merge changes into" are blank, can't select anything, therefore can't submit a PR. 😿

Version: 1.61.2
Commit: 6cba118ac49a1b88332f312a8f67186f7f3c1643
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:93.0) Gecko/20100101 Firefox/93.0
Embedder: github.dev

I was facing this on Firefox as well. But not on Chrome. Let me know if you need any logs etc.

This only seems to be occurring on http://vscode.dev / http://github.dev and not on GitHub CodeSpaces.
This screenshot is from the same firefox, but running codespaces:

I'm no longer seeing this occur in Firefox.


Adding constraint instances adds those instances to all constraint definitions.

This seems to be a problem only with the GUI. The backend seems to save the Instances correctly.


error: use of undeclared identifier 'sycl' while building portBLAS with AdaptiveCpp

I am trying to build portBLAS with AdaptiveCpp but I keep getting error: use of undeclared identifier 'sycl'.
Build command:
export ACPP_TARGETS=generic

cmake -GNinja -DTUNING_TARGET=NVIDIA_GPU -DCMAKE_BUILD_TYPE=Release       -DSYCL_COMPILER=adaptivecpp -DACPP_TARGETS=$ACPP_TARGETS       -DBLAS_ENABLE_BENCHMARK=ON ..

Environment:
Ubuntu 20.04 (WSL2)
Clang17
AdaptiveCpp built from source
Full stacktrace: https://pastebin.com/k0iW2HqR

Hi @krishvishal,
we recently merged PR #513 that fixes the issue you encountered. Could you please update your portBLAS repository and try again?


Warning message to user about representative image and visibility options

Adds guiding message about permissions and display for filesets on the fileset edit page.

Coverage remained the same at 74.497% when pulling 150cf3bd6a71325f60557b464747c8d81059bdf6 on image-1048 into 230b19e57eda141dfb31201b2f242309aadee2b7 on develop.


[V1] How to send email to gmail (not in spam) and configure dkim and dmarc

I am using https://github.com/tomav/docker-mailserver/tree/v1 and I am trying to send email in Google ( not in spam folder )
I am reading at : https://support.google.com/mail/answer/81126#authentication

Why is it important to authenticate your messages?

Authentication ensures that your messages can be correctly classified. Emails that lack authentication are likely to be rejected or placed in the spam folder, given the high likelihood that they are forged messages used for phishing scams.

In addition, unauthenticated emails with attachments may be outrightly rejected, for security reasons.

To ensure that Gmail can identify you:

Use a consistent IP address to send bulk mail.
Keep valid reverse DNS records for the IP address(es) from which you send mail, pointing to your domain.
Use the same address in the 'From:' header on every bulk mail you send.
We also recommend the following:

Sign messages with DKIM. We do not authenticate messages signed with keys using fewer than 1024 bits.
Publish an SPF record.
Publish a DMARC policy.

I am using v1 of mail-server,  have configured spf for my main domain domain0.com and aliases domain, I want to know how I can configure opendkim, opendmarc with aliases domain :
The following appear in my logs:
At start
opendkim[512]: 586AC4AC1485: DKIM-Signature field added (s=mail, d=domain0.com)
opendkim[512]: E075E4AC148F: DKIM-Signature field added (s=mail, d=domain0.com)

and
opendkim[512]: OpenDKIM Filter v2.10.3 starting (args: -x /etc/opendkim.conf -u opendkim -P /var/run/opendkim/opendkim.pid)
opendmarc[523]: OpenDMARC Filter v1.3.1 starting (args: -c /etc/opendmarc.conf -u opendmarc -P /var/run/opendmarc/opendmarc.pid -p inet:54321@localhost)
opendmarc[523]: additional trusted authentication services: mail.domain0.com

Is it possible to configure opendkim/opendmarc with aliases domains ?

The V1 i'm not sure (before my time) But take a look at: https://github.com/tomav/docker-mailserver/wiki/Configure-DKIM

Where can I found the v1 documentation?

You need to use SPF and DKIM.

You need both

Could we please recreate branch v1 ? I can't troubleshoot correctly, I know I should have fork but we are not supposed to delete tag aren't we?


[Fix]No Binding NPE

Originally in isSupportedBasicExpression, some expression had not been resolved to a table correctly, causing a NPE problem which prevent the expression from push down. Fixed this by adding a resolve to all the expression.

@ilovesoup PTAL


FEATURE: Auto assign feature

Description
By implementing the Auto Issue Assignment feature using GitHub Actions, the issue assignment process becomes automated and efficient. Issues matching specific comment-based criteria will be automatically assigned to the user who made the relevant comment, improving collaboration and task management within the repository.
Implementation:

.github/workflow

Screenshots
NA

Hey @agamjotsingh18 @Arnabdaz
Please review my pull request #271
Thank you

Hey @Arnabdaz
Could you kindly review it when you have a moment?
Pull request - #271
Thank you


Follow IMP recommendations better

Namespaces
Currently, namespaces are separated by _ in this repository.
With the new linting naming rules, we can enforce using a scheme like namespace.name for things like scoreboard objectives, team names, etc.
This what is recommended byhttps://github.com/Arcensoth/imp-spec/blob/master/docs/imp_core.md#2-namespace-everything.
It has the advantage, that it is easier to see the namespace in a name and makes it less likely to cause conflicts.
Storages
The IMP conventions suggest using namespace.<transient_name> namespaces for transient storages (i.e. storages that store temporary data). E.g. bingo.__temp__

I'm happy with the current conventions used in the repo. No need to keep track of this anymore.


integrate kernel constants with inference and ssa

update: Johnni started working on the integration with global inference (see https://dart-review.googlesource.com/c/sdk/+/97240)

update: CFE constants are integrated in inference and ssa but work is still needed to fully support platform code with the new constants. This work (see https://dart-review.googlesource.com/c/sdk/+/97634) has already shown some bugs but more may come.

current status: Johnni tested enabling CFE constants on on the bots and believes most backend work is done.Only one issue with native typed arrays showed up, which he'll investigate tomorrow.

@johnniwinther - do you believe there is anything else to do on the dart2js side? Can we close this issue?

We're done.


Add HPCS secret name to Ceph and NooBaa CR

For HPCS KMS, UI creates a Secret (containing IBM_KP_SERVICE_API_KEY & IBM_KP_CUSTOMER_ROOT_KEY) in openshift-storage namespace. Name of that Secret needs to be passed in:
cephCluster.Spec.Security.KeyManagementService.TokenSecretName
nooBaa.Spec.Security.KeyManagementService.TokenSecretName

BZ#2047201
Signed-off-by: SanjalKatiyar<EMAIL_ADDRESS>

/release-4.10

/cherry-pick release-4.10


Added SciPy DCIP notebook files to tutorials folder.

Prior to merging, we should

[ ] move data and model files to the cloud
[ ] add links so that these can be downloaded

Also, since there are several notebooks in this folder, adding an index notebook with a brief explanation of what the others contain and a suggested order to work through them would be helpful!

also, for the file extensions on the pickled objects, lets follow convention and use

[ ] .p

[ ] in DCinv_Gradient_singleSrc_Clean.ipynb can we point to scripts in this repo rather than the presentations one ?

The files are here:

https://storage.googleapis.com/simpeg/tkc_synthetic/dcip/DCresults.p
https://storage.googleapis.com/simpeg/tkc_synthetic/dcip/IPfwd.p
https://storage.googleapis.com/simpeg/tkc_synthetic/dcip/IPresults.p
https://storage.googleapis.com/simpeg/tkc_synthetic/dcip/VTKout_DC.dat
https://storage.googleapis.com/simpeg/tkc_synthetic/dcip/VTKout_eta.dat

@fourndo : could you point @micmitch to how you are downloading files from the cloud? Thanks!

The images are now available at:

https://storage.googleapis.com/simpeg/images/tutorials/tkc_synthetic/DCIP_Diamond.png
https://storage.googleapis.com/simpeg/images/tutorials/tkc_synthetic/dc.png

so they can be removed from the repo, and we can just pass them as urls

@micmitch : can you take a look at how the files are being downloaded for the potential fields story (https://github.com/simpeg/tutorials/blob/master/docs/content/stories/PF_MAG.rst) and associated notebook? We can then remove the large files from this pull request and just download them from the cloud. It would be great to get this example in!


Add NodeJS

Oh well, NodeJS Looking Good, Even For Discord it's for life of the bots, Please add it

node.js has been part of piston for like 3 years now. in fact, it was the first language ever implemented.


Support indexing events without specifying a contract address

At present you have to specify the address from which events will come. For some event types (for instance, ERC20 transfer), it's beneficial to be able to listen to all matching events, regardless of the address that emitted them.

This may be easy to add support for.
Let's assume we make the address field under dataSources > source optional but keep the abi field so we can identify what ABI the event relates to.
We would now call eth_getLogs without an address, which is fine. We should still be able decode event parameters using the ABI (don't need the contract address for that). We may have to relax some checks to allow events to get through to to the mapping runtime but that's easy. Beyond that, no changes should be needed. You should even be able to access the contract by binding ABI + event address.
Let me double-check the related code. My gut feeling is this won't be hard.

Any update on this?

ERC20 will also need internal transaction triggers / function call triggers, as the standard doesn't require contracts to implement events. We're actively working on this right now.
Still, making address optional can be useful on its own. I'll put it on next week's development agenda.

@sorawit @Arachnid Support for this has been merged into graph-node, but using it in subgraphs also requires https://github.com/graphprotocol/graph-cli/pull/240 (+ a new graph-cli release). You can expect a release this week.

Thank you!
Re ERC20 - transfer events are mandatory, but the spec doesn't specify a mandatory event for minting/burning.

@Arachnid Yep, you're right about Transfer.

Just wondering, but wouldn't it be trivial for me to make an unrelated contract with a different address but identical Event signatures, and alter data for a subgraph indexing in an address-agnostic way like this?

This is very useful, but if there is a document to explain how can I use this? Very thanks！

Hi! Is it possible to define source with callHandlers but without contract address? I know it's possible with eventHandlers but when I try to use callHandlers I got error subgraph source address is required

Hi I don't think this is possible currently, and would also not be performant given the current reliance on trace_filter for fetching calls

@FlacoJones 100% - ideally a bytecode protection would be great for this usecase.


BE-874 Fix script to specify the BC Exprorer process

"stop.sh" uses "ps" command to identify the process ID of Explorer.

ps aux | grep -v "awk" | awk '/name - hyperledger-explorer/ {print $2}'

But Alpine Linux, which is used by Explorer container, returns the following values for "ps" command.

PID USER TIME COMMAND
45 root 0:12 node app/main.js name - hyperledger-explorer

So correct process ID cannot be obtained by the above method. (Misidentify "root" as process ID.)
It works correctly by specifying the format of the output of "ps" command like follows.

ps -eo pid,args | grep -v "awk" | awk '/name - hyperledger-explorer/ {print $1}'

Close it.


aria-expanded should be set to "true" when dropdown is shown

Thank you for this great project, and glad to see accessibility has been considered!
I feel like the list of choices's container which has aria-expanded="false" should have it set to "true".

Ah ha! Good spot. I'll fix this ASAP.

This has been resolved as of version 2.7.7!

@jshjohnson  is there a way that we can set this programmatically ? Not always needed the area-expanded as 'true' (it increases the page height)


Content type headers not passed onto request

Hello. We have a nextjs project with api-platform as backend and was trying out the generated client from openapi v3.1. I have noticed that content type is not parsed from the schema and resulting in the use of crazy logic in the interceptors (we need ld+json and merge+patch, which is all described in json). I have tried several different generators and they seem to respect it.
Also, its not very clear how the TOKEN works, as I cant quite make it work. Putting my token in header from interceptors does the trick. I really enjoy using this, but since its on an early stage, examples are old and docs are alim, I thought i'd ask and potentially help others looking for this.
Thanks for this great project!

Thanks @wizzyto12! Are you able to share the spec and expected results where it fails? Might be easier for me to fix content types that way

Hey!
So this is my interceptors
`OpenAPI.interceptors.request.use(async (request) => {
request.headers = {
...request.headers,
Accept: "application/ld+json",
"Content-Type":
request.method === "PATCH"
? "application/merge-patch+json"
: "application/ld+json",
};
// Here I suppose that this can probably set the token without me adding it to the header manually. I also tried setting it as a string.
// OpenAPI.TOKEN = async () => {
//   const session = await auth();
//
//   return session.user.access_token;
// };
const session = await auth();
// This is what works, but its obvious it will.
if (session?.user?.access_token) {
request.headers = {
...request.headers,
Authorization: Bearer ${session.user.access_token},
};
}
return request;
});`
This is an entry from my openapi json file, which is openapi v3.1
"\/api\/booking_sessions": {       "post": {         "operationId": "api_booking_sessions_post",         "tags": [           "BookingSession"         ],         "responses": {           "201": {             "description": "BookingSession resource created",             "content": {               "application\/ld+json": {                 "schema": {                   "$ref": "#\/components\/schemas\/BookingSession.jsonld-thing.get"                 }               }             },             "links": {}           },           "400": {             "description": "Invalid input"           },           "422": {             "description": "Unprocessable entity"           }         },         "summary": "Creates a BookingSession resource.",         "description": "Creates a BookingSession resource.",         "parameters": [],         "requestBody": {           "description": "The new BookingSession resource",           "content": {             "application\/ld+json": {               "schema": {                 "$ref": "#\/components\/schemas\/BookingSession.jsonld-booking-session.post_thing.post"               }             }           },           "required": true         },         "deprecated": false       },       "parameters": []     },
We can see that it has the appropriate "application/ld+json", which is not respected, or I don't know how to make it work.
Usage in my code, random example
`async function login(credentials: { email: string; password: string }) {
const response = await LoginCheckService.appAuthPost({
requestBody: credentials,
});
if (response.token) {
const decodedToken = jwtDecode(response.token) as DecodedToken;
return { .... } as User;

}
return null;
}`
I get a 406 from my API if the content is not set in the interceptors accordingly.
If thats not clear enough (I know its all around the place with the codes, but still), I can create an MVP example of api platform and nextjs to try to recreate the issue in a clean project.
Thanks for the fast reply!

That should be plenty, thank you!

@mrlubos In case this is helpful, I have debugged it and turn out the reason the header is not attached is because the header constructor, it don't get the object passed in. As the code below, the first console.log (label  headers 164) have enough headers property (including content-type), but the second log is empty.
export const getHeaders = async (
	config: OpenAPIConfig,
	options: ApiRequestOptions
): Promise<Headers> => {
	const [token, username, password, additionalHeaders] = await Promise.all([
		resolve(options, config.TOKEN),
		resolve(options, config.USERNAME),
		resolve(options, config.PASSWORD),
		resolve(options, config.HEADERS),
	]);

	const headers = Object.entries({
		Accept: 'application/json',
		...additionalHeaders,
		...options.headers,
	})
		.filter(([, value]) => value !== undefined && value !== null)
		.reduce(
			(headers, [key, value]) => ({
				...headers,
				[key]: String(value),
			}),
			{} as Record<string, string>
		);

	if (isStringWithValue(token)) {
		headers['Authorization'] = `Bearer ${token}`;
	}

	if (isStringWithValue(username) && isStringWithValue(password)) {
		const credentials = base64(`${username}:${password}`);
		headers['Authorization'] = `Basic ${credentials}`;
	}

	if (options.body !== undefined) {
		if (options.mediaType) {
			headers['Content-Type'] = options.mediaType;
		} else if (isBlob(options.body)) {
			headers['Content-Type'] = options.body.type || 'application/octet-stream';
		} else if (isString(options.body)) {
			headers['Content-Type'] = 'text/plain';
		} else if (!isFormData(options.body)) {
			headers['Content-Type'] = 'application/json';
		}
	}

  console.log('headers 164', headers)
  console.log('new Headers(headers)', new Headers(headers))

	return new Headers(headers);
};

Hey, I am going to close this issue as it pertains to legacy clients. These are now in maintenance mode and I am open to pull requests, so if anyone sees this and would like to fix it, please open a pull request. The recommended approach moving forward is to use standalone clients that do not suffer from this type of problems.


Get peers from github ARK-Peers

For issue #46 .
I used await to make the request to github so I had to put everything inside an anonymous async function.

I think this is better to be implemented in https://github.com/ArkEcosystem/ark-client/tree/develop for the next version.
@wownmedia Have you already thought about where you would put the peer loading?

Stupid me, this was already implemented in the latest PR of @wownmedia as he copied over my code from ARK-JSON-RPC.
https://github.com/ArkEcosystem/ark-client/blob/develop/lib/services/network.js#L141-L147

Yes, in my opinion /lib/config/networks.js. For now I followed what you did with the RPC and used the same file as in ARK-JSON-RPC, then when V2 is released we can see how and what to implement in case anything changes. @air1one would that work for you too?

OK my bad didn't checked the develop branch. I'm good with @wownmedia PR 👍


C# Export attribute error

Godot version
v4.1.dev.mono.custom_build [f717cc0a3]
System information
Windows 10, Vulkan API 1.3.236 - Forward+ - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1050 Ti, Intel i5-8300H
Issue description
When I use the "Export" attribute to export variables in C#, regardless of whether they are of type "Node" or "Resource", the creation dialog in the inspector in the editor is always the one for creating a "Resource", and it contains all "Resource" types instead of the matching ones.

Steps to reproduce
"Open the "Test" scene in the "Demo" project, which contains two "Export" variables - one is of type "Texture2D" and the other is of type "Node". Please check and create them in the inspector of the editor interface."
Minimal reproduction project
Demo.zip

This was caused by https://github.com/godotengine/godot/pull/69988 when adding a new member to the PropertyHint enum made the PROPERTY_HINT_NODE_TYPE change its value from 34 to 35.
The C# source generator that generates the property info that is retrieved by the inspector is still generating 34 because the enum wasn't updated in the C# side.
This kind of change breaks compatibility, because libraries that were compiled against an older version of GodotSharp will use the old enum values. So I think the fix would be to move the new enum member to the end of the enum to prevent breaking compatibility while we still can before the 4.1 release.


InputButton Fix

I found out there is a bug in InputButton
I managed to fix it by calling AddTextEntry()
AddTextEntry("FMMC_MPM_NA", (windowTitleEntry and windowTitleEntry or "")) LINE 422
and
DisplayOnscreenKeyboard(1, 'FMMC_MPM_NA', '', defaultText or '', '', '', '', maxLength or 255) LINE 423

Could you please provide code to reproduce your issue?
windowTitleEntry is not a window title string, it's a text entry - see native docs.

Oh gotcha, sorry. i tough it was the message box title...
I am sorry 👍


feat(transactions): export cvToHex, hexToCV, parseReadOnlyResponse

Description
Description
As a blockstack developer, I would like to use the blockstack-api-client library. The method SmartContractsApi.callReadOnlyFunction requires to hex encode clarity values, the result of that method needs to be parsed. In  this library (stacks-transactions-js) these methods already exists.
These methods, defined in utils.ts are now exported.
For details refer to issue https://github.com/blockstack/stacks-transactions-js/issues/126
Type of Change

[x] New feature
[ ] Bug fix
[ ] API reference/documentation update
[ ] Other

Does this introduce a breaking change?
No
Are documentation updates required?
yes, documentation for new methods needs to be published
Testing information
No code features changed.
Checklist

[x] Code is commented where needed
[x] Unit test coverage for new or modified code paths
[x] npm run test in transactions passes
[ ] Changelog is updated - where is that?
[x] Tag @yknl for review

Where will these methods be documented? We currently don't have any documentation for them. In the txjs repo, we would use the README file. @yknl - will these methods be in the new docs for the monorepo?

There are typedoc generated reference docs but it's better to have it in the form of a smart contract tutorial.

@friedger could you add some docs updates for this PR?

I have provided the documentation in the code. @agraebe isn't it enough to regenerate the docs? I'm using these methods also in the tutorial

I think I wanted to make a broader point for documentation inside our docs as opposed to just inside typedoc. most developers will find it very buried if it is only mentioned in the typedoc. I think it's okay if we tackle the broader issue outside of this PR.
The only relevant place I can think of (right now) would be https://docs.blockstack.org/stacks-blockchain/integrate-stacking#step-6-display-stacking-status


Current User

Current user
Gets currently authenticated user.
GET /user

{
  "id": 1,
  "username": "john_smith",
  "email"<EMAIL_ADDRESS>  "name": "John Smith",
  "private_token": "dd34asd13as",
  "state": "active",
  "created_at": "2012-05-23T08:00:58Z",
  "bio": null,
  "skype": "",
  "linkedin": "",
  "twitter": "",
  "website_url": "",
  "theme_id": 1,
  "color_scheme_id": 2,
  "is_admin": false,
  "can_create_group": true,
  "can_create_project": true,
  "projects_limit": 100
}

This could also be used for a test of authentication since it requires no admin rights or parameters.


PHP 7

Hi,
Does it works on PHP7 ?

kristijan, I haven't had much time to tinker with it yet, but I have installed piphp/gpio on php 7.0.19 last night. Created a simple test script to access a couple of pins and had no issues. At a glance, I don't see anything that would be an issue in 7.0.x.

I can confirm our tests are passing and the library should work on 7.2, 7.3, and likely 7.4.


Use more JA3 fingerprints from another repository

Is your feature request related to a problem? Please describe.
I'm currently searching for a Python library capable of using the same JA3 fingerprints of the most important browsers to prevent Cloudflare from detecting my bots. I found your repository and another based on this one (https://github.com/rawandahmad698/noble-tls). The other allows me to impersonate very recent versions like chrome_124 or even using fingerprints of okhttp, which works for preventing the detection (old versions are blocked), and periodically that repository gets updated to include more fingerprints.
Describe the solution you'd like
However, I prefer using your repository because it has a nice syntax very similar to the requests library. Also, your repository works very good with files, but the other can't manage them. Since that repository is based on yours, it's possible for you to use it to upgrade your repository adding more fingerprints? Sincerely I didn't investigate how yours projects work exactly.
By the way, thank you for developing this project. You helped me a lot :)
Edit: Before while testing on a website protected with Cloudflare I noticed that if I enable using a random tls extension order (option available in the noble-tls project and in tls-client) I would be able to bypass the security, but if disabled, not. So maybe the solution for the problem I detected would be adding that option too, combined with allowing more JA3 fingerprints.

You can use Chrome 124 now, with the beta version 0.7.0b6. The release process was stalled by Windows build issues, which is resolved now.
As for as I know, this is the only project based on curl-impersonate. Most of other projects are based on golang tls-client. The most important differences is that curl-impersonate uses real world TLS library BoringSSL, which chrome uses, too.
Support for other customized fingerprints will be the next thing to implemente, once 0.7 is released.

Thank you!


Docker Vulnerabilities

Current these are the vulnerabilities found using Azure/container-scan:

Dashing:

CVE-2020-12351
CVE-2020-12351

Make sure your kernel version has the proper patch for each one of them.

We added these CVE's to the allowed list
https://github.com/orise-robotics/ros_ws/blob/b82a17bf0be17f9ee5e0b7f4be0064cee96ea337/.github/containerscan/allowedlist.yaml#L2-L5

This issue was greatly solved with #35. I'm closing it


Create pt-discourse.json

For the newly created Portugal category on the Discourse forum.

Thanks @waldyrious 👍


Add govuk_env_sync configuration for the Content Data API

The Content Data API is the new name for the Content Performance
Manager. At the same time as the name is being changed, the service is
also being migrated to AWS.
This commit adds both the data syncs to from AWS Production to AWS
Staging and Integration, but also a sync task to pull data from
Carrenza Production which is being used for the migration.
The govuk_env_sync script doesn't handle changing database ownership,
so to try and make the dual running situation easier, the Content Data
API in AWS is configured to use the
content_performance_manager_production database, and
content_performance_manager user. Immediately before switching to AWS,
the database can be renamed, and the ownership changed.

Trello card: https://trello.com/c/2XFTquZv/1180-3-aws-migration-get-data-syncs-working


Php auth interop test

added PHP service account credentials auth interop test
introduced dependency to php auth library repo via composer
the auth test is passing against the cloud server
all other tests are passing against a local Node server

Fixes #1306


Use Cow for element attribute values

I think this should help with using owned values (e.g. from format!) without requiring unnecessary clones

This is good, and probably could use the same for children too? Would be good to add some test cases:
#[test]
fn owned_string() {
    use pretty_assertions::assert_eq;
    use render::{component, html, rsx};

    #[component]
    fn Welcome<'kind, 'name>(kind: &'kind str, name: &'name str) {
        rsx! {

            {format!("Hello, {}", name)}

        }
    }

    assert_eq!(
        html! { <Welcome kind={"alien"} name={"Yoda"} /> },
        r#"
Hello, Yoda
"#
    );
}

Maybe split the support for children into another commit if it makes sense.

I'm not familiar with Cow enough. But like @daaku said, let's add a test to verify that behavior? 😃

Added test from comment

Not sure why github actions are not testing it, but I was expecting the test to not pass as-is since children support isn't built yet.. Does it work for you?
If it doesn't, I suggest removing the part of the test that tests for Cow children and merge this for attributes to start with.

The test passes


Validate people's launch.json

To help make sure that people's settings for launching the debugger are valid and will not lead to e.g. a confusing error message.

e.g. a confusing error message.

Like what?

Closing in favour of https://github.com/microsoft/vscode-python/issues/1987.


regarding Helm Best Practices move CRDs from /templates/crds to /crds

Resolves #144

@kuritka did you test the installation end-to-end?

@kuritka did you test the installation end-to-end?

@ytsarev, I successfully went through local playground demo


Manifest changes for Hashicorp.Vagrant version 2.2.9

Resolves https://github.com/microsoft/winget-pkgs/issues/29878
Microsoft Reviewers: Open in CodeFlow

Did a bit of digging. Looks like 3010 is now a "banned" return code for MSI: https://github.com/microsoft/winget-cli/pull/1421


Missing repository in Cargo.toml

Hi! As part of our efforts to verify that crates.io releases are reproducible ^1, we have discovered that one or more crates hosted in this repo are missing the repository field ^2 in Cargo.toml.
The repository field helps users and automated tools find the link to the git repository hosting the source code. While not required by crates.io, we believe it should always be set.
We suggest that the repository field be configured for every crate published to crates.io. Thank you very much.

Hello Paolo,
thank you for the feedback. I have added the repository section to both projects stable-fs and ic-wasi-polyfill.

Thank you @wasm-forge


Mekanism Basic Logistical Transporter Crash

So I accidentally  placed a basic logistical transporter next to my bed while I was right clicking to sleep. Every time I try to brake it with any tool or even my hand it crashes the game after I have reached 9%( I can see that in Waila). You can stop breaking it before it reaches 9% and it won't crash. The building its in, is a generated roguelike dungeon and its placed on the top floor (I haven't destroyed any of the blocks around it.).  It is adjacent to a bed and a polished granite block. I tried using a configurator tool and it can change the color without crashing, so its really weird. Every time the game crashes the world gets messed up a bit (for example pipes are missing, machines are turned around, chests are missing etc.). I am playing on the beta-3 of 1.10.2 Minecraft and the Launchers version is <IP_ADDRESS>.Also i am playing on a Laptop, a Sony Vaio SV1521V!EW. Here are some crash reports, I hope I provided enough info to help solve this issue.
crash-2016-11-19_17.49.34-client.txt
crash-2016-11-19_18.14.14-client.txt
crash-2016-11-19_18.15.05-client.txt
crash-2016-11-19_18.16.01-client.txt
crash-2016-11-19_23.08.59-client.txt
crash-2016-11-19_16.58.19-client.txt
I placed another basic logistical transporter next to it and the same thing happens. Here is the crash report for that.
crash-2016-11-19_23.43.17-client.txt

I am going to replace the pipes with air so that I can continue playing without it bugging me, but I will keep a backup of the world in case it is useful.

This is the best bug report I've ever received lol. Bravo. Ill look into it and forward what I can to the author.
Can you break the bed without crashing? Can you break the bed or all blocks touching any face of the machine and then break it without crashing?

I will try.

So, i tried breaking all blocks adjacent to the pipe and i didn't crash, but the problem persisted. I also tried putting a pipe down next to a vanilla chest to see what would happen, but I could remove the pipe without crashing at 9%. I could change the pull/push side of it. there are the crash reports for both of the cases:
crash-2016-11-21_17.48.05-client.txt
crash-2016-11-20_23.30.43-client.txt

Should be fixed in our next update.

@Gsdigger try to reproduce with beta4 please

FIXED!


GitHub Action to lint Python code

Output: https://github.com/cclauss/flask-state/actions
Run pytest .
ImportError while loading conftest '/home/runner/work/flask-state/flask-state/tests/conftest.py'.
tests/conftest.py:4: in <module>
    from src.flask_state import DEFAULT_BIND_SQLITE
E   ModuleNotFoundError: No module named 'src'
Error: Process completed with exit code 4.

Added tests/__init__().py
Added redis to pip installed dependencies.  Should it also be added to setup.py?
pytest fails on tests/test_utils.py ....F..

redis  is a pluggable dependency, so we do not plan to add it to setup.py. The redis ImportError will be fixed in the next version.

Dear @cclauss, we sincere thanks for your those improvement code review. Have a good day!


Backport of ES auto expanding changes

Summary of the changes / Why this improves CrateDB
See individual commits.
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[X] Touched code is covered by tests
[X] CLA is signed
[X] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

New FilteringAllocationIT tests are failing?

New FilteringAllocationIT tests are failing?

Seems to be related to the jenkins nodes. I added assert busy and increased the timeout, seems to be ok now.


Alpine based images create the data folder to the OS disk

I have a StatefulSet With a PersistentVolume mounted to /var/lib/postgresql. Debian based Postgres images work as expected. Database is initialized and /var/lib/postgresql/data folder is stored in the PersistentVolume.
$ grep image statefulset.yaml | grep -v \#
          image: postgres:15.0

$ kubectl apply -k .
namespace/test created
statefulset.apps/postgres created
$ kubectl exec -it -n test statefulset/postgres -- bash

bash-5.1# df -h
Filesystem                                              Size  Used Avail Use% Mounted on
overlay                                                 126G   34G   87G  28% /
tmpfs                                                    64M     0   64M   0% /dev
/dev/mapper/rl-root                                     126G   34G   87G  28% /etc/hosts
shm                                                      64M  1.1M   63M   2% /dev/shm
/dev/longhorn/pvc-7eab52f7-918e-4791-a924-74072998cd26  7.8G   28K  7.8G   1% /var/lib/postgresql
...

With Alpine based images the initialization process creates a separate mount for the /var/lib/postgresql/data folder to the OS disk. This will lead to data loss when the pod is recycled (data is not in the PersistenVolume).
$ grep image statefulset.yaml | grep -v \#
          image: postgres:15.0-alpine

$ kubectl apply -k .
namespace/test created
statefulset.apps/postgres created

$ kubectl exec -it -n test statefulset/postgres -- bash
bash-5.1# df -h
Filesystem                Size      Used Available Use% Mounted on
overlay                 125.4G     33.0G     86.7G  28% /
tmpfs                    64.0M         0     64.0M   0% /dev
/dev/mapper/rl-root     125.4G     33.0G     86.7G  28% /etc/hosts
/dev/mapper/rl-root     125.4G     33.0G     86.7G  28% /dev/termination-log
/dev/mapper/rl-root     125.4G     33.0G     86.7G  28% /etc/hostname
/dev/mapper/rl-root     125.4G     33.0G     86.7G  28% /etc/resolv.conf
shm                      64.0M      1.0M     63.0M   2% /dev/shm
/dev/longhorn/pvc-7a73320e-4498-403a-a5ec-33d3c4f8ae76
                          7.8G     28.0K      7.8G   0% /var/lib/postgresql
/dev/mapper/rl-root     125.4G     33.0G     86.7G  28% /var/lib/postgresql/data
...

Not sure if this is a configuration problem in the StatefulSet itself. Atleast it is suprising behaviour which is hard to detect at first. It took me couple of times losing the data before suspecting something is wrong. It is not Longhorn related, I have seen the problem happen with AWS based storage too.
To reproduce checkout the following repository.
https://github.com/tuupola/postgres-alpine/
For documentation purposes this is the StateFulset config.
---
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: postgres
spec:
  replicas: 1
  serviceName: postgres
  selector:
    matchLabels:
      app.kubernetes.io/name: postgres
      app.kubernetes.io/component: database
      app.kubernetes.io/instance: test-postgres
  template:
    metadata:
      name: postgres
      # These are not needed by considered a good practice
      labels:
        app.kubernetes.io/name: postgres
        app.kubernetes.io/component: database
        app.kubernetes.io/instance: test-postgres
        app.kubernetes.io/version: "15.0"
    spec:
      containers:
        - name: postgres
          #image: postgres:16.0
          #image: postgres:16.0-alpine
          #image: postgres:15.0
          image: postgres:15.0-alpine
          #image: postgres:14.0
          #image: postgres:14.0-alpine
          env:
            # These are created automagically
            - name: POSTGRES_USER
              value: postgres
            - name: POSTGRES_PASSWORD
              value: example
            - name: POSTGRES_DB
              value: test
          resources:
            requests:
              cpu: 250m
              memory: 256Mi
            limits:
              cpu: 1000m
              memory: 1024Mi
          volumeMounts:
            - name: data
              mountPath: /var/lib/postgresql
  volumeClaimTemplates:
    - metadata:
        name: data
      spec:
        accessModes:
          - ReadWriteOnce
        resources:
          requests:
            storage: 8Gi
        storageClassName: longhorn

I think this:
          volumeMounts:
            - name: data
              mountPath: /var/lib/postgresql

Needs to be:
          volumeMounts:
            - name: data
              mountPath: /var/lib/postgresql/data

(The mount you're seeing at /var/lib/postgresql/data is likely an anonymous volume created by your underlying container runtime, which I'm guessing is most likely Docker, since I don't think any other runtime will auto-create anonymous volumes from the image config like that.)


TypeId panic selecting double column from MySQL

thread 'main' panicked at 'not implemented: encountered unknown field type id: TypeId(5)

as far as I can tell, the DOUBLE is missing here: https://github.com/launchbadge/sqlx/blob/60d45ac44b107c0b8079db344f190c453a90a309/sqlx-core/src/mysql/protocol/row.rs#L109

This is now available as v0.2.5


pyreadstat

fixes using pyreadstat with --standalone (issue #980)

Note that this makes all my test scripts work (minimal.py and test.py) when using --standalone (or --onefile) but I am unsure whether or not there should be other imports added. I just had a look at pyreadstat cython files and they import a bunch of other stuff. I guess it works because pandas is required and it imports those too, but I wonder whether or not I should add those imports too. FWIW, here are the import blocks of those cython files:
pyreadstat/pyreadstat.pyx
import multiprocessing as mp

import pandas as pd

from readstat_api cimport readstat_parse_sas7bdat, readstat_parse_dta, readstat_parse_sav
from readstat_api cimport readstat_parse_por, readstat_parse_xport
from readstat_api cimport readstat_parse_sas7bcat
from readstat_api cimport readstat_begin_writing_dta, readstat_begin_writing_por, readstat_begin_writing_sav
from _readstat_parser cimport py_file_format, run_conversion
from _readstat_writer cimport run_write
cimport _readstat_parser, _readstat_writer
from copy import deepcopy
from worker import worker

pyreadstat/_readstat_parser.pyx
from cpython.datetime cimport import_datetime, timedelta_new, datetime_new
from cpython.exc cimport PyErr_Occurred
from cpython.object cimport PyObject
from libc.math cimport NAN, floor

#from datetime import timedelta, datetime
from collections import OrderedDict
import os
is_pathlib_available = False
try:
    from pathlib import Path
    is_pathlib_available = True
except:
    pass

import pandas as pd
import numpy as np
#from pandas._libs import Timestamp

from readstat_api cimport *

# necessary to work with the datetime C API
import_datetime()

pyreadstat/_readstat_writer.pyx
import os
import numpy as np
#cimport numpy as np
import pandas as pd
import datetime
import calendar
IF PY_MAJOR_VERSION >2:
    from datetime import timezone as _timezone
ELSE: # not available on python 2
    import pytz as _timezone
from libc.math cimport round, NAN
is_pathlib_available = False
try:
    from pathlib import Path
    is_pathlib_available = True
except:
    pass

from readstat_api cimport *
from _readstat_parser import ReadstatError, PyreadstatError
from _readstat_parser cimport check_exit_status

I am going to include it in a hotfix that is coming


Allow to drop elements into the page over the layout

This is a simpler solution than the one in #270 for the same problem

Dropping the component outside the page area is working as expected by any other d&d afterwards seems to be broken.
See this is the behavior in the main branch:
https://github.com/user-attachments/assets/5334b00a-5c38-4b06-986f-23292f539117
And with the changes in this branch:
https://github.com/user-attachments/assets/92ac1893-8545-4c6e-ae7b-1feb481401a3


Frequent BackUp of metadata table to S3 or other data store

There is currently no backup of metadata table to backup data store. If this thing gets deleted or goes down then all your work is lost. Possible solutions would give admin users the rights to backup tables at defined intervals as well as ad-hoc data backup
*recurring
api.manager.backup_metadata_table(table_name='my_metadata_table', backup_location='s3', interval='hourly')
*ad-hoc
api.manager.backup_metadata_table(table_name='my_metadata_table', backup_location='s3')

I'd also be totally happy if this were just something you did from the AWS management console. An admin-facing method would be awesome though!

This is an area of active development outside the DataFS project (nice going @jgerardsimcock!). While we may include support and/or documentation for this at some point in the future, I'm going to say this is out of scope for the time being.


[SILGen] Fall back to module lookup on invalid conformance when building subst. maps for vtable thunks

class Base<T> {
  func foo<U>(arg: U) where T == Int
}
class Derived<T>: Base<T> {
  func foo(<U>(arg: U) where T: Protocol
}

Here, as usual, we combine the override substitutions T := Int with the invocation generic signature <T where T: Protocol> to form a new substitution map for the reabstraction thunk, only to realize that we cannot recover the Int : Protocol conformance from just the original substitution map. As a consequence, we end up with an invalid conformance, and a crash later on once we actually emit the thunk. As ugly as it is, falling back to module lookup in SubstitutionMap::get fixes the issue.
### FIX 2
Now that we allow purely contextual where clauses, we can face non-generic methods
with generic signatures that constrain all outer params to concrete types. E.g.
class Base<T> {
  func foo() where T == Int
}
class Derived<T>: Base<T> {
  func foo() where T: Protocol
}

Remove the assertion guarding against these generic signatures on SILFunctionType
so that we always retain the override substitutions needed to transform the derived
method's formal type when we emit a reabstraction thunk.

cc @slavapestov and @jckarter

Can you try to come up with fixes that don't break these existing invariants?

I see, so we do lose performance.. I take it whatever eventually causes the performance regression does not check if the generic signature is at all useful.

Can you try to come up with fixes that don't break these existing invariants?

I did try something, but hit another one saying ≈ non-generic SIL function type not allowed to have a generic environment. I am not well versed in this yet, so I would experiment a bit more in that direction if you think it is worth exploring as an alternative to your old PR.

If you want to pursue the second fix further, you should finish off my old PR that removes the assertion: #23291
The hard part was that we lose performance because existing optimizations are currently disabled for
generics (such as inlining) but we still want them to take place in the fully-concrete case. So some
optimization passes will need changes to handle this.

Optimization passes are interesting, but I'm hesitant about the idea of making the optimizer responsible for collecting unused generic signatures (unused because the function type is already fully substituted through canonicalization) just because we need that signature in another place within the SILGen library. Does this make sense to you?

I am going to rip off the second fix so that we can proceed with the first one. If it looks sane, that is. I have no idea how often the module lookup fallback is going to be hit in the wild though, and this does fix an edge case, so maybe we should test compiler performance?

FWIW, I refactored the first part into a (hopefully temporary) targeted fix to make sure we don't end up getting performance regressions elsewhere.

It would be more correct if we could capture the conformance for Int: Protocol we saw at the point we establish the override. Falling back to conformance lookup in the module is a bug, since it's an opportunity to potentially end up with a different answer from the first time we established the conformance.

Falling back to conformance lookup in the module is a bug, since it's an opportunity to potentially end up with a different answer from the first time we established the conformance.

Do you have an example of such a scenario? I'm curious.

If Protocol is defined in a module that doesn't itself conform Int to Protocol, then two different modules could each declare their own retroactive conformance of Int: Protocol.

Yeah, I think @jckarter's answer is right. We need to record 'override substitutions' when we check the override, just like we record substitutions for protocol witnesses.

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


Check for null in focus-trap focus() method

Putting this up just in case this ends up being the appropriate fix for this dialog issue. Else just close it.

Thanks! I'll check out this branch and add some tests that hopefully expose the error and prevent it from happening again. Also want to have some better tests for that initial focus placement.

The main issue here is dialogs that are opened as their initial state (i.e. <d2l-dialog opened>), and that initial focus happening before anything has rendered -- including the focus trap. Emil's fix prevents that error, and focus then later gets properly applied after rendering has completed. Wondering though whether that should be happening in firstUpdated instead of after a rAF. 🤷
I've added some unit tests for dialog's focus management code, since the logic around which element should be initially getting focus is pretty clear. I've confirmed that the one test would have caught this.

Looking into the Safari failures... I have a feeling initial focus handling may have never been working here and we just didn't notice.

Too many issues with the new dialog focus management tests in Firefox & Safari, so I'm going to skip them for now so we can get the fix to the JS error that's breaking FACE released. Then we can tackle those issues -- which aren't new AFAIK -- later.


Update ports section on vs-2017.3.host.json

We broke VS as part of the changes to enable HTTPS and launchSettings.json by default. VS is not passing a port for HTTP, so it's getting generated randomly (VS does some extra checks to ensure that the port is available).
Bad
{
 "ports": [
    {
      "name": "HttpsPort",
      "useHttps": true
    }
}

Good
{
 "ports": [
    {
      "name": "HttpPort",
      "useHttps": false
    },
    {
      "name": "HttpsPort",
      "useHttps": true
    }
}

Approved for preview1.


C# How to get/understand state and conversation data from storage table And Cosmos DB

Bot Info

SDK Platform:  .NET
SDK Version: SDK version 3
Deployment Environment: local development with Emulator?

Issue Description
I'm trying to understand and use Storage accounts and CosmosDB.
1. About the state
Following this , It seems my state is saved in Cosmos DB successfully. But I cannot understand contents. Is there any documents to help?

2. About the conversation data
Using the codes below, it seems to have succeed in saving conversation data using storage accounts without adding any other codes. Is there any similar way to save conversation data while using CosmosDB. And how to retrieve data from Cosmos DB or Storage Table?
protected void Application_Start()
{
var connectionString = ConfigurationManager.ConnectionStrings["StorageConnectionString"].ConnectionString;
var storageAccount = CloudStorageAccount.Parse(connectionString);
        Conversation.UpdateContainer(builder =>
        {
            builder.RegisterModule(new TableLoggerModule(storageAccount, "MessageLog"));
        });

        GlobalConfiguration.Configure(WebApiConfig.Register);
    }
}

3. About Compress
It seems a lot of data is compressed. when I check DB, it's difficult to understand. Is there any easy way to recommend?

Hi, unfortunately there is sparse documentation for managing custom state.
First you need to get reference to the storage account from the container.
using (var scope = DialogModule.BeginLifetimeScope(Conversation.Container, message))
{
    var botDataStore = scope.Resolve<IBotDataStore<BotData>>();
}

Using the botDataStore you can get various types of states (user state, conversation state etc)
var key = Address.FromActivity(activity);
var conversationData = await botDataStore.LoadAsync(key, BotStoreType.BotUserData, CancellationToken.None);

Notice, that it uses a key of type Address. You only need to pass current activity  to get a unique key for the state.
Then you can simply, read, write or flush the state.
string userEmail = conversationData.GetProperty<string>(Constants.KEY_EMAIL); // Read
conversationData.SetProperty(Constants.KEY_EMAIL, email);
await botDataStore.SaveAsync(key, BotStoreType.BotUserData, conversationData, CancellationToken.None); // Write

You're right, there is not much documentation on BotBuilder-Azure, but the code is open source so you can look it over and figure out how it works: https://github.com/Microsoft/BotBuilder-Azure/blob/master/CSharp/Library/Microsoft.Bot.Builder.Azure/DocumentDbBotDataStore.cs

The TableLoggerModule does not save conversation data, but is saving conversation history.  It is trivial to create an IActivityLogger implementation.  There is not currently a CosmosDb (i.e. DocumentDb) example, but it should not be difficult to create one based on the Azure Table example.

You would need to either reference the source directly or duplicate in your project, and change it so the compression no longer runs.  https://github.com/Microsoft/BotBuilder-Azure/blob/master/CSharp/Library/Microsoft.Bot.Builder.Azure/TableLogger.cs#L429


Impossible to add v2.3.4 TAPO Camera plugin in Homebridge

Describe Your Problem:
When I try to install the v2.3.4, I have this error message :
npm ERR! code ENOENT
npm ERR! syscall spawn git
npm ERR! path git
npm ERR! errno -4058
npm ERR! enoent An unknown git error occurred
npm ERR! enoent This is related to npm not being able to find a file.

 Click the "Preview" tab before you submit to ensure the formatting is correct. -->

GIT needs to be installed to work

GIT needs to be installed to work


Investigate what happens if different HL7 encoding characters are used

User Story
As a report stream sender, I would like to know what happens if I send HL7 encoding characters that are different than the recommend ones.
Description/Use Case
The encoding characters MSH.2 are recommended to be ^~\&, but can technically be any characters.  As we encountered with ELIMS, some implementors might not stick to those; in the elims case they adopted the 2.7 encoding character # for truncation.  We should research what happens if an entirely different set of encoding characters are used.
Risks/Impacts/Considerations
Dev Notes
You'll likely need to create an HL7 with different encoding yourself
Acceptance Criteria

[ ] It is investigated and documented in the repo what happens if different encoding characters are used for a message going through the universal pipeline

I tested this last week with something simple like %~\& instead of ^~\&, and it kinda worked, but with some errors. My impression at the time was that it wouldn't be worth the effort to investigate and figure out until we had a user who wanted to use non-standard encoding characters (the usage of ^~\%# seems common, but other than that, I haven't seen anyone deviate from the standard, and it's hard to think of a good reason to do so). Still agree it would be good to at least document whatever we can find with another round of testing on this topic.


[1255] add publishing panel

Context
Course published status
Changes proposed in this pull request
Add published status panel to the description page
Guidance to review

Ignore white space

After

 Course description page - publishing panel

Looks fine but is there test for the status panel?

Agreed. There's a lot of logic in here, and if we're not going to separate that out into a decorator then we need some testing. Not to keep banging on it, but just another reason we should use ... (blah blah blah ya we know).


Enable compile for batched RLHF generation utils

see parent issue. This issue is a real-time log of an amateur descent into compile debugging.
I'm compiling my rlhf generation utils, which uses kv-cacheing and iterative decoding (pls don't judge the code too hard, it needs cleaning up). However, compiled runs OOTB are significantly slower. It's worth noting the main generation utils in torchtune do not exhibit this behaviour. The main difference in the RLHF generation utils is that it supports batched generation with varying sequence lengths which may be left-padded (thus needing custom attention masks and input position ids).
(tune) salman@combuter:~/torchtune$ python generate_test.py rlhf_compile
Warmup run!
Time for warmup: 23.83 sec
V0823 13:04:27.827000 148928 torch/_dynamo/guards.py:2688] [0/1] [__recompiles] Recompiling function generate_next_token_with_logits in /home/salman/torchtune/torchtune/modules/rlhf/_generation.py:37
V0823 13:04:27.827000 148928 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     triggered by the following guard failure(s):
V0823 13:04:27.827000 148928 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     - 0/0: tensor 'L['input_pos']' stride mismatch at index 0. expected 130, actual 192
Output:

...
Time for inference: 27.96 sec total, 21.78 tokens/sec
Bandwidth achieved: 112.15 GB/s
Memory used: 5.90 GB

(tune) salman@combuter:~/torchtune$ python generate_test.py rlhf
Output:

...
Time for inference: 5.49 sec total, 110.99 tokens/sec
Bandwidth achieved: 571.61 GB/s
Memory used: 5.21 GB

What's going on here? Let's set the backend to cudagraphs and log:
I0823 14:22:20.582000 276899 torch/_inductor/cudagraph_trees.py:364] [__cudagraphs] recording cudagraph tree for graph without symints
V0823 14:22:20.726000 276899 torch/_inductor/cudagraph_trees.py:2099] [__cudagraphs] Running warmup of function 0
Time for warmup: 11.63 sec
V0823 14:22:20.848000 276899 torch/_dynamo/guards.py:2688] [0/1] [__recompiles] Recompiling function generate_next_token_with_logits in /home/salman/torchtune/torchtune/modules/rlhf/_generation.py:37
V0823 14:22:20.848000 276899 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     triggered by the following guard failure(s):
V0823 14:22:20.848000 276899 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     - 0/0: tensor 'L['input_pos']' stride mismatch at index 0. expected 356, actual 610
I0823 14:22:30.863000 276899 torch/_inductor/cudagraph_trees.py:364] [__cudagraphs] recording cudagraph tree for graph without symints
V0823 14:22:30.865000 276899 torch/_inductor/cudagraph_trees.py:2099] [__cudagraphs] Running warmup of function 1
V0823 14:22:30.955000 276899 torch/_inductor/cudagraph_trees.py:2101] [__cudagraphs] Running eager of function 1 because ancestor needed to warm up
/home/salman/.pyenv/versions/3.11.9/envs/tune/lib/python3.11/site-packages/torch/_inductor/cudagraph_trees.py:2282: UserWarning: Unable to hit fast path of CUDAGraphs because of pending, uninvoked backwards. Consider running with torch.no_grad() or using torch.compiler.cudagraph_mark_step_begin() before each model invocation
  warnings.warn(
V0823 14:22:31.048000 276899 torch/_inductor/cudagraph_trees.py:2101] [__cudagraphs] Running eager of function 1 because ancestor needed to warm up
V0823 14:22:31.141000 276899 torch/_inductor/cudagraph_trees.py:2101] [__cudagraphs] Running eager of function 1 because ancestor needed to warm up
V0823 14:22:31.229000 276899 torch/_inductor/cudagraph_trees.py:2101] [__cudagraphs] Running eager of function 1 because ancestor needed to warm up
... continues on max_generated_tokens times

wth??? It's not warmed up enough? And what's going on with this line?
/cudagraph_trees.py:2282: UserWarning: Unable to hit fast path of CUDAGraphs because of pending, uninvoked backwards. Consider running with torch.no_grad() or using torch.compiler.cudagraph_mark_step_begin() before each model invocation

generate_with_logits is decorated with inference_mode and I've wrapped with torch.no_grad
Anyway. I tried bumping up the number of generated tokens from 2 to 8 in the warmup run, but no avail!
What about... another identical warmup run?? Let's switch back to inductor to test this out, too.
Warmup run!
Time for warmup: 23.64 sec
Warmup run2!
Time for warmup: 0.08 sec
V0823 14:31:30.352000 291181 torch/_dynamo/guards.py:2688] [0/1] [__recompiles] Recompiling function generate_next_token_with_logits in /home/salman/torchtune/torchtune/modules/rlhf/_generation.py:37
V0823 14:31:30.352000 291181 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     triggered by the following guard failure(s):
V0823 14:31:30.352000 291181 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     - 0/0: tensor 'L['input_pos']' stride mismatch at index 0. expected 356, actual 482
...
Time for inference: 26.04 sec total, 18.47 tokens/sec
Bandwidth achieved: 95.14 GB/s
Memory used: 5.95 GB

Okay... what about 2 warmup runs, but in the second warmup run, we generate an additional token?
Warmup run!
Time for warmup: 22.77 sec
Warmup run2!
V0823 14:34:46.736000 296583 torch/_dynamo/guards.py:2688] [0/1] [__recompiles] Recompiling function generate_next_token_with_logits in /home/salman/torchtune/torchtune/modules/rlhf/_generation.py:37
V0823 14:34:46.736000 296583 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     triggered by the following guard failure(s):
V0823 14:34:46.736000 296583 torch/_dynamo/guards.py:2688] [0/1] [__recompiles]     - 0/0: tensor 'L['input_pos']' stride mismatch at index 0. expected 356, actual 357
Time for warmup: 21.48 sec
Output:
...
Time for inference: 3.12 sec total, 154.03 tokens/sec
Bandwidth achieved: 793.24 GB/s
Memory used: 5.95 GB

????
@ebsmothers @joecummings

relevant issues?
https://github.com/pytorch/pytorch/issues/120309#issuecomment-1991902257
https://github.com/pytorch/pytorch/issues/110666


chore: replace CircleCI status with GHA action status badge

Run ci.yml on master push and replace CircleCI status with GHA action status.

I think I figured out why this circle notification keeps popping up - I selected stop builds on circle. So hopefully this circle notification stops showing up.


Add a dummy CSI driver that can be used for testing controller/sidecar interactions

By using a dummy CSI-driver that implements CSI-Addons operations for known names of PVCs, it should be possible to test the whole process from user-input to updates status in the CRs.
This fake CSI-driver does not need to use any actual storage. It can just return success/failed responses without doing anything.

Instead of writing a fake CSI-driver, csi-driver-host-path can probably be used.


Move config to Sass

Currently highly experimental and incomplete.
Based around a conversation with @davidcmoulton a while back, I said I'd investigate moving all the configuration to Sass, and export to other levels (rather than Sass being exported to). Also refs https://github.com/libero/libero/issues/151#issuecomment-496418642.

Going to revert all the changes; the new/changed config is something we should do, but not right now.

I have the file being read by Twig in Browser, but we've hit https://github.com/libero/libero/issues/190.

Going to take the Twig exports out too.


Zero-shot generating of pre-trained mBART25 behaves weirdly.

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
I tried to run zero-shot generating with pre-trained mbart.cc25/model.pt. According to the description of paper, I expected it to generate something similar with source tokens. However, all it can generate are "," and "...".
Do you have any idea about this problem?
Thank you.
Code

What have you tried?
I've checked the format of sample['net_input']['src_tokens'], sample['net_input']['prev_output_tokens'], and sample['target']. They are as follows:

src: ▁risk s </s> [en_XX]
prev_output: [en_XX] ▁risk s </s>
target: ▁risk s </s> [en_XX]

It seems like the correct input format that matches the pre-training objective of mbart. But it just could not generate legal sentences. Instead, it would output something like ,,,,,,,,....
What's your environment?

fairseq Version: master:
PyTorch Version: 1.6
OS: Linux
How you installed fairseq: source
Build command you used (if compiling from source): pip install -e .
Python version: 3.7
CUDA/cuDNN version: 10.1
GPU models and configuration: V100
Any other relevant information:

Hi @yongchanghao, I am facing a similar probelm.  Any suggestions how did you solved to the problem. Thank you!

I didn't solve this problem actually. I pre-trained a new model and didn't find this problem. But the fine-tuning results are similar.

Hi @yongchanghao, Thank you for your reply.  This seems a feasible solution. Are your code and pre-trained models are available openly?  Please share if possible.


[mongo-c-driver] update to 1.27.0

Library name
mongo-c-driver
New version number
1.27.0
Other information that may be useful (release notes, etc...)
Release notes: https://github.com/mongodb/mongo-c-driver/releases/tag/1.27.0

C driver 1.27.1 has also been released. I can file a separate issue/PR, but wanted to mention in case updating https://github.com/microsoft/vcpkg/pull/38608 is preferred.

@kevinAlbs Thanks for your comment. Since the https://github.com/microsoft/vcpkg/pull/38608 has been merged, I will submit a new pr to update mongo-c-driver to 1.27.1.


[Version 3.6] Add GenerateCoverageViz pipeline step to dag.

Also slight change to BlastContigs pipeline step to generate blast.top.m8 file.
Testing steps:

Tested the benchmark sample with this dag branch on staging, and it succeeded as expected. https://staging.idseq.net/samples/12576
Tested a minimal sample with no assembled contigs with this dag branch on staging, and it also succeeded (this tests the case where some files are empty) https://staging.idseq.net/samples/12575

The coverage viz displays correctly using the files generated by this pipeline step.
Step takes about 1 minute to run for the benchmark sample, based on cloudwatch logs.

I think b) is good and is actually pretty easy to do. You change the step,
calculate length and output it to a new sqlite3 file. It can also serve
other purposes if people are trying to curate NT/NR.
On Wed, Apr 10, 2019 at 1:20 PM Charles de Bourcy<EMAIL_ADDRESS>wrote:

@cdebourcy commented on this pull request.
In idseq_dag/steps/download_accessions.py
https://github.com/chanzuckerberg/idseq-dag/pull/133#discussion_r274139349
:

   with open(accession_summary_tab, 'w') as ast:

       for accession_id, _ in accession_dict.items():

           entry = loc_dict.get(accession_id)

           if entry and len(entry) == 3:

               # The seq_len includes newline characters in the fasta file, so we must correct for this.

               seq_len = math.floor(FASTA_SEQ_LINE_LENGTH / (FASTA_SEQ_LINE_LENGTH + 1) * int(entry[2]))

If I understand correctly, the issue is that we cannot use the size of the
byterange as the sequence length, because byterange includes new-line
characters. I can see a few different options, some of which are much
better than others:
(a) Calculate all the sequence lengths from the downloaded sequence files.
This would be super wasteful to do at pipeline run time.
(b) Determine the actual sequence length while generating nt_loc_db (this
is part of the index update operation) and store it in the dict once and
for all so it can just be looked up. Probably the best option, but requires
the development/testing overhead of building a new nt_loc_db.
(c) Keep the current setup with assumed line length of 80, but do a quick
spot check in the code to confirm sequence files actually satisfy this
condition. This code can probably live in the step that generates a new
nt_loc_db when we update the index. Include an assertion with a very
explicit / verbose message for the developers if the condition fails, so
that they know how to address the problem (for example, "you need to change
the value of FASTA_SEQ_LINE_LENGTH in download_accessions.py"). For what
it's worth, I do think the line length of 80 will be a good assumption for
a long time to come. Wikipedia article says it's something of a standard:
https://en.wikipedia.org/wiki/FASTA_format
Anything I missed?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/chanzuckerberg/idseq-dag/pull/133#discussion_r274139349,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABoAQnVNdCYV_wKpD8Jxf7H2Oq5T37N2ks5vfkdwgaJpZM4cWpDr
.

In general, this is rad!!

Also, could you please test a few large, real-life samples? In order to assess performance and results for different distributions of reads across contigs and accessions.
The benchmark has a low number of reads and was generated from a small number of species covered really really well which makes it easy to assemble them into small numbers of contigs. Real samples will have a much much longer tail of singletons

Regarding utils vs static methods, it's pretty arbitrary.
The static methods are mostly reading files, and the util methods are calculations and JSON object compilation.
One other distinction is the util methods are all tested with unit tests, and the static methods aren't since I wasn't sure how to mock files.

Talked to @yunfangjuan today about this PR.
She had two concerns:

I wasn't taking paired reads into account, and she remembered some tricky logic in alignment-viz and ReadViz.jsx involving paired reads. I dug into this, and it turns out paired reads can affect the indices for the read, but not for the reference sequence. Since I only need the indices for the reference sequence to figure out where to add coverage to account for the read, I don't need to take paired reads into account for coverage viz. (in alignment viz, we also need to extract the exact aligned sequence from the read, in the correct orientation, which is why it's important)
She was concerned about my putting a lot of functions into the utils folder that were specific to the CoverageViz step. She was also concerned that my code was a bit hard to read because I didn't group my functions together well enough. I'll do a pass to package/modularize my functions into neater modules with clear inputs/outputs.

Latest commit:
Improve code organization for coverage viz pipeline step.
Removed util/coverage and moved functions into PipelineStepGenerateCoverageViz,
since those functions are specific to that one pipeline step.
Reorganized functions in PipelineStepGenerateCoverageViz so that the "main"
functions are at the top of the file.
Consolidated tests into one file.
Verified that:

Tests still pass.
The pipeline step still outputs the exact same JSON as before.


v0.4.0

Coverage remained the same at 100.0% when pulling fd6fdaffc388d81c281fb3c010350fd2a5e7247e on develop into 80688801be7a8764659ee6c9e0237139dd41a26a on master.


Console Spam with DungeonsXL

What is your Quests version / build number (do not say "latest")?
3.8.5-b64
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
https://pastebin.com/CaUBGyAm
I'm experiencing issue when I enter DungeonsXL with someone that has different version than the other (didn't test on other versions except for 1.8.9, 1.15.1, 1.15.2). My server is on 1.8.8 and uses viaversion.
The crash shows  "Could not pass event PlayerMoveEvent" when someone moves.
It seems like it crashes also when someone punches.
It doesnt happen when player with any version enters dungeon alone.
This doesnt occur when Quest isnt installed, but still occurs when DungeonsXL module isn't there (I assume that's probably problem, because Quest cant determine where player is in the world, since the world is temporary)

Edit: Players that use the same client version (1.8.8) spam the console with "event PlayerMoveEvent" "event PlayerInteractEvent".

I've moved your console error to a Pastebin. Please follow the issue instructions more carefully next time.
The commit I've pushed should resolve the issue. Look forward to its inclusion in the next release.


How to label the same object in the picture?

For example, if there are multiple people in a picture, they are labeled as person 1,person2,person3?

You'll have to add that yourself, but I can get you started. You want to modify this line to include the index of the person:
https://github.com/dbolya/yolact/blob/d50616f36002c48eda0a1e34a75eac12ef70aae0/eval.py#L187
Use some data structure to keep track of how many of each class have been detected beforehand and just put that number in the string next to the class. If you want to do a quick and dirty solution you can just replace that with text_str = '%s %d' % (_class, j) but that's not exactly what you want.

I'm sorry, I didn't make it clear, but I'm asking how do I label the same kinds of objects when I make my own data set.

Oh, just create an annotation object for each of them with the same image ID. So if "person" is category id 1, and this label is for image id 1234, you'd have both
{
"id" : someIDForPerson1,
"image_id" : 1234,
"category_id" : 1,
"segmentation" : ...
},
{
"id" : someIDForPerson2,
"image_id" : 1234,
"category_id" : 1,
"segmentation" : ...
}, ...

You can look at the annotation files that COCO provides for reference.

Thank you so much!


Automatic Backup doesn't work

Seems simple enough but it doesn't do anything:

Have those checked but doesn't backup to cloud or local. How is it triggered?

It seems to work for me -- everytime you save your .kdbx file, it will make a backup of the previous copy, for example, I keep a Remote copy, and it creates the backups right next to my remote file:
PasswordFile.kdbx
PasswordFile_Backup_2017-07-29-17-06-34.kdbx
PasswordFile_Backup_2017-08-01-23-14-40.kdbx

More details of the behavior are in #61.

My bad, it is actually Backup on Remote not Backup to Remote.

Thanks for pointing me to this unclear wording. I will improve documentation and/or wording for this.


[#1906] Fixed bugs within connected service editor.

#1906
===
There was a bug in the ConnectedServiceEditor code that was causing botframework-config to throw, which then broke the saga that originally opened the add service context menu. This subsequently broke all future interactions with the service sagas, preventing the context menu or service buttons from working anymore.
I added code to initialize the QnA Maker service editor with a sample host name to satisfy botframework-config and prevent the error from being thrown.
There was also another bug where the learnMoreLinkButton function was being called like a getter even though it was a function, and so none of the service editors were actually rendering the learn more links, which is now fixed.

Coverage increased (+0.1%) to 68.673% when pulling 64dfdaf7515f7f612fcdd285be8fb9552853dc0f on toanzian/acc-#1906 into c2bee8bf2f68f790aee0a5ab18a66d4d7dc1b1d4 on master.


Support gRPC on standby masters

What changes are proposed in this pull request?
Implemented a RpcServerStandbyGrpcService which runs gRPC server on both standby and primary masters.
Why are the changes needed?
We are going to implement a feature which allows workers registering to all masters to speed up the master failover process. This requires standby masters to enable gRPC servers so that the works can rpc. This PR made these changes.
Does this PR introduce any user facing changes?
N/A

alluxio-bot, merge this please


Refactor Callable Bonds using ObserveAt

Also ensure that negative date offsets work afterwards.

Depends on https://github.com/digital-asset/daml-finance/issues/767


New ParisP2P Logo (alternative)?

@ernest25 suggested some changes on the logo:

use the bottom of the image as "the earth"

remove the stars in the bottom
trace an arc (not necessarily with a border)
use a different color, more blue for the bottom

non-Eiffel towers stars should be constellations

they should be in yellow
they should also have edges (but less thick than the Eiffel Tower)

A basic example quickly demonstrating the idea:

Cc @ernest25, @alexsland

@ernest25 added some photos as examples here: https://www.meetup.com/fr-FR/Paris-P2P/photos/30240035/483818312/?photoId=483818312&photoAlbumId=30240035&chapter_analytics_code=UA-124224137-4#483818312

mhhh we could also use the iconic network topology graph that are used in most P2P network (centralised / meshed / distributed) talk as "constellation"


Use port getter methods consistently

I was working on a fix for #1149 and noticed inconsistencies in how I/O
ports are referred to. This simply replaces are direct references to
fm.ports[] with the appropriate getter method. Go inlines the methods so
this doesn't incur any runtime overhead.

Oops, I've actually done exactly the same, but forgot to push. Thanks but I'll just close this now.


check_code_format / Formatting Check doesn't seem to work

See discussion on https://github.com/3rdparty/eventuals-grpc/pull/58 : the "check_code_format / Formatting Check" check seems to perpetually have the status "Expected — Waiting for status to be reported". It seems broken.

Note that we seem to have 3 Formatting Check checks: 2 that work (one push, one pull), and 1 that doesn't:

I think that the problem here is that we had a branch protection rule configured to required a "check_code_format" check that didn't actually run.
I've updated the branch protection rules to instead treat the working formatting check rules as required, and to not expect the non-existent version of the formatting check:


chore: add token package, fix prettier, trial nx-release

I'm going to close this off as per: https://github.com/leather-wallet/mono/pull/23/files#r1407523553
I'll take the good work done and create a cleaner PR


Embedding Docs

Finally, understood what's wrong with skip gate: it multiplies hidden memory with a number from 0 to 1, causing vanishing gradients. So needed to refactor this behavior. The new version is working, need to clean it up, and make different flags working for ablation tests

next: start ablation tests

I wouldn't (for now) delay this again to do more ablation tests. Let's get a working version into rasa core and then we can still iterate on it

Agree, ablation tests are just more for actual testing if several flags (on and off)  are working together without exception errors.

ah ok understood :)

Update: @tmbo reviewed the code. I'll refactor it according to his comments and add explanatory comments why certain thing were included. These comments will create the basis for the embed policy docs.

The refactor is ready! Next stop a new scheme of the policy for the docs


Definitions for webpack module API

webpack/webpack-env.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/webpack
http://bower.io/search/?q=webpack
others?

[X] has a test file? (webpack/webpack-env-tests.ts or others)
[ ] pass the Travis-CI test?

@tkqubo could you review this PR?

@vvakame I checked the changes and they look OK :rocket:
@use-strict Thanks for PR!

thanks all!


[Editor] Remove the class fooEditing from the layer when destroying it

and simplify the way to handle the different types of editors.

These type tests are painful...

/botio integrationtest


cannot install autopy

ERROR: Could not find a version that satisfies the requirement autopy==3.0.0 (from versions: 0.5, 0.51, 1.0.0, 1.0.1, 1.1.0, 1.1.1, 2.0.0, 2.1.0, 3.0.0, 3.0.1, 3.1.0, 4.0.0)
ERROR: No matching distribution found for autopy==3.0.0
This is likely caused by a bug in autopy.

i also cant able to install autopy


[REQUEST] All or nothing progress bar showing indeterminate state with elapsed time

I have a mixture of tasks which I want to display, most of which I can update the progress for, and am showing the elapsed time and NofM:
PROGRESS_BAR_COLUMNS = [
    TextColumn("[progress.description]{task.description}"),
    BarColumn(),
    TaskProgressColumn(),
    # Removing TimeRemainingColumn() from defaults, replacing with:
    TimeElapsedColumn(),
    # Add this last as have some out of N and some out of N^2:
    MofNCompleteColumn(),
]

This gives me something like this, starting with an empty (grey) bar, then a red partly filled bar, and finally a full green bar:
Indexing ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% 0:00:00  0/10
Indexing ━━━━━━╺━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━  20% 0:00:18  2/10
Indexing ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╺━━━  90% 0:00:58  9/10
Indexing ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:01:00 10/10

That is all good so far.
However, some tasks have a known number of steps (say 10), but I cannot monitor their progress, so what to show the indeterminate animated state AND the elapsed time, and then when finished show the standard green 100% bar:

I can get the indeterminate state using start=False, but then the elapsed time does not show:

In ASCII art:
Indexing ━━━━━<<━━>>━━━━━━━<<━>>━━━━━━━━<<━>>━━━━   0% -:--:--  0/10

I can get the indeterminate state using total=None, but I do have a value, and would like the % place holder.

In ASCII art total=None gave:
Indexing ━━━━━<<━━>>━━━━━━━<<━>>━━━━━━━━<<━>>━━━━  0:00:10  0/?

I'd like something like that visually but with the time, and perhaps ?/total at the end:
Indexing ━━━━━<<━━>>━━━━━━━<<━>>━━━━━━━━<<━>>━━━━   0% 0:01:23  ?/10

Is this currently not possible? Thank you!
Tested with rich 13.9.4 on Python 3.12 on macOS.

Perhaps TaskProgressColumn should show ?% for indeterminate?
Indexing ━━━━━<<━━>>━━━━━━━<<━>>━━━━━━━━<<━>>━━━━   ?% 0:01:23  ?/10


About staked

When I executed "delegated --cpu=40 --net=40", I found that the number of "STAKED" was 840000, not 800000. And I can not undelegate this extra part. Why?

Please tell us your account name so that we can look into it.

Please tell us your account name so that we can look into it.

jxnc.ptrsoft

Below is your account status, I can not verify your issue.
===========================================================
memory:
quota:     60.55 KiB    used:     4.016 KiB
net bandwidth:
staked:          4.0000 SRP           (total stake delegated from account to self)
delegated:       0.0000 SRP           (total staked delegated to account from others)
used:                 0 bytes
available:        9.588 MiB
limit:            9.588 MiB
cpu bandwidth:
staked:          4.0000 SRP           (total stake delegated from account to self)
delegated:       0.0000 SRP           (total staked delegated to account from others)
used:               354 us
available:        14.54 min
limit:            14.54 min
SRP balances:
liquid:           95.0000 SRP
staked:            8.0000 SRP
unstaking:         0.0000 SRP
total:           103.0000 SRP
producers:
aipress         bigone          bruce
foxone
===========================================================
If you can, please add more description for this issue.
For example,  you can paste the full prs-atm command you run when have this issue.

@taoran777 If there is no further progress. I will close this issue next week. Thanks.


*: uses Jsonnet to generate argo workflow

Signed-off-by: mahjonp<EMAIL_ADDRESS>What problem does this PR solve?
Handwriting argo workflow is very troublesome.
close #355 close #208  and also #321 case writing part:
编写完用例后，需要编写相应的 yaml 文件
yaml 文件大部分内容是 重复 的，而且文件行数比较多，编写、review、维护 都很心累

What is changed and how does it work?
This PR uses Jsonnet to make life easy.
For example, If I have written a test case named list-append which has three parameters: tablecount, read_lock, txn_mode.
To let it runnable, I need to fill on the metadata about list-append case on lib/case.jsonnet just like below:
list_append(args={ tablecount: '7', read_lock: '"FOR UPDATE"', txn_mode: 'pessimistic' })::
  [
    '/bin/append',
    '-table-count=%s' % args.tablecount,
    '-read-lock=%s' % args.read_lock,
    '-txn-mode=%s' % args.txn_mode,
  ],

Looks simple right? Jsonnet is a simple extension of JSON, on above I define a function named list_append which has three arguments, and returns an array.
After fill on the testcase metadata, I should write a workflow file on workflow/list-append.jsonnet, it's content:
{
  _config+:: {
    case_name: 'list_append',
    image_name: 'hub.pingcap.net/qa/tipocket',
    args+: {
      // k8s configurations
      'storage-class': 'local-storage',
      // client configurations
      client: 5,
      'request-count': 100000,
      round: 10,
    },
    command: { tablecount: '7', read_lock: '"FOR UPDATE"', txn_mode: 'pessimistic' },
  },
}

Explain a little here, _config+ means override some key-value defined on lib/config.jsonnet.
On the args+ part, I set storage-class, client and round etc parameters declared on the fixture.go file which is the common flag all over test cases.
Then, the command field is set to invoke list_append case with 7 tables, use FOR UPDATE as the read_lock.
Now I could run make build_workflow to generate the workflow file.
$ make build_workflow
find run -name "*.jsonnet" | xargs -I{} jsonnetfmt -i {}
rm -rf run/manifest/*
find run/workflow -name "*.jsonnet" -type f -exec basename {} \;  | xargs -I% sh -c 'jsonnet run/workflow/% -J run/lib | yq eval -P - > run/manifest/%.yaml'

Lucky, everything goes well. ☺
Check List
Tests

Unit test
E2E test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has Go code change
Has CI related scripts change
Has Terraform scripts change

Side effects

Breaking backward compatibility

Related changes

Need to update the documentation

Does this PR introduce a user-facing change?:

NONE

This is great! Perfect work.


Can't search the sidebar w/ x86 latest build of VCS

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

VS Code Version:
Version: 1.89.0 (Universal)
Commit: b58957e67ee1e712cebf466b995adf4c5307b2bd
Date: 2024-05-01T02:10:10.196Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Darwin x64 23.4.0
OS Version:
14.4.1 (23E224)

Steps to Reproduce:

type command + f to find a file or directory in the sidebar

Typing command + f does not allow me to search within the files/folders in the sidebar. The command does not even bring up a search dialog box.
This seems to be an issue with the x86 build as my m1 & m2 computers don't share this behavior. Typing command + f on either of those computers displays a searach box and they will highlight matching content.

@ArturoDent  Thanks, I missed that note. Thank you!


Fixing covscan issues

Hi Till,
I ran static analysis by coverity scan for cups-filters-1.20.0, which we have in Fedora 28, and fixed several issues which were reported and they are even in current version. I'll send you initial results of scan by email (in case there are some issues, which can be exploitable).
Does it look good to you? Would you mind adding it to the project?

@zdohnal, thank you very much for doing the tests and providing the patches. I have merged your patches now.

Np, all issues in report which sent you were counted as important, so there are other issues, but coverity doesn't count them as important.


reserve-with-timeout seem to be buggy

I observed that reserve with time out produces inconsistent behaviour when clients reconnect. Needs further investigation

Can you reproduce the bug?

yeah just run the server locally and run reserve with timeout from multiple clients. I can give you code to reproduce soon.


perf: memoize number formatter

Adds number formatter memoization to improve restore performance.
new Intl.NumberFormat() is very expensive operation especially for Hermes
Profiler (1-2 seconds recording):

Some more context

@headfire94 @mvayngrib Captured: https://app.asana.com/0/1202170937152401/1206553661682635


[EDITION]: R Edition

Proposal to create R versions of all solutions as appropriate:

[ ] Base README-R.md - About R (An introductory lesson)
[x] Regression: Tools
[x] Regression: Data
[x] Regression: Linear
[x] Regression: Logistic
[x] Classification: Classifiers 1
[x] Classification: Classifiers 2
[ ] Clustering: Visualize
[ ] Clustering: K-means
[ ] NLP: Translation
[ ] NLP: Reviews 1
[ ] NLP: Reviews 2
[ ] Time Series: Intro
[ ] Time Series: ARIMA
[ ] Reinforcement: QLearning
[ ] Reinforcement: gym

@R-icntay

oK, I will try to add as many R versions as I can.. @jlooper

hi @ravindranath-sawane - @R-icntay is leading this but I'd recommend chatting with him on the bulletin boards. I'll create a thread for you!

Okay, @jlooper. Definitely, we will do it together @R-icntay

@R


Juice-shop error while installing in RootTheBox

Hi there,
For juice box installation in RTB, when I am importing xml file. It is giving a database error.
I am running mysql database & latest RTB.

@jeffg2k, are you aware of any recent (?) changes to RTBs data format that might have caused this? Juice Shop CTF extension wasn't changed for a while. Last known version of RTB where the XML import worked was 3.3 according to my docs.

So far, I haven't been able to reproduce it on docker or my test instance.  From the error, it's having an issue importing the unicode of ⭐ emoji, which we included for the difficulty level.  It doesn't look like a python error though.  It looks like a database error.  @PunitTailor55 when you created your database, did you set the correct character set?  create database rootthebox character set utf8mb4;


no_gaps_when_only bug with multiple monitors

Config
dwindle {
	pseudotile=0
	no_gaps_when_only = 1
}

Screenshot

Screenshot with scaling (Now the border is weirdly offset)

Expected behavior: The second monitor's border doesn't burn into the first one

Do I understand correctly that with no_borders_when_only the border is rendered on the other monitor?

Yes

should be fixed with bec75013658836e24d4bc9d3a5a96d328b8c7bf0 but please test as i cannot atm