Boss Level: Generate a DSL

Surprisingly, this might actually be tractable.
Scalably typed already does this in JS land. So it would be perfectly possible to have a typesafe DSL in scala JS.
However, the implementations dead end at js.native calls. Point is, that we don't need those. Vega is declarative, so we only need to be able to serialise the object and properties to JSON. We can discard it's actual implementation!
Solution sketch :

fork scalably typed
replace it's js types with Jim ones <-- boss level.
find a way to serialise it's objects...

Alternatively, the Vega JSON schema is published here:
https://github.com/vega/schema
If you can generate case classes from JSON schema (surely someone has solved this problem?) then you can use circe on JVM and JS to encode/decode the DSL.

Like this: https://cchandurkar.github.io/json-schema-to-case-class/

@armanbilge dude I think you just speedran my boss level!
I'll have a look at this when I get a little time to devote to this projcet once more, but if that generator works as advertised, I can't see any reason this wouldn't work.
This project is currently wedded to ujson / upickle... but they work in scala JS land too...

Alas, life is not quite so simple!
Limitations

As of now, it does not support a few latest JSON Schema features such as allOf/ anyOf/ oneOf.

But maybe we can do this;
https://transform.tools/json-schema-to-typescript
And then wheel in scalably typed... ?

So this does indeed appear to be a proper boss level problem, having spent a little time with it

scalably typed is amazing
converting typescript schema to scala case classes, does not appear to be that easy.

There's still a lot of options to explore :) I guess my point is, focus on language/platform agnostic technologies.
For example, you posted this tool above: https://transform.tools
I see that it has features to transform JSON schema to:

Protobuf, which you can convert to Scala with https://scalapb.github.io/
OpenAPI schema (aka Swagger), which you can convert to Scala with https://github.com/joan38/kubernetes-client/blob/4a884aa5e4f0cd1f879f4ccb5a06aeeb9462c470/project/SwaggerModelGenerator.sc

Just tossing ideas out there ... there are many language-agnostic protocols, and many converter tools out there, and many many people with this problem. So, surely, there must be a way, but I am certainly it won't involve Typescript nor ScalablyTyped because those are specific to JavaScript which is exactly what you want to avoid.

I'm listening, and I originally thought the same. In this case, that tools doesn't work with schema -> protobuf. And the API literally comes out the same as I paste it in .
The swagger generator is new for me though. I will give that route a go...

This is the most promising route I have so far;
https://app.quicktype.io
Apparently generates Kotlin classes from a JSON schema. It looks like it does a pretty good job.
Kotlin syntax, is not so far from scala...
I also think that all modifications would be syntactic in nature, and therefore can be done in serial, i.e. without needing to parse a (complicated) syntax tree...
As that bit, is already done... which is attractive...

Very nice find! Looks like once-upon-a-time there was a PR to add Scala to that tool:

https://github.com/quicktype/quicktype/pull/843

I looked at this project and concluded that I'd start from kotlin...
https://github.com/quicktype/quicktype/pull/1932
There's a certain amount of low hanging fruit above that I've already picked.
Let's see if it gets any response from the maintainers...

This is quite the journey. I believe the PR above produces syntactically valid scala code now, using intersection types for unions. Or at least I believe the case classes "compile".
But intersection types turned out to be a giant gotcha.
In order to include scala3 in their test suit, I believe I'd need to serialise, then deserialise their test suite. I have been  bemused to discover that, as far as I can make out, scala JSON libraries, do not currently support intersection types. I guess it's a niche use case :-)!
So, I need to either revisit the way of encoding these types... stuck...

By "intersection types" do you mean | as in union types? :)
https://dotty.epfl.ch/docs/reference/new-types/union-types.html
(I believe intersection types are & https://dotty.epfl.ch/docs/reference/new-types/intersection-types.html)
In theory it should be possible to derive encoder/decoders for union types, but not sure in practice. Off the top of my head I can think of two problems:

Lack of compiler support e.g. https://github.com/lampepfl/dotty/issues/14770

How to distinguish between types. E.g. consider:
 case class A(foo: Int, bar: Option[Int])
 case class B(foo: Int, baz: Option[String])

If you want to decode { "foo": 42 } to A | B, which concrete class should you instantiate? It could be either. If you give A preference for being first, it breaks the commutativity of |.
Perhaps this is a contrived example, and such situations are rare in practice, but it does mean this is tricky to implement in general.

So, I need to either revisit the way of encoding these types... stuck...

Out of curiosity, is it possible to just used sealed trait like before the advent of the Scala 3 union? E.g.
sealed trait AorB
case class A(...) extends AorB
case class B(...) extends AorB

By "intersection types" do you mean | as in union types? :)
I do. I can't seem to change the "type intersection" and "set intesectoin" Venn diagrams. I am probably going to call these the wrong names forever. Not good. I think I have the right mental model though, so I guess that's something.

Out of curiosity, is it possible to just used sealed trait like before the advent of the Scala 3 union? E.g.
It must be. It is ... less beautiful, but I think I need to simply accept this. I either wait or get dragged into problem which are not in my wheelhouse.

Current status; I have

a first implementation that spits out types only,
a test harness...
200 unit tests which pass... if I don't de/serialize the examples...
... And 200 unit tests that fail if I do  :-(...

Need to do the union implementation... but after that... I'm backing you circe...

https://github.com/com-lihaoyi/upickle/issues/386
and the same problem with circe.
https://scastie.scala-lang.org/Quafadas/Pay4wq3zSMCger0ScI9wdQ/38
That's game over for the time being...

The -Xmax-inlines? What happens if you add the compiler flag? :)
https://github.com/armanbilge/gcp4s/blob/bd8949adc84dc2b01a76a272bcb5c26df4880916/build.sbt#L31

I can barely believe it, but this appears to work!
i.e. we now have a complete, type safe DSL that works exactly the same in both JS and JVM land.
To be released after a little playing...

@Quafadas that is absolutely mind-blowing 🤯 Phenomenal! Exactly, you thought I was sticking my nose in here for selfless reasons? 😉

It's so close...
https://github.com/Quafadas/dedav4s/issues/21
I think that's a bug in the Vega spec... but it reproduces all other parts of the spec beautifully!


RDKBDEV-2972: use sysctl_iface_set() to write to /proc files

Reason for change:
This function enhances code reusability, promotes safety through buffer overflow prevention and error handling.
Test Procedure: Sanity.
Risks: None.
Signed-off-by: Andre McCurdy<EMAIL_ADDRESS>

Internal ticket for tracking https://ccp.sys.comcast.net/browse/RDKCOM-5108


Upload the hand-tools QB software

This is required to create a hardware interface for the hand, and build upon the ros_control framework. The current ROS node for the QB tools should be discarded when this is functioning.
@laurentio86 Lorenzo, could you upload the current hand API as we talked long time ago? Thanks.

The API was added as git submodule.


Re-add bower.json

Would it be possible to re-add bower.json?
Removed here: https://github.com/chartjs/Chart.js/commit/c6318749da660f6d8599b466b3871f437123c1d0
It breaks several things, like:
https://github.com/2amigos/yii2-chartjs-widget/issues/18
https://github.com/fxpio/composer-asset-plugin/issues/242
https://github.com/bedezign/yii2-audit/issues/159

You can still install with bower through the bower-npm-resolver as explained in the installation portion of the docs.

Further, with the build system changes this cannot be installed via bower because the release source code zip doesn't contain the built files

We're installing the plugin via https://github.com/fxpio/composer-asset-plugin - so I don't think npm-bower-resolver would work here. Probably the best option for the widget is to switch to npm.
Just to be curios, what was the reason behind dropping bower-support, like maintaining two package managers or did bower just not work fine?

I am curious as well as to why bower support was dropped.

Indeed, dist folder was useful for dependencies management, angular wrappers or whatever

You can get the dist folder through building with gulp build.

Could you gulp build for us?
This adds another step to the deployment process. And I'm not planning to add gulp just for one dependency.

There is a way to include the dist folder only on releases even if this folder is ignored by default, this way it will be available when checkout releases tags, popular repos like jQuery or full calendar manage distribution on releases like this.
I guess you are currently manually adding the compiled files on each release. However you could add this behavior automatically in your release task in gulp, this way everybody will be happy and no dist folder will be in source control, only on releases :)
What do you think?

@miguelmich the release already happens automatically. @simonbrunel set it up so that it adds the built files once the build finishes. As far as I know, there is no way to have those files inside the source zip (which is what bower uses) without checking those files in.

@schmunk42 @fabianTMC: main reasons why we removed dist files (and thus bower support):

not obvious which version was compiled (in master/branches)
confusing because these files were not always up to date with latest sources
pollutes commit history and PR since many people were committing their own built files in PR
makes more complicated manipulating git history for developers, such as rebasing

