Basic argument parsing

Generally when writing scripts, you don't need complex argument parsing and want something that can easily be figured out by looking at the code. You also don't need help text. For that reason, I think an approach like the following might be more appropriate than something more featureful like deno_std's argument parser.
$.args.on("prepare", () => {
  console.log("Preparing...");
});

$.args.on("build", () => {
  console.log("Building...");
});

> deno run -A script.ts prepare
Preparing...
> deno run -A script.ts build
Building
> deno run -A script.ts other
Unknown command: other

Possibilities:
  - prepare
  - build

More complex example:
const args = $.args(); // shorthand for `const args = $.args(Deno.args);`

await args.on("build", async (args) => {
  await args.on("data", (args) => {
    console.log(1);
    if (args.hasFlag("force") || args.hasFlag("f") || hasChanged()) {
     await buildData();
     console.log(2);
    }
  });

  // can be sync or async
  args.on("info", (args) => {
    console.log(3);
  });
});

args.on("version", (args) => {
  console.log(4);
});

// no match here causes a non-zero exit code that shows all the found options

> deno run -A script.ts build data
1
2
> deno run -A script.ts build data
1
> deno run -A script.ts build data --force
1
2
> deno run -A script.ts build data -f
1
2
> deno run -A script.ts build info
3
> deno run -A script.ts build other
Unknown command: build other

Possibilities:
  - build data
  - build info
> deno run -A script.ts version
4
> deno run -A script.ts invalid
Unknown command: invalid

Possibilities:
  - build
  - version

People could call a function on args to prevent the "unknown command" error (maybe args.markHandled() or something).
$.args.hasFlag("force") for --force and $.args.hasFlag("f") for -f
$.args.get("time"): Arg | undefined -- always requires an equals sign
$.args.getOrThrow("time"): Arg | undefined

$.args.getOrThrow("time").number()

$.args.throwUncapturedFlags() - Throw for any flags that haven't been captured (maybe)

People could call a function on args to prevent the "unknown command" error (maybe args.markHandled() or something).

One idea I've been toying with was "immediate mode" argument parser, with the API along the lines of
const force = args.bool("-f/--force", "ignore errors")
const path = args.string("--path", "path to operate on", { required: true })
args.done()

The trick would be that required --path does not return the error immediately -- it returns "" (or maybe even undefined (just blatantly lying to the type system)). The actual error is raised in .done(). The benefit here is that .done() knows the full grammar for the CLI, so it actually can emit a help message.
From the other angle, clap-style derives are actually inconvenient for quick scripts, as they force you to declare a struct type for the CLI args, while, in the script, you often need only local variables.
And I think low-effort option to add help text (or just to print the options themselves, without any human-authored help) is pretty valueable -- a common use-case are internal tools for a team of software engineers, where you don't necessary want to polish everything to be fit for the end-user, but you also don't want to bug your collegue-author in slack to learn how to use the tool.

Not going to lie, but this feels outside the scope of dax. cliffy has a really good interface that is really easy to understand and create good CLI interfaces.

I think it's in scope. Dax is meant to be a swiss army knife for automation scripts and being able to understand inputs to that script is important part of that. Both cliffy and deno_std don't really provide the kind of API I'm looking for in this case.

Also agree out of scope. There is no necessity to couple such functionality, as it can just be composed. You could write your desired functionality as your own module.
If you want something existing, I wrote this arg parser https://github.com/bevry/argument based on the patterns from a decade of writing hundreds of commands in Dorothy https://github.com/bevry/dorothy - which I'm thinking of using dax for as a replacement for bash

Yeah, I'm going to close this for now (mainly because I don't have the bandwidth atm)


samples: export data to BigQuery

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #74 ☕

I suspect that the sample failed because of an issue in the Insights client library. I filed https://github.com/googleapis/java-contact-center-insights/issues/93


Issue about creating

Description
When I try to create that streak stats it doesn't shows it, and I think it is because of link because my old code that I wrote before is also giving an "error" showing just broken image icon and the name "Github Streak".

Same problem for me

This problem is associated with some specific themes, I suppose.

When there is a lot of traffic to the Heroku server, the GitHub API seems to stop sending back content and the responses become very slow.
To avoid this issue, I recommend self-hosting the project. There are steps in the readme, I recently added a new method for deploying with Vercel entirely online. It's free and shouldn't take more than a couple minutes to set up.
It should also start working on its own, but there isn't much I can do to prevent it from going down in the future, so self-hosting is still an option to get it to stay up more reliably.


feat(query-core): Add previousQuery to placeholderFn

This PR adds another parameter to the placeholderFn called previousQuery to useQuery. This parameter can be used to inspect the query which contained the last defined data for that observer.
Pending:

Tests for the new parameter

Codecov Report
Patch coverage: 100.00% and project coverage change: +15.20 :tada:

Comparison is base (a9d7a37) 74.93% compared to head (482dd16) 90.13%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##            alpha    #5358       +/-   ##
===========================================
+ Coverage   74.93%   90.13%   +15.20%
===========================================
  Files          23       76       +53
  Lines         363     2676     +2313
  Branches      107      740      +633
===========================================
+ Hits          272     2412     +2140
- Misses         70      232      +162
- Partials       21       32       +11

Impacted Files
Coverage Δ

packages/query-core/src/queryObserver.ts
86.12% <100.00%> (ø)

... and 70 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


IRIS 2023.1.3 Illegal Regular SQL identifier 'Module'

Error on importing/compiling zpm-0.7.0.xml stating that 'Module' is a SQL reserved word.

I believe this error is related to my customer's disabling of "support delimited identifiers"

This is a duplicate of #440 (maybe actually the same root case, given that the two were reported the same day?). There may be a number of side effects related to this, but it boils down to "IPM should work with 'support delimited identifiers' disabled"


longer short flex

I would like to use the AIY vision setup in conjunction with another peripheral attached in between the Vision Bonnet and the RPi. Sadly the short flex is a wee bit short to accommodate this. Are there somewhat longer versions of this flex available anywhere?

Any of these 22POS 0.5MM pitch FFC cables should work:
https://www.digikey.com/products/en/cable-assemblies/flat-flex-ribbon-jumper-cables/457?k=FFC&k=&pkeyword=FFC&pv1989=0&FV=1640055%2Cii22|2166%2Cffe001c9%2C40800d1&quantity=0&ColumnSort=0&page=1&pageSize=25

@dmitriykovalev Hi I am trying to use the Pi 3 B with the vision bonnet. Can you recommend a flex cable between the Pi 3 B and bonnet that has a short enough connector that it would fit? The USB ports are too close to the bonnet for the standard cables.
I even clipped the peeled hard black piece in half but it still seems too squished, may have shorted the board because it wouldn't even power on.
Thanks for any suggestions!


Unit tests fail on Ubuntu 17.10

The unittests fail when run on Ubuntu 17.10 / GCC 7.2 / binutils 2.29.1:
./preconfig
./configure CFLAGS=-O1
make
./coredumper_unittest
  ...
