Promisify XHR functions or replace XHR functionality with third-party lib that uses promises.

Moved to LumaGL


feat(price-feeder): Convert remaining providers to the Websocket Controller

Description
Converted the remaining providers over to use the Websocket Controller. For each provider I cross referenced the current API docs with how we currently handle their websocket and tuned it to maintain a stable connection.
Changes other than just converting providers to use the Controller:

Coinbase: Added a ping loop to keep the connection alive longer
Crypto.com: Changed the pricing info to parse into strings instead of float to correctly read prices
Huobi: Add the HuobiSubscriptionResp to handle subscription response messages and prevent an erroneous error
Added a second context to the WebsocketController to kill the old ping loop after a reconnect happens. Otherwise we risk running multiple ping loops for a single provider.
Added an integration test that tests creating an active connection, receiving and parsing prices for each provider websocket we have implemented.

closes: https://github.com/umee-network/umee/issues/1507

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

woo! Checking this out @zarazan

oh, and changelog please!

With this implementation, peggo cannot update the price-feeder version anymore, basically peggo does not know which prices will have to query when instantiating new providers. The new tickers might appear when someone tries to cross a different token
Or maybe every time a new price comes, we would need to kill the providers and instantiate again

this is so much cleaner than it was before, but thinking of price-feeder as a package for others, would be good to have an func to subscribe new pairs as needed with the provider running

this is so much cleaner than it was before, but thinking of price-feeder as a package for others, would be good to have an func to subscribe new pairs as needed with the provider running

cc @zarazan

@zarazan please lmk once rafael's concerns are addressed 🚀

Do these need to be backported for price feeder v1 branch or is it all v2 stuff?


Exclude com.sun.jersey from docker-minimal-bundle.

This probably is not the correct fix long-term as we should find the specific module that is pulling in the bad jersey-1 version. But until we can track down what the specific offending module is, this will have to do.

cc @ifilonenko @erikerlandson

rerun integration tests please

2/2 successful so far

rerun integration tests please

rerun integration tests please

rerun integration tests please

rerun integration tests please

Looks like the right fix to me.