@koxon: we already build them for you, no need to deal with gulp or even npm. You can get it from the Git release page or the CDN (or the npm package if you use npm). We provide many formats, see #2555 for details.
@miguelmich: the only manual process is the Git tag creation, everything else is automate: build, git release (attached files), NPM release and deployment to the CDN (described in #2555). But I see what you mean with jQuery, they have the Git tag automatically created and in the same time they push dist files only for the tag. So finally, dist files are never on master/branches, neither PRs. Could be something interesting to investigate :)
I don't know bower enough, but I'm sure there are different ways to grab libraries from URL (Git release downloads or CDN) without relying on Git repository, right?

Hello @etimberg @simonbrunel,
Ok, now I understand better your workflow, however one of the main concerns about using direct urls is that we can't manage version tags.
It is also possible to include the dist folder only in the releases tags and master branch will be clean always for PRs like mentioned here: (https://github.com/fullcalendar/fullcalendar/issues/2989#issuecomment-170825613)
This could be an addition to the current build process instead of some kind refactor, If you want I can make a research more deeply about this and give you a better idea of how it could work.
Thanks

Hello guys,
I want to share what I found so far; The behavior mentioned above is possible to achieve following these steps in the release task:

Run build task
Checkout a detached state
Add dist folder files to commit
Commit these files
Create a tag
Push tag to origin
Reset the detached state
Checkout HEAD

This process is explained in more detail here, and these two repos are using the same releasing workflow as you can see in their release tasks:
https://github.com/fullcalendar/fullcalendar/blob/master/build/build-release.sh
https://github.com/melonjs/melonJS/blob/master/tasks/release.js
Both are using a symbolic reference pointing to HEAD (git symbolic-ref HEAD) in order to return to the previous branch prior to the detached state.

Thanks @miguelmich, will look it more closely, however our current build/release process is triggered from the git tag creation, so this solution doesn't seem to work out of the box and would require other major changes in our workflow.
The first link you posted is from 2014, but since there are alternatives, such as bower-npm-resolver. So I wondering why this solution is not suitable?

Ok I understand your point of view, but personally I don't like to install additional plugins to handle a default behavior in bower using a "workaround" just for one dependency.
If you guys decided to not support bower you should have your reasons and I respect that :)

I too respect your decision to drop bower support. What's naughty though is that you guys do that without respecting semver. Many bower based projects use the semver range ^2.x.x, expecting braking changes to occur in major version changes only. Now many bower based projects install version 2.2.x with the braking change of not having build files.

@miguelmich: nothing decided yet, we need to be sure that there is no other way than dealing with dist files in the repository tags to support Bower. It will potentially impact the whole Chart.js release workflow, so it needs to worth it (meaning that alternatives solutions are really not acceptable). I can understand why you don't want to use bower-npm-resolver and I'm not sure why Bower doesn't support npm packages natively since it requires npm itself. IMHO, bower-npm-resolver should be built in Bower.

... main concerns about using direct urls is that we can't manage version tags

What do you think about using npmcdn.com for version management, would it work for you? For example, you could use:

bower install<EMAIL_ADDRESS>bower install<EMAIL_ADDRESS>...

@pkaske you absolutely right :\ I guess the confusion came from that the lib itself is still compatible with 2.x.x, but only Bower integration needs adaptation.
Note: reopening this issue to keep track of the discussion about Bower support.

Yes, I think I can use the cdn url since it's only one file

Another small plug for native bower support.  We include bower packages inside of rails using rails-assets.org, which re-packages bower packages as Ruby gems.  This only works if it's a native bower package and doesn't use any extra plugins like npm-resolver or custom urls.  I'm not suggesting that you do any extra work for just that reason, because this workflow is pretty niche, but this is one more small reason why native bower support with a dist folder would be nice for some people

I'm use bower install npm:chart.js --save, but wiredep (gulp plugin) in my project incorrect inject chartjs:
<script src="/bower_components/chart.js/src/chart.js"></script>
and i give error: chart.js:4 Uncaught ReferenceError: require is not defined

@700ghz You are using the wrong version of Chart.js. The only versions that you should be using for production are in the dist folder. You should change your src to be /bower_components/chart.js/dist/Chart.min.js. (use Chart.bundle.min.js if you want to use time scale)

In file "bower.json" we have:
{
  "name": "chart.js",
  "description": "Simple HTML5 charts using the canvas element.",
  **"main": "src/chart.js",**
  "license": "MIT",
  "homepage": "http://www.chartjs.org",
  "repository": {
    "type": "git",
    "url": "https://github.com/chartjs/Chart.js.git"
  },
  "dependencies": {},
  "devDependencies": {}
}

In line:
 "main": "src/chart.js"
should be:
"main": "dist/Chart.js"
Now gulp wiredep work correctly.

We are implementing a new system very soon that should allow full bower implementation, just as it used to be before the recent update.

@zachpanz88 Awesome!  I'm excited to be able to upgrade to the latest version.  Thanks!

Fixed in version 2.3.0-rc.1, no need to use bower-npm-resolver anymore, native Bower support is back.

Thank you @miguelmich for pointing us on that process to push dist/* files along release tags :) (our implementation is quite similar to the fullcalendar one and is automated by Travis).

@simonbrunel I'm glad to hear that, you have done a great work guys.


f_open may mess up cyw43_gpio_irq_handler?

Problem
Seems like the f_open may override others interrupt? I am totally not sure about this.
Setup
I have an "almost minimal" example attached here. But let me sum it up first.
My application will run both core:

core 0 is for bluetooth and its setup.
core 1 will initialize the sd card, then open a file using f_open when i press a button, and close it when i press again, and do it infinitely.

Step to reproduce
So during the run. I try to connect using the rfcomm command from linux:
sudo rfcomm bind /dev/rfcomm0  "your pico bluetooth address" 1, then open that port /dev/rfcomm0 using minicom
What went wrong

So, if i try to connect with minicom right at the start, then it worked normally, i can still receive the packet through the packet handler.(It does not actually connect, but the point here is the packet handler is called normally).
BUT, if before that, i press the button once or twice. Then the packet handler will not be triggered AT ALL.  Added a breakpoint at cyw43_gpio_irq_handler  in cyw43_driver.c, and noticed that normally a gpio will trigger this handler, but after f_read, it cannot be triggered anymore!

multicore_example.zip

I don't have time to study the example at the moment, but you could try changing the IRQ number used by no-OS-FatFS-SD-SPI-RPi-Pico. Which branch are you running?
In the sdio branch, you can specify a DMA_IRQ_num in your hw_config.c. This is documented in the README:

DMA_IRQ_num Which IRQ to use for DMA. Defaults to DMA_IRQ_0. The handler is added with irq_add_shared_handler, so it's not exclusive. Set this to avoid conflicts with any exclusive DMA IRQ handlers that might be elsewhere in the system.

In the master branch there is a set_spi_dma_irq_channel call you can make. See spi.h.

I don't have time to study the example at the moment, but you could try changing the IRQ number used by no-OS-FatFS-SD-SPI-RPi-Pico. Which branch are you running?
In the sdio branch, you can specify a DMA_IRQ_num in your hw_config.c. This is documented in the README:

DMA_IRQ_num Which IRQ to use for DMA. Defaults to DMA_IRQ_0. The handler is added with irq_add_shared_handler, so it's not exclusive. Set this to avoid conflicts with any exclusive DMA IRQ handlers that might be elsewhere in the system.

In the master branch there is a set_spi_dma_irq_channel call you can make. See spi.h.
I am using master branch, so
We set it right before the stdio_init_all right?

That feature was added by MCJack123 in #14, "Add DMA IRQ channel selection", but I think you could do it any time before the first f_mount.
Meanwhile, I have merged in a fix to the DMA IRQ handling that was missing in the master branch. It was using a common interrupt handler for DMA_IRQ_0 and DMA_IRQ_1 but only matching on the channel number, not the IRQ number. I made a couple of other improvements to the IRQ handling while I was at it. So, you could try pulling the latest and see if that makes any difference.

I am terribly sorry that i ignored the comment, really. I am pulling the latest now to see if it work. Update in a moment.

Currently testing, but it is still not working.

I have a lead on the problem.
Read the bluetooth CYW43439 datasheet yesterday, and it seems like the Bluetooth interface must pull its Chip Select Line, which is GPIO 25 in Pico W, low and only then can it write data to the Pico through the GPIO24 line. SO what i did is after every f_read, and f_open, i manually set the CS line high again, so the bluetooth device can pull the CS line down again. And it did work. ( I hope the explanation I just gave makes sense). But again it is just a workaround.
So the problem now is

Why do f_read and f_open operations pull GPIO 25 low? (Other operations might do the same, but I'm unsure.) If this isn't intended, it might be a bug.
I'm having trouble pinpointing the code causing this. Could you provide some direction on where to look for this bug? If I can fix it, I'll submit a pull request.

What is your no-OS-FatFS-SD-SPI-RPi-Pico hardware configuration (hw_config.c)?
Maybe there's supposed to be a pull on GPIO 25, e.g. gpio_pull_up(25);.

Yeah i just found this in your code at sd_driver/sd_spi.c
    static void sd_spi_deselect(sd_card_t *pSD) {
        gpio_put(pSD->ss_gpio, 1);
        LED_OFF();
        /*
        MMC/SDC enables/disables the DO output in synchronising to the SCLK. This
        means there is a posibility of bus conflict with MMC/SDC and another SPI
        slave that shares an SPI bus. Therefore to make MMC/SDC release the MISO
        line, the master device needs to send a byte after the CS signal is
        deasserted.
        */
        uint8_t fill = SPI_FILL_CHAR;
        spi_write_blocking(pSD->spi->hw_inst, &fill, 1);
    }

Why the LED_OFF though?

Oh, yeah, that must be it. The LED was used just to show activity on the SD card. It's something familiar to those of us who remember the early PCs. I don't really use it anymore and had more or less forgotten about it.
You should be able to fix it by simply defining NO_PICO_LED. Something like
add_compile_definitions(NO_PICO_LED)

in CMakeLists.txt should do it.

Oh, yeah, that must be it. The LED was used just to show activity on the SD card. It's something familiar to those of us who remember the early PCs. I don't really use it anymore and had more or less forgotten about it.
You should be able to fix it by simply defining NO_PICO_LED. Something like
add_compile_definitions(NO_PICO_LED)

in CMakeLists.txt should do it.

i think i will open a pull request instead (after i am done with my thesis) to make it work out of the box on Pico W.  But for now, I will close the issue.
Thank you a lot for the guidance!


Remove starting application installation step

In Elixir >= 1.4 we no longer need to add the application because it is
configured in the dependency.
Even knowing of the change to Elixir 1.4 I got caught up on this because
I was focused on getting wifi setup. Since I added applications to my
application config I ran into other modules not working on my device. In
my host config I didn't have that issue so the problem was further
exacerbated by the modules being available on the host iex session.
Amos King @adkron<EMAIL_ADDRESS>

@adkron thanks!


Hotkeys require window focus

I've been trying to get this to work on my machine and one particularly frustrating caveat is that the hotkeys only work when the window is focused. I've got other OBS plugins that can get key inputs just fine (input-overlay) so this appears to be specific to the livesplit extension, although I could be wrong!
I'm running on Arch Linux with the latest git version of OBS and this extension, and running on GNOME (Wayland). I tried setting the hotkeys to a bunch of different ones but it appeared that they were only being registered when the window was focused.

The hotkeys are entirely managed by OBS, so this would probably be an upstream bug. You could try to do some debug prints to see if they at least reach our Rust code (confirming it's a bug on our side).

It's been a while since I filed this, but I believe that the issue is that there's no really good protocol at the moment for hotkeys under Wayland, and GNOME and OBS certainly don't implement any in common. So, I'm kinda stuck dealing with that issue, and it's probably not this.
Will close.


Community notes data is again out of date – most recent contribution was 7 days ago

A similar issue occurred not very long ago – I flagged it in #254.
It looks like it is happening again – the note on https://x.com/i/communitynotes/download-data currently says:

Data includes contributions up until 2:03 AM · Aug 18, 2024

I.e. 7 days ago.
It’s still useful to have these bulk data downloads even with this delay, but it would be useful to know if this length of delay should be expected going forward.

Retrospectively closing this, as it was resolved.
There does seem to be a general problem, though, that community notes data always lags by more than the documented one day, and sometimes by significantly more.

it's been down again for the last 5 or so days.
stuck on 1437535 notes

Thanks, @plopnl – I’ve reopened this issue accordingly.

In fact, I’ll close this (because it was resolved at the time) and will open a new issue.


Feat: Added link of dynamic list of contributors

Fixes #61
Hi @engineerjoe440,
Please review and let me know if any changes are required. : )
Thanks

