.NET 6 C# - ScriptPropertiesGenerator fail with overridden initialized property on master

Godot version
v4.0.beta.mono.custom_build [56522f7f8] (with double)
System information
Arch Linux, Vulkan
Issue description
C# projects with a parent class including a virtual property that is then overridden & initialized in a child class appear to cause ScriptPropertiesGenerator & ScriptSerializationGenerator to fail to generate source.
The problem appears to be caused by #70483, the Source Generators run fine if built immediately before this PR was committed.
I'm not sure how to debug the Source Generators for a more useful error/stack trace than below, sorry.
Parent Class
using Godot;
public partial class TestParentClass : RigidBody3D
{
    public virtual int NodeType { get; set; } = 1;
}

Child Class
If the initialized property override below is commented then the Source Generators work fine.
public partial class TestChildClass : TestParentClass
{
    public override int NodeType => 10;
}

Warnings/Errors produced
Warnings produced when a child class overrides the initialized property from its parent:
0>CSC: Warning CS8785 : Generator 'ScriptPropertiesGenerator' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'NullReferenceException' with message 'Object reference not set to an instance of an object.'
0>CSC: Warning CS8785 : Generator 'ScriptSerializationGenerator' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'NullReferenceException' with message 'Object reference not set to an instance of an object.'

Any [Export] properties then throw build errors as ScriptPropertiesGenerator failed:
0>csharpbug_03c26d6/Godot.SourceGenerators/Godot.SourceGenerators.ScriptPropertyDefValGenerator/Nothing_ScriptPropertyDefVal.generated.cs(9,33): Error CS0117 : 'Node.PropertyName' does not contain a definition for 'TestInt'
0>csharpbug_03c26d6/Godot.SourceGenerators/Godot.SourceGenerators.ScriptPropertyDefValGenerator/Nothing_ScriptPropertyDefVal.generated.cs(11,33): Error CS0117 : 'Node.PropertyName' does not contain a definition for 'TestFloat'

Steps to reproduce
Build the solution in the minimal reproduction project.
Minimal reproduction project
csharpbug_03c26d6.zip

I didn't consider that there may be non-readonly properties with a null SetMethod. https://github.com/godotengine/godot/pull/71128 should fix it.

