Make it easier to round-trip custom tags.

At the moment, as far as I can tell it's pretty tricky to get<EMAIL_ADDRESS>to parse and stringify a document containing custom tags in general, but particularly to preserve the tags (e.g. for parsing, modifying / stitching the data, stringifying).
Describe the solution you'd like
Preferably something as simple (to use) as:
const source = `
a: !B c
d: !E [f, g]
`;
const value = yaml.parse(source, { preserveTags: true });
assert(value.a instanceof yaml.Tag);
assert(value.a.tag === "!B");
assert(value.a.value === "c");
assert(value.d.tag === "!E");
assert(value.d.value[0] === "f");
assert(value.d.value[1] === "g");

value.h = new yaml.Tag("!I", "j");

assert(yaml.stringify(value) === `\
a: !B c
d: !E [f, g]
h: !I j`);

Describe alternatives you've considered
So I did get this working with:
class YAMLTag { // note, *not* a yaml node.
  constructor(value) {
    this.value = value;
  }

  toString(...args) {
    return this.value.toString(...args);
  }
}

class AWSRef extends YAMLTag {
  static tag = "!Ref";
}

class AWSGetAttr extends YAMLTag {
  static tag = "!GetAttr";
}

...

const options = {
  customTags: [AWSRef, AWSGetAttr, ...].map((Tag) => ({
    tag: Tag.tag,
    resolve(value) {
      return new Tag(value);
    },
    identify(value) {
      return value instanceof Tag;
    },
  })),
};

But this is pretty messy, needs you to know about all the tags ahead of time, and I can't figure out how to get it to handle collections properly, let alone the ambiguous format AWS uses, where !GetAttr Foo.Bar is a shorthand for !GetAttr [Foo, Bar], so you have to support both formats.
Additional context
Seems like the latter issue might be related to #246 ?

Have you tried using YAML.parseDocument()? It's sort of designed for keeping this kind of data within the structure, while allowing the document contents to be manipulated.

That would make sense, but it means all the rest of the code had to be aware of yaml, and use methods rather than native object property manipulation. So I started with that then I ended up with a bunch of code based on that, then hit needing to support tags, and came up with a hacky solution that seemed to work in 2.0.0-3 by extending from Node from yaml/types.
None of this is your problem, of course!
I guess really what I'm asking for is a more flexible tags interface, that doesn't have to give a list of tags up front and can handle ambiguous scalar/collection tag content, so I could easily add this myself. I'm not sure what the constraints are here, though: in particular why scalar and collections are treated so separately.

Closing this, as it's not really clear what the actionable bit here would be. Happy to talk more about improving the custom tag API, but it might be good to open a new issue or discussion for that.


Close #141. Change about to goals until we have an about page

Sorry. Already fixed this. I really don't think things this tiny need pull-requests from the core web team like us... it just slows things down and makes project hard.

@bnvk if it would have been in a pull request, it would have been visible that it was already done. ;)

What if we did middle of the road where if it's a small thing you do a PR and merge it if it's minor?

That way people get notified of the fix, but you still get to move on.

@simonv3 that's what I mentioned at the Summit, am in favor. :)


Implement login mechanism to load settings from different devices

Issue Type: Feature Request
Hi
It would be great if we could login to vs code to be able to save all local settings and extensions in a smilar way as chrome browser does, so each time I'm in a different machine, I could login and get specially all extensions automatically installed but also local settings.
Thanks
Regards
VS Code version: Code 1.28.2 (7f3ce96ff4729c91352ae6def877e59c561f4850, 2018-10-17T00:23:51.859Z)
OS version: Windows_NT x64 10.0.17763

Oh, I realized this feature is already duplicated several times. sorry


I've also found that "pkg" is more stable when called like that
pkg update -f && pkg upgrade -y && pkg install -y ...
however, it might be times when I played with "snap" images.
should we leave "pkg" just as is ?

@chipitsine
BTW, could you consider making a new release before long?
The latest release v4.32-9731-beta of Stable Edition includes some fixes by Nobori-san submitted at new year's day. However, 5.01.9672 doesn't. At this point, stable edition is ahead of developer edition.

Regarding pkg update & pkg upgrade -y, it is plausible. I'll do some experiments.