I was about to share one screenshot. Anyway thanks for merging.


PPO mini-batches with tensors (instead of datasets)

This PR seems to make the tests fail and has some tricky indexing and integer division issues. Closing for now, let's re-open a new one once that is corrected.


Feature Request - Add Policy for "Deploy Diagnostics settings for Log Analytics to Log Analytics workspace"

It is possible to send diagnostic logs and metrics to for a Log Analytics workspace to a central Log Analytics workspace.

Since individual Landing Zones often deploy their own Log Analytics workspaces, it would be nice to include Log Analytics to the Azure Policy initiative Deploy-Resource-Diag deployed by ALZ.
It could then configure diagnostic logs even for the central Log Analytics workspace deployed by ALZ, and in practice send the diagnostic logs to itself.

Hey @janegilring,
Good call. Do you want to contribute this via a PR following this as an example: https://github.com/Azure/Enterprise-Scale/blob/main/src/resources/Microsoft.Authorization/policyDefinitions/Deploy-Diagnostics-Bastion.json and then add to this initiative https://github.com/Azure/Enterprise-Scale/blob/main/src/resources/Microsoft.Authorization/policySetDefinitions/Deploy-Diagnostics-LogAnalytics.json?
Let us know
Thanks
Jack

@jtracey93 Definitely, happy to contribute 👍

Cool ill assign this to you, let me know if you need any support.


fig 3

why don't we show the green rectangle and dot in fig 3, just like we do at the bottom of fig 1?
that is a real question, is it because we have never thought of it?
if we show it (and it looks good), it seems to me that it makes our paper stronger?
i vote we investigate, and decide once we've looked at it, whether to include it, or try to get the referees to request it.

ok, I will plot the largest rectangle (or dot) containing the highest power scales and show you

updated figure & caption on overleaf, what do you think?
Note that the green dot is the highest power, where the green box is the largest rectangle such that all scales within is at most 0.1 power different from the max power. (increasing 0.1 will enlarge the box, reducing 0.1 will reduce the box. If too small, it will just be a dot, so I set it to 0.1 for better visual)

it looks good, but i wanted to show the performance of sample MGC, eg,
generating the green dot/box the same way that we do for figure 4, know
what i mean?
On Sun, Dec 18, 2016 at 7:51 PM, cshen6<EMAIL_ADDRESS>wrote:

updated figure & caption on overleaf, what do you think?
Note that the green dot is the highest power, where the green box is the
largest rectangle such that all scales within is at most 0.1 power
different from the max power. (increasing 0.1 will enlarge the box,
reducing 0.1 will reduce the box. If too small, it will just be a dot, so I
set it to 0.1 for better visual)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/171#issuecomment-267860140, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjcmcYOy11X6nCubSX_BhPiyNwoPBuks5rJdUdgaJpZM4LQCnU
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.

there is no power map associated with sample MGC though.
Also, we generate 1000 dependent pairs for power computation, which there are 1000 sample MGC statistic for the alternative, and 1000 green dots & boxes when doing permutation test. There is no good way to show them?

i know, just show 1 sample estimate, to demonstrate that our estimate of
the optimal scale is reasonable.
On Sun, Dec 18, 2016 at 9:02 PM, cshen6<EMAIL_ADDRESS>wrote:

there is no power map associated with sample MGC though.
Also, we generate 1000 dependent pairs for power computation, which there
are 1000 sample MGC statistic for the alternative, and 1000 green dots &
boxes when doing permutation test. There is no good way to show them?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/171#issuecomment-267865990, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjci8qOagOS_6B0e26pZoXRbOBcZ4_ks5rJeWfgaJpZM4LQCnU
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.

Having some admin stuff to do first....I will update figure 3 tomorrow morning (US time).

Ok I tried this one.
However, if we just show the green box & green dot based on one sample data, I find that they are not always reasonable, i.e., may not align well with the deep blue colors for some of them.
I think if we can find a way to show combined green boxes & dots from multiple data, they should fit better with the power map...But I suggest we drop this idea for now, since it is difficult to show & does not align well with the scale discovery section anyway.
What do you think?

wait, seriously, the one in there now is using sample mgc?
i think it looks totally amazing, no?!
what's wrong with it?
On Tue, Dec 20, 2016 at 7:43 AM, cshen6<EMAIL_ADDRESS>wrote:

Ok I tried this one.
However, if we just show the green box & green dot based on one sample
data, I find that they are not always reasonable, i.e., may not align well
with the deep blue colors for some of them.
I think if we can find a way to show combined green boxes & dots from
multiple data, they should fit better with the power map...But I suggest we
drop this idea for now, since it is difficult to show & does not align well
with the scale discovery section anyway.
What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/171#issuecomment-268234318, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjcjxixaQpVXhAC4VFze_YBUBCS8sEks5rJ82CgaJpZM4LQCnU
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.

no the current one is based on maximizing power, i.e., the true optimal scales.
if you want to keep that it will be fine.
but the box by sample mgc will be very different when based on one sample data. In the extreme, if MGC does not detect dependency on the given data, there will be no green box....so I think it will be confusing to show the greenbox for one sample data, unless we run a few and purposely select the one that matches the best with the true optimal scale. But then it is better to just show the true optimal scale.
What do you think?

please put in the sample scales and let me know when done.
On Tue, Dec 20, 2016 at 5:59 PM, cshen6<EMAIL_ADDRESS>wrote:

no the current one is based on maximizing power, i.e., the true optimal
scales.
if you want to keep that it will be fine.
but the box by sample mgc will be very different when based on one sample
data. In the extreme, if MGC does not detect dependency on the given data,
there will be no green box....so I think it will be confusing to show the
greenbox for one sample data, unless we run a few and purposely select the
one that matches the best with the true optimal scale. But then it is
better to just show the true optimal scale.
What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/171#issuecomment-268384241, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjct2o_wDiU5orh0mlhUiMNSZi-c8bks5rKF3MgaJpZM4LQCnU
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.

put in the 1d case for now, see last figure in appendix. (Now it is not too bad, as I force data re-generation and re-run if sample MGC fails to test significance)
if you like it, I will put in the hd case?

i think i love it!
do you not love it?

hmmm, it is a bit away from true optimal though, but I can accept it.
I will update the hd heatmap in a few hours after an errand.

ok updated figures and codes. Looking great now!
still, you may want to take a look at the figure 3 caption & the scale discovery section, polishes are necessary for my additions there :-)

it looks great, but the spiral one has got to show a local scale that it finds!
and why did you remove the green dots/boxes from the 1D figure?

setting 20 box is weird.

the green dots are there til the code is screwed...fixed now, will upload to git and overleaf in a while.
setting 20 is independent, so there shouldn't be any green box and dot anyway, right?

ok updated figure 3.
note that although most boxes are of high power, there are usually one to two plots whose box are not very close to optimal, each time we re-generate.
If you want me to artificially generate a perfect figure I can do as well, but I think the current one is mostly ok?

i think its great!!!


One solution of "Following genes from MutSig results are not available in MAF"

Hi, dear @PoisonAlien ,
One can often see warning messages like this：

I found a reason for this problem.
When we run mutsigcv, we first need to prepare the input file with the preparemutsig() function of maftools. Just like:
foo <- prepareMutSig(maf = maf)

This function uses the corresponding table in “hugo_to_mutSigSymbol.txt.gz” to convert “Hugo_Symbol” to "MutSig_Synonym". However, the oncoplot function does not have the function of turning "MutSig_Synonym" back to “Hugo_Symbol”. As a result, some genes will be lost.

Hi,
ideally you should use the same maf that was used for mutsigCV to avoid such problems.

Using mutsigCV's MAF is really a solution.
I wrote a function to convert the gene name back.
: )


Add Audio support

expected that there might be call for multipart/form-data or something to keep it DRY


Fix loading button issue after submitting empty file reference

closes #502
📑 Description
✅ Checks

[ ] I have updated the documentation as required
[x] I have performed a self-review of my code

ℹ Additional context
works as expected after change

@petar-cvit can you please review this pr?


rfcomm: extend indicators

some (old) bluetooth headsets require additional indicators.  One of
these headsets is a Plantronics 510.
Otherwise, the headset never requests indicator statii, halting state
transitions to the "connected" state.
I noticed that this headset requested the indicator statii when connected to an android phone.  When inspecting the bluetooth capture, this was the difference.  Doing the same in bluealsa fixed the issue.

Many thanks!


[Bug] local installation fails to launch

OS: Debian Bookworm
I cloned the latest master and built and installed it. lauch frog gets's me the following error:
/usr/local/bin/frog
Traceback (most recent call last):
  File "/usr/local/bin/frog", line 59, in <module>
    from frog import main
ModuleNotFoundError: No module named 'frog'

Another thing I noticed: meson references release 1.3.0 but GH has only 1.2.0.
Furthermore, meson emits a warning about outdated features:

Project version: 1.3.0
Host machine cpu family: x86_64
Host machine cpu: x86_64
meson.build:16: WARNING: Project targets '>= 0.50.0' but uses feature deprecated since '0.48.0': module python3.

The failure is due to a Debian-specific packaging layout, described in:
https://github.com/mesonbuild/meson/issues/8739
The deprecation warning is due to using the old meson "python3" module rather than its replacement, "python". I've made a PR in #115

Since it is a Debian-specific problem I close this issue.


teach flutter drive about --no-build option

flutter run knows about it but flutter drive doesn't.

I don't remember exactly what this was about, but I think --use-existing-app covers this to some degree already. Closing until it comes up again.


SocketIO not confirming session_request in v2.3.x

Rasa version: 2.3.x
Python version: 3.7.8
Issue: SocketIO not confirming session_request in v2.3.x
Hi,
There seems to be a bug( or undocumented change) in rasa v2.3.0 and v2.3.1. When trying to connect to a rasa project via sockets there is no session_confirmation send back from the server to the client.
What did I do:

New python environment with python 3.7.8 and pip 21.0.1
pip install rasa
rasa init
changed credentials.yml to allow socketio with “user_uttered” and “bot_uttered” set as events
rasa run --cors “*” --debug
used the webchat/botfront code snippet to test (GitHub - botfront/rasa-webchat: A feature-rich chat widget for Rasa and Botfront 1)

