Use command -v to detect git

Version details:
Version: 1.39.1
Commit: 88f15d17dca836346e787762685a40bb5cce75a8
Date: 2019-10-10T23:35:11.314Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0

What I expect to happen
VS Code's Git extension finds + uses Git
What actually happens
VS Code says it cannot find Git:

Steps to reproduce
TL;DR: VS Code should use command -v to detect commands, not which.
This is a bit tricky, so bear with me:
There is a node package called which:
https://www.npmjs.com/package/which
If this package is installed through Node, and a user is using asdf, this which-from-node can take precedence in $PATH. This will cause issues if there is no default Node version specified for asdf. An error like this can appear:
asdf: No version set for command which
you might want to add one of the following in your .tool-versions file:

nodejs 8.14.0

(I suspect this is only an issue on 'zsh' terminals -- I am unable to reproduce it in Bash)
To work around this which-conflict, we should use the built-in 'command' command. This comes standard in every shell program and is a viable alternative to 'which'.
When my shell works correctly, here's the output of which -a which:
which: shell built-in command
/Users/ryanbigg/.asdf/shims/which
/usr/bin/which

However, when it is misbehaving, I suspect the order is like this:
/Users/ryanbigg/.asdf/shims/which
which: shell built-in command
/usr/bin/which

And that's why it's trying to use the node which. When no node version is configured for asdf, then it fails to run that node-which, which then causes this extension to claim that it can't find git.
This PR (likely) fixes #81287 -- although I'm not 100% sure of that.

👀

So... what happens if the user has a command command ahead in their PATH variable, just like which?

Closing.

So... what happens if the user has a command command ahead in their PATH variable, just like which?

I believe that wouldn't be a problem because the shell builtin command should take precedence over things from $PATH.
From bash manual: Shell Builtin Commands

When the name of a builtin command is used as the first word of a simple command (see Simple Commands), the shell executes the command directly, without invoking another program.

Isn't which a builtin command?

Isn't which a builtin command, as was mentioned above?

In csh which is indeed a builtin, but not in sh/bash, @joaomoreno

Got it. Keeping my opinion: this is not worth the trouble.


fix documentation

At https://pruvious.com/tutorial/01-project-setup
This line doesn't open the browser automatically.
To fix it: npm run dev -- -o instead of npm run dev -o

Fixed. Thank you.


Don't call RecyclerView methods in setTimeout call when component is already unmounted

Relate to any issue?
Nope
Breaking change?
Nope
What this PR does?
Because the following code is happening in setTimeout call:
this._recyclerListView._pendingScrollToOffset = null;
this._recyclerListView.scrollToTop(false);

what can happen and actually happen to us is that the component itself might be already unmounted but the setTimeout callout will still happen trying to access those fields and crash.
This was happening in following situation:

React-Native-Emoji-Picker is opened in Modal Window.
User starts typing in search input.
User immediately hits back button on Android to close the modal.
Component gets unmounted but setTimeout call is already queued so it will happen anyway.

That if just guards access to those fields only if component is still mounted.

I would suggest renaming the value to _isMounted which is given by the react tutorial.

Will push change today / tomorrow.

Have to close this one in favor of:
https://github.com/sskhandek/react-native-emoji-input/pull/33
Since I don't have access anymore to organization account from which I have created original PR


I think its time to move on.

The all-famous 2303 Controller is no longer easily available in the market.
Does it mean that this "Flaw" has been fixed in the more common 2307?
No one in 2 years have worked on Phison 2307 controllers and i doubt if someone is going to.
Conclusion : BADUSB vulnerability has been fixed

"BadUSB" is not a vulnerability -- it is the ability to alter the firmware on a USB flash drive, and that most certainly has not changed on the 2307.
What prevents people from easily using this on the 2307 (and even some 2303s) is the patches, which need to be tweaked to work properly on different firmware versions. Rather than simply hard-code patch addresses, the code in this repository tries to be a tiny bit smart and look for code patterns, so that there'd be some hope of it working on other firmware versions, but it does not work well enough on the 2307 or some 2303 versions.
Studying the disassemblies and making the appropriate changes is all that's necessary -- and I've seen links on this very issue tracker (such as https://github.com/brandonlw/Psychson/issues/137 ) of people who have done exactly that, so it is certainly possible.

Thank You for finding time to reply.
i never said it is not possible, but no one is committed enough to work further on this and more importantly on other controllers.
only if the developers who discovered this 'ability' could find time for this project and somehow tweak through the changes for 'further analysis and research'

Also i went thoroughly on the Japanese guy who sorted this out, but he is not actually providing documentation or the proper walk-through of how to do this on 2307


191-teampage

Fixes #191

Excessive space at the top of team page
**before

**after

Font weights of content are not looking appropriate
**before

**after

Space required between Our team heading and the content
**before

**after

Excessive Margin between our team section and board section
**before

**after

Margin After the last Show all button and the footer
**before

**after

@smaranjitghose Hey :) Kindly check!

@anushbhatia Hey! Why is this PR labeled with the easy tag?
Before this for opening a new issue and working on the changes my PR was labeled as medium or hard by @smaranjitghose
Here in this PR I have made 5 changes.
Kindly review :)

Only Bugs were resolved and enhancement was done. When u do a major push in the repo like changing the UI and making it more attractive that gets a medium tag or you create a custom-designed element.
That complete page was given a hard or medium tag not the individual problems.


When starting jupyter the conda environment is  not activated

Select a conda environment as the environment for Jupyter server.
Ensure auto start is turned on
Open a python file.
When debugging I found that the conda environment is not activated when auto starting jupyter.
I think this is a problem even during regular starts.

Validated.


fix: curl retry with exponential backoff

Retry public key test if HTTP response fails.  With these options, curl will retry 14 times, with exponential backoff, for a total duration of ~1 hour.  Each attempt will fail if it takes longer than 2 seconds to connect or 5 seconds to receive the key.
Partially fixes #100 and #102.
Reverts #94 and #103.

That’s a nice shortcut … removing -s on curl likely exhibit everything the end user needs. 👍🏻


Fix dropSlotHeight null check to allow a height of 0

Use a strict check for undefined. Fixes #489

I made the change in my local environment. Thanks for the PR!


Add some sort of implementation of a Preview/Debug aware CacheTagHelper

Hi @warrenbuckley this is what I had hacked together following our conversation about adding an Our Umbraco Tag Cache Helper that was at least aware of Preview and Debug modes, and which maybe cleared it's cache whenever something got published, as per Umbraco.CachedPartial - a very blunt mechanism, but kinda what Umbraco people are used to having.
This doesn't really clear the Cache, just varies the cache key, when something is published, which is a bit of a hack / how Dot Net expects you to expire a Tag Cache Helper! I do not know!
But still think it's useful to have a Tag Cache Helper that is aware of Preview and Debug mode
See what you think anyway!
There is hopefully a 'scribe' you can access here, which shows how to test it...
https://scribehow.com/shared/Workflow__2aX25AlzSAWrzDYuXC3l-Q

@marcemarc let me know when you try the new changes we talked about and collab'd on over Discord are working out for you?

Looks good to me :)

@warrenbuckley @marcemarc  Do we have a date for this to be merged in?


Content in  not rendered correctly

In markdown, we could sometimes have
<details>
<summary>Summary of the details</summary>
- detail 1
- detail 2
- detail 3
</details>

rendered like this:

Summary of the details

detail 1
detail 2
detail 3

However, it's actually rendered and display as plain text... (actually not 100% sure if it's a theme specific problem 😳 )
Thank you!!

I think this is how kramdown works – once it enters an HTML tag (in this case, <details>) it stops parsing the content as markdown. So you'll need to use <ul> manually.