[Context: This bug is so elusive and hard to catch, that I cannot reproduce it reliably enough to isolate a proper issue. Here's a way it happened to me if it helps anyone. Using Godot 4.3-stable and a self-compiled build of 4.3-stable with PR #79731 baked in. Both of them worked the same. This makes it C#/Mono exclusive.]
The one thing that triggered this bug is this asset from the Asset Library. It's not the asset itself that causes this. This bug happened unexpectedly (while on a game jam), even if under the same conditions it worked before.
The conditions were:

Implementing from a class below any level of abstraction that required overriding an auto-set virtual property. (same as shown above)
Have, or not, any kind of [Export] attribute attached to the derived class. Could derive from any Godot object, or anything below done by the programmer. (same circumstances as described on the issue, but on a larger scale)
The [Export] property were both on auto-set property and fields. This one happened on the asset above.
Once you got an error on one, it propagated to any previously-functioning class with similar conditions. Probably because CSC gave up, like the error shown on the issue.
Not even a dotnet clean could fix anything.

The only solution was:

Removing every virtual property and exported field from all files on the game.
Manually editing the project.godot file and the .tscn files to remove some broken script references.
Converted my virtual properties to abstract.
Changed to .NET 8.0 as a target.
Delete the whole .godot folder.
Regenerate the mesh import database.
dotnet clean from the terminal.
dotnet build from the terminal.
Build _from within Godot.
Play.

It might've been a game-breaker for many. Some steps might look unnecessary but they were in this particular case. The game was so tainted by several days of having this time bomb lurking that I did lose some code, and thankfully I did have some working backups to fall back into.
I did find a comment on the ScriptPropertyDefValGenerator.cs file that kind of makes me think of something.
I am not a professional C# programmer, nor expert in the field, but I did find a pattern:

Godot did let me, at some point, export a field/property that was read-only, had a private setter, or was, as described here, virtual and overridden later on. Not a single previous warning in MSBuild, the engine or previous warning.
Godot did not know it happened, or that it was bad, until something triggered the error above and then it cascades down below to parts I did not were wrong.
The code generator got stuck in such a way that anything, even things completely unrelated, became erratic. Exported fields/properties became so unpredictable that builds sometimes failed, sometimes passed, sometimes changed slightly. It's all because CSC failed at some point. My game at some point booted with the camera completely inverted for one person. On several it caused the audio to revert to -0dB (in my case, something that was almost tinnitus-inducing). Compiles became completely non-deterministic. I can blame that to my own code, though... but it does seem weird the random way it manifests.
Of course, the stack trace in Godot .NET is so sparse (to put it very mildly) that it took me around 6 hours to figure this out. The issue in godot-propsals/issues/6811 was in full swing. The only way I could notice it was where it was failing: that asset. And it was working until suddenly it didn't.

I write this after quite a while of figuring out the details of this bug... but I cannot reproduce them 100% reliably, mainly because the aforementioned evasive nature of it. But my gut feeling is that it's not resolving this specific combination of abstraction correctly, and that, worst case, we should get a warning this pattern is not recommended and can cause later harm.


Setting custom colors

The default gray for ABSTAIN labels is difficult to see on my screen. I wondered if there was an easy way to set custom color per label.

On the recipe (i.e. simple_annotator) level there currently isn’t. For ABSTAIN the light gray color is kind of the point — ABSTAIN is the unlabeled background upon which labeled colors are applied.
If you’d like a custom color palette, the best place to do it is probably a Recipe class. Let me give more thought to this.
Best regards,
Haochuan

From: Kwame Porter Robinson @.>
Sent: Sunday, August 21, 2022 1:22:59 AM
To: phurwicz/hover @.>
Cc: Subscribed @.***>
Subject: [phurwicz/hover] Setting custom colors (Issue #45)
The default gray for ABSTAIN labels is difficult to see on my screen. I wondered if there was an easy way to set custom color per label.
—
Reply to this email directly, view it on GitHubhttps://github.com/phurwicz/hover/issues/45, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEZTDWVQTRIASFS544AWHNDV2EH7HANCNFSM57DQYN6A.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Great, looking forward to it. I actually thought the Bokeh icon was a color wheel and clicked it to modify the current label color.
As an aside, maybe a roadmap needs-help-issue, UI accessibility will be greatly enhanced if users are able to control color settings and defaults. For example, Bokeh put accessibility related concerns on their roadmap, https://github.com/bokeh/bokeh/issues/12259, and other products let you specify color settings, such as the commercial prodi.gy that has a crude bulk labeling script. Libraries like colorcet provide ready made perceptually accurate pallets for dense data visualizations (colorcet.holoviz.org) too.
On Aug 20, 2022, at 9:36 PM, Harry Wei @.***> wrote:

On the recipe (i.e. simple_annotator) level there currently isn’t. For ABSTAIN the light gray color is kind of the point — ABSTAIN is the unlabeled background upon which labeled colors are applied.
If you’d like a custom color palette, the best place to do it is probably a Recipe class. Let me give more thought to this.
Best regards,
Haochuan

From: Kwame Porter Robinson @.>
Sent: Sunday, August 21, 2022 1:22:59 AM
To: phurwicz/hover @.>
Cc: Subscribed @.***>
Subject: [phurwicz/hover] Setting custom colors (Issue #45)
The default gray for ABSTAIN labels is difficult to see on my screen. I wondered if there was an easy way to set custom color per label.
—
Reply to this email directly, view it on GitHubhttps://github.com/phurwicz/hover/issues/45, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEZTDWVQTRIASFS544AWHNDV2EH7HANCNFSM57DQYN6A.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Hi @robinsonkwame, custom colors are now supported 🎉
# enter the hex code that works best for you
hover.config['visual']['abstain_hexcolor'] = "#dcdcdc"

for more information, you can
hover.config.hint()


'window-all-closed' event on darwin

The quick-start code provided for main.js in the docs makes an exception for darwin related to the 'window-all-closed' event triggering app.quit():
https://github.com/atom/atom-shell/blob/fdc07dea177f52ae0943db320358ee1ea8da5107/docs/tutorial/quick-start.md
Is this exception still necessary?  I've got some code triggered by the 'before-quit' event that is not getting run on OSX due to this exception.

just need to update the docs, you can remove the exception.

On OS X it is a convention that most apps do not quit when all windows are closed, that code is to follow this convention.
You can create a separate issue if you observed a bug.

Ahh, Forgot abt tat, thanks!

@zcbenz does this event window-all-closed still exists? It doesn't fire on MacOS, no matter if:

there's an app window and I window.close()
there's an app window and user press cmd+W
there's an app window and user press cmd+Q


Twitch clone

🛠 Fixes Issue
Closes #590

👨💻 Changes proposed
Added twitch clone

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Remove package.lock-json file

Removed


ARXIVNG-499 added tests, changes to patch method, to update event-based submission object with classic DB status

This is mainly just verifying that the core package represents submission state correctly when publication-related changes have occurred.

Test

Thanks @mhl10 ! @DavidLFielding anything else to tweak before we merge?


Case insensitive search

Description
Hello Team,
I have the following use case.
Store/Index data without tokenize (use StringField) and maintain the same string while storing (maintain case sensitivity while storing data). The problem I face is during search. If I store field called name as 'Lucene' and if I try to search with 'lucene', I'm not able to get the store data with 'Lucene'.
I'm trying to use Lucene 9.5.0
`public class LuceneExample {
public static void main(String[] args) throws IOException, ParseException {
// Create index
KeywordAnalyzer analyzer = new KeywordAnalyzer();
String indexDirectoryPath = new StringBuilder("/temp/").append("test").toString();
// create the directory to store the index
Directory directory = FSDirectory.open(Paths.get(indexDirectoryPath));
IndexWriterConfig config = new IndexWriterConfig(analyzer);
IndexWriter writer = new IndexWriter(directory, config);
    Document doc = new Document();
    doc.add(new StringField("name", "John Smith", Field.Store.YES));
    writer.addDocument(doc);
    writer.close();

    // Search index

    // create a reader to read the index
    IndexReader reader = DirectoryReader.open(directory);
    IndexSearcher searcher = new IndexSearcher(reader);
    QueryParser parser = new QueryParser("name", analyzer);
    Query query = parser.parse("john smith");
    TopDocs results = searcher.search(query, 10);
    for (int i = 0; i < results.scoreDocs.length; i++) {
        Document hitDoc = searcher.doc(results.scoreDocs[i].doc);
        System.out.println(hitDoc.get("name"));
    }
}

}`
Am I missing something here? Could you please help me out.
Thanks in advance!

You are. The prescribe method for dealing with this sort of pre-processing is through using the analysis chain.
If you add this line (and associated imports) to the top of this snippet you should be good to go:
StandardAnalyzer analyzer = new StandardAnalyzer();
Keep in mind that the standard analyzer does not process things like stop words, so the text should be largely unaffected. If you only want to affect case strictness, you can look into the LowerCaseFilter applied at the character level. Be sure to apply at index- and query-time.

[SrikanthMedisetti] please use TextField instead. Not an issue.


Are Google Pay and Apple Pay supported ?

Are Google Pay and Apple Pay supported ?

No, unfortunately currently not supported


Add alias operator

Suggestion
Add alias operator, this would create a link to the original object.

🔍 Search Terms
"alias"
"alias operator"
"alias prefix"

List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.
"alias"
✅ Viability Checklist

My suggestion meets these guidelines:

[Y] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[Y] This wouldn't change the runtime behavior of existing JavaScript code
[Y] This could be implemented without emitting different JS based on the types of the expressions
[Y] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[Y] This feature would agree with the rest of TypeScript's Design Goals.

Also this would be easy to implement, since u can just replace the names of the aliases with the originals.
⭐ Suggestion

Add an alias operator that would be used like this:
var examplea = 10;
alias exampleb = examplea; //exampleb is now linked to examplea
examplea += 10; //examplea is now 20
console.log(exampleb); //will print "20", since exampleb is linked to examplea
exampleb -= 5; //exampleb is now 15, thus, examplea is now 15 too
console.log(examplea); //will print "15"

this would compile to:
var examplea = 10;
examplea += 10;
console.log(examplea);
examplea -= 5;
console.log(examplea);

📃 Motivating Example
would allow multiple names for a function, for example:
if i have a class called color, i could add an alias called colour, which would be nice

💻 Use Cases
allowing multiple names for one object and making it easy to add aliases.
currently there are alot of problems trying to do it, like when u want to make a class color and colour, you'ld need to either duplicate classes, or make one of them the other, which is not the same.

Emphasis mine:

This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)


Dropdown not working on bootstrap 4 Alpha 6

Dropdown is not expanding using Bootstrap v4.0.0-alpha.6. According to https://github.com/twbs/bootstrap/issues/20939 on Alpha.6 the class .open was renamed to .show:
"Renamed some .active, all .in, and all .open classes for our JavaScript powered components for more consistent naming. For example, instead it's now .collapse.show instead of .collapse.in and .show > .dropdown-menu instead of .open > .dropdown-menu to better communicate the visibility of elements."
Currently ng2-bootstrap dropdown is still adding the .open class. If I change it manually to .show it is working, so it should be updated.
Thank you.

see #1458 and #1553

thanks, closing as this is duplicate


Introduce new JSON schema

New JSON schema
Examples migrated
New helper functions
Refactored jq magic and iterations in the shell scripts
Pass on cluster and addon_config objects as terraform -var=... iso x amount of vars
Add support for Argo FQDN exposure
...

not suggesting that this is an ideal approach: https://github.com/tetrateio/tetrate-service-bridge-sandbox/pull/309/commits/0d9cecaaa459a8390955698ec74cccf8628797bb#diff-a86cc0bb6aa6cd0d6d4ed8384e11316a90d232f310ed9cfa0d7100032b7b2265, i.e. I had to apply in order to unblock the testing...
however having local.tetrate_defaults is a map to have a parity with var.tetrate as map not as object type, does simplify the merge between, however object type does provide a better representation of how the object should be defined, I hit the same dilemma when crafting tse sandbox. Basically having an open map as any gives an ability to instantiate empty variable that can be after on expanded with defaults.
Technically map can be involved to something like to have variable constraints defined as well as per chatgpt...
variable "tetrate" {
  description = "An object containing global tetrate configuration"
  type        = map(any)
  default     = {}

  validation {
    condition     = can(regex("^.+\\..+\\..+$", var.tetrate.fqdn))
    error_message = "The fqdn must be a valid fully qualified domain name."
  }

  validation {
    condition     = length(var.tetrate.password) >= 8
    error_message = "The password must be at least 8 characters long."
  }

  // Add additional validations as necessary
}

as of now based on the applied changes no issues with make k8s step, hitting the below when applying make tsb will continue tomorrow...
│ Error: Insufficient features blocks
│
│   on <empty> line 0:
│   (source code not available)
│
│ At least 1 "features" blocks are required.
╵
make: *** [tsb_mp] Error 1

vars file
{
  "k8s_clusters": {
    "aws": [],
    "azure": [
      {
        "region": "eastus",
        "tetrate": {
          "control_plane": true,
          "management_plane": true
        }
      },
      {
        "region": "eastus",
        "tetrate": {
          "control_plane": true
        }
      }
    ],
    "gcp": []
  },
  "tags": {
    "tetrate_owner": "sergey",
    "tetrate_team": "ce"
  },
  "name_prefix": "r172dev7",
  "tetrate": {
    "fqdn": "r172dev7.azure.sandbox.tetrate.io",
    "image_sync_apikey": "<REDACTED>",
    "image_sync_username": "<REDACTED>",
    "organization": "tetrate",
    "password": "<REDACTED>",
    "version": "1.7.2"
  }
}

Resolved conflicts, have been independently reviewing with @shamusx , need tomorrow to do manual testing and we should be good to merge


Emulator is not familiar with the LUIS endpoints outside of North America regions

If you add a LUIS service, the emulator does not expect a LUIS URL different from https://www.luis.ai and it cannot connect to https://eu.luis.ai and other endpoints mentioned here - https://docs.microsoft.com/en-us/azure/cognitive-services/luis/luis-reference-regions.
We need an additional configuration parameter asking for the LUIS URL

Which version of the emulator does this pertain to?

4.0.15-alpha. Thanks!

Ran into the same issue yesterday. The emulator expects all LUIS services to run under https://www.luis.ai.

Tracking this here


Account type

Hi,
Im also having issues authenticating. I suspect my account to be the problem though. I have the SHIELDAlarm account. Do you expect this to work with that account type?
If not, maybe that could be your next integration project ;-)
Kjeld-

It works on my old G4S Everhome system

This onlu works with the G4SSMARTa larm system. I unfortunately don't  have access to any of their other systems so I can't develop for them


⚠ St. Lawrence Global Observatory CKAN has degraded performance

In 4ac907e, St. Lawrence Global Observatory CKAN (https://catalogue.ogsl.ca/en/) experienced degraded performance:

HTTP code: 200
Response time: 9834 ms

Resolved: St. Lawrence Global Observatory CKAN performance has improved in 7ddc682 after 15 minutes.


How to use sendSPLTokens

Hi, first of all thanks for this great library.
I'm trying to use sendSPLTokens to transfer custom token to another wallet address, and I'm a bit stuck on what it means by fromPublicKey and destinationAddress
I tried either wallet address and token account address for fromPublicKey, but I got the below error

Transaction simulation failed: Attempt to debit an account but found no record of a prior credit

extension Action {
    public func sendSPLTokens(
        mintAddress: String,
        from fromPublicKey: String,
        to destinationAddress: String,
        amount: UInt64,
        allowUnfundedRecipient: Bool = false,
        onComplete: @escaping (Result<TransactionID, Error>) -> Void

In web3.js I can understand that the from and to refer to the token account address, not the wallet.
splToken.Token.createTransferInstruction(
        splToken.TOKEN_PROGRAM_ID,
        fromTokenAccount.address,
        toTokenAccount.address,
        fromWallet.publicKey,
        [],
        0
      )

I'm using testnet and my wallet has enough custom token https://explorer.solana.com/address/BPkzsoetXHfH228V8B74xpZC22SNVgu3NX8hiwNPbEpH?cluster=testnet
I know this is not really an issue with the library, but how to correctly use this method.
Thanks, any help is appreciated 🙏

Probably the allowUnfundedRecipient needs to be true (if the destination doesn't already hold some of that token).

Thanks for the answer. That was the issue 👍


Mega Menu: Fix top padding. Fix brochure icon. Hide menu column header.

Changes

Fix top padding above the overview link, so that the hover has correct height.
Change order brochure link icon to be a document instead of a complaint.
Hide redundant menu item group header.

Testing

Pull branch and gulp clean && gulp build
Enable MEGA_MENU_VAR_1 flag.
Visit homepage and resize desktop to view extra large and regular desktop sizes and see that hover for overview link is equal height on the top and bottom. Also see that second column in menu item groups has a hidden heading. Also see that the Order Brochures icon has changed to a document.

Screenshots

@chosak Thanks for catching that. Pushed up a fix.


Implement Content Security Policy

Should we enable this?
Contra
Argo CD does not implement this.
https://developer.mozilla.org/en-US/docs/Web/HTTP/CSP

@jannfis thoughts?


<EMAIL_ADDRESS>extract_dir error

Installing '7zip' (19.00) [64bit]
Loading 7z1900-x64.msi from cache
Checking hash of 7z1900-x64.msi ... ok.
Extracting 7z1900-x64.msi ... ERROR Could not find '7-Zip'! (error 16)

/cc @niheaven

ref: https://github.com/lukesampson/scoop/issues/3198#issuecomment-470772222
This may be caused by lessmsi, and will be fixed in https://github.com/lukesampson/scoop/pull/3204


Update Maven to 3.6.3

@leantk could you take a look at this?

LGTM

@colemarkham can you remove the change to the readme file (which should get rid of the conflict)? Once that's done I'll merge it in. Thanks!

@alepauly Fixed. I only updated the README because I noticed it was wrong. It says 3.6.1, but the version in the current image is 3.6.2.

@alepauly Fixed. I only updated the README because I noticed it was wrong. It says 3.6.1, but the version in the current image is 3.6.2.

Thanks for noticing it. It should become accurate soon (if it isn't already) as we go through finalizing the repo and making sure our processes are updating it correctly.


Rscript is not installed properly.

Rscript version 4.3.0 fails all the Rscript checks in SCAFE.
Rscript --version
Rscript (R) version 4.3.0 (2023-04-21)

https://github.com/chung-lab/SCAFE/blob/90afd96279c51302ea31521622b468954cc54df7/scripts/scafe.tool.cm.filter#L716C2-L716C58
https://github.com/chung-lab/SCAFE/blob/90afd96279c51302ea31521622b468954cc54df7/scripts/scafe.tool.cm.annotate#L693C26-L693C26

I have also run into this problem.  The Rscript version message has changed between Rv3 and 4.  If R v4 works fine, could you suggest a fix ?


feat: add pagination support for listing repos

Description
This PR fixes #41.
It adds pagination support for retrieving the list of repos of the organization.
Additionally it cleans up the temp directory created for cloning a repo.
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

The failed workflow run seems to be a known issue of the actions/checkout workflow with a recent release:
https://github.com/actions/checkout/issues/1448

Hey @netomi,
thanks a lot for this contribution! Was definitely needed.
The lazy "just take 100, we don't have more right now" really wasn't a sustainable solution :)

The failed workflow run seems to be a known issue of the actions/checkout workflow with a recent release:
actions/checkout#1448
130 thumbs up after 24 minutes :-)
Second re-run made it work.

Yes we experienced this in other repos as well

I am surprise that the otherwise great go-github library does not have pagination support out of the box. There are some ideas floating around which I took as inspiration, but having to do that yourself for every api that potentially only returns paged results is certainly a pain.


Automatic deploy to IPFS

Decentralization

Requested removal of workflow as it seems to be consistently failing
https://github.com/web3privacy/web/actions/workflows/ipfs-deploy.yml


How to install GenomeWorks on local machine?

Hello everyone,
I am a research scholar, and I need to test GenomeWorks  on sample data. Could anyone please explain how to install and run on my system?
System details:
UBUNTU 18.0
6GB CUDA -enabled NVIDIA GTX 1660Ti
Intel core-i7 9th Gen
16GB RAM
Thanks in advance.

Hi @kountaydwivedi - the setup instructions for GenomeWorks are described in https://github.com/clara-parabricks/GenomeWorks#genomeworks-setup .
Are you interested in using the C++ APIs or python APIs?

Thanks @tijyojwad for  your response.
I have installed GenomeWorks as per the setup instructions. But I'm unable to execute any command or tool in the terminal, unlike  the GATK4 application, in which I could easily access each commands after installation. For eg:
I am trying to access-
$~ cudamapper
it is showing "command not found"
Also, I am going to use python APIs.

Is is possible to use GenomeWorks with Anaconda on Windows/Ubuntu?

Hi @kountaydwivedi  - we don't install our binaries/libraries into any default system location. The easiest to add the binary to your path would be (on Ubuntu e.g.) export PATH=$PATH:<build/install/bin>.
We support Ubuntu but not windows. And an Anaconda package is on our roadmap but we don't have one generated yet. We plan to add that soon!
For the python API you would need to follow the instructions under pygenomeworks folder to install from source, or you can simply install from PyPi - e.g. https://pypi.org/project/genomeworks-cuda-10-1/ for CUDA-10.1

Thanks @tijyojwad for your response. I would try to solve the issue. If there would be a problem, I'll reply to you.
Thank you once again.

Hi @kountaydwivedi could you solve the problem? If so, we would close the issue. If not, happy to help again.

Hi @kountaydwivedi could you solve the problem? If so, we would close the issue. If not, happy to help again.

Am having trouble installing Genome works on a Jetson nano... Anyone with tips on how to go about it?

Am trying to setup genome works on the jetson nano ... but it keeps exiting at the 'make -j install' stage with an error

Thank you.....
Finally I was able to install it.....
My interest was cudamapper and I was able to run it successfully and get
the expected results.....
On Monday, 8 February 2021, ahehn-nv<EMAIL_ADDRESS>wrote:

Hi @PMuchina https://github.com/PMuchina.
First of all, running GenomeWorks on NVIDIA Jetson Nano is not supported,
since GenomeWorks requires a GPU with CUDA compute capability >= 6.0 and
NVIDIA Jetson Nano has compute capability 5.3.
However, you might be able to try GenomeWorks on NVIDIA Jetson Nano
nevertheless, if you modify the CMakeLists.txt files and replace all
occurrences of "-arch=compute_60" with "-arch=compute_53". (If I remember
correctly, the only occurrences are in cudaaligner/CMakeLists.txt and
cudapoa/CMakeLists.txt.) Please note that this has not been tested at
all, is not supported officially, and may show poor performance.
Unfortunately, it is hard to understand what went wrong in your case
specifically, since you did not mention what error you got during the
build. I might be able to help if you post the message.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clara-parabricks/GenomeWorks/issues/593#issuecomment-775413177,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANGRD6D4KGHMMLI5E7UPEF3S6BAPTANCNFSM4TCQWEGA
.


[Compute] vm extension set: Fix bug where users could not set an extension on a VM with --ids

Issue #9592
Fix bug where users could not set an extension on a VM with --ids.
Changes:

Move --name and --extension-instance-name outside Resource Id Arguments group
Update --extension-instance-name help text
Add example

It's a small breaking change. According to data explorer, --ids usage:
30 day, 13 request
1 day, 9 requests, (I ran tests)
So no problem.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

@Juliehzl please help with this PR.

az vm extension set -h

Command
    az vm extension set : Set extensions for a VM.
        Get extension details from `az vm extension image list`.

Arguments
    --name -n      [Required] : Name of the extension.  Values from: az vm extension image list.
    --publisher    [Required] : The name of the extension publisher.
    --extension-instance-name : Name of the vm's instance of the extension. Default: name of the
                                extension.
    --force-update            : Force to update even if the extension configuration has not changed.
    --no-auto-upgrade         : If set, the extension service will not automatically pick or upgrade
                                to the latest minor version, even if the extension is redeployed.
                                Allowed values: false, true.
    --no-wait                 : Do not wait for the long-running operation to finish.
    --protected-settings      : Protected settings in JSON format for sensitive information like
                                credentials. A JSON file path is also accepted.
    --settings                : Extension settings in JSON format. A JSON file path is also
                                accepted.
    --version                 : The version of the extension.

Resource Id Arguments
    --ids                     : One or more resource IDs (space-delimited). If provided, no other
                                'Resource Id' arguments should be specified.
    --resource-group -g       : Name of resource group. You can configure the default group using
                                `az configure --defaults group=<name>`.
    --subscription            : Name or ID of subscription. You can configure the default
                                subscription using `az account set -s NAME_OR_ID`.
    --vm-name                 : The name of the Virtual Machine. You can configure the default using
                                `az configure --defaults vm=<name>`.

Global Arguments
    --debug                   : Increase logging verbosity to show all debug logs.
    --help -h                 : Show this help message and exit.
    --output -o               : Output format.  Allowed values: json, jsonc, none, table, tsv, yaml.
                                Default: json.
    --query                   : JMESPath query string. See http://jmespath.org/ for more information
                                and examples.
    --verbose                 : Increase logging verbosity. Use --debug for full debug logs.

Examples
    Add a user account to a Linux VM.
        az vm extension set -n VMAccessForLinux --publisher Microsoft.OSTCExtensions --version 1.4 \
            --vm-name MyVm --resource-group MyResourceGroup \
            --protected-settings '{"username":"user1", "ssh_key":"ssh_rsa ..."}'

Zunli's questions demonstrate that this command is quite confusing. The command name set and parameter names --name, --extension-instance-name, --vm-name are not instructive.

Zunli's questions demonstrate that this command is quite confusing. The command name set and parameter names --name, --extension-instance-name, --vm-name are not instructive. However, vm extension set has 26737 times usage. Not easy to redesign.

You are right that this isn't a clear command. Maybe adding some more examples using different parameter sets / groups could help clarify the command.


Revert "Merge pull request #2902 from yebblies/typeofva_argsave"

This reverts commit f07a77d106432d70468bb86e34f2ef5b8cb12d61, reversing
changes made to 4bd949406af646370504208cd569ace0707f2b82.
It's breaking std.outbuffer unittest in Win64 platform.

Auto-merge toggled on

@yebblies Ping.

Thanks for fixing this.


Allow passing number | string to order endpoints

Currently, the values in NewSpotOrderParams only accept numbers. Since all numbers in JS are floats, this leads to some numbers having recurring decimals when converting to base 10, tripping the filter "Precision is over the maximum defined for this asset." even after normalizing the number (with Math.floor(qty / precision) * precision).
Allowing passing numberInString or simply string (as numberInString | number) would allow the user to trim the passed value with Number.toFixed.
eg:
const qty = 10/3; //  => 3.3333333333333335
const precision = 0.0001;
const normalized = Math.floor(qty / precision) * precision; //  => 3.3333000000000004 (rejected)
const trimmed = normalized.toFixed(4) //  => "3.3333" (accepted)

JS can be quite bad at precise math, as you also noticed here. Even when trimming and rounding to try and work around this, you're likely to see this happen again and cause other problems in your workflows.
If you're dealing with sensitive math (such as order amounts, values, prices, anything money related), I really recommend using a decimal library such as big.js or decimal.js for any calculations that require precision. Once your calculations are done you can still extract a number (or a string if you prefer) from this special type.

That part is sorted out. My suggestion is just to change the typings to allow strings, so people don't need to use hacks like this to circumvent the requirement for some values to be numbers:
{
  quoteOrderQty: amount.toFixed(4) as unknown as number
}

That part is sorted out. My suggestion is just to change the typings to allow strings, so people don't need to use hacks like this to circumvent the requirement for some values to be numbers:
{
  quoteOrderQty: amount.toFixed(4) as unknown as number
}

If the API accepts strings for these fields (even the docs say numbers only), I'm absolutely open to that. It was also suggested here - just need to update the type to accept number | string in those fields, as long as each field has been tested to support that.

Before using your library, I was passing everything as strings because the request is made using form data and every param is sent as string anyways (as opposed to JSON, that has bools, numbers, null etc).
I'd love to be able to change that but I can't make the tests pass as is in my environment. Are there any documentation on that?

Before using your library, I was passing everything as strings because the request is made using form data and every param is sent as string anyways (as opposed to JSON, that has bools, numbers, null etc).

That's a good point!

I'd love to be able to change that but I can't make the tests pass as is in my environment. Are there any documentation on that?

The tests take API credentials for the private portion. I have a sub-account dedicated to these tests, although I did notice the API key expired (since it doesn't have IP whitelisting enabled), so I'm fixing that shortly.
What're your tests failing with? Missing credentials?
If you're in a unix or macOS shell, you can just pass API credentials using env vars as such:
API_KEY_COM="yourapikeyhere" API_SECRET_COM="yourapisecrethere" npm run test:watch

That's how I normally do full test runs locally and that's how the CI run executes these tests too.

(just as a clarification about my suggested type: I suggested numberInString because it's an existing type used widely on the project that resolves to number | string)

Also a good point. I'm undecided about using it in a request scenario where either of the two types is OK...because it's currently used for responses, but only responses with the beautifier enabled will always return a number for this field, or if the beautifier is disabled they will always return a number in a string.
I had hoped to eventually improve how types resolve so that typescript knows "this client was instanced with the beautifier enabled, so all numberInString fields are definitely number and never string. I don't have an immediate solution for this but it's a point I've wanted to reach with that specific response property type. If that point is reached, I'm not sure it would cause conflict or confusion for requests fields that are not either or but can be both...

I had hoped to eventually improve how types resolve so that typescript knows "this client was instanced with the beautifier enabled, so all numberInString fields are definitely number and never string.

I don't know enough about TS to help you with that right now, but I think it would be a fun subject to study! :)
I'll let you know as soon as I have a working example.

I had hoped to eventually improve how types resolve so that typescript knows "this client was instanced with the beautifier enabled, so all numberInString fields are definitely number and never string.

I don't know enough about TS to help you with that right now, but I think it would be a fun subject to study! :)
I'll let you know as soon as I have a working example.

That would be extremely helpful! I'm also very curious how one might resolve that. It would be easy if the boolean controlling the beautifier were included in a function call (although that would be an ugly design choice too in my opinion), but right now any API call would somehow need to know that this part is going via the beautifier so any numberInString is actually a number...
https://github.com/tiagosiebler/binance/blob/master/src/util/BaseRestClient.ts#L244
Maybe passing in a generic TNumber. If via the beautifier, any numberInString<TNumber> would actually be numberInString<number> (thus returning only a number type), else numberInString<string> (thus this property contains a string, since the beautifier didn't resolve it).
Tricky but interesting challenge. Open to design changes too if it makes it easier to solve, within reason.

Good new is that it seems possible with conditional types:
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBDAnmApnA3nAogRwK4CGANgDTYAeqAxvAL5wBmUEIcA5AAJKoC0VAFsSIoAdgHMUAZwD0eGMCKS2AWABQa7mgBCKAnOANgKACYBhIkZHwAvBjgAjXfoaIAXAih40tDcjQAlAgB3c0sbO0c9eRd3BmJJb19UOFDRcJ0ogyMzCzS4AB84QJDcqzUktAA5PBBHKAB5KABlGChgcQAeABU4FHIYUWNJFNKYAD44Wx6+gZEhuAznbNSrOAB+OBEaurh3SVb2sXLVTTgqAgTh2wBtNTgKahgO3EIiDuralAbmg870SOcbg8XloYzIW0+UDGYLuDxQNGe+GI722X0aLTafwB0SBcUUKFBZH2mLE0JIagAukA

I thought it might be in that direction, good stuff, though I still wonder how the implementation might look on something like the rest clients...does each method need a pass-through generic in the return value?

The type of the client can be inferred by the values in the constructor like this:
https://www.typescriptlang.org/play?#code/C4TwDgpgBAQghgV2ASwGbIgEwApwE5wC2UJAvFAN5QBGEiKqIAXFMHgtAL4CwAUKJCgAlOAHdcBYiWnTyVWvTTMoqOABsAzlz4DoAYTUYAdsAlENMqOQAU8JGgw58RKAB9hYs4QCUUAGSUAPbAABYQeCwARGDOhBqRPLw64NAAcgiEtHgA8ngAymzIRgDmADwAKlAQAB7AEEaYFgbGprEaAHxWUJU1dQ0WdgyOXlAA-FBGGVlQLBqFJXx8mBAAxmr40GtwGk2G9cAVVbX1jVDN+14dlHwkK4FGc+wrwIF41jGSGizl3gDcN1BihBgAA1dQcazeFjpTLhXIFPBFMrldr-XiJPh3B7AKBrFoAQS6RggojOexM1goCnsjBYbA4ABooMEwhEoNE2glvJjycB8QA6IGg8EQSGLXhYua43kwIkksktSnUhjKVSaCBMlnhKIfcxcnktGCC4FgtQQ7m8IA

Ah that is quite nice, all that's left then is passing in the generic to the type return by each method. Nice!


Incorrect tripple-slash directive in sub-directory

🔎 Search Terms
tripple-slash directives references
🕗 Version & Regression Information
v4.x did not emit these directives at all.
⏯ Playground Link
https://github.com/icholy/tsc-tripple-slash-bug
💻 Code

src/types/fs.ts

declare module 'fs' {}

src/a/x.ts

import * as fs from 'fs';

export { fs };

src/a/b/x.ts

import * as fs from 'fs';

export { fs };

🙁 Actual behavior
Emitted files:

lib/a/x.d.ts

/// <reference path="../types/fs.d.ts" />
/// <reference types="node/fs" />
import * as fs from 'fs';
export { fs };

lib/a/b/x.d.ts (using types instead of path).

/// <reference types="lib/types/fs" />
/// <reference types="node/fs" />
import * as fs from 'fs';
export { fs };

🙂 Expected behavior
Emitted files:

lib/a/x.d.ts

/// <reference path="../types/fs.d.ts" />
/// <reference types="node/fs" />
import * as fs from 'fs';
export { fs };

lib/a/b/x.d.ts

/// <reference types="../../types/fs" />
/// <reference types="node/fs" />
import * as fs from 'fs';
export { fs };

Additional information about the issue
This seems to be related to module resolution.

Looks like the same issue as https://github.com/microsoft/TypeScript/issues/56571

@fatcerberus that does look like the same issue. But I'm not clear on why lib/a/x/d.ts outputs a path="..." while lib/a/b/x.d.ts outputs a types="..." directive?

@RyanCavanaugh do you have any insight about why the two files contain different variations of the directive?


Linux/ARM?

Hi folks,
Is it on the roadmap to release a Linux/ARM binary?
Thank you,

See #1.

I guess I should follow good open source practices and read existing issue first 😆
Thank you

The reason for this request is to have Docker Selenium containers that work for M1 and other ARM machines where Chrome is used instead of Chromium.
https://github.com/seleniumhq-community/docker-seleniarm

Thank you for the additional context on your use case!


ISO 8601 formats

We need to coordinate across the Jisc specs for the particular variant of ISO 8601 to be used for datetime. See jiscdev/analytics-udd#302

Now agreed to have a standard ISO 8601 format of:
YYYY-MM-DDThh:mm[:ss.mmm]Z
Seconds and milliseconds [:ss.mmm] are optional and will be zero filled where not given. Examples:
2012-03-29T10:05Z
2012-03-29T10:05:00Z
2012-03-29T10:05:00.000Z


RehypeRaw is incompatible with RehypeSlug

As the title says. If the rehypeRaw plugin is added (in order to use GFM), it works fine until you use the rehypeSlug plugin, in which case the inline HTML is not properly rendered. They work separately.

I think it works fine.
https://stackblitz.com/edit/svelte-exmarkdown-rehype-slug?file=src%2FApp.svelte&terminal=dev
Please share reproduction sample!


Format of schematics file

Hi, I'd love to see the schematics of your preamp, but I can't open the SCHX file (LiveSpice format? I'm on Mac). Can you post it in PDF or other cross-platform file format please? Thanks!

Sure thing, eventually I will upload a more universal netlist, but in the mean time I can post an image.

Image now uploaded

Image now uploaded

Am 27.05.2024 um 06:19 schrieb rossbalch @.***>:
Image now uploaded
—
Reply to this email directly, view it on GitHub https://github.com/rossbalch/cc-reactor/issues/1#issuecomment-2132612850, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYO3P7UYLWN6NAQ6RRW2L4TZEKX5HAVCNFSM6AAAAABIFW2E6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSGYYTEOBVGA.
You are receiving this because you authored the thread.

Thanks! :)
Ingo


fix: add code coverage npm scripts

Prerequisites checklist

[X] I have read the contributing guidelines.

What is the purpose of this pull request?
Add npm scripts to collect code coverage from tests. Run with npm run test:coverage in a package directory.
What changes did you make? (Give an overview)

Added a c8 config for the whole repo.
Added c8 dependency and test:coverage script to each package.json file.

Related Issues

Is there anything you'd like reviewers to focus on?
This should trigger a new release.

should we also update the ci to run test:coverage？

This should trigger a new release.

I don't think it's a bugfix, nor should it trigger a release. maybe ci or chore?


Node not working in parallel flow

First thanks for the node. This is exactly what I need in my project. I love the comprehensive documentation as well.
But I have a strange problem with the node when I try to read I2C registers in parallel.
Case:
I have a PCF8574 attached to a raspberry PI. This chip has I/O Pins that can be used for switches etc. If anything changes on the I/O Pins an interrupt will be generated on a GPIO PIN (PIN 11 in my case). After the PI receives this interrupt, the register of the 8574 can be read via the I2C bus to determine which I/O port changed the status.
Flow:
When I start the following flow by pressing a button, only the Read register part will be executed. The "Button events" node receives the msg object, but does not process the data as exepected. I checked that the node gets the msg object with the VSC remote debugger by setting a breakpoint in the input routine. But was not able to trace why the node is not working because the debugger goes way deep into several .js files and I was not able to follow.
In the following flow only the upper streams works. the 254 indicated that the I/O PIN 0 has been changed and brought back to normal after I released the button. The 0 and 1 come from the debug node at the bottom.

When i 'cut' the wire from the trigger to the read register, the "Button events" node works perfectly. Single and double clicks are recognized reliably.

I'd like to run the detection of the button press in parallel due to the requirement to read the I2C register as fast as possible and I'd like to avoid multiple nodes between the GPIO Node and Button events node.
Versions:
Welcome to Node-RED
===================

6 Apr 18:02:38 - [info] Node-RED version: v2.2.2
6 Apr 18:02:38 - [info] Node.js  version: v14.19.1
6 Apr 18:02:38 - [info] Linux 5.15.30-v7l+ arm LE

Expected behaviour:
Node works as designed
Behaviour:
Node not working
[{"id":"7118ec3979daf874","type":"tab","label":"Flow 1","disabled":false,"info":"","env":[]},{"id":"d98e10d19293939d","type":"i2c in","z":"7118ec3979daf874","name":"Read register 0x3f","busno":"1","address":"63","command":"","count":"1","x":650,"y":80,"wires":[["fd69ee1d0f28b5e2"]]},{"id":"fd69ee1d0f28b5e2","type":"debug","z":"7118ec3979daf874","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"false","statusVal":"","statusType":"auto","x":870,"y":80,"wires":[]},{"id":"ab594a152238e00e","type":"button-events","z":"7118ec3979daf874","name":"","outputs":2,"inputField":"payload","inputFieldType":"msg","outputField":"clickmode","outputFieldType":"msg","downValue":"0","downValueType":"num","upValue":"1","upValueType":"num","idleValue":"1","clickedInterval":200,"pressedInterval":200,"debounceInterval":30,"events":[{"type":"clicked"},{"type":"double_clicked"}],"x":420,"y":160,"wires":[["56c5eb89f356fad0"],["70e6ba374db38c31"]]},{"id":"70e6ba374db38c31","type":"debug","z":"7118ec3979daf874","name":"","active":false,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","statusVal":"","statusType":"auto","x":590,"y":180,"wires":[]},{"id":"96eaec33084d06eb","type":"pi-gpiod in","z":"7118ec3979daf874","name":"","host":"localhost","port":8888,"pin":"17","intype":"PUD_UP","debounce":"240","read":false,"x":140,"y":120,"wires":[["39fe7ca11fc54102","36bd7c0ed4a1e5d4","ab594a152238e00e"]]},{"id":"36bd7c0ed4a1e5d4","type":"trigger","z":"7118ec3979daf874","name":"","op1":"1","op2":"","op1type":"str","op2type":"nul","duration":"120","extend":false,"overrideDelay":false,"units":"ms","reset":"","bytopic":"topic","topic":"AllBinary","outputs":1,"x":420,"y":80,"wires":[[]]},{"id":"56c5eb89f356fad0","type":"debug","z":"7118ec3979daf874","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","statusVal":"","statusType":"auto","x":590,"y":140,"wires":[]},{"id":"39fe7ca11fc54102","type":"debug","z":"7118ec3979daf874","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"false","statusVal":"","statusType":"auto","x":410,"y":240,"wires":[]}]

Hi @HarterHorst,
Thanks for the nice analysis of the problem!  That makes thing much easier to follow...
That is a strange problem.  So it seems that the first chain of nodes (trigger, read register, ...) has impact on the second chain of nodes (button events, ...).  Cannot immediately explain this.  Because - unless things have changed recently - NR works like this when 3 wires are connected to an output:

The original message is send to the first wire
The message is cloned and that clone is send to the second wire
The message is cloned again and that clone is send to the third wire

Since the button-event event node will receive a copy of the original message, how on earth can there be a relation...
Could you do some extra experiments please:

In the second flow, add a debug node to your trigger node.  And then test, to see whether there is a problem at the output of the trigger node.
If you remove the trigger node, and you replace it by the read-register node.  Does my button-event node still work?
Others ...

Hopefully a bell will start ringing when I have extra feedback...
Bart

Hi, glad to hear that you struggle with this in the same way. First I thought that I missed  fundamentals regarding objects in flows.
Your interpretation of the flow is correct.
Here are the results from the additional test:

Add debug node after trigger node. You can see, that the trigger node modifies the msg.payload and makes it a string. Beside this no signs of trouble.

No changes in the behaviour when I remove the trigger node.

As I wrote, I connected to the nodered service with the vsc debugger and set a breakpoint at your node. Here are the results. On the left hand you can see the object, which looks good to me.

You can find the flow in my original post.

Since I don't use the gpiod nodes, it is not possible to reproduce your problem.
So I have replaced your gpiod node by a trigger node, to simulate the behaviour:
[{"id":"99b05d5ec7a850ba","type":"i2c in","z":"b72068aa707669b0","name":"Read register 0x3f","busno":"1","address":"63","command":"","count":"1","x":710,"y":260,"wires":[["cf70786923288cba"]]},{"id":"cf70786923288cba","type":"debug","z":"b72068aa707669b0","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"false","statusVal":"","statusType":"auto","x":930,"y":260,"wires":[]},{"id":"16f81ac983ae6d0a","type":"button-events","z":"b72068aa707669b0","name":"","outputs":2,"inputField":"payload","inputFieldType":"msg","outputField":"clickmode","outputFieldType":"msg","downValue":"0","downValueType":"num","upValue":"1","upValueType":"num","idleValue":"1","clickedInterval":200,"pressedInterval":200,"debounceInterval":30,"events":[{"type":"clicked"},{"type":"double_clicked"}],"x":480,"y":340,"wires":[["02013e600b986b47"],["2eace1ebe45a1e4e"]]},{"id":"2eace1ebe45a1e4e","type":"debug","z":"b72068aa707669b0","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","statusVal":"","statusType":"auto","x":650,"y":360,"wires":[]},{"id":"d3d2db86c70a3c09","type":"trigger","z":"b72068aa707669b0","name":"","op1":"1","op2":"","op1type":"str","op2type":"nul","duration":"120","extend":false,"overrideDelay":false,"units":"ms","reset":"","bytopic":"topic","topic":"AllBinary","outputs":1,"x":480,"y":260,"wires":[["99b05d5ec7a850ba"]]},{"id":"02013e600b986b47","type":"debug","z":"b72068aa707669b0","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"true","targetType":"full","statusVal":"","statusType":"auto","x":650,"y":320,"wires":[]},{"id":"a6f84e9dac9c30db","type":"trigger","z":"b72068aa707669b0","name":"","op1":"0","op2":"1","op1type":"num","op2type":"num","duration":"120","extend":false,"overrideDelay":false,"units":"ms","reset":"","bytopic":"topic","topic":"AllBinary","outputs":1,"x":260,"y":260,"wires":[["16f81ac983ae6d0a","d3d2db86c70a3c09"]]},{"id":"576ea356da93195c","type":"inject","z":"b72068aa707669b0","name":"0","props":[{"p":"payload"}],"repeat":"","crontab":"","once":false,"onceDelay":0.1,"topic":"","payload":"0","payloadType":"num","x":90,"y":260,"wires":[["a6f84e9dac9c30db"]]}]

But that still works for me, even when the 'read register' node is available.
Could you please update the timings in my flow so that I can use it to reproduce the problem.


Add datasets with tasmax / tasmin

It's been a couple times I would have liked to have datasets (at least two simulations) with tasmax, tasmin variables. Something like datasets in sdba (which have tasmax, pr).
Should I add new datasets with tasmax/tasmin, or update existing datasets in sdba with a 3rd variable?

How breaking of a change would it be to the existing SDBA tests? Would you be willing to update all associated tests?

As far as tests are concerned, I think only test_stack_variables would need to be changed.
Also, sdba{_advanced}.ipynb assume we only work with pr, tasmax, in this case I would propose to simply drop tasmin from the start and keep the same structure.

It's not really related to tests, so maybe I should avoid complicating the structure. I was just using this to test methods for R&D projects, so it's maybe not a real need for xclim_testdata.

I ended up downloading a few lon/lat points from datasets on the server. I would be happy to add tasmin datasets in the test datasets if we judge we should have this, but I'm not sure if this is necessary.
When checking if adding tasmin would pose problems with xclim, I realized that the datasets in xclim-testdata are intented for tests (duh), so my use case of trying methods for new bias corrections is not what we intend to cover with xclim-testdata (correct?). For instance, it would also have been useful for me to have access to radiation datasets, but I don't think we want this if we are to keep xclim_testdata minimal.
So, if you guys agree, feel free to close the issue, if not I can add the tasmin datasets.

Indeed, the point of this repo is mainly for testing. But I use it too for development as it gives an easy access to well-formatted data.
As for adding variables to the SDBA test datasets, I think it would make sense once we have multivariate methods to test. Currently, tasmin  would be no use because in an univariate context you could just do tasmin = ds.tasmax.rename('tasmin'). But in a mutlivariate context, it would be interesting to have "real-life" data with all the noisy correlation and other perks.

I think it would make sense once we have multivariate methods to test.

We do some multivariate tests in sdba.ipynb, but for now tasmax / pr is sufficient for our needs.

serves both the unit tests and the performance tests of xclim

This would be neither of those currently.

Unless you're adding a significant amount of data (>3 MiB would be significant)

Ok, yes I was thinking something the same size as the current sdba datasets. I'll go forward with this when I have a down time.


Revamp Your Website: Stylish Navbar Menu for Contributing to an Open-Source Project for a Sleek and Modern Design

Before Changes

After Changes

Hi @Vin205
I recently updated the navbar menu for your project to make it more attractive and user-friendly. Enhance readability, improve navigation, and give your site a visually appealing touch that stands out with these simple yet impactful design updates. I've attached screenshots of the changes, and I'd love to contribute to this enhancement to improve the overall design.
Please assign me to this task. It would be a great opportunity to contribute while enhancing my skills.
I'm looking forward to hearing back from you!
Best regards,
@khurshed07
https://github.com/khurshed07

i want to work on this issue .assign me this project


Here's a new site I'd like you to consider blocking: facebook

Here's a new site I'd like you to consider blocking:
www.facebook.com
Unfortunately the plugin doesn't work with FB.

Facebook is one of a few sites that I've given up on. The names of the elements on the page are dynamic and change too often to deal with.
I don't know that it blocks comments, but you might want to check out Fluff Busting Purity, an extension that deals just with Facebook.


Unhandled act Account.Steam.get_user_id

Everything has been working fine since the beginning of this mod for me.
But since the new update, I got this strange message and the game is just infinite loading.
Any idea what could be the problem ?

Account.Steam.get_user_id has always been unhanded. If you updated to v1.2.0 it is not compatible with v1.11 of YgoMaster. v1.12 has been released which is compatible with the v1.2.0 update.


[SPARK-24537][R]Add array_remove / array_zip / map_from_arrays / array_distinct

What changes were proposed in this pull request?
Add array_remove / array_zip / map_from_arrays / array_distinct functions in SparkR.
How was this patch tested?
Add tests in test_sparkSQL.R

Test build #92351 has started for PR 21645 at commit 87100ec.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/491/
Test PASSed.

Test build #92351 has finished for PR 21645 at commit 87100ec.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

@HyukjinKwon @felixcheung
Could you please review the changes? Thank you very much in advance!

Test build #92891 has started for PR 21645 at commit 1b6a113.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/873/
Test PASSed.

Merged to master.


PSPGP HashAlgorithmTag "SHA256"

Hi Przemysław Kłys, thanks so much for this great ps for pgp. Have a question of how to change HashAlgorithmTag?
As read PgpCore, the default of HashAlgorithmTag is SHA1. I want to change it using SHA256.
Any parameter of your ps or other way that can achieve it?
Thanks.

This is currently not supported. WE would need to expand Protect-PGP and once it connects  $PGP = [PgpCore.PGP]::new($EncryptionKeys) we need to allow people to change those values

It doesn't seem like a hard thing to do. Feel free to submit PR or I'll fix it at some point

Protect-PGP -FilePathPublic $PSScriptRoot\Keys\PublicPGP.asc -String "This is string to encrypt" -HashAlgorithm Sha256 -CompressionAlgorithm Zip

I've exposed different parameters and it should be possible to use it now


Update VSIX to remove API filtering.

This update removes filtering of specific API's in the error/warning build event telemetry.  The update includes a rebuilt binary for the VSIX.

This change is

msvc/language/BuildMonitor/source.extension.vsixmanifest was updated, replacing BuildMonitor metadata with the VSIX's own information. Expected?

:shipit:


fix: do not check invalid node_modules folder.

There are packages (resolve) that distribute its test code in the package.
https://github.com/browserify/resolve/issues/171
And in those test code it can contain the folder node_modules which messed up the calculation.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


README: simple typo correction

Reference Issues/PRs

No specific issue. Test first PR
What does this implement/fix? Explain your changes.

Found a typo with a dead link in README file.
Does your contribution introduce a new dependency? If yes, which one?

No
PR checklist for new estimators

Any other comments?

Thanks @hiqbal2, will merge this directly!


Add simple, standalone client

Adds a new function lychee::check(), which removes
a lot of boilerplate for simple cases. Adjusted the code,
tests, and documentation.
The downside is that check now returns a Result, so
we have to use ? to get to the response. That's because
we have to account for the case where the given string is
not a valid URI.

So this PR includes a few more changes:

Add doc-comment tests
Change exclude behavior to make includes more intuitive
Improve progress visualization
Add support for colored output

About 2.:
Previously an include alone would not mean that only the included
patterns were checked. Only if an exclude was given as well,
the includes would make a difference. Now, the includes on their
own will work as expected.
Moved the exclude methods into the Exclude mod.
Also changed the order of exclude tests to do the fast lookup
ones before the regex ones.
Added tests to guarantee behavior in the future.
I could move those changes into a separate PR. Then again the benefit would be very low at this point.
So I'll go ahead and merge it as part of this existing PR.
What the changes all have in common is that they improve ergonomics in a wider sense, so I'll go ahead and update the title before merging.


Correction for raw affiliation Department of Obstetrics and Gynecology, Western University

Correction needed for raw affiliation Department of Obstetrics and Gynecology, Western University
raw_affiliation_name: Department of Obstetrics and Gynecology, Western University
new_rors: 02grkyz14
previous_rors: 05cgtjz78
works_examples: W4389177972;W4389172677
contact: 6c870721bc0a9db3aef5c7b3b8c20e93:4d3d70fcdba440df @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Datepicker bug with year.

I found a weird behavior with the datepicker input.
Changing the year on the dropdown change the year in the date preview on the left even though a new date hasn't been selected.
The year should stay the same until a new date is selected.
I am using version beta release 0.100.
EDIT: when I say "the date preview", I am talking about the blue panel.
EDIT 2: I made a quick video to better illustrate the issue https://vimeo.com/226567416

It happened with me too. I use the 0.100.1.

And how do you preview where you are currently in the calendar?

@philipraets

@gusbemacbe I am describing a different issue.

@17tlhon please provide screenshots, video / gif to show the exact issue.

@DanielRuf, were you asking me or to @17tlhon?
If me, then I put the calendar inside the form, using s5 xl12. I could not see the year when I open the calendar. And if you touch everywhere or near the calendar in a page, the calendar always opens itself.

@gusbemacbe see the handle (after @) in my comment. Regarding your issue, if it is not the same please open a new issue with more information.

@DanielRuf I include a video see the original post.

I updated the latest commit, using terminal @Dogfalo solved the issue and I saw the moth and year now. It worked.

The date and month will still change before any date has been selected, however now once a date has first been selected and either the month or year view is changed, the preview will not update until a new date has been selected
Fixed in befe826c45b07afe70c1b32bdce74ebf99161e59


numpy.einsum support

opt_einsum has the capability of detecting some contraction patterns that:

cannot be computed with tensordot
might be more efficiently computed with other functions

In these cases, opt_einsum will try to call the einsum function of our library if it provides one. We should implement a einsum function that detects this kind of patterns and calls the appropiate optimized function. In case no pattern is found, it will fallback to tensordot.
This functionality is critical for general compatibility of rosnet with array-like interface consumers; e.g. opt_einsum, numpy, quimb, ...
Patterns
A list is shown in https://numpy.org/devdocs/reference/generated/numpy.einsum.html.

[ ] trace np.trace
[ ] diagonal np.diag*
[ ] array axis sum np.axis
[ ] transposition / permutation np.transpose
[ ] matmul/dot np.matmul np.dot
[ ] vector inner/outer product np.inner, np.outer
[ ] tensor contraction np.tensordot
[ ] broadcasting, element-wise and scalar multiplication np.multiply
[ ] chained array operations np.einsum_path

Current solution is to use helper functions from opt_einsum that can compute FLOP count and input/output array sizes. We may want to exhaustively match einsum patterns with array operations for better dispatching.


change the name from enterprise scale

@mosabami : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mosabami: I'm sorry - only the author of this article, @BrianBlanchard, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@mosabami - is the date the only change?

#label:"aq-pr-triaged"

#sign-off  the only change is the date. this has been reviewed by the content team.

@JanetCThomas: I'm sorry - only the author of this article, @BrianBlanchard, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


[EventPage] Adds 'Add to Calendar' integration

Replicate of https://github.com/wncc/InstiApp/pull/286

[x] Android

[x] Fix manifest permissions

[x] iOS

[x] Fix manifest permissions

Overall working is fine now.

[ ] Add error handling when a calendar is deleted
[ ] Add setting in SettingsPage to clear don't ask again defaults

events are added to calendar by default


fix Fix result wrong when query rewrite by mv if query contains null_unsafe equals expression (#39629)

Proposed changes
commitId: 5d4ad028
pr: https://github.com/apache/doris/pull/39629

run buildall

run buildall


Clarify information about error results

One could (and did) interpret that cqe->res mathes the return value of preadv2 and pwritev2.
Clarify that it matches the errno values used by these two.

Looks good! Might be worth making it fully complete and mention that cqe->res is -errno, not just for read/write operation but in general for any request type that is a mirror of the equivalent system call?

Thanks for the tips! I should have some time to look deeper during the weekend.

I can pull this one for now, just send an updated one when you have time. Thanks!

Pulled it, and added a commit on top. Hopefully this will do it... Change in question:
https://git.kernel.dk/cgit/liburing/commit/?id=4c23254cc770505816a307ed8c4f17f9a4aaefb9


Move cargo-clippy into cargo

Cargo-side tracking issue for moving cargo-clippy into cargo
See full details at https://github.com/rust-lang/rust-clippy/issues/3837

Closing this since we have decided not to do this.  See https://github.com/rust-lang/rust-clippy/issues/3837#issuecomment-617413116 for more details.


SKPaint with more font weights such as a Thin, Light, Heavy, etc

Hi again!  I am wondering if SkiaSharp can support more font weights? SKTypefaceStyle seems to only support normal, bold, italic and bolditalic.  Have I missed something?
I think this is an important capability for modern apps as we have moved past the days of 'bold' and 'not bold' being enough choice for our user interfaces.
iOS has good native support:
public enum UIFontWeight
{
  UltraLight = 0,
  Thin = 1,
  Light = 2,
  Regular = 3,
  Medium = 4,
  Semibold = 5,
  Bold = 6,
  Heavy = 7,
  Black = 8,
}

Android native support is (a bit weird) via the family name:
var Thin = Android.Graphics.Typeface.Create("sans-serif-thin", Android.Graphics.TypefaceStyle.Normal);
var Light = Android.Graphics.Typeface.Create("sans-serif-light", Android.Graphics.TypefaceStyle.Normal);
etc

Wpf has good native support:
System.Windows.FontWeights.Thin;
System.Windows.FontWeights.ExtraBold;
etc

WinRT (UWP) also has good native support:
Windows.UI.Text.FontWeights.Thin;
Windows.UI.Text.FontWeights.ExtraBold;
etc

The C++ API does not seem to support this.
Could you try using the Android-approach?
SKTypeface.FromFamilyName ("sans-serif-thin");

And report back?

I can confirm that the Android approach works with SkiaSharp.

Ok, documented this.

Is there a way to get Open Sans Light working? I've tried all kinds of combinations and none seem to work.
SKTypeface.FromFamilyName("Open Sans")
SKTypeface.FromFamilyName("Open Sans-Light")
SKTypeface.FromFamilyName("Open Sans Light")
SKTypeface.FromFamilyName("OpenSans-Light")
SKTypeface.FromFamilyName("open-sans-light")


fix: resolve issue with date picker properties (resolves #832)

This should resolve type issues with the date picker component. To test:

Log in as a test regulated organization user<EMAIL_ADDRESS>password = 'password'
Create a new project. Ensure that the project date fields don't cause any errors during the initial editing process.
Edit the user's existing project. Try empting year/month/day fields individually and in different combinations and saving.

Date picker still throws error when a string is entered to the year and day. I think loosening type for year and day might be a good approach here.
Error message:
Cannot assign string to property App\Http\Livewire\DatePicker::$year of type ?int

Date picker still throws error when a string is entered to the year and day. I think loosening type for year and day might be a good approach here.
Error message: Cannot assign string to property App\Http\Livewire\DatePicker::$year of type ?int

My apologies, I was testing this on a wrong branch, and the date picker is working fine even with string input.

When I enter "0" under day (with the start month being may and end month being september), I save and it successfully saves - except the start date is turned into april 30 and august 31. I'm thinking this should trigger an error instead.


fix:EnableOverridingThemeColor

This change enables to change theme-color with ReactHelmet that's currently introduced to this theme.
Without this gatsby-plugin-manifest inserts a theme-color meta tag into the site head. This means that you can't use React Helmet to set the theme_color.
References
https://github.com/gatsbyjs/gatsby/issues/9977
https://www.gatsbyjs.com/plugins/gatsby-plugin-manifest/#remove-theme-color-meta-tag

@EmaSuriano

@EmaSuriano hi I am also the one who would like to customize theme color using React Helmet so could you review this pull request please?


Swift 3 Support

Hello - would it be possible to add Swift 3 support?
Cheers,
Andrew

Sure, I'll work on it in the next few days, I have a bit of catching up to do on all the other libs.

Awesome - thanks!
Andrew
On September 15, 2016 at 5:34:19 AM, Andrea Mazzini (notifications@github.com(mailto:notifications@github.com)) wrote:

Sure, I'll work on it in the next few days, I have a bit of catching up to do on all the other libs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub(https://github.com/andreamazz/GearRefreshControl/issues/7#issuecomment-247281684), or mute the thread(https://github.com/notifications/unsubscribe-auth/AJxUH2ikaWxW-vDvU8XpQBP7ejnS71aBks5qqREbgaJpZM4J9AWd).

Done, check out version 1.0.0

Thanks!  Will do :) Appreciate your efforts.
On September 20, 2016 at 4:29:36 AM, Andrea Mazzini (
<EMAIL_ADDRESS>wrote:
Done, check out version 1.0.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andreamazz/GearRefreshControl/issues/7#issuecomment-248237856,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJxUH_3FV6ROKGQZzgPMBYt58Uh0_cEmks5qr5lwgaJpZM4J9AWd
.


Added measurement variable for inkscape import support

Now, in netDxf 2.3.0 you can add your own header variables to the document, see the sample AddHeaderVariable() in the TestDxfDocument project.


Website has incorrect instructions for language

The handbook has the new instructions for language as follows:
Field name	language
Column Contains	Language
Year/s Used 	Wales 1891 – 1911 and Isle of Man 1911 only
What Can Be Entered 	Up to 2 letters or blank
Description and Rules	In many censuses there was a question about language. This occurred in Wales and the Isle of Man. Enter the appropriate letter or letters for the language spoken here. Leave this entry blank if it has not been filled in on the census.
Use a code from the list below:
•	E 	English
•	G 	Gaelic
•	GE 	Gaelic and English
•	I 	Irish
•	IE 	Irish and English
•	M 	Manx
•	ME	Manx and English
•	W 	Welsh
•	WE 	Welsh and English
If you are using the England and Wales spreadsheet for a County that is not in Wales or the Isle of Man leave this column blank.
Note: If the language is enumerated as Both Transcribe the appropriate double entry from the list above. Do not use B as it will produce an Error when tested by CSVProc.
The website still has the old instructions from FC1 as follows:

This is causing confusion. Can the old instructions be taken down please and replaced with a note to refer to the Handbook.
Geoff

@geoffj-FUG I can make the changes you request, but let me check that I understand the situation correctly.
The spreadsheet fields is for volunteers using the VLD system, yes? So, are you saying that transcribers using the VLD system need to follow the same instructions for the Language field as those using CSVProc?
In which case, I suggest that the instructions be changed in situ, rather than referring volunteers to the Handbook (a lengthy, probably unfamiliar document at this stage).
If no-one is using the VLD system any longer, then the whole page would need to be taken down.
I also note that the Language field applies to some Scotland census returns (even though we do not have access to those images for now), not only to Wales and the Isle of Man.


✨ Add core typography tokens

Core typography tokens in Figma. There are currently 4 scales, ranging from tight to relaxed, where the difference is the line-height. Line-heights are calculated based on font-size and rounded up to the nearest 4px to fit with the layout grid.
Known issues: Figma Tokens does not add new line at the end of the file, I will notify the developers about this

Not sure where these tokens best belong

I think putting them in packages/eds-tokens/ makes sense. The tokens can be modified outside of Figma, as long as the format remains the same. The plugin checks if there are changes in the repo when it’s started, and asks the user if s/he wants to pull the latest changes.
I can add a README with an instruction on how to load the tokens in the Figma Tokens plugin as well.

Does it check against the whole repo or can it specify against a given path? 🤔

You can specify path and branch


Purpose of “DIGIT … FULL STOP” Unicode in Display output?

5b210380 landed a number of non-ASCII UTF-8 characters, and I'm not clear on the intended meaning.

U+2488 DIGIT ONE FULL STOP

Includes a limitation of liability.
May be used for commercial purposes.
Source code must be made available when the software is distributed.

U+2489 DIGIT TWO FULL STOP

Changes made to the code must be documented.
Does not grant trademark rights.
May be distributed.

U+248a DIGIT THREE FULL STOP

Does not provide any warranty.
May be modified.
The license and copyright notice must be included with the software.

U+248b DIGIT FOUR FULL STOP

Does not provide any rights in the patents of contributors.
Provides an express grant of patent rights from contributors.
Users who interact with the software via network are given the right to receive a copy of the source code.

U+248c DIGIT FIVE FULL STOP

May be used for private purposes.
Modifications must be released under the same license.

This feels a bit like the attribute grouping on choosealicense.org, but your groups are a bit different.  And I don't understand the need to go beyond ASCII here, because “1.” looks about as nice or nicer than “⒈” unless you happen to be using a font that includes support for these uncommon characters.  Can you give a bit more background on the intended semantics?

Yeah I agree, these should probably just be ASCII.


espi: it8xxx2: enable configuration of Chromebook

This enables the below configuration so the AP and EC are able to
communicate over eSPI:
CONFIG_ESPI_PERIPHERAL_EC_HOST_CMD
CONFIG_ESPI_PERIPHERAL_ACPI_SHM_REGION
CONFIG_ESPI_PERIPHERAL_CUSTOM_OPCODE
Signed-off-by: Dino Li<EMAIL_ADDRESS>

Rebase and fix conflict of linker.ld.

@tejlmand Can you please re-review?

@brockus-zephyr can you please review.

@tejlmand Could you please re-review?

@Dino-Li I believe if you rebase, it should fix the documentation failure.  I believe the doc failure was fixed by PR#41865

@Dino-Li I believe if you rebase, it should fix the documentation failure. I believe the doc failure was fixed by PR#41865

@keith-zephyr Done.


Etcher is still sending some usage/tracking data though disabled

Etcher version: 1.0.0
Operating system and architecture: Win 7, 64bit

I have statistic sending disabled in settings but etcher is still sending some usage/tracking data via HTTP (http://api.mixpanel.com/track/?data=xxx).

@schlamar That's weird, thanks for reporting. Can you paste the logs from DevTools (Ctrl+Alt+I), so we can see at which part the events are being sent?

Looks like every event is sent, e.g. navigating to settings and back.
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 document not ready yet, trying again in 500 milliseconds...
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 Download the React DevTools and use an HTTP server (instead of a file: URL) for a better development experience: https://fb.me/react-devtools
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51  _____ _       _
|  ___| |     | |
| |__ | |_ ___| |__   ___ _ __
|  __|| __/ __| '_ \ / _ \ '__|
| |___| || (__| | | |  __/ |
\____/ \__\___|_| |_|\___|_|

Interested in joining the Etcher team?
Drop us a line at<EMAIL_ADDRESS>C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 Wed May 24 2017 15:29:56 GMT+0200 (Mitteleuropäische Sommerzeit) Application start
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 Wed May 24 2017 15:29:56 GMT+0200 (Mitteleuropäische Sommerzeit) Checking for updates ({"currentVersion":"1.0.0","releaseType":"PRODUCTION","updateSemverRange":"<2.0.0","includeUnstableChannel":false})
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 {"cause":{"code":"UNABLE_TO_VERIFY_LEAF_SIGNATURE"},"isOperational":true,"code":"UNABLE_TO_VERIFY_LEAF_SIGNATURE"}
console.(anonymous function) @ C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 Wed May 24 2017 15:30:18 GMT+0200 (Mitteleuropäische Sommerzeit) Navigate ({"to":"settings","from":"main"})
C:\Apps\Etcher\resources\app.asar\node_modules\raven-js\dist\raven.js:51 Wed May 24 2017 15:30:23 GMT+0200 (Mitteleuropäische Sommerzeit) Navigate ({"to":"main","from":"settings"})

Could you open the DevTools "Application" tab, click file:// under the "Local Storage" section and show me the stringified JSON contents of the "redux" key? For example:

{"unmountOnSuccess":true,"errorReporting":false,"sleepUpdateCheck":false,"lastUpdateNotify":1494444978918,"unsafeMode":false,"downloadPath":"/Users/jviotti/Downloads","ejectOnSuccess":true,"validateWriteOnSuccess":false,"useUnstableUpdateChannel":false}

That {"cause":{"code":"UNABLE_TO_VERIFY_LEAF_SIGNATURE"},"isOperational":true,"code":"UNABLE_TO_VERIFY_LEAF_SIGNATURE"} error is very interesting as well. I'll send a PR for it.

Regarding the UNABLE_TO_VERIFY_LEAF_SIGNATURE error, are you behind a proxy?

Content of "redux":
{"unsafeMode":false,"errorReporting":false,"unmountOnSuccess":true,"validateWriteOnSuccess":true,"sleepUpdateCheck":false,"includeUnstableUpdateChannel":false,"lastUpdateNotify":null}

Regarding the UNABLE_TO_VERIFY_LEAF_SIGNATURE error, are you behind a proxy?

Yes

Thanks for the information. I'm sending a fix for the UNABLE_TO_VERIFY_LEAF_SIGNATURE error, and @Shou can probably help us out getting to the bottom of that ugly analytics issue.

I confirm this issue is being fixed by https://github.com/resin-io/etcher/pull/1604


No longer have access to sprockets instance in template

I see in v4.0.0.rc.1 that this is using the new pattern for Middleman extensions. I've been working to update an app from v3 and realized I'm using something like the following in one of my layouts to see if an image exists before including a default image:
topic_image_path = "some/image/path.jpg"
default_topic_image_path = "some/default/path.jpg"
topic_image = sprockets.find_asset(topic_image_path) ? topic_image_path : default_topic_image_path

However, because the new version of the extension only exposes the sprockets instance to the config file, I'm getting an error. Is adding the following a possibility if I were to do a PR?
expose_to_template sprockets: :environment

Or is there a better way to accomplish this?

Hmmm, yeah I know I have used that in a few places to dump svgs as well. I'll look into it -- for you should be able to do this though:
topic_image = extensions[:sprockets].environment.find_asset(topic_image_path) ? topic_image_path : default_topic_image_path

That code sample, exposing that var to the template sandbox, should work.

Yep, @robertwbradford if you use the gem from master you should be good now:
gem 'middleman-sprockets', github: 'middleman/middleman-sprockets'

Beauty! I'll have to look at this later, but thanks for the change. Middleman has been a great platform to work in, btw.


View and update your profile within a game

On the you screen, you can view your details, which will be specific for a particular game.
You will have your username which is fixed (dummy used for now), but also have your character name, which is randomised to start with, and a bio, which starts empty.
Your character name and bio can be changed by pressing an edit button, which will change the text field to an editable text field, and back when confirmed.
Entering a different game will have a different associated character name and bio.

Can now add and retrieve characters for a specific game and user.
Need to:

Add additional edge cases for adding a new character.
Add updating a character, with edge cases.
Consider removing cross references where one-to-many relationships used, with foreign keys.

Also, UI concerns:

On the you screen, you can view your details, which will be specific for a particular game.
Your character name and bio can be changed by pressing an edit button, which will change the text field to an editable text field, and back when confirmed.
Entering a different game will have a different associated character name and bio.

Next:

UI concerns (starting with ViewModel)

Do last:

Consider removing cross references where one-to-many relationships used, with foreign keys.


riscv64: Add RISC-V support

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #223. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.

How could the RISC-V build be added to TAOS?

How could the RISC-V build be added to TAOS?

We will need Tizen-GBS ready for RISC-V and all the dependency packages ready for RISC-V in Tizen, first.

@MarekPikula If you want this merged, please mark "Ready for review".


Support emitting sourceRoot in generated source map

What problem does this feature solve?
I've searched google and the issues within this repository, I couldn't find anything on this topic. Apologies if I missed an ongoing discussion about this!
In my mono-repo structure (using Nx) I have a project that I am bundling with rspack which is truly blazingly fast :)
I'm running the bundled project with Nx's Node executor @nx/js:node, but this is probably irrelevant.
I am emitting a source map main.js.map alongside my main.js in a distribution folder.
config.devtool = 'source-map nosources';
My IDE (vscode) is not resolving the sources listed in the source map file, because the code that generates the source map uses relative paths in the map file. I think rspack_plugin_devtool is responsible for generating the source map.
If the dev tool plugin supported emitting the "sourceRoot" key in the map file, my IDE will be able to correctly resolve the source maps, and my breakpoints will bind. I've tested this by manually adding the "sourceRoot" key to the emitted main.js.map file. This solves my issue, but is a manual step.
Here's my folder structure:
Workspace root: workspace/
Sources: workspace/apps/my-project/src/*
Bundle output: workspace/dist/apps/my-project/* (holds bundled main.js and main.js.map)
Currently, the map file be emitted with relative paths from the workspace root. So a source file in workspace/apps/my-project/src/my-file.ts gets emitted as ./apps/my-project/src/my-file.ts in the map. This causes my IDE to resolve the path relative to the map file's directory, in my case workspace/dist/apps/my-project/main.js.map. This means the IDE searches for the source at workspace/dist/apps/my-project/apps/my-project/src/my-file.ts which is incorrect.
What does the proposed API of configuration look like?
Webpack's SourceMapDevToolPlugin has an option to explicitly set the sourceRoot that will be emitted in the file.
https://webpack.js.org/plugins/source-map-dev-tool-plugin/#options. I'm not sure how this directly translates to rspack's dev tool configuration. It seems rspack's dev tool configuration is pretty limited from what I can gather, where the only configuration being config.devtool, which is just a string of flags.
I see that the Devtool struct only has an inner member for the flags (parsed from config.devtool).
If the sourceRoot configuration should be adjacent to the current config.devtool config, I feel that the config.devtool would have to be expanded from a string, possibly to an object like {flags?: string, sourceRoot?: string}
However, automatically calculating the sourceRoot may be a nice place to start. The sourceRoot would have to be whatever the "context directory" or cwd is when the relative paths in the sources list is being generated.

we need to support SourceMapDevToolPlugin first to provide this functionality. cc @ahabhgk

I have also run into this issue. The best I could find was changing the config.context to the directory of my entry point then all my modules imported from my libs would work, but the source map for the app would still be broken.


Detection of running Docker containers doesn't work as expected

Detection of running Docker containers reports that a container is running if it exists, rather than if it is actually running.  This needs to be fixed.

Fixed in #29


WMS ID 11390 - added more steps

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

@ramumg this branch is out-of-date, please merge this to the latest updates.


CircleCI testing with test collection [testing]

Testing here at the moment, until working and if landed
https://circleci.com/gh/yschimke/okhttp/1

Appears to be hanging in mockwebserver tests :(

With JDK 11 - it progresses but fails on 2 tests
https://circleci.com/gh/yschimke/okhttp/3#tests/containers/0
But great demo of the test results!

Working now with submodules
https://circleci.com/gh/yschimke/okhttp/4


Chapter 0 Persian Translation First Draft

Added chapter 0 and updated _toctree.yml

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@lewtun I am having trouble viewing the preview live version at https://moon-ci-docs.huggingface.co/course/course/pr_95, which is the address that gets mailed to me. It auto redirects back to https://moon-ci-docs.huggingface.co/course/pr_34/en/chapter1/1 and when I manually change the URL to something like https://moon-ci-docs.huggingface.co/course/pr_95/fa/chapter2/1, the request just stalls. If this is the same bug you told me about in the last PR, please ignore. I want this mainly to see how it renders with full style, since my local builds throw a sed warning and seem to lack the necessary stylesheets. I will try to see if I can fix that locally later.

Hi @jowharshamshiri yes this is a current limitation in the way we render the website - basically the fa/chapter1/1.mdx file needs to exist.
My suggestion would be to see if @schoobani is still working on that section - if not, you could add it as part of this PR and then we can really see the build :)

yep. I am on it. I will let you know very soon. @jowharshamshiri

Hi @schoobani, please review.

Hi @lewtun, please merge.

Thanks for adding this and apologies for the delay - merging now!

@jowharshamshiri [Sorry to comment on a closed PR!]
You are using  tags for the subheaders of the text, but instead you should use #.
this causes the right menu not to appear correctly. please fix it in the next PR. thanks.

@jowharshamshiri [Sorry to comment on a closed PR!] You are using tags for the subheaders of the text, but instead you should use #. this causes the right menu not to appear correctly. please fix it in the next PR. thanks.

Hey just saw this, I am doing it bc in the current version of the Hugging Face docs, we need this for the page to render correctly in the preview. It will be LTR if we don't do it.


Fix #234: redefinition of \FA, upgrade to fontawesome5

My previous PR #235 was closed as 'invalid duplicate bug', which I'm not quite sure what means, and this seems to remain unfixed to a lot of people including me.
I tried master again, but still had the issue; rebased my changes, and it still fixes it. (Tested Linux & macOS.)
If this fix is 'invalid' for some reason, or it should work as-is I'm just doing something wrong, please, let me know - I'd love to be using awesome-cv properly so that I don't have to maintain my fork, or if a fix is needed just not this one, I can try to improve it.
Thanks.

I wish I had checked PRs before spending hours trying to figure out why I couldn't compile the PDFs. Anyway, I approached the same issue you're talking about in a different way so it's definitely a problem that exists and has not been addressed.
It may be useful for you to merge my approach with yours so that Github Actions auto generates PDFs on each new push, saving you the trouble of having to have the TeX environment set up locally.

@vpatel9202 I'm not sure that relates to this issue, what is there to render from awesome-cv itself?
Funnily enough though my own CV is also GitHub Action, and formerly Circle CI, rendered/released. I have an auto-updating texlive image at OJFord/docker-texlive for the purpose too, in case that's of any help.
I don't think this PR needs anything to do with how it's rendered though, this is just fixing an error that occurs however (so far as I can tell) the rendering is done.

The way I solved the font issue was to install the font package into the docker container that does the rendering (which, admittedly seems like a roundabout way to do it comparing to how you remedied the situation). So we're having the same issue with TeX throwing errors when it can't finding FontAwesome, but we solved it in different ways. I think your approach is more direct and solves problem with Awesome-CV itself while I was utilizing a docker to get around the problem.

Should the symlink to fontawesome.sty in examples/ removed as well? I can't see what purpose it's serving after this merge but it's possible I'm missing something.

Quite possibly @dcsim, I'll check later (writing from phone). Thanks for all your help in issues recently by the way!

Oh yes, for sure. Thanks!


Feat default response paths

Add special paths that will be called an case of an error, like 404, 500, etc

also need to add custom error handler option

I'll leave custom error path for later
it doesn't make as much sense as with 404 pages and is much harder to code without producing spaghetti


keep old date when using datetimepicker for just picking time

I'm using the picker just for time picking (format: 'HH:mm')
The problem is: when I change the time the date is changed too and set to today but I want it to stay at the old date.
Example:
Orig. date is: 2017-08-01 11:00:00
I want to change from 11:00 to 12:00
But the date is also updated to the current day which results in 2017-08-07 12:00:00
How can I prevent datetimepicker to update the date?

Hi, can you provide a fiddle showing your case you can use whatever you prefer JSFiddle or a JSbin

Thanks for the answer, the problem seems to be somewhere else.
I use the picker in combination with knockout and found the problem there
Sorry!


How can I handle durable.engine.MessageNotHandledException properly?

from durable.lang import *

def risk7_callback(c):
    print('risk7 fraud detected')

dynamic_ruleset = { 'risk7': {
    'suspect': {
        'run': risk7_callback,
        'all': [
            {'m': {'$gt': {'succ': 30}}}
        ],
    }
}}

get_host().set_rulesets(dynamic_ruleset)

post('risk7', {'succ': 31, 'fail': 11})
post('risk7', {'succ': 30, 'fail': 11})

Hi, I try your module for checking condition with dynamic rule load from json.
I planned rule file & data from redis and insert it to rule engine and send alert to any other DBs or API.
But when I try above code, at last line, I met below Exception.
durable.engine.MessageNotHandledException: {'succ': 30, 'fail': 11}
I think this will be your intended work but don't know how can I handle that properly.
When met last line I hope nothing happen or call some other callback function.
Could you guide it?

Hi, thanks for asking the question. MessageNotHandledException is raised when the object posted doesn't satisfy any rules. In that case, the rules engine does not keep a copy of it. You can can handle the exception and ignore:
try:
    post('risk7', {'succ': 31, 'fail': 11})
    post('risk7', {'succ': 30, 'fail': 11})
except MessageNotHandledException as error:
    pass


Unable to export patched game

It would be nice to be able to export the game so it doesn't have to be run in debug mode with the FPS counter on. It appears to crash when I try to do that at the current time.
The executale created is just nw and a lot of the metadata in the project seem to have default values instead of OMORI. This made me wonder if some assets were missing, but when I copy folders from my windows OMORI install it just crashes. This made me think perhaps there is encryption stuff going on.
this is the error running the exported game:

I must say it is really awesome that you made this. regardless of if it has a couple issues it's a great starting point

It seems notes.pluto is replaced by notes.yaml after being decrypted. I'll let you know if I find a way to fix this :D


Use CUID2 instead of CUID?

CUID is officially deprecated
https://github.com/paralleldrive/cuid

How about NanoId?
This post has a nice comparison: https://dev.to/harshhhdev/uuidguid-cuid-nanoid-whats-the-difference-5dj1

There is cuid2 BTW.

I'm not using paralleldrive/cuid, I'm using paralleldrive/cuid2 :)
You almost gave me a heart attack 😆 Thanks anyway!

As for Nanold, our use cases are pretty simple so cuid2 is good for us :)

Sorry just decision docs talk about CUID and not the CUID2, so I guessed that CUID 1 was used.