[Travis] Test R on Travis

[Travis] Reorganization, now almost all instructions are part of the matrix
[Travis] Need manual ./configure to copy C++ files
[Travis] Pseudo-env-var TRAVIS_LANG used to describe the builds in the Travis UI
[Travis] Coverage checking, please visit https://codecov.io and sign up with GitHub
Add README badges.
[R] Run document()
[R] Remove documentation of S3 methods
[R] Improve test coverage, output and some corner cases still missing

Current coverage is 75.00%

Branch #87 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

Merged. thank you!


merge: stage to master

merge: stage to master

:tada: This PR is included in version 0.41.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed implicit cast from Uri to Url

Closes #1966

:shipit:

FFFFFFFFUU! :sob:

Phew :sweat_smile:


chore(robot-server): upgrade Pydantic to support latest mypy

Overview
We want to upgrade mypy, but our current version of Pydantic has a bug in its mypy plugin that makes it incompatible with later versions of mypy.
Acceptance criteria

[ ] Robot server uses latest version of Pydantic
[ ] opentrons package accepts latest version of Pydantic

If it seems responsible / possible, we should expand the version of Pydantic for the api package rather than setting it to something exact

Moved to Jira: RSS-49


Chore/update readme - Added the Web3 conference Lagos event

Fixes #68

Thanks for the massive efforts @Tonie-NG ✊🏾🚀


Add non-root-POM feature to Maven plugin

Introduces ability to the Maven plugin to work with projects which lack a root-level POM file. Changes to specific classes are as follows:
RefactorFirstMavenReport

Set requiresProject parameter to false. A true value will result in MissingProjectException being thrown if no root-level POM file is present. A falsy value is required to enable scanning of projects without a root-level POM file.
Added check (starting line 133) for default project name & substitution. The MavenProject dependency will use a default project name of "Maven Sub Project (No POM)" if a root-level POM file is not present. If that name is detected, replace it with the name of the project's root directory.
Added handling (starting line 202) for an upstream bug in MavenProject dependency. This class's getBasedir method returns null instead of a File instance if no POM file is present in the proejct's root dir. This, in turn, was causing a chained getPath call to fail. Added a check to see if a getBasedir call returns null, and set projectBasedir conditionally.

RefactorFirstMavenCsvReport

Set requiresProject parameter to false.
Added check (starting line 79) for default project name & substitutio.
Added handling (starting line 89) for an upstream bug in MavenProject dependency.

ReportWriter

In the absence of a root-level POM file, the MavenProject class was returning "${project.basedir}/target/site" as the output directory. Added call to replace to scrub the output directory of the default.

RefactorFirstMavenJsonReport

Added handling (starting line 41) for an upstream bug in MavenProject dependency.

Tested these changes in the following scenarios, all of which produce expected/correct behaviour:

projects containing a root-level POM file
projects lacking any POM files
projects without a root-level POM, but containing sub-projects which DO have their own POM files

PR looks great!  Thank you for the great work and your thorough comments in the PR. 😄


JsonValueMapValue breaks when space is present

Required Information

[X] AEM Version, including Service Packs, Cumulative Fix Packs, etc:  All
[X] ACS AEM Commons Version: 4.2.2
[x] Reproducible on Latest? yes

via Rajat Pachouri on Disqus:
The JsonValueMapValue Injector injection breaks when there is a space between the string.
e.g it works for "Hello" but fails at "Hello there" as GSON library consider it as malformed. Could be fixed by setting up GSON.setLenient(true).
Having this issue makes this practically unusable as its highly likely that you will add space in what you are authoring.
https://adobe-consulting-services.github.io/acs-aem-commons/features/sling-model-injectors/json-valuemap-value/index.html#comment-4549882259

Damn! Never realized this. Il fix this asap

Well. I tried to reproduce but I cannot. I will need more details. Please give me the sling model + exact underlying json stored as property that you are trying to inject.

I will try to reproduce today and give you guys more details. sorry for the delay.

Closing for now but please open if we can reproduce this with a test case.

Hey,
I am importing acs-common-4.7.0 bundle in my project. I can see other annotations available like aem object etc, but JsonValueMapValue is not available.
When I open the jar, I can see the interface for JsonValueMapValue is available but when I try to open impl file, I see the internal error.


Add partitioned table and materialized view to db backup

Will allow tests for #59 and #58.

Closing this because I forgot to add composite types.


Standard type extraction

In Schema we now added ability to extract types like: typeof Person.Type that makes it really nice to get the types, the same should be applied to things such as tags so one can do typeof Service.Type and to Effect so that one can do typeof Effect.Success etc.
I do feel though that unless we prefix those fake properties somehow it can become confusing, one might argue that defining them in uppercase makes it clear that they are meant to be types and not real values.
This issue is to discuss on a standard.
cc @gcanti @tim-smart @IMax153 @datner @patroza @fubhy

So far I think that the capitalization is sufficient. Types are generally capitalized. Uppercase generally signifies constants. JSDoc should take care of the rest.

If we wanted to be even more pedantic, we could add a Type suffix, so:
Effect.SuccessType
Effect.ErrorType
Effect.ContextType

but to be honest I think this is unnecessary and overly verbose. I agree with @patroza that the capitalization should generally be sufficient, along with a JSDoc explaining what the value is intended to be used for.

One other option is a namespace:
typeof someEffect.Types.Success

But I think I prefer .Success etc directly on the type.

while
typeof someEffect.Types.Success

would be the most obvious, the more nested it is the less benefit it has over Effect.Effect.Success<typeof someEffect>
Like, if we just alias it to the module level then compared with an Effect.Success<typeof someEffect> it's barely any different.
It has to be
typeof someEffect.Success

if at all 🤔 (name pending, I only mean the nesting)

I would also offer to consider lifting the stand-alone type accessors from the namespace to the top-level.
I often use bare types for polymorphism, and my codebase is full of
const exchangeCtx = yield* Effect.context<
        Layer.Layer.Success<ExchangeLayer>
      >()

However, there is nothing preventing us to have Layer.Success, because there are no conflicts on the top-level namespace. In the end of the day – having a nested namespaces feels cumbersome

@dilame I would very confidently place the usage of Effect.context to be in the bottom slots of the "very rare" bracket of utils, so I can't quite imagine how you get a 'codebase full of' it. Regardless, the question at hand is not about eliminating the nested namespace. That would not happen most likely as the purpose of it is to enrich the base type Effect (aka Effect.Effect) with utilities like Effect.Success (aka Effect.Effect.Success). The question is about not needing utilities at all.
In your case, it would mean typeof ExchangeLayer.Success, not Layer.Success<typeof ExchangeLayer>. If you seek the latter, just change
- import { Layer } from "effect"
+ import { Layer } from "effect/Layer"

to get Layer.Success<typeof ExchangeLayer>

I think Effect.Success, Effect.Error and Effect.Context is fine

@datner it will work when you are relying on runtime entities, but will not work when you use Layer as kind of OOP polymorphism (which i believe is the original intention of layers)
export type ModeLayer = Layer.Layer<
  | ModeKlineImpl
  | ModeOrderCancelImpl
  | ModeOrderCreateImpl
  | ModeOrderFetchImpl
  | ModeOrderListenImpl
  | ModePriceImpl,
  never,
  Layer.Layer.Success<ExchangeLayer>
>;

@dilame I honestly have no clue what you're showing me, I don't have code that looks like that..
Remember that effects patterns are still emerging, there are very few blessed ways to use effect and they are largely technically driven and don't force a specific design so it's not odd we'll find different patterns, but I have no idea why would you want / need that code 😅
I'm just saying that it's a different conversation, and currently it's impossible to merge namespaces in typescript (we tried, really hard). Lets refocus please, do you have any opinions regarding typeof myEffect.Succeed? Do you think it should exist?

@datner it will work when you are relying on runtime entities, but will not work when you use Layer as kind of OOP polymorphism (which i believe is the original intention of layers)
export type ModeLayer = Layer.Layer<
  | ModeKlineImpl
  | ModeOrderCancelImpl
  | ModeOrderCreateImpl
  | ModeOrderFetchImpl
  | ModeOrderListenImpl
  | ModePriceImpl,
  never,
  Layer.Layer.Success<ExchangeLayer>
>;

Very much not the intended usage of layers. A layer represent a layer in the sense defined by the onion architecture, so a progressive translation of implementation from most abstract to least. Never seen a layer being manually typed, and also having *Impl in the signature is kind of bad, it should be the generic interface not the specific impl

Very much not the intended usage of layers. A layer represent a layer in the sense defined by the onion architecture

I believe that OOP polymorphism and Effect Layer are both low-level abstractions aimed at achieving the goals of high-level architectures like Onion, Ports and Adapters, Hexagonal, Whatever. These paradigms emphasize separation of concerns and dependency inversion. Thus, I see Effect Layer as an alternative to OOP polymorphism within these architectural frameworks. They are not opposing concepts but tools that serve the same purpose in maintaining a clean architecture.

it's not odd we'll find different patterns, but I have no idea why would you want / need that code 😅

You are so tactful and real at the same time, i like it!
I have no doubt that typeof myEffect.Succeed should exist.
Practically speaking, to get the type of myEffect at runtime, we have to write typeof myEffect anyway. If we extract generics using another generic type, the best case would look like this:
import { Success } from 'effect/Effect'

Success<typeof myEffect>

This adds Success and two extra symbols < and >. Typically, imports are import { Effect } from 'effect', making the extraction code even longer.
typeof myEffect.Succeed just adds single dot and requires no extra imports. I vote with three hands for typeof myEffect.Succeed.

Very much not the intended usage of layers. A layer represent a layer in the sense defined by the onion architecture

I believe that OOP polymorphism and Effect Layer are both low-level abstractions aimed at achieving the goals of high-level architectures like Onion, Ports and Adapters, Hexagonal, Whatever. These paradigms emphasize separation of concerns and dependency inversion. Thus, I see Effect Layer as an alternative to OOP polymorphism within these architectural frameworks. They are not opposing concepts but tools that serve the same purpose in maintaining a clean architecture.

Even following your logic a sentence like @datner it will work when you are relying on runtime entities semantically has no meaning (makes no sense), code like the one you posted is not common and I'd go as far as to say it's even an anti-pattern. Anyway all of this is fairly off topic, feel free to continue the discussion via discord, this issue only talks about type extractors and I think we reached agreement on the team that type of Effect.Success is a valid option

Hmm we will have to change the variance in some scenarios to make this work:

@tim-smart I think this technique is applicable only to covariant and invariant type parameters (as is the case with @effect/schema)

