Any interest in packaging as an executable?

I love the project! I don't by default have node or npm installed at work so getting it setup felt like a lot of overhead.
I was thinking that packaging it with something like pkg would be helpful although does open the project to supporting different architectures.
If you're interested I can look at opening a PR.

Great idea! I was looking into ways to support at least older node versions, incl. those that are available on Ubuntu 20+ via apt. But packaging using pkg might be even better.

I couldn't get it to work with pkg. I also tried nexe but that failed, too.
Looks like node.js now has experimental support for single-executable application built-in. Thinking of trying that next but it will require to first transpiling from ESM to commonJS.

I also have not had luck with pkg or transpiling for node's built in. Still fiddling with it when I find time.

I'd love this as a packaged binary so it can be distributed in homebrew. what issues are you running into with pkg etc at the moment?

I don't remember what the issue with pkg was, but at this point I'm more interested in learning about the new node.js native support for creating executables. The next step for that is transpiling to commonJS.


[recipe] bump to 0.1.5

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


Extend generatePath api with an optional pathFunctionOptions argument

Version
v.5.2.0
Problem
Hey guys, using react-router a lot among my projects, you rock!
My team faced an issue with generatePath function in v5:
Use case is that we need to encode our query parameters with something like encodeURIComponent function, while generatePath uses encodeURIComponentPretty which is a slightly modified encodeURI function with escaped characters of /, ?, and #, due to hard-coded { pretty: true } argument inside the current implementation of generatePath.
I simply added { pretty: true } as a third default argument called pathFunctionOptions to generatePath function and wrote some tests to make sure that updated API is backward compatible.
I also updated docs with specifying differences between encodeURIComponentPretty and encodeURIComponent functions in terms of which symbols are escaped and which are not.
What I tended to achieve was { encode: encodeURIComponent } from the newest version of path-to-regexp, but as far as I know, v6 will not use this dependency which is quite great news I guess.
I choose argument name from path-to-regexp d.ts to make sure that is transparent between the libraries and looks similar to previous implementation, however, I did not yet updated react-router d.ts as long as these are cached from DefinitelyTyped and I was not sure how to perform that correctly.
The behaviour of function will change that way:
// default behaviour will remain the same as current
generatePath("/password/restore?email=:email", {
  email<EMAIL_ADDRESS>});
// Will return<EMAIL_ADDRESS>
// However it is possible to specify the `{ pretty: false }` directly
generatePath(
  "/password/restore?email=:email",
  { email<EMAIL_ADDRESS>},
  { pretty: false }
);
// Will return "/password/restore?email=emailWith%2BSign%40gmail.com"

Here are the 8 characters encoded differently between flags { pretty: true } and { pretty: false }

I would appreciate any feedback and thanks again for your job. Cheers!

I just realised that last v5 release was on may 2020 and it looks like there were no further movement in v5 version 😅
Are there any plans on v5.x releases or it would be only major v6.x updates in the future?

We'll do 5.x releases as needed. We did 3.x releases well into the 5.x release cycle (for peerDep updates, mostly).
I think this looks good. I'll give a chance for others to comment on it before merging it in.

Thanks for everybody's work on this. I'm closing issues that are old, if they are still problems please open a new issue and reference this one. Thanks again!


[frontend] Disable submit buttons for all in-flight requests

Simultaneous requests lead to race condition, causing this incident.
The race condition comes from serializer is_valid and save being not an atomic operation so validators might be invalidated in the meantime between the two function calls. It looks like the official view from rest_framework is that this should just be tolerated (see, eg. their issue 3876). The suggestion is perhaps to add an exception handler to re-raise it as a HTTP 400 / 409.
We can mitigate this from frontend by avoiding accidental simultaneous calls: let's disable all buttons as soon as the request is in-flight.

I think a robust fix involves a frontend refactor. (We have this disabling for a few of the buttons, but it's a lot of duplicated code. It's probably time to make a robust "loading-button" class or smth)
If this increases in urgency, then consider a cheap quick fix and refactor later


[Bug]: ChromaClient times out on Vercel's Edge network

What happened?
const client = new ChromaClient({
    path: process.env.CHROMA_URL,
  });
console.log(client.heartbeat())

Is timing out on Vercel's Edge network, without sending a request through to the server that's hosting Chroma. I can access the server fine from other projects, or with the same file but not using the edge runtime. But not on the Edge.
There's no error returned, it just hangs.
I'm guessing its something to do with the APIs supported on the Edge by Vercel: https://vercel.com/docs/functions/edge-functions/edge-runtime#supported-apis ??
@moreno1123 reported a similar problem on langchain, but I think it's a chromaclient issue, hence raising here.
https://github.com/langchain-ai/langchainjs/issues/3521
Versions
ChromaDB 1.6.1

That is interesting, do you have a stack trace you could share? Not sure what the env differences on vercel edge functions could be.

I also raised this with the Vercel team
On Tue, Dec 12, 2023 at 9:01 AM Hammad Bashir @.***>
wrote:

That is interesting, do you have a stack trace you could share? Not sure
what the env differences on vercel edge functions could be.
—
Reply to this email directly, view it on GitHub
https://github.com/chroma-core/chroma/issues/1502#issuecomment-1852443057,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGZWEGEJCIMB4K7HHB7UPDYJCEV5AVCNFSM6AAAAABARQNTHWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJSGQ2DGMBVG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks both.
@HammadB I attempted to get a stack trace using this code, but nothing particularly useful in the logs.
// Connecting to Chroma with a timeout to try and catch the stack
  const timeout = (ms: any) => new Promise((_, reject) => setTimeout(() => reject(new Error('Operation timed out')), ms));

  try {
    const client = new ChromaClient({
      path: process.env.CHROMA_URL,
    });
    console.trace();
    // Use Promise.race to throw an error if the heartbeat takes longer than 5 seconds
    const result = await Promise.race([
      client.heartbeat(),
      timeout(5000) // 5 seconds timeout
    ]);
    console.log(result);
  } catch (error) {
    const e = error as Error;
    console.error('Error occurred:', e.message);
    // Couple of different attempts at getting a stack trace
    console.error('Stack Trace:', e.stack);
    console.trace();
  }
```
Logs
```
Error occurred: Operation timed out
Stack Trace: Error: Operation timed out
    at (app/api/chat/route.ts:57:82)