The webchat-widget is able to create a socket connection to rasa, but rasa does not respond to the “session_request” event.
I tested different rasa versions. Rasa 2.3.1 and 2.3.0 seem to have the problem. Every version before that works just as expected. (<=2.2.10).
For now, I will be using v2.2.10 until this is fixed, or did I miss any documented changes?
Ben
PS: I deleted the bug template since it does not apply to this bug.
I originally asked the question on the rasa forum here
The problem might be related to an update to the python-socketio library source

Thanks, @desmarchris and @b-quachtran will look into this to figure out whether it's the socket library.

@SirBenedick can you also provide your webchat definition, so we know what version of that code its using?

@SirBenedick can you also provide your webchat definition, so we know what version of that code its using?

Sure, its exactly the example code from here, with a modified socketUrl. I did not explicitly define the version to be used, I am assuming it is 1.0.0.
The exact same Webchat works with rasa versions <=2.2.10
<html>
<head>
<script>!(function () {
  let e = document.createElement("script"),
    t = document.head || document.getElementsByTagName("head")[0];
  (e.src =
    "https://cdn.jsdelivr.net/npm/rasa-webchat/lib/index.js"),
    (e.async = !0),
    (e.onload = () => {
      window.WebChat.default(
        {
          customData: { language: "en" },
          socketUrl: "http://localhost:5005",
          // add other props here
        },
        null
      );
    }),
    t.insertBefore(e, t.firstChild);
})();
</script>
</head>
<body></body>
</html>

It looks to me like rasa-webchat should upgrade to the latest socket.io library. I opened an issue there.


Encode as UTF-8 when posting

We had an encoding/umlaut problem in a project, which is solved by encoding the payload correctly.
Please review and merge.

Very nice!
Have you checked if we use the default encoding (which is ISO-8859-1 set in DEFAULT_CONTENT_CHARSET) elsewhere? If so, just ignore and merge.


2.0.x Not responding to touching right after close animation

After close animation (removing a screen) finish, the screen freeze for about half second, not responding to touching. Only happens when animationEnabled is true, and open animation (adding a screen) is not affected. Not sure if it's related to TransitionSpec because I feel like using TransitionIOSSpec is a lot worse (freeze longer) than FadeOutToBottomAndroidSpec.

I think this commit fixed this issue
https://github.com/react-navigation/navigation-ex/commit/15fe3ebb51266fcd4b9a4c0baf59424bd92ff054
but I tried to quickly navigate after back using gesture in IOS, it feels jagged because gesture trigger StackAction.pop() and called a bit latter after the animation being done, I tried fix in this commit:
https://github.com/react-navigation/navigation-ex/pull/244
cc: @satya164

Thanks! Haven't tested yet. @Jekiwijaya could you elaborate a bit more why removing PointerEventsView helps?

It's because previously every screen on the stack the pointerEvent are disabled except the topmost screen. And now the commit only disabled while the screen is closing

Same for me. Works in 1.10.2, doesn't work in 2.0.15
#353 related

I believe this shouldn't be an issue anymore. Please try the latest version and let me know.


Last is not update when switching to last project

# last = x
pr y
pr x
# last = y
pr y
# last = y

This has been fixed during other work 🤷


Support for CooperativeMatrix extension?

I know this is Nvidia only extension at the moment, but is there a
plan to support cooperative matrix extensions (GL_NV_cooperative_matrix, GL_NV_integer_cooperative_matrix)?
Examples are here:
https://github.com/jeffbolznv/vk_cooperative_matrix_perf/tree/master/shaders
More and more shaders are using this extension.

Hi, I  also started using GL_NV_cooperative_matrix recently in a project. When using the extension, spirv-reflect kept crashing.
My fix was to add this case to ParseNodes(SpvReflectPrvParser*) in spirv_reflect.c:
--- a/extern/SPIRV-Reflect/spirv_reflect.c
+++ b/extern/SPIRV-Reflect/spirv_reflect.c
@@ -762,6 +762,7 @@ static SpvReflectResult ParseNodes(SpvReflectPrvParser* p_parser)
       case SpvOpTypePipe:
       case SpvOpTypeAccelerationStructureKHR:
       case SpvOpTypeRayQueryKHR:
+      case SpvOpTypeCooperativeMatrixNV:
       {
         CHECKED_READU32(p_parser, p_node->word_offset + 1, p_node->result_id);
         p_node->is_type = true;

In my understanding of spirv-reflect this just interprets the fcoopmatNV as an opaque type without any introspection, but allows the parser to not fail in parsing the file.
I do not know enough about spirv-reflect to know if this is a good idea. Maybe a developer can look into this and decide if this fix can be merged?


Add :db.type/{instant, uri, uuid} types to bootstrapper. Fixes #201

This is missing a v3.db to run in the test_open_current_version test in db/src/db.rs,but looking for a sanity check here. Commenting out that test, everything else passes. Unlikely I'll tackle the creation of a v3 DB in ./fixtures and the SQL update migration for v1 -> v2 -> v3

Rebased, but still failing in the test that uses the v2 database @ db/src/db.rs:884

Obsoleted by #438.


Android build fails due to missing namespace

This issue occurs on 2.9.0, but not 2.8.0. I suspect it is related to this code change: https://github.com/chyiiiiiiiiiiii/flutter_zendesk_messaging/commit/88d086473f570d7f96aef81e2719bebfdaf0538b#diff-197b190e4a3512994d2cebed8aff5479ff88e136b8cc7a4b148ec9c3945bd65aL29-L32
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':zendesk_messaging'.
> Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl.
   > Namespace not specified. Specify a namespace in the module's build file: /home/runner/.pub-cache/hosted/pub.dev/zendesk_messaging-2.9.0/android/build.gradle. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace.

     If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant.

Thanks. I think it does not appear in every project. But the new version v2.9.1 has recovered for setting.

Looks good in 2.9.1. Thank you @chyiiiiiiiiiiii!


Would be nice to override all default blueprints to create .ts files

I don't think we need to actually override any blueprints, just expand what we are currently doing on post install to make sure we:

Convert dummy app files to .ts
Add a tsconfig.json to dummy app
Add denali-typescript to dummy app package

I guess what I meant is when we do denali generate migration foo-bar it still produces a .js file and then I have to go manually convert it.

Ah. To fix that, denali-typescript would need to ship with all the blueprints Denali core does. That's certainly doable, and unfortunately I don't see a more efficient way of doing it.
We also need to think through precedence here. If both denali and denali-typescript ship with an action blueprint, for example, and I type denali g action, which should run? Right now you can always prefix the addon name to be explicit, i.e. denali g denali-typescript:action, but I'm wondering what the default behavior should be (not sure what it is right now).

Hmm, good point. What if denali cli had some sort of events that fire through the lifecycle of calling a blueprint that typescript could hook into?
i.e.
afterBlueprint()?
Then in there denali-typescript could just go through and run .js -> .ts on all the files again.

That might work for some use cases, but I think the ideal solution for denali-typescript users would be if the generated blueprints weren't just JS files with the TS extension, but were actually typed Typescript code.
Another way to think of this: how would denali-coffeescript work, since unlike Typescript, regular JS isn't valid Coffeescript?

Ah good point.
On Mon, Aug 28, 2017, 11:41 Dave Wasmer<EMAIL_ADDRESS>wrote:

That might work for some use cases, but I think the ideal solution for
denali-typescript users would be if the generated blueprints weren't just
JS files with the TS extension, but were actually typed Typescript code.
Another way to think of this: how would denali-coffeescript work, since
unlike Typescript, regular JS isn't valid Coffeescript?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/denali-js/denali-typescript/issues/8#issuecomment-325441655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACDIwqeBrlmTBI6loqIJxYp5tcaX637dks5scwnqgaJpZM4PAUDS
.


Add chrome stable 86 and Edge 86

Chrome 86 was released as stable 4 days ago, and yesterday Edge followed suit

They are (now?):

Yes this was added in a patch 14 hrs ago. Closing as fixed


[winpr,sspi] skip IP addresses for kerberos auth

kerberos requires hostnames to authenticate, if we connect with a IP address do not try kerberos based authentication

@giox069 can you test this branch and tell if it works for you?
simple check if we have an IP address as hostname and then fail kerberos early, should be sufficient for your case.

@akallabeth it works, now connection is successful when I connect via IP address.
But an attempt to contact kerberos server is done even if I used an ip address for /v.
Kerberos is not 100% disabled... in my case I can accept it, and this patch should solve all my problems.
But I think I have a case where I cannot reach a KDC when connecting to a win11 PC... i will test it later.

@giox069 yes, kerberos is not disabled here (as we only know that the machine name is an IP address later on), but at least it fails gracefully with NTLM fallback, which was the intention here.


Add variable for cursor

Add variable for cursor on buttons? :+1:  :)
I'm not sure about the justification for having the pointer cursor on buttons, but it doesn't sit quite right for me, and I'd love to have the option to change it without hacking it. The comments even say it's "opinionated" :)

I think the code you're referring to @emilmuller relating to cursor: pointer being added to buttons is this:
.btn:not(:disabled):not(.disabled) {
    cursor: pointer;
}

Buttons and anchors should ideally have cursor: pointer to aid with web accessibility and whilst adding some variables for cursors on buttons might be a good idea, personally, I think we should add some utility classes for some of the popular cursor types which would also allow for styling 'fake' buttons/anchors which might not otherwise be real anchors (e.g - a <span> tag within a anchor)
I'd be curious to know what others think about this, but regarding the utility classes, I think the following would be a good start:

pointer
default
help
text
not-allowed

We've talked several times about when to use each cursor. Maybe having cursor utilities is a good idea.
cc/ @patrickhlauke what do you think?

frankly, i tire of the back-and-forth on this point ;)
also, personally not a fan of adding classes for everything. at some point, we'll end up with a class to reflect every single CSS property at this rate...
deferring to @mdo and others


Create: 12-Isomorphic-Strings.java

File(s) Modified: 12-Isomorphic-Strings.java
Language(s) Used: java
Submission URL: _https://leetcode.com/submissions/detail/832488698/_

Thanks, @tharunkanna14!


[BACKLOG-37633] Fixed 'What's New in 9.5' and fixed 'Avro_Output' links

:white_check_mark: Build finished in 7m 55s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
assemblies/static,plugins/avro-format/core

:ok_hand: All tests passed!
Tests run: 42, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Bug: no-missing-import doesn't use the require algorithm when TS has CJS moduleResolution?

Environment
Node version: v20.13.1
npm version: v10.8.1
ESLint version: v9.11.1
eslint-plugin-n version: v17.10.3
Operating System: Windows 10
What rule do you want to report?
no-missing-import
What did you expect to happen?
The plugin should find imports from TS barrel files.
Participation