We might be able to use the same trick as the Iterators to disconnect the type helpers from the type variance. Will have a play around tomorrow.

We might be able to use the same trick as the Iterators to disconnect the type helpers from the type variance. Will have a play around tomorrow.

Not sure that's possible

An alternative would be to have a general GetType helper to use like: GetType<typeof program>["Success"] which can be made by enhancing the supported types with an extractor symbol that preserves the variance, it's a bit more involved than simply typeof program.Success but it can be made truly general, ideally we could even shorten it to like Type<typeof program>["Success"]

Maybe just moving them to the top level is the simplest approach?


Food effect procs cause incorrect state

Effect procs like for On Kill Food cause additional buffs with "Nourishment" name to be applied, which results in incorrect buff states.
This is likely the same for Res Utilities.

Procs for the Condi variants like Saffron Mushroom do not cause this issue since their buff seems to be named "Malice" instead.

Fixed for Dragon's Breath Bun.
Other procs using the "Nourishment" or "Enhancement" names will need to have their ids added to the data in order to be ignored.


How to make a modal's content scrollable, if there is more content

I'm submitting a instructions or feature request, if it doesn't exist yet.

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[x] Support request
[ ] Other... Please describe:

Current behavior
If i lower the height of the screen, the content which was before in modal window dissapears, instead of changing state to scroll. I tried adding overflow-y: auto to all the html components in my modal window, but it doesn't solve the issue.
Expected behavior
When there is more content in the modal window, the scroll should appear. Instead of this, currently the content just dissapear
Minimal reproduction of the problem with instructions
You just add a lot of content inside the
....................
, like adding just a lot of lorem ipsum will also reproduce the problem. You just need a lot of content inside modal window, and the content would go over a modal window, instead of showing a scroll.
What is the motivation / use case for changing the behavior?
It would be a great feature, since I will have many more modal windows in my applicaton, which will have more content
Environment

Angular version: 15

Browser:
- [X] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [X] Chrome (iOS) version XX
- [X] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [X] Edge version XX

For Tooling issues:
- Node version: v16.20
- Platform:  Linux, Windows

I solved it by myself.
In styles.scss file, I added those two styles:
ngneat-dialog {
  .ngneat-dialog-backdrop {
    overflow: hidden !important;
    .ngneat-dialog-content {
      overflow: auto !important;
    }
  }
}

Great to hear


Smarter CHECKS

I'd like to precompile my app before actually starting up the server, so my Procfile may look something like this:
web: make build && make start

Which will take longer than 35 seconds. I could update the timeout, but that would be a guess and I'd have to keep updating it as the app grows.
It'd be nice to either wait until the command fully finishes, or if that's not possible, curl before releasing to make sure that you are not getting a "Bad Gateway" error from nginx.

I wrote a plugin to do this in the pre-deploy phase. I believe there is another out there as well.
https://github.com/michaelshobbs/dokku-app-predeploy-tasks

Also this is in process. https://github.com/progrium/dokku/pull/1056
Feel free to chime in there as well.

Closed in favor of #1056


Web applications that takes longer than 10s to load are killed

Have taken https://github.com/webosose/test-apps/tree/master/com.webos.app.test.youtube and just replaced:
window.location = "http://www.youtube.com"

with
window.location = "http://svtplay.se"

And modified other strings by just replacing "Youtube" with "SVT Play". Just for testing.
On a RPI4 with webOS OSE 2.5.0 it looks like it very often takes longer than 10s for svtplay.se to load. I can see the following in the journal:
$ grep  "Changed:\|timeout\|killApp" journal-svt-does-not-start.txt
Jul 15 09:05:01 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][75bf8a7a-5408-453a-bb4b-4451e93b52fd0] Changed: com.luxoft.app.test.svtplay.0 (stop ==> splashing)
Jul 15 09:05:11 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][75bf8a7a-5408-453a-bb4b-4451e93b52fd0] Changed: com.luxoft.app.test.svtplay.0 (splashing ==> splashed)
Jul 15 09:05:11 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][75bf8a7a-5408-453a-bb4b-4451e93b52fd0] Changed: com.luxoft.app.test.svtplay.0 (splashed ==> launching)
Jul 15 09:05:11 raspberrypi4 sam[607]: [W][RunningApp][onKillingTimer][75bf8a7a-5408-453a-bb4b-4451e93b52fd0] Transition is timeout
Jul 15 09:05:11 raspberrypi4 WebAppMgr[603]: [] [pmlog] WAM LUNA_API {"APP_ID":"com.luxoft.app.test.svtplay.0","API":"killApp"} reason :

Looks like application spends too long (>10s) in "launching" state and is killed in https://github.com/webosose/sam/blob/8a1443b41a75e23f75cca247b3f010190f439d7e/src/base/RunningApp.cpp#L257 .
If I try to launch the application again soon after it very often succeeds, e.g. the web application only spends 4s in launching state before it goes to foreground state:
Jul 15 09:08:40 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][02ca65f6-a470-4b63-aad8-7ad4ffa035800] Changed: com.luxoft.app.test.svtplay.0 (stop ==> splashing)
Jul 15 09:08:40 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][02ca65f6-a470-4b63-aad8-7ad4ffa035800] Changed: com.luxoft.app.test.svtplay.0 (splashing ==> splashed)
Jul 15 09:08:40 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][02ca65f6-a470-4b63-aad8-7ad4ffa035800] Changed: com.luxoft.app.test.svtplay.0 (splashed ==> launching)
Jul 15 09:08:44 raspberrypi4 sam[607]: [I][RunningApp][setLifeStatus][02ca65f6-a470-4b63-aad8-7ad4ffa035800] Changed: com.luxoft.app.test.svtplay.0 (launching ==> foreground)

Perhaps it loads faster the second time due to web content being cached locally, not sure.
Also not sure that this is something that should be fixed in SAM or in WAM. Or is there e.g. a way to make the web application behave better (say that it should be brought to foreground before loading has finished). Have not found any documentation for how to handle web apps that take a long time to load on https://www.webosose.org yet.

Just noticed that the timestamps in the log above when the application fails is weird. It is 09:05:11 both for splashed ==> launching and Transition is timeout which does not make any sense. Not sure what is going on there. Some buffering in log framework?
Anyway, here is a log that shows the application being killed with timestamps that make more sense (launching 10s before timeout is struck and app is killed):
Jul 16 14:54:46 raspberrypi4 sam[610]: [I][RunningApp][setLifeStatus][0c52f033-fd4b-47d3-9587-958481578d360] Changed: com.luxoft.app.test.svtplay.0 (stop ==> splashing)
Jul 16 14:54:46 raspberrypi4 sam[610]: [I][RunningApp][setLifeStatus][0c52f033-fd4b-47d3-9587-958481578d360] Changed: com.luxoft.app.test.svtplay.0 (splashing ==> splashed)
Jul 16 14:54:46 raspberrypi4 sam[610]: [I][RunningApp][setLifeStatus][0c52f033-fd4b-47d3-9587-958481578d360] Changed: com.luxoft.app.test.svtplay.0 (splashed ==> launching)
Jul 16 14:54:56 raspberrypi4 sam[610]: [W][RunningApp][onKillingTimer][0c52f033-fd4b-47d3-9587-958481578d360] Transition is timeout
Jul 16 14:54:56 raspberrypi4 WebAppMgr[613]: [] [pmlog] WAM LUNA_API {"APP_ID":"com.luxoft.app.test.svtplay.0","API":"killApp"} reason :


Updated changelog to reflect correct release date

Updated changelog to reflect correct release date

API changes have been detected in azure-cosmos. You can review API changes here


Requirement of connecting to -admin url should be mentioned

Add-PnPSiteCollectionAppCatalog does not work by usual method of connecting to target site via Connect-PnPOnline and executing the "Add" cmdlet.
It needs connection to the tenant -admin url, a fact noted in the PnP-PowerShell issue.
As this is a departure from the usual method, and not something which is obvious, it should be mentioned in the docs page.

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

ID: 5e6077b2-2003-44d4-1dc0-37513a3eaaba
Version Independent ID: 80ae9151-994d-86f7-f071-ac52ab798adb
Content: Add-PnPSiteCollectionAppCatalog (sharepoint-pnp)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPSiteCollectionAppCatalog.md
GitHub Login: @techwriter40
Microsoft Alias: kirks

@officedocsbot assign @yogkumgit

Hello @AbhishekGarg How are you?
Thanks for your feedback.
I need to kindly redirect you to open PnP cmdlets issues here, a member of the PnP community will handle your request there.
Thank you!

Hello @get-itips, I am fine, thank you :)
Hope you are good as well!
Thank you for pointing me in the right direction. I'll raise an issue there.
I am not 100% sure if this issue can be solved only once the PnP docs are updated, or if this will remain as such. Please close if this should be closed.

@get-itips  Thank you very much for the contribution and sharing this explanation. @AbhishekGarg
Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[openssl] Build error on x64-windows

Copy issue body from D:/vcpkg-master/installed/vcpkg/issue_body.md

Embarrassing!


PSX Porsche Challenge rumble not working on v24.10 other PSX titles ok

BlueRetro firmware version
24.10
BlueRetro firmware specification
HW1
BlueRetro firmware variant
System specific
BlueRetro hardware type
External adapter dongle (1 port only)
Manufacturer
PSUnoRetro (Willsconsolemodifications)
System used
Sony PlayStation 2
Bluetooth controller brand & name
Sony DualShock 4
What is problem? (only list ONE problem per report)
I reported #155 long ago and you recently made changes to support dual motor rumble. I updated and tried it but strangely the rumble in PSX Porsche Challenge is not working, at least on PS2 via backwards compatibility (I don't have a real PSX to try). This works fine with a regular DualShock2, so if blueretro emulates this then it ought to work. I tried setting the system type from Auto to PS2 in the advanced config but that didn't help. Rumble support in Gran Turismo works fine for instance, so it may be some kind of edge case with this particular game.
What did you expect to happen?
Rumble in game when the Analog mode is toggled for the controller.
Attach files like logs or Bluetooth traces here
No response

What region of that game? I tried US one on a PS1 and no rumble on either OG analog, DS1 or DS2

Ok saw its was PAL in other bug, tried that and rumble does work. I'll check why BlueRetro doesn't work.

Work now with this build:
https://github.com/darthcloud/BlueRetro/actions/runs/11302189293

Thanks, will try today. I should also have mentioned that the analog control worked fine.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jacques Gagnon @.>
Sent: Saturday, October 12, 2024 4:26:31 AM
To: darthcloud/BlueRetro @.>
Cc: patters-match @.>; Author @.>
Subject: Re: [darthcloud/BlueRetro] PSX Porsche Challenge rumble not working on v24.10 other PSX titles ok (Issue #1093)
Work now with this build:
https://github.com/darthcloud/BlueRetro/actions/runs/11302189293
—
Reply to this email directly, view it on GitHubhttps://github.com/darthcloud/BlueRetro/issues/1093#issuecomment-2408331006, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANEJPSURMNPMHBF67DATAGTZ3CJGPAVCNFSM6AAAAABPZL6ZUCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYGMZTCMBQGY.
You are receiving this because you authored the thread.Message ID: @.***>

Woohoo - success! 😃  Thanks for the rapid response!


Not working on NodeOS

When launched on NodeOS, cursor keeps blinking and CPU gets to 100% (maybe an infinite loop somewhere?), but nothing more happens. This is not a blessed or terminator issue since now by using -curses instead of nographic blessed demos are being shown (with bad characters, but at least it's not an empty screen). I've also lauched it with slap --help and slap --logger.level debug with no result, so seems the problem would be before this steps.

@piranna I was trying to get a more recent NodeOS running on my computer but could not, as the NodeOS docker image very old. Is there a NodeOS IRC channel or some other support channel?

Docker images are unmaintained, but you have easy and updated instructions for QEmu, and also now it can run on real hardware... :-) On QEmu you'll need to use the curses flag, but on real hardware it should work as is.

I can't build NodeOS on OSX for a variety of reasons. First it complained about elf.h, then byteswap.h. I remember it failing on an Ubuntu VM as well, but I'll try again.

It can't be build on OSX, but some people got to build it on an Ubuntu VM. I'm thinking if using crosstools-ng would allow to build it on other systems like OSX or Windows without problems...

@piranna here's my traceback after building on a fresh Ubuntu 14.04 VM: https://gist.github.com/secrettriangle/9bb9175ba631c33f30e0

From the traceback I can't be able to identify the problem, but definitelly it's possible to build it on Ubuntu 14.04 VM (it's what's being used on SemaphoreCI servers). Since the process is killed, maybe it's an out of memory problem? How much memory has the virtual machine?
By the way, maybe you are interested in using the autogenerated releases instead of generate them yourself, and also someone could be able to convert them to VirtualBox... :-)

Output of run.sh as well as npm install

There's something really awful on your virtual machine... Dumb question, but you have exec bin/install-dependencies previously to build NodeOS with npm install?
The problem of run.sh is because it's prepared to exec QEmu on a real machine, KVM is not available inside virtual machines. Just remove the KVM support and try again, it will be slower but you should be able to run it :-)

dan@dan-VirtualBox:~/Downloads$ ls
bzImage            README.md  usersfs.img
initramfs.cpio.gz  run.sh     x86_64-pc_qemu.zip
dan@dan-VirtualBox:~/Downloads$ ./run.sh
./run.sh: 5: ./run.sh: --kernel: not found

Same thing with an absolute path to bzImage in run.sh.
After running install-dependencies I get the exact same error.

Sh*t, that's annoying! :-( What's the actual content of run.sh after changing it?

OK, I'm able to run it now... Had to remove the kvm line, not just comment it out. Everything is very slow in this VM-within-a-VM, but I will do some debugging when I can.

Does NodeOS support environment variables? Also where is /home? slap relies on process.env.HOME.

OK, I'm able to run it now... Had to remove the kvm line, not just comment it out.

Great! :-D Yeah, bash scripting sucks, but for run.sh could be the best one, since this allow people to test NodeOS without need to have Node.js installed. Maybe add an option to disable KVM?

Everything is very slow in this VM-within-a-VM, but I will do some debugging when I can.

Yeah, it's normal :-P It will last more time to boot it but it's usable anyway :-)

Does NodeOS support environment variables?

Yes, you can access them opening a node shell and executing process.env. nsh (the NodeOS shell) currently don't support to define them... definitely, nsh needs to get a huge priority status...

Also where is /home? slap relies on process.env.HOME.

Your $HOME is / :-) It's an OverlayFS filesystem stacking the initramfs and your home directory, and later chroot'ed to it at /. This way users have the feeling to have all the filesystem to themselves while at the same time can't be able to access others $HOME to sneak their files ;-) Cool, eh? :-D

Interesting -- if I comment out the KVM line in run.sh, I can run it natively on OSX. FYI :)

Hum? Define "natively"... using an OSX QEmu binary? Yeah it would makes sense, I don't know but KVM probably would be a Linux-expecific virtualization mechanism... :-)

@piranna it works using a prebuilt binary by running ./run.sh after only removing the KVM line.

Oh cool, then maybe we should set KVM it as an opt-in instead as by default?

Exactly what I was thinking... maybe add a command-line option, like --with-kvm (if it's easy enough).
Also the zip should container a folder with files rather than a bunch of files. (IMO)

KVM is a linux virtualization module, without KVM, QEMU just does emulation as far as I know. It's a bit slower, but it will work.

Also IMO the zip should container a folder with files rather than a bunch of files. NodeOS/NodeOS#142

Maybe it's a question of taste, I don't like when I de-compress a file and inside there a lonely folder, I find it redundant... :-P

KVM is a linux virtualization module, without KVM, QEMU just does emulation as far as I know. It's a bit slower, but it will work.

Yes, definitely the flag seems to be a good thing :-)

What's the easiest way to add/change files in the NodeOS VM?

At this moment, probably to edit the build scripts and rebuild, specially node_modules/nodeos-usersfs module. Also, the npm packages are defined in .txt files as a list, just add there the name of the module and it will be downloaded. Probably you'll need to delete the obj & out folders or comment th if statement to force to recreate them...

I've just tested slap 0.1.28 on NodeOS (I couldn't test any more recent because we don't support nan2 yet) running on real hardware... and it works! :-D Seems the problem was due to an out-of-memory error, since the default QEmu virtual machine has only 128mb and after booting NodeOS it left free only 40mb (I need to fix the memory consumption...). The same happened to npm, that "at least" I've been able to exec npm -v without crashing :-)
It has showed an strange error though, specially because all seems to work correctly after that:

usrbinenv is an implementation of /usr/bin/env in Javascript, and since it's a common item it has too restrictive permissions in NodeOS (no read or write permissions, only execution to allow to traverse the directories). This work correctly, that's not the problem, but I don't understand why it try to read its package.json file... :-/

@piranna so glad to hear it! Wish it would support a newer version of slap but 0.1.28 will do. :P
The only reason I can see that error appearing is because slap checks all global installed packages to see if they have a slap-plugin keyword to load slap plugins. Maybe that's the issue?

@piranna so glad to hear it! Wish it would support a newer version of slap but 0.1.28 will do. :P

Me too, there's a lot of packages that has upgraded to nan2 and latest Node.js versions. Don't know if another more recent will work, I was going back in time each month until I found the version of May 15 worked :-) There's a bug on v8 that don't allow to use a newer version and also it's needed gcc 4.8 or bigger (we are using 4.7.3), but @heavyk has done some huge progress on this aspects :-) I hope to find some time to git bisect the v8 bug, I think it will be a something really stupid bug...

The only reason I can see that error appearing is because slap checks all global installed packages to see if they have a slap-plugin keyword to load slap plugins. Maybe that's the issue?

usrbinenv is installed globally for all the system, and the package.json file don't have read permissions, so it makes sense. Maybe you would check the EACCESS error and ignore if? It's not an usual situation, but it's a harmless check for this use case anyway...

I've given read permissions to the usrbinenv module and now the error don't appear :-) Next steps, mouse support and upgrade Node.js version to be able to exec last version of slap :-P

Now that NodeOS use latest Node.js 4.3.1 I've been able to install latest version of Slap (hoorray!!! :-D) but I'm not able to use it because it doesn't respond to the keystrokes, also Ctrl+Q to quit. What could I be able to look for to make it work? :-)

That is strange indeed. Try passing --logger.level silly and tailing ~/.slap/slap.log. Does it acknowledge keystrokes at all?

Does it acknowledge keystrokes at all?

Regular shell works flawlessly. Are you talking about blessed-based applications?

I'm talking about in ~/.slap/slap.log.


  title String @db.VarChar(255) leads to Attribute not know<EMAIL_ADDRESS>

Currently for default schema on https://prismaliser.ovy.cloud/

Removing the native type definition fixes the problem of course.
Maybe db is not defined as the datasource name that is sent in as validation? Could be that you have to add that (or just include it in the default schema for the validation to be able to pick it up from there).
Update: Yep, adding this to the default schema makes it work:
datasource db {
  provider = "mysql"
  url = "mysql://foo"
}

Update 2: Maybe this is just using a cached schema vs. the current default that already would fix this problem?

Yeah it's the cached schema. I fixed this earlier during testing the fix for #1 while I came across it, but it means that previous users still have the old one cached


Update README Structure and typo

What happened
Update the README structure for clearer content.
Insight
n/a
Proof Of Work
n/a

@byhbt FYI the failing tests are fixed on https://github.com/nimblehq/elixir-templates/pull/82 :)

@byhbt FYI the failing tests are fixed on #82 :)

Thanks @andyduong1920 for the information.
Do we need to merge the PR https://github.com/nimblehq/elixir-templates/pull/82 into develop branch first then rebase this PR ?

Do we need to merge the PR #82 into develop branch first then rebase this PR ?

Yes @byhbt, that PR needs 1 more approval 🙏 so we can merge that to develop


Help

What is your question?
How do i flip health bar im making a mod that you play on left side and need to flip health bar pls

setProperty('healthbar.angle', 180)
try that

setProperty('healthbar.angle', 180)
try that

That flips the colors around, not reverse the health

setProperty('healthbar.angle', 180) try that

thats what i need, imma just reverse the icons now


missing profile for QNCX 16A plug

I created a dump file for my QNCX 16A plug.
Is it possible for you to create a profile and schema for my device?
Thank you very much!
dump_qncx-16a-cb2s.zip

Added as QNCX 16A Smart Plug


chore(main): release 1.0.6

:robot: I have created a release beep boop
1.0.6 (2023-03-17)
update

更改promise返回类型 (b2ad5a4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/CRM-Vue/releases/tag/v1.0.6 :sunflower:


Channel Name Font-Size

The font of the channel's name is smaller than other texts like the views and upload date

The font of the channel's name is smaller than other texts like the views and upload date
css span{     font-size:70%; }
was the issue


Create cupertino_tk.arb

Turkmen language support

CC @shihaohong

Hi @kea2288, thank you for providing your suggestions for the Turkmen language! Currently, Flutter uses an internal Google system for managing message translations used in the Flutter SDK (for example, the Material and Cupertino widgets' messages).
Currently, we do not take PRs for adding unsupported translations, but we'd certainly consider them if there is enough interest. If you'd simply like to use your own translations for these Material/Cupertino widgets, you can look at this resource for supporting a new language to add your versions of the Turkmen translations to your application.
I'm going to close this PR for now, but please do not hesitate if you have any questions!


[DOC] Improve for multi-tenancy using Tenant edge or "tenant_id" field

Summary 💡
I'm opening this to improve docs about multi-tenancy using Tenant edge or "tenant_id" field.
Motivation 🔦
@a8m suggested me to use Privacy Filter Rules and I did but with a few issues.
If you can confirm the code I used is OK I'll PR to improve docs for (not very smart :smile:) people like me who will come in the future.
Issues (with fix)

[ ] I'm using this code for privacy rule (is it OK?):
 func (TenantMixin) Policy() ent.Policy {
   return privacy.Policy{
     Query: privacy.QueryPolicy{
       rule.FilterTenantRule(),
     },
     Mutation: privacy.MutationPolicy{
     	rule.FilterTenantRule(),
     },
   }
 }

 func FilterTenantRule() privacy.QueryMutationRule {
   type TenantsFilter interface {
     WhereTenantID(entql.Uint64P)
   }

   return privacy.FilterFunc(func(ctx context.Context, f privacy.Filter) error {
     tf, ok := f.(TenantsFilter)
     if !ok {
       return privacy.Denyf("unexpected filter type %T", f)
     }

     tf.WhereTenantID(entql.Uint64EQ(GetTenantID(ctx)))

     return privacy.Skip
   })
 }

[ ] To avoid always using the .SetTenantID() method in the .Create() I used a hook only for the OpCreate like this:
func (TenantMixin) Hooks() []ent.Hook {
  return []ent.Hook{
    hook.On(
      func(next ent.Mutator) ent.Mutator {
        return ent.MutateFunc(func(ctx context.Context, m ent.Mutation) (ent.Value, error) {
          if s, ok := m.(interface{ SetTenantID(uint64) }); ok {
            s.SetTenantID(GetTenantID(ctx))
          }

          return next.Mutate(ctx, m)
        })
      },
      ent.OpCreate,
    ),
  }
}

Issues (unresolved)

[ ] If I use WhereTenantID(uint64) the type assertion doesn't work, why?
 type TenantsFilter interface {
   WhereTenantID(uint64) // this doesn't work, why?
   // WhereTenantID(entql.Uint64P) // this works
 }

The example was improved and the website was updated according to the new changes. Please, see https://entgo.io/docs/privacy#multi-tenancy and https://github.com/ent/ent/tree/master/examples/privacytenant.
Closing


Update submodule

IMPORTANT: Please search among the Pull requests before creating one.
Description
Updates the notebooks submodule
How has this been tested?

Closes

Hi @michalk8, I merged this, hope that was alright.


Question(s) about uninstalling...

I am trying to remove tailwind from my project (yea I know - why the heck would I want to do that??) I keep getting this error when I attempt to push to Heroku:
Enumerating objects: 92, done.
Counting objects: 100% (92/92), done.
Delta compression using up to 4 threads
Compressing objects: 100% (59/59), done.
Writing objects: 100% (71/71), 7.13 KiB | 1.42 MiB/s, done.
Total 71 (delta 43), reused 0 (delta 0), pack-reused 0
remote: Compressing source files... done.
remote: Building source:
remote:
remote: -----> Building on the Heroku-20 stack
remote: -----> Using buildpack: heroku/ruby
remote: -----> Ruby app detected
remote: -----> Installing bundler 2.2.33
remote: -----> Removing BUNDLED WITH version in the Gemfile.lock
remote: -----> Compiling Ruby/Rails
remote: -----> Using Ruby version: ruby-3.0.3
remote: -----> Installing dependencies using bundler 2.2.33
remote:        Running: BUNDLE_WITHOUT='development:test' BUNDLE_PATH=vendor/bundle BUNDLE_BIN=vendor/bundle/bin BUNDLE_DEPLOYMENT=1 bundle install -j4
remote:        You are trying to install in deployment mode after changing
remote:        your Gemfile. Run `bundle install` elsewhere and add the
remote:        updated Gemfile.lock to version control.
remote:
remote:        If this is a development machine, remove the /tmp/build_ef4e16b0/Gemfile freeze
remote:        by running ``.
remote:
remote:        You have deleted from the Gemfile:
remote:        * tailwindcss-rails (~> 2.0)
remote:        Bundler Output: You are trying to install in deployment mode after changing
remote:        your Gemfile. Run `bundle install` elsewhere and add the
remote:        updated Gemfile.lock to version control.
remote:
remote:        If this is a development machine, remove the /tmp/build_ef4e16b0/Gemfile freeze
remote:        by running ``.
remote:
remote:        You have deleted from the Gemfile:
remote:        * tailwindcss-rails (~> 2.0)
remote:
remote:  !
remote:  !     Failed to install gems via Bundler.
remote:  !
remote:  !     Push rejected, failed to compile Ruby app.
remote:
remote:  !     Push failed
remote: Verifying deploy...
remote:
remote: !       Push rejected to thawing-sands-95218.
remote:
To https://git.heroku.com/<myURL>.git
 ! [remote rejected] main -> main (pre-receive hook declined)
error: failed to push some refs to 'https://git.heroku.com/<myURL>.git'

I tried running ./bin/bundle remove tailwindcss-rails which returned the following:
Removing gems from /Users/isaactait/Desktop/Web_Dev/rubyOnRailsApp/Gemfile
tailwindcss-rails (~> 2.0) was removed.
....<more code run>
Fetching tailwindcss-rails 2.0.8 (x86_64-darwin)
Using turbo-rails 1.0.1
Using web-console 4.2.0
Using rails <IP_ADDRESS>
Installing tailwindcss-rails 2.0.8 (x86_64-darwin)

I am new to Ruby on Rails so what am I missing in the uninstall step so that I can successfully push the build to Heroku?

I solved the error by deleting my Procfile.dev file (the only thing I had missed in the clean up 😬)


ValueError: Trying to export a florence2 model

Hello,
I am attempting to export and quantize the Florence-2 model for CPU usage but encountered the following error:
ValueError: Trying to export a florence2 model, that is a custom or unsupported architecture, but no custom onnx configuration was passed as `custom_onnx_configs`. Please refer to https://huggingface.co/docs/optimum/main/en/exporters/onnx/usage_guides/export_a_model#custom-export-of-transformers-models for an example on how to export custom models. Please open an issue at https://github.com/huggingface/optimum/issues if you would like the model type florence2 to be supported natively in the ONNX export.
i am trying to quntize florence 2 model for cpu but its show this error

Based on the error message, it seems that the Florence-2 model is not natively supported for ONNX export, and a custom configuration is required.
Could you please provide guidance or support for exporting and quantizing the Florence-2 model using a custom ONNX configuration? It would be highly beneficial for my project to have native support for this model type, or at least detailed instructions on how to proceed with the custom export.

Would be really nice to be able to convert it to onnx as well, as currently
optimum-cli export onnx --trust-remote-code --model microsoft/Florence-2-large ms_flor_onnx

throws error
KeyError: "Unknown task: image-text-to-text. Possible values are: `audio-classification` for AutoModelForAudioClassification, `audio-frame-classification` for AutoModelForAudioFrameClassification, `audio-xvector` for AutoModelForAudioXVector, `automatic-speech-recognition` for ('AutoModelForSpeechSeq2Seq', 'AutoModelForCTC'), `depth-estimation` for AutoModelForDepthEstimation, `feature-extraction` for AutoModel, `fill-mask` for AutoModelForMaskedLM, `image-classification` for AutoModelForImageClassification, `image-segmentation` for ('AutoModelForImageSegmentation', 'AutoModelForSemanticSegmentation'), `image-to-image` for AutoModelForImageToImage, `image-to-text` for AutoModelForVision2Seq, `mask-generation` for AutoModel, `masked-im` for AutoModelForMaskedImageModeling, `multiple-choice` for AutoModelForMultipleChoice, `object-detection` for AutoModelForObjectDetection, `question-answering` for AutoModelForQuestionAnswering, `semantic-segmentation` for AutoModelForSemanticSegmentation, `text-to-audio` for ('AutoModelForTextToSpectrogram', 'AutoModelForTextToWaveform'), `text-generation` for AutoModelForCausalLM, `text2text-generation` for AutoModelForSeq2SeqLM, `text-classification` for AutoModelForSequenceClassification, `token-classification` for AutoModelForTokenClassification, `zero-shot-image-classification` for AutoModelForZeroShotImageClassification, `zero-shot-object-detection` for AutoModelForZeroShotObjectDetection"


talk: no place to view all notifications

Describe the bug, optionally with screenshots
With the new talk app being separate from groups yet able to view all of my various group chat channels, I find myself using it almost constantly (mainly sorted by recent messages). That said, it means that I only can see that a channel has unread messages, not any other notifications. This means that I'll often miss mentions in groups chats, particularly for channels that I participate in less often which have stayed unread and drifted to the bottom of my channel list.
To Reproduce
Steps to reproduce the behavior:

Go to Talk
Click on nothing
See giant blank frame with chat channels on the lefthand panel.
Wish for notifications feed.

Expected behavior
I'd love to be able to see my notifications feed (maybe with an option to filter by X options, but not necessary at present), similar to the experience when first opening Groups.
Additional context:

Ship type: Planet
Instance type: self-hosted
OS/browser: Brave

@urcades

Tracking internally as LAND-384


Refactor DSN and transport configuration

Passing a transport via a scheme prefix is now deprecated in favor of explicit class loading:
>>> Client(dsn, transport=HttpTransport)

Additionally DSN-related configuration is now held in the RemoteConfig helper, which is bound on Client.remote.
This removes raven.conf.load but otherwise should be backwards compatible.
Fixes GH-587

There will be a followup backwards incompatible change that removes binding of Transports to a URL. They will become generic reusable for any HTTP connection. It will only cause problems for people with custom transports (which is likely very few people).

Changes Unknown when pulling 12bb73770747ae8a6cc688bff32279257362e76b on dsn-refactor into ** on master**.


(+VIRALˇ𝘝𝘐𝘋𝘌𝘖𝘴:.ＳＥＸ) Minahil Malik Nazeba 𝙻𝚎𝚊𝚔𝚎𝚍 𝚅𝚒𝚍𝚎𝚘 MMS On Social Media X Twitter [LINK√FULL]

13 sec ago Minahil Malik Nazeba 𝙻𝚎𝚊𝚔𝚎𝚍 𝚅𝚒𝚍𝚎𝚘 Viral On Social Media. (+VIRALˇ𝘝𝘐𝘋𝘌𝘖𝘴:.ＳＥＸ) Minahil Malik Nazeba 𝙻𝚎𝚊𝚔𝚎𝚍 𝚅𝚒𝚍𝚎𝚘 MMS On Social Media X Twitter [LINK√FULL]

Minahil Malik Nazeba's latest video has taken the internet by storm, igniting conversations across various social media platforms. The rising digital star, known for her engaging content—especially her recent Spiderman-themed clip—has experienced a remarkable boost in popularity, with fans eagerly seeking the full version. Viewers are flocking to Instagram, X (formerly Twitter), and Discord to engage with her latest creation.
As excitement grows, fans are passionately sharing their reactions online, emphasizing the crucial role social media plays in propelling creators like Minahil Malik Nazeba into the spotlight. With her unique style and innovative approach, she has quickly made a notable impact in the digital landscape, leaving many curious about her next moves.


setSendAt applying to the first email

Issue Summary
I'm setting up an email to go to multiple recipients, but they shouldn't see each other on the To line. That's working. But when I add a setSendAt to the email object it's being applied to the first email, the 0 index in the personalization array.
Steps to Reproduce
Here's the function I'm using to add and send the emails. It's been working as expected until I needed to add in the send schedule. I feel like I might just be missing something obvious, I'd really appreciate any guidance!
	public function setupEmailAndSend($template_id, $company_id) {
		$this->recipients = self::populateRecipients($company_id); //returns an array of To objects
		$this->from = new From($this->sender_email, $this->sender_name);
		$this->email = new Mail(
			$this->from,
			$this->recipients
		);
		$this->email->setSendAt(self::getTodayAtTwoUnix());
		$this->email->setTemplateId($template_id);
		$this->status = self::sendMail();
		return self::testSuccess($this->status);
	}

getTodayAtTwoUnix() just returns the unix timestamp, as an INT, for today at 2pm EST.
The top-level Mail object var dumps:
["send_at":"SendGrid\Mail\Mail":private]=>
  NULL

but personalization ->0 has the correct info:
["send_at":"SendGrid\Mail\Personalization":private]=>
      object(SendGrid\Mail\SendAt)#23 (1) {
        ["send_at":"SendGrid\Mail\SendAt":private]=>
        int(1570644000)
      }

The rest of the personalization objects are send_at null.

Ok, so to answer my own question, here's how I've accomplished what I'm looking to do. Please let me know if there's a more logical solution that I've missed. Thanks!
		$this->send_time = $this->getTodayAtTwoUnix();
		$this->recipients = $this->populateRecipients($company_id);
		$this->from = new From($this->sender_email, $this->sender_name);
		$this->email = new Mail(
			$this->from,
			$this->recipients
		);
//once the recipients are set they have personalization objects built, so now I get those objects and fill in the send at directly for each
		foreach ($this->email->getPersonalizations() as $key => $value) {
			$this->email->setSendAt($this->send_time, $key, $value);
		}

Something I do still have a question about:
If I set up an email to send at 2pm EST and it's after that time, 2:30 EST. Does it send right away?

Hello @bjornmann,
Thanks for taking the time to share the solution! We don't have a sendAt specific use case written up just yet, but in the meantime, hopefully the kitchen sync example is useful.
In my testing, if the sendAt time is in the past, it will send immediately.
With best regards,
Elmer


Logged in user flow in Explorer

create paths for a logged in user
Differences/Updates (from demo user experience)
Logged in user can:

[x] change url params for Sample Searches
[x] change url params for keyword search
[ ] select any media as provided in selectMedia page - already it's own issue #445
[x] select any date range
[x] the back end requires a log in for these api calls

dates are required or the query will fail, how do we handle default for logged in users? let it fail or provide the two-week range?

closing and continuing towards source selection widget


Proper way to handle csv input for cpu training?

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 16.04
TensorFlow installed from (source or binary):
pip
TensorFlow version (use command below):
('v1.3.0-rc2-20-g0787eee', '1.3.0')
Python version:
Python 2.7.12 from Anaconda

Describe the problem
I am training some classification model with an 32-cpu Ubuntu machine and one of the problem is to feed data fast enough to the training process.
I am trying to read data from some csv file but the default tf.csv or tf.data module seems to be slow.
A speed test for reading 1000000 row * 17 column csv file shows a speed like :

tf.decode_csv with queue and theads  :   ~192 seconds
tf.data :  ~164 seconds
hand write cpp reading op :  ~25 seconds
pure python code with help from pandas : ~23 seconds

It is fast enough to use pandas for one single file, but it might face the GIL problem if try to speed up with more threads.
Codes can be found below. I am not sure if I use it the right way, is there any official benchmarks or guidelines for this?
Source code / logs
https://github.com/littleDing/mini_csv_reader
The speed test is run through speed_test.py

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!

I think it is a feature request.
There is a simple demo on the official website about reading data, but never a best-practice under different scenes, or any official benchmarks. I could run according the demos with small datasets but once it became some million or billions it would be an nightmare.
It took me very long time to figure out how to feed csv data into training process efficiently. But as @mrry suggested in https://github.com/tensorflow/tensorflow/issues/13883, there might still be some upgrade that might be useful, so I do the test and post this issue.
To be honest,  I am still confused now and I think lots of developers would feel the same before any official guidelines is posted.


CoFHCore is required

Forge: 1448
RFTools 3.31
McJtyLib 1.3.0
[Client thread/ERROR] [FML]: The mod rftools (RFTools) requires mods [CoFHCore] to be available

Suggestion:  Edit related projects under RFTools 3.31 to point to CoFHCore as a required library like what you have done with McJtyLib

Fixed some time ago.


Add a changelog for all packages

:books: Documentation preview :books:: https://metatensor--403.org.readthedocs.build/en/403/

Yes will be useful for the future. I would also add a PULL_REQUEST_TEMPLATE.md to the .github and do we also want the changelog to be displayed in the docs?

Yes to both, I'll have a go at it ASAP


Updating configuration..yaml file ONLY doesn't publish changes to expected environment

Hello all,
During my testing I noticed that if I only change the configuration.prod.yaml file and commit that change it doesn't update the environment as I would expect.  The reason seems to be related to the fact that nothing actually changed in the artifacts/named values/ folders.
As a workaround, I was able to update one of the namedValueInformation.json files along with the configuration.prod.yaml file in a single commit and that change DID get applied to the prod environment as expected.  This is not a workable solution in my opinion as it will also update the lower environment since the namedValueInformation.json file had to be updated.
Doesn't work when this is included in the commit:
apiops/configuration.prod.yaml -> file changed and committed but nothing published to the prod environment.
Works when this is included in the commit:
apiops/artifacts/named values/environment/namedValueInformation.json -> file changed, committed, and published to lower environment
apiops/configuration.prod.yaml -> file changed, committed, and published to higher environment
Let me know if you need additional details.

The fix would be to go through the config.yaml and republish all the APIM objects specified in the config.yaml. Currently, since the config file isn't considered a APIM object just changing that doesn't publish anything.

Correct. Working on a fix.

This has been fixed and published part of release v0.2.0

@waelkdouh, this resolved my issue...thanks all!


Introduce koverrides

Introduce the concept of koverrides, which provide a lightweight interface for supplying environment-specific configuration settings in Kubernetes YAML files(s).
Similar to the ko:// prefix, YAML nodes containing a koverride:// prefix will be preprocessed. Users can then use the --koverrides flag to provide configuration overrides (if necessary).
See README updates in this changeset for more details.
(copy pasting README changes below)
"koverrides"
When using ko in the context of Kubernetes, you may wish to reuse your YAML file(s)
across several different environments.
As a lightweight solution for environment-specific configuration, you can leverage
"koverrides" (a.k.a. ko overrides).
Using koverrides, you can prefix values with koverride:// and supply a default value:
apiVersion: v1
kind: ConfigMap
metadata:
  name: demo
data:
  db-host: koverride://db.host/mysql.default.svc
  db-port: koverride://db.port/3306
  db-user: koverride://db.user
  db-pass: koverride://db.pass

The default value can be defined as everything after the first forward slash (/)
character. If there is no default, then an empty string will be used ("").
Nested keys are supported using dot (.) notation.
Consider the following koverrides file, demo.yaml:
db:
  host: db.example.com
  user: secretuser
  pass: secretpass

Now we run ko resolve (or ko apply) using the
--koverrides flag, pointing to the file:
ko resolve -f config/ --koverrides demo.yaml

Which produces the following YAML:
apiVersion: v1
kind: ConfigMap
metadata:
  name: demo
data:
  db-host: db.example.com
  db-port: "3306"
  db-user: secretuser
  db-pass: secretpass

For practical purposes, all resulting values are coerced to strings
(e.g. 3306 -> "3306").
This feature is particularly useful for Kubernetes resource types designed
for configuration purposes, such as ConfigMaps or Secrets.

Interesting!
Is there a way a user could do this with some existing YAML templating tool like kustomize today, that koverrides supports better somehow? In general I'd prefer to have ko remain a relatively single-use tool, rather than embarking down the path of more complex YAML templating/overrides, especially while existing tools exist for that.

Yeah I'd punt this to kustomize or skaffold.

Perhaps you didn't notice, but I included unit tests for this feature.

Codecov Report

Merging #583 (be7b84c) into main (1425e4b) will increase coverage by 0.56%.
The diff coverage is 70.49%.

@@            Coverage Diff             @@
##             main     #583      +/-   ##
==========================================
+ Coverage   48.58%   49.14%   +0.56%
==========================================
  Files          43       43
  Lines        2221     2281      +60
==========================================
+ Hits         1079     1121      +42
- Misses        956      974      +18
  Partials      186      186

Impacted Files
Coverage Δ

pkg/build/gobuild.go
57.14% <0.00%> (-0.46%)
:arrow_down:

pkg/build/gobuilds.go
71.15% <0.00%> (-1.40%)
:arrow_down:

pkg/commands/options/filestuff.go
0.00% <0.00%> (ø)

pkg/commands/resolver.go
29.59% <0.00%> (-1.11%)
:arrow_down:

pkg/internal/testing/fixed.go
59.25% <0.00%> (-7.41%)
:arrow_down:

pkg/build/recorder.go
87.50% <100.00%> (+1.78%)
:arrow_up:

pkg/build/shared.go
95.00% <100.00%> (+0.26%)
:arrow_up:

pkg/build/strict.go
100.00% <100.00%> (ø)

pkg/resolve/resolve.go
86.88% <100.00%> (+14.47%)
:arrow_up:

Continue to review full report at Codecov.

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


Add API to contribute problem matchers (with fixits) as JS functions

I am writing a small expect testing library for my project. After I run the test, I want to see a :bulb: in the editor with suggestions to update the expected value in the tests.
Ideally, I'd just print failure messages in a special format to stdout on test failure, and have the editor automatically convert that into a diagostic and associated fix.
The diagnostic part is possible today with regex problem matchers. However for providing fixits, it seems like a more flexible API is required.

Take a look at the code action api: https://github.com/microsoft/vscode/blob/11fda69d237b9aa006eb53da724fa509afe92e0a/src/vs/vscode.d.ts#L2122
This is what extensions such as eslint use to implement fixes

Yup, I can use CodeActions to provide fixits. The problem is, I can't consume a Task output. I can work around this by writing substantially more code and managing process execution myself, but then it won't be integrated nicely with VS Code tasks UI.
Or am I missing something and there's some kind of onTaskOutput(line: string) event somewhere?

Tasks generate diagnostics which your code actions can use.
There are no plans to generate fixes from task output. For more advanced use cases like your, you'll need to write adapter code in your extension. It would probably be easier not to use tasks at all here and just have your extension generate diagnostics directly. The logic for generating the diagnostics and fixes can still live in a separate library or process


Keras Layer With Trainable=False Flag Creates Variables With Trainable=True

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
System Version: macOS 10.15.2 (19C57)
Kernel Version: Darwin 19.2.0
TensorFlow installed from (source or binary): binary (pip)
TensorFlow version (use command below): v1.15.0-rc3-22-g590d6eef7e 1.15.0
Python version: 3.7.0

Describe the current behavior
When I create a Keras model using layerwise structure in tf.keras.Model, and set one or more of the layers to trainable=False, this results in a situation where the keras model believes those variables/parameters to be non-trainable (and reports them as such in model.non_trainable_variables and model.summary) but doesn't set the actual trainable flag on the created Tensorflow variables to be False. This can lead to unintuitive behavior if other systems are reasonably expecting var_obj.trainable to correspond with the model's actual trainable variables.
Describe the expected behavior
When a layer is set to trainable=False, Tensorflow variables associated with that layer also have the trainable flag set to False
Code to reproduce the issue
inputs = tf.keras.layers.Input(shape=(10,))
hiddens = tf.keras.layers.Dense(15,  trainable=True, name="trainable_layer")(inputs)
output = tf.keras.layers.Dense(5, trainable=False, name="nontrainable_layer")(hiddens)
model = tf.keras.Model(inputs, output)
model.compile(optimizer='rmsprop', loss='categorical_crossentropy')
print(f"Model trainable variables: {model.trainable_variables}")
print(f"Model non-trainable variables: {model.non_trainable_variables}")
print(f"Trainable flags on model non-trainable variables: {[v.trainable for v in model.non_trainable_variables]}")

@decodyng  Could you please confirm if the issue still persist in the nightly version gist shared

@Saduf2019 I'm not sure if you intended me to independently try to install and check for the bug on the nightly version, but in the linked notebook, it does appear that variables in the non_trainable_variables set are still having their trainable flag incorrectly set to True.


Component Licensing?

I notice that both Public Domain and Apache 2 are listed in this template. My understanding is that Public Domain is not a license at all and contains no restrictions except perhaps the convention of source attribution. The Apache 2 license is quite permissive but it does have certain restrictions.
My assumptions are:

If an already licensed component is packaged then that component would be under its existing license.
If a component is the partial work of non-CDC folk it may fall under a license depending on what the team decides unless the work is separable between federal and non-federal contributors. Separable work by federal employees should be Public Domain.
Highly derivative work (simple port or copy) of an existing licensed software will fall under said license.
If the component is the sole work of a Federal employee, then indeed it is required to be Public Domain.
Public Domain software can be included in a licensed work BUT I am unfamiliar with any mechanism that would require separable, new contributions from non-federal employees to be dedicated to public domain.
Modification of public domain code bases likely should be able to retain their status as per the guidance in the template.

I am wondering if the language could be improved to specify how components fall under a license or not (public domain). This may require modules and components be granted separate licensing or public domain dedications, but the package as a whole is generally under the most restrictive license, is it not?
IANAL. Would be grateful for guidance from someone who has more experience in this area.

Thanks for the comment and thinking about this. I want to improve it, but am kind of stuck. The thing about the public domain (and other parts reference cc0) is because that's in there because pretty much everything cdc produces is public domain as a government work. That addresses copyright but isn't a license. So we also specify Apache2 (or MIT, or BSD, or GPL or whatever) to make our work easier for others to use and reduce questions. It also helps with contributions because it can get confusing to contributors who may not understand how to release in public domain but do no how to license their contributions under the particular license.
There was also some thought around while public domain exists, the no warranty notices were useful to explicitly state and reduce confusion by people who might think software does have warranty when it almost never does.
We worked with OGC on this version and the GSA code.gov group and have been stuck at this current version for a few years. I agree I'd like it to be less confusing (and thus more useful and thus more CDC software available) but not sure what action to take as everything got into project specific particulars that didn't generalize into this template.
The current wording applies to the entire project and all components within it but projects can customize if particular parts are under other licenses or restrictions. I feel like there are some examples in CDCGov but couldn't find any to link here. So the entire package would fall under the license unless otherwise stated. Depending on the complexity, it may make it easier to use to just store modules as separate projects that explicitly note their language and then link to them as needed.
If we can come up with specific questions, I can start a new conversation with OGC as they're super smart and have been very helpful with this so far.

Thanks for the thoughtful response.
I have indeed started borrowing some of the language you all produced and customized it slightly to refer to individual files/scripts rather than the repo as a whole. I dedicate the individual scripts/modules/classes as public domain in the header comments but license the overall project based on component requirements. Some of our components are GPLv3, so the entire package ends up under GPLv3 given the viral license. I note to users that there is additional licensing flexibility for individual files but that the project is generally under license X.
I agree that it makes sense to split up repos with incompatible licensing where the boundaries make sense (software-wise). However, I wouldn't split a project solely for the readability of the license terms. I can provide some more examples when I get further down this road. Most of our software is still on-prem, partially because of the confusion regarding clearance and licensing. The software that has been released pre-dates CDC's expanded use of github and involves explicit naming of component licenses.
Thanks for the distinction between copyright and license, that was a piece I sometimes falsely equivocate.

Glad this is helpful and that's a really good point about GPL licenses. I think the convention is as long as items are clearly marked (in readme or in specific folders) it doesn't really matter to have them in the same repo. I spend a lot of time trying to figure out what clear enough is, and that's where we can loop in OGC to confirm or suggest something. I think the package should be clear so that a user understands whether it's in github, some other site or distribution, or they just find it in a folder.
As you go through these, we could add links to example projects that might be useful references for others.
I'm glad you're building software with this in mind. As I join ID, I'm really interested in how ID packages are used as OSS and we can make them more used in the future.


Fix for 'PDOException with message 'SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: Name or service not known'

Our Command Bash is limited to default network...
By Default you are not using the default network...
since it will create a new name of network based on your app Name
using php command will always yield an error
PDOException with message 'SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: Name or service not known'

So I made a new Command called artisan
function artisan() {
	docker run -it --rm -v $(pwd):/var/www/app -v $A_COMPOSER_MOUNT -v $A_SSH_PHP_MOUNT --network=$1 --link $2:mysql ambientum/php:7.0 php artisan "$3"
}
alias artisan=artisan

we can use it as follows
artisan network_name container_id_or_name artisan_commands

so for example
artisan docker53_default docker53_mysql migrate

For running commands on a compose defined network, its probably a better ideia to run it against a container that is compose defined.
That's how you can do it:
docker-compose run [service-name] [command]

so for example if you want to migrate
docker-compose run app php artisan migrate

on this case, i'm considering the web service name is app, name it as you named the service inside docker-compose container
you can alias artisan in a better way
alias artisan='docker-compose run app php artisan'

and them just use
artisan migrate

This way, the container will have connection to all the links defined inside docker-compose.yml, and the .env file will be valid against both


Fix "Prefer format() over string interpolation operator" issue

This pull request automatically fixes all occurrences of the following issue:
Issue type: Prefer format() over string interpolation operator
Issue details: https://www.quantifiedcode.com/app/project/gh:Miserlou:Zappa?groups=code_patterns/%3A4ACGxFj1
To adjust the commit message or the actual code changes, just rebase or cherry-pick the commits.
For questions or feedback reach out to<EMAIL_ADDRESS>Legal note: We won't claim any copyrights on the code changes.
Cheers,
Cody - Your code quality bot

Coverage remained the same at 90.568% when pulling 799e4d4a3732201b2f56ba25ed9c312ad0fc1f00 on autofix/wrapped2_to3_fix-0 into 32c45486f2460237bddf6ba48cedb498a0f8b742 on master.


refactor(RadioButton): lightweight styling radiobutton

GitHub Issue: #991 #1067
PR Type
What kind of change does this PR introduce?

Refactoring (no functional changes, no api changes)

Description

Refactors non-brush resources for lightweight styling.

PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[x] Tested UWP
[ ] Tested iOS
[ ] Tested Android
[x] Tested WASM
[ ] Tested MacOS
[x] Contains No breaking changes

If the pull request contains breaking changes, commit message must contain a detailed description of the action to take for the consumer of this library. As explained by the Conventional Commits specification

@mergifyio backport legacy/2x


exported video is lagging

Describe the bug
i have created video with the help of render package but video is so much lagging
a widget that i want to render in video its contains some text widget with animation
Expected behavior
video should play smooth.

This package is very well documented, please refer to the Using settings to keep up performance heading to check what you can do to keep performing at high quality. Due to hardware limitations, you must compensate for either the process duration, video quality, or fluency (what you have described). I recommend taking longer rendering time instead of fluency.
I have thought about making automatically adjusted settings (where you just set your preferences of compensation), but it is hard to find documentation to check the current FPS of the device. I wasn't able to archive that yet.
I am gonna close this issue, as I think generally this is a settings issue. If you want to get invested in building an automatic solution, please don't hesitate to reach out to me for help (eg. pull request).

yes i have checked your documentation and i have tested will all possible combination but unable to achieve desired output

Could you please provide me with a mockup code of what you are trying to archive? If your UI widget is big, it might also be unnecessary to set a high pixel ratio.
I recommend you lower the pixelRatio and lower the simultaneousCaptureHandlers while setting your frameRate to your desired output. Then slowly adjust settings to fit even more to your needs, without lagging.
Note that lagging is not caused by the post-capturing processing, but by too many simultaneous processes during capturing,  which makes the whole application lose its frame rate and therefore also the UI images that can be repainted (due to too many handlers, images being stored, etc.). Therefore, it is critical to keep the device framerate high, while post-processing usually can help you out by adjusting the image quality through upscaling.


Sailsjs Validate one model atribute only.

Hello everyone.
I am pleased to use this framework for a lot of projects and it's simply fantanstic.
Today i had a situation where i was in need to validate just some attributes of my model. For example, in my user model, i want to validate just the user password and confirmation attributes, instead of causing a global validation, by calling Model.update. Maybe there is a Model.validate or Model.validadeAttribute method that i am missing?
My question is,  how can this be done, without developing new code, by using the built in methods?
And thanks for this wonderful tool that you guys are mantaining.

@netrevisanto has your question been resolved?


Harmonize ventilator modes

Datasets have different encodings for ventilator modes. These are extracted in the blendedICU dataset, but not harmonized (yet).

Hi,
I am wondering how long would it take for you to have the ventilator modes harmonized and integrated in your pipeline?
Best,
Mostafa

Hi,
I am not actively working on this right now, but I will when I get some down time on other projects. I expect it to be integrated in a few weeks.
Thanks for your interest in the project !


The error occurred during compileInlining.

java.lang.NullPointerException: getTypes(...) must not be null
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileInlineStep.compile$lambda$1(SwaggerSchemaCompileInlineStep.kt:30)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:29)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:37)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:33)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:36)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:33)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:37)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:33)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences(SwaggerSchemaCompileUtils.kt:37)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileUtils.resolveReferences$default(SwaggerSchemaCompileUtils.kt:24)
   at io.github.smiley4.schemakenerator.swagger.steps.SwaggerSchemaCompileInlineStep.compile(SwaggerSchemaCompileInlineStep.kt:23)

...
install(SwaggerUI) {
    ...
    schemas {
        generator = {
            it.processReflection()
                .generateSwaggerSchema()
                .withTitle(TitleType.SIMPLE)
                .compileInlining() // error here
        }
    }
    ...
}

This works fine in 1.4.3, but the above error occurs in 1.5.0.
However, if I replace compileInlining with compileReferencingRoot, no error occurs, so I think this is an issue with compileInlining?

Hi,
that defenitly looks like an issue with compileInlining in schema-kenerator. Could you perhaps provide some model which i can use to reproduce the error?

Hi, that defenitly looks like an issue with compileInlining in schema-kenerator. Could you perhaps provide some model which i can use to reproduce the error?

This is the code I found to reproduce the issue minimally:
// any sealed class
sealed class SealedClass
// any class that extends the sealed class
class A: SealedClass()
// any class that has a nullable field of the sealed class
// must be a nullable field
class B(val a: SealedClass?)

fun main()
{
    val type = typeOf<B>()
    type
        .processReflection()
        .generateSwaggerSchema()
        .withTitle(TitleType.SIMPLE)
        .compileInlining() // error here
}

Ah, thank you.
i fount the issue. The bug should be fixed in the next version :+1:


Sum of nodes on the longest path from root to leaf node

Description
Code added for Sum of nodes on the longest path from root to leaf node in C++
Fixes #1913
Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

@ravikr126 Kindly review my PR under LGMSOC'21.


Blazor WASM components in Server-hosted scenario

Fixes #18431

This has been placed on hold until engineering advances the feature in the framework.


Using Lines to connect two sets of Markers?

First of all - thanks for creating this amazing library! This is more of a 'best-practice' question v. an issue...happy to supply some examples of my code of course! I am building a map that shows bookings...I am trying to show a marker with the activity that was booked, and a line connecting to a second marker for the billing address of the customer. I currently have two sets of markers rendering, would I use Lines to connect the two with start and end coordinates? Would there be a better way to do this? Thanks for any insights!

Hi @Unionindesign,
Yep, probably best to use lines for this. It's an interesting use case. Would love to see examples of this.


chore(deps): bump https://github.com/zeebe-io/zeebe-cluster-helm from 0.0.147 to 0.0.148

Update zeebe-io/zeebe-cluster-helm from 0.0.147 to 0.0.148
Command run was jx step create pr chart --name zeebe-cluster --version 0.0.148 --repo https://github.com/zeebe-io/zeebe-full-helm.git

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign salaboy
You can assign the PR to them by writing /assign @salaboy in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


CLexer misclassifies mulitline comment

(Original issue 656 created by None on 2011-04-05T15:11:31.092441+00:00)
For input in C:
{{{
/* PIPS include guard begin: #include <stdio.h> /
#include <stdio.h>
/ PIPS include guard end: #include <stdio.h> */
}}}
after performing code highlighting (Clexer & HtmlFormatter) I get wrong output:
{{{
/* PIPS include guard begin: #include <stdio.h> /
#include <stdio.h>
/ PIPS include guard end: #include <stdio.h> */
}}}
First line is wrongly classified as a Comment.Preproc, while it should be Comment.Multiline.
I've extracted also tokens:
{{{

Token.Comment.Preproc   u'    /* PIPS include guard begin: #include<stdio.h> /\n    #'
Token.Comment.Preproc   u'include<stdio.h>'
Token.Comment.Preproc   u'\n'
Token.Text      u'    '
Token.Comment.Multiline u'/ PIPS include guard end: #include<stdio.h> */'
Token.Text      u'\n'
}}}
First line ends not with '\n', but '#'. What happened?

(Original issue 656 last updated on 2012-02-05T11:28:07.065850+00:00)
(Issue automaticaly closed due to status in Bitbucket: duplicate)

(Original comment by madika on 2011-04-06T09:08:09.798555+00:00)
After further investiogation, it looks like the problem lies in the combination of two regexes in
pygments/lexers/compiled.py :

Line 42: _ws = r'(?:\s|//.?\n|/[].?[]/)+'

This regex matches a white space or a comment.

Line 51: ('^' + _ws + '#', Comment.Preproc, 'macro'),

This regex will match the first line and the leading "#" on line 2,
because in a multiline matching context, the comment on line 1 is
considered as whitespace preceding the "#".

(Original comment by birkenfeld on 2012-02-05T11:28:07.095226+00:00)
Duplicate of #688.


Add support to File data store for files that do not have a directory prefix in their path

e.g. 'image.jpg' instead of '123/456/78/image.jpg' - previously an invalid argument exception was being raised when purging empty directories.

Thanks man!


Create cli command to serve frontend

Overview
In order, to serve the post install screen, we will create a command to be chained on the end of create, that will show the tools added post-installation on a dashboard.
For that to work:

Installer will download a zip/tar file with post-built UI artifacts
New Command will host the UI static content
New Command will have an API to translate read requests to viper file.

The command will allow use the viper(.kubefirst) to populate ui fields.
As reference, this code may help: https://github.com/kubefirst/kubefirst/blob/main/internal/reports/section.go
It provide the formula for URLs and Viper pointers.

Installers has ui available under create, using a flag.
Details: #348


Improve indexing performance

I did some testing and was able to improve times for sending documents from ~11.2 seconds to ~9.5 seconds by sending the documents concurrently and doing the settings update concurrently. Admittedly my testing was not proper bench marking,  I used time pytest build_index.py and took and average.
Additionally updating the settings before sending the documents should be faster on the Meilisearch side.

This looks great, and makes a lot of sense! I'll look into adding further testing but for now I'm very much doing a similar average timing across runs to get an idea of performance. Thanks for writing such a great async package!


Ability to set custom public gateway

I'd like the ability to define a custom gateway in the settings like you can with the IPFS Companion browser extension, so the link copied when a file/screenshot is shared or when "view on IPFS gateway" doesn't have to be from share.ipfs.io or ipfs.io respectively.

Hmm, View on IPFS gateway is truly part of IPFS Web UI. @olizilla I think this would be easy to implement on Web UI. Shall we move the issue there?


Test prod script

Small script to generate a mini prod tree on the cluster with its paths config

Imo, I don't think this script is needed in the lib.

We'll need this script in the future, I'd like to save it somewhere. Where should I put it ?

data_test directory at LP doesn't contain off0.4deg gammas (I added by hand). If we create a test_subsample, please include all particles to check a real (although small) workflow.

Regenerate using the up-to-date script please.


Bug (and fix) in nodes.py that affects custom filters

According to http://jinja.pocoo.org/docs/dev/api/#writing-filters we can use the decorator @environmentfilter on a custom filter function:

"...for marking environment dependent filters. The current Environment is passed to the filter as first argument."

However a bug in nodes.py (see Filter.as_const at approx. line 568) causes the Environment to be passed as the SECOND parameter under certain conditions.  Specifically, when the filter is called on a static string, the as_const() method does not work properly.
The solution is simple:

Immediately after the line:
args = [x.as_const(eval_ctx) for x in self.args]

Add two new lines:
main_args = []
main_args.insert(0, obj)

Then replace the line:
-args.insert(0, eval_ctx)

Replace with::
main_args.insert(0, eval_ctx))

Then replace the line:
-args.insert(0, self.environment)

Replace with::
main_args.insert(0, self.environment)

Then replace the line:
-return filter_(obj, *args, **kwargs)

Replace with::
return filter_(main_args, *args, **kwargs)

This seems to solve the problem.
You can easily see the error in the original code:  return filter_(obj, *args, **kwargs) is not going to pass in the Environment as the first argument.
The fix simply uses a new local variable main_args to properly sequence obj, eval_ctx and _self.environment as needed, and use these in the call to filter_().  Separately. *args and **kwargs are passed through as well.
If I am misunderstanding something, please let me know.
Otherwise, please consider including this fix in a future release.  I am currently using Jinja 2.8

Just to clarify (after further testing), without the fix described above the arguments to a custom filter are passed in the wrong order when the filter is called on a static string.
For example, given a filter named myFilter (that is decorated with @environmentfilter):
{{ "someText"|myFilter }} will cause:
The static string "someText" to be passed as the first argument to myFilter
The Jinja environment to be passed as the second argument to myFilter
On the other hand, if the filter is called on Jinja data, the behavior is different / is as documented:
{{ data.someValue|myFilter }} will cause:
The Jinja environment to be passed in as the first argument to myFilter
The value of data.someValue to be passed in as the second argument to myFilter
The fix described above corrects the behavior when calling the filter on a static string:  the Jinja environment is passed to the filter as the first argument, and the static string is passed as the second argument.


Support Unicode file names in MultipartMediaUploader

Fixes https://github.com/google/googleapis.dart/issues/238
Also add a regression test to the integration suite

@jonasfj – I plan to publish ASAP after approval!


Pass through Lua code in script

This allows the literal characters % and balanced pairs of { } to appear in either command (\script{...}) or environment (\begin{script}...\end{script}) forms of the script command and be passed through as-is to Lua for evaluation.
Todo before merging:

[ ] Make sure we're passing regression tests on everything that used to build.
[x] Update documentation to describe new  allowed syntax and limitations.
[ ] Refactor LPEG capture groups or add AST tree cleanup so we're not passing as many arbitrary data types further down the parsing pipeline than we need to be.

This will close #450.

Hmm it looks like the only reason escape sequences aren't allowed currently is because of something I overlooked in the LPEG grammar. It should be possible to get them working properly so 100% of valid Lua code would be allowed, minus whatever the closing tag (or unmatched closing brace) is.

I updated this to fix handling Lua string escape sequences (and remove the caveat about them from the docs).

Refactoring the LPEG groups is going to be a different kind of job that this PR, so I'm nixing that item. It looks like the right way to do that is to compress the tree one more level than it is (so what is now passed around as content[1] from TeXlike functions becomes just content). This would also normalize the use of function  from Lua vs. from TeX-flavor markup, but that's a major change and I don't see any simpler way of cleaning up the grammar. So I cleaned up the coding style and we'll call it good for this PR.
I've tested this on my own machine and the metrics I get for the current master branch. This doesn't seem to introduce any new regressions.


Trigger a warning when regression trainer is not given an array

When you train a network for regression and that you want to predict only one value it is very tricky that the trainer should take an array of one value:
var stats = trainer.train(line.x, [line.y]);

i just added a console log so that beginners users realise this quickly (i've seen two people fall in this trap in a short interval).

You're right, it's a tricky part. I burned myself a few times too. Is there an off by one error in your code? Should you be subtracting -1 from the index?

oops sorry about that. just corrected

Thanks!


#200: Updated PeerListProvider interface to extend Listenable, update…

…d implementations.
Issue #, if available: #200
Description of changes:
The hash ring needs to be rebuilt whenever the list of peers changes. Hooked in to Armeria's Listenable interface for this purpose.

Updated PeerListProvider interface to extend Armeria's Listenable interface
Updated static and DNS implementations

Testing:
Started a cluster w/ DNS discovery, scaled from 1 node to 3 and observed logs:
...
1113 [main] INFO  com.amazon.dataprepper.plugins.processor.peerforwarder.HashRing  – Building hash ring with endpoints: [<IP_ADDRESS>]
...
5783 [armeria-boss-http-*:21890] INFO  com.linecorp.armeria.server.Server  – Serving HTTP at /<IP_ADDRESS>:21890 - http://<IP_ADDRESS>:21890/
5783 [main] INFO  com.amazon.dataprepper.plugins.source.oteltrace.OTelTraceSource  – Started otel_trace_source...
5783 [main] INFO  com.amazon.dataprepper.pipeline.Pipeline  – Pipeline [entry-pipeline] - Submitting request to initiate the pipeline processing
5884 [entry-pipeline-process-worker-1-thread-1] INFO  com.amazon.dataprepper.pipeline.ProcessWorker  –  entry-pipeline Worker: No records received from buffer
7914 [raw-pipeline-process-worker-3-thread-1] INFO  com.amazon.dataprepper.pipeline.ProcessWorker  –  raw-pipeline Worker: No records received from buffer
15534 [entry-pipeline-process-worker-1-thread-1] INFO  com.amazon.dataprepper.pipeline.ProcessWorker  –  entry-pipeline Worker: Processing 1 records from buffer
15640 [service-map-pipeline-process-worker-2-thread-1] INFO  com.amazon.dataprepper.pipeline.ProcessWorker  –  service-map-pipeline Worker: Processing 1 records from buffer
...
61119 [armeria-common-worker-epoll-2-1] INFO  com.amazon.dataprepper.plugins.processor.peerforwarder.HashRing  – Building hash ring with endpoints: [<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>]

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Something that stands out to me is that there is some considerable logic added, but no tests are added to confirm that the new code will function as expected. Is the logic/functionality covered by existing tests?

Fair point - we have 2 new methods which intentionally do nothing, and 2 new methods which make 1-line calls to an Armeria object (which should be covered by its own unit tests). I can add tests to at least verify the Armeria object was called correctly however.


Support specifying which IP addresses to listen on

Allow to define which IP addresses to bind the WebSocket server on. This is especially useful to limit which addresses the add-on will listen on since it runs in host network context.

see the linter error from mypy

see the linter error from mypy

Ugh, yeah I see TCPSite supports only a single string as host, whereas the underlying loop.create_server supports a list. We've run into this limitation in the Core already (see https://github.com/home-assistant/core/blob/2024.2.0b8/homeassistant/components/http/web_runner.py#L14).
I've now added a similar implementation here too.
@bdraco do you think we can add something like that to aiohttp? :pleading_face:


[HUDI-4813] Fix infer keygen not work in sparksql side issue

Change Logs
Fix KEY_GENERATOR_CLASS_NAME will be assigned default as ComplexKeyGenerator in org.apache.spark.sql.catalyst.catalog.HoodieCatalogTable#extraTableConfig
The org.apache.hudi.DataSourceOptionsHelper#inferKeyGenClazz is not used at all：
at org.apache.hudi.DataSourceWriteOptions#keyGeneraterInferFunc, because not set default value, will not be called；at org.apache.spark.sql.hudi.command.SqlKeyGenerator#getRealKeyGenClassName, also not be called, because beforeKeyGenClassName != null && beforeKeyGenClassName.nonEmpty is excluded from org.apache.spark.sql.hudi.command.SqlKeyGenerator#originKeyGen
Impact
SparkSql will also follow infer way when KEY_GENERATOR_CLASS_NAME not user defined.
Risk level: none | low | medium | high
none
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Hi @danny0405 , after my test, I find the inferFunc of https://github.com/apache/hudi/pull/5815 not be called, please correct me if I am not right.

I will check this ci failed issue local.

Seeems TestHoodieSparkSqlWriter.testToWriteWithoutParametersIncludedInHoodieTableConfig:868 » NoSuchElement not related to this pr.

Hi @TJX2014 , the PR overall looks good, but there are test failures, can you check and take a look ?

cc @TJX2014 , nice ping, you may need to check the CI failure.

@danny0405 Ok, I will check the CI right now.

@hudi-bot run azure

Still some test failure here, can you check again ~

A little hard to reproduce, please give me some time.

A little hard to reproduce, please give me some time.

No hurry, thanks for the patience.

@hudi-bot run azure

@hudi-bot run azure

@danny0405 Please help me review, seems ok in the last ci test.

The CI shows that TestSavepointsProcedure#Test Call rollback_to_savepoint Procedure is still failing.

TestSavepointsProcedure

Hi @danny0405  seems the failed ci record is triggered in two days ago, the failed test is：

Test Create Hoodie Table With Options *** FAILED ***
Expected "....apache.hudi.keygen.[Complex]KeyGenerator", but got "....apache.hudi.keygen.[Simple]KeyGenerator" (TestCreateTable.scala:139)
Total number of tests run: 186
Suites: completed 46, aborted 0
Tests: succeeded 185, failed 1, canceled 0, ignored 0, pending 0

Seems not the recent one, the failed one is what I have corrected yesterday.

Can you rebase with the latest master code and force push to re-trigger the tests.

Hi, @danny0405，https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=11379&view=results this is the latest right, seems ci will retain at least one failed record：
https://github.com/apache/hudi/commit/81546636356aa5b224b87087bad14a1da1a06a4d Azure: FAILURE
https://github.com/apache/hudi/commit/f32cf8515a4984c75bc26641abd3a5b042ebe372 Azure: PENDING
FYI : )

Hi @danny0405 ci passed : )


ci: add Debian:11 (Bullseye) build files

The release is imminent, it is easy to set up a build for it. In about a
year we should retire the Debian:9 (Stretch) builds as this is under LTS
support and will be EOL in June.

This change is

May want to update https://github.com/googleapis/google-cloud-cpp/blob/main/ci/generate-markdown/generate-packaging.sh too

May want to update https://github.com/googleapis/google-cloud-cpp/blob/main/ci/generate-markdown/generate-packaging.sh too

I was of two minds on this, but that nudge is enough. Done.  I also fixed some stale and ambiguous release names.


chore(deps): upgrade execa to 2.1.0

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #6542 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #6542   +/-   ##
=======================================
  Coverage   95.67%   95.67%
=======================================
  Files          79       79
  Lines        2708     2708
  Branches      699      699
=======================================
  Hits         2591     2591
  Misses        101      101
  Partials       16       16

Flag
Coverage Δ

#e2e
100% <ø> (ø)
:arrow_up:

#fixtures
50.59% <ø> (ø)
:arrow_up:

#unit
92.31% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Data science

Completed data_science exercise

Nice work on this!  I've added some additional comments throughout about best practices and style.  Take a look at the comments, refactor and resubmit when ready.  Feel free to talk to Zamir or myself if you have questions.


Add changelog for 1.16.0

This is WIP, as I'll be fixing it as new PRs come in. There are a few I want to land before we release, but this gets it started so we can finally get a release out.
Also modified the changelog generation, as I'd like to try and automate as much of this as possible, so we can get releases out more frequently than we have been.

2 notes:

Created a milestone for 1.16.0 for the last few things I'd like to land before we release.
I'm going to rebase these commits after I'm done, but I'll be pushing incrementally.

I wasn't aware of this before, but we should also adjust the .npmignore file because of #1781.

Rebased & ready for review.

Unrelated: I know what the first thing I will do after we publish 1.16.0.

Good call. While sorting, I also realized I had Java twice. I also love that we have both Markup & Markdown as languages 😆.


from PyQt5 import QtWidgets, uic import sys from PIL import Image from PyQt5.QtGui import QPixmap from PyQt5.QtWidgets import QFileDialog import cv2 from gray import rgb_to_gray import numpy as np import keras from keras.models import load_model from keras.preprocessing.image import load_img,img_to_array import matplotlib

from PyQt5 import QtWidgets, uic import sys from PIL import Image from PyQt5.QtGui import QPixmap from PyQt5.QtWidgets import QFileDialog import cv2 from gray import rgb_to_gray import numpy as np import keras from keras.models import load_model from keras.preprocessing.image import load_img,img_to_array import matplotlib

@Bouzian29
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!