coredumper_unittest: ../src/coredumper_unittest.c:396: CheckExtraNotesWithReadElf: Assertion `note_index == kExtraNotesCount' failed.

The problem is that the readelf on Ubuntu 17.10 / binutils 2.29.1 is buggy: the output of readelf -a is missing a newline after printing the description data.  If you run readelf -a on the core-test file you'll see the incorrect output:
  GOOGLE               0x00000010       Unknown note type: (0xdeadbeef)
   description data: 61 62 63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 70   FOOO                 0x00000009  Unknown note type: (0x00f00ba7)
   description data: 71 77 65 72 74 79 75 69 6f   BAR                  0x00000000       Unknown note type: (0x00000000)
  LARGE                0x00000400       Unknown note type: (0x00001234)

Note that "FOOO" and "BAR" are printed on the same line as the description data.  It should look like this:
  GOOGLE               0x00000010       Unknown note type: (0xdeadbeef)
   description data: 61 62 63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 70
  FOOO                 0x00000009       Unknown note type: (0x00f00ba7)
   description data: 71 77 65 72 74 79 75 69 6f
  BAR                  0x00000000       Unknown note type: (0x00000000)
  LARGE                0x00000400       Unknown note type: (0x00001234)

I tried a readelf from binutils 2.30 built myself from upstream and it gave the correct output and the coredumper unit test worked properly.
I don't know if this issue exists in upstream binutils or if it's an artifact of Ubuntu's version.


VBCorTests project failures

I'm submitting a bug report
Library Version: git master@e12acb5
Please tell us about your environment:

Operating System: Windows 10 & VB6 IDE
Installed https://github.com/kellyethridge/SimplyVBUnit/releases/tag/v5.0 from .exe

Current behavior:

When I build the project group, or also just the VBCorTests project, after successfully building
the Source\CorLib\VBCor3.dll and the Source\Fakes\Fakes.dll, I get a compilation error in
trying to build the VBCoreTests project:

Compile error:
Variable uses an Automation type not supported in Visual Basic
And it highlights the phrase GuidToUse As Guid on line 120 of GuidTests.cls.

When I try the "Run with full compile" command on the VBCorTests project, I get a different error:

Object library not registered

When I try to just "Run" the VBCorTests project, I get yet a different error:

Compile error:
Method or data member not found

Expected/desired behavior:
I want to be able to run the test suite to validate that I have a good compilation of the current git master dll.
Am I missing a dependency somewhere? Do I need to register the .dlls that I've built?

FWIW, I just noticed that l the IDE did update my .vbp files so that they no longer had relative path references, and they were replaced with relative paths to the dll files.

Bah, I just realized that I hadn't registered the TypeLib\CorType.tlb and VBVM6Lib.tlb. After doing that from an elevated shell, I get:

And it drops me into GuidTests.cls at line 86, in the "Equals_WithNonGuidValues_ReturnsFalse" test case.

@kellyethridge Any ideas on why I might not be able to run your unit test suite? I'm still blocked on the most recently reported problem.

HI,
Sorry about the delay. I don't get much time for this.
Anyways, to the Guid issue at the top of this thread. It's most likely because the VBCorLib Guid is in a naming conflict and the reference to VBCorLib needs to be moved up in priority in references.
I'm not sure was the Error 91 is coming from, I'll have to look more in to that. I did however find an issue with SimplyVBUnit that will cause tests to fail with a compiled VBCorLib that will be addressed.

For whatever reason, after installing the latest SimplyVbUnit v5.0.1, I can now run the test runner successfully... And I get 11 of failing tests. I assume this isn't expected, and I'd love to report them, but I'm not sure how I'd get the results out of the test runner app...
FWIW, most of the failures are directory-related and seem to be triggered by my project living under my e:\.

I'm glad the tests are actually running now. As for capturing the output of the tests I've been using a logger class. I've included it as an attachment. You will need to change the extension back to cls. I added it in the From_Initialize event of the test runner.
Private Sub Form_Initialize()
    Me.UIRunner1.Init App
    Me.UIRunner1.AddListener New OutputLogger
End Sub

This will generate a log file of errors and failing tests.
OutputLogger.txt

I, too, am having failed tests but I think I understand the nature of the problem - I think the root of all of them is this:
CorLib.Tests.System.TimeZoneTests.Constructor_WhenCalled_ReturnsExpected: Time zone 'US Mountain Standard Time' is not supported by tests.
I'm in Phoenix and am definitely using US Mountain Standard Time as my timezone (we're in the small minority that ignores DST changes)

@kellyethridge I sent you a pull request for US MST issues.  Only one left:
EnvironmentTests.GetCommandLineArgs_WhenCalled_ReturnsArgs failed: Expected: 2 but Was: 1
It might have more to do with the way I'm running the tests.  I'm loading up the CorLib.Solution.vbg in the Source directory.  Is that the problem?  Is there a better way to load up so as to avoid this issue?

Found the reason for my above error. Opening VB6 first, and then browsing and opening CorLib.Solution.vbg means that the command line arguments only contain one element: the VB6.exe.  However, if one were to close VB6 and double-click CorLibSolution.vbg instead, the test passes.  PR submitted.


Export datavec schema to Spark SQL schema

Issue Description
Having the ability to export a datavec schema to a spark-sql schema - and vice-versa - would be very valuable for data exploration and munging cases. For example, when investigating class imbalance you could limit a dataset for specific columns, or only query records that meet a specific criteria.
This would also greatly assist complex ETL cases where CSVs (and other formats) may not be in a desirable state.
Version Information
master
Contributing
@AlexDBlack thoughts here? I could greatly use this, having some guidance I can get on this.

An alternative here is to instead adapt a SQL parser and utilize our own functionality - which may be more ideal for binary (and other) data formats as per @huitseeker.
Suggestions include:

https://github.com/JSQLParser/JSqlParser
https://github.com/brunoribeiro/sql-parser

https://github.com/deeplearning4j/deeplearning4j/issues/5204


Analyzer fails to report error for import prefix hidden by class member (when used to name a type)

Consider the following code:
import 'dart:core';
import 'dart:core' as core;

class A {
  core.List get core => null; // (1)
}

Based on my reading of the spec, there should be a compile-time error at (1), because the declaration of the getter core inside of class A hides the import prefix declared at top level. So the type core.List should be invalid.
Oddly, the analyzer accepts this code without complaint, but it considers the return type of the getter to be the type dynamic.
Note that the front end's handling of this case isn't correct either--see #34498.

This should indeed be a compile-time error.

Test case here: https://dart-review.googlesource.com/c/sdk/+/75280

Note: a similar issue is #33005 (in that case, a type parameter fails to shadow the library prefix; in this one, a getter fails to shadow the library prefix).

Analyzer now reports:

error • The prefix 'core' can't be used here because it is shadowed by a local declaration. • a.dart:5:3 • prefix_shadowed_by_local_declaration

Analyzer now reports:

error • The prefix 'core' can't be used here because it is shadowed by a local declaration. • a.dart:5:3 • prefix_shadowed_by_local_declaration


[zh-cn] Security Checklist document is missing

This is a Feature Request

What would you like to be added

add and translate Chinese document:
content/zh-cn/docs/concepts/security/security-checklist.md
English:
https://kubernetes.io/docs/concepts/security/security-checklist
Why is this needed
Support for more localization.
Comments

/language zh
/assign

/triage accepted


Aragon?

This is really cool! Love the idea of using passive income to incentivize and reward a community. Do you guys happen to have a rough idea of how this might work as an Aragon DAO?

Thanks @burrrata - the Aragon DAO can deposit funds into the virtue DAO through a proposal. Basically a virtue DAO allows people to self organize around their common goal from continuous funding or chunks of funding. Happy to chat more about it.

If an Aragon DAO can deposit funds into the VirtueDAO, then really that just means that any address can deposit funds into a VirtueDAO right? In that case, the VirtueDAO is more like a sidecar/app that any Ethereum entity can interact with than something that directly integrates. Is that correct?

If an Aragon DAO can deposit funds into the VirtueDAO, then really that just means that any address can deposit funds into a VirtueDAO right?

Yup, any address can deposit funds into a virtue DAO - e.g. Aragon, DAOStack, Moloch, individual accounts.

In that case, the VirtueDAO is more like a sidecar/app that any Ethereum entity can interact with rather than something that directly integrates.

Yes, that's one use case. There could be a deeper integration. The sidecar (nested DAO) approach is the simplest one.
Some alternate non-sidecar configurations would be:

Replace the proposal module in an Aragon DAO with a VirtuDAO
Investor/product purchase/grant $ -> VirtueDAO (no parent DAO)
rDAI interest -> VirtueDAO (no parent DAO)


When defining onSelecting, the same slot can't be dragged over using a mouse or long pressed using touch events multiple times in a row.

Do you want to request a feature or report a bug?
report a bug
What's the current behavior?
When I use the onSelecting, selectable, and onSelectSlot props together with each other, I can't do a drag selection of just 1 slot or a long press of 1 slot on a touch device multiple times back-to-back and have the onSelectSlot callback fire.

https://codesandbox.io/s/719rxl98m6
Browser: Chrome 66.0.3359.181
react-big-calendar @v0.19.1
What's the expected behavior?
When I use the onSelecting, selectable, and onSelectSlot props together with each other, I should be able to do a drag selection of just 1 slot or a long press of 1 slot on a touch device multiple times back-to-back and have the onSelectSlot callback fire.

@brenthosie Are you saying you want to be to able to select and drag going from left to right instead of just top to bottom?

@arecvlohe no, the problem is that once you start a drag and only select 1 slot, then the callback of onSelectSlot stops firing reliably.

Ah I see. @jquense will probably have more insight into this than I do.

@arecvlohe any word on this?


Migrate to Null Safety

@duyduong please look into this PR

Thanks guys!
I am still in busy mode that don't really have time for maintaining. But I will be back soon to make a good update on this plugin, especially supporting better way to custom the ad content


Large depth value at forehead position

Other than that, the depth maps look fine.

Hi, could you please provide some visualization cases? We can analyze them and further improve our model :)


AssertionError: Wrong input formats

Traceback (most recent call last):
File "/home/zippo/Video-CCAM/demo.py", line 23, in
response = mllm.generate(texts=[question], videos=[frames])[0]
File "/home/zippo/Video-CCAM/model/videoccam.py", line 297, in generate
assert len(et) == 2, f'Wrong input formats for {t}'
AssertionError: Wrong input formats for hello

No matter how I change the question, I will get this error.

@JaronTHU I encounter this problem too. It seems that the chat prompt is not consistent with DEFAULT_IMAGE_TOKEN and DEFAULT_VIDEO_TOKEN. How should I fix this error?

Sorry for not replying in time. You encounter the problem in sample reference code, right? As @zd11024 says, prepending DEFAULT_VIDEO_TOKEN before the question can fix this problem.


chore(main): release 0.6.2

:robot: I have created a release beep boop
0.6.2 (2023-12-16)
Bug Fixes

Bump marked from 11.0.0 to 11.0.1 (#279) (e179cfb)
Bump marked from 11.0.1 to 11.1.0 (#286) (1e1479d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/nzdjb/static-site-generator/releases/tag/v0.6.2 :sunflower:


Rotating 2D object in animation causes rotation to tare out

Godot version
4.1.0
System information
Linux 64-bit (Garuda/Arch)
Issue description
When animating in 2D, I rotated an object and keyed it, but then the editor reset the rotation angle to 0, but kept the actual rotation. This is not only throwing off the rotation, but the positioning, too.

Steps to reproduce
I haven't been able to reproduce it, but I also haven't been able to undo it.
What hasn't worked:

Turning it off and back on again
Right-click the object
Right-click drag the object
Right-click the rotation
Deleting and re-creating the same object (in this case, CollisionShape2D)
Disabling
Enabling One-Way Collision

Minimal reproduction project
BugReport.zip
I kept the spiders just because the malfunctioning piece in question is a hitbox. One hit should kill them.

No bug in here, that's a user error. The parent of the mentioned CollisionShape2D is rotated, hence the CollisionShape2D remains rotated if its own rotation is zero etc.


Polygon implementation

I have implemented the Polygon usage, to verify if a point is contained in it or not used in geofences, also the calculation of the polygon area and perimeter.
Please review the code and accepts this contribution =)

Hey Paul,
many thanks for your work! I merged the PR locally and I'm writing some unit tests and adjusting the coding style for PSR-2 now.
When I'm done, a new version 0.3 of phpgeo will be released.
Marcus

I'm glad to contribute with your work, has been very usefull for us. I'll be aware of the new release.
Greetings Marcus.

Hey @paulvl
the Polygon implementation is now merged and released.
There was a bug in the getSegments() method: the last segment between the last and the first point was missing. I fixed it and added some unit tests.
The area calculation gives us wrong results currently as the calculations aren't done on the ellipsoid but on an flat surface. The proper area calculation is a quite complex beast (you may check out implementation details here).
Again, many thanks for your help!
Marcus

@mjaschen I was looking the GeographicLib solution for area calculations, seems interesting but as you said, is a quite complex beast. Hope I can implement the correct area calculation soon.
Thanks for merging.


🐛 Bricked 'Account Settings | Save Settings' Button – Tina Cloud UI

Describe the issue that you're seeing. Any Loom videos or screenshots usually help a lot!
I am going to this link https://app.tina.io/account/settings, making changes and when clicking "Save Settings" there's no response.
After refreshing the page, the new settings I entered aren't persisted either. I've reproduced this issue on both Firefox and Safari.
Cheers –
https://github.com/user-attachments/assets/8a0c41d2-01e9-48f7-94b2-a7038e8b5fe2
Reproduction
N/A
Steps to reproduce
Go to https://app.tina.io/account/settings.
Change setting fields.
Click 'Save Settings'.
Refresh the page.
⚠ The changed fields have returned to original values.
System Info
System:
OS: macOS 13.6.1
CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Memory: 40.41 MB / 8.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 20.16.0 - ~/.nvm/versions/node/v20.16.0/bin/node
Yarn: 1.22.22 - /usr/local/bin/yarn
npm: 10.8.1 - ~/.nvm/versions/node/v20.16.0/bin/npm
pnpm: 9.6.0 - /usr/local/bin/pnpm
Browsers:
Safari: 16.6
Validations

[x] Follow our Code of Conduct
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug.

Moving this one to TinaCloud


Google Drive File Stream

Google Drive File Stream installs fail with a Team ID mismatch.
`./Installomator.sh googledrivefilestream

no blocking processes defined, using Google Drive File Stream as default

Changing directory to /var/folders/bs/d7hl_c0d5fs8gr19vgy1bj1h0000gn/T/tmp.6NAWwcQ8

could not find Google Drive File Stream.app

no more blocking processes, continue with update

Downloading https://dl.google.com/drive-file-stream/GoogleDriveFileStream.dmg to Google Drive File Stream.dmg

Mounting /var/folders/bs/d7hl_c0d5fs8gr19vgy1bj1h0000gn/T/tmp.6NAWwcQ8/Google Drive File Stream.dmg

Mounted: /Volumes/Install Google Drive File Stream

Verifying: /Volumes/Install Google Drive File Stream/”GoogleDriveFileStream.pkg”

Team ID:  (expected: EQHXZ8M8AV )

Team IDs do not match!

Deleting /var/folders/bs/d7hl_c0d5fs8gr19vgy1bj1h0000gn/T/tmp.6NAWwcQ8