[ ] I am willing to submit a pull request for this issue.

Additional comments
No response

How to add Barrels in TypeScript

Right, that concept I’m aware of, not sure that’s the common name for it though, never really heard that term before.
Is this limited to TS or goes for all files that does that kind of thing?
And what error are you getting?
From the rule description:

This rule checks the file paths of import and export declarations

But “should find imports from TS barrel files” sounds like it’s not about paths in your case but about what’s imported? Path validation shouldn’t care if it’s a “barrel file” or non-barrel one

Following throws errors for me:
import {Foo} from '../../foo/bar';

where bar is existing folder containing index.ts exports.

Right, then the issue is probably this:
You use TS-syntax that compiles to CommonJS syntax, since only CommonJS (require()) supports resolving ./something to ./something/index.js
ESM imports are required to include the full path, including file extensions.
Have you specified your tsconfig? https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/no-missing-import.md#tsconfigpath

No, I am not using CommonJS anywhere in the application. I am using ESM everywhere and everything compiles just fine.
node-ts for development and esbuild for production builds.
I am not using full paths anywhere and everything transpiles just fine.

Can you break this example using a barrel?
https://eslint-online-playground.netlify.app/#eNp1kM1uwyAQhF8FcbFUJXZ7ipKotz5A7yGVEKwtt2ZBBreuLL97F3DtqD8+GMR8M+wwcd+rqsUAfS0V+ErJUL56fuIwOtsHpix6+svAHllBS3Hmu+SBURrXQYZbk+ApgdKz8GHZzOreGlaUt/EtahjJs8b8oW2X360ReaziLPCXpG3zT2BWfnah09iFluQB35GlJK1uF3xpg1bDczc0Lea7xMLuXTrco+AC14E01HLoArsIZPRt5iXaXwSvOxmqHpQ1Bqis3hurhw4Ev+4EXuM0Tqo32cRXtUiTTDFM8PDpCDrR7ttAfBQ0vD+Bi1moWvCRSZao5WGz7eVYHsr7xbaJN00S9XAojxGL1Cxw5vMX0wG2aw==
(Just paste the new url here once you have it not working)

So it's exactly what @voxpelli said:

You use TS-syntax that compiles to CommonJS syntax, since only CommonJS (require()) supports resolving ./something to ./something/index.js
ESM imports are required to include the full path, including file extensions.

You will need to specify the full path to the file path (including file extension)
So, this will error:
import { cat } from './interfaces';
And this will not:
const { cat } = require('./interfaces');

Ok, if TS code with barrel imports is not supported and there are no plans to introduce the support, feel free to close this as "won't fix".

It should be doable, the basics is just adding mainFiles.push("index") to https://github.com/eslint-community/eslint-plugin-n/blob/master/lib/util/import-target.js#L293. However, we would need to ensure we don't break other ts files (eg if ts gets compiled to esm).
The other option would be an option called allowIndexImports that is user configurable.
A final option would be to expose the resolver options found here:
https://github.com/eslint-community/eslint-plugin-n/blob/master/lib/util/import-target.js#L296, but that is a little trickier

allowIndexImports sounds like a good workaround to me.

The functionality is called Using folders as modules in the CJS docs of Node.js, and is considered legacy with subpath imports being the recommended replacement.
The requirement to specify index.js in ESM-files is also documented here: https://nodejs.org/api/esm.html#mandatory-file-extensions
I think rather than allowIndexImports it should be useRequireResolution or such, as the algorithm that require() supports more special cases than index files.

ts-node supports ESM barrel imports without any issue out-of-the-box without any special flags. It's only an issue with vanilla nodejs ESM imports.
I do not think full require() algorithm should be supported and useRequireResolution is a strange name for the config - nobody will understand what it does.

ts-node supports ESM barrel imports without any issue out-of-the-box without any special flags.

Have you investigated why / how it does that?
Does it still support it if you set these in your tsconfig.json?
    "module": "nodenext",
    "moduleResolution": "nodenext",

Yes, it works fine with module resolution set to NodeNext. Here is my tsconfig:
{
    "compilerOptions": {
        /* Base Options: */
        "esModuleInterop": true,
        "skipLibCheck": true,
        "allowJs": true,
        "resolveJsonModule": true,
        "isolatedModules": true,
        "strictNullChecks": true,
        "strictPropertyInitialization": false,
        "forceConsistentCasingInFileNames": true,
        "allowSyntheticDefaultImports": true,
        /* Strictness */
        "strict": true,
        /* Decorators */
        "experimentalDecorators": true,
        "emitDecoratorMetadata": true,
        "target": "ES2021",
        "module": "NodeNext",
        "moduleResolution": "NodeNext",
        "outDir": "dist",
        "types": [
            "./typings.d.ts"
        ]
    },
    "include": [
        "./srv"
    ]
}

According to ts-node, they require set "type": "module" in package.json and "module": "ESNext" in tsconfig.json to use ESM output, and then it uses the new experimental ESM loaders and otherwise they use the old require.extensions that has been deprecated since Node.js v0.10.6 and carries the following note:

Avoid using require.extensions. Use could cause subtle bugs and resolving the extensions gets slower with each registered extension.

Relatedly: I did an investigation related to this now, as they have done some work on enabling ESM-files to be loaded through require() , and ran into some odd findings, but that should not be relevant here: https://github.com/voxpelli/investigation-esm-require I did still ask the author of that feature about it on Twitter: https://twitter.com/voxpelli/status/1841818608713826693

My code is ESM but for production I am transpiling to CJS. So, the output is CJS.
Anyway, why does it matter for this rule how the transpiled code looks like? I am not running the linter on the minified production bundle.

why does it matter for this rule how the transpiled code looks like

It matters because different module resolution is applied when a javascript file is loaded through require and when its loaded through import and TS annoyingly supports both in their import so its important to know if its a bug in the implementation of either of the module resolutions or a matter of the wrong module resolution being applied.
Sounds to me like no-missing-import is applying the wrong module resolution in your case.
If you switch to "module": "commonjs" in your tsconfig.json and sets "type": "commonjs" in your package.json – does the issue still exist?

If you switch to "module": "commonjs" in your tsconfig.json and sets "type": "commonjs" in your package.json – does the issue still exist?

Yes, I get exactly the same errors.

TS annoyingly supports both in their import

Sorry, but I have to strongly disagree. The annoying one here is nodejs which for a decade or so can't implement proper support for ES modules.  CJS should have been deprecated years ago and everyone should have moved to ESM. Why runtimes like bun can support both CJS and ESM module systems in a single file and nodejs plain refuses to implement this? Just look at ESLint v9 - do you know how many different config extensions it has now? 6 different variants. That's just plain ridiculous.

Yes, I get exactly the same errors.

Then that could be a bug. @scagood When a TS-file is configured to have CJS module resolution, does no-missing-import use the require resolution algorithm rather than the import one?

Why runtimes like bun can support both CJS and ESM module systems in a single file and nodejs plain refuses to implement this?

Node.js supports importing CJS files in ESM files by using import and will be able to import most ESM files in CJS files using require when Node 23 is launched (already possible in latest Node 22 when using --experimental-require-module) and CJS files can already import ESM files by using await import()
The issue here though isn't whether require / import supports both ESM and CJS syntax or whether both ESM and CJS files implement them both.
The issue is that TypeScript in some cases apply the require algorithm to the import statements, which is straight up incorrect, and in your case no-missing-import isn't adapting to that and giving you errors.
Only thing that remains to be figured out is why no-missing-import isn't adapting.

Node.js supports importing CJS files in ESM files by using import and will be able to https://github.com/nodejs/node/issues/52697 in CJS files using require when Node 23 is launched

But you still can't use require() and import() in the same file like with bun? That's the issue. There shouldn't be any divide and there shouldn't be any need for cjs and mjs file formats.

But you still can't use require() and import() in the same file like with bun?

You can use await import() in CJS files and you can create a require() in ESM files using createRequire():
import { createRequire } from 'node:module';
const require = createRequire(import.meta.url);

// sibling-module.js is a CommonJS module.
const siblingModule = require('./sibling-module');

There shouldn't be any divide and there shouldn't be any need for cjs and mjs file formats.