```

The route.ts in question is here for  context: https://github.com/DaveCBeck/Chat-with-LLM/blob/VercelEdge/frontend/app/api/chat/route.ts  - its a very-slightly-adapted version of a Vercel template.

any updates?

Just for info @moreno1123, as a temporary workaround (assuming Vercel or Chroma will sort this at some point) you can comment out the edge portion of the code and it runs fine. Obviously not for production use but for testing or low-volume use I've not had any problems.
app/api/chat/route.ts
// off the Edge for now, because otherwise the ChromaClient times out without sending a request to the server.
//export const runtime = 'edge'
export const maxDuration = 300; // comment this out if running on the edge


Create storage account

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?

Why?

Additional context

Checklist

[x] Implementation tested (with at least one cloud provider)
[ ] Error handling code meets the guideline
[ ] Logging code meets the guideline (TODO)
[ ] OpenAPI and Postman files updated, client regenerated (make generate-client) (if needed)
[ ] User guide and development docs updated (if needed)
[ ] Related Helm chart(s) updated (if needed)

To Do

[ ] If the PR is not complete but you want to discuss the approach, list what remains to be done here

@waleed5544 can you elaborate more on the use case this PR tries to solve?


Appendicies do not appear in TOC appropriately

Appendices, depending on how the section numbering is set up, may appear in tables of contents with incorrect letters or with a phantom . before the letter.

Can't reproduce anymore


Automate test with github actions

Had to create seperate branch, for some reason clicking the "Create a branch for this issue" did not work with making a branch for making the github action. Not sure why.

Hi,
I just saw this video and apparently, you can set this up using the PkgTemplate https://youtu.be/C9hGkFbJLl4?t=897
I have tried to use the template to create the actions including a tagbot which we are going to need later and the actions to automatically create a documentation page. I have attached the files here.
I think we should copy the actions from here and the docs folder. Then we would also follow the standard layout and ect.
SARProcessing.zip


Support enums on array parameters

Describe the PR
Adds support for including enums on array query parameters. If the parameter type is an array, the enum values are added to the items field on the parameter instead.
Relation issue
https://github.com/swaggo/swag/issues/918
Additional context
None

Thanks @sdghchj! Do I need to do anything further to trigger the CI checks?

@lorenzodonini Thanks for your contribution!


[Extensibility Request] Page 96 "Sales Cr. Memo Subform" - Procedure from Local to Global

Hello, we need the following currently Local procedure to be set as Global. Can you please check this? Thank you!
procedure ExplodeBOM()

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release.
We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues.
If you have follow-up questions or requests, please create a new issue where you reference this one.


How to prevent groupby.agg(list) produce columns __reduced__ ?

I have to remove those __reduced__  in case
df = df[[c for c in df.columns if c !='__reduced__']]

Hi @eromoe, thanks for your report.
Could you please elaborate on your use case of groupby.agg(list)? I've tried a couple of common scenarios for an aggregation specified by a list and didn't get any __reduced__ columns in the result:
import modin.pandas as pd

df = pd.DataFrame({"a": [1, 2, 2], "b": [3, 4, 5]})
print(df.groupby("a").agg(["sum", "mean"]))

series = pd.Series([1, 2, 3])
print(series.groupby([1, 2, 2]).agg(["sum", "mean"]))

Output:
    b
  sum mean
a
1   3  3.0
2   9  4.5

   sum  mean
1    1   1.0
2    5   2.5

@eromoe feel free to reopen if needed


Printing mistake ADD->Add

Printing mistake ADD->Add

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

ID: 431df4fc-4168-cc8f-d053-9b264d2971c1
Version Independent ID: cd157d04-d596-83fa-6edb-24d52193eb34
Content: Add, download, and delete user data to Identity in an ASP.NET Core project
Content Source: aspnetcore/security/authentication/add-user-data.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Visual Studio UI uses ADD. Include a screen shot of what's wrong.


Added missing accessors support, join() and list*(), unit tests updated

Hello! This PR is to add support for:

accessor operators
join() function
improve list*() function with support of different resources

To sum up, LARK grammar was updated, typing.py and transformer.py to parse new types of values.
Please review :)

you have to run pre-commit run -a then it will fix the linting issues for you. I sadly have no access to your branch to do the fix on my own

you have to run pre-commit run -a then it will fix the linting issues for you. I sadly have no access to your branch to do the fix on my own

Branch is in our company's Github organization, not sure if access can be granted, but I can ask.
So far, I reviewed pre-commit logs from CI run and committed few changes to the code.

Codecov Report

Merging #137 (96ff61c) into master (8b856c3) will decrease coverage by 0.24%.
Report is 8 commits behind head on master.
The diff coverage is 96.90%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #137      +/-   ##
==========================================
- Coverage   99.75%   99.52%   -0.24%
==========================================
  Files           5        5
  Lines        1210     1253      +43
==========================================
+ Hits         1207     1247      +40
- Misses          3        6       +3

Flag
Coverage Δ

unittests
99.52% <96.90%> (-0.24%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

pycep/transformer.py
99.46% <94.73%> (-0.54%)
:arrow_down:

pycep/typing.py
100.00% <100.00%> (ø)


Release 0.2.3

Hey @dyoo, would you want to add more documentation or settings before we release this? I tested it on Comprehensive Rust and the auto-skipping of code blocks removes ~800 lines from the PO files (out of ~20k lines). We will be able to remove more lines when we start adding skip comments and when https://github.com/google/comprehensive-rust/issues/531 is fixed.

I can't think of anything else to add at the moment.  Please proceed!  :)
(Apologies; didn't have as much time last week, and probably this one,  to help contribute as much as I'd like.)

I can't think of anything else to add at the moment. Please proceed! :)

Thanks @dyoo!

(Apologies; didn't have as much time last week, and probably this one, to help contribute as much as I'd like.)

All good, your contributions are already very helpful, so thank you for that!


Is this gem still currently supported and working on the latest version of rails as of 7/9/15?

I'm using it in a Rails 5 project. 12/7/16. Works for me.

It seems to me that the owner of this repo @mixonic got into JavaScript and Ember.js development and is no longer really maintaining this?

@bxt I'm still around.

Is this gem still currently supported?

As with most OSS, "support" is a vague status. Do I still have access to the repo? Yes. Have changes landed to master in the last year? Yes. Do I have any active projects myself using this library? No. Are there open PRs that are behind on review? Probably.
Is that enough for your needs? Only you know.

Is this gem still working on the latest version of rails?

I have no idea. You can find a very detailed list of what library versions are supported in the travis.yml file.
I'm happy to help anyone who uses this library in a real app and would like to step up as a maintainer. You can find my email pretty readily :-D


POST API chain issue, observation?.includes is not a function on large data

Hi, i am using post api chain, i am retrieving property information from an api, each property has around 100 fields that i am getting,sending payload and getting response, issue is when the api response contain large data like 10 properties returning i am facing error of "observation?.includes is not a function" when i ask questions in chatbot, but when it has 2 or 3 properties it predicts the answer right and doesn't throw error.
On large data response i am getting issue.
On small api response:

On large api response:

On both cases the format of response is same.

same


Needs some updates

Get-AzureADPolicyAppliedObject -ObjectId should be Get-AzureADPolicyAppliedObject -id
Remove-AzureADApplicationPolicy -ObjectId should be Remove-AzureADApplicationPolicy -id
"Step 2: Locate the service principal to which to assign the policy" In this section it should be added that it can be obtained by running get-azureadserviceprincipal.

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

ID: fc5a36a7-2acb-c7d5-4629-e8c1f026cad3
Version Independent ID: 1749bb9d-766e-c639-20a2-c22749448822
Content: Configure sign-in auto-acceleration for an application using a Home Realm Discovery policy
Content Source: articles/active-directory/manage-apps/configure-authentication-for-federated-users-portal.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @msmimart
Microsoft Alias: mimart

@stephenc01  Thanks for your feedback! We will investigate and update as appropriate.

Thank you for that information @stephenc01 ! We are currently making those changes and appreciate you bringing these issues to our attention. As this has been addressed, will be closing this issue out now.


Implement interfaces to allow mocking for unit testing

Would it be possible to implement interfaces of the SimpleTcpClient and Message classes to allow mocking in unit tests.

I suppose we could, what's your unit test scenario where that would be beneficial?

I'm writing tests to ensure that the SimpleTcpClient is called with the expected ip and port number, and set with the expected properties (Delimiter for example).
I can get so far wrapping the SimpleTcpClient in our own code but this method falls down due to Message having an internal constructor. So the simplest solution would be to provide interfaces.
I'm happy to add them and raise a PR?


Ticket install does not replace existing ticket

When installing a ticket that is already existing (ie. to replace a bad ticket with a good one) FBI will show a successful install yet will not actually replace the ticket, nor does it give any indicator of the fact. Need to delete the old ticket before installing the new one.

I've always observed the same behavior when installing older firmware files as well. Get prompted to delete the old one first, but the install doesn't happen. Always have to install a second time (so the deleting part does work, in that case, just not the install after the delete.

Already taken care of in latest source.


BLD: fix ReadTheDocs build.  Outdated NumPy gave a conflict with MPL.

There are official NumPy Windows wheels now, so no need for installing
a very old version from a Rackspace server.
[ci skip]

Just going to merge this so I can see if the RTD build now works.

that worked


README: Clarified IndifferentAccess docs

Tried to make the IndifferentAccess documentation clearer.
Closes #340

:+1: Thanks!


Failed to send some messages ; Negative SMTP reply: 501

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
This issue has been discussed several times though not necessarily recently nor in a way to solve my particular  issue.
Expected behavior
email should be sent
Actual behavior
cannot send email.   when trying to send an email, this error comes up Failed to send some messages ; Negative SMTP reply: 501; sometimes with an invalid email address and sometimes with a syntax error.  I have several email accounts and it happens now on two of them only.  I send to the exact same email address, from contacts in all cases. so i cannot be typing the email address wrong.
Steps to reproduce
1.click new email, enter recipient first few letters then select name (from contacts list)  Also tried manually typing in address not in contacts list
2.provide a subject, some text
3.send - then error and message is moved to drafts folder.  Outbox
Environment
K-9 Mail version:5.203
Android version:6
Account type (IMAP, POP3, WebDAV/Exchange):imap

Additional information.  I was probably using BCC addresses and no TO and no CC.  when it first started - not 100% certain.  Also, when I try to send now, is says  Sending 0/2.   even though I am only sending one.  Based upon some of the other threads on this I've read, something is stuck in the "out" but the outbox is empty and I don't know what or where else to clear it from.

Please record a debug log while trying to send a message. https://github.com/k9mail/k-9/wiki/LoggingErrors

Probably, this issue can be closed, as no debug log was provided. I've tried to reproduce it on newer k-9 version (5.403), and email is actually sent.


Update-Help fails to update 'Whea' module

Windows Server 2016 v1607 Build 14393.2155
PS C:\Users\Administrator> update-help
update-help : Failed to update Help for the module(s) 'Whea' with UI culture(s) {en-US} : Unable to retrieve the HelpInfo XML file for UI
culture en-US. Make sure the HelpInfoUri property in the module manifest is valid or check your network connection and then try the command
again.
At line:1 char:1

update-help

  + CategoryInfo          : ResourceUnavailable: (:) [Update-Help], Exception
  + FullyQualifiedErrorId : UnableToRetrieveHelpInfoXml,Microsoft.PowerShell.Commands.UpdateHelpCommand

same on windows server 2012 v6.2.9200

+1 I'm using 2016 too

@officedocsbot assign @yogkumgit

I think these questions are more suitable for support services, @yogkumgit can you kindly take over? Thanks

@kukucz Thank you very much for posting this issue. Considering the nature of the issue, it is advised that please open a service ticket in your tenant and follow with them for the resolution. If you wish to know how to open the service ticket, please follow this link:
https://docs.microsoft.com/powershell/#pivot=main&panel=community
We will be very happy if you can share the outcome or resolution with us. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1894 ms

Resolved: Bofrost.at performance has improved in e450145 after 6 minutes.


[Bug]: Prompt for inconsistency with the design draft when the name is duplicated or starts with a space

ODC version
odc424
OB version

What happened?

What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?
Create a custom environment with duplicate names
Anything else we need to know?
No response
Cloud
No response

Fixed In Commit f403ce9d29fa6429e0389d00f6fcd204da56e0e8
------
陈朝俊
Tue Feb 20 18:36:45 2024 +0800

Inconsistent with the design draft when the name is duplicated

Fixed In Commit 575dbf9bf5215cf46ea8f34aa9a017c66f659bc5
------
陈朝俊
Wed Mar 6 17:34:24 2024 +0800

pass


version 2022.5.0 regression error: ConnectTimeoutError: Connect timeout on endpoint URL: "http://<IP_ADDRESS>/latest/api/token"

ConnectTimeoutError: Connect timeout on endpoint URL: "http://<IP_ADDRESS>/latest/api/token" error occurs with version 2022.5.0 but not with 2022.3.0.
Failing code running in a databricks notebook
Python 3.7.5
%pip install openpyxl s3fs==2022.5.0

import pandas as pd
import openpyxl
#import s3fs
import copy
import numpy as np
import datetime

pdf = pd.read_csv("s3://bucket/path/file.txt", sep = '|', header = 'infer')

This looks like it is happening deep inside aiobotocore. Can you please check is something similar has been reported with them, or if you see the same error with different versions of aiobotocore (with matching botocore, of course).

Also seeing this.
aiobotocore==2.3.2
boto3==1.21.21
botocore==1.24.21
s3fs==2022.5.0
More details to follow

Sorry I don't have any clear way to debug this. I hope we hear back from aiobotocore.

Sorry I don't have any clear way to debug this. I hope we hear back from aiobotocore.

Indeed. may be fixed now https://github.com/aio-libs/aiobotocore/commit/14f2dd1231c6aa84655b6e0211c367ab10980d6c


Browse for list of tag providers?

Node-RED @ 3.1.0
node-red-contrib-ignition-nodes @ 1.5.10
Ignition @ 8.1.31
Ignition Node-RED Module @ 1.5.15
I am trying to replicate the results obtained by calling the following within Ignition (8.1) script console:
print system.tag.browse("")
Using 'ignition tag browse' node, I am struggling to obtain a list of tag providers.
If I configure an 'inject' node with the following:
msg.payload = {     "defaultTagProvider": "",     "tagPath": "" }
I receive an error:

No tag path supplied

If I configure the 'inject' node with the following:
msg.payload = {     "defaultTagProvider": "",     "tagPath": "/" }
I receive an error:

Error executing command: java.lang.Exception: provider not found: edge

Interestingly, within the Ignition Server Configuration Node, I've left the 'Default Tag Provider" blank (the text 'default' is shown as default text - in grey). I've confirmed that adding a default provider here changes the error received above.
Is it possible to browse for tag providers?

Fixed this issue in 1.5.16:
https://github.com/IgnitionModuleDevelopmentCommunity/IgnitionNode-RED/releases/tag/1.5.16


scrappy should reward those with 7+ streak not troll them

Made a few edits to the code. Yay!!! Thanks for the PR :)


Readme and release updates

Let's use this pull request to do last minute housekeeping on the documentation before we merge into master and do a new release

@shirubana  I updated v0.2.4 to use absolute module imports, and rebased this branch Readme-and-Release-Updates to bring it up to speed with v0.2.4.  See if this fixes your import issues in Jupyter.

@cdeline so I updated the readme and the bifacial_radiance header notes on this version. I thought it had not been merged since the branch was still moving forward ....


@magic-sdk/provider EthNetworkName missing "goerli"

✅ Prerequisites

[x] Did you perform a cursory search of open issues? Is this feature already requested elsewhere?
[x] Are you reporting to the correct repository (magic-sdk)?

✨ Feature Request
@magic-sdk/provider EthNetworkName is missing "goerli".

🧩 Context

Thanks for opening this issue. it got fixed in<EMAIL_ADDRESS>closing this issue.

Wow. That was fast. Thanks a lot @harryEth!


Fix android test targetSdk

Fixing failures from https://github.com/square/okhttp/pull/8156

@swankjesse requesting post review

LGTM


Use std::env::consts::ARCH when CNB_TARGET_ARCH isn't set

This is a workaround for the specific CNB lifecycle and platform API versions the salesforce-functions builder must support.

For posterity, this is to fix:
===> DETECTING
Timer: Detector started at 2024-05-23T14:50:05Z
heroku/jvm                  5.0.0
heroku/maven                5.0.0
heroku/jvm-function-invoker 5.0.0
Timer: Detector ran for 30.837415ms and ended at 2024-05-23T14:50:05Z
===> RESTORING
Timer: Restorer started at 2024-05-23T14:50:05Z
Timer: Restorer ran for 337.339µs and ended at 2024-05-23T14:50:05Z
===> BUILDING
Timer: Builder started at 2024-05-23T14:50:05Z
thread 'main' panicked at buildpacks/jvm/src/main.rs:121:22:
arch should be always parseable, buildpack will not run on unsupported architectures.: UnsupportedArchError("")

Seen in:
https://github.com/heroku/cnb-builder-images/actions/runs/9209191519/job/25336538201#step:6:23


Avoid logging for non-TFJob pod

In AddPod, it logs "This pod's job does not exist" even for non-TFJob pod. we only need to log for TFJob pod.

This change is

Travis tests have failed
Hey @jian-he,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
2nd Build

     gometalinter --config=linter_config.json --vendor ./...

cmd/tf-operator.v2/app/options/options.go:1::warning: file is not goimported (goimports)

3rd Build

     gometalinter --config=linter_config.json --vendor ./...

cmd/tf-operator.v2/app/options/options.go:1::warning: file is not goimported (goimports)

travis_time:end:1c89f9dc:start=1535146275530537533,finish=1535146398383676277,duration=122853138744

Coverage decreased (-0.04%) to 57.986% when pulling 6e8b71481aecd35c223be4b9d722d8e489f98da7 on jian-he:logging into 3061493b14c0a9a8c8d7ba0661818304f33438e0 on kubeflow:master.

@ScorpioCPH / @gaocegege
The build passed, could you check again.


tikv-ctl compact run infinitely

Bug Report
When tikv-ctl --host is set with url with a schema like <IP_ADDRESS>:20161, compact runs successfully and returns soon. But once the url has a schema like http://<IP_ADDRESS>:20161, compact will run infinitely and actually it does nothing.

I would like to take this
 pub fn secure_connect(mut self, addr: &str, mut creds: ChannelCredentials) -> Channel {
            let args = self.prepare_connect_args();
           // does the http prefix has to be removed?
            let addr = CString::new(addr).unwrap();
            let addr_ptr = addr.as_ptr();
            let channel = unsafe {
                grpc_sys::grpc_secure_channel_create(
                    creds.as_mut_ptr(),
                    addr_ptr,
                    args.args,
                    ptr::null_mut(),
                )
            };

            Channel::new(self.env.pick_cq(), self.env, channel)
        }

does the http prefix has to be removed on connect?

@Connor1996 We need to find out why it can not work when has http:// prefix.

@sch00lb0y only remove http prefix is not elegant, any invalid url will cause it run infinitely. IMO, it is better to return some error info.

Looks like grpc dns resolution failed.
GRPC_VERBOSITY=debug RUST_BACKTRACE=1 tikv-ctl --host http://<IP_ADDRESS>:2379 compact
I1012 18:30:00.103591814   19552 ev_epoll1_linux.c:93]       grpc epoll fd: 3
D1012 18:30:00.103678277   19552 ev_posix.c:107]             Using polling engine: epoll1
D1012 18:30:00.103723979   19552 dns_resolver.c:301]         Using native dns resolver
I1012 18:30:00.157116868   19554 dns_resolver.c:170]         dns resolution failed (will retry): {"created":"@1539340200.157073771","description":"OS Error","errno":-2,"file":"/home/jenkins/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.2.3/grpc/src/core/lib/iomgr/resolve_address_posix.c","file_line":104,"os_error":"Name or service not known","syscall":"getaddrinfo","target_address":"http://<IP_ADDRESS>:2379"}
D1012 18:30:00.157149170   19554 dns_resolver.c:176]         retrying in 1.000000000 seconds
I1012 18:30:01.206784191   19554 dns_resolver.c:170]         dns resolution failed (will retry): {"created":"@1539340201.206747007","description":"OS Error","errno":-2,"file":"/home/jenkins/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.2.3/grpc/src/core/lib/iomgr/resolve_address_posix.c","file_line":104,"os_error":"Name or service not known","syscall":"getaddrinfo","target_address":"http://<IP_ADDRESS>:2379"}
D1012 18:30:01.206818784   19554 dns_resolver.c:176]         retrying in 1.000000000 seconds
I1012 18:30:02.258244236   19554 dns_resolver.c:170]         dns resolution failed (will retry): {"created":"@1539340202.258199163","description":"OS Error","errno":-2,"file":"/home/jenkins/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.2.3/grpc/src/core/lib/iomgr/resolve_address_posix.c","file_line":104,"os_error":"Name or service not known","syscall":"getaddrinfo","target_address":"http://<IP_ADDRESS>:2379"}
D1012 18:30:02.258287685   19554 dns_resolver.c:176]         retrying in 1.000000000 seconds
I1012 18:30:03.312975073   19554 dns_resolver.c:170]         dns resolution failed (will retry): {"created":"@1539340203.312935103","description":"OS Error","errno":-2,"file":"/home/jenkins/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.2.3/grpc/src/core/lib/iomgr/resolve_address_posix.c","file_line":104,"os_error":"Name or service not known","syscall":"getaddrinfo","target_address":"http://<IP_ADDRESS>:2379"}
D1012 18:30:03.313031626   19554 dns_resolver.c:176]         retrying in 1.000000000 seconds

@Connor1996 @zhangjinpeng1987 @overvenus I did some investigation on this with help of log.
Error is happening here https://github.com/grpc/grpc/blob/master/src/core/lib/iomgr/resolve_address_posix.cc#L87
cause for the bug is not parsing the hostname and port properly.(https://github.com/grpc/grpc/blob/master/src/core/lib/gpr/host_port.cc#L41)
They passing http://localhost:80 in getaddrinfo instead of localhost:80
I'm not sure whether it is not convention to use http.
But the problem is not in tikv, It's in grpc core library.

And I've tried this with sample server. It was panicked and exited.
When I tried to compile tikv with latest grpc-rs,I was not able to compile.
I think, This issue will be solved with the new grpc-rs

@overvenus /cc

Yes, it is solved with grpc-rs 0.4.
GRPC_VERBOSITY=debug RUST_BACKTRACE=1 ./tikv-ctl --host http://<IP_ADDRESS>:2379 metrics
D1017 00:01:47.164783511   66453 ev_posix.cc:145]            Using polling engine: epollex
D1017 00:01:47.164902734   66453 dns_resolver.cc:339]        Using native dns resolver
D1017 00:01:47.165200193   66453 dns_resolver.cc:280]        Start resolving.
I1017 00:01:47.217677672   66454 dns_resolver.cc:210]        dns resolution failed (will retry): {"created":"@1539705707.217636686","description":"OS Error","errno":-2,"file":"/home/jenkins/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.4.0/grpc/src/core/lib/iomgr/resolve_address_posix.cc","file_line":108,"os_error":"Name or service not known","syscall":"getaddrinfo","target_address":"http://<IP_ADDRESS>:2379"}
D1017 00:01:47.217708128   66454 dns_resolver.cc:221]        retrying in 1000 milliseconds
DebugClient::metrics: RpcFailure(RpcStatus { status: Unavailable, details: Some("Name resolution failure") })

@sch00lb0y Thanks a lot😄


Add page for PLATEAU documents

Please help update relaton.org to show how to cite PLATEAU documents. Thanks.
Originally posted by @ronaldtse in https://github.com/relaton/relaton-plateau/issues/3#issuecomment-2249458295

Added

@andrew2net thanks. Next time can you make a PR and then in the commit write “fixes #N” then when the PR merges it automatically closes? Thanks.


MatchSpec parses incorrectly when channel is passes as a URL

What happened?
While using the py-rattler library, I discovered a possible problem with how MatchSpecs are parsed when passing the channel name as a URL
>>> from rattler import MatchSpec
>>> m = MatchSpec("https://conda.anaconda.org/conda-forge::python[version=3.9]")
>>> m
MatchSpec("*")
>>> print(m.channel)
None

The following happens when specifying the channel as a MatchSpec key:
>>> from rattler import MatchSpec
>>> m = MatchSpec("python[channel=https://conda.anaconda.org/conda-forge,version=3.9]")
InvalidMatchSpecException                 Traceback (most recent call last)
Cell In[21], line 1
----> 1 m = MatchSpec("python[channel=https://conda.anaconda.org/python/conda,version=3.9]")

File ~/opt/testminiconda/envs/pyrattler/lib/python3.12/site-packages/rattler/match_spec/match_spec.py:85, in MatchSpec.__init__(self, spec, strict)
     83 def __init__(self, spec: str, strict: bool = False) -> None:
     84     if isinstance(spec, str):
---> 85         self._match_spec = PyMatchSpec(spec, strict)
     86     else:
     87         raise TypeError(
     88             "MatchSpec constructor received unsupported type" f" {type(spec).__name__!r} for the 'spec' parameter"
     89         )

InvalidMatchSpecException: invalid bracket key: channel

What should happen?
I would expect rattler to be able to parse full URLs properly. This is currently also a bug in conda as reported here:

https://github.com/conda/conda/issues/14074

But, micromamba appears to be parsing this correctly.

Just to verify. The channel name of https://software.repos.intel.com/python/conda should be conda right? Not python/conda.

We just merged #792 which fixes:
>>> from rattler import MatchSpec
>>> m = MatchSpec("https://conda.anaconda.org/conda-forge::python[version=3.9]")
>>> m
MatchSpec("conda-forge::python ==3.9")
>>> m.channel
Channel(name="conda-forge", base_url="https://conda.anaconda.org/conda-forge/")

Parsing python[channel=https://conda.anaconda.org/python/conda,version=3.9]  still fails because we dont recognize the channel key yet. That will be next.


bug: mainnet bounty shows wrong text when logged on rinkerby

Issue
Mainnet bounty shows wrong text when logged on rinkerby network via metmask

Steps to Reproduce
Login into the test net (rinkerby) via metamask and open a bounty funded on the mainnet like
https://beta.bounties.network/bounty/v1/486
Fix
Update the content to use the current active network rather than hardcoding the message

@villanuevawill opened up the issue here ! Sorry about that

villanuevawill commented 2 hours ago
@thelostone-mc - 485 and 487 show up. This is a pretty extreme edge case where 486 does not exist on the rinkeby testnet. Most ids will exist from the mainnet that exist on rinkeby. We should def. update this, but I think there are only 3 ids that don't exist in rinkeby that exist in mainnet.

Basically the way the Bounties Network operates is that we use whichever Ethereum network gets injected by MetaMask, whether it's MainNet or Rinkeby. A fix for this could entail us embedding the network type directly in the URL, but this would likely break our existing URL structure.


Swipe on Sidenav

I got it to work, still need to make it more flexible and think about a solution for multiple sidenavs. But first, do you want it?

I will use / like it.

Yeah! It will be a very good feature. But, first, are you using a custom library, like Hammer.js? Or you built the feature by hand?

No library

Nice. I would love to see a PR with this improvement.

So, @igor-ribeiro, still no PR ? I must say I am looking forward to it 🙂

Sorry, guys. So busy at work. I will send a PR this week!


#157 Add viem support and related tests

Related issue: https://github.com/cgewecke/hardhat-gas-reporter/issues/157
This PR

adds viem support and tests
upgrades typescript
replaces EGRDataCollectionProvider with SnifferProvider
integrates extendEnvironment to inject sniffer provider

@cgewecke any feedback on this?

@muhammetaliakbay @gionn Thanks so much for this and apologies for taking so long to review.
Could someone link to the relevant code at viem or hardhat where the freezing described below happens?

Injecting the modified provider in TASK_TEST_RUN_MOCHA_TESTS sub task is too late as viem reads and freezes the provider instance on the module load

I'm a a little worried about configuring the provider for gas logging in an extendEnvironment hook because AFAIK.it will execute for every task hardhat runs. It might come into conflict with other plugins and is difficult to enable/disable conditionally.
That said there is also a new extendProvider hook available in more recent HH versions which seems to be designed for this kind of application. Does using that have same freezing problem?
(Will pull this PR down this week and take a closer look to see if there are any other issues...thanks for all the work you've already put into this).

pls merge this


[Feature Request]: Feat:adding a new page to display all contributors✨

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add a page to showcase all the contributors of this website.
@kartikayasijaa can you assign me this issue,

Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@T-Rahul-prabhu-38 New page meaning

ill create a seperate page to show the repo stats and top contributors

Add features like integrating auto correct feature in chat raise issue I will assign it
@T-Rahul-prabhu-38

Closing not required


fix: give instructions for cmd too

I am skittish about this PR potentially not handling something like spaces in paths, I don't have time to think through and test it.


Clarify and Clean Up Use of Klf in CFE

As discussed in #220, all the CFE implementations currently use an expressly provided value for Klf - the lateral flow parameter for the soil reservoir.  The value is either hard-coded or an external configuration parameter.   However, there are several things that suggest this is not how CFE should function.
Code was written for deriving the Klf value from other parameter values:
// Equation 10 in parameter equivalence document.
lateral_flow_linear_reservoir_constant=2.0*assumed_near_channel_water_table_slope*NWM_soil_params.mult*
                                       NWM_soil_params.satdk*NWM_soil_params.D*drainage_density_km_per_km2;   // m/s
lateral_flow_linear_reservoir_constant*=3600.0;  // convert to m/h

Further, in the latest revision of the original stand-alone version, this is the line where the soil reservoir struct's member variable for Klf is set:
soil_reservoir.coeff_secondary=0.01;  // 0.0 to deactiv. else =lateral_flow_linear_reservoir_constant;   // m per h
Clarification is needed on how this should be implemented, with the code adjusted if needed and any misleading/unnecessary comments and code cleaned up.

@madMatchstick Would you take a quick look at this in relation to the current noaa-owp/CFE code and see if this is still relevant to the  current state of the code there?  If so, we may want to migrate this issue.


Remote patron activity sync celery task (PP-1164)

Description
This PR updates our Patron API that sync with remote providers to do the sync as an async celery task, rather then in the main web thread worker. To coordinate these patron activity sync tasks, I wired up redis as a service that we can use to store data, not just as a celery broker.
Redis is used to store the state of the patron activity sync and as a distributed lock, so that only one task is doing a sync at a time.
🚨 This needs a PR to update our deployments, since we have new environment variables. I'm going to leave it in draft until its ready to be merged.
Motivation and Context
This should improve our speed and responsiveness when syncing the loans feed and will help with the issues we have been seeing in CA where a provider going down can cause the whole CM to become unresponsive because we are waiting for so long for the remote requests.
JIRA:

PP-1164
PP-1113

Both those tasks in JIRA ended up being worked on together in this one, since it wasn't easy to pull them apart.
How Has This Been Tested?

Tested locally
Unit tests

Checklist

[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.

I'm doing some local testing on this, and getting the PR ready for our hosting environment, but I think while that is happening this is ready for a code review.

Once https://github.com/ThePalaceProject/circulation/pull/1899 goes in, there will be a couple minor conflicts that need to be resolved with this one.


http-parser-js can throw?

It's a little unclear to me whether and when http-parser-js.execute will return or throw errors. See, https://github.com/creationix/http-parser-js/blob/master/http-parser.js#L126.
Any thoughts? Do we need a try/catch around parser.execute(...)?

Nevermind. It only happens when calling the virtual methods.


Fix the call to undefined function

The function getUserIDs() is called getResultIDs() in the UserList class.

Thanks!


Uploaded diagrams for illustrating basic pod architecture.

I’m uploading some new diagrams in SVG and AI format to help address issue #84

Thanks a ton, @tpryan !


Problema com desempenho

 Environment data

Language Server version: 2024.5.102
OS and version: win32 x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: true
python.analysis.typeCheckingMode: off

 Code Snippet

XXX

 Repro Steps

XXX

 Expected behavior
XXX
 Actual behavior
XXX
 Logs

XXX

Thanks for the report. Unfortunately, this is not enough information for us to investigate this issue.
If this continues to be a problem, would you please reopen the issue and add the information described in the troubleshooting guide?


[3.0-Triple] fix grpc-message messy code

What is the purpose of the change
https://github.com/apache/dubbo/issues/9342
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #9375 (7194035) into 3.0 (78fd454) will decrease coverage by 0.22%.
The diff coverage is 63.63%.

@@             Coverage Diff              @@
##                3.0    #9375      +/-   ##
============================================
- Coverage     65.28%   65.06%   -0.23%
  Complexity      342      342
============================================
  Files          1202     1202
  Lines         51878    51825      -53
  Branches       7729     7674      -55
============================================
- Hits          33870    33721     -149
- Misses        14391    14487      +96
  Partials       3617     3617

Impacted Files
Coverage Δ

...bbo/rpc/protocol/tri/InboundTransportObserver.java
44.44% <0.00%> (ø)

...ache/dubbo/rpc/protocol/tri/UnaryClientStream.java
27.14% <0.00%> (ø)

.../org/apache/dubbo/rpc/protocol/tri/GrpcStatus.java
45.45% <60.00%> (+0.19%)
:arrow_up:

.../apache/dubbo/rpc/protocol/tri/AbstractStream.java
70.96% <100.00%> (+0.31%)
:arrow_up:

...a/org/apache/dubbo/rpc/filter/AccessLogFilter.java
24.35% <0.00%> (-44.88%)
:arrow_down:

...va/org/apache/dubbo/rpc/support/AccessLogData.java
53.16% <0.00%> (-37.98%)
:arrow_down:

...pache/dubbo/common/function/ThrowableFunction.java
71.42% <0.00%> (-28.58%)
:arrow_down:

...bo/rpc/cluster/support/FailbackClusterInvoker.java
59.37% <0.00%> (-14.07%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

... and 48 more

Continue to review full report at Codecov.

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


i need help with dx11

when I try to use exclusive Fullscreen it got back to windowed mode do u know how to fix?

Are you using any overlays or anything?

no

Are you using any overlays or anything?

I should mention that exclusive fullscreen only does that if another window pops on screen, as that's what it's supposed to do. Some other window or something is likely trying to show itself, which is probably why Roblox is doing that. That's just my guess though. I'll close this since this isn't a Bloxstrap problem.

okay thx for telling e that


Add ability to use extensions in docs

At the moment, simulators in the docs cannot use extensions as noted in Microsoft/pxt-arcade#242.
However, there is a way to include external packages for blocks rendering using the ```package macro.
This uses those packages and sends them to the embedded simulator through the url like the code is sent.
Once received, when generating the initial files for the project, this adds those dependencies to the pxt.json so it loads them just like a normal project.
fixes Microsoft/pxt-arcade#242

