[BUG] TEST

Please ping me on discord if you write up an issue. I will see the discord notification long before I see github notification. My discord username is claudekennilol.
Describe the bug
A clear and concise description of what the bug is.
E.g. "When I have a fortune buff and attack with "Super Awesome Axe" it does not roll twice"
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
What version of Roll Bonuses and PF1 are you using?
Roll Bonuses:
core:
Additional context
If there's a specific actor you're having issues with, please attach that actor here. To export an actor, go to your actors tab and right click on the actor. Upload the json file here.
Providing me with the actor means I will be able to import and reproduce the issue is. If there is not already a buff/feature/item configured with your problem on this actor, it will be much harder for me to reproduce and fix it.

done


Xcode 15.0 beta 7 Failed to build module 'GoogleMapsPlatformCombine'; this SDK is not supported by the compiler

Environment details

macOS 13.4.1
Library version: 0.3.1, with SPM on Xcode Xcode 15.0 beta 7

Steps to reproduce

add the 0.3.1 version to an Xcode using SPM in Xcode 15.0 beta 7 project
import GoogleMapsPlatformCombine, you will see this error message
"Failed to build module 'GoogleMapsPlatformCombine'; this SDK is not supported by the compiler (the SDK is built with 'Apple Swift version 5.5 (swiftlang-13<IP_ADDRESS> clang-13<IP_ADDRESS>)', while this compiler is 'Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.106 clang-15<IP_ADDRESS>)'). Please select a toolchain which matches the SDK."

If you would like to upvote the priority of this issue, please comment below or react on the original post above with :+1: so we can see what is popular when we triage.
@Kareem-Badr Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.


Can I reconfigure the wifi via the shell? [Support Needed]

I've received the device recently and set it up for my home network. I've now brought it to work and want to use it with the corp wifi. Is there a way to reconfig the wifi w/out repeating the setup?
I've shell'ed into the device and tried doing it manually using the command line, but it doesn't seem to work, following instructions such as these (that work on my Pi's):
For Wep or no password:
https://askubuntu.com/questions/294257/connect-to-wifi-network-through-ubuntu-terminal
For WPA:
https://askubuntu.com/questions/138472/how-do-i-connect-to-a-wpa-wifi-network-using-the-command-line
Sorry for the newbie question.

Hi @lokijota is the device MAC address registered on https://openwifi/ ?

Hi @lokijota unfortunately, the only way to reconfigure Wi-Fi settings is via setup and pressing and holding the power button on the back for 5+ seconds until the device goes into SoftAP mode. Then Connect and reconfigure from there.

I eventually gave up trying and went with the UI, it's just not very convenient, as I keep alternating between the office and home, and the process takes a while. If it were scriptable it'd probably be much faster and avoid the networking switching process. Isn't there any direct way of doing it in the shell, some configurations file? I mean, I assume the web UI is making some changes, right?
(regarding openwifi, yes, I did that before starting with attempt)

We haven't found any other way to do it. The OOBE / setup should only take a few minutes.
We'll make note to improve user experience for WiFi configuration.

If you do find a way to update WiFi settings via config file, please let us know.

WiFi configuration file isn't accessible for security reasons.

It's like re-configuring Alexa every time I want to use it :p . Thanks anyway for the help, I'll try to find a way anyway.


fix: use slim rpc cap lib to save on deps

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

1.2.8
GitHub release
1.2.8

Your semantic-release bot :package::rocket:


map & priority cause params to return lists

Given ruby 1.9.2, padrino 0.10.3
In app.rb:
get :test, map: "/test/:id", priority: :low do
  params[:id].inspect
end

Request:
/test/something
Expected Result:
"something"
Actual Result:
["test", "something"]
Without the priority, it works as expected...

If it's possible can you write a failing test to capture this issue?

So just removing the priority fixes the issue consistently?

Yes, I first encountered this in 0.10.2 and retested w/ a vanilla 0.10.3 to confirm...

not having much luck adding a test that actually fails.. it works fine on the mock infrastructure..

That's unfortunate, means its probably a reloader issue. Just curious put disable :reload in your app and see if priority routes work again?

Disabling reload doesn't help.. on further testing, it seems like map doesn't matter either.. the following simpler config is also broken:
get "/test/:id", priority: :low do

@nesquena weren't you gonna bring in the forking reloader?

It actually doesn't make a difference in this case:

Disabling reload doesn't help..

Apparently disabling reload doesn't help :(

below coode fixing the issue, but the problem somewhere deeper.
i.e., catch-all routes with normal priority break env['router.params'] for routes with lower priority
def default_routes!
  configure :development do
    get '*__sinatra__/:image.png', :priority => :low do
      content_type :png
      filename = File.dirname(__FILE__) + "/images/#{params[:image]}.png"
      send_file filename
    end
  end
end

this is definitely the same issue as https://github.com/padrino/padrino-framework/issues/738

This is the same bug as in here:
https://github.com/joshbuddy/http_router/issues/22

There is no :priority in the http_router, this is Padrino issue


Versions inconsistence

Good day,
I've met with the next issue.
On http://outdatedbrowser.com/en site Google Chrome the latest version is shown as 51.

But 52 version is installed indeed.

Hi @Eolga, thanks for notice that. We've fixed the version number on Chrome and Firefox and Opera too.
Cheers!
Bruno.


[bug]It doesn't work when I add " const fs = require('fs');" to my custom tool

It doesn't work when I add the following to my custom tool:
const fs = require('fs');
const path = require('path');
why?

Can't custom tools introduce packages about files and paths? So how to operate local files?

its not allowed by default, because then it exposes huge risk for others to gain access to your file system. However, you can still import it via TOOL_FUNCTION_BUILTIN_DEP env variable - https://docs.flowiseai.com/configuration/environment-variables#built-in-and-external-dependencies

Flowise support different environment variables to configure your instance. You can specify the following variables in the .env file inside packages/server folder
I did not find the above folder, mine is a windows system
"C:\Users\msrobot\AppData\Roaming\npm\node_modules\flowise\node_modules\flowise-ui.env"
I only found this file, but adding the environment variable had no effect on the task.

I did not install it with docker. Please help me point out the location of the .env file. I have been looking for it for a long time but could not find it. Thank you.

@musuiot you have to create a new file .env file at C:\Users\msrobot\AppData\Roaming\npm\node_modules\flowise folder


Support absolute file paths for spaces

Augment the space info and space packet endpoints to accept url
path encoded (https://golang.org/pkg/net/url/#PathEscape) paths.
Also:

Add some testing to search and space info endpoints.
Remove space list endpoint.

I hear your concerns. We’re doing this right now because it’s the simplest
way to get this working w the requirements for brim: we need to be able to
create a space next to wherever in the file system the user selected a pcap
On Fri, Feb 21, 2020 at 10:29 AM aswan<EMAIL_ADDRESS>wrote:

@aswan commented on this pull request.
In zqd/search/endpoint.go
https://github.com/brimsec/zq/pull/340#discussion_r382737538:

query, err := parseSearchRequest(r.Body)

if err != nil {
httpError(w, err.Error(), http.StatusBadRequest)
return
}

//logger.Debug("parseSearchRequest", zap.Stringer("query", query))
dataPath := filepath.Join(".", query.Space, "all.bzng") //XXX need root dir param

//XXX We do not support root dir param right now. If query.Space is an
// absolute path we will ignore the root dir param. For now relative paths

I'm going to change this to localhost. Does that resolve your concerns?
It lessens them a little bit, but unauthenticated requests to read any
file still seems like a bad idea. Can we invert this discussion: what's the
product requirement here? from there maybe we can figure out a solution
that wouldn't be so trivially exploitable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brimsec/zq/pull/340?email_source=notifications&email_token=AAQMJXI6IGL4LUGUX7OONKTREAMQRA5CNFSM4KYWFP22YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCWQFLJI#discussion_r382737538,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQMJXLA3J6LV4YWGLDRPT3REAMQRANCNFSM4KYWFP2Q
.

we need to be able to create a space next to wherever in the file system the user selected a pcap

I don't understand this sentence.  Is the requirement that all the temporary files we create (e.g., all.bzng, extracted pcaps, etc) be put in the same directory as the pcap file they are dervied from?  That seems like a strange requirement as it means the app wouldn't be able to handle a file in a read-only directory.  ..which seems like a scenario that is quite likely to occur in practice.

I would vote for keeping the space API we had and augmenting it with a way to upload a pcap.    The advantage of this is our existing tools and code just works and we don't make unnecessary work for us.
Also, the existing work we did on the APIs was fairly well thought out and, in the very near future, when we want brim to reach over the network to a remote zqd, we'll start putting back in then what we took out now.
It seems like we have convolved ux design of a desktop-scale open-brim experience with a re-do of the REST API.  I don't think we need to make changes to the existing API;  it just needs augmentation.  e.g., GET /space went away and broke tip of brim as well as my scripts.
Here are a few suggestions:

keep the local spaces directory in ~/Library/Brim (or equivalent for non mac)
put back the GET /space endpoint (the space names need not be exposed in the UI, but probably later)
add POST /space/{space}/pcaps which takes an absolute path of the pcap on the local filesystem as a parameter in the JSON body (support just a single post for now).  (this could also post a binary pcap over the network instead of using the file path... but later)
add GET /space/{space}/pcaps to return a list of the absolute file names of pcaps that have been ingested into that space

On startup, the app could list the spaces and get each absolute path to allow the user to open preloaded pcaps.
On pcap open, the app could generate a space name from the file name (or auto generate one if they aren't exposed to the user, space1, space2, etc) and create the space, then post the pcap file.  (We could stream back json status updates on the POST request, later.)
If we want a leave-no-traces ux (probably not given the zeek bottleneck), the app could do DELETE space/* on exit.
This approach accomplishes all the ux we've talked about while maintaining compatibility with our existing network-services-oriented API and results in fewer new moving parts.
Also, this sets us up to later support:

posting multiple pcaps at different times to the same space
post pcaps over the net
do everything we can do locally over the net to a zqd running on a server
support multiple connections to different data sources with tabbed searches running to different locations
etc

GET /space went away and broke tip of brim as well as my scripts.

Out of curiosity what were your scripts doing? Were you running brim against this branch?

add POST /space/{space}/pcaps which takes an absolute path of the pcap on the local filesystem as a parameter in the JSON body (support just a single post for now). (this could also post a binary pcap over the network instead of using the file path... but later)

Does this endpoint just the path of that pcap in the space?

GET /space went away and broke tip of brim as well as my scripts.

Out of curiosity what were your scripts doing? Were you running brim against this branch?

I have aliases that I run; script was the wrong word.  Also, we have existing code that we may want to cut and paste into an open-source repo that uses the logic of the existent API (fetch a list of spaces then do something with them).

add POST /space/{space}/pcaps which takes an absolute path of the pcap on the local filesystem as a parameter in the JSON body (support just a single post for now). (this could also post a binary pcap over the network instead of using the file path... but later)

Does this endpoint just the path of that pcap in the space?

I just spoke to @alfred-landrum and I realized I didn't finish my stream of consciousness here... the ux of putting the data next to the pcap can be accomplished for the pcap(s)-in-the-same-dir approach by having each space directory have a json config file that points to the .brim dir next to the pcap, essentially keeping the existing state management on the services side (I have a list of places I put pcaps and zeek logs and meta-info associated with them) instead of the app side (I keep a list of .brim dirs in localstorage).  The app can query the space list just as easily as the loading the list of recent pcaps from local storage.    This lets us keep the general model while giving the desktop ux experience so we'll be teed up to do zqd over the network before we know it.

Closing whilst I rework this for new changed scope.


Updated navbar-clone to mimic navbar

I am not a coding guy; I am only able to do some minor fix by just guessing the code structure. And this is my first ever pull request. I am doing it after watching a youtube video. Hope I did not mess things up. I've tested it and it works.
Basically, I've copied pieces from the navbar.html to navbar-clone.html in the root/layouts/partials folder.
If this goes through, I will also fix the footer menu where DOCS and BLOGROLL elements are duplicated.

This needs to be incorporated.  What is the status of this?


udocker load in rhel cluster doesn't work on saved tar docker image

Hi,
I have built an image on my pc using wsl with docker and I wanted to move it to a gpu cluster I have through the organization and run it there.
The organization uses udocker. udocker version returned 1.3.8.
Here is the output from the attempt at loading the image:
$ udocker load -i CompareRL/comparerldocker.tar
Info: already installed, installation skipped
Info: repository and tag already exist comparerldocker latest
Error: load failed

apologies if you tried this already
udocker images - does it have the image registered there already?
if yes, udocker rmi
if not
udocker -D load -i CompareRL/comparerldocker.tar
to get more debug info and send us
another possibility is for you to send us (somehow) the tar file IF it's not private of course

inactive for several months, please reopen if necessary


Fix const correctness for SparseMatrix methods EliminateCols() and GetSubMatrix()

It is natural to have const specifier for the input Array which stores indices in EliminateCols() and to make the method GetSubMatrix() a const one.

Please hold on merging in next -- we will flush it after the 3.3.2 release.


Unknown pragma when compiling with GCC

I pulled down the latest dev drop today.
It appears there is a clang pragma in cmsis_gcc_m.h (maybe others too) that generates a build error.

If I delete this line, then it builds fine

Thanks for reporting this. I'd assume GCC to ignore pragmas for other compilers. Which version of GCC do you use?
In this case, the pragma got added long time ago when the only way to use Clang was via GCC compatibility. Nowadays, we have separate headers for Clang and could (probably should) remove these pragmas from GCC headers.
Would you like to contribute the changes that via pull-request?

We are using ARM GNU Toolchain 12.2. And are treating warnings as errors.
Sure, I can make a pull-request


SYS_calls should be bound

Since there’s a binding to syscall function, SYS_* constants should also be bound. Perhaps in their own module, since there’s so many of them.

[ ] Linux (???)
[ ] Android where _NR are defined in unistd{32,64};
[ ] FreeBSD;
[ ] NetBSD;
[ ] OpenBSD;
[ ] Apple

Sounds good to me, should be relatively easy enough to add at any time!

I am willing to take this on. I've been wanting to script getting constants anyway—at least on Linux— and this is a great excuse.

Also re Linux, I found the easiest place to get the sycall numbers is actually in from musl source, eg:

http://git.musl-libc.org/cgit/musl/tree/arch/aarch64/bits/syscall.h
http://git.musl-libc.org/cgit/musl/tree/arch/powerpc/bits/syscall.h

It'll be worth putting these in a joint musl / other location though. Probably unix::notbsd::linux::syscalls, which will be included by others.

For now I'd prefer to just stick with duplication, it's not that hard to have and it presents a much more understandable organization hierarchy

Sounds good.

:+1: for scripting this and getting consistent, complete coverage.

@alexcrichton I think this is addressed as there are tons of SYS_* constants now in libc.

Indeed!


Investigate 500 errors on mobile

Per the crawl error report below, pages that should serve 404 errors on mobile are currently serving 500 errors. It’s important that Google registers that 404s on desktop are also 404s on mobile so that the crawl budget is prioritized appropriately.
We should fix this so that the correct 404 errors are shown to Google, and investigate for other causes in addition to the merge of the m.artsy.net and artsy.net code bases. @christine-artsy will bring this up in the next GMV meeting as well.
https://www.google.com/webmasters/tools/crawl-errors?siteUrl=https://www.artsy.net/&utm_source=wnc_628900&utm_medium=gamma&utm_campaign=wnc_628900&utm_content=msg_723500&hl=en#t2=31

cc @craigspaeth @christine-artsy @tayapage6


Detect Zebra crossings in CARLA

Hello,
I am working on detecting crosswalks in CARLA data. I want to create ground truths with bounding 2D box data of crosswalks. How can i get the crosswalks road marking objects?

@XGodina Continuing on my previous comment. Is There a way to give separate labels to Zebra crossings in City Palette. So that i can easily draw bounding boxes on the Zebra crossings. If it is not possible, Is there a way to mark bounding boxes on Zebra crossings from Carla itself

Hello @bhuvanharlapur
Currently, the crosswalk is included in the Land Markings label because they are the same geometry.
A good way to do that will be to cut the geometry (crosswalk part) and create a new tag for semantic segmentation.
Here you have documentation on how to do that.
https://carla.readthedocs.io/en/0.9.13/tuto_D_create_semantic_tags/#create-semantic-tags


👌auto select download tab based on user system

personal message
I loved the project, the website.
and because I learnt a lot from Bucky Roberts and thenewboston channel, I saw the download page and I was like, why the website doesn't know I use linux? so I did just that, small change but nice user experience, I just wanted to give a little thing back to Bucky and thenewboston cimmunity.
PR details
when Download page loads, now the Download component try to guess user system to
calculate a UserOS number variable which represents the index of the
tab that shows the OS specific download instruction
which is based on the index of each component in tabs array :

Windows
Mac
Linux

default is still windows if the regex /Windows|Linux|Mac/gi doesn't mach anything in navigator.userAgent
thus closes #819

@akamfoad , nice job, thanks for contributin. Please reference the issue that this solves, or if there is no issue, please create one and make sure to leave your account number so we'll be able to reward you.
Also, for future reference, submit only PRs that are solving reported issues.

thanks for the instructions, I will, I created the issue #819 and added it to the PR message.


fix: stop Utils#createTopic trying to create topic if admin client gets closed

Type of change

Bugfix

Description
Break out of the createTopic creation loop if the RetriableException message indicates that the Admin client is closed.  This prevents a thread pointlessly spinning in the background, failing forever.
Please describe your pull request
Additional Context
Why are you making this pull request?
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@SamBarker what do you think to committing this change?  I think in the long term, the Kafka Client needs a fix, but this would improve the behaviour in the meanwhile.

I'll comment the nastiness with a note about https://issues.apache.org/jira/browse/KAFKA-15507.


Invalid HTTP method: PATCH

Facing a problem, where a REST API has a "PATCH" method
using the jakarta.ws.rs.PATCH annotation i can very well setup such a call, but it fails with
Exception in thread "main" java.net.ProtocolException: Invalid HTTP method: PATCH
	at java.base/java.net.HttpURLConnection.setRequestMethod(HttpURLConnection.java:491)
	at java.base/sun.net.www.protocol.http.HttpURLConnection.setRequestMethod(HttpURLConnection.java:598)
	at java.base/sun.net.www.protocol.https.HttpsURLConnectionImpl.setRequestMethod(HttpsURLConnectionImpl.java:343)
	at si.mazi.rescu.HttpTemplate.configureURLConnection(HttpTemplate.java:182)
	at si.mazi.rescu.HttpTemplate.send(HttpTemplate.java:123)
	at si.mazi.rescu.RestInvocationHandler.invokeHttp(RestInvocationHandler.java:152)
	at si.mazi.rescu.RestInvocationHandler.invoke(RestInvocationHandler.java:111)

with older java versions one could work around it with a pretty dirty trick: https://stackoverflow.com/questions/25163131/httpurlconnection-invalid-http-method-patch
but with the newer java versions (like ie 22.0.1) the workaround does not work any more
any idea how to bypass the problem?

See also #97.

here is an idea
https://github.com/mmazi/rescu/pull/148


Generating differntly each time function is called

Hi Erik,
the first time the pdf generator function is triggered, the file looks completly different than it should. Calling the function the second time it works as it should....
I really don't now what to do....
Thank you for your help!
Kind regards...

Same issue, basically on a fresh page load - pdf exports only generates the pdf content in browser and on second the pdf export starts working as well.

Same issue, basically on a fresh page load - pdf exports only generates the pdf content in browser and on second the pdf export starts working as well.


"Command not found" - but it is in my current directory?

This is weird. When I have a certain command in my current directory, it won't find nor execute it. But with ./ it does. Here an example with gulp:
~/D/s/n/.bin ❯❯❯ gulp
zsh: command not found: gulp
~/D/s/n/.bin ❯❯❯ ./gulp
[14:02:53] Requiring external module coffee-script/register
... // it works

btw: ls -la shows gulp in this directory

This is intended, as it is not in your path. This is also not a prezto issue. You need to stop making issues on the prezto issue tracker for things that are not specific to prezto. This is the fourth time you have done this.

export PATH=$PATH:.

thanks - yeah i know i ask a lot but the problem is that i often do not know what really caused it. never had the above issue before i installed prezto, that's why my initial guess.

That's a potential security issue to add the current directory in your PATH, never do that.

Exactly my point


ci: Add Pre-release action to synchronize branches

Instructions

PR target branch should be against development
PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

This action synchronizes a release version of the web sdk between all necessary branches

Testing Plan

[ ] Was this tested locally? If not, explain why.
N/A This is a CI/CD Update

Reference Issue (For mParticle employees only.  Ignore if you are an outside contributor)

Closes https://go.mparticle.com/work/SQDSDKS-5965

The commit name should also probably not be Pre-release, as it is the actual CDN release step.


Spin-telemetry: consider eliminating the use of set_global_default

Today spin_telemetry::init() calls registry::init() to build a tracing registry subscriber. Inside of that API, it calls dispatcher::set_global_default(). According to its documentation:

Note: Libraries should NOT call set_global_default()! That will cause conflicts when executables try to set them later.

When used as a library, spin_telemetry should not call set_global_default as it might cause conflicts with executables that depends on it. E.g. the spin shim.
Alternative

consider using opentelemetry tracer instrumentation instead of that of tracing.
consider using set_default() for every thread that spin runtime creates (albert much more invasive to the code)

Note: Libraries should NOT call set_global_default()! That will cause conflicts when executables try to set them later.

spin_telemetry::init() is meant to be called by binaries, not libraries.

consider using opentelemetry tracer instrumentation instead of that of tracing.

This would only cover part of the functionality of spin-telemetry and would still normally involve setting a global.

consider using set_default() for every thread that spin runtime creates (albert much more invasive to the code)

This may be plausible by hooking into tokio, but I'm not sure I understand how this would work with the shim
Could you say more about the issues you are seeing? I think we should be able to find less invasive options for shim interop.

Could you say more about the issues you are seeing?

Yes, I had long discussions with @calebschoepp on this topic and the main issue we are seeing is that, in containerd spin shim, there are two places where set_global_default is invoked. One at the entrypoint of the shim binary, which is supposed to collect traces from shim functions. Another is within the container sandbox that initializes a spin trigger and runtime, which is to collect traces from spin runtime.
Note that the shim process is the parent of the spin runtime process, but since it's invoked by clone3() the spin runtime process inherits the global states the shim process has, including global states for the tracing collector. Hence the panic saying a second invocation of the set_global_default.
The tricky part of this case is that both the spin shim and spin runtime processes are multi-threaded.

spin_telemetry::init() is meant to be called by binaries, not libraries.

You are right that it should be called by the binary. but in the case of spin shim, spin_telemetry::init() is invoked at the point where spin runtime is initializing. you may ask, why don't we call spin_telemetry::init() at the main() function of the spin shim binary? That's because at that moment, the binary hasn't parsed the container spec to get OTLP environment variables such as the endpoint, and it's not within the sandbox that is supposed to contain the spin runtime.

My understanding is that opentelemetry crate supports namespaced trace instrumentation, right? It should help the case I described above.

Namespaced trace instrumentation would potentially help us get around setting a global, but I think switching Spin from tracing to opentelemetry is a non-starter. Beyond just producing traces we also rely on tracing in Spin for:

Producing metrics
Implementing WASI Observe

since it's invoked by clone3() the spin runtime process inherits the global states the shim process has, including global states for the tracing collector.

Is this intentional or accidental behavior? Subprocesses do not usually inherit memory like this...


PyChOpMarg not starting

(pychopmarg) C:>pychopmarg
matplotlib not found while setting up plotting
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\dghan.venv\pychopmarg\Scripts\pychopmarg.exe_main.py", line 4, in
File "C:\Users\dghan.venv\pychopmarg\Lib\site-packages\pychopmarg\cli.py", line 35, in
@cli.command(context_settings=dict(help_option_names=["-h", "--help"]))
^^^
NameError: name 'cli' is not defined
I installed matplotlib to eliminate the first error, but the others remained.  I'm using Python 3.11.9.

The GUI is broken.
Jupyter notebook use only, for now.
-db

On Nov 2, 2024, at 12:23 PM, Dennis Han @.***> wrote:
(pychopmarg) C:>pychopmarg
matplotlib not found while setting up plotting
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\dghan.venv\pychopmarg\Scripts\pychopmarg.exe_main.py", line 4, in
File "C:\Users\dghan.venv\pychopmarg\Lib\site-packages\pychopmarg\cli.py", line 35, in
@cli.command(context_settings=dict(help_option_names=["-h", "--help"]))
^^^
NameError: name 'cli' is not defined
I installed matplotlib to eliminate the first error, but the others remained. I'm using Python 3.11.9.
—
Reply to this email directly, view it on GitHub https://github.com/capn-freako/PyChOpMarg/issues/10, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAH7XGRBYVOTNLPIXIR7QULZ6T36ZAVCNFSM6AAAAABRB3OV66VHI2DSMVQWIX3LMV43ASLTON2WKOZSGYZTANJYGY4DKNQ.
You are receiving this because you are subscribed to this thread.


Connection reset

Environment
HikariCP version: 2.5.1
JDK version     : 1.8.0_65
Database        :MySQL
Driver version  : 5.1.41

com.mysql.jdbc.exceptions.jdbc4.CommunicationsException: Communications link failure

The last packet successfully received from the server was 4,317,240 milliseconds ago.  The last packet sent successfully to the server was 4,317,240 milliseconds ago.
	at sun.reflect.GeneratedConstructorAccessor16.newInstance(Unknown Source)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
	at com.mysql.jdbc.Util.handleNewInstance(Util.java:425)
	at com.mysql.jdbc.SQLError.createCommunicationsException(SQLError.java:989)
	at com.mysql.jdbc.MysqlIO.send(MysqlIO.java:3746)
	at com.mysql.jdbc.MysqlIO.sendCommand(MysqlIO.java:2509)
	at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2680)
	at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2501)
	at com.mysql.jdbc.PreparedStatement.executeInternal(PreparedStatement.java:1858)
	at com.mysql.jdbc.PreparedStatement.executeUpdateInternal(PreparedStatement.java:2079)
	at com.mysql.jdbc.PreparedStatement.executeUpdateInternal(PreparedStatement.java:2013)
	at com.mysql.jdbc.PreparedStatement.executeLargeUpdate(PreparedStatement.java:5104)
	at com.mysql.jdbc.PreparedStatement.executeUpdate(PreparedStatement.java:1998)
	at com.zaxxer.hikari.pool.ProxyPreparedStatement.executeUpdate(ProxyPreparedStatement.java:61)
	at com.zaxxer.hikari.pool.HikariProxyPreparedStatement.executeUpdate(HikariProxyPreparedStatement.java)
	......
Caused by: java.net.SocketException: Connection reset
	at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:113)
	at java.net.SocketOutputStream.write(SocketOutputStream.java:153)
	at java.io.BufferedOutputStream.flushBuffer(BufferedOutputStream.java:82)
	at java.io.BufferedOutputStream.flush(BufferedOutputStream.java:140)
	at com.mysql.jdbc.MysqlIO.send(MysqlIO.java:3728)
	... 17 more

Everything seems ok,  but this error just periodically to occurs

@RustJason Set your maxLifetime to at least one minute shorter than the MySQL wait_timeout as per this article.

thanks @brettwooldridge

If there is additional network infrastructure, such as load-balancers, between your application and the database, they may be imposing a separate (shorter) timeout.  The pool maxLifetime should be one minute shorter than the shortest timeout in the infrastructure path.

Yeah, I understand that, I Think it's the LVS at the first layer and behind that, it's haproxy

But I wonder why it is one minute shorter, can we just set 5 or 10 sec shorter than that?

Under heavy load Java timers can fire many seconds off the mark.  We typically recommend one minute as a completely safe margin that accounts for load, NTP clock adjustment, and network delay.  Why would you want to try to cut the margin so short?

because longer lifetime will reduce the connection time, Do you think this is a bug?

[DEBUG][2017-03-22T09:50:58.267+0800][pool-1-thread-1 - com.zaxxer.hikari.HikariConfig:857 - 19623-huzhou-yinqing]_pubtrans_undef||_msg=maxLifetime.....................120000||
[DEBUG][2017-03-22T09:40:54.660+0800][HikariPool-56 connection adder - com.zaxxer.hikari.pool.HikariPool:445 - ]_pubtrans_undef||_msg=HikariPool-56 - Added connection com.mysql.jdbc.JDBC4Connection@60db6cc8||
[WARN ][2017-03-22T09:50:21.484+0800][main - com.zaxxer.hikari.pool.ProxyConnection:156 - ]_pubtrans_undef||_msg=HikariPool-56 - Connection com.mysql.jdbc.JDBC4Connection@60db6cc8 marked as broken because of SQLSTATE(08S01), ErrorCode(0)||
[DEBUG][2017-03-22T09:50:21.485+0800][HikariPool-56 connection closer - com.zaxxer.hikari.pool.PoolBase:128 - ]_pubtrans_undef||_msg=HikariPool-56 - Closing connection com.mysql.jdbc.JDBC4Connection@60db6cc8: (connection is broken)||
[DEBUG][2017-03-22T09:50:21.485+0800][HikariPool-56 connection closer - com.zaxxer.hikari.pool.PoolBase:137 - ]_pubtrans_undef||_msg=HikariPool-56 - Closing connection com.mysql.jdbc.JDBC4Connection@60db6cc8 failed||

This connection 60db6cc8 did not closed after passed the maxLifetime
I was expecting something like this:
[DEBUG][2017-03-22T09:36:12.369+0800][HikariPool-13 connection closer - com.zaxxer.hikari.pool.PoolBase:128 - ]_pubtrans_undef||_msg=HikariPool-13 - Closing connection com.mysql.jdbc.JDBC4Connection@75501ddf: (connection has passed maxLifetime)||

But didn't see it until error

If the connection is in use at the maxLifetime event, it cannot be evicted.  In that case, it will be marked for eviction and evicted when it is returned.
Can you enable the leakDetectionThreshold, also set for two minutes?

@RustJason It means you need to close the connection when you are finished with it.  You do not need to check the lifetime, just ensure that you don't hold connections open longer than the lifetime.
It is always recommended to close connection using Java's try-with-resources blocks:
try (Connection connection = dataSource.getConnection()) {
   ... SQL here
}

Getting a connection from the pool is extremely cheap, on the order of nanoseconds, so there is almost no reason to hold connections out of the pool for long periods of time.


flutter create package template reports an android embedding v2 error

Steps to Reproduce

flutter create --template=package testpackage

Output
 flutter create --template=package testpackage
Creating project testpackage...
Running "flutter pub get" in testpackage...
Resolving dependencies in testpackage...
+ async 2.10.0 (2.11.0 available)
+ boolean_selector 2.1.1
+ characters 1.2.1 (1.3.0 available)
+ clock 1.1.1
+ collection 1.17.0 (1.17.1 available)
+ fake_async 1.3.1
+ flutter 0.0.0 from sdk flutter
+ flutter_lints 2.0.1
+ flutter_test 0.0.0 from sdk flutter
+ js 0.6.5 (0.6.7 available)
+ lints 2.0.1
+ matcher 0.12.13 (0.12.14 available)
+ material_color_utilities 0.2.0 (0.3.0 available)
+ meta 1.8.0 (1.9.0 available)
+ path 1.8.2 (1.8.3 available)
+ sky_engine 0.0.99 from sdk flutter
+ source_span 1.9.1
+ stack_trace 1.11.0
+ stream_channel 2.1.1
+ string_scanner 1.2.0
+ term_glyph 1.2.1
+ test_api 0.4.16 (0.4.18 available)
+ vector_math 2.1.4
Changed 23 dependencies in testpackage!
This app is using a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to migrate
this app to the V2 embedding.
Take a look at the docs for migrating an app:
https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects
Wrote 13 files.

All done!
Your package code is in testpackage/lib/testpackage.dart

In my specific case, if you include device_info_plus which requires android embedding v2, tests are unrunnable.

flutter pub add device_info_plus
flutter test

Test Output
Running "flutter pub get" in testpackage...
Resolving dependencies...
  async 2.10.0 (2.11.0 available)
  characters 1.2.1 (1.3.0 available)
  collection 1.17.0 (1.17.1 available)
  js 0.6.5 (0.6.7 available)
  matcher 0.12.13 (0.12.14 available)
  material_color_utilities 0.2.0 (0.3.0 available)
  meta 1.8.0 (1.9.0 available)
  path 1.8.2 (1.8.3 available)
  test_api 0.4.16 (0.4.18 available)
Got dependencies!
This app is using a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to migrate
this app to the V2 embedding.
Take a look at the docs for migrating an app:
https://github.com/flutter/flutter/wiki/Upgrading-pre-1.12-Android-projects
The plugin `device_info_plus` requires your app to be migrated to the Android
embedding v2. Follow the steps on the migration doc above and re-run this
command.

Looking through the help of flutter, I notice there is plugin and package. I realise I may have use the wrong template. It was my understanding that I should use plugin when I am writing native code. Since I'm only writing dart code, I chose package. Is this correct?

Flutter Doctor
[✓] Flutter (Channel stable, 3.7.7, on macOS 12.6.2 21G320 darwin-arm64, locale
    en-GB)
    • Flutter version 3.7.7 on channel stable at /usr/local/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2ad6cd72c0 (13 days ago), 2023-03-08 09:41:59 -0800
    • Engine revision 1837b5be5f
    • Dart version 2.19.4
    • DevTools version 2.20.1

@NebulaFox
There's a similar open issue describing your case, https://github.com/flutter/flutter/issues/116320 so follow-up in it for further updates. Closing this as a duplicate.


How to stop sounds from playing again on mouse hover?

Describe your problem here.
hey there! was making a new menu state for my mod and wanted to make the scroll sound to play when mouse is over the button, but whilst the mouse is over said button, the sound keeps playing and playing.
Video Attached.
https://user-images.githubusercontent.com/88951878/193193540-29226a0c-280a-4446-bcf5-add044249dc7.mp4
Here's the code that plays the sound.
		//elixio icon
		if(FlxG.mouse.overlaps(elixioIcon)){
			FlxG.sound.play(Paths.sound('scrollMenu'));
			elixioIcon.alpha = 1;
			quote1.alpha = 1;
		}
		else
		{
			elixioIcon.alpha = 0.5;
			quote1.alpha = 0;
		}

		//beebop icon
		if(FlxG.mouse.overlaps(beebopIcon)){
			FlxG.sound.play(Paths.sound('scrollMenu'));
			beebopIcon.alpha = 1;
			quote2.alpha = 1;
		}
		else
		{
			beebopIcon.alpha = 0.5;
			quote2.alpha = 0;
		}

sorry if this is kinda a dumb/obvious question, Im a newbie in haxeflixel lul :d
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Made a custom credit menu.

Describe your problem here.
hey there! was making a new menu state for my mod and wanted to make the scroll sound to play when mouse is over the button, but whilst the mouse is over said button, the sound keeps playing and playing.
Video Attached.
2022-09-29.23-57-14.mp4
Here's the code that plays the sound.
		//elixio icon
		if(FlxG.mouse.overlaps(elixioIcon)){
			FlxG.sound.play(Paths.sound('scrollMenu'));
			elixioIcon.alpha = 1;
			quote1.alpha = 1;
		}
		else
		{
			elixioIcon.alpha = 0.5;
			quote1.alpha = 0;
		}

		//beebop icon
		if(FlxG.mouse.overlaps(beebopIcon)){
			FlxG.sound.play(Paths.sound('scrollMenu'));
			beebopIcon.alpha = 1;
			quote2.alpha = 1;
		}
		else
		{
			beebopIcon.alpha = 0.5;
			quote2.alpha = 0;
		}

sorry if this is kinda a dumb/obvious question, Im a newbie in haxeflixel lul :d
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Made a custom credit menu.

	//elixio icon
	if(FlxG.mouse.overlaps(elixioIcon) && elixioIcon.alpha != 1{
		FlxG.sound.play(Paths.sound('scrollMenu'));
		elixioIcon.alpha = 1;
		quote1.alpha = 1;
	}
	else
	{
		elixioIcon.alpha = 0.5;
		quote1.alpha = 0;
	}

	//beebop icon
	if(FlxG.mouse.overlaps(beebopIcon) && beebopIcon.alpha != 1){
		FlxG.sound.play(Paths.sound('scrollMenu'));
		beebopIcon.alpha = 1;
		quote2.alpha = 1;
	}
	else
	{
		beebopIcon.alpha = 0.5;
		quote2.alpha = 0;
	}

this happens.

Describe your problem here.
hey there! was making a new menu state for my mod and wanted to make the scroll sound to play when mouse is over the button, but whilst the mouse is over said button, the sound keeps playing and playing.
Video Attached.
2022-09-29.23-57-14.mp4
Here's the code that plays the sound.
		//elixio icon
		if(FlxG.mouse.overlaps(elixioIcon)){
			FlxG.sound.play(Paths.sound('scrollMenu'));
			elixioIcon.alpha = 1;
			quote1.alpha = 1;
		}
		else
		{
			elixioIcon.alpha = 0.5;
			quote1.alpha = 0;
		}

		//beebop icon
		if(FlxG.mouse.overlaps(beebopIcon)){
			FlxG.sound.play(Paths.sound('scrollMenu'));
			beebopIcon.alpha = 1;
			quote2.alpha = 1;
		}
		else
		{
			beebopIcon.alpha = 0.5;
			quote2.alpha = 0;
		}

sorry if this is kinda a dumb/obvious question, Im a newbie in haxeflixel lul :d
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Made a custom credit menu.

	//elixio icon
	if(FlxG.mouse.overlaps(elixioIcon) && elixioIcon.alpha != 1{
		FlxG.sound.play(Paths.sound('scrollMenu'));
		elixioIcon.alpha = 1;
		quote1.alpha = 1;
	}
	else
	{
		elixioIcon.alpha = 0.5;
		quote1.alpha = 0;
	}

	//beebop icon
	if(FlxG.mouse.overlaps(beebopIcon) && beebopIcon.alpha != 1){
		FlxG.sound.play(Paths.sound('scrollMenu'));
		beebopIcon.alpha = 1;
		quote2.alpha = 1;
	}
	else
	{
		beebopIcon.alpha = 0.5;
		quote2.alpha = 0;
	}

try this

This happens.
https://user-images.githubusercontent.com/88951878/193232173-5473b4ad-dc1e-4c5d-8fab-8ba3318dade5.mp4

put this outside the override thingies, and make sure it's in the class. var isOverlayed:Bool = false;
	//elixio icon
	if(FlxG.mouse.overlaps(elixioIcon) && !isOverlayed{
		FlxG.sound.play(Paths.sound('scrollMenu'));
		elixioIcon.alpha = 1;
		quote1.alpha = 1;
                isOverlayed = true;
	}
	else
	{
		elixioIcon.alpha = 0.5;
		quote1.alpha = 0;
               isOverlayed = false;
	}

	//beebop icon
	if(FlxG.mouse.overlaps(beebopIcon) && !isOverlayed){
		FlxG.sound.play(Paths.sound('scrollMenu'));
		beebopIcon.alpha = 1;
		quote2.alpha = 1;
                isOverlayed = true;
	}
	else
	{
		beebopIcon.alpha = 0.5;
		quote2.alpha = 0;
                isOverlayed = false;
	}

Not sure what I'm doing wrong, but every time I do that this error appears.
source/CreditsState.hx:36: characters 2-4 : Expected }
Heres the code.
class CreditsState extends MusicBeatState
{

	var bg = new FlxSprite().loadGraphic(Paths.image('creditsMenu/bgGradient'));
	var elixioIcon = new FlxSprite(300, FlxG.height * 0.38).loadGraphic(Paths.image('creditsMenu/elixioIcon'));
	var beebopIcon = new FlxSprite(735, FlxG.height * 0.38).loadGraphic(Paths.image('creditsMenu/beebopIcon'));

	//elixio icon
	if(FlxG.mouse.overlaps(elixioIcon) && !isOverlayed{
		FlxG.sound.play(Paths.sound('scrollMenu'));
		elixioIcon.alpha = 1;
		quote1.alpha = 1;
        isOverlayed = true;
	}
	else
	{
		elixioIcon.alpha = 0.5;
		quote1.alpha = 0;
               isOverlayed = false;
	}

	//beebop icon
	if(FlxG.mouse.overlaps(beebopIcon) && !isOverlayed){
		FlxG.sound.play(Paths.sound('scrollMenu'));
		beebopIcon.alpha = 1;
		quote2.alpha = 1;
                isOverlayed = true;
	}
	else
	{
		beebopIcon.alpha = 0.5;
		quote2.alpha = 0;
        isOverlayed = false;
	}

Not sure what I'm doing wrong, but every time I do that this error appears. source/CreditsState.hx:36: characters 2-4 : Expected }
Heres the code.
class CreditsState extends MusicBeatState
{

	var bg = new FlxSprite().loadGraphic(Paths.image('creditsMenu/bgGradient'));
	var elixioIcon = new FlxSprite(300, FlxG.height * 0.38).loadGraphic(Paths.image('creditsMenu/elixioIcon'));
	var beebopIcon = new FlxSprite(735, FlxG.height * 0.38).loadGraphic(Paths.image('creditsMenu/beebopIcon'));
	var isOverlayed:Bool = false;

	//elixio icon
	if(FlxG.mouse.overlaps(elixioIcon) && !isOverlayed{
		FlxG.sound.play(Paths.sound('scrollMenu'));
		elixioIcon.alpha = 1;
		quote1.alpha = 1;
        isOverlayed = true;
	}
	else
	{
		elixioIcon.alpha = 0.5;
		quote1.alpha = 0;
               isOverlayed = false;
	}

	//beebop icon
	if(FlxG.mouse.overlaps(beebopIcon) && !isOverlayed){
		FlxG.sound.play(Paths.sound('scrollMenu'));
		beebopIcon.alpha = 1;
		quote2.alpha = 1;
                isOverlayed = true;
	}
	else
	{
		beebopIcon.alpha = 0.5;
		quote2.alpha = 0;
        isOverlayed = false;
	}

you gotta put those in an override update(elasped:Float)
except for these
	var bg = new FlxSprite().loadGraphic(Paths.image('creditsMenu/bgGradient'));
	var elixioIcon = new FlxSprite(300, FlxG.height * 0.38).loadGraphic(Paths.image('creditsMenu/elixioIcon'));
	var beebopIcon = new FlxSprite(735, FlxG.height * 0.38).loadGraphic(Paths.image('creditsMenu/beebopIcon'));
	var isOverlayed:Bool = false;

your welcome


Issue when inserting a div before an editable div (contentEditable)

When I insert a new div before an editable div, preact throws an error. When clicking the button in this example:
class TestContentEditable extends Component {
  constructor (props) {
    super(props)

    this.state = {
      inserted: false
    }

    this.insert = () => {
      this.setState({inserted: true})
    }
  }

  render () {
    if (this.state.inserted) {
      // => this will insert a new div before the editable div, this gives an error
      return

not editable?

edit here...

    }
    else {
      return

edit here...

        <button onClick={this.insert}>Insert div before editable div</button>

    }
  }
}

This throws the following error in Chrome:
preact.js:158 Uncaught SyntaxError: Failed to set the 'contentEditable' property on
'HTMLElement': The value provided ('null') is not one of 'true', 'false',
'plaintext-only', or 'inherit'.

See also this similar virtual-dom issue: https://github.com/Matt-Esch/virtual-dom/issues/176

Nevermind, I see I didn't use the latest version of preact, and this issue is fixed in v5.6.0...
I'm really happy with preact btw, thanks! :)

Oh wow, that's awesome! Thanks for the issue report anyway, I'll try to add a test for this so that future changes to this stuff don't break your use-case. Cheers!

Thanks!


adds additional imported module namespaces to be added in codegen

The purpose of this change is to give codegen the capability to add custom import statements to generated operations.
This change would enable the consumer to share fragments across generated packages.
Addresses this thread:
https://github.com/apollographql/apollo-ios/issues/3284

@hemel: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


Update SSH access docs ('balena ssh' usage with production OS variants)

Connects-to: balena-io/balena-cli/pull/2457
Connects-to: balena-io/balena-cli/pull/2461
Change-type: patch

@balena-ci I self-certify!


Cache handling needs improvement

caching everyting to ~/bioimageio_cache with the current pattern of only analysing the url fails for updates to the remote resource, e.g. a 'master' branch on github, etc...

currently, the default cache folder is still defined in bioimageio.spec. There are also use cases for a cache there, so I opened a new issue to discuss cache solutions there: https://github.com/bioimage-io/spec-bioimage-io/issues/289


[Feature] Gemini with lRU catch - result api

gemini 放入LRU catch, 完成 result api

took 2 hours
done


fix: create a warning when project already exists

This PR was deployed with mocked data in the following URLs for your appreciation:

Frontend: https://fabric8-launcher-frontend-pr-1292-app.surge.sh
Welcome App: https://fabric8-launcher-frontend-pr-1292-welcome-app.surge.sh
Storybook: https://fabric8-launcher-frontend-pr-1292-storybook.surge.sh

According to https://docs.openshift.com/container-platform/3.11/architecture/core_concepts/projects_and_users.html there isn't an owner concept in Projects, just roles that the user has on it., So I think the warning message needs some thought

@gastaldi what I mean is that you can only continue if the project is yours, or am I wrong? If you had something else in mind, I'm happy to change it to that

AFAIK You must have the admin role in the project to be able to modify it.
That role is added when you create the project but can also be assigned to
anyone using oc adm commands
Em Sex, 3 de mai de 2019 10:37, Erik Jan de Wit<EMAIL_ADDRESS>escreveu:

@gastaldi https://github.com/gastaldi what I mean is that you can only
continue if the project is yours, or am I wrong? If you had something else
in mind, I'm happy to change it to that
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8-launcher/launcher-frontend/pull/1292#issuecomment-489098029,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANG5MTJWOFZ7KYONIOJNDPTQ52BANCNFSM4HKTYEIA
.


Block queries seem to return different sealed results

When interacting with the most recently sealed blocks, queries appear to return different values for what the latest sealed block actually is.
I am running into this problem on two different occasions.
I request the latestSealedBlock using:
let toBlock = await fcl.send([fcl.getBlock(true)]).then(fcl.decode);
then I get the newest events up that block:
let results = await fcl.send([fcl.getEventsAtBlockHeightRange(metadataBuilder.getTokenMintedEventType(), fromBlock, toBlock)]);
The results will not always contain events all the way up to the "toBlock". Sometimes results only contains events that exist in blocks at "toBlock-3" or before.
The second error occurs when my code loops around and now "fromBlock" was the previous "toBlock". When I try to call getEventsAtBlockHeightRange again, I get an error saying that fromBlock is several blocks in front of the latest current block.
This has been disastrous to my code because now there are arbitrary events missing all over.

I have the same issue... looks like the isSealed is ignored. True or false returns the same block.

Hi @satosheth,
What version of FCL or the JS-SDK are you using when you run into this issue?
Could you please try again using the latest alpha version of FCL<EMAIL_ADDRESS>.
The block heights passed into getEventsAtBlockHeightRange must be heights of sealed blocks. In the latest FCL alpha version, any issues with querying for the latest sealed block should be resolved, so your query fcl.getBlock(true) should now work as expected.
Thank you opening this issue! Happy building :)

It has been 3 months since we asked for more information, FCL has progressed 5 versions since then. No-one else has raised this.


update deleteNamedspaceJob due to api change

k8s api changed caused deleteNamedspaceJob not removing orphan pods.  Wrote a test case to detect  future changes. Not sure how to catch this kind of API changes in general for js though.

Pull Request Test Coverage Report for Build 1723

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 66.845%

Totals

Change from base Build 1718:
0.02%

Covered Lines:
6754

Relevant Lines:
9508

💛  - Coveralls


Improvements

Really cool job @barbara-chaves  !!!
Here you have some tips to improve the exercise:
JavaScript

This is beauty https://github.com/Adalab/m2-evaluacion-intermedia-barbara-chaves/blob/master/_src/assets/js/main.js#L12
[x] You should use parseInt here https://github.com/Adalab/m2-evaluacion-intermedia-barbara-chaves/blob/master/_src/assets/js/main.js#L32
[x] Remember to convert input value string to a number (always)
[x] Use a variable to store counter

Advanced

[x] You are writing many times "trackEl.innerHTML = 'something';", what about making a function that accepts a string as param and paints it into 'trackEl'?

HTML is perfect!!!!! Congratulations!!!

Steps to finish the game (for those far from the solution)
You are close to the exercise solution, but here you have a steps guide to complete it anyway ;)

Save a reference to DOM elements in variables
Generate the random number and save it to a variable
Init counter of trials to 0
Define a function that executes when the user clicks on the button, that

increments the counter and paints its value in the HTML
collect the value of the HTML input and compare it with the random number

if they are equal, draw a 'You Win' message
if it's lower, draw a 'Too short' message
if it's higher, draw a 'Too long' message

Assign a listener to the button click to execute the previous function

Further challenges
Then you can improve the game as you please, here you have some ideas:

Add new feedback when the number is bigger than 100 or smaller than 0.
Manage when the user hits the button and the input is empty.
Make the input work when hitting enter key.
Add a reset button that cleans the input, the counter, writes the initial feedback and generates a new random number to play again!
Whatever you want!

Let me know if you have any doubts ;)
Go for it!!

Thank you for the review! If it's not good please tell me.


XSLT StyleSheet for ISO missing latest update

Alberta was shown an updated xslt stylesheet to format the XML metadata into a more readable format.  With the latest 2.6.0 release there is an old 2016 stylesheet in the code.
geoportal/src/main/resources/metadata/details/iso-details/xml-to-html-ISO.xsl
Please push the new stylesheet to GitHub. Thank you

I've done a bunch of work on an ISO to HTML transform that uses some Javascript to display a location map, and puts a schema.org JSON-LD script in the HTML that should improve indexing by Schema.org aware search engines.  The code is in this repo https://github.com/usgin/metadataTransforms/tree/master/ISO19139toHTMLandSDO
you can see what it looks like with searches at
http://catalog.iedadata.org (map not implemented, slightly older version). Click on 'full metadata record' link in search results.
or
http://cinergi.sdsc.edu/geoportal (current version) click on the 'html' link in search results.

@KathleenPate  the change was actually posted to github last February, what issue are you having?

@Zguo Earlier this year my coworkers had a video conference with Redlands staff and were shown a new ISO html layout developed for Alberta. It is my understanding that Alberta requested the Field names be converted to more user friendly terms. Example: CI_Address: would simply be Address:
I have never seen the layout, but my coworkers who were in the presentation tell me that what they see in the Sandbox does not match what they saw in the presentation. Does that ring a bell?
My coworkers will be back in the office Tuesday and I can get more specifics from them if this does not make sense.

#283

We did two things to fix up the display of ISO metadata display; add a stylesheet, and, add new display elements.  File path: /geoportal/src/main/resources/metadata/details/iso-details/
We modified the field connections in elements-ISO.xml  to fix display issue.
For Example we added an option without the gmd: in the tag name because our docs didn't have gmd prefixes.
<name tagname="abstract">Abstract</name>
Add Bootstrap Stylesheet found in the GeoPortal into the xml-to-htm-ISO.xsl

	<link rel="stylesheet" href="/geoportal/lib/calcite-bootstrap/css/calcite-bootstrap.css"/>


When changing scroll direction, restart animation from current position instead of future position (includes Wizzel1's scroll speed commits)

I also added a fix to the trackpad gestures not working when switching between mouse wheel and trackpad (and when first opening the widget)

Also fixed scrollview not locking when hovering cursor on top of the scrollbar or holding it


Ability to modify the main text colour

Hey! I've had a look through the compiled code to see if there was a way to modify the white text to a sligh offwhite as it's quite strong against the black background.
Are you able to offer any way of doing this? I've installed using yarn. Thanks!

Set a different color in your terminal with ANSI color codes
Customize a theme in settings:

@dsgnr It is done, please close this issue!


createFileSessionStorage not working

Reproduction
https://github.com/EvgeniyBudaev/remix-2-9-2-session
npm i
npm run dev
open in browser
System Info
System:
    OS: Linux 6.5 Ubuntu 22.04.4 LTS 22.04.4 LTS (Jammy Jellyfish)
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11700K @ 3.60GHz
    Memory: 46.99 GB / 62.57 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.14.2/bin/yarn
    npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
  Browsers:
    Chrome: 126.0.6478.61
  npmPackages:
    @remix-run/dev: ^2.9.2 => 2.9.2
    @remix-run/express: ^2.9.2 => 2.9.2
    @remix-run/node: ^2.9.2 => 2.9.2
    @remix-run/react: ^2.9.2 => 2.9.2
    @remix-run/serve: ^2.9.2 => 2.9.2
    vite: ^5.1.0 => 5.3.1

Used Package Manager
npm
Expected Behavior
I'm waiting for the creation of a session directory with files

Actual Behavior
session directory with files not created

i fixed this


Update gov.txt directly from zone

In #107, we updated gov.txt (a copy of the zone file_ each day from the version we publish each day to CZDS. Instead of pulling once daily from a copy, we should pull from our DNS provider, Cloudflare, which we could also poll more frequently since the provenance of the data is the real source. Let's start with once every 6 hours.
One question to answer is our commit approach. We want to see the individual commits but summarize them into one (like a PR, though we don't want to need to approve a PR).
Acceptance criteria

[ ] Get read-only API token from Cameron
[ ] Determine commit approach
[ ] Decide if we want to strip the zone file's comment header (the order of this file is slightly different than CZDS)
[ ] gov.txt is updated daily via Cloudflare
[ ] ADR is updated with any necessary changes based on this work

@h-m-f-t - in Fireside, we reviewed this and wanted your feedback on the actual priority on this item. It's fairly high in the backlog now.

It's true this isn't super urgent. However, one of my reasons for it being high in the backlog is that it is similar to the work recently done to archive the CZDS zone file and so it may be easy to pick up.

I lightly updated the issue to de-scope questions about multiple commits in a PR. (When we poll more frequently, we'll want to see individual daily commits but summarize the daily diff into one (perhaps this is a PR, though we don't want to approve a PR before merging).


Support Banana PI

Hello! Does your library supports Banana PI?

If the banana pi supports SPI (what does ls -l /dev/spi* show?) and RPi.GPIO (or there is an equivalent) that works, then it should work.

@RomanGoncharov any feedback?

No feedback, closing.


Its not discovering my mdns service in  my nodemcu .

Showing no results.

After fixing a small typo in my arduino code , I got it fixed .


Feat/shuffling explorers with deeplinks

Shuffling is already in main. This was just to test something :)


ipyvuetify DatePicker and allowedDates

I m struggling with the datePicker widget, I have a list of dates in python and I would like to display the widget with only theses dates, but how I m supposed to provide that array with ipyvuetify ?
This work fine, but it only hides odd days.
class DatesPicker(v.VuetifyTemplate):

date = traitlets.Unicode(default_value= '2020-01-01').tag(sync=True)
#allowedDatesArray = traitlets.List(default_value= ["2020-01-01","2020-01-02", "2020-01-05","2020-01-06"]).tag(sync=True)

traitlets.default('template')
def _template(self):
    '''<template>
<v-row justify="center">
<v-date-picker
  v-model="date"
  :allowed-dates="allowedDates"
  class="mt-4"
  min="2016-06-15"
  max="2020-01-15"
></v-date-picker>
</v-row>
</template>

<script >
export default {
data: () => ({
}),

methods: {
  allowedDates: val => parseInt(val.split('-')[2], 10) % 2 === 0,
},
}
</script>

'''
So how can I pass "allowedDatesArray" to methods, or is there an other way to deal with allowedDates based on an array ? (With trailets only ?)

UP! How can I link python's list of allowed-dates ?


Migrate to rocm-docs-core, updated README

This PR includes these changes:

Streamlined README.md
Added docs/ folder with reformatted docs. Prometheus/Grafana and Slurm docs moved to dedicated pages.
Added necessary files for rocm-docs-core (toc file, read the docs config, sphinx config)

Got your suggestions incorporated @farshadghodsian :+1:

Matt is this Exporter documentation covering all Variations . I see install on   K8s Exporter only and standalone docker install info .    I dint see info on  Operator/exporter  or Kube-rbac info  and Debian based install
Also  Slurm config files still needs to be  updated , currently it  is pointing to old files ...Prolog=/path/to/custom/slurm-exporter.sh
Epilog=/path/to/custom/slurm-exporter.sh

Matt is this Exporter documentation covering all Variations . I see install on K8s Exporter only and standalone docker install info . I dint see info on Operator/exporter or Kube-rbac info and Debian based install

@bhunuk I didn't write any new docs for this, only updated existing docs and integrated some of the guide Farshad put together. These docs are focused on standalone usage. Using the metrics exporter w/ k8s is covered in the GPU Operator docs (https://dcgpu.docs.amd.com/projects/gpu-operator/en/latest/metrics/exporter.html and https://dcgpu.docs.amd.com/projects/gpu-operator/en/latest/metrics/kube-rbac-proxy.html). I'm unclear on when the .deb package will be publicly released, but we can add those docs in when the installer is available.

Also Slurm config files still needs to be updated , currently it is pointing to old files ...Prolog=/path/to/custom/slurm-exporter.sh Epilog=/path/to/custom/slurm-exporter.sh

What are the correct files to use?

This is ready for review @sajmera-pensando @farshadghodsian @yansun1996


[BUG] [LP] !important prevent change font-family heading with cssclass

Describe the bug

I can't "manually" change the font of heading with a cssclass because of an !important :
.cm-s-obsidian .cm-header-1,.cm-s-obsidian .cm-header-2,.cm-s-obsidian .cm-header-3,.cm-s-obsidian .cm-header-4,.cm-s-obsidian .cm-header-5,.cm-s-obsidian .cm-header-6,.cm-s-obsidian .HyperMD-header-1,.cm-s-obsidian .HyperMD-header-2,.cm-s-obsidian .HyperMD-header-3,.cm-s-obsidian .HyperMD-header-4,.cm-s-obsidian .HyperMD-header-5,.cm-s-obsidian .HyperMD-header-6{font-family:var(--font-heading-editor)!important}

To Reproduce
Steps to reproduce the behavior (this is an example):

Create a snippet cssclass with :

.draft .cm-line,
.draft .markdown-source-view,
.draft .cm-s-obsidian:not(span.cm-comment),
.draft .markdown-preview-view,
.markdown-reading-view .draft,
.draft {
  font-family: 'Fantasque Sans Mono' !important;
}

Use it in a file.

Expected behavior

Screenshots

Version [Desktop or Mobile]:
→ The two version (mobile and desktop LP)
Editor
Live Preview CM6 Editor
Additional context
Removing the important doesn't seem to be a problem here.

I have removed the !important in my dev version of the theme, so it won't be there in the next update. In the meantime, and for reference, you can always try changing the custom property value locally, like so:
.draft .cm-line,
.draft .markdown-source-view,
.draft .cm-s-obsidian:not(span.cm-comment),
.draft .markdown-preview-view,
.markdown-reading-view .draft,
.draft {
    --font-heading-editor: 'Fantasque Sans Mono';
}

You shouldn't have this problem in the latest update. :)


Fixes #37 doc update for zf2 vs zf3 set shared manager on EventManager

Fixes #37

Closing in preference to #44, which I find more succinct.


Couldn't find valid bootstrap when overwriting entrypoint in lambda console

Hi,
I have a docker image created using this Dockerfile
ARG FUNCTION_DIR="/opt/"
FROM node:10.13-alpine@sha256:22c8219b21f86dfd7398ce1f62c48a022fecdcf0ad7bf3b0681131bd04a023a2 AS BUILD_IMAGE
ARG FUNCTION_DIR
RUN apk --update add cmake autoconf automake libtool binutils libexecinfo-dev python2 gcc make g++ zlib-dev
ENV NODE_ENV=production
ENV PYTHON=/usr/bin/python2
RUN mkdir -p ${FUNCTION_DIR}
WORKDIR ${FUNCTION_DIR}
COPY package.json yarn.lock ./
RUN yarn --frozen-lockfile
RUN npm prune --production
RUN yarn cache clean
RUN npm cache clean --force
FROM node:10.13-alpine@sha256:22c8219b21f86dfd7398ce1f62c48a022fecdcf0ad7bf3b0681131bd04a023a2
ARG FUNCTION_DIR
ENV NODE_ENV=production
ENV NODE_OPTIONS=--max_old_space_size=4096
RUN apk update \
    && apk upgrade \
    && apk add mongodb-tools fontconfig dumb-init \
    && rm -rf /var/cache/apk/*
RUN mkdir -p ${FUNCTION_DIR}
WORKDIR ${FUNCTION_DIR}
COPY --from=BUILD_IMAGE ${FUNCTION_DIR}/node_modules ./node_modules
COPY . .
RUN if [ -f  core/config/local.js ]; then rm core/config/local.js; fi
RUN cp core/config/local.js.aws.readonly core/config/local.js
USER node
EXPOSE 8080
ENTRYPOINT ["/usr/bin/dumb-init", "--"]
CMD ["node", "app.js", "--app=search", "--env=production"]

I use this Dockerfile to generate an image (called core-a) that run our application in K8s.
I've added some code inside my application to handle the case our application is launched from a lambda function and i've created another Dockerfile like the one above but using custom ENTRYPOINT and CMD setting this values.
ENTRYPOINT [ "/usr/local/bin/npx", "aws-lambda-ric" ]
CMD [ "apps/search/index.handler" ]

Than i deployed this image called core-b to a lambda function and everything works as expected.
After that i thought that i can use the entrypoint overwrite and CMD overwrite in lambda function, so i modified Lambda function's image pointing to core-a and modified the values from the console to overwrite ENTRYPOINT and CMD, this way

But every value i insert in those fields i always get an error from lambda
{
  "errorMessage": "RequestId: b34c7d1c-9870-4390-87d4-a3ac41030ff8 Error: Couldn't find valid bootstrap(s): [\"/usr/local/bin/npx\"]",
  "errorType": "Runtime.InvalidEntrypoint"
}

It was an issue in the aws documentation that states that each parameter should be enclosed between quotation marks... removed the quotation marks everything worked


a little question

Hello, after seeing the architecture diagram, I am thinking about how region sends the required information to other clients, such as Aliyuncli, to prepare for their access. Or I want to know whether my thinking is wrong. Could you please explain？

Region maintains a data structure called cloudaccount, which stores the credential to access a cloud provider. For example, a cloudaccount for Aliyun stores the AcessKey/SecretKey for an Aliyun account. The information of cloudaccount was input by administrator of the platform. Region will call API of cloud provider with the credential to pull list of resources or take action on resources, e.g. create a VM instance or start a VM.

Region maintains a data structure called cloudaccount, which stores the credential to access a cloud provider. For example, a cloudaccount for Aliyun stores the AcessKey/SecretKey for an Aliyun account. The information of cloudaccount was input by administrator of the platform. Region will call API of cloud provider with the credential to pull list of resources or take action on resources, e.g. create a VM instance or start a VM.

Hello, I would like to ask another question, using croma Start and run to schedule and process tasks, but I see that adding tasks to the task queue is only done once in startService, not in the multi-coroutine way, so how does the task stream into the task queue

We have an in-memory task queue, the code is at https://github.com/yunionio/cloudpods/blob/master/pkg/appsrv/workers.go. A task is enqueue by calling Run method. The going routine will consume tasks in queue.
There is another task manager whose state is maintained by database. The code in at https://github.com/yunionio/cloudpods/tree/master/pkg/cloudcommon/db/taskman. A task is scheduled by calling NewTask. The states are stored in database. The task proceeds by stages. The initial OnInit stage is called immediately after the task is created. The stage finishes by calling a remote procedure call, using at another node, e.g. a host agent. The next stage is invoke asyncronously by calling the task handler when the previous remote procuedure call finished at host agent.

We have an in-memory task queue, the code is at https://github.com/yunionio/cloudpods/blob/master/pkg/appsrv/workers.go. A task is enqueue by calling Run method. The going routine will consume tasks in queue.
There is another task manager whose state is maintained by database. The code in at https://github.com/yunionio/cloudpods/tree/master/pkg/cloudcommon/db/taskman. A task is scheduled by calling NewTask. The states are stored in database. The task proceeds by stages. The initial OnInit stage is called immediately after the task is created. The stage finishes by calling a remote procedure call, using at another node, e.g. a host agent. The next stage is invoke asyncronously by calling the task handler when the previous remote procuedure call finished at host agent.

Well, I know that task joins the task queue in that file, I would like to ask the above mentioned region to send command, is through cloudAccount and call API such as Ali cloud API, However, I found that region obtains the cloudaccount task from the database only when it starts the region service. In this case, if I want to add another cloudaccount later, do I need to restart the service service？

Not necessarily to restart thge service. Every time a new cloudaccount is created, the cloudaccount task will be started for this account.


no obvious age difference between orignial stylegan and this

I used previous stylegan and stylegan in interfacegan to change age. But I cannot tell obvious difference between these two.
the boundary used is "stylegan_f    fhq_age_w_boundary.npy". Could you provide any help?

Do you use the tool edit.py? And have you tried adjusting start_distance and end_distance in file edit.py?

Yeah... I met the same phenomenon use the pretrained ffhq-stylegan model.  I follow the default setting in edit.py. Should I modify some parameters when I use stylegan?

You should use boundary in Z space.

@ZhouDongliang @mehameha998 If you want to use W/W+ space, you should first generate images from Z space, and then use the intermediate latent codes from W/W+ space (which will be automatically saved by generate_data.py). Please do not generate W/W+ vectors randomly, since W/W+ space is not subject to standard normal distribution. See NOTE in edit.py for more details.

@ZhouDongliang @mehameha998 If you want to use W/W+ space, you should first generate images from Z space, and then use the intermediate latent codes from W/W+ space (which will be automatically saved by generate_data.py). Please do not generate W/W+ vectors randomly, since W/W+ space is not subject to standard normal distribution. See NOTE in edit.py for more details.

OK, thank you for your reply.


[SPARK-34649][SQL] org.apache.spark.sql.DataFrameNaFunctions.replace() fails for column name having a dot

What changes were proposed in this pull request?
Use resolved attributes instead of data-frame fields for replacing values.
Why are the changes needed?
dataframe.na.replace() does not work for column having a dot in the name
Does this PR introduce any user-facing change?
None
How was this patch tested?
Added unit tests for the same

Can one of the admins verify this patch?

cc @cloud-fan FYI

ok to test

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40440/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135858/

Test build #135860 has started for PR 31769 at commit b03acab.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40442/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40442/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40442/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135865/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40448/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40448/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40448/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40451/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135868/

Test build #135860 has finished for PR 31769 at commit b03acab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135860/

Test build #135873 has started for PR 31769 at commit 07582ae.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40456/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40456/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40456/

Test build #135873 has finished for PR 31769 at commit 07582ae.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135873/

Test build #135877 has started for PR 31769 at commit 0f0f48e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40460/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40460/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40460/

Test build #135877 has finished for PR 31769 at commit 0f0f48e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135877/

Let's add an item in the migration guide to describe the behavior change.

Could you address the @cloud-fan comment above? You need to update docs/sql-migration-guide.md, too.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135891/

retest this please

Test build #135898 has started for PR 31769 at commit 58679bd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40478/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40478/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40478/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135896/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40481/


objectstore/cos: add endpoint field for config vpc internal endpoint

Signed-off-by: Jimmie Han<EMAIL_ADDRESS>

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Add endpoint field to COS object store config.
Add document and validate test.

Verification

Unit test.
Real depoyment test.

CI is broken, it will fixed by https://github.com/thanos-io/thanos/pull/5000.


Use num_bytes instead of numel in min-cut partitioning

As Title
@Chillee

Let's error out for now. We can keep adding it as we test more models. And once a steady state is reached, we can default to float.


org rename in 2.3 branch

The related issue: https://github.com/stolostron/backlog/issues/18746
Signed-off-by: haoqing0110<EMAIL_ADDRESS>

/assign @elgnay

/approve
/lgtm


Problem with Tuning

Hello,
I am having problems with cartographer's tuning.  I'd be very thankful for any help!

here's the rosbag_validate

all the lua/lauch/urdf files

bag

Thank you!

Lidar works at 25 Hz. All sensors are "simulated" in UnReal.
How can I tune laser SLAM? On which parameters should I work?

There is no specific advice I can give you because this depends very much on your custom setup. But you can have a look at the "tuning" sections of the documentation pages.


Error in the documentation

Issue Summary
The documentation has a typo.
Steps to Reproduce:
At https://docs.mathjax.org/en/latest/web/typeset.html , the sentence "The second, Mathjax.typesetPromise(), performs the typesetting asynchronously, and returns a promise that is resolved when the typesetting is complete." should be replaced with "The second, MathJax.typesetPromise(), ..." with the upper J in MathJax. This looks trivial, but I have been looking for a bug everywhere in my Vue app before catching it.
Technical details:
Not relevant.
Supporting information:
Not relevant.

Thanks for the report.  I'll fix it in the next pass over the documentation (I know it has been a long time).


[Hotfix] Fix TCI build

Purpose
Fix TCI build
Changes

Edit tests to conform with expected behavior based on code

Alternate solutions
Because I'm not certain "expected behavior based on code" equates to "expected behavior", two possible alternatives were considered:
diff --git a/admin/users/views.py b/admin/users/views.py
index 0d2dde4..b68d40a 100644
--- a/admin/users/views.py
+++ b/admin/users/views.py
@@ -372,13 +372,11 @@ class UserWorkshopFormView(PermissionRequiredMixin, FormView):

     @staticmethod
     def get_user_logs_since_workshop(user, workshop_date):
-        query_date = workshop_date + timedelta(days=1)
-        return NodeLog.objects.filter(user=user, date__gt=query_date)
+        return NodeLog.objects.filter(user=user, date__gt=workshop_date)

     @staticmethod
     def get_user_nodes_since_workshop(user, workshop_date):
-        query_date = workshop_date + timedelta(days=1)
-        return Node.objects.filter(creator=user, date_created__gt=query_date)
+        return Node.objects.filter(creator=user, date_created__gt=workshop_date)

     def parse(self, csv_file):
         """ Parse and add to csv file.

and
diff --git a/admin/users/views.py b/admin/users/views.py
index 0d2dde4..3682397 100644
--- a/admin/users/views.py
+++ b/admin/users/views.py
@@ -372,12 +372,12 @@ class UserWorkshopFormView(PermissionRequiredMixin, FormView):

     @staticmethod
     def get_user_logs_since_workshop(user, workshop_date):
-        query_date = workshop_date + timedelta(days=1)
+        query_date = workshop_date + timedelta(days=1) - timedelta(hours=workshop_date.hour, minutes=workshop_date.minute, seconds=workshop_date.second, microseconds=workshop_date.microsecond)
         return NodeLog.objects.filter(user=user, date__gt=query_date)

     @staticmethod
     def get_user_nodes_since_workshop(user, workshop_date):
-        query_date = workshop_date + timedelta(days=1)
+        query_date = workshop_date + timedelta(days=1) - timedelta(hours=workshop_date.hour, minutes=workshop_date.minute, seconds=workshop_date.second, microseconds=workshop_date.microsecond)
         return Node.objects.filter(creator=user, date_created__gt=query_date)

     def parse(self, csv_file):

The former assumes that any logs/nodes created immediately after the workshop date are desired, even if only by seconds.
The latter assumes that any logs/nodes created anytime on a day following the workshop date are desired, and is convoluted (rather than just calling .date()) to avoid issues with date/datetime type conversion and timezone naïveté.
Side effects
None expected
Ticket
None

The current behavior does seem a bit off, but this PR is good to go for the time being. I've created a ticket to investigate that node/log calculation: https://openscience.atlassian.net/browse/OSF-8089


(URGENT) Word: next term too close to previous term's last NOTE

The Word layout places the next term too close to the last NOTE of the previous term.

The PDF looks reasonable:

Can we add the same amount of space after the last NOTE as with a new term?
Can we slightly reduce the margin between the term definition and the first NOTE?

No, and no. I am not mucking around with the styles defined in the IEEE template. That's they styles they've published, that's what we use. Closing ticket as vexatious.


html/template: data race in Execute

go version go1.14.3 darwin/amd64

Building the below program with go build -race and then running it fails:
package main

import (
	"fmt"
	"html/template"
	"io/ioutil"
	"log"
	"sync"
)

func main() {
	var wg sync.WaitGroup

	jsTempl := `
{{- define "jstempl" -}}
var foo = "bar";
{{- end -}}
<script type="application/javascript">
{{ template "jstempl" $ }}
</script>
`

	tpl := template.New("")
	_, err := tpl.New("templ.html").Parse(jsTempl)
	if err != nil {
		log.Fatal(err)
	}

	const numTemplates = 20

	for i := 0; i < numTemplates; i++ {
		_, err = tpl.New(fmt.Sprintf("main%d.html", i)).Parse(`{{ template "templ.html" . }}`)
		if err != nil {
			log.Fatal(err)
		}
	}

	for i := 1; i <= 10; i++ {
		wg.Add(1)
		go func() {
			defer wg.Done()
			for j := 0; j < numTemplates; j++ {
				templ := tpl.Lookup(fmt.Sprintf("main%d.html", j))
				if err := templ.Execute(ioutil.Discard, nil); err != nil {
					log.Fatal(err)
				}

			}
		}()
	}

	wg.Wait()
}

WARNING: DATA RACE

WARNING: DATA RACE
Write at 0x00c00011c540 by goroutine 32:
  runtime.mapassign_faststr()
      /Users/bep/dev/go/dump/go/src/runtime/map_faststr.go:202 +0x0
  text/template.(*Template).associate()
      /Users/bep/dev/go/dump/go/src/text/template/template.go:227 +0x1a8
  text/template.(*Template).AddParseTree()
      /Users/bep/dev/go/dump/go/src/text/template/template.go:133 +0x2cc
  html/template.(*escaper).commit()
      /Users/bep/dev/go/dump/go/src/html/template/escape.go:810 +0x2c5
  html/template.escapeTemplate()
      /Users/bep/dev/go/dump/go/src/html/template/escape.go:38 +0x324
  html/template.(*Template).escape()
      /Users/bep/dev/go/dump/go/src/html/template/template.go:102 +0x349
  html/template.(*Template).Execute()
      /Users/bep/dev/go/dump/go/src/html/template/template.go:119 +0x3c
  main.main.func1()
      /Users/bep/dev/go/bep/temp/main.go:46 +0x177
Previous read at 0x00c00011c540 by goroutine 28:
runtime.mapaccess1_faststr()
/Users/bep/dev/go/dump/go/src/runtime/map_faststr.go:12 +0x0
text/template.(*state).walkTemplate()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:399 +0x13b
text/template.(*state).walk()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:268 +0x2be
text/template.(*state).walk()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:263 +0x195
text/template.(*Template).execute()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:220 +0x2fb
text/template.(*Template).Execute()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:203 +0xd2
html/template.(*Template).Execute()
/Users/bep/dev/go/dump/go/src/html/template/template.go:122 +0x7f
main.main.func1()
/Users/bep/dev/go/bep/temp/main.go:46 +0x177
Goroutine 32 (running) created at:
main.main()
/Users/bep/dev/go/bep/temp/main.go:42 +0x2de
Goroutine 28 (running) created at:
main.main()
/Users/bep/dev/go/bep/temp/main.go:42 +0x2de
==================
WARNING: DATA RACE
Write at 0x00c0000ac0b0 by goroutine 32:
text/template.(*Template).associate()
/Users/bep/dev/go/dump/go/src/text/template/template.go:227 +0x1bd
text/template.(*Template).AddParseTree()
/Users/bep/dev/go/dump/go/src/text/template/template.go:133 +0x2cc
html/template.(*escaper).commit()
/Users/bep/dev/go/dump/go/src/html/template/escape.go:810 +0x2c5
html/template.escapeTemplate()
/Users/bep/dev/go/dump/go/src/html/template/escape.go:38 +0x324
html/template.(*Template).escape()
/Users/bep/dev/go/dump/go/src/html/template/template.go:102 +0x349
html/template.(*Template).Execute()
/Users/bep/dev/go/dump/go/src/html/template/template.go:119 +0x3c
main.main.func1()
/Users/bep/dev/go/bep/temp/main.go:46 +0x177
Previous read at 0x00c0000ac0b0 by goroutine 28:
text/template.(*state).walkTemplate()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:399 +0x14e
text/template.(*state).walk()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:268 +0x2be
text/template.(*state).walk()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:263 +0x195
text/template.(*state).walkTemplate()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:413 +0x3cd
text/template.(*state).walk()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:268 +0x2be
text/template.(*state).walk()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:263 +0x195
text/template.(*Template).execute()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:220 +0x2fb
text/template.(*Template).Execute()
/Users/bep/dev/go/dump/go/src/text/template/exec.go:203 +0xd2
html/template.(*Template).Execute()
/Users/bep/dev/go/dump/go/src/html/template/template.go:122 +0x7f
main.main.func1()
/Users/bep/dev/go/bep/temp/main.go:46 +0x177
Goroutine 32 (running) created at:
main.main()
/Users/bep/dev/go/bep/temp/main.go:42 +0x2de
Goroutine 28 (running) created at:
main.main()
/Users/bep/dev/go/bep/temp/main.go:42 +0x2de
Found 2 data race(s)

@bep thanks for the issue. Are you able to check Go 1.13 and see if it is affected? (This will determine if this is a blocker for 1.15 or not)
/cc @dmitshur

It also fails on go1.13.12.

/cc @empijei

/cc @mikesamuel

Some additional info:

This does not affect "text/template"
This also affects ExecuteTemplate so it is not just an issue with the lookup+execute calls.
This does not happen if the templates are executed at least once before the concurrent execution. This is probably due to a lazy computation since the race happens in a escapeTemplate call, which is only executed once per template.
My best guess is that the computation is lazy because it involves calling nested templates but I'd need to take a better look at this. The escape should be happening while holding a lock or should be executed at parse time if possible.

As a quick workaround running the templates once before they are run concurrently addresses the issue (see code below), but we need to properly fix this.
Thanks for reporting.
package main

import (
	"fmt"
	"html/template"
	"io/ioutil"
	"log"
	"sync"
)

func main() {
	var wg sync.WaitGroup

	jsTempl := `
{{- define "jstempl" -}}
var foo = "bar";
{{- end -}}
<script type="application/javascript">
{{ template "jstempl" $ }}
</script>
`

	tpl := template.New("")
	_, err := tpl.New("templ.html").Parse(jsTempl)
	if err != nil {
		log.Fatal(err)
	}

	const numTemplates = 20

	for i := 0; i < numTemplates; i++ {
		_, err = tpl.New(fmt.Sprintf("main%d.html", i)).Parse(`{{ template "templ.html" . }}`)
		if err != nil {
			log.Fatal(err)
		}
	}

        // Temporary workaround to prevent the race.
	for j := 0; j < numTemplates; j++ {
		if err := tpl.ExecuteTemplate(ioutil.Discard, fmt.Sprintf("main%d.html", j), nil); err != nil {
			log.Fatal(err)
		}
	}

	for i := 1; i <= 10; i++ {
		wg.Add(1)
		go func() {
			defer wg.Done()
			for j := 0; j < numTemplates; j++ {
				if err := tpl.ExecuteTemplate(ioutil.Discard, fmt.Sprintf("main%d.html", j), nil); err != nil {
					log.Fatal(err)
				}
			}
		}()
	}

	wg.Wait()
}

This appears to be the same data race causing gohugoio/hugo#7293, where it is triggered from text/template.
Hugo have a temporary fork of text/template included in their codebase. I attempted to fix the data race in the fork: gohugoio/hugo#7507. They'd rather have it upstreamed though.

I pulled the patch out of Hugo's fork of text/template and changed it a bit:

Separate lock for tmpl instead of reusing muFunc—not sure if that's best
Use lock everywhere tmpl is accessed

Personally, I'd prefer a single lock as it's simpler and safer. What do you think, should I file a PR?

Do you have a proposal to fix the race described in this bug report? If so please do send a CL my way :smile:

It looks to me like html/template's nameSpace.mu needs to be an RWLock and must be RLock'ed during t.text.Execute.
t.escape will need to RLock to decide if escaping is necessary and then RUnlock/Lock if so.

I guess this will not get fixed in Go 1.16?

The problem in text/template seems different to me.  That seems to require calling AddParseTree concurrently, but that method is not documented as supporting concurrent calls.  Maybe we should change that, but it doesn't seem related to this problem.

See previously #2439, which was apparently the same problem but regressed due to the lack of a regression test (which is understandable given that the race detector debuted over a year later).

@bcmills sure, but you (and with you I don't mean specifically you) cannot at the sam time say that data races are always a serious bug which need fixing and then brush this on to a future release when there is a working fix in place.
This whole situation means that I need to somehow maintain a fix for this in a fork, which is not something I do with joy and certainly not something I get paid to do.
/cc @spf13

@bep

you (and with you I don't mean specifically you) cannot at the sam time say that data races are always a serious bug which need fixing and then brush this on to a future release when there is a working fix in place.

I understand that the length of time this has been open is frustrating, but I don't see anybody saying this.
Are you able to check whether https://golang.org/cl/274450 fixes the problems that you are seeing?

I think we're going to have to roll back the fix for this.  I don't see a way to avoid #43855.

I think we're going to have to roll back the fix for this.  I don't see a way to avoid #43855.

The fix was rolled back, so reopening this issue.

The fix was rolled back, so reopening this issue.

The basic issue here is that html/template expects to be able to call the text/template Template methods Execute and AddParseTree concurrently.  We can in principle fix this by adding locking to html/template, but it's hard because user-defined template functions, including calls to data type methods, can themselves call Execute and, therefore, AddParseTree.
My attempt to fix this, following the suggestion at https://github.com/golang/go/issues/39807#issuecomment-703966128, failed because nothing released the read lock held during a method call, so if the method call called Execute, it led to a deadlock.
Given html/template's needs, I think we kind of do have to fix this in text/template, along the lines of https://golang.org/cl/257817.  It's certainly an abstraction violation to change text/template to fix an html/template problem.  But given that html/template needs to concurrently call Execute and AddParseTree, I don't yet see a different workable approach.

The basic issue here is that html/template expects to be able to call the text/template Template methods Execute and AddParseTree concurrently.  We can in principle fix this by adding locking to html/template, but it's hard because user-defined template functions, including calls to data type methods, can themselves call Execute and, therefore, AddParseTree.
My attempt to fix this, following the suggestion at https://github.com/golang/go/issues/39807#issuecomment-703966128, failed because nothing released the read lock held during a method call, so if the method call called Execute, it led to a deadlock.
Given html/template's needs, I think we kind of do have to fix this in text/template, along the lines of https://golang.org/cl/257817.  It's certainly an abstraction violation to change text/template to fix an html/template problem.  But given that html/template needs to concurrently call Execute and AddParseTree, I don't yet see a different workable approach.

Following my comment in https://golang.org/cl/c/go/+/274450/4/src/html/template/template.go#138, I think it is possible to fix this race by escaping all of the templates at the first call to Execute, guarded by a sync.Once. But an attempt to do that apparently causes TestAddParseTreeHTML to fail.
I suspect that the failure observed in TestAddParseTreeHTML could be addressed by escaping the templates in topological (instead of arbitrary) order.
However, I suspect that that failure indicates a fundamental inconsistency in the API: if the template b cannot be escaped outside the context of its invocation within template a, doesn't that imply that the same template might need to be escaped multiple times, in different ways depending on the context? But the map data structures in the html/template package do not appear to include that context as part of the map key — so either the failure to escape the template in the test is spurious, or a different test ought to reveal a condition in which the template is mis-escaped.

Following my comment in https://golang.org/cl/c/go/+/274450/4/src/html/template/template.go#138, I think it is possible to fix this race by escaping all of the templates at the first call to Execute, guarded by a sync.Once. But an attempt to do that apparently causes TestAddParseTreeHTML to fail.
I suspect that the failure observed in TestAddParseTreeHTML could be addressed by escaping the templates in topological (instead of arbitrary) order.
However, I suspect that that failure indicates a fundamental inconsistency in the API: if the template b cannot be escaped outside the context of its invocation within template a, doesn't that imply that the same template might need to be escaped multiple times, in different ways depending on the context? But the map data structures in the html/template package do not appear to include that context as part of the map key — so either the failure to escape the template in the test is spurious, or a different test ought to reveal a condition in which the template is mis-escaped.

I don't think there is any inconsistency in the API.  When a top-level template is escaped, all the {{template NAME}} invocations are rewritten to invoke a template escaped in the proper context.  That escaping is indeed context-dependent, the escaped template is given a context-dependent name (see the mangle method in context.go), and the escaped template is not marked in a way that is seen if the template is invoked at the top level or in some other context.
A topological sort might work, if we don't care about the extra work.

I don't think there is any inconsistency in the API.  When a top-level template is escaped, all the {{template NAME}} invocations are rewritten to invoke a template escaped in the proper context.  That escaping is indeed context-dependent, the escaped template is given a context-dependent name (see the mangle method in context.go), and the escaped template is not marked in a way that is seen if the template is invoked at the top level or in some other context.
A topological sort might work, if we don't care about the extra work.

@ianlancetaylor if a building is about to tip over, you put some support pillars in place. I reviewed a perfectly fine CR (https://golang.org/cl/257817) that would have made a great pillar until you could get a proper foundation in place. Would you have released Go 1.16 with a similar bug in the net/http package?

@ianlancetaylor if a building is about to tip over, you put some support pillars in place. I reviewed a perfectly fine CR (https://golang.org/cl/257817) that would have made a great pillar until you could get a proper foundation in place. Would you have released Go 1.16 with a similar bug in the net/http package?

If the net/http bug had existed for several releases, and if it were this late in the release cycle, then, yes, I would not have blocked Go 1.16 from being released with such a bug.

If the net/http bug had existed for several releases, and if it were this late in the release cycle, then, yes, I would not have blocked Go 1.16 from being released with such a bug.

@ianlancetaylor https://github.com/golang/go/issues/43822#issuecomment-768639142

@bep I'm sorry you are upset about this issue.  I understand that it is frustrating.  It's frustrating to me too.  I've probably put 40 hours into fixing this issue, and so far I have failed.  That's a lot of time for me.
That said, the comment you mention is different.  I'm not blocking 1.16 because of that bug.  And, to use your metaphor, that CL is not a case of putting in support pillars until there is a proper foundation.  That CL is, I believe, the correct fix.

OK, I think this is finally fixed, and the fix will be in the 1.17 release.  I hope.  Sorry for the lengthy work on this.


changed where Code of Conduct was at

"How to succeed" shouldn't start with "how to fail".

Awesome.  Thanks @brianisbrilliant