Unmounting /Volumes/Install Google Drive File Stream

"disk2" ejected.`

fixed in dev branch 1b999a11018cbbc91b9abc6bad8168b40b17ac4e


Debuginfo for release profile

Is there some reason to have debug=true for release profile?

line numbers in backtraces for the default config that most users will use. That helps debugging and bug reports.

There's not much info in the commit message that added it. Agree with @bluss that it's helpful for bug reports.


Statistics

@BenKaufmann this pull request adds support for user statistics to the clingo APIs. What I do not like about it (on the clingo-side) is that currently there is a function in the Control class to register a callback to update statistics. At the moment we already have an event handler that can be passed to the solve() function. To not end up with too many functions to register callbacks, I would prefer to add an update_statistics event there. Is it guaranteed from the clasp-side that statistics callbacks are fired only during solving?

@rkaminsk
Stats callbacks registered added via ClaspFacade::addStatisticsCallback() are called exactly once at the end of each solving step. So yes, they are only fired during solving.

Thanks, it should be easy then to put it in clingo's event handler.


2465 fix issues related to ellipsis logic for recordedit inputs foreign key boolean file iframe

@jrchudy This PR is ready to be merged. But before merging it I think you should also take a look at the changes as this modifies multiple components.
For testing the recordedit changes, I added a test-table to the defaults schema on ermrestdatautils (This is the catalog that I created which you might have access to). And for the rest of the changes (facet and related section) I used staging.atlas-d2k.org


[WIP] CMake refactoring

This PR is similar to the arrayfire CMake refactoring. It currently builds the library and all the examples. There still needs to be plenty of work that needs to be done to properly handle dependencies and other build examples.

This is being taken care of in https://github.com/arrayfire/forge/pull/150


Add NDArray histogram functionality

Histogram functionality similar to numpy would be useful:
http://docs.scipy.org/doc/numpy/reference/generated/numpy.histogram.html
Though we probably don't need all of the options theere... for now options for N equal sized bins and binning based on the specified bin starts/ends would be useful.

Actually, maybe we just move this to ND4J, and extend it to allow for specified bin starts/ends:
https://github.com/deeplearning4j/deeplearning4j/blob/master/deeplearning4j-ui-parent/deeplearning4j-ui/src/main/java/org/deeplearning4j/ui/weights/HistogramBin.java

This was added some time ago: https://github.com/deeplearning4j/nd4j/blob/ff268e4c92e263f9e821def3cdca4881227faaf4/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ops/impl/transforms/Histogram.java


added toggle theme

#19 Adding a Dark-Mode
This PR does the following job :-

Toggle button in the Navbar
changes the theme of the website
will retain the theme even if refreshed [usage of LOCAL-STORAGE]
No major changes in the UI of the website.

PS. Please judge this PR based on the functionality and not by its UI . I have completed the task to introduce the feature of toggling theme and have used LOCAL-Storage .

@AADESHak007
Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor.
Great contribution to HACKTOBERFEST'24! 🎉


Fix importing dask_ml.cluster with scikit-learn = 0.24.0

Fixes #768

Thanks, and sorry for the delay.
I'll look into the other CI failures another time.

Thanks, and sorry for the delay.
I'll look into the other CI failures another time.


private-cloud

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: private-cloud
Document link: _index.md
My Issue/Suggestion
When we license, we sell environments:
3 Request Both Operator and Runtime license  -> but in our license page we call it runtime licenses.

@pietervercammen Thank you for reaching out. Can you link the license page that you mean, to make sure I'm looking at the right thing?

https://docs.mendix.com/developerportal/deploy/private-cloud/#licensing
--
Pieter Vercammen
Solutions Architect
+32 (0)498 616513
mendix.com
[signature_1809295879]http://www.mendix.com/
From: katarzyna-koltun-mx @.>
Date: Friday, 13 October 2023 at 18:33
To: mendix/docs @.>
Cc: Pieter Vercammen @.>, Mention @.>
Subject: Re: [mendix/docs] private-cloud (Issue #6648)
@pietervercammenhttps://github.com/pietervercammen Thank you for reaching out. Can you link the license page that you mean, to make sure I'm looking at the right thing?
—
Reply to this email directly, view it on GitHubhttps://github.com/mendix/docs/issues/6648#issuecomment-1761795526, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZUMI4FARZGZRRRP76SKL3X7FUOJAVCNFSM6AAAAAA57DQUNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRRG44TKNJSGY.
You are receiving this because you were mentioned.Message ID: @.***>

@pietervercammen I was actually referring to this part: "but in our license page we call it runtime licenses." <-- what is the "license page" that you mention here? If it's a different document, I'd like to take a look and compare it to the one that you linked.


Add admission webhook and configuration for it

The admission webhook is watching secrets with specific labels
"createdForResourceType=providers" to be created or updated.
Once the secret is created/updated the webhook triggers a call for the controller
with a said secret in the request. Inside the controller, we start the
validation and connection test.
Related to:

https://github.com/konveyor/forklift-controller/pull/460
https://github.com/konveyor/forklift-ui/pull/982

Replaced by https://github.com/kubev2v/forklift/pull/54


{Compute} az vm create: Update help message for --public-ip-address

Submitting request here as suggested in below pull request:
https://github.com/Azure/azure-cli/pull/26350#pullrequestreview-1440820818
Related command
--public-ip-address
Name of the public IP address when creating one (default) or referencing an existing one. Can also reference an existing public IP by ID or specify "" for None ('""' in Azure CLI using PowerShell or --% operator).
Description
We have tested with different versions of OS and PowerShell core version 7.3.4 to test the public-ip-address parameter behavior and below is our findings when we have to create VM without public IP:
For PowerShell core version 7.2.* & below === --public-ip-address '""'
For PowerShell core version 7.3.4 ===== --public-ip-address ‘’ or --public-ip-address “” ---------------------------------------- Here we just have to use single field i.e., either single inverted comma (‘’) or just double inverted comma (“”).
Since, cloud shell also uses PowerShell core edition so here also we saw the same behavior in past. This not specific to any OS version as such, we just need to do slight modification in command while using PowerShell core version 7.3.4.
Testing Guide
az vm create --name newvm --resource-group testazcli --image Win2022Datacenter --admin-username sinhamanisha --admin-password Manishasinha*10 --subnet /subscriptions/6b26da86-6cfd-4198-afde-fb754b11c54e/resourceGroups/testazcli/providers/Microsoft.Network/virtualNetworks/testvnet/subnets/testsubnet --location southindia --size Standard_E32s_v3 --public-ip-address ''
OR
az vm create --name newvm --resource-group testazcli --image Win2022Datacenter --admin-username sinhamanisha --admin-password Manishasinha*10 --subnet /subscriptions/6b26da86-6cfd-4198-afde-fb754b11c54e/resourceGroups/testazcli/providers/Microsoft.Network/virtualNetworks/testvnet/subnets/testsubnet --location southindia --size Standard_E32s_v3 --public-ip-address ""

Thank you for your contribution! We will review the pull request and get back to you soon.

/azp run

Hello Team,
Please let me know if anything else is required from my end.
Thanks!

Thank you for the response!
I am still unable to see changes in public doc.


Non UTF-8 encoding for read_file?

Adding support for different encodings to read_file by reading bytes, then decoding
(also consider using with open() instead of manually opening and closing the file)

implemented in d30930aa9515384d61fe6cbba17fabc181d5c8a5


An error occurred in the program

I opened the sln file and the form program after using visual studio 2015 and 2015 versions. After inputting ID and secret, I clicked Authenticate button to here comes an error

Can you try with a newer Visual Studio? 2015 version is no longer supported with recent .NET versions.


Ask a question about iterators

How to query the data in row n. Or row m after the nth row.
My current requirement is to query the m row data after the nth row.
thank you.

Just iterate n-times.


Last page of wrapped should have a different path

To make sharing wrapped easier, the final "everything-on-one-page" page should not be part of the slide show, and the share button should share a link to that final page instead.

Fixed in 61126df


Remove and disable package-lock.json

package-lock.json was added in #111 even though we have and use yarn.lock. This removes the package-lock.json file and adds a .npmrc file which disables creation of the lock file. npm 5 does not work on windows and the presence of it's lock file only causes more issues in CI in my experience. Which is why I'd vote for continuing to use yarn. This also updates the yarn.lock file for newly added deps.

i think it's considered bad practice for a library/reusable module to hold a package-lock.json anyway.  I didn't look too deeply into why, but i'm pretty sure it helps with package duplication and reduces bloat. sindresorhus and shellscape both follow that pattern.  My understanding is that it's recommended for applications.
didn't know npm 5 had issues on windows.  Thanks for mentioning hat

Oops, my mistake.
I'm fine removing it and leaving yarn.lock. Just to clarify a couple points, though:
@sudo-suhas "npm 5 does not work on windows": I think a lot of people use it successfully on Windows; and it looks like the CI failures are about test coverage not installation failures, right? (e.g. https://ci.appveyor.com/project/davidtheclark/cosmiconfig/build/job/3erl53ej7wt4e5a1)

i think it's considered bad practice for a library/reusable module to hold a package-lock.json anyway

It's encouraged to commit the lockfile to your repo (both for Yarn and for npm). Some people don't commit lockfiles in order to avoid the annoying merge conflicts they can produce when you change dependencies. But I think both Yarn and npm have since released ways to easily manage those conflicts. If a module like cosmiconfig is installed by an end user, its lockfile doesn't have any impact on the installation — just the semver ranges in package.json. I guess I'm not aware of outstanding reasons not to commit the lockfile.

hmm - I can't find any good reasons to forego lock files then.  Sorry about that.  Maybe sindresorhus just considers it noise like you mentioned

I am unsure about npm, but yarn  according to yarnpkg.com/blog/2016/11/24/lockfiles-for-all:  "Lockfiles within your dependencies will be ignored".
I think the argument against using lockfiles in libraries would be if an upstream package breaks something, the library's tests wouldn't catch it, but the end user's app would break. Pretty much the opposite reason you want to use lock files in your application.

@sudo-suhas "npm 5 does not work on windows": I think a lot of people use it successfully on Windows; and it looks like the CI failures are about test coverage not installation failures, right?

We had faced issues due to npm in lint-staged (mostly when the node_modules cache was enabled) like this one - https://ci.appveyor.com/project/okonet/lint-staged/build/1.0.79/job/c1u2k57uwn2dn91o. I had also got EPERM issues on my local Windows system quite a few times. Which was why I had to switch to yarn. I'll concede that it has been some time since I last used npm so the issues might have been solved. However, considering yarn works quite well, I haven't had the motivation to try npm again(there are other issues when trying to use it with nvm-windows).

I'm not aware of outstanding reasons not to commit the lockfile

Let's say you declared the dependency pkg-a: ^1.1.0 and this was locked to 1.1.3. If a newly released version, say 1.2.0, breaks any of the tests, only the end-user will be able to see it because when we run any test in CI, yarn will install 1.1.3 and not 1.2.0. But this is an edge case and I don't think this is a real risk for cosmiconfig.

I am unsure about npm, but yarn  according to yarnpkg.com/blog/2016/11/24/lockfiles-for-all:  "Lockfiles within your dependencies will be ignored".

Here's a relevant article - https://jpospisil.com/2017/06/02/understanding-lock-files-in-npm-5.html. package-lock.json present in dependencies are ignored similar to yarn.lock.

But this is an edge case and I don't think this is a real risk for cosmiconfig
Yeah and without something like greenkeeper too, it'd be lucky if we even caught updated dependency incompatibilities

Which version of yarn does this repo use? When running yarn with the most recent version 1.5.1 it updates the yarn.lock file.

Yarn is fully deterministic as long as all your teammates are using the same Yarn version.

When using yarn, everyone needs to use the same version according to https://yarnpkg.com/blog/2017/05/31/determinism/
This might be another reason to remove yarn.lock?

This might be another reason to remove yarn.lock?

I'm fine removing Git-ignoring yarn.lock if others want to.

I don't have a strong opinion but if I had to vote, I'd keep the lock file.

Looks like the --pure-lockfile option might be relevant here. If you're not actually updating dependencies, you could use yarn --pure-lockfile to use the lockfile as the source of truth, ignoring package.json.


Predefined styles

Predefine sets of styles (primarily colour palettes).
Document visuals showing example outcome and 'styleguide' of unique aspects.
Provide easy method of incorporating these styles into an existing pinout project.

This is sort-of achieved with implementation of an auto-generating css file when no style-sheet is added to a diagram. Colour palette is pseudo-randomly selected with results filtered to match the criteria of text/background colour contrast ratio greater than 3 (see wcag accessibility recommendations for more details.)

on reflection I feel this might be out of scope for the core project.
closing with the thought of creating sample/demo/ style guide with documented colours and styles as another related project (along with PCB graphical components.)


Fix retval in commands.go /Post

Replace log.Fatalf  with return fmt.Errorf(..

Can you explain why you want to change this? Especially for the last of the two?

Can you explain why you want to change this? Especially for the last of the two?

I wanted to call postEvent() (I thought) from a go program without exiting on error. I Cannot do that, so I have replicated the postEvent implementation instead. You can discard the patch . Maybe put the postEvent code into the library, so others can call it ?

I wanted to call postEvent() (I thought) from a go program without exiting on error. I Cannot do that, so I have replicated the postEvent implementation instead. You can discard the patch . Maybe put the postEvent code into the library, so others can call it ?

These are the action functions for the command line client, and are not a part of the library. They depend on the command line context to operate. This function just generates an API call based on command line parameters so it isn't a general function that should be imported anywhere else.

These are the action functions for the command line client, and are not a part of the library. They depend on the command line context to operate. This function just generates an API call based on command line parameters so it isn't a general function that should be imported anywhere else.


Fixed IBM broken link in ReadMe

Looks like they're no longer using Slate? I think another PR managed to fix this, but thanks for the suggestion!


Remove 'use strict' from source

Removes the 'use strict' statement from the source code. This is no longer required as modules are strict by default, and Rollup will therefore automatically add 'use strict' to the CommonJS and UMD bundles.

Perhaps one should add strict to the tsconfig in this PR also?
Such that TypeScript also operates under the assuption of strict mode.
https://www.typescriptlang.org/tsconfig#strict

TypeScript strict more dictates the strictness of types, it has nothing to do with strict mode in JavaScript.

Yes it turns on the alwaysStrict flag in addition to more type checking strictness as you say.

Ensures that your files are parsed in the ECMAScript strict mode, and emit “use strict” for each source file.

From StackOverflow Strict vs alwasStrict

It is a good idea to not only include alwaysStrict, but to include the strict flag that enables alwaysStrict and also noImplicitAny, noImplicitThis, strictBindCallApply, strictNullChecks, strictFunctionTypes, strictPropertyInitialization and useUnknownInCatchVariables which are even more important.

Does not matter seems that strict is enabled by default either way:

Personally i would prefer to be explicit about it being enabled imo.

Seem that it is already enabled 😂 https://github.com/auth0/jwt-decode/blob/beta/tsconfig.json#L13


Feedback page/ Contact us page

I would like to build a page that lets users gives the moderator feedback about the website i.e the positive things that they like about the site , the issues that they are facing as well as some features that they would like to add to the site .

I'm a GSSOC'21 participant and I would love to work on it.
Thank You.

@RubenFalcao14 Hey, if you want to make something like that, I can give you something related to it.

Here is what you have to do, below the search bar in the above image, you have to put review cards of customers/users who have given the review.
Something like this:-https://youtu.be/O-QUBZuZlXM
Let me know if you can do it in a couple of days.

I'll try my best , Thank You.

Most Welcome. All the Best :)

It's almost done , I just need to sort out some minor issues I'll send you some pics , in about 1 day's time . Sorry for the delay .

Sir is it okay ?@yash982000

Sir is it okay ?@yash982000

Works but there should be a pop-up kinda thing when the mouse pointer hover over it.

I'll try to add it by today and send you a pic.

@yash982000 Sir , I have added the hover effect to the review cards let me know if it's okay .

@yash982000 Sir , I have added the hover effect to the review cards let me know if it's okay .

@RubenFalcao14  It looks good. Can you add the code now?
Also, I hope you have made the CSS file too and it's responsive.

@yash982000 Sir, have made 2 files 1 .html file and 1 .css file. Could you tell me where I should add them in the repo .

@yash982000 Sir, have made 2 files 1 .html file and 1 .css file. Could you tell me where I should add them in the repo .

Make a pull request by adding these files in a new folder named templates or similar.


Add note that only 64bit builds are available

From<EMAIL_ADDRESS>
I'm trying to install Nuvola Apps in Xubuntu 16.10 but it's impossible because my terminal says that: "No such ref 'app/eu.tiliado.Nuvola/i386/stable' in remote eu.tiliado.Nuvola-6-origin"
Could you help me?

$ flatpak install --from https://nuvola.tiliado.eu/eu.tiliado.Nuvola.flatpakref
Installing: eu.tiliado.Nuvola/i386/stable
Warning: Can't find dependencies: No entry for app/eu.tiliado.Nuvola/i386/stable in remote summary flatpak cache
Installing: eu.tiliado.Nuvola/i386/stable from eu.tiliado.Nuvola-6-origin
error: No such ref 'app/eu.tiliado.Nuvola/i386/stable' in remote eu.tiliado.Nuvola-6-origin


gzipped input?

hi, thanks for releasing 2.0! I successfully installed and ran it on Mac OS X 🎉 (and presumably Linux will just work :).
Our workflows generally produce gzipped FASTQ files after preprocessing. It seems like cuttlefish does not currently support them; thought I'd drop this issue here to check in and let you know that we'd love that support :).

Hello Titus,
Cuttlefish do support gzipped inputs! Could you please paste here what commands are you trying to use for cuttlefish, and what error messages are you getting?

excellent! I double-checked and (of course) you're right. I think I was trying on a gzipped FASTA file rather than a FQ file at first, and I never went back and checked with a proper FQ file. Thanks!


Auto promoted Freight Error Patching ArgoCD Application

Description
After updating from 0.8.4 to 0.8.6 I'm consistently hitting this error for automatically promoted freight only. A subsequent manual promotion of the same freight, or an initial manual promotion of freight into the same stage is successful.
Promotion Errored
error executing Argo CD promotion mechanism: error patching Argo CD Application "<app-name>": failed to patch the object: Operation cannot be fulfilled on applications.argoproj.io "<app-name>": the object has been modified; please apply your changes to the latest version and try again

Steps to Reproduce

Warehouse subscribed to a git repo and image repo
Stage with promotionMechanisms.gitRepoUpdate and promotionMechanism.argoCDAppUpdates configured with multiple ArgoCD Applications

The stage can allow direct freight from the warehouse or require another stage
annotate for authorized stage on the applications
autoPromotionEnabled for the stage on the project

New image tag discovered and freight generated
Auto promotion begins, freight changes are committed to git
Error occurs as the argocd app refresh/sync process starts
Manually promote the same freight into stage succeeds

Version
Client Version: v0.8.7
Server Version: v0.8.6
Logs
"error executing Promotion" error="error executing Argo CD promotion mechanism: error patching Argo CD Application \"<app-name>\": failed to patch the object: Operation cannot be fulfilled on applications.argoproj.io \"<app-name>\": the object has been modified; please apply your changes to the latest version and try again" freight=cd45f097cc7d86b05091f22181b96c087523f888 namespace=<app-ns> promotion=canary.01j6axmchf8jyx54svs7a0mb6a.cd45f09 stage=canary

This seems to have been a result of having argocd auto sync policy enabled on the applications managed by the stage. Up to this point, I hadn't seen any particular reason to turn it off/on, as I assumed it didnt matter what was triggering the sync, since the branch the argocd app is associated with only gets updated when new freight is ready/pushed to the branch. With the optimistic locking for argocd app sync added in 0.8.5, I see now that it interferes with kargo's ability to trigger the sync.

Never mind - this persists sporadically with auto-sync disabled. I'm wondering if it's the application being refreshed (not synced) by Argocd while kargo is trying to trigger its own refresh/sync? They get refreshed fairly quickly because they are configured with webhooks, but I imagine if that were the case, even without webhook configured, a stage that's responsible for enough Applications would run into that eventually when a polling refresh coincided with a kargo promotion.
Either way, curious if it make sense to have some kind of retry here getting the resource, generating the patch, and attempting to apply, so single occurrences don't error the promotion.

Any chance you can catch the individual versions of the object from around the timeframe the issue occurs?
Asking because I would like to understand the core of the issue better, before trying to address it.

Got all apps before, and the failed app after. Only thing that stands out in the diff is status.resources with an HPA recommendation. The latest item in status.history is a previous kargo-controller triggered action(about 15 minutes before the current promotion) that was successful. The status.reconciledAt, however is about 20 seconds after the failed promotion began.

Sounds good, let me know if I can provide any more details to replicate. Even with the chance of it happening being slim, I'm curious in what case we wouldn't want to retry? Maybe I'm misunderstanding the goal of the write lock, but if it is just to prevent overwriting changes made by some other tooling, should something else interacting with the argo application be a potential blocker for a promotion?


remove default space calc from book

This should be done once anchor 0.23.0 is released

closed by #47


[hi] Localized content/en/docs/tasks/administer-cluster/access-cluster-api.md in hindi

fixes: #40757

Since, a long time there isn't any respond/activity from your side on this PR @KashishLakhara04. Kindly, respond or implement changes so that this PR can either be merged or closed.

@KashishLakhara04 : I'll be closing off this issue since there has been no response for ~8 months.
/close


Force Powers purchase bug

When trying to purchase a Force Power it does not analyze the maximum value to allow the purchase.

Done, will be released in 1.902. Thanks for the report!


Multiple css properties output on a single color?

Issue description:
I know this has been asked before (https://github.com/aristath/kirki/issues/72) but the current solution is not acceptable, especially if you have more than a few properties to set:
Version used:
latest (<IP_ADDRESS>)
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
For example, I have these rules in my css (defaults):
.header {
	border: solid 1px $borderColor;
	-webkit-box-shadow: 0 -1px 0 0 $borderColor inset;
	-moz-box-shadow: 0 -1px 0 0 $borderColor inset;
	box-shadow: 0 -1px 0 0 $borderColor inset;
}

Now, I have added a color control in Customizer that I can use to change the border color, but to apply the change on the above selector (please imagine that's not the only one I have), I would have to do this:
'output' => array(
	array(
		'element'  => '.header',
		'property' => 'border-color',
	),
	array(
		'element'  => '.header',
		'property' => '-webkit-box-shadow',
               // yes, using the "pattern_replace"
	),
	array(
		'element'  => '.header',
		'property' => '-moz-box-shadow',
               // yes, using the "pattern_replace"
	),
	array(
		'element'  => '.header',
		'property' => 'box-shadow',
               // yes, using the "pattern_replace"
	),
       // and so on...

And this code would only get bigger....
The most reasonable solution would be to have "property" as an array as well, so for each entry in the array, apply the value...
@aristath Can you please have a look over this? I don't think it would be difficult to implement this functionality.
Thanks a bunch and keep up the good work!
Regards,
Costin

you can always write your own javascript function when using the 'output' => 'postMessage'-setting as described here: http://aristath.github.io/kirki/docs/modules/postmessage.html
#2026

Marking as a feature request. I agree, using an array for the property would simplify things.
However, the particular use-case from above doesn't really need to be like that. Kirki automatically adds vendor-prefixes here: https://github.com/aristath/kirki/blob/v3.0.34/modules/css/class-kirki-modules-css-generator.php#L241-L258 so there will work cross-browser without any extra code from you: 'border-radius', 'box-shadow', 'box-sizing', 'text-shadow', 'transform', 'background-size', 'transition', 'transition-property'


Computing vector distance functions (like vss_inner_product) incorrectly parses the data

I'm dumping some raw bytes into a vector column, but when sqlite-vss parses the BLOB into a vector, it checks if it starts with a v\x01 and if so, these two bytes are treated a header.
The problem is, I have some vectors actually do start with this header (as part of the data) and now it fails to parse the data correctly.
https://github.com/asg017/sqlite-vss/blob/8fc44301843029a13a474d1f292378485e1fdd62/src/sqlite-vector.cpp#L68-L76
One thing I can do is to prepend the header to every row, but that feels a really bad solution and it would be great if we can fix this.

Try vector_from_raw() instead. The vector_from_blob() function was a poor attempt at a new vector format, but it made things more complex for no reason (like you saw). The vector_form_raw() on the other hand should handle raw blobs correctly (ie 4 bytes per float vectors).
I'll likely deprecate vector_from_blob() in the next release


Insertion Sort

Description
This is a code to sort an array with the help of insertion sort
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

Are you contributing under any open-source program ?
hacktoberfest

Ma'am would you please assign me this Issue ?

Description
This is a code to sort an array with the help of insertion sort
Programming language

[ ]  C
[x]  C++
[ ]  Java
[ ]  Python

Are you contributing under any open-source program ?
hacktoberfest

Assigning this to you. Inside algorithms folder there is sorting algorithm folder inside that make a file Insertion_Sort.md and explain everything neatly.

Maam along with the md file do i also add the code file ?

I would like to contribute in this using C

I would like to contribute in this using C

Wait for this pr to be merged then you can go to improve a doc template and open a new issue for this.


Try to get data from morph table

Laravel Version: 7.18.0
PHP Version: 7.3.22
Database Driver & Version: MySQL 5.7

Description:
I try to get favorite data from favorites morph table and get error:
Integrity constraint violation: 1052 Column 'id' in order clause is ambiguous
Steps To Reproduce:
$client = Client::findOrFail($request->client_id);
$client->getFavoriteItems('App\Models\Vendor')->get();

public function getFavoriteItems(string $model)
{
	return app($model)->whereHas('favoriters', function ($query) {
		return $query->where('client_id', $this->getKey());
	});
}

SQL:
and getFavoriteItems function return:
SELECT
	*
FROM
	`vendors`
WHERE
	EXISTS (
		SELECT
			*
		FROM
			`clients`
			INNER JOIN `favorites` ON `clients`.`id` = `favorites`.`client_id`
		WHERE
			`vendors`.`id` = `favorites`.`favoriteable_id`
			AND `client_id` = 1
			AND `favoriteable_type` = 'App\\Models\\Vendor'
			AND `clients`.`deleted_at` IS NULL
		ORDER BY
			`id` DESC)
		AND `vendors`.`deleted_at` IS NULL
	ORDER BY
		`id` DESC;

correct sql looks like this (this is hard code)
SELECT
	*
FROM
	`vendors`
WHERE
	EXISTS (
		SELECT
			*
		FROM
			`clients`
			INNER JOIN `favorites` ON `clients`.`id` = `favorites`.`client_id`
		WHERE
			`vendors`.`id` = `favorites`.`favoriteable_id`
			AND `client_id` = 1
			AND `favoriteable_type` = 'App\\Models\\Vendor'
			AND `clients`.`deleted_at` IS NULL
		ORDER BY
			`clients`.`id` DESC)
		AND `vendors`.`deleted_at` IS NULL
	ORDER BY
		`id` DESC;

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!


Cache misbehaves on version 1.3.4

Description of the bug
We have simulations with and without header matchers. We noticed that on version 1.3.4 we get wrong results back from Hoverfly on requests that do not contain header matchers.
Steps to reproduce the issue
Difficult to describe in detail, but feel free to reach out I can demonstrate in a call (or expand the description here).
I'll try to illustrate, cannot share the entire recordings.
Request 1:
{
				"request": {
					"path": [
						{
							"matcher": "exact",
							"value": "/s4hanacloud/sap/opu/odata/sap/API_OUTBOUND_DELIVERY_SRV/$metadata"
						}
					],
					"method": [
						{
							"matcher": "exact",
							"value": "GET"
						}
					],
					"destination": [
						{
							"matcher": "exact",
							"value": "sandbox.api.sap.com"
						}
					],
					"scheme": [
						{
							"matcher": "exact",
							"value": "https"
						}
					],
					"body": [
						{
							"matcher": "exact",
							"value": ""
						}
					],
					"query": {
						"sap-documentation": [
							{
								"matcher": "exact",
								"value": "all"
							}
						]
					}
				},
				"response": {
					"status": 200,
					"body": "<Some xml>",
    				"encodedBody": false,
					"headers": {
						"Cache-Control": [
							"max-age=0"
						],
						"Content-Type": [
							"application/xml"
						]
					},
					"templated": false
				}
			},

Request 2:
				"request": {
					"path": [
						{
							"matcher": "exact",
							"value": "/path/to/resource"
						}
					],
					"method": [
						{
							"matcher": "exact",
							"value": "GET"
						}
					],
					"destination": [
						{
							"matcher": "exact",
							"value": "service.tld"
						}
					],
					"scheme": [
						{
							"matcher": "exact",
							"value": "https"
						}
					],
					"headers": {
						"SpecificHeader": [
							{
								"matcher": "exact",
								"value": "Specific-Value"
							}
						]
					}
				},
				"response": {
					"status": 200,
					"body": "<Some json>",
					"encodedBody": false,
					"headers": {
						"Content-Length": [
							"..."
						],
						"Content-Type": [
							"application/json"
						]
					},
					"templated": false
				}
			},

Observed result
Requests (with Hoverfly as proxy):

GET https://service.tld/path/to/resource (with correct header)

Returns <Some json> (correct result)

GET https://sandbox.api.sap.com/s4hanacloud/sap/opu/odata/sap/API_OUTBOUND_DELIVERY_SRV/$metadata?sap-documentation=all

Returns <Some json> (incorrect result)

GET https://sandbox.api.sap.com/s4hanacloud/sap/opu/odata/sap/API_OUTBOUND_DELIVERY_SRV/$metadata (missing query)

No match found (correct result)

To me it looks like the lookup is done correctly (result is only returned on requests that exist in the simulation) but the returned result does not come from the cache if it matched, instead one of the simulations with a header matcher is returned (it seems to always be the same one).
Best guess: Related to https://github.com/SpectoLabs/hoverfly/pull/1006
Expected result

GET https://sandbox.api.sap.com/s4hanacloud/sap/opu/odata/sap/API_OUTBOUND_DELIVERY_SRV/$metadata?sap-documentation=all

Returns <Some xml>

Additional relevant information

Hoverfly version: v1.3.4
Anything that might help us to diagnose the problem: does not happen on v1.3.3

The workaround for now is to disable caching: hoverfly -disable-cache. I will try to get a bug fix version out as soon as I can


[Bug] Docker image cannot be built

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[x] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
main branch https://github.com/open-mmlab/mmdetection3d
Environment
Docker env
Reproduces the problem - code sample
docker build -t mmdetection3d docker/

Reproduces the problem - command or script
docker build -t mmdetection3d docker/

Reproduces the problem - error message
=> [5/6] RUN conda clean --all     && git clone https://github.com/open-mmlab/mmdetection3d.git -b dev-1.x /mmdetection3d     && cd /mmdetection3d     && pip install --no-cache-dir -e .                  361.2s
=> => #   Downloading attrs-24.1.0-py3-none-any.whl (63 kB)
=> => #   Downloading attrs-23.2.0-py3-none-any.whl (60 kB)
=> => #   Downloading attrs-23.1.0-py3-none-any.whl (61 kB)
=> => #   Downloading attrs-22.2.0-py3-none-any.whl (60 kB)
=> => #   Downloading attrs-22.1.0-py2.py3-none-any.whl (58 kB)
=> => #   Downloading attrs-21.4.0-py2.py3-none-any.whl (60 kB)
Additional information
Installer cannot find suitable version for pip packages.

34 |     # Install MMDetection3D
35 | >>> RUN conda clean --all
36 | >>>     && git clone https://github.com/open-mmlab/mmdetection3d.git -b dev-1.x /mmdetection3d
37 | >>>     && cd /mmdetection3d
38 | >>>     && pip install --no-cache-dir -e .
29316.2 WARNING: jsonschema 3.2.0 does not provide the extra 'format-nongpl'
29318.0   Downloading jupyter_events-0.6.1-py3-none-any.whl (18 kB)
pip._vendor.resolvelib.resolvers.ResolutionTooDeep: 2000000

Added RUN python -m pip install --upgrade pip in Dockerfile to resolve this issue

Indeed, updating pip works. @joelkny97 Thank you!

This did work, but why isn't this already in the official Dockerfile?


I have installed it from the releases page

And I don't see any plugin except translate and calc.
For example, I don't see the ip plugin

And gif plugin.

Do I need to install anything?

Type plugins :wink:

Thanks.


Destination path already exists and is not an empty directory using --terragrunt-source-update

Hi everyone,
We have experienced the following error while trying to run terragrunt plan-all --terragrunt-source-update --terragrunt-non-interactive --terragrunt-include-external-dependencies
error downloading<EMAIL_ADDRESS>/usr/bin/git exited with 128: fatal: destination path '..project-terragrunt/secrets/.terragrunt-cache/M062klXZFj2B2dKtXGow5FueqdY/ZS8XBy97Un6SDm3LMf87-Dx8xEY' already exists and is not an empty directory.
Our project has the following structure:
...
├── lambda
│   ├── LambdaA
│   │   └── terragrunt.hcl
│   ├── LambdaB
│       └── terragrunt.hcl
├── secrets
│   └── terragrunt.hcl
...

And we were trying to execute the command in the lambda folder.
LambdaA and LambdaB have:
terraform {
  source = "git::git@github.com:project-terraform//lambda/LambdaA?ref=branch"
}
dependency "secret" {
  config_path = "../../secrets"
}

We are currently avoiding this issue changing the reference to a certain commit and removing the flag --terragrunt-source-update, so we think the problem should be in the cache folder cleanup.

Could you share more of the log output? Seeing the surrounding context for your error would be helpful.

Hi Yevgeniy!
I attach an output of the log where we see the error:
output-terragrunt-test.txt
This is a smaller version than the original, which is too long. If you want, we can attach it too.

Hi everyone!
We still have this issue. Were able to conduct any research on it?

In this case, when running a plan-all, Terragrunt runs concurrent plans for each lambda, and both lambdas has the same dependencies, thus using  --terragrunt-source-update will be trying to clone the source twice and at the same time.
A workaround can be to iterate with a loop using  plan instead of plan-all. For example:
for D in lambda/*/;
  do
    cd "${D}"
    terragrunt plan --terragrunt-source-update --terragrunt-non-interactive -no-color
    cd ../..
  done

Another approach can be to delete the .terragrunt-cache directory to force downloading the sources for each dependency and module. For example, using a before_hook like this one:
terraform {
  before_hook "interpolation_hook" {
    commands     = ["plan"]
    execute      = ["rm", "-rf", "${get_terragrunt_dir()}/.terragrunt-cache/" ]
 }

But these are just workaround. Terragrunt should be smart enough to resolve the dependencies,  and download it just one time and then run a plan for each lambda.

Hi!
I have the same issue.
Second workaround throw me the following panic on runtime error (Terraform 0.12.12, Terragrunt 0.20.4):
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x199a682]

goroutine 8 [running]:
github.com/gruntwork-io/terragrunt/cli.runTerraformWithRetry(0xc0002cc2c0, 0x3, 0x0)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/cli/cli_app.go:475 +0x142
github.com/gruntwork-io/terragrunt/cli.runTerragruntWithConfig.func1(0xc00048d400, 0x1)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/cli/cli_app.go:428 +0x2a
github.com/gruntwork-io/terragrunt/cli.runActionWithHooks(0x1c060d9, 0x9, 0xc0002cc2c0, 0xc000111000, 0xc00050bd30, 0xc00024c960, 0xc000496010)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/cli/cli_app.go:439 +0x28e
github.com/gruntwork-io/terragrunt/cli.runTerragruntWithConfig(0xc0002cc2c0, 0xc000111000, 0x0, 0x0, 0x0)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/cli/cli_app.go:427 +0x2c7
github.com/gruntwork-io/terragrunt/cli.runTerragrunt(0xc0002cc2c0, 0x1c1076d, 0x15)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/cli/cli_app.go:315 +0x4fb
github.com/gruntwork-io/terragrunt/configstack.(*runningModule).runNow(0xc000305540, 0x0, 0x0)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/configstack/running_module.go:238 +0x16f
github.com/gruntwork-io/terragrunt/configstack.(*runningModule).runModuleWhenReady(0xc000305540)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/configstack/running_module.go:201 +0x6a
github.com/gruntwork-io/terragrunt/configstack.runModules.func1(0xc000584ef0, 0xc000305540)
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/configstack/running_module.go:171 +0x5b
created by github.com/gruntwork-io/terragrunt/configstack.runModules
	/private/tmp/terragrunt-20191014-59879-vdrekm/src/github.com/gruntwork-io/terragrunt/configstack/running_module.go:169 +0xe1

I'll take a detailed look at this today since I implemented the fix.

Hi, I decided to revert the problematic change for the time being. Can you see if that fixes the issue? I created an integration testing release for convenience: https://github.com/gruntwork-io/terragrunt/releases/tag/v0.21.2-alpha.1

I've just tried with that release and it worked :+1:

I've tried it now - worked as charm :)

Hi @gcrespi and @zelig81 , thanks for confirming that fix! I would also appreciate it if you can give https://github.com/gruntwork-io/terragrunt/releases/tag/v0.21.2-alpha.2 (https://github.com/gruntwork-io/terragrunt/pull/937) a try as well. Unlike the previous fix which rolls back the change, this will do proper synchronization of the terragrunt output call (and cache the results) so that we can keep the original change that is causing problems. If this works, we will most likely go with this version of the fix.

Released the latest alpha fix as v0.21.2 (https://github.com/gruntwork-io/terragrunt/releases/tag/v0.21.2). Binaries will show up shortly. The original alpha releases will be deleted once the binaries for v0.21.2 show up.
I will keep this open until I have confirmation that this fixes the root issue.

After upgrading to 0.21.2 the problem still persists :(
In order to work around the problem I've remove the cache files using find . ..... solution from the read.me but it did not help...
With the alpha version that is still on my computer the problem continues to be solved, so meanwhile I continue to use it :)
Directory structure:
.
├── production
│   └── environment.tfvars.json
├── staging
│   ├── infrastructure
│   │   ├── sg
│   │   │   ├── sg-db-access-from-vpn
│   │   │   │   └── terragrunt.hcl
│   │   │   ├── sg-https-from-vpn
│   │   │   │   └── terragrunt.hcl
│   │   │   ├── sg-ssh-from-vpn
│   │   │   │   └── terragrunt.hcl
│   │   │   ├── sg-vpc-traffic
│   │   │   │   └── terragrunt.hcl
│   │   │   └── sg-web-traffic
│   │   │       └── terragrunt.hcl
│   │   └── vpc
│   │       └── terragrunt.hcl
│   └── environment.tfvars.json
├── README.md
├── commons.tfvars.json
└── terragrunt.hcl

The log with errors:
[terragrunt] 2019/11/03 09:51:00 Encountered the following errors:
Hit multiple errors:
error downloading<EMAIL_ADDRESS>/usr/bin/git exited with 128: Cloning into '/Users/ilyag/code/infra-deployment/staging/infrastructure/vpc/.terragrunt-cache/Ef-6lmvbZXkKDLUhRJi2mrsUqtQ/e--jc6WhBhtnKuyeyTkOxcY52Do'...
fatal: cannot copy '/Library/Developer/CommandLineTools/usr/share/git-core/templates/description' to '/Users/ilyag/code/infra-deployment/staging/infrastructure/vpc/.terragrunt-cache/Ef-6lmvbZXkKDLUhRJi2mrsUqtQ/e--jc6WhBhtnKuyeyTkOxcY52Do/.git/description': File exists

Hit multiple errors:
error downloading<EMAIL_ADDRESS>/usr/bin/git exited with 128: fatal: destination path '/Users/ilyag/code/infra-deployment/staging/infrastructure/vpc/.terragrunt-cache/Ef-6lmvbZXkKDLUhRJi2mrsUqtQ/e--jc6WhBhtnKuyeyTkOxcY52Do' already exists and is not an empty directory.

unlinkat /Users/ilyag/code/infra-deployment/staging/infrastructure/vpc/.terragrunt-cache/Ef-6lmvbZXkKDLUhRJi2mrsUqtQ/e--jc6WhBhtnKuyeyTkOxcY52Do: directory not empty
Hit multiple errors:
error downloading<EMAIL_ADDRESS>/usr/bin/git exited with 128: fatal: destination path '/Users/ilyag/code/infra-deployment/staging/infrastructure/vpc/.terragrunt-cache/Ef-6lmvbZXkKDLUhRJi2mrsUqtQ/e--jc6WhBhtnKuyeyTkOxcY52Do' already exists and is not an empty directory.

[terragrunt] 2019/11/03 09:51:00 Unable to determine underlying exit code, so Terragrunt will exit with error code 1

@zelig81 Thanks for trying the new release. Can you share the entire log output, not just the error section, for a run with the environment variable TG_LOG=debug? This will help see other possibilities for the issue. It is unlikely that we will merge in the rolled back version, so I would much rather roll forward with the proper fix.
FWIW, I have been able to reproduce the issue in test and successfully validated that the issue is fixed in https://github.com/gruntwork-io/terragrunt/pull/937, but it is possible that you have a different setup that is causing this to happen that the fix in 0.21.2 isn't handling. The extended debug logging will help in identifying that potential problem.

Sorry - took time to relaunch.
folders structure of infra-deployment:
.
├── production
│   └── environment.tfvars.json
├── shared
│   └── route53
│       └── public-hosted-zone
│           └── terragrunt.hcl
├── staging
│   ├── infrastructure
│   │   ├── rds
│   │   │   └── aurora-online
│   │   │       └── terragrunt.hcl
│   │   ├── route53
│   │   │   └── private-hosted-zone
│   │   │       └── terragrunt.hcl
│   │   ├── sg
│   │   │   ├── sg-db-access-from-vpn
│   │   │   │   └── terragrunt.hcl
│   │   │   ├── sg-https-from-vpn
│   │   │   │   └── terragrunt.hcl
│   │   │   ├── sg-ssh-from-vpn
│   │   │   │   └── terragrunt.hcl
│   │   │   ├── sg-vpc-traffic
│   │   │   │   └── terragrunt.hcl
│   │   │   └── sg-web-traffic
│   │   │       └── terragrunt.hcl
│   │   └── vpc
│   │       └── terragrunt.hcl
│   └── environment.tfvars.json
├── README.md
├── commons.tfvars.json
├── output.txt
└── terragrunt.hcl

folders structure of modules:
../tf-modules
├── rds
│   ├── README.md
│   ├── main.tf
│   ├── outputs.tf
│   └── variables.tf
├── route53
│   └── hosted-zone
│       ├── README.md
│       ├── main.tf
│       ├── outputs.tf
│       └── variables.tf
├── sg-generic
│   ├── README.md
│   ├── main.tf
│   ├── outputs.tf
│   └── variables.tf
├── vpc
│   ├── README.md
│   ├── main.tf
│   ├── outputs.tf
│   └── variables.tf
└── README.md

attached output of export TG_LOG=debug; terragrunt plan-all --terragrunt-source-update --terragrunt-source ~/code/tf-modules/ --terragrunt-non-interactive
debug.log

Hi thanks for sharing the logs. For some reason, it looks like the synchronization locking mechanism introduced in v0.21.2 isn't working for your environment. This is possible if the dependency config path isn't properly overlapping, but the logs seem to indicate that the path is cleaned correctly.
I just pushed a new release (https://github.com/gruntwork-io/terragrunt/releases/tag/v0.21.5) with additional debug logging which will give me a definitive answer for this. If you don't mind, can you run export TG_LOG=debug; terragrunt plan-all --terragrunt-source-update --terragrunt-source ~/code/tf-modules/ --terragrunt-non-interactive again for me?
And I really appreciate your patience in helping out here!

Of course, hope I'll do it today or tomorrow

after upgrading to the new version of terragrunt it is suddenly started to work :)
terragrunt version v0.21.5
Terraform v0.12.13
I can confirm that the issue is solved. Thank you for your patience and time!!!!

Thank you for your patience and time!!!!

And thank you for your patience helping me with debugging this!
Going to go ahead and close this now that it is confirmed fixed. If others continue to have issues, we can open a new ticket.

root@1a1237fc6310:/# git clone<EMAIL_ADDRESS>fatal: destination path 'alx-pre_course' already exists and is not an empty directory.


Unable to compile 'Windows' branch

Using Visual Studio 2013.
Error   1   error MSB3073: The command "asm\fasm.exe asm\xenoncat_AVX.asm asm\xenoncatavx1.obj
asm\fasm.exe asm\xenoncat_AVX2.asm asm\xenoncatavx2.obj
:VCEnd" exited with code 9009.  C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.CppCommon.targets  127 5   cpu_xenoncat
Error   2   error LNK1104: cannot open file 'pthreads\pthreadVC2.lib'   D:\Apps\Crypto\nheqminer-Windows\cpu_tromp\LINK cpu_tromp
Error   3   error C1083: Cannot open include file: 'CL/cl.h': No such file or directory d:\apps\crypto\nheqminer-windows\ocl_xpm\opencl.h   12  1   ocl_xpm
Error   4   error C1083: Cannot open include file: 'CL/cl.h': No such file or directory d:\apps\crypto\nheqminer-windows\ocl_xpm\opencl.h   12  1   ocl_xpm
Error   5   error C1083: Cannot open include file: 'CL/cl_ext.h': No such file or directory d:\apps\crypto\nheqminer-windows\ocl_xpm\ocl_device_utils\cl_ext.hpp    168 1   ocl_xpm
Error   6   error LNK1104: cannot open file 'cuda_tromp.lib'    D:\Apps\Crypto\nheqminer-Windows\nheqminer\LINK nheqminer

if you can find where fasm.exe gets called , add -m524288 to the command line
from; http://board.flatassembler.net/topic.php?t=12800

Hi @skive. I'll push the fix now. .gitignore blacklisted the FASM bin

Pull the latest commit it should work now. Reopen if it does not work.

Fasm error fixed with last commit. Other erroros still firing.
Error	103	error C1083: Cannot open include file: 'CL/cl_ext.h': No such file or directory	d:\apps\crypto\nheqminer-windows\ocl_xpm\ocl_device_utils\cl_ext.hpp	168	1	ocl_xpm
Error	104	error C1083: Cannot open include file: 'CL/cl.h': No such file or directory	d:\apps\crypto\nheqminer-windows\ocl_xpm\opencl.h	12	1	ocl_xpm
Error	105	error C1083: Cannot open include file: 'CL/cl.h': No such file or directory	d:\apps\crypto\nheqminer-windows\ocl_xpm\opencl.h	12	1	ocl_xpm
Error	106	error LNK1181: cannot open input file 'pthreads\pthreadVC2.lib'	D:\Apps\Crypto\nheqminer-Windows\cpu_tromp\LINK	cpu_tromp
Error	107	error LNK1181: cannot open input file 'cuda_tromp.lib'	D:\Apps\Crypto\nheqminer-Windows\nheqminer\LINK	nheqminer

I will have to update the README file. What is/are your target platform devices (CPU/NVIDIA/GPU)?

Thanks. Target is CPU/AMD.

@skive OK. I have updated the master branch again to fix 3rdParty libs. If you don't want to compile against CUDA you can go to nheqminer C/C++ > Preprocessor properties and remove USE_CUDA_TROMP. If there are any more issues let us know on this thread. If you get things building then close it. Good luck.

there is no USE_CUDA_TROMP at stated area

Hm... can you please double check. Make sure you have solution configuration set to Release when you build that then build with ReleaseSSE2.

i was changed my os to ubuntu,now revert back after your message and now im downloading visual studio 2013 community.i will let you know when i test.sorry again.

No problem... no need to apologize.

Platform: Windows 10 x64, Visual Studio Community 2013
Target architecture: avx2 cpu's, nvidia, amd
Configuration: Debug
Build runs successfully, but nheqminer show's 0 sol/s always
Configuration: Release
Error	56	error LNK1181: cannot open input file 'cpu_tromp_SSE2.lib'	D:\Apps\Crypto\nheqminer-Windows\nheqminer\nheqminer\LINK	nheqminer

Configuration: ReleaseSSE2
Error	56	error LNK1181: cannot open input file 'cpu_tromp_AVX.lib'	D:\Apps\Crypto\nheqminer-Windows\nheqminer\nheqminer\LINK	nheqminer

Last commits fixed this. Thank you for help.


Paket update does not update .csproj file correctly anymore (Regression)

Description
We encountered a new problem with one of our packages. When we pulled on it, paket did not add a reference to the dll in the project file.
Repro steps
Here is a repro sample;
https://github.com/franknarf8/paket-issue-repro/tree/b363bf2d50031e71ff9bb3a39ca471b44b64f8c7
Here, we are trying to install WindowsNative in the project Sandbox2.0.
The commands listed below do not have the same effect on the .csproj file. We were expecting the old behavior. If the new behavior is the correct one, why is it so?
Expected behavior
When we ran : .paket\paket.exe.old.exe update (<IP_ADDRESS>)
Paket used to create a reference to that dll in the .csproj file.
Actual behavior
Now, when we run : .paket\paket.exe.new.exe update (<IP_ADDRESS>)
The reference is removed from the .csproj file.
Known workarounds
Link directly to the .dll (bad workaround)

Ok I tried to reproduce. which lib is missing?

the wk-component/WindowsNative package seems empty after extraction

oh I just extracted it with 7zip:

so it looks like unzip is broken here. WTF!?


Duplicate ParquetNodeToType()

https://github.com/polarsignals/arcticdb/blob/450025ff363cb7241d9a038dcaf74b8f2779f2ac/pqarrow/arrow.go#L379
https://github.com/polarsignals/arcticdb/blob/0c97de2410a804308b52574cdb1d5073d436ae64/query/logicalplan/expr.go#L127
There are two ParquetNodeToType functions in pqarrow and logicalplan with the same name and the same functionality, so it would make sense to make one function to be used around.
However, there are perhaps two problems with it.

From its functionality,  pqarrow package would be the appropriate place ( it means parquet-arrow, doesn't it? ), but it imports logicalplan, which causes an import cycle.

To avoid using panic(), we need to change to DataType(*dynparquet.Schema) (arrow.DataType, error) in the expr interface.
https://github.com/polarsignals/arcticdb/blob/1bd737630d2c68487961bd7d52c50503924e5ad9/query/logicalplan/builder.go#L82

Any thoughts?

From its functionality, pqarrow package would be the appropriate place ( it means parquet-arrow, doesn't it? ), but it imports logicalplan, which causes an import cycle.

Yes, that's exactly why we initially duplicated it.

To avoid using panic(), we need to change to DataType(*dynparquet.Schema) (arrow.DataType, error) in the expr interface.

Perfectly happy with that. The panic was actually a left over from prototyping when we wanted to fail hard in cases that we didn't fully implement through.
Maybe we can just extract it into another package to break the cyclic import? Maybe a convert sub-package under pqarrow?


26: Fix error output in json2ts.sh

Close #26

@SSSKrut I made the following changes:

use --no-deprecation instead of  NODE_NO_WARNINGS cause it's more precise and general warnings can be useful
remove waiting of intermediate jobs (jobcount %30 this crap), we don't have so much models
increase sleep time so it works properly on my machine

@SSSKrut can we close it?


Add timer

I want to use some delayed action in my Java -> Javascript code.
This is why i want to implement the following classes:
java.util.Timer
java.util.TimerTask
The javascript (not sync version) equivalent is setTimeout and setInterval.
The code will only use the Date class functionality.

This is related to #32, so need the #33 to work this correctly


Modifies dragging logic to be more graceful and enforce separation of concerns

Moves event listener up to the window scope to remove extra logic revolving the cursor leaving the shape too quickly.

 Kirby: Change event handling for DraggableShapes


configure website http://www.bitcoin101.xyz to work with GitHub

This issues was dropped from the list due to lack of team consensus on the importance/need of the described feature


Transforming jest tests to migrate to latest API

Just landed plugin that will simplify transaction to latest version of jest with help of code transformation using Putout.
What it does is changes:
jest.addMatchers({
    fail: (a, b, c) => {
        return {
            pass: false,
        };
    },
});

jest.resetModuleRegistry();
jest.runTimersToTime(1000);

Into:
expect.extend({
    fail: (a, b, c) => {
        return {
            pass: false,
        };
    },
});

jest.resetModules();
jest.advanceTimersByTime(1000);

Thank you guys for such a useful test runner :)!

Hey @coderaiser, sorry I missed this! Wanna send a PR to awesome-jest adding a link? I think it makes sense under "migration", but maybe not? https://github.com/jest-community/awesome-jest/blob/main/README.md#migration


Warn when frames are dropped

Redo of #792 because it was undone by a merge or rebase

Oh sorry about that. I thought it had been merged and then removed on accident, but sounds like that wasn't the case

No issues, we appreciate your contribution. However, we cannot merge this at the moment


⚠ Bofrost.de has degraded performance

In b05ae43, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1860 ms

Resolved: Bofrost.de performance has improved in 2e7332c.


Accessibility: alerts are missing headings and an indication that a message is an 'alert'

Current alerts used on Canada.ca are often missing headings and are skipped by screen readers when one pulls out headings and links.
The alert governance does not include a requirement to have:
-a heading
-invisible text to explain what type of alert this is
There are also 4 different documents providing guidance on the use of alerts

https://design.canada.ca/common-design-patterns/contextual-alerts.html
https://design.canada.ca/crisis/alerts.html
https://blog.canada.ca/2020/07/15/alert-fatigue.html
https://docs.google.com/presentation/d/1kpDFOVUAW1YdWMGZdmZaG1ksRXOCS9aHb1mnrvIHcrE/edit#slide=id.g71f30593e9_0_10

I've combined the governance from the three and included additional governance to address accessibility issues here:
https://docs.google.com/document/d/1JkjLhLWrT989QcrHiXJDaseBEUB9JB2NIse8ztqx6vc/edit?usp=sharing
We should encourage different departments to test their alerts and share research evidence to improve this governance.

This issue should be folded in with a visual presentation of how the component should be used (How to implement) https://github.com/canada-ca/design-system-systeme-conception/issues/107.

Is there an opportunity to create another segment in the Design System that lists the component Anatomy, similar to this https://www.carbondesignsystem.com/components/notification/usage/#main-elements.
This is where the elements of the component can be shown. It's a quick way to show what the essential pieces are. In the case of alerts, a heading will be essential.

Hi Ksenia - The new model we were working towards had something very similar: https://design.canada.ca/components/alerts.html

That's great, thanks for sharing David!
Also, this discussion was just shared with me https://github.com/alphagov/govuk-design-system-backlog/issues/2#issuecomment-673356072 it highlights the importance of 'verbal indication' for each colour of banners due to the difficulty in differentiating between the types of banners.
Here the suggestion is to use "role" to describe the type of message. Status for green, Warning for red, for example.
Also, there might be an issue about placing banners at the top of the page, even with headings, quote:
"Clarification: When a screen reader user uses the list of headings they would not miss it, the banner heading will be in that list. But it's likely they would miss it if they skip to the h1 first as a means to skip to the main content and then start reading the content from there (or skip to the next heading etc)."


getting started optimisations

is missing how to enable termguicolors
Plug 'ishan9299/modus-theme-vim', {'branch': 'stable'} -- only if you are not using nightly I guess one can remove branch stable, if one is not on nightly. This could be more explicit in the formulation.
recommended init.lua setup tutorial would be great
use 1 setup option and move the other to package_setup.md (with linking to it)

is missing how to enable termguicolors

The colorscheme will work even if termguicolors is not set but it is enabled automatically maybe because of colorbuddy.

recommended init.lua setup tutorial would be great

The README does state how to set the colorscheme and it is in lua.
Also will update the README on how to setup the colorscheme.

is missing how to enable termguicolors

The colorscheme will work even if termguicolors is not set but it is enabled automatically maybe because of colorbuddy.

recommended init.lua setup tutorial would be great

The README does state how to set the colorscheme and it is in lua.
Also will update the README on how to setup the colorscheme.


SS-3009 Do not use a timeout when the request is associated with a co…

…ntext.

Code Coverage is 82.3%, details at https://s3.amazonaws.com/rs-code-coverage/rsc/2016-01-26_560_Coverage-82.3.txt (sometimes the URL can take a few minutes to be available)


Ingress example not working

Hello,
The ingress example isn't working because now the api used is the networking v1 and the syntax doesn't seem to be recognized
Tested with GKE 1.21.5
Thank you.
Name:             hello-kubernetes-ingress
Namespace:        hello-kubernetes
Address:
Default backend:  default-http-backend:80 (<IP_ADDRESS>:8080)
TLS:
  hello-prod terminates XXXXXXXXXXX
Rules:
  Host        Path  Backends
  ----        ----  --------
  *
              /hello-world(/|$)(.*)   hello-kubernetes-hello-world:80 (<IP_ADDRESS>:8080,<IP_ADDRESS>:8080)
Annotations:  acme.cert-manager.io/http01-edit-in-place: true
              cert-manager.io/cluster-issuer: letsencrypt
              kubernetes.io/ingress.class: nginx
              nginx.ingress.kubernetes.io/rewrite-target: /$2
Events:
  Type     Reason          Age                   From                      Message
  ----     ------          ----                  ----                      -------
  Warning  Rejected        <invalid>             nginx-ingress-controller  hello-kubernetes/hello-kubernetes-ingress was rejected: with error: spec.rules[0].host: Required value
  Warning  Rejected        <invalid>             nginx-ingress-controller  hello-kubernetes/hello-kubernetes-ingress was rejected: with error: spec.rules[0].host: Required value
  Warning  Rejected        <invalid>             nginx-ingress-controller  spec.rules[0].host: Required value
  Warning  Rejected        <invalid>             nginx-ingress-controller  spec.rules[0].host: Required value

The ingress example probably shouldn't be there at all as Ingress configuration is typically cluster-specific.

My adapted / working example on K3S 1.23
`# hello-kubernetes-ingress.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: hello-kubernetes-ingress
annotations:
kubernetes.io/ingress.class: nginx
nginx.ingress.kubernetes.io/rewrite-target: /$2
spec:
rules:

http:
paths:

pathType: Prefix #Exact
backend:
service:
name: hello-kubernetes-hello-world
port:
number: 80
path: /hello-world(/|$)(.*)
pathType: Prefix #Exact
backend:
service:
name: hello-kubernetes-custom-message
port:
number: 80
path: /custom-message(/|$)(.*)`


add whitelist for lambda code

add whitelisting rule for lambda aws console code

:+1:

Codecov Report

Merging #526 into master will increase coverage by 0.04%.
The diff coverage is 94.11%.

@@             Coverage Diff              @@
##             master     #526      +/-   ##
============================================
+ Coverage     55.52%   55.56%   +0.04%
- Complexity      533      535       +2
============================================
  Files           136      136
  Lines          3112     3115       +3
  Branches        213      214       +1
============================================
+ Hits           1728     1731       +3
  Misses         1303     1303
  Partials         81       81

Impacted Files
Coverage Δ
Complexity Δ

...ndo/stups/fullstop/plugin/lambda/LambdaPlugin.java
82.5% <94.11%> (+1.41%)
9 <2> (+2)
:arrow_up:

Continue to review full report at Codecov.

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

:+1:

:+1:


[usbdpi] Introduce fault reporting

Description
In many places the DPI model (now) catches unexpected data/timing/behavior in the USB traffic to/from usbdev but does not cease operation and indicate failure. In a few places it still has assertions that are not directed purely at catching faults in its internal logic.
Any fault that may be induced by traffic on the USB, rather than being purely an internal failure, shall be signalled to the test, such that the software on the simulated Ibex CPU may terminate the test with a failure status in the normal manner. This mechanism is yet to be decided, and a couple of approaches may be required, but perhaps a vendor-specific control transfer to endpoint zero may be used to return the test status, fault code and ASCII text description (including file/line within usbdpi), at least for higher-level faults.
It is suggested that perhaps a bus reset to the device to force it, and more importantly the usb_testutils software, into a known state before transmitting the control sequence, may be a good way to ensure that this mechanism is more robust, since this is clearly a fairly high-level operation, requiring the USB itself to be functioning.

estimate 8

The remaining effort here can be considered a V2 effort.

Discussed in triage, moving to M5 because this seems to be a cleanup-only task


Added custom dropdown in the search panel

Fixes #1865
Describe the changes you have made in this PR -

Added class name and cursor: pointer to the search icon so that it looks clickable.
Added custom drop-down menu styles so that it looks the same in all the different browsers.

Screenshots of the changes (If any) -
Chrome (Before)

Chrome (After)

Firefox (Before)

Firefox (After)

@dipanshparmar you branched from new_UI and created the PR to master branch. What you should be doing is branch from master branch and create a PR to master branch. Thanks!
Will close this PR for now

Coverage decreased (-0.06%) to 87.962% when pulling ee69aef219a590c980107ee2b3acee9f3a57b052 on dipanshparmar:new_UI into 74b330a6b9a1513648faea2fa51f1619c80de3c7 on CircuitVerse:master.

@dipanshparmar you branched from new_UI and created the PR to master branch. What you should be doing is branch from master branch and create a PR to master branch. Thanks!
Will close this PR for now

@satu0king do you want me to merge all the changes to master and then make a pull request direct to master from my master branch?

@dipanshparmar I suggest re creating the whole PR from scratch.

Yeah @satu0king, I'm looking forward to do the same but can you please guide me that which branches should I use for my PR. I mean which branch should I choose at my side to make the changes and to which branch should I push the changes to CircuitVerse side? It is not mentioned in contributing.md so I'm little confused.

Do this,

Make sure your master branch is upto date with CircuitVerse master
Create a new branch from your branch all it fix/custom-dropdown
Make your change in fix/custom-dropdown and commit
Send a PR!

😄

Do this,
1. Make sure your master branch is upto date with CircuitVerse master

2. Create a new branch from your branch and call it fix/custom-dropdown

3. Make your change in fix/custom-dropdown and commit

4. Send a PR to our master branch from fix/custom-dropdown!

smile

Thanks :smile:, it is clear now.


I'm getting "failed to receive data from Create. Check if robot is powered!" error

Hi
I'm using create2 robot. To start the robot I use create_2.launch from ca_driver. However, I always get  "failed to receive data from Create. Check if robot is powered!" error. I also tried to switch on the robot by pressing the middle button, but it didn't work.
However, if I remove the battery from the base and insert it back, it magically starts working. But after sometime again it stops working. Is anyone facing the similar issue ?
Is there something I'm doing wrong?
Thanks for your help

I am having the same issue with create 1. I will try the battery trick and get back to you though.

i faced to the same issue several time during my project. the battery was
removed and was fitted to solve this problem
Thank you.
Best Regards,
Isanka Diddeniya
S. I. A. P. Diddeniya
Research Assistant
Department of Physics,
Faculty of Applied Sciences,
University of Sri Jayewardenepura,
Gangodawila, Nugegoda,
Sri Lanka.
B.Sc. Honors (USJP)
Tell : +94712869344, +94767155757
On Tue, Nov 13, 2018 at 4:59 PM Shrijal Pradhan<EMAIL_ADDRESS>wrote:

I am having the same issue with create 1. I will try the battery trick and
get back to you though.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AutonomyLab/create_autonomy/issues/54#issuecomment-438233553,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AboyMjOnNVcbyWXgfK91QNHIlehOlzT1ks5uuq0ogaJpZM4XWnOm
.

Still now working for me. I am using Ubuntu 16.04 on a RaspberryPi 3B and ROS Kinetic. Could that be a problem?
I have tried to connect it with a PC with the same version of ROS and Linux but to no avail.
Can it be anything to do with the baudrate the systems are suppose to communicate in?

Can it be anything to do with the baudrate the systems are suppose to communicate in?

This could be the culprit. You could try changing the baud argument for the driver or on the Create itself (see the manual for instructions). The Create communicates with 57600 or 19200 (default) baud.

For me all it took was to uncomment the baudrate parameter line in the xml. Left everything else 'as is', even the baud rate.  I'm using a Roomba 560 running on a Pi 3.

I have encountered a similar issue on 18.04 + ROS Melodic where the driver would fail to connect with serial error - Operation cancelled.
The solution in this case was a fix in libcreate:
In libcreate/include/create/serial.hpp:
#include <boost/enable_shared_from_this.hpp>
class Serial : public boost::enable_shared_from_this<Serial> {
and in libcreate/src/serial.cpp, replace all uses of
boost::bind(&Serial:onData, this, _1, _2)

with
boost::bind(&Serial::onData, shared_from_this(), _1, _2)

I am unable to reproduce this issue with a Create 1 platform on Ubuntu 18.04, but I would be willing to accept autonomylab/libcreate#38 if others on this thread can confirm it fixes the issue for them.

I'm going to close this with https://github.com/AutonomyLab/libcreate/pull/38 merged. Let me know if there's still an issue.


Feature request: Vitepress-style callout markdown support

Hello! I’m a new user to Nextra but it is hands-down the best React-powered docs solution I’ve run across. I love how quickly it is to get up-and-running with almost no config, and I love how great everything looks, feels, and works ❤.
Feature Request
Nextra has Callouts, but requires importing the component manually. Vitepress, instead, has a more automatic built-in syntax:
::: info
This is an info box.
:::

::: tip
This is a tip.
:::

::: warning
This is a warning.
:::

::: danger
This is a dangerous warning.
:::

::: details
This is a details block.
:::

This is not only easier to compose; it also works in regular ol’ .md files for more flexibility / portability. And not that the Nextra project needs to be compatible with Vitepress or anything, but in general the more parity this project has with others, the easier it is for people to migrate to it.
If this syntax seems inline with the project, I’d love to add a PR hooking up this syntax with the Callout component.

Thank you for your feedback, I am not fun of this syntax, but if you want to work on this feature feel free (considering use v3 as the base branch since I do not plan to add new features to v2)

The upcoming twoslash feature may satisfy this request.

The upcoming twoslash feature may satisfy this request.

Ooh I love that too! As long as you can place components or other markdown within it (I think that’s a major reason for the ::: blocks. But that looks even cleaner.

Hey, this feature has been added in<EMAIL_ADDRESS>