This fix (#420) passed all integration-testing runs (except one that failed for some unrelated reason).  I'm still curious why #419 passed 5/6 times without addressing the jar conflict at all.
Regardless, LGTM

Thoughts on whether we should use this fix or the one in #419 ?

I think we should merge #420.  Merging #419 might also be useful; it reduced failures by ~80%, which makes it a potentially good defensive change against CI failures from lengthy startup times.


Add ManagedIdentityCredentialOptions

Add ManagedIdentityCredentialOptions type and use it to configure the Managed Identity Credential

Added in linked PR #6432


The Account Edit page is not guarded

The Account edit page is not guarded. An unauthenticated user is able to access the account/edit page.

@bryanparmelee do you want to incorporate this into your #19 pull request before merging?

That's the hope! Unfortunately this approach doesn't work in nested pages like account/edit in Next 13, so I've been working on a different solution.

Pull request 36 introduces changes to protected the Account Edit page from user-less and unauthenticated access.


Build and test for Linux/ARM64

Description
For aarch64 platform, I have built " spring-cloud-function " from source.
Steps
Following STEPS were used for the building and installation:
$ git clone https://github.com/spring-cloud/spring-cloud-function.git  &&  cd spring-cloud-function/

$ git checkout v4.1.1

$ Install JDK and setup JAVA_HOME.

$ Install MVN and setup Maven

$ mvn clean install

Outcome
Maven build, test and install is successful.
All this information is also added to Software Ecosystem Dashboard for Arm.

Closing it as there is no action to perform


[New Site] SC

URL： https://secret-cinema.pw
Site based on Gazelle
SC as target and source site. It's similar to PTP.  I can provide an invite.

Thanks, I'll test it.
I see it didn't auto filled English title.  At "Movie title" field goes original title like "Qiuqiu ni, biaoyang wo", at "Alternate title" should go English title like "Gimme Kudos".
I'm thinking isn't somehow possible to auto fetch torrent files too? Because SC site can take any torrent file from other tracker and it auto transforms it by changing "Source" to SC in the torrent file to change hash.

"Quick search" is not searching by imdb id. URL to search by id: https://secret-cinema.pw/torrents.php?cataloguenumber=tt8772262

Icon is bigger than the other icons:

Now I tried to transfer but script doesn't fill anything, tried from PTP and KG. I see stuff in the URL but nothing is filled.

Now I tried to transfer but script doesn't fill anything, tried from PTP and KG. I see stuff in the URL but nothing is filled.

Firefox?

I'm thinking isn't somehow possible to auto fetch torrent files too? Because SC site can take any torrent file from other tracker and it auto transforms it by changing "Source" to SC in the torrent file to change hash.

That's can't be done

Firefox?

Waterfox Classic, it's based on  Firefox.
Nothing unusual in browser's console.

I looked at code, I think problem is new JS optional chaining "?." insc.js, few lines like with "imdbData?.details?" and "?.[1]".
It's better to refrain from very new JS methods, as it drastically narrows compatibility.

@Purfview you can check the dist file,which is transformeb

Ah, syntax is transformed in dist, then I dunno where problem is.

Ah, syntax is transformed in dist, then I dunno where problem is.

G3.2.3 (64-bit) MacOS works fine.

Can you make a script with console.log at some points to debug and upload it as a test branch?

You can leave your following information:

Operating system:
Browser：
Browser Version ：
Script manager extension[e.g. tampermonkey Violentmonkey]：

Can't reproduce.Closed for now.


Use ISteamUser/GetPlayersSummaries to fetch additional claims #6

I used profile service to add the claims manually.
This makes these claims accessible: picture (steam avatar), given_name, website (steam profile URL) and nickname.
It's my first time working with C# so probably there're things to improve.

Did you test it too?

Did you test it too?

Yes, I had tested it manually, and I was able to get the new claims after successfully logging in

Thank you for your contributions


List required permissions

Could the documentation include the permissions needed at the resource group or subscription level?

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

ID: 2064767f-5304-275a-df9b-e74199b53408
Version Independent ID: 57dcf6b8-7125-1e0b-c446-34a0a1366a92
Content: Tutorial - Stream Azure Active Directory logs to an Azure event hub (preview)
Content Source: articles/active-directory/reports-monitoring/tutorial-azure-monitor-stream-logs-to-event-hub.md
Service: active-directory
GitHub Login: @priyamohanram
Microsoft Alias: priyamo

@Jackson-Miller Thanks for your feedback! We will investigate and update as appropriate.

@Jackson-Miller global administrator or security administrator permissions are required at Azure AD tenant level to use this feature which is already listed in the current documentation.

@SaurabhSharma-MSFT Those are the permissions required to enable the feature.  However the admin making the change does not have access to the subscription that houses the event hub resource, then it does not show up in the drop down list to pick the event hub resource from.  So some level of access is needed at either the resource group or the subscription.

@Jackson-Miller Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

can you please provide ARM template code snippet example for Stream Azure AD logs to an Azure event hub.

@Jackson-Miller : Thanks for reporting the issue! You would need ListKeys permissions on the target resource (i.e. event hub) in order to stream logs to it. You can find out more about Azure Monitor subscription-level permissions in this article. I'll update the documentation with this information shortly. Please feel free to open a new issue if you have any further questions.
#please-close


$GITHUB_ENV: ambiguous redirect

  -  name: Test
      uses: addnab/docker-run-action@v3

      with:
        image: alpine
        options: --rm -i -v ${{ github.workspace }}:/app -w=/app
        run: |
          echo "Start"
          echo "DEPLOY_STATE=successful" >> $GITHUB_ENV

gives ambiguous redirect exception, could workaround this by adding another trailing echo

I've been running into this and have tried working around this and can't seem to figure it out. What do you mean by this being worked around with a "trailing echo". Do you just mean running an additional echo command in your run block?

I've been running into this and have tried working around this and can't seem to figure it out. What do you mean by this being worked around with a "trailing echo". Do you just mean running an additional echo command in your run block?

Yes


Incorrect Schema for Service_Forwarding

Environment

Application Services Version: 3.20.0
BIG-IP Version: <IP_ADDRESS>

Summary
Declaration fails when using policyNat as show in the documentation as the schema references it to be excluded.
https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/latest/refguide/schema-reference.html#service-forwarding-policynat
https://github.com/F5Networks/f5-appsvcs-extension/blob/916a5cf62c5ff0896edfad281c50044cc67b575b/schema/3.20.0/as3-schema-3.20.0-3.json#L27006
Service_Forwarding can be compared to Service_L4 which also has policyNat - https://github.com/F5Networks/f5-appsvcs-extension/blob/916a5cf62c5ff0896edfad281c50044cc67b575b/schema/3.20.0/as3-schema-3.20.0-3.json#L14304
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

    "TEST_Service_Forwarding_123": {
        "class": "Tenant",
        "Application": {
            "class": "Application",
            "template": "generic",
            "test.item-foo.123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234": {
                "class": "Service_Forwarding",
                "policyNAT": {
                    "use": "natPolicy2"
                },
                "virtualPort": 8080,
                "virtualAddresses": [
                    "<IP_ADDRESS>"
                ],
                "forwardingType": "l2",
                "layer4": "tcp",
                "profileL4": {
                    "bigip": "/Common/apm-forwarding-fastL4"
                },
                "translateServerAddress": false,
                "translateServerPort": true
            },
            "natPolicy2": {
                "class": "NAT_Policy"
            }
        }
    },

Observe the following error response:

Error: Error while Sending/Posting http request with AS3 json :{"code":422,"errors":["/TEST_Service_Forwarding_123/test.item-foo.123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234: # is NOT valid"],"declarationFullId":"","message":"declaration is invalid"}  <nil>

Expected Behavior
policyNat should be allowed and configured in as3.
Actual Behavior
policyNat is no declared and as3 fails.

Thank you for your feedback. I have added AUTOTOOL-1699 to our internal product backlog to look into this further.

Hi! Is there any info when this is available? I would need this feature to work like it should.

This was resolved with the 3.31.0 release of AS3.


racecar will silently crash if an exception is raised from Cli#new

Problem Statement
exe/racecar silently discards exceptions because it assumes Cli#run has already rescued and logged them.
rescue
  # Exceptions are printed to STDERR and sent to the error handler
  # in `Racecar::Cli#run`, so we don't need to do anything here.
  exit(1)

But if an exception is raised in Cli#new, this assumption won't be true. The exception will be silently discarded.
Steps to Reproduce
Cause an exception to be raised from Cli#new. For example, cause a typo in a --require:
$ bundle exec racecar --require typo ConsumerClass || echo failed

When you run the above, you will see failed output but not the exception that caused it.
failed

Expected Behavior
When the above is run, the exception should be displayed on stderr (and the error_handler run, if one has been registered):
=> Crashed: NameError: undefined local variable or method `typo' for Cli#...
<backtrace>

failed

Suggested Fix
The easiest and most general fix for this would be to move the rescue block from here out to exe/racecar. This is best practice in case some other code path is later introduce that might also raise. And it would would have an added benefit of removing any risk of redundantly logging, then raising, then logging the same exception again.
I'd be happy to make a PR with the suggested change if you'd like.

Please open a PR and I'll review it!

@dasch Will do!

PR: https://github.com/zendesk/racecar/pull/189


Strange effect when swiping the last slide on mobile

Issue description:
Strange effect when swiping the last slide on mobile.
This happens only if we use fixedWidth and loop is disabled.
Demo link/slider setting: https://jsfiddle.net/n1a5kwcu/4/
Tiny-slider version:   2.3.5
Browser name && version:  Safari
OS name && version:   iOS 10.3.2

Thanks @mqnrk , this should be fixed in v2.3.6

It is now working as expected 🙂  Thanks @ganlanyuan !


RuntimeError due to Invalid Seed Function Evaluation

When running ReEvo with the command python main.py problem=tsp_aco, an error occurs during the evaluation of the seed function, leading to a RuntimeError. The error message suggests that the seed function is invalid, but it doesn't provide specific details about what caused the invalid evaluation.
Steps to Reproduce:

Set up the environment as described in the README.
Run the command python main.py problem=tsp_aco.
Observe the RuntimeError related to the invalid seed function evaluation.

Error Logs:
[2024-05-16 12:38:34,259][root][INFO] - Workspace: /remote-home/iot_liuguangyi/llm/LLM-as-HH/outputs/main/2024-05-16_12-38-34
[2024-05-16 12:38:34,259][root][INFO] - Project Root: /remote-home/iot_liuguangyi/llm/LLM-as-HH
[2024-05-16 12:38:34,259][root][INFO] - Using LLM: gpt-3.5-turbo
[2024-05-16 12:38:34,259][root][INFO] - Using Algorithm: reevo
[2024-05-16 12:38:34,668][root][INFO] - Problem: tsp_aco
[2024-05-16 12:38:34,668][root][INFO] - Problem description: Solving Traveling Salesman Problem (TSP) via stochastic solution sampling following "heuristics". TSP requires finding the shortest path that visits all given nodes and returns to the starting node.
[2024-05-16 12:38:34,668][root][INFO] - Function name: heuristics
[2024-05-16 12:38:34,668][root][INFO] - Evaluating seed function...
[2024-05-16 12:38:34,669][root][INFO] - Seed function code:
import numpy as np
def heuristics_v2(distance_matrix: np.ndarray) -> np.ndarray:
return 1 / distance_matrix
[2024-05-16 12:38:34,669][root][INFO] - Iteration 0: Running Code 0
[2024-05-16 12:38:36,349][root][INFO] - Iteration 0: Code Run 0 successful!
[2024-05-16 12:38:56,351][root][INFO] - Error for response_id 0: Command '['python', '-u', '/remote-home/iot_liuguangyi/llm/LLM-as-HH/problems/tsp_aco/eval.py', '50', '/remote-home/iot_liuguangyi/llm/LLM-as-HH', 'train']' timed out after 19.99986495007761 seconds
Please let us know if you need further information or assistance in resolving this issue. Thank you!

Hi there. Thanks for your interest!
It seems the execution timed out after 20 seconds. You may try to set a longer allowed execution time, e.g.:
python main.py problem=tsp_aco timeout=60

Hi there, thank you for your patience and detailed information. Despite my attempts to increase the execution time to 60 seconds and 200 seconds, the problem remains unresolved.
Error Logs with timeout 60:
[2024-05-17 01:54:54,061][root][INFO] - Workspace: /remote-home/iot_liuguangyi/llm/LLM-as-HH/outputs/main/2024-05-17_01-54-54
[2024-05-17 01:54:54,061][root][INFO] - Project Root: /remote-home/iot_liuguangyi/llm/LLM-as-HH
[2024-05-17 01:54:54,061][root][INFO] - Using LLM: gpt-3.5-turbo
[2024-05-17 01:54:54,061][root][INFO] - Using Algorithm: reevo
[2024-05-17 01:54:54,509][root][INFO] - Problem: tsp_aco
[2024-05-17 01:54:54,509][root][INFO] - Problem description: Solving Traveling Salesman Problem (TSP) via stochastic solution sampling following "heuristics". TSP requires finding the shortest path that visits all given nodes and returns to the starting node.
[2024-05-17 01:54:54,509][root][INFO] - Function name: heuristics
[2024-05-17 01:54:54,513][root][INFO] - Evaluating seed function...
[2024-05-17 01:54:54,513][root][INFO] - Seed function code:
import numpy as np
def heuristics_v2(distance_matrix: np.ndarray) -> np.ndarray:
return 1 / distance_matrix
[2024-05-17 01:54:54,513][root][INFO] - Iteration 0: Running Code 0
[2024-05-17 01:54:55,797][root][INFO] - Iteration 0: Code Run 0 successful!
[2024-05-17 01:55:55,798][root][INFO] - Error for response_id 0: Command '['python', '-u', '/remote-home/iot_liuguangyi/llm/LLM-as-HH/problems/tsp_aco/eval.py', '50', '/remote-home/iot_liuguangyi/llm/LLM-as-HH', 'train']' timed out after 59.9999217200093 seconds
Error Logs with timeout 200:
[2024-05-17 01:58:19,536][root][INFO] - Workspace: /remote-home/iot_liuguangyi/llm/LLM-as-HH/outputs/main/2024-05-17_01-58-19
[2024-05-17 01:58:19,536][root][INFO] - Project Root: /remote-home/iot_liuguangyi/llm/LLM-as-HH
[2024-05-17 01:58:19,536][root][INFO] - Using LLM: gpt-3.5-turbo
[2024-05-17 01:58:19,536][root][INFO] - Using Algorithm: reevo
[2024-05-17 01:58:19,934][root][INFO] - Problem: tsp_aco
[2024-05-17 01:58:19,934][root][INFO] - Problem description: Solving Traveling Salesman Problem (TSP) via stochastic solution sampling following "heuristics". TSP requires finding the shortest path that visits all given nodes and returns to the starting node.
[2024-05-17 01:58:19,934][root][INFO] - Function name: heuristics
[2024-05-17 01:58:19,934][root][INFO] - Evaluating seed function...
[2024-05-17 01:58:19,934][root][INFO] - Seed function code:
import numpy as np
def heuristics_v2(distance_matrix: np.ndarray) -> np.ndarray:
return 1 / distance_matrix
[2024-05-17 01:58:19,934][root][INFO] - Iteration 0: Running Code 0
[2024-05-17 01:58:21,266][root][INFO] - Iteration 0: Code Run 0 successful!
[2024-05-17 02:01:41,266][root][INFO] - Error for response_id 0: Command '['python', '-u', '/remote-home/iot_liuguangyi/llm/LLM-as-HH/problems/tsp_aco/eval.py', '50', '/remote-home/iot_liuguangyi/llm/LLM-as-HH', 'train']' timed out after<PHONE_NUMBER>6002445 seconds

Could you please check out the stdout file in outputs/main/2024-05-17_01-58-19/problem_iter0_stdout0.txt and provide more information?

@henry-yeh Thank you for your response. By checking the file outputs/main/2024-05-17_01-58-19/problem_iter0_stdout0.txt, I found that the issue was still due to the timeout being too short. I resolved the problem by setting it to 1000. Could you explain the meaning of timeout and the rules for setting it?

On my Mac, running the seed evaluation takes about 3 seconds. So it's really weird it takes that much time for you. Maybe your CPU is occupied. The rule of setting it really depends on your hardware.
If the time of evaluating a heuristic exceeds timeout, it is killed. We set this parameter because LLM may generate code that involves infinite loops.

I've upgraded to an i7-14700K, and now everything runs smoothly. Thank you for your help!


Show how to include a custom function in the data model

What to solve
Sometime you want to include some custom functionality into the generated models, and here is how.
This can be done through a custom preset using the hook additionalContent.
Read more about presets here https://github.com/asyncapi/modelina/blob/master/docs/presets.md
Use the following code in the example:
generator = new TypeScriptGenerator({
  presets: [
    {
      class: {
        additionalContent({ content }) {
          return `${content}
public someAdditionalFunctionForClasses(): string {
  return 'Some test';
}`;
        },
      }
    }
  ]
});

Duplicate the folder https://github.com/asyncapi/modelina/tree/master/examples/TEMPLATE and rename it to include-custom-function. Update any relevant documentation.

Some folks want more specific tasks to complete, or use this if you get stuck - CLICK ME

This issue is specially designed for first-time contributors, that want to contribute their very first code. If you are accustomed to how to solve issues in AsyncAPI, you can skip some of the steps.
What to know before getting started
Before actually getting started with solving the issue, there are a few things that need to be set up:

Make sure you familiarize yourself with our code-of-conduct and contributor guidelines
We work on forks of repositories and branches in forks, if you are unsure how to set this up you can follow the guide on our git-workflow setup.

Feel free to reach out either here in the issue, or on slack under #tooling if you have any questions or are stuck, we will gladly help.
Getting started
This is a task list, that you copy into your PR description to ensure everything is included, this list walks you through each step as to how to solve this issue. It assumes that you have already set up your fork and branch and are ready to commit code.

[ ] Duplicate the folder https://github.com/asyncapi/modelina/tree/master/examples/TEMPLATE and rename it to include-custom-function. This will give you a good base to start out with.
[ ] Adapt the documentation about the feature and ensure the example are linked correctly.

The docs that need to change are the following: docs/advanced.md#include-a-custom-function-in-the-data-model
The documentation must reference an actual example that is tested and contain no code itself. It is because the docs will inevitably become desynchronized and display incorrect code.

[ ] Adapt the examples overview to include your new example.
[ ] Now it is time to adapt the test code (in examples/include-custom-function/index.spec.ts) and the implementation (in examples/include-custom-function/index.ts).
[ ] Make sure that the test succeeds, by standing in the new example folder and execute npm run test.
[ ] Make sure that there are no linting errors, by standing in the root folder and execute npm run lint.
[ ] Create your PR, by selecting your branch that should be merged with the master branch - https://github.com/asyncapi/modelina/compare
[ ] It is good practice to double-check your PR before requesting a review from someone else, the same way you read an assignment before handing it in.

Do you have any examples I can check for reference?
We have currently 4 example documentations that you can look for guidance on how to achieve the desired output:

How to add logging to the library
Generate models from AsyncAPI documents
Generate models from JSON Schema draft 7 documents
Generate an interface instead of class

Hi, i would like to work on this issue✨

Hi, i would like to work on this issue✨

Awesome! Ping me if you have any questions or problems 🙂

Happy Hacktoberfest, everyone! ✨✨ Feel free to reach out to me for any questions you may have about adding documentation on these open issues. 🙂
Tag me in your AsyncAPI Doc PRs 📄, my GitHub handle is /alequetzalli 🐙.

@jonaslagoni  I guess this is solved by @mahakporwal02


[Qt5] switch MUSIC to Qt5

Qt4 is going to be disable/not available in the next Linux LTS release.

(you mean Ubtuntu)
The current Ubuntu LTS releases (14.04 and 16.04) are valid until April 2019 and April 2021 so it's not urgent. The whole point of an LTS is to avoid projects like ours having to upgrade their libraries every time a new version of Ubuntu comes out.

(the transition to QT5 will be done with the upgrade to medInria 3 anyways)

Actually I looked this up and qt4 is still supported in ubuntu 18.04. It's supposed to be removed from 18.10.

Already done in https://github.com/medInria/medInria-public, closing


Run the check with javascript files I want to choose

In your document, you said "This Rake task runs JSHint across all the JavaScript assets within the following three folders to ensure that they're lint free.
your-rails-project/app/assets/javascripts
your-rails-project/vendor/assets/javascripts
your-rails-project/lib/assets/javascripts"
How can we use the regex to include and exclude some javascript files?
Is that possible? Thanks.

@damian: How do you check files in rails-project/spec/javascripts/*? @ysihaoy already asked this question but got no answer.


Permissions Denied: Missing or insufficient permission

In https://github.com/flutter/codelabs/tree/master/firebase-get-to-know-flutter I am having trouble with Firestore, even though, I have already set up Firestore in the Firebase Console.
In my main.dart, I have already imported the import 'package:cloud_firestore/cloud_firestore.dart';
I have followed the example and setup as follows.

This is the error I keep getting.

This isn't something related to flutter or the cloud_firestore package itself. It most probably has to do with your firestore rules.
Make sure they're set up correctly in your Firebase console.

@JamesVanWaza Did @dancamdev's comments help?

Yes, will keep testing I will let you know.
You can close issue

Thanks @JamesVanWaza


Plugin should not enable the start button while VM creation provision is in progress

Environment: Windows/IntelliJ14
In the portal when VM creation is in progress start button is disabled.

In the plugin when VM creation is in progress start button is enabled.

If we know from the API if the VM is provisioning then we can disable the Start in the menu.

Should we follow Visual Studio behavior that allows to operate as if it's already started, while in fact it is still provisioning

Or should we mimic the Portal's behavior

I would say portal behavior
From: Leonardo M. Tilli<EMAIL_ADDRESS>Sent: Wednesday, March 25, 2015 10:34 AM
To: MSOpenTech/msopentech-tools-for-intellij
Cc: Beatrice Oltean (MS OPEN TECH)
Subject: Re: [msopentech-tools-for-intellij] Plugin should not enable the start button while VM creation provision is in progress (#213)
Should we follow Visual Studio behavior that allows to operate as if it's already started, while in fact it is still provisioning
[Image removed by sender. image]https://cloud.githubusercontent.com/assets/1890830/6830798/f546331e-d2fa-11e4-9ad7-4279b9c15f27.png
Or should we mimic the Portal's behavior
[Image removed by sender. image]https://cloud.githubusercontent.com/assets/1890830/6830812/164e848a-d2fb-11e4-8e6b-56993c27d718.png
—
Reply to this email directly or view it on GitHubhttps://github.com/MSOpenTech/msopentech-tools-for-intellij/issues/213#issuecomment-86134432.

This issue is fixed and functionality is working as expected in the latest build.


LA HCO Results Profile

Story
As LA HCO, to ensure we'll receive all needed results data to an EHR, we need to relate the ReportStream results information to what will be received by the EHR.
Pre-conditions

none

Acceptance Criteria
Acceptance Criteria

[ ]  Approved message profile

Tasks

[x] Obtain publish message profile (if available)
[x] Obtain sample messages
[x] Document data elements of interest

[x] Optionality
[x] Cardinality
[ ] Vocabulary requirements

[ ] Requested HCO feedback on our analysis
[ ] Confirm baseline profile requirements with Sender

Decisions

Optional: Any decisions we've made while working on this story

Notes

Optional: Any reference material or thoughts we may need for later reference

Blocked: HCO waiting for SOW documentation to be signed. Per PHL procurement process takes 6-8 weeks. Adrian to update details.

Currently held up by the procurement process.  Oschner needed to sign attestation form confirming who at the lab has authority to sign SOW on behalf of the HCO.  Now contract is with LADPH legal moving through the process.  Expecting 6-8 weeks, may be longer.  Unclear how long Oschner will need to ramp up once contract is in place.

Still blocked. Waiting on Oschner

Still waiting for approvals

Several rounds of review have been completed with Oschner. They did not have a results profile in place, so they will be updating their results to match the labs. Once the lab's results profile is done, they will be able to adapt the same profile. It will just need final confirmation.


Azure Kubernetes missing from Naming Rules and Best Practice

Page does not include any references to the Azure Kubernetes Service.
It is also excluded from the naming and tagging best practice article:
https://docs.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/naming-and-tagging

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

ID: 374ec8eb-64d4-3766-5c5e-39137a262ffd
Version Independent ID: 3abeeae9-6854-651a-483c-a1673eea9f9f
Content: Resource naming restrictions - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/resource-name-rules.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@hiscjo01 Thanks for your feedback. Assigning the issue to content author @tfitzmac  for doc enhancement.

@hiscjo01 - the resource provider namespace for AKS is Microsoft.ContainerService. Within it is a resource type called managedClusters which I believe is the resource type for AKS. Is that what you were looking for?
I'll add a link to the article that maps resource providers to services.

@tfitzmac - That is exactly what I was looking for.
Thank you for the quick response - much appreciated


std.random.uniform return value depends on integer size

dlang-bugzilla (@CyberShadow) reported this on 2015-10-03T18:25:09Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=15147
CC List

greensunny12
jrdemail2000-dlang

Description
With an RNG seeded with a constant value, uniform(0, 100) and uniform(0, 100L) will return different values.

Demonstration:

////////////////// test.d //////////////////
import std.random;
import std.stdio;

void test(T)()
{
    Mt19937 rng;
    foreach (n; 0..10)
        write(uniform(0, T(100), rng), " ");
    writeln();
}

void main()
{
    test!int();
    test!uint();
    test!long();
    test!ulong();
}
////////////////////////////////////////////

This will print:

12 2 34 85 4 91 29 85 98 3
12 2 34 85 4 91 29 85 98 3
54 49 75 69 11 25 66 64 28 18
54 49 75 69 11 25 66 64 28 18

This can be a problem for applications that expect seeded RNGs to behave predictably on all platforms (e.g. to generate something procedurally than must be in sync), and pass a size_t to uniform. E.g.:

string sample = arr[uniform(0, $, rng)];

greensunny12 commented on 2018-03-31T18:09:58Z
I just ran into this when trying to add more/better public examples for std.random (https://github.com/dlang/phobos/pull/6393).

Normally, examples can be done by seeding the RNG and thus _knowing_ the sequence of the PRNG.
The current behavior is really nasty.

jrdemail2000-dlang commented on 2018-10-10T05:55:50Z
I'm interested in this, as I use these properties extensively for unit tests in randomization routines. Is it also a capability intended to be supported by some of my tools.

The formulation of the unit tests added as part of https://github.com/dlang/phobos/pull/6393 raised a couple questions for me. Those unit tests are qualified as follows:

version(X86_64) // Issue 15147
assert([0, 1, 2, 4, 5].randomShuffle(rnd2).equal([2, 0, 4, 5, 1]));

My questions:
* Is the lack of consistency strictly related to integer size (e.g. 32-bit vs 64-bit), or can these differences surface between different platforms even with the same integer size?
* For routines like std.random.randomShuffle and std.random.randomCover, is the issue that these are passing the range length, normally size_t, to std.random.uniform et.al. The issue being that the size of size_t is platform specific?


Alternative nix

While I was working on #380 I (with the help of some team mates) updated the nix setup. @IvanMalison updated the nix setup before I cleaned up my changes and put them in a PR. I thought I should still raise this version, as it takes a different approach that may be preferable.
The key difference here is that nixpkgs is (by default) pinned to a specific commit, such that this setup should always work regardless of the user's NIX_PATH (where <nixpkgs> comes from).
I've tested this with both nix-build and nix-shell --run "cabal v2-build github".

@ajmcmiddlin We could easily pin with the setup that I uploaded by pinning in default.nix. I actually deliberately decided NOT to pin (in the overlay would be fine to pin in default.nix) because this is a library that is presumably used by many different people, who might want to pin to different versions of nixpkgs themselves.
The overlay is generally useful enough that I think people will be able to use it with somewhat different versions of nixpkgs, and it automatically plugs in nicely with existing nix setups.

Yeah, that all makes sense. I'll use your setup and maybe raise a PR for taking nixpkgs as an argument in default.nix, defaulting it to a pinned version. I think I'll need that in my current project.


Applies hotfixes to the RSS reader Edge App

What does this PR do?

[x] Removes fields (whose value is an empty string) from the manifest file
[x] Adds the entrypoint field in the manifest file
[x] Applies styling changes to the Edge App
[x] Modifies docs so that it will include a list of RSS links that do or don't need the CORS proxy to be enabled
[x] Adds support for a loading spinner

Checklist

[x] Update the screenshots.

Screenshots (WIP)
800x480

1280x720

1920x1080

3840x2160

4092x2160

Hello @MuratUrsavas & @L4cko. I'd like to request a review. Thanks.

Looks good @nicomiguelino but can you please fix linting.


Unnecessary ‘high’ stack usage in aes_gen_tables

“aes_gen_tables” puts 2 arrays (‘pow’ and ‘log’) each with 256 elements and of type ‘int’ onto the stack. In these arrays only values in the range from 0 to 255 are stored therefore an ‘unsigned char’ would be sufficient. On systems where ‘int’ is 4 bytes 1.5kB of stack space is wasted which can be a problem on systems with limited RAM resources.
Kind regards
Christian

ARM Internal Ref: IOTSSL-1165

I think this can be reduced further be simply reusing the tables. For example it looks like you can use table RT2/RT3 instead, then there is no huge tables allocated on stack. By the time RT2/RT3 are generated it's done with the log/pow table.

Added PR & suggestion from @colinoflynn (in my opinion also correct) because RT2 and RT3 are assigned on the end of function


Feature Use tap-spec in place of tap-nyancat

Ahhh, so much faster.

BTW, is there a way to reduce the verbosity of tap-spec so it only shows failing tests?

Needed to happen, but I will be sad to see nyan go :(

@SaintPeter Not sure. Didn't look into it


Extending ak.max/min functionality to first N entries

It would be convenient to have a function to get the first N highest/lowest entries as opposed to just the first max/min. From our side discussion, this could be ak.top/bottom.

I labeled this "performance" because it's equivalent to sort-and-slice.

It looks like Pandas calls this nlargest: https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.nlargest.html


CubeMX with CMake

closes #140
Key changes
CubeMX generates for CMake instead of Makefile!

This greatly simplifies the build system since we don't need to rip data out of the Makefile.
Also improves robustness since we can use the generated flags instead of fixed flags in Toolchain.cmake
On that note, toolchain.cmake is no longer needed as it is generated on-the-fly by cubemx. No other platforms used it so I completely removed it
CubeMX generation is executed in mcal_conf.cmake (instead of platforms/stm32cubemx/CMakeLists) because we need the toolchain to exist before CMake starts looking at subfolders (specifically, before project() is called in firmware/CMakeLists.txt)

I had to do a few hacky things to the generated code because CubeMX is quite rigid. I modify cubemx's generated toolchain file (gcc-arm-none-eabi.cmake) to create racecar-toolchain.cmake:

CMAKE_*_COMPILER_FORCE is commented out as it prevents clangd from finding system headers
The linker path is modified since our CMAKE_SOURCE_DIR is not the cubemx dir (it is firmware)

These modifications happen automatically whenever cubemx regenerates so there is no impact or responsibility to the developer.
Removes the bindings CMake target.
#### BEFORE ####
# firmware/CMakeLists.txt
add_executable(main)
add_library(bindings)
...
target_link_libraries(main PRIVATE bindings)

#platforms/.../CMakeLists.txt
target_sources(bindings PRIVATE bindings.cc)
target_link_libraries(bindings PUBLIC mcal-stm32f767)

#### AFTER ####
# firmware/CMakeLists.txt
add_executable(main)

#platforms/.../CMakeLists.txt
target_sources(main PRIVATE bindings.cc)
target_link_libraries(main PRIVATE mcal-stm32f767)

It was causing the static initialization order fiasco and causing duplicate compilation since sources which should have been compiled with main were compiled on the side.
Now, in platforms/.../CMakeLists.txt, all sources, mcal layers, and cubemx sources are linked directly to the main target
Also removes the os target - files are just linked to main directly. This removes this ugly block from firmware/CMakelists.txt

# Only link/include RTOS if the project supports RTOS
# The project needs to provide an underlying OS target
# in the platform/${PLATFORM_NAME}/CMakeLists.txt
if(TARGET os)
    target_link_libraries(main PRIVATE os)
endif()

Why make these changes?
The Makefile based system was very hacky. It only existed because cubemx did not support CMake generation until very recently.
Why now?
Our projects are getting more complex and have more source files. The current system doesn't (easily) allow for you to break mcal or shared code into .h and .cc. Since mcal and shared were independent CMake targets, they did not inherit the compiler flags required for stm so they could not be linked to the other files.
In particular, this is necessary for my work on the CAN drivers which, for circular dependency reasons, require the CAN sources be split into header and source files.
Known Issues
Clangd is getting confused by some of the file names. In particular, CubeMX generates peripheral files like gpio.h, while we have shared/periph/gpio.h and mcal/stm32f767/periph/gpio.h. Part of clangd's strategy is to associate source and header files with similar names (ie. the same prefix, as far a I know), so clangd thinks that our c++ gpio.h files are actually C files. THIS DOES NOT AFFECT COMPILATION, only the language server. I looked through the clangd docs and forums and couldn't find a solution. There's no clangd flag or anything which disables this behaviour.
To fix this, we will need to rename our header files. This means either changes the name ex gpio.h -> shared_gpio.h or the extension gpio.h -> gpio.hpp. I'm in favour of the latter since it is less intrusive, and it guarantees that we won't ever collide with a C file (in case cubemx decides to add shared_gpio.c). EDIT: See #334
Testing
@samparent97 @TylerStAmour @CameronBeneteau @Corbin-Jarrett @sokosam @AmanRV @luaibash @Tegh25
This commit includes the relevant updates to all of the projects and their platforms, with the exception of raspi/sil platforms. I have tested the stm32f767 platforms on the board and they work, but I want some people to try the new build system on their windows / mac machines and verify it all compiles.
You must run deepclean before building to clear out the old Cubemx files
make PROJECT=... PLATFORM=... deepclean
Replace the ... as needed
Please add a PR comment with your OS, which projects/platforms you tested, whether it worked, and any other notes

OS: Windows 11
Tested: Everything in Demo except RTOS
Platform: CLI
Compiled and works as intended

The debug projects are missing these changes (even the non-sil ones), so they won't compile

Yes I didn't convert them. Those projects will be removed in the next few weeks so I'm not worried about them.

OS: Windows 10
Builds were successful for all Demo, FrontController, LVController, and TMS projects for the cli and stm platforms.

OS: Windows 11
Tested: All demos, and FrontController, LVController, and TMS projects
Platform: CLI
Everything compiled as expected.


Rust × RDF

This is a tracking issue for the RDF loader generated in Rust. This loader should be able to load RDF documents in various formats (JSON-LD in particular) into a data structure generated from the TreeLDR layouts definitions.

[ ] Import from RDF dataset
[ ] Define static JSON-LD contexts
[ ] Import from JSON-LD

Import from RDF dataset
Any data structure generated from TreeLDR should define a method to instantiate the data structure from an RDF dataset (using the grdf library). This would be the common representation for various input formats like JSON-LD.
Define static JSON-LD contexts
This is outside the direct scope of TreeLDR, but necessary to have the best experience with it. The goal here would be to statically define the set of known JSON-LD context as a type, that can then be used hen loading a JSON-LD document.
#[derive(JsonLdContext)]
pub enum Context {
  #[context("https://json-ld.org/contexts/person.jsonld")]
  Person,
  #[context("https://www.w3.org/2018/credentials/v1")]
  Credentials
}

pub struct StaticContextLoader;
pub struct UnknownContext(IriRefBuf);

impl json_ld::context::Loader for StaticContextLoader {
  type Error = UnknownContext;
  type Output = Context;

  fn load_context(&mut self, iri: Iri) -> Result<Context, UnknownContext> {
    if iri == iri!("https://json-ld.org/contexts/person.jsonld") {
      Ok(Context::Person)
    } else if iri == iri!("https://www.w3.org/2018/credentials/v1") {
      Ok(Context::Credentials)
    } else {
      Err(UnknownContext)
    }
  }
}

Import from JSON-LD
Once static contexts defined, one should be able to load a JSON-LD document like so:
#[treeldr("example.tldr")]
mod schema {
   #[prefix("https://example.com/")]
   pub example;
}

fn main() {
  let json_ld_input = json_ld::load("input.json-ld").unwrap();
  let doc = json_ld_input.expand::<Context, _>(&mut StaticContextLoader).unwrap();
  let dataset: grdf::HashDataset = doc.rdf_quads().into();
  let instance = schema::example::Type::import(dataset, node_id).unwrap();

  ...
}

One open question is how to get the identifier (an IRI or blank node identifier) node_id of the node we want to import from the dataset. It correspond to the top level objects in a JSON-LD document. For now it can be manually fetched once the JSON-LD document is expanded. One just need to be sure to manually assign a blank node id to anonymous nodes or else a random one will be assigned in the dataset, and will be harder to find. We may need to write a helper function to do all that.

The JSON-LD part necessitate timothee-haudebourg/json-ld/pull/37 to be done.


[in progress] Update dependencies, add possibility to work with tcp

This PR is just hotfixes for getting things done (I wanted to push messages through tcp and I've added the code), but in general it all could be done more neatly. For example, for now there is hardcoded port and only one atom for all the system. Plus it's a logger, right? All log tasks should be ran asynchronously not to stop main code execution.

https://github.com/t0xa/gelfj/blob/master/src/main/java/org/graylog2/GelfMessage.java#L106 ah, I did it wrong... tcp message should be different from udp one. Ok, tomorrow I guess I'll fix it

Ok, thanks

@narkisr please look at the last commit, is it acceptable? I'm feeling like i'm rewriting all the library. Should I still make it as pull request or should I cancel it and make my own library?

From reading the last commit it looks pretty nice, I don't think you need to separate this to another library
How did you check that its working? did you use logstash and Kibana4?

I use graylog itself. It still not finished and not tested at all :) I added [in progress] to header and will clean this up in the next couple of days if you don't mind.

Sure go ahead, let me know when your done and ill test against kibana
Thanks

@narkisr I tested tcp and it works pretty well, but can't test udp. I must be doing something wrong, but your initial version doesn't work for my server as well as my new one (I actually got your code for gzip/chunking and rewrote https://github.com/t0xa/gelfj/blob/master/src/main/java/org/graylog2/GelfUDPSender.java this one for sending and tried to combine them). So please test and/or fix it.
I'll fit timbre adapter.

Ok, ill look into this and let you know if there are any changes required
Thanks!

Hi @narkisr, any luck with testing?

@s-mage I'm on it, had a busy couple of weeks but ill attend to this on the weekend

@narkisr that would be great!

Ok, iv tested UDP (btw its quite easy you use this and run the midje facts) and it seems to work fine (I didn't test the TCP one yet).
Iv also added a bunch of comments on the pull request that id appreciate if you could apply
Thanks~!


Entity addresses loading time

It takes very long to load and inspect the addresses associated with large entities entity:
https://demo.graphsense.info/#!btc/entity/546360778
Maybe use smaller pages?

Done


Duplicate type error when both c++/winrt app and c++ windows runtime component are using WinUI framework package

How to repro:
Download and compile BlankApp2.zip
Both BlankApp2 {Blank App(C++/WinRT) template} and RuntimeComponent1 {Windows Runtime Component (C++/WinRT) template} are cppwinrt project which is created by https://marketplace.visualstudio.com/items?itemName=CppWinRTTeam.cppwinrt101804264
Both projects have reference to WinUI.
Here is the reference relationship:
BlankApp2  -> RuntimeComponent1  -> WinUI
BlankApp2  -> WinUI
Error on build:

Severity
Code
Description
Project
File
Line
Suppression State
Suppression State

Error
MDM2009
Duplicate type   Microsoft.UI.Xaml.Automation.Peers.AnimatedVisualPlayerAutomationPeer found   processing metadata file

...\source\repos\BlankApp2\Debug\RuntimeComponent1\Microsoft.UI.Xaml.winmd,   type already exists in file ...\source\repos\BlankApp2\packages\Microsoft.UI.Xaml.2.2.190917002\lib\uap10.0\Microsoft.UI.Xaml.winmd. | BlankApp2 | ...\source\repos\BlankApp2\BlankApp2\MDMERGE | 1 |   |

@Scottj1s looks like this error comes from mdmerge, invoked from 2>C:\Users\danzil\Desktop\BlankApp\packages\Microsoft.Windows.CppWinRT.2.0.190530.8\build\native\Microsoft.Windows.CppWinRT.targets(340,9): error MSB3073: The command "mdmerge @"ARM\Debug\mdmerge.rsp"" exited with code 2.

It is surprising to me that Microsoft.UI.Xaml.winmd is in the RuntimeComponent1 output location. Is that expected @scott1js?

Can we reevaluate with the latest C++/WinRT vsix/nuget/templates?  @jlaanstra has made some significant improvements to the msbuild support, addressing duplicate type definition errors like this, among others.

@jevansaks @Scottj1s VS tends to copy Reference entries to the output folder which will then be picked up by mdmerge, because it uses the output directory as a metadata_dir. I have a OPR out internally to make mdmerge support files so we can be more specific about what we pass to mdmerge.exe.
In the meantime you can use this as a workaround:
<ItemDefinitionGroup>
    <Reference>
        <Private Condition="'$(ConfigurationType)' != 'Application'">false</Private>
    </Reference>
</ItemDefinitionGroup>

There is also an open issue to make duplicate type definitions (those that clearly don't violate ODR, such as from identical files) benign in mdmerge.


Good aligned size

Hi there,
Thanks for this very useful FFT library!
For my application I need to find an FFT length that is both fast and SIMD aligned.
I managed to do this by making the following modifications to the header file:
Line 405: static POCKETFFT_NOINLINE size_t good_size_cmplx(size_t n, int align=1)
Line 422:              if ((x<bestfac)&&(x%align==0)) bestfac=x;

If speed would be too much impacted by the modulo operation, one could also only check powers of 2, i.e.:
Line 405: static POCKETFFT_NOINLINE size_t good_size_cmplx(size_t n, int align=0)
Line 422:              if ((x<bestfac)&&(x&((1<<align)-1)) bestfac=x;

If you think this could be useful to others as well, feel free to add this to the header file.

Thanks a lot, that is an interesting feature!
I have come across the need for something like this in a slightly different context; I had the requirement "the result size must ne even", not so much "the result size must not break an alignment of x", but de facto there isn't a real difference between these two.
I wonder if this can be implemented a little bit more efficiently, by doing
Line 405: static POCKETFFT_NOINLINE size_t good_size_cmplx(size_t n, size_t required_factor=1)
Line 410:   for (size_t f11=required_factor; f11<bestfac; f11*=11)

(plus some small adjustments in lines 407 and 409 to make sure we don't get into endless loops.)
Would this also work for you?

Thanks for the quick reply, yes this solution would work for me too!

Great! Can you please give the enhanced_good_size branch a try?

Ah, sorry! I think this implementation is still buggy. Hopefully I'll have something better soon!

Now it should hopefully be correct.

It works! Thanks again for the very quick response 😄!

Merged!


Adding support to ***inq functions

Close #798 by implementing the ``***inq``` functions
Check list:

[x] Mapdl implementations (console, corba and grpc).
[x] Mapdl refined grpc implementation
[x] Documentation. (Pretty much scrapped copied from the MAPDL internals. Therefore, we should probably make sure the info is releasable.
[x] Test units. They are going to be very small, since they are undocumented and we are not ensuring fully support. I'm just going to make sure the commands run and get an output with valid format.

Test units. They are going to be very small, since they are undocumented and we are not ensuring fully support. I'm just going to make sure the commands run and get an output with valid format.

Agreed.

As mentioned before, there are quite a lot of info in the docstrings which I'm not sure if we want to share. Some guidance is appreciated.
Nevertheless, if we are intending PyMAPDL becomes somehow the new way of interact with MAPDL, I see that this documentation is going to be the reference, not only for checking PyMAPDL commands but also MAPDL commands (given the similar syntax and the lack of official public documentation).
Hence I believe we should be ambitious regarding the scope of this documentation and give as much info as possible.


Error when instzlled with the git and pip

Traceback (most recent call last):
File "/usr/bin/i3-workspace-names-daemon", line 33, in
sys.exit(load_entry_point('i3-workspace-names-daemon', 'console_scripts', 'i3-workspace-names-daemon')())
TypeError: main() missing 1 required positional argument: 'argv'

I get the same thing :(


fix: remove the remaining isBeta instances

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

The isBeta flag was removed by https://github.com/freeCodeCamp/freeCodeCamp/pull/54491, but there are still instances left (they didn't exist when #54491 was created).
This PR removes those instances to unblock CI.

CI will still be blocked - this is included in #54537 to unblock everything in one go.


APU debug window

Fixes #21
Right now is just a pretty simple display of each channel.  Not sure what else I can put in here though.
It's certainly pretty to look at.
Doesn't come across the best in a 8fps GIF, really need to see it in person, but here goes:

Codecov Report

Merging #36 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   95.35%   95.38%   +0.02%
==========================================
  Files          44       44
  Lines        5624     5655      +31
==========================================
+ Hits         5363     5394      +31
  Misses        261      261

Impacted Files
Coverage Δ

src/emulator/apu/mod.rs
95.7% <ø> (+1%)
:arrow_up:

src/emulator/apu/synth.rs
76.53% <ø> (ø)
:arrow_up:

src/emulator/mod.rs
81.9% <ø> (ø)
:arrow_up:

src/emulator/memory.rs
93.19% <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 1db4f4b...53e43f2. Read the comment docs.


Fault Tolerance Specifications page

Please go to this doc and fill in the needed information to create each page.
https://docs.google.com/document/d/1tFhT1inI2G3PdU2QawmCexc5aUmK8E9ZHgpqCZ_KFAE/edit#heading=h.4kitmexapqcp

I updated the page on WP> Please review all links and content and put comments here with screenshots:
https://microprofile.io/specifications/fault-tolerance/
password: MPreview
@Emily-Jiang

The content looks good. For the title, can you update to MicroProfile Fault Tolerance 4.1?

@Emily-Jiang why would we put 4.1 when no other pages have versions?

@rstjames all of the other pages need to have versions.

For review:
https://microprofile.io/specifications/fault-talerance/
Password: MPreview
What I need are links to git for old versions
@Emily-Jiang @jclingan @edbratt @aeiras

For review: https://microprofile.io/specifications/fault-talerance/ Password: MPreview
What I need are links to git for old versions
@Emily-Jiang @jclingan @edbratt @aeiras

https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/1.0
https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/1.1.4
https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/2.0.4
https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/2.1.1
https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/3.0
https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/4.0.2
https://github.com/eclipse/microprofile-fault-tolerance/releases/tag/4.1

Updated


Swap material and mesh bind groups

Objective

Materials should be a more frequent rebind then meshes (due to being able to use a single vertex buffer, such as in #10164) and therefore should be in a higher bind group.

Changelog

For 2d and 3d mesh/material setups (but not UI materials, or other rendering setups such as gizmos, sprites, or text), mesh data is now in bind group 1, and material data is now in bind group 2, which is swapped from how they were before.

Migration Guide

Custom 2d and 3d mesh/material shaders should now use bind group 2 @group(2) @binding(x) for their bound resources, instead of bind group 1.
Many internal pieces of rendering code have changed so that mesh data is now in bind group 1, and material data is now in bind group 2. Semi-custom rendering setups (that don't use the Material or Material2d APIs) should adapt to these changes.

As a followup PR, we'll probably want to sort draws by pipeline id.

Hmm pretty sure someone flipped these for some reason awhile back (with some reasonable justification for going against the  "mesh before material" bind frequency advice) . Iirc it was some sort of technical details thing (ex: architectural limitation, platform support), not a performance thing. Worth trying to find that context.

cart from the past says:

Pretty sure for performance reasons we need to sort by bind frequency / assume that "later" bind groups will be rebound if earlier ones change. Given that we should be sorting by material and then drawing many meshes with the same material, I think the current order is correct

the current order used to be (potentially) better for performance before we batched meshes. now it makes more sense to put mesh first as the batch of meshes won't need frequent rebinding.

Right so after moving mesh data (not mesh vertex data, just the MeshUniforms) to GpuArrayBuffer, the bind group should almost never need to be rebound (for normal meshes).
What does need rebinding is:

When the type of mesh data changes (regular model only, skinned mesh, mesh with morph targets)
I think an extra dynamic index for each skinned/morph target mesh, but I'm not 100% sure having never really touched that code

(Note: We probably want to sort draws so regular/skinned/morph meshes are grouped together, after material sorting)
So the only way this PR loses out currently is for e.g. skinned meshes, where you may have two different skinned meshes with the same material. This PR would cause you to have to rebind the material for each unique skinned mesh, which is expensive. But this PR is better for regular meshes...

For CPU-driven rendering, I guess maybe it makes more sense to keep this as-is to avoid expensive material (pipeline) rebinds. For GPU-driven, where the pipeline is the only rebind, I can just put materials in bind group 2 anyways. I'm thinking we should close this.

Cool yeah that all makes sense!

@JMS55 I'm not so sure. I think this is useful for CPU-driven as well. Considering the two bind groups:

The mesh bind group has to be bound once for all non-skinned/non-morphed mesh entities, and rebound for each individual skinned / morphed mesh entity currently.
The material bind group has to be rebound for every single material instance (as in, each material asset regardless of its type) currently.

The usual case for mesh assets is lots of non-skinned + non-morphed mesh entities and few skinned / morphed mesh entities. So prioritising the defaults for non-skinned / non-morphed entities makes sense. And those require one binding as all the data they need is in one buffer.
The usual case for material assets with the current solution is a bit less clear:

One material asset per mesh entity - common for non-instanced models with textures and no texture atlas
One material asset per many mesh entities - instanced models, or simpler usage with flat colour materials. I don't really feel like we should optimise for the latter as it's simple enough anyway.

Either way, rebinding for a material asset is at best once per group of mesh entities, and at worst once per mesh entity. The only situation I can think of where it would be 'worse' would be for entities using different skinned/morphed mesh assets which have the same material. Unless I'm missing something, that seems very niche and unlikely?
I think at this point it does make sense to swap the material and mesh bind groups. I'd suggest reopening this and merging it, as well as in a separate PR, changing the sorting of the opaque and alpha mask passes. If we do swap the binding order, then we would want to sort by pipeline id, mesh bind group id + dynamic offset, material bind group id, z front to back. Though we'd want to test the impact on sorting performance for non-trivial configurations to see whether we could cut down on those sort key fields.

Would it be possible to move the skinning and morph data into storage buffers? More perf, and we'd avoid this whole issue to begin with.

i haven't checked performance - i don't expect any regressions but maybe we ought to benchmark?

Shouldn't be any regressions atm I don't think, as we don't sort primarily by pipeline/bind group yet. It'll limit the max perf we can have for skinned meshes/morph targets in the future though (as they use a separate vertex buffer per mesh), but that's a tradeoff we'll have to accept, unless we want to add a toggle to the shader code to switch the group orderings (which we could also easily do).

Would it be possible to move the skinning and morph data into storage buffers? More perf, and we'd avoid this whole issue to begin with.

I have a branch locally that does this for skinning. Morph uses textures so it's a different beast. And we still can't avoid it because the current approach is the best we can do for uniform buffers and not everyone has access to storage buffers.


Update sshd config syntax highligher

to current openssh 8.4p1

Have you tried contacting the maintainer?

Opening a PR does not count as contacting a maintainer?

No, each runtime file is maintained by each maintainer.
https://github.com/vim/vim/blob/master/CONTRIBUTING.md#syntax-indent-and-other-runtime-files

I contacted the maintainer without any answer. I also sent the email to vim-dev without any feedback for three weeks. So what is the procedure to get some changes in?

would you volunteer to maintain this file then?

Sure, if you point me to some guidelines/documentation what is expected from me if I would maintain this file.

Change the header to make Dominik "previous maintainer" and add yourself as the maintainer.
I haven't heard from Dominik for more than a year, and then he was very slow to respond.
It would be preferred to have a more active maintainer.

Sure, if you point me to some guidelines/documentation what is expected from me if I would maintain this file.

There are no actual guidelines.
Besides what Bram said, if you intend to maintain this file on a public repo, put the repo inside the runtime file so users can find it. Also, if new keywords are added by the openssh project, send updates as you feel comfortable with them.
Then of course it helps to be re-active and react to questions or issues that might arise. :)
But in general, this is not too much of an additional effort. I get barely any messages for e.g. the sqloracle syntax file that I maintain.
Also, please update the CODEOWNERS file.

Thanks for comments. I added couple of more missing algorithms and added myself as a new maintainer. Hope I will be able to cut some time for that with every new upstream release of OpenSSH.
I am fine updating it directly here rather than setting up a new repository for this.

I'll include it, thanks.


Correct HTML attribute parsing

Fix #6870.
Handling all possible character reference is I suppose definetely something external library comes into play.
There are several npm packages on this topic, and I chose entities judging from popularity and performance.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

closing in favour of #6891


Add Lingjie's Open Source Contribution

Great!  Your translation skills would be particularly helpful- that helps expand communities beyond just English speakers, which is more and more of a problem for global software projects.  I hope you consider going deeper over the summer.


Fix "only mail on mention" bug

fix #12774

Codecov Report

Merging #12775 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master   #12775   +/-   ##
=======================================
  Coverage   43.21%   43.22%
=======================================
  Files         649      649
  Lines       72000    72007    +7
=======================================
+ Hits        31117    31122    +5
- Misses      35839    35842    +3
+ Partials     5044     5043    -1

Impacted Files
Coverage Δ

models/user.go
53.87% <100.00%> (+0.31%)
:arrow_up:

services/mailer/mail_issue.go
57.83% <100.00%> (ø)

services/mailer/mail_release.go
80.64% <100.00%> (ø)

models/unit.go
46.57% <0.00%> (-2.74%)
:arrow_down:

modules/process/manager.go
72.50% <0.00%> (-2.50%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
53.84% <0.00%> (-1.54%)
:arrow_down:

models/gpg_key.go
54.81% <0.00%> (-0.59%)
:arrow_down:

models/issue_comment.go
53.75% <0.00%> (-0.16%)
:arrow_down:

modules/git/repo.go
50.76% <0.00%> (+1.01%)
:arrow_up:

modules/indexer/stats/queue.go
76.47% <0.00%> (+11.76%)
:arrow_up:

... and 1 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 1fbc50f...2460158. Read the comment docs.

Please send backport to v1.12


Multi-scale training and prediction

I have used ssd to detect texts in which the training input is 300x300, but the prediction input needs to be in different scales. But in practice, when I have changed the input scale, the prediction results is not right at all. After inspecting the caffe ssd implementation, I have found the reason:
min_dim = 300
...
min_ratio = 20
max_ratio = 90
step = int(math.floor((max_ratio - min_ratio) / (len(mbox_source_layers) - 2)))
min_sizes = []
max_sizes = []
for ratio in xrange(min_ratio, max_ratio + 1, step):
  min_sizes.append(min_dim * ratio / 100.)
  max_sizes.append(min_dim * (ratio + step) / 100.)
min_sizes = [min_dim * 10 / 100.] + min_sizes
max_sizes = [min_dim * 20 / 100.] + max_sizes

In caffe, the sizes of the anchor boxes are measured in pixels, while in mxnet, they are measured in proportion by scaling to 0-1 of input size. This is very important in multi-scale prediction because the receptive field of one convolution kernel does not change regardless of input size. So the model will hold in another input size only if the anchor box size is fixed, not depending on input size.
After changing the anchor box sizes to pixels, the MultiboxPrior layer should be modified according to caffe implementation:
        net[name] = L.PriorBox(net[from_layer], net[data_layer], min_size=min_size,
                clip=clip, variance=prior_variance, offset=offset)

The input data symbol should be included in arguments to cast pixel anchor size back to 0-1.
After making these changes, Textboxes can be reproduced with good performance.
Maybe the official mxnet example can also be revised likewise.

This issue has been recognized long time ago. What you have mentioned is a correct way to solve it, but will break the backward compatibility. I am still seeking a way to retain the current model compatibility.

Backward compatibility is really a headache. Maybe at least add a notice in REAME, it takes me a lot of time to find the reason.

@arcadiaphy
I  met the same problem ,  would you like to share the code you modified?
thx.

@goodtogood
I have port my modification to the latest ssd code, please refer to my forked repo.
The backward compatibility is totally broken. Here is an example to setup MultiBoxPrior:
    sizes = [[30., 42.4], [60., 82.7], [114., 138.4], [168., 193.1], [222., 247.5], [276., 301.8]]
    ratios = [[1, 2, 3, 5, 7, 10], [1, 2, 3, 5, 7, 10], [1, 2, 3, 5, 7, 10], [1, 2, 3, 5, 7, 10], [1, 2, 3, 5, 7, 10], [1, 2, 3, 5, 7, 10]]
    anchors = mx.contrib.symbol.MultiBoxPrior(
        *[from_layer, data], sizes=sizes[0], ratios=ratios[0],
        clip=True, name=name, steps=[], textboxes=True)

Changes to MultiBoxPrior layer:

add image input variable data to argument
sizes should be measured in pixels, not normalized to 0-1
steps should be set to [] in multi-scale prediction to enable automatic calculation

@arcadiaphy   thanks a lot !

Hi @zhreshold would you recommend leaving the current implementation as is for backward compatibility in that case?


fix: esbuild.charset not work with css plugin.

Description
Vite@4 has set the default config esbuild.charset to utf8, and in #12565 , the default configuration for plugin:css has also been set to utf8
This operation may lead to the generation of iconfont files being escaped, causing display errors in some cases. Although I can modify the esbuild.charset in vite.config.ts, it seems that the parameter is not passed to the plugin:css. I need to make it work.
Additional context
This is the effect of using iconfont(css) after Vite 4.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Would you move the code inside resolveEsbuildMinifyOptions?

@sapphi-red
Sorry, I don't intend to put this code in resolveEsbuildMinifyOptions because I just want to fix the code which the original author missed.
Additionally, charset is not part of the minify options, so I should not be retrieving properties which are not relevant to resolveEsbuildMinifyOptions.
If there are more places in the future that need to use options related to both charset and minify, I would consider creating a new function, such as resolveEsbuildOptionsForCssPlugin.

I should have put #9210 in minifyCSS function or should have put #12565 in resolveEsbuildMinifyOptions. That mistake is making the inconsistency here.
If we put charset in minifyCSS, then I think log* options should be in minifyCSS too. If we put charset in resolveEsbuildMinifyOptions, then I think log* options should be in resolveEsbuildMinifyOptions. I prefer placing charset in resolveEsbuildMinifyOptions because it'll make the diff smaller.

I should have put #9210 in minifyCSS function or should have put #12565 in resolveEsbuildMinifyOptions. That mistake is making the inconsistency here.
If we put charset in minifyCSS, then I think log* options should be in minifyCSS too. If we put charset in resolveEsbuildMinifyOptions, then I think log* options should be in resolveEsbuildMinifyOptions. I prefer placing charset in resolveEsbuildMinifyOptions because it'll make the diff smaller.

I understand. I was going to ask you to fix it.
But I was hoping that it would be merged soon to solve the problem in my project, so I raised a PR myself.
I still have<EMAIL_ADDRESS>in my project.
I'm glad you noticed the problem, I'm going to close the PR and let you fix the code.
Maybe you can rename resolveEsbuildMinifyOptions to resolveEsbuildOptions
Translated with www.DeepL.com/Translator (free version)


Create talent homepage -> a profile dashboard with talent read and write operation

Is anyone working on this issue, yet?

@jojothomas1515

Yes, I think Jojo is still working on this


Install: add Ubuntu Eoan, and remove old non-LTS versions

Eoan (Ubuntu 19.10) packages are now up; remove old non-LTS versions
from the list, as they reached EOL, and packages may now be outdated.
fixes https://github.com/docker/docker.github.io/issues/8891
fixes https://github.com/docker/docker.github.io/issues/10037

ping @usha-mandya @StefanScherer ptal


Basis-Cash Improvement Proposal #5: Lowering Seignorage TWAP Threshold

BIP-5
Title: Lowering Seignorage TWAP Threshold
Author: liquidity_king, @liquidity_king
Category: Economic
Summary:
Currently, Basis Cash only mints additional BAC to BAS/BAB holders when the TWAP for BAC is over 1.05  This proposal is to lower the seignorage threshold to a lower value voted on by the community.
Motivation:
Early on in an algo coins lifecycle, investors are primarily speculators who are betting on continued expansion, while having confidence that the bond mechanism backstop will encourage buying below peg. BAC does not have a natural source of demand, with demand primarily coming from speculation that the network will expand.
Speculators rely on the seignorage dynamic to support BAS prices, which ultimately drive demand for BAC. They take comfort that the BAB mechanism will support them when prices fall below peg. However, with the current parameter of 1.05 there is a gap from 1.0-1.05 where investors are not supported by the protocols incentives. Speculators buying BAC at 1.04 are not supported by BAB demand for another 4 cents, and they are also not going to be receiving any seignorage income from their farmed BAS. The 1.05 parameter also effects BAB redemptions, meaning that BAB buyers need to have faith that TWAP will reach 1.05 in a reasonable time frame. Lowering this threshold gives BAB more potency which is the most critical incentive for the protocols long term survival. Finally in the very long term having a more narrow threshold should help us target a more narrow peg which is the ultimate value proposition of the project.
Voting Options for BAS/BAB Seignorage Threshold:
*1.05 (current)
*1.04
*1.03
*1.025
*1.02
*1.01
*1.0

1000% support

i think 1.001 or 1 is ok !!!

1.01 perfect

1.01

0,01

1.01 or 1.02


cue: cue eval hangs up on recursive definition with default value

Cue hangs up (presumably with an infinite loop) when trying to run cue eval on the following code:
package tst

x: {
	a: A
	b: A
}

A :: string | [A]

This is with commit 275105a3c865a7444b9552bb115ab1c7db3c088b.

This issue has been migrated to https://github.com/cue-lang/cue/issues/217.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Option to override container name or to add prefix

Chart update request: Need the ability to override the container name with a value from values file
proxy as a container name is too generic which is causing issues in autodiscovery with our monitoring tools. We would like to name it as kong-proxy to differentiate it from other proxies.
https://github.com/Kong/charts/blob/main/charts/kong/templates/deployment.yaml#L129

What is the autodiscovery tool in question and what is its behavior?
In general I don't think container names should be a concern, since they're internal to a Pod and not really designed for consumption by other tools. The Pods do have more descriptive labels.

I collect metrics per each container of a pod and having standard names prefixed on container names allows me to group them easily since you cannot label individual containers within a pod.

With which tool? Is it not able to group containers by Pod or add Pod metadata to them?


Lua plugin resources are not bundled

Self help steps before reporting the issue:
[x] Check Solar2D forums for a solution
[x] If Android: make sure Java8 is installed and reported when running java -version and javac -version if not, download and install Java 8
Before reporting
resources bundled in Lua plugins are ignored
Platforms involved
Building on: macOS 10-15-16/Windows 7-8-10/
Building for: iOS/Android
repro
Build https://schroederapps.github.io/progress_ring_sample.zip, note that resulting apps does not contain plugin resources.

Could java 8 be included in the msi?

@mark-sgs thats a great ideal. I looked around and it seems that we totally can bundle it together, for both mac and windows, same versions Android Studio uses. It'll increase distribution size by about 100mb, but totally worth it, I think.


Update honeycomb section for OTLP export

Honeycomb recently changed to a "services-first" architecture that simplifies setup for tracing. Updating docs here to reflect those changes.

Thanks @vasireddy99!


Action submission buttons

Add a button "What's it worth?" to the left of the "I'll Do It!" button.  When evaluate action is pressed, do a Get request to find out the number of points, cost and savings. Show the explanation on screen or in a pop-up box with the following text:  If points is positive, "This action could save around xxx lbs CO2 emissions annually." If zero "No emission savings for this action."  but if negative "This action could cause xxx lbs CO2 emissions annually." (show the number as positive value).   If the cost is positive, add " Initial cost could be $xxx or more.", if zero add "Little or no cost to do".  If the savings is positive, add "It could save about $xxx each year.", if negative "It could cost you an extra $xxx each year".  If zero don't add anything.

@wkleiman - can you check out whether the explanation is correct on the screen, in the different cases?


Fixes to examples. Fixes #9

This change is

Hi @mbreakey3, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Thanks for your contribution!

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


do i have to install firebase first or i just have to install this library?

i did not install anything beside this library, i can receive notification token all , but can not receive remote notification from firebase console or firebase, but i can receive messages from cloudkit push notification,? does anyone know what is the problem, i set up everything

have you found a solution for this @ninjia0 ?


fix: be less strict about what's a bad tx

closes https://github.com/paradigmxyz/reth/issues/2257
Don't consider mistakes like insufficient funds or outdated nonce as malicious.

Codecov Report

Merging #2268 (b5edde8) into main (ec097fa) will decrease coverage by 54.62%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2268       +/-   ##
===========================================
- Coverage   70.75%   16.13%   -54.62%
===========================================
  Files         472      472
  Lines       56721    56738       +17
===========================================
- Hits        40132     9156    -30976
- Misses      16589    47582    +30993

Flag
Coverage Δ

integration-tests
16.13% <0.00%> (-0.01%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

crates/transaction-pool/src/error.rs
0.00% <0.00%> (ø)

... and 331 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


overhaul breeze dark

now breeze dark looks more like Breeze (even though with a darker colour scheme) thanks to QBreeze, the old theme is still available as "MultiMC-Dark"

I assume the reason it's called this was a misunderstanding?
I don't think it has anything in common with breeze...

I assume the reason it's called this was a misunderstanding? I don't think it has anything in common with breeze...

you mean this or the old one? for this I changed the color scheme to make it a bit darker, this is how it would look without it

but i think it looks worse. Maybe I should change name

I renamed it to Breeze-ish-dark btw

Looks pretty good visually but there are a few issues
Screenshots are invisible:

Launch button is weird when hovering:

(also tool buttons in general seem to shift 1px to the right when hovering)
Checkbox states only appear when selected or hovered:
https://github.com/user-attachments/assets/4800908a-b38f-4f44-9341-3237b07216db


Add docker build command in Makefile.

Add docker build command in Makefile so developer can build docker images as the following command.
$ make all docker

Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

make all docker won't work on osx or windows machines because the binary will be compiled for the host machine architecture

it is currently not possible to compile drone on osx or windows for use in a linux container, because it uses CGO to include SQLite. While I appreciate the pull request, my concern is that it will result in developer confusion when they run make all docker and it creates an OSX binary file that gets added to the Docker image, which will fail to start

@bradrydzewski Yes.
I want debug on osx. I need the following steps:

make all
docker build --rm -t drone/drone .
docker-compose up

this won't work because make all runs make build_static which compiles the binary using CGO and results in a binary that only runs on OSX architecture. Adding this binary to the image will cause the image to fail to start.
note that you can run Drone perfectly fine on OSX. It does not require a linux runtime, just a valid Docker daemon url

We should add a shell and bat script that builds the binary within a docker container and than builds the docker container afterwards that can be used

With that we can even get rid of the makefile like we did for plugins

when I develop drone locally I use go install && drone server

@tboerger 👍
Because I run drone on linux machine and debug using https://github.com/drone-demos/drone-on-docker-compose
When I modified the drone code base, I need run three command as below.
make all
docker build --rm -t drone/drone .
docker-compose up

So I think we can add docker build command in Makefile for docker user.

Different users, different commands I'm using make builds static && ./release/drone server

@tboerger I know the ./release/drone server command. I just want to run docker command in Makefile.

make all is btw a waste of compile time because it builds ALL binaries instead of the one you need for development or testing.

@appleboy in this case the command is not cross platform, and would likely result in osx developers trying to use it and logging bugs when it doesn't work. You should definitely feel free to create local shell scripts that help you develop more efficiently, but in some cases like this we may choose not to merge.

@tboerger I know make build_static.
I just want to simplify the steps as below on Linux maction:
make build_static
docker build --rm -t drone/drone .
docker-compose up

to
make build_static docker
docker-compose up

For docker user.

I know what you mean. But as I said above, I think building .sh and .bat files should be better because they are usable on all supported platforms.

@bradrydzewski @tboerger
OK. Thanks for response quickly. 👍


uv python pin should find .python-version file in a workspace

If working in a child directory of a workspace, uv python pin should support finding the .python-version file in the workspace root.
Related #4970

I thought of changing version_files.rs to use Workspace::discovery() internally to find the version files. Though uv-distribution can't be added as a dependency of uv-python as that would creation a cycle.
Should I instead change all functions in version_files.rs to accept a path as argument and search the version files there? Or is there a simpler solution that I'm missing cause of lack of context?

Yeah based on that description I'd probably add requests_from_version_file_in(root: PathBuf) and keep the workspace discovery over in the uv crate.
Thanks for looking into it!

Should uv python pin 3.12 also only write to the project root or the workspace root? At present, it writes to the current directory, but this means it won't be picked up by other commands like uv python install unless they're run in that specific directory.

This is a good (and open) question. Do users opt-in to operating on the project or operating on the workspace? When working in the workspace root, we require --project <name> to affect a project. When in a project directory, we should probably require --workspace to mutate the workspace?

I'm assuming that a workspace can contain more projects, correct me if I'm wrong. Given that I would image this behaviour.
If called in a project subfolder it would pin for the first closest project.
If called in a workspace folder it would pin for the workspace only.
If you want to change this behaviour you'd use the --project <name> or --workspace options as you propose.
The --isolate option ignores all the above and pins in the current working directory as you said in https://github.com/astral-sh/uv/pull/5035#issuecomment-2226960463.
It makes sense in my head but maybe it could be confusing to some? 🤔


[Q] Create matplotlib and save as wandb.Image

In my imitation learning experiments, I am logging a video of the gym environment's rollout, obtained at each iteration of my model's training. Together with the video, I want to log a plot (which also should be updated at each training iteration) representing some latent variables added by myself to this gym environment. However, to create this plot I need to use matplotlib or seaborn, but the core get dumped as soon as I use these.
latents = np.zeros(10)
fig, ax = plt.subplots() # FAILS HERE
ax.plot(latents)
fig.canvas.draw()
# Convert the canvas to a raw RGB buffer
buf = fig.canvas.tostring_rgb()
ncols, nrows = fig.canvas.get_width_height()
image = np.frombuffer(buf, dtype=np.uint8).reshape(nrows, ncols, 3)
self.wandb_run.log({"rollout/latent": wandb.Image(image)})

How could I add a plot relating to the particular rollout?

Hi @liubaoryol, this might be better to raise in Matplotlib repo if it is failing before running any wandb code unless you suspect it might be caused by the wandb library?
Thank you,
Nate

WandB Internal User commented:
nate-wandb commented:
Hi @liubaoryol, this might be better to raise in Matplotlib repo if it is failing before running any wandb code unless you suspect it might be caused by the wandb library?
Thank you,
Nate

Hi! Thank you for your response. I found some workarounds to this, so I will not be needing this anymore


Error while trying to delete single message through API

Hi guys!
First of all thanks for great work with creating MailCatcher! It works like a charm, but I'm having problems with deleting single message through the API. I can easily delete all of them, but not a single one. When trying to do that I get the following error:
SQLite3::RangeException at /messages/1
bind or column index out of range
file: statement.rb location: bind_param line: 41
The thing is that the message with id 1 does exist and I can retrieve the body by calling GET http://intranet_hosl/messages/1.json or 1.html or whatever. But DELETE http://intranet_host/messages/1 still fails.
If that helps I can attach whole error message (HTML) returned by Mailcatcher.
Would be greateful for help as I'm using MC with some automated test cases and I need to be able to delete single messages, not all of them.
Cheers,
Bartek

This is very strange as I can't replicate this issue. Does it happen all the time, for every message?

Yes, all the time and for every message I tried deleting (meaning: not even once deletion of a single message via API succeeded).
Would attaching error page I get when calling API help in any way?

I still can't reproduce this.
Please try upgading to 0.6.0 with gem install mailcatcher. If you're still having issues please feel free to reopen this issue.


One-QA V1 release: BooleanQA: scorer

jsmc created the following on Apr 12:
The version of Tydi scoring used in the MRC script only supports factoid questions, not boolean.
Needs to add support and refactor so that it can be used to evaluate the Tydi-with-boolean system.
Original issue: https://github.ibm.com/ai-foundation/PrimeQA/issues/105

On May 19, jsmc commented:
done


Magnify lens stays active on mouse leave

I'm currently using Magnifier on 'inside' mode. When the user's mouse leaves the thumbnail the magnifier stays active over the thumb and onThumbLeave never fires. This only happens with very slow mouse movements.

Sometimes it stays for me, too. But not always.

how can we fixed it


profile import failure

Howdy
file: \Commands\Common\NewCommand.php
import was missing:
use Distill\Distill;
extract structure was wrong, so changed from Lines 561-562:
$extractPath = $this->projectDir . DIRECTORY_SEPARATOR . '.' . uniqid(time()) . DIRECTORY_SEPARATOR . 'pwprofile';       $extractionSucceeded = $distill->extractWithoutRootDirectory($profile, $extractPath);
to
$extractPath = sys_get_temp_dir().DIRECTORY_SEPARATOR.uniqid(time()).DIRECTORY_SEPARATOR;       $extractionSucceeded = $distill->extract($profile, $extractPath);

@Stikki Thanks for reporting the fix - I just hit this issue too.

Thanks for reporting, I have corrected it now.


all Themes color fixes for unbound reporting

missing .query-error color for all themes and some other fixes for perfection ;-)

@AdSchellevis would be great if you could merge this color fixes before next release. thx. have a nice weekend. regards rené

Thanks René! With a bit of luck it's in time for the next release, but I'll gladly leave that to @fichtner

@AdSchellevis Thank you anyway. Yes, I'll see if it's included in the next release. If not, it's not a problem either. Nevertheless, thanks for the quick implementation. Greetings Rene


Added SuperMarioBros2Japan-Nes and HangOn-Sms

Thanks for the PR!  I'll look at the build failure, which does seem unrelated.  I'm afraid we are not accepting new game integrations currently: https://github.com/openai/retro/blob/master/CONTRIBUTING.md
I added a section to the docs about how to add your own path for integrations: https://github.com/openai/retro/blob/master/docs/integration.rst#using-a-custom-integration-from-python


Add Japanese translation of README

This PR includes:

Translated README (ja_JP)

With this translated README, Japanese language users will easily access Lima.

Signed-off-by: KOSHIKAWA Kenichi <reishoku.misc @ pm.me>

Thanks for commends. Fixed all.


Removed version declaration from Composer file

Composer recommends excluding the version and instead it can infer the version from the latest tags.

Awesome! Thanks for the pull request.

Packagist seems a little finicky about Git tags having a v prefix. I retagged 0.2.0. Let me know if you have any issues with it.

Can confirm that 0.2.0 is working well. Thanks!


cannot read property onDidStop of null

VSCode Version: 1.4.0
OS Version: win 10 ver 1601

I see error :   cannot read property onDidStop of null
Steps to Reproduce:

debugging powershell,

I wonder it could be related to the power shell extension on VS Code. I would suggest to check with the extension - https://github.com/PowerShell/vscode-powershell
Please reopen the issue if not so. Sorry for not being very helpful here.

i upgraded to 0.7.1 today. It actually makes it worse :-)
The program hangs when I hit F5

This issue was moved to PowerShell/vscode-powershell#265

@maxcli I moved this issue to Powershell repository


feat: @uploadthing/expo

TODOs

[ ] validate upload progress, I don't seem to get any progress indications on native. I've tested uploading the same file from web and it does work so seems related to RNs implementation of XHR?
[ ] verify it works deployed? I don't have an Apple Developer Account to test this with though...

Demo Example
https://github.com/pingdotgg/uploadthing/assets/51714798/dc449522-a367-4f15-a6b7-425225f77f7f

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #579

PR #583   👈

This stack of pull requests is managed by Graphite.

Excited about this

React native for web, the File response from image picker, does not have a size property, which is required for startUpload

I suggest this should work with RN for Web. I can help with supporting that, there are some things that I don't think work like the File response from image picker, does not have a size property, which is required for startUpload

What??? I would have imagined web being the place where you could expect them to follow standards the most.... Another evidence RNW is a mess

Pushing out another round of canary. You shouldn't have both @uploadthing/react and @uploadthing/expo, just the Expo one. (If you do end up having both make sure it's the same version used by the expo lib)

@younes200 I improved the URL resolving a bit to work more ootb on web


[feat] cargo deb

I would be awesome if cargo-deb is available into the image https://github.com/kornelski/cargo-deb

Extending your dockerfile to install cargo-deb, and start packaging gives  /root/.cargo/bin/cargo-deb: 1: Syntax error: word unexpected (expecting ")")