I wanted to finish with PPP patch https://github.com/SoftEtherVPN/SoftEtherVPN/pull/1072 , memory leak https://github.com/SoftEtherVPN/SoftEtherVPN/issues/1069 and chocolatey stuff.
still did not find a spare time :(

Oops I removed openssl111 at 1e601ebf6595e8c663830c707c0fd0b96be77c46 but it re-appears unintended.

Rebased to current master and force-pushed.

but we can release anyway, you are right, January fixes are good.


[Finishes #146028835] Allowed users to duplicate services

@miq-bot add_label enhancement
@miq-bot add_label fine/no
@miq-bot add_label catalogs / dialogs
@miq-bot add_label  ux/review
See https://github.com/ManageIQ/manageiq-ui-service/issues/937

Here is a screenshot of what duplicating an existing service from our Order explorer looks like.  From a UX perspective it wasn't much of a change.

@serenamarie125 @Loicavenel for your review!

@chalettu Video is confusing, you are taking are clicking on Order but ending up in the Service, I don't see how you select a service in the order?

Updated Screenshot for reordering of a service.  @Loicavenel we tried to push the button further right but can't due to limitations of it being an expanded content row of the Order above it. There is a certain amount of padding in the patternfly component that we can't override.

@serenamarie125 here is the updated screenshot . For your review.

Cool @chalettu ... any chance that Reorder button can be right aligned on the list view ?

@serenamarie125 , that button is pushed as far right as it can get.  It is an issue with the patternfly listview and having listviews in the expanded contents.  We looked into it and its a matter of all the padding that gets added into the patternfly list view component

@serenamarie125 , please let me know if my answer from yesterday makes sense and if we can proceed.

@serenamarie125 does @chalettu make sense, or should I setup a meeting to chat about that?
/cc @Loicavenel

@serenamarie125 does @chalettu make sense, or should I setup a meeting to chat about that?
/cc @Loicavenel

@chalettu please rebase and fix conflicts.

@chriskacerguis , with greenkeeper updating as frequently as it is , I was going to hold off on rebasing frequently until we get the green light from UX on this PR.    It makes a lot of sense to only do one maybe two rounds of rebasing just to save time from having to do it daily until we get approval.

@chalettu understood on the placement of the button.  We can handle that later.
Couple questions - is the reorder flow embedded in a dialog ?  I can't tell.  Also can't see the buttons to actually add to the shopping cart, so curious how this works?

@serenamarie125 , The reorder flow is demonstrated in that screenshot.  We reused the same screen that was built for when you attempt to order a new service from the service catalog screen.  The difference is that we are filling the dialogs with the values that existed for the service you are attempting to duplicate.  The add to shopping cart button is at the top right of the dialog which hasn't changed from the existing screen we currently have for ordering a service.  It can be seen briefly in that screenshot demo in the top right part of the screen before we scroll down in the demo.   The flow for this process matches the exact flow for how a new service is ordered.

That's great about using the same screen, etc and filling the default values.  Exactly what we want!
@chalettu Sorry, it's tough for me to differentiate if that is a dialog or embedded in the page.  If in a dialog, the action buttons should be on the bottom right and consistent with the rest of the SUI dialogs.

@chalettu one question on my side, does it use the previous valued entered (what we want) or the reset to default? thanks

@serenamarie125 the screenshot is using our dialog component on a page.  In the context of your last comment, are you intending to say the difference between being a modal vs being part of the page?   This particular screenshot is part of a larger page and is not a modal.  That being said, are we fine with the button placement ?

@Loicavenel This uses the previous value entered for the service you are trying to duplicate

Champagne!!

@chalettu If it's not a modal, what does the Navigation look like? Can you increase the size of the image so we can see that, as well as show where the user is left after they click on "Add to Shopping Cart"

@serenamarie125 here is a bigger screen shot of the actual add to cart process.

@serenamarie125 , did you have any more thoughts on this PR after reviewing the last screenshot and comments I submitted?

@chalettu just to make sure i understand, the user goes to "My Orders", finds a request to Reorder, and then is moved to the Service Catalog area?  I think that's a bit funky, and that's why I was suggesting that it be put into a modal.  @Loicavenel what are your thoughts here, are you ok with the current flow?

@serenamarie125 moving to the service Catalog does not make any sense. the way it was implemented in the second animation is perfect for me.  You go to My oder, you find an order, you expend it, then you select the service you want to re-order... click re-oder, show the dialog right there with overlay..

@miq-bot remove_label ux/review

@miq-bot add_label ux/approved

@AllenBW had previous approved this (but, it was dismissed due to a UX change), so going to merge.

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1502307


clones graphml not stored

New release introduced an issue that graphml clones are not stored properly. Problem in calling dnapars from R script clonal_analysis.R

it was an executable problem, this is solved in my fork now and will be included in the next release.

fixed by #59


installation

Hi Richard,
This looks like a super useful package!! I have been trying to fit occupancy models with splines in Stan and Jags - but its slow.
However, I have a problem. I am trying to install the package on our HPC server (since I have lots of species) but it isnt working. I discussed with our IT guys and they said:
"The problem is, that for whatever reason, this R package is trying to install itself to where R is installed, not to your home directory. That's why you get the error about not having write permissions to the location where R is installed.
I even tried from a clean slate, i.e. I removed my ~/R directory. Normally, when using install_packages or install_github, R will ask you where to install the packages, and suggests ~/R if you don't have it already. That's not the case with this R package and that's why it's broken.
"
Any ideas of how to fix this?
Cheers,
Diana

This worked for me:
devtools::install_github("r-glennie/occuR", type = "source", lib = "directorywhereyouwanttoinstall")
We also have permission issues with the default R package library at our HPC (for any package, not just occuR), so we need to have a directory with the installed packages under our personal directory.

thanks @patchcervan  that did it!

Thanks @patchcervan for working this out!


feat(ci): automatically sync maintainers in the team

Closes #112

wait i didn't add an action oops

oh no GITHUB_TOKEN

I'll convert to a draft until @sgoudham updates the GitHub token, since he owns the secret.


Roles script of view with dll script in postgresql database

it would be great to be able to take roles script of view with view ddl script in postgresql database
Here is the some ss for pgadmin behavior and dbeaver behavior when we take create script of view

Hi @emretuna01 ,
seems that it is implemented in 7.1.5v.
You could check Show permissions checkbox in Generated SQL dialog

that's excellent work thanks for that


test

Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
test
Module Details
test #closethis
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

"#closethis"


fixed passport-oauth2 export error

Will be available on NPM as v2.0.1 (Forgot to add travis for auto-publish on v2.0.0). It'll be up later today since I need to get ahold of @kmariano to get me publish rights to that NPM package.

It's published now


"python -m pip install requirements.txt" results: "Could not find a version that satisfies the requirement requirements.txt" (IDFGH-1670)

Typo likely:
The master version of the readme.md:
Setup Build Environment
says:

Run python -m pip install requirements.txt to install Python dependencies

I get an error doing that,  it seams that pip is trying to install requirements.txt as opposed to reading the contents.
Adding the -r works for me: Run python -m pip install -r requirements.txt to install Python dependencies.

try to install this  https://dl.espressif.com/dl/esp-idf-tools-setup-2.3.exe


Implicit 2 classes names at same time

Looks link there is no Implicit 2 classes names at same time
for ex.
table.table,.table-hover
table(.table,.table-hover)
then tab
i think it suppose to give me html like this

but it works only in one class

sorry it is working i was adding ,
table.table.table-hover


Set empty routes for test on current store

The test it_returns_an_empty_array_when_no_routes_are_configured might fail if the current store is default instead of admin. Simply setting the testsettings on the current store instead of admin makes the test pass.

Thanks bjoern-tantau,
Apologies for the delay. We will review this pull request and your other one in the coming weeks.
Regards,
Colin


Update README.md

Added links to mpt as a library wrapped via SWIG based on GGML and Nethermind/Mpt-Instruct-DotNet-S C# wrapper effort

Hi, sorry this slipped through the cracks. We appreciate the contributions, but aren't going to add them to the webpage at this time.


Travis

Hi @matt-faber thanks for the contribution. Can you update the title and the description with some detail on what you are trying to achieve here?


Add a convenient gradle option to automatically set other, appropriate options for best-practice use in CI/CD pipelines

Basically, do one better than https://github.com/gradle/gradle/issues/17511 by providing a new command line option --ci that does whatever voodoo has to be done in order for gradle to perform best in a CI/CD environment.
Expected Behavior
Optimal default gradle performance in a CI/CD environment out of the box with a simple, convenient --ci gradle option argument.
Current Behavior
The gradle user has to dig, dig, dig to find the best incantation of gradle in a CI/CD pipeline, and they often fail at it, myself included.
Context
I've tried every combination of --no-daemon, --no-build-cache, --no-watch-fs & others to no avail in an effort to improve the performance of my build pipelines.
For example, on gitlab.com, It takes ~2+ minutes just to start the gradle daemon.  Even if I create a new, "prebuild" Docker image FROM gradle:jdk16 in which I perform a gradle testClasses on my volume-mounted src to cache dependencies, it has no effect on performance when I use the prebuild image on subsequent builds.

Not stale

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team, but this area of Gradle is currently not in focus. It might take a while before it gets implemented.

Blocked by

https://github.com/gradle/gradle/issues/17511


SAK-49975 Kernel prevent EntityManager from adding non loadable entities to unresolvable roots cache

https://sakaiproject.atlassian.net/browse/SAK-49975

Just reading the description of the issue - this sounds like a really important thing to fix :) ❤


D455 + Android Connection Issues

Required Info

Camera Model
D455

Firmware Version
<IP_ADDRESS>

Operating System & Version
Android 10

SDK Version
2.36.0

Segment
Tablets / Phones

Issue Description
The D455 is not recognized as a USB device on Android. We tested this on a Samsung Galaxy Tab S6 (which has a USB 3.1 compliant USB-C port).
The tablet seems to completely ignore the D455. No Android / system logs indicate its presence. For example when connecting a D415 we immediately get system logs indicating the presence of a new USB device. None of that happens with the D455. It's as if no device was connected to USB.
We have not tested this on any other Android devices so it may be an issue specific to the S6 tablet.

@rafaelspring We tested on Android 8.1 RK3399. D455 can be recognized and start streaming successfully. What's the output of "lsusb |grep 8086" in adb shell of your setup when you get the issue? Looking forward to your update. Thanks!

@radfordi Did you get chance to try "lsusb |grep 8086" in adb shell? Looking forward to your reply. Thanks!

@radfordi Any update from your side? Thanks!

@radfordi We've got your NDA ticket and resolved this. Close it accordingly. Thanks!


Deprecate ElementType

Due to legacy design, the ElementType is an enum with values that sort of, but not exactly, overlap with DataCategory. In practice, ElementTypes are only semantic:

In DualDatasets, we expect to have two image elements, but they need to have separate types to be distinguishable (e.g. ElementType == low_res or high_res in super-resolution). When we convert to PyTorch, if we don't do this, we'll have collisions (both low and high-res will be converted to the 'image' key)
In case of DisplayEngines, two element types with the same category can be displayed differently depending on context (e.g. source and target)

The go-to solution is to just use strings, but we're not fans of magic strings everywhere. Also, there's a need to synchronize between element types in datasets (e.g. 'image', 'class_label') and display engines, so we don't get 'image' vs 'img' or 'class_label' vs 'class' mismatches.

Closed with #12


Update & Re-Enable entrypoint.sh Readarr script

This is a continuation of the work @stavros-k did on https://github.com/elfhosted/containers/pull/594.
In addition to re-enabling the entrypoint.sh code it also required adding back the config.xml.tmpl file that was removed for the application in the Dockerfile:
COPY ./apps/lidarr/config.xml.tmpl /app/config.xml.tmpl

To fix the authentication errors from Readarr the config.xml.templ needed to be updated for AuthenticationMethod and AuthenticationRequired; External and DisabledForLocalAddresses, respectively.  This matches the config template for Lidarr as well.  I also set the UpdateMechanism to Docker to remove a log entry about doing the same during startup.
The xmlstarlet  package also needed to be added to the Docker image to support the entrypoint code replacing the config.xml.
I have tested this locally for both a new installation creating a new config.xml and using an existing config directory and, after dealing with line ending issues, both worked ok.
Command:
docker run -it --rm -e "READARR__PORT=30045" -e "READARR__INSTANCE_NAME=readarr-2" -p 30045:30045 -v ${PWD}:/config readarr-local

Log:
...
[Info] Microsoft.Hosting.Lifetime: Now listening on: http://[::]:30045
[Info] QualityProfileService: Setting up default quality profiles
[Info] MetadataProfileService: Setting up standard metadata profile
[Info] MetadataProfileService: Setting up empty metadata profile
[Info] Microsoft.Hosting.Lifetime: Application started. Press Ctrl+C to shut down.
[Info] Microsoft.Hosting.Lifetime: Hosting environment: Production
[Info] Microsoft.Hosting.Lifetime: Content root path: /config
[Info] ManagedHttpDispatcher: IPv4 is available: True, IPv6 will be disabled

Generated Config:
<Config>
  <LogLevel>info</LogLevel>
  <UrlBase></UrlBase>
  <Branch>develop</Branch>
  <ApiKey></ApiKey>
  <AuthenticationMethod>External</AuthenticationMethod>
  <AuthenticationRequired>DisabledForLocalAddresses</AuthenticationRequired>
  <InstanceName>readarr-2</InstanceName>
  <PostgresUser></PostgresUser>
  <PostgresPassword></PostgresPassword>
  <PostgresPort>5432</PostgresPort>
  <PostgresHost></PostgresHost>
  <PostgresMainDb></PostgresMainDb>
  <PostgresLogDb></PostgresLogDb>
  <PostgresCacheDb></PostgresCacheDb>
  <Port>30045</Port>
  <BindAddress>*</BindAddress>
  <LaunchBrowser>False</LaunchBrowser>
  <EnableSsl>False</EnableSsl>
  <SslPort>6969</SslPort>
  <SslCertPath></SslCertPath>
  <SslCertPassword></SslCertPassword>
  <UpdateMechanism>Docker</UpdateMechanism>
  <AnalyticsEnabled>False</AnalyticsEnabled>
</Config>

Thank you for considering these changes!

I think we can remove the whole script!
https://github.com/Readarr/Readarr/pull/3482
Check this out!

You always find the best ways of doing things! ❤  I thought it was pretty silly how this was modifying the config file.  I'm always hesitant to make more drastic/breaking changes on projects I don't run though.
I'll let @elfhosted decide to close but agree the ENV is the better way forward 👍


playground, phosphorjs discontinued

https://github.com/phosphorjs/phosphor
https://github.com/eclipse-theia/theia/issues/6501
https://golden-layout.com/
https://github.com/jupyterlab/jupyterlab/issues/8092
https://github.com/jupyterlab/lumino

looks like golden-layout is the way to go, but last release is 3 years ago, however looks like there is some traction to get a new release out

closing, going towards https://github.com/abaplint/abaplint/issues/828 insetad


Fix #3566: Improve LinePreservingCodeReplacer.ReplaceCode

This also fixes https://github.com/Microsoft/PTVS/issues/3662 (at least it correctly works with repro sample), but I'm not sure if we should merge this into 15.6 or into master.

Looks like this is only used by FormatCode, so worse case is we regress some more in that area, best case some broken formatting now works as expected and doesn't delete user code.
You should run CodeFormatterTests.* and add new ones for the code snippets in #3662. Looking at the tests, it's hard for me to feel confident even if they pass.

Thanks for the hint on tests, I've found one issue (result of copy-paste).
I've tried it on test_descr.py. I haven't noticed issues in the algorithm itself, though I've noticed that we change code in comments. Not sure if it is the right thing to do.

I think there's a rule to split semicolon separated statements, and one to wrap long lines. But I think they’re fine if the formatted code has new lines, it’s if there’s a race (for example, if inserting a snippet triggered format).

Merging with Alex's permission.


Intellij  does not succeed JAVA_HOME

Hello, I need some help!
I install sdkman successfully and install Java and gradle with sdkman!
java -version
openjdk version "1.8.0_131"
OpenJDK Runtime Environment (Zulu <IP_ADDRESS>-linux64) (build 1.8.0_131-b11)
OpenJDK 64-Bit Server VM (Zulu <IP_ADDRESS>-linux64) (build 25.131-b11, mixed mode)

gradle --version

------------------------------------------------------------
Gradle 3.5
------------------------------------------------------------

Build time:   2017-04-10 13:37:25 UTC
Revision:     b762622a185d59ce0cfc9cbc6ab5dd22469e18a6

Groovy:       2.4.10
Ant:          Apache Ant(TM) version 1.9.6 compiled on June 29 2015
JVM:          1.8.0_131 (Azul Systems, Inc. 25.131-b11)
OS:           Linux 4.9.0-2-amd64 amd64

all good!
now I download IntelliJ ce
I decompress the file on /opt/Idea and run bin/idea.sh all fine but when run intellij from the icon menu i have this

modified my .bashrc
#THIS MUST BE AT THE END OF THE FILE FOR SDKMAN TO WORK!!!
export SDKMAN_DIR="/home/juanib/.sdkman"
[[ -s "/home/juanib/.sdkman/bin/sdkman-init.sh" ]] && source "/home/juanib/.sdkman/bin/sdkman-init.sh"

export JAVA_HOME="/home/juanib/.sdkman/candidates/java/current"
export GRADLE_HOME="/home/juanib/.sdkman/candidates/gradle/current"

close the sesion and login but not found!
some tips!

Not sure why this is happening, it could be because the JDK you installed is not an Oracle JDK. Please do not add these export statements to the end of your .bashrc, as these are generated dynamically by SDKMAN. Not the right above #THIS MUST BE AT THE END OF THE FILE FOR SDKMAN TO WORK!!!.
In future, please consider trying our Gitter user-issues channel before opening an Issue here.

Same here, but still no love.

Did you find the solution?

This is because sdkman gets sourced in the shell and you are starting
IntelliJ outside of the shell. Try running IntelliJ from the shell using
the idea.sh and IntelliJ will pick it up.
On Sun, 14 Oct 2018 at 06:51, stevo<EMAIL_ADDRESS>wrote:

I have the same problem.
IntelliJ IDEA 2018.2.4
[image: screen shot 2018-10-13 at 10 40 04 pm]
https://user-images.githubusercontent.com/2980220/46913019-009ad500-cf3a-11e8-8323-2169205d7d85.png
here my .bash_profile
THIS MUST BE AT THE END OF THE FILE FOR SDKMAN TO WORK!!!
export SDKMAN_DIR="/Users/stefan/.sdkman"
[[ -s "/Users/stefan/.sdkman/bin/sdkman-init.sh" ]] && source "/Users/stefan/.sdkman/bin/sdkman-init.sh"
export JAVA_HOME="~/.sdkman/candidates/java/current"
$ java -version
openjdk version "11" 2018-09-25
OpenJDK Runtime Environment 18.9 (build 11+28)
OpenJDK 64-Bit Server VM 18.9 (build 11+28, mixed mode)
Did someone figure it out?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/sdkman/sdkman-cli/issues/584#issuecomment-429598835,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI0Buny6sh2J2pus1e1YjoUgFUyTVRCks5uktD1gaJpZM4NqwUf
.

--
Marco Vermeulen
+44 7757 510 608

@marc0der did it exactly that way, unfortunately I have exactly the same problem: IntelliJ Idea does not show any SDK. A pitty :(

Right, we are talking about two different things here. The first is that
intellij does not pick up Java and can thus not start up.
The second is that IDEA does not pick up installed JDKs for use in your
projects. The solution for this is to simply add it manually. All your JDKs
are stored under ~/.sdkman/candidates/java.
We have an open ticket on IDEA's youtrack for this very issue so please
vote for it:
https://youtrack.jetbrains.com/v2/issue/IDEA-193914
On Sat, 20 Oct 2018, 02:44 Kannan Thambiah<EMAIL_ADDRESS>wrote:

@marc0der https://github.com/marc0der did it exactly that way,
unfortunately I have exactly the same problem: IntelliJ Idea does not show
any SDK. A pitty :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sdkman/sdkman-cli/issues/584#issuecomment-431538859,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI0BiF1QVCMpg08ClChbFTMPH1q6_S3ks5umoAYgaJpZM4NqwUf
.

I know this is closed, but you can set JAVA_HOME globally on MAC OS with launchctl
launchctl setenv JAVA_HOME $(/usr/libexec/java_home -v 1.8)

or
launchctl setenv JAVA_HOME ~/.sdkman/candidates/java/current

Not the best practice, but fix this error globally on any Linux
Please ensure JAVA_HOME points to JDK rather than JRE.
Put something like this:
JAVA_HOME=/home/agilob/.sdkman/candidates/java/current

to /etc/environment just change your username

It's definitely not good practice to place anything pointing to your home directory in /etc/environment or /etc/profile. Please do not do this.

If you guys still struggling with this, maybe one solution (the one that worked with me) is creating a symbolic link to some visible folder from the .sdkman/candidates/java/current/

For those still looking to fix this / those arriving from search engines. The steps below worked for me.
First create a symlink to the ~/.sdkman folder.
ln -s ~/.sdkman ~/sdkman
Your desired JDK version can be selected afterwards by navigating to the output of the command below in the IntelliJ UI
echo "/Users/$(whoami)/.sdkman/candidates/"


Use Nuxt's DevOnly Component

Instead of checking manually if we're in dev environment or not, we can use Nuxt's <DevOnly> component that only renders a component in dev environment.

Apparently this only works when you run the app with nuxt dev, while we run our development deployments in production mode with nuxt build and node .output/server/index.mjs so wrapping the component with DevOnly won't work.


broken pypi requirements

[Enter feedback here]
Hello. There is problem with broken pypi reqiirements in the intruction:
ERROR: cookiecutter 1.7.2 has requirement requests>=2.23.0, but you'll have requests 2.22.0 which is incompatible.
But after upgrade requests package we take
ERROR: botframework-connector 4.9.1 has requirement requests==2.22.0, but you'll have requests 2.23.0 which is incompatible
So you should upgrate botframework-connector requirements or freeze cookiecutter==1.7.0
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 30a20df2-727c-1afd-2b87-836ea1ea4f7a
Version Independent ID: 157fc262-fb1f-f12d-0890-e90119feb444
Content: Create a bot using Bot Framework SDK for Python - Bot Service
Content Source: articles/python/bot-builder-python-quickstart.md
Service: bot-service
GitHub Login: @emgrol
Microsoft Alias: kamrani

Thanks for pointing this out @burguyd. The file has been updated with the correct package version in PR 2258 and the changes should go live soon.
Related issue: https://github.com/microsoft/botbuilder-python/issues/1070

been updated with th

Ok, thank you


Version update 0.3.0

Modulus Pull Request
Description

Updating versions inside package
Checklist

[ ] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is
up to date with these changes.

Dependencies

/blossom-ci

/blossom-ci


Error When Compiling

I get that error when compiling.
By the way, solid work...kudos

I'm not familiar with this error. but I coded this RAT using VS 2019. maybe you should update your VS.

Yeah that's 2019....the latest update

I'll try a fix...if it's done, I'll update here for others.

I simply closed VS, deleted the ".vs" folder and re-opened the solution and re-compiled.  Just like that........
I can't say what caused the earlier error but it's working like a charm now!


KafkaBindingProperties has no documentation (so no popup help in IDE)

Also related nested objects: KafkaConsumerProperties and KafkaProducerProperties.

@dsyer KafkaBindingProperties is a pojo used in a map from KafkaExtendddBindingProperties here. Per @snicoll map of pojo won't get metadata support in IDE, but there is an issue to support that in boot. It lists the available properties, but no pop-up docs help. Or are you referring to something else here? Thanks!

I see them in the IDE with "no description". That suggests to me that the IDE would happily add a description if there was one. Whether the AP can generate them is another question (but I don't really see why not).

There never was documentation for Maps of POJO, see https://github.com/spring-projects/spring-boot/issues/9945

Maybe the IDE would display Javadocs if they were available in the POJO (that's what the popup in Eclipse suggests to me at least)? The properties need to be documented IMO anyway, so it's worth doing. The only question is whether to do it in a format acceptable to the AP, in case one day it decides to support this as well.

OK, so I tested it. Eclipse (STS) will show the Javadocs from the getter method (not the field) in a POJO. I think we should put some docs in there. And maybe when the AP catches up it can look there as well.

@dsyer Is it only for the POJO's inside the map that Javadocs need to be added on the getter methods and not on the fields? We usually add Javadocs on the fields only.

Yes. Fields is what the AP supports. But the AP does not support mapped POJOs currently.


chore: update Graphback to 0.16

Opening as draft as we are waiting for a final stable 0.16 release. But 0.16.0-alpha2 should do it.

Merging


provide refactoring to convert full module name to relative module name in import

provide a refactoring for users to move between the relative path and absolute path in imports.
some users prefer using a relative path and some users like absolute paths in the import statement.
let users to easily convert between two to improve productivity.

this is related to "add import" feature - https://github.com/microsoft/python-language-server/pull/1656


STM32G4 support

There is currently no zephyr support for the STM32G4 family
It would be great if there was support for these. Any sort of timeline for support would be greatly appreciated

@rosterloh, there is currently no plan to add this series in Zephyr, but for sure it would be welcome!
Help can be provided to any effort to do this porting work.

Ok, I've started work on it. Submitted a PR for the hal and working on the dts

Here is the PR, if others are looking for it aswell: https://github.com/zephyrproject-rtos/zephyr/pull/18806

Closing as G4 is supported now. Thanks!


Update dependabot.yml

Pull Request Test Coverage Report for Build c37abe321bf89b52248e39ded8b85e289ecaa5df-PR-111

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.508%

Totals

Change from base Build ec6705486af0f1c88dea8862bebbacd3421d7160:
0.0%

Covered Lines:
285

Relevant Lines:
354

💛  - Coveralls


[HOLD] Add Onfido SDKs + Enable Payments flow

Details
Still Holding on
Auth to deploy:

[x]  https://github.com/Expensify/Auth/pull/5520

Web-Secure changes to be merged/deployed:

[ ] https://github.com/Expensify/Web-Secure/pull/1944

Fixed Issues
Fixes https://github.com/Expensify/Expensify/issues/161733
Tests

Follow the setup instructions in https://github.com/Expensify/Web-Secure/pull/1944 to get a "pending" wallet

Log into E.cash

Verify you can access the /enable-payments via url or deep link + that the userWallet appears in localStorage and that you are able to see the Onfido screen after a hopefully brief loading spinner. The wallet itself should have a SILVER status.

Complete the Onfido flow (without using any of your own ID if you can avoid it - Onfido recommends to not use real info) then verify you are brought to the next screen once the flow is completed.

Fill out additional details and try to break stuff. Errors should appear on the fields that have issues.

Before submitting we must mock IDology so it succeeds by altering the line mentioned here

Next we will should be brought to the Terms placeholder screen (Note: Terms will be added in a later step)

Check the boxes and submit

Verify you are brought to a screen that shows the status of your wallet.

QA Steps
❌  - This is not yet QA able but probably will have to be production QA and tested in connection with the Web-Secure PR
Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Hey, I noticed you changed some webpack configuration files. This can break production builds. Did you remember to run a production build locally to verify they still work?

Taking this of draft but it's still on HOLD for Web-Secure

@nickmurray47 @MariaHCD @stitesExpensify - adding y'all as reviewers on this PR as the Web-Secure PR can't be tested on it's own.
Let me know if there are any questions 1:1 about how to test or anything else 🙇♂

updated to resolve merge conflict

The web-secure PR has been merged so I removed the hold here. @MariaHCD @stitesExpensify @nickmurray47 Is there anyway you can start reviewing this? I'm taking over for Marc while he is OOO. I think everything is working. There was a mention that mobile-web might not be working, but it seems to work for me, with the exception of about 70 pixels being cut off the bottom of the Onfido UI. I don't think mobile-web is an important flow to launch with and I couldn't find anyway to fix this except for removing the modal header (which also prevents the user from being able to cancel out of the Onfido UI).

The last time I tested both flows on web and iOS they both tested well for me! Will try to get a quick turn-around on the review.

FYI, I just fixed the problem of the Onfido window being cut off in mobile-web

OK, I've updated the PR to address Rory's comments (thank you!).
I need to go through testing again this afternoon as I haven't tested after any of the recent changes. @roryabraham Would you also be able to test this out a little? If not, that's cool... it's a doozy to test. I have to jump into a meeting now for a bit though.

OK, resolving conflicts again. This should be ready to merge as long as the tests pass

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging in version: 1.0.63-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production in version: 1.0.64-0🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


add *.hpp.in to the file.associations in vscode out of the box

*.hpp.in is a very common extension for C++ file to be processed by cmake. also *.h.in. It would be nice to add it to vscode so it can highlight it in c++ out of the box.
Thank you.

Pointer for starting: https://github.com/Microsoft/vscode/blob/master/extensions/cpp/package.json#L21-L22

@alexr00 , I will push this change :)


Update TaskRun status docs

Changes
When a TaskRun is cancelled on purpose, the completionTime will be set, this pr update the docs to reflect that. The current documentation states that the taskRun will not get a CompletionTime, which I think is not correct.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

hi @qu1queee thanks for opening the PR, could you sign the CLA?

/ok-to-test
/kind documentation

@dibyom done


Bug: nmcbit statistics

BH is reading the nmc shares instead of the btc shares from the stats page

Simplecoin doing this as well.


Add project to list of projects that uses go-prompt.

Suggestions
Hi, my cf-shell project uses go-prompt. Can you add it to the list of projects that uses go-prompt?
https://github.com/zhusulai/cf-shell
Thanks.

It looks your repository is forked. Should I add https://github.com/dgruber/cf-shell ?


Update to value.yaml and deployment.yaml with breaking changes for provider deployment

Update deployment.yaml with breaking changes
If you are using the latest Virtual Kubelet image you get the following notification in the logs:
Flag --taint has been deprecated, Taint key should now be configured using the VK_TAINT_KEY environment variable
So, the changes introduced are breaking changes, because the node taint has to be modified and some other spaces like the Virtual Kubelet port.
Furthermore a nodeSelector was added to make sure that the IoT Edge Provider will land on a Linux node in Kubernetes cluster with multiple agent pools having different node OS types.
Update value.yaml with breaking changes
So, the value.yaml is reflecting the changes that are made in the deployment.yaml.
rbac.install is switched now from false to true, because AKS cluster are deployed per default with Kubernetes RBAC enabled.
vk.image.tag is switched to latest from 0.3 to reflect the changes that are made to the deployment.yaml
Added the taint section.

I will update the example .yaml files for container deployment in the next days.

Updated the README.md and the sample-deployment.yaml and sample-pod.yaml to reflect the initial proposed changes the IoT Edge Provider deployments.


Start-AzureRmNetworkWatcherResourceTroubleshooting Error: This gateway does not belong to the specified subscription

I am trying run a Runbook for monitoring the status of a VPN connection. But it failed with the following error message:
Start-AzureRmNetworkWatcherResourceTroubleshooting : This gateway does not belong to the specified subscription
StatusCode: 400
ReasonPhrase: Bad Request

The powershell run book is based on the following Microsoft article see here
I have also tried it using Network Watcher VPN Troubleshoot from Azure portal and the result is the same error message.
I found this error report link but I'm not sure if it is the same problem that I have experienced.
I hope you can help me to find out the problem.

I'm having the same error performing this today in the portal on a subscription where this was working for someone else yesterday. Be curious to see what the fix is

@rsedano , @rsedano  thanks for your comments. We are actively investigating and will get back to you shortly.
@jimdial twe have 2 reports of breaking change to the runbook referenced in this article
@georgewallace  - Fyi

Facing the same problem.

As this is causing issues in the portal as well, it is safe to say this is not an Azure Automation issue. I have reached out to the product group to see if we have an answer,

#in-process

We have identified the issue reported below, made the appropriate repairs, and the fix has been fully deployed.
Please let us know if you are still experiencing this issue.

#please-close

@rsedano We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

We have identified the issue reported below, made the appropriate repairs, and the fix has been fully deployed.
Please let us know if you are still experiencing this issue.

Working like a charm now. Thank you very much!


Browserless use of IITC?

I'm trying to see if it's possible to integrate parts of iitc for use with a slack integration.  I'm a developer, but new to web stuff.  I'm not even sure how to go about doing that, would the slack integration need its own ingress account, then you mimic a login via javascript and call iitc functions from there?
Could you just call the scanner API without logging into the intel map?
Mainly, I'm just trying to see if there's an easy way to check the status of portals located at various common farm areas, although there could be lots of fun other uses.
Thanks

Sounds like everything you are proposing are strictly against the Ingress Terms of Service.

The IITC project is strictly a replacement user interface within the web browser environment.


pulse: when default device changes update the info

The problem was that after a default device change the cubeb_get_max_channel_count() returned the wrong number of channels; it returned the device info (including channel count) of the initial default device. In order to update the default device info dynamically, a subscription to the  event is created during context initialization and the device info is updated accordingly.

I assume we'll need to make the same fix to cubeb-pulse-rs too.

Sorry, I misunderstood the patch.  You've already handled the situation I'm concerned about by including PA_SUBSCRIPTION_MASK_SERVER when registering via pulse_register_device_collection_changed and never unregistering the callback.

Thank you, I am working on the rust side.


How to use plugins in callback [ios]

Hello, can you please add the documentation on how to use the third-party plugins inside the callback function for IOS?

Sadly I don't know a way to do that, it fix a lot of problem if we could find a way to do that.

@ozz-rjq Do you mean to use for example local notification plugin in the callback? Or a plugin for making server requests?

@dacianf yes, plugins like local notifications or audioplayers

@ozz-rjq Have you tried to do it? What errors gave you? I'm asking these because in my current project I use two plugins in the callback function, local notification and shared preferences and they work perfect.

@dacianf How did you register your plugins on AppDelegate?, because I also want to use shared preferences but the app crashes each time I run the service.

@SandeMK what errors have it shown? Did you gave permissions to your ios app to run in background (inside signing & capabilities tab in xcode)? Can you post here the errors?

@dacianf yes I did give permissions just as shown on that pic, I think what caused the crashing was the plugin registrant callback method. I changed it to
`if (!registry.hasPlugin("BackgroundLocatorPlugin")) {
GeneratedPluginRegistrant.register(with: registry)
}'
The app doesn't crush now, but I get location update only once, and that's when I'm starting the service. Is this the case in IOS when you are stationary?(I'm debugging on a simulator), does IOS update only when the device is moving?
because with the same code, Android does update.

I just updated the wiki page for using other plugins in iOS. Please check it out Wiki

I'm having a similar problem. I'm using this plugin and a geofencing plugin found here. In the location callback function I create geofences based on the user's location. This works fine on Android, but on iOS calling any methods from the geofencing plugin in the location callback has no effect.
Below is my AppDelegate.swift. Any help would be much appreciated!

func registerPlugins(registry: FlutterPluginRegistry) -> () {
  if (!registry.hasPlugin("BackgroundLocatorPlugin")) {
    BackgroundLocatorPlugin.register(with: registry.registrar(forPlugin: "BackgroundLocatorPlugin"))
  }
  if (!registry.hasPlugin("GeofencingPlugin")) {
    GeofencingPlugin.register(with: registry.registrar(forPlugin: "GeofencingPlugin"))
  }
  if (!registry.hasPlugin("FlutterLocalNotificationsPlugin")) {
    FlutterLocalNotificationsPlugin.register(with: registry.registrar(forPlugin: "FlutterLocalNotificationsPlugin"))
  }
  if (!registry.hasPlugin("FLTSharedPreferencesPlugin")) {
    FLTSharedPreferencesPlugin.register(with: registry.registrar(forPlugin: "FLTSharedPreferencesPlugin"))
  }
  if (!registry.hasPlugin("FlutterSecureStoragePlugin")) {
    FlutterSecureStoragePlugin.register(with: registry.registrar(forPlugin: "FlutterSecureStoragePlugin"))
  }
}

@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
  override func application(
    _ application: UIApplication,
    didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
  ) -> Bool {
    GMSServices.provideAPIKey("$GOOGLE_MAPS_API_KEY")

    GeneratedPluginRegistrant.register(with: self)

    GeofencingPlugin.setPluginRegistrantCallback(registerPlugins)
    BackgroundLocatorPlugin.setPluginRegistrantCallback(registerPlugins)

    // iOS notification setup
    if #available(iOS 10.0, *) {
      UNUserNotificationCenter.current().delegate = self as? UNUserNotificationCenterDelegate
    }
    // Prevent "old" notifications showing up after reinstallation
    if(!UserDefaults.standard.bool(forKey: "Notification")) {
      UIApplication.shared.cancelAllLocalNotifications()
      UserDefaults.standard.set(true, forKey: "Notification")
    }
    return super.application(application, didFinishLaunchingWithOptions: launchOptions)
  }
}

@ozz-rjq Have you tried to do it? What errors gave you? I'm asking these because in my current project I use two plugins in the callback function, local notification and shared preferences and they work perfect.

Can You please provide code on how You actually use "flutter_local_notifications" plugin in callback?
I am having problems with this plugin in callback (in the "callback" function inside "locations_services_repository.dart"), and only in iOS. It never gets executed and no error is thrown. Otherwise this plugin is working normally anywhere in the app and on Android it is working also in callback.
Also, for example, "flutter_ringtone_player" works for me in callback without registering it in AppDelegate.swift
If I try and register plugin as shown in an example in wiki page, then following error is thrown.
error: cannot find 'FlutterLocalNotificationsPlugin' in scope
FlutterLocalNotificationsPlugin.register(with: registrar(forPlugin: "com.dexterous.flutterlocalnotifications"))

@ozz-rjq Have you tried to do it? What errors gave you? I'm asking these because in my current project I use two plugins in the callback function, local notification and shared preferences and they work perfect.

Can You please provide code on how You actually use "flutter_local_notifications" plugin in callback?
I am having problems with this plugin in callback (in the "callback" function inside "locations_services_repository.dart"), and only in iOS. It never gets executed and no error is thrown. Otherwise this plugin is working normally anywhere in the app and on Android it is working also in callback.
Also, for example, "flutter_ringtone_player" works for me in callback without registering it in AppDelegate.swift
If I try and register plugin as shown in an example in wiki page, then following error is thrown.
error: cannot find 'FlutterLocalNotificationsPlugin' in scope
FlutterLocalNotificationsPlugin.register(with: registrar(forPlugin: "com.dexterous.flutterlocalnotifications"))

@dacianf Got it working.
There is a difference in code on wiki page https://github.com/rekab-app/background_locator/wiki/Use-other-plugins-in-callback and in the AppDelegate.swift code on line 30: https://github.com/rekab-app/background_locator/blob/239a850832766be20036552e81ccd956b1afbbaa/example/ios/Runner/AppDelegate.swift#L30
Adding "as! FlutterPluginRegistrar" resolved my issue and I can use "flutter_local_notifications" plugin both in background mode and when app is terminated.
I guess this should be fixed in documentation on wiki page so it matches the code in example.

@dacianf Got it working.
There is a difference in code on wiki page https://github.com/rekab-app/background_locator/wiki/Use-other-plugins-in-callback and in the AppDelegate.swift code on line 30: https://github.com/rekab-app/background_locator/blob/239a850832766be20036552e81ccd956b1afbbaa/example/ios/Runner/AppDelegate.swift#L30
Adding "as! FlutterPluginRegistrar" resolved my issue and I can use "flutter_local_notifications" plugin both in background mode and when app is terminated.
I guess this should be fixed in documentation on wiki page so it matches the code in example.


[Bug] Seashelf crafting recipe not working

Hi, Me and a friend made a custom modpack using Apotheosis called Eldritch Machinizations 2 and I have just discovered the seashelf crafting recipe is broken. The mod version we use is: apotheosis-0.2.81.jar. We run this on the Fabric API if that makes any difference.

There is no fabric version of apotheosis.


TestingKPconvAutomatization

Student, group: Chervak Anton, 3530901/90202
URL:  https://github.com/Houston613/TestingKPconvAutomatization
Git approach: git-flow

`+


TNT Not Falling Bug

Hello! I have tried the first version but when you stand on edges, TNT does not fall, so I was wondering if the newest version have already fixed that bug?

No, that bug still exists. I can also not find a proper solution for it.. When i remove more blocks, players suddenly start falling for no reason (block under them disappears), when i change it to the blocks the AABB of the player collides with, the same also happens.. Imagine running and the block you are jumping to disappears..

So is there no way to fix this problem yet? If so that's just sad :(

It works alot better now, you can not stay in the same spot for some seconds and not on corners

thx!


[bug] Grist self-hosted is not compatible with SSL

Hi all and happy New Year!! Sorry to begin 2022 with a bug report ;o))

I have taken advantage of the holidays to switch all our self-hosted applications to SSL, using Nginx Proxy Manager and the free Cloudflare infrastructure.
Everything worked just fine for most of our apps except for Grist.
I am using the very last version of Grist Core (continuously updated with Watchtower)
When accessing Grist from the newly created https address I arrive an error page:

If I click Go to man page, nothing happens
If I select Sign in from the upper right menu, I get this error:

And the browser Console log says: Failed to load resource: the server responded with a status of 502 ()
Please help as I would like to allow other employees to access our main worksheets that have all been transferred to Grist.
PS. I can send you logs on your private email address if needed.

Hi @asitemade4u, happy new year!
One thing you could try is to set environment variables APP_DOC_URL and APP_HOME_URL to whatever way your site will be accessed by the user. If your site is now at https://example.com, you would set those two variables to that (including the https).
502 is a bad gateway error. Could you check what request the browser was making? It may be in the console just before the error, or failing that in the network tab (once you reload the page).

Hi Pau, and thanks for the prompt response.

So I changed the docker-compose.yml file as recommended and it worked: I could connect to the main page (with the documents) and Sign in to my Grist account
But, when I try to open a document I get a blank page and an error message:

BUT, asyou may know, there is a setting in Nginx Proxy Manager that allows for Web Socket support.
I checked it and I now can access the documents.
Case closed! Thanks!

Even if everything seems fine, I have left the console opened and I see errors each time I open an existing document.
As the log includes private info, i will send it to your private email address.

Great, glad it worked and you figured out the websocket setting. Will watch my mailbox for the remaining issue.

@asitemade4u I think the error you are seeing is a failure to load browser-check.js, a small utility to check for unsupported browsers and give a little warning in them. I wouldn't expect its absence to cause trouble. I'll look into it, thanks for reporting.

@asitemade4u browser-check.js should be loading well in latest docker version. Do you have any remaining problems in using your own domain name?

Hi Paul,
No problem except for the remaining (and quite stressful) issue with losing the path to documents at each update (I use Watchtower and you issue a lot of updates).
Actually, the (small) issue has gone worst since I have my own domain and certificate: instead of having to Sign in here is how it goes:

I arrive on an error page (Something went wrong)
I sign in and arrive on a page where my documents are not displayed (this is the stressful part)
I sign in again and now view the documents

@asitemade4u I think this is a symptom of losing session information. You can control where session information is stored with the environment variable GRIST_INST_DIR. I think setting it to GRIST_INST_DIR=/persist may work in your case, if /persist is a directory mapped onto the host. With that, after you log in, the log in should survive across docker restarts. You'll see an extra grist-sessions.db file. It is also possible to direct Grist to store session information in an external redis instance by setting REDIS_URL.

OK, will try and let you know

Hi Paul,
It did not work, at least with Cloudflare + Nginx Proxy Manager.
So:

I changed the docker-compose.yml file according to your recommendations and added - GRIST_INST_DIR=/home/stsinc/dkr/gst/aut to the environment: section
I unlocked the "cloud" icon on Cloudflare's DNS line (I "grey-clouded" it)
I emptied the cache of my browser

I get a Bad Gateway error

@asitemade4u is there a volumes section in your docker-compose.yml for the directory /home/stsinc/dkr/gst/aut? If not, you'll need one so the container has access to it. And make sure to use the mapped directory name in the environment variable. For example, when your docker-compose.yml was as you had it in https://github.com/gristlabs/grist-core/issues/90, you'd need GRIST_INST_DIR=/persist.

@asitemade4u in latest image GRIST_INST_DIR is now set by default to /persist.

GRIST_INST_DIR is now set by default to /persist in latest docker image, meaning session information will survive container upgrades.

@asitemade4u is there a volumes section in your docker-compose.yml for the directory /home/stsinc/dkr/gst/aut? If not, you'll need one so the container has access to it. And make sure to use the mapped directory name in the environment variable. For example, when your docker-compose.yml was as you had it in #90, you'd need GRIST_INST_DIR=/persist.

I'm just getting started, trying to self-host grist and am wondering if there is a complete list/documentation of all env variables somewhere?
I saw quite a few mentioned here and there in different issues but no comprehensive list.

@ovizii good point. We have an internal list for our hosted service, here's a quick stab at pulling out the relevant parts.
Grist environment variables
Grist is sensitive to several environment variables.  Here is a list:

Variable
Purpose

APP_UNTRUSTED_URL
URL to serve plugin content from, or where to expect to find it

GRIST_APP_ROOT
directory containing Grist code, including subdirectories such as sandbox, static, and bower_components.

GRIST_DATA_DIR
directory to search for document list.

GRIST_DOC_BUCKET
s3 bucket where Grist documents are stored, for hosted Grist.

GRIST_DOMAIN
in hosted Grist, Grist is served from subdomains of this domain.  Defaults to "getgrist.com".

GRIST_HOST
hostname to use when listening on a port (not respected for electron).

GRIST_INST_DIR
path to Grist instance configuration files, for Grist server.

GRIST_LOGIN_REDIRECT_HOST
host of cognito-based login helper, if applicable (usually login.getgrist.com).

GRIST_MAX_UPLOAD_ATTACHMENT_MB
Max allowed size for attachments (0 or empty for unlimited).

GRIST_MAX_UPLOAD_IMPORT_MB
Max allowed size for imports (except .grist files) (0 or empty for unlimited).

GRIST_SESSION_COOKIE
if set, overrides the name of grist's cookie

GRIST_SESSION_DOMAIN
if set, associates the cookie with the given domain - otherwise defaults to GRIST_DOMAIN

GRIST_SESSION_SECRET
the key used to encode the session

GRIST_TAG
internal grist urls are /v/<GRIST_TAG>/....

GRIST_TESTING_SOCKET
a socket used for out-of-channel communication during tests only.

GRIST_TEST_HTTPS_OFFSET
if set, adds https ports at the specified offset.  This is useful in testing.

GRIST_TEST_SSL_CERT
if set, contains filename of SSL certificate.

GRIST_TEST_SSL_KEY
if set, contains filename of SSL private key.

GRIST_TEST_LOGIN
allow fake unauthenticated test logins (suitable for dev environment only).

GRIST_USER_ROOT
an extra path to look for plugins in.

HOME_PORT
port number to listen on for REST API server; if set to "share", add API endpoints to regular grist port.

PORT
port number to listen on for Grist server

REDIS_URL
optional redis server for browser sessions and db query caching

GOOGLE_CLIENT_ID
set to the Google Client Id to be used with Google API client

GOOGLE_CLIENT_SECRET
set to the Google Client Secret to be used with Google API client

GOOGLE_API_KEY
set to the Google API Key to be used with Google API client (accessing public files)

GOOGLE_DRIVE_SCOPE
set to the scope requested for Google Drive integration (defaults to drive.file)

Database variables:

Variable
Purpose

TYPEORM_DATABASE
database filename for sqlite or database name for other db types

TYPEORM_HOST
host for db

TYPEORM_LOGGING
set to 'true' to see all sql queries

TYPEORM_PASSWORD
password to use

TYPEORM_PORT
port number for db if not the default for that db type

TYPEORM_TYPE
set to 'sqlite' or 'postgres'

TYPEORM_USERNAME
username to connect as

Grist hosting:

Variable
Purpose

APP_DOC_URL
doc worker url, set when starting an individual doc worker (other servers will find doc worker urls via redis)

APP_STATIC_URL
url prefix for static resources (all servers need this)

APP_HOME_URL
url prefix for home api (home and doc servers need this)

APP_DOC_PORT
port to be used by an individual doc worker

APP_STATIC_PORT
port to be used by an individual static server

APP_HOME_PORT
port to be used by an individual home server

APP_DOC_START
set to "1" if container should run a doc worker

APP_STATIC_START
set to "1" if container should run a static server

APP_HOME_START
set to "1" if container should run a home server

GRIST_ADAPT_DOMAIN
set to "true" to support multiple base domains (careful, host header should be trustworthy)

GRIST_BACKUP_DELAY_SECS
wait this long after a doc change before making a backup

GRIST_DEFAULT_EMAIL
if set, login as this user if no other credentials presented

GRIST_ID_PREFIX
for subdomains of form o-, expect/produce o-${GRIST_ID_PREFIX}.

GRIST_SINGLE_ORG
set to an org "domain" to pin client to that org

GRIST_ORG_IN_PATH
if true, encode org in path rather than domain

GRIST_HOME_INCLUDE_STATIC
if set, home server also serves static resources

GRIST_ROUTER_URL
optional url for an api that allows servers to be (un)registered with a load balancer

GRIST_MANAGED_WORKERS
if set, Grist can assume that if a url targeted at a doc worker returns a 404, that worker is gone

GRIST_SERVE_SAME_ORIGIN
set to "true" to access home server and doc workers on the same protocol-host-port as the top-level page, same as for custom domains (careful, host header should be trustworthy)

GRIST_SUPPORT_ANON
if set to 'true', show UI for anonymous access (not shown by default)

GRIST_TEST_ROUTER
if set, then the home server will serve a mock version of router api at /test/router

GRIST_THROTTLE_CPU
if set, CPU throttling is enabled

GRIST_EXPERIMENTAL_PLUGINS
enables experimental plugins

Sandbox related variables:

Variable
Purpose

GRIST_SANDBOX_FLAVOR
Can be pynbox, unsandboxed, docker, gvisor, macSandboxExec (note: not all options in grist-core). If set, forces Grist to use the specified kind of sandbox.

GRIST_SANDBOX
A program or image name to run as the sandbox. See NSandbox.ts for nerdy details.

PYTHON_VERSION
Can be 2 or 3. If set, documents without an engine setting are assumed to use the specified version of python. Not all sandboxes support all versions.

PYTHON_VERSION_ON_CREATION
Can be 2 or 3. If set, newly created documents have an engine setting set to python2 or python3. Not all sandboxes support all versions.

Thank you very much for the effort!

Got the same issue here. I set up Grist with Cloudflare tunnel as the following docker-compose
version: '3.3'
services:
  grist-core:
    container_name: grist-core
    ports:
      - "<IP_ADDRESS>:8484:8484"
    volumes:
      - "~/grist-core:/persist"
    environment:
      - GRIST_SESSION_SECRET="xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"

    image: gristlabs/grist
    restart: unless-stopped
  tunnel:
    container_name: cloudflared-tunnel-grist-core
    image: cloudflare/cloudflared
    restart: unless-stopped
    network_mode: host
    command: tunnel --no-autoupdate run
    environment:
      - TUNNEL_TOKEN=xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx

Chrome Developer tab shows some incorrect API calls:
http://mydomain:8484/o/docs/api/session/access/active
http://mydomain:8484/o/docs/api/session/access/all
http://mydomain:8484/o/docs/api/log

The correct root URL should be https://mydomain/
What environment should I set? (@paulfitz )

APP_HOME_URL would be the key variable to set @duythongle - see https://support.getgrist.com/self-managed/#how-do-i-run-grist-on-a-server

@paulfitz
I added APP_HOME_URL="https://mydomain". It throws an error when running
Welcome to Grist.
In quiet mode, see http://localhost:8484 to use.
For full logs, re-run with DEBUG=1
2023-09-03 19:29:58.225 - info: == Grist version is 1.1.3 (commit unknown)
2023-09-03 19:29:58.231 - debug: skipping incomplete language fa (set GRIST_OFFER_ALL_LANGUAGES if you want it)
TypeError [ERR_INVALID_URL]: Invalid URL: "https://mydomain"
    at new NodeError (internal/errors.js:322:7)
    at onParseError (internal/url.js:270:9)
    at new URL (internal/url.js:346:5)
    at new FlexServer (/grist/_build/app/server/lib/FlexServer.js:153:73)
    at main (/grist/_build/app/server/mergedServerMain.js:39:20)
    at main (/grist/_build/stubs/app/server/server.js:124:54) {
  input: '"https://mydomain"',
  code: 'ERR_INVALID_URL'
}

Please note that I run the app on localhost only ports: "<IP_ADDRESS>:8484:8484" to hide the origin server from public internet. Cloudflare tunnel helps publish the app to Cloudflare platform with mydomain. The issue can be re-produced easily with my docker-compose above.

@duythongle I think somehow the quotes are getting included in your URL, since if I try an invalid URL:
$ docker run -e APP_HOME_URL=zig -it gristlabs/grist
Welcome to Grist.
In quiet mode, see http://localhost:8484 to use.
For full logs, re-run with DEBUG=1
2023-09-03 19:52:40.200 - info: == Grist version is 1.1.3 (commit unknown)
2023-09-03 19:52:40.206 - debug: skipping incomplete language fa (set GRIST_OFFER_ALL_LANGUAGES if you want it)
TypeError [ERR_INVALID_URL]: Invalid URL: zig

There are no quotes in the error message.

@paulfitz
Removing the quotes to APP_HOME_URL=https://mydomain helps resolve the issue. The app works as expected. Many thanks.


[WIP] Work on settings for the robot

[x] Import required settings code from LR, which was initially taken from the Android settings source code and changed

[x] Settings button on main page

[x] Actual settings

[ ] Everything can be changed without crashing

Settings from LR are now integrated, but there is some work that still needs to be done such as listing the different controls that can be used. Right now selecting anything in the robot page crashes the app

fixes https://github.com/remotv/controller-for-android/issues/8

fixes https://github.com/remotv/controller-for-android/issues/4


📃 Documentation Update: Added the security

Describe the documentation update.
No response
Why is this needed?
No response
Additional context
No response

@Dipanita45 add details, don't open empty issue request like this.!


How to resolve warning: initialization of 'uint32_t (*)(const TIM_TypeDef ), from incompatible pointer type 'uint32_t ()(TIM_TypeDef *)

Origin
How to resolve Warning, while building Multi_Timers in 3.2.0 Zephyr version.
zephyr-v3.2.0/zephyr/drivers/counter/counter_ll_stm32_timer.c:40:9: warning: initialization of 'uint32_t (*)(const TIM_TypeDef )
{aka 'unsigned int ()(const TIM_TypeDef )'} from incompatible pointer type 'uint32_t ()(TIM_TypeDef )' {aka 'unsigned int ()(TIM_TypeDef *)'}
[-Wincompatible-pointer-types]
Type
zephyr-v3.2.0
board: nucleo_h743zi
Module
Multi_Timers example
nucleo_h743zi.overlay
&timers2 {
st,prescaler = <95>;
counter {
compatible = "st,stm32-counter";
status = "okay";
};
};
Purpose
Resolve below warnings,
zephyr/drivers/counter/counter_ll_stm32_timer.c:40:9: warning: initialization of 'uint32_t (*)(const TIM_TypeDef )'
{aka 'unsigned int ()(const TIM_TypeDef )'} from incompatible pointer type 'uint32_t ()(TIM_TypeDef )' {aka 'unsigned int ()(TIM_TypeDef *)'} [-Wincompatible-pointer-types]
40 |         LL_TIM_OC_GetCompareCH1, LL_TIM_OC_GetCompareCH2,

zephyr-v3.2.0\zephyr\drivers\counter\counter_ll_stm32_timer.c
		/** Channel to compare get function mapping. */

Line No:38 #if !defined(CONFIG_SOC_SERIES_STM32F4X)
Line No:39 static uint32_t(*const get_timer_compare[TIMER_MAX_CH])( const TIM_TypeDef *) = {
Line No:40 	LL_TIM_OC_GetCompareCH1, LL_TIM_OC_GetCompareCH2,
Line No:41	LL_TIM_OC_GetCompareCH3, LL_TIM_OC_GetCompareCH4,
};
#else
static uint32_t(*const get_timer_compare[TIMER_MAX_CH])(TIM_TypeDef *) = {
LL_TIM_OC_GetCompareCH1, LL_TIM_OC_GetCompareCH2,
LL_TIM_OC_GetCompareCH3, LL_TIM_OC_GetCompareCH4,
};
#endif
Note: After removed "const" in static uint32_t(*const get_timer_compare[TIMER_MAX_CH])( const TIM_TypeDef *), warnings are not getting, but not correct way to remove const.

@Niranjan-eaton You have selected the wrong category for this, if this is an issue then it should be opened with "bug" not "TSC" - the technical steering committee of zephyr does not need to read this or vote on it.

@Niranjan-eaton  Can you please check your version of the modules/hal/stm32/stm32cube/stm32h7xx
I guess the pb you see is fixed with version v1.11.0 of the modules/hal/stm32/stm32cube/stm32h7xx
And it will come with the next zephyr 3.3.0
Maybe in between you could west update and get the hal_stm32  master branch c36136b253a46ed8dd63525eeee659393047ce83

Tried, west update in zephyr-v3.2.0>west update , but not resolved.
west_update_log.txt
Please help, suggest/Idea to remove warnings.


get kafka_brokers_sasl from instance.extensions

Description
request from BNPP
From: PRIOU Frantz <[frantz.priou@bnpparibas.com](mailto:frantz.priou@bnpparibas.com)>
Date: Thursday, 24 October 2024 at 12:49
To: Uche Nwankwo <[Uche.Nwankwo@ibm.com](mailto:Uche.Nwankwo@ibm.com)>
Cc: BERANTOSON Manda <[manda.berantoson@bnpparibas.com](mailto:manda.berantoson@bnpparibas.com)>
Subject: RE: [EXTERNAL] ER 835 - More information needed

Hello Uche,

It is a terraform feature request.

So we need to get bootstrap cluster when cluster deployment is terminated.

To get bootstrap cluster (variable kafka_brokers_sasl) we must create a first topic. If we don’t create a topic on cluster.. kafka_brokers_sasl variable is empty.

So we would like to be able to get variable kafka_brokers_sasl without having to create a first topic.

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

PR pipeline failing with below error, waiting on this fix to get merged.
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │ summary: "resourceIBMEventStreamsTopicCreate createSaramaAdminClient: kafka: client
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   has run out of available brokers to talk to: 6 errors occurred:\n\t* kafka server:
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   SASL Authentication failed: Authentication failed, invalid credentials\n\t* kafka
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   server: SASL Authentication failed: Authentication failed, invalid credentials\n\t*
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   kafka server: SASL Authentication failed: Authentication failed, invalid credentials\n\t*
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   kafka server: SASL Authentication failed: Authentication failed, invalid credentials\n\t*
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   kafka server: SASL Authentication failed: Authentication failed, invalid credentials\n\t*
TestRunUpgradeExample 2024-10-25T06:12:53Z logger.go:66: │   kafka server: SASL Authentication failed: Authentication failed, invalid credentials\n"

/run pipeline

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

GitHub release
v2.5.1

Your semantic-release bot :package::rocket:


Resetting after fileCache expires cannot immediately obtain values

What steps will reproduce the problem?
$rs = Yii::$app->cache->set('k', 'v', 5);
var_dump($rs);
var_dump(Yii::$app->cache->get('k'));

What is the expected result?
bool(true)
string(1) "v"
What do you get instead?
bool(true)
bool(false)
Additional info
The first execution returned no problem. When k was reexecuted after 5 seconds expired, the returned result was wrong and I couldn't get the value immediately.

Q
A

Yii version
2.0.50

PHP version
8.1.27

Operating system
openEuler

What kind of cache is it?

Is it file, DB, memcache, Redis?

What kind of cache is it?

file
'components' => [
    'cache' => [
        'class' => 'yii\caching\FileCache'
    ],
]

Is it file, DB, memcache, Redis?

file

I confirm this. Problem lies in the internal caching of file stats. Cache get returns previous cached set time which can yield false if duration is really short and cache is called immediately as in this case.
To be honest I'm not sure if this is worth fixing.

Short-lived file cache is pointless as file accessing is slow.
We could unlink the file but it adds overhead.
We could clear stats cache but again it adds overhead.

I recommend just adding the warning in the docs so it can be taken care of by the developer if that short duration on file cache is really necessary.

We could clear stats cache but again it adds overhead.

We only need clearstatcache() call in setValue(), I don't think that overhead should be noticeable comparing to already existing logic in this method.
Also, I don't think that this is related to cache duration. You should get such behavior always when you have consecutive set() and get() calls for expired cache.

Also, I don't think that this is related to cache duration

What do you mean? I logged set timestamp and received filemtime and it was always showing previous set value on rapid calls.

I assumed this happens when:

cache expired, but file still exists with mtime from the past.
you call get() which caches mtime by PHP for that file.
you call set() which overwrites file ad its mtime.
you call get() which uses mtime cached in step 2. and detects cache as expired.

In my tests it showed:
fresh cache set to X (timestamp)
cache get before X - success
next spin:
cache set to Y
cache get before Y but returns X - fail

I can confirm it works like described in https://github.com/yiisoft/yii2/issues/20300#issuecomment-2565243249 and clearstatcache() fixes this problem.
var_dump(Yii::$app->cache->get('__METHOD__'));
Yii::$app->cache->set('__METHOD__', 'test2', 10);
var_dump(Yii::$app->cache->get('__METHOD__'));
Yii::$app->cache->set('__METHOD__', 'test3', 10);
clearstatcache();
var_dump(Yii::$app->cache->get('__METHOD__'));

result:
bool(false)
bool(false)
string(5) "test3"

PR: https://github.com/yiisoft/yii2/pull/20304


Introduce MetricsMiddleware for OWin & Netcore

Here a not tested dotnet core aproach for implement the /metrics endpoint via a middleware (Owin). Not tested but perhaps a solution that works with ASP.MVC and NancyFx.
I use the csproj Tooling of netcore

fyi @stefc  If you're interested in collaborating,
https://github.com/dustinchilson/PrometheusNet.Owin


updated the ArcGIS Runtime SDK for Android version to 100.6.0

added RELEASE.md

for issue: https://github.com/Esri/mapbook-android/issues/34
The change is to just update this sample so that it works with the latest version of ArcGISRuntime.

@gunt0001 I have update the target and compiler sdk versions(there is an error in the commit message stating min sdk version) . The latest support lib version out is 28.0.0

@esreli can you please review.


Build cache is required, but could not be located

What version of Hugo are you using (hugo version)?

$ hugo version

0.91.0
Run peaceiris/actions-hugo@v2
Hugo version: 0.91.0
/usr/bin/tar xz --warning=no-unknown-keyword -C /home/runner/actions_hugo/_temp -f /home/runner/work/_temp/f496a467-32a5-47a2-81a4-8b300f6e61b7
/home/runner/actions_hugo/bin/hugo version
hugo v0.91.0-D1DC0E9A+extended linux/amd64 BuildDate=2021-12-17T09:50:20Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
I guess yes!
https://github.com/chengjun/starter-hugo-academic/runs/4603137806?check_suite_focus=true
Run hugo --gc --minify
build cache is required, but could not be located: GOCACHE is not defined and neither $XDG_CACHE_HOME nor $HOME are defined
Error: failed to download modules: failed to execute 'go [mod download]': failed to execute binary "go" with args [mod download]: build cache is required, but could not be located: GOCACHE is not defined and neither $XDG_CACHE_HOME nor $HOME are defined
 *errors.errorString
hugo: collected modules in 897 ms
Total in 898 ms
Error: Process completed with exit code 255.

I run hugo within github action:
name: Deploy to chengjun.github.io
on:
  push:
    branches:
      - master  # Set a branch name to trigger deployment
  pull_request:

jobs:
  deploy:
    runs-on: ubuntu-latest
    concurrency:
      group: ${{ github.workflow }}-${{ github.ref }}
    steps:
      - uses: actions/checkout@v2
        with:
          submodules: true  # Fetch Hugo themes (true OR recursive)
          fetch-depth: 0    # Fetch all history for .GitInfo and .Lastmod

      - name: Setup Hugo
        uses: peaceiris/actions-hugo@v2
        with:
          hugo-version: 'latest'
          extended: true

      - name: Build
        run: hugo --gc --minify

      - name: Deploy
        uses: peaceiris/actions-gh-pages@v3
        with:
          deploy_key: ${{ secrets.ACTIONS_DEPLOY_KEY }}
          external_repository: chengjun/chengjun.github.io
          publish_branch: main  # default: gh-pages
          publish_dir: ./public

Update
When I downgrade the version to 0.90.0, it works well. I guess it is the problem of the new version.

@chengjun Can you run this command on your computer? If so then the bug is from github actions so I recommend you report it in the corresponding repo, if not then this is the right place for the report.

What's the go version used here?

@bep the hugo version is 0.91.0.


Added unix domain socket support to http.proxy

changed the HTTPReverseProxySettings class to hold a destination URL instead
of (host, port)
added reverseProxyRequest overload with URL
updated the proxy handler logic
made getFilteredHost() public

I have addressed the comments.


[FR] add API to get available reviewers & assignees

expose repo.GetReviewers() as GET /api/v1/repos/<owner>/<repo>/reviewers
expose repo.GetAssignees() as GET /api/v1/repos/<owner>/<repo>/assignees
https://github.com/go-gitea/gitea/blob/a4148c0f12fe5a93d2c9a40f24d4813bcfef4ff8/models/repo.go#L617-L624
This would be helpful for tea issue create

blocking

https://codeberg.org/gitnex/GitNex/issues/200
https://gitea.com/noerw/tea/src/branch/issue-create-opts/modules/interact/issue_create.go#L118


Consider soft deletes (e.g. deleted_at)

It could be nice to add this kind of feature to core, so you get filtered views without cluttering up your application queries
Open questions:

Would it be slow to add this filter to all queries?
Can we do it only when we need it?
Would this be better handled in Photon?

I'm not sure whether this should be a core feature or rather be a pattern that's easy to be implemented in "user land" e.g. by using scoped queries.

I have a slight preference for this being a userland thing, as I've definitely mixed "hard delete" and "soft delete" models in the same system (e.g. soft delete Article records, but hard delete Tag records)


Split the README

Currently our README.md is giant. It contains lots of useful information but it may be easier to navigate if we split it into separate files with links to them from the main README. Feel free to update any of the wording too.

I will start looking into this, not sure if I can assign to myself - but just putting a note here so you're aware I'm working on it.


fix: account for possible -1 value in ClassLabel feature in statistics computation

will fix https://github.com/huggingface/datasets-server/issues/1833
It appeared that ClassLabel feature might have -1 (int) value which means "no label", I didn't know that so I end up having the last value of ClassLabel.names instead of it lol.
What would be the best way to represent it? It's not a null value. Now I convert it to string form (like "-1"). I thought of converting it to something like "no_label" or "(no label)" (as the viewer currently displays them in rows) but in theory it might match to the existing label specified in ClassLabel.names explicitly. Here is an example of response:
 {
        "nan_count": 0,
        "nan_proportion": 0.0,
        "n_unique": 2,  # num unique values stays the same as when there is no `no label` values
        "frequencies": {
        	"cat": 10
        	"dog": 5,
        	"-1": 10,
        }
    }

Alternatively, I can add a new field "num_no_labels" to store this number instead of having it as a key in "frequencies".
wdyt?

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (1de1c70) 89.40% compared to head (bc9aeb0) 92.56%.

:exclamation: Current head bc9aeb0 differs from pull request most recent head 12d0d8e. Consider uploading reports for the commit 12d0d8e to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1870      +/-   ##
==========================================
+ Coverage   89.40%   92.56%   +3.15%
==========================================
  Files         158       83      -75
  Lines        8890     6036    -2854
==========================================
- Hits         7948     5587    -2361
+ Misses        942      449     -493

Flag
Coverage Δ

jobs_cache_maintenance
?

jobs_mongodb_migration
?

libs_libapi
?

libs_libcommon
?

services_admin
?

services_api
?

services_rows
?

services_search
?

services_sse-api
?

services_worker
92.56% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

...worker/job_runners/split/descriptive_statistics.py
94.83% <100.00%> (ø)

services/worker/tests/fixtures/datasets.py
100.00% <ø> (ø)

...s/job_runners/split/test_descriptive_statistics.py
98.17% <100.00%> (ø)

... and 238 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Maybe this ?
 {
        "nan_count": 0,
        "nan_proportion": 0.0,
        "nolabel_proportion": 0.0,
        "n_unique": 2,  # num unique values stays the same as when there is no `no label` values
        "frequencies": {
        	"cat": 10
        	"dog": 5,
        }
    }

@lhoestq i like it this way! the question is is it okay to the frontend to change the fields set? @severo

Sure!

nit: no_label instead of nolabel

@severo i also renamed n_unique to num_unique - is it okay? because we have num_examples, to be consistent.
can revert though

@severo i also renamed n_unique to num_unique - is it okay? because we have num_examples, to be consistent.
can revert though

hmmm: it would require handling both field names while we recompute all the cache entries. I would prefer avoiding the pain

@severo ouch true, thanks, reverting then

i don't understand why e22 tests fail with timeout :( any ideas? @severo @AndreaFrancis @lhoestq

any ideas?

no, but let's create a dedicated issue for that

also, should I increment the version of the worker or better rerun manually only for datasets that has ClassLabel feature? @severo @AndreaFrancis

rerun manually only for datasets that has ClassLabel feature

would be a lot better!

also, should I increment the version of the worker or better rerun manually only for datasets that have ClassLabel feature? @severo @AndreaFrancis

no need imo, we can refresh the main ones manually

@severo since the e2e tests failures are not related to this pr, can we merge?


Rename tool directory in piped-base image

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Could you update the documentation as well?
https://pipecd.dev/docs/operator-manual/piped/installation/

Yes, I will do it in the next pull request. This PR updates only the piped-base image and the next one includes

update to use the new base image
fix the code
update the docs

Fine!
/approve


Generate ssh keys before parsing install args

https://github.com/Shippable/pm/issues/10310

this is done, closing


add "nouuid" option to XFS mounts

Restored snapshots inherit the XFS UUID of the original source. If mounted
on the same node as the original without the "nouuid" option, XFS will
complain about a duplicate UUID and refuse to mount.

Hm, or should we try to generate a new one? maybe should even LINSTOR do that for us?
Should we have this one as "quick fix" as it is trivial and the other one as LINSTOR feature request?

Hm, or should we try to generate a new one? maybe should even LINSTOR do that for us?
Should we have this one as "quick fix" as it is trivial and the other one as LINSTOR feature request?

Since LINSTOR is only concerned about the block volumes, I don't see any clear path to add this as a new feature.
I also considered the xfs_admin -U generate route, but that would add a new complication to the read-only mode: There would need to be a window where only one node executes the xfs_admin command, while the others wait for the change to propagate.
This second issue is a result of the way we lazily initialize the file system. If we could find a way to manage the file system already in the provisioning stage, we could also run xfs_admin there.

Since LINSTOR is only concerned about the block volumes, I don't see any clear path to add this as a new feature.
I also considered the xfs_admin -U generate route, but that would add a new complication to the read-only mode: There would need to be a window where only one node executes the xfs_admin command, while the others wait for the change to propagate.
This second issue is a result of the way we lazily initialize the file system. If we could find a way to manage the file system already in the provisioning stage, we could also run xfs_admin there.

Since LINSTOR is only concerned about the block volumes, I don't see any clear path to add this as a new feature.

actually it isn't. It has a feature to create file systems. So conceptionally it could also extend that to a "if the parent had the 'create file system option' set, then also make sure the children have a new UUID".

There would need to be a window where only one node executes the xfs_admin command, while the others wait for the change to propagate.

True, LINSTOR would be that central lock

This second issue is a result of the way we lazily initialize the file system.

That could be done by LINSTOR I guess. The "guess" is the important part here, I'd really have to look into when the FS is created how, especially with snapshots.
Don't know if you knew that LINSTOR can create FSs with options and if that changes anything. The fix as is is fine with me, feel free to merge. Just (re)consider if moving FS creation/UUID generation to LINSTOR is possible.

Since LINSTOR is only concerned about the block volumes, I don't see any clear path to add this as a new feature.

actually it isn't. It has a feature to create file systems. So conceptionally it could also extend that to a "if the parent had the 'create file system option' set, then also make sure the children have a new UUID".

There would need to be a window where only one node executes the xfs_admin command, while the others wait for the change to propagate.

True, LINSTOR would be that central lock

This second issue is a result of the way we lazily initialize the file system.

That could be done by LINSTOR I guess. The "guess" is the important part here, I'd really have to look into when the FS is created how, especially with snapshots.
Don't know if you knew that LINSTOR can create FSs with options and if that changes anything. The fix as is is fine with me, feel free to merge. Just (re)consider if moving FS creation/UUID generation to LINSTOR is possible.

actually it isn't. It has a feature to create file systems
TIL

I'll have to look into this LINSTOR feature. For now I'll merge this while thinking about how we could benefit from using LINSTOR in this case.

actually it isn't. It has a feature to create file systems
TIL

I'll have to look into this LINSTOR feature. For now I'll merge this while thinking about how we could benefit from using LINSTOR in this case.


SP5: refactor MultiAxisLimits

We should remove AxisLimits from RenderDetails and replace it with a new object that stores axis limits for all axes in a plot
Unfortunately there's already a class named MultiAxisLimits but that should probably be renamed to MultiAxisLimitsManager or something because it's stateful

In 4cc579f I moved what was MultiAxisLimits to ScottPlot.Control.MultiAxisLimitManager since it's purpose isn't for passing around multi-axis limits, but rather snapshotting them so they can be restored later (useful for click-drag operations)
Although this issue is resolved for now, there's still room in the future to create some type of structure analogous to AxisLimits ... it's confusing though because axis limits aren't paired (X axes and Y axes aren't paired except inside plottables) so it'll probably be more like GetAxisRanges() but then why not just GetAllAxes() and interrogate their Range properties? I guess this problem is fully resolved after all.