Status?

On the back-burner while working on micro:bit bugs. I should be able to incorporate the changes over the weekend.


CSimple language support

Fix #2036
A draft to discuss.
Depends on https://github.com/apache/camel/pull/4708

I am improving some parts after the feedback from @jamesnetherton

Already cherry-picked in camel-master


Bump rust-rocksdb to 0.21.0

Description
rust-rocksdb 0.21.0 ships several build fixes compared to 0.19.0 and uses RocksDB 8.0.0, with more bug fixes and performance improvements.
Linked Issues
None
Testing
No testing, we shouldn't see any behavior difference.

I wonder to see this PR has impact on benchmark we have.

I don't expect to see much of a difference, but who knows. I read that they changed how CPU feature detection is done at compile-time, that might have some impact. I'll look into it 🙂


Provide a per-client-thread mutable bucket for arbitrary data

When working with Locust, you can arbitrarily add fields to the client object, and this is available anywhere the client is invoked.  This ultimately allows the sharing of arbitrary data between tasks within a taskset.
My general idea is to add a single field to the GooseClientState struct, which itself is either a HashMap or a BTreeMap of mutex-wrapped-generics. Then, when writing a load test, you could optionally create a custom Struct with whatever fields you want to share between tasks within a taskset and read/write from/to it as needed.
There are advantages to both structures, likely it would be best to add two buckets to the load test: one that's a HashMap, and one that's a BTreeMap. If a load test doesn't use a bucket, there's not enough overhead to matter. In general the preference would be to use the HashMap for best performance, and the BTreeMap when you need to do more complex comparisons etc.
A nice overview of the two collection types, and the advantages/disadvantages of each:
https://doc.rust-lang.org/std/collections/index.html