Those currently are only needed for the type that differs from the "type" of the package.json.
As mentioned, starting in Node 23 – and already by using --experimental-require-module in latest Node 22 – it will no longer be needed in many cases (whenever no top level await exists in the dependency chain) as Node.js will then try requiring a file as CJS and if that fails it will retry it as ESM. (It can still be advisable to use "type" or .mjs for ESM-files when using require though, as the performance will be a bit better, as it won't have to try requiring it as CJS first)

That said, lets focus on supporting the require algorithm in import for such TS projects, as looking at the tests and code it seems like there are no logic for that case at all at the moment

@hakimio You could try import/no-unresolved / import-x/no-unresolved as they are the more advanced version of n/no-missing-import / n/no-missing-require and since eslint-plugin-import / eslint-plugin-import-x focuses on nothing but this very functionality it should have wider support for the cases, including full support for TypeScript

What's the difference between those two?

Then that could be a bug. @scagood When a TS-file is configured to have CJS module resolution, does no-missing-import use the require resolution algorithm rather than the import one?

Currently, the rule does not distinguish between common and module resolution, it just does esm resolution on no-missing-import and node resolution on no-missing-require 👀
I have been thinking about merging these two together into just no-missing, maybe we could perform the package.json#type check then

What's the difference between those two?

eslint-plugin-import-x is a fork of eslint-plugin-import that aims to provide a more performant and more lightweight version of the original plugin.

Currently, the rule does not distinguish between common and module resolution, it just does esm resolution on no-missing-import and node resolution on no-missing-require 👀 I have been thinking about merging these two together into just no-missing, maybe we could perform the package.json#type check then

That would be nice, for this specific case its more a matter of checking moduleResolution than package.json#type though. We could check what https://github.com/import-js/eslint-import-resolver-typescript is doing and copy the necessary parts if we want to entertain that path – or we simply say that TS-specific resolution algorithms is out of scope of eslint-plugin-n

or we simply say that TS-specific resolution algorithms is out of scope

Considering that there was effort put to improve TS support (#139) that would big step backwards considering the fix for the issue is quite simple:

It should be doable, the basics is just adding mainFiles.push("index")

Especially when the fix for the issue is quite simple

That's an incorrect fix. The correct fix is to implement the correct algorithm informed by what TS considers to be the correct algorithm.
That involves a few things:

Figure out what TS considers for the current project
Interpret what algorithm that translates to
Start using the require algorithm in the cases where its needed

That might be way beyond this rule's scope...

Exactly my feeling, especially as import/no-unresolved etc has a good pluggable file resolution system with proper TS support


Rescaling function: Replace magic number with value relative to image size

This isn't really a bug per-say so I didn't label it as such, but is more of a code smell that risks causing an issue for certain cases in the future.
In the rescaling function of data_augmentation.py, the function returns the original patch if the rescaling results in a difference of an arbitrary/magic value 5 voxels or less (apparently to avoid having a certain parameter = 0):
https://github.com/neuropoly/axondeepseg/blob/150cf46c8e8326f1ebbc4bdd6230ecfdcfbfbd3e/AxonDeepSeg/data_management/data_augmentation.py#L73-L76
While this value was likely well-defined for the patch sizes used in the paper, the user (me) isn't aware of the range of patch sizes that this function is well-behaved. E.g. if the patch sizes are small, even "large" rescaling factors won't do any rescaling at all. I think that this magic number would be better off being replaced by a value relative to the patch size.

This was resolved, but is no longer used


Multiple instances

Is there a way to have multiple instances of Inquiry running at a time?
Example:
I have a Service that is running in background and sometimes needs to write to a database. An Activity is reading from the database every time it is opened or a refresh is triggered. The problem here is that I can't use Inquiry.init() and Inquiry.deinit() in my Activity's/Service's lifecycle methods as the activity may still be running when the service has finished its work. It shouldn't be any problem if I had one Inquiry instance for the service and one for the activity

This has been fixed in my fork: https://github.com/HeinrichReimer/inquiry/issues/2

Done

https://github.com/afollestad/inquiry/releases/tag/3.0.0


Expand all node by one click

How to expand all node by one click?

How to expand all node by one click?

Hello @doan281,
I don't remember what was going on there but now when checking the code I can recommend a place to start:
This is the expanding function:
d is the node to be toggled
if (d.children) {
    d._children = d.children;
    d.children = null;
} else {
    d.children = d._children;
    d._children = null;
}

update(d);

You should loop recursively through the tree from the root node and set all _children to children and children to null then call update on the root node.

Thanks!
I have found the solution:
`
function expand(d){
if (d._children) {
d.children = d._children;
d._children = null;
}
var children = (d.children)?d.children:d._children;
if(children)
children.forEach(expand);
}
function expandAll(){
expand(root);
update(root);
}
function collapseAll(){
root.children.forEach(collapse);
collapse(root);
update(root);
}
`
Ref https://stackoverflow.com/questions/28754273/d3-js-tree-expand-all-and-collapse-all


TC.3.1 This method should be called whenever a user tries to make a move

637f09692773a0d4433d5b637a75dbb3ba2e40e4


Add Vari-Lite VL4000 Spot fixture definition

This is the first fixture definition I'm attempting to add, I'm happy to fix any functional or quality problems with it. It follows the official DMX tables for the fixture, minus the color wheel "mid-positions" (between two colors) which aren't described by the tables, but which were also used by the other Vari-Lite fixtures.

Hi, thanks for this.
Being so huge I think for now it's mostly functional.
It misses a few details (e.g. gobos, channels relationships) and I think I need to add some new presets for framing systems


Improve WebSockets support

Currently, TetherFi doesn't support WebSockets which is used in several web applications(e.g. Discord and IRCcloud) that implement this relatively modern technology.  As more and more applications adopt this feature(and drop support to legacy counterparts like long polling) we need to investigate whether we can support such usecase.
Reverse proxy implementations like NGINX seems to have already implemented such support, maybe it is also applicable to forward proxies as well?
Implement SOCKS proxy(#101) may make this feature request obsolete, though...

So there's two pieces here, the WS and wss protocol. For WS, we would have to implement the HTTP 1.1 upgrade flow, which I believe is straightforward but not the easiest.
For wss, I think all we are able to do is respond to the initial CONNECT tunnel, past that we are just a delegate for proxies network data. In theory, wss protocol should already work, as evidenced by web chat usually working when accessed through a mobile browser.
For fun, here are some notes about apps which I wrote up while using the proxy on a plane.
 I do not believe there is any further work to do to support Websockets.
Sadly, since third-party apps on the device are allowed by the OS to "go around" the proxy, things like WSS requests from a client library would not be intercepted by the hotspot proxy in all cases, which leads to the inconsistent behavior in chat apps like Discord. Not sure what else I can do here unfortunately.

@pyamsoft

it looks like the existing proxy server code does support the WS and WSS protocol

Unfortunately I unable to reproduce the same behavior while using the Discord web app:

I wonder whether it is rather caused by #298, would you mind check it out?

Another case using the IRCCloud web app:

debug log: tetherfi-debug-0202405241408.log.gz

Confirmed a memory leak from never closing sockets does at least effect this issue.
The next version will have a tweak to close inactive socket connections after 1 minute (if a network connection is started, but no data is transferred for a full minute, it will close the connection). This should help us clean up and reduce memory usage, which should avoid the IOException that causes these rejected connections.

Version 43 was just approved on the store and includes a new option:
While this is OFF by default, I recommend you turn on the Enable Idle Timeout tweak, which will clean up memory after a connection has not had any activity for a whole minute.
I will continue to investigate memory usage, but as far the original issue about websockets in third-party apps there is very little we can do. Mobile websites will usually work, but the actual apps themselves do NOT use the system proxy on Android (only the default browser app does). Sadly, this may be all that we can do for now, until we find a more encompassing solution. Sorry :(
Thank you for your contribution to the project!

Thanks for your work!  I no longer have problems using the web versions of the WebSockets applications at the moment so it's all fine.


Zoom: Add Navigation

If zoomend into a step, we need to show some form of navigation to zoom out. Also a the Name of the Parent as a headline for the current zoom state would be nice

Issues was written for first iteration of lambdui which is no more.


Add Maybe utility: maybe_str

Interface: (T) -> Maybe[str]
In case T cannot be converted to str Nothing is returned

Duplicates #5


fixes iob_data props check

Ok, i'll then work on dev branch(my fault).
iob_data.bolussnooze is used
var bolusContrib = iob_data.bolussnooze * sens; and bolusContrib is used to calculate snoozeBG that is then return. In my opinion is always better to check if iob_data.bolussnooze is not undefined, otherwise it will be NaN.

What conditions could cause it to be missing? What is the impact on temp basal decisions if snoozeBG is NaN?
I don't want to add any strict checks that might result in the loop stopping when it would be safer to continue looping despite the error condition, as halting the loop usually results in a reduction of overall safety to the user, unless it's a fatal error that makes it impossible to calculate a temp basal.

Closing this one.  Please feel free to open a new PR targeted against dev if you think additional checks are needed there.


Function await picker.getImage(...) from image_picker shows no dialog to select an image and never returns

Hi, I am trying to use the image_picker plugin in flutter web and have the following problem. When the user clicks a button to load an image I call the function
Future<void> foo() async
{
   PickedFile img = await picker.getImage(source: ImageSource.gallery);
   ...
}

The call to getImage however shows no dialog to select an image and never returns.   Messages on this channel are being discarded in FIFO
fashion.  The engine may not be running or you need to adjust the buffer size if of the channel.
MissingPluginException(No implementation found for method pickImage on channel plugins.flutter.io/image_picker)

What could be a cause of this problem? I am running the web version with flutter run web.

you can try https://pub.dev/packages/image_picker_web plugin for web version

Hi @mzimbres
image_picker doesn't support web,
but there's a first party web implementation image_picker_for_web
If the issue persists,
please provide your flutter doctor -v,
your flutter run -d chrome --verbose,
your pubspec.yaml
and a minimal reproducible code sample.
Thank you

@iapicca image_picker_for_web works like a charm, thanks for suggesting it, I did not know it existed.

@iapicca THE DOOC OF IMAGE-picker_for_web says to just add it to the dependencies and that it should work after. Is there anything else that has to be done ? Like importing said package somewhere ? It still hits me with the MissingPluginException


ANIM and GLOW flags don't work together

/yamipa give MikChan test.gif 1 4 2 ANIM,GLOW

The image glows but is not animated
/yamipa give MikChan test.gif 1 4 2 GLOW,ANIM

The image is animated but doesn't glow

That's because you're overwriting the default flags (see #54).
If you want to give image items that glow, you need to add (not overwrite) the "GLOW" flag:
/yamipa give MikChan test.gif 1 4 2 +GLOW

Please, read the documentation regarding flags.


fixed 'append' saving option for par1 vs par2 case

In the previous PR, it was left out the case "saving" : "append" when we are looking at "variation": true

Re-tested following 'append' cases:

1 par

"variation": false and limits=[None, None]
"variation": true and limits=[A_%, B_%]!=[None, None]

2 pars, one vs another one (gave up, we'll look again tmrw)

"variation": false and limits_1=[None, None] , limits_2=[None, None] -> it mixes the two params
"variation": false and limits_1=[A1_%, B1_%] , limits_2=[A2_%, B2_%] -> it mixes the two params / plot info sbagliato per primo parametro

Note: checked that the saved plot info dictionary is compatible with the previous version (all relevant keys for plotting are there and correct for each variation and number of parameters case)


Application sends push notifications of older app versions (iOS / Android)

Our application successfully sends push notifications, only the push tokens seem to stay after a reinstall of the application.
After reinstalling the application on iOS and sending a push notification, we get two notifications. It goes as far as more than 50 notifications. This is really strange, as I understand that after an uninstall Apple should automatically receive a note that the push token should be rejected next time. I think this is a pretty recent bug, perhaps on iOS in combination with this plugin?
On Android we get the same problems, although not tested as thoroughly yet.
Could this be a bug in the plugin or has anyone encountered something like this? Thanks in advance for any advice you can give.

Update: This seems to have been an error in the push service. Issue can be closed.


GoLang Scrypt Package "Pull Request"

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
Latest

What operating system and processor architecture are you using (go env)?
This is an Online IDE Cloud9IDE:

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ubuntu/workspace"
GORACE=""
GOROOT="/opt/go"
GOTOOLDIR="/opt/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

Im adding a New Go-Lang function for the scrypt package:
A link on play.golang.org is best.

What did you expect to see?
This is a pull request.... Not expecting to see anything. Im just making sure somone just fixes this.

What did you see instead?
Like i said above..

Other info.. I preety much pasted this in gerrit code review too:
Im preety much adding a function to check if Hashed Password is equal to the Password they inputed.. This is the function
func checkPassword(passCheck string, HashedPass string , salt []byte ,N, r, p, keyLen int) error{
    checkFunc,err := scrypt.Key([]byte(passCheck), salt, N, r, p, keyLen)
    if err != nil{
      errOrg = errors.New("Could Not Activate Key.")
    }
    if string(checkFunc) == HashedPass{
        errOrg = nil
    }else{
        errOrg = errors.New("ACCESS DENIED")
    }
    return errOrg;
}

I could not represent a go playground or any online simple compiler since i need to import the scrypt package.. But here is a working test:
package main;

import(
    "fmt"
    "errors"
    "golang.org/x/crypto/scrypt"
    )

var errOrg error;
func main(){
    //func Key(password, salt []byte, N, r, p, keyLen int) ([]byte, error)
    dk,err := scrypt.Key([]byte("Password"), []byte("dfs"), 16384, 8, 1, 32)
    check(err)
    fmt.Println(string(dk))

    checkPassword("Password", string(dk), []byte("dfs"), 16384, 8, 1, 32)
    if errOrg == nil{
        fmt.Println("*******ACCESS GRANTED********")
    }else{
        panic(errOrg)
    }
}

func check(e error){
    if e!=nil{
        panic(e);
    }
}

func checkPassword(passCheck string, HashedPass string , salt []byte ,N, r, p, keyLen int) error{
    checkFunc,err := scrypt.Key([]byte(passCheck), salt, N, r, p, keyLen)
    if err != nil{
      errOrg = errors.New("Could Not Activate Key.")
    }
    if string(checkFunc) == HashedPass{
        errOrg = nil
    }else{
        errOrg = errors.New("ACCESS DENIED")
    }
    return errOrg;
}

I think that function is trivial enough that callers can implemented it as needed. We try not to add too many helper functions unless they're needed often.


go fmt the source files and add travis check for go fmt

Ran go fmt on the repo. Also added a travis check for go fmt so that incoming patches will fail if go fmt is not honored.

Thanks! The travis-ci check is great.


fix: tracks not draggable in touch devices

Also having this issue. If we could get this merged that'd be great


Program consistently hangs when evaluating the Llama2-7B model using AutoDAN

Hi, I collected 80 validation test cases of AutoDAN on Llama2-7b, but when running the step2, it consistently hangs. I've tried reducing batch size or using more gpus but neither works. I am using 4 Nvidia A100 gpus 81920MiB.

@SCccc21 , what are the last error messages before it hangs? Can you provide full cmd that you used and the example logs file when it hang?

In fact it does not give any error message. It just got stuck at generating the first batch. (The 'torchvision.io' warning seems not to be an issue because I got autodan to run for zephyr model and have also seen this warning)
Below is the full log:

(harmbench) [chensi@tc-gpu002 HarmBench]$ python scripts/run_pipeline.py --methods AutoDAN --models llama2_7b --mode local --step 2 --verbose
CUDA_VISIBLE_DEVICES 0,1,2,3
The token has not been saved to the git credentials helper. Pass add_to_git_credential=True in this function directly or --add-to-git-credential if using via huggingface-cli if you want to set the git credential as well.
Token is valid (permission: read).
Your token has been saved to /projects/ai_safe/si/cache/token
Login successful
python -u generate_completions.py --model_name=llama2_7b --behaviors_path=./data/behavior_datasets/harmbench_behaviors_text_val.csv --test_cases_path=/projects/ai_safe/si/HarmBench/results/AutoDAN/llama2_7b/test_cases/test_cases.json             --save_path=/projects/ai_safe/si/HarmBench/results/AutoDAN/llama2_7b/completions/llama2_7b.json --max_new_tokens=512
/home/chensi/miniconda3/envs/harmbench/lib/python3.10/site-packages/torchvision/io/image.py:13: UserWarning: Failed to load image Python extension: '/home/chensi/miniconda3/envs/harmbench/lib/python3.10/site-packages/torchvision/image.so: undefined symbol: _ZN3c1017RegisterOperatorsD1Ev'If you don't plan on using image functionality from torchvision.io, you can ignore this warning. Otherwise, there might be something wrong with your environment. Did you have libjpeg or libpng installed before building torchvision from source?
warn(
Namespace(model_name='llama2_7b', models_config_file='./configs/model_configs/models.yaml', behaviors_path='./data/behavior_datasets/harmbench_behaviors_text_val.csv', test_cases_path='/projects/ai_safe/si/HarmBench/results/AutoDAN/llama2_7b/test_cases/test_cases.json', save_path='/projects/ai_safe/si/HarmBench/results/AutoDAN/llama2_7b/completions/llama2_7b.json', max_new_tokens=512, generate_with_vllm=False, generate_with_adv=False, generate_with_dpo=False, incremental_update=False)
Warning: num_gpus in config (1) does not match num_gpus available (3). Using 3 GPUs.
model_config {'model_name_or_path': '/projects/ai_safe/si/data/Llama-2-7b-chat-hf', 'use_fast_tokenizer': False, 'dtype': 'float16', 'chat_template': 'llama-2', 'num_gpus': 3}
Using HF generation
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████| 2/2 [00:03<00:00,  1.99s/it]
Tokenizer.pad_token is None, setting to tokenizer.unk_token
tokenizer.pad_token
Found Instruction template for /projects/ai_safe/si/data/Llama-2-7b-chat-hf
{'description': 'Template used by Llama2 Chat', 'prompt': "[INST] <>\nYou are a helpful, respectful and honest assistant. Always answer as helpfully as possible, while being safe. Your answers should not include any harmful, unethical, racist, sexist, toxic, dangerous, or illegal content. Please ensure that your responses are socially unbiased and positive in nature.\n\nIf a question does not make any sense, or is not factually coherent, explain why instead of answering something not correct. If you don't know the answer to a question, please don't share false information.\n<>\n\n{instruction} [/INST] "}
Generating completions for 80 test cases
Generating completions...
0%|                                                                                | 0/2 [00:00<?, ?it/s]/home/chensi/miniconda3/envs/harmbench/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:515: UserWarning: do_sample is set to False. However, temperature is set to 0.6 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature.
warnings.warn(
/home/chensi/miniconda3/envs/harmbench/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:520: UserWarning: do_sample is set to False. However, top_p is set to 0.9 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset top_p.
warnings.warn(

It might be the auto_find_batch_size from accelerate that we used to auto-define the batch size for generation, which can take time to define the batch size. I think a quick fix is to change the generation back end to vllm with
--generate_with_vllm in [generate_test_cases.py] and with 1 GPU. (https://github.com/centerforaisafety/HarmBench/blob/main/generate_test_cases.py). If you use pipeline.py then you can add this arg here.


%h«$x» is a trap, maybe

I've never seen this before, and I'm actually surprised that somebody came up with this.
Code:
my %h = ‘hello’ => 42;
my $x = ‘hello’;
say %h«$x»

Result:
42

So it works, right? Wrong.
Code:
my %h = ‘hello world’ => 42;
my $x = ‘hello world’;
say %h«$x»

Result:
((Any) (Any))

Of course, the right way to do it is to use %h{$x}. If you insist on «» for some reason, then %h«“$x”».
See this: https://irclog.perlgeek.de/perl6/2017-06-07#i_14699923

OK, this is a pitfall, because our docs are full of this:
doc/Language/5to6-nutshell.pod6:    say %calories«$key»;  # Perl 6 - double angles interpolate as a list of Str
doc/Language/5to6-nutshell.pod6:    say join ',', %calories«$keys»;          # Perl 6 the split is done after interpolation
doc/Language/operators.pod6:    say %color«cherry $fruit».perl;   # OUTPUT: «("red", "green")␤»
doc/Language/rb-nutshell.pod6:    say %calories«$key»;    # Perl 6 - double angles interpolate as double-quotes
doc/Language/rb-nutshell.pod6:    say %calories«$keys».join(','); # Perl 6, interpolated split
doc/Language/subscripts.pod6:    %hash«foo $var»;    # same as %hash{ «foo $var» }
doc/Type/Hash.pod6:    dd %h«oranges $fruit»;


Use lerna and cssnext

Travis failed:
$ rake
rake aborted!
No Rakefile found (looking for: rakefile, Rakefile, rakefile.rb, Rakefile.rb)
@xcatliu

@cht8687 Please ignore this. The new version doesn't require ruby or sass anymore.

@cht8687 Fixed travis issue.


Warning: Task "compass" not found. Use --force to continue.

Hey,
Is anyone able to help me fix this error?

Yours hopefully,

I had the same error, which I solved by installing the compass ruby gem: sudo gem install compass

I also had the same problem on Windows.  After installing compass, make sure your path includes the path to ruby.

I am also getting the same Warning: not found: compass Use --force to continue.

i tried installing the compass seperately as well, no luck.  Also, what do you mean by "make sure your path includes the path to ruby."?

Your PATH environment variable needs to include the directory where Ruby was installed.

I install yo and yo angular. Here is a angular project, I run "grunt server" > get the same error.

I had the sam problem on Windows and running gem install compass fixed the problem.

try the solution https://github.com/Compass/compass/issues/2129#issuecomment-334047499


Error deserializing

Hi,
following your instruction in creating a parcel this is the error we are getting:
The data contract type 'SendCloudApi.Net.Models.Country' cannot be deserialized because the required data members 'from_id, from_iso_2, from_iso_3, from_name' were not found.'
This is the code:
var parcel = new CreateParcel             {                 Name = "John Smith",                 CompanyName = "ACME",                 Address = "Wellingtonstreet 25",                 City = "Wellington",                 PostalCode = "3423 DD",                 Country = "NL",                 RequestLabel = true,                 ShippingMethod = 8, // Shipping method, get possibilities from client.ShippingMethods.Get()                 OrderNumber = "ORDER2014-52321",             };             try             {                 var response = await client.Parcels.Create(parcel);             }             catch (SendCloudException e)             {                 throw new Exception(e.Message);             }

Should be fixed now.
Set IsRequired to false for Country fields

Thank you!


must create tray icon in main thread on macOS

It appears that macOS requires the tray icon to be created/updated from the main thread of an application. I've seen something similar in this python library (https://pystray.readthedocs.io/en/latest/reference.html?highlight=macos#pystray.Icon.run_detached). I have tried the following, but not having any luck. This code runs, but no icon is created on macOS.
#include <thread>

// tray icon macros and includes
#if defined SYSTEM_TRAY && SYSTEM_TRAY >= 1
#if defined(_WIN32) || defined(_WIN64)
#define TRAY_ICON WEB_DIR "images/favicon.ico"
#elif defined(__linux__) || defined(linux) || defined(__linux)
#define TRAY_ICON "tray-icon"
#elif defined(__APPLE__) || defined(__MACH__)
#define TRAY_ICON WEB_DIR "images/tray-icon-16.png"
#include <dispatch/dispatch.h>
#endif
#include "tray/tray.h"
#endif

#if defined SYSTEM_TRAY && SYSTEM_TRAY >= 1
void tray_cb(struct tray_menu *item) {
  // code
}

// Tray menu
static struct tray tray = {
  .icon = TRAY_ICON,
#if defined(_WIN32) || defined(_WIN64)
  .tooltip = const_cast<char *>("Tooltip"), // cast the string literal to a non-const char* pointer
#endif
  .menu =
    (struct tray_menu[]) {
      { .text = "Open", .cb = tray_cb },
      { .text = nullptr } },
};

// function to create system tray, run in detached thread
int create_tray() {
  if(tray_init(&tray) < 0) {
    // logging
    return 1;
  }
  else {
    // logging
  }

  while(tray_loop(1) == 0) {
    // logging
  }

  return 0;
}
#endif

int main(int argc, char *argv[]) {
  // some code

#if defined SYSTEM_TRAY && SYSTEM_TRAY >= 1
  // create tray thread and detach it
#if defined(__APPLE__) || defined(__MACH__)
  // on macOS we need to create the tray on the main thread
  dispatch_async(dispatch_get_main_queue(), ^{
    create_tray();
  });
#else
  std::thread tray_thread(create_tray);
  tray_thread.detach();
#endif
#endif

  // more code

  // stop system tray
#if defined SYSTEM_TRAY && SYSTEM_TRAY >= 1
  tray_exit();
#if !defined(__APPLE__) || !defined(__MACH__)
  if(tray_thread.joinable()) {
    tray_thread.join();
  }
#endif
#endif

  // even more code

  return 0;
}

The most relevant part being
#if defined SYSTEM_TRAY && SYSTEM_TRAY >= 1
  // create tray thread and detach it
#if defined(__APPLE__) || defined(__MACH__)
  // on macOS we need to create the tray on the main thread
  dispatch_async(dispatch_get_main_queue(), ^{
    create_tray();
  });
#else
  std::thread tray_thread(create_tray);
  tray_thread.detach();
#endif
#endif

If I use create_tray(); directly, instead of within the dispatch_async the icon is created, but of course the remainder of the code does not execute due to the endless while loop.
Using the separate tray_thread is working great on Windows and Linux.
I'd be happy to provide more of the code if it will help.
Could you offer any workaround or guidance?

Hi @ReenigneArcher , this is interesting. I've only tested MacOS Catalina myself. Which version of MacOS fails for you?
I've reviewed your patch to LizardByte, seems like you ended up with compiling tray by your own CMake scripts with some extra C++ harness. Could you please create a PR for us with your changeset? It would be great to see what features you miss and how we could integrate them.

@dmikushin I was likely testing builds in macOS 13, but I think it's been the same behavior in macOS for a while.
I ended up disabling the tray icon for macOS as I couldn't figure out how to get it to run without blocking the main thread, so unfortunately I have no solution to provide.
I am not very experienced with C++. There were some issues with my initial PR that were fixed with the following PRs.
https://github.com/LizardByte/Sunshine/pull/1187 (Fixes issues when running as a service in Windows... cgutman may have already submitted the relevant changes to your repo... I see you merged one of his PRs)
https://github.com/LizardByte/Sunshine/pull/1208 (use cmake install prefix for icon instead of /usr)

Hi @ReenigneArcher ,
Hold on, I think I start to understand the source of your confusion. So, when you approach this code:
  while (tray_loop(1) == 0) {
    printf("iteration\n");
  }

, do you imply that it is going to block the main thread in the real usecase?
You are right, it will block is this simple example. But it does not mean that this code should be copy-pasted "as is" into the real application. When we work with GUI, main thread somehow maintains an event loop. Every GUI flavor does it differently: Win32, Qt, gtk, Cocoa - there is no common code that could be written for all of them. But what remains common is they all have some form of event loop. So I think your goal is actually to plant the tray_loop(1) call somewhere into the actual event loop that you use, where it's appropriate. Does this make sense?


meowpuffygottem

totally not just an R# keybind over the entire project

does this compile? probably
was that my goal? no lol

blessed


Switch to libsmce 1.3 and dynamically link it

Goal is to sync up with smce and use its shared lib instead of statically linking to it as hacked together boost static boost libs arent shipped along anymore.
While dynamically linked I opted to ship it along via cpack so that lesser platforms dont have to worry about installing libsmce, so on paper this change shouldnt be noticed by the end user.

Note: this also removes auto downloading libSMCE, you do have to provide your own install for development. Might need to document this..


sync with master

Current coverage is 100%

Merging #615 into v22.5.7 will not change coverage

@@           v22.5.7   #615   diff @@
=====================================
  Files           29     29
  Lines          439    439
  Methods        111    111
  Messages         0      0
  Branches         0      0
=====================================
  Hits           439    439
  Misses           0      0
  Partials         0      0

Powered by Codecov. Last update 14f22c7...1216291


Breathinganimation

The cards from the breathing exercises have a PlayButton added. This opens an animation with which the users can synchronize their breath. Thanks for checking!

Nice work, I just left some remarks to consider 🚀

Thank you 🙏 Everything is cleaned up now 😊


EventProcessorClient: CreateConnection() Exception not surfaced

Describe the bug
When a user calls StartProcessing() then that function subsequently makes the following call to CreateConnection. That call, despite not being enclosed in a try catch, can and will throw an exception and when it is it seems:

processing never starts
no error is raised
there's no indication of a problem
the application using the client has no way to know a fatal error occurred, if, say, events are seldomly received.

Expected behavior
A fatal error, that can't be automatically or eventually recovered, should be surfaced immediately upon death
Actual behavior (include Exception or Stack Trace)
The offending call is started here and, if either StopProcessing() or Dispose() is called then it will surface to the user, far too late.
To Reproduce
The aforementioned exception was thrown in my case here, when eventHubName wasn't specified using this constructor
Environment:
<PackageReference Include="Azure.Messaging.EventHubs" Version="5.0.1" />
<PackageReference Include="Azure.Messaging.EventHubs.Processor" Version="5.0.1" />
.NET Core SDK (reflecting any global.json):
 Version:   3.1.200
 Commit:    c5123d973b

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.17763
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   c:\Program Files\dotnet\sdk\3.1.200\

Host (useful for support):
  Version: 3.1.2
  Commit:  916b5cba26

.NET Core SDKs installed:
  2.2.207 [c:\Program Files\dotnet\sdk]
  3.1.100 [c:\Program Files\dotnet\sdk]
  3.1.102 [c:\Program Files\dotnet\sdk]
  3.1.200 [c:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.14 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.16 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.8 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.14 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.16 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.8 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.0 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.2 [c:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.14 [c:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.16 [c:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.8 [c:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.0 [c:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.2 [c:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.0 [c:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.1.2 [c:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET Core runtimes or SDKs:
  https://aka.ms/dotnet-download

Hi @serpentfabric.   We're sorry that you're experiencing difficulties and appreciate you reaching out to let us know.  The scenario that you've identified was indeed a bug in the processor; the connection should have been created within a try/catch that surfaced any issues to the error handler.
I'm happy to report that this was something that had been identified and has since been fixed.  The next release of the processor contains the fix, along with other fortifications around resilience and error visibility.  It is expected to be released to preview shortly.


ARROW-8645: [C++] Missing gflags dependency for plasma

The documentation build fails because gflags is not installed and CMake doesn't build the bundled version of it.
Introduced by https://github.com/apache/arrow/commit/dfc14ef24ed54ff757c10a26663a629ce5e8cebf

@github-actions crossbow submit ubuntu-docs

@github-actions crossbow submit test-ubuntu-18.04-docs

FYI: I'm working on this at #7067 .

Didn't see that, closing in favor that then.


matplotlib may create 2 windows when run through spyder

The following script (doublewindow.py) creates 2 windows when launched from the run icon in a ipython console, ie runfile(...). When it is launched from the same console using "%run -i doublewindow.py" it runs fine, ie only one window is open as expected. I could track down the issue from the call to plt.ishold(). If you comment this single line, both runs create a single window. PS: use "%matplotlib wx" first.
import matplotlib.pyplot as plt

from matplotlib.backends.backend_wx import FigureCanvasWx as FigureCanvas
from matplotlib.figure import Figure

import wx

class MyPanel(wx.Panel):
  def __init__(self, parent):
    wx.Panel.__init__(self, parent)
    self.figure = Figure()
    self.canvas = FigureCanvas(self, -1, self.figure)
    self.axes = self.figure.add_subplot(111)

if __name__ == "__main__":
  if not 'app' in locals() or app==None: app= wx.App()
  frm = wx.Frame(None,-1,'Test')
  MyPanel(frm)
  frm.Show()
  plt.ishold()  ### <-------------------- issue is here
  #app.MainLoop()

It occurs with spyder 2.3.8 and 1 week ago dev version.
All my libraries are the last ones (matplotlib,wx,ipython...). I use python 2.7 (win32) on windows 7.
Thanks.

I can reproduce this bug, but it's a problem that we can solve ourselves. This is a bug in the qtconsole package that we just embed in Spyder.
So please report this problem here:
https://github.com/jupyter/qtconsole

Actually I cannot reproduce the bug pasting this code on a "jupyter qtconsole" (and uncommenting "#app.MainLoop()"). Only one window pops-up.
How do you know it is from qtconsole, how can I justify it for this group?
Thanks. My qtconsole is 4.2.0.

I reproduced it in qtconsole on Linux :-) But maybe this is not a bug on Windows, I don't know.

Would you tell me how to generate the bug on a qt-console, the qt-console guys asked for it :-)
What does runfile(...) is translated to? Because within spyder %run -i doublewindow.pyworks.


Incentivizing managers

We predict that nodes will come online only when they need to, i.e. when they want to broadcast a new opinion toward other peer, which can last a few seconds. If every node in the network does the same, then the network will not run properly. The real incentives for nodes to be constantly online and calculate the reputation scores for others are needed.
We will use the concept of domains:

There will be a special domain reserved for being a good Eigen Trust manager (e.g. domain 0).
When a peer requests a trust score for its neighbor, it will look for peers that have good reputation in domain 0, and make a request to that peer.
If a peer is a good Eigen Trust peer, i.e. is providing correct trust scores for the peers they manage, other peers will give it a good score in domain 0.
If a peer is a bad Eigen Trust peer, i.e. is not providing correct trust scores for the peers they manage, other peers will give it a bad score in domain 0.

Will be explained in the docs: https://filips-books.gitbook.io/eigen-trust/