Automated cherry pick of #3863: baremetal: fix init baremetals fatal exit

Cherry pick of #3863 on release/2.13.
#3863: baremetal: fix init baremetals fatal exit

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@57eabfe). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #3864   +/-   ##
==============================================
  Coverage                ?   6.93%
==============================================
  Files                   ?     499
  Lines                   ?   84344
  Branches                ?       0
==============================================
  Hits                    ?    5852
  Misses                  ?   77996
  Partials                ?     496

Flag
Coverage Δ

#aFlag
6.93% <0%> (?)

Impacted Files
Coverage Δ

pkg/baremetal/agent.go
0% <0%> (ø)

pkg/baremetal/manager.go
0.51% <0%> (ø)

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 57eabfe...ab3556a. Read the comment docs.

/lgtm

/approve


build: update to system 0.20

Fix #1785

@patrickmichalina what changes you had to make in your custom fork after the upgrade?

Had to add this to the project.config in order for systemjs to get it working. Previously it worked without this entry. Obviously no fault of the seed, its a systemjs thing.
{
  name: '@nguniversal/express-engine',
  path: 'node_modules/@nguniversal/express-engine',
  packageMeta: {
    defaultExtension: 'js'
  }
}

the import being used is:
import { REQUEST } from '@nguniversal/express-engine/tokens';

Waiting for @meriturva to confirm that this works for him before I merge.

Okay, merging it. In case anyone experiences issues we can always revert it back.

@mgechev sorry ... i'm tying to pass to systemjs 0.20 right now but big issue with: https://github.com/auth0/angular-jwt now. So i will send you feedback as soon as possible.

@meriturva is it not working on your setup with 0.20? I have it working with angular-jwt.

@patrickmichalina could you please share your config? i had to back switch systemjs to 0.19 version for now!

let additionalPackages: ExtendPackages[] = [
      {
        name: 'angular2-jwt',
        path: 'node_modules/angular2-jwt/angular2-jwt.js'
      }, {
        name: 'js-cookie',
        path: 'node_modules/js-cookie/src/js.cookie.js'
      }, {
        name: '@nguniversal/express-engine',
        path: 'node_modules/@nguniversal/express-engine',
        packageMeta: {
          defaultExtension: 'js'
        }
      }];

    this.addPackagesBundles(additionalPackages);

I have found that big problem is typescript with tsconfig module: system and angular2-jwt on our big project.
Really strange.


Syntax lookup: Module

Suggestion for #162
PR should be editable, please feel free edit.

LGTM
Welcome back Kevan  🎉