I believe the problem w/ my proposed solution is how we're storing client states in a lazy_static. I believe this requires us to define the object that will be in the HashMap and/or BTreeMap at compile time, which makes it far less useful.
Perhaps this could be solved through some unsafe code, it needs further thought/architecture.

Here's the direction I was originally thinking -- this doesn't compile as we have to declare the type in the lazy_static (which is visible in the first line of the patch) and this would prevent users from declaring their own arbitrary structures as desired:
diff --git a/src/lib.rs b/src/lib.rs
index 0dd1a3a..52cd63a 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -340,7 +340,7 @@ lazy_static! {
     static ref CLIENT: RwLock<Vec<GooseClientState>> = RwLock::new(Vec::new());
 }

-struct GooseClientState {
+struct GooseClientState<T> {
     /// A Reqwest client, wrapped in a Mutex as a read-write copy is always needed to
     /// manage things like sessions and cookies.
     client: Mutex<Client>,
@@ -350,8 +350,9 @@ struct GooseClientState {
     /// Integer value indicating which task within the current sequenced bucket is currently
     /// running.
     weighted_bucket_position: AtomicUsize,
+    storage: HashMap<String, T>,
 }
-impl GooseClientState {
+impl<T> GooseClientState<T> {
     // Initialize one client per thread.
     async fn initialize(clients: usize) {
         // Grab a write lock to initialize state for each client.
@@ -377,6 +378,7 @@ impl GooseClientState {
                 client,
                 weighted_bucket: AtomicUsize::new(0),
                 weighted_bucket_position: AtomicUsize::new(0),
+                storage: HashMap::new(),
             });
         }
     }

Discussed with @LionsAd who proposed something like:
pub struct MyUserData {
  count: u32;
}
impl GooseUserData for MyUserData {
}
impl MyUserData {
    pub fn new() -> Self {
        MyUserData {
          counter: 0,
        }
    }
}
my_user_data = MyUserData::new();
client.set_user_data(Box::new(my_user_data));

Inspired by:
https://github.com/terry90/rs-simple-proxy/blob/master/src/lib.rs

Fixed by https://github.com/tag1consulting/goose/pull/355


Update Readme - correct link to gh-pages source

Thanks for fixing this.  Merged.

oops;  missed one at bottom of the page; one more PR coming!


Documentation using ReadTheDocs and MyST

I can use the same setup as shot-scraper: https://shot-scraper.datasette.io/en/stable/ / https://github.com/simonw/shot-scraper/tree/main/docs

I just made this change: https://github.com/simonw/dclient/blob/8a205d81a3d0d6525d59d69f0f45d82c4918dc31/dclient/cli.py#L8-L9
So having dclient.datasette.io is a good idea as it makes that a bit more self-documenting.

Now live at https://dclient.datasette.io/


Normalize uid argument to set_prognoses

set_prognoses is either
def set_prognoses(self, sim, target_uids, source_uids=None)

or
def set_prognoses(self, sim, uids, source_uids=None)

Let's pick one and be consistent.

I don't especially like the name set_prognoses -- I feel this is maybe a holdover from Covasim where we pre-computed the prognoses for the entire disease course, but we won't necessarily be doing that here. In particular, what I see the set_prognoses function doing is:

Appending to the infection log
Modifying state values (e.g. self.susceptible[uids] = False)
In some cases, then actually setting prognoses (e.g. self.ti_primary[uids] = sim.ti + rr(self.pars.dur_exposed.rvs(uids) / sim.dt)

I wonder if we should rename make_new_cases to choose_cases, and set_prognoses to make_cases or something like that?
I also don't love "cases". For infectious diseases, "case" sometimes means only a diagnosed infection. I propose that we use the term infection in Infection and its subclasses, and do not define this method for Disease, since it may or may not be applicable. For example, I could imagine an Obesity module wouldn't involve a make_new_cases method that would flip agents from one state to another, but would instead track continuous variables, and over a certain threshold agents would be defined as being a "case".

Closed by #527


Add variable width histogram aggregation

Closes #5400

Failure in 7.6 is a bit odd and I believe unrelated. I can replicate using the exact same seed only. Needs some investigation but not related to this PR.


prevent travis from randomly failing

travis is randomly failing because of: constantTimeCompare
I guess we don't want to increase the alpha time
What about we change the travis script to
npm run check || npm run check || npm run check
So the tests are running at least three time before failing

If the failing test is constantEquals exposes no timings, then the correct thing to do would be to either make the alpha larger (e.g. 0.005) or increase iterations.  There's a trade-off; making alpha larger makes the test less sensitive.  Increasing iterations takes longer.
It is probably preferable to increase the iterations.
It probably would not make sense to run the tests three times, which is roughly the same as saying "there's no timing attack ... 1 out of 3 times." :smile:
The other possibility is that constantTimeCompare is not actually constant time.  Likely if that has happened it is because of optimizations by the interpreter.  This'll happen regularly because optimizations are changing all the time, and vary on engine; the only correct long-term solution will eventually be to use WebCrypto API.

Yeah, true, good point :smile:
What about the iterations depends whether 'CI' in process.env is true
so keep 2500 for local
And maybe 25000 for travis ?

@mastilver That's a great idea.

I'm reopening this one as tests are still failing randomly
@ericelliott Could you disable greenkeeper while we figure out how to make test more stable

Done.

The underlying reason that the tests still fail is, quite possibly, because the constant-time comparison is never constant.
The Node.js folks discussed this at some length some time ago, with the upshot being:

In Javascript there's no way to get around the optimizations of the compiler, interpreter, and processor, so constant time is nigh-unachievable; and
a wisely-implemented Double-HMAC may achieve the equivalent by blinding the timing channel.

I would err on the side of preferring double-HMAC to constant-time comparison.  It doesn't look like a big fix, but one would have to wrap their head about it.
With a blind timing channel the stats tests ought to pass.

I'm open to a PR.

Closing all stale issues and PRs. Please update, ensure all tests pass, and reopen if you really want this in.


Nova wallet link doesn't work

Problem
When I tried to install "Nova wallet" extention through our dapp, I cann't  install it with "install" button.

It forwards me to the wrong page.
Suggestion
It happens because there is wrong URL. When i copy URL to click-board,  I got this - "hhttps://novawallet.io". As you see there is extra "h" as you can see.
Please, fix it.

fixed


Suggestion : Advanced scaling / scheduling options

At points I reach a limit where I would like to see more options in regards to scaling.
For example ;
For several design's, I want to scale in terms of physical resources by adding (virtual/cloud) machines. I've setup my services so that they a container runs on each host. This renders me a template/blueprint per host, of the services that should run on it. Though this has the disadvantage that I can only run 1 container per service per host. I've often pondered if having the option to select # containers per host wouldn't solve that for me.
Another approach would be to enhance the scheduling rules and allow mathematic formula's. Where one could say to run a minimum & maximum amount of containers of X services. The drawback would be that you would have to have an additional setting at the service, where you would say "fill to max" (or something like that), as otherwise the service might always run in degraded mode.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


fcntl F_DUPFD_CLOEXEC has different values on Solaris and Illumos

Some changes occurred in solarish module
cc @jclulow,@pfmooney

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) soon.
Please see the contribution instructions for more information.

@bors r+

:pushpin: Commit 89aacef2eca3434e68afc1804279c3c29bd8c5a7 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 89aacef2eca3434e68afc1804279c3c29bd8c5a7 with merge 6ab03598bce2308c77ecf03bb67ea71c395601a6...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 6ab03598bce2308c77ecf03bb67ea71c395601a6 to master...


Re-org XPU finetune images

Description

Rename xpu finetune image from ipex-llm-finetune-qlora-xpu to ipex-llm-finetune-xpu.
Add axolotl to xpu finetune image.
Upgrade peft to 0.10.0, transformers to 4.36.0.
Add accelerate default config to home.

1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] N/A
[ ] Unit test
[ ] Application test
[ ] Document test
[ ] ...

5. New dependencies

[ ] New Python dependencies
- Dependency1
- Dependency2
- ...
[ ] New Java/Scala dependencies and their license
- Dependency1 and license1
- Dependency2 and license2
- ...

need a quick start for "Finetune with docker on (multi-)GPUs"
https://github.com/analytics-zoo/nano/issues/1361#issuecomment-2105506189

need a quick start for "Finetune with docker on (multi-)GPUs" analytics-zoo/nano#1361 (comment)

Yes. We can open another PR for this.


Tweak common.c code, remove some duplication

close #160

Merging, thank you for the review @jfclere


String interpolation different color than strings.

String interpolation should be a different color than the rest of the strings. If this is already a feature, what would the syntax for custom themes be?

Are you referring to format! etc?

no. like
println!("{}", x)
the {} and what ever is inside it should be a different color

I think this is done.


Grt: Add -drc_report_iter_step option

Report the DRC on each iteration of overflow. By default only the final report is generated. (Discussed in #3583)

@eder-matheus any comments?

@luis201420 What is the default behavior? In the https://github.com/The-OpenROAD-Project/OpenROAD/issues/3583 issue, it is required that the reports should be generated by default. If I understand correctly, this code is not generating the reports by default.
@oharboe What should be the default interval between iterations to generate the congestion report?

@eder-matheus The OpenROAD tool should not change behavior, but I have discussed with @maliberty the ORFS policy on when and how to generate interim .odb files, so I suggest you discuss with him. I am not the only user who has concerns on this and Matt is a bit more on top of other inputs.

I think it  should be off by default at the OR level and we can discuss the default in ORFS


Is this library intended to be browser-compatible?

The browser logos in the readme would seem to imply so, but https://github.com/es-shims/RegExp.prototype.flags/commit/c3a37276764d99c1e4f7e9467ad636fce8c92c58 introduced a transitive dependency on has-symbols (via es-abstract/helpers/callBind) which references a property on global, which doesn't exist in browser environments.
@ljharb just wondering if this is expected, or an oversight.

It's not an oversight at all; every bundler transforms global to window for the browser, and if they don't, then they'll be unable to correctly bundle a great many npm dependencies.

every bundler transforms global to window for the browser

Webpack currently "polyfills" global (along with other Node-isms) by default, but this feature might be removed in v5, which is around the corner. It's been encouraged to disable it manually anyway.

if they don't, then they'll be unable to correctly bundle a great many npm dependencies

I'm not sure what you mean by "a great many", but this doesn't seem consistent with my experience. We've had Webpack's automatic polyfill feature disabled at my company for ~6 months, and across a few web apps bundling thousands of runtime npm dependencies<EMAIL_ADDRESS>is the first one for which this has been an issue. (I'm sure it's not the first one to reference global, but typically I've seen modules check for global's existence before accessing properties on it.) I'm sure some other modules do this too but it seems overwhelmingly uncommon, at least amongst packages which intend to be browser-compatible. Do you have other examples in mind?
Thanks for your work maintaining these shims, btw 😊 and no problem if you don't have the time or interest to continue the conversation.

If webpack does decide to remove it in v5 - which I very much hope they don't - then obviously I'd have to update has-symbols to use https://npmjs.com/globalthis.
I reference global in a number of my modules, so while your experience is interesting I think it's anecdotal.


Random 454 4.7.1 Relay access denied, no logging for this error

Problem

Getting random rejections:  454 4.7.1 Relay access denied

Sometimes right after starting the container
Sometimes after 3 or more retries
Sometimes if I insist retrying, I get only rejections

None of these rejections are displayed in the output log, not even with debugging enabled (POSTFIX_debug_peer_list=<IP_ADDRESS>/0)
Any otherinfo, like if dkim has worn permissions on the key file or delivery successes are displayed.

Deployment

As a pod, in kubernetes 1.18
Configuration is almost basic, nothing fancy. Did relax everything that I could.

/etc/postfix/main.cf
# See /usr/share/postfix/main.cf.dist for a commented, more complete version

# Debian specific:  Specifying a file name will cause the first
# line of that file to be used as the name.  The Debian default
# is /etc/mailname.
#myorigin = /etc/mailname

smtpd_banner = $myhostname ESMTP $mail_name (Debian/GNU)
biff = no

# appending .domain is the MUA's job.
append_dot_mydomain = no

# Uncomment the next line to generate "delayed mail" warnings
#delay_warning_time = 4h

readme_directory = no

# See http://www.postfix.org/COMPATIBILITY_README.html -- default to 2 on
# fresh installs.
compatibility_level = 2

# TLS parameters
smtpd_tls_cert_file=/etc/ssl/certs/ssl-cert-snakeoil.pem
smtpd_tls_key_file=/etc/ssl/private/ssl-cert-snakeoil.key
smtpd_use_tls=yes
smtpd_tls_session_cache_database = btree:${data_directory}/smtpd_scache
smtp_tls_session_cache_database = btree:${data_directory}/smtp_scache

# See /usr/share/doc/postfix/TLS_README.gz in the postfix-doc package for
# information on enabling SSL in the smtp client.

smtpd_relay_restrictions = permit_mynetworks permit_sasl_authenticated permit_inet_interfaces reject_unauth_destination
myhostname = revomatico.com
alias_maps = hash:/etc/aliases
alias_database = hash:/etc/aliases
mydestination = localhost
relayhost =
mynetworks = <IP_ADDRESS>/0
mailbox_size_limit = 0
recipient_delimiter = +
inet_interfaces = all
inet_protocols = all
smtp_tls_security_level = may
smtpd_tls_security_level = none
milter_protocol = 2
milter_default_action = accept
smtpd_milters = inet:localhost:12301

/etc/opendkim.conf
InternalHosts <IP_ADDRESS>/0, ::/0
KeyTable refile:/etc/opendkim/KeyTable
Mode sv
SigningTable refile:/etc/opendkim/SigningTable
Socket inet:12301@localhost
Syslog yes
UMask 002

I am so sorry for opening this, just found the actual issue:

Our kubernetes configuration tried to use this smtp server by an external IP.
This external IP would reach in theory the kubernetes smtp service
On the physical kuberentes nodes there was also the default postfix service
This would sometimes answer the connections on the external IP, and since it was configured to be restrictive, would reject conenctions.

So, closing this. Thank you for this awesome project!

No problem! good you fixed it


(? - 23w31a) Highlighted text field

This texture file was added in 23w32a, so I presume this was a hardcoded, generated texture before this point.


implement ddim demo

Implemented a demo for generating pictures based on ddim sampling
reference
https://github.com/luping-liu/PNDM
https://github.com/ermongroup/ddim

Closing this pull request since it will be supported in https://github.com/open-mmlab/mmediting/issues/1260.
@plyfager Please involve @yanniangu  in the new pull request.


Ref to a DOM Element

How to refer to dom created by yew like React ref callback.
React Refs and the DOM

#332 is a duplicate but contains more info, I'd move the conversation there.


Add Chinese Simplified Translation

I translate it, thank you for your nice app!
Here's some screenshot

Hello, thanks for your contribution. I've added one more string in the latest beta, could you please translate it? I'll be happy to get the translation into the next release then.

OK I will take care of them

Thanks, LGTM


Star Info - Timers changed again

As of the October 25, 2023 update, all star tiers last exactly 7 minutes.
Ref: https://secure.runescape.com/m=news/forestry-part-two?oldschool=true

@pwatts6060 I opened a PR which should have everything necessary for the latest update. I dramatically simplified the code since a lot of it isn't needed anymore. I've also updated the README and version number so it should be as easy as merging it and then creating the PR in the runelite plugin-hub repository.


Feature: client url upload

Add upload_from_url method to Python client

Code changes look good. Just one comment about docstring. Also looks like CI is failing because numpy 2.0+ is getting installed now and numpy 2.0 removes np.float_ and related types

@ulya-tkch I'll go ahead and mass update to np.float64. let me know if you have any issue with this. thanks for flagging @axl1313


How to Share from PowerShell to Javascript?

The package and version I'm asking about:

Question
How do I share variables from PowerShell to Javascript? All of the documentation uses C# and F# as the example. I assumed it would work similarly, but I cannot get it to work.
I know it should be possible since these [enhancement] tagged issues are closed:

https://github.com/dotnet/interactive/issues/1648
https://github.com/dotnet/interactive/issues/1551

But there doesn't seem to be any documentation on how to share between the PowerShell kernel and Javascript kernel.
When I try to just use the #!share magic in the same way as I would for C# or F#, it doesn't work.

I used pwsh worked. Did try just test in the js cell. Changed to the console.log, and then it double printed.

Thanks @dfinke! Looking at your screenshot helped solve it. I didn't realize that the !#share magic is language specific. None of the documents I read (unless I just overlooked it) mentioned that. Using the UI to select the language works, as does running the #!share magic after using a language selection magic.

Thanks again for the assistance, closing this out.


Adding optional income/tax chart

pull bot closed this 1 minute ago
frankly speaking, don't know what does it mean


[Enhancement] implement exit command on path input

Would be cool to exit the program if you enter exit when asked for profiles.ini path
shadowfox_windows_amd64.exe V1.40

Since v1.4.1 the input field no longer exists, so I guess this can be closed


[ADD] xifetch and xifetch-rs

I did not copy the whole project to this repo. I just used submodule, it may have problems.
~THanks!!!

I think there are .git directories which you added. Could you delete them and also .gitmodule. Rename the directories with the technology's name. Such as python or rust

If I try to enter xifetch directory that redirects me to https://github.com/reilinux/xifetch. Please make sure that everything is right

that is why i used submodules

You shouldn't make submodules. Just copy your own neofetch project into this repo. and push it. See no one did like this

Will you commit?

~OKAY!!!


Support Verification in SDK - SD-JWT-VC

Our aim is to bring verification capabilities across the different SDK platforms. We will do so by allowing Edge Wallets to initiate a ProofRequest to a known Identifier (DID), also this same Edge wallet need to be capable of Generating the Proof and send it back. This would basically allow any Edge wallet to initiate and complete verification requests by itself with Credential Type SD JWT. As this is a Selective Disclosure JWT, during the verification process the Verifier will receive both the shared disclosures and the SD JWT Credential.
User Cases
High level description SD-JWT-Based Verifiable Credentials: An Introduction (criipto.com)
Current Draft Spec https://datatracker.ietf.org/doc/draft-ietf-oauth-selective-disclosure-jwt/
EBSI Spec https://hub.ebsi.eu/vc-framework/did/selective-disclosure-sd-jwt
Age Verification: For services that require age verification, such as venues where entrants need to be over 18. Entrants can share their driving licence as verifiable presentation and security can receive on a mobile device and automatically confirm that they are over the age of 18 without actually disclosing their age.
Voting Systems: In digital voting systems, edge wallets can store digital identities, allowing citizens to vote securely and anonymously, ensuring the integrity of the electoral process.
Using Presentation exchange protocol
Linked Issues

[x] hyperledger/identus-cloud-agent/issues/1421
[x] hyperledger/identus-edge-agent-sdk-ts/issues/315
[ ] hyperledger/identus-edge-agent-sdk-kmp/issues/214
[x] hyperledger/identus-edge-agent-sdk-swift/issues/177

linked issue https://github.com/hyperledger/identus-cloud-agent/issues/1421

https://github.com/hyperledger/identus-cloud-agent/issues/1430


Trouble getting chalet to work with svelte-kit/vite

Trying to figure out how to get a svelte-kit app (https://kit.svelte.dev) working in chalet. I can get the app running, but it keeps reloading. Svelte-kit apps use vite (https://vitejs.dev) for hot-module-reloading in development, and vite is trying to connect to wss://my-app.localhost:3000 and failing, so it tries reloading, infinitely. Not sure how to get this working so that chalet will forward those web socket connections through to the vite server. Here's my app config:
{
  "cwd": "/Users/pascal/Projects/my-svelte-app",
  "cmd": "npm run dev",
  "xfwd": true,
  "out": "app.log",
  "env": {
    "PATH": "[omitted]",
    "PORT": 3000
  }
}

I have self-signed SSL working, so the app loads at https://my-app.localhost, but it can't reach wss:my-app.localhost:3000
Will keep trying and report what I find but if anyone has any guidance I'd appreciate it.

Follow up: I was trying again to get vite working in chalet, and I noticed that some of the example servers in the readme include --port $PORT, so I tried that with my vite app and it works! Including wss/hmr (web sockets / hot module replacement)
{
  "cwd": "/Users/pascal/Projects/vite-chalet",
  "cmd": "npm run dev -- --port $PORT",
  "env": {
    "PATH": "[omitted]"
  }
}

Opened a PR to aid future travelers down this road https://github.com/jeansaad/chalet/pull/44


PIXI.ParticleContainer and unexpected interaction behavior from children

If I create a particle container
var container = new PIXI.ParticleContainer(15000, {
  scale: true,
  position: true,
  rotation: true
});
// Instead of a regular container
// var particles = new PIXI.Container();

And set
container.interactiveChildren = true;

And also create a sprite:
function onMouseDown() {
  alert('on mouse down!');
}

sprite = new PIXI.Sprite.fromImage('zaku.gif');
sprite.scale.x = sprite.scale.y = Math.random() * 2 + 1;
sprite.position.x = width * Math.random();
sprite.position.y = height * Math.random();
sprite.rotation = Math.PI * 2 * Math.random();
sprite.interactive = true;
sprite.mousedown = onMouseDown;
container.addChild(sprite);
}

It will call the onMouseDown() callback only if you click in the top, left corner of the screen, no matter where the sprite is.
I would expect either interaction to not be supported for PIXI.ParticleContainer child sprites or for the mousedown callback to only fire when you actually click the sprite like a regular PIXI.Container.

I thin this might have something to do with the child sprite's worldTransform not being updated which is necessarily for containsPoint() to be calculated correctly.

Interactions and other advanced features do not work on particle container children. All it does is draw, really fast.

Makes sense. So I shouldn't have even set interactive=true for the Sprite. Any behavior observed after that is undefined. 👍


Using vnc backend configured to use ssl throws an invalid rfb version error

When I'm connecting to x11vnc started with -ssl it fails and the connection closes
The log I get from xrdp at the point of error looks like:
[20190826-22:41:41] [DEBUG] xrdp_wm_log_msg: VNC started connecting
[20190826-22:41:41] [DEBUG] xrdp_wm_log_msg: VNC connecting to <IP_ADDRESS> 5900
[20190826-22:41:42] [DEBUG] xrdp_wm_log_msg: VNC tcp connected
[20190826-22:41:43] [DEBUG] VNC error 1 after security negotiation
[20190826-22:41:43] [DEBUG] VNC error before sending share flag
[20190826-22:41:43] [DEBUG] VNC error before receiving server init
[20190826-22:41:43] [DEBUG] VNC error before receiving pixel format
[20190826-22:41:43] [DEBUG] VNC error before receiving name length
[20190826-22:41:43] [DEBUG] VNC error before receiving name
[20190826-22:41:43] [DEBUG] xrdp_wm_log_msg: VNC error - problem connecting
[20190826-22:41:43] [DEBUG] Closed socket 20 (AF_INET6 ::1 port 34984)
and on the x11vnc side the logs look like:
02/08/2019 01:55:36 check_vnc_tls_mode: waited: 1.422010 / 1.40 input: (future) RFB Handshake
02/08/2019 01:55:36 check_vnc_tls_mode: version: 3.3
02/08/2019 01:55:36 check_vnc_tls_mode: invalid version: 'RFB 003.003'
02/08/2019 01:55:36 SSL: ssl_helper[11580]: exit case 2 (ssl_init failed)

VNC with SSL is not supported in xrdp.

So SSH tunneling only option here. Thanks
On Tue, 29 Oct 2019, 02:38 metalefty<EMAIL_ADDRESS>wrote:

VNC with SSL is not supported in xrdp.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neutrinolabs/xrdp/issues/1396?email_source=notifications&email_token=AC3XNEY5DGJMZCQT366TBDTQQ6OZ3A5CNFSM4IPXBJNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECPBEOY#issuecomment-547230267,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3XNE7ROP5SFIT6CU63RLLQQ6OZ3ANCNFSM4IPXBJNA
.


Probability

Hello, I'm trying to learn processing and I'm working on this exercise :
int [] stuff = new int[5]
stuff[0] = 1;
stuff[1] = 1;
stuff[2] = 2;
stuff[3] = 3;
stuff[4] = 3;
int index = int(random(stuff.length));
But I have an error : "Syntax error, maybe a missing ] character?"
Could you help me ?

You miss the ";" at line 1 or maybe instead of int [] you should write int[]

You missed a " ; " perhaps is that

Thanks for the answer ! Now my code is :
int[] stuff = new int[5];
stuff[0] = 1;
stuff[1] = 1;
stuff[2] = 2;
stuff[3] = 3;
stuff[4] = 3;
int index = int(random(stuff.length));
And I have another error : "Missing right curly bracket "}" " and "unexpected token"...

"unexpected token: [" exactly.

Ok, I had to add {} : int[] stuff = new int[5]; {
stuff [0] = 1;
stuff [1] = 1;
stuff [2] = 2;
stuff [3] = 3;
stuff [4] = 3;
int index = int(random(stuff.length));
}
but why ?


10-decimal float32 parse error

There is a bug in parsing numbers with 10 decimal digits.
To reproduce, add next test case to TestDecoderFloat32
{
	name:           "10-decimal-digit-float",
	json:           "0.9833984375",
	expectedResult: 0.9833984,
},

output:
--- FAIL: TestDecoderFloat32 (0.00s)
    --- FAIL: TestDecoderFloat32/10-digit-float (0.00s)
        decode_number_float_test.go:925:
                Error Trace:    decode_number_float_test.go:925
                Error:          Expected nil, but got: "Cannot unmarshal JSON to type 'int32'"
                Test:           TestDecoderFloat32/10-decimal-digit-float
                Messages:       Err must be nil
        decode_number_float_test.go:928:
                Error Trace:    decode_number_float_test.go:928
                Error:          Not equal:
                                expected: 983398
                                actual  : 0
                Test:           TestDecoderFloat32/10-decimal-digit-float
                Messages:       v must be equal to 0.983398

First problem, it shouldn't fail. Second - error message "Cannot unmarshal JSON to type 'int32'" when parsing float32 is misleading (should it be moved to a separate issue?).
This particular error can be fixed by changing line 383 in decode_number_float.go (expI >= 11 instead of expI >= 12). I wanted to submit a pull request with the fix, however there is similar logic with numbers in exponential format (line 347) which probably should be changed too. IMO, there is not enough test cases for this part of the code to make such changes with confidence.

Hi, thanks for raising the issue.
I'm working on it.

The bug has been fixed by #116
I'm closing the issue. Sorry for the delay.


It is allowed to generate an invalid configuration file (static response with no backends)

Right now, it is possible to create a new route without any backend, and export the configuration file without any warning or error, while the configuration file is rejected by KrakenD because it contains no backends
Reproduction steps:

Create a service
Create a new route
Export the configuration file

My use case was that I was trying to create a fixture route, returning always static data. I exported the configuration that was rejected and I had to add some fake backends to the configuration to make it work

The button is disabled now until no backends are added. I have also corrected the JSON checker for the provided object.
It will be available online as soon as I merge the 0.9 branch. I will also tag the docker image as 0.9.

Docker image tagged as 0.9
Fix is live!


Migrate all tooling that refers to "module" to names that refer to "application"

Add2App introduced a new project structure known as "module" and therefore we now have multiple tooling references to "module".
Along the way the decision was made to use this new project structure for all future projects.  Therefore, this project structure, and most of the tooling around it, is not about "modules".  We should rename this tooling to use the term "project" to match the eventual --template term that will be used for it, which will read "flutter create --template project"

@Hixie FYI - this ticket is available for help from others if someone has time.  It's essentially a glorified renaming project (including renaming and updating device lab tests associated with the template tooling).

cc @gspencergoog this would fit under your tool remit quite well. :-)

After discussing offline, Matt, Ian and I decided to rename "create -t module" to "create - t application", leaving the old "create -t app", but deprecating it.

And as for the method of upgrading: I was going to have it just detect which type of existing project was there, and use that for the default type if no type was specified, but if they specify a type, then generate that type. I'm not sure about this though.  If you specify "application" in an existing "app" project, what should it do? Ideally, it would attempt to convert between the two, but I'm not sure how feasible that is. It seems like it would be complicated and error prone.

Does this include upgrading the apps in flutter/flutter/examples to use the module/application template rather than the old app template (i.e. to not have materialized hosts in them)?

@mit-mit migrating those projects makes sense to me.  They're partially a marketing tool so I suppose we should keep them up with the latest and greatest.  I'll file a ticket in the Add2App project for that.

OK, here's what I'm going to do for flutter create:

If the destination directory doesn't exist, then I'll create an "application" (née module) project by default.
If they ask for a particular template (i.e. '-t application'), and the destination doesn't exist, then I'll create that.
If they ask for a particular template (i.e. '-t application'), and the destination already exists, then I'll see if the existing project is indeed one of those, and I'll update it if is, and error out if it isn't.
If they don't ask for a particular template (the default), and the destination already exists, then I'll figure out the type of the existing project and I'll update it using what it is currently (any of "app", "application", "plugin" or "package") as the template.

I'm also going to add a "project_type" to the .metadata file that we write, and I'll make the templating system overwrite that file even if it already exists, so that when we update a project, we also update the flutter version used to update it. This will make it easier in the future to determine the type of project that is written.  For projects that don't have this, I will detect the type and then write it.

CC @devoncarew @stevemessick @DanTup to verify that @gspencergoog's proposal won't break the IDEs.

I don't think anything here impacts VS Code. Our only interaction with create is the Flutter: New Project command and it doesn't pass -t and just uses the default.

To clarify - am I right in thinking that a standard app (run from Flutter) and what was called "module" (to be used within an existing native app) are the same? If so, I guess https://github.com/Dart-Code/Dart-Code/issues/1178 is not required anymore?

Yes, I think you're correct.  What used to be created with flutter create (the "app" template) is no longer going to be the default, and instead we'll create what used to be called a module (now called the "application" template), and we'll be deprecating the old "app" template. There will no longer be a "module" template.

The next release of the Flutter plugin for IntelliJ will no longer have a
Module option in the new project wizard, so that will bring it in line with
these changes.
However, "Module" is used in a couple different contexts. We have UI
support for Android Studio that links a Flutter module/application into an
Android app. We're hanging that off the "New Module" menu item. I think
that makes sense from and Android programming viewpoint but would be
interested to know if others agree.
On Mon, Oct 1, 2018, 11:57 PM Greg Spencer<EMAIL_ADDRESS>wrote:

Yes, I think you're correct. What used to be created with flutter create
(the "app" template) is no longer going to be the default, and instead
we'll create what used to be called a module (now called the "application"
template), and we'll be deprecating the old "app" template. There will no
longer be a "module" template.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/22344#issuecomment-426168566,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIH6jcdzKGk4XW5VeCJy-J5txBrHTZgCks5ugw5ZgaJpZM4W8O7V
.

This should be fine. It's --template options going away that will cause problems w/o planning. New --template options would also need to be planned out, to know when we could safely add them to the new project wizards.

OK, cool. In that case, just to be clear, this change will effectively rename the --template=module to --template=application, and changes the default (running flutter create with no template argument) to application. The old app template type is still there.

@gspencergoog are we truly renaming --template=module to --template-application?  Or do we currently have both doing the same thing?  I thought we might want to leave --template=module without any public docs for one cycle just in case.  Although if others feel confident that we can nuke it right now then I'm fine with that, too.

Well, my PR truly does the rename, but I could make an alias for "module" for a bit if we need it. It sounded to me, however, (from the above discussion) like IntelliJ was the only one that used it, and that's going away.

If we merge this to master tomorrow, for example, if a dev is on master wouldn't IntelliJ be broken for them in the project creation use-case?

Yes, probably.  We can either wait to merge my PR until IntelliJ isn't using module anymore, or I can make an alias as I said above.

Yes, that's going away in the next release, at the end of the month. Probably best to leave it and delete it next month.

OK, will do.


Case insensitive device id

fixes https://github.com/Comcast/codex-gungnir/issues/32

Codecov Report

Merging #38 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   56.25%   56.25%
=======================================
  Files           5        5
  Lines         208      208
=======================================
  Hits          117      117
  Misses         88       88
  Partials        3        3

Impacted Files
Coverage Δ

primaryHandler.go
89.13% <100%> (ø)
:arrow_up:

deviceStatus.go
91.66% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update on debounce does not work

Description
The input is not formatted on debounce, only on blur.
Expected outcome
50000 does not turn into $50,000 on debounce.
Actual outcome

50000 should turn into $50,000 on debounce.

Live Demo
https://jsbin.com/duzeyopose/edit?html,output
Steps to Reproduce

Add a field with a large debounce value
Change the fields value
Click off of the field


Investigate and Resolve Project Service High Usage

Summary
Investigate and Resolve Project Service High Usage
Background
One of the EasyCLA services is calling the project service more than expected. This is resulting in extremely high usage, 1+ million requests per day.  This appears to be related to a CRON job we are running for pre-calculating metrics.
Most of the reported API requests are for GET /project-service/v1/projects/{project_sfid}
Tasks

[ ] Identify, diagnose, and document in this ticket the Project Service high-usage logic.
[ ] Review the offending code. Devise a plan of action to reduce the rate at which we query the project service API.
[ ] Implement API optimizations that will reduce the number of API queries.

Screen Shots

Expected behavior
EasyCLA should not make excessive API calls to the project service.
Environment
Please complete the following information:

Environment: DEV, STAGING, and PROD

Additional Context

Reported by @niravpatel27 and Prabodh Chaudhari

Acceptance Criteria
The "done" criteria when this feature or problem is resolved. Such as:

Unit Tests added and running in CI
Functional Tests updated to cover feature, if applicable
Demonstrate the set of capabilities to the product team while the code is running in the STAGING environment.

References
(Optional) Provide any code or specification references that would be helpful for the developer implementing this feature.

Log insights report from Onkar Kathane:
https://app.zenhub.com/files/194341141/05aee5c4-1d3d-4d39-aa19-1399b6a66090/download

@dealako @rinkeshbhutwala
Below is the update from Onkar
As compare to the previous count, now its very low, previously we are getting 150k  request every hr, but now its 250 request we are getting.
But now also cla hitting project service in every hr.

This is fine for now and acceptable.


[Question] the learning rate scheduler may not reach zero, right?

The implementation doesn't seem to follow the description in the paper:

Learning rate decays linearly from initial value to zero

When running the LT_glue.py command shown in README, the script sets --num_train_epochs as 30 and it means training each pruned subnetwork takes 3 epochs (and the [0%, 10%, ...90%]-pruned subnets take 30 epochs in total).
However, the learning rate scheduler depends on the --num_train_epochs (30) rather than 3:
t_total = len(train_dataloader) // args.gradient_accumulation_steps * args.num_train_epochs
...
scheduler = get_linear_schedule_with_warmup(
        optimizer, num_warmup_steps=args.warmup_steps, num_training_steps=t_total
)

So, the last learning rate after 3 epochs is 1.8e-5 (= 2e-5 * (1 - 3/30)) (rather than zero) if we start from 2e-5.
Is my understanding correct?
Sure this minor difference doesn't hurt the great contributions of the work. I just wanna confirm it and know the detail for reproduction and my own further research. Thanks!

@Tianlong-Chen when you have time, could you please take a look?


Is it possible to track media plays?

Really sorry, this isn't a bug - but I'm wondering if anyone knows of a way to track media (specifically video) plays on Nextcloud?
I've just managed to set this up, being non-technical it took some effort - so chuffed with myself there! What I'm looking for is to be able to monitor and count the number of plays of video's stored in my nextcloud. I need this as the video's contain licensed material and I need to provide the Music Licensing Company a count of plays of the files so they can pay the correct royalties.
This isn't something it seems you can do in Nextcloud directly, so I was hoping to do it using Matomo... Do you know if this would be possible, and any suggestions as to how?
I know this isn't the best place/way to ask. But would really appreciate any help you can give!
Thanks

Tracking such kind of stuff is tricky, because you have to make sure that it is not easily preventable. For example someone could disable javascript, download the file once and view it multiple times and so on. If this is no concern for you, you can watch for start/stop events on the video element. If you can't implement that by your own, you can also purchase the media analytics plugin from Matomo.

Hi, OK... so that's a step too far for me (Sooo confused). I've added the media analytics plugin on a trial to see if that helps, but that's not recording any plays. According to the docs it should 'just track' without any configuration. Is there something in this code which needs adding so that media plays can be tracked?

I'm not familiar with that plugin and it's a bit out of scope for this app. If you need some paid consulting, please write a mail to<EMAIL_ADDRESS>


Enable python3.7 debugging

Description
Currently python3.7 debugging is blocked but debugging does work as it is documented for python functions here - https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/serverless-sam-cli-using-debugging-python.html
I would also propose to allow the user to pass in a custom --bootstrap argument via the --debug-options argument. This would allow the following workflow...

User creates a debug/ folder.
Install ptvsd into that folder - pip install ptvsd -t debug/..
Create a file called debug_bootstrap with the following contents:

#!/var/lang/bin/python3.7

import ptvsd

ptvsd.enable_attach(address=('<IP_ADDRESS>', 5890), redirect_output=True)
ptvsd.wait_for_attach()

import sys
sys.path.insert(0,'/var/runtime')

from bootstrap import main

if __name__ == '__main__':
    main()

chmod +x debug/debug_bootstrap
Run sam local start-api -d 5890 --debugger-path debug/ --debug-args "--bootstrap /tmp/lambci_debug_files/debug_bootstrap"

@billyshambrook Thanks for the issue and getting it kicked off/providing a solution. We were in the process of doing the same and have a way that wouldn't require a customer to override through a debug_bootstrap (which matches with how we currently support debugging. It is out in PR now #1055.
Updating labels to reflect the current state.

Perfect, thanks for the update @jfuss

Support was adding in v0.14.1.
Closing


[Epic] Features required for Back-end Model B

Scenario: A Chinese learner wants to understand an article and hence want to interact and discuss with Chatbot regarding the article. (you can check model A code for langchain reference)
[ ] Add a sample Chinese article that user wants to learn and interact with the chatbot. Eg 國王的新衣
[ ] Add RAG to store the article as a vector base (you decide what suits the best).
[ ] Add a demo on how user can interact with this article, e.g., user may want to ask the chatbot that: (user may input english or chinese to ask about it)

"what is the meaning of this sentence xxx in the article?" ,
"What does 國王 mean?",
"What does the word 這套 mean in the article, and how is it different from 一件?",
"Why is it called 伊索寓言, not 伊索預言?"

[ ] Able to switch on or off the 中英對照回答 functionality (or maybe we can instead create a different page that can be shown or hidden to users to achieve the same effect)

llm-comparsion.pdf


Consider holding a unique_ptr to the Document Window inside melatonin_inspector.h

Pros:

Simplifies installation instructions (applications just hold a normal member, no need for users to make_unique or define onClose)
Could attach key listener attached to root app so ctrl/cmd-I constructs and opens the inspector

@FigBug Do you care at all? Mainly motivated for the key command, I can also just add on my app side.

Also might help do some work on the way to #56

I'm not too picky how the api looks, as long as I can have my menu item that creates / destroys the inspector.