Thank you for the explanation!
For people who landed here, I managed to make it work by adding markdown: GFM to _config.yml. While it's working for me, I'm guessing this might have other trade-offs; but might worth trying... 🤔
(I found out about it from here: https://docs.github.com/en/pages/setting-up-a-github-pages-site-with-jekyll/setting-a-markdown-processor-for-your-github-pages-site-using-jekyll)


Gitignore *.json.publisher_v2.valid files

These seem to be a new addition. We couldn't work out why they were
polluting our git status when we were generating new schemas and why
we'd never seen anything like them before, then we found this file and
determined they'd been missed when they were added.

I suspect its a result of https://github.com/alphagov/govuk-content-schemas/pull/164

Seems good to merge to me, @danielroseman?

:+1: but needs rebasing.

Rebased!

Thanks @davidbasalla for the rebase! :star:


how to visualize multiple generated obj objects Simultaneously

Dear Prof；
Thanks for your excellent paper. I just wanted to know how to visualize the multiple generated objects simultaneously just like your paper Figure 1

Or can you provide the colab script for your scene generation in paper figure 1?

For Figure1 all the shapes are individually generated and then loaded and placed in a Blender scene manually by me - they are not simultaneously generated


Community usage and support

Hey @BigstickCarpet, I wanted to say thanks for this publishing this project.
We have been using it for years to validate and bundle dozens of yaml files into our various OpenAPI/Swagger specifications.
It still works great and we would love to see it continue to evolve and support the upcoming specification changes.
I haven't looked since last fall, but I have not found other tools that do quite the same thing.
cc @alexvolsh

Hi @stevenscg.  Thanks for your kind words.  I know it's been a long time since I've done any work on this project.  Same goes for my other Swagger-related projects too.  The reason is that when I originally wrote them, I was being paid to do it as part of my day job because my employer needed them.  But I've changed jobs since then, so now any work I do on these projects is limited to my personal time.
That said, with the recent release of Swagger 3.0, I'm now eager to update all my Swagger projects to support the new 3.0 spec (in addition to continuing to support v2.0).  So hopefully you'll see these projects come back to life soon

Sounds great!
We are still on OAI 2.X, but will be monitoring the spec progress and updating as well.
@ mention me for testing help or feedback if you need it.
I think I marked a few of the open issues as being potentially handy for us. I'll try to +1 those when I find them.

I just released Swagger-CLI v1.0.0, which includes the latest version of Swagger-Parser.


Sidebar out map div

Hi eb,
Can I put the Sidebar out of th map div?
I tried, this is my webmap with sidebar inside the map div, everthong works:
www.mainjoin.eu/map/sidebar_in_map_div
And this is the webmap when I put the sidebar out the map div:
www.mainjoin.eu/map/sidebar_out_map_div
But here the point and polygon in map disappear.
My goal is confine the sidebar in a 50% left div:
div id="sideleft" style="width:50%;float:left;"
and confine the map in a 50% right:
div id="map" style="width:50%;float:right;"
Thanks for any tip
Valerio

Hi eb,
Can I put the Sidebar out of th map div?
I tried, this is my webmap with sidebar inside the map div, everthong works:
www.mainjoin.eu/map/sidebar_in_map_div
And this is the webmap when I put the sidebar out the map div:
www.mainjoin.eu/map/sidebar_out_map_div
But here the point and polygon in map disappear.
My goal is confine the sidebar in a 50% left div:
div id="sideleft" style="width:50%;float:left;"
and confine the map in a 50% right:
div id="map" style="width:50%;float:right;"
Thanks for any tip
Valerio

hello @valeriodeluca  , i have a question about your webmap which i found so brilliant , my question is how you made the content of the sidebar change dynamically when you click diffrent marker, i have been trying doing it but i can't figure out a way, thanks

Bonjour, je voudrais ajouter du contenu dynamique. pouvez vous nous aider ???
Merci d avance


error: invalid use of incomplete type 'PyFrameObject' {aka 'struct _frame'}

Hi. Unfortunately it fails to build Pangolin in the final step which is ./install_dpvo.sh. Could you please double check if your instruction is correct at the moment. Thank you. The following is a screenshot of the error report:

@jiaqingxie
Hi, I got the same error. here is my solution.

Problem

PyFrameObject was removed in python 3.11.
Python version of (dpvo) env is 3.10, while (base) env in the docker container is 3.11.
Then cmake .. in ./install_dpvo.sh selected wrong interpreter of (base).
-- Selected Python: '/root/miniconda3/bin/python3.11'. cmake --build . -t pypangolin_pip_install to use pypangolin module.

Solution
Downgrade python version of (base) to 3.10, and retry ./install_dpvo.sh

conda deactivate
conda install python=3.10
conda activate dpvo

Now cmake .. successfully selected the interpreter of (dpvo), showing
-- Selected Python: '/root/miniconda3/envs/dpvo/bin/python3.10'. cmake --build . -t pypangolin_pip_install to use pypangolin module.

I could run the demo with this modification!

Thank you for the suggestion, I have also found that its being related to the version of CMAKE. Because its been rewritten in the header file. The downgrading of python env also means the downgrading of cmake version, which works for me.


Standardize releases pipeline

We should standardize our releases pipeline. Right now, Github releases are usually not made prior to PyPI publishing and the github action for automatic PiPY publishing is not working.
I'm opening a PR where we can discuss possible ways of doing that.

So, for v0.8.1, you had to manually upload that to PyPI, right? The GitHub release didn't automatically handle it?

@CodyCBakerPhD that's right


fix(PageHeader): Correctly add display prop

Closes: https://github.com/github/primer/issues/4481
Changelog
Changed

Fixed bug for incorrectly displaying display prop for title

Rollout strategy

[x] Patch release
[ ] Minor release
[ ] Major release; if selected, include a written rollout or migration plan
[ ] None; if selected, include a brief description as to why

Integration tests are failing due to unrelated errors


Clean up the nixos option tree

Is there a guideline on where to put new options?
I can find categories like ec2, gnu, qtPlugins and zramSwap in the option root, on the same level as e.g. hardware, nix and lib.
Apart from fulltext option search it is hard to find anything in the current structure.
I’m all for flat hierachies, mind you, but designed systematically.

Ouch, that's very ugly. Those definitely should not be at top-level.

A strategy for the cleanup is needed. Since renaming options causes a lot of user breakage, a sensible deprecation model is needed first. So this depends on https://github.com/NixOS/nixpkgs/issues/15357

There are still zramSwap, gnu, dysnomia and powerManagement that look out of place to me. Do we have a deprecation model by now ?

I’ll close this for now, because I don’t think it’s an extremely important thing to do. Grepping for keywords is much easier in any case.


span enrichment tests added and validated

Added simple heuristic tests for the metadata, metrics and feedback setting methods.

running tests locally to validate no regressions were introduced


allow valid set() calls without a callback to proceed.  These were fa…

…iling silently for no reason other than the fact that a callback was not provided.
This fixes issue #248

I will note that the build failures are not with my test/code.  I was getting these same failures with a fresh install.

@3rd-Eden Any updates on this PR?

This just burned me as well. I agree that the callback should be optional but regardless silent failure isn't the right behavior.

@ronkorving @3rd-Eden  Just FYI, I've been running this in my production services without issues.
If you see the posts lots of people keep hitting this bug.  If there's anything else I can provide in the pull request please let me know.


System wide rc / non interactive rc.

People need a place to setup module paths and other things in a system wide rc script that is run even on non interactive scripts.
We should probably just study how POSIX shell does it, and imitate that.

Done.


prescient--save does not set print-length, or print-level.

I was testing out the prescient--save facility, and discovered this code was written:
(funcall prescient-cache-callback :version (quote 5) :history (quote #s(hash-table size 65 test equal rehash-size 1.5 rehash-threshold 0.8125 data ("ivy-prescient-mode" 2 "prescient.el" 1 "prescient--save" 0 ...))) :frequency (quote #s(hash-table size 65 test equal rehash-size 1.5 rehash-threshold 0.8125 data ("ivy-prescient-mode" 0.991026973 "prescient.el" 0.994009 "prescient--save" 0.997 ...))) :serial-number (quote 3))

The specific problem is the :history value -- as you can see, the list is terminated with the string ..., which is the result of the print-level variable truncating the output; you can demonstrate this yourself with:
(print (cl-loop for n from 1 to 50 collect n))
=> (1 2 3 4 5 6 7 8 9 10 ...)

To correctly save things, set both print-length and print-level to nil, which allows unlimited volumes of content to be produced.  (This controls the number of elements in a single list, and the number of nested list levels, respectively, before they truncate)
(cl-letf ((print-length nil) (print-level nil))

I implemented this locally with this advice:
  (define-advice prescient--save
      (:around (real-function &rest args) :name "save-correctly")
    "Force `print-length' and `print-level' nil to correctly
write all the content correctly in the code."
    (let ((print-length nil) (print-level nil))
      (apply real-function args)))

This now correctly saves the content, without truncation.  I'd suggest applying that code ASAP, as the default print-length of 10 is going to truncate for most people...

Thank you for the report. I applied the change you suggested.
However, in my emacs -Q 26.1, the default value of print-length is nil (this explains why I have not encountered any problematic behavior with prescient.el). Was the default value non-nil in some other version of Emacs, or is there something I am missing?
Also, thank you for informing me of the existence of define-advice. Very neat!

It occurred to me that your use of define-advice seems wrong. The argument spec is
(define-advice SYMBOL (WHERE LAMBDA-LIST &optional NAME DEPTH) &rest BODY)

whereas your usage has NAME as a keyword argument. Just an FYI.

Oh, just so!  I'm impressed how resilient it turns out to be, since it worked fine and I didn't notice, but I'll fix that locally. :)
Thanks for that.  and yeah, nadvice is much more pleasant than the old system.


Remove version: FineprintSoftware.FinePrint version 12.10

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


Query not returning Tag info when using Daily or Hourly periodicity.

Here is the query:
"ccexplorer get aws -g DIMENSION=SERVICE,TAG=Name -e {end} -s {start} -m DAILY -l -p csv"
Running from Jupyter - see attached for an screenshot of output.

@japoeder are you getting the same issue when sending to stdout?  Have you configured  the tags as user-defined cost allocation tags?

So, one point I should clarify is that technically the output is $Name for every record, not blank as it appears in the attached image.  When the tag does show up I remove that from the string.  '$Name' shows up in every record with stdout as well.
Regarding the configuration - I think (and correct me if I'm wrong) the answer to your question is yes, because I'm able to see the expected output when setting the periodicity to monthly.

Is this an AWS-generated cost allocation tag? There is a limitation with the number of active tag keys per report which is 500 but in this case you have a single tag.
When you filter by a specific $Name are you seeing the tags?
"ccexplorer get aws -g DIMENSION=SERVICE,TAG=Name -e {end} -s {start} -m DAILY -f Name="<some app name>" -l -p csv "

@japoeder I would recommend either deleting those links or removing the account information contained within. I'm unable to recreate the issue you described locally but given that you can get the information when the granularity is Monthly I would be wondering if there is some nuance associated with the metering on your account. Is it possible that metering applied to your account is non-standard?  Feel free to fork this repo and try find root cause.

Thanks for the heads up!  Where would I find the metering info related to cost explorer?  I went down a rabbit hole trying to find it and didn't have any luck...

Hey - turns out I was able to get it working when I use the repo, but not the version installed with brew.  Thanks for your help on the matter.

Good to hear. Out of interest what version were you using , you can check with brew list --versions -v . thanks


[RFC] add test helper

We decided, that this feature doesn't provide the amount of benefit to the users that would justify the additional maintenance effort that is introduced by it. Hence this PR is closed and we won't further pursue the topic "testing helpers", but will rather focus on improving the documentation with regards to testing.


feat: make some ops new be const fn

Which issue does this PR close?
Rationale for this change

What changes are included in this PR?
make some new be const fn

Are there any user-facing changes?
No

Maybe should define a helper macro to reduce root related code.

We may not randomly mark fn new as const fn just because it can be. This is a one-way door that revoke the const modifier would be a breaking change.
Please explain why those fn SHOULD BE constant function.


Orbstack is crashing after migration via Migration Assistant

Describe the bug
On the new compute, Orbstack is getting very upset with a series of error:

It prompted me to fix some permission issue: sudo chown -R $USER ~/.orbstack/data

Then, it crashed consistently with:

 vmgr | time="12-11 17:08:55" level=info msg="forwarding SSH agent" sock=/private/tmp/com.apple.launchd.jSkYBZzcLa/Listeners
 vmgr | time="12-11 17:08:55" level=info msg="starting VM"
 vmgr | time="12-11 17:08:55" level=info msg="starting host services"
 vmgr | time="12-11 17:08:55" level=info msg="reading user environment variables"
2024-12-12T01:08:56.088158Z  INFO devices::virtio::block::debug: read id="vdb" offset=0, len=4096, isect_start=0, isect_end=4096 ok: sector_hashes=[13308243557328531395,<PHONE_NUMBER>0707039239,<PHONE_NUMBER>0240488170, 0, 0, 0, 0, 0]
 system |  [*] Booting OrbStack  (+0ms)
 system |   -  Kernel version: 6.11.10-orbstack-00282-g72f45320fe21
 system |  [*] Network  (+1ms)
 system |  [*] Apply system settings  (+1ms)
 system |  [*] Late tasks  (+0ms)
 system |      [*] Emulation
 system |      [*] Memory
 system | thread '<unnamed>' panicked at src/startup.rs:1473:38:
 system | called `Result::unwrap()` on an `Err` value: missing data partition: No such file or directory (os error 2)
 system |
 system | Caused by:
 system |     No such file or directory (os error 2)
 system | note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
 system |   -  Using Rosetta
 kernel | [    0.134765] zram0: detected capacity change from 0 to 32834416
 system | thread 'main' panicked at src/startup.rs:1488:21:
 system | called `Result::unwrap()` on an `Err` value: Any { .. }
 kernel | [    0.152302] Adding 16417204k swap on /dev/zram0.  Priority:32767 extents:1 across:16417204k SSDsc
 kernel | [    0.152645] Kernel panic - not syncing: Attempted to kill init! exitcode=0x00006500
 kernel | [    0.152665] CPU: 3 UID: 0 PID: 227 Comm: vinit Not tainted 6.11.10-orbstack-00282-g72f45320fe21 #14
 kernel | [    0.152678] Hardware name: orbstack,virt (DT)
 kernel | [    0.152686] Call trace:
 kernel | [    0.152690]  dump_backtrace+0x94/0xf0
 kernel | [    0.152701]  show_stack+0x1c/0x30
 kernel | [    0.152709]  dump_stack_lvl+0x48/0x88
 kernel | [    0.152715]  dump_stack+0x14/0x20
 kernel | [    0.152724]  panic+0x114/0x2e8
 kernel | [    0.152735]  do_exit.cold+0x378/0x438
 kernel | [    0.152749]  do_group_exit+0x90/0xb8
 kernel | [    0.152762]  get_signal+0x328/0x418
 kernel | [    0.152780]  do_signal+0x7c/0x878
 kernel | [    0.152794]  do_notify_resume+0x148/0x170
 kernel | [    0.152803]  el0_svc+0x4c/0x50
 kernel | [    0.152815]  el0t_64_sync_handler+0x40/0xe8
 kernel | [    0.152823]  el0t_64_sync+0x14c/0x150
 kernel | [    0.152829] SMP: stopping secondary CPUs
 kernel | [    0.154833] Kernel Offset: 0x2d8e43e00000 from 0xffff800080000000
 kernel | [    0.154846] PHYS_OFFSET: 0xffffddd700000000
 kernel | [    0.154850] CPU features: 0x00,00000002,00051528,020e7403
 kernel | [    0.154856] Memory Limit: none
2024-12-12T01:08:56.169067Z  INFO krun::machine: VM stopped
 vmgr | time="12-11 17:08:56" level=info msg="[VM] stopped"

Check logs for more details.```

Eventually, I was able to workaround by running:

/Applications/OrbStack.app/Contents/MacOS/bin/orb reset

### To Reproduce

_No response_

### Expected behavior

_No response_

### Diagnostic report (REQUIRED)

OrbStack info:
  Version: 1.9.0
  Commit: fa066983fe619a7cecfd01801b08681fae797750 (v1.9.0)

System info:
  macOS: 15.1.1 (24B2091)
  CPU: arm64, 16 cores
  CPU model: Apple M4 Max
  Model: Mac16,5
  Memory: 64 GiB

Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-12-12T01-17-21.340098Z.zip

### Screenshots and additional context (optional)

_No response_

hi! unfortunately, migration assistant doesn't work with our data.img: https://docs.orbstack.dev/faq#data-img

hi! unfortunately, migration assistant doesn't work with our data.img: https://docs.orbstack.dev/faq#data-img. OrbStack should work again if you delete ~/.orbstack/data/data.img. you can use tar to transfer the file from your old computer if you need to

👍 got it. I missed this section from the docs


Initial suggestions for plugin release

Thanks for this amazing plugin @graza-io 😄 . I have been playing around the tables and they were very informative 👍 . The structure looks great. We have a few minor suggestions before we can push the plugin to the hub, please let us know if you have any questions.

Could we use the following query in the README and the docs/index.md files:

select
  id,
  name,
  color
from
  clickup_team;

It is essentially the same as select * from clickup_team but I guess it would look better to have all the columns listed out. We can also use this query in social graphics(Simple yet powerful).

Could we add the contributing section at the end of the README file? Please refer to https://github.com/turbot/steampipe-plugin-pagerduty#contributing for more information

Could we add the following query in the docs/index.md file along with sample output:

select
  id,
  name,
  color
from
  clickup_team;

Could we also update # ClickUp + Turbot Steampipe to # ClickUp + Steampipe?

Could you please add the Credentials section in the docs/index.md file, mentioning the permission scope (if required) to query the tables?

The config/clickup.spc should look something like this:

connection "clickup" {
  plugin = "theapsgroup/clickup"

  # ClickUp API token
  # This can also be set via the `CLICKUP_TOKEN` environment variable.
  # token = "pk_xxxxxxx_xxxxxxxxxxxxx"
}

The formatting in the table docs is slightly different from what we follow across all the documents. Could you please update it as follows:

Existing:
### Get a folder by ID

```sql
select
    id,
    name,
    order_index,
    hidden,
    space_id,
    task_count,
    archived,
from
    clickup_folder
where
    id = '7fsd72'

Updated:
### Get a folder by ID

```sql
select
  id,
  name,
  order_index,
  hidden,
  space_id,
  task_count,
  archived,
from
  clickup_folder
where
  id = '7fsd72';

The semicolons are also missing at the end of query examples, could you please add them at the end of query examples?

The current table examples are great but they still feel a bit light. Could we have 1-2 additional example queries per table?

The table descriptions are missing, could we please add them? Currently, the inspect command returns the following result:

> .inspect clickup
+-------------------------+-------------+
| table                   | description |
+-------------------------+-------------+
| clickup_folder          |             |
| clickup_folderless_list |             |
| clickup_goal            |             |
| clickup_list            |             |
| clickup_list_member     |             |
| clickup_space           |             |
| clickup_task            |             |
| clickup_task_assignee   |             |
| clickup_task_watcher    |             |
| clickup_team            |             |
| clickup_team_member     |             |
+-------------------------+-------------+

I did check the SDK for pagination, but couldn't find any. Does the SDK support pagination, backoff, and retry functionality? Please refer to https://steampipe.io/docs/develop/plugin-release-checklist#data-ingestion for more information.

Overall the plugin looks great, after the above suggestions are addressed we can push the plugin to the hub 👍 .
Thanks a lot for the plugin @graza-io, eagerly waiting to push it to the hub 👍 .

https://github.com/theapsgroup/steampipe-plugin-clickup/commit/7e0179a315b16909a22bd5b6836f9f064f13ab3a - covers 1,2,3,4,5
https://github.com/theapsgroup/steampipe-plugin-clickup/commit/7dc1d9a13bf84b56229ca89280902e48f2cea2ed - covers 8
https://github.com/theapsgroup/steampipe-plugin-clickup/commit/1aa30af6d588db65eada0bf2c02a9d41a6b55afc - covers 6

9 - pagination is only really in one place https://github.com/theapsgroup/steampipe-plugin-clickup/blob/1aa30af6d588db65eada0bf2c02a9d41a6b55afc/clickup/table_task.go#L36

As for 7, more examples... that's where things become difficult, since all the tables (except teams) require a filter to be applied, you generally want all the records (even if you only want a subset of columns)... will think on it and attempt to get something in place soon.

I'd actually like to get #2 ironed out before we publish to the hub, since it'll be a breaking change to current code.

Some outstanding issues, albeit mostly with the underlying SDK, issue raised see here


Other small suggestions that could improve gameplay:

Firstly, I hope you're having a good start to the year, mi friend.
1.
Sound for Plant Death:
It would be good if you could add a sound for when a plant dies. This could be optional in the accessibility menu, but it could be very helpful in cases where a plant is crushed, killed by a zombie/plant in Zombotany, by a catapult, or squashed.
2.
Sounds for Zombie Movement:
Implement short and simple sounds for when zombies move across the columns. This should be a non-intrusive sound, as users will hear it frequently. It could be useful for better understanding zombie speed and improving strategies for beginners, such as knowing how to place a Potato Mine correctly without it being eaten before it's ready to explode. A brief, high-pitched click-like sound could work well to distinguish it from the deeper sounds typically used.
3.
Notifications for Garden Needs:
Whenever a plant has a need, play a sound to indicate it, and have a screen reader verbalize a message specifying which plant needs attention and what it needs. Additionally, propose messages related to the snail, like when the chocolate loses its effect.
4.
Locating Fire and Ice Balls on the Board:
This might be more challenging, but the idea is to allow players to locate the fire and ice balls thrown by Dr. Zomboss on the game board. This could help understand how much and how quickly the ball advances. Similar to finding zombies and other objects when navigating the grid, it would be beneficial if fire and ice balls could be located.
Thank you very much for reading.

Same to you :)
Those are all good suggestions, though a zombie movement sound might be too much.
Many zombies move at the same time, so it would be like the automatic sonar without a delay, at a much quieter volume level.
It may be too overwhelming for most players, but it may also be very useful for some players.
All the other suggestions are objectively useful for all players though, so are definitely being added to the todo list.
I'm planning on starting a mod for another game soon, so development on this mod will probably slow down a lot.
I won't stop working on this one, but it might be a while before the next feature update.

Thank you very much for considering my suggestions. By the way, there's something I'd like to ask you about. I saw this in the to-do list that you set for yourself: "SFX panning (zombie groans, impacts, etc.)." Do you have a specific idea in mind to achieve this, or is it just something you left open-ended as something you would like to achieve? Recently, I was watching gameplay of Plants vs. Zombies 2 and noticed that the game does have panned sounds. Peashooters fired from the left, and the zombie impact was heard on the right. Do you think you can do something like that? It sounds like it could be your own final boss in creating this mod, right? I hope you can achieve this, even if partially. By partially, I mean having three audio channels: left, center, and right. It might be challenging to make the zombie or object move through the stereo field, but perhaps it's easier if the sound plays on one of these three channels. I don't think it's essential for the sound to change channels while playing. The sounds in Plants vs. Zombies are short, and the gameplay doesn't have too much movement. Anyway, panned sound, while interesting as an experiment, is more than we could ask for, my friend. Thank you very much for the mod and the opportunity to translate it. I'm already working on a Spanish translation for my friends! By the way, there are some texts that don't seem to be in the English language files. Maybe I'll open a new # to explain it better later.

It's something I would love to add, but at the moment, I have no idea how to do it.
I'm not sure how the game plays sound effects, or where that data is stored in memory.
It will take a lot of time to reverse engineer the game, figure out if it's possible with the game's audio engine, then somehow adjust the left/right volume of sound effects before they're played.
I might need to intercept the audio files as they're being played, or heavily patch the game's audio engine to allow this.
Or maybe each sound effect already has a unique left/right volume, and it would be as simple as overwriting the volume levels in memory.
Without access to the game's code, it's hard to say if it would take me a couple of days, or a couple of months.
That's why it's a lower priority than the other features.


[Meta] A "docker deploy" roadmap needed

Someone probably needs to start a roadmap issue for getting a solution to current problems with "docker deploy" (ie. where the feature is planned to go) because so far, I cannot find anything, let alone an issue, describing current progress.
Including:

docker deploy
docker stacks
The planned relation between Swarm Mode / Docker Compose / docker deploy
The planned file formats

Ping @mgoelzer!

@icecrime I'm on it!

Any Progress?

👍

@techninja1008 @busbyjon Not forgotten, but not quite ready yet.  It's coming.

Roadmap posted in #26876


Incorrect HTML formatting with "html.format.indentHandlebars": false

Here is an extract of HTML (Angular 2) code:
  <app-root>Loading...</app-root>
{{#each scripts.polyfills}}
  <script src="{{.}}"></script>
{{/each}}

After format with"indentHandlebars": false :
  <app-root>Loading...</app-root>
  {{#each scripts.polyfills}}
  <script src="{{.}}"></script> {{/each}}

instead of :
  <app-root>Loading...</app-root>
{{#each scripts.polyfills}}
  <script src="{{.}}"></script>
{{/each}}

I expect unchanged Angular/handlebar indentation with "indentHandlebars": false
Copied from original issue: Microsoft/vscode#8144

Has the same cause as #1103.

@aeschli  could you show a bit more of your example?
The current output for the input you provided is:
<p>
    <app-root>Loading...</app-root>
    {{#each scripts.polyfills}}
    <script src="{{.}}"></script>
    {{/each}}
</p>

Is that sufficient?

@pierredewilde Can you help @bitwiseman ?

Yep, the example is sufficient to show the correct output.
Before the fix, the output was
  <app-root>Loading...</app-root>
  {{#each scripts.polyfills}}
  <script src="{{.}}"></script> {{/each}}

After the fix, the output is
  <app-root>Loading...</app-root>
  {{#each scripts.polyfills}}
  <script src="{{.}}"></script>
  {{/each}}

Ideally, in Visual Studio Code with "html.format.indentHandlebars": false, the output should be untouched
  <app-root>Loading...</app-root>
{{#each scripts.polyfills}}
  <script src="{{.}}"></script>
{{/each}}

but the current fix is OK.

This would now be an enhancement.   If we do not indent handlebars, they are treated as content, and then get indented like content.  I see how the name could be confusing.


I have the same issue

I have the same issue, I'm using python3
Originally posted by @jerald-jacob in https://github.com/Tessellate-Imaging/monk_v1/issues/60#issuecomment-744372926

Ho @jerald-jacob ,
Could you please mention the versions of torch, torchvision and pillow installed in your work environment?


Fix typo from 'AVFounation' to 'AVFoundation'

:)

You'r contribution is highly appreciated, thank you! :octocat:


Allow code search by filename

This is a large and complex PR, so let me explain in detail its changes.
First, I had to create new index mappings for Bleve and ElasticSerach as the current ones do not support search by filename. This requires Gitea to recreate the code search indexes (I do not know if this is a breaking change, but I feel it deserves a heads-up).
I've used this approach to model the filename index. It allows us to efficiently search for both the full path and the name of a file. Bleve, however, does not support this out-of-box, so I had to code a brand new token filter to generate the search terms.
I also did an overhaul in the indexer_test.go file. It now asserts the order of the expected results (this is important since matches based on the name of a file are more relevant than those based on its content).
I've added new test scenarios that deal with searching by filename. They use a new repo included in the Gitea fixture.
The screenshot below depicts how Gitea shows the search results. It shows results based on content in the same way as the current version does. In matches based on the filename, the first seven lines of the file contents are shown (BTW, this is how GitHub does it).

Resolves #32096

The year of copyright in new files should be 2024.

The year of copyright in new files should be 2024.

Done

Please do not force push, it's hard to follow changes this way

Please do not force push, it's hard to follow changes this way

Sure. My bad 😔

This requires Gitea to recreate the code search indexes

Ah, right - for this to happen, we need to increase issueIndexerLatestVersion in the affected indexers

Can you add a search test with filename as keyword?

Can you add a search test with filename as keyword?

I have included some., I'll provide some comments on the test cases to better describe their underlying scenarios. What do you think ?

Can you add a search test with filename as keyword?

I have included some., I'll provide some comments on the test cases to better describe their underlying scenarios. What do you think ?

Hey @lunny, I've added some comments to the test cases to better describe their scenarios :)


add predict_obs_cov to dual_lofi

Copy https://github.com/probml/rebayes/blob/main/rebayes/low_rank_filter/lofi.py#L174 to dual_lofi

Already exists in
https://github.com/probml/rebayes/blob/ca72fbb91e92dcbb61b4abe44c1092df0bd07371/rebayes/low_rank_filter/dual_lofi.py#L188


NetworkServer.dontListen improvements and fixes

changes double negative 'dontListen=false' to 'listen=true'
fixes #2536

MERGE, DONT SQUASH

for a simple solution, we could change OnTransprotConnectec to ignore connections while not listening

for a simple solution, we could change OnTransprotConnectec to ignore connections while not listening

You'd need to call Transport.ClientDisconnect if listen == false so the transport kicks the connection off.

This in NetworkServer::IsConnectionAllowed
            if (dontListen)
            {
                Debug.LogError($"Server not listening, client {connectionId} will be kicked");
                return false;
            }

@MrGadget1024 pushed. no more shutdown. just not accepting connections.
that should be the most simple solution here.

@miwarnec Failing test

@miwarnec Failing test

fixed


fixed right order for EHLO handler

What do these changes do?
Fix in EHLO handler, where the 250 HELP response line, means without dash should be the last one.
My tetsing postix server do not accept nre EHLO extensions after 250 HELP
Are there changes in behavior for the user?
There is no change for user, just fix according the RFC https://datatracker.ietf.org/doc/html/rfc5321#section-<IP_ADDRESS>
Related issue number
None
Checklist

[x ] I think the code is well written

I need to double check this - I think it's the right fix, I think I may have overridden this method within my own code for that same reason.

Confirmed that yes, this is the right fix.
However, we don't have a test failing for this case, so we really should create that test case. @hudecof is that something that you could tackle this week or next?

HELP

I did not want to close this ;)
I could try, to write the test, but bit sure about the time line ;(

Running the GHCI to test correctness.
Until we create the test to ensure this won't happen again, I won't merge it though.
I hope I can find some time to write the necessary tests if you're occupied @hudecof

This still use the old GHCI definition file, resulting in very slow qa_docs (and usually ends up in failure)
If you can rebase this to the latest master, @hudecof , I can rerun the tests.
Though again, until tests are written to prevent same issue from happening, I won't merge this.
But at least we can be sure that the changes do not cause any regression.


Overrule Windows Service account

This is related to this question.
I learned that when TM1py script is executed via TI, the TM1py script runs with the user of the Windows service.
As per out IT infrastructure guidelines, we have separate service accounts for each environment (DEV/QA/PROD) and I therefore can only make this work if I add all three service accounts as ADMIN in TM1 (currently not the case). This means we would need to get additional licenses from IBM.
Wouldn't it be possible to somehow overrule this Windows service account and define another ADMIN account that TM1py can use instead of Windows service account?

You can't use SSO though.
Pass the service account username to user parameter and password to the password parameter.
gateway parameter is not required.
You can also pass a base64 encoded password.

Clear, thanks @rkvinoth


Support for Groovy

I would be awesome if you can add support for Groovy :)

@AKolakowska47 are you okay with this?
If so, can you plz close this issue.
Thanks

@o2sh Your commit seems to be OK, but  I'm waitin' for binary to check it ;)
When the new one (2.1.1.)  will be available?

@AKolakowska47 This binary was built from 90bdc4ecb48d71e23871e4266a96e5d381c7d226

Onefetch 2.2.0  is now available.


Challenge 109 - Replace Text "replace with." with "replace."

Challenge Name

https://www.freecodecamp.com/challenges/change-text-inside-an-element-using-jquery
Issue Description

Possible error in diction in text.  Suggest replacing "replace with." with "replace."
Browser Information

Not relevant.

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code
// If relevant, paste all of your challenge code in here

Screenshot

@VincentDiCarlo thanks for looking into this but I don't think there needs to be a change.

In other words, this function will not evaluate any HTML tags passed to it, but will instead treat it as text you want to replace with.

The text you pass into the function is the "text you want to replace with" rather than just the "text you want to replace." You want to replace the existing text. Maybe it could be changed to:

In other words, this function will not evaluate any HTML tags passed to it, but will instead treat it as the text you want to replace the existing text with.

@erictleung In your suggestions, the word text is too repetitive. What about:

In other words, this function will not evaluate any HTML tags passed to it, but will instead treat it as text you want to replace the existing content with.

@atjonathan sounds good to me!
Those wanting to fix this, this line needs to be changed to the following:

In other words, this function will not evaluate any HTML tags passed to it, but will instead treat it as the text you want to replace the existing content with.

I have submitted a PR to address this issue.

I will take this as a first timers :).

@l-tong #8884 has already addressed this issue!

Got it.  thanks!


line comment toggle not working

When I press Ctrl + /  on a regular CSS file, a particular line is being commented on, but this is not working when using tcss files and this extension. Is it an issue with the plugin or does one just need to change something in local vs code settings?

Hmm interesting.
VS Code doesn't know what tcss files are and so it doesn't know how to comment them.
What we can do is tell VS Code what “commenting a line” means via the extension.
We'll look into it.

Moving this Todo. That could get annoying.

grate to hear and looking forward to this small enhancement 💪

This should be fixed in 1.1.0.

@rodrigogiraoserrao works like a charm! Many thanks 🙌


Address apiview comments for Compute, Network and Cdn

[Compute] Rename Vm prefix to VirtualMachine
[Cdn] SslProtocol -> DeliveryRuleSslProtocol
[Cdn] SslProtocolMatchCondition -> DeliveryRuleSslProtocolMatchCondition
[Network] HttpConfiguration -> NetworkHttpConfiguration
[Network] HttpConfigurationMethod -> NetworkHttpConfigurationMethod
[Network] HttpMethod -> NetworkWatcherHttpMethod
[Network] IPConfigurationProfile -> NetworkIPConfigurationProfile
[Network] IPConfigurationBgpPeeringAddress -> NetworkIPConfigurationBgpPeeringAddress
[Network] IpAllocationType -> NetworkIPAllocationType
[Network] IpAllocationListResult -> NetworkIPAllocationListResult

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Compute


DDC fails when running on Dart VM with fixed-size integers

Reproduction:
DART_VM_OPTIONS=--limit-ints-to-64-bits tools/test.py -m release -c dartdevc -r chrome --checked --strong corelib_2/int_parse_radix_test/none

Failure:
Runtime error:
Expect.equals(expected: <-0.0>, actual: <-9223372036854776000>) fails.
at Object.dart.throw (http://<IP_ADDRESS>:35118/root_build/gen/utils/dartdevc/js/amd/dart_sdk.js:4912:29)
    at Function._fail (http://<IP_ADDRESS>:35118/root_dart/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.js:365:17)
    at Function.equals (http://<IP_ADDRESS>:35118/root_dart/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.js:121:21)
    at int_parse_radix_test_none.main (http://<IP_ADDRESS>:35118/root_dart/out/ReleaseX64/generated_compilations/dartdevc-checked-strong/multitest_corelib_2_int_parse_radix_test_none/int_parse_radix_test_none.js:69:20)

It looks like DDC generated Dart literal '-9223372036854775808' to JS as '-null'.

I don't see a failure anymore (with either dartdevc or dartdevk). Assuming stale.


DllReferencePlugin purpose in aspnet core spa app?

No idea why this plugin is used in aspnetcore-spa apps inside webpack.config.js. Webpack documentation doesn't help either. Do we have to use this?  what is manifest.json file? do we have to run the command (webpack --config config/webpack.config.vendor.js) to create it before running the asp.net core app?
new webpack.DllReferencePlugin({
            context: __dirname,
            manifest: require('../wwwroot/dist/vendor-manifest.json')
        })

Manifest json specify your third party dependencies. It specify what exact *.js files should be included in page from node_modules and how the mapped inside of our concatted file. Dependency  described in webpack.config.vendor.js
inside of vendor entry:
 entry: {
        vendor: [
            'bootstrap',
            'bootstrap/dist/css/bootstrap.css',
            'style-loader',
            'jquery',
            '@angular/common',
            '@angular/compiler',
            '@angular/core',
            '@angular/http',
            '@angular/platform-browser',
            '@angular/platform-browser-dynamic',
            '@angular/router-deprecated',
            '@angular/platform-server',
            '@angular/router-deprecated',
        ]
    },

In a fact you could use any configuration of your webpack (this one is just jump start  suitable
for most of projects). The only restriction for webpack config right now - you have to add entry with name 'main'.

Thanks @laskoviymishka
By default webpack knows how to load modules based on the dependencies, e.g if my entry file index.js references angular, it will load, bundle, minify, based on webpack configuration provided in webpack config file.
Still didn't get the benefit/purpose of webpack.config.vendor.js and manifest output from this configuration file. Alos, do you know why the project uses DllReferencePlugin ?

@asadsahi well, you can remove it. Probably you right and this stuff could and should be removed. Right now we need to made some additional command before we could run ng2 app - webpack --config webpack.config.vendor.js if we remove this dll we probably remove this call also.

@asadsahi you are right, this is not really usefull here. It could be removed.

You could even remove webpack.config.vendor at all.
In this case your webpack.config will be like:
var path = require('path');
var webpack = require('webpack');
var merge = require('extendify')({ isDeep: true, arrays: 'concat' });
var ExtractTextPlugin = require('extract-text-webpack-plugin');
var extractCSS = new ExtractTextPlugin('styles.css');
var devConfig = require('./webpack.config.dev');
var prodConfig = require('./webpack.config.prod');
var isDevelopment = process.env.ASPNETCORE_ENVIRONMENT === 'Development';

module.exports = merge({
  resolve: {
    extensions: ['', '.js', '.ts']
  },
  module: {
    loaders: [
          { test: /\.(png|woff|woff2|eot|ttf|svg)$/, loader: 'url-loader?limit=100000' },
          { test: /\.ts$/, include: /ClientApp/, loader: 'ts-loader' },
          { test: /\.html$/, loader: 'raw-loader' },
          { test: /\.css/, loader: extractCSS.extract(['css']) }
    ]
  },
  entry: {
    main: ['./ClientApp/boot-client.ts']
  },
  output: {
    path: path.join(__dirname, 'wwwroot', 'dist'),
    filename: '[name].js',
    publicPath: '/dist/'
  },
  plugins: [
      new webpack.ProvidePlugin({ $: 'jquery', jQuery: 'jquery' }), // Maps these identifiers to the jQuery package (because Bootstrap expects it to be a global variable)
      new webpack.optimize.OccurenceOrderPlugin(),
      extractCSS
  ]
}, isDevelopment ? devConfig : prodConfig);

And don't forget to add bootstrap style into client boot:
import 'bootstrap';
import 'bootstrap/dist/css/bootstrap.css';

In this case everything will be bundled in single file.

Sweet and Simple :+1: , will give it a try in a bit and share my experiece as well.

I also think that the webpack configuration right now is too complicated and certainly not easy to understand it. For development there is no point to split the bundles between vendor and rest. The most common approach is to create just a js file which contains everything. Then during preparation for production environment is better to split vendor dependencies with the app's own code. I am trying to create a sample for ReactRedux in ES6 and not in TypeScript and have to say that webpack configuration in hard to follow.

@laskoviymishka , works like a charm with no webpack.config.vendor.js
@xabikos Agree with you on webpack's configuration being complex in aspnetcore-spa generator.
BTW, feeling sad after you mention that sample project you are working on is in ReactRedux in ES6 and not in Angular 2 & typescript :). I like your webpack middleware at somepoint, but I think aspnetcore-spa is converging the same idea of middleware into generator.
Right now I am currently working on a project template which is mix of:

aspnetcore-spa generator (for server side stuff)
Angular 2 webpack starter (I have picked some webpack stuff from this project, I like the way vendor/polyfills and application level providers and directives are organized in this this starter template, also, this project has lazy loading of bundles using es6-module loader with webpack, so that is inspiring too)
Angular 2 seed (Picked angular 2 styleguide following this project)

@SteveSanderson , Lets see if Steve likes the above ideas implemented in aspnetcore-spa generator as well :)
Here is my webpack.config.js with no reference of jQuery or javascript element of bootstrap and sass loader:
var path = require('path');
var webpack = require('webpack');
var merge = require('extendify')({ isDeep: true, arrays: 'concat' });
var ExtractTextPlugin = require('extract-text-webpack-plugin');
var extractCSS = new ExtractTextPlugin('styles.css');
var devConfig = require('./webpack.config.dev');
var prodConfig = require('./webpack.config.prod');
var isDevelopment = process.env.ASPNETCORE_ENVIRONMENT === 'Development';

module.exports = merge({
    resolve: {
        extensions: ['', '.js', '.ts']
    },
    module: {
        loaders: [
            { test: /\.ts$/, include: /Client/, loader: 'awesome-typescript-loader' },
            { test: /\.html$/, loader: 'raw-loader' },
            { test: /\.css/, loader: extractCSS.extract(['css']) },
            { test: /\.scss$/, loaders: ['raw-loader', 'sass-loader?sourceMap'] },
            { test: /\.(png|woff|woff2|eot|ttf|svg)$/, loader: 'url-loader?limit=100000' }
        ]
    },
    entry: {
        main: ['./Client/boot-client.ts']
    },
    output: {
        path: path.join(__dirname, '../wwwroot', 'dist'),
        filename: '[name].js',
        publicPath: '/dist/'
    },
    plugins: [
        new webpack.optimize.OccurenceOrderPlugin(),
        extractCSS
    ]
}, isDevelopment ? devConfig : prodConfig);

@asadsahi take my :+1:
Right now yo generator is really simple, and it still to much stuff to enhance. It may be great if there will be an option like one which we use in react and react-redux. IMHO angular2-seed and  Angular 2 webpack starter looks really scary and for some case it overkill. But sometimes it really best starters (especially webpack version i use it in one prod app).

@asadsahi I am not into both Angular or TypeScript and there is already a template for Angular 2 in TypeScript. Now about the middleware I have created here indeed is simpler and spa services provide way more functionality. My goal is to just make the usage of webpack as module bundler really easy in any asp.net core application. Even if we don't use React or Angular.
I really like this project and effort of having server side rendering of JavaScript apps but in order to succeed I think a template in ES6 should be present for sure. In general TypeScript is not a very popular choice in React community and it's a pity not to have an option for pure JavaScript devs.

@laskoviymishka agree with angular 2 seed and webpack angular 2 starter being complex beasts and I have had few iterations in last week to somehow get most of the functionality working in aspnetcore-spa template from those projects, but trust me you don't want to waste time like me. Thiking about 80/20 rule in webpack world, we may not need alot of customisations. Therefore, I am picking the better parts from those two projects only.
@xabikos it makes sense. Different tooling in different situations. :)

The reason for the "vendor" bundle (and the DllReferencePlugin which enables it) is just to make your builds much faster. All the really huge dependencies (e.g., Angular, in your case) only need to go through Webpack's static analysis process when you explicitly rebuild your "vendor" module, and can be skipped entirely for most builds.
In practice I find this shortens the build time from about 15 seconds to about 3 seconds. This is especially important when you're using Hot Module Replacement, because you want your code changes to appear in the browser pretty much instantly, not 10 seconds after you make them.
I appreciate that it makes the webpack setup more complex, but the benefit is pretty huge. If anyone has suggestions for a simpler webpack setup that retains such fast builds, please let me know!
Please reopen if you think this explanation is inadequate or there's a better solution!

That perfectly explains it @SteveSandersonMS .
Reason is strong and I am in favour of keeping it.
Thanks


index is incremented in Watcher; remove double-increment

Remove double-increment, closes #5741.
Watch index is incremented in both etcd-watch client and in Watcher method from keys.go. It makes more sense in Watcher(), so removed from etcd-watch.

@dannysauer Can you change the commit message to
etcdctl: index is incremented in Watcher; remove double-increment

Add etcdctl as a pkg prefix.

Commit message adjusted.  Thanks for the practice. ;)

@dannysauer Thanks for your patience!


Add stream_mode to kinesis

Wanted to add in this option so users can have the option to allow for kinesis to autoscale rather than be provisioned

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

Signed

Confirmed! @jordan-tr has signed the Contributor License Agreement. Thanks so much.


Scrollbars do not reappear after disabling and re-enabling ScrollContainer scrolling.

Godot version:
master
OS/device including version:
Windows 10
Issue description:
If you disable horizontal or vertical scrolling on a ScrollContainer node, the scrollbar will not reappear upon re-enabling scrolling.
Steps to reproduce:
Create a ScrollContainer node with some content. Add the following script:
extends ScrollContainer

func _input(event):
	if event is InputEventKey:
		if event.pressed and not event.echo:
			scroll_horizontal_enabled = not scroll_horizontal_enabled
			scroll_vertical_enabled = not scroll_vertical_enabled

			# uncomment for workaround
#			set_visible(false)
#			set_visible(true)

Run the project and press any key. The scrollbars will be hidden. Press any key again and the scrollbars do not return as expected. (Resizing the window will make them reappear.)
Minimal reproduction project:
ScrollContainer_scrollbar_visibility_bug.zip

when changing scroll_horizontal_enabled or scroll_vertical_enabled property of ScrollContainer,
the minimum width or height of ScrollContainer is changed to fit child node.
you can easily see what happens when changing it on the editor.
it's not for just changing the visibility of scrollbars.
https://github.com/godotengine/godot/blob/61dd7748cafafca08b8f87183ddf42de06b58911/scene/gui/scroll_container.cpp#L40-L72
probably need discussion?

I am having the same issue in godot 3.2.3
func _input(_event): 	if Input.is_action_just_released("control"): 		scroll_vertical_enabled = true 		set_visible(false) 		set_visible(true) 	if Input.is_action_pressed("control"): 		scroll_vertical_enabled = false 		var scroll_value = (Input.get_action_strength("scroll_up") - Input.get_action_strength("scroll_down")) * 100 		panel.rect_min_size.y += scroll_value*
from the moment scroll_vertical_enabled = false runs both scrollbars disappear (even though I only disabled vertical not horizontal) and won't re-appear


fix(@angular/cli): command runner is not working on windows

fs is unable to read the normalized paths from @angular-devkit/core  on windows as the disk drive is replaced from c:/ to /c/
Closes #11928
Another alternative is to use getSystemPath in all the places where the is readFileSync within the runner. But I found this to be cleaner.
This was not caught in CI, as in AppVeyor we are not running test-cli-e2e

FYI @hansl looks like it was introduced in your https://github.com/angular/angular-cli/commit/2ce11551587407769d0a51c126bb62db3c5d6a08#diff-53bc4547164f6bfb4fee01f58a298892

On another note, is it really required to remove the : when normalizing paths?

@alan-agius4 there's a misunderstanding here;

I did not catch the change in time in my PR, sorry about that. The original PR was the one introducing the error, but I should have caught it.
normalize and Path are devkit concepts. They should never be mixed with Node fs methods. Core does not understand Node fs, so it's okay. All those Path types are useable in devkit's hosts and general methods. They're meant to be platform independent, so no : or http: or anything weird like that. They just represent paths; absolute or relative, directory names separated by /. That's all. They should never be used outside of core methods.
I'd like to move the Path type at some point to not be a subtype of string, but be a unique symbol. That way people won't make that mistake again. But in the meantime we have to deal with this.

@hansl thanks for the clarifications. 👍


Add a utility class for parsing GeoJson

To download tiles based on Offline Imagery selections, we need to parse geojson files for tile specifications (polygons) and determine which tiles in a given geojson file intersect with the bounds of an offline imagery selection. This utility class handles both requirements.
This PR also contains changes to the Tile model--we need to update our approach to modeling tiles now that we download them by regional intersection instead of direct selection.

gino-m@ Mind taking another look? This may still not be totally optimal, but we're getting closer!

Mind taking one more look, @gino-m? 👼

Weird, checks are failing to run with:
Finished Step #2 - "Copy .git directory for versioning"
ERROR
ERROR: build step 2 "gcr.io/cloud-builders/git" failed: step exited with non-zero status: 128

Are you able to run checkCode locally?

Weird, checks are failing to run with:
Finished Step #2 - "Copy .git directory for versioning"
ERROR
ERROR: build step 2 "gcr.io/cloud-builders/git" failed: step exited with non-zero status: 128

Are you able to run checkCode locally?

Oh weird, I'll take a look

Weird, checks are failing to run with:
Finished Step #2 - "Copy .git directory for versioning"
ERROR
ERROR: build step 2 "gcr.io/cloud-builders/git" failed: step exited with non-zero status: 128

Are you able to run checkCode locally?

Oh weird, I'll take a look

Sure enough, I just cloned this to my home machine and I can't get gradle to sync b/c of the following error:
ERROR: android.defaultConfig.versionCode is set to -1, but it should be a positive integer.

which in fact sounds like it would result from something like the original failure:
Finished Step #2 - "Copy .git directory for versioning"
> > ERROR

If it failed to copy the git dir for versioning it'd break out versions.
Not really sure how or why this happened or how to debug this--Idk anything about gitVersioner (that's what we're using) I'll try to figure it out

Sooooo this is proving tricky. I cloned the repo to my home laptop and cannot get this thing to build at all. gitVersioner refuses to identify it as a git repo even though it is one and every git command works as expected. I've even tried stepping back a couple commits and that doesn't solve anything--there doesn't seem to be any corruption in the repo itself.....so I'm fairly stumped.

I do notice that we use the variable $_HEAD_REPO_URL to copy the git file (see cloudbuild.yaml) --I'm guessing that's set in our cloud build setup? Could that have been messed up at some point? Though I don't see how that'd result in local issues building....
As a last effort for now I'm going to remove the gitVersioner plugin entirely and see if I can get this to build.

Finally got this to build on my home machine. Had to remove the gitVersioner plugin completely which revealed another error that  android.enableSeparateAnnotationProcessing=true is experimental and not supported...commenting that out finally did the trick.
Unfortunately, I don't think these local remedies solve any problems on the cloud build side. We probably still have to diagnose this. It'll be interesting to see whether these same issues crop up when I try to build on my other machine tomorrow.
Not pushing any fixes since it seems to me that the fix I've found is less than ideal and there's probably some other root cause--maybe we have to change something about the way we use gitVersioner? Or perhaps there's something problematic in the .git folder copying script?

Did some digging. The last time we touched gitVersioner was back in December--and we hadn't encountered this up till now--so I suspect this issue ultimately stems from either a bug in gitVersioner itself or some other strangeness in git on my local fork. Checking things out on my other machine (where I originated these commits) will hopefully be instructive.
@shobhitagarwal1612 FYI

@gino-m woo! Checks passed


RNIap.requestSubscription returns successful result when subscription is not confirmed (Amazon)

Version of react-native-iap
5.2.0
Version of react-native
0.63.3
Platforms you faced the error (IOS or Android or both?)
Android
Expected behavior
RNIap.requestSubscription waits for successful subscription
Actual behavior
RNIap.requestSubscription returns successful result even though the user has not subscribed the app yet
Tested environment (Emulator? Real Device?)
Real device, GalaxyS9, Android 10
Steps to reproduce the behavior

Install an app from Amazon Store
Pick a subscription item (RNIap.requestSubscription) is called
RNIap.requestSubscription returns the result immediately

Basically, this behavior is different from the cases when apps are installed through Google Play. Normally, RNIap.requestSubscription waits until the users subscribes (selects card and clicks OK)

I am facing the same issue with RNIap.requestPurchase. By default (Google) it shows the pop up to cancel or confirm the purchase and in that case it returns a Json with its information. But when I use the ones from Amazon, await RNIap.requestPurchase automatically returns true.
I have tried different configurations in Amazon App Tester but the only thing that changes is that the IAPs load or not. The app is still in Live App Testing, and the IAP is a consumable that is Live.
Using react-native 0.62.2.
Real device: Pocophone F1, Android 10
Installed from Android studio and using RNIap.setFallbackInstallSourceAndroid(RNIap.InstallSourceAndroid.AMAZON); to use Amazon's.

I am facing the same issue with RNIap.requestPurchase. By default (Google) it shows the pop up to cancel or confirm the purchase and in that case it returns a Json with its information. But when I use the ones from Amazon, await RNIap.requestPurchase automatically returns true.
I have tried different configurations in Amazon App Tester but the only thing that changes is that the IAPs load or not. The app is still in Live App Testing, and the IAP is a consumable that is Live.
Using react-native 0.62.2.
Real device: Pocophone F1, Android 10
Installed from Android studio and using RNIap.setFallbackInstallSourceAndroid(RNIap.InstallSourceAndroid.AMAZON); to use Amazon's.

@iaphub May you help this out?

@avoloshko You're right currently the Amazon is working a little bit differently even if it is not an issue to use Amazon since you can listen to purchase & errors events.
Amazon is still in 'beta' and will definitely need some love in the coming months for all these little improvements.
I'll take the time to work on it in the future but @avoloshko @MarcosHY feel free to push a PR if you would like 👍

Actual. On android request return true on moment when I just see native popup with Subscribe button.


[no-restricted-paths] Add options to ignore certain import kinds

This adds a new option called "allowedImportKinds" to the rule no-restricted-paths.
This option defaults to an empty array and can be set to an array of strings.
This option allows imports of the specified kind(s) to be ignore by this rule.
Example use:
{
  // Allow type imports.
  "allowedImportKinds": ["type"],
  "zones": [ {
    "target": "./tests/files/restricted-paths/server/one",
    "from": "./tests/files/restricted-paths/server",
    // Override above `allowedImportKinds`, don't make an exception for any import kinds.
    "allowedImportKinds": [],
  } ]
}

Coverage decreased (-2.3%) to 95.574% when pulling 22a95d8eb23668efc96c297c1466dbe836ec153f on RebeccaStevens:no-restricted-paths/allowed-import-kinds into fef718cb134016855afe61682e32c63ba3ed9281 on benmosher:master.

@ljharb

I'm not sure if we need an enum - will there ever be another kind of import that's not value or type?

In this PR I haven't enforced the option being an enum, any strings can be given. With regard to whether there will be any other kinda of import, most likely not - but it's probably best to keep things flexible just in case.

@ljharb Think you can help me resolve the failing build? It's failing in older versions of eslint as node.parent is undefined.


chore(main): release 1.5.0

:robot: I have created a release beep boop
1.5.0 (2024-03-27)
Features

add nacos (392c7a0)
add prompthub (db63b2c)

Docs

make doc better (4adfede)
make doc better (1043f62)
make doc better (349d68e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/WGrape/sparrow/releases/tag/v1.5.0 :sunflower:


reharden Caravel using open GF180MCU pdk

This is still an ongoing work. The PnR branch has the current views.

#95


Change load route in ShopServiceProvider with remove web middleware shop routes

Hi @milwad-dev,
Thanks for the contribution.
There are more packages in Bagisto like admin, velocity, and many more, please check that also.
So that consistency should be maintain in all the packages.

Hi @devansh-webkul .
Yes, I want to add separate PR, but whatever you say

@devansh-webkul
Complete.


focus-manager: prevents tabbing to elements added after dialog initial render

Bug
I’m having an issue with Dialog and the Focus Manager (love the idea and implementation of the focus manager btw). The issue is preventing me from tabbing to items inside my modal.
Basically FocusManager is not updating what it thinks is the last focusable element when the contents of the dialog update after the initial render. This prevents you from tabbing to anything that was dynamically added below the initial last-focusable element. Any ideas on what we can do to address this?
Here’s a CodeSandbox that illustrates the problem.
Here’s the actual dialog having the issue (you can’t tab past the search box)
Expected behavior
Should be able to tab to all content in dialog
Actual behavior
Cannot tab to elements added after the dialog initial render
Related packages

dialog
focus-manager

Environment

OS - MacOS
Browser - Chrome

Hey Kaiden, thanks for the bug notice.  We'll take a look at it.

Any idea when you'll be able to get to this issue?

Hi @jaketrent, we upgrade to the latest version of ps-design-system-dialog (which as you know pulls in focusmanager) but we're still seeing this issue. Happy to jump on a zoom call or something to show you. Thanks!

:sad-trombone:  well, that's no good. I'll put this back on my list today to repro. Perhaps you could try to do the same. The repro codesandbox that kaiden sent was great last time. Mind doing that again with the latest ver?

I took the old sandbox and used the new version of Dialog that has the new version of FocusManager in it. Same as you, it doesn't work. I put it into our storybook, still doesn't work. Sits right next to another dynamic child story that I thought was proving it worked. I'll figure out what the diff is...

@izikandrw k, found the issue. PR is now in https://github.com/pluralsight/design-system/pull/1813


Fix typo in part-8-modern-redux.md

Fix typo

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
What docs page needs to be fixed?

Section: Using createSlice
Page: https://redux.js.org/tutorials/fundamentals/part-8-modern-redux#using-createslice

What is the problem?
Missing code block for createSlice

My solution was not correct


Navbar links not working (Issue #134) fixed

@khushi-joshi-05 I have fixed the links in navbar and also linked the other files in html-files to content.css in Css-files as you asked. Thank you for giving me the opportunity to contribute to this project.

Solve your Conflicts!


Adds resource tags to downloadDashboardsImage (Fixes #21428)

What this PR does / why we need it:
In order to be able to use the dashboard funtionality when resourcequotas are in place you need to specify the resources for all container (including all initcontainers)
Which issue this PR fixes

fixes #21428

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @zanhsieh

@njoaquim
Could you bump the chart version to latest please? Thank you.


Weird behaviour of graphics.drawTriangles in HTML5

Basically in HTML5 when the last triangle is drawn using graphics.drawTriangles() something else is drawn above it.
I created a little project to test it. (see Flash vs HTML5)
GraphicsDrawTrianglesBug.zip

Thanks for the test! I found the issue, it's a matter of automatically closing shapes, with a pending fill. drawTriangles in this renderer already commits the result to the canvas, meaning we don't need a pending draw later

Wow. That was quick. Thank you very much!


Unused ConfigMap in deploy manifest

The deploy manifest includes a ConfigMap that does not appear to be used (it is not mounted to the Deployment). Can it be removed?
https://github.com/GoogleCloudPlatform/gke-autoneg-controller/blob/b8b9a87c65f3bfc07aebc60649b4f5c4af7d3230/deploy/autoneg.yaml#L189-L222

Yeah, I don't think that is important - happy to take a patch to remove it. I think it made it's way into the Terraform example as well.


The first segment contains response mode of form_post but the immediate examples show response mode of fragment

[Enter feedback here]
The first segment contains response mode of form_post but the immediate examples show response mode of fragment which is confusing.

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

ID: c54adceb-9510-aeb5-0c61-c0d052459cda
Version Independent ID: 0ff6b6b2-2802-9534-8963-796b9c4a2a76
Content: Web sign-in with OpenID Connect - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/openid-connect.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@kumarsumitrudeboy- Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

Dear @kumarsumitrudeboy, thanks for the feedback. I'm updating the article with the relevant response mode parameter. A new version will publish later today, and this GitHub issue will be closed automatically (before the doc is published). If you have any questions or need further information, please open new GitHub issue.


set_network_addresses_tuple doesn't work for IPv6 on Windows 11

The netsh syntax appears to be wrong. I'm not sure if it worked on older versions of Windows? But on my Win11 dev system it doesn't work.
This does work: netsh interface ipv6 set address interface="My Wintun Tunnel" address=f000::f000
So I'm working around it here https://github.com/firezone/firezone/pull/3120/files#diff-d3087e924a318d4651ba5e2d08ebc753563e4cd3d2a0f6acccf2475936595551R86-R95
I'm not sure if the CREATE_NO_WINDOW flag helps or not - It worked for Powershell windows I was trying to hide.

Thanks for flagging!
I won't have access to a Win11 machine for a few weeks so feel free to submit a PR if you need this fixed faster

I wonder if these Win32 functions could help: https://learn.microsoft.com/en-us/windows/win32/iphlp/managing-ip-addresses-using-addipaddress-and-deleteipaddress

No problem! For now I'm just shelling out to netsh directly instead of using the wintun function.
Yeah I've ended up using a lot of those IpHelper functions from the Windows crate in the Firezone client.
If anyone's reading this and doesn't want to shell out, ChatGPT recommends CreateUnicastIpAddressEntry for both IPv4 and IPv6 - https://microsoft.github.io/windows-docs-rs/doc/windows/Win32/NetworkManagement/IpHelper/fn.CreateUnicastIpAddressEntry.html


Include a link to the hands-on lab on the Containerized Single Instance use-case

A lab is now available in the LiveLab format with detailed instructions to run the Containerized Single Instance database on an OCI trial account, using both dynamic block storage and NFS storage.
It might be interesting to include a link to this lab in the documentation.

@janleemans this is really good. Thanks will include the link.

@janleemans Addressed this in PR #30. Closing this issue.


error  on build mycurve25519

i download the solution and i have errors with the mycurve25519 cpp project
"Cannot open include file: 'pch.h': No such file or directory"

i used the soolv on #151
but now i have this error
Severity	Code	Description	Project	File	Line
Error	LNK1104	cannot open file 'dummy.lib'	mycurve25519	D:\whatsApp\Chat-API-NET-master\mycurve25519\mycurve25519\LINK	1

there's a folder called extra in the mycurve25519 project, makes sure there's a file called dummy.lib in this folder and ensure you've linked it correctly in the project. refer to the image attached here


🐛 Bug Report: Undefined method Container::add() in Utopia DI Container

👟 Reproduction steps
When attempting to run the PHP development server, encountering a fatal error due to an undefined method add() in the Utopia DI Container class.
Here is the code (used straight from readme):
<?php

require_once __DIR__.'/vendor/autoload.php';

use Utopia\DI\Container;
use Utopia\DI\Dependency;
use Utopia\Http\Http;
use Utopia\Http\Request;
use Utopia\Http\Response;
use Utopia\Http\Adapter\FPM\Server;

// Creating the dependency injection container
$container = new Container();

// Adding a user dependency to the container
$user = new Dependency();
$user
    ->setName('user')
    ->inject('request') // We can insert and use other injections as well
    ->setCallback(fn (Request $request) => $request->getHeader('x-user-id', 'John Doe'));

$container->add($user);

// Defining Route
Http::get('/hello-world')
    ->inject('request')  // Auto-injected each request
    ->inject('response') // Auto-injected each request
    ->inject('user')
    ->action(
        function(Request $request, Response $response, string $user) {
            $response
              ->addHeader('Cache-Control', 'no-cache, no-store, must-revalidate')
              ->addHeader('Expires', '0')
              ->addHeader('Pragma', 'no-cache')
              ->json(['message' => 'Hello World', 'user' => $user]);
        }
    );

Http::setMode(Http::MODE_TYPE_PRODUCTION);

$http = new Http(new Server(), $container, 'America/New_York');
$http->start();

?>

👍 Expected behavior
It should not throw a error.
👎 Actual Behavior
Error message:
➜  testing-swoole php -S localhost:8000 index.php
[Mon Dec 23 01:29:16 2024] PHP 8.4.2 Development Server (http://localhost:8000) started
[Mon Dec 23 01:29:21 2024] [::1]:51534 Accepted
[Mon Dec 23 01:29:21 2024] PHP Fatal error:  Uncaught Error: Call to undefined method Utopia\DI\Container::add() in /Users/chiragagg5k/Desktop/Coding_Stuff/testing-swoole/index.php:22
Stack trace:
#0 {main}
  thrown in /Users/chiragagg5k/Desktop/Coding_Stuff/testing-swoole/index.php on line 22
[Mon Dec 23 01:29:21 2024] [::1]:51534 Closing

🎲 Utopia HTTP version
Version 0.18.x
🐘 PHP Version
PHP 8.0
💻 Operating system
MacOS
🧱 Your Environment
PHP 8.4.2
"utopia-php/http": "^1.0"
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

This might be a documentation error, not sure


User/garydan/sysvad

prerelease drop of Sysvad, which includes the loopback tappoint feature

This project would require Insiders SDK/WDK on the canary channel (build 26040 or greater). My recommendation would be to update the documentation to mention this, but given that this is a repository with limited scope, partners can be notified of this directly.


ectopia lentis - new synonym

Add new synonym for ectopia lentis:
NEW SYN: ectopic lens

Synonym has been added.


Update Composer dependencies (2021-07-22-00-06)

Visual regression test passed!
View the visual regression test report


Add secret names as environment variables on connection

Also, need to allow <resource>#Name as a property ref (currently not possible due to Name not being an official property but rather part of the ID).

why do we need to allow the Name to be referenced? Because the pulumi name could have the uuid appended i would argue we dont want to allow the klotho resource name to be referenced as a property ref.


Build fails on fresh install

I'm new to Solana and struggling to find a single example repo that actually builds and works.

Build fails with
Error: `idl-build` feature is missing. To solve, add
[features]
idl-build = ["anchor-lang/idl-build"]
in `/path/to/CRUD-dApp/anchor/programs/journal/Cargo.toml`.
Warning: run-commands command "anchor build" exited with non-zero status code

 ——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  NX   Ran target anchor for project anchor (47s)

         With additional flags:
           build

    ✖    1/1 failed
    ✔    0/1 succeeded [0 read from cache]

Adding the idl-build = ["anchor-lang/idl-build"] line also fails with:
Error: IDL doesn't exist
Warning: run-commands command "anchor build" exited with non-zero status code

 ——————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  NX   Ran target anchor for project anchor (17s)

         With additional flags:
           build

    ✖    1/1 failed
    ✔    0/1 succeeded [0 read from cache]

I downgraded to anchor 0.29.0 and it seems to build now.

Hey @stoooops
You're experiencing transcript error. It seems your query would be best dealt with by the support team.
I'll have to refer you to the official support Requests team with the ticket ID SOL07638
Please see the link below to our dedicated support line:
solana Support Requests
Note; Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.


Unable to call delete since body is typed as never

Given the schema below, I am unable to call delete on it since the data param is typed as never.
Schema
 {
    method: 'get',
    path: '/api/customers/:customerId/subscriptions/:subscriptionId/resources/ifm/:resourceId',
    requestFormat: 'json',
    parameters: [
      {
        name: 'customerId',
        type: 'Path',
        schema: z.string().uuid()
      },
      {
        name: 'subscriptionId',
        type: 'Path',
        schema: z.string().uuid()
      },
      {
        name: 'resourceId',
        type: 'Path',
        schema: z.string().uuid()
      }
    ],
    response: IFMResource
  },

this is a get path. You can call delete by passing undefined for body and it should work.

Oh, sorry, pasted the wrong schema, here's the correct one:
´´´typescript
{
method: 'delete',
path: '/api/customers/:customerId/subscriptions/:subscriptionId/resources/intunefourme/:resourceId',
requestFormat: 'json',
parameters: [
{
name: 'customerId',
type: 'Path',
schema: z.string().uuid()
},
{
name: 'subscriptionId',
type: 'Path',
schema: z.string().uuid()
},
{
name: 'resourceId',
type: 'Path',
schema: z.string().uuid()
}
],
response: z.void()
},
´´´

Sorry, pasted the wrong schema, here's the correct one:
 {
    method: 'delete',
    path: '/api/customers/:customerId/subscriptions/:subscriptionId/resources/intunefourme/:resourceId',
    requestFormat: 'json',
    parameters: [
      {
        name: 'customerId',
        type: 'Path',
        schema: z.string().uuid()
      },
      {
        name: 'subscriptionId',
        type: 'Path',
        schema: z.string().uuid()
      },
      {
        name: 'resourceId',
        type: 'Path',
        schema: z.string().uuid()
      }
    ],
    response: z.void()
  },

But still get this:

Update, I created a new project with just zodios and an empty schema and added the faulting endpoint, which worked. Then added the other endpoints one by one, when I added another delete endpoint which had a body defined, I got the error again:
Here's an example that reproduces the erorr:
import { makeApi, Zodios, ZodiosPathsByMethod, type ZodiosOptions } from "@zodios/core";
import { z } from "zod";

const DeleteRoleAssignmentCommand = z.object({ tenantId: z.string().uuid(), objectId: z.string().uuid() });

const endpoints = makeApi([
  {
    method: "delete",
    path: "/api/customers/:customerId/",
    requestFormat: "json",
    parameters: [
      {
        name: "customerId",
        type: "Path",
        schema: z.string().uuid(),
      }
    ],
    response: z.void(),
  },
  {
    method: "delete",
    path: "/api/customers/:customerId/roleassignments/:roleId",
    requestFormat: "json",
    parameters: [
      {
        name: "body",
        type: "Body",
        schema: DeleteRoleAssignmentCommand,
      },
      {
        name: "roleId",
        type: "Path",
        schema: z.string().uuid(),
      },
      {
        name: "customerId",
        type: "Path",
        schema: z.string(),
      },
    ],
    response: z.void(),
  },
]);

export const api = new Zodios(endpoints);

export function createApiClient(baseUrl: string, options?: ZodiosOptions) {
  return new Zodios(baseUrl, endpoints, options);
}

const client = createApiClient("http://localhost:3000/");

client.delete("/api/customers/:customerId/", undefined, { params: { customerId: "123", subscriptionId: "456", resourceId: "789" } });

If you have a body, put the body instead of undefined.
I see no bug.

Ok, so to be clear I have endpoint A that accepts a body and endpoint B that doesn't accept a body. I want to call endpoint B,
but it seems like having the other endpoint with a body in the endpoints array cause the typing to resolve to never.
Using zodios version 10.9.0.

ok, i reproduced the issue. it's triggered in some specific use cases. fix incoming.
in a few minutes

can you check new version v10.9.1 and close the issue if it's fixed with this new version ?

Yes, it works fine now. Thanks for the quick response and fix! 🚀


'Hierarchy - Top' layout is not applying hierarchical physics

This is known good in https://github.com/grtjn/ml-visjs-graph.js/tree/0.4.0 and known bad in whatever version @grtjn  is running locally :-).
Creating this issue to help with debugging.

You are right. I observed this when I was working on #15, and edge styling was getting lost altogether. I was trying to pass through complete options in setOptions to resolve this, but that just made things worse. In the end I am still keeping a shadow options object, but passing in diffs only to setOptions (except initially). The culprit with the edges was, that I had to pass through a complete edges.smooth object, not just a loose edges.smooth.type property.

It was actually related to #11, or maybe both 11 and 15..


Test Issue created at Fri, 04 Jun 2021 09:59:12 GMT

Test issue body  Fri, 04 Jun 2021 09:59:12 GMT

Comment on issue at Fri, 04 Jun 2021 09:59:13 GMT


Refactor website landing page

The current website is a little awkward. There is an about page which is the landing zone (e.g. https://ibm.github.io/the-mesh-for-data/v0.1/), however, most of the initial user information is here: https://ibm.github.io/the-mesh-for-data/v0.1/concepts/introduction/ which is a little hard to find.
Originally posted by @butler54 in https://github.com/mesh-for-data/mesh-for-data/discussions/588

Tasks:

[ ] Create mesh-for-data.github.io website
[ ] Add support for a blog on mesh-for-data website
[ ] Move base content to root mesh-for-data homepage


Please provide installation steps

Hello @CaptainEven ,
ThankYou for providing this code. I have followed the exact installation steps from https://github.com/ifzhang/FairMOT.
conda create -n MCMOT
conda activate MCMOT
conda install pytorch==1.2.0 torchvision==0.4.0 cudatoolkit=10.0 -c pytorch
cd ${MCMOT_ROOT}
pip install -r requirements.txt
We use DCNv2 in our backbone network and more details can be found in their repo.
git clone https://github.com/CharlesShang/DCNv2
cd DCNv2
./make.sh
Can you please confirm if you used the same steps?
I was able to successfully run the code from FairMOT repository, but i am facing issues while running the code in this repository.
ThankYou

@abhaygargab , were you able to perform a correct installation? Could you provide the steps? Thanks!


Intellisense fires within comments

Inside a comment, I finish a sentence with a "." and the intellisense popup appears with "append", etc:

I can imagine some use for intellisense in comments, but in this case it shouldn't have appeared at all.
If there's some agreement, I'll create an issue.

I agree this should never occur. There is code in place to prevent this, but you must have hit a condition that our checks don't cover. If you're able to find a minimal, self-contained code sample, that would be really useful.

Hmm, I'm not able to repro the problem following those steps. Are you sure that these completion suggestions are coming from pylance? Do you have other extensions installed that might be generating these completion suggestions? The icons next to these items in your screen shot don't look familiar.

Ahhh, it was a different extension (frhtylcn.pythonsnippets).  I went through the ones I had installed, and toggling that one triggers this issue.
Sorry for the confusion, thanks for the help!


PGERC721 Deploy, View, WhaleView

Modified the contract a bit, but PGERC721s are a thing, populated to a view, linked with their respective WhaleUI.
this uses whale and collection view from nft-retroactive-funding / @austintgriffith
To Change:

Token URIs should be entered manually, and we pull json / file list from there
Whale functions on the View NFT page
Page styling could be improved
Inflation is currently entered in 1000x format, so 3 percent inflation is "1030". I'll fix this.

Issues:
ViewNFT is receiving the "burner" address, so collection view, minting, redeeming, are not working correctly. I'm really actually stumped as to why this is happening.

@simplemachine92 is this wip?


I have created a stage in lua script with an animated object

Describe your problem here.
my problem is that I have created a stage in lua script and when I test it it gives me the error: mods / stages / 1.lua: 7: ')' expected near 'mii' the stage code is:
function onCreate ()

background shit
makeLuaSprite ('stageback', 'stageback', -500, -300);
setLuaSpriteScrollFactor ('stageback', 0.9, 0.9);
  makeAnimatedLuaSprite ('mii,' mii ', -500, -300)

setLuaSpriteScrollFactor ('mii', 1.17, 1.17);
scaleObject ('mii', 1, 1);
addAnimationByPrefix ('mii', 'float', 'stagecurtains', 24, true)

sprites that only load if Low Quality is turned off
if not lowQuality then

end
addLuaSprite ('stageback', false);
    objectPlayAnimation ('buttons', 'float', false)

close (true); --For performance reasons, close this script once the stage is fully loaded, as this script won't be used anymore after loading the stage
end
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
yes i created a stage in lua script

makeAnimatedLuaSprite ('mii,'
you've put